Determine a way to unambiguously re-export types without silent conflicts.

:heavy_check_mark: @arlyon here! For a fix, please see this comment https://github.com/arlyon/async-stripe/issues/154#issuecomment-1311619880. This issue is now about determining the best way to fix the aliasing issue.

Hi!  I just updated from rc3 to 0.13
I'm trying to create a subscription with SubscriptionPaymentBehavior::DefaultIncomplete
When I pass SubscriptionPaymentBehavior::DefaultIncomplete to CreateSubscription (wrapping inside Option appropriately), the compiler complains that I'm passing the wrong one:
expected enum stripe::resources::generated::billing::subscription::SubscriptionPaymentBehavior, found enum SubscriptionPaymentBehavior
I'm using stripe::SubscriptionPaymentBehavior
If I try using stripe::resources::generated::billing::subscription::SubscriptionPaymentBehavior, the compiler complains that resources is a private module
My features: async-stripe = { version = "0.13", features = ["runtime-tokio-hyper", "webhook-events"] }

I managed a temporary fix by combining the SubscriptionPaymentBehavior in subscription_item.rs to with SubscriptionPaymentBehavior in subscription.rs
I haven't really wrapped my mind around the code generation yet, so I don't know where to start to contribute a fix back to this repo.

Hey dgramop.  They problem is not in the code generation, it's in the exported modules.  Code isn't exported automatically in Rust, you have to include the appropriate struct in the correct file (module) to make code visible.  However, that means that our code generation could in theory produce duplicates, that people can't find until they show up.  I have a very little bit of time, let me take a look and see if I can spot the double code name and figure out a solution for you to try.

Okay, looks like this is a common problem with subscription and subscription_item...if you look into resources.rs you can see that they renamed several of the other modules so that they don't conflict.
The quick fix is to do the same thing for your SubscriptionPaymentBehavior.  You do the following:

Fork master
Create a branch called SubscriptionPaymentBehavior-visiblity-fix or something similar
Change resources.rs so that the names for SubscriptionPaymentBehavior in subscription and subscription_item modules don't conflict with the as keyword.  You can see this being done as an example around lines 183ish in resources.rs.
Commit, Test, and make a pull request with this repo.

If you run into trouble, push your unfinished work to your forked branch, ping me in this repo and I'll help.

Oh, sorry, I had found the duplicated struct and fixed that error last night and made a quick fix, I should have reported that back to save you a few minutes. I was still working on my subscription integration when I found another inconsistency that was causing my expanded Subscription to not deserialize: the generated prefix for invoice line items was wrong - it seems to always be "li_" from stripe.
Would the PR changes I suggest be overwritten if openapi code generation is run again? I had also rolled back to the previous version (0.13.0-rc3), so I'll have to make those changes again.
Additionally, I noticed the version that's on crates.io (0.13.0) is referencing a commit that doesn't seem to exist in this repo: .cargo_vcs_info.json from docs.rs source
git checkout 815db5e4b54e952c539e4841d4cce616b4c7a029
fatal: reference is not a tree: 815db5e4b54e952c539e4841d4cce616b4c7a029

So, a few things.

The only thing that will be overwritten on an openapi code generation is any changes that you make in the generated folder.  And you should not make any changes to those files to get something to work for that reason.

The reason the 0.13.0 isn't referencing the right thing is that Arlyon rewrote some history recently on master for some reason or other (I'm not sure why).  @arlyon Can you update the crate that's on crates.io so that it's something on the current history?

Finally, yeah, I've discovered that the add_id(String) method doesn't actually work.  We should probably open that up as it's own issue.

If you want up to date code, you have to use the github.  Arlyon is not paid to maintain this and larger companies with money I think prefer something with more support like python, or something else, so the "releases" are when he can find time and feels like enough work has been done, which takes time.

Regarding the nonexistent commit, I bumped the verison, published to crates IO, realised the readme changes were not included in the commit (and so the docs on crates io are incorrect) so I amended the commit with the intention to overwrite. Apparently that is not possible, even just for docs changes, and because I chose to amend it locally (we don't have automated releases set up so publishing / versioning is manual atm) the commit was lost. Sorry!
I will push a new version with the changes to default to address this :)
Part two, yes, some places in the openapi generate almost identical structs with similar names that need to be manually exported. This is mainly due to low test coverage (and huge api surface area) so thanks for the report! If you have time to submit a PR I will make sure to merge it asap.

grep -r "struct " . | sort | uniq -d
grep -r "enum " . | sort | uniq -d
I'm using this to find identically named structs and enums for manual verification that they are indeed duplicates and for merging. I'm currently working on removing these duplicates.
I'm sure there's some way we could have some kind of limited parser that checks fields and enum variants to do this automatically at generation-time, but I'll do it manually for now

4. f you want up to date code, you have to use the github.  Arlyon is not paid to maintain this and larger companies with money I think prefer something with more support like python, or something else, so the "releases" are when he can find time and feels like enough work has been done, which takes time.

Absolutely! I'm familiar enough with how OS works to be very grateful for the time and effort you guys have put into this repo 💛

I made the PR, and I made another branch in that repo that has a hacky script to remove duplicates (see combine.py)
The python script doesn't really follow some of the standards so well, but it maintains functionality. I tested it with an older cached 0.13.0-rc3 off crates.io (which is what my application uses). I also built it with cargo make openapi-install and adjusted the Makefile.toml accordingly
I'm new to this build process so please let me know if I missed anything!

Any progress on this, am running in to the same issue

Hi! There is a PR to try and dedupe a number of types but I haven't had a chance to get round to reviewing it. In the mean time, you can clone the repo and work locally, following the pattern for subscription and subscription_item in src/resources.rs to manually export the conflicting types :)

Just ran into this myself.  For users of the library, you can hack/workaround it by letting the compiler infer the type when deserializing the equivalent json.
let mut to_create = CreateSubscription::new(customer);
// async-stripe just reexports the generated openapi structs, and sometimes they have duplicated names.
// hack: let the compiler infer which one to use.
let behavior = serde_json::from_str("default_incomplete").unwrap();
to_create.payment_behavior = Some(behavior);

For a fix on the library side, it sounds like reviewing the renaming PR has been pending for a while.  Would it be a reasonable smaller fix to make the stripe::resources::generated module hierarchy public so that the users can import the specific struct from there?

I think that that is a fair request.

Partially addressed in https://github.com/arlyon/async-stripe/pull/296

This is partially addressed by the above but also rustc now warns on these.


remove the duplicated styled-jsx dep

Fix the FOUC of the page which is caused by duplicate installed styled-jsx
After removing it:

@huozhi thanks for the fix! I feel like I read somewhere that styled-jsx stopped being bundled with Next.js. I guess it still is!

@coryetzkorn Would you mind sending me the link if you find it again someday? We really wanna clarify the usage that styled-jsx could be used out of box in next.js app. Thanks in advance 🙏


rwt_config_generator: 0.0.1-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository rwt_config_generator to 0.0.1-0:

upstream repository: https://github.com/DLu/rwt_config_generator.git
release repository: https://github.com/wu-robotics/rwt_config_generator-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: null

I'd love to see more doc on that :)

Coming soon to a theatre near you.


update utils/demo.py and examples/demo_ct.py

demo.py: add pytorch dataset for CT100 matlab database.
demo_ct.py: add example of CT image reconstruction with DeepInv.
Checks to be done before submitting your PR

[ ] python3 -m pytest tests/ runs successfully.
[ ] black . runs successfully.
[ ] Updated docstrings related to the changes (as applicable).
[ ] Added an entry to the CHANGELOG.md (as applicable) (to come).

Thanks a lot @edongdongchen! I'm closing this one as this is coming from the wrong repo I guess; would you mind opening one after checking out from the main of this repo?


HBASE-28734 Improve HBase shell snapshot command Doc with TTL option

https://issues.apache.org/jira/browse/HBASE-28734

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
4m 11s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  5s

master passed

+1 :green_heart:
spotless
0m 45s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 54s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
rubocop
0m  3s

No new issues.

+1 :green_heart:
spotless
0m 44s

patch has no errors when running spotless:check.

@Apache9 Could you take a look? Thanks.


V2

Coverage decreased (-33.4%) to 49.254% when pulling 9f2cb41276f519fc30ad1bc2c6639ca2cb570fc7 on v2 into 5b40d44566a9c0aa895407a46538997acdd28ba5 on master.

Coverage decreased (-33.4%) to 49.254% when pulling 9f2cb41276f519fc30ad1bc2c6639ca2cb570fc7 on v2 into 5b40d44566a9c0aa895407a46538997acdd28ba5 on master.

Coverage decreased (-33.4%) to 49.254% when pulling 9f2cb41276f519fc30ad1bc2c6639ca2cb570fc7 on v2 into 5b40d44566a9c0aa895407a46538997acdd28ba5 on master.

Coverage increased (+17.3%) to 100.0% when pulling 54b817213b2ad32fc56ca6eafe7c0612293a5471 on v2 into 5b40d44566a9c0aa895407a46538997acdd28ba5 on master.

Coverage increased (+17.3%) to 100.0% when pulling 289a6cd504074f807f6045bb99027fddcd752b4e on v2 into 5b40d44566a9c0aa895407a46538997acdd28ba5 on master.


Prepping Thrombin to reproduce TIES paper

Hiya,
If I want to prep Thrombin to reproduce the TIES paper what is the best way to include the disulfide bridges? The standard TIES protein prep pipeline with tleap does not add these. How was this protein prepped for the paper? https://pubs.acs.org/doi/abs/10.1021/acs.jctc.0c01179
Cheers,
Alex

Hey Alex,
In order for tleap to add disulphide bridges, the best way could be to (a) define all of them (SG-SG bonds) in the input PDB after the coordinates (see example provided below), and (b) rename all CYS residues involved to CYX. After this, tleap should automatically add disuphide bridges.

Cheers,
Agastya

Hey Agastya,
I put the CYX as they describe here https://ambermd.org/tutorials/basic/tutorial7/index.php but did not see the bridges at the end. They also discuss using the bond command in LEaP to define which CYX are bound to each other which would need modification to the default tleap script in TIES?
I will try again and leave the connect information in the PDB. I hope with CYX and the connect tleap will add the bridges automatically.
Cheers,
Alex

hey Alex,
That's right. Using the bond command would require modification to the code base, this is why I prefer the approach I suggested. Please give it a go and let me know if you face issues with that.
Cheers,
Agastya

Okay nice thanks for the advice this works well to build the bridges :)


RsaSecurityKey.Parameters should be nullable.

While it is not possible now to make it nullable.
Another property should be added perhaps RsaParameters? RsaParameters that is nullable.

don't need this.


python3Packages.maxcube-api: init at 0.4.3

Motivation for this change
Required for the home-assistant integration of our local hackerspaces radiators.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 138833 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.maxcube-api
python39Packages.maxcube-api


Web Component:  Card - Pictogram component variations

Corresponding React epic (#4448)
The web components version will be developed first.  The React version will be a wrapper of the web component.
The problem
The default card component becomes very plain and repetitive for landing and gateway pages, which often feature over 12ish cards. We need a new visual to

add differentiation to cards themselves for easier scanning, and add differentiation between sections of cards so to keep engagement on all links and destinations
break up the monotony with visual interest

Pictogram card
Summary
Pictogram card is designed for page navigation. Cards are restricted to link out to internal URLs only. This component can be a variation to the default card component, using the same typography but add a custom pictogram.
Design and spec
Collection link on Abstract: https://share.goabstract.com/96c3a064-a525-47b7-83e7-a89aeee948e2

Pictogram card Jan 4, 2021 design specs
Pictogram card functional specs

Variations
— 2:1 aspect ratio, pictogram bottom (default)
— 2:1 aspect ratio, pictogram top
— 4:3 aspect ratio, pictogram bottom
— 4:3 aspect ratio, pictogram top

Content model
Pictogram Card:

Pictogram (Required) - Choose from dropdown
Headline (Required) - Simple Text (Max 40 characters)
Description (Optional) - Simple Text (Max 75 characters) - If provided, description is shown on hover
URL (Required) - should be restricted to link out to internal URLs only
Pictogram Alignment (Default = True) - True: Pictogram on bottom. False: Pictogram on top.
Inverse (Default = False) - False: Not inversed. True: Inversed
Size (Default = True) - True: 4:3 size ratio. False: 2:1 size ratio.
Pictogram Color (Required) - Choose from dropdown. Default = Gray

Pictogram Grid:

Pictogram Alignment (Default = True) - True: Pictogram on bottom. False: Pictogram on top. - This overrides all card values
Pictogram Color (Required) - Choose from dropdown. Default = Gray - This overrides all card values
Defined Values for all cards - Inverse = False, Size = True
The above values are defined because at the time we see no need for an inverse pictogram grid or a 2:1 card grid, but want to enable these in the card content model to support future patterns

Additional information
Frequently asked questions about this design:
Q: Should there be arrows to indicate interactivity?
A: We removed it during yesterday's discussion. The reason is it's clashing with the picto and taking away space for text. The over all card becomes more clustrophobic. Felt safe removing because 1. The cards all have a hover state (and potentially a more expressive hover state with gradient & colors), 2. Card area is large, so hard to miss. 3. we use icon / picto as a indicator for interactivity relatively often (buttons / tiles / accordion)
Q: Should we pick 1 and not have all 4 variations?
A: Pictogram top or bottom has proven to be useful for different contexts. Picto top allows for better scanning in situ where an image is strongly associated with the content, for example, Industries. It also helps with breaking up the default heading-first design used most of the time. Picto below allows text to lead, provide more precise communication, but harder to scan for returning users and very monotonous and boring by the end as this is a very common design. In conclusion both designs are needed, and should be a toggle for page authors to choose based on context.

Designs are still being refined.  Scheduling the func specs and design refinement work to the first sprint of 2021 and will move earlier, if needed.

Content model has been added to this feature request

Here's the design specs link: https://ibm.box.com/s/ykv2ukr3twyfx2nrro3rn7ge4iqoz48x

@photodow As you know, the designs have been posted by Lily.  Wonil has included review of the designs in his design meeting on Friday, Nov 20.  Work to create the func specs is scheduled in the next sprint, Nov 23 - Dec 4.  Please reach out to Shixie and Michael with questions as you are working on the specs.  We'll be in touch to decide when you are ready to present to the engineers for their sizing.  Shixie and Michael should be included in that review meeting.  Thanks.
cc:  @lily-kenney

I haven't seen an update from last week's meeting, so I'll just add a quick one here from my perspective.
So we met last Friday to go over the pictogram cards to see how it fits in with the current design system and determine whether we can move forward with it or not. The designs looked great, and everyone saw a need for it. There were some concerns though around adding too many variations of the card, and whether this was a whole separate card or part of our existing one. There were also some concerns about it's alignment to the current card style guide that was released recently and agreed upon by both teams. Because of some of this, we decided to move forward with a trimmed-down version of the pictogram card. Adding the pictogram to our existing card in the bottom left of the card, and save some of the pictogram positionings and hover effects for a later iteration after we've been able to take a look at our cards from a higher perspective. After talking with Roberta and Linda, it sounds as if we will review this pivot with the development team on Monday.
Next steps:

Move forward with pictogram card MVP adding pictogram to the bottom left of the card.
Conduct an audit of existing cards, desired, and needed cards to see how we can scale our cards better to meet the growing needs we're continuing to see. https://github.com/carbon-design-system/digital-design-ideation/issues/184
Reach out to Shixie and team to continue to work with them on the next iteration of the pictogram card.
Determine if style guid needs to be updated, or iterated on to allow for these types of cards.

@wonilsuhibm @oliviaflory @JennySanchez

Added line to account for pictogram from a functional specs standpoint. Probably should reformat the card functional specs in the near future. It's kind of confusing to navigate right now.
https://ibm.ent.box.com/file/749150538784

Based on the comments from Jina, the designs were finalized on Jan 4, 2021. See notes included in issue (#4445)   The final designs are in Box here:
https://ibm.ent.box.com/file/760349403349?s=x815734pg1ktgbiavbdhy84hm5dqpjh5
The additional work to make the changes included in these designs will now be scheduled.
cc:  @jyoon25 @lily-kenney @shixiedesign @jeffchew @wonilsuhibm @ljcarot @photodow

Based on the comments from Jina, the designs were finalized on Jan 4, 2021. See notes included in issue (#4445)   The final designs are in Box here:
https://ibm.ent.box.com/file/760349403349?s=x815734pg1ktgbiavbdhy84hm5dqpjh5
The additional work to make the changes included in these designs will now be scheduled.
cc:  @jyoon25 @lily-kenney @shixiedesign @jeffchew @wonilsuhibm @ljcarot @photodow

CECC / ISCEP adopter has another Card - Pictogram feature request that we are moving forward with under issue (#4927 ).  Closing this issue.


Proofpoint On demand(POD) Timer too long

Describe the bug
In Solutions/Proofpoint On demand(POD) Email Security, the function is now pulling logs every 1 hour, many times the information received is too much and ends with a timeout.
To Reproduce
Steps to reproduce the behavior:

Deploy the function.
Wait till the function is executed.
If the information received is big, will crash.

Expected behavior
If the pulling time is reduced, then this problem will be avoided.
Additional context
Suggested time: 10 minutes.

Hi @dreamerfuzzy, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates.Thanks!

Hi @dreamerfuzzy, Did you get a chance to check on above request. Thanks!

Hi @v-sudkharat,
Not sure what changes have been made in the package you sent but I do not see any change after applying your patch.
On my end it's still executing every hour

Thanks.

Hi @dreamerfuzzy,
Based on the error mentioned into the issue description for function timeout, we have made change in function timeout to 10 min, which help to not crash the app.
If request is for the run the function app to every 10 min, please update the below URL in we in which we have changes the function execution time which earlier was every hour now it set for every 10 min:
https://github.com/Azure/Azure-Sentinel/raw/9b0bbba7d8421a13b8311cba6635280c29112266/Solutions/Proofpoint On demand(POD) Email Security/Data Connectors/ProofpointSentinelConn.zip
Once update, restart the function app. Thanks!

Thanks for your quick response @v-sudkharat,
I've tried but as far I can see, you changed the timer, so it's being execute every 10 minutes but the function is retrieving very time the last 60 minutes, so we're getting duplicate events.

Thanks.

Hi @dreamerfuzzy, WEBSITE_RUN_FROM_PACKAGE - https://github.com/Azure/Azure-Sentinel/raw/c32f4c8347e79ed128f76aa76ff099d133997114/Solutions/Proofpoint On demand(POD) Email Security/Data Connectors/ProofpointSentinelConn.zip
Update the above shared link in function app WEBSITE_RUN_FROM_PACKAGE:
Once it updated, save the changes are restart the function app. Please let us know if your issue resolved.
Thanks!

@dreamerfuzzy,

Hi @dreamerfuzzy, WEBSITE_RUN_FROM_PACKAGE - https://github.com/Azure/Azure-Sentinel/raw/c32f4c8347e79ed128f76aa76ff099d133997114/Solutions/Proofpoint On demand(POD) Email Security/Data Connectors/ProofpointSentinelConn.zip
Update the above shared link in function app WEBSITE_RUN_FROM_PACKAGE:
Once it updated, save the changes are restart the function app. Please let us know if your issue resolved.
Thanks!

@dreamerfuzzy , to avoid duplication issue, you can go with this package where the timeout added 10 min, which help to not crash the app
Thanks!

Hi @dreamerfuzzy, Did you get a chance to check on above comment. Please let us know if it help to resolve your issue. Thanks!

@dreamerfuzzy, Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond to it in the next 2 days. If we don't receive a response by 06-11-2024 date, we will be closing this issue.
Thanks!

Hi @dreamerfuzzy, since we have not received a response in the last 5 days, we are closing your issue as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


[Bug] Where is save all button?

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.85.2
OS Version: Windows 10

Steps to Reproduce:
Where is the save all button? They removed the only button that was useful for anything

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Updated to January 2024 (version 1.86.0).

Were you using an extension to add this?

No.
My vscode was updated and the save all button is gone

Yes, its only in the "Open Editors" view.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Update TID generation

For TID, instead of uuid (letters and numbers) we want 'TXXX' with 3 random numbers for XXX.
I reverted my change to genereate_uuid in case we still want the existing functionality for other things, and changed where TID's are generated so that they're in the right format.

Closing - Not needed now


2023 XCDR CP Integration

Integrate CP-ITI-1014-04ballot61, CP-ITI-1219-04-ballot62, CP-ITI-1227-03ballot62

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed
[ ] I have selected a committee co-chair to review the PR

ℹ Additional Information

@mattblackmon please review

@mattblackmon review please


A "navigationItem.rightBarButtonItem" Exception"

I define an enable RACSignal ,which is bind to RAC(RAC(self.navigationItem.rightBarButtonItem,enabled), at same time , its also be a enableSignal of "self.navigationItem.rightBarButtonItem.rac_command".
Then the excetpion is thrown:
OG: error: session_id=3EFB33D463E8F6453C31794E3A35EECA, context=Signal name: is already bound to key path "enabled" on object , adding signal name: is undefined behavior
(null)
((
0 CoreFoundation 0x0000000182126dc8 + 148
1 libobjc.A.dylib 0x000000018178bf80 objc_exception_throw + 56
2 CoreFoundation 0x0000000182126c80 + 0
3 Foundation 0x0000000182aac1c0 + 88
4 ReactiveCocoa 0x00000001006d2684 -[RACSignal(Operations) setKeyPath:onObject:nilValue:] + 2216
5 ReactiveCocoa 0x00000001006d1da4 -[RACSignal(Operations) setKeyPath:onObject:] + 116
6 ReactiveCocoa 0x00000001006fa4d8 -[UIBarButtonItem(RACCommandSupport) setRac_command:] + 308
7 Neighbours 0x000000010003f834 -[FillMobileVC bindRac2ThisVC] + 800
8 Neighbours 0x000000010003f3b8 -[FillMobileVC viewDidLoad] + 368
The Code throw Exception is listed below, if "RAC(self.navigationItem.rightBarButtonItem,enabled) = enablSginal" is removed , it works fine.
RACSignal* enablSginal = [self.mobileTextField.rac_textSignal map:^id(id value) {
NSString* mobile = (NSString*)value;
BOOL nextEnabled = [MobileFormatUtil isValideMobile:mobile];
return @(nextEnabled);
}];
RAC(self.navigationItem.rightBarButtonItem,enabled) = enablSginal; //The bug Line//
__weak FillMobileVC* weakSelf = (FillMobileVC*)self;
self.navigationItem.rightBarButtonItem.rac_command = [[RACCommand alloc]initWithEnabled:enablSginal signalBlock:^RACSignal *(id input) {
return [weakSelf requestVeiryCodeSignal:weakSelf.mobileTextField.text];
}];

Setting the rac_command will automatically set the enabled state of the button item. That's why RAC is complaining—the enabled state of the button is getting bound twice.
If you remove "the bug line", then it should work as desired.


Set-PnPGroupPermissions stoped working today

Up until this morning I ran scripts using
Set-PnPGroupPermissions -Identity $visitorsGroup.Title  -List "DPC Documents" -AddRole @('Contribute') -Connection $projConn

With no problem
today the same statement shows error:
Set-PnPGroupPermissions : Method not found: 'System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable`1<Boolean>
Enumerator.MoveNextAsync()'.
At line:1 char:1
+ Set-PnPGroupPermissions -Identity $visitorsGroup.Title  -List "DPC Do ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (:) [Set-PnPGroupPermissions], MissingMethodException
    + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.Principals.SetGroupPermissions

What is the version of the Cmdlet module you are running? 1.11.0
Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

What can cause it?

No

I would recommend rolling back to 1.10 in the short term, while this problem is reproduced and fixed.

hi @Ofer-Gal , we havent changed anything from our end here, are you able to reproduce this issue on different tenants ? Looking at the error, seems like some other module is causing a conflict, so maybe you can try with loading just PnP PowerShell in your PS session and try ?

Closing this due to lack of response.
We haven't made any changes as such to the cmdlet.

I discovered the same issue :-s
The error:
Set-PnPGroupPermissions : Method not found: 'System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable`1
Enumerator.MoveNextAsync()'.
At line:372 char:9

    Set-PnPGroupPermissions -Identity $DeputiesGroupName -List "S ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Set-PnPGroupPermissions], PSInvalidOperationException
FullyQualifiedErrorId : InvalidOperation,PnP.PowerShell.Commands.Principals.SetGroupPermissions

Why isn't anybody responding? For us, this is urgent, as it is part of an automation script!

@mvanders this issue has been closed for over a year and also references an old version of PnP.PowerShell
Please create a new issue with all the relevant information.


Expose the item's alternative call number individual parts for inclusion in a label template

Hello!
We want to use SpinoMatic for our labels in our library and have a few questions regarding the customization of a field label.
Our labels look this:

It includes from top to bottom:

System point: NGB
Sequence number: 2;
Serial number/year: -1
Edition: (Ed.50)
counted copy: +2

So far we can get the needed label with this field: {{field:item_data.alt_call_no}}
The problem is, that it's arranged in line instead of from top to bottom, like in the image.
Our questions are:

Is it possible to somehow arrange the information in the field from from top to bottom like in the image?
And if yes, is it possible to change the font size  in each line?
Would it also be possible to skip lines while maintaining the layout if a label didn't include a serial number and counted copy, for example? Like in this image:

I wish you a nice day.
Best regards,
Adrian

@AdrianMenti, unfortunately only the parsed call number's individual parts are available for inclusion in a template. To add the same functionality for the item's alternative call number is an enhancement. I've changed the summary of this issue and labeled it an enhancement.

@AdrianMenti, you could include the "item_data.alternative_call_number" in your template, then in the "Call number" options turn on the checkbox for "Line breaks between call number parts" and select "Split By Space" for the "Call Number Parser" field. However, this won't support formatting/font options for different parts of the call number, nor keeping spacing when certain parts are missing.

Hello mgobat
Thank you for your reply and suggestion! This won't work, since we have formatted all our labels so far.
Would an enhancement be possible? Just asking, so I can present the answer to my boss.
Best regards,
Adrian

Hi @AdrianMenti. This is a potential enhancement for the Cloud App. Let me summarize how we might be able to implement it. Please let me know if it solves the problem. Thank you.
New fields in the "Add field" dropdown when editing a template:

Parsed alt call number part one
Parsed alt call number part two
Parsed alt call number part three
Parsed alt call number part four
Parsed alt call number part five
Parsed alt call number part six
Parsed alt call number part seven
Parsed alt call number part eight
Parsed alt call number part nine
Parsed alt call number part ten

Adding any of these new fields to the template would take the corresponding part number from the "Parsed alternative call number" (item_data.alt_call_no) field.
Each part could have its own font, font size, font characteristics (bold, etc), and alignment (left, center, right).
Checking the "Keep label spacing when field empty" checkbox would keep blank lines in the label if a particular part didn't exist. Note that this would require the parsed alternative call number to have empty parts.
Here's an example template:

Hello mgobat
Thank you for your answer and sorry for the late reply! Yes, this would be perfect but I have a few questions.

"Note that this would require the parsed alternative call number to have empty parts."
- In ALMA we have written our labels like this: PC 13:-1(Ed.7)+2
- So we would have to change our labels like this for example "PC 13: -1 (Ed.7) +2" for it to work. Is that correct?

Would it cost something to implement this enhancement? If yes, how much would it cost?

I wish you a nice day.
Best regards,
Adrian

Hello @mgobat
Thank you for looking into it!
Of course, we are possibly the only ones asking for this enhancement. Could you give me a rough estimate of how long it could take to implement the enhancement?
I wish you a nice day.
Best regards,
Adrian

The latest beta version, v1.4.0-beta-01, has the ability to include individual parts of the alternative call number in a label template. To try the beta, click the "i" (information) icon on the SpineOMatic tile in the "Cloud App Center" in Alma, then click the "Try v1.4.0-beta-01" link.
Please add feedback/comments to this issue. Thank you.


Remove version: REALiX.HWiNFO version 8.14

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/hwinfo/files/Windows_Installer/hwi64_814.exe/download
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/hwinfo/files/Windows_Installer/hwi64_814.exe/download
Status Code: 200
(Automated message - build 903)

Close with reason: Duplicate;


--capabilities option for Linux apps does nothing

Problem
In ChipLinuxAppInit we parse command line args like so:
    err = ParseArguments(argc, argv, customOptions);

which for --capabilities will do:
    case kDeviceOption_Capabilities:
        LinuxDeviceOptions::GetInstance().payload.rendezvousInformation.SetRaw(static_cast<uint8_t>(atoi(aValue)));
        break;

Then back in ChipLinuxAppInit we do:
    err = GetPayloadContents(LinuxDeviceOptions::GetInstance().payload, rendezvousFlags);

where rendezvousFlags is based on some build-time defines, and that does:
    aPayload.rendezvousInformation = aRendezvousFlags;

which overrides the value that came from the command line.
Proposed Solution
What is the actual intended behavior of this command-line option?  Is it meant to change the behavior of the application, or just the pairing codes that get printed?
@yufengwangca

This issue fails 3 test-cases of Device discovery that is TC-DD-3.11, TC-DD-3.12, TC-DD-3.13


physics

Hi, I am trying to create a player that can move and look like a regular player, I was wondering how I can add the physics as the player currently just floats. I was also wondering how to simulate the inputs (like W, A, S, D, Jump, etc) to go along with the physics.

I know how to move a player using
new ClientPlayerPositionPacket()
but I don't know how to stop the player from floating and to make the player move and step up on to slabs automatically.

You're going to need to implement the physics on the client side like the vanilla client does. This is out of the scope of MCProtocolLib.


Add ActionElement tests to

fixes some of the console errors that were happening as well with this test

also could add https://codeclimate.com/ as a hook to show updates to the code coverage on pull requests

That would be awesome!
Thanks,
Pavan

On Oct 11, 2016, at 6:23 PM, maniax89<EMAIL_ADDRESS>wrote:
also could add https://codeclimate.com/ as a hook to show updates to the code coverage on pull requests
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Documentation updates

Updates based on feedback:

Avoid making promises about future releases ... (so) avoid the word "yet."

We assume that users are reading the documentation while they use the product, so we write in the present tense.

Avoid the "Note" label because it doesn't tell users anything about the content of the note. Users can't tell whether the note will present crucial information or be just a tip. In this case, we could use the "Important" or "Restriction" label instead.

This is based on feedback in https://github.com/puppetlabs/osp-docs/pull/353


Remove Remote and NYC emphasis in job outro

@nicks : Does it make sense to remove this? I don't think there's any clear plans to go back to an office in NYC anytime soon right? And 4 of us aren't in NYC right now anyways right?

can you explain your concerns a bit more? I do several phone screens a week, and the geographic distribution of the team is a people ask about / want to talk about, so i'd be in favor of adding more info rather than less

@nicks : Sure. I just find the "Remote is OK" thing a bit weird, especially in pandemic times when everyone is doing remote. I'm worried it would be turning away candidates that think we don't really like remote or think it's a second-class citizen. If we just removed "Remote is OK" and leave the remark about NYC, I'm fine with that too. (Since the line below gives the geographical constraint already.)

How about something like:
"About half our team is in NYC and working from home during the pandemic. About half is remote, each in a different city. We believe you can be successful in this role remotely."
fwiw, we've had plenty of strong remote candidates apply, so i'm not super concerned about that, more concerned that we're upfront about the relevant information

Sounds good to me @nicks . Updated PR with your suggestion.


Arduino syntax highlighting could be better :)

Only the Arduino keywords are blue. Could you add syntax highlighting for strings, digits, comments, if, for... ?

Fixed.

Sounds great @epixode !


Update to nGraph v0.9.0

TESTNOW

TESTNOW

I had forgotten to push the ngraph-mxnet-bridge update

Instead of https://jira01.devtools.intel.com/browse/NGRAPH-3111 ( long term fix), this PR is LGTM


Dig points disappear under transparent skin surface using mne coreg GUI

Description of the problem
Hi! Looks like there's something quirky going on with mne coreg.
Additional points in a head digitization are disappearing from view when they sink beneath the skin surface, regardless of the opacity of the surface. While cHPI coils and EEG sensor locations persist (red & pink dots), additional dig points (white dots) disappear.
Steps to reproduce
1. Set your subjects dir path to point to the "subjects" directory of the MNE sample data set.
2. In the command line, open `mne coreg`
3. In the "path to info" field, set "sample_audvis_raw.fif" from the mne-python MEG sample dataset as your digitization source.
4. Using the translation buttons, decrease the y-translation value (hit the down arrow by "tY") until the digitization points in the back of the head cross the head surface.

Link to data
No response
Expected results
White dots should remain visible underneath the skin surface.
Actual results

Additional information
`>>> mne.sys_info()
Platform             Linux-6.2.0-34-generic-x86_64-with-glibc2.35
Python               3.11.5 | packaged by conda-forge | (main, Aug 27 2023, 03:34:09) [GCC 12.3.0]
Executable           /home/erica/miniconda3/envs/sprint_2023/bin/python
CPU                  x86_64 (24 cores)
Memory               62.7 GB
Core
├☑ mne               1.6.0.dev88+gb9e907667
├☑ numpy             1.26.0 (unknown linalg bindings (threadpoolctl module not found: No module named 'threadpoolctl'))
├☑ scipy             1.11.2
├☑ matplotlib        3.8.0 (backend=QtAgg)
├☑ pooch             1.7.0
└☑ jinja2            3.1.2
Numerical (optional)
├☑ nibabel           5.1.0
├☑ dipy              1.7.0
├☑ pandas            2.1.1
└☐ unavailable       sklearn, numba, nilearn, openmeeg, cupy
Visualization (optional)
├☑ pyvista           0.42.2 (OpenGL 4.6 (Core Profile) Mesa 23.0.4-0ubuntu1~22.04.1 via AMD Radeon RX 570 Series (polaris10, LLVM 15.0.7, DRM 3.49, 6.2.0-34-generic))
├☑ pyvistaqt         0.11.0
├☑ vtk               9.2.6
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
├☑ qtpy              2.4.0 (PyQt5=5.15.2)
└☐ unavailable       ipympl, pyqtgraph, mne-qt-browser, ipywidgets, trame_client, trame_server, trame_vtk, trame_vuetify
Ecosystem (optional)
├☑ mne-bids          0.14.dev0
├☑ mne-connectivity  0.6.0dev0
└☐ unavailable       mne-nirs, mne-features, mne-icalabel, mne-bids-pipeline
`

Running this to try to rule some stuff out
MNE_COREG_HEAD_OPACITY=0.25 MNE_COREG_ADVANCED_RENDERING=false MNE_COREG_MARK_INSIDE=false mne coreg --subjects-dir=~/mne_data/MNE-sample-data/subjects -s sample -f ~/mne_data/MNE-sample-data/MEG/sample/sample_audvis_raw.fif --trans ~/mne_data/MNE-sample-data/MEG/sample/sample_audvis_raw-trans.fif

and then clicking "lock fiducials" and setting the EEG weight to zero (so the EEG electrodes disappear) it's still clear that many extra points are missing:

Will have to look into why this is the case... this is where the plotting happens if you want to look into it before I get a chance to in the coming days @nordme
https://github.com/mne-tools/mne-python/blob/c54d229913c5fe1bdb7a76b70930f66a89601c95/mne/viz/_3d.py#L1280-L1332

Hi!! I have pulled the fix, but I'm still getting the disappearing points.

Here's my mne sys_info output again:
$ mne sys_info
Platform             Linux-6.2.0-35-generic-x86_64-with-glibc2.35
Python               3.11.5 | packaged by conda-forge | (main, Aug 27 2023, 03:34:09) [GCC 12.3.0]
Executable           /home/erica/miniconda3/envs/sprint_2023/bin/python3.1
CPU                  x86_64 (24 cores)
Memory               62.7 GB
Core
├☑ mne               1.6.0.dev88+gb9e907667
├☑ numpy             1.26.0 (unknown linalg bindings (threadpoolctl module not found: No module named 'threadpoolctl'))
├☑ scipy             1.11.2
├☑ matplotlib        3.8.0 (backend=QtAgg)
├☑ pooch             1.7.0
└☑ jinja2            3.1.2
Numerical (optional)
├☑ nibabel           5.1.0
├☑ dipy              1.7.0
├☑ pandas            2.1.1
└☐ unavailable       sklearn, numba, nilearn, openmeeg, cupy
Visualization (optional)
├☑ pyvista           0.42.2 (OpenGL 4.6 (Core Profile) Mesa 23.0.4-0ubuntu1~22.04.1 via AMD Radeon RX 570 Series (polaris10, LLVM 15.0.7, DRM 3.49, 6.2.0-35-generic))
├☑ pyvistaqt         0.11.0
├☑ vtk               9.2.6
Warning: Ignoring XDG_SESSION_TYPE=wayland on Gnome. Use QT_QPA_PLATFORM=wayland to run on Wayland anyway.
├☑ qtpy              2.4.0 (PyQt5=5.15.2)
└☐ unavailable       ipympl, pyqtgraph, mne-qt-browser, ipywidgets, trame_client, trame_server, trame_vtk, trame_vuetify
Ecosystem (optional)
├☑ mne-bids          0.14.dev0
├☑ mne-connectivity  0.6.0dev0
└☐ unavailable       mne-nirs, mne-features, mne-icalabel, mne-bids-pipeline

Can you try with MNE_COREG_MARK_INSIDE=false in your env? That should at least show them

unrelated note:

Python 3.11.5 | packaged by conda-forge | (main, Aug 27 2023, 03:34:09) [GCC 12.3.0]
Executable /home/erica/miniconda3/envs/sprint_2023/bin/python3.1

executable is reported as python 3.1. I think this is not actually wrong (on my system too, there's a python3.1 in the env bin folder that is a symlink to python3.10; guessing that it's a shim put in by conda/mamba to deal with short-sighted minor version parsing code).  But ideally the symlink should get followed before writing the path, i.e. Path(sys.executable).resolve().

@nordme never mind, I can replicate on my macOS machine. It's an issue with the scalars affecting not just the color but also the glyph scaling even though we set scale=False somewhere. So when we use 0 and 1 to set things to use color0 and color1, we end up scaling the color0 points (inside the head) to zero. I'll investigate further... unclear why this doesn't happen on my Linux machine but does on macOS.

@larsoner  Yeah, sorry I forgot to mention that it happens regardless of what MNE_COREG_MARK_INSIDE is set to.


Delete unused source translation

https://github.com/gbif/portal16/blob/7d3a839d397287fb3d5c16b4a068f52932d8cdf7/locales/source/en-DK/components/downloadReport.json#L19
as per https://github.com/gbif/portal16/issues/1819

Ok, I'll give it a try next.


Add autobinding for React methods

Expected behaviour
React methods are bound to the instance of the class. For example, this.doThing() in a component should be in the context of the component.
This functionality is implemented in dan1elhughes/yaab.
Actual behaviour
Methods are not called in the component's context.
Steps to reproduce behaviour
Add a component method which refers to this without calling this.thing = this.thing.bind(this) in constructor.

@morena needs more time to work on this.
@peasandwell to catch up today and update this issue.


Example Problems UndefVarError: C not defined

Hi,
Thanks for developing this it looks very useful.
I've just got it installed and tried to test it with the examples you've supplied but once I get to either the f = (α - x)^2 + b*(y - x^2)^2 line in Create NLP Solver or opti.minimize( (y - x^2)^2 ) in Using Opti Stack, I get an error UndefVarError: C not defined, from Line 12 in mathops.jl
I'm assuming I've got something setup wrong, if you could help that would be great.
Thanks!

Hi, I run the examples without problems but there is certainly a case that I configured sth wrongly 🤔... Which Julia version do you use? Do you use a manually installed version of casadi or the one automatically downloaded from conda?

Hi,
I'm using Julia 1.5.3 with the miniconda and casadi that are automatically installed.
Thanks!

Hi,
Having looked a bit more at it, it seems that it's only a problem with the multiplication section of the file.
Thanks!

Hi @Liberty009, sorry for the late reply! I believe b79043259927ee43fe261e3ead0865a0355c7eb1 fixes the issue :smiley:


Adaptations for Control Cluster

If the Peer Educator is working in a health center which belongs to a control cluster, the app should behave differently in the following ways:

Preference assessments must not be accessible.
No next ART Refills must be done.
In regular intervals (6-month or 12-month cycle) an "Endpoint" form must be filled in. This questionnaire is similar to (/the same as) the "ART Refill Not Done" form. It contains questions about whether the patient has died, has been hospitalized, moved to another clinic etc.
Consequently: no flagging for next preference assessment and ART Refill in main screen. Instead flagging and sorting according to when the next Endpoint form has to be filled in.

The list of health centers belonging to the control cluster will be provided by the study supervisor.

Retrieve the user data with DatabaseProvider().retrieveLatestUserData() and then use userData.healthCenter.studyArm to check whether the Peer Educator works in an intervention or control cluster.
Make sure the intervention and control clusters are correctly defined in lib/database/beans/HealthCenter.dart's _studyArm map.


Use a record type Globals collecting all scopes

It was getting rather annoying repeating these three scopes again and again

(this should be merged after https://github.com/jespercockx/agda-core/pull/7)


Migrate fully from npm to yarn

Right now there is a yarn and an npm lock file in the code. In some places yarn is used to install dependencies (Dockerfile) in some other places npm is used to run scripts (package.json).

Migrate npm run to yarn
Remove package-lock.json
Update yarn.lock

done


feat: update grafana config to prod

Just copy the grafana config from dev to prod.

@hi-rustin PTAL~

What a big PR.  🤣

What a big PR. 🤣

This is the accumulated update on dev. It seems that separating this part of the configuration files that should be shared will bring us some trouble. For example, after the pod image is updated, should we test the prod image on dev? However, we still need to do some development on dev.

/merge


Set escape directive to backtick in Nano Server Dockerfiles.

Steps to reproduce the issue

Open a Dockerfile for Nano Server. For example, https://github.com/dotnet/dotnet-docker-nightly/blob/master/2.0/sdk/nanoserver/amd64/Dockerfile
Observe that the escape directive is not specified. This means / (backslash) will be the default escape character.

Expected behavior
Use the escape directive, as suggested at (https://docs.docker.com/engine/reference/builder/#escape) to  ` (backtick).

The motivation behind this change is coming from the upcoming Windows Server 1709 release.  With this release the Nano Server image no longer includes PowerShell.  As a result the current .NET Core Dockerfiles need to be refactored accordingly.  These changes can be seen in the Dockerfiles for the Windows Insiders preview.  With these changes, if the default escape directive is used (/), there is a case which needs to double escape the / which is pretty ugly and confusing.  Switching to use the back tick is cleaner and easier to read/understand.  It would be nice to have consistency across all of the .NET Core Nano Server Dockerfiles.
It is also important to note that the escape directive is scoped to the Dockerfile in which it is declared therefore this change will not have an impact on any Dockerfiles that use the .NET Core Docker images as their base image.

Issue moved to dotnet/dotnet-docker #311 via ZenHub


Adds a DownloadFileSize when using Apple Data Source

Connects #94 (Partially addressed)
When using the Apple datasource, this data is easily available. Added to info pain under a Download Size header. It will hide when the version is downloaded, as it is kind of irrelevant information.
Added a quick bytes to readable string converter. Just grabbed a quick one from the internet.

XcodeReleases.com doesn't have this information. In order to fully support it with both datasources, we should possibly PR it for them.

Updated from feedback.
One downside with the Apple Datasource is that there is a check in updateAvailableXcodes that checks the beta site and if it's available, it would chose the data from beta site over /more.
This causes the fileSize not to show as the beta site doesn't include it.

One downside with the Apple Datasource is that there is a check in updateAvailableXcodes that checks the beta site and if it's available, it would chose the data from beta site over /more.
This causes the fileSize not to show as the beta site doesn't include it.

We could change that to "merge" the two AvailableXcodes, taking the Version from /downloads and the file size from /downloads/more, instead of only taking the data from /downloads.


CVE-2022-31129

Hi Christian,
Any chance to update the version of @sap/cds-dk in this library?
Whitesource is failing because of vulnerable moment-2.29.3 lib.

thanks,
Jonatan

Thanks for letting me know.
As this one needs to be fixed by @sap/cds-dk, and there is no need to react here, I will close this issue.  We will follow up internally.


Ruby Basics Exercises

@aristidebamenou I believe you may have intended to push these to your fork, but regardless we don't intend for students to have the answers to the exercises already filled out so I will be closing this PR.


Consider removing phpunit/php-invoker as a dependency

ext-pcntl library is not installed by default in many PHP environments and support for this feature requires a recompile of PHP.
That being said, perhaps phpunit/php-invoker shouldn't be used as a dependency.
btw Not even sure whether its features are used in unit tests of this library at the moment.

It is not. It is an old relic that I had totally forgotten about. Thanks for pointing it out.


WP3.7.4 Finalise software to output metrics for tracking of improvements to the pipeline sky subtraction

Change Request C025 resulted in a change of title, focus and due date.

As explained at 02/SEP/21 WP-PM meeting this task is now finished.


Version 0.3.0

Codecov Report

Merging #9 (e9b6b85) into master (0e807f8) will increase coverage by 6.84%.
The diff coverage is 90.06%.

@@            Coverage Diff             @@
##           master       #9      +/-   ##
==========================================
+ Coverage   78.44%   85.28%   +6.84%
==========================================
  Files          13       22       +9
  Lines         218      367     +149
==========================================
+ Hits          171      313     +142
- Misses         47       54       +7

Impacted Files
Coverage Δ

lib/src/model/errors.dart
20.00% <ø> (ø)

lib/src/model/glossary_info.dart
54.54% <54.54%> (ø)

lib/src/model/glossary_language_pair.dart
60.00% <60.00%> (ø)

lib/src/model/glossary_entries.dart
89.74% <89.74%> (ø)

lib/src/model/glossary_info.g.dart
100.00% <100.00%> (ø)

lib/src/model/glossary_info_list_api_response.dart
100.00% <100.00%> (ø)

...b/src/model/glossary_info_list_api_response.g.dart
100.00% <100.00%> (ø)

lib/src/model/glossary_language_pair.g.dart
100.00% <100.00%> (ø)

...model/glossary_language_pair_list_api_reponse.dart
100.00% <100.00%> (ø)

...del/glossary_language_pair_list_api_reponse.g.dart
100.00% <100.00%> (ø)

... and 3 more

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 0e807f8...e9b6b85. Read the comment docs.


Build scripts mix up SDL 1.2 and SDL2 when cross-compiling

In file included from /home/david/src/mxe/usr/i686-w64-mingw32.shared/include/SDL2/SDL_video.h:32:0,
                 from /home/david/src/mxe/usr/i686-w64-mingw32.shared/include/SDL2/SDL_events.h:33,
                 from /home/david/src/mxe/usr/i686-w64-mingw32.shared/include/SDL2/SDL.h:41,
                 from /home/david/src/mxe/usr/i686-w64-mingw32.shared/include/SDL2/SDL_net.h:42,
                 from /home/david/src/eel/src/eelium/eel_net.h:29,
                 from /home/david/src/eel/src/eelium/eel_net.c:26:
/home/david/src/mxe/usr/i686-w64-mingw32.shared/include/SDL2/SDL_pixels.h:264:9: error: enumerator value for 'SDL_PIXELFORMAT_YV12' is not an integer constant
         SDL_DEFINE_PIXELFOURCC('Y', 'V', '1', '2'),
         ^
[...]

Could this have something to do with Audiality 2 being built for SDL2, whereas EEL (still) only supports SDL 1.2...?

Of course, this isn't going to be a problem once I've migrated all my active projects to SDL2 - and EEL is the last one now.


Remove calls to d.Chk.True. Fixes #2741

This doesn't really fix the problem because PanicIfTrue also takes a var args of interface{}, so it'll box the string param (and all following)
Can we change panic if true to be
func PanicIfTrue(cond bool, message string)
so as to make the point that you shouldn't be doing work in an assertion?

Yep. Even as I was making that change, I didn't understand the benefit of it either. Now I see what you are asking. Also, I woke up last night realizing that d.Chk.True() and d.PanicIfTrue() are opposites. I'll fix these and send it to you again.

PTAL @rafael-atticlabs

@rafael-atticlabs btw, I ran go fmt on the entire source tree. So you may see a couple of files with just formatting changes.

Diff Coverage
File Path

0%
go/merge/candidate.go

0%
go/datas/local_batch_store.go

0%
go/types/ordered_sequences.go

0%
go/merge/three_way.go

0%
go/chunks/chunk_serializer.go

0%
go/merge/three_way_ordered_sequence.go

0%
go/types/opcache.go

0%
go/types/validating_batching_sink.go

0%
go/types/sequence_concat.go

0%
go/chunks/leveldb_store.go

Review all 44 files changed

Powered by Codecov. Last update 11cc65c...09ef58c

nice. lgtm. watch the benchmark bots. i'll bet some benchmark somewhere will tick up slightly.


Enter and Space should activate long presses

Provide a general summary of the feature here
Enter and Space are both used as alternatives in usePress(). However, keyboard events are explicitly filtered out from useLongPress(). The documentation does warn that you should provide an alternative, which makes sense given that some users will not be able to hold a key easily.
But there's still a use case for allowing Space and Enter to perform a long press, for example it's a common interaction for "temporary pausing" such as a in carousels or 'Stories' features in many social apps.
🤔 Expected Behavior?
Space and Enter should activate long presses
😯 Current Behavior
Space and Enter do not activate long presses
💁 Possible Solution
No response
🔦 Context
Had to reimplement useLongPress() in order to get this behavior
💻 Examples
No response
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Are those actually long presses? I tried Stories with touch, and it pauses until the press is ended. But long press finishes after about half a second, which would be a different behavior. If you used useLongPress for this, it'd only pause the video for half a second and then it would resume. Seems like useLongPress isn't the right thing to use for this interaction pattern to begin with?
It also doesn't take a long press for the keyboard version of the interaction, just a quick tap of the 'Space' key will pause a video.
Maybe you can provide more details about what you're building, or an example app? Maybe some interaction patterns are being conflated. For instance, I think you can just use usePress with onPressStart and onPressEnd and use the interaction mode to determine what actions to take.
Something along these lines?
Touch: pause on press start, resume on press end
Mouse: toggle pause on press
Keyboard: toggle pause on press

We did not include keyboard interactions in useLongPress by default because they often differ between components. For example, in a button with both normal and long press mouse/touch interactions, you might use Alt + Enter or Alt + ArrowDown to open a long press menu. We could potentially make it configurable, but it doesn't seem too hard to use useLongPress in combination with your own onKeyDown listener either.


Feature: AutoResizing of iframe

Looks good! Left a few comments, could you also add a unit test for this case?
Thanks!

One more thing that came to mind -- can we make this configurable? Like I should be able to turn it on or off, as the component author:
https://github.com/krakenjs/xcomponent/issues/7

Yup. We could do that. Cool idea.
On Jun 27, 2016 4:30 PM, "Daniel Brain"<EMAIL_ADDRESS>wrote:

One more thing that came to mind -- can we make this configurable? Like I
should be able to turn it on or off, as the component author:
#7 https://github.com/krakenjs/xcomponent/issues/7
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/krakenjs/xcomponent/pull/28#issuecomment-228906711,
or mute the thread
https://github.com/notifications/unsubscribe/ALjTmlN5UVRlOAqF9Q3uZUBMFfh0JWf0ks5qQF0hgaJpZM4I-JB7
.

It would also be good to squash these commits into one so we end up with a clean commit for the feature:
http://eli.thegreenplace.net/2014/02/19/squashing-github-pull-requests-into-a-single-commit

Hi Vishaka, added a few more comments -- mostly just code organization, the logic looks really good :) thanks for taking care of this!
Few more things that didn't fit into any other specific place:

For features like this, can you please squash commits? http://eli.thegreenplace.net/2014/02/19/squashing-github-pull-requests-into-a-single-commit Then the history is clean, and we keep tests passing from one commit to another

Can you add the autoResize option (See https://github.com/krakenjs/xcomponent/issues/7) before we merge this in? This should probably default to false, since many components are likely to just be a static size.

Thanks!
Daniel

Looks perfect! Merging :)

@bluepnume Is this feature still working on master? I'm trying to embed using an iframe, but it is not auto resized:

Am I missing something in the configuration? thx


Automated cherry pick of #9644: feature(climc): policy save data into file

Cherry pick of #9644 on release/3.4.
#9644: feature(climc): policy save data into file

Codecov Report

Merging #9647 (9cdd7dd) into release/3.4 (fc16c37) will decrease coverage by 2.63%.
The diff coverage is 0.44%.

@@              Coverage Diff               @@
##           release/3.4   #9647      +/-   ##
==============================================
- Coverage         6.19%   3.56%   -2.64%
==============================================
  Files              761     306     -455
  Lines           118405   62969   -55436
==============================================
- Hits              7330    2242    -5088
+ Misses          110412   60571   -49841
+ Partials           663     156     -507

Flag
Coverage Δ

aFlag
3.56% <0.44%> (-2.64%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
1.78% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/capabilities.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudaccounts.go
0.47% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudregions.go
1.52% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudsync.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_skus.go
2.79% <0.00%> (-0.04%)
:arrow_down:

pkg/compute/models/domainquota.go
23.80% <0.00%> (ø)

pkg/compute/models/elasticcache_skus.go
3.13% <0.00%> (-0.05%)
:arrow_down:

pkg/compute/models/elasticipresource.go
0.00% <0.00%> (ø)

pkg/compute/models/elasticips.go
1.07% <0.00%> (-0.02%)
:arrow_down:

... and 452 more

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 8549df1...9cdd7dd. Read the comment docs.


CCDr algorithm can't run

Describe the bug
- cdt version: '0.5.23'
- python version: 3.6.9
- no Gpu
when I run CCDr algorithm in ipython,  it got stucked by hours，the code is snippet in doc:
import networkx as nx
from cdt.causality.graph import CCDr
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
obj = CCCDr()
output = obj.predict(data)

if I use 'ctrl + C' the output is below, it seem   RPackages.sparsebn got stucked:

KeyboardInterrupt                         Traceback (most recent call last)
 in
3 from cdt.data import load_dataset
4 data, graph = load_dataset("sachs")
----> 5 obj = CCDr()
6 output = obj.predict(data)
~/.local/lib/python3.6/site-packages/cdt/causality/graph/CCDr.py in init(self, verbose)
87     def init(self, verbose=None):
88         """Init the model and its available arguments."""
---> 89         if not RPackages.sparsebn:
90             raise ImportError("R Package sparsebn is not available.")
91
~/.local/lib/python3.6/site-packages/cdt/utils/R.py in getattribute(self, name)
130         out = object.getattribute(self, name)
131         if out is None and not object.getattribute(self, 'init'):
--> 132             availability = self.check_R_package(name)
133             setattr(self, name, availability)
134             return availability
~/.local/lib/python3.6/site-packages/cdt/utils/R.py in check_R_package(self, package)
145         """
146         test_package = not bool(launch_R_script(Path("{}/R_templates/test_import.R".format(os.path.dirname(os.path.realpath(file)))),
--> 147                                                      {"{package}": package}, verbose=True))
148         return test_package
149
~/.local/lib/python3.6/site-packages/cdt/utils/R.py in launch_R_script(template, arguments, output_function, verbose, debug)
192             output = subprocess.call([str(rpath), "--vanilla", str(scriptpath)],
193                                     stdout=subprocess.DEVNULL,
--> 194                                     stderr=subprocess.DEVNULL)
195         except Exception as e:
196             print("R Call errored, is R available ?")
/usr/lib/python3.6/subprocess.py in call(timeout, *popenargs, **kwargs)
287     with Popen(*popenargs, **kwargs) as p:
288         try:
--> 289             return p.wait(timeout=timeout)
290         except:
291             p.kill()
/usr/lib/python3.6/subprocess.py in wait(self, timeout, endtime)
1475                         if self.returncode is not None:
1476                             break  # Another thread waited.
-> 1477                         (pid, sts) = self._try_wait(0)
1478                         # Check the pid and loop as waitpid has been known to
1479                         # return 0 even without WNOHANG in odd situations.
/usr/lib/python3.6/subprocess.py in _try_wait(self, wait_flags)
1422             """All callers to this function MUST hold self._waitpid_lock."""
1423             try:
-> 1424                 (pid, sts) = os.waitpid(self.pid, wait_flags)
1425             except ChildProcessError:
1426                 # This happens if SIGCLD is set to be ignored or waiting
KeyboardInterrupt:

Hello,
Thanks for the feedback, have you tried on other, smaller datasets ? This seems like a inherent CCDr issue (internal to the R library).
Best regards,
Diviyan

I have tried other datasets, only about 1000 data. It got stucked too


⚠ ClamAV has degraded performance

In ca45ad1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 21117a3.


[ROADMAP] Updates to 0.7.x

Is your feature request related to a problem?

[ ] Yes

Please describe the problem
For tracking purposes, here is my list for changes to be done for next release.
As some of these imply a bog change to the integration, I consider a major version change and jump to 0.7.x
Describe the solution you'd like
Major Goals for Updates
FRONTEND

Fix AC commands and Climate Control (as REQ #44)
Redo AC control and rename to AC Blowing (or similar)
Fix Rear Window Defrost Button and Action/Service(currently not fully working)
Move "Open Boot" button to Lock, to avoid accidental calls.
Rename Battery Voltage to Ancillary Battery Voltage
Show only Battery Heating Status sensor if vehicle has Battery Heating capabilities
Add lastKeySeen id sensor. (might be useful for some users)

BACKEND

Revise code and sort classes alphabetically, for better understanding, and breakdown by data type (status, charging, etc..)
Reorganize and refactor some code sections to improve handling and performance.
Add a different Update Interval to when the vehicle is Powered On/Driving (changing update intervals to 10 mins, for example).
Try to re-login during updates if token expired, for example, by using the iSmart app.

Describe alternatives you've considered
No response
Additional context
Should any user have additional requirements or suggestions, feel free to jump in.

Closed as all features are implemented on 0.7.0
Outstanding only the Show only Battery Heating Status sensor if vehicle has Battery Heating capabilities has I did not find a way to identify this feature on vehicle info. Kept for next updates if possible


Bump up version to v0.0.1.dev4

Summary
Bump up version to v0.0.1.dev4.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.47%. Comparing base (90ad4db) to head (ceee6dd).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage   84.47%   84.47%
=======================================
  Files         103      103
  Lines       11313    11313
=======================================
  Hits         9557     9557
  Misses       1756     1756

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


bug: doesn't handle image exif.orientation

Hi, I am using #3.11.0, I add an image with exif.orientation=6. It should adjust the image to make it right, but it does. I have attached the pptx. Here is my git link
react-demo.pptx
Thanks for the great product.

If you're expecting the images added to presentations to honor exif attributes, that's not how this library or OOXML works. You'll need to use the API provided to set scaling, orientation, etc.
See the demos for examples.


use mode when creating overviews for integer columns

when a table has an integer column the avg is being calculated when you usually need the mode (they are categorical values)

The problem is integer columns may represent categorical data or any other kind of magnitude (think population...)

the final decision is:

when the column has less than 20 different values -> mode for aggregation
more than 20 -> avg


missing package prefix un pyorangecloud_launch.py

raise PyOrangeCloudError should by raise pyorangecloud.PyOrangeCloudError

Thanks! Just fixed. The pyorangecloud_launch.py file has been updated.


Update GitHub Action to Use .pr_agent.toml

PR Type:
Refactoring

PR Description:
This PR updates the GitHub action to use the '.pr_agent.toml' file from the default branch instead of the head branch. It also includes logging enhancements to provide more information about the application of repository settings. The changes are as follows:

The '.pr_agent.toml' file is now fetched from the default branch.
Added logging to indicate the application of repository settings.
The 'apply_repo_settings' function is now used in the GitHub action runner.

PR Main Files Walkthrough:
 files:
pr_agent/git_providers/github_provider.py: The '.pr_agent.toml' file is now fetched from the default branch instead of the head branch.
pr_agent/git_providers/utils.py: Added logging to indicate the application of repository settings.
pr_agent/servers/github_action_runner.py: The 'apply_repo_settings' function is now used in the GitHub action runner, and additional logging has been added.

/describe

Not really breaking, but it does make a bit annoying that you need to merge the config with the default branch in order to test the config.


Having trouble getting it to work

Hi, I'm trying to get this to work, but without much result.
Here's my setup:
Craft version 3.7.8
Craft Vite plugin version 1.0.20
Local server: MAMP PRO
Local URL: http://localhost/test.com/public_html/
First question would be: after installing the Craft Vite plugin, do I need to manually install Vite.js ? Or does the plugin do this for me? (I manually installed it using npm install vite --save-dev)
Second question: do I need to manually start the Vite server? After adding the scripts dev, build and preview to the package.json I'm able to do npm run dev and Vite will start a server at  http://localhost:3000/. But following that link results in an ERR_CONNECTION_REFUSED error.
Am I missing something in the docs? Or should the above simply work and am I doing something wrong?
Thanks in advance!

Yes, you have to install Vite.js yourself and run it however you want to run it.
You should not access the Vite dev server directly via http://localhost:3000/, you should access your website however you normally do in your local dev environment.
The Vite dev server is only for serving assets like JS, CSS, etc.
I think the issues you've having are unrelated to the Vite plugin itself.


Make sure mod id doesn't contain : when using mavenGroup:id format.

Basically using the following would try to look up org.quiltmc::quilt_loader rather than org.quiltmc:quilt_loader
"depends": [
      "org.quiltmc:quilt_loader",
]

Example error message:

Added some tests, although this code is unlikely to break.

Looks good, thanks!


Please provide a man page for Clangd

Manpages are helpful to know what clangd can do if we can not view online documentation.

Closing seems there is no need.

Closing seems there is no need.

FWIW, on my (debian) system there is a clangd man page - it seems to be autogenerated from the --help so it just covers the basic flags.

FWIW, on my (debian) system there is a clangd man page - it seems to be autogenerated from the --help so it just covers the basic flags.


routeChangeStart alternative in new angular router?

Hello,
I'm trying to implement logic which scrolls to the top of the page whenever the route changes, I used to achieve this by listening to the routeChangeStart event but its not there any more in the new router .. Any solutions?
Thanks.

Have a look at the livecycle hooks. In your case, the can(de)Activate hooks. Not exactly the same tough.
However, if you really need this, you can create an service that monitors $location and spawns events/callbacks when needed.
Not too sure if this is something I want in the new router tough.

Late to the party but playing with new router which is to say scouring source quite heavily at the moment. Basically as @SanderElias mentions I think a $watch is the answer here based on how the new router works. What's missing of course is due to the newness, documentation.
Anyway doing a $watch of $location.path() is exactly how the RootRouter kicks things off anyway. Probably don't even need a service could likely live in the RouterController.
Once some examples are out there I think this concern largely goes away. Add to that since EventEmiiters are not used all over the place, the sugar in 1.x ($routeChangeStart etc) that merely said hey I stated or stopped via broadcasts may be now out of place.
Feel feel to correct me if I've missed something, however this is how I see it I investigate the new router.


patch mesh validate

Signed-off-by: Rudraksh Pareek<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Does quick stuff so mesh validate waits for the events endpoint
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3584 (0c66693) into master (008287e) will decrease coverage by 2.44%.
The diff coverage is 0.36%.

@@            Coverage Diff             @@
##           master    #3584      +/-   ##
==========================================
- Coverage   12.63%   10.18%   -2.45%
==========================================
  Files          37       37
  Lines        2929     3299     +370
==========================================
- Hits          370      336      -34
- Misses       2467     2896     +429
+ Partials       92       67      -25

Flag
Coverage Δ

gointegrationtests
?

unittests
10.18% <0.36%> (-1.22%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/config/config.go
0.81% <0.00%> (-0.11%)
:arrow_down:

mesheryctl/internal/cli/root/perf/perf.go
26.66% <0.00%> (-23.34%)
:arrow_down:

mesheryctl/internal/cli/root/system/channel.go
16.49% <0.00%> (-2.33%)
:arrow_down:

mesheryctl/internal/cli/root/system/config.go
1.11% <ø> (ø)

mesheryctl/internal/cli/root/system/login.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/reset.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/restart.go
2.32% <0.00%> (-0.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.20% <0.00%> (-0.31%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (ø)

... and 8 more

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 b439d75...0c66693. Read the comment docs.

@dhruv0000 should we hold these changes until the enhancements in the events streamer API? :thinking:

Fixed in https://github.com/meshery/meshery/pull/3689


add color-reset to ascii logo drawing

reset the color after drawing the ascii logo.
should fix #1065 and similar issues. This was probably not noticed before because ascii-drawing is the first called function and many other after it do reset the color.
Note: line_break() also does not reset the color, but I think that's fine, since it does not use color.

LGTM, thanks!


A11y_VisualStudioCodeClient_Customizations and settings_Keyboard:Using keyboard tooltips are not appearing for "Record keys" and "Sort by Precedence" icons.

GitHubTags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #Visual Studio Code Client; #BM-VisualStudioCode-Win32-Nov2021; #Win32; #Keyboard; #WCAG2.1.1; #AINotInScope; #FTP; #STP;
Environment Details:
Application Name: Visual Studio Code Client
Windows Version: Windows 11
Repro Steps:
1.Open Visual studio code editor.
2.Naviagte to setting icon
3)Select "Keyboard shortcuts options".
4)Verify whether the tooltip is appearing for the "Record keys" and "Sort by Precedence" icons using keyboard or not.
Actual Result:
Using keyboard tooltips are not appearing for "Record keys" and "Sort by Precedence" icons.
Expected Result:
Using keyboard tooltips should be appear for "Record keys" and "Sort by Precedence" icons.
User Impact:
Keyboard Users will miss the additional information within tooltip for the control.

Dup of https://github.com/microsoft/vscode/issues/132344


QR Code

🛠 Fixes Issue #398

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
📷 Screenshots

?

@DragonUncaged the exact same extension already exists


building for production...cp: no such file or directory: static/*

I just followed up instructions written and tried to run npm run build and got this error
  building for production...cp: no such file or directory: static/*

however I see generated output in console but files are not written on disk, I use OS.
Any ideas what might be wrong? Should I update config files or something?

Initially the folder is empty (except a .gitkeep). So that's correct - * matches nothing. Once you put something in it, images or whatever, the error may go away.

duplicate of #401

I'm puzzled by the same behavior.
The console output suggests there are files generated, where are those files? I suppose they should be added to the static folder - right?

@vaske Upgrading shelljs to 0.7.6 fixed it for me.

Close (in-activity)


Added support to specify the network used by container

I'm using ddev and need to have the testcontainer in the same network, so with this PR you can specify the docker network it should use.
Also useful in other cases where docker compose is used.
Basically like: https://github.com/testcontainers/testcontainers-java/blob/main/core/src/main/java/org/testcontainers/containers/Container.java#L283

Anything i need to fix for the piplines to pass? Looks like an unrelated issue

Yes it is unrelated

Ok, cool any chance to get it tagged? to use non git url in my composer to install :)

sure I tagged it :) https://github.com/shyim/testcontainer/releases/tag/0.1.1
thank you again! :)


Open full history of trades

We should be able to click a button near latest trades that opens up a paginated view of  the full history of trades (on another page).

Duplicate of https://github.com/zeitgeistpm/ui/issues/1908


Locale automation skript

Run once to extract all missing msgstr
Translate missing ones (manually)
Run skript again to insert all translated strings
I want to merge this change because...
It will make localisation much easier
Screenshots
Translate extraced strings:

Re-run skript, and:

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).

If, by context you mean file and line, then yes.

File and line do not result in separate translations. Context (msgctx) does and that's intentional.

Transifex remains our primary tool for managing translations and I'd like to stick to the flow it provides.


Extending Helper Class

It would be nice to have the ability to easily extend the helper class.
In my case, I cannot write to the file system. Instead, I would extend Helper's acquireLock(), releaseLock(), & getLockLifetime() functions to make them read / write to a table.
BackgroundJob accepts a third parameter for the helper class, but run-job does not pass anything for that parameter.
My alternative is to extend Jobby to make the property 'script' a different script that would do the same thing as run-job, but pass BackgroundJob a new Helper class. Is this the route you would recommend?

@creativebum2 That sounds right to me. You would need to wrap all locking queries in a transaction to ensure you don't get multiple jobs accessing the same lock at once. You could also make it an abstraction to support different databases e.g. mysql, pgsql, etc.


Gulp: don't use huna.min.js

Implement a flag so that you can run without including huna.js entirely, so no errors are caught and sent to the server. Extended version of #23

Running the following to strip the script tag from index.html:
gulp build --nohuna
gulp start --nohuna

Can be used in combination with --dev and all error are raw and uncut wherever you program them.

Doesn't work with the start task.


Multilingual application - dynamic text

Hi,
I wanted to ask if it is possible and if so how to manage the fact of having a multilingual tour.
For example, how can you manage to make the text in dynamic :
buttons: [     {       text: ???,       action: tour.next     }   ]
and also title and text in every single step?
Thank you

@amic81 you'll have to use some sort of i18n package then use that to set your text accordingly.


fix: maven config to the right directory

Pull request description
Closes: https://github.com/kubeshop/testkube/issues/3001
Checklist (choose whats happened)

[ ] breaking change! (describe)
[ ] tested locally
[ ] tested on cluster
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Breaking changes

Changes

Fixes

Looks good, but will this work with https://github.com/kubeshop/testkube-executor-maven/blob/main/pkg/runner/runner.go#L97 ?

This is a good point, Thanks. It is working on the testing I've made.

Looks good, but will this work with https://github.com/kubeshop/testkube-executor-maven/blob/main/pkg/runner/runner.go#L97 ?

it seems that the maven dependency manager runs in a different environment context and is not affected by this change

Looks good, but will this work with https://github.com/kubeshop/testkube-executor-maven/blob/main/pkg/runner/runner.go#L97 ?

it seems that the maven dependency manager runs in a different environment context and is not affected by this change

That is very good news! Glad it works now \o/


Tag property

The purpose of the Tag property is for you to use it for any purpose you want. You can safely store anything in there you want.

tks! :dagger:


Fix normal sign on contact reporting for circle/poly collisions

This pull request fixes a hidden bug where contact points are computed for circle/polygon collisions. It is related to issue #160 where I described the contact points were generated far away from each other, and not correctly resembling the real contact points.

Huh, nice catch. Sorry this took so long to get merged.


Modernize FontDescriptor's Hashable

Make FontDescriptor use the new hash(into:) method.
Also made TextStyle an OptionSet.
Closes #8.

Thanks! It's better that way.

Something I overlooked: the equality operator for TextStyle is redundant and can be removed.

Done.
I hadn't declared it in the right place, I wasn't used with operators at the time.


Kernel dies upon initialization

Hi,
I have an IPython notebook running some code in Julia 0.5.1. A few days ago it was working, but now when I initialize it it throws the error 'the kernel appears to have died'. I get the same error when I create a new notebook.
Any idea on what could be the cause?
Thanks.

See https://github.com/JuliaLang/IJulia.jl#debugging-ijulia-problems

Closing as not reproducible.


Release notes for 0.53.3 has a broken link

Provide a description of requested docs changes
The release notes for 0.53.3 have a link to release 0.53.1 for more information, but that hyperlink has a typo, with an extra . between the 0 and 5:
Broken link: https://github.com/microsoft/PowerToys/releases/tag/v0..53.1
Intended link: https://github.com/microsoft/PowerToys/releases/tag/v0.53.1

Fixed!  good catch


readlink: illegal option -- f

scripts don't work on Mac OS 13.6 using macports

see e.g. https://stackoverflow.com/questions/1055671/how-can-i-get-the-behavior-of-gnus-readlink-f-on-a-mac


create_token XLA error on GPU (and rarely CPU)

I ran example from readme on version 0.2.7 (installation for other versions fails because of this issue). Below is  my code and error:
Am I doing something wrong?
code:
import jax
import mpi4jax

comm = MPI.COMM_WORLD
a = jax.numpy.ones((5,4))
b = mpi4jax.Allreduce(a, op=MPI.SUM, comm=comm)
b_jit = jax.jit(lambda x: mpi4jax.Allreduce(x, op=MPI.SUM, comm=comm))(a)

error:
  File "test.py", line 7, in <module>
    b = mpi4jax.Allreduce(a, op=MPI.SUM, comm=comm)
  File "/opt/conda/lib/python3.7/site-packages/mpi4jax/validation.py", line 90, in wrapped
    return function(*args, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/mpi4jax/collective_ops/allreduce.py", line 66, in Allreduce
    return mpi_allreduce_p.bind(x, token, op=op, comm=comm, transpose=_transpose)
  File "/opt/conda/lib/python3.7/site-packages/jax/core.py", line 282, in bind
    out = top_trace.process_primitive(self, tracers, params)
  File "/opt/conda/lib/python3.7/site-packages/jax/core.py", line 628, in process_primitive
    return primitive.impl(*tracers, **params)
  File "/opt/conda/lib/python3.7/site-packages/jax/interpreters/xla.py", line 238, in apply_primitive
    compiled_fun = xla_primitive_callable(prim, *unsafe_map(arg_spec, args), **params)
  File "/opt/conda/lib/python3.7/site-packages/jax/_src/util.py", line 198, in wrapper
    return cached(bool(FLAGS.jax_enable_x64), *args, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/jax/_src/util.py", line 191, in cached
    return f(*args, **kwargs)
  File "/opt/conda/lib/python3.7/site-packages/jax/interpreters/xla.py", line 288, in xla_primitive_callable
    compiled = backend_compile(backend, built_c, options)
  File "/opt/conda/lib/python3.7/site-packages/jax/interpreters/xla.py", line 352, in backend_compile
    return backend.compile(built_c, compile_options=options)
RuntimeError: Internal: Unexpected shape kind for %after-all.2 = token[] after-all(), metadata={op_type="allreduce_mpi" op_name="allreduce_mpi[ comm=<mpi4jax.utils.HashableMPIType object at 0x7fd1025dfa20>\n               op=<mpi4jax.utils.HashableMPIType object at 0x7fd1025df9b0>\n               transpose=False ]"} and shape index {}

what version of jax and jaxlib do you have?
I believe this is Jax having changed something again... They are breaking create_token often nowadays..

More probably it's jax starting to validate token data, which we do not properly propagate along our operations.
I opened Jax#5707 for that.
@fgvbrt I'll try to quickly fix mpi4jax once jax people tell us how to treat this case.
in the meantime, the only workaround is to use an older version of jax, libjax and mpi4jax.

@fgvbrt they confirmed it's a recent bug introduced in XLA (and therefore jaxlib). I suspect it will take at least 2 weeks for a fixed libjax to be released.

Thanks for investigating it. I am using jaxlib==0.1.59, I will try jaxlib==0.1.57

Fixed in the new jaxlib release.


prevent connections from starting transaction

SQLAlchemy inserts an implicit BEGIN (starts a transaction)
This can cause an error if connection is closed remotely/ disconnected even if connection is not used again
Also prevents disconnect of a long running query that is running during some types of publishes (table updates)
isolation_level available in SQLAlchemy 0.8.2

Adding to master


Track Sensor WRITE_BYTES for Counter Mutation on leader replica

Implementation notes:

The counter mutation on non-leader replica will be accommodated for out of the box because they translated into regular mutations (accounted for in WRITE_BYTES sensors)
The response from leader replica -> coordinator will only encode the leader measured sensors values (that is WRITE_BYTES_REQUEST.<counter>" & WRITE_BYTES_TABLE.<counter> in the custom headers. Please note the the leader replica (not the coordinator) is responsible for forwarding the counter mutation to other replicas and today, we don;t have a mechanism to rollup replica responses on the leader replica
Counter cache bytes are not accommodated for - cache is limited to counter_cache_size_in_mb

I switched to a simpler impl that relies on replica counts instead of parsing the actual headers from non-leader replicas to send sensor values back to coordinator https://github.com/datastax/cassandra/pull/1056/commits/e68e732886fc3a613ee5d2cd3e835b9ec6a8fb6f (doesn't require wrapping response handler in CounterWriteReponseHandler anymore.
The reason I switch from passing the handler (could be the vanilla WriteResponseHandler now) is to simplify testing. Mocking replica plans inside handlers with the static assureSufficientLiveReplicasForWrite method at play was not possible.
@sbtourist PTAL

Resolved conflicts https://github.com/datastax/cassandra/pull/1056/commits/358441b215627f651a243410cc67c84afe2096c3, merging.


Clarify usage of ChannelIndex objects

This PR updates the documentation of the neo.ChannelIndex class based on discussions in issues https://github.com/NeuralEnsemble/python-neo/issues/309 and https://github.com/NeuralEnsemble/python-neo/issues/350 and offers a new starting point to clarify the intended use of neo.ChannelIndex objects and directly update the corresponding documentation. I would be happy about some feedback and suggestions / for additional use cases to include in the docstring.

Coverage remained the same at 49.62% when pulling da022ddf35b35877aba805a603662cc95f3fb595 on JuliaSprenger:doc/channel_index into 37f974463136bb2c85b2ebfa8686385ed42c9955 on NeuralEnsemble:master.


Search execution trace

Trace API should offer functionality for indexing the trace and letting clients search for matching expressions. For example, the user might search for all calls to JQuery methods that manipulate the DOM on the DOM element 'button5'. Support should be provided for querying based on both identifiers and runtime values.

Dana, here it is an example tha illustrates what we have to trace https://jsfiddle.net/luminaxster/g07c57o1/1/

Thanks, David.
Perhaps, I did not make it clear. I understand the part you explained with examples. My question is trying to see how we will present the results. But, I guess it is for the Execution Visualization and I should not be too concerned.

Based in our session today:
TODO LIST
0. Provide a list of test cases that we can agree on.

Create a list of all JS (and Jquery) DOM API and classify them by read and write (getters and setters)
Create a list of syntax types (look the full list in Syntax array defined at the beginning od esmorph.js)
Analyze the stackdata entry stackDataEntry in line 18 of this file.
Store statckdata in the JSLINQ format
Provide queries that satisfy the API requirements

This issue was not meant to be in Sprint 2. We need to change the milestone.

TODO: make more specific and clarify what is part of milestone 4.


Remove need for separate EgardiaServer setup

Description:
We removed the need for the user to run a separate Egardiaserver component. This is a breaking change, but makes the usage of the component much easier (and it can now also run on HASS.io).
This is a replacement for https://github.com/home-assistant/home-assistant/pull/11322
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4282
Example entry for configuration.yaml (if applicable):
alarm_control_panel:
  - platform: egardia
    host: XX.XX.XX.XX
    username: XX
    password: XX
    report_server_enabled: True
    report_server_port: 52010

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

See my comments in the old PR about home assistant event listener for stop.

Thanks for your help Martin!


grabserial: add package

This is the grabserial python utility for timestamping serial port output.
Homepage: https://github.com/tbird20d/grabserial
Please let me know if the package location in nixpkgs needs to be changed or any other change needs to be made.
Thanks and Regards,
Venkat

@pSub I fixed the syntax errors in the meta information. I should have caught that before submitting the pull request.
@bjornfor I have updated the version number string to the last commit date.
Thanks and Regards,
Venkat

The Travis CI build failed because of a build issue unrelated to grabserial. Can anyone please help me resolve this issue?
Thanks and Regards,
Venkat
==> Merging PR into base
error: cannot import ‘/nix/store/zapaadgbk6pbfbxdiq1pk6ax62dra3b0-nixui/default.nix’, since path ‘/nix/store/01rsibv0w4h8yhcm8fv0qfv0p3sw655v-nixui.drv’ is not valid, at "/home/travis/.nox/nixpkgs/pkgs/tools/package-management/nixui/default.nix":13:12

@vmandela: Don't worry about Travis CI, we're used to it fail a lot :-)
Btw, I see that grabserial-1.6.0 was released 2014-01-07 (changelog in grabserial script). What's the reason for going with git HEAD instead of the release?

Ah, upstream doesn't do releases it seems (no tags or archives...).

Ooops, forgot we're in 2015! Never mind previous comment :-)

Pushed to master with small fix (01fc3e0525c6c6c20cbd0b8f40e9a939505a5e36). Thanks!


Why is the conversion of OpenIdConnect to oAuth enabled by default?

SecuritySchemes of type openIdConnect are supported in OpenAPI 3.0.x. I can only guess that this is to address limitations in code-gen tooling?

fixed in 0.8.0 -- this conversion is now optional and enabled with the --oidc-to-oauth2 option or the --scopes option.


Bump version to 0.2.68

New release to make F_OFD_* constants available (from #1686).

r? @JohnTitor
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks! Waiting for Alex's decision. Also, CI failed due to some demoted targets. I fixed CI on master so it'd be great if you could rebase here to make CI happy :)

Rebasing done :)

Okay, Alex says "it's okay" and I can make a release on crates.io now. Let's move on.

Thank you!