Ensure cached env not written when remember flag off

@birdandbees

Working on the tests.

Looking at this now.

The issue is a little more complicated than I originally thought. Will continue working on this at the end of the day.

Couple more hrs when I can revisit this.

This one is good to go. slightly over raw data * 2

Great, fixing tests now.

@robertzk if you are too busy, maybe @kirillseva can give it a shot?

@robertzk @birdandbees tests are still broken

They pass locally. Rn debugging Travis-CI
Sent from my iPhone

On Dec 8, 2015, at 09:17, Tong<EMAIL_ADDRESS>wrote:
@robertzk @birdandbees tests are still broken
—
Reply to this email directly or view it on GitHub.

I've narrowed it down a problem with purrr but not sure exactly how yet.

@robertzk same failures....


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 215 ms

Resolved: ClamAV performance has improved in 9c327fb.


Update Unity ID to easy

https://support.unity.com/hc/en-us/articles/360000878446-How-do-I-permanently-delete-my-Unity-account

Thanks for your contribution!


always got the time out when try to download upb file.

Firstly try to build TP  2.3 branch in windows 10 using the following command, always got the time out when download upb file.
can I manully download the file or disable the download operation? or modify the connect_time_out option?
bazel build //tensorflow/tools/pip_package:build_pip_package
ERROR: no such package '@upb//bazel': java.io.IOException: Error downloading [https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz] to C:/users/86178/_bazel_86178/26orbg4z/external/upb/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz: connect timed out
INFO: Elapsed time: 69.014s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)

ystem information
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): source
TensorFlow version: v2.3.0
Python version: CPython 3.8
Installed using virtualenv? pip? conda?: conda
Bazel version (if compiling from source): download bazel-3.1.0-windows-x86_64.exe

ystem information
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): source
TensorFlow version: v2.3.0
Python version: CPython 3.8
Installed using virtualenv? pip? conda?: conda
Bazel version (if compiling from source): download bazel-3.1.0-windows-x86_64.exe

I tried to download https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz directly using IE, got the file- upb-9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz. THen put to the filefolder - C:/users/86178/_bazel_86178/26orbg4z/external/upb.
but after type the command - bazel build //tensorflow/tools/pip_package:build_pip_package,
it will automatically delete the files in filefolder - C:/users/86178/_bazel_86178/26orbg4z/external/upb.
questoin: How to disable the "delete" behaviour?

I tried to download https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz directly using IE, got the file- upb-9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz. THen put to the filefolder - C:/users/86178/_bazel_86178/26orbg4z/external/upb.
but after type the command - bazel build //tensorflow/tools/pip_package:build_pip_package,
it will automatically delete the files in filefolder - C:/users/86178/_bazel_86178/26orbg4z/external/upb.
questoin: How to disable the "delete" behaviour?

The tf supporters are probably all on Christmas or new years break because they are not responding to any of the latest issues being raised.

The tf supporters are probably all on Christmas or new years break because they are not responding to any of the latest issues being raised.

The tf supporters are probably all on Christmas or new years break because they are not responding to any of the latest issues being raised.

Happy new year. I am not familar with bazel command options.......

The tf supporters are probably all on Christmas or new years break because they are not responding to any of the latest issues being raised.

Happy new year. I am not familar with bazel command options.......

INFO: Found applicable config definition build:windows in file d:\tensorflow.bazelrc: --copt=/w --copt=/D_USE_MATH_DEFINES --host_copt=/D_USE_MATH_DEFINES --cxxopt=/std:c++14 --host_cxxopt=/std:c++14 --config=monolithic --copt=-DWIN32_LEAN_AND_MEAN --host_copt=-DWIN32_LEAN_AND_MEAN --copt=-DNOGDI --host_copt=-DNOGDI --linkopt=/DEBUG --host_linkopt=/DEBUG --linkopt=/OPT:REF --host_linkopt=/OPT:REF --linkopt=/OPT:ICF --host_linkopt=/OPT:ICF --experimental_strict_action_env=true --verbose_failures --distinct_host_configuration=false
INFO: Found applicable config definition build:monolithic in file d:\tensorflow.bazelrc: --define framework_shared_object=false
INFO: Repository upb instantiated at:
no stack (--record_rule_instantiation_callstack not enabled)
Repository rule http_archive defined at:
C:/users/86178/_bazel_86178/26orbg4z/external/bazel_tools/tools/build_defs/repo/http.bzl:336:16: in
WARNING: Download from https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz failed: class java.io.IOException connect timed out
ERROR: An error occurred during the fetch of repository 'upb':
java.io.IOException: Error downloading [https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz] to C:/users/86178/_bazel_86178/26orbg4z/external/upb/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz: connect timed out
ERROR: no such package '@upb//bazel': java.io.IOException: Error downloading [https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz] to C:/users/86178/_bazel_86178/26orbg4z/external/upb/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz: connect timed out
INFO: Elapsed time: 56.606s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
(base) D:\tensorflow>

INFO: Found applicable config definition build:windows in file d:\tensorflow.bazelrc: --copt=/w --copt=/D_USE_MATH_DEFINES --host_copt=/D_USE_MATH_DEFINES --cxxopt=/std:c++14 --host_cxxopt=/std:c++14 --config=monolithic --copt=-DWIN32_LEAN_AND_MEAN --host_copt=-DWIN32_LEAN_AND_MEAN --copt=-DNOGDI --host_copt=-DNOGDI --linkopt=/DEBUG --host_linkopt=/DEBUG --linkopt=/OPT:REF --host_linkopt=/OPT:REF --linkopt=/OPT:ICF --host_linkopt=/OPT:ICF --experimental_strict_action_env=true --verbose_failures --distinct_host_configuration=false
INFO: Found applicable config definition build:monolithic in file d:\tensorflow.bazelrc: --define framework_shared_object=false
INFO: Repository upb instantiated at:
no stack (--record_rule_instantiation_callstack not enabled)
Repository rule http_archive defined at:
C:/users/86178/_bazel_86178/26orbg4z/external/bazel_tools/tools/build_defs/repo/http.bzl:336:16: in
WARNING: Download from https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz failed: class java.io.IOException connect timed out
ERROR: An error occurred during the fetch of repository 'upb':
java.io.IOException: Error downloading [https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz] to C:/users/86178/_bazel_86178/26orbg4z/external/upb/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz: connect timed out
ERROR: no such package '@upb//bazel': java.io.IOException: Error downloading [https://github.com/protocolbuffers/upb/archive/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz] to C:/users/86178/_bazel_86178/26orbg4z/external/upb/9effcbcb27f0a665f9f345030188c0b291e32482.tar.gz: connect timed out
INFO: Elapsed time: 56.606s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
(base) D:\tensorflow>

@huangyhg
Please, refer similar issues #37811 , link and see if it helps you. Thanks!

@huangyhg
Please, refer similar issues #37811 , link and see if it helps you. Thanks!

@huangyhg
Please, refer similar issues #37811 , link and see if it helps you. Thanks!

yw, what is " .netrc setup"?
I need use IE with one proxy to download the file manully .

@huangyhg
Please, refer similar issues #37811 , link and see if it helps you. Thanks!

yw, what is " .netrc setup"?
I need use IE with one proxy to download the file manully .

https://raw.githubusercontent.com/vokins/yhosts/master/hosts
solved

https://raw.githubusercontent.com/vokins/yhosts/master/hosts
solved

@huangyhg,
Closing this issue as it is resolved. Thanks!

@huangyhg,
Closing this issue as it is resolved. Thanks!

yw, please close it, thanks.

yw, please close it, thanks.


The template instructions can use a csproj

Moving from https://github.com/NuGet/Home/issues/7389
by @PureKorne
Hi 👋
I'm trying to see if it's possible to package a nuget from a .csproj where the package type is a Template.

NuGet product used: VS UI | dotnet.exe
dotnet.exe --version: 2.1.403
VS version: 15.8.7
OS version (i.e. win10 v1803 (17134.345)):
So I'm trying to create a new custom template using the dotnet new template. Part of the steps include creating a .nupkg. The example documentation says to do this using nuget pack <nuspec file>.
I'm trying to do this with dotnet pack instead. nuget.exe isn't in my path (yes, I know I can download it, etc) so I'm trying to do with the existing tools possible / least amount of friction.
In the .nuspec file it says we need to define the nuget package to be a  template like:
<packageTypes>
  <packageType name="Template" />
</packageTypes>

Is there a way we can define this in a .csproj file, so then I might be able to use dotnet pack instead?
by @rrelyea
It looks like setting the csproj property "PackageType" will be persisted as a PackageType when dotnet pack is used. NuGet.Build.Tasks.Pack.Targets defines a Target named "GenerateNuspec" which will pass $(PackageType) into the PackTask.
If this works well, or even better, please consider doing a doc PR or a doc issue to "The example documentation" page you link to in your initial comment.
[Edited by gewarren to add document details below.]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d9cb6bd4-1194-91fa-014c-2e72d809e772
Version Independent ID: c3fddda9-24c1-f20c-a1b6-6aeea61d6e53
Content: Create an item template for dotnet new - .NET Core CLI - .NET Core
Content Source: docs/core/tutorials/cli-templates-create-item-template.md
Product: dotnet-core
GitHub Login: @Thraka
Microsoft Alias: adegeo

This was already completed with the rewrite of the tutorials a while back. Cheers!


Update expfmt.NewDecoder usage

:+1:

After some thinking, I'm considering to let ResponseFormat return FormatUnknown instead of an error. On that NewDecoder can then return an error – less redundant error checking. Thoughts @juliusv ?

Shouldn't we have a fallback format to try in case of an invalid header?
I remember I just fell back to text format at several occasions, and only give up if the content cannot be parsed as text format.
I'm thinking here of ad-hoc exposition using the text format where setting up the right headers might be complicated.

Yes, that would be even one step further, i.e. NewDecoder not returning an error either and defaulting to the text decoder. Also worth considering.

Yeah, when the headers are missing completely, it'd make sense to fall back to the text format. If they are set to something unknown, I'd go for the FormatUnknown return. But don't feel strongly about any of these alternatives.

Closing as this is invalidated by prometheus/common#6


Add my.tccd.edu

Tarrant County College

URL - https://www.tccd.edu/
IT Course - https://catalog.tccd.edu/preview_program.php?catoid=4&poid=876
Email Portal - https://mail.google.com/a/my.tccd.edu

@theg1nger I am sorry, I have to close this request without merging. This domain is blacklisted because the college allows to register an email address there even if you are not a student of the college. If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


Fix: Quickly press different menu items opens them one by one

Details

In Settings page, quickly press multiple menu items will open them all, one by one. This PR only allows the first pressed item to be opened.
Fixed Issues

$ https://github.com/Expensify/App/issues/23867
PROPOSAL: https://github.com/Expensify/App/issues/23867#issuecomment-1659542744
Tests

Login with any account
Open Settings page
Press different menu items as quick as possible
Verify that only the first menu is navigated

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

Offline tests

NA
QA Steps

On Android, there's a slow delay when navigating so it's hard to reproduce on other platforms but it's fine to test on Android only.

Login with any account
Open Settings page
Press different menu items as quick as possible
Verify that only the first menu is navigated

[ ] 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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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 code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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://github.com/Expensify/App/assets/113963320/62e1c11b-ccf4-4d20-93f3-4e3d5d21290f

Mobile Web - Chrome

https://github.com/Expensify/App/assets/113963320/4126caaf-c864-47a0-a316-8dc23805680e

Mobile Web - Safari

https://github.com/Expensify/App/assets/113963320/841ffc54-08cb-4347-8935-bdb635d7e3ad

Desktop
https://github.com/Expensify/App/assets/113963320/f0965c63-12df-4a64-9ab5-3d414d554622

iOS

https://github.com/Expensify/App/assets/113963320/f237737a-19d2-464a-a4c2-c47af8c39f5c

Android

https://github.com/Expensify/App/assets/113963320/b62b626c-0f5b-4040-880c-2d1d319968b3

I was just reviewing the code. Code changes look okay, but I am a bit concerned about the PressableWithFeedback change. I'll need time for QA.

Reviewer Checklist

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

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] 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)
[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 checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] 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
[x] I verified proper code patterns were followed (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 the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[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] 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 grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is 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 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)
[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] 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 code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[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 the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[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 reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/3069065/11870494-ab97-41ba-9006-1a8b60a804c9

Mobile Web - Chrome
https://github.com/Expensify/App/assets/3069065/a8ea1462-87d3-4a1f-9cf1-c55faf96e83c

Mobile Web - Safari
https://github.com/Expensify/App/assets/3069065/72ff27e4-a1e6-41ba-a847-4856edd302aa

Desktop
https://github.com/Expensify/App/assets/3069065/5ed5ef5e-0e97-4ed0-af70-a87593e15f09

iOS
https://github.com/Expensify/App/assets/3069065/dc997f64-c0d3-4f20-bc73-4101a8c30f9f

Android

@tienifr Earlier changes from PressableWithFeedback were added to avoid creating multiple workspaces on back to back click. I am trying this on our branch and it looks like continuous clicks are creating multiple workspaces?
Unfortunately my cocoapods has crashed and I am unable to test this on latest main. Can you please confirm if we introduced a regression here?
https://github.com/Expensify/App/assets/3069065/15ad90db-4854-437e-a41d-3acb5d9521d6

Thanks for the fix @tienifr. I'll be testing this today.

@tienifr yeah this is working fine.
https://github.com/Expensify/App/assets/3069065/26c59e39-dac7-420c-83d8-af3ee3afd48c

@youssef-lr This is ready for your review.
🎀 👀 🎀 C+ reviewed.

: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/youssef-lr in version: 1.3.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.58-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/youssef-lr in version: 1.3.59-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.59-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


update: Add FrontendPractice resource to README

Frontend Practice has HTML and CSS challenges for all skill levels

Thanks for your contribution @MazontheMoon


"npm run build" prints error in console for forks of Cesium

I see this printed out to the console when I run npm run build in a fork of Cesium because it doesn't have our tags.  My first immediate reaction was "oh no, why did the build fail?".  Is there anything we can do to avoid having "fatal" printed out to the console?  Can we check if the tag exists before making the diff call?  Otherwise, maybe we should print out a message saying to ignore that failure, the build is fine.
fatal: ambiguous argument '1.48': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
Failed to retrieve list of new Sandcastle demos from Git.

Just did a clean fork and build and couldn't reproduce this.

Hmm, thanks @zbennett10!  The build still succeeds even when this error prints out so it doesn't effect anything.  I just think it's a little confusing to see that in the console.  I'll have to double check to see if it still happens for me.

I think a new clean fork includes new tags.  This problem likely affects forks only after they get old enough to be missing a couple release tags.

We should fix this, or at least print a more useful informational messege so users don't think it's an error.


Setting align: left for base font still renders justified

The doc for asciidoctor-pdf indicates that left aligned is default for the base font, however I have only seen justified.
I have a request for a document with left alignment. I have tried setting align: left and also leaving the field blank, and in both cases the pdf still renders text as justified.

I believe this works in the latest version. If not, please reopen and submit a theme and sample document I can test.
The syntax should be:
base:
  align: left


Due to Flashattention, inference cannot be performed on v100

Description / 描述
FlashAttention only supports Ampere GPUs or newer.
Case Explaination / 案例解释
Due to Flashattention, inference cannot be performed on v100

done with caseid 592863


There seems to be a problem with the Auth part

Change the TestAuthWithoutSecure test like below code, modify the password you will get unexpected result.
It will return invalid authPluginName, because the nativePassword function return bytes with 0x00, [17 167 145 105 109 173 51 88 102 33 107 0 67 175 113 232 40 58 206 120].
So when you try to read authResponse, you just get the part of the authResponse. And the code continues to execute, the order changes. Actually you will get the database name as the plugin name, that's why we got invalid authPluginName error.
proto/auth_test.go
func TestAuthWithoutSecure(t *testing.T) {
	want := NewAuth()
	want.charset = 0x02
	want.authResponseLen = 20
	want.clientFlags = DefaultClientCapability &^ sqldb.CLIENT_SECURE_CONNECTION &^ sqldb.CLIENT_PLUGIN_AUTH_LENENC_CLIENT_DATA
	want.clientFlags |= sqldb.CLIENT_CONNECT_WITH_DB
	want.authResponse = nativePassword("password", DefaultSalt)
	want.user = "root"
	want.database = "test_db"
	want.pluginName = DefaultAuthPluginName

	got := NewAuth()
	err := got.UnPack(want.Pack(
		DefaultClientCapability&^sqldb.CLIENT_SECURE_CONNECTION,
		0x02,
		"root",
		"password",
		DefaultSalt,
		"test_db",
	))
	got.authResponseLen = 20
	assert.Nil(t, err)
	assert.Equal(t, want, got)
}

Thanks for pointing that out.
This is a bug when we generate the scremble with NULL char.
We will fixed it in the next version.
BTW, We have fixed it in ClickHouse on this commit:
https://github.com/ClickHouse/ClickHouse/pull/10140/files

看 id 感觉是国人，随手 Google 了下，还真是。。

看 id 感觉是国人，随手 Google 了下，还真是。。

If you have any problems with go-mysqlstack, you can also find me:
<EMAIL_ADDRESS>Thanks.

Please the re-pull the master again, we have fixed this issue.


Propose increasing severity for sec-high/sec-critical bugs that are lower than S2

This may fit as a part of the severity_underestimated tool.
@marco-c wdyt?

@suhaibmujahid that would work, though the query might become too complex? Maybe better to have a separate script like the accessibility one, so the queries can be much simpler.
Another option would be to run multiple separate queries as part of the same script.

I query the keywords, it seems that all bugs with sec-high and sec-critical keywords are resolved (0 bugs open, 4056 bugs closed).
Based on this, it seems that the script will not have an impact on current bugs.
@marco-c do I miss something?

@suhaibmujahid sec-high and sec-critical are only visible if you have security permissions, there are actually some that are not S2.


Unable to stop debugger in vsscode when using javascript debug terminal

Describe the bug
https://github.com/vitest-dev/vitest/issues/3569
The issue is closed but it still happening, when i disconnect/stop debugger, it does not stop debugging session
Cancelling test run. Press CTRL+c again to exit forcefully.
...
close timed out after 10000ms

then terminal basically unusable
i have to delete terminal and create a new one which kind of annoying
Reproduction
its obvious
System Info
vscode windows
"vitest": "1.2.1"

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

there is no code to reproduce, just run vitest in vscode with "javascript debug terminal"
press ctrl-c 2 times


Error when creating a new vendor

I am trying to create a new vendor. Here is a code snippet
vendor = Vendor()
vendor.CompanyName = "Test Name"
try:
    vendor.save(qb=client)
except QuickbooksException as e:
   print(e.detail)

The following is printed:

Property Name:Can not instanti specified is unsupported or invalid

However the Vendor object has no property Name. What am I doing wrong?

I checked the Quickbooks vendor documentation and it looks like you need to set DisplayName or one of the other name fields:

The DisplayName attribute or at least one of Title, GivenName, MiddleName, FamilyName, or Suffix attributes are required during create.

Also, you can print out the json for the object you are working with like this:
vendor.to_json()
Then paste it into the QB API explorer to get a better idea of the issue.

Thanks 👍


Images don't refresh on new search

To recreate:

search
gallery view
change query (don't leave gallery view)
search again

Images don't change until you go to table/list and back to gallery.

apparently not fixed


Node-RED on learn.openshift.com

When using the operator on the OpenShift 4.4 playground here https://learn.openshift.com/playgrounds/
The Node-RED operator installed and and I went to create a Node-RED instance but the default YAML does not find the storage class.
Can this either be corrected or instructions created for modifying the YAML for the playground?

Hello Mark,
Can you try creating an instance with out storage? You will have to set this in the "NodeRed" CR
persistentvolume:
enabled: false
Note without storage backup and recovery will not be possible.

Thanks. I made that change and it works. It would be useful to add that to the instructions.

Today, I published two articles about the Node-RED operator on OpenShift.

How to use Node-RED Operator on RedHat OpenShift
https://kazuhitoyokoi.medium.com/how-to-use-node-red-operator-on-redhat-openshift-e5a516de1e17
Human pose estimation using Node-RED
https://kazuhitoyokoi.medium.com/human-pose-estimation-using-node-red-69fdb53bd7

The discussion in this issue was helpful for me to write the articles. Thank you!


Schemas pr-152

Implements https://github.com/Sage-Bionetworks/rocc-schemas/pull/152

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
809 unchanged lines in 47 files lost coverage.
Overall coverage decreased (-14.6%) to 46.311%

Files with Coverage Reduction
New Missed Lines
%

openapi_server/dbmodels/user.py
1
76.92%

openapi_server/dbmodels/challenge_platform.py
2
80.0%

openapi_server/encoder.py
2
87.5%

openapi_server/models/challenge_status.py
2
80.0%

openapi_server/typing_utils.py
2
33.33%

openapi_server/dbmodels/challenge.py
3
78.57%

openapi_server/dbmodels/organization.py
3
78.57%

openapi_server/dbmodels/person.py
3
76.92%

openapi_server/dbmodels/tag.py
3
66.67%

openapi_server/dbmodels/grant.py
4
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-14.6%

Covered Lines:
1136

Relevant Lines:
2453

💛  - Coveralls


add alternate hosts to ndsz output in proxy

Currently we do not add alternate hosts generated for k8s in ndsz output. Sometimes it would be useful to have that for debugging. It increases the ndsz output so if there are concerns with it (I can introduce a flag to url and only add if asked for). It is only added for proxy currently - as it involves more complication to do that at Istiod, but if it is useful, I can try that in a separate PR

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
47dd2d90a924eedc0177afe166c09250c509a8cd
link
unknown
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio

@ramaraochavali: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
47dd2d90a924eedc0177afe166c09250c509a8cd
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

earlier we do not add alt hosts generated after nametable is built. This PR adds them


Add option to automatically assign port

Is your feature request related to a problem? Please describe.
I allow my friends to create custom minecraft server. They have full power of creating them but still it's not pain-free. They will run into issues when starting server for first time because default port (25565) is already assigned to different one.
Describe the solution you'd like
Allow user with admin permissions to:
a) create list of available ports
or
b) dynamically create list of open ports on server
When creating new server Pufferpanel would look at this list and choose non-used port. This way it would be easy to create new server without any issues even for non-techsavy friends.
Describe alternatives you've considered
I considered creating some form of list but this needs to be maintenanced and realtime updated, that is possible, but it will be a lot of cooler if Pufferpanel could do it automatically.

I mean it depends on the template you can prewrite the config file for each version of the server as it is up to the server to assign the default port not the wrapper.

I mean it depends on the template you can prewrite the config file for each version of the server as it is up to the server to assign the default port not the wrapper.

Alright, what will happen when 2 users uses same template? there will be 2 server with same port, right?

make it a variable option in the template and just make them have to read a already taken list before choosing

as you said it was friends correct?

make it a variable option in the template and just make them have to read a already taken list before choosing

Oh, I get it now. It's the almost same solution as creating list I mentioned in main post, right?

right but pufferpanel wouldn't do any of it unless you had some detailed stuff on what you wanted it todo
but it can app be configured in the template

right but pufferpanel wouldn't do any of it unless you had some detailed stuff on what you wanted it todo
but it can app be configured in the template

That's why I wrote this feature request. Just an idea.


[Core] Multiprocessing Pipeline Parallel support

Only for single-node.

There are some issues with the (daemon) miultiprocessing processes not being terminated in the CI, I think due to the test server that spawns them running in a ray actor. This is causing OOM as these processes accumulate. Need to look into this further.

Hey @njhill any luck here?

@andoorve I think this is finally ready if you'd like to take another quick look, sorry for the delay!

Thanks @andoorve! I'm actually not sure whether it's possible to re-trigger individual tests once they have passed :thinking:

I think this command works locally:
./.buildkite/run-multi-node-test.sh /vllm-workspace/tests 2 2 public.ecr.aws/q9t5s3a7/vllm-ci-test-repo:f0d5e5cdb0f9da33cfc7ac596de0be4155b594e9 "VLLM_TEST_SAME_HOST=0 torchrun --nnodes 2 --nproc-per-node=2 --rdzv_backend=c10d --rdzv_endpoint=<IP_ADDRESS> distributed/test_same_node.py && pytest -v -s distributed/test_pipeline_parallel.py" "VLLM_TEST_SAME_HOST=0 torchrun --nnodes 2 --nproc-per-node=2 --rdzv_backend=c10d --rdzv_endpoint=<IP_ADDRESS> distributed/test_same_node.py"


Allow dynamic content resize

Added a way to change the collapsible's height.
Added example.

Is this still usable with RN40 ?

Closing this as inactive as there's been no activity for over 30 days. Please open a new pull request if you're still desirous of merging this PR.


Create basic story for < FunctionArguments />, < InformationBox />

Monday ticket
Add basic story by structure similar to existing ones
https://github.com/mondaycom/vibe-storybook-components/blob/master/COMPONENTS_DOCUMENTATION_GUIDELINES.md

I would like to take this up too.

Thanks a lot, @Dhoni77 !
It's yours :)

Hi @SergeyRoyt In the <FunctionArgument/> component I could see that the

Closed thanks to @Dhoni77 !


fixed config.yml

fixed config.yml

👷 Deploy Preview for quiet-pie-34a01b processing.

Name
Link

🔨 Latest commit
953edfb272e51a441766cd075f1f91139856e0e6

🔍 Latest deploy log
https://app.netlify.com/sites/quiet-pie-34a01b/deploys/65464e942d875e00081e7a35


updated edit date

Docs Build status updates of commit 07396c6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/add-holographic-remoting.md
:warning:Warning
View
Details

mixed-reality-docs/add-holographic-remoting.md

Line 18: [Warning] Illegal link: `<a href="index.md#news-and-notes">coming soon</a>` -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


configure terragrunt

Hi Guys,
please help me to configure this part:
Install Terraform, and let Terragrunt know where to find it using one of the following options:
Place terraform in a directory on your PATH.
Caution: this makes it easy to accidentally invoke Terraform directly from the command line (thus bypassing the protections offered by Terragrunt).
Specify the full path to the Terraform binary in the environment variable TERRAGRUNT_TFPATH.
Specify the full path to the Terraform binary in --terragrunt-tfpath each time you run Terragrunt (see CLI Options).
Install Terragrunt by going to the Releases Page, downloading the binary for your OS, renaming it to terragrunt, and adding it to your PATH.

Which of those options are you trying to do?

Option 2.I'd like to know all of them,if possible.I added terragrunt file to /usr/bin directory.is it correct?

PATH is an environment variable that contains a list of paths where the OS will look for executables. When you type in a command like grep or terraform, the OS will loop through the list of paths in PATH and call the first executable in those paths that matches your command (or exit with an error if a match can't be found).
/usr/bin and /usr/local/bin are part of the default PATH on many operating systems, so putting the Terragrunt and Terraform binaries in one of those will work. Alternatively, you could put them in a new folder (e.g. /foo/bar/terraform) and add that folder to PATH:
export PATH=/foo/bar:$PATH

The code above prepends /foo/bar to the previous value of PATH. Note that if you just run the command above, it only affects your current shell. If you want to set an environment variable that's available in every shell you open, see here.
If you want to set the TERRAGRUNT_TFPATH variable, you take a similar approach:
export TERRAGRUNT_TFPATH=/foo/bar

thank you man
my os is:Ubuntu(16.04) x86_64
I downloaded this file:terragrunt-0.10.2.tar.gz(Source code (tar.gz))
I added Terragrunt and Terraform binaries to /usr/bin and /usr/local.(and two other methods)
I got this error after ran terragrunt plan:
payamzowghi@ubuntu:~/Downloads/terraform_0.8.6/chapter3$ terragrunt plan
bash: /usr/bin/terragrunt: cannot execute binary file: Exec format error
then I downloaded the other files(terragrunt_linux_386 or terragrunt_linux_amd64).they aren't binary.
I don't know what is the problem :(

I downloaded this file:terragrunt-0.10.2.tar.gz(Source code (tar.gz))

Why did you download the source code? You want the binary. For example, for your OS, you want terragrunt_linux_amd64.

I downloaded this file and changed the name to terragrunt and copy in /usr/bin
payamzowghi@ubuntu:~/Downloads/terraform_0.8.6/chapter3$ ls /usr/bin | grep terra
terraform
terragrunt
but it didn't work as the command. It seems that it is not the binary :(
I'm confused.

You need to give it execute permissions too.
chmod u+x /usr/bin/terragrunt
chmod u+x /usr/bin/terraform

thank you it works but it should be:
chmod +x /usr/bin/terragrunt

Hi,
I am getting the below error on my MAC while executing : terragrunt
/usr/local/bin/terragrunt: cannot execute binary file
I have downloaded terragrunt_linux_amd64 and renamed it to terragrunt and moved to /usr/local/bin
Lastly, executed chmod +x /usr/bin/terragrunt
But I am still getting error as mentioned above.
Could someone please suggest ? I am trying to install Terragrunt v0.18.3. thanks in advance for your help.

Hi,
I am getting the below error on my MAC while executing : terragrunt
/usr/local/bin/terragrunt: cannot execute binary file
I have downloaded terragrunt_linux_amd64 and renamed it to terragrunt and moved to /usr/local/bin
Lastly, executed chmod +x /usr/bin/terragrunt
But I am still getting error as mentioned above.
Could someone please suggest ? I am trying to install Terragrunt v0.18.3. thanks in advance for your help.

@kiran266   Check the kernel version with uname -a. It is Darwin for mac. Please download the file specific to darwin "terragrunt_darwin_amd64" and that should solve your problem


retry as stale read on replica

Issue #17422
We have seen that retrying stale read as follower read caused raft channel clogging which increases the blast radius and impact other regions.
This would be the flow with this change

Try on replica in same AZ
Retry in leader if original request is not on leader and leader is marked serverIsBusy and DeadlineExceeded
Retry on replica as stale read

I expect requests to be succeeded in leader read if the leader is not slow. We should retry it as follower read if leader already slow.
Unit Test

I injected the error between TiDB-->TikV and confirmed that reads are retried as stale reads.

Test results

zyguan, you06 please review these changes


How to change 5173 port?

How to change the defaut 5173 port to others? for example 7060 because 5173 port was used by other application?
i want use localhost:7060 to access the main page.

@jamsnrihk  Are you talking about running locally?

If you are using docker-compose.yaml then its easy to change its right there in the ports section for each container.
Just like @Vaibhav91one  suggested, if we can get more information on how you are deploying it,  it would be great.

Yes, I am running locally using docker.
On Mon, 16 Oct 2023 at 01:19, Vaibhav Tomar @.***>
wrote:

@jamsnrihk https://github.com/jamsnrihk Are you talking about running
locally?
—
Reply to this email directly, view it on GitHub
https://github.com/arc53/DocsGPT/issues/613#issuecomment-1763452248, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A76VNQTW3JLMT2P2Z65LHMDX7QLJBAVCNFSM6AAAAAA6A7XGBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGQ2TEMRUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
P Help save paper - do you need to print this email?

@dartpain
I am using docker. following is docker-compose.yaml, but, seems either localhost:7060 or 192168.23.100:7060, both not working. If change back to 5173, everthing is ok. please advise whether any other config file need to change?
version: "3.9"
services:
frontend:
build: ./frontend
environment:
- VITE_API_HOST=http://<IP_ADDRESS>:7091
- VITE_API_STREAMING=$VITE_API_STREAMING
ports:
- "7060:7060"

depends_on:
  - backend

backend:
build: ./application
environment:
- API_KEY=$API_KEY
- EMBEDDINGS_KEY=$API_KEY
- CELERY_BROKER_URL=redis://redis:6379/0
- CELERY_RESULT_BACKEND=redis://redis:6379/1
- MONGO_URI=mongodb://mongo:27017/docsgpt
ports:
- "7091:7091"
volumes:
- ./application/indexes:/app/application/indexes
- ./application/inputs:/app/application/inputs
- ./application/vectors:/app/a

pplication/vectors
depends_on:
- redis
- mongo
worker:
build: ./application
command: celery -A application.app.celery worker -l INFO
environment:
- API_KEY=$API_KEY
- EMBEDDINGS_KEY=$API_KEY
- CELERY_BROKER_URL=redis://redis:6379/0
- CELERY_RESULT_BACKEND=redis://redis:6379/1
- MONGO_URI=mongodb://mongo:27017/docsgpt
- API_URL=http://backend:7091
depends_on:
- redis
- mongo
redis:
image: redis:6-alpine
ports:
- 6379:6379
mongo:
image: mongo:6
ports:
- 27017:27017
volumes:
- mongodb_data_container:/data/db
volumes:
mongodb_data_container:

@dartpain
I am using docker. following is docker-compose.yaml, but, seems either localhost:7060 or 192168.23.100:7060, both not working. If change back to 5173, everthing is ok. please advise whether any other config file need to change?
version: "3.9"
services:
frontend:
build: ./frontend
environment:
- VITE_API_HOST=http://<IP_ADDRESS>:7091
- VITE_API_STREAMING=$VITE_API_STREAMING
ports:
- "7060:7060"
depends_on:
- backend
backend:
build: ./application
environment:
- API_KEY=$API_KEY
- EMBEDDINGS_KEY=$API_KEY
- CELERY_BROKER_URL=redis://redis:6379/0
- CELERY_RESULT_BACKEND=redis://redis:6379/1
- MONGO_URI=mongodb://mongo:27017/docsgpt
ports:
- "7091:7091"
volumes:
- ./application/indexes:/app/application/indexes
- ./application/inputs:/app/application/inputs
- ./application/vectors:/app/application/vectors
depends_on:
- redis
- mongo
worker:
build: ./application
command: celery -A application.app.celery worker -l INFO
environment:
- API_KEY=$API_KEY
- EMBEDDINGS_KEY=$API_KEY
- CELERY_BROKER_URL=redis://redis:6379/0
- CELERY_RESULT_BACKEND=redis://redis:6379/1
- MONGO_URI=mongodb://mongo:27017/docsgpt
- API_URL=http://backend:7091
depends_on:
- redis
- mongo
redis:
image: redis:6-alpine
ports:
- 6379:6379
mongo:
image: mongo:6
ports:
- 27017:27017
volumes:
- mongodb_data_container:/data/db
volumes:
mongodb_data_container:

Change it like this please
ports:

"7060:5173"
Keep original port on the right side, and port you want to map to on the left

Thanks, it works.


Structure of business directory

I have come to issue with current structure of business directory (business/data) when i need to query and return association between user and product (in my case user and role) with current structure if i want to return Products info (models) based on user ID and my func is in business/data/user pkg I have to create another struct of Products in user pkg and duplicate product struct then I can return result of query to my handler I can not import product struct from product pkg (import cycle not allowed).
example.com/users/::USER_ID::/products --- should return list of products associated with user.
and if I need to do same with product to return user association (if many to many)
example.com/products/::PRODUCT_ID::/users -- should return list of users associated with product.
So a options I can suggest:
Adding inside business/data/APP_NAME as in app folder and it make sense becasue this kind of monorepo structure
example:
business/data/sales-api/user
business/data/sales-api/product
and split models to own separate pkg so other pkg can import and avoid import cycle issue.
example:
business/data/sales-api/models
business/data/sales-api/models/user.go
business/data/sales-api/models/product.go
if you have better option please let me know :)

Nooo, none of this is good. Let's get on a call to discuss. Send me an email<EMAIL_ADDRESS>


fix: add 3ds frictionless faq

Description
Title

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Install procps to make sysctl command available.

Since 6ec2880f6ae8a0384eed7c93c6c76970c4fed885, sysctl command is used by vr-bgp. This PR installs the command as it is missing in debian:stable image.

Weird that we have containers for which this works... but don't seem the harm in explicitly installing.

BTW, I found out why we had it.. I temporarily built some container manually and based them on ubuntu for testing purposes.. forgot to fix that afterwards


add generate_tracing test

Signed-off-by: Evan Flynn<EMAIL_ADDRESS>
add ros2_tracing to the generate_trace test

closes #16

@christophebedard would love your input here to know if I am missing something in the launch configuration of Trace().
right now after running colcon test the tracing data is put into a tracing folder in the colcon_ws directory, here are the resulting profile directories and their sizes:

@christophebedard I wasnt sure if I needed to add these settings to properly configure the tracer. what exactly do these settings do?

@christophebedard I wasnt sure if I needed to add these settings to properly configure the tracer. what exactly do these settings do?

@flynneva they configure the LTTng userspace events, kernel events, and context fields that will be enabled for tracing. There are default values: https://gitlab.com/ros-tracing/ros2_tracing/-/blob/ad79dad92013e554b902bfcb460003038f14a8a3/tracetools_launch/tracetools_launch/action.py#L48-L71. It's always better to set them explicitly, but the default values should be fine here, at least for now.


Canary releases and dynamic routing done right

We faced with the problem that we need to do such thing as dynamic routing and canary releases. So, for example, we deploy microservice microservice-1. Then, when someone finished a big feature we want to deploy it as a microservice microservice-1.1.
Question
Is it possible to dynamically reroute requests using information (for example from Headers) to use version microservice of version microservice-1.1 instead on microservice-1?
For example, someone needs this feature and he will modify/add specific Header and for all requests, he will use new microservice-1.1. And if that Header is missing then the current microservice-1 version should be used.
If it's not possible, please advise. I am interested in Zuul of course. For service discovery, I am using Eureka. Right now I am looking at https://linkerd.io/ but there is no support for Eureka.

There is no reason why you couldn't write a pre-filter to do that kind of routing.  If that needed to be more dynamic, you could use a dynamic property to choose the header.

@mikeycohen any good examples/techniques that you could advise/show? Looked through the internet and other related question and mostly right now what I could find is kubernetes.


Add catch-all Binary Viewer for non-editable types

Fixes https://github.com/mozilla/thimble.mozilla.org/issues/2374.  This adds a catch-all "Binary" viewer, for file types we don't know what to do with.  The goal here is to not dump the user into a binary file (e.g., Word Doc) and have the editor lock up trying to read it as UTF8.
At the same time, we want to give them options.  They can download it, open it in a new tab, or tell Brackets to let them edit it directly:

@flukeout this is only a skeleton, and I assume we'll want to clean up my code a bit, and especially fix what we show.  Maybe you can iterate on it.
Also, I noticed something when doing this that might be responsible for some of the weird bugs we've seen with viewers.  When a file is renamed, we aren't removing it from the _viewers object cache.  We should be, so that we don't try and open/refresh it.  I'll file a bug on that for the other viewers.

Will be testing this today and will also start work on a styling PR

@humphd I'm encountering an error, here are the steps...

Open the editor via src/hosted
Clear your cache etc
Add a file with an unrecognized extension bling.blam
The Editor switches to show the new view
Reload the editor before doing anything else
Editor doesn't load, stuck on spinning square loader with no error messages in console

Let me try to reproduce here...

Confirmed, I'll debug a bit and report back.

@flukeout I think I fixed it, try testing with my latest commit.

It would be cool if this gave two "open" options: "try to open anyway" and "edit as binary data" that fires up a hex editor - a utility I constantly wish I had available in all my editors, because it makes it really easy to both demonstrate how files work, and do edits that would otherwise need big tools (like resizing a jgp in-file... just change 2 bytes and "magic" that you can teach others has happened, without ever using an image editor)

@humph - confirmed that your latest commit fixes the issue I found.

@Pomax I mean...https://xem.github.io/hex/...we could?

Man, there are a lot of these around http://mikach.github.io/hex-editor/

@humphd...

I made a PR with some styling and localization updates against this branch

https://github.com/humphd/brackets/pull/12

Here's the Thimble PR with the required strings

https://github.com/mozilla/thimble.mozilla.org/pull/2428

Let's merge this when I get back from holiday.

Ping @humphd - I think this is good to go now.

OK, landed.  Let's test this on staging this week, and make sure we're happy with how it all happens now.


Fix bug in Gruneisen DensityEnergyFromPressureTemperature()

PR Summary

@Gerbeset wasn't able to use the DensityEnergyFromPressureTemperature() function with ambient conditions and properties for aluminum. This pointed to a bug in the implementation.
As far as I can tell, the bug in question was in the bounds for the root find. The big issue I think was that the upper bound was outside the applicable maximum density of the EOS. Now that we have a maximum density implemented, this can bound the root find appropriately.
EDIT - It turns out that the issue was thermodynamic consistency. For a given temperature, there is a maximum in the pressure since the cold curve energy doesn't increase with increasing density. As a result, at temperatures/energies below the reference curve, there will be a regime where the pressure decreases as a function of density. This is not a phase change, but rather a result of thermodynamic consistency breaking. The root find assumed a monotonic $P(rho, T)$ curve for a given temperature, so it was failing to converge.

See above for a plot of the pressure for various isotherms. This should be monotonic.
This MR introduces a new function that bounds the density to a range where the EOS is thermodynamically stable when performing the P-T lookup.
An open question is whether to apply this domain limiting procedure to all lookups or not.
Less important changes
I also switched from a reference capture lambda to our PORABLE_LAMBDA.
To make the implementation easier to follow (and more general), I decided to just use density-temperature lookups everywhere. As a result, I find the density that satisfies the pressure-temperature point first and then use that density in the energy lookup. This isn't strictly necessary since the EOS assumes that energy is constant along an isotherm in density space (violating thermodynamic consistency), but I figured I'd do it this way anyway to be more correct. I think there should be negligible performance difference if any. Since this is generally an initialization function, I doubt that matters.
I also added a test to make sure that the call is returning correct values.
PR Checklist

[x] Adds a test for any bugs fixed. Adds tests for new features.
[x] Format your changes by using the make format command after configuring with cmake.
N/A Document any new features, update documentation for changes made.
[x] Make sure the copyright notice on any files you modified is up to date.
[x] After creating a pull request, note it in the CHANGELOG.md file.
[x] LANL employees: make sure tests pass both on the github CI and on the Darwin CI

@chadmeyer and @jonahm-LANL can you review?

Found another bug... working on it. Will make draft for now

@chadmeyer and @Yurlungur I had to make a few more changes.
It looks like the primary issue is that the Gruneisen EOS pressure is not monotonic along an isotherm. I didn't realize that subtraction of the reference energy curve will eventually overwhelms the pressure reference curve, causing a maximum in the pressure.
I introduced a function to calculate the location of this maximum and essentially bound the EOS to the region up to this maximum (in P-T space). We could think about imposing this maximum on the other lookups too, but it would add an expensive root find. The P-T lookup is most likely at initialization so it's okay to be a bit more expensive there.

I'm converting back to draft. I'm encountering a situation where dPres_drho_e is returning a negative value and I'm not quite sure why.

I'm converting back to draft. I'm encountering a situation where dPres_drho_e is returning a negative value and I'm not quite sure why.

I'm converting back to draft. I'm encountering a situation where dPres_drho_e is returning a negative value and I'm not quite sure why.

More specifically, the reference pressure in the beginning of the algorithm is of order -1e-13. Something is very weird here maybe in how the EOS parameters are being set up.

I'm converting back to draft. I'm encountering a situation where dPres_drho_e is returning a negative value and I'm not quite sure why.

More specifically, the reference pressure in the beginning of the algorithm is of order -1e-13. Something is very weird here maybe in how the EOS parameters are being set up.

I figured out the issue... it was a unit conversion issue with the input specification. I think this MR is fine.

@chadmeyer please take a look

Good catch! I do think we should try to limit root finding generically in equations of state. However, it probably suggests each EOS should have additional introspection.

I will observe here that RE of PT is generally going to be a root find for most EOSs, and isn't meant to be called much outside of certain special cases (i.e. not in an inner loop outside of initialization/remapping of an EOS).

I think this seems like a reasonable change, given the issue seen. Is there a more thermodynamically consistent form of the Gruneisen we could implement at some point?

@chadmeyer , we actually already have one that @aematts provided in https://github.com/lanl/singularity-eos/blob/main/singularity-eos/eos/eos_mgusup.hpp (https://lanl.github.io/singularity-eos/main/src/models.html#mie-gruneisen-linear-u-s-u-p-eos). The only downside of that EOS is that it is limited to $b = 0, S_2 = 0, S_3 = 0$, referencing the parameters in this EOS.
Ann has also provided a power law Mie-Gruneisen (https://lanl.github.io/singularity-eos/main/src/models.html#mie-gruneisen-power-expansion-eos) that offers more flexibility in the shape of the Hugoniot reference curve, but could not use the Steinberg parameters directly.
More generally though, I think the only way to make this EOS thermodynamically consistent is to numerically integrate a temperature reference curve (i.e. the temperature along the Hugoniot reference curve). You could imagine doing this a priori and then either fitting it to a function or tabulating it (and maybe storing it in a spiner table). You could also do this in-line, but that would probably be expensive.

It's worth pointing out that this Gruneisen is specifically in the code because it's not thermodynamically consistent but it is consistent with several host code implementations.

I think this seems like a reasonable change, given the issue seen. Is there a more thermodynamically consistent form of the Gruneisen we could implement at some point?

@chadmeyer , we actually already have one that @aematts provided in eos_mgusup.hpp (documentation). The only downside of that EOS is that it is limited to b = 0 , S 2 = 0 , S 3 = 0 , referencing the parameters in this EOS.
Ann has also provided a power law Mie-Gruneisen that offers more flexibility in the shape of the Hugoniot reference curve, but could not use the Steinberg parameters directly. I think this version also has restrictions on the functional form of the Gruneisen parameter, possibly limiting accuracy off of the principal Hugoniot (re-shocks or rarefactions).
More generally though, I think the only way to make this EOS thermodynamically consistent is to numerically integrate a temperature reference curve (i.e. the temperature along the Hugoniot reference curve). You could imagine doing this a priori and then either fitting it to a function or tabulating it (and maybe storing it in a spiner table). You could also do this in-line, but that would probably be expensive.

@jhp-lanl , @chadmeyer , @Yurlungur : It is true that all three of the EOS's I have implemented uses $\Gamma \rho = const$, that is $b_0=0$. It is mainly because then it is fairly easy to derive an entropy for a MG type EOS. I can think about how to modify my derivation to use a Gruneisen type $\Gamma$ relation.
I also want to mention the Extended Vinet EOS that is based on an isotherm (any temperature) and might be better for off-Hugoniot cases. It is a full EOS and not the cold curve form that is commonly named Vinet at LANL.

One last edge case I found: if the reference temperature is absurd (i.e. a unit conversion issue), it's possible for the maximum pressure to occur at a density less than the reference density. In this case, all compression states are unstable, which is bogus. I added an error to help with this case.
Will merge when tests pass.


Reduce memory usage and increase performance for convolution on iOS

We've had lots of problems with large convolutions hitting memory limits on iOS. This new implementation of the operator breaks the work into chunks so we never use more than 16 MB, and uses Apple's Accelerate framework to optimize the matrix multiplication.
Testing shows that it's between 5% to 10% faster than the existing implementation on various models, and keeps memory usage to a minimum.

For quantized models will be used gemmlowp also on IOS? /cc @wangyida

@bhack gemmlowp can be used on iOS, though we haven't investigated optimizing it for those devices in particular so I expect we'll need to do more work there. This is primarily a fix for memory issues when running float models.

Jenkins, test this please.

Could be interesting to benchmark this against BNNS.

@bhack I can apply gemm on FC layer in tiny-dnn of IOS platform now, some memory issue seems related to the batch size and network structure rather than the parametric model itself.

Looks good to me (my latest round of comments were minor)...

Once the tests have passed, could the admins merge this since we have LGTMs?


Create kragers.c

Greetings of the day @hanshulll,
Please review the pr. I've created c implementation of kragere's algorithm for finding min cut

@shikharvashistha
Don't worry it will be approved soon as you may know their are a lot PR's right now and also it's a one month program so don't worry it will be approved as soon as possible also I can't merge all PR's blindly as it may result in the repo being excluded from the program like many other repo so be patient full you PR will be accepted soon. Happy Hactoberfest

@shikharvashistha Don't worry it will be approved soon as you may know their are a lot PR's right now and also it's a one month program so don't worry it will be approved as soon as possible also I can't merge all PR's blindly as it may result in the repo being excluded from the program like many other repo so be patient full you PR will be accepted soon. Happy Hactoberfest

Great


test ambient with kubernetes netpol

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
https://github.com/istio/istio/issues/49277 was found running a new proposed integ test in an environment with Cilium CNI installed instead of the KinD default. Ideally we should add integ tests against network policy configurations we test.
https://github.com/istio/istio/pull/49255 seeks to lay groundwork for an integ platfrom based on KinD+Calico. Cilium would also be plausible for CI.
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[X] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

I'd like to work on this issue. Can someone assign me?

The results of the discussions ended up that accepting a firm dependency on a cni that enforces net pol for basic testing was undesirable.

okay, but we still need to accept a firm dependency on Cilium for basic CNI conformance testing generally with Cilium.
If we wanna do that but not test netpol, that's fine, it doesn't really change the need.

I'll just create a new issue with a different title.

@danehans I opened https://github.com/istio/istio/issues/51467 and put you on that one - it's a bit more targeted.
Closing this one.


Translated Auth Flow Help Text for Cell Field

@marahml suggested additional help text for MX + BR cell fields, because they won't immediately receive a welcome SMS.
@mikefantini and I thought about adding help text to the translation, but we're not sure how difficult this is knowing that "cell" pops up in numerous.
Mentioned briefly to @weerd, but maybe @sbsmith86 you have some ideas?

Per our convo, here's a copy suggestion:
"We're working on getting texting up in your country. When we do, we'll send volunteer opportunities right to your phone!"

@julielorch yea we can add help text to that field based on country code on the registration form. Where else does this field pop up?

@marahml i'm a bit concerned about how long that copy is given that it needs to fit on this form (where optional is) especially on mobile devices.

@sbsmith86 i tried adding it in to see what it would look like through inspect element but couldn't figure out how to get the text to wrap. Would it be relatively quick for you to put this together in browser and put a screenshot in this issue? both on desktop and the mobile breakpoint?

Totally! Your call on how many characters would make most sense here. Here's a couple more options:

"We'll let you know as soon as DoSomething texts are available (country)!"
"We're working on getting texting up in (country)!" (<--- though, not 100% users will understand what that means)
"As soon as we have texting available in your country, we'll send volunteer opps to your phone!"
"You might not here from us asap, but we're working to be able to text our (country) members soon!"

@mikefantini I can do that, is there a particular one you want to test? It might take just a little bit longer to try out all of the options marah put together.

@mikefantini @julielorch @marahml I tried out a few options on mobile and desktop, take a look:
"We're working on getting texting up in your country. When we do, we'll send volunteer opportunities right to your phone!"

"We're working on getting texting up in (country)!"

"As soon as we have texting available in your country, we'll send volunteer opps to your phone!"

Huh I was totally thinking this would go next to the "(optional)" - which would have been really tight spacing - so I think this makes more sense. Thanks @sbsmith86!
@julielorch totally defer to you on your thoughts on this from a UX perspective.
Also @DFurnes want to make sure breaking up the form like this jives from a Forge perspective.

Yeah, totally cool with me. :+1: It might be worth considering using the Footnote pattern so it shows up as "subtext" more clearly when scanning?

Agree with @DFurnes to use Footnote, otherwise it's a bit bold.

@mikefantini @julielorch what text did you want?

"Get volunteer opportunities to your phone! Texting available in (country) soon"

Let's change "to" --> "on" so it makes sense without having to add in "sent" to your phone. So:
"Get volunteer opportunities on your phone! Texting available in (country) soon"

Closing per #5538


[Map] GHG installations are no longer showing up on the map

Describe the bug
Expect to see installations from the link on the map landing page. It's still working in prod, but not on autopush or dev
To Reproduce
URL:  https://autopush.datacommons.org/tools/map#%26sv%3DAnnual_Emissions_GreenhouseGas_NonBiogenic%26pc%3D0%26denom%3DCount_Person%26pd%3DgeoId%2F48%26ept%3DCounty%26ppt%3DEpaReportingFacility%26mp%3D1
Screenshots
Prod:

Autopush:

Additional info:
There doesn't seem to be API errors, and the request for EpaReportingFacility is still being made with results returned.

Relatedly, glacier's are also missing (but this is also missing in prod)
https://datacommons.org/tools/map#%26sv%3DDifferenceRelativeToBaseDate2006_Max_Temperature_RCP45%26pc%3D0%26denom%3DCount_Person%26mapsv%3DSurfaceArea%26pd%3Dasia%26ept%3DIPCCPlace_50%26ppt%3DGlacier%26mp%3D1
https://datacommons.org/api/choropleth/map-points?placeDcid=asia&placeType=Glacier  returns {}

Describe the bug
Expect to see installations from the link on the map landing page. It's still working in prod, but not on autopush or dev
To Reproduce
URL: https://autopush.datacommons.org/tools/map#%26sv%3DAnnual_Emissions_GreenhouseGas_NonBiogenic%26pc%3D0%26denom%3DCount_Person%26pd%3DgeoId%2F48%26ept%3DCounty%26ppt%3DEpaReportingFacility%26mp%3D1
Screenshots
Prod:
Autopush:
Additional info:
There doesn't seem to be API errors, and the request for EpaReportingFacility is still being made with results returned.

I think this actually is an API error. The API for getting the data values for the installations is returning empty for both dev and autopush, but has values for prod. This is true for both the flask endpoints and the Mixer endpoints.
The flask endpoint: https://autopush.datacommons.org/api/stats/within-place?parent_place=geoId/48&child_type=EpaReportingFacility&stat_vars=Annual_Emissions_GreenhouseGas_NonBiogenic
The Mixer endpoint: https://autopush.api.datacommons.org/stat/set/within-place?parentPlace=geoId%2F48&childType=EpaReportingFacility&statVars=Annual_Emissions_GreenhouseGas_NonBiogenic

Ah, yes - we have not pushed out Glaciers, which is why that is missing from prod.
I'll move this issue over to the mixer repo

argh i can't transfer this issue to mixer.
so it seems it's not aggregating up to state / country, and this is happening on autopush too.
Both work
dev - zip level
autopush - zip level
dev - county level
autopush - county level
Both doesn't work
dev - state level
autopush - state level
Note: this works in prod / staging

Relatedly, glacier's are also missing (but this is missing in prod, working in dev) https://datacommons.org/tools/map#%26sv%3DDifferenceRelativeToBaseDate2006_Max_Temperature_RCP45%26pc%3D0%26denom%3DCount_Person%26mapsv%3DSurfaceArea%26pd%3Dasia%26ept%3DIPCCPlace_50%26ppt%3DGlacier%26mp%3D1
https://datacommons.org/api/choropleth/map-points?placeDcid=asia&placeType=Glacier returns {}
That map doesn't load on autopush because this is returning 500: https://autopush.datacommons.org/api/stats/set/series/within-place?parent_place=asia&child_type=IPCCPlace_50&stat_vars=Count_Person
This is pretty strange behavior..

Glaciers are not pushed to prod yet, but there is something strange happening with the API causing the map to not load on autopush due to api/stats/set/series/within-place returning 500.
It looks like the /stat/set/series/within-place endpoint on autopush (both dev and prod are fine though) is complaining about missing places argument although that is not an argument for the endpoint:
autopush
dev
prod

Relatedly, glacier's are also missing (but this is missing in prod, working in dev) https://datacommons.org/tools/map#%26sv%3DDifferenceRelativeToBaseDate2006_Max_Temperature_RCP45%26pc%3D0%26denom%3DCount_Person%26mapsv%3DSurfaceArea%26pd%3Dasia%26ept%3DIPCCPlace_50%26ppt%3DGlacier%26mp%3D1
https://datacommons.org/api/choropleth/map-points?placeDcid=asia&placeType=Glacier returns {}
That map doesn't load on autopush because this is returning 500: https://autopush.datacommons.org/api/stats/set/series/within-place?parent_place=asia&child_type=IPCCPlace_50&stat_vars=Count_Person
This is pretty strange behavior..

It looks like the map not being returned on autopush due to api/stats/set/series/within-place might be a data/API problem. It looks like on autopush (dev and prod are fine), api/stats/set/series/within-place is getting an empty list when calling GetChildPlaces for IPCC50 in asia

Thanks for the investigation. I sent a fix in https://github.com/datacommonsorg/mixer/pull/754. The issue is due to a partial cleanup of PlacesIn API and a non-merging version is used now.

Fixed in autopush now. I will push to dev soon.

thanks for the quick fix!


Support a custom method execution via Mapper interface

Hi MyBatis developers and users,
I've implemented mechanism for adding custom implementation method.
I use a Mapper interface of MyBatis to implement a Repository of DDD design.
Hence; i hope to add mechanism for adding custom method in Mapper method.
For example, i plan to add following custom method using this mechanism.

Fetch page object which contain total count and page content.
Save a Entity with hide actual operation(insert or update)
Save with entity and related entity at same time.
etc ...

My prototype is below:
For details, refer to javadoc of @CustomMethod.
Note that English might be wrong because it is not native language.
How to define :
public interface PersonRepository {

    Person findOne(int id); // Normal mapper method

    @CustomMethod // Mark a custom method using annotation
    Page<Person> findPage(Pageable pageable);

    @CustomMethod
    Person save(Person person);

}

How to implement a custom method :
public class PersonRepositoryImpl {
    public Page<Person> findPage(SqlSession sqlSession, Pageable pageable) {
        // total count and page content using SqlSession APIs or Mapper interface
        // ...
        return new PageImpl(content, pageable, totalCount);
    }

    @CustomMethod
    public Person save(SqlSession sqlSession, Person person) {
        // insert or update suing SqlSession APIs or Mapper interface
        // ...
        return person;
    }

}

Please give me free opinions.
Thanks.

I've fixed errors for compatibility with Java 6.

Here is a common Mapper: https://github.com/abel533/Mapper
You just need to extends the base Mapper, then you can use all the methods in the base Mapper.

There was a proposal fron Franta (see him in the contributors list) to be able to load classes as mappers. So you can code methods if you want and let mybatis add the implementation for others (using cglib).
The feature is cool indeed but in my opinion it is good to have cool features but it is also good to have not too many of them :) so we do not get too far from simplicity.
I would say that you can achieve this with common java coding. You will indeed code a bit more but you won't need to learn about a new specific MyBatis feature.
So in my opinion, the feature is not really needed.
As always kazuki, sorry for my reply but a big thanks for your contributions, which are great (including this one).


HELP

can someone make a video on how to run it on windows?

Setup WSL https://www.youtube.com/results?search_query=setting+up+wsl+on+windows+10
Install Ubuntu inside Windows https://www.youtube.com/results?search_query=ubuntu+wsl
Make sure you know how to access the windows drives inside WSL https://www.youtube.com/results?search_query=How+to+access+external+drives+from+WSL
Setup cmake https://www.youtube.com/results?search_query=ubuntu+setup+cmake
Setup git https://www.youtube.com/results?search_query=ubuntu+install+git
Inside WSL:
git clone https://github.com/madMAx43v3r/chia-plotter
cd chia-plotter
git submodule update --init
./make_devel.sh
export POOL_KEY="ac8e049..."
export FARMER_KEY="85b956c22..."
export TMP_DIR="/mnt/d/"
export WINDOWS_USERNAME="username"
export THREADS="$(expr $(nproc) / 2)" # this detects nr of threads automatically. You can simply use 'export THREADS="4"' instead
./chia_plot $POOL_KEY $FARMER_KEY $TMP_DIR $TMP_DIR $THREADS 7 2>&1 | tee /mnt/c/Users/$WINDOWS_USERNAME/.chia/mainnet/plotter/chia-plotter-$(uuid).log

Both WSL and docker options are quick to setup and work out of the box.
BUT the performance will be VERY LOW because of the way the filesystem will be accessed.
The sole objective of this project is to be fast, so these options are NO GO.
When it comes to msys2/mingw64, I'll be happy if you could share a working solution.
I'm struggling making both the compilation and execution work to the final plot.

@galmiza It is still saturating my SATA SSD.
WSL works reasonably fast enough for SATA SSDs when the ubuntu installation is excluded from Windows Defender and Realtime Protection is off.
Let me see if I can figure out how to compile this with msys2/mingw64. Someone has already done it, but did not share how (yet)

no luck for me also did not work for me. someone should create a windows version, will make our life easier

when the ubuntu installation is excluded from Windows Defender and Realtime Protection is off.

Can you elaborate how to do it please? Thanks you.

when the ubuntu installation is excluded from Windows Defender and Realtime Protection is off.

Can you elaborate how to do it please? Thanks you.
https://medium.com/@leandrocrs/speeding-up-wsl-i-o-up-than-5x-fast-saving-a-lot-of-battery-life-cpu-usage-c3537dd03c74

someone just released a windows version #74. can you give me the steps to set it up?

someone just released a windows version issue74. can you give me the steps to set it up?

You extract the contents of plotter. Navigate to the directory of the file using powershell, then run the chia_plotter.exe with the options. If you run it without anything it will pop up with the options that need to be added.


Add Entity.hurt(object, object);

if its implemented, a custom hurt AI should work better, right now ill investigate how to add that function from C++

it should be object1->doHurtTarget(object2);
from Mob::doHurtTarget(Entity*);

Java_net_zhuoweizhang_mcpelauncher_ScriptManager_nativeHurtTo, u found this :D @Khristian-kun.

gotta love kill aura -

umm no


Where is or How to generate the annotation file: nuscenes_infos_train_mono3d.coco.json in FCOS3D ?

I am training fcos3d and the config file indicates the annotation file path is data/nuscenes/nuscenes_infos_train_mono3d.coco.json, but I do not find it? Is this needed to be generated first by myself(if so, how?) or is it somewhere else? Thanks.

you can generate json file by running nuscenes_converter.py in mmdetection3d/tools/data_converter folder

you can generate json file by running nuscenes_converter.py in mmdetection3d/tools/data_converter folder
OK, thank you for replying


Move 'Jsonable' to core-interfaces

Rationale: Jsonable type has generic utility outside of datastores.  'core-interfaces' exists at the right layer (zero-dependencies) and is already included by almost all current users.  (I think there was one exception.)

is it needed on the server? thats really the only things that should be is common

breaking changes should be done in the next branch


Liveness and Readiness Probe Failing with Meilisearch v.0.17.0 on production

With Meilisearch v.0.17.0, when production is enabled, the frontend is disabled.
The readiness and liveness Probe in Kubernetess will fail, because it is a httpGet on the frontend

I may have found a Solution.
Using           tcpSocket instead as httpGet in the probes in Statefulset.
Gonna do a PR tomorrow

Hi @HelmutLety
Well spotted, you are absolutely right! An httpGet on / now returns a 404, which makes the livenessProbe broken. Thanks for this feedback and will be glad to check your PR anytime, please ping me :)

We should be using the get health route IMO


Include contact information in the footer (discuss with Ashok)

[ ] Include contact information in the footer (discuss with Ashok)

@jayasanka-sack @anjula-sack @RandilaP


include all majors

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.026%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
426

Relevant Lines:
468

💛  - Coveralls


rework CRYPTO_POLICY check to work with fedora

since fedora uses different version scheme it was not covered by previous
check. Also add checks for ssh client, to see if we successfully
override CRYPTO_POLICY there
Signed-off-by: Martin Schurz<EMAIL_ADDRESS>

Do you consider that can be merged ?

I think so, yes.
I ran this locally on all kitchen hosts in https://github.com/dev-sec/ansible-ssh-hardening/blob/master/.kitchen.yml and it didn't generate unexpected errors.
I'm currently preparing another update on ansibe-ssh-hardening to solve the fedora setup correctly. So there would be currently one expected error, since the ansible role doesn't handle fedora the right way.
Do you have any improvements or recommendations regarding the code?

No, it seems ok for me. Do you have merge permissions or do you want me to do it ?


Cant open Login page when opend with testcafe

What is your Test Scenario?
If i open the Login page by url, everthing works fine. If testcafe opens the page there is an error and it does not load.
What is the current behavior?
when the test starts the opened window shows just loading spinner and the console output says :
:55579/h5ycmw2PF!s!utf-8/http://localhost:4200/vendor.js:177380 Uncaught SyntaxError: Unexpected token ';'
What is the Expected behavior?
no error and login page loads
What is your web application and your TestCafe test code?
its an angular SPA
test.ts

fixture('password-module')
    .disablePageCaching
    .page(baseUrl)

// Single test to check login functionality
test('login', async t => {
    const getLocation = ClientFunction(() => document.location.href)
    await t
    .useRole(roles.admin.role)
     .expect(getLocation()).contains('password')
})

roles.ts
import { Role, Selector } from 'testcafe'
import { btn } from './patterns'

const isProd = false

export const baseUrl =  'http://localhost:4200'
export const dBInput = Selector('input').withAttribute('name', 'database')
export const nameInput = Selector('input').withAttribute('name', 'username')
export const passwdInput = Selector('input').withAttribute('name', 'password')

export const roles = {

  admin: {
    role: Role(baseUrl + '/authentication/login', async t => {
      await t
        .maximizeWindow()
        .typeText(dBInput, 'Test')
        .typeText(nameInput, 'TestUser')
        .typeText(passwdInput, 'Testpasswd', { replace: true })
        .click(await btn('Login'))
        .wait(10000)
    }, { preserveUrl: true }
    )
  }
}

Your website URL (or attach your complete example):

Your complete test code (or attach your test files):

Your complete configuration file (if any):

Your complete test report:

Running tests in:
 - Chrome 84.0.4147.89 / Windows 10

 password-module
 × login
   1) - Error in Role initializer -
      A JavaScript error occurred on "http://localhost:4200/".
      Repeat test actions in the browser and check the console for errors.
      If you see this error, it means that the tested website caused it. You can fix it or disable tracking JavaScript errors in TestCafe. To do the latter, enable the "--skip-js-errors" option.
      If this error does not occur, please write a new issue at:
      "https://github.com/DevExpress/testcafe/issues/new?template=bug-report.md".

      JavaScript error details:
      SyntaxError: Unexpected token ';'

      Browser: Chrome 84.0.4147.89 / Windows 10

Screenshots:

Steps to Reproduce:

unfortunately is it not possible for you to reproduce :( I'll give my best to get you all the infos you need
the problem is in these lines:

childView.rootNodes.forEach((element) => {
--
  | if (element.nodeType === 1) {
  | this.renderer.addClass(element, DX_TEMPLATE_WRAPPER_CLASS);
  | }
  | Object(devextreme_events__WEBPACK_IMPORTED_MODULE_6__["one"])(element, 'dxremove', ( _hh$temp195 , params) => {
  | var; if (!params \|\| !params._angularIntegration) {
  | childView.destroy();
  | }
  | });
  | });
  | return childView.rootNodes;

var; - here is some name missing i guess
Your Environment details:

testcafe version:    1.8.8
node.js version:     v14.2.0
command-line arguments: chrome test,ts
browser name and version: Chrome 84.0.4147.89
platform and version:   Windows 10
other:

@nao3301
Hello,
Yes, it looks like we have a script processing issue here:
{ var;

Could you please share this unprocessed block of code with us? You can get it from your web application without TestCafe. I assume you will find the required code in the childView.rootNodes.forEach callback function.


Make elyra work on kubeflow notebooks inside opendatahub/openshift

Describe the issue
Elyra currently doesn't run on  kubeflow notebooks inside opendatahub/openshift
To Reproduce
Steps to reproduce the behavior:

Provision OpenShift in IBM Cloud
Install OpenDataHub Kubeflow
Start notebook using elyra/kf-notebook:3.7.0 (custom image)
find appropriate pod and get logs

Screenshots or log output
Expected behavior
Elyra runs properly (or at least gives same error as described here)
Deployment information

openshift:4.8.31_1546 on IBM Cloud
ODH operator 1.1.2 stable
Kubeflow version: build version dev_local (version number can be found at the bottom left corner of the Kubeflow dashboard):
kfctl version: (use kfctl version):
Kubernetes platform: openshift
Kubernetes version: openshift:4.8.31_1546
Elyra 3.7.0 (see image above)

@akchinSTC as discussed, I've used this kfdef

Container images for Kubeflow Notebook servers are now available for ODH. See https://github.com/elyra-ai/elyra/discussions/3006 for details.


Allow extending the schema

Hi,
I'm building RaspberryPI3 images with linuxkit through linuxkit-server.
To make building the sdcard as easy as possible, I added to linuxkit-server capability to request the output in img-file what users can easily flash to sd-card for example with dd.
I would like to build support for defining disk partitions in the linuxkit yaml-file, but cannot because linuxkit build fails because it doesn't allow any extra fields to exist in the yaml-file.
My options are:

Change moby/tool schema to allow extra fields (change here additionalProperties=true)
unmarshal yaml, drop extra fields, marshal back to []byte and call Moby
Something else?

It's quite common for example in JSON APIs to ignore extra fields, so maybe also, in this case, it make sense. What do you think? It would allow the community to build extra features on top of Moby/Linuxkit what is not suitable to be merged into the core.

We originally added validation as ignoring typo fields is annoying and hard
to debug. I think this still maybe an issue but we could add some options
eg --allow-extra or ignore fields that match a pattern like x-* headers.
On Sat, 24 Mar 2018, 06:00 Erno Aapa<EMAIL_ADDRESS>wrote:

Hi,
I'm building RaspberryPI3 images with linuxkit
https://github.com/linuxkit/linuxkit through linuxkit-server
https://github.com/ernoaapa/linuxkit-server.
To make building the sdcard as easy as possible, I added to
linuxkit-server capability to request the output in img-file what users can
easily flash to sd-card for example with dd.
I would like to build support for defining disk partitions in the linuxkit
yaml-file, but cannot because linuxkit build fails because it doesn't allow
any extra fields to exist in the yaml-file.
My options are:

Change moby/tool schema to allow extra fields (change here
https://github.com/moby/tool/blob/master/src/moby/schema.go#L7
additionalProperties=true)
unmarshal yaml, drop extra fields, marshal back to []byte and call
Moby
Something else?

It's quite common for example in JSON APIs to ignore extra fields, so
maybe also, in this case, it make sense. What do you think? It would allow
the community to build extra features on top of Moby/Linuxkit what is not
suitable to be merged into the core.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/moby/tool/issues/209, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPD158AierMXozw1PkBmxNsMq8NKCks5thm2agaJpZM4S5ydf
.

Validation sounds like a good reason and because of that, made me think that maybe it's good to keep it as it is. I just need to find a way, how can I drop the extra property before I pass the yaml bytes to the moby.Build.


Setting Min/Max on Range Slider Arrows

Problem Description
Hi All,
I'm creating a range slider in a module with a max range of 23. The slider works correctly but there isn't a max on the arrows. This, incorrectly allows the user to select a range above the max. Am I missing a setting? Thank you.
Code Used
'dwot_monday_open_hours' => array(
'label'           => esc_html__( 'Opening Time: Hour', 'dwot-opening_times' ),
'type'            => 'range',
'range_settings' => array(
'min' => '00',
'max' => '23',
'step' => '1',
),
'fixed_unit'      => ':00',
'default' => '9:00',
'option_category' => 'basic_option',
'toggle_slug'     => 'main_content',
'show_if' => array(
'dwot_monday_switch' => 'on',
),
),
Screenshot

have you tried 'fixed_range' => true ?

Thanks for the reply Jan. I've tried that, unfortunately, it didn't work.

I guess thats a bug then. The workaround would be to simply do a min/max check in your own code to force the number into that range.

Thanks Jan. I'll try to work out how to do that. Hopefully ET will pick it up!

In php you would do something like:
$my_value = min(23, max(0, $my_value));
In JS it would be:
my_value = Math.min(23, Math.max(0, $my_value));

I'll have a play around with those. Thanks very much.


Remove track! from the global scope

Vanity has overall been a boon. Thank you for your work on it.
One request would be to remove track! from the global scope. Vanity.track! would avoid polluting the global namespace and would also be easier to read months down the line or by someone unfamiliar with Vanity in the project.

Thanks @daniel-nelson, there's definitely a lot more work to be done!
I tend to agree. I think a better approach is to make it available on the Vanity object and also make it includable in the ApplicationController so it's easily accessible in controllers. Thoughts?

I don't see the benefit of a global track! method. Analytics track, too, so it isn't clear what track! belongs to/does. But as long as it wasn't exposed in controllers by default, I would just avoid including and always access via the class method.


Add missing terms for secretion of bodily substances

A more complete set of. body fluid terms are defined in Uberon:

Note - it makes sense for at least some of these to be under 'secretion by cell'.

CC @caroline-99

Hi @Caroline-99
To create new terms, we need PMIDs in definition cross references and genes that will be annotated to these terms.
Thanks, Pascale

Hi @Caroline-99
To create new terms, we need PMIDs in definition cross references and genes that will be annotated to these terms.
Thanks, Pascale

Hi @pgaudet
Thank you for the information. I am currently working on gathering the references and genes for annotation and will update the ticket.
Best,
Caroline


Fixed retain cycles, version number bumped to 1.1.1

While I used your library I discovered that it has multiple retain cycles, which results in memory leak. I fixed them, and created a pull request. I hope you'll find it useful.

Hi, Thank you for your collaboration. You saw an important point. I'll merge and it will be ready for the next release.

Can you make a release?

Hi,
New version released.


Issue in an Add the dependency to gradle file:

i try below but not any of them is working:
implementation 'com.lapism.search:search: ASAP'
implementation 'com.lapism.search:search:1.0.0-alpha05'
implementation 'com.lapism.search:search:5.0.0-alpha05'
and many other but it not working. Please help

Yes, because I had to you close the old repository. But new dependency will be out asap. In the meantime you have to fork this repository.


Deepdive Version Check

Hi, I'm new to deepdive and I'm wondering how to check the version of my deepdive.
Thanks

deepdive version does the trick