CLI Flags, JSON configuration files, and default values

The current setup with some parameters specified as CLI flags and others in the JSON file is not optimal. There are two competing needs:

We would like to group related parameters in a JSON file so that it's easy to share model configurations with the community. Attaching the JSON file to a saved checkpoint makes it trivial to replicate experiments and test networks that others with more resources have trained, thus helping democratize the platform.
Changing individual parameters during development should be very, very easy.

We should find a way to accommodate both. I propose:

All parameters are defined as flags.
A list of JSON files are read sequentially and merged, and finally the supplied flags are merged with the combined JSON dictionary for manual overrides.
We would have a JSON file with model parameters, a JSON file with training parameters (learning rate, etc.), and users could add another one with their specific overrides.

It still doesn't feel optimal, though. Any suggestions?

As mentioned in #80, whatever we do should either be provided by an external library or include proper tests.

@lemonzi

All parameters are defined as flags.

I like this (I'm assuming "flag" is synonymous with a command-line argument). I observe that with json, there's no place to specify what the possible allowed values are. Whereas with command-line arguments, one can specify an entire description of what it does and the allowed values are, which is useful.

It still doesn't feel optimal, though. Any suggestions?

Is there a need for a list of json files? A list suggests there's a need of a plurality of sources of information, and a single .json file already allows for a hierarchy of objects within it. Seems it could be a bit simpler if we used the hierarchy of json files to simplify the structure and allow a single json file to specify all command-line arguments.
Until we converge on an "ultimate" solution, I'll add any configurable option as a command-line argument


Test with Pint 0.23, other updates

Codecov Report

Merging #83 (e3c2612) into main (89578fd) will decrease coverage by 43.83%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files


default PI to current user when creating new request

Added to the layout serialized user context, so now in addition to roles we always know the current user details (could come in handy in other scenarios).  When creating a new request we just take the user details out of the context and make them the default PI.
closes #241

@jSylvestre this first name thing is an interesting one -- in particular it's a CAS problem, since CAS should provide first name but it isn't.  But still, we should still handle this case.  I'll look into maybe returning an empty string or something if this happens.


Taxbit is being finicky

I am chatting with support about why my exported csv wont upload correctly

I see you!

LOL. I wish I knew how to fix the problem! I hope our support team gets you sorted out though. No shared chat mediums. 😢

When Importing transactions on taxbit if the fee amount is 0 or null the fee currency should also be null/empty.  Also when the address in question is the receiver sending source should be null.


[Swagger validation] CI script runs validation on json files that are not swagger specs

Some services would like to separate definitions of models in additional files, and then refer to those from the swagger file. CI is currently running on these definition files, applying the rules, model and semantic validation and reporting the failures.
I believe we want this type of "definition" files to exist, if so, our CI script should not run for those files. I'd expect the content of the files to be validated, if the definitions are referenced from a swagger spec, since validation should follow the reference.

this should already fixed in latest CI pipeline


contrib/kube: Restrict example-app servicemonitor to default ns

The prometheus-frontend role of the example app kubeprometheus section
is scoped to the default namespace. Thereby the frontend Prometheus
instance is not able to discover anything outside of the default
namespace. We might as well restrict the front end service monitor to
the default namespace too.

Wouldn't this be the same if it's deployed in the default namespace? I feel like we should just set the namespace for all of these objects just to make sure the user actually deploys it in the default namespace. It's an example after all, it is just supposed to serve for demonstration purposes, if people want to use it further, they can investigate. Maybe we should also be more clear in the kube-prometheus docs about this.

Setting the namespace to default for all objects is done in https://github.com/coreos/prometheus-operator/pull/1111.
Any other concerns @brancz?


Figure out the unique ID mechanics for assets

Turns out the TypeID can't be used, because Rust is happily keeping it the same if the structure changes. Still unique within the program, but not what we expect it to be.

It can also change when recompiled with the same structure.
You may wish to use a hash of data layout as ID.
Or generate ID and put it onto type with a macro.

Yep, that's what I'm going to do

I suppose this is referring to this line in blade-asset?
https://github.com/kvark/blade/blob/450bb7d91bccc766c546b806114856fa6f4e0d77/blade-asset/src/lib.rs#L380
I created a very basic proc-macro that just hashes the names of types of the fields of a struct for a similar use-case we had. We are using hot-code reloading to reload our game as a dynamic lib so TypeId is especially contentious for us as it might not even be stable during a single run.

proc-macro to hash type-signatures of fields
#[proc_macro_derive(FieldHash)]
pub fn derive_field_hash(stream: TokenStream) -> TokenStream {
    use std::hash::{Hash, Hasher};

    let input = syn::parse_macro_input!(stream as syn::DeriveInput);
    let mut hasher = std::collections::hash_map::DefaultHasher::new();

    match &input.data {
        syn::Data::Enum(data) => {
            for variant in data.variants.iter() {
                match &variant.fields {
                    syn::Fields::Named(ref fields) => {
                        for field in fields.named.iter() {
                            expand_type(&field.ty).hash(&mut hasher);
                        }
                    }
                    syn::Fields::Unnamed(unnamed) => {
                        for it in unnamed.unnamed.iter() {
                            expand_type(&it.ty).hash(&mut hasher);
                        }
                    }
                    syn::Fields::Unit => {}
                }
            }
        }
        syn::Data::Struct(data) => match &data.fields {
            syn::Fields::Named(ref fields) => {
                for field in fields.named.iter() {
                    expand_type(&field.ty).hash(&mut hasher);
                }
            }
            syn::Fields::Unnamed(unnamed) => {
                for it in unnamed.unnamed.iter() {
                    expand_type(&it.ty).hash(&mut hasher);
                }
            }
            syn::Fields::Unit => {}
        },

        _ => {}
    }

    let mut type_name = input.ident.to_string();
    type_name.hash(&mut hasher);
    let hash = hasher.finish();
    let mut output = format!(
        r#"
impl {type_name} {{
    pub const FIELD_TYPE_HASH: u64 = {hash};

    pub const fn field_type_hash(&self) -> u64 {{
        Self::FIELD_TYPE_HASH
    }}
}}"#
    );
    output.parse().unwrap()
}

fn expand_type(ty: &syn::Type) -> String {
    quote!{#ty}.to_string()
}


fix: tab behaviour in selecting from emoji menu and fix console error

Details
Fixes the tab cycling behaviour in the emoji menu (supports Tab and Shift+Tab) and removes a console error that happened when using it. Using arrow keys should work similar to before.
Fixed Issues

$ https://github.com/Expensify/App/issues/16348
https://github.com/Expensify/App/issues/16348#issuecomment-1479616603
Tests

Enter the app using the browser or desktop
Open a chat and open the emoji menu
Try to select an emoji with arrow keys and the tab key (or Shift+Tab to go back)
Check that the behaviour is as expected and no console error appears

[X] Verify that no errors appear in the JS console

Offline tests

Not applicable
QA Steps

Enter the app using the browser or desktop
Open a chat and open the emoji menu
Try to select an emoji with arrow keys and the tab key (or Shift+Tab to go back)
Check that the behaviour is as expected and no console error appears

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/25725586/226987444-45504792-8719-4770-b276-48448f49ef3e.mp4

Mobile Web - Chrome
https://user-images.githubusercontent.com/25725586/226987523-0dc0a598-5dcd-48fe-8481-b853f4f18f45.mov

Mobile Web - Safari
https://user-images.githubusercontent.com/25725586/226987634-66c57fb8-c7c8-457a-a4e8-b87ee27ba279.mp4

Desktop
https://user-images.githubusercontent.com/25725586/226987906-356eef93-e449-4f41-84b2-f45a8dfe5370.mp4

iOS
https://user-images.githubusercontent.com/25725586/226987976-629e8671-48ad-4871-bb14-f54560d159ae.mov

Android
https://user-images.githubusercontent.com/25725586/226988034-c239bbcb-4593-4b54-aced-a63b79697e28.mov

@mountiny @Santhosh-Sellavel One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

Will do today!

@BeeMargarida This is not yet fixed, the crash still occurs check the video for steps.

Hi, it should be fixed now with the latest commit. There was an edge case not being handled, which involved pressing Shift+Tab when going from the emoji list to the category shortcut (header emojis). It required adding the logic to detect if the Shift key is being pressed.
Please let me know if you still want to go forward with this solution.
https://user-images.githubusercontent.com/25725586/227486559-04256fb4-b746-4969-b513-e17e1b428140.mp4

@Santhosh-Sellavel thanks for thorough testing 🙇 would you be able to checkout this solution again?

@BeeMargarida Can you investigate this again, and check if there is better alternate solution for this
Apart from this early return
if (!this.emojiList) {
        return;
    }

Nice! Thank you @BeeMargarida 🙇
@Santhosh-Sellavel what do you think now?

When I press the tab the first time, the first emoji header icons get focused. But there is unable to focus on any other header icons though.

That's weird, on my side everything is working fine. I'm using a chromium based browser. Are you using default chrome?
https://user-images.githubusercontent.com/25725586/228171350-14bf2348-d060-4e55-a525-141cd7fdcacd.mp4

At any point, either the emoji header icon or emoji should be focused not both.

I did not notice this edge case, I made a commit to fix this (it's small).

Pressing Enter should perform the action for the item which is focused.

This is already the behaviour implemented.
The only weird behaviour is that the arrow keys go straight to the emojis without passing through the category header emojis. This was already the default behaviour so I didn't change it, but left the tab logic to access this, since they are focusable elements.

@BeeMargarida Could we make the arrow keys work for the sections too? I just tested and I agree tis a bit weird if you can access it with the TAB but not with the arrow keys.

Although it still kinda breaks with the categories, basically if you choose some category using tab, it scrolls down, but then if you continue with the TAB you will be taken to the first emoji in the list. So it will be kinda half baked, but I think polishing this is an accessility issue and we already got a lot more working than we needed
I think we dont have to spend more time on this as we will be working on accessibility issues later in future as its own project.

Updated!

Could we make the arrow keys work for the sections too?

Will this be postponed to the accessibility issues to be fixed later? This would require unifying the tab and arrow keys logic, so at least a significant amount code in the file would have to be changed.

Will this be postponed to the accessibility issues to be fixed later? This would require unifying the tab and arrow keys logic, so at least a significant amount code in the file would have to be changed.

Yeah let's leave that for now, its not perfect now and even if we would make the sections buttons work it would not be perfect. Once we will be working on accessibility issues this will have to be refactored

On it now

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

🚀 Deployed to production by https://github.com/luacmartins in version: 1.2.92-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix "Price=free" giving 0 results in Following query

Issue
Fixes #4477
Remarks
I didn't consult the documentation, but simply did a trial-and-error.
"=0" -- didn't work
"<=0" -- work
Disclaimer
I have no idea what I'm doing, but it works.  I have a feeling this might be an SDK problem.

Adding changelog (assuming fix is correct):

Hmm, mine is working on master with just =0.  On one of my accounts, I did see a query timeout...let me track that down. Maybe that's what you were seeing and not 0 results. I'm not sure how this fix would help...but maybe some strange sql quirk on the SDK.

Closing this PR.  Continuing discussion in the original Issue.


Make OPTIMIZE be able to merge all parts with FINAL.

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

You have implemented it only for MergeTree, not for Replicated variants.

It should work in about the same way.
There is some considerations in usability. When you execute an OPTIMIZE query it will create many merges in the replication queue and merges of new parts won't be created for a long period of time. If you have inserts in your table, they will end up efused with the message "Too many parts..."


Task Misc/Voting did not complete within 1 attempts.

Not sure what the issue here is, the script has been running (mostly) fine for the last few days, and anytime an error similar to this pops up I can just run the script again and it continues, but this issue is persisting.
_> loopgyou
Running loopgyou version [4977e41] in KoLmafia r26722
logPreferenceChange => true
logPreferenceChangeFilter => _lastCombatStarted,libram_savedMacro,maximizerMRUList,testudinalTeachings
Preference logPreferenceChangeFilter changed from to _lastCombatStarted,libram_savedMacro,maximizerMRUList,testudinalTeachings
Preference battleAction changed from attack with weapon to custom combat script
autoSatisfyWithMall => true
Preference autoSatisfyWithMall changed from false to true
autoSatisfyWithNPCs => true
Preference autoSatisfyWithNPCs changed from false to true
autoSatisfyWithCoinmasters => true
Preference autoSatisfyWithCoinmasters changed from false to true
dontStopForCounters => true
Preference dontStopForCounters changed from false to true
hpAutoRecoveryTarget => 0.0
Preference hpAutoRecoveryTarget changed from -0.05 to 0.0
mpAutoRecoveryTarget => 0.0
Preference mpAutoRecoveryTarget changed from -0.05 to 0.0
autoTuxedo => true
Preference autoTuxedo changed from false to true
autoPinkyRing => true
Preference autoPinkyRing changed from false to true
autoGarish => true
Preference autoGarish changed from false to true
allowNonMoodBurning => false
Preference allowNonMoodBurning changed from true to false
Preference customCombatScript changed from default to grimoire_macro
CCS set to grimoire_macro
mpAutoRecoveryItems => phonics down;knob goblin superseltzer;mountain stream soda;magical mystery juice;knob goblin seltzer;cherry cloaca cola;soda water;black cherry soda;doc galaktik's invigorating tonic
Preference mpAutoRecoveryItems changed from phonics down;knob goblin superseltzer;mountain stream soda;magical mystery juice;knob goblin seltzer;cherry cloaca cola;soda water to phonics down;knob goblin superseltzer;mountain stream soda;magical mystery juice;knob goblin seltzer;cherry cloaca cola;soda water;black cherry soda;doc galaktik's invigorating tonic
hpAutoRecoveryItems => scroll of drastic healing;cannelloni cocoon;tongue of the walrus;lasagna bandages;doc galaktik's homeopathic elixir
Preference hpAutoRecoveryItems changed from scroll of drastic healing;cannelloni cocoon;tongue of the walrus;lasagna bandages to scroll of drastic healing;cannelloni cocoon;tongue of the walrus;lasagna bandages;doc galaktik's homeopathic elixir
choiceAdventure1106 => 3
Preference choiceAdventure1106 changed from 0 to 3
choiceAdventure1107 => 1
Preference choiceAdventure1107 changed from 0 to 1
choiceAdventure1340 => 3
Preference choiceAdventure1340 changed from 0 to 3
choiceAdventure1341 => 1
Preference choiceAdventure1341 changed from 0 to 1
Executing Misc/Voting [Free action]
Executing Misc/Voting
CCS set to grimoire_macro
Autoattack is disabled.
Visiting The Voting Booth
logPreferenceChange => false
Preference logPreferenceChange changed from true to false
logPreferenceChangeFilter =>
autoSatisfyWithMall => false
autoSatisfyWithNPCs => false
autoSatisfyWithCoinmasters => false
dontStopForCounters => false
hpAutoRecoveryTarget => -0.05
mpAutoRecoveryTarget => -0.05
autoTuxedo => false
autoPinkyRing => false
autoGarish => false
allowNonMoodBurning => true
CCS set to default
(but battle action is currently set to attack with weapon)
mpAutoRecoveryItems => phonics down;knob goblin superseltzer;mountain stream soda;magical mystery juice;knob goblin seltzer;cherry cloaca cola;soda water
hpAutoRecoveryItems => scroll of drastic healing;cannelloni cocoon;tongue of the walrus;lasagna bandages
choiceAdventure1106 => 0
choiceAdventure1107 => 0
choiceAdventure1340 => 0
choiceAdventure1341 => 0
JavaScript exception: Task Misc/Voting did not complete within 1 attempts. Please check what went wrong. (file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js#7378)
at file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js:7378 (checkLimits)
at file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js:7108 (execute)
at file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js:12647 (execute)
at file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js:7011 (run)
at file:/C:/Users/jiggl/AppData/Local/KoLmafia/scripts/loop-gyou/loopgyou.js:17369 (main)_

Do you actually have the voting booth IoTM?
If you do have it, not sure what's going on. You could do the voting manually to fix it for today; once you have the I Voted sticker, the script should continue. (If you have the sticker and the script is still stuck here, try refresh all in the Mafia CLI).
If you don't have it, this might be a Mafia issue. You can check with prefref voteAlways in the CLI if Mafia thinks you have the IoTM or not. If it is mistakenly set to true, then set voteAlways=false should fix it.

I do have the IoTM, I guess it just wasn't reading me as having the sticker properly, a refresh resolved it! Thanks!

Well, not sure about the source of the desync. But I added the refresh as part of the script to try and workaround. Thanks!


scala-cli fails to start with a "bloop server failed to start" error message

Version(s)
0.1.20
Describe the bug
When attempting to execute code with scala-cli, it fails with:
[E] Failed to load project from /path/to/wd/scala-build/.bloop/project_724f53bbeb-402a56f8c4.json
Error: java.lang.RuntimeException: Bloop BSP connection in /home/nrinaudo/.cache/scalacli/bsp-sockets/proc-304411 was unexpectedly closed or bloop didn't start.
For more details, please see '/path/to/wd/scala-build/stacktraces/1677076006-2251002204469956629.log'

I've included a more complete error log at the end of this.
To Reproduce
Attempt to run Scala 3 code via a command such a:
scala-cli ./file.sc

Expected behaviour
I would expect not to get a bloop related error.
Complete error log
This was obtained using -v -v -v:
Fetching List(Dependency(org.scala-lang:scala3-compiler_3, 3.2.2, Configuration(), Set(), Publication(, Type(), Extension(), Classifier()), false, true)), adding List(IvyRepository(Pattern(List(Const(file:///home/nrinaudo/.cache/scalacli/local-repo/v0.1.20//), Var(organisation), Const(/), Var(module), Const(/), Opt(List(Const(scala_), Var(scalaVersion), Const(/))), Opt(List(Const(sbt_), Var(sbtVersion), Const(/))), Var(revision), Const(/), Var(type), Const(s/), Var(artifact), Opt(List(Const(-), Var(classifier))), Const(.), Var(ext))), None, None, true, true, true, false, None, true))
Found 13 artifacts:
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/scala3-compiler_3/3.2.2/scala3-compiler_3-3.2.2.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/scala3-interfaces/3.2.2/scala3-interfaces-3.2.2.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/scala3-library_3/3.2.2/scala3-library_3-3.2.2.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/tasty-core_3/3.2.2/tasty-core_3-3.2.2.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/modules/scala-asm/9.3.0-scala-1/scala-asm-9.3.0-scala-1.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-sbt/compiler-interface/1.3.5/compiler-interface-1.3.5.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-reader/3.19.0/jline-reader-3.19.0.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-terminal/3.19.0/jline-terminal-3.19.0.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/jline/jline-terminal-jna/3.19.0/jline-terminal-jna-3.19.0.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/scala-library/2.13.10/scala-library-2.13.10.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/com/google/protobuf/protobuf-java/3.7.0/protobuf-java-3.7.0.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-sbt/util-interface/1.3.0/util-interface-1.3.0.jar
  /home/nrinaudo/.cache/coursier/v1/https/repo1.maven.org/maven2/net/java/dev/jna/jna/5.3.1/jna-5.3.1.jar

Fetching List(Dependency(org.scala-lang:scala3-library_3, 3.2.2, Configuration(), Set(), Publication(, Type(), Extension(), Classifier()), false, true)), adding List(IvyRepository(Pattern(List(Const(file:///home/nrinaudo/.cache/scalacli/local-repo/v0.1.20//), Var(organisation), Const(/), Var(module), Const(/), Opt(List(Const(scala_), Var(scalaVersion), Const(/))), Opt(List(Const(sbt_), Var(sbtVersion), Const(/))), Var(revision), Const(/), Var(type), Const(s/), Var(artifact), Opt(List(Const(-), Var(classifier))), Const(.), Var(ext))), None, None, true, true, true, false, None, true))
Fetching List(Dependency(org.virtuslab.scala-cli:stubs, 0.1.20, Configuration(), Set(), Publication(, Type(), Extension(), Classifier()), false, true)), adding List(IvyRepository(Pattern(List(Const(file:///home/nrinaudo/.cache/scalacli/local-repo/v0.1.20//), Var(organisation), Const(/), Var(module), Const(/), Opt(List(Const(scala_), Var(scalaVersion), Const(/))), Opt(List(Const(sbt_), Var(sbtVersion), Const(/))), Var(revision), Const(/), Var(type), Const(s/), Var(artifact), Opt(List(Const(-), Var(classifier))), Const(.), Var(ext))), None, None, true, true, true, false, None, true))
Found 1 artifacts:
  /home/nrinaudo/.cache/scalacli/local-repo/v0.1.20/org.virtuslab.scala-cli/stubs/0.1.20/jars/stubs.jar

Checking for a running Bloop server at /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Attempting to connect to Bloop server /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Connection attempt result: Some(Right(java.nio.channels.SocketChannel[connected local= remote=/home/nrinaudo/.local/share/scalacli/bloop/daemon/socket]))
nailgun debug: Sending arguments '' to Nailgun server
nailgun debug: Sending environment variables to Nailgun server
nailgun debug: Sending working directory /home/nrinaudo/dev/nrinaudo/test to Nailgun server
nailgun debug: Sending command to about Nailgun server
nailgun debug: Finished sending command information to Nailgun server
nailgun debug: Starting thread to read stdin...
nailgun debug: Received action Print([B@2367a06c) from Nailgun server
nailgun debug: Received action Print([B@1779b229) from Nailgun server
nailgun debug: Received action Print([B@551c38de) from Nailgun server
nailgun debug: Received action Print([B@7f721d2) from Nailgun server
nailgun debug: Received action Print([B@2d4acc61) from Nailgun server
nailgun debug: Received action Print([B@699869e3) from Nailgun server
nailgun debug: Received action Exit(0) from Nailgun server
nailgun debug: Waiting for stdin thread to finish...
nailgun debug: Waiting for heartbeat thread to finish...
nailgun debug: Returning exit code...
Checking for a running Bloop server at /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Attempting to connect to Bloop server /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Connection attempt result: Some(Right(java.nio.channels.SocketChannel[connected local= remote=/home/nrinaudo/.local/share/scalacli/bloop/daemon/socket]))
Checking for a running Bloop server at /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Attempting to connect to Bloop server /home/nrinaudo/.local/share/scalacli/bloop/daemon ...
Connection attempt result: Some(Right(java.nio.channels.SocketChannel[connected local= remote=/home/nrinaudo/.local/share/scalacli/bloop/daemon/socket]))
nailgun debug: Sending arguments '' to Nailgun server
nailgun debug: Sending environment variables to Nailgun server
nailgun debug: Sending working directory /home/nrinaudo/dev/nrinaudo/test to Nailgun server
nailgun debug: Sending command to about Nailgun server
nailgun debug: Finished sending command information to Nailgun server
nailgun debug: Starting thread to read stdin...
nailgun debug: Received action Print([B@78777449) from Nailgun server
nailgun debug: Received action Print([B@15836fd0) from Nailgun server
nailgun debug: Received action Print([B@51926cb) from Nailgun server
nailgun debug: Received action Print([B@53cec9e4) from Nailgun server
nailgun debug: Received action Print([B@10553749) from Nailgun server
nailgun debug: Received action Print([B@677f53fa) from Nailgun server
nailgun debug: Received action Exit(0) from Nailgun server
nailgun debug: Waiting for stdin thread to finish...
nailgun debug: Waiting for heartbeat thread to finish...
nailgun debug: Returning exit code...

Exception in thread "main" java.lang.RuntimeException: Bloop BSP connection in /home/nrinaudo/.cache/scalacli/bsp-sockets/proc-306146 was unexpectedly closed or bloop didn't start.
	at scala.sys.package$.error(package.scala:27)
	at scala.build.blooprifle.internal.Operations$$anon$1.openSocket(Operations.scala:390)
	at scala.build.bloop.BloopServer$.create$1(BloopServer.scala:126)
	at scala.build.bloop.BloopServer$.connect(BloopServer.scala:142)
	at scala.build.bloop.BloopServer$.bsp(BloopServer.scala:165)
	at scala.build.bloop.BloopServer$.buildServer(BloopServer.scala:184)
	at scala.build.compiler.BloopCompilerMaker.$anonfun$1(BloopCompilerMaker.scala:32)
	at scala.build.compiler.BloopCompiler.<init>(BloopCompiler.scala:13)
	at scala.build.compiler.BloopCompilerMaker.create(BloopCompilerMaker.scala:34)
	at scala.build.compiler.BloopCompilerMaker.create(BloopCompilerMaker.scala:16)
	at scala.build.compiler.ScalaCompilerMaker.withCompiler(ScalaCompilerMaker.scala:29)
	at scala.build.compiler.ScalaCompilerMaker.withCompiler$(ScalaCompilerMaker.scala:7)
	at scala.build.compiler.BloopCompilerMaker.withCompiler(BloopCompilerMaker.scala:11)
	at scala.build.Build$.build(Build.scala:597)
	at scala.cli.commands.run.Run$.runCommand(Run.scala:280)
	at scala.cli.commands.run.Run$.runCommand(Run.scala:58)
	at scala.cli.commands.default.Default.runCommand(Default.scala:58)
	at scala.cli.commands.default.Default.runCommand(Default.scala:44)
	at scala.cli.commands.ScalaCommand.run(ScalaCommand.scala:273)
	at scala.cli.commands.ScalaCommand.run(ScalaCommand.scala:266)
	at caseapp.core.app.CaseApp.main(CaseApp.scala:150)
	at caseapp.core.app.CommandsEntryPoint.main(CommandsEntryPoint.scala:120)
	at scala.cli.ScalaCliCommands.main(ScalaCliCommands.scala:125)
	at scala.cli.ScalaCli$.main0(ScalaCli.scala:198)
	at scala.cli.ScalaCli$.main(ScalaCli.scala:102)
	at scala.cli.ScalaCli.main(ScalaCli.scala)

The numerous messages like
[W] Scala instance jar /home/nicolas/.cache/coursier/v1/https/repo1.maven.org/maven2/org/scala-lang/scala3-library_3/3.2.0/scala3-library_3-3.2.0.jar doesn't exist!

are suspicious. Are these files really there? Why could they have disappeared?

Woah. Something really weird here. Didn't even check the path, but.. /home/nicolas/ is not the path of my home:
15:46:20 ~ $ pwd
/home/nrinaudo

There is no /home/nicolas directory on my setup.

Why would scala-cli look for that path? Is there some configuration path somewhere that might contain invalid data, quite possibly through some bad manipulation on my part?

I'm not sure… Could there be a stale .scala-build directory alongside your script? (Although Scala CLI should override files there - like Bloop project files - if it think they're stale, so I'm not sure which ones would retain stale data.)

I also remember running into weird home directory paths in some containerized environments, with maybe the JVM reading the home dir from /etc/password while HOME in the env was different, or something along those lines.

Well there you go. There was a .scala-build directory, I trashed it, and now things work as expected.

Thank you, @nrinaudo, for reporting. In #1916, we added a warning for users. If this error occurs again, we suggest running the clean command, which I believe, that may help in certain cases


waitfor.js example - use JSDoc

specifying parameter type helps reading code and prevents usage error

Given that this is an example and the requirements are clearly documented in the text, I don't think it adds terribly much.  Would you be interested in fleshing out the stub documentation on the actual website, instead?  There's lots of that.

No, thanks, I would only fix or improve when I run into


NeoActive File Updates

Added new background as well as missing portable systems such as
Game.com and GP32.

Thank you


Enable command decoration smoke tests on Windows

Only on windows, command decorations aren't showing up even though shell integration is getting activated correctly.

Skipping those on windows for now
Originally posted by @meganrogge in https://github.com/microsoft/vscode/issues/148056#issuecomment-1109032288

Might repro locally

Should we close this @Tyriar ?

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Use goblin to pre-check plugins

We should use goblin here to not require us to load the library before actually checking if it's a memflow plugin. This would also prevent us from potential security issues by loading random libraries into the process.
This should also be a separate tracking issue.
Originally posted by @ko1N in https://github.com/memflow/memflow/pull/48#discussion_r567438322

Linux implementation is in - windows and mac still require testing.

Tests added to ci and merged to next.


Random spaces are added when printing Array

Description
It seems some random spaces are added when printing arrays using both the built in print() and  jax.debug.print.
A minimal, reproducible code example is provided below:
import jax
from jax import random

for i in range(3):
    key = random.PRNGKey(i)
    x = random.normal(key, (2, 1))
    print(x)
    jax.debug.print("{}", x)

The output exhibits the issue, showing extra spaces:
[[-0.78476596]
 [ 0.85644484]]
[[-0.78476596]
 [ 0.85644484]]
[[-0.11617039]
 [ 2.2125063 ]]
[[-0.11617039]
 [ 2.2125063 ]]
[[-1.0433589 ]
 [ 0.71743053]]
[[-1.0433589 ]
 [ 0.71743053]]

What jax/jaxlib version are you using?
jax: 0.4.23
jaxlib: 0.4.23
Which accelerator(s) are you using?
CPU
Additional system info?

numpy: 1.26.2
python: 3.11.6 (main, Oct  2 2023, 13:45:54) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]
platform uname:

system: 'Darwin'
release: '23.2.0'
version: 'Darwin Kernel Version 23.2.0: Wed Nov 15 21:53:18 PST 2023; root:xnu-10002.61.3~2/RELEASE_ARM64_T6000'
machine: 'arm64'

NVIDIA GPU info
No response

JAX delegates its array printing to NumPy, so this is how NumPy would format these arrays.
Can you say more about why this is a bug? It looks like it's working as intended to me: the printing is aligning the brackets and the leading digits.

Certainly, as you mentioned, spaces were simply added to align the appearance. I apologize for any confusion.


Possible breaking change with ReactNative upgrade (0.57 to 0.58) ?

Hello,
I just recently upgraded my react-native version from 0.57.7 to 0.58.1.
If everything was working well on both my app on iOS and Android on 0.57.7, just after upgrading, it appears that an API call at the startup of my app, didn't returned anything.
Weird, because with iOS it's working fine even on the new version, and only on Android, it's not...
Here is the general syntax :
[...Actions and Reducers involved before...]
export function* watchStartup(api) {
  yield call(doSomething, api);
}

function* doSomething(api) {
{
         const response = yield call(api.doGetOnAPI);
         ==> response not returning anything...
        if (response.ok)
        {...}
}

Does anyone reported similar issues when upgrading React-Native ? The API call is a very simple one, a simple get, without ID's or headers needed...
Still working fine on 0.57.7, but completely stuck with the upgrade to 0.58
Infos:
    "redux": "4.0.1",
    "redux-logger": "^2.7.4",
    "redux-persist": "^4.6.0",
    "redux-saga": "^0.16.0",
    "redux-thunk": "^2.3.0",
    "reduxsauce": "^0.4.1",
    "react": "16.6.3",
    "react-native": "0.58.1",

Thank's for the help 🙏🏻🙏🏻🙏🏻🙏🏻🙏🏻🙏🏻

Hi! Experiencing same issue only on Android too.
take doesn't catch any actions.
Did you found a workaround ? Or a breaking change we missed ? thank's
    "react": "16.8.3",
    "react-native": "0.59.10",

    "react-redux": "7.1.0",
    "redux": "4.0.4",

    "redux-immutable": "4.0.0",
    "redux-localstorage": "1.0.0-rc5",
    "redux-localstorage-debounce": "0.1.0",
    "redux-persist-filesystem-storage": "1.4.0",

    "redux-saga": "1.0.5",

    "reselect": "4.0.0"

This really seems like some environmental problem. redux-saga doesn't do anything special in node, react-native or in the browser - it's an isomorphic package. I'm afraid we can't help you much, because we don't have the react native expertise - you have to dig into this on your own.

@Andarist Thank's for your really quick answer, yes it's an environmental problem. Just trying to found some kind of similar issue on different redux related project.
I found my problem, just in case someone get the same : it was related to redux-persist-filesystem-storage I downgraded it to 1.3.2.

I'm seeing a lone yield call(myFunc); that stops execution indefinitely. No errors.
So I stepped through the functions to try to find the issue, and watched them all complete successfully. But still execution is never resumed after the function is called. So I tried putting return; immediately at the top of the problem function - and the behavior is exactly the same. Works everywhere for me except React Native (Android and iOS).
Not really a redux-saga problem, but @Andarist if you have any thoughts about debugging this I'd appreciate it greatly. Not sure what to make of this one.

I'm afraid I don't have any immediate tips for debugging this. If myFunc is a "sync" generator (no async stuff involved) you should be able to step through redux-saga internals and reach your log.warn - I can't imagine how it would not be reached unless there is some bug in redux-saga (rather unlikely, given its stability) or that there is some error thrown there which you miss (for any reason, maybe its suppressed somehow?)

I figured as much but I've been stuck mightly on this one. React Native's debugging tools leave a lot to be desired.
I did get it working just a minute ago. I cannot understand for the life of me why the simplest possible test case failed. I tried stepping through to the log.warn statement, but after a couple hundred steps I gave up.
No need to read beyond this point, but for posterity in case it helps someone:
Oddly, while the example I gave inexplicably continues to fail in all scenarios (no debugger/browser debugger/React Native Debugger) - I was able to find the issue in our original code working by inspecting in the browser debugger and changing a sequential call & take to a yield all([ call..., take... ]). No idea why this wasn't apparent to me when I stepped through the code in React Native Debugger.
Since I got the real thing working, I won't question why my test case failed, but wow; React Native development is worse than getting teeth drilled sometimes.

Closing due to inactivity.  If this is still an issue then please reply and I will reopen and try to help debug.  Thanks!


Ignore trailing characters in the token file

This PR fixes something annoying. It is a very primitive solution.
Previously if the token file ended in a newline (which is pretty much
the standard) the code would not work.
This commit just truncates the token to the length of 32 characters
which is the standard token length.

More advanced solution would probably be to use strings.TrimSpace

Yes, but AFAIK that would add the "strings" library (or whatever they are called in go) as an additional dependency.
Here we use go's primitive string type

If I understood correctly "strings" is part of the standard library like "fmt" and "time" so it's not a "additional dependency"

That might be true, I'm not a go-dev

I would probably use the method @Chicken mentioned, because it would allow empty lines anywhere in the file.
strings is indeed in core libraries, and will not add any additional dependencies.
I will look into this further.

So be it then. I'll update my branch

So @romeq do you want me to use the strings.TrimSpace() method or do you wan't me to implement a specific reader for the token file that only reads the first 32 bytes?

I would say it's up to you. I will review both solutions with a high chance of merging as far they look good to me.
More user-friendly method would be to use the TrimSpace function. However, the token shouldn't be modified by the end user so it really doesn't matter in normal use, and IIRC the code doesn't add any newlines at the end. If you'd like, you can add this feature too.
In theory reading a specific number of bytes from the file would be more efficient. For example, if someone went to fill the token file with millions of newlines at the end of the file. This is user's own fault, though, and code doesn't need to care for that.


Upgrade to Vaadin 8

Please upgrade you great extension to Vaadin 8.

Work for this is in progress. I still have to implement the replacement for PagedContainer for V8, then it should be good to go and releasable

Any chance this plugin can be released with or without the PagedContainer any time soon?
It would be great not having to clone and build this plugin manually..

2.0.0 released to Vaadin Directory, contains a PagedDataProvider to replace PagedContainer.


JavaScript: types allowed when decorator is added to a JavaScript file

TypeScript Version: nightly (2.1.0-dev.201xxxxx)
When I add a decorator to a .js file, I no longer get the 'types' can only be used in a .ts file error. Is this expected behavior?
Code

@SomeDecorator
class SomeClass {
    foo(x:number) {

    }
}

Expected behavior:
'types' can only be used in a .ts file
Actual behavior:
no errors

This is a duplicate of https://github.com/Microsoft/TypeScript/issues/11719. @jramsay should have a fix for this in soon.

looking at this again, do not think this is a duplicate.

No errors are being produced by tsc at all, so I don't think it's a dupe. To be clear, I actually like this behavior, but I'm not sure if I should rely on it working this way.

It is a bug. So I would not rely on this behavior.


GraphQL handler not applying operationHeaders on mesh serve

Describe the bug
Upon running mesh serve, the payloads are arriving without any changes to the headers.
To Reproduce
Steps to reproduce the behavior:
Copy paste the graphql handler example from the docs site and connect to a graphql endpoint with auth required.

Expected behavior
HTTP request to arrive with headers as defined in the yml file.

Environment:

OS: Windows
@graphql-mesh/graphql:
NodeJS: 15

Additional context
Have been trying without success to connect to the API provided by Braintree.
https://payments.sandbox.braintree-api.com/graphql
I have tried everything I can find online, but not able to connect successfully.

If you provide the same headers under both schemaHeaders and operationHeaders, does it work?

Closing this issue for now. Feel free to open a new with a reproduction.
Also you can check https://github.com/the-guild-org/Stack/blob/master/CONTRIBUTING.md#a-typical-contributor-workflow


fix(static_centerline_generator): fix test depend

Description

fix(static_centerline_generator): fix test depend
autoware_static_centerline_generator: Cannot locate rosdep definition for [behavior_path_planner]

Tests performed

rosdep install
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

will fix in https://github.com/autowarefoundation/autoware.universe/pull/7442


QuickMask: hemisphere masks based on VisuAlign registration

I tested the QuickMask tool for generating left/right hemisphere masks and it works well. However, it would be really great if the hemisphere masks could be generated based on the nonlinear registration and not just the QuickNII output.

Currently, QuickMask generates masks with white on the left side and black on the right side so that the colour corresponding to the area to include in the analysis will vary depending on whether the brain is anchored to the right or left hemisphere. Brianna has requested that QControl generate masks with a set colour corresponding to the area to include in the analysis, irrespective of whether the sections are anchored to the right or left hemisphere. I guess this may be possible in relation to implementing support for the nonlinear registration?

Currently, QuickMask generates masks with white on the left side and black on the right side so that the colour corresponding to the area to include in the analysis will vary depending on whether the brain is anchored to the right or left hemisphere. Brianna has requested that QControl generate masks with a set colour corresponding to the area to include in the analysis, irrespective of whether the sections are anchored to the right or left hemisphere. I guess this may be possible in relation to implementing support for the nonlinear registration?

Status on 03.09.2021. Priority is on WebAlign. We will return to this

Gergely has implemented support for nonlinear alignment. It works well. Close this issue. 06.01.22.


fix selected text drop issue in Firefox #219

Moved e.preventDefault() outside of the isDraggingNativeItem check, as the default must be prevented in Firefox for non Native items too.
Should isFirefox be checked here? I tested in Chrome and found no issue with preventing the default.

@gaearon Any movement on this? Looks like it's not quite a dupe of #242 as the default must be prevented at capture AND bubble for our separate issues

Hey, sorry, I don't currently have time to review this. I might have some time in September.
As you can see from React DnD API, the backend is pluggable.
Feel free to fork HTML5 backend for now, publish it as a separate package, and use it with your fix.

I can confirm that this works for me. I was seeing the issue mentioned in #221, and using this patched solved the problem.

Hey @ryanhiebert, I think #221 is subtly different. @Enet4 found my patch didn't help.
There is another PR -  #242 - which cancels Firefox's event handling at bubble. That fixed #221 for Enet4 but didn't help me with #219

It may be subtly different, but it did solve my case, which is what was described in #221. I found that if I dropped an avatar which was a part of my drag layer, it would open the url to the image. Using your patch, I no longer have that issue.

Thank you very much for the PR. Your PR did not merge cleanly because you included the dist file—there is no need to do this in PRs. I cherry-picked your change as a different commit so I'm closing this via 2299ece.
Cheers!

The fix for this issue was released in<EMAIL_ADDRESS>Please verify that it works.

Hey @gaearon - yep, works as expected. Sorry about the updated dist & thanks for the sort-of-merge!


Get error: "Unknow blob:preparing" when docker push in docker v1.20

Latestly, I upgrade my docker to v1.20, but when push image, I get error:

unknown blob: Preparing

and i lookup docker logs, found message as follows:
time="2016-09-30T11:22:26.091049762+08:00" level=error msg="Attempting next endpoint for push after error: Get https://ip:port/v2/: http: server gave HTTP response to HTTPS client"

I' m using debian 7.9, docker version:
 Client:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8f09b43-unsupported
 Built:        Mon Sep 26 15:33:43 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8f09b43-unsupported
 Built:        Mon Sep 26 15:33:43 2016
 OS/Arch:      linux/amd64

docker info:
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 13
Server Version: 1.12.0
Storage Driver: overlay
 Backing Filesystem: extfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options:
Kernel Version: 3.18.20-nce-amd64
Operating System: Debian GNU/Linux 7 (wheezy)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.871 GiB
Name: qa-control-pub-qa-build
ID: 7RAW:6IC4:HGWF:MC67:SE4H:JWXW:Y2SP:U3BW:ZIUA:4N7Q:3VSX:NV4S
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>:5000
 <IP_ADDRESS>:5001
 <IP_ADDRESS>:5002
 registry.hz.netease.com
 <IP_ADDRESS>/8

/etc/default/docker configuration as follows:
DOCKER_OPTS="-H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock -b=docker0 -s=overlay --insecure-registry ip:port"

Please follow these instructions to help us diagnose your issue

copy the output of:

docker exec <registry-container> registry -version

copy the command line you used to launch your Registry
restart your docker daemon in debug mode (add -D to the daemon launch arguments)
reproduce your problem and get your docker daemon logs showing the error
copy your registry logs that show the error
provide any relevant detail about your specific Registry configuration (e.g., storage backend used)
indicate if you are using an enterprise proxy, Nginx, or anything else between you and your Registry

@xautjzd Looks like there is no activity on this issue for awhile. I'm closing it accordingly. Let me know if you still have questions.


[#113996483] Ensure all components have consolidated logging

Ensure all components have consolidated logging
What
Acceptance criteria:
if it logs and it is a system component
then its logs should appear in logsearch
or there should be a story to implement the logs
After reviewing our logsearch implementation we found that we are not logging all events most notably we are missing the all the logs under /var/vcap/*/*.log
This PR
Installs and configures logsearch-shipper-boshrelease
to ensure all the logs under /var/vcap/*/*.log are sent to logsearch
ingestor.
How this PR should be reviewed
I've drafted this PR to be reviewed in the following context:

I would like to:

Install the logsearch-shipper bosh release
Add the logsearch-shipper to each node
Configure the logsearch-shipper to send logs to the logsearch ingestor node
Ensure that the logsearch deployment includes the parser rule to tag the collectd metrics that logsearch-shipper

How to Review this PR
Deploy or update your existing deployment. Create ssh tunnel via deployer concourse to kibana:
ssh -L 5601:<IP_ADDRESS>:5601 -i $session/id_rsa vcap@$DEPLOYER_CONCOURSE. You should be able to browse logs and see them appearing in real time.
You can also connect to one of the nodes and verify that the logs are being parsed by the logsearch-shipper process.
Who can review this PR
This PR can be reviewed by a android who is dreaming of electric sheep if a product of the Tyrell corporation can not be found then any member of the corp team will do with the exception of @actionjack and @tmower

Rebased with master

I am concern that this PR setups collectd to send the metrics to ElasticSearch as log events.
After discussing with @bleach  and @alext, we agreed that it makes sense to disable the metrics feature from collectd by setting logsearch.metrics.enabled=false.
That option set to false will prevent collectd to start. It does not matter that the package and config is installed, and it won't conflict with other deployments.
If we disable metrics, probably we can get rid of that filter in logstash_parse.filters

I have found out that there is a issue in the upstream bosh release which makes it not work disabling the collectd service.
I do think that we can live with this issue for the time being until they fix it upstream.

LGTM, merging

/cc @timmow @actionjack
There are some things I missed in the PR and that I noticed now:

Not all the jobs have the logshipper template defined. Is that intended?
We parametrized the job name rather than the release name (https://github.com/alphagov/paas-cf/blob/master/manifests/cf-manifest/deployments/030-cf-jobs.yml#L50). It should be:

  - name: logsearch-shipper
    release: (( grab meta.logsearch_shipper.release.name ))

not
  - release: "logsearch-shipper"
    name: (( grab meta.logsearch_shipper.release.name ))


Use iptables -w flag

To avoid problems with iptables locking, we need to use the -w flag, if available. Kubernetes's pkg/util/iptables has code to check for its availability and use it if so, so the simplest fix is to just switch to using that.
(Closes https://bugzilla.redhat.com/show_bug.cgi?id=1267670)

LGTM

How likely is this to be a problem without this fix? We may pull this into
OSE, I know we've had a few reports of this.
On Oct 2, 2015 5:32 PM, "Dan Williams"<EMAIL_ADDRESS>wrote:

LGTM
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/openshift-sdn/pull/173#issuecomment-145160098
.

How likely is this to be a problem without this fix? We may pull this into OSE, I know we've had a few reports of this.

openshift restarts docker as part of its own startup, and both openshift/kubernetes and docker fiddle with iptables rules at startup. So it's a race condition but it will definitely happen now and then.


Batch: Simpler method signature?

I think it would be good to be able to run simply:
batch.runForNodes(nodes, processorFunction) or
batch.runForQuery(query, processorFunction)

and having batchName, numberOfThreads, numberOfProcessedItems using defaults.
Is it possible to have a syntax like this?:
batch.runForNodes({
  batchName : "someName",
  numberOfThreads : 4,
  numberOfProcessedItems : 10,
  nodes : someNodesArray,
  processorFunction : function() {...},
  runAsSystem : true
});

For the syntactic sugar version, I recommend https://github.com/ciber/alfresco-js-batch-executer. I will ask him to join my project here to publish it as one extension instead of two...

I have integrated the project into this project.


Communication with substraTEE-registry

Get list of enclaves
Register/un-register enclave

Worker registers his enclave. Finished with commit 790a68acf1a23da81ca5788a9cacdaed555eedf1.
Get list and un-register not implemented as not needed for now.


Added support for Ruby 2.5; updated documentation

Coverage remained the same at 85.454% when pulling e1313da66da783d471105fd59fd9b61edd9f4133 on upgrade-ruby into 085514e286d2f7385172e43e46c79a9bb25f4f37 on master.


Cmd arrow

resolve #23

It would be awesome to port this to https://github.com/kscalelabs/sim

It would be awesome to port this to https://github.com/kscalelabs/sim

I opened a PR already

@henri123lemoine could you please add a argument flag like --command-arrow so it's not turned on every time?

@henri123lemoine could you please add a argument flag like --command-arrow so it's not turned on every time?

Done.
Also cleaned up a few other things, and added a command manager class, mainly for modularity and to make future improvements easier.


Only one result is returned if multiple items returned from contract method

Take below method for example, it returns 3 uint, but I can only get one from returnValues
function getAccountLiquidity(address account) public view returns (uint, uint, uint) {
}

Hey can you give me a working index.js / index.ts file so I can test this out with your example code please

FWIW I have also found this issue to happen. Requesting from this contract, for example, claimableRewards returns an array of length two with the required values, however chaining in another call to withdrawableBalance returns one bignumber and lockedBalances the same, when they should be returning more values.

Here's an example, check the comments in the index.js for what is not returning properly. I grabbed a random address for that, if it doesn't return results at all when you try, you should be able to grab one from here.
multicall example.zip

Hey, thanks a lot for the example this helped me track down the issue.
It should be fixed in version 2.1.1 which I have just published!
Let me know if it's working for you now.

Going to check, I just sat down to code around this, thanks for the quick work!

Thanks a million @joshstevens19!! Appreciate the effort on a Sunday, saved me coding around this.

Hey @mashup-purveyor no worries thanks for giving me an example! Glad we sorted it. Let me know if you get anymore issues.


docs: Show examples for other languages

Is your feature request related to a problem? Please describe.
May be useful to show examples in the README of use of watched-files for repo languages other than JS. Case in point, this Java project:
https://github.com/mufajjul/aml-mlflow-java-example/issues/2
The languages I could think of off-hand would be:

[ ] Java, pod.xml?
[ ] Golang, e.g. go.mod, go.sum
[ ] Python, requirements.txt

Suggestions very welcome.
Describe the solution you'd like
Would like to formulate a few samples and validate them with contributors.  Given enough samples, may be better to make a folder of them and link from the README.
cc @bdougie @0-vortex @mufajjul

would go with python first, go modules have multiple rules, not sure about the java path but can investigate further


Proposal : make a synchronous form would be senseful!

See as the title, and if has any solution, please give me some suggest, thinks!

hey @newbie-tan , I'm not sure I get correctly what enhancement you want, but Forms are explained here: https://fastapi.tiangolo.com/tutorial/request-forms/
In this tutorial the function is async, but nothing prevents you from writing it synchronously removing the async keyword
@app.post("/login/")
def login(*, username: str = Form(...), password: str = Form(...)):
    return {"username": username}

tks, butThe reality is that  my front colleagues pass the information to the backend  by using the form table all the place, i want to make it easy to packaging as a native python  data structure.
from starlette.request import Request
@router.post('/demo')
async def get_file(request:Request)
    form = await request.form()
    file_text = from["file"].file._file.read().decode('utf-8')
    path = "xxx"
    and so on ...
    return {"msg":"success"}

it is my solution , help it work in a synchronous project

@tony-is-coding I imagine you want to use synchronous path operations.
You can use normal def instead of async def: https://fastapi.tiangolo.com/async/
And if you use files (https://fastapi.tiangolo.com/tutorial/request-files/#uploadfile) you can access file.file to get the synchronous version.

@tony-is-coding Is the point that you want to be able to access the result of await request.form() without the await? If so, you should be able to accomplish that by defining a wrapper dependency:
async def get_form_data(request: Request):
    return await request.form()

@router.post('/demo')
def get_file(form = Depends(get_form_data)):
    file_text = form["file"].file._file.read().decode('utf-8')
    path = "xxx"
    and so on ...
    return {"msg":"success"}

FastAPI will evaluate the dependency, then inject its result into the threadpool call to get_file.

Does that solve your use case @tony-is-coding ?


feat: endpoint to get details for all namespaces

Get details for all namespaces in the form of:
{
	"ns1": {
		"db1": {
			"coll1": {
				"schema": {},
				"size": x,
			}
		},
		"db2": {
			...
		}
	},
	"ns2": {}
}

Codecov Report
Base: 31.79% // Head: 31.65% // Decreases project coverage by -0.14% :warning:

Coverage data is based on head (57e32fe) compared to base (254ec91).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #608      +/-   ##
==========================================
- Coverage   31.79%   31.65%   -0.15%
==========================================
  Files          96       96
  Lines       11276    11327      +51
==========================================
  Hits         3585     3585
- Misses       7369     7420      +51
  Partials      322      322

Impacted Files
Coverage Δ

api/server/v1/tx.go
0.00% <ø> (ø)

server/request/request.go
13.87% <ø> (ø)

server/services/v1/management.go
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix handling an empty list of versions

This patch addresses this review comment by treating null and an empty list of versions differently. An empty collection passed should return no results, rather than ignoring it and returning all results.
Changes

Update the condition to handle the null and empty versions differently
Clarify javadocs on the behavior of empty versions
Add tests for the empty versions in different classes, which would fail without this patch

This PR has:

[x] been self-reviewed.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[x] been tested in a test Druid cluster.

Thanks for the quick review, @kfaraz!


See post you're replying to

This will be probably be negated by the reply box in conversation view, but if that doesn't happen (or the separate reply view persists as well) I'd like to be able to see the content of the post I'm replying to, I guess above the text box. I use this a lot on other platforms.

Yeah, I think you're right that this is needed if we keep the new post screen for replies.


lakeFSFS with Presigned URLs

This is a continuation of #5105.
Description
This feature will introduce presigned mode to lakeFSFS. In this mode, lakeFSFS will request presigned storage URLs from the lakeFS server. This will be the only way in which lakeFSFS authenticates with the storage. In particular, running in this mode will not require any storage credentials on the client side. The only credentials required are the ones to lakeFS.
To use this mode, the user will need to set a configuration property to true.
Requirements

Correctness: Passes all contract tests, integration tests and otherwise behaves the same as lakeFSFS.
Security and ease-of-use: Client side doesn't require storage permissions.
Performance: Try not to be embarassing.
Cloud agnostic: Keep the usage of Cloud-specific SDKs to a minimum. Works with Azure Blob as the backing storage.

Bonus points

Works with GCP.

Plan

[x] 1. Implement naive version: no multipart-uploads, reads will be saved in memory before streamed (1d).
[x] 2. Head-to-head benchmarking with classic lakeFSFS (1d).
[x] 3. If results are satisfactory, test and fix to work on Azure (2d).
(stop point: make more detailed plan, document how it works, etc.)
[ ] 4. Make reads seekable (3d).
[ ] 5. Testing: make it pass contract tests and others (tests will need to run on both classic and presigned modes) (2d)
[ ] 6. Integration tests for Azure (2d).
[ ] 7. Document it (0.5d).
[ ] 8. Hope we can get away with not implementing multipart uploads (0d).


remove duplicate declaration of cudaMalloc

closes #2027

Jenkins, test this please.

Jenkins CI test (for commit 5af1243b39e3f598f3774ff47310a60812619c8f, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM! Test failures are not related to this PR.

Jenkins CI test (for commit 5af1243b39e3f598f3774ff47310a60812619c8f, target branch master) succeeded!


Roll Engine from 1ef10b240e28 to e16bc5002672 (9 revisions)

https://github.com/flutter/engine/compare/1ef10b240e28...e16bc5002672
2020-09-16<EMAIL_ADDRESS>Roll Skia from 3e72b3ff8ca7 to 2610e8261e9e (4 revisions) (flutter/engine#21210)
2020-09-16<EMAIL_ADDRESS>Roll Dart SDK from c835079edda4 to 7476b58cc9d0 (8 revisions) (flutter/engine#21209)
2020-09-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 8q8xxvY60... to VpUd_W9Oi... (flutter/engine#21206)
2020-09-16<EMAIL_ADDRESS>Verify Flutter clang module, add hook for verifying consumer warnings (flutter/engine#21203)
2020-09-16<EMAIL_ADDRESS>Roll Skia from a195d101f96c to 3e72b3ff8ca7 (7 revisions) (flutter/engine#21205)
2020-09-16<EMAIL_ADDRESS>Add Linux Wayland support (flutter/engine#20629)
2020-09-16<EMAIL_ADDRESS>[darwin] Header #import hygiene (flutter/engine#21193)
2020-09-16<EMAIL_ADDRESS>Fix NPE in PlatformPlugin.getClipboardData() (flutter/engine#21189)
2020-09-15<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from -iBBiPj1C... to sXR21ye3r... (flutter/engine#21197)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from -iBBiPj1CeUA to sXR21ye3rKeL
fuchsia/sdk/core/mac-amd64 from 8q8xxvY60Jci to VpUd_W9Oi9zl
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


"yarn run eject" ejected succcessfully, but followed by "The batch file cannot be found."

Can you reproduce the problem with latest npm?
Yes.
Description
I started a fresh project with create-react-app. yarn start and yarn run build runs as expected. However, when running yarn run eject, it shows Ejected successfully!, but followed by
Ejected successfully!

Please consider sharing why you ejected in this survey:
  http://goo.gl/forms/Bi6CZjk1EqsdelXk1

The batch file cannot be found.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I also tried npm run eject where it shows more error message:
Ejected successfully!

Please consider sharing why you ejected in this survey:
  http://goo.gl/forms/Bi6CZjk1EqsdelXk1

The batch file cannot be found.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>eject: `react-scripts eject`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>eject script 'react-scripts eject'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the testcra7 package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     react-scripts eject
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs testcra7
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls testcra7
npm ERR! There is likely additional logging output above.

npm ERR!      C:\path\to\npm-cache\_logs\2017-04-25T05_03_07_216Z-debug.log

Should I worry about this?
yarn start and yarn run build still run fine after eject tho.
Expected behavior
No error message after Ejected successfully!
Actual behavior
Got error message after Ejected successfully!
Environment

npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>

node -v: 6.10.2

npm -v: 4.5.0

Operating system: Windows 7 (behind corporate firewall)

Browser and version: Chrome 53.0.2785.116

Can you share this “complete log”?
npm ERR!      C:\path\to\npm-cache\_logs\2017-04-25T05_03_07_216Z-debug.log

yeap, here it is.
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\apps\\nodejs\\node.exe',
1 verbose cli   'C:\\apps\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'eject' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'preeject', 'eject', 'posteject' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for preeject, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\apps\nodejs\node_modules\npm\bin\node-gyp-bin;C:\path\to\github\testcra7\node_modules\.bin;"C:\apps\curl\bin";"C:\apps\jdk8u51x64";"C:\apps\jdk8u51x64\bin";"C:\apps\PythonPortable\App";"C:\apps\nodejs";"C:\apps\GitPortable\bin";"C:\apps\GitPortable\usr\bin";"C:\apps\Atom\resources\app\apm\bin";C:\apps\cmder\bin;C:\apps\cmder\vendor\conemu-maximus5\ConEmu\Scripts;C:\apps\cmder\vendor\conemu-maximus5;C:\apps\cmder\vendor\conemu-maximus5\ConEmu;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\Java\jre1.5.0_15\bin;C:\Program Files (x86)\Wave Systems Corp\bin\;C:\Program Files\Wave Systems Corp\Wave TSS\bin\;C:\Program Files (x86)\1E\NomadBranch\;C:\Program Files\TortoiseSVN\bin;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\Java\jre1.5.0_15\bin;C:\Program Files (x86)\Wave Systems Corp\bin\;C:\Program Files\Wave Systems Corp\Wave TSS\bin\;C:\Program Files (x86)\1E\NomadBranch\;C:\apps\curl\bin;C:\apps\jdk8u51x64;C:\apps\PythonPortable\App;C:\apps\nodejs;C:\apps\GitPortable\bin;C:\apps\GitPortable\usr\bin;C:\path\to\AppData\Local\atom\bin;C:\apps\GitPortable\usr\bin;C:\apps\GitPortable\usr\share\vim\vim74;C:\apps\cmder\
10 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\path\to\github\testcra7
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'react-scripts eject' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec eject script
14 verbose stack Error<EMAIL_ADDRESS>eject: `react-scripts eject`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (C:\apps\nodejs\node_modules\npm\lib\utils\lifecycle.js:279:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (C:\apps\nodejs\node_modules\npm\lib\utils\spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:886:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd C:\path\to\github\testcra7
17 verbose Windows_NT 6.1.7601
18 verbose argv "C:\\apps\\nodejs\\node.exe" "C:\\apps\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "eject"
19 verbose node v6.10.2
20 verbose npm  v4.5.0
21 error code ELIFECYCLE
22 error errno 1
23 error<EMAIL_ADDRESS>eject: `react-scripts eject`
23 error Exit status 1
24 error Failed at the<EMAIL_ADDRESS>eject script 'react-scripts eject'.
24 error Make sure you have the latest version of node.js and npm installed.
24 error If you do, this is most likely a problem with the testcra7 package,
24 error not with npm itself.
24 error Tell the author that this fails on your system:
24 error     react-scripts eject
24 error You can get information on how to open an issue for this project with:
24 error     npm bugs testcra7
24 error Or if that isn't available, you can get their info via:
24 error     npm owner ls testcra7
24 error There is likely additional logging output above.
25 verbose exit [ 1, true ]

This is very odd. I don't know what's causing this. 😞

maybe related? http://stackoverflow.com/questions/26240414/jenkins-deleting-batch-file-before-its-completed it's jenkins but the symptoms pretty similar
@phoa do you able to consistently replicate this problem? eg. tried to create another CRA instance and eject it?

@viankakrisna It consistently happens when I eject (10 tries). Same thing happens when I tested it on my colleague's computer.
The ejected package runs fine tho. So far I do not see any problem with it.
I kinda suspect that this may be caused by some corporate settings applied to our computers.
This is not happening with my personal computer.

I get the same error, running Windows 10.  Exact same entries in my log file.

This is happening in windows CI for PR #2400
Looks like it might be to do with running the latest version of yarn (0.24.5) as thats the only change that the PR makes

Got this message too, Windows 10.

Got same error, Windows 8.

Got same error on windows 10. FWIW, sharing the logs too.
$ npm run eject

><EMAIL_ADDRESS>eject C:\Users\mihir.karandikar\Source\Repos\DWH\asdfasdfasdf\src\asdfasdfasdf\Client\asdfasdfasdf
> react-scripts eject

? Are you sure you want to eject? This action is permanent. (y/N) y
? Are you sure you want to eject? This action is permanent. Yes
Ejecting...

Copying files into C:\Users\mihir.karandikar\Source\Repos\DWH\asdfasdfasdf\src\asdfasdfasdf\Client\asdfasdfasdf
  Adding \config\env.js to the project
  Adding \config\paths.js to the project
  Adding \config\polyfills.js to the project
  Adding \config\webpack.config.dev.js to the project
  Adding \config\webpack.config.prod.js to the project
  Adding \config\webpackDevServer.config.js to the project
  Adding \config\jest\cssTransform.js to the project
  Adding \config\jest\fileTransform.js to the project
  Adding \scripts\build.js to the project
  Adding \scripts\start.js to the project
  Adding \scripts\test.js to the project

Updating the dependencies
  Removing react-scripts from devDependencies
  Adding autoprefixer to devDependencies
  Adding babel-core to devDependencies
  Adding babel-eslint to devDependencies
  Adding babel-jest to devDependencies
  Adding babel-loader to devDependencies
  Adding babel-preset-react-app to devDependencies
  Adding babel-runtime to devDependencies
  Adding case-sensitive-paths-webpack-plugin to devDependencies
  Adding chalk to devDependencies
  Adding css-loader to devDependencies
  Adding dotenv to devDependencies
  Adding eslint to devDependencies
  Adding eslint-config-react-app to devDependencies
  Adding eslint-loader to devDependencies
  Adding eslint-plugin-flowtype to devDependencies
  Adding eslint-plugin-import to devDependencies
  Adding eslint-plugin-jsx-a11y to devDependencies
  Adding eslint-plugin-react to devDependencies
  Adding extract-text-webpack-plugin to devDependencies
  Adding file-loader to devDependencies
  Adding fs-extra to devDependencies
  Adding html-webpack-plugin to devDependencies
  Adding jest to devDependencies
  Adding object-assign to devDependencies
  Adding postcss-flexbugs-fixes to devDependencies
  Adding postcss-loader to devDependencies
  Adding promise to devDependencies
  Adding react-dev-utils to devDependencies
  Adding react-error-overlay to devDependencies
  Adding style-loader to devDependencies
  Adding sw-precache-webpack-plugin to devDependencies
  Adding url-loader to devDependencies
  Adding webpack to devDependencies
  Adding webpack-dev-server to devDependencies
  Adding webpack-manifest-plugin to devDependencies
  Adding whatwg-fetch to devDependencies

Updating the scripts
  Replacing "react-scripts start" with "node scripts/start.js"
  Replacing "react-scripts build" with "node scripts/build.js"
  Replacing "react-scripts test" with "node scripts/test.js"

Configuring package.json
  Adding Jest configuration
  Adding Babel preset
  Adding ESLint configuration

Running yarn...
yarn install v0.24.5
[1/4] Resolving packages...
[2/4] Fetching packages...
warning<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check.          Excluding it from installation.
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Saved lockfile.
Done in 17.93s.
Ejected successfully!

Please consider sharing why you ejected in this survey:
  http://goo.gl/forms/Bi6CZjk1EqsdelXk1

The batch file cannot be found.

npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\         node_modules\\npm\\bin\\npm-cli.js" "run" "eject"
npm ERR! node v6.10.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>eject: `react-scripts eject`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>eject script 'react-scripts eject'         .
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the asdfasdfasdf pack         age,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     react-scripts eject
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs asdfasdfasdf
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls asdfasdfasdf
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\mihir.karandikar\Source\Repos\DWH\asdfasdfasdf\src\asdfasdfasdf\Client\asdfasdfasdf\npm-debug.         log

Does anyone want to look into why this happens? I don't know enough about Windows.

I can't reproduce this on a Windows laptop. Tried both Cmd and PowerShell, both npm and yarn.
Can anybody who reproduces this think more about what makes your setup special?

@gaearon - my setup is as follow:

using standalone NodeJS Windows 64 Binary 6.10.2 and NPM 4.6.1 from NPM Github Release Page
yarn is installed via npm install -g yarn
i'm behind corporate-proxy, so my NPM and Yarn have proxy and https-proxy config setup
there are corporate-specific scripts running behind the scene, which may or may not contribute to the error

A colleague of mine has the same setup, and he also got the same error message.
Up till now, the ejected version is still running fine without problem for me, despite the error message during eject.

Same, works fine despite the message.
@gaearon

Windows 8.1
Node v7.5.0 and NPM 4.1.2
yarn installed using .msi
firewall disabled completely

Can't find anything special about my setup apart from that.

same prob... any solution ... I dont running yarn together :(


Deleting multiple note using selection block the app in selection mode

If you select multiple note then move them to trash, the deleted files stays selected and there is no way to unselect them. The None option doesn't unselect the notes.

You have to close the app to work around this issue.


feat(paginator): Add functionality to jump to first and last page

Add two buttons which allow you to jump to the first and last page.
Hidden by default and can be toggled with showHideFirstButtons
Fixes #9278, contains the following fixes and is therefore blocked by: #9597 and #9604 (and therefore also #9584)

There is a bug in rtl mode in this feature. Please fix it. Thank you 😘😅

@bitbeter @crisbeto has replaced the CCS icons with SVG elements in #9776 which should fix it for you!


Allow subdomains, block or allow subdirectories

Closes https://github.com/penge/block-site/issues/12
Closes https://github.com/penge/block-site/issues/9
Closes https://github.com/penge/block-site/issues/5
Inspired by https://github.com/penge/block-site/pull/11 and https://github.com/penge/block-site/pull/7
Huge thanks to @clairdl as I revisited his suggestions on making simpler descriptions, and I completely agree. Also, I'd like to thank to @lumoe for the initiation that inspired me to continue on that and solve currently open issues.

I'd like to invite @lumoe and @clairdl for a review.

Thank you for your effort, also your solution to support filtering by pathname is great. Tested the subdirectories branch with different filter settings on my machine (Ubuntu 20.04 and Chrome 97).
Two things I would suggest:

There should be an example in the README showing that it is possible to have rules with pathname in it
Run the URL normalizing function getDomainUrl also on the rules. Having the rule https://youtube.com/ currently does not block the site. After running the function it would become  youtube.com which would then result in a matching rule.

@lumoe
Thank you for your great suggestions! I have updated the rules to support fully qualified URLs as well, example:
https://youtube.com/
!https://music.youtube.com/

This may be more natural to some, as simple copy & paste URL will now work. Should we document it and provide more examples?
On one side, I don't want to confuse the user with too many examples.  What we could do, is to separate examples with an empty line, and put same examples with fully qualified URLs below. What do you think?
I have added your !reddit.com/r/MachineLearning which I think is great. There's one more example we need. That is, blocking subpath (subdirectory) only. Any suggestions? @lumoe @clairdl

Going to ship it. Summarizing the new features here. You can now:

Exclude subdomain from blocking (https://github.com/penge/block-site/issues/12), example:

youtube.com
!music.youtube.com

The above example would block youtube.com, but allow music.youtube.com.

Block subdirectories only (https://github.com/penge/block-site/issues/9), example:

youtube.com/user/BBC

The above example would block youtube.com/user/BBC only, and so allow youtube.com.

Allow subdirectories only (https://github.com/penge/block-site/issues/5), example:

youtube.com
!youtube.com/results?search_query

The above example would block youtube.com, but allow youtube.com/results?search_query.

Note: Fully qualified URLs will work too (including https or https, www). Trailing slash is optional.


Fix cargo add behaving different when translating package name

Fixes #13702
Fixes #10680
TODOs

[ ] Fuzzy match registry dependencies in cargo remove as well
[ ] Don't duplicate name permutation generation
[ ] Shall we reject cases where multiple different permutations match?
[ ] Add comments that explain the behavior

This PR looks to be basically ready. @dohse, is there anything else you need to do before a final review? I see an unfinished TODO list in the description and one review comment that may not be resolved yet.

Yes, this PR should be ready. I need feedback to my responses (@epage). I would take another look at my TODOs when the general direction of the fix is clear.

I responded to your comments - I will also rebase this PR.

In looking at the responses, its not clear to me what I needed responding to.

Sorry - this PR became confusing, because it contains two possible solutions. I was mostly referring to this comment, that I broke out a function as suggested.

I hope that I addressed the comment.
I am also now only check permutations that are unique

I changed the comments according to the discussion.

I updated the comment.
I also removed the .dedup() of the combinations - it would not prevent all lookups of the unmodified original basename.

Thanks!
@bors r+

:pushpin: Commit 9391bfb34d307ead02c00261de9f506feb45cb76 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 9391bfb34d307ead02c00261de9f506feb45cb76 with merge 1ed3596675d9338a0c215701051e64064b14b633...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 9391bfb34d307ead02c00261de9f506feb45cb76 with merge 40b6638df6ee1360680a7577168eb1be4a167a83...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 40b6638df6ee1360680a7577168eb1be4a167a83 to master...


Fixes #1405. Marks connections with specific 'reserved' keyword as used

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[x ] I have tested this change with a run of GEOSgcm (if non-trivial)
[x ] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x ] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

Yes, it is very similar to what I showed you yesterday, but this line is a different procedure (this on is in APPEND, which puts the items in the container). Yesterday, I had similar logic inside varIsListed

Running tests with this now after merging in #1411

This looks to be zero-diff as well. Tested at C48, AMIP and Replay.


Is it possible to host a UWP control inside a WinUI application?

Describe the bug
I am using a third party UWP control, but it is not available in the WinUI platform yet, can I host it in the WinUI application? if so how?
Steps to reproduce the bug
This is a question, not a bug.
Expected behavior
No response
Screenshots
No response
NuGet package version
None
Windows version
No response
Additional context
No response

Hi @JeganRavichandran. No, it is not possible to host a UWP control inside a WinUI3 application: having two versions of XAML loaded in the same process has a variety of problems.

Closing this issue as it appears to have been answered.


M1 Max MPS F32 / F16 Issues

Hello @BlinkDL,
As per your recommendation, I was able to run this on MPS at half precision. It gets stuck on MPS at full precision.
On 64 GB M1 Max CPU, 14B model gives pretty good results, but it is pretty slow. When I made a few changes to get it working on MPS, it's very fast. But the results are the worst.  In 2018, Bezos defeated Musk in the semi-finals, while Gates took down both of them. So now it’s Bezos and Musk who are playing each other.

What am I missing ?
The most significant change was in sample_logits function, where I used the same probability algorithm for both MPS and CPU. Rest of the changes included only changing the device from CUDA to MPS.

"very fast" means the inference is buggy and not really computing the correct result. MPS is known to be still buggy.

Thanks @BlinkDL. How do I debug this ? Where should I start ?
Can you give me a few pointers ?

@okpatil4u Can you show how to make it work in mps?   When I try to set it in mps(MBP intel x86 CPU + eGPU[RX6800 16G]), it return a very big token, just like below:
Bob: 企鹅会飞吗
Alice: 企鹅是不会飞的。企鹅的翅膀短而扁平，更像是游泳时的一对桨。企鹅的身体结构和羽毛密度也更适合在水中游泳，而不是飞行。

Bob: hi

 len(tokens), tokens[]: 8 [26845, 27, 14260, 187, 187, 2422, 547, 27]
Alice:
 token, out: -9223372036854775808 tensor([ -5.4473, -25.4831,  -6.7508,  ...,  -7.4079,  -6.1975,  -4.1842],
       device='mps:0')

 len(tokens), tokens[]: 1 [-9223372036854775808]
Traceback (most recent call last):
  File "/Users/ppt/Github/ChatRWKV/v2/chat.py", line 474, in <module>
    on_message(msg)
  File "/Users/ppt/Github/ChatRWKV/v2/chat.py", line 387, in on_message
    out = run_rnn([token], newline_adj=newline_adj)
  File "/Users/ppt/Github/ChatRWKV/v2/chat.py", line 163, in run_rnn
    out, model_state = model.forward(tokens[:CHUNK_LEN], model_state)
  File "/Users/ppt/Github/ChatRWKV/v2/../rwkv_pip_package/src/rwkv/model.py", line 563, in forward
    x = w['emb.weight'][tokens if seq_mode else tokens[0]]
RuntimeError: Expected !is_symbolic() to be true, but got false.  (Could this error message be improved?  If so, please report an enhancement request to PyTorch.)
(ChatRWKV) ppt@pptdeMacBook-Pro v2 % python chat.py

ChatRWKV v2 https://github.com/BlinkDL/ChatRWKV

Chinese - mps:0 fp32 - /Users/ppt/Github/ChatRWKV/v2/prompt/default/Chinese-2.py
Loading model - ./fsx/BlinkDL/HF-MODEL/rwkv-4-pile-1b5/RWKV-4-Pile-1B5-EngChn-testNovel-done-ctx2048-20230225
Traceback (most recent call last):
  File "/Users/ppt/Github/ChatRWKV/v2/chat.py", line 133, in <module>
    model = RWKV(model=args.MODEL_NAME, strategy=args.strategy)
  File "/Users/ppt/miniconda/envs/ChatRWKV/lib/python3.10/site-packages/torch/jit/_script.py", line 292, in init_then_script
    original_init(self, *args, **kwargs)
  File "/Users/ppt/Github/ChatRWKV/v2/../rwkv_pip_package/src/rwkv/model.py", line 84, in __init__
    raise ValueError("Invalid strategy. Please read https://pypi.org/project/rwkv/")
ValueError: Invalid strategy. Please read https://pypi.org/project/rwkv/
(ChatRWKV) ppt@pptdeMacBook-Pro v2 %


Can't see BCOs

Describe the bug
A clear and concise description of what the bug is.
Objects are not listing.
To Reproduce
Steps to reproduce the behavior:
Click on BioCompute Objects in top menu
Expected behavior
A clear and concise description of what you expected to happen.
A listing of BCOs
Screenshots

Desktop (please complete the following information):

OS:  Windows10

Browser  Tried in Brave, Edge, Firefox, Chrome...

This is not a bug, just needs documentation.

Duplicate of #134