Feature request: Add support for doc strings

Thanks for the great package! It was just what I needed. It would be great if it supported doc strings. I think just one docstring per "for" that would be repeated among all the combinations generated by gen_ops_ex.
Yours,
Carl
https://github.com/CarlKCarlK & https://medium.com/@carlmkadie

Hello Carl,
Sorry for the long delay in reply. I got busy with other things.
I believe it's possible to implement this feature. I'll look into this soon.
Regards,
K Vinayak

Amazing Article!
I appreciate for recommending this crate. Thanks.
For the implementation, did you want a use case like this?
struct Complx(i32, i32);

gen_ops!(
    types Complx, Complx => Complx;
    /// Adds two complex numbers
    for + call |a: Complx, b: Complx| Complx(a.0+b.0 , a.1+b.1);
)

impl std::ops::Add for Complx {
    type Output = Complx;

    #[doc = r" Adds two complex numbers"]
    #[inline]
    fn add(self, other:Complx) -> Complx {
        //... call the closure here
    }
}

I was testing a little bit and saw that rust-analyzer doesn't always show the docs when hovered over the operator. It just shows default docs for the operator instead. I'll look into that once I add the feature to the core macros.

That looks great. Thanks for working on this.
From: K Vinayak @.>
Sent: Saturday, April 15, 2023 9:02 AM
To: K-Vinayak/gen_ops @.>
Cc: Carl Kadie @.>; Author @.>
Subject: Re: [K-Vinayak/gen_ops] Feature request: Add support for doc strings (Issue #1)
Amazing Article!
I appreciate for recommending this crate. Thanks.
For the implementation, did you want a use case like this?
struct Complx(i32, i32);
gen_ops!(
types Complx, Complx => Complx;

/// Adds two complex numbers

for + call |a: Complx, b: Complx| Complx(a.0+b.0 , a.1+b.1);

)
impl std::ops::Add for Complx {
type Output = Complx;

#[doc = r" Adds two complex numbers"]

#[inline]

fn add(self, other:Complx) -> Complx {

    //... call the closure here

}

}
I was testing a little bit and saw that rust-analyzer doesn't always show the docs when hovered over the operator. It just shows default docs for the operator instead. I'll look into that once I add the feature to the core macros.

Reply to this email directly, view it on GitHubhttps://github.com/K-Vinayak/gen_ops/issues/1#issuecomment-1509880000, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABR65P2BL2FARXT6OGD57CTXBLA6JANCNFSM6AAAAAAU4GZFVQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

It's working. Thanks!


Build and release arm64 artifacts

This PR improves the CI/CD workflow to release arm64 artifacts for Linux and macOS. This helps M1 macOS users to download native executables with no Rosetta 2 overhead. Tested release artifacts here.

Thank you for your contribution.
I don't own an M1 Mac, but it looks nice. :shipit:


Set some defaults and cache key salt

Made the local setup easier and added a way to set the cache key salt so Redis doesn't loose the cache in production on each deployment.

Pull Request Test Coverage Report for Build 519

0 of 0   changed or added relevant lines in 0 files are covered.
17 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.7%) to 98.315%

Files with Coverage Reduction
New Missed Lines
%

cmd/phpcs-server/main.go
17
94.06%

Totals

Change from base Build 517:
-1.7%

Covered Lines:
992

Relevant Lines:
1009

💛  - Coveralls


Integration Test - Release - 2022/03/11 07:39:48

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

GitHub user PRMergerTest2 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#please-close

#unassign PRMergerTest1

#unassign NonAllowedUser


samples: nats: Check return value and convert to ztest

Check return value for the function and convert test case
to ztest framework
Signed-off-by: Punit Vara<EMAIL_ADDRESS>

I didn't have networking device to check this. If someone can test it I can appreciate @tarunkum

So, if the idea is to fix a coverity issue, it should be done with just normal if + printk.

ok thanks @pfalcon

Codecov Report

Merging #8175 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8175      +/-   ##
==========================================
- Coverage   64.53%   64.53%   -0.01%
==========================================
  Files         420      420
  Lines       40101    40101
  Branches     6762     6762
==========================================
- Hits        25881    25880       -1
- Misses      11103    11104       +1
  Partials     3117     3117

Impacted Files
Coverage Δ

samples/philosophers/src/main.c
96.87% <0%> (-1.57%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 265f502...2b12aaa. Read the comment docs.

This was already fixed by another PR thus closing it.


Document the usage of "local_file: true" bundle option

Documenting this: https://github.com/kairos-io/kairos/issues/2000

Waiting for the feature to land in a release, to update the docs.


proper handling of openMP

@jurajHasik, is it ok that openMP is enabled by default?

Right now, the only openmp parallelism comes through lapack/mkl.
So if ITensor is compiled with those, openmp kicks in. I would leave it
on as default for now

ok, so can we merge?


Implement a dummy impl. of trait ColonyChain

Type of this PR

Select at least one type

[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Bug fix
[ ] Dependency, Environment, Build related update
[ ] Documentation
[ ] Other tiny fix

Describe your changes
Issue ticket number and other helpful resource
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] Leave reference about this pull request, if exist (ex. discord message, google docs, etc)
[ ] If it is a core feature, I have added thorough tests.

Checklist after creating a pull request

[ ] Mention reviews in discord channel directly

아 그리고 커밋 이름과 PR 제목을  Implement a dummy impl. of trait ColonyChain 뭐 이런식으로 바꿔주세요.

Commit 이름은 Update가 아니라 Add new line at end of file 이런식으로 정확하게 뭘 했는지 설명하는게 훨씬 도움됩니다.
너무 자잘한 커밋들은 squash가 필요합니다.


tr: Correctly "fill" set2 in case of complement

In a STRING with 256 given characters and [y*], this "repeated character" should expand to the empty string "". With only 255 given characters, it should expand to "y", i.e. a single character.
$ x226="xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
$ echo -n 'hello' | ../gnu/src/tr -c '[:upper:]' ${x226}'[y*]xxx' # "[y*]" expands to "y", and now set2 contains two distinct characters.
../gnu/src/tr: when translating with complemented character classes,
string2 must map all characters in the domain to one
[$? = 1]
$ echo -n 'hello' | ../gnu/src/tr -c '[:upper:]' ${x226}'[y*]xxxx' # "[y*]" expands to "", so set2 contains only one distinct character.
xxxxx
$ echo -n 'hello' | cargo run tr -c '[:upper:]' ${x226}'[y*]xxxx' # We handle the "" case correctly, accidentally.
xxxxx
$ echo -n 'hello' | cargo run tr -c '[:upper:]' ${x226}'[y*]xxx' # \342\200\246 but we fail in the "y" case.
xxxxx

There are some other ways to detect it, but this is probably the easiest way.
(Original issue: https://github.com/uutils/coreutils/issues/6133)

Fixed by #6472. Thank you!


prepare Reproducible Builds

not sure this is sufficient, but at least this is required for Reproducible Builds...

removing platform-dependant data from daemon common breaks tests...
I'm convinced that such things should not be at that place: perhaps only in client, because client is the only aspect that is native at the end?

#630 should help with the build.properties file.


Get historical reproduction numbers for Lithuania

Hello developers,
thank you for a great analysis! I would like to get an effective reproduction numbers for Lithuania starting from 2020-02-17. Is it possible to get this kind of information using your provided resources?
Thanks in advance!

Hi @ramashka328 ,
sorry for the slow response. There are a few options:

Change the data window from 12 weeks (https://github.com/epiforecasts/covid-rt-estimates/blob/master/R/utils.R#L75) and run just Lithuania from the global datasets using -i "cases/Lithuania, deaths/Lithuania". You could do this properly using #102 if you fancied tackling a ticket or just hack it in for your own use locally.

within the next few months we are thinking we will change infrastructure and able to run the full historic datasets every so often (this is wishlist so no guarantees). This might include the global cases / deaths datasets that would include a top level number for Lithuania.

find a data source that gives breakdown within Lithuania and incorporate it into the project formally - https://github.com/epiforecasts/covid-rt-estimates/blob/master/SMG.md should give some guidance but do ask for more advice if you need it. In combination with 1)/#102 this would give you more detailed figures for the country but may be more than you need.

find some relevant data, massage it into the right format and mash it into the script here: https://github.com/epiforecasts/covid-rt-estimates/blob/master/R/update-regional.R#L121 - this is a quick and nasty version of 3 but absolutely fine for one off work. You are then really using the EpiNow2 library but you can use this project as an outline of how to call it (copy/paste for the win).

I hope this helps
Joe

Hi @ramashka328 thanks for opening this,
If taking option 4 you might find the docs for EpiNow2 helfpul: https://epiforecasts.io/EpiNow2/dev/
@joeHickson (hope you had good AL) - we are getting increasingly freqeunt requests for historical Rt values so it might be a good idea to bump historical Rts up the food chain in terms of priority.
Sam

+1 for historical Rt trajectories as a priority.

Whilst we work on historical estimates here is a case study from generating Rts from scratching for a country in the ECDC data: https://gist.github.com/seabbs/163d0f195892cde685c70473e1f5e867
Note this needs to be adapted to use realistic delays for the country of interest.


ember-power-selected selected option test

The documentation says:

The power-select's selected, disabled and placeholder properties and the onchange action are already wired up to the controlling form.element for you

I tried, it did not work. So I tried to poke deeper and found these:

https://github.com/kaliber5/ember-bootstrap-power-select/blob/6566374b18ce372a255b87f2fac76cebc28f0f06/addon/templates/components/bs-form/element/control/power-select.hbs#L4
https://github.com/kaliber5/ember-bootstrap-power-select/blob/6566374b18ce372a255b87f2fac76cebc28f0f06/addon/templates/components/bs-form/element/control/power-select.hbs#L31

So I tried to alter my code to pass value property and it worked:
{{#bs-form model=this as |form|}}
    {{form.element controlType="power-select" options=options value=selected}}
  {{/bs-form}}

So I'm proposing this failing test to alter the behaviour to be in line with official ember-power-select docs. And I think that property should be changed to from value to selected.

There is some misunderstanding here. power-select has a selected property, the one you linked to. ember-bootstrap's FormElement does not have a selected, the equivalent is indeed value, see its API: http://www.ember-bootstrap.com/api/classes/Components.FormElement.html#property_value. So either using property as in the docs, or (if you really need that) using value.

Just for historical documentation: I can't / don't want to use property, because whatever I'm passing as options is not part of the model and therefore won't work properly with selected property -> initial selection would be wrong, selecting anything won't work.
Why do I have weird stuff in options? Because I need some more complicated data to render the individual option rows in the select box. In my example it was attribute style binding which needs to be sanitised first, so I can't do it on template level.
Thinking about that ... probably a correct way is to use selected property and call a component for every individual option and do the sanitising there. 🤷♀


Consistent treatment of energy across models

Is your feature request related to a problem? Please describe.
At the moment the animal model tracks energy directly, while the soil and litter models track carbon, which is an imperfect  proxy for energy. This becomes a problem when exchanges between models take place, as conversions have to be made. At the moment the one example of this conversion (animal excrement to metabolic litter) is done in a very handwavey manner. This is something we need to fix for more complete versions of the Virtual Rainforest.
Describe the solution you'd like
The fact that different models track different quantities is unavoidable as they are drawn from different traditions with different objects of study. We could enforce all models tracking exactly the same quantities, but this doesn't allow us to escape the problem as we would still need to find sensible conversions between quantities.
I think we need to decide on a consistent approach to conversions of this type, be it making a particular set of assumptions, literature search or something else. I feel the key thing to minimise the problem is that we take a consistent approach to these conversions both within and across models.
Describe alternatives you've considered
I don't feel there is any quick way to fix this, so feel that eventually having an extended discussion of the problem will be necessary at some point in the future.

The animal model is now mostly reworked such that the currency is wet mass. It should be easier to do the conversion from wet mass to carbon mass.


Return XingApi::Response object for 204 responses

Related to https://github.com/xing/xing_api/issues/15

well done!

thx :smile:


Add Documentation for Linear Search

Explain the basic concept of the Algorithm in the README.md file in the folder!
Ground rules:

Whoever claims the Issue first, should send a Pull Request within 24 hours else it'll be assigned to another claimant and further Pull Requests regarding that issue will not be entertained.
Pull Requests sent without claiming and referencing the Issue will not be entertained.
Make sure your Pull Requests have just one commit in them. Squash multiple commits into one if you have to.

I want to claim this

i want to claim this

i want to claim this

I want to claim.

i claim this.


Architecture independant API

I'm using this crate for riscv64 and x86, but the API is different. Is there a way to have only one API for all archs ?
Maybe a struct ?

The problem with the x86 approach is that you

Can’t have exit success due to the way the qemu side is implemented.
You always need two arguments for the ISA exit device, there is now way around it.

Since in all cases it’s only a single function call, is it too inconvenient to work with #[target] in the client code?

The problem with the x86 approach is that you
1. Can’t have exit success due to the way the qemu side is implemented.

Maybe an other value like 255 will do the trick see

2. You always need two arguments for the ISA exit device, there is now way around it.

On riscv64 also. We could just not use the second argument on architectures which does not required it ?

Since in all cases it’s only a single function call, is it too inconvenient to work with #[target] in the client code?
That not too inconvenient but require a wrapper in order to have one API on the client code. So why not directly do it here ?

Maybe something like this:
pub struct QemuExitHandle {
    pub addr: usize,
    pub succes_code: u64,
    pub failure_code: u64,
}

pub trait QemuExit {
    fn new(addr: usize) -> Self;
    fn exit<T>(&self, code: T) -> !
    where
        T: Into<u64>;
    fn exit_success(&self) -> !;
    fn exit_failure(&self) -> !;
}


Consider sending None as revision to checker

The current practice where even the latest revision is sent as a number causes some unnecessary database meddling in the checker. It should be better to send None, and then stamp the answer with the latest revision number after checking instead.

Changed. Latest revision is saved into the answer object, but sent as None to the checker in courses.views.check_answer.


Incompatble with trashslot

Minecraft Version: 1.16.5
Mod Version:  1.16.2-1.1.2
Single Player or Server:
Singleplayer
Describe problem (what you were doing; what happened; what should have happened):
I installed it with trashslot, the trashslot disappeared.

Trashslot has a hotkey to hide it. You need to look under your controls to figure out how to re-enable it.


Bump BC to 1.69

The new version has already been released. Upgrade once the release hits maven central and hopefully enjoy FLAWLESS TRAVIS BUILDS :)

We can tackle this, once https://github.com/bcgit/bc-java/issues/969 is resolved.


Fix checkbox click area (#358)

Fix for #358: If we use block here then labels span the width of the page. If we use inline-block then they collapse, but wrap together side by side. If we use table then they collapse but still stack under one another = win!
Checklist:

[ ] Documentation N/A
[x] Tests
[x] Ready to be merged

Codecov Report

Merging #392 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #392   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          77     77
  Lines         358    358
  Branches       16     16
=====================================
  Hits          358    358

Impacted Files
Coverage Δ

components/checkbox/src/index.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dca99e3...eaeadfe. Read the comment docs.

When adding bugfixes can you try to think if there’s a unit test that could show the issue beyond a snapshot test?

Also I’m not sure this is something we need to fix.
What is the default html behaviour?
What is the govuk-frontend and predecessor behaviour?

It's caused by the wrapping label element, which I believe can be fixed by setting inline-block on it, not sure why that didn't fix it if that was attempted?

It might be worth considering using flexbox here as it allows better customisation of the contained elements?

If we set inline-block just on the label element then it stops the full-width block behaviour but does mean all the labels wrap together on the same line.

I can't see why you would need a LabelContainer using flexbox?

display flex and flex direction column on the label would work?

Yea it doesn't need an additional container, but it suffers from the same problem that display block does.
If we are really going to copy GDS CSS and we want things like this to work, we will probably need to copy the markup as well (no biggy really imho), as it is simply the behaviour of the label element that is kicking off.

Yes, having now seen the govuk-frontend examples I notice each component has a wrapping div which contains the input and label, whereas those in govuk-react don't include this. It also uses a label, not a span, for the text, and sets the for attribute on it and an id on the checkbox to link the two.
I've just added a commit to make the structure of ours resemble this. The tests could probably be improved a bit but take a look, also in Storybook - working well! n.b. It does require that wherever this is used an id is sent in.

Just remembered labels are actually one place where we're intentionally doing something to govuk-frontend.
In govuk-frontend, labels don't wrap form elements, but this requires form elements to have unique IDs.
As React doesn't require form elements to have IDs, and due to the complexity of producing consistent IDs for form fields that would (a) work with universal rendering (b) be easy for the parent application to implement in all scenarios, we decided to make labels wrap the form element.
Perhaps that's what the issue is here?

Yes, that's certainly related, @penx, as per my commit above. (btw I just forced pushed an update to pass tests - ESLint does not pick up all files with the current glob.)
If we want to avoid ids and keep the label on the outside, containing the checkbox, we can give this a bit more thought - probably just add a wrapping div.

If we want to avoid ids and keep the label on the outside

Yep, well not avoiding them altogether, just not making them mandatory in order for label association to work.

Sometimes the simplest things are best! Check out the latest commit compared with the previous fix. This works no problem so perhaps we are there.

I'm not a fan of anonymous divs personally, but not enough to fight for it - I'd personally find a way to link the label and the input in a decent, instance related way, but yea, each to their own! : )

I'm not a fan of anonymous divs personally

Are you against what we are trying to achieve, i.e. an inline block inside a block? If not, an anonymous div seems like the best approach, so not sure why you'd be against it for this usage.
Yes I think the className should be on the outer div.
Happy to recommend we pause this until we can all discuss in person if you're not happy :-)

Hi. Perhaps the above will be best - we end up with this:
<StyledContainer className={className}>
  <StyledLabel>
    <StyledInput type="checkbox" {...props} />
    <StyledSpan>{children}</StyledSpan>
  </StyledLabel>
</StyledContainer>

I'm only against it if it doesn't have the className prop on it as it is no longer anonymous!

I've taken a look at this too now...
I'm not a fan of the approach taken.  Adding in more markup to work around a problem always feels slightly dodgy to me.
Additionally I disliked the original styling solution too - as overriding font choices wasn't possible.
This is my preferred approach: https://codesandbox.io/s/43p4wm4p89

This seems to use label's default inline-block behaviour and <br> tags to achieve line breaks between each label? However when I try two of them the display isn't looking right. https://codesandbox.io/s/8lpoy2z5yl
Maybe this is not an issue if these labels are always expected be used inside divs.

Steve's confirmed that with his suggested implementation the labels would align on one line due to the inline-block behaviour of labels, and line breaks would be achieved by using container divs at implementation level.
This doesn't match the govuk-frontend structure, which I gather is something @penx wanted to do (that has a wrapping div) - but then that is also different with its for attribute approach. :)
My last suggestion has been approved but required tests haven't completed so I can't merge. @stevesims Maybe you could implement your suggestion here instead or merge this.

@gavinorland closing this for now. Feel free to revisit comments above and reopen later


migrate the other CI to python 3.11

(namely, additional CI and upstream-dev CI)
python=3.11 has been released more than a year ago and python=3.12 is out as well, which means that it is a good idea to migrate sooner than later.
Regarding python=3.12, usually it is numba that keeps us from testing on a new python version for some time, where numbagg and sparse are the only dependencies that would use it. Should we create a environment without those two dependencies and switch back to the normal one once numba supports the new python version?
We still have the special environment files for python>=3.11 because the normal ones still include cdms2. We deprecated that back in May – not sure which release that ended up in – but since cdms2 will be abandoned end of this year, that's when we're free to drop support and merge both environments (though maybe we can justify dropping support earlier?)

Sounds great on all counts!

Shall we merge?

before merging we probably need to silence the cgi deprecation warning raised by one of pydap's dependencies (pydap is imported in xarray.tests.__init__).


Make TTL configurable via config and reconcile it

We currently default to 1 week and tell users to customize it using alter table default.flows_raw modify ttl Date + toIntervalDay(20).
https://github.com/monogon-dev/NetMeta/blob/2bab7d91da7c8fd97e8a414989f42c8b8dfd2b69/deploy/single-node/schema/0001_create_flows_raw.sql#L79
Now that we auto-reconcile, we should make this a config item and have the reconciler apply it automatically (+making sure it gets applied to old partitions, maybe CH even does this automatically now).

This is harder than it seems as we have to prevent users to accidentally nuke all their data. My current Idea is to move all operations to the reconciler including the creation of flows_raw but still keep migrations around for destruction of things. Also if the reconciler wants to reduce the ttl, delete anything etc. it should cancel and throw a warning.
Maybe we can create a table where we write log messages into and display them inside grafana to not have the user run in the dark?


A question about pre-computed predictions of LAGO and fCLSWGAN

Dear @yuvalatzmon,
first of all let me thank you for having shared the code of your CVPR'19 paper, which I found very interesting!
I was playing with your code, which works perfectly, but I have a few questions that I would like to humbly ask you. I am referring to the .npz files in which you stored the predictions of LAGO and fCLSWGAN (XianGAN) backbones.

To my understanding, in "pred_gzsl_test.npz" you concatenated the predictions carried out on test instances from both seen and unseen classes. Can you please tell me in which order you stacked the two groups? I suppose that you stored seen first and then unseen ones, but I would like to double-check this.

When loading "pred_gzsl_val.npz" from XianGAN, I am experiencing a mismatch in  the dimensions with the actual validation set proposed in [The Good, the Band and the Ugly of ZSL, Xian et al. CVPR 2017]. For instance, on AWA1, val_loc indexes have 7926 entries, when I load pred_gzsl_val.npz I am having a matrix with 11330 logits. I am right in saying that there should be a precise match in those dimensions?

Thank you very much in advance for your reply.

Dear Jacopo,
Thank you for your interest in our paper!
About Q1:
The labels in data['Y_GZSLtest'] correspond with the samples of
"pred_gzsl_test.npz". You can see in Line 112 of main_cosmo.py
https://github.com/yuvalatzmon/COSMO/blob/master/src/main_cosmo.py#L112,
that I first stack the samples of the seen classes and then the samples of
the unseen classes.
As another alternative, you can infer it from that labels by doing
np.in1d(data['Y_GZSLtest'], data['seen_classes_test'])
About Q2:
There are two sources of (legit) mismatch with val_loc:
(1) Our GZSLval split add samples of seen classes to the validation set,
which we take from the training set. The original split by Xian does not
has samples of seen classes.
(2) Our validation set uses 6372 samples of unseen-val classes
(print(np.in1d(data['Y_GZSLval'], data['unseen_classes_val']).sum()).
This is lower than 7926 samples (in Xian's val_loc),  because 1554
samples in val_loc overlap with the seen test set. Therefore, we take
them out in order to avoid overfitting. You can see it in line 254 in
load_xian_data.py
https://github.com/yuvalatzmon/COSMO/blob/master/src/load_xian_data.py#L254
https://github.com/yuvalatzmon/COSMO/blob/master/src/load_xian_data.py#L254
On Sat, Oct 19, 2019 at 11:30 AM JacopoMath<EMAIL_ADDRESS>wrote:

Dear @yuvalatzmon https://github.com/yuvalatzmon,
first of all let me thank you for having shared the code of your CVPR'19
paper, which I found very interesting!
I was playing with your code, which works perfectly, but I have a few
questions that I would like to humbly ask you. I am referring to the .npz
files in which you stored the predictions of LAGO and fCLSWGAN (XianGAN)
backbones.

To my understanding, in "pred_gzsl_test.npz" you concatenated the
predictions carried out on test instances from both seen and unseen
classes. Can you please tell me in which order you stacked the two groups?
I suppose that you stored seen first and then unseen ones, but I would like
to double-check this.
2.
When loading "pred_gzsl_val.npz" from XianGAN, I am experiencing a
mismatch in the dimensions with the actual validation set proposed in [The
Good, the Band and the Ugly of ZSL, Xian et al. CVPR 2017]. For instance,
on AWA1, val_loc indexes have 7926 entries, when I load
pred_gzsl_val.npz I am having a matrix with 11330 logits. I am right
in saying that there should be a precise match in those dimensions?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yuvalatzmon/COSMO/issues/1?email_source=notifications&email_token=AD6DAM3ORVPT4MO5RNTQGU3QPLASJA5CNFSM4JCOZUTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HS5CSVQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DAMZK5UXFB2PDXQCHV5LQPLASJANCNFSM4JCOZUTA
.

Dear @yuvalatzmon,
thank you very much for your quick response!! I hope not to abuse your patience with another quick question that I would like to kindly ask you.
In practice, your validation set is the concatenation of two set of indexes: 1) the val_loc (once you elimitated any overlap with test_seen_loc) and 2) the 2:1 split that you computed here https://github.com/yuvalatzmon/COSMO/blob/a8875607c7dde60ad05a7e7ddb733a2a2af0168a/src/gating.py#L32 and for which you did not applied any shuffling. Am I correct in saying that?
Thank you again!

Hi Jacopo,
Yes, I did not shuffle the validation set data.
Yuval
On Sun, Oct 20, 2019 at 7:52 AM JacopoMath<EMAIL_ADDRESS>wrote:

Dear @yuvalatzmon https://github.com/yuvalatzmon,
thank you very much for your quick response!! I hope not to abuse your
patience with another quick question that I would like to kindly ask you.
In practice, your validation set is the concatenation of two set of
indexes: 1) the val_loc (once you elimitated any overlap with
test_seen_loc) and 2) the 2:1 split that you computed here
https://github.com/yuvalatzmon/COSMO/blob/a8875607c7dde60ad05a7e7ddb733a2a2af0168a/src/gating.py#L32
and for which you did not applied any shuffling. Am I correct in saying
that?
Thank you again!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yuvalatzmon/COSMO/issues/1?email_source=notifications&email_token=AD6DAMY3UGKVOSNJRVNBSWDQPPPZLA5CNFSM4JCOZUTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYCLOA#issuecomment-544220600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DAM5YDTBIK3M4RI43EHDQPPPZLANCNFSM4JCOZUTA
.

Thank you very much, but I still see something unclear to me: on AWA1, if I take the train_loc of Xian et al. CVPR 2017 and perform your recommended 2:1 subsampling, I get a 8432-dimensional vector. Then, I concatenate to it all indexes which are in val_loc once the commonalities with test_seen_loc are removed. As a result, I obtain a total validation set of 14804 elements.
Unfortunately, I still see a mismatch with the shape of the 11330 logits that you provided in your pred_gzsl_val.npz file (XianGAN, AWA1).
I am sorry to still bothering you, but I am really puzzled.
Thank you in advance for your reply.

Hi Jacopo,

if I take the train_loc of Xian et al. CVPR 2017 and perform your
recommended 2:1 subsampling, I get a 8432-dimensional vecto

That's not what we are doing. At the first phase of cross validation, we
take the seen train set (minus seen val set and seen test set)  to train
the experts. (see "X_seen_train" at link
https://github.com/yuvalatzmon/COSMO/blob/master/src/load_xian_data.py#L326
an
link2
https://github.com/yuvalatzmon/COSMO/blob/master/src/cosmo_misc.py#L464)
The 2:1 sub sampling is only used to train the "gating" model. For that we
sub sample the "GZSLval" *validation *set, since it helps us to simulate
response of the experts to "unseen" classes.
I apologize for my slow responsiveness,  I am before a tight deadline and
my bandwidth is limited.
On Sun, Oct 20, 2019 at 5:18 PM JacopoMath<EMAIL_ADDRESS>wrote:

Thank you very much, but I still see something unclear to me: on AWA1, if
I take the train_loc of Xian et al. CVPR 2017 and perform your
recommended 2:1 subsampling, I get a 8432-dimensional vector. Then, I
concatenate to it all indexes which are in val_loc once the
commonalities with test_seen_loc are removed. As a result, I obtain a
total validation set of 14804 elements.
Unfortunately, I still see a mismatch with the shape of the 11330 logits
that you provided in your pred_gzsl_val.npz file (XianGAN, AWA1).
I am sorry to still bothering you, but I am really puzzled.
Thank you in advance for your reply.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yuvalatzmon/COSMO/issues/1?email_source=notifications&email_token=AD6DAM2GX666DSD7D45BDVLQPRSBTA5CNFSM4JCOZUTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYLD4I#issuecomment-544256497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DAM43OIKFLDWTQ3ZXDH3QPRSBTANCNFSM4JCOZUTA
.

Thanks again for your kind reply, I tried to go back in your code exploiting the links you provided, I appreciated it! Unfortunately, it is still unclear to which data do the 11330 logits in /COSMO/data/XianGAN_predictions/AWA1/pred_gzsl_val.npz  refer to.

Hi Jacopo,
I apologize again for the delayed reply. I'll try to clarify better.
train_loc has 16864 samples, which overlaps with test_seen_loc by 3404
samples. This leaves us with 13460 samples that we can use for training /
validation.
Of those samples, we sample 4958 for the seen validation set. We join
these samples with the 6372 unseen validation samples (that we discussed
earlier) and get a total of 4958+6372=11330 GZSLval samples.
As a side note: Taking 4958 samples for validation leaves us with
13460-4958=8502 samples for training the first phase of cross validation.
I hope it is helpful
Best!
Yuval
On Mon, Oct 28, 2019 at 5:22 AM JacopoMath<EMAIL_ADDRESS>wrote:

Thanks again for your kind reply, I tried to go back in your code
exploiting the links you provided, I appreciated it! Unfortunately, it is
still unclear to which data do the 11330 logits in
/COSMO/data/XianGAN_predictions/AWA1/pred_gzsl_val.npz refer to.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yuvalatzmon/COSMO/issues/1?email_source=notifications&email_token=AD6DAM3P7F355AHFQNGXLA3QQZLG7A5CNFSM4JCOZUTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECLSKEQ#issuecomment-546776338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6DAMZ3K5S2KQ2ZAE5BIE3QQZLG7ANCNFSM4JCOZUTA
.

Thank you very much Yuval (and sorry for my late reply too). I think that now things are clear. Thanks for your time!


allow_dependency_failure: true doesn't apply to timed out builds

Hey folks, we ran into an issue where a job timed out and it didn't trigger a dependency who's allow_dependency_failures flag was set to true.

Our YAML looks something like this:
  - label: Deploy
    agents:
      queue: ${QUEUE}
    timeout_in_minutes: 30
    concurrency: 1
    key: deploy-step
    ....
  - label: Send Slack Alerts
    command: ".buildkite/scripts/deploy_alerts_to_slack.rb"
    depends_on: deploy-step
    allow_dependency_failure: true

Is this a known issue? If not, are we doing something wrong?

It feels like a discrepancy, at minimum. Given the following pipeline:
steps:
  - label: Time out
    key: time-out
    timeout_in_minutes: 1
    command: sleep 120

  - wait: ~
    key: continuing-wait
    continue_on_failure: true

  - label: Test A
    command: "true"

  - wait:

  - label: Test B
    depends_on: time-out
    allow_dependency_failure: true
    command: "true"

  - label: Test C
    depends_on: continuing-wait
    allow_dependency_failure: true
    command: "true"

Test A passes
Test B sits there with the "waiting" clock, like your example above
Test C passes

Hey,
That's correct; timeout is treated as a failure regardless of what you've been accepted as a failure because we presume an error that needs manual interventions and is not something that can be gracefully recovered. It's a design decision.
Nevertheless, we will discuss this behavior in more detail with the team.

Hi folks, I’m going to close this issue as it isn’t so much an issue in the Buildkite Agent but a property of the Buildkite Pipelines product. We appreciate you taking the time to open an issue to give us this feedback :bow:


Write about network in readme

Added network line in readme https://github.com/grebtsew/dcDeployControl/commit/fbf84ce19f73779423c05158a017bc280d4bc620


Validation

Whatever you do here, lets submit an identical PR for vagrant on vaquero-examples

@stephenrlouie, good point. Removed (already added to examples)


Cannot create a client properly via API (500 server error)

I am using Chef server 12.3.1 on CentOS 7.
Related API document: https://docs.chef.io/api_chef_server.html#id19
The client is Ubuntu 12.04, chef is on 12.5.1 and is using this command to register a client:
knife exec -u $validator_username -c /etc/chef/client.rb -k /etc/chef /etc/chef/validator.pem << EOF
client_desc = {
  "name" => "CHEF-TESTVM",
  "admin" => false
}
new_client = api.post("/clients", client_desc)
puts new_client
EOF

Response:
{"uri"=>"https://chef-server.local/organizations/test/clients/CHEF-TESTVM"}

That's it. There is no private key nor public key shown like the API document says.
The server log:
2015-11-28_19:49:38.37621 Started PUT "/organizations/test/clients/CHEF-TESTVM" for <IP_ADDRESS> at 2015-11-28 19:49:38 +0000
2015-11-28_19:49:38.37826 Processing by ClientsController#update as JSON
2015-11-28_19:49:38.37827   Parameters: {"organization_url"=>"/organizations/test", "id"=>"CHEF-TESTVM", "name"=>"CHEF-TESTVM", "public_key"=>"-----BEGIN PUBLIC KEY-----", "validator"=>false, "admin"=>false, "json_class"=>"Chef::ApiClient", "chef_type"=>"client", "access"=>{"read"=>true, "create"=>true, "grant"=>true, "update"=>true, "delete"=>true}, "private_key"=>true, "organization_id"=>"test", "client"=>{"organization_url"=>"/organizations/test", "id"=>"CHEF-TESTVM", "name"=>"CHEF-TESTVM", "public_key"=>"-----BEGIN PUBLIC KEY-----", "validator"=>false, "admin"=>false, "json_class"=>"Chef::ApiClient", "chef_type"=>"client", "access"=>{"read"=>true, "create"=>true, "grant"=>true, "update"=>true, "delete"=>true}, "private_key"=>true}}

In the GUI, the client is successfully registered, but when I click that client: it gives: An error occured fetching the details of the client. Please try again later.
When I do get clients/CHEF-TESTVM it gives 500 server error.
Related server error:
Started GET "/organizations/test/clients/CHEF-TESTVM" for <IP_ADDRESS> at 2015-11-28 19:51:50 +0000
2015-11-28_19:51:51.00438 Processing by ClientsController#show as JSON
2015-11-28_19:51:51.00440   Parameters: {"organization_id"=>"test", "id"=>"CHEF-TESTVM"}
2015-11-28_19:51:51.04504 Client data contained neither a certificate nor a public_key.
2015-11-28_19:51:51.04505 /opt/opscode-manage/embedded/service/opscode-manage/app/controllers/clients_controller.rb:45:in `object_for_id'
...

Found out what problem is.
When request, the param can be set as:
client_desc = {
  "name" => "CHEF-TESTVM",
  "admin" => false,
  "create_key" => true
}

Please update the doc, thanks!

@chef/chef-tech-writers @jamescott (i hope i have the right team)

PR has been made https://github.com/chef/chef-web-docs/pull/766

As per the doc https://docs.chef.io/api_chef_server.html#clients seem docs have been updated with
 "create_key": true
params:
{
  "name": "name_of_API_client",
  "create_key": true
}

So we are good to close this issue. Thanks!


locallog Refreshing a player's locallog when they leave/get kicked (Errors)

Errors.

Still not fixed

Better hecc'in fix this already or reopen this issue.

ugh what

how bout now?


fix: use FakeWeakRef when WeakRef is unavailable

FakeWeakRef is a useful construct for clients that lack WeakRef support. In particular, there are still many Android devices that don't have WeakRef available.
This change adds FakeWeakRef fallback to Restorer and Modalizer.

I see, that makes sense. We support the last 4 major Android versions and 2 major iOS versions.
From what we've seen WeakRef support is present in Android browser versions >=84, consistent with the support matrix you linked to. Since 4 versions ago was Android 11 and Chrome 84 was released before Android 11, it sounds like our supported devices are in alignment.


qbittorrent login failed, reason: self signed certificate

I setup my docker compose and setup the config as needed. When i run docker compose up -d. I am getting the following error, due to my qbittorrent using Https self signed certificate. Is there a way to skip SSL verification that can be added to the config?
error: qBittorrent login failed: request to https://<IP_ADDRESS>:8888/api/v2/auth/login failed, reason: self signed certificate

Self signed untrusted certificates for internal https communication are nothing more than security theater and there's really no use case to do that.

If your LAN is being ease-dropped on....you've already lost the game since they presumably already have local and physical access


Cannot use ionic-app-scripts custom config files in command line

What version of the CLI are you using?
2.2.1
Short description of the problem:
Cannot use ionic-app-scripts custom config files path in command line.
If you run
ionic build android --release --prod -s ./config/sass.config.js -r ./config/rollup.config.js
Build fails with error
Error: Unknown platforms: ./config/sass.config.js, ./config/rollup.config.js
What behavior are you expecting?
Ionic command line should accept ionic app scripts parameters
Steps to reproduce:

copy  config files from \node_modules@ionic\app-scripts\config to config directory
run

ionic build android --release --prod -s ./config/sass.config.js -r ./config/rollup.config.js

Post the output of ionic info below please
ordova CLI: 6.5.0
Ionic Framework Version: 2.2.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v6.4.0
Xcode version: Not installed
Workaround: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
You can change  \node_modules@ionic\app-scripts\config directly

This issue was moved to driftyco/ionic-app-scripts#862


bug(python): Error trying to create Colbert reranker

LanceDB version
0.14.0
What happened?
I used a code sample in these docs, but get an error:
ImportError: Please install rerankers

Details below.
Are there known steps to reproduce?
I tried this on a fresh conda env
conda create -n lancedb_reranker python=3.11.5
conda activate lancedb_reranker

in py shell:
Python 3.11.5 (main, Sep 11 2023, 08:31:25) [Clang 14.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from lancedb.rerankers import ColbertReranker
>>> colbert = ColbertReranker()
Traceback (most recent call last):
  File "/Users/tleyden/miniconda3/envs/lancedb_reranker/lib/python3.11/site-packages/lancedb/util.py", line 154, in attempt_import_or_raise
    return importlib.import_module(module)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/tleyden/miniconda3/envs/lancedb_reranker/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1140, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'rerankers'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/tleyden/miniconda3/envs/lancedb_reranker/lib/python3.11/site-packages/lancedb/rerankers/colbert.py", line 41, in __init__
    super().__init__(
  File "/Users/tleyden/miniconda3/envs/lancedb_reranker/lib/python3.11/site-packages/lancedb/rerankers/answerdotai.py", line 50, in __init__
    rerankers = attempt_import_or_raise(
                ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/tleyden/miniconda3/envs/lancedb_reranker/lib/python3.11/site-packages/lancedb/util.py", line 156, in attempt_import_or_raise
    raise ImportError(f"Please install {mitigation or module}")
ImportError: Please install rerankers
>>>

Thanks for reporting.. The error asks you to install rerankers. It can be installed by pip install rerankers. We'll update the docs to better point out extra/optional dependencies for each reranker.

thanks - would be good to have the error message also emit run "pip install reranker"


Support g++ extensions to <special-name> that are de facto standard

Total number of libxul symbols:                       274346
Number of libxul symbols parsed:                      274319 (99.99%)
Number of libxul symbols demangled:                   274319 (99.99%)
Number of libxul symbols demangled same as libiberty: 196365 (71.58%)


Centralise handling of unfurling a bit and unfurl replies

Previously, we would not unfurl responses to our own messages, which
meant we ended up with messages like "<@U12341|foo>: hi" instead of
"@foo: hi". This fixes that.

:+1:


ReferenceError: assignment to undeclared variable field

priceToFloat generate this error.
Even demo page http://flaviosilveira.com/Jquery-Price-Format/ shows the error selecting price to float

What of your browser and version @ClaPro?

Firefox 54.0.1


Encoder-Decoder Architecture Issues

Hello, author, I am very interested in your work. This is a great discovery. I would like to ask why this paper only uses one Encoder architecture to outperform other Encoder-Decoder architectures. This is what I am confused about. I look forward to your reply.

In theory, shouldn't the features extracted using the Encoder-Decoder architecture be better? I look forward to your reply. Thanks again.

Hi, I think it depends on the problem. Maybe this paper would be helpful.

Okay, thank you very much
Yong Liu @.***> 于2024年6月14日周五 15:29写道：

Hi, I think it depends on the problem. Maybe this paper
https://arxiv.org/abs/2204.05832 would be helpful.
—
Reply to this email directly, view it on GitHub
https://github.com/thuml/iTransformer/issues/88#issuecomment-2167408966,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7D5MMOS5PI4ZXJ5HDGKR5LZHKLXDAVCNFSM6AAAAABJJZAOSWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRXGQYDQOJWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


[2pt] set up your Kanban board

set up your Kanban board
PROJECT LINK
ASSIGNED TEAM MEMBERS

Team member N.1: @andym80
Team member N.2: @AmaliaGM
Team member N.3: @carlosmuerto

Project requirements

Join a Zoom call with your team. Only one person needs to copy the board on behalf of the entire team but everybody should help during the call.
Create an empty repository for your group project.
Follow the steps described in curriculum-tools-copy-projects in order to copy the [TEMPLATE] Ruby Group Capstone Project board.
Open the GitHub project (Kanban board) in your team repo.
Open project requirements and read them carefully.
Open project requirements divided by team member and compare it with the project requirements.
Create tasks on your Kanban board based on the project requirements divided by team member.
Each task should be a separate card that:

Has a descriptive title.
Has the number of points in the [Xpt] prefix. That number should indicate the number of points that show how complex the task is. 1 point = 1 hour of estimated work. You shouldn't change the points after dividing up cards, even if the actual work takes longer.
Has a description of the task.

Each of the team members should assign themselves to the cards as divided in the project requirements divided by team member (and time estimate distributions should be fair - no single person should be doing all of the most time-consuming tasks). Decide who will be the assignee for the cards marked as group tasks. All cards on your Kanban board should have assignees.
Open a new issue in your repository.

In its description, paste a link to your GitHub project.
Add a note telling the reviewer how many people there are in your team.
Add a note telling the reviewer which team member was assigned to tasks listed for: Team member #1, Team member #2, Team member #3 Team member #4 (if your team has 4 members). Use your GitHub usernames.

Use the issue link to ask for a review. A code reviewer will leave you feedback in the issue's comment.

As it is your first time using a GitHub project and Kanban board, we need to double-check your setup to avoid problems while working on your project.

FULL README

Hi Team,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights

good job touching every subject with the cards

Required Changes ♻

[ ] The columns used should be the same as in the template provided by Microverse
[ ] Each card should have a description of the task
[ ] not all cards have points assigned to them for example: "Implement startup actions: Present the user with a list of options to perform."
[ ] the differences in workload (points) per student are too big 44, 33 & 38,5 please make sure to divide the work more evenly.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @carlosmuerto @andym80 @AmaliaGM 👋,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are still some which aren't addressed yet.
Highlights

All information of team members provided 👍
Link to board correctly 👍

Required Changes ♻

As mentioned by the previous CR kindly add descriptions to all your tasks as some tasks don't have descriptions 🙏

Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @davidosky007 in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @carlosmuerto ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Product update

Fixes #9
Roadmap

[ ] Fix updates when only localised fields are updated. When only localised fields are updated, the query is stripped from data because no data were sent to the buildQuery function, which considered there wasn't any data to update

[GitHub Actions]
Deployment SUCCESS
Commit 2fe0f2ad559ae8e8979e583ef5177304acd201ae successfully deployed to https://nrn-admin-3e0ftphg9.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit 0f29ff0712fd7a82d60c313a47dcbb031cf426ae successfully deployed to https://nrn-admin-ji976sfxe.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit 6a0af3a894b26f8c3a38f2fe9ee085cac6343759 successfully deployed to https://nrn-admin-hrj9rzicf.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit fd24840f3166760e606047cb6f5637e646e5fd4e successfully deployed to https://nrn-admin-61p7swyyk.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
Deployment SUCCESS
Commit 44f1a2dd136638e10102906c664c5e61936185a0 successfully deployed to https://nrn-admin-awpydqb0k.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top

[GitHub Actions]
Deployment SUCCESS
Commit bd3f6dc6dd03504e0200a853e42d1e6a08cf634c successfully deployed to https://nrn-admin-a8zk953d7.now.sh
Deployment aliased as https://nrn-admin-product-update.now.sh

[GitHub Actions]
E2E tests FAILED
Download artifacts (screenshots + videos) from checks section at the top


Cannot replace a form value, for a form field, where multiple occurrences of the form field exist

Hey @Hopding,
Thank you for the awesome repository, it has been really useful.
I've created a demo using pdf-lib to fill out form fields in an AcroForm PDF to look to alternatives to an existing solution using iTextPDF. The functions have come from examples in some of the other issues, and would actually be quite nice exposed as part of the pdflib api! :) Happy to help do this, as I think it would be really useful!
Issue:-  Cannot replace a form value, for a form field, where multiple occurrences of the form field exist
Expected Behaviour:- A single name reference can be provided, and this value is used to fill in all form fields matching that name reference.
Actual Behaviour:- An exception is thrown where we have two fields with the same name, as they are referenced as instalments#0 & instalments#1 in the adobe form
Error: Expected instance of PDFArray, but got instance of undefined

Steps to reproduce:-
I have created a minimal example here - https://github.com/YOU54F/pdf-form-node
It contains a test.pdf with 3 form fields.

Filling in a form field, where it only exists once in a document
To run:
npm install
npm run single

Reads the test.pdf template and outputs as filled_single.pdf.
QPDF is called to correct filled_single.pdf and outputs as fixed_single.pdf

Filling in a form field, where it exists more once in a document
To run:
npm install
npm run multiple

Reads the test.pdf template and outputs as filled_multiple.pdf.
QPDF is called to correct filled_multiple.pdf and outputs as fixed_multiple.pdf

I actually think this will work, as it has correctly read there are three fields, when I have just ran it against the above pdf. Will test out and report back!
https://github.com/Hopding/pdf-lib/issues/349#issuecomment-585508339

So that has found the three fields, just trying to fill values on the kids now
➜  pdf-form-node git:(master) ✗ ts-node nested.ts
rootField
<<
/AP <<
/N 105 0 R
>>
/BS <<
/S /S
/W 0
>>
/DA (/Helv 6 Tf 0 g)
/DV (this is a single field)
/F 4
/FT /Tx
/MK <<
>>
/P 19 0 R
/Rect [ 264.379 488.376 504.748 528.144 ]
/Subtype /Widget
/T (single)
/Type /Annot
/V (this is a single field)
>>

rootField
<<
/DA (/Helv 6 Tf 0 g)
/DV (this is text to be replaced entered in a mulitple form fields, the text get shown on all fields)
/FT /Tx
/Kids [ 35 0 R 36 0 R ]
/T (multiple)
/V (this is text to be replaced entered in a mulitple form fields, the text get shown on all fields)
>>

Total fields: 2
childField
<<
/AP <<
/N 105 0 R
>>
/BS <<
/S /S
/W 0
>>
/DA (/Helv 6 Tf 0 g)
/DV (this is a single field)
/F 4
/FT /Tx
/MK <<
>>
/P 19 0 R
/Rect [ 264.379 488.376 504.748 528.144 ]
/Subtype /Widget
/T (single)
/Type /Annot
/V (this is a single field)
>>

childField
<<
/AP <<
/N 104 0 R
>>
/BS <<
/S /S
/W 0
>>
/F 4
/MK <<
>>
/P 19 0 R
/Parent 102 0 R
/Rect [ 264.775 379.39 505.144 419.158 ]
/Subtype /Widget
/Type /Annot
>>

childField
<<
/AP <<
/N 103 0 R
>>
/BS <<
/S /S
/W 0
>>
/DA (/Helv 6 Tf 0 g)
/F 4
/MK <<
>>
/P 19 0 R
/Parent 102 0 R
/Rect [ 266.098 274.644 506.467 314.412 ]
/Subtype /Widget
/Type /Annot
>>

Total childFields: 3

Looks like the kids don't have this property
    PDFName { encodedName: '/T' } => PDFString { value: 'single' },

but could look up that valuee on the parent, as it has a parent value
    PDFName { encodedName: '/Parent' } => PDFRef { objectNumber: 102, generationNumber: 0, tag: '102 0 R' },

found a field matching multiple: acroField: TRUE which has the the fieldName undefined

PDFDict {
  dict: Map {
    PDFName { encodedName: '/AP' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/BS' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/F' } => PDFNumber { numberValue: 4, stringValue: '4' },
    PDFName { encodedName: '/MK' } => PDFDict { dict: Map {}, context: [PDFContext] },
    PDFName { encodedName: '/P' } => PDFRef { objectNumber: 19, generationNumber: 0, tag: '19 0 R' },
    PDFName { encodedName: '/Parent' } => PDFRef { objectNumber: 102, generationNumber: 0, tag: '102 0 R' },
    PDFName { encodedName: '/Rect' } => PDFArray { array: [Array], context: [PDFContext] },
    PDFName { encodedName: '/Subtype' } => PDFName { encodedName: '/Widget' },
    PDFName { encodedName: '/Type' } => PDFName { encodedName: '/Annot' }
  },
PDFDict {
  dict: Map {
    PDFName { encodedName: '/AP' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/BS' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/DA' } => PDFString { value: '/Helv 6 Tf 0 g' },
    PDFName { encodedName: '/F' } => PDFNumber { numberValue: 4, stringValue: '4' },
    PDFName { encodedName: '/MK' } => PDFDict { dict: Map {}, context: [PDFContext] },
    PDFName { encodedName: '/P' } => PDFRef { objectNumber: 19, generationNumber: 0, tag: '19 0 R' },
    PDFName { encodedName: '/Parent' } => PDFRef { objectNumber: 102, generationNumber: 0, tag: '102 0 R' },
    PDFName { encodedName: '/Rect' } => PDFArray { array: [Array], context: [PDFContext] },
    PDFName { encodedName: '/Subtype' } => PDFName { encodedName: '/Widget' },
    PDFName { encodedName: '/Type' } => PDFName { encodedName: '/Annot' }
  },
found a field matching single: acroField: TRUE which has the the fieldName (single)
{
  dict: Map {
    PDFName { encodedName: '/AP' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/BS' } => PDFDict { dict: [Map], context: [PDFContext] },
    PDFName { encodedName: '/DA' } => PDFString { value: '/Helv 6 Tf 0 g' },
    PDFName { encodedName: '/DV' } => PDFString { value: 'this is a single field' },
    PDFName { encodedName: '/F' } => PDFNumber { numberValue: 4, stringValue: '4' },
    PDFName { encodedName: '/FT' } => PDFName { encodedName: '/Tx' },
    PDFName { encodedName: '/MK' } => PDFDict { dict: Map {}, context: [PDFContext] },
    PDFName { encodedName: '/P' } => PDFRef { objectNumber: 19, generationNumber: 0, tag: '19 0 R' },
    PDFName { encodedName: '/Rect' } => PDFArray { array: [Array], context: [PDFContext] },
    PDFName { encodedName: '/Subtype' } => PDFName { encodedName: '/Widget' },
    PDFName { encodedName: '/T' } => PDFString { value: 'single' },
    PDFName { encodedName: '/Type' } => PDFName { encodedName: '/Annot' },
    PDFName { encodedName: '/V' } => PDFString { value: 'this is a single field' }
  },

Got this working now nicely, will update my repo when I get time and post back the solution! Thanks again for the great repo

pdf-lib now has form creation and filling APIs that should be used instead of the above example(s). See the form filling JSFiddle for a working example. Additional information is available in the README and API docs. The PDFTextField.setText method is of particular relevance to this issue.

Thanks again @Hopding ! Look forward to trying this out 👍🏽


fix using nil error problem

This PR is a

[ ] Feature Implementation
[X] Bug Fix
[ ] Documentation

What this PR does / why we need it: fix using nil error problem
Which issue(s) this PR fixes

Fixes #
Please leave this checklist in the PR comment so that maintainers can ensure a good PR.
Merge Checklist:

[ ] New feature

[ ] Tests
[ ] Documentation

[ ] SVCat CLI flag
[ ] Server Flag for config

[ ] Chart changes
[ ] removing a flag by marking deprecated and hiding to avoid
breaking the chart release and existing clients who provide a
flag that will get an error when they try to update

/ok-to-test
/lgtm

/ok-to-test
/approve

/lgtm


Compile error on Flutter 2.2.2

Describe the bug
Compile error on Flutter 2.2.2
Config
Flutter 2.2.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d79295af24 (6 days ago) • 2021-06-11 08:56:01 -0700
Engine • revision 91c9fc8fe0
Tools • Dart 2.13.3
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.2.2, on Microsoft Windows [versÃ£o 10.0.19041.1052], locale pt-BR)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] IntelliJ IDEA Community Edition (version 2021.1)
[√] Connected device (3 available)
Error
Error: The parameter 'event' of the method 'AlwaysWinPanGestureRecognizer.addAllowedPointer' has type 'PointerDownEvent', which does not match the corresponding type, 'PointerEvent', in the overridden method, 'DragGestureRecognizer.addAllowedPointer'.

'PointerDownEvent' is from 'package:flutter/src/gestures/events.dart' ('/C:/src/flutter/packages/flutter/lib/src/gestures/events.dart').
'PointerEvent' is from 'package:flutter/src/gestures/events.dart' ('/C:/src/flutter/packages/flutter/lib/src/gestures/events.dart').
Change to a supertype of 'PointerEvent', or, for a covariant parameter, a subtype.

I just realized this is a flutter_colorpicker-0.5.0 package issue.
"(...).pub-cache/hosted/pub.dartlang.org/flutter_colorpicker-0.5.0/lib/src/hsv_picker.dart:730:43: Error: The parameter 'event' of the method 'AlwaysWinPanGestureRecognizer.addAllowedPointer' has type 'PointerDownEvent' "

Try these:

Downgrade to v2.2.0 instead of v2.2.0+1-dev.1

If that doesn't work then

Set pubspec to

dependencies:
   html_editor_enhanced: ^2.2.0

dependency_overrides:
   flutter_colorpicker: ^0.4.0

Those changes fixed the issue.
Everything is working great now.
Thank you.

Good to hear, sorry for the inconvenience.


Add Shell scripts for simulation

Init script add
Update .gitignore
Update requirements.txt
Add instructions in README.md

  File "/home/chrizandr/hydra-flock/hydra-flock-demo/sim_drone3/flock_drone/mechanics/distance.py", line 1, in <module>
    from haversine import haversine
ImportError: No module named 'haversine'

Also getting these errors

@chrizandr Forgot to add it to requirements, please run pip install haversine for now


Does TabNine support vscode-cpptools as its C++ semantic completion backend?

Cquery is slow on my computer. It sometimes takes 100% cpu to index files and produces thousands of index files. So I wonder if I can switch the sem backend to Microsoft VSCode C++ Extension? Since it seems that vscode-cpptools also implements the LSP according to https://microsoft.github.io/language-server-protocol/implementors/servers/

+1 on this. I know you can add language servers in the .config, I'm just not sure what the args needed for cpptools are

I recommend customizing the language server using the instructions at https://tabnine.com/semantic.


Halo2 circuit in mopro (redesigned)

Proposition on how we could add Halo2 Circuits to redesigned Mopro.
Overview
I attempted to be as consistent as possible to Circom implementation, resulting in defining two types Halo2ProveFn and Halo2VerifyFn which we expect the user to implement locally and then use the key_halo2_circuit_map as a way to map between different circuits and their implementations of prove and verify functions.
_Note that as Halo2 is a library, there is very little shared components, hence Mopro only defines these two types as well as uses key_halo2_circuit_map to invoke correct function at run time. _
Halo2 Module in test-e2e
I have also added a halo2 module in the test-e2e crate. It does not contain a halo2 circuit logic (I have extracted it into another repository as discussed before - https://github.com/ElusAegis/halo2-fibonacci-sample.git, which can be forked into Mopro organisation). What it has is logic to interface with the circuit, implementing logic to parse and prove the values.
As each circuit can use its own types, I did not see a way to abstract it away without enforcing strict user limitations. So halo2 is an example which users would need to reproduce. (Halo2 requires quite a bit of background knowledge).
Halo2/Circom Features
Currently, all users must use both Halo2 and Circom (define key_halo2_circuit_map and zkey_witness_map.  I have given a thought to how we could be able to disable and enable each of them, however with current design that does not seem to be possible unless we add something like halo2_placeholder!() which users need to specify if they do not use Halo2. This does not seem ideal UX.
Hence @vimwitch @vivianjeng , perhaps you have some views on how we could be able to do this? We may need to do something different than the current approach of making a user define a particular function.

@vimwitch @vivianjeng I have finished the PR. Can you please have a look.
Key changes were to app macros, to enable to have different adapters.
I have it now recursively call halo2_app and circom_app, which each have a fallback implementation when their respective feature is not implemented.
I have also played with how the test-e2e/lib.rs would look like, considering we would need conditionally compile it based on the flags, and I ended up doing it using macros, which are conditionally expanded, thus avoiding to write  #[cfg(feature = ...)] before each line.
I have added tests for iOS, however I have not updated Android as I am still unable to compile it.

@vimwitch thank you for the review, I agree with all of it besides how we show the setup to users. Will get to improving the

There are further 3 samples which I have created to show how Halo2 would work with Mopro:
- RSA Full Mopro example  (https://github.com/ElusAegis/halo2-rsa-mopro.git)
-  Keccak256 Mopro Halo2 Circuit (with use example in mopro/test-e2e) https://github.com/ElusAegis/halo2-keccak-stable.git
-  Fibonacci Mopro Halo2 Circuit (with use example in mopro/test-e2e) https://github.com/ElusAegis/halo2-fibonacci-sample.git

Benchmarking:
Keccak256:

M1 Pro iPhone simulator: 116s to prove and 1.4s to verify
iPhone 15 Pro: 200s prove time, 1.1s verify time

RSA:

M1 Pro iPhone simulator: ~450s proving , ~1s verifying time (without proving/verifying key generation)
iPhone 15 Pro:  Crashes due to memory limit exceeded (needs around 5GB RAM, with iPhone limiting to 3GB)

@vivianjeng thank you for the review, I have implemented the suggestions. However, after I have merged with the main and updated Cargo.lock it no longer runs the  iOS-app-simulator successfully. Yet it does on my local machine with a clean start.
I do remember we had some problems with non-determinism, could it be one of them? Or was there something in the recent PR that was merged that could have impacted it? I looked at it and nothing seems to be able to break the process at that point.

The iOS simulator on github has intermittent failures 😭. Usually if you read the log it will say something like "timed out" or similar.


How to use ESP-12S and blynk for get data for Ant smart BMS Thank you

Were you able to connect the bms to an esp32 or esp8266?


LND Account for Pending Closes

I have my LND test node and forced closed a bunch of channels. During the force close period the sats get not accounted for as "Incomming" - see what we can change to display those sats as incomming balance.

OK it gets now displayed on the LCD


Specifying the manifest path ignores .cargo/config.toml

Problem
When specifying --manifest-path=dir/Cargo.toml the .cargo/config.toml files appear to be ignored. It doesn't seem to matter whether .cargo/config.toml is (e.g., in my crates, workspace, or the directory from which I am running Cargo). It is always ignored.
Steps

Place some configuration in a .cargo/config.toml file. For example, I have:

[build]
target = "wasm32-unknown-unknown"

So that Cargo compiles to WebAssembly by default.
2. Run cargo build --manifest-path=path/to/Cargo.toml
3. Observe that instead of WebAssembly, Cargo is producing shared libraries (i.e., .so on Linux)
Possible Solution(s)
No response
Notes
No response
Version
cargo 1.57.0 (b2e52d7ca 2021-10-21)
release: 1.57.0
commit-hash: b2e52d7cab0a286ee9fcc0c17510b1e72fcb53eb
commit-date: 2021-10-21
host: x86_64-unknown-linux-gnu
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.79.1-DEV (sys:0.4.49+curl-7.79.1 vendored ssl:OpenSSL/1.1.1l)
os: Arch Linux Rolling Release [64-bit]

It should, unless something else is overriding it.  Can you show a complete example?  For example:
~/Temp> mkdir foo
~/Temp> cd foo
~/Temp/foo> mkdir .cargo
~/Temp/foo> cat > .cargo/config.toml << EOF
> [build]
target = "wasm32-unknown-unknown"
> EOF
~/Temp/foo> cargo new myproj
     Created binary (application) `myproj` package
~/Temp/foo> cargo build --manifest-path myproj/Cargo.toml
   Compiling myproj v0.1.0 (/Users/eric/Temp/foo/myproj)
    Finished dev [unoptimized + debuginfo] target(s) in 1.42s
~/Temp/foo> ls -al myproj/target/wasm32-unknown-unknown/debug/myproj.wasm
-rwxr-xr-x  1 eric  staff  1664827 Jan 17 08:40 myproj/target/wasm32-unknown-unknown/debug/myproj.wasm


ModuleNotFoundError  when enabling add-on after installation.

Installed the addon in blender. Then I checked the box to enable it and knock out the error that the module was not found.

rename addon folder to "Sollumz" (case-sensitive)


New feature request - Allow control of camera zoom from Valkyrie Editor

Description of Problem or Question
After a brief discussion with scrubbles on Discord I suspect I am not the only person who experiences very slow zooming using the middle mouse wheel, both in the Valkyrie Editor and also when playing custom content (Descent).
The main reason for needing to scroll is when large maps are placed. The user usually needs to pan and zoom to be able to see and therefore place the whole map. Slow zooming make this a real pain. One solution would be for the content creator to be able to set the camera zoom (e.g. on an event) in the editor. The camera can be zoomed out and the position set (as it is today) so that the entire map is visible.
Valkyrie Version
2.4.09
Attach Save, Quest or Scenario
Log File:
%LOCALAPPDATA%Low\NA\Valkyrie\output_log.txt
Saves:
%appdata%/Valkyrie/MoM/Save
%appdata%/Valkyrie/D2E/Save
Quest:
%appdata%/Valkyrie/D2E/Editor
Scenario:
%appdata%/Valkyrie/MoM/Editor

I'd like this function, it could just be a box you toggle. Min, Default, Max. Or have a number box 1-10 to set the camera zone with the default being prepopulated.
Anyway +1 for this request.

This ticket has overlap with #971


Add naming requirements to metadata for _validation of objects

As part of putting automation on top of the Azure SDK for Python, I found one of the missing pieces of information in the object metadata is the naming rules, for example maximum name length, allowed naming pattern.  This information is in the documentation, but not present in the generated code.  It would be really nice if the autorest-ed classes could add to validation, for example here is a mocked up example from class Subnet with "name" content and length rules (I made up the content, it is not based on any actual documentation):
      _validation = {
          'etag': {'readonly': True},
          'name': {'max_length': 64, 'regex': ...},
          'private_endpoints': {'readonly': True},
          'ip_configurations': {'readonly': True},
          'ip_configuration_profiles': {'readonly': True},
          'resource_navigation_links': {'readonly': True},
          'service_association_links': {'readonly': True},
          'purpose': {'readonly': True},
          'provisioning_state': {'readonly': True},
      }

One reason I would like this is to automatically compress and disambiguate names to improve automation success.  For example, some objects have a max name length of 64, some 80, some 16.  The process of fixing up those names to conform is much easier if the SDK advertises what the naming rules are.  And yes, even if this is in a "private" variable, it's still incredibly useful to consume for automation.

Hi @jeking3 this is supported by the code generator, and will generate those (you can find a lot of example in the repo). regex is called pattern, but otherwise you got max_length right :).
In your case, the specification in input didn't have validation information:
https://github.com/Azure/azure-rest-api-specs/blob/0eb469cc310ad03f1466eaf861bb01923b80f33f/specification/network/resource-manager/Microsoft.Network/stable/2021-05-01/virtualNetwork.json#L1681


Implement Python script to search random videos on YouTube

The script should be able to search random videos on YouTube which should be logged with a person's gmail id. The motive of the script is to break the bubble that the YouTube algorithm forms for any individual.
Here, bubble refers to the YouTube feeding similar type of content to the user based on his search and watch history.

So I have to give my youtube credentials along with with the search keyword in python?


Model Property Names using Custom Json Contract Resolver

Hi,
Cant find a way to use a custom json contract resolver with swashbuckle.
My stripped down example below;
var config = new HttpConfiguration();
config.Formatters.JsonFormatter.SerializerSettings.ContractResolver = new LowercaseUnderscoreContractResolver();
Swashbuckle.Bootstrapper.Init(config);
app.UseWebApi(config);
public class LowercaseUnderscoreContractResolver : DefaultContractResolver
{
protected override string ResolvePropertyName(string propertyName)
{
var regex = new Regex("(?!(^[A-Z]))([A-Z])");
return regex.Replace(input, "_$2").ToLower();
}
}

Please help me decide which Interface to use, IModelFilter or ISchemaFilter
https://github.com/domaindrivendev/Swashbuckle/blob/f5f3a187e1648e818778eddbdb4e1b6e35b6d83d/Swashbuckle.Core/Application/SwaggerDocsConfig.cs#L147
says
// NOTE: In next major version, ModelFilter will completely replace SchemaFilter
but
https://github.com/domaindrivendev/Swashbuckle/blob/07a7a247ef33ef9ba3afcd70b4e4c34937839427/README.md#transitioning-to-swashbuckle-50
says
v5
IModelFilter is now ISchemaFilter
and trying to figure out how to write a IModelFilter and looking at
https://github.com/domaindrivendev/Swashbuckle/blob/953f9803bdf4a5051dd4ad90f8a6bc0ad8b6af3b/Swashbuckle.Core/Swagger/XmlComments/ApplyXmlTypeComments.cs
as it doesn't use ISchemaFilter yet ....
Any suggestions on how to update it's JsonSerialisationConfiguration for this type as the JsonObjectContract isn't accessible?


make install-tools error

I have a new (intel based)mac and when I run make install-tools I get the following error:
Installing bin/fzn-or-tools 9.2.9972
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100     9    0     9    0     0     18      0 --:--:-- --:--:-- --:--:--    18
tar: Error opening archive: Unrecognized archive format
rm: or-tools__flatzinc_MacOsX-12.2.1_v9.2.9972.tar.gz: No such file or directory
rm: or-tools__flatzinc_MacOsX-12.2.1_v9.2.9972.tar.gz: No such file or directory
make: *** [hack/tools/bin/fzn-or-tools] Error 1

Duplicate of https://github.com/fybrik/fybrik/issues/1525

closed as duplicated


Why the Endless Tunnel sample has the dependency constraint-layout

Some of the samples (Native Activity) seem to have dependencies that should be related to Java apps. Why does for example the "Endless Tunnel" sample have these dependencies:
dependencies {
    compile fileTree(dir: 'libs', include: ['*.jar'])
    compile 'com.android.support:appcompat-v7:25.2.0'
    compile 'com.android.support.constraint:constraint-layout:1.0.1'
}

True, this probably due to the sample project template was used  during the porting process.  This project does not have any Java source code( yet:  should add some to handle run-time permission for writing disk because it does try to write out files in certain scenarios, just not time for it )
If you have chance, please add a patch. Otherwise it will be removed in next checkin here

fixing it with: https://github.com/googlesamples/android-ndk/pull/387


Why are 2 convolutions used in every res-block in a resnet?

This is more a general question about resnets.
I am wondering why there are always 2 convolutional layers used in every res-block instad of just one? To me just one convolution with a skip connection seems more like the natural choice. So why did they use 2 convs in every block in the official paper? What does happen if I use just one? And are there any studies about using just one compared to two?

Hmm. There could be something worth investigating here.
One reason for having two convolutional layers was that we could do some dimensionality reduction to add a bottleneck. This forces each building block to compress the information a little. That way, each building block is a bit more complicated than an ordinary linear transformation. But that feels like the sort of intuition that's handwave-y and i'm not sure if there's much validation behind that.
You might like the FractalNet paper as one logical conclusion of "what happens if we replace each building block with a smaller copy of the network architecture." I don't think this model is very widely understood or validated though. It's certainly an interesting idea.


Feature request for Promise.loop()

This is a feature request for the ability to call some async operation N times in sequence.  The idea is like Bluebird's existing Promise.reduce(), but there is no source array to iterate over.
I see this requested pretty regularly on StackOverflow when people are trying to call an API with some sequence of requests where each request increases an index value.  There is no underlying iterable they are iterating, just a for loop with an index.
I think of this as Promise.loop(), but it could certainly have any appropriate name.  Here's the general idea:
    // Create a sequenced loop from startLoop to endLoop (inclusive)
    // The iterator is called with whatever the previous promise returned and
    //    must return a promise when it is done.
    // The iterator is called like this: iterator(result, index); where:
    //     result is the fulfilled value of the prior promise in the chain
    //     index is the current loop index
    Promise.loop = function(startLoop, endLoop, increment, iterator, initialVal) {
        var p = Promise.resolve(initialVal);
        for (var i = startLoop; i <= endLoop; i+= increment) {
            (function(index) {
                p = p.then(function(result) {
                    return iterator(result, index);
                });
            })(i);
        }
        return p;
    };

Sample usage example here: https://jsfiddle.net/jfriend00/rodrz6o3/

Hey, nice to see you here @jfriend00 , any particular reason this should not be a separate module?
Alternatively, mind linking to some of those questions?

Obviously it works as a separate module.  It just seemed to be of similar or more utility as some of the other utilities you had built in and I thought it was one of the sequential iteration structures you had not yet included.
I can look around for some of the links.  The one I came across today which triggered my thought that this should be a prewritten iteration structure because I've seen it a bunch of times is this one: http://stackoverflow.com/questions/36191637/returning-looped-get-requests-sequentially-using-q-js/36193971#36193971.  I've even seen people make up a dummy array just so they could use something like Promise.mapSeries() on it.

Also a point about using a library, I trust more in code that's commited to petkaantonov/bluebird rather than to some unknown repo, If I see some utility function in bluebird I'm more likely to use it.

Let's sum this up, I'm -1 on it.
@petkaantonov @spion ?

I think this should be handled by bluebird streams, once they become real.
Example crawler that finds the first bluebird link from reddit (with imaginary API):
iterate(page => getNextPage(page), getFirstPage('/r/programming'))
.flatMap(page => page.links)
.skipUntil(link => link.innerText.contains("bluebird"))
.first()
.then(link => console.log("Article about bluebird", link.innerText, link.href));

I'm sort of +0, this gets asked pretty often, but the ease at which you can do loops simply using recursion is why I'm not entirely +1.

Here's a slight different variation of need.  Whereas the original proposal was to sequence async items in a loop, this is a situation where you want to iterate with a for loop, but you want to know when all the async ops in the for loop are done and they can all be run in parallel.  Yes, you can manually collect the promises in an array and use Promise.all, but it could be useful to have a utility that does that for you.  Here's the example.  http://stackoverflow.com/questions/38572767/for-loop-in-promise-then.   Maybe one utility could be made to let you manage the async for loop results in either sequence or parallel.

I believe all you need is
function* loop(start, end, increment=1) {
    for (var i=start; i<end; i+=increment) yield i;
}

that you can use with Promise.map and Promise.mapSeries. No need for dummy arrays when we have iterators :-) I don't think we need an extra utility function for this.

Yeah, after playing with it a little I think I'm -1 on this. Given spion is -1 on this and petka is +0 I'm going to close at this point - but feel free to reopen or to publish this as a utility lib.


AttributeError: 'CosmosClient' object has no attribute 'ExecuteStoredProcedure'

I'm trying to write a fairly simple piece of code that invokes a stored procedure in one of my containers.
# Required imports
from azure.cosmos import exceptions, CosmosClient, PartitionKey

# Initialize the Cosmos client
endpoint = "https://***.documents.azure.com:443/"
key = '***'

# Get a Cosmos client
client = CosmosClient(endpoint, key)

# Database name
database_name = 'SomeDBName'

# Collections to perform the hard delete
list_of_collections = ["SomeContainerName"]

# Name of sproc to invoke
sproc_name = "delete_by_query"

# Query to pass into the sproc
query = "select * from c where c.active = false"

# Invoke the sproc
for collection in list_of_collections:
    client.ExecuteStoredProcedure("dbs/{}/colls/{}/sprocs/{}".format(database_name, collection, sproc_name), query)

When running this, I get the following error:
AttributeError: 'CosmosClient' object has no attribute 'ExecuteStoredProcedure'

Please advise :)

Your import should be :
from azure.cosmos.cosmos_client import CosmosClient

@vchalmel Just tried, and still running into the same error.

Which version of the package are you running ?

I'm using version 4.0.0b6

@sjhermanek We don't have a sample, but here is a test showing how to execute a stored procedure in the v4 SDK https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/cosmos/azure-cosmos/test/test_crud.py#L622-L644
execute_stored_procedure is a method that exists on the container.scripts object. We just released 4.0 and are still working on getting all the docs updated.
If you run into any further issues, you can open an issue in our central Python repo https://github.com/Azure/azure-sdk-for-python. All active development has moved to that repo.


drawHeader only executes once when returning true

I'm using the following code to draw a custom header.  In this use case, I'm passing a pipe-delimited string as the header text, but I want to split on the delimiter and render multi-line text.  ie 'Mon|01-02-2022'
const drawHeader: DrawHeaderCallback = (args) => {
    const { ctx, rect, column, theme } = args;

    if (!args.column.title.includes('|')) {
      return false;
    }

    const [title, subTitle] = column.title.split('|');
    const numberX = rect.x + rect.width / 2;

    ctx.save();
    ctx.beginPath();
    ctx.clearRect(rect.x, rect.y, rect.width, rect.height);

    ctx.fillStyle = theme.bgCell;
    ctx.fillRect(rect.x, rect.y, rect.width, rect.height);
    ctx.save();

    ctx.textAlign = 'center';
    ctx.fillStyle = theme.textDark;

    ctx.font = `bold 11px ${theme.fontFamily}`;
    ctx.fillText(title, numberX, rect.y + rect.height / 2);
    ctx.font = `11px ${theme.fontFamily}`;
    ctx.fillText(subTitle, numberX, rect.y + rect.height / 2 + 17);

    ctx.save();
    ctx.restore();

    return true;
 };

The first header I'm rendering does not need this treatment, thus the return false early on.  However, once the second header is rendered (thus hitting the return true), the grid stops drawing.  I can put console.logs in this block and they all print as expected, however it appears to just....stop.  Attached is a screenshot from my running app showing that it renders one of my custom headers exactly as intended, but then no further header cells are rendered.

Is there something I'm clearly not doing correctly in the example above?  The code provided is the exact thing I'm trying which is rendering the above image.  I've had success creating CustomCells for the rest of the grid; it would be neat if I could do the same thing for Header customization.

👍


chore(master): release 0.11.2

:robot: I have created a release beep boop
0.11.2 (2024-12-27)
Bug Fixes

use PAT instead of GITHUB_TOKEN to trigger workflow to npm (a0e98e4)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.11.2
:sunflower:


panic: Error downloading snapshot file... x509: certificate signed by unknown authority

Hi,
Using either docker-compose or docker build, I get the following error:
panic: Error downloading snapshot file: Get "https://ls.manapotion.io/export.bin": x509: certificate signed by unknown authority
Ubuntu 18 with latest docker-ce installed.
Ran the tutorial at https://docs.iota.org/docs/hornet/1.0/tutorials/install-hornet-docker. It looks pretty straightforward but this error keeps showing. Untouched config.json or other file.
Full logs attached.

Fixed with #506


fix(tekton): fix code smells

Resolves:
https://github.com/janus-idp/backstage-plugins/issues/584

@debsmita1 sonarCloud is complaining about code duplication

@debsmita1 sonarCloud is complaining about code duplication

The debounce.ts file now exists in topology and shared-react. I will raise a separate PR to remove the debounce file from the topology plugin


Issue in dep check with files dropped by .gitignore

What version of dep are you using (dep version)?
dep 0.5.0
What dep command did you run?
dep check
Issue: We are trying to integrate dep check into our ci/cd workflows. But we face a problem when of our vendor directories have files that are included in the .gitignore. When we run dep ensure in our local machine, the computed hash will always take that ignored file into account. In our ci/cd, however, the gitignored file will not be present and during the dep check, the computed hash will always end up being different from the hash in the gopkg.lock.
I feel that this issue poses a challenge for those of us who'd want to use dep check in ci/cd pipelines. One solution could be to just prune the files that are included in .gitignore.

@sdboyer  @carolynvs  Is it possible to have some discussion about this? dep check is very useful for us and this issue precludes us from using it in our CI processes.
I am happy to implement the required changes through a PR if we come to some consensus on the required approach here.

For us it happens for this repository: https://github.com/go-errors/errors
It contains cover.out file which is by default ignored by .gitignore files, especially for Go: https://github.com/github/gitignore/blob/master/Go.gitignore#L12

dep ensure
git add --force vendor/
git commit

That should do the trick.

hi! i'm sorry, but there's no reasonable way to make dep check accommodate .gitignore files. It accounts for your prune settings - that's as good as can be done, and is usually good enough when it comes to trimming down the size of vendor. (Unfortunately, using .gitignore in this way was a bit of a hack from the beginning).
If there are specific projects that you're trying to exclude from verification checks because there are .gitignored parts, you can use the new noverify Gopkg.toml field. This will make your dep ensures less efficient, as it will revert to the old behavior of always writing out that project on every run. But you'll be able to use dep check without erroneous failures in CI.
For cases like @ondrej-fabry's, where a default .gitignore is ignoring a file, the one-time operation @AlekSi describes is the best solution - thanks for that!

@sdboyer  @AlekSi , Thanks for your inputs.
To clarify, my case is similar to that of @ondrej-fabry 's. In my case we are gitignoring an .editorconfig file and one of our vendored repositories has that file added to their repo. This case will not be uncommon I am sure.
While the solution provided works, it seems very hacky. To other devs who may fork/clone our repositories, this may not be obvious and can possibly cause confusion esp. If they wanted to add a new dependency which had gitingnore conflicts. They would have to force add the vendor as well. Developer/User experience wise, this is counter-intuitive. I am also reluctant to normalize the practice of git force adds. It's certainly not elegant.

I get it. I do - again, there's the noverify workaround, if needed. I'd also be happy to accept a PR that makes a note on the command docs suggesting that .gitignore may be the problem.
i could also imagine even just a shell script that checks for any files being ignored in such a way, as a helper.
but the whole design of dep check is based on the idea that any input that could affect the state in vendor is structured and well-defined - e. g. prune opts change, or the package list, or the version. it is that property that lets us trivially determine when toml/code/lock/vendor are out of sync.
gitignore, OTOH, is arbitrary, which means we can't tell from one dep run to the next if its contents have changed, and therefore vendor should be regenerated even if hashes match.
its convoluted to think about, I know. think about it this way - if pruneopts wasn't in the lock, how would we know when it changed in the toml, thereby necessitating regenerating vendor?

@ondrej-fabry: Unfortunately that only works for stuff ignored in the same .gitignore file (and probably levels above). If the dependencies have their own .gitignore files it doesn't override them, at least for me. I'm using git version 2.18.0.

Not sure I understand your problem correctly. You mean one of the deps in vendor directory has its own .gitignore file in which they defined some files to be ignored and still commit the file anyway? Because otherwise I'm not sure how the different hash could be calculated.

@sdboyer  Appreciate your detailed response. I think I am fine with Ondrej's solution. I might just go forward with that. Like you had mentioned earlier, I am happy to raise a PR to highlight these dep-check related caveats in the dep docs.
@ondrej-fabry Thanks! This workaround certainly seems pragmatic.

Did you consider to ask maintainers to remove files that should not be committed, or were committed and then gitignore patterns were erroneously updated to include them?


Some sites fail with segmentation fault

see e.g. safe://safe-blues.jpl (browser crashes straight away), safe://optomoon.draw (browser crashes after 9s)

I'd wager this is to do with partial content headers, as it's with mp3 and video that it's failing.
I'll take a deeper dive shortly and investigate as/when i'm on the streaming implementation.
In the meantime, if you're able to garner any useful logs, please post them.
Thanks @JPL1!


[Doc Improvement][Update card actions][2475762]

added sample code.

Docs Build status updates of commit f2f6a17:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/cards-actions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Ensure last entry in keymap popup is displayed

The downside is there appears to be a blank line at the bottom of the keymap
popup.  If there is another way to ensure the last line is included in the
popup without adding a blank line, that solution would be ideal.  Because of
this nitpick, I marked this PR as a draft.

Could you describe the problem you're seeing without this? For me, there are no missing lines in the popup. Do you use customized positioning?

Sorry for the delay.  Thank you for the feedback! :)
Below are some screenshots highlighting the issue this PR attempts to solve.  I've also attached some version information at the very end.
I've also linked my current configs (see modal.lua in particular), though I intend on trying to reproduce the issue in Xephyr with a reduced awesome config. I hope to have this tested in such a way for this weekend.
I hope this helps clarify the weird issue I've encountered.  As mentioned above, I'm not entirely certain this is the best way to address the issue.  I'm looking forward to better understanding what's going on here.
Screenshots
In the next two screenshots the very bottom will show a key entry in the first screenshot, which is present in the second screenshot.
Here's a screenshot of what happened on my present setup without this commit:

Here's the same modal with the git commit applied:

 Version information
winston@snowcrash ~/.../awesome/modalbind $ awesome --version
awesome v4.3 (Too long)
 • Compiled against Lua 5.1.5 (running with Lua 5.1)
 • D-Bus support: ✔
 • execinfo support: ✔
 • xcb-randr version: 1.6
 • LGI version: 0.9.2
winston@snowcrash ~/.../awesome/modalbind $ uname -a
Linux snowcrash 5.7.13 #1 SMP Wed Aug 12 03:22:48 CDT 2020 x86_64 AMD FX(tm)-8350 Eight-Core Processor AuthenticAMD GNU/Linux

I experimented a bit with your config and noticed, that the problem only occurs, if there is only a single entry after the last separator. Also, the problem seems to be that the whole text is then placed lower inside the wibox. This is also visible in your screenshots, the one with the missing entry has more space above the "Manage Awesome" title.
May be related to: https://github.com/awesomeWM/awesome/issues/3126
The only other solution I've found is to add at least 2 pixels to the height here. But 2 pixels seems arbitrary and is probably dependent on font/dpi/whatever else, so I wouldn't say that workaround is any better...


When Creating a new Product Content Item in BagPart, ProductPartIndex don't update. (OCC-220)

Describe the bug
When Creating a new Product Content Item in BagPart, ProductPartIndex don't update.
To Reproduce
Steps to reproduce the behavior:

Create a new ContentType and name "BagTestPage", add BagPart in it
Set "Contained Content Types" of Bag in BagTestPage to "Product"
Create a new Content Item of BagTestPage and name "ProductBags"
Create a couple of Product content items in ProductBags, these Product content items was not updated to ProductPartIndex.

Jira issue

This is not a bug, but a limitation of the bag part. It's not related to OrchardCore.Commerce either, for example the child content doesn't appear in the stock OC indexes like ContentitemIndex either.
Even if these indexes were created, they wouldn't work as expected, because all the child items in the bag part are stored in the holding content item's document. I don't suggest placing Products inside bag parts at all. Consider using a List part instead.


Show copy-ready address of inspected object

When inspecting object, show it's memory address even if description if not available
Allow copying this address in the same manner as object description

Pardon my debugging message in dealloc method.

Could you post a screenshot of what it looks like?

Sure, of course!

 "Copy" tooltip appears on long press, just like with "Description" field

What do you think of this?

That's a great idea! I love it, much cleaner solution.