texi2html: Fix cross-platform build

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build texi2html
@GrahamcOfBorg build pkgsCross.aarch64-multiplatfrom.texi2html

maybe you can add strictDeps = true to make regressions less likely?
This applies to many of your cross-fixing PRs.

@GrahamcOfBorg build texi2html
@GrahamcOfBorg build pkgsCross.aarch64-multiplatform.texi2html

maybe you can add strictDeps = true to make regressions less likely?
This applies to many of your cross-fixing PRs.

Done (other will follow).

Please target master. This is not a mass rebuild.

Please target master. This is not a mass rebuild.

@SuperSandro2000 can you please for future reference clarify what constitutes as a mass rebuild and ideally have that documented clearly (as in 'objectively') in the staging branch section of the nixpkgs manual? I have asked that question multiple times and in different spaces and have never gotten a definitive answer. Thanks.

Please target master. This is not a mass rebuild.

@SuperSandro2000 can you please for future reference clarify what constitutes as a mass rebuild and ideally have that documented clearly (as in 'objectively') in the staging branch section of the nixpkgs manual? I have asked that question multiple times and in different spaces and have never gotten a definitive answer. Thanks.

500+ plus is a mass rebuild.

Please target master. This is not a mass rebuild.

@SuperSandro2000 can you please for future reference clarify what constitutes as a mass rebuild and ideally have that documented clearly (as in 'objectively') in the staging branch section of the nixpkgs manual? I have asked that question multiple times and in different spaces and have never gotten a definitive answer. Thanks.

500+ plus is a mass rebuild.

Anyone knows why @GrahamcOfBorg requested my review? Maybe some recent history (the mindustry updates in particular, since there's @petabyteboy too) got briefly included while switching branch and force pushing?

Anyone knows why @GrahamcOfBorg requested my review? Maybe some recent history (the mindustry updates in particular, since there's @petabyteboy too) got briefly included while switching branch and force pushing?

Result of nixpkgs-review pr 97093 run on x86_64-darwin 1

4 packages marked as broken and skipped:

btrbk
frescobaldi
lilypond-unstable
udiskie

11 packages built:

asciidoc-full
asciidoc-full-with-plugins
ccache
ccacheStdenv
knightos-genkfs
knightos-kimg
knightos-mktiupgrade
lilypond
lilypond-with-fonts
rep
texi2html

Result of nixpkgs-review pr 97093 run on x86_64-darwin 1

4 packages marked as broken and skipped:

btrbk
frescobaldi
lilypond-unstable
udiskie

11 packages built:

asciidoc-full
asciidoc-full-with-plugins
ccache
ccacheStdenv
knightos-genkfs
knightos-kimg
knightos-mktiupgrade
lilypond
lilypond-with-fonts
rep
texi2html

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 97093 run on x86_64-linux 1

2 packages failed to build and are new build failures:

spring: log https://termbin.com/jxy1
springLobby: log was empty

35 packages built:

adapta-gtk-theme
asciidoc-full
asciidoc-full-with-plugins
btrbk
ccache
ccacheStdenv
ccacheWrapper
chrome-gnome-shell
clevis
denemo
disorderfs
frescobaldi
gnome3.gnome-session (gnome3.gnome_session)
gnome3.gnome-shell (gnome3.gnome_shell)
gnome3.gnome-terminal (gnome3.gnome_terminal)
gnome3.gnome-tweaks (gnome3.gnome-tweak-tool)
gnome3.pomodoro
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
herbstluftwm
knightos-genkfs
knightos-kimg
knightos-mkrom
knightos-mktiupgrade
lilypond
lilypond-unstable
lilypond-with-fonts
luksmeta
pantheon.elementary-session-settings
rep
solfege
tang
texi2html
udiskie

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 97093 run on x86_64-linux 1

2 packages failed to build and are new build failures:

spring: log https://termbin.com/jxy1
springLobby: log was empty

35 packages built:

adapta-gtk-theme
asciidoc-full
asciidoc-full-with-plugins
btrbk
ccache
ccacheStdenv
ccacheWrapper
chrome-gnome-shell
clevis
denemo
disorderfs
frescobaldi
gnome3.gnome-session (gnome3.gnome_session)
gnome3.gnome-shell (gnome3.gnome_shell)
gnome3.gnome-terminal (gnome3.gnome_terminal)
gnome3.gnome-tweaks (gnome3.gnome-tweak-tool)
gnome3.pomodoro
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
herbstluftwm
knightos-genkfs
knightos-kimg
knightos-mkrom
knightos-mktiupgrade
lilypond
lilypond-unstable
lilypond-with-fonts
luksmeta
pantheon.elementary-session-settings
rep
solfege
tang
texi2html
udiskie

Anyone knows why @GrahamcOfBorg requested my review? Maybe some recent history (the mindustry updates in particular, since there's @petabyteboy too) got briefly included while switching branch and force pushing?

Yes, one commit that was pushed to the branch, https://github.com/NixOS/nixpkgs/commits/0fd3afc1b2fb053d6b6f67db659b0a236d60cbf8, also contains a mindustry change, but it's not included in the diff anymore.

Anyone knows why @GrahamcOfBorg requested my review? Maybe some recent history (the mindustry updates in particular, since there's @petabyteboy too) got briefly included while switching branch and force pushing?

Yes, one commit that was pushed to the branch, https://github.com/NixOS/nixpkgs/commits/0fd3afc1b2fb053d6b6f67db659b0a236d60cbf8, also contains a mindustry change, but it's not included in the diff anymore.


Redesign the blog

The current grid used for the listing of the post is just too damn large. Font sizes are freaking scary.
The redesign should fix all of this including a revamped look. The index page isn't bad and can be left as is. But I'd be inclined to scrape out the contact form anyways

This has been fixed in #4


WumpusKnowledgeBase

There is a little error inside tellTemporalPhysicsSentences method line 344
    for (int x = 1; x <= caveXDimension; x++) {
        for (int y = 1; y <= caveYDimension; y++) {
            tellSuccessorStateLocationAxiom(t, x, y);
            // Optimization to make questions about unvisited locations faster
            tell(new ComplexSentence(
                    newSymbol(LOCATION, t + 1, x, y),
                    Connective.IMPLICATION,
                    newSymbol(LOCATION_VISITED, x, y)));
        }
        tellSuccessorStateOrientationAxioms(t);
    }

tellSuccessorStateOrientationAxioms(t) is inside de loop but it not necessary. Anyway the sentences are added one time in the KB ...

Should be fixed now. Thank you!


clear all blocklist with fail2ban

Is there a way to release all everyone in the blocklist?
this command   Rack::Attack.clear! is not releasing  the blocklist.
I want to release everyone from blocklist

Were you able to figure this one out?

Rack::Attack.clear! does not release the users in the blocklist.
Rather, it clears the the configuration for throttle, blacklist, safelist and track.
Although I am not very sure how to release everyone from the blocklist, maybe clear from the cache?

I created a hack to solve it.
I used dynamic prefix for the key and when I wanted to clear  I just change the prefix key

Similar issue also discussed here: https://github.com/kickstarter/rack-attack/issues/113
email =<EMAIL_ADDRESS>epoch_time = Time.now.to_i
Rack::Attack.cache.store.delete "#{Rack::Attack.cache.prefix}:#{epoch_time / 30.minutes}:logins/email:#{email}"

We don't want specific, I want to clear all the blacklist

Rack::Attack.clear! was misleading and was changed to Rack::Attack.clear_configuration in https://github.com/kickstarter/rack-attack/pull/344.

FWIW we're considering adding a Rack::Attack.reset! method to reset Rack::Attack's state in other issues (https://github.com/kickstarter/rack-attack/issues/389, https://github.com/kickstarter/rack-attack/issues/249#issuecomment-539029878).

adding it?

Rack::Attack.reset! was released in v6.3.0. Be aware it resets every Rack::Attack state, no only fail2ban.

When you say Rack::Attack state, are you talking about the cache and/or the rules?
In case, Is there a way to reset the rules? I'm doing some debugging on a staging console and I'm still searching for a way of redefining rules :)


Would be a good idea to check whether the user has generated the apis

Fix https://github.com/mpociot/laravel-apidoc-generator/pull/789


No visible way to set endpoint

Hi, I installed v1.4.4 of the plugin to edit a graphql request in an IntelliJ scratch buffer.
I created a new scratch buffer and set the language to GraphQL. Now I can edit with syntax checking, but there is no schema set and I can not set/select an endpoint URL.
The tool windows title bar with "Edit the GraphQL endpoints in graphql.config.json" etc is greyed out.
Maybe I need to create a graphql.config.json, but I don't know where or what to put into it. I cannot find any way to create one by using the plugin.
So: there should be a way to specify an endpoint after installing the plugin so that the plugin can fetch the schema and issue requests.

Hi Dirk.
Thanks for using the plugin.
It might be the case that a scratch file doesn't trigger the editor notification that should help you create the graphql.config.json. Here's an excerpt from the FAQ in the README.md:

How do I configure the plugin in a project
The plugin is activated as soon as you view or edit GraphQL in the editor. This includes GraphQL inside Relay.QL and gql templates in JavaScript and TypeScript. You can also use .graphql physical files and scratch files. These files allow you to query your endpoint directly from your IDE.
An editor notification bar should prompt you to "Create a graphql.config.json". Accept and edit this config file to point the plugin at your local schema.json or your endpoint for introspection. The plugin uses the schema to provide completion and error highlighting.

I suggest you try and create a physical .graphql file or use "Relay.QL" or "gql" tagged templates in .js files.
Let me know how it goes.
/Jim

Hi Jim,
thanks for the reply!
I managed to get the editor title bar prompt to create a graphql.config.json by creating a regular graphql file. I then could customize the config to my needs. Thanks!
It seems you're right, creating a scratch file or scratch buffer and switching its language to graphql doesn't trigger this prompt.
While experimenting, I deleted the graphql.config.json again, but then did not get the editor prompt even when creating a new .graphql file. Only after restarting the IDE would again give me this prompt.
I think it would be a good idea to be able to trigger the graphql.config.json creation by a user action (button or menu).
Also, a UI element to trigger re-loading the schema would be great. I had an error in my schema (which apparently is undetected by graphiql!) To reload the modified schema, I modified the graphql.config.json file by inserting a blank somewhere and saving it. That's a bit awkward.
Otherwise, so far as I can judge from the short usage now, you plugin seems great! Thank you!

Hi Dirk.
Thanks for the update.
You can reload schema as outlined in the FAQ:

In the the GraphQL tool window, select the "Current Errors" tab and click the "Restart JS GraphQL Language Service" button.

I'll see if the scratch file detection and action to create the config can find it's way into a future release.
/Jim

Ah. I did read the readme, but somehow missed that last FAQ to reload the schema. Thanks again!

Hi Dirk.
The 2.0 release features a GraphQL Schemas tool panel that allows you re-run introspection queries to update your schema. See https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/releases/tag/2.0.0-beta-2
Let me know if you have any issues.
Best regards,
Jim.


update gradle wraper from 2.2.0-alpha5 to 2.2.3

fix #27
Updated the build.gradle and gradle-wrapper.properties files for each project to meet android studios minimum requirements. I'm working through the course now using these versions and will inform if there are any incompatibilities.

Awesome! This was on my to do list. Thanks for contributing!

:D Glad to help.


More unit testing

For read_* functions and spectra processing.
TODOs
manage_lib and match_spectrum() still need testing routines.
@wincowgerDEV Could you maybe produce a set of 100 random Raman spectra for testing purposes and add them to OSF as test_library.rds and test_metadata.rds?

@wincowgerDEV Do you have an idea why read_spa() keeps failing on Windows? I don't have a Windows machine at hand to check and debug.
I guess, the problem is somewhere here:
https://github.com/wincowgerDEV/OpenSpecy/blob/1feb5a0fbbb8c7e8039e5ad7c53b1067e5e1220c/R/read_spectra.R#L50-L57
Is mgn empty?

I commented the test out for now that checks pass but we shouldn't merge until this is fixed.

For read_* functions and spectra processing.
TODOs
manage_lib and match_spectrum() still need testing routines.
@wincowgerDEV Could you maybe produce a set of 100 random Raman spectra for testing purposes and add them to OSF as test_library.rds and test_metadata.rds?

I just added "augmented_ftir.rds" and "augmented_raman.rds" to the osf repo.

Basically what these are is an augmented version of the libraries. I just added random noise and baseline to the spectra in there so you can merge the sample_name to the metadata files and get all their metadata.
New variable explanations:
intensity_og is the original intensity in the library spectra just normalized.
intensity_aug1 is the original intensity with noise added.
intensity_aug2 is the original intensity with noise and baseline sift added.

I commented the test out for now that checks pass but we shouldn't merge until this is fixed.

I am testing it out on my device.
The original read_spa function from the open specy website works on my device, but this new formatted function doesn't work. Did the original function work on your device? Was there a reason you made the changes you did?

I just added "augmented_ftir.rds" and "augmented_raman.rds" to the osf repo.

Ah no, .. seems like we talked past each other :see_no_evil: sorry!
My idea behind the testdata was not to have random/augmented spectra but to have a smaller subset of the current reference library (max. 100 spectra randomly picked from the large one) to run unit tests on them. This is because downloading the complete library just to test if e.g. get_lib() or match_spectrum() works as expected will just take too long and uses up traffic.
Hope producing the augmented data set didn't take too much of your time.


Fix double spaces in IOU confirmation message

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/14737
Tests

Logged in from Account A, request money from Account B.
Pay the money request using either elsewhere, PayPal, or Wallet.
Verify there are no double spaces before the payment method, e.g. Settled up elsewhere

[ ] Verify that no errors appear in the JS console

Offline tests

Same as above.
QA Steps

Logged in from Account A, request money from Account B.
Pay the money request using either elsewhere, PayPal, or Wallet.
Verify there are no double spaces before the payment method, e.g. Settled up elsewhere

[ ] 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] 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/9680864/216427361-0951d559-59a2-470c-9f6b-8da638d6ab4f.mov

Mobile Web - Chrome

See recording above, this should work on all platform as we're just changing text.

Mobile Web - Safari

See recording above, this should work on all platform as we're just changing text.

Desktop

See recording above, this should work on all platform as we're just changing text.

iOS

See recording above, this should work on all platform as we're just changing text.

Android

See recording above, this should work on all platform as we're just changing text.

Removing C+ as this is a pretty simple change 😅

This works, but do we want to take this opportunity to translate paymentMethodMessage which isn't currently translated?

Good point, but, the final IOU confirmation message actually comes from the backend, and replaces the optimistic one. Hence if we translate the optimistic message, it will be overwritten by the message coming from the backend. And this also answers your question below:

Separately, on main when I do this, it appears with the double space first, but then if I move to another view and back (e.g. clicking on the payment details) it then fixes itself and removes the double space. It doesn't seem to mess up this PR, so it's probably fine, but did you experience that and/or do you know why it's happening?

The message coming from the backend doesn't have double spaces. Check this function here responsible for creating the IOU confirmation message.

Oh, okay - that makes sense. I didn't realize it was coming from two sources and this was just the optimistic one. All good then!
(I can make a separate issue/bug report to translate that text. I wonder if we did it only in English since the grammar might be different in Spanish and so the string replacement won't work.)

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.
[ ] 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

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/dangrous in version: 1.2.64-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.2.64-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.2.64-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.2.64-7 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Exception when using custom layout

I'm new to using react and I can't solve this issue. I've tried everything but nothing seems to work. When I use the Liform element I can render forms easely but when I try to render the fields using the renderField method I get the following error:
Error: Field must be inside a component decorated with reduxForm()
    at new Field (createField.js:39)
    at ReactCompositeComponent.js:294
    at measureLifeCyclePerf (ReactCompositeComponent.js:75)
    at ReactCompositeComponentWrapper._constructComponentWithoutOwner (ReactCompositeComponent.js:293)
    at ReactCompositeComponentWrapper._constructComponent (ReactCompositeComponent.js:279)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:187)
    at Object.mountComponent (ReactReconciler.js:45)
    at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:370)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:257)
    at Object.mountComponent (ReactReconciler.js:45)

This is my form in json:
{
  "schema": {
    "title": "stage",
    "type": "object",
    "properties": {
      "name": {
        "type": "string",
        "title": "name",
        "propertyOrder": 1
      }
    },
    "required": [
      "name"
    ]
  }
}

And this is my component:
import React from 'react'
import {connect} from 'react-redux'
import {fetchForm} from "../../actions/stage"
import {DefaultTheme, renderField} from 'liform-react'

class Create extends React.Component {
    constructor(props, context) {
        super(props, context);
    }

    componentDidMount() {
        this.props.dispatch(fetchForm(this.props.baseUrl));
    }

    render() {
        const {schema, fetching} = this.props;
        if (!schema || fetching) {
            return <div className="loader loader-primary"/>;
        }

        return (
            <form onSubmit={console.log}>
                {renderField(schema, null, DefaultTheme)}
                <button className="btn btn-primary" type="submit">Submit</button>
            </form>
        )
    }
}

export default connect((state) => ({
    baseUrl: state.appState.baseUrl,
    schema: state.appState.form.stage,
    fetching: state.appState.form.stageFetching
}))(Create)

If there is anyone that can help me with this issue I would appreciate it

Sorry, My fault, I forgot to read the part where you inserted the form into the Liform element. Closing this issue.


st_numberOfCharactersInATweet counting Emoji's as 2 characters while Twitter accepts as 1

Hi,
When including Emoji's in the tweet text the st_numberOfCharactersInATweet method is counting these as 2 characters while Twitter accepts them as 1 character.
As a temporary fix i have modified -

(NSInteger)st_numberOfCharactersInATweetWithShortURLLength:(NSUInteger)shortURLLength shortURLLengthHTTPS:(NSUInteger)shortURLLengthHTTPS

as follows -
//    NSInteger count = [s length];
__block NSInteger count = 0;
[s enumerateSubstringsInRange:NSMakeRange(0, s.length) options:NSStringEnumerationByComposedCharacterSequences usingBlock:^(NSString *subString, NSRange subStringRange, NSRange enclosingRange, BOOL *stop) {
count++;
}];
This seems to give a correct count for Twitter but would be interested to get your take on this as I am not overly familiar with the Twitter API.
Thanks.
/jh

You are perfectly right, thank you for spotting this error.


aws-mfa: init at v0.0.12

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

odd they don't do an sdist

@jonringer who is they? What's an sdist ?

@jonringer done,
Question, do you know why after I applied the changes you requested the following command to install the package stopped working:
nix-env -f pkgs/top-level/all-packages.nix -i aws-mfa
I have to use:
nix-env -f . -iA aws-mfa
The first example would otherwise give me the error:
error: cannot auto-call a function that has an argument without a default value ('lib')
This is actually why I didn't do it the way you suggested in the first place.
`

if you want to test a build, you shouldn't use nix-env, but rahter nix-build, and just do nix-build -A aws-mfa

@GrahamcOfBorg eval

@jonringer yeah sorry. I understand that but I was actually trying to install it as I wanted to use the package now.

@jonringer who is they?

they is upstream, the ones who created aws-mfa

What's an sdist ?

short for "source distribution", its one of the ways you can publish a package. A .whl file is a bdist_wheel format

If you need it locally for packaging another package, you can use an overlay https://nixos.wiki/wiki/Overlays
If you just need an environment where you can call it, you could idiomatic python method of using a venv to install and then use it within the venv. https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/python.section.md#how-to-consume-python-modules-using-pip-in-a-virtual-environment-like-i-am-used-to-on-other-operating-systems

@jonringer closing as it seems this is not wanted. Thanks.

i would use this.


AndroidTV recognized at Chromecast causing warn-logs

I didn't know how to most precisley describe this. I got a Sony Android TV with built in Chromecast and the adapter seems to be unable to connect to and keeps throwing errors. Multiple per second.
Is there any possibility I can exclude this device from the list?
ALso got a "normal" Chromecast which I don't want to loose in ioBroker by uninatalling the adaptor.
Here's an excerpt from the log:

`chromecast.0
2018-03-28 23:15:30.797
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:26.781
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:23.766
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:21.753
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:20.737
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:20.695
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:20.443
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:20.374
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:19.522
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:19.497
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:19.445
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:18.906
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:18.879
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:17.921
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:16.910
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:16.904
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:16.902
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:15.970
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:15.878
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:15.770
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:14.920
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:14.881
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:13.947
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:13.918
info
Wohnzimmer - client disconnected

chromecast.0
2018-03-28 23:15:13.677
info
Wohnzimmer - Connected client

chromecast.0
2018-03-28 23:15:11.667
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.

chromecast.0
2018-03-28 23:15:07.654
warn
Wohnzimmer - Client error: Error: read ECONNRESET at exports._errnoException (util.js:1020:11) at TLSWrap.onread (net.js:580:26)

chromecast.0
2018-03-28 23:15:04.637
warn
Wohnzimmer - Client error: Error: socket hang up at TLSSocket.onHangUp (_tls_wrap.js:1124:19) at TLSSocket.g (events.js:292:16) at emitNone (events.js:91:20) at TLSSocket.emit (events.`

I can add a "disabled" state to each Chromecast so that if it is set to true the adaptor does not try to connect.
Until I find a few minutes to code and test it you might want to disable the adapter logs as circumvention.

That would be absolutely great.
Vegetto<EMAIL_ADDRESS>schrieb am Do., 29. März 2018, 06:18:

I can add a "disabled" state to each Chromecast so that if it is set to
true the adaptor does not try to connect.
Until I find a few minutes to code and test it you might want to disable
the adapter logs as circumvention.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angelnu/ioBroker.chromecast/issues/33#issuecomment-377116619,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFF_GpvoXE-Py47oeP_0vldX1nabOcdnks5tjGCIgaJpZM4S_YO2
.

@Haileid - please try version 1.4.3. It is only in git -> I will push to npm after hearing from you.
In your case you need to set Wohnzimmer.enabled to false

Thanks for your effort. Will try and report back. I also played around
yesterday and figured out ths real problem. I had a TV and Google home with
the same name interferrring. After giving unique names the initial problem
also seems to be solved.
Vegetto<EMAIL_ADDRESS>schrieb am Di., 3. Apr. 2018, 02:26:

@Haileid https://github.com/Haileid - please try version 1.4.3. It is
only in git -> I will push to npm after hearing from you.
In your case you need to set Wohnzimmer.enabled to false
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/angelnu/ioBroker.chromecast/issues/33#issuecomment-378089516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFF_GgLrdB-RD3iJtjTH23nGwo4dVBoAks5tksG_gaJpZM4S_YO2
.


updated container setup.

fixed urls
upgraded to jdk17 and latest lib versions
better signal handling (made sure container stop signal propagates to tomcat)

i leave this as draft until we have a tag for 6.1, so i can change the git checkout line.

@snoopdave could we get a roller-6.1.0 tag for the recent release?

@mbien I pushed a roller-6.1.0 tag today and cleaned up all the old RC tags.

perfect. I changed the containerfile to checkout tags/roller-6.1.0 instead of master.


feat: ignore built-in workloads during generating the Spec

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 54.957%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/engine/api/generate/generator/generator.go
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
8891

Relevant Lines:
16178

💛  - Coveralls


This page isn’t working!!>>

Once I had completed my HTML I clicked on Go Live but my Chrome it's showing this page isn't working
please help me out

I believe the developer unfortunately died. the last commit for his previous work (live server) was 2 years ago.
If he's not dead, he would've atleast telling people that he need someone to maintained it.


fix('FindingsAPI'): Latest generation using openapi-sdkgen

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


release: 0.13.0

There are quite many deprecations that should go from warning to error and error to removal.

This has already been done for this release.

And, it would be nice to use the new APIs in the sample app aswell, meaning in the :conference module.

There's really only DegradationPreference API, which is somewhat niche. We can cover that later if there's a demand. ReferConferenceEvent is implemented.

It is mostly related to the Messenger API.
w: file:///Users/tpett/dev/pexip-android-sdk/sample/src/main/kotlin/com/pexip/sdk/sample/conference/ConferenceAction.kt:224:58 'message(String): Unit' is deprecated. Use Conference.messenger.send() instead.
w: file:///Users/tpett/dev/pexip-android-sdk/sample/src/main/kotlin/com/pexip/sdk/sample/conference/ConferenceEventsScreen.kt:43:33 'MessageReceivedConferenceEvent' is deprecated. Use Messenger to receive messages instead.
w: file:///Users/tpett/dev/pexip-android-sdk/sample/src/main/kotlin/com/pexip/sdk/sample/conference/ConferenceEventsScreen.kt:103:12 'MessageReceivedConferenceEvent' is deprecated. Use Messenger to receive messages instead.
w: file:///Users/tpett/dev/pexip-android-sdk/sample/src/main/kotlin/com/pexip/sdk/sample/conference/ConferenceEventsScreen.kt:121:22 'MessageReceivedConferenceEvent' is deprecated. Use Messenger to receive messages instead.
w: file:///Users/tpett/dev/pexip-android-sdk/sample/src/main/kotlin/com/pexip/sdk/sample/conference/ConferenceService.kt:54:9 'stopForeground(Boolean): Unit' is deprecated. Deprecated in Java

In Conference, InfinityConference and ConferenceEvents the deprecations for messenger was last adjusted 28th of february.

In Conference, InfinityConference and ConferenceEvents the deprecations for messenger was last adjusted 28th of february.

Everything that was planned for this release was already changed.


Add release workflow

https://github.com/snowdrop/narayana-spring-boot/commit/b7e3752fe2d8e905592eadf7a514659edb2039bc


Fever API password should be hashed in the database

Currently all passwords for Integrations are stored as cleartext in the database. For most of them, this is necessary: Instapaper's "simple" API doesn't have token auth, while Wallabag's API does have token auth but they've implemented OAuth incredibly badly and so you still need a password anyway. It's ridiculous.
However, Miniflux acts as a Fever server when the Fever API compatibility is active, not a client. This means it doesn't need access to a cleartext version of the Fever password - a hash would suffice for verifying the password when requests are made to the Fever API. So this password in particular should be hashed.

Fever uses token which is md5 of username and password, and it's already stored in the database.
I made a PR to remove password column from database.

PR #824 has been merged.


versions: Update Clear Containers image

version: 18800

Changes in package clear-containers-agent (from
1fa147836736824c32a46889d6fb59402d4e58bd-14 to
9adc9d49378aa0a19b85d02c447b3eb1e2b87774-14):

Jose Carlos Venegas Munoz - new agent version 9adc9d

https://download.clearlinux.org/releases/18800/clear/RELEASENOTES

version: 19010

Changes in package systemd (from 234-153 to 234-154):
Miguel Bernal Marin - version bump from 234-153 to 234-154
Victor Rodriguez - Fix CVE-2017-15908

https://download.clearlinux.org/releases/19010/clear/RELEASENOTES
version: 19050

Changes in package systemd (from 234-154 to 234-157):

Arjan van de Ven - don't do transient hostnames; we set ours already
Arjan van de Ven - move vconsole to the console subpackage

https://download.clearlinux.org/releases/19050/clear/RELEASENOTES

version: 19060

Changes in package clear-containers-agent (from 9adc9d49378aa0a19b85d02c447b3eb1e2b87774-14 to
243e2aefa4f9ff5a1bd32967a213e8533dab54df-15):

Jose Carlos Venegas Munoz - version bump from 243e2aefa4f9ff5a1bd32967a213e8533dab54df-14 to
243e2aefa4f9ff5a1bd32967a213e8533dab54df-15
Jose Carlos Venegas Munoz - New agent version 243e2ae

Changes in package systemd (from 234-154 to 234-157):
-Arjan van de Ven - don't do transient hostnames;
we set ours already Arjan van de Ven - move vconsole to the console subpackage

https://download.clearlinux.org/releases/19060/clear/RELEASENOTES

Fixes: #708
Signed-off-by: Jose Carlos Venegas Munoz<EMAIL_ADDRESS>

lgtm


Update vagrant guide VirtualBox version

Vagrant does not yet support VirtualBox 5, so the previously-linked page will not work.

I signed it!

LGTM


ability to dragdrop components without selecting them

dragula can differentiate between regular clicks and dragstarts, so we should take advantage of that and allow users to drag around components without having to select them first.
Problems:

how would users highlight text in a component?
how do we handle components that contain component-lists?

done in #448


Question about ConfigurationDbContext migrations from 1.0.1 to 2.0.0

I have an IdentityServer4 instance using IdentityServer4.EntityFramework version 1.0.1 with migrations generated for the ConfigurationDbContext.
I'm in the process of upgrading to IdentityServer4 2.0.2 and IdentityServer4.EntityFramework version 2.0.0 and I'm trying to do it like so...

Upgrade nuget packages
Add a new migration against the ConfigurationDbContext from the new nuget package version

This works fine for the PersistedGrantDbContext (few or no changes), but for the ConfigurationDbContext I get an error when attempting to apply the migrations:
Cannot change column 'Id': used in a foreign key constraint 'FK_IdentityClaims_IdentityResources_IdentityResourceId'
Is there a better method for such an upgrade?

I think it's possible that I'm seeing a bug from Pomelo MySQL provider like this: https://github.com/PomeloFoundation/Pomelo.EntityFrameworkCore.MySql/issues/362

All set on this issue -- can we close?

I'm also seeing this issue at the moment, did you find a reasonable workaround @dls314 ?

I never worked through caleblloyd's recommendation over in #423 about adding code manually to the migration, but I'm sure it's an answer.

Thanks @dls314 I'll give that a go 👍


suggested interface

brief proposal for interface based on call i just had with anraj:

add a "upload file" command to discord, to upload a python file (this is not instead of git)
add a "run file". response is sent back on discord as an ascii message
add a kill file" command, to stop the command
each rover has its own tag. so "$run iamz1 raise_your_leg PARAM1 PARAM2..."
we can add a "git xxx" which will pull from the git repository into a directory with same name.
all files loaded into directories with the username of the discord user (or something)
video feedback we still need to solve

update on proposal:
each python code that can run starts  command of the format rover= Rover(camera='R',legs='W',time=30)
means - want to get a rover object and i need rights of R for camera and W of legs. and i want 30 minutes. this information is put  in a common table.
each bot has a different model which si served locallyt for example, for spiderPI it includes camera, servos, legs and gaits. so:
the call to Rover gives you a submodel of this model, the  part you have access to.
the command rover.leg1.tipto((x,y,z)). will check if leg1 is part of the model and if you have the  right to do "tipto()" and if yes, move the tip of the leg to that position.
video stream (when we decide how to do, maybe twitch?) will also be such a resource, which disconnects when the time allocated is done.
you can also have complex rights like maximum power, maximum speed of a particular serveo, etc. so not only RW
Output is in log file put on discord
You also get a command to send back messages

two more items:

watchdogs. can be uploaded. they are able to stop any prog from interacting with rover parts. not clear what is best way to do it. a message to the Rover object might do it.
communication. we will have a text-type channel between anything (also rovers) and rovers. one example can be to have a flask server on vultr and use webhook to communicate. resulting IP may be used for direct communication maybe.

status now is that we have general uploads and also "cmd" uploads which are shared


Variation tests generate incorrect JSON path to array items

Portman versions tested: 1.16.1 + 1.18.0
One of the API operations I'm testing defines a request body that is an object, with a property that is an array of strings. The strings have a maxLength, and the server should return a 400 if that length is exceeded. Below is an example open API spec that will reproduce:
openapi: 3.0.1
info:
  title: Test Spec
  description: Test
  contact:
    email<EMAIL_ADDRESS>  license:
    name: Fake Software License
    url: 'https://api.fake.com/LICENSE'
  version: '0.1'
servers:
  - url: 'https://api.sandbox.fake.com'
tags:
  - name: Test
    description: Test
security:
  - API-ID: []
    API-Secret: []
paths:
  /foo:
    post:
      tags:
        - Test
      operationId: testOperation
      summary: Test operation
      description: |
        Test
      requestBody:
        required: true
        content:
          application/json:
            schema:
              description: Test request body
              type: object
              properties:
                testArray:
                  description: |
                    A test array of strings
                  type: array
                  items:
                    type: string
                    pattern: '[a-zA-Z0-9\-]+'
                    maxLength: 16
                    minLength: 1
                  example:
                    - foo
                    - bar
                testString:
                  description: |
                    A test string
                  type: string
                  maxLength: 16
                  minLength: 1
              required:
                - testArray
                - testString
              example:
                testString: foo
                testArray:
                  - foo
                  - bar
      responses:
        '400':
          description: Bad request
          type: object
          properties:
            error:
              example:
                code: 400
                message: Bad request
              description: Error details
              type: object
              properties:
                message:
                  type: string
                  description: Error message
                code:
                  type: string
                  description: Error code
              required:
                - message
                - code

So from this example, I would expect a maxLengthFields variation test to target testArray[0], and generate a request body for the test that looks something like this:
{
  "testString": "foo",
  "testArray": [
    "fooffffffffffffff",
    "bar"
  ]
}

Instead, the test targets testArray[0].items and generates a request body that looks like this:
{
  "testString": "foo",
  "testArray": [
    "foo",
    "bar"
  ]
}

The same happens with minLengthFields tests.
Tests that target string properties within objects work correctly. So in my example above, the min and max length tests that target testString work as expected.

@torque-smackey
Could you share you Portman configuration, with the variation tests settings?
This will allow us to reproduce the issue.

hi @torque-smackey
Any chance, that you could share your Portman config, with the variation tests settings?
We want to be able to investigate the reported issue.

Closing due to inactivity.
Since there hasn't been any feedback for the past months, I'll close the item. If you're still experiencing problems, please feel free to reopen this issue or create a new one.


MaskZeroCriterion:recursiveMask fails when mask is empty

My model and criterion are as follows:
model = nn.Sequential()
model:add(nn.Convert(ds:ioShapes(), 'bwh'))
model:add(nn.SplitTable(1, 2))

model:add(
  nn.Sequencer(
    nn.MaskZero(
      nn.Sequential()
        :add(nn.FastLSTM(inputSize, hiddenSize))
        :add(nn.Dropout(0.2))
        :add(nn.Linear(hiddenSize, outputSize))
        :add(nn.LogSoftMax())
    , 1)
  )
)

criterion = nn.ModuleCriterion(
  nn.SequencerCriterion(
    nn.MaskZeroCriterion(
      nn.ClassNLLCriterion(), 1
    )
  ),
  nn.Identity(),
  nn.Sequential()
    :add(nn.Convert())
    :add(nn.SplitTable(1, 1))
)

My dataset is like:
local samples = 10
local maxLen = 20
inputs = torch.Tensor(samples, maxLen, inputSize)
targets = torch.LongTensor(samples, maxLen)

Inputs are padded by zeros in the very beginning of each sequence.
When learning error occurs:
/home/evgeny/torch/install/bin/luajit: ...ny/torch/install/share/lua/5.1/rnn/MaskZeroCriterion.lua:34: bad argument #1 to 'index' (sizes do not match at /home/evgeny/torch/extra/cutorch/lib/THC/generic/THCTensorCopy.c:47)
stack traceback:
	[C]: in function 'index'
	...ny/torch/install/share/lua/5.1/rnn/MaskZeroCriterion.lua:34: in function 'recursiveMask'
	...ny/torch/install/share/lua/5.1/rnn/MaskZeroCriterion.lua:76: in function 'forward'
	...y/torch/install/share/lua/5.1/rnn/SequencerCriterion.lua:39: in function 'forward'
	...eny/torch/install/share/lua/5.1/dpnn/ModuleCriterion.lua:23: in function 'forward'
	...torch/install/share/lua/5.1/dp/propagator/propagator.lua:168: in function 'forward'
	.../torch/install/share/lua/5.1/dp/propagator/optimizer.lua:50: in function 'propagateBatch'
	...torch/install/share/lua/5.1/dp/propagator/propagator.lua:127: in function 'propagateEpoch'
	...torch/install/share/lua/5.1/dp/propagator/experiment.lua:110: in function 'run'
	conf_rnn.lua:246: in main chunk
	[C]: in function 'dofile'
	...geny/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
	[C]: at 0x00406670

As far as I understood this error caused by empty mask as every sequence in batch is started with zeros.
My questions are:
Is my model or criterion used in right way?
How can I avoid empty mask using MaskZeroCriterion?

@notimesea This should be fixed by https://github.com/Element-Research/rnn/commit/4c4895a2da501ac2b6c66eec4d174ca12c3f5a1d . Let me know if this works for you. Thanks!


Inability to Stake The Collateral if its older than 25days

As the tittle suggests you cant stake the collateral if its older than 25days
I have tried everything.I shut off the vpss, the FSs are not on the list,i waited for a day staking
but still nothing.
There seems no problem staking the collateral if its newer than 20
it stakes immediately without having to wait for the FSs to go out of the list which suggests there is definitely a bug.The bug exists a long time now since 3.3 at least.I though i was doing something wrong but after talking to other members of the community they are having the same problem.Please fix

Same issue with not being able to stake the 5000 D FortunaStake collateral sometimes. I kept thinking this is a weight against network weight thing, but maybe its a time thing?
Maybe some others could try leaving 5k in a FS for 21 days, then delete that single line in fortunastake.conf and try staking that single 5k collateral.

Just stopped a 41d old node and got same issue. no stacking reward from the 5k collateral.
For the next 15 minutes after breaking the node i recived a transaction every 2 minutes with the supposed stacking reward, but all of those failed reporting as "?".
In few days can break more nodes, as soon as they mature 21-22 days.

This is somewhat normal operation, but you must STOP your FS node, add mnconflock=0 and fortunastake=0 to your conf and then you must wait for it to drop off the list to actually start staking the funds. For @manosv issue, I will need to get with you further to see what is going on exactly.
If you are seeing ? (Generated/Not Accepted) you are staking, but someone else beat you to solving that particular block, just keep trying ;)

@carsenk in my example i was reporting that the stacking weight accumulated from the node during his 41 days life, is not paid once you break it.

On my wallet when I took a few FS's off to stake, the rest of the wallet stopped staking. Only once I moved those 5k's that wouldn't stake to a new txid, would the wallet start staking again.
On more recent FS's that I setup they staked fine, maybe about a week or 2 old. So its either time or weight.

i can add that if you add the removed node back, the rest of the wallet start to stake again as it does moving the coins to a new txid

its time to close this.The issue is resolved


Roll engine 56bad48ab2ed..56139397e6ae (2 commits)

https://github.com/flutter/engine/compare/56bad48ab2ed...56139397e6ae
git log 56bad48ab2ed..56139397e6ae --no-merges --oneline
56139397e Roll src/third_party/skia 7ba1d64f0706..5f0726b01019 (12 commits) (flutter/engine#6104)
47a1ce0e6 Allow embedders to set the root surface transformation. (flutter/engine#6085)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://flutter-dashboard.appspot.com/build.html

Commit queue failed; closing this roll.


[Snyk] Upgrade org.apache.commons:commons-lang3 from 3.12.0 to 3.14.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.apache.commons:commons-lang3 from 3.12.0 to 3.14.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 3 months ago, on 2023-11-18.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

📚  View the storybook of this branch here

Closed in favor https://github.com/gravitee-io/gravitee-api-management/pull/6761


Pass radionuclide parameters from config

Adds ability to pass radionuclide parameters from the configuration file, and to turn off radiogenic heating entirely (for both SPIDER and Aragog). This is done by specifying concentrations (ppmw) for naturally occurring Th, U, K, and the reference age to which these concentrations correspond. Other physical quantities (heat production, half life, isotopic abundance) are specified in constants.py.
Closes #276.
Requires https://github.com/ExPlanetology/aragog/pull/33 to be merged first.
I have compared the results against SPIDER, and we get the exact same heat production [W/kg] when using the same parameters. We also get reasonable values for the globally-averaged heat flux F_radio which is ~0.8 W/m^2.

Yes, definitely. Thanks Tim!

Oh... I was mistaken. It works fine to run with current version and radiogenic_heat turned to false. I had issues in the past running with empty radionucleide array that made me creating a dummy one, but it seems it has been fixed in the meantime.

Fair enough. Thanks for pointing this out though - I had not considered this condition!


Command line flags should be additive

where multiple flags are allowed i.e. the attestor flag -a attestors added at the command line should be in addition to those defined in the config file.

Need to really understand the UX of this so we end up with predictable behavior


Idea: Fix styles 'small-device-button-sidebar'

It looks like small-device-button-sidebar styles is broken. It would be nice to add proper padding and fixed position svg element inside of the button.
Let me know if it need to be fixed.

Sure, give it a shot! Should be centered


Suggestion for dependabot

May I suggest you to use the groups feature in dependabot?
https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups
This could reduce the number of PRs
You can have a look at what was done in this repository when I suggested it
https://github.com/Zxilly/go-size-analyzer/commit/ec9c02833120b5ccca104f6b3a4f8366a5dc04fd

Thanks, I would have a look at it


Try to improve performance with multiple HTML pages

Fixes #208
This breaks index.html in to multiple pages in order to hopefully improve readability as well as performance.
Signed-off-by: Scott Johnson<EMAIL_ADDRESS>

Since this is a pretty big change to the user viewpoint, it would be good to get a second opinion from @warrenjp . Take your time.

Updated to also handle "old style" study parameters, such as index.html?cherokee. These now get directed to the appropriate webpage (ie. index_cherokee.html in this case)

I suggest changing the word "peoples" to "nations" in the opening text in index_cherokee.html

I have address several of the comments, all but the "Advance" one. I'm not sure I like Continuous +, though I agree we can do better than Advance. I'll file an issue.
Note that your suggestion spurred me to test the feature of adding hyperlinks to popup text, which works nicely. Good suggestion @warrenjp .


es5 support?

https://github.com/postcss/postcss-selector-parser/issues/137
This library uses only a few features from es6, but it makes it hard to use in some cross-browser deployments still. Would you accept one of the following patches?

Use babel to compile es6 -> es5 as part of the publish process
Use babel to compile an alternate version that would be released in addition to the es6 version for browsers to consume?
Change the source to use var instead of const, & let. Use function instead of =>

Thumbs up on this.  Makes configuration life a lot easier that way actually.

@mathiasbynens Wow that was a quick, and really appreciate it!!  Thank you!

🎉


Cannot load local images in 0.63.4 in iOS simulator

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
After adding image to source tree, when using code like <Image source={ require('./path_to_image.png') } /> the image is not rendered and no error is shown. If I use a URI or icon, it renders.
React Native version:
Run react-native info in your terminal and copy the results here.
System:
OS: macOS 11.1
CPU: (8) x64 Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz
Memory: 500.91 MB / 32.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 14.5.0 - /usr/local/bin/node
Yarn: 1.22.5 - ~.yarn/bin/yarn
npm: 6.14.9 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.0 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.3, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
Android SDK: Not Found
IDEs:
Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
Xcode: 12.3/12C33 - /usr/bin/xcodebuild
Languages:
Java: Not Found
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.4 => 0.63.4
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
See above.
Expected Results
Image renders

Hi @plnii thank you for the issue. Could you test this PR and see if it helps your case? https://github.com/facebook/react-native/pull/29595
You could use patch-package to test it out. https://github.com/ds300/patch-package

Hi @plnii thank you for the issue. Could you test this PR and see if it helps your case? https://github.com/facebook/react-native/pull/29595
You could use patch-package to test it out. https://github.com/ds300/patch-package

@safaiyeh I'm not sure exactly how to apply this patch. But I edited the RCTImageLoader.mm file. Then used Patch-Package to build and apply the fix. Using React Native 0.63.4. Here's part of the build errors:
`/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:102:13: error: property implementation must have its declaration in interface 'RCTImageLoader' or one of its extensions
@synthesize turboModuleRegistry = _turboModuleRegistry;
^
/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:1145:106: error: no type named 'InitParams' in 'facebook::react::ObjCTurboModule'

(std::shared_ptrfacebook::react::TurboModule)getTurboModule:(const facebook::react::ObjCTurboModule::InitParams &)params
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:629:12: error: use of undeclared identifier '_turboModuleRegistry'
&& ![_turboModuleRegistry moduleForName:"RCTNetworking"]) {
^

@safaiyeh I'm not sure exactly how to apply this patch. But I edited the RCTImageLoader.mm file. Then used Patch-Package to build and apply the fix. Using React Native 0.63.4. Here's part of the build errors:
`/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:102:13: error: property implementation must have its declaration in interface 'RCTImageLoader' or one of its extensions
@synthesize turboModuleRegistry = _turboModuleRegistry;
^
/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:1145:106: error: no type named 'InitParams' in 'facebook::react::ObjCTurboModule'

(std::shared_ptrfacebook::react::TurboModule)getTurboModule:(const facebook::react::ObjCTurboModule::InitParams &)params
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
/app/node_modules/react-native/Libraries/Image/RCTImageLoader.mm:629:12: error: use of undeclared identifier '_turboModuleRegistry'
&& ![_turboModuleRegistry moduleForName:"RCTNetworking"]) {
^

This seems to indicate that more than one file needs to be changed but the PR only has one file. I'm not sure how to track down the dependencies or get them into my project without a lot of work on my end. Could I pull the latest release candidate via yarn and patch that? Or maybe a nightly build?

This seems to indicate that more than one file needs to be changed but the PR only has one file. I'm not sure how to track down the dependencies or get them into my project without a lot of work on my end. Could I pull the latest release candidate via yarn and patch that? Or maybe a nightly build?

@plnii Did you try import instead of require? BTW, what is the use case why did you use require?

@plnii Did you try import instead of require? BTW, what is the use case why did you use require?

I'm just trying to load a bunch of local images as per https://reactnative.dev/docs/images
It's been a few weeks, but I tried every way I could find to get images to load and had no success with local images. Temporarily I'm loading from an external URI, which is not idea. I'm not sure specifically what you mean by import. Do you think a particular way of importing will work?

I'm just trying to load a bunch of local images as per https://reactnative.dev/docs/images
It's been a few weeks, but I tried every way I could find to get images to load and had no success with local images. Temporarily I'm loading from an external URI, which is not idea. I'm not sure specifically what you mean by import. Do you think a particular way of importing will work?

@plnii I see, you can try this file structure.
.
|--src
|---App.js
|---images
|----test1.jpg
|----test2.png
|----...
index.js

Let say I'm inside App.js, then I want to load those two or more images
Access local files via import
import Image1 from './images/test1.jpg';

<Image source={Image1} />
Access local files via require
<Image source={require('./images/test2.png')} />

@plnii I see, you can try this file structure.
.
|--src
|---App.js
|---images
|----test1.jpg
|----test2.png
|----...
index.js

Let say I'm inside App.js, then I want to load those two or more images
Access local files via import
import Image1 from './images/test1.jpg';

<Image source={Image1} />
Access local files via require
<Image source={require('./images/test2.png')} />

I tested the above approach it seems to have worked. It turned out the defect I was experiencing was with React Native Elements, which use the same Image tag. I see they fixed the defect here: https://github.com/react-native-elements/react-native-elements/pull/2717

I tested the above approach it seems to have worked. It turned out the defect I was experiencing was with React Native Elements, which use the same Image tag. I see they fixed the defect here: https://github.com/react-native-elements/react-native-elements/pull/2717


Incorrect parsing of files in a directory under Windows7

as the initial directory for scan is set to "/" - that cannot properly be processed:

E:\gopath\src\github.com\nelsam\vidar>vidar 1.txt
2016/04/09 13:30:09 Received error reading directory /: open /: Access is denied.

This should be solved.  Please check and close this issue if it's working for you.


feat(mypy): add mypy run to the CI workflow (TDE-184)

Change Description:
Mypy configuration is not checking all the current errors by default. That means mypy . run on the current code base will not raise any error.
Mypy run is added to the CI workflow. mypy . will be run at every build on the GitHub repository.
Notes for Testing:
Run mypy . at the project root. Make sure that it returns Success: no issues found in [...]

This should be a ci change not a feat as this is not adding any sort of feature to the codebase and is only modifying the ci pipeline.
Also scopes are generally for areas of your codebase, so its somewhat weird to have a scope of (mypy), I dont think we have any discussed or defined "scopes" inside of this repo.
otherwise LGTM


controls: false gives "TypeError: undefined is not an object (evaluating 'An.children')"

My fairly simple tns code:
var carousel = tns({
    container: document.querySelector('.carousel'),
    controls: false,
});

Gives me the error "TypeError: undefined is not an object (evaluating 'An.children')". Changing controls to true, or deleting the line, solves this.

Having a similar problem with controlContainer.

Sample CodePen: http://codepen.io/jsit/pen/XMKzwr

I will look into it tmr or next Monday
On Sat, Mar 4, 2017, 5:05 PM Jay<EMAIL_ADDRESS>wrote:

Sample CodePen: http://codepen.io/jsit/pen/XMKzwr
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ganlanyuan/tiny-slider/issues/23#issuecomment-284188039,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AExGmzATYvVWtOqFLkGvTzvI9HrTA7l2ks5rieAMgaJpZM4MTLcv
.

Hey,
Just fixed an issue on selecting controlsContainer, could you check it again?
BTY, seems using https://raw.githubusercontent.com/ganlanyuan/tiny-slider/master/dist/tiny-slider.js don't work for me on codepen, I made a copy with inline styles and scripts.
http://codepen.io/ganlanyuan/pen/bqexjZ

Seems to work fine now, thanks!


fix: entrypoint fork checkout conflicting names

Continues work from b2eecca
We need to set the modified branch name to track the upstream (unmodified) branch name.
Before
 >>> GITHUB_WORKSPACE: /github/workspace
>>> Running steps for PR from fork
>>> BASE BRANCH: main
>>> GITHUB HEAD REF: main
error: branch 'main_fork' not found.
Switched to a new branch 'main_fork'
fatal: branch 'main' does not exist
There is no tracking information for the current branch.
Please specify which branch you want to merge with.
See git-pull(1) for details.

    git pull <remote> <branch>

If you wish to set tracking information for this branch you can do so with:

    git branch --set-upstream-to=origin/<branch> main_fork

HEAD is now at 7efb132 Merge pull request #834 from whegedus/doc/directory-structure
HEAD is now at 7efb132 Merge pull request #834 from whegedus/doc/directory-structure
origin	https://<redacted>/ops/prometheus_rules (fetch)
origin	https://<redacted>/ops/prometheus_rules (push)
From https://<redacted>/ops/prometheus_rules
 * branch            main       -> FETCH_HEAD
Already up to date.
Previous HEAD position was 7efb132 Merge pull request #834 from whegedus/doc/directory-structure
Switched to branch 'main_fork'
>>> Running: pint --config=./.pint.hcl  ci --base-branch=main

After
>>> GITHUB_WORKSPACE: /github/workspace
>>> Running steps for PR from fork
>>> BASE BRANCH: main
>>> GITHUB HEAD REF: main
error: branch 'main_fork' not found.
Switched to a new branch 'main_fork'
branch 'main_fork' set up to track 'origin/main'.
Already up to date.
HEAD is now at 5680333 ci: test pint-action fix
HEAD is now at 5680333 ci: test pint-action fix
origin	https://<redacted>/ops/prometheus_rules (fetch)
origin	https://<redacted>/ops/prometheus_rules (push)
From https://<redacted>/ops/prometheus_rules
 * branch            main       -> FETCH_HEAD
Already up to date.
Previous HEAD position was 5680333 ci: test pint-action fix
Switched to branch 'main_fork'
Your branch is ahead of 'origin/main' by 12 commits.
  (use "git push" to publish your local commits)
>>> Running: pint --config=./.pint.hcl  ci --base-branch=main

Somewhat related to https://github.com/cloudflare/pint/pull/864


Add link to KubeCon Virtual EU 2021

This PR adds link to KubeCon Virtual EU 2021 in the index.html

/assign @onlydole

Markdown looks OK
/lgtm
aside I think we could use headless bundles to represent events and generate pages that use them. Maybe even JSON-LD as well.

@sftim I think it should be dealt with as a new issue. I would not mind if you wanna close it and try the new way for this time

Do we need a separate PR to deal with the bundle? Or will we need to solve here there?

Let’s approve for now, and treat headless bundle implementation as a separate issue.
/approve


Data Science Settings leaves old developer toolbars behind

Updated to VS Community 2015 Update 3 and installed RTVS 0.5
Ran "R Tools > Daya Science Settings" expecting to get the toolbar layout depicted in the Install Guide.
Instead, my toolbar looks like this:

I don't think we provide toolbar layout, only window layout.

VS has no ability for the window layout to say 'close everything but this' (i.e. whitelist case). It does not touch windows that particular layout does not know about or were opened by the user. For example, if you open C# interactive and reset window layout, the window will not be closed. Similarly, if some other team adds a new toolbar in Update N, it will appear as DS layout does not explicitly say 'close it'.
So effectively this is by design.


Terminal seems to flicker more than ever

I feel that the terminal flickers more when I resize it, not sure it ever did flicker so much actually:

Did something change in the past weeks?
My terminal related settings:
"terminal.integrated.fontFamily": "Menlo",
    "terminal.integrated.fontSize": 13,
    "terminal.integrated.lineHeight": 1.4,
    "terminal.integrated.windowsEnableConpty": false,
    "terminal.integrated.showExitAlert": false,
    "terminal.integrated.tabs.enabled": true,
    "terminal.integrated.shellIntegration.enabled": true,
    "terminal.integrated.stickyScroll.enabled": false,
    "terminal.integrated.commandsToSkipShell": [
        "workbench.action.openPreviousEditorFromHistory",
        "workbench.action.startTerminalSpeechToText",
		"workbench.action.stopTerminalSpeechToText"
    ],
    "terminal.integrated.initialHint": false,

MBP M1, 2020. 16 GB RAM