minor copyedits from external contributor (original PR #744)

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/concepts/multiple-qubits.md
73
44
48
link
:white_check_mark:

articles/contributing/style-guide.md
81
57
2
link
:white_check_mark:

articles/install-guide/pyinstall.md
94
76
0
link
:white_check_mark:

articles/install-guide/qjupyter.md
87
73
0
link
:white_check_mark:

articles/install-guide/standalone.md
88
61
0
link
:white_check_mark:

articles/libraries/machine-learning/design.md
77
45
0
link
:white_check_mark:

articles/machines/resources-estimator.md
87
51
0
link
:white_check_mark:

articles/quickstart.md
86
62
3
link
:white_check_mark:

articles/quickstarts/qrng.md
88
75
0
link
:white_check_mark:

articles/user-guide/language/expressions.md
85
52
0
link
:white_check_mark:

articles/user-guide/language/types.md
81
52
0
link
:white_check_mark:

articles/user-guide/using-qsharp/operations-functions.md
81
57
4
link
:white_check_mark:

articles/user-guide/using-qsharp/testing-debugging.md
81
56
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

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

File
Status
Preview URL
Details

articles/concepts/multiple-qubits.md
:white_check_mark:Succeeded
View

articles/contributing/style-guide.md
:white_check_mark:Succeeded
View

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

articles/install-guide/standalone.md
:white_check_mark:Succeeded
View

articles/libraries/machine-learning/design.md
:white_check_mark:Succeeded
View

articles/machines/resources-estimator.md
:white_check_mark:Succeeded
View

articles/quickstart.md
:white_check_mark:Succeeded
View

articles/quickstarts/qrng.md
:white_check_mark:Succeeded
View

articles/user-guide/language/expressions.md
:white_check_mark:Succeeded
View

articles/user-guide/language/types.md
:white_check_mark:Succeeded
View

articles/user-guide/using-qsharp/operations-functions.md
:white_check_mark:Succeeded
View

articles/user-guide/using-qsharp/testing-debugging.md
:white_check_mark:Succeeded
View

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

Thank you @Bradben for addressing this. Will you be editing the api files to update the .yml files in #744 ?

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/concepts/multiple-qubits.md
73
44
48
link
:white_check_mark:

articles/contributing/style-guide.md
81
57
2
link
:white_check_mark:

articles/install-guide/pyinstall.md
94
76
0
link
:white_check_mark:

articles/install-guide/qjupyter.md
87
73
0
link
:white_check_mark:

articles/install-guide/standalone.md
88
61
0
link
:white_check_mark:

articles/libraries/machine-learning/design.md
77
45
0
link
:white_check_mark:

articles/machines/resources-estimator.md
87
51
0
link
:white_check_mark:

articles/quickstart.md
86
62
3
link
:white_check_mark:

articles/quickstarts/qrng.md
88
75
0
link
:white_check_mark:

articles/user-guide/language/expressions.md
85
52
0
link
:white_check_mark:

articles/user-guide/language/types.md
81
52
0
link
:white_check_mark:

articles/user-guide/using-qsharp/operations-functions.md
81
57
4
link
:white_check_mark:

articles/user-guide/using-qsharp/testing-debugging.md
81
56
0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 80b5ee1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/concepts/multiple-qubits.md
:white_check_mark:Succeeded
View

articles/contributing/style-guide.md
:white_check_mark:Succeeded
View

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

articles/install-guide/standalone.md
:white_check_mark:Succeeded
View

articles/libraries/machine-learning/design.md
:white_check_mark:Succeeded
View

articles/machines/resources-estimator.md
:white_check_mark:Succeeded
View

articles/quickstart.md
:white_check_mark:Succeeded
View

articles/quickstarts/qrng.md
:white_check_mark:Succeeded
View

articles/user-guide/language/expressions.md
:white_check_mark:Succeeded
View

articles/user-guide/language/types.md
:white_check_mark:Succeeded
View

articles/user-guide/using-qsharp/operations-functions.md
:white_check_mark:Succeeded
View

articles/user-guide/using-qsharp/testing-debugging.md
:white_check_mark:Succeeded
View

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

Yes, they are PRs in the qsharp-runtime and quantumLibraries repos.


Reedux removes any existing state

When adding a new reducer using reedux any existing state in the store is removed which wasn't the expectation as the readme states

requires no modifications to the existing project

If I create a store with state which is loaded when the app is loaded
const store = createStore(reducer, { app: { user: {}}};
state at this point is
{
    app: {
        user: {}
    }
}

then later load more of the site using react-loadable and reedux to attach the reducer all my existing state is removed e.g.
const addReducer = addStorePath('test', { x: 0, y: 0 });
I would expect state to look like:
{
    app: {
        user: {}
    },
    test: {
        x: 0,
        y: 0
    }
}

but instead the state is:
{
    test: {
        x: 0,
        y: 0
    }
}

If you add the following test to the reedux.spec.js you can see this in action as the test will fail:
 it('should preserve any existing store state', () => {
    const existingStore = createStore( s => s, { router: { location: { path: '/' } } });
    assert.equal(existingStore.getState().router.location.path, '/');

    const addStorePath = reedux(existingStore);
    const addReducer = addStorePath('test', { x: 0, y: 0 });

    assert.equal(existingStore.getState().router.location.path, '/');
    assert.equal(getState().test.x, 0);
  })

When looking into this I managed to keep the initial state but only by creating a store which had no state at all and then adding the initial state through reedux unfortunately this didn't work when trying to implement connected-react-router as internally its looking for state but not finding it.

Nope. It's a problem. Let me check it out.

I saw you updated the package and pulled it down to try but still had a problem.
I've attached a zip with minimal app that recreates the issue.
Appreciate you looked at this quickly as for the actual application we are using this is a blocking issue

Wanted to give a detailed explanation before announcing that it's fixed
I already opened a new bug to update the docs
This is purpose built for larger settings where it makes sense to start breaking things down, and creating a different store, managed exclusively through Reedux, made sense
I don't really fancy the idea of having two competing libraries managing the same store. It won't work
But if all you have redux and reedux, now you should be able to reedux(store, reducer) - in the app you sent, file src/store.js, last line
As for the for the forth route in the sample app, I'm afraid that's not very well set up
The quickest fix you can do is
const loadFourth = Loadable({
  loader: () => new Promise(r => r(() =>
test
)),
  loading() {
    return Loading
  }
});

Thanks for the feedback 👍


"Error: Error: Internal Server Error" when publishing to OpenVSX

publish-vscode-extension Version
Operating System

1.6.2
Ubuntu (github runner)

Description

I'm receiving an internal server error when I try to publish to the OpenVSX registry.
The error message is not very precise:
Error: Error: Internal Server Error
    at IncomingMessage.<anonymous> (/home/runner/work/_actions/HaaLeo/publish-vscode-extension/v1.6.2/dist/index.js:90479:40)
    at IncomingMessage.emit (node:events:531:35)
    at endReadableNT (node:internal/streams/readable:1696:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:[21](https://github.com/coq-community/vscoq/actions/runs/9644258882/job/26596304259#step:6:24))

Expected Behavior

Publish the extension to the openVSX registry
Actual Behavior

It is failing to do so and blocking my CI
https://github.com/coq-community/vscoq/actions/runs/9644674144/job/26597227513
Steps to Reproduce

Context

Ah sorry for the noise, seems to be unrelated to you ! See https://github.com/eclipse/openvsx/issues/945


An error occurred: Timeout

Hey so my discord-screenshot just stop working randomly i have no clue why, im getting "An error occurred: Timeout" in the discord

This happens when the screenshot upload takes more than 1 minute, which usually means that the player has slow internet or your server is having bandwidth problems. If you restart you server does it work normally?

If you mean normally like if you can connect throw and stuff like that yes that works everything works fine, just that im getting that timeout thing and my internet its not that bad so maybe a port blocking it?

I was referring to the script itself, but if the problem was a blocked port, it wouldn't work.

Yes, and i have no error comming from the scrip

All players or just you?

All

What host provider are you using?
Have you taken any screenshots before the error, if so, how many?
Sorry for making so many questions.

Im using zaphosting,
Yes i was enable to take a screenshots before, like 10+
You can ask how many you want bc i have tried everything i can understand why

When i join the server and took a screenshot this come up im not sure if it means anything

Gettings this problem now

Both warnings are from screenshot-basic and I can't do anything about them. If you want to see more details about the last one take a look at the issue #18.

Could you try restarting both resources (screenshot-basic first and discord-screenshot after) and test if they work?

Im just getting this now

Both warnings are from screenshot-basic and I can't do anything about them. If you want to see more details about the last one take a look at the issue #18.

^^

Are you using the screenshot-basic that cames with discord-screenshot?

Yes, but then im just getting the "(node:2052) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead."
I have check the links you gave me going to read more about when i have time


Feature/replace string ref with string view

Scope & Purpose
Replace arangodb::velocypack::StringRef with std::string_view. This will be necessary to merge https://github.com/arangodb/velocypack/pull/103 into arangodb/arangodb.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[x] :hammer: Refactoring/simplification
[ ] :book: CHANGELOG entry made

Testing & Verification

[ ] This change is a trivial rework / code cleanup without any test coverage.
[ ] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

@jsteemann
we have to ensure that we've switched to clang-12 for 3.10

Yes please! I came across that issue as well. Keep in mind that apple-clang still doesn't support it (in any version) and we use afaik apple-clang for mac builds. We could however, switch to real clang for mac 🤷🏽♂

We could however, switch to real clang for mac
Yes please! There are unfortunately also plenty of other features that apple-clang does not yet support...

IResearch part is LGTM


Update doc wording for IMPLICIT_COMMAND_DEPEDENCIES

Reword, remove a bit of redundancy by describing actions containing && in one place instead of repeatedly. Actually mention the case of setting the cvar to True.  Doc-only change.
Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated src/CHANGES.txt (and read the README.txt in that directory)
[X] I have updated the appropriate documentation

AAArgh.... I push a branch with one file changed, make a PR, and it says 12 files changed.  AFTER explicity doing  "git diff upstream/master currentbranch" and seeing it looked just like it should. Withdrawing again.


Include more icons

Is it possible to include more icons to be shown in status bar? Currently vscode supports only octoicons, which might not have all required icons.
Including https://github.com/vorillaz/devicons/ would be a good idea too.

I am in favour of more glyphs, tho unsure about the devicons cos the a missing license. Do you  have other suggestions?

Devicons iconic font is free to use and licensed under MIT.

Anyways there are plenty to choose from. Most popular is - http://fontawesome.io/icons/

@jrieken fyi we could update to a newer octicon version, I think they added some new icons and updated existing ones. we are on a quite old version.

@bpasero, Maybe then remove extra font versions (ttf, eot, woff) (we have one "browser" - Chrome)?

also removes
screen-normal
screen-full
Unsure how much of a breakage that is...

That sucks, but maybe we can detect this and offer some kind of migration. Or actually build our own fork of Octicon that still includes these removed icons to have a more complete set.

Yeah, we could bundle both and use the old if one of the three removed was used (have a special class name maybe)

Hey just stumbled upon this and don't necessarily mean to self promote but could nerd fonts be of help for this?

@aeschli Any API with which I can reuse your existing work on icon sets?

We should think of making an icon contribution point where an extension can publish a list of named icons. Such a named icon could then be used at places like the status line (maybe even in the completion proposals list?)
The icons can be defined in a font or be an SVG located in the extension.
The file icon theme could also use that contribution point. Right now the file icon theme definition file contains both: Icon definitions and file to icon associations.

Right now the file icon theme definition file contains both: Icon definitions and file to icon associations.

Would it make sense to break thing up into icon set and icon association with this milestone? Then we don't end up with two stories of defining icons that are reusable and those that aren't reusable?

@aeschli ping.

Moving to September as this grew larger but also into a more powerful future-proof story.

@aeschli

The icons can be defined in a font or be an SVG located in the extension.

Is this currently possible or are you planning on supporting it in a near future? Because many of the file icon theme repos I've seen contain icons in .png format.

File icon sets can also use png.

@aeschli Right. My question was more about font-faces. I know .png files can be used, I was asking about whether you could use something like FontAwesome or Devicons on a custom theme.

Proposal
I'd love to reference icons from an Icon Theme in the status bar. We could use the $(theme_id:icon_id) syntax, provided that : isn't a valid id character... which likely isn't true; but we can surely find some syntax. Both SVG/PNG and font glyphs should be supported.

theme_id matches the Icon Theme identifier contributes.iconThemes[].id
icon_id matches the one of that theme's icons in the iconDefinitions section

@aeschli @jrieken thoughts?

I like proposal and given my understand of @aeschli's work it should be doable like that

What if a theme id can not be found? Or does it mean that practice you can only reference the minimal and seti theme?

If both ids don't match anything, no icon will be shown. Possibly we can print a message to the console. It should be possible to reference any icon from any icon theme.

Wouldn't you rather want to specify an abstrace icon identifier ('folder', 'errorIcon') that then can be provided by any icon theme?
That would give us theming.

I see. How about:

$(theme:icon) picks a specific icon
$(:icon) picks a themeable icon from the active theme (file, folder, etc)

Yes, sounds good!

Since this issue is about using additional icon packs, i'm going to merge this into https://github.com/microsoft/vscode/issues/92791 where extensions can replace the default icons.


Docs App Configuration - Key Values & Health Indicator

Adds doc sections for Key Values: Placeholders, Json, Key Vault Reference and Health Indicator.

@mrm9084 , could you please merge this PR?

@mrm9084 , could you please merge this PR?

@AlexandraKemperMS has only been able to look at one of the files. The other file still needs to be reviewed.


Place reserved_keywords.txt not in root.

Do not place reserved_keywords.txt in the root
as this can give environment errors.
This was raised in: https://github.com/spulec/moto/pull/2885#discussion_r415150276
From testing in virtual_env it avoids placing the one in the root and there will only be one in the parsing module directory.
@kislyuk could you verify whether this works for your setup as well?

Coverage remained the same at 94.385% when pulling dd725bd60495651a839128b3369d3286232e9298 on pvbouwel:data_files_fix into 6a41573eb8aecd58a77151c6ff9f99e85f841df4 on spulec:master.

@pvbouwel no, I still get the following:
ERROR: Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/usr/local/moto'
Consider using the `--user` option or check the permissions.

I believe you need to add your file to https://github.com/spulec/moto/blob/master/MANIFEST.in instead of using data_files. See this comment: https://github.com/jupyter/jupyter-packaging/issues/11#issuecomment-295262893. (There are lots of scattered tidbits of information online but the gist of it is that data_files is for files that the package (for some extraordinary reason) wants to install system-wide, while package_data is for files attached to the package; to use package_data, you set include_package_data=True in setup.py and EITHER set package_data or add your files to MANIFEST.in; since moto already has a MANIFEST.in and sets include_package_data=True, the correct solution is to just add to MANIFEST.in).

@pvbouwel no, I still get the following:
ERROR: Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/usr/local/moto'
Consider using the `--user` option or check the permissions.

I believe you need to add your file to https://github.com/spulec/moto/blob/master/MANIFEST.in instead of using data_files. See this comment: jupyter/jupyter-packaging#11 (comment). (There are lots of scattered tidbits of information online but the gist of it is that data_files is for files that the package (for some extraordinary reason) wants to install system-wide, while package_data is for files attached to the package; to use package_data, you set include_package_data=True in setup.py and EITHER set package_data or add your files to MANIFEST.in; since moto already has a MANIFEST.in and sets include_package_data=True, the correct solution is to just add to MANIFEST.in).

Thanks for educating me. I didn't know about the difference but after the explanation it does make sense to prefer using package_data over data_files!  Removed the data_files entry and added it to the manifest.

@pvbouwel OK, I ran pip install git+https://github.com/pvbouwel/moto@0bd586e and it no longer triggers this error for me:
Successfully installed moto-1.3.15.dev634

I learned something new today as well! Thanks for the PR @pvbouwel, and thanks for helping out @kislyuk!


General improvements of the TimeVarying DCM planner

This PR introduces some general improvements for the TimeVarying DCM planner.

https://github.com/dic-iit/bipedal-locomotion-framework/commit/5da3568d21c6f91dcf098ba8e899e5728b3ec0a9 convert the Casadi::MX element in Casadi::SX object. Thanks to this modification the desired DCM trajectory is computed faster.
For instance, the test gives the following results:

Without the PR:******************************************************************************
This program contains Ipopt, a library for large-scale nonlinear optimization.
 Ipopt is released as open source code under the Eclipse Public License (EPL).
         For more information visit http://projects.coin-or.org/Ipopt
******************************************************************************

      solver  :   t_proc      (avg)   t_wall      (avg)    n_eval
       nlp_f  | 595.00us (  8.50us) 597.40us (  8.53us)        70
       nlp_g  |  56.22ms (803.11us)  22.99ms (328.42us)        70
    nlp_grad  |   1.91ms (  1.91ms) 727.76us (727.76us)         1
  nlp_grad_f  | 489.00us ( 19.56us) 493.73us ( 19.75us)        25
  nlp_hess_l  |  71.06ms (  3.38ms)  28.67ms (  1.37ms)        21
   nlp_jac_g  |  81.14ms (  3.01ms)  31.11ms (  1.15ms)        27
       total  | 501.12ms (501.12ms) 374.23ms (374.23ms)         1

With the PR:******************************************************************************
This program contains Ipopt, a library for large-scale nonlinear optimization.
 Ipopt is released as open source code under the Eclipse Public License (EPL).
         For more information visit http://projects.coin-or.org/Ipopt
******************************************************************************

      solver  :   t_proc      (avg)   t_wall      (avg)    n_eval
       nlp_f  | 815.00us ( 15.38us) 822.17us ( 15.51us)        53
       nlp_g  |   1.21ms ( 22.91us)   1.18ms ( 22.20us)        53
    nlp_grad  | 107.00us (107.00us) 106.85us (106.85us)         1
  nlp_grad_f  | 737.00us ( 32.04us) 738.33us ( 32.10us)        23
  nlp_hess_l  |   1.48ms ( 73.80us)   1.50ms ( 74.94us)        20
   nlp_jac_g  |   1.40ms ( 58.13us)   1.41ms ( 58.79us)        24
       total  | 276.92ms (276.92ms) 277.00ms (277.00ms)         1

https://github.com/dic-iit/bipedal-locomotion-framework/commit/393f668e6d47883ad50311de5260bd6d189efbb5 introduces the possibility to switch the linear solver used by the IPOPT.
https://github.com/dic-iit/bipedal-locomotion-framework/commit/f9a4cc397e63a82d5ad0dda99c81cf0af937ec5d improves the documentation

CI is failing on windows with this error that this not related with this PR
2020-10-03T02:13:45.5270628Z -- Acquiring MSYS2...
2020-10-03T02:13:45.5271845Z -- Downloading https://sourceforge.net/projects/msys2/files/Base/x86_64/msys2-base-x86_64-20190524.tar.xz/download...
2020-10-03T02:13:45.5273331Z -- Downloading http://repo.msys2.org/msys/x86_64/msys2-keyring-r21.b39fb11-1-any.pkg.tar.xz...
2020-10-03T02:13:45.5275261Z -- Downloading http://repo.msys2.org/msys/x86_64/msys2-keyring-r21.b39fb11-1-any.pkg.tar.xz... Failed. Status: 28;"Timeout was reached"
2020-10-03T02:13:45.5276616Z CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:176 (message):
2020-10-03T02:13:45.5277185Z
2020-10-03T02:13:45.5277545Z       Failed to download file.
2020-10-03T02:13:45.5278263Z       If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
2020-10-03T02:13:45.5282223Z       variables to "***your-proxy-ip-address:port/".
2020-10-03T02:13:45.5283011Z       Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues
2020-10-03T02:13:45.5283441Z
2020-10-03T02:13:45.5283740Z Call Stack (most recent call first):
2020-10-03T02:13:45.5284241Z   scripts/cmake/vcpkg_acquire_msys.cmake:86 (vcpkg_download_distfile)
2020-10-03T02:13:45.5284848Z   scripts/cmake/vcpkg_fixup_pkgconfig.cmake:281 (vcpkg_acquire_msys)
2020-10-03T02:13:45.5285431Z   ports/cppad/portfile.cmake:39 (vcpkg_fixup_pkgconfig)
2020-10-03T02:13:45.5285909Z   scripts/ports.cmake:79 (include)
2020-10-03T02:13:45.5286181Z
2020-10-03T02:13:45.5286311Z
2020-10-03T02:13:45.5286706Z Error: Building package cppad:x64-windows failed with: BUILD_FAILED
2020-10-03T02:13:45.5287284Z Please ensure you're using the latest portfiles with `.\vcpkg update`, then
2020-10-03T02:13:45.5287941Z submit an issue at https://github.com/Microsoft/vcpkg/issues including:
2020-10-03T02:13:45.5288478Z   Package: cppad:x64-windows
2020-10-03T02:13:45.5288819Z   Vcpkg version: 2020.06.15-nohash

CI is failing on windows with this error that this not related with this PR
2020-10-03T02:13:45.5270628Z -- Acquiring MSYS2...
2020-10-03T02:13:45.5271845Z -- Downloading https://sourceforge.net/projects/msys2/files/Base/x86_64/msys2-base-x86_64-20190524.tar.xz/download...
2020-10-03T02:13:45.5273331Z -- Downloading http://repo.msys2.org/msys/x86_64/msys2-keyring-r21.b39fb11-1-any.pkg.tar.xz...
2020-10-03T02:13:45.5275261Z -- Downloading http://repo.msys2.org/msys/x86_64/msys2-keyring-r21.b39fb11-1-any.pkg.tar.xz... Failed. Status: 28;"Timeout was reached"
2020-10-03T02:13:45.5276616Z CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:176 (message):
2020-10-03T02:13:45.5277185Z
2020-10-03T02:13:45.5277545Z       Failed to download file.
2020-10-03T02:13:45.5278263Z       If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
2020-10-03T02:13:45.5282223Z       variables to "***your-proxy-ip-address:port/".
2020-10-03T02:13:45.5283011Z       Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues
2020-10-03T02:13:45.5283441Z
2020-10-03T02:13:45.5283740Z Call Stack (most recent call first):
2020-10-03T02:13:45.5284241Z   scripts/cmake/vcpkg_acquire_msys.cmake:86 (vcpkg_download_distfile)
2020-10-03T02:13:45.5284848Z   scripts/cmake/vcpkg_fixup_pkgconfig.cmake:281 (vcpkg_acquire_msys)
2020-10-03T02:13:45.5285431Z   ports/cppad/portfile.cmake:39 (vcpkg_fixup_pkgconfig)
2020-10-03T02:13:45.5285909Z   scripts/ports.cmake:79 (include)
2020-10-03T02:13:45.5286181Z
2020-10-03T02:13:45.5286311Z
2020-10-03T02:13:45.5286706Z Error: Building package cppad:x64-windows failed with: BUILD_FAILED
2020-10-03T02:13:45.5287284Z Please ensure you're using the latest portfiles with `.\vcpkg update`, then
2020-10-03T02:13:45.5287941Z submit an issue at https://github.com/Microsoft/vcpkg/issues including:
2020-10-03T02:13:45.5288478Z   Package: cppad:x64-windows
2020-10-03T02:13:45.5288819Z   Vcpkg version: 2020.06.15-nohash

Associated issue: microsoft/vcpkg#13286
cc @traversaro

That error was fixed in https://github.com/microsoft/vcpkg/pull/13889 . I am generating a new archive in https://github.com/robotology/robotology-superbuild-dependencies-vcpkg, but indeed there are regressions. For the time being I suggest to bump the vcpkg tag to  76a7e9248fb3c57350b559966dcaa2d52a5e4458 as done in https://github.com/robotology/robotology-vcpkg-ports/pull/16 .

Thank you @traversaro. In https://github.com/dic-iit/bipedal-locomotion-framework/pull/127/commits/e82a5e32c43de688fddf7b4ae8c1a5a3bb3ca3df I bumped the vcpkg commit.  Let's see if it solves the problem.

e82a5e3 fixes the problem. Merging


Repo sync for protected CLA branch

The pull request is created from master637453168869015017 to master to fix git push error for protected CLA branch

@cmcclister : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cmcclister : Thanks for your contribution! The author(s) have been notified to review your proposed change.


CocoaPods could not find compatible versions for pod "Folly"

Hi guys, I have the problem trying to run the command "pod install". This is my issue. Anyone help me? Thanks a lots!

Try the code prompted by the pod.
pod update Folly --no-repo-update

Try the code prompted by the pod.
pod update Folly --no-repo-update
Thanks sir, worked!

You closed and reopened but based on your comment I assume this is resolved.

Try the code prompted by the pod.
pod update Folly --no-repo-update

Thanks Issue resolved!

after apply the code new error
'UMReactNativeAdapter/UMModuleRegistryAdapter.h' file not found


s3 multipart upload does not work properly

I was playing around and also have some difficulties when trying to make the multipart upload work
i get the following error
The XML you provided was not well-formed or did not validate against our published schema. (Service: S3, Status Code: 400, Request ID: 168ED5E4224995A4, Extended Request ID: null)

i can imaging something changed at aws sdk? the extended request id: null is probably giving it away, but im still very unfamiliar with the sdk, so i will have to dig a bit deeper to find out what's going on.

@wim82 i recently upgrader AWS SDK dependency on latest one, can you check with latest fs2-aws SNAPSHOT ?


Repeating events

This is the implementation I suggested in #6. I've made it a little more general than initial proposal.
- input_name: '...'
  remappings:
    BTN_EXTRA:
    - code: REL_WHEEL
      type: EV_REL        # Overrides received event type [optional]. Defaults to EV_KEY
      value: 1            # optional. Defaults to the value of received event.
      value: [1, ...]     # multiple values are supported; they will be applied in sequence.
      repeat: true/false  # optional, defaults to false
      count: xx           # number of repetitions (optional, defaults to 0).
                          # If count is 0 it will repeat until key/button is depressed which covers most use-cases I think
                          # If count > 0 it will repeat specified number of times
      rate: 0.1           # Repeat rate in seconds [optional, default:0.1]_

These changes should be backwards compatible with initial config format.

flaked :D

Great. Could you please squash the changes and provide a change description explaining the new features.

done

I expanded the change description and merged it manually. github isn't smart enough to detect that, so I have to close the pull request manually.
Thanks for contributing!


Improve the ui/ux

You might have noticed that the UI of the whole application kinda looks similar to Spotify, to the web developers, you can create a PR to change the UI of the whole application without changing its functionalities or adding any bug.

I have been using your project for a couple of months now and i came across this issue which I want to help solve by contributing in it. PLease assign this issue to me.

Assign this issue to me


Add support for techlef SpacingRange feature

The GF180MCU PDK has the SpacingRange features (?) in its techlef file. It would be good for the OpenROAD tooling to support this.
See report from @kareefardi @ https://github.com/RTimothyEdwards/open_pdks/issues/277

The related rule appears to be here:
LAYER Metal1
    # ...

    MINWIDTH 0.230 ;                   # Mn.1  (n=1)
    WIDTH 0.230 ;                      # Mn.1  (n=1)
    SPACING 0.230  ;                   # Mn.2a (n=1)
    SPACING 0.300 RANGE 10.005 999.00 ; # Mn.2b

    # ...
END Metal1

This will very rarely if ever impact DRT correctness because it only applies to metals with width > 10.005 um, which is way out of range of the 0.230 um default width.
That being said, since we support spacing tables, I don't think this should be too difficult to implement?

This is only going to be relevant for pdngen @arlpetergadfort

@maliberty the warning about not being supported is in DRT:
https://github.com/The-OpenROAD-Project/OpenROAD/blob/5140e61103f7c01ccad05718f746fc598c0581bb/src/drt/src/io/io.cpp#L1633-L1634

@maliberty if it's implemented in https://github.com/The-OpenROAD-Project/OpenROAD/blob/bbf00fa504cb847866c9f91d641a4a0ad53f45ed/src/odb/include/odb/db.h#L7032
PDNGEN would honor it.

I don't think this represents a real barrier to gf180 bring up.  PDN supports it and drt will only hit it with some extreme NDR of the sort we never use.  I'll leave it open but as a low priority.

As GF180 implemented with OpenROAD flow, is the issue still relevant?

We still don't support it in drt but I don't think this will ever get to the top of the priority list.


Sweep: Do not install latest npm in github workflow npm-build.yml

Do not install latest npm in github workflow npm-build.yml
https://github.com/deep-foundation/workflows/blob/d12184a1d834a379abd2d907e50e7c555887a66c/.github/workflows/npm-build.yml#L23

Sweep: retry

Sweep: retry

@FreePhoenix888 what is the problem with installation of latest npm? It usually fixes lots of problems, so it may safely stay.
I suggest to close this issue as not planned.


SINGA-12 Supprt Checkpoint and Restore

The checkpoint is done in the Worker class and controlled by two model configuration fields: checkpoint_after and checkpoint_frequency.
Only do checkpoint for the Params owning the param values from the first group. The name, version and values of one Param are dumped onto disk (the path is /checkpoint/step-worker.bin). It is possible that the snapshot is separated into multiple files because the neural net is partitioned into multi workers.
The checkpoint files can be used:

application 1: to restore (resume) the training by setting the command line argument -resume = true.
The Resume function of the Trainer will find the files for the latest snapshot and add them to the model.conf's checkpoint filed. It also set the model config's step field to the snapshot step (extracted from file name).

application 2: as the pre-training result of another model. Users have to config the new model's checkpoint field to add the paths of all checkpoint files of the same snapshot.

The Worker's InitLocalParam will init Param's from checkpoint files if available. Otherwise it randomly init them using user configured init method. It matches the Param objects based on name. If the Param is not configured with a name, NeuralNet class will automatically create one for it based on the name
of the layer to which the Param belongs. In this case,

for application 2, users have to configure the names of the new model params carefully to
match the names of params from the pre-trained model

for application 1, the worker-server topology cannot be changed.

Restore for params which are partitioned due to model partitioning is not supported. Because if the pre-training is done using 2 workers, while the new model is trained with 3 workers,
then the same original param is partitioned in different ways and hence cannot be matched.
TODO: update the sing-run.sh to include the command line argument for resume, as
./bin/singa-run.sh -conf=.. . -resume=...

For the  cifar10 example, if we configure the model.conf as:
...
checkpoint_after: 100
checkpoint_frequency: 300
...

Then after training for 700 steps, under cifar10/checkpoint folder, there would be two checkpoint files:
step400-worker0.bin
step700-worker0.bin

We can resume the training from the last checkpoint (i.e., step 700) by:
./bin/singa-run.sh -conf=examples/cifar10 -resume=true

We can also use the the snapshot from step 400 as the pre-trained model for a new model by configuring the model.conf of the new model as:
...
checkpoint : examples/cifar10/checkpoint/step400-worker0.bin
...

the launch command is the same as starting a new job
./bin/singa-run.sh -conf=...


Could it support 2 deployments?

I found an issue when I try to deploy two stacks in the same workflow (github action):
mkdir: cannot create directory '/github/home/.aws': File exists
-
    name: "Deploy analytics celery stack"
    uses<EMAIL_ADDRESS>    env:
      TEMPLATE: 'amazon-aws/cloudformation/prod-ecs-celery-analytics.yml'
      AWS_STACK_NAME: 'prod-ecs-celery-analytics'
      AWS_REGION: 'eu-west-1'
      AWS_ACCESS_KEY_ID: ${{secrets.AWS_ACCESS_KEY_ID}}
      AWS_SECRET_ACCESS_KEY: ${{secrets.AWS_SECRET_ACCESS_KEY}}
      AWS_DEPLOY_BUCKET: ${{secrets.AWS_CLOUDFORMATION_BUCKET}}
  -
    name: "Deploy async celery stack"
    uses<EMAIL_ADDRESS>    env:
      TEMPLATE: 'amazon-aws/cloudformation/prod-ecs-celery-async.yml'
      AWS_STACK_NAME: 'prod-ecs-celery-async'
      AWS_REGION: 'eu-west-1'
      AWS_ACCESS_KEY_ID: ${{secrets.AWS_ACCESS_KEY_ID}}
      AWS_SECRET_ACCESS_KEY: ${{secrets.AWS_SECRET_ACCESS_KEY}}
      AWS_DEPLOY_BUCKET: ${{secrets.AWS_CLOUDFORMATION_BUCKET}}

2 deployments is supported in 2 different jobs, not 2 steps.


Add Stella and Odysseus Planet Pack from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of Thomas988, to add Stella and Odysseus Planet Pack to CKAN.
Please direct questions about this pull request to Thomas988.

This depends on some arcane version of KopernicusTech called KopernicusCore, forum thread link on KS is a link to a single post in large forumthread (52 pages) which is now locked and as such prevents questions to clarify dependencies. Seems to depend on (at least) RSS, KittopiaTech and KopernicusCore.
Not enough information available to add to CKAN and we don't have KopernicusCore (probably won't since it is officially cancelled) indexed and as such I'm closing this.


EOFError not catched in Instaloader.check_if_committed()

For some reason I get the Could not find "window._sharedData" in html response error.
All packages are up to date:
Requirement already satisfied: instaloader in /usr/local/lib/python3.6/site-packages (4.2.10)
Requirement already satisfied: requests>=2.4 in /usr/lib/python3.6/site-packages (from instaloader) (2.14.2)
Requirement already satisfied: urllib3>=1.21.1 in /usr/lib/python3.6/site-packages (from requests>=2.4->instaloader) (1.25.1)

The log looks like this:
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
profile: JSON Query to profile/: Could not find "window._sharedData" in html response.
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
profile: JSON Query to profile/: Could not find "window._sharedData" in html response.
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
JSON Query to profile/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
profile: JSON Query to profile/: Could not find "window._sharedData" in html response.
Traceback (most recent call last):
  File "/usr/local/bin/instaloader", line 11, in <module>
    load_entry_point('instaloader==4.2.10', 'console_scripts', 'instaloader')()
  File "/usr/local/lib/python3.6/site-packages/instaloader/__main__.py", line 414, in main
    storyitem_filter_str=args.storyitem_filter)
  File "/usr/local/lib/python3.6/site-packages/instaloader/__main__.py", line 189, in _main
    download_stories, fast_update, post_filter, storyitem_filter)
  File "/usr/local/lib/python3.6/site-packages/instaloader/instaloader.py", line 1039, in download_profiles
    downloaded = self.download_post(post, target=profile_name)
  File "/usr/local/lib/python3.6/site-packages/instaloader/instaloader.py", line 462, in download_post
    self._committed = self.check_if_committed(filename)
  File "/usr/local/lib/python3.6/site-packages/instaloader/instaloader.py", line 1151, in check_if_committed
    load_structure_from_file(self.context, filename)
  File "/usr/local/lib/python3.6/site-packages/instaloader/structures.py", line 1094, in load_structure_from_file
    json_structure = json.load(fp)
  File "/usr/lib64/python3.6/json/__init__.py", line 296, in load
    return loads(fp.read(),
  File "/usr/lib64/python3.6/lzma.py", line 200, in read
    return self._buffer.read(size)
  File "/usr/lib64/python3.6/_compression.py", line 99, in read
    raise EOFError("Compressed file ended before the "
EOFError: Compressed file ended before the end-of-stream marker was reached

With the latest 4.3a1 I get the EOFError again.
Traceback (most recent call last):
File "/usr/local/bin/instaloader", line 11, in
load_entry_point('instaloader==4.3a1', 'console_scripts', 'instaloader')()
File "/usr/local/lib/python3.6/site-packages/instaloader/main.py", line 417, in main
storyitem_filter_str=args.storyitem_filter)
File "/usr/local/lib/python3.6/site-packages/instaloader/main.py", line 81, in _main
instaloader.load_session_from_file(username, sessionfile)
File "/usr/local/lib/python3.6/site-packages/instaloader/instaloader.py", line 426, in load_session_from_file
self.context.load_session_from_file(username, sessionfile)
File "/usr/local/lib/python3.6/site-packages/instaloader/instaloadercontext.py", line 177, in load_session_from_file
session.cookies = requests.utils.cookiejar_from_dict(pickle.load(sessionfile))
EOFError: Ran out of input


annoying issue : Uncaught (in promise): invalid link: TabsPage

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
ionic serve loading old version
every time it shows the error Uncaught (in promise): invalid link: TabsPage, i have to stop serve and rerun ionic serve, this is really annoying
What behavior are you expecting?
Build the latest chnages in pages, do not load old vesion of page
Steps to reproduce:

create ionic 3 tab project
ionic serve
sometime it shows this error when building changes

insert any relevant code between the above and below backticks

Which @ionic/app-scripts version are you using?
1.3.4

I just upgraded app-script to 1.3.7 , checking now

Please provide a set of actions to recreate this issue. I cannot recreate with the latest app-scripts.
Thanks,
Dan

I had this issue with<EMAIL_ADDRESS>and upgrading to<EMAIL_ADDRESS>resolved it.

Have issue intermittently too
@ionic/cli-utils  : 1.8.1
ionic (Ionic CLI) : 3.8.1
global packages:
Cordova CLI : 7.0.1

local packages:
@ionic/app-scripts : 2.1.3
Cordova Platforms  : none
Ionic Framework    : ionic-angular 3.6.0

System:
Android SDK Tools : 25.2.5
Node              : v8.2.1
npm               : 5.3.0
OS                : macOS Sierra
Xcode             : Xcode 8.3.3 Build version 8E3004b

i am sharing my experience as in my case i had declared class like this
@IonicPage()
@Component({
selector: 'page-login',
templateUrl: 'login.html',
})

export class logininfo {
 name: string;
}

export class loginPage {
  // code
}

as seen here the declared class is before the declaration of page class that's why it was saying invalid error, but after remiving the logininfo class the link was working


Add proper "requires" support

The modules api already supports requires it just needs to be implemented to auto import missing modules.

Added in d79d68ab50742cf8c4815eed79c064f5ebebdfd8


Fix build

/cc @myronmarston this is the hopefully final step in getting 1.9.2 working across the board, we also need to merge #1770 to get the main builds working, although that won't fix rspec-rails which is still segfaulting on 1.8.7

LGTM

Merging as 1.9.2 is fixed, leaving only 1.8.7


[Android]: Back Button doesn't work

Hi,
First of all I would like to thank you for this cool plugin.
I used it in my application and it saved me a huge amount of my work. I have noticed, that for some reason, when I clicked on the hardware "Back" button application doesn't react.
Thanks in advance,
Pavel

Correct cause if you are on the first page it closes the app instead of trouwens it to the background.
If you see the sourcecode of this plugin you see in the background-mode.js a line with:
// Override back button action to prevent being killed
document.addEventListener('backbutton', function () {}, false);
so the backbutton doesnt do anything anymore.
You need to make your custom backbutton function so when it is on the first page of your app it wont close your app but put it to background and to android main screen or something like that. You can comment this line in the javascript plugin code but remember if the app doenst got history pages it will close the app.

Hi I have the same problem with the back button. How to do enable the back button ?

@ljadrbln The plugin does not overwrite the back-button handler anymore.


Color Picker: behavior when multiple fields are selected

Bug Description
Originally reported by Sam:
When 2+ elements are selected, the color picker's input field contains the placeholder (Multiple). When you click on it, the color picker is opened.
Expected Behaviour

I expect to be able to type and not open the color picker. The rule is clicking on text should behave like an input field and clicking on the color swatch, opens the color picker

Steps to Reproduce

Select 2 or more text elements with different colors
Click on the (Multiple) placeholder in the color picker input field.

Screenshots

Additional Context

Plugin Version: master / 1.0.0-alpha.10
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Somewhat related to this is the label displayed for gradients in the same place.
@samitron7, what should happen when you click the label for "radial" or "linear" for a radial or linear gradient? Should clicking that label also switch to an input field or should it open the color picker (as it does now)?

It should open the color picker. The only time you get the input field is if you have a hex value there.

@barklund @bmattb FYI I've pulled this into 1.2 since the sprint backlog is empty.

Verified in QA

Verified in UAT


why compile_commands.json not generated?

Output of 'strings libarm_compute.so | grep arm_compute_version':
23.11
Platform:
Operating System:
Ubuntu

Problem description:
I use scons Werror=1 -j8 debug=0 asserts=1 neon=0 opencl=1 embed_kernels=1 os=linux arch=armv8a export_compile_commands=True, but compile_commands.json is not generated in build folder, Could someone help me ?

Add
export CMAKE_EXPORT_COMPILE_COMMANDS=1
Before compiling

Hi @Shipley1105
You can output the compile commands during the SCons build by appending export_compile_commands=1 build/compile_commands.json to your build command.
For example:
scons arch=armv8a os=linux neon=1 export_compile_commands=1 build/compile_commands.json
You also need to have SCons >= 4.0 installed and available in your environment.


Backbone with FPN on top of Swin Transformers

🚀 The feature
Adding a backbone with FPN on top of swin transformers similar to the already available resnet_fpn and mobilenet_fpn backbones (https://github.com/pytorch/vision/blob/main/torchvision/models/detection/backbone_utils.py). I do have a working version (need to be confirmed by the authors  if it is what was used in the paper for comparisons) in a separate branch. However, as per protocol, I am opening an issue first to see if it is a worthwhile addition.
Motivation, pitch
It lets us use swin transformer based backbones for the object detection frameworks (FasterRCNN, MaskRCNN..).
Alternatives
No response
Additional context
No response

Thanks for offering to help @vsuryamurthy. There are certainly lots of interesting newer backbones that can be combined with FasterRCNN/MaskRCNN. Because the number of such combinations can be easily explode, we currently offer variants that were included on the original paper. This is because the effort of configuring and training new variants is non trivial and can quickly add up. Nevertheless we acknowledge that offer new variants for good detection models can be useful to users and that's why we offer examples on our documentation on how to do this. I think what we can do is leave this issue open and if we decide to offer more variants, we can reach out to you for help. What do you think?

Yes understood and I completely agree with you. My only motivation was the fact that swin transformer is quite popular and has high score with MaskRCNN but I can imagine that being surpassed in a short duration and we are left with another state of the art. Lets do according to what you proposed. Thank you :)

I'm interested in this @vsuryamurthy, if you wouldn't mind sharing. I wrote an fpn neck on top of swin-t backbone in the manner outlined above as well. Things run fine, but when I create a MaskRCNN model using my swin-t backbone with FPN neck, I get some index out of range error when I try to run.


bpf: Add enable_local_redirect_policy config

Wrap the datapath code by the config when the feature is enabled.

/test


Want to make shin-chan anime with python

Description
It will be animation draw with the help of python we will create shin-chan character drawing

Tech Stack
Python
turtle module of python
I would like to make this project
I'm a GSSOC22 contributor
Reference
No response
Are you creating this issue for yourself?

[X] Yes
[ ] No

@manisha-27 The PR is made #292 . Kindly review and merge it.
Thank You


.gitignore update

My work on another issue resulted in accumulation of files that needn't be pushed.
May I fix this by updating the current .gitnore with this file?
https://github.com/github/gitignore/blob/main/Python.gitignore
Thank you!

@claytonjhamilton yes please go ahead, feel free to open a new PR. Thanks.


Triggering subsequent github actions without a PAT

I wanted to point out a workaround and ask if you see any downsides to using it for triggering subsequent github actions from release please without having to use a personal access token.
The referenced documentation from github (https://docs.github.com/en/actions/using-workflows/triggering-a-workflow#triggering-a-workflow-from-a-workflow) mentions that workflow_dispatch can be used to trigger subsequent actions.
So I've set up the following workflow for release please:
name: release-please

on:
  push:
    branches:
      - main

permissions:
  actions: write
  contents: write
  pull-requests: write

jobs:
  release-please:
    runs-on: ubuntu-latest
    steps:
      - uses: googleapis/release-please-action@v4
        id: release-please
        with:
          release-type: python
          token: ${{ secrets.GITHUB_TOKEN }}
      - name: Trigger Workflow
        if: steps.release-please.outputs.release_created == 'true'
        uses: actions/github-script@v7
        with:
          script: |
            github.rest.actions.createWorkflowDispatch({
              owner: context.repo.owner,
              repo: context.repo.repo,
              workflow_id: 'release.yml',
              ref: '${{ steps.release-please.outputs.tag_name }}',
            })

This will trigger my release.yml workflow when a release is created, and works like a release.published event would have. It requires an additional permission, actions: write and my release.yml must trigger on workflow dispatch.
Do you see any issues with using this workaround? For our use case using a PAT seems strange as all releases would be attributed to a person and we'd need to stay on top rolling it periodically.
Are there any other methods you recommend for handling this? Do you recommend using the probot instead for use cases where PAT does not fit?

While this is good for the release dispatch, no ci is run on the release pull request which in the end didn't work out for me so I think this suggestion is unlikely to work out for many people. I went with a token created for a github app


LabelValue undefine when use view type 'Tick'

Hai, I have an issue here about Tick type, when I use that type it is not allowed me to render my projects, and the console show me this message

please anyone here,

Thank you for your question. I suppose this is not current anymore but it seems like you did not fill the label property in descriptions, which is required.


IAs can delete a project

This introduces a new role for users that is a higher privilege than
the GDS Editor. Only IAs can delete projects and its dependent project
content items.
Trello

I've tested this locally, and it works for me :)

This change will need the Information architect role to be created in Signon Production & Integration. Hopefully @tijmenb can comment on that.

Based on what @tijmenb just said about the GDS Editor role, it seems that the additional role isn't necessary, so I'm going to remove the Information architect permission.


Filters' search/user input is lost after option selection

Screenshot
Native filters:

Filter box:

Description
In a situation where a filter contains a lot of options and also the "Dynamically search all filter options" option is activated, users will type in the search filter to get to the desired value. However, they might want to select multiple options from the newly updated options list. This is hard to do because: 1) using native filters the options list will reset after a given time; 2) on the filter box component the list will disappear after any option is selected. This leads to users having to retype their search to select all desired options.

Having a hard time deciding if this is a bug or a feature request... but I can confirm that this still happens in the latest Superset, and it can be annoying.
Pinging @kasiazjc in case any obvious design solution jumps out.

Hi @rusackas, @kasiazjc and @aspedrosa,
Any updates on this topic?
We have tested a little bit and detected that:

For a variable (of type: FLOAT), it has added the filter >= {{introduced_value}}.
For other variables (of other types and also FLOAT), it has not applied any filters when a manual value was introduced on the filter.

Any ideas?
Thanks,
Josep

I think this behaviour has now changed on 3.0.1.
Although I stumbled on this GitHub issue while looking for a bug report on the current behaviour.
I.e. I consider the 3.0.1 behaviour to be buggy.

@rusackas This issue is most likely resolved at this point

Thanks all! Closing :)


Prepare for 0.3 release

For the 0.3 version, we’re going to drop support for older rubies and older Rails versions.
Let’s do this!

:+1:


Concatenate layer error unhashable type: 'Dimension'

I was trying to do a simple test; Concatenating two layers.
Below is the code
inputs = Input(shape=(seq_length, latent_dim)) # seq_length = 3, latend_dim=2
reshaped_d_inputs = Reshape((inputs.get_shape()[1] * inputs.get_shape()[2], ))(inputs)
print(reshaped_d_inputs.get_shape()) # (?, 3, 2)
repeat_d_repeat = RepeatVector(seq_length)(reshaped_d_inputs)
repeat_d = Reshape((seq_length, inputs.get_shape()[1], inputs.get_shape()[2]))(repeat_d_repeat)
print(repeat_d.get_shape()) # (?, 3, 3, 2)

permuted_e = Permute((2, 1))(inputs)
reshaped_e_inputs = Reshape((inputs.get_shape()[1] * inputs.get_shape()[2], ))(permuted_e)
permuted_e_repeat = RepeatVector(seq_length)(reshaped_e_inputs)
repeat_e = Reshape((seq_length, inputs.get_shape()[2], inputs.get_shape()[1]))(permuted_e_repeat)
repeat_e = Permute((1, 3, 2))(repeat_e)
print(repeat_e.get_shape()) # (?, 3, 3, 2)

outputs = Concatenate(-1)([repeat_d, repeat_e])
model = Model(inputs=inputs, outputs=outputs)
model.compile(optimizer='rmsprop', loss='mse')
output_array = model.predict(input_array)

But it emits an error like below

TypeError                                 Traceback (most recent call last)
 in
14 print(repeat_e.get_shape())
15
---> 16 outputs = Concatenate(-1)([repeat_d, repeat_e])
17 model = Model(inputs=inputs, outputs=outputs)
18 model.compile(optimizer='rmsprop', loss='mse')
d:\igs_projects\nlp_nlu\venv\lib\site-packages\keras\engine\base_layer.py in call(self, inputs, **kwargs)
429                                          'You can build it manually via: '
430                                          'layer.build(batch_input_shape)')
--> 431                 self.build(unpack_singleton(input_shapes))
432                 self.built = True
433
d:\igs_projects\nlp_nlu\venv\lib\site-packages\keras\layers\merge.py in build(self, input_shape)
355         for i in range(len(reduced_inputs_shapes)):
356             del reduced_inputs_shapes[i][self.axis]
--> 357             shape_set.add(tuple(reduced_inputs_shapes[i]))
358         if len(shape_set) > 1:
359             raise ValueError('A Concatenate layer requires '
TypeError: unhashable type: 'Dimension'

I do not know if it is me doing something wrong or Keras bug...
After digging a while I realize that the tensor of right after the RepeatVector could be Concatenate but won't be concatenated once the posterior code is added.

Please fill the issue template here. Could you update them if they are relevant in your case, or leave them as N/A? Along with the template, please provide as many details as possible to find the root cause of the issue. It would be great if you can provide a small code to reproduce the error. Thanks!

I am facing the same issue. The resolution provided https://github.com/keras-team/keras/issues/5972  does not work.

It looks like you haven't used a template to create this issue. Please resubmit your issue using a template from here. We ask users to use the template because it reduces overall time to resolve a new issue by avoiding extra communication to get to the root of the issue. We will close this issue in lieu of the new one you will create from the template. Thank you for your cooperation.

@gilgarad I sloved it in #12963

@gilgarad Are you able to solve this problem? I am also facing the same problem.


I'd like to join this project's community

Your public repo
https://github.com/engn33r/gov4git-identity-public.git
Your public branch
main

Have you contributed yet to the project?

Yes, with https://github.com/pluralitybook/plurality/pull/965

Approved.

 Follow up
On Monday, 12-Aug-24 06:00:32 UTC by Gov4Git dev
@engn33r was added to the community.


Documentation

Expected:
The documentation would be consolidated to one place and when new features are added/updated, they are added to the documentation.
Actual:
Right now, using ember-engines requires a bit of juggling between Github and the guides, which can give the impression that documentation is missing.
Proposed solution:

Make the README.md a table of contents and link to sections on the ember-engines guides/site.
When something new is merged, it can be added to the README (table of contents), even if the author doesn't have time to write the documentation. This will also serve to identify where documentation still needs to be added.
Add a roadmap section to make it visible what the "roadmap to 1.0" looks like

@MelSumner can we keep ember-engines site or will to do a new site with Ember Addons Docs?

I would like to maybe contribute some things to the engines docs, but am unsure about where to go/how to start.


Launch motor does not run to full rpm at power setting 1 or -1 for either launch or intake

Appears to receive rapid sequence of conflicting power setting commands (-1 0 1 rather than 1 1 1 0 or -1 -1 -1 -1 -1 0 where 0 stops the motor and allows the wheel to coast)

Feeder wheel, CIM from KitBot Kit of Parts is fine


Pubsub retry config not generated

This script: https://github.com/googleapis/google-cloud-dotnet/blob/master/apis/Google.Cloud.PubSub.V1/midmicrogeneration.sh
Needs to also edit the retry config correctly to make the names match. At the moment it looks like there's a v1 -> v2 change, which is probably not correct.
Retry config file here: https://github.com/googleapis/googleapis/blob/4d52dfb72078000b13de923c1dadec19f3a64ad1/google/pubsub/v1/pubsub_grpc_service_config.json

It seems like this might not have been correctly generated. StreamingPull should have 5 retryable error codes, but it seems like none were added. Am I missing something perhaps?

I'll look into this tomorrow.

Actually, this may be deliberate, because retry for streaming RPCs just doesn't work - at least not for gRPC, and I think not for any language. We've discussed prohibiting it from the service config in the past, but I suspect we just deliberately ignore it in the C# generator. Will check tomorrow.

Jon is correct, streaming RPCs cannot have retry automatically applied, so any config is ignored.


adding herzing college

herzing.ca

@KeelanV First stage of verification passed, official domain is herzing.ca as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@KeelanV Thank you for your patience while waiting for a review and processing your request. Pull request merged.


dev: remove turbofish syntax

https://github.com/starkware-libs/cairo/commit/8e0293c888cb9aea54c930d2f5fc38d9c79d23d3
We can now declare types without the turbofish syntax. This is a great improvement for readability

wrong repo :/


doc: Update AccessControl.md with repository information

Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[x] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

/thaw


V1.2 release preperation

Update window title to IntelliJournal.
Update jar file name to IntelliJournal.jar.
Update version to v1.2 in logger.

Codecov Report

Merging #104 into master will increase coverage by 2.69%.
The diff coverage is 54.28%.

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


Receiving message in HA log that media_player.template is using depreciating features that will be unavailable in 2025

The message in my log file is:
(MainThread) [homeassistant.helpers.entity] Entity media_player.now_playing (<class 'custom_components.media_player_template.media_player.MediaPlayerTemplate'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <MediaPlayerEntityFeature.TURN_ON|TURN_OFF|VOLUME_STEP|SELECT_SOURCE|SELECT_SOUND_MODE: 68992>, please create a bug report at https://github.com/Sennevds/media_player.template/issues and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation
I love this template and is making my life so much easier.  Is this on anyone's radar to be fixed?

Hi.  Touching base on this issue again.  I have been seeing it for a while and haven't seen any comments or updates
Is anyone maintaining this code any longer?
Thanks!

Hi, chiming here too.
Am getting these warning messages post upgrade to 2024.3.0
Logger: homeassistant.helpers.entity
Source: helpers/entity.py:1597
First occurred: 8:42:57 PM (2 occurrences)
Last logged: 8:43:04 PM

Entity media_player.tdai_1120 (<class 'custom_components.media_player_template.media_player.MediaPlayerTemplate'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <MediaPlayerEntityFeature.VOLUME_SET|VOLUME_MUTE|TURN_ON|TURN_OFF|VOLUME_STEP|SELECT_SOURCE|SELECT_SOUND_MODE: 69004>, please create a bug report at https://github.com/Sennevds/media_player.template/issues and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation
Entity None (<class 'custom_components.smartir.climate.SmartIRClimate'>) is using deprecated supported features values which will be removed in HA Core 2025.1. Instead it should use <ClimateEntityFeature.TARGET_TEMPERATURE|FAN_MODE: 9>, please report it to the author of the 'smartir' custom integration and reference https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation

SUPPORT_TURN_OFF was used from media_player_template, this is a deprecated constant which will be removed in HA Core 2025.10. Use MediaPlayerEntityFeature.TURN_OFF instead, please report it to the author of the 'media_player_template' custom integration
SUPPORT_TURN_ON was used from media_player_template, this is a deprecated constant which will be removed in HA Core 2025.10. Use MediaPlayerEntityFeature.TURN_ON instead, please report it to the author of the 'media_player_template' custom integration
SUPPORT_VOLUME_MUTE was used from media_player_template, this is a deprecated constant which will be removed in HA Core 2025.10. Use MediaPlayerEntityFeature.VOLUME_MUTE instead, please report it to the author of the 'media_player_template' custom integration
SUPPORT_VOLUME_SET was used from media_player_template, this is a deprecated constant which will be removed in HA Core 2025.10. Use MediaPlayerEntityFeature.VOLUME_SET instead, please report it to the author of the 'media_player_template' custom integration
SUPPORT_VOLUME_STEP was used from media_player_template, this is a deprecated constant which will be removed in HA Core 2025.10. Use MediaPlayerEntityFeature.VOLUME_STEP instead, please report it to the author of the 'media_player_template' custom integration

And...
SUPPORT_NEXT_TRACK was used from media_player_template, this is a deprecated constant ...
SUPPORT_PAUSE was used from media_player_template, this is a deprecated constant ...
SUPPORT_PLAY was used from media_player_template, this is a deprecated constant ...
SUPPORT_PLAY_MEDIA was used from media_player_template, this is a deprecated constant ...
SUPPORT_PREVIOUS_TRACK was used from media_player_template, this is a deprecated constant ...
SUPPORT_SEEK was used from media_player_template, this is a deprecated constant ...
SUPPORT_SELECT_SOUND_MODE was used from media_player_template, this is a deprecated constant ...
SUPPORT_SELECT_SOURCE was used from media_player_template, this is a deprecated constant ...
SUPPORT_STOP was used from media_player_template, this is a deprecated constant ...

By the way, looks like this is already fixed (https://github.com/Sennevds/media_player.template/commit/b3157342bc06b67b191f9bc8c9c5b005d7b217c5) but hasn't been published to HACS.
Hi @Sennevds , are we going to have an update on HACS?
Thanks!

Sorry for the late reaction. I've just created a new release.


Added CI for windows (fixes #68)

@HeeManSu I ran the CI and the linter fails, we should review that.


GreyNoise API configuration issue

Hi,
I've a GreyNoise API key and I want to test it on the platform, but as soon as I set it up on the config.py file, the APP does not work, is not able to download any feed. There is required any format, quotations or something like that?
PS: I would like to add also the twitter API ones, but firts I prefer to identify how to configure it
Thanks in advance

Thanks for reporting this. Are you seeing any errors in Splunk? Also, please ensure that all value, with the exception of 'None', is surrounded by quotes. Commit d7ec95c39d667bd6bfc7579466e5297342fa3d84 was just pushed to show this in the comments of the config.py.
Let me know if this helps.

That works! I'm sorry to come with this newbie question :)


E AssertionError: False is not true : Thread message count in restored archive is off

A failing test that seems to be coming rather bizarrely from https://github.com/datactive/bigbang/pull/585 though tests passed on that PR.
The issue seems to be in the test_bigbang.py test.
Namely:
def test_mailman_chain(self):
        name = "bigbang-dev-test.txt"

        # archive loaded from mbox
        arx = archive.Archive(name, archive_dir=CONFIG.test_data_path, mbox=True)

        arx.save("test.csv")

        # archive loaded from stored csv
        arx2 = archive.load("test.csv")

        ....

        self.assertTrue(
            [t.get_num_messages() for t in arx.get_threads()] == [3, 1, 2],
            msg="Thread message count in mbox archive is off",
        )

        self.assertTrue( <---- ERROR HERE
            [t.get_num_messages() for t in arx2.get_threads()] == [3, 1, 2],
            msg="Thread message count in restored archive is off",
        )

The restored email archive is showing a different number of threads with a different number of messages: [4]
This is super weird.
$ python -m pytest tests/unit/test_bigbang.py
========================================================================= test session starts ==========================================================================
platform linux -- Python 3.8.13, pytest-7.3.1, pluggy-1.0.0
rootdir: /home/sb/projects/bigbang/tests
configfile: pytest.ini
plugins: anyio-3.6.2
collected 8 items

tests/unit/test_bigbang.py ...F....                                                                                [100%]

=============================================================================== FAILURES ===============================================================================
____________________________________________________________________ TestArchive.test_mailman_chain ____________________________________________________________________

self = <tests.unit.test_bigbang.TestArchive testMethod=test_mailman_chain>

    def test_mailman_chain(self):
        name = "bigbang-dev-test.txt"

        # archive loaded from mbox
        arx = archive.Archive(name, archive_dir=CONFIG.test_data_path, mbox=True)

        arx.save("test.csv")

        # archive loaded from stored csv
        arx2 = archive.load("test.csv")

        print(arx.data.dtypes)
        print(arx.data.shape)

        self.assertTrue(
            arx.data.shape == arx2.data.shape,
            msg="Original and restored archives are different shapes",
        )

        self.assertTrue(
            (arx2.data.index == arx.data.index).all(),
            msg="Original and restored archives have nonidentical indices",
        )

        self.assertTrue(
            [t.get_num_messages() for t in arx.get_threads()] == [3, 1, 2],
            msg="Thread message count in mbox archive is off",
        )
>       self.assertTrue(
            [t.get_num_messages() for t in arx2.get_threads()] == [3, 1, 2],
            msg="Thread message count in restored archive is off",
        )
E       AssertionError: False is not true : Thread message count in restored archive is off

This seems to be due to differences in how 'None' fields are represented in the original and restored archive.
(Pdb) arx.data['In-Reply-To'].iloc[0] is None
False
(Pdb) arx2.data['In-Reply-To'].iloc[0] is None
True

These fields are actually missing from the original test data.

Fixed with https://github.com/datactive/bigbang/commit/7ee437e1dc78af7b9a939ee0195de0a18d5c88aa


Update /wmg layout to be three panels

Worth checking out how Clever Canary did it for Search/Filter feature, since I think it's the same design? Hopefully we can reuse or copy/paste their code somehow 😆
Figma: https://www.figma.com/file/htICFl7zihfs1z9eIYVwzw/Gene-Search?node-id=2402%3A423263

@tihuan Can you attach a mock of some sort? Even if it's just a screenshot of CC's Search/Filter feature.

Morning Seve! Harley is gonna finalize the V1 mock today, but I think the layout is likely staying the same 👌
https://www.figma.com/file/htICFl7zihfs1z9eIYVwzw/Gene-Search?node-id=2402%3A423263
Is that what you're looking for?
Thank you!

Right! Thank you, forgot you mentioned that yesterday afternoon!
Just to clarify/refine, the layout is more akin to explorer with a main visualization book-ended by two sidebars, yes? I'm being nitpicky just to establish the difference in priority between the two headspaces: three panels with equal thirds of importance, vs main single importance viz with two supplementing sidebars.

Ooh good point that Explorer has the three panel layout too 💡
That's a great question, from the mock it doesn't seem like three panels will be equal in size, but they will be collapsible!
Let's confirm with @hthomas-czi on the actual specs just in case 🙏
And please feel free to reach out to him directly if he doesn't respond here, since I know some people don't check GitHub notif frequently 😄
Thanks a lot!

@seve Yes, that's the right way to think about the interface (main workspace + 2 sidebars). It differs from Explorer in the ways @tihuan described — namely, they're asymmetrically and collapsible.

Hey @seve !
Harley finalized the panel layout in the latest Figma 🚀
What we discussed in the kickoff meeting this morning is that the left panel expand/collapse will reposition the middle panel (instead of an overlay), but won't resize the middle panel. And the right panel expand/collapse will overlay the middle panel, so it could cover the middle panel partially when expanded and that's fine
Hope this makes sense!
Thank you!

I just updated the DoD, please feel free to edit it as you see fit! Thanks!

@seve also just to be super explicit, the three panel layout doesn't include contents in the panel! We have other tickets for those 👍
Please feel free to check https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/1844 for all the tickets and help see if I missed anything 😆 Thank you!

confirmed in staging!


adding missing python dep for dockerfile

[x] adding missing docker python dep
[x] split dockerfile into build and runtime segments
[x] members api returns correct output
[x] depcheckrc added for pg
[x] package.json deps cleanup
[x] npm ci --production

From the helm test:
Invalid environment variables:
    AUTH_ISSUER: Invalid url: "aHR0cHM6Ly92aXRhbGFtLmV1LmF1dGgwLmNvbS8="
    AUTH_JWKS_URI: Invalid url: "aHR0cHM6Ly92aXRhbGFtLmV1LmF1dGgwLmNvbS8ud2VsbC1rbm93bi9qd2tzLmpzb24="
    AUTH_TOKEN_URL: Invalid url: "aHR0cHM6Ly92aXRhbGFtLmV1LmF1dGgwLmNvbS9vYXV0aC90b2tlbg=="
 Missing environment variables:
    API_HOST: undefined

where has this come from? They are definitely not valid urls


Generating multiscale pyramids for raw CMIP6 data

Background
We're interested in generating a decent sized collection of CMIP6 data, reprojected into web-mercator for use in @carbonplan/maps. This issue is meant to track the initial prototypes for that process.
Initial approach
The following code sample generates multiscale data pyramids for raw CMIP6 data.
import intake

import rioxarray
from ndpyramid import  pyramid_reproject

# open cmip6 data catalog
col_url = "https://cmip6downscaling.blob.core.windows.net/cmip6/pangeo-cmip6.json"
cat = intake.open_esm_datastore(col_url).search(member_id=['r1i1p1f1'], table_id=["day"], grid_label=['gn'])

# get a dataset
for key, entry in cat.items():
    print(key)
    ds = entry(storage_options={'account_name': 'cmip6downscaling'}).to_dask()
    ds = ds.squeeze('member_id', drop=True)
    ds = ds.rio.write_crs('EPSG:4326')
    break  # remove when its time to loop over the full collection

# generate the pyramid
pyramid = pyramid_reproject(ds, levels=2)
pyramid.to_zarr(...)

Warning, I suspect that this will not be particularly performant and that we may want to use a different method for doing the reproduction.
Passing this off to you @freeman-lab. cc @katamartin

See also: https://github.com/carbonplan/ndpyramid/issues/10

this has been implemented via ndypramid's pyramid_regrid() functionality.


languages not loaded

Hello,
At
https://github.com/tesseract-ocr/tesseract/wiki
I saw there is a windows installer for Tesseract 3.05-dev from Tesseract at UB Mannheim at
https://github.com/UB-Mannheim/tesseract/wiki
I installed it. It works with English. On the last page there is a link to "download the appropriate training data" for another language at
https://github.com/tesseract-ocr/tessdata
On this page it says "These language data files only work with Tesseract 4. Get language data files for Tesseract 3.04 or 3.05 from the 3.04 tree.". The new link is:
https://github.com/tesseract-ocr/tessdata/tree/3.04.00
I downloaded from there bul.traineddata and deu.traineddata and copied to
. . . \Tesseract-OCR\tessdata
. Now tesseract --list-langs shows
bul
deu
eng
osd
But it works really with English. For Bulgarian and for German the message is:
Failed loading language 'deu'
Tesseract couldn't load any languages!
Could not initialize tesseract.
Any advice - what could be wrong?

The language files bul.traineddata and deu.traineddata were downloaded incorrectly.


pcluster manager - shutdown procedure ?

Hi,
I have successfully launch pcluster manager.
As there are many nested resources, is there a single point of entry where we can do a clean shutdown to ensure that all the resources spurn up for a given pcluster is all gone and does not continue to charge to our account ?
Cheers

Hi @nyue -
Yes the pcluster-manager main stack will delete everything.

If I have created compute cluster via pcluster manager, would the deletion of the main stack also delete all the created cluster resources ?

No it won't. you'll need to delete those stacks as well. This is by design


rpm problems: Missing +x permissions on all os/linux64 binaries

Describe the bug
All the /usr/share/ghidra*/**/os/linux64/* binaries are missing the executable permissions
when being installed via the fedora 31 rpm
leading to error 13 errors on analyze, ...
To Reproduce
Steps to reproduce the behavior:
sudo rpm -i https://forensics.cert.org/cert-forensics-tools-release-31.rpm
sudo dnf --enablerepo=forensics install ghidra

ls -al /usr/share/ghidra_9.1.2_PUBLIC/GPL/DemanglerGnu/os/linux64/demangler_gnu
/usr/share/ghidra_9.1.2_PUBLIC/GPL/CabExtract/os/linux64/cabextract
/usr/share/ghidra_9.1.2_PUBLIC/Ghidra/Features/Decompiler/os/linux64/decompile
/usr/share/ghidra_9.1.2_PUBLIC/Ghidra/Features/Decompiler/os/linux64/sleigh

RELEASE PUBLIC_20200212.1.fc31
linux64
version 9.1.2

If I remember correctly the starter script
/usr/share/applications/ghidra.desktop
pointed to /usr/share/ghidra_9.1.2/ without the _PUBLIC suffix.

this is a distribution issue, not ghidra.  The only official binary release for ghidra is ghidra-sre.org.  File a bug with forensics.cert.org or whoever packaged it.

Ah thanks. I thought so after I bugged you here :)


Enabled all debug checks

Please use builds from this branch to fix debug check violations in Electron.
This branch enables all debug checks including V8 and general Chromium ones, and as such presents the greatest challenge because Electron triggers those checks often.
There's another branch which enables only some debug checks whose violations should be easier to fix:
https://github.com/electron/libchromiumcontent/pull/326
Tip: Symbol files for libcc are stored at the same location as the published libchromiumcontent.zip file, only the file is named libchromiumcontent-dbg.zip

Rebased on current master.

@alespergl Do you still need this PR?

Since there's no response from @alespergl and the libcc repo is going to be archived soon, I'm going to close this PR.


Unnecessary privileged stacks with CONFIG_USERSPACE=y

Describe the bug
When building with userspace enabled, gen_priv_stacks.py generates privileged stacks for system stacks that do not need it.  For example, on Cortex-M platforms, there is no need for a privileged shadow stack of _interrupt_stack, z_main_stack, or z_idle_stack.
To Reproduce
Using recent code as of g219d9fc0829
west build -p auto --board=stm32f411e_disco samples/userspace/prod_consumer
~/zephyr-sdk/arm-zephyr-eabi/bin/arm-zephyr-eabi-nm build/zephyr/zephyr.elf |grep -e "priv_stack_[0-9]+"
20004ea0 b priv_stack_20001800
20004aa0 b priv_stack_20002000
200046a0 b priv_stack_20002800
200042a0 b priv_stack_20003000
20003ea0 b priv_stack_20003400
~/zephyr-sdk/arm-zephyr-eabi/bin/arm-zephyr-eabi-nm build/zephyr/zephyr.elf |grep -e _interrupt_stack -e z_main_stack -e z_idle_stack
20002800 B _interrupt_stack
20003400 B z_idle_stack
20003000 B z_main_stack
Impact
This does not really cause any functional issues but does waste memory which can be scarce on certain platforms.  I can postprocess the elf and remove the unneeded stacks to get around this for now.
I assume that the proper fix would involve fixing up gen_priv_stacks.py to somehow bypass the system related stacks, but I am not sure of a clean way to do that.  That seems like it would need a blacklist of the system stacks.  Another option I was thinking about would create a new macro and type for the system stacks, but that would require touching a lot of core code and does not seem like a good option either.
I'm willing to go attack this problem if I can get a better course of action that the above ideas.
Thanks.

Duplicate of #16210, and will be solved by #13637

@bbolen maybe close this one (you could add this report info in #16210)

Sorry for the duplicate.  Thanks for the response.


Stable default location

As discussed in https://github.com/adoptium/installer/pull/789 I created a separate PR with effectively the same changes taking into account the proposal from https://github.com/adoptium/installer/pull/789#issuecomment-2306065205.
The default location should now be temurin-<ver>-<jdk|jre> in the installer with these changes.
I did not find any variable / localization reference in the installer code base that contains the required temurin string. I can change that from the current hardcoded string, of course, but would need a little input/guidance on what to do.

I'm good with this change as long as we send it out with a release note as it will deviate from the previous installs.

Is there some way/doc how to gather such requirements? (future release notes)

@douph1 this is a fair criticism and I am willing to help with getting it addressed, however, I am unable to building such logic into the installer by myself. If any of you can help with providing pointers etc., that'd be immensely helpful and I would try to incorporate this.

Just my 2 cents :
As already said, I'm think this is a bad idea to have only a fixed path with only major version as it can break some running process.
If done we must ensure :

no running jvm are already running from the same folder. This PR does not include any change to secure that.
Or
warn the user that running jvm can be unstable, and ask to restart it asap.

Did the installer override/upgrade the previous version without asking/warning ?
It is OK to always use only major version in path if no previous install exists.
As suggested, two installer with different minor number and fixed path must be created to be able to test in place upgrade while running jvm.
I'm probably not able to make a reproducible test case where running jvm is affected by background change of files, but I'm certain I have already been impacted by that.

I'm not sure we need to cover the case for the running JVM? IIRC - Typically with Windows installers the OS expects you to shut down the process of the application that you're upgrading. Perhaps a warning in the installer would be sufficient.


[Concurrent Segment Search] ContextIndexSearch handles CollectionTerminatedException

ContextIndexSearch searchLeaf method seems to return instead of continuing when it hits the CollectionTerminatedException.
Ref: https://github.com/opensearch-project/OpenSearch/blob/74788c3f45895a90d030b2434539beac7a1b1723/server/src/main/java/org/opensearch/search/internal/ContextIndexSearcher.java#L323

This looks like the right behavior to me.
https://github.com/opensearch-project/OpenSearch/blob/828730f23377e78dd1df0fa53939ed7f2486800e/server/src/main/java/org/opensearch/search/internal/ContextIndexSearcher.java#L281-L345
There's no return in the catch blocks on L317 and L336 because those are in the if/else blocks and would not make a difference. In fact IntelliJ probably marks those returns as unnecessary and prompts the user to remove them.
@yigithub @sohami I think we can close out this issue?


Refactor isEndOfStreamBlock to isTerminateBlock

Fixes todo task refactoring isEndOfStreamBlock() in TransferableBlock.java to isTerminateBlock()

Codecov Report
Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 63.67%. Comparing base (59551e4) to head (0b9bde3).
Report is 1264 commits behind head on master.

Files with missing lines
Patch %
Lines

...apache/pinot/query/mailbox/GrpcSendingMailbox.java
0.00%
0 Missing and 1 partial :warning:

...he/pinot/query/mailbox/InMemorySendingMailbox.java
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #14338      +/-   ##
============================================
+ Coverage     61.75%   63.67%   +1.91%
- Complexity      207     1467    +1260
============================================
  Files          2436     2660     +224
  Lines        133233   145685   +12452
  Branches      20636    22291    +1655
============================================
+ Hits          82274    92759   +10485
- Misses        44911    46061    +1150
- Partials       6048     6865     +817

Flag
Coverage Δ

custom-integration1
?

integration
100.00% <ø> (+99.99%)
:arrow_up:

integration1
100.00% <ø> (+99.99%)
:arrow_up:

integration2
0.00% <ø> (ø)

java-11
0.00% <ø> (-61.71%)
:arrow_down:

java-21
63.67% <75.00%> (+2.04%)
:arrow_up:

skip-bytebuffers-false
63.65% <75.00%> (+1.90%)
:arrow_up:

skip-bytebuffers-true
63.64% <75.00%> (+35.91%)
:arrow_up:

temurin
63.67% <75.00%> (+1.91%)
:arrow_up:

unittests
63.66% <75.00%> (+1.91%)
:arrow_up:

unittests1
55.32% <75.00%> (+8.42%)
:arrow_up:

unittests2
34.19% <0.00%> (+6.45%)
:arrow_up:

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

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


lib.rs with integration test ex

lib.rs holds the tests that execute when you run cargo test
integration_test.rs holds the integration tests that excuse when you run cargo test --test integration_test -  purpose is to test whether many parts of your library work together correctly

@thea-exe commit signing?

rebased on my changes. moirging em

whoops


vLLM always places spaces between special tokens

Implement huggingface's spaces_between_special_tokens in _convert_tokens_to_string_with_added_encoders.
Fixes #1347

Thank you for your contribution, can you undo the naming change request -> req so the diff is cleaner to view?

Yes, no problem.  Done.


Mention helm chart in installation guide

It looks like etcd-operator is packaged as a Helm chart:
https://kubeapps.com/charts/stable/etcd-operator
https://github.com/kubernetes/charts/tree/master/stable/etcd-operator
Is there any reason to not to document it in the https://github.com/coreos/etcd-operator/blob/master/doc/user/op_guide.md ?

@ahmetb Would you like to update the doc?

@hongchaodeng I have to follow some internal approval process to contribute. It might be faster for you to do it.


ng serve/test/e2e differential loading and IE support

🐞 Bug report
- [ ] new
- [ ] build
- [x] serve
- [x] test
- [x] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
To me this sound as a regression. ng serve/test/e2e used to work out of the box for Internet Explorer on a freshly created project.
Now it simply doesn't without tweaking the configuration (as proposed here) (which I clearly do not want to embrace... see my points in description below)
Description
I would like to be able to develop and test on legacy platforms in a transparent way. But I also want to benefit from differential loading.
ng build supports differential loading both in dev and prod mode, apparently ng serve and ng test don't. (ng e2edoes not either as being build on top of ng serve)
I am fine for ng serve not doing it but I still do want to have ng test/e2e working with real differential loading. It is even mandatory for anyone that needs to test Internet Explorer support.
Workarounds proposed:

Revert back to pre ng-cli-8 era (ie target: es5 for everyone in tsconfig.json and no differential loading).
Maintain 2 different build paths / configuration in my package.json/angular.json

None of these workarounds suit my need.
🌍 Your Environment

Angular CLI: 8.0.1
Node: 10.9.0
OS: darwin x64
Angular: 8.0.0
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.800.1
@angular-devkit/build-angular     0.800.1
@angular-devkit/build-optimizer   0.800.1
@angular-devkit/build-webpack     0.800.1
@angular-devkit/core              8.0.1
@angular-devkit/schematics        8.0.1
@angular/cli                      8.0.1
@ngtools/webpack                  8.0.1
@schematics/angular               8.0.1
@schematics/update                0.800.1
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.30.0

Anything else relevant?
Trying to dig down all these things, I also ended up looking at the code of this commit
https://github.com/angular/angular-cli/blob/6ec09919b5c2695dee784ce0c3accee7f9754bb0/packages/angular_devkit/build_angular/src/angular-cli-files/plugins/karma-debug.html#L32-L33
What's the point of doing that? Any old browser will end-up loading both of them. Line 32 shouldn't have a type="module" ?
Even more, what's the point at all to load an es5 compatible bundle where all the other one are only es2015 ?
Are my below statements right ?

On an es2015 compatible platform, this polyfills-es5.js should not be needed.
On an old es5 compatible platform, if polyfills.js (even runtime.js which comes before) is es2015 it would'nt fly and would crash with syntax error (which is the case)

I am really confused about this change performed in this commit... 😞

Hi, @benouat
Let me answer some of your questions :)

On an es2015 compatible platform, this polyfills-es5.js should not be needed.
Indeed this is true, infact it won't be downloaded because of the nomodule attribute.

On an old es5 compatible platform, if polyfills.js (even runtime.js which comes before) is es2015 it would'nt fly and would crash with syntax error
The polyfills.js should always be downloaded irrespective if the browser supports es2015 or not, because in this file there are polyfills which are always required for the application to work such as zone.js.

The polyfills file JavaScript version will be the same script target of the build, ie if it's an ES5 build the polyfills content will be in ES5 Js, while for an ES2015 build it will be ES2015.
If you want to test on non es2015 browser you'd need to provide a different configuration as per my comment https://github.com/angular/angular-cli/issues/14455#issuecomment-493739768,
I hope the above makes some more sense :)
I think it's best to keep the discussion in a single thread, being https://github.com/angular/angular-cli/issues/14455. If we see that this has a big impact on the community we will re-open and propose a better alternative for this,
Thanks.

Thanks for your clarification ! 👍

I think it's best to keep the discussion in a single thread, being #14455.

Nonetheless, keeping a discussion thread on a closed issue seems a bit strange to me.


"Find" - add more options

In "Find" dialogue I miss the features

start search always on top
or
continue search when reaching end / start of file
What your opinion, Chris?

Ctrl+home is pretty easy...


Add status to ImageDigest/TagMirror API to pass ci

Add satus to ImageDigest/TagMirror API. This will make it pass CI: blob:https://prow.ci.openshift.org/c17448be-31e0-46a9-a834-cf85814f7d5d
Signed-off-by: Qi Wang<EMAIL_ADDRESS>

/label px-approved
/label qe-approved
/label docs-approved

something is up with someone's whitespace, but this seems fine otherwise.
/lgtm


removing tincan role

I've removed the tincan_role capistrano role, this will now be defined within the application itself so we have more control over where we want this running.

This PR should be merged alongside this: https://github.com/captainu/captainu/pull/1708


flutter initializing  gradle forever

Analyzer Feedback from IntelliJ
Version information

IDEA AI-182.5<IP_ADDRESS>14842
2.2.0-edge.0a7dcf17eb5f2450480527d6ad1e201fb47f1e36
AI-182.5<IP_ADDRESS>14842, JRE 1.8.0_152-release-1248-b01x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1366x768

/move to flutter/flutter-intellij


Send request with credentials ( cookie )

Describe the bug
I have on setting. " Send request with credentials ( cookie ).
I succeful login by my mutation and get all info with cookie. But nest query with guard where im checking id user from cookie cant see any cookie delivered by request. In Apollo studio, Graphi and Graphql Playground its works perfect.
To Reproduce
Server with mutation to Login and Query with UseGuards with checking idUser in cookie
Desktop (please complete the following information):

OS:MacOs 12.3
Browser chrome
Platform: desktop app
Version latest

Additional context
Add any other context about the problem here.

Can you show some logs and/or screenshot of the settings on Altair and the Request headers in the network tab when you are sending the request in Altair?

Going to close this issue as stale. Feel free to comment here if that is not the case.

We have the exact same issue - this is a regression, it worked in prior versions (half a year ago).
If i set a header 'Cookie: myname' with a value, it failed the request with
`failed to execution set request header on XMLRequest: 'Cookie: myname'.. is not a valid HTTP header field name

@EugenMayer The question here is: what form factor are you using Altair in? Desktop app? Web app? Browser extension?
I just manually tested adding a cookie header with myname value in the desktop application and it worked - as expected.
If you're trying this from a web application or browser extension, it wouldn't work, and it has never worked there. We've had this article published (for quite a while now) explaining this and recommending the desktop applications.
If you are having this issue in the desktop application, and you confirm your application version is recent, I can re-open this issue, or you can create a new issue if the problem there is more specific.

Desktop/electron app (arch build)

Do you have a specific version? It might be easier to create an issue with related screenshots so I can better understand the issue here. Considering I tested this locally with the latest Altair version, it might be something more nuanced to your use case.

It's 4.4.2 (the snap variant).
Cannot provide a screenshot right now, since i'am on holidays - but there is nothing to see realy. The error string is show weiredly since the error message is not in the expected format (i guess).
With the desktop variant it should be reproducable against a normal graphql endpoint.
I can us a curl request using the same POST payload with the cookie and it will work, using the same cookie with Altair no longer works. Also maybe to make this clear, it used to work less half a year ago. We do not use Altair regurarly, that is why i cannot tell it more specificly.
Hope this helps

'''
Failed to execution set request header on XMLRequest: 'Cookie: myname'. is not a valid HTTP header field name
'''

Yes but where do you receive that message? In a terminal? Notification alert within the app? Developer tools console?


Assembly make_simple implementation

Part simplified model
At the moment, a sphere Part may be implemented as follows:
import cadquery, cqparts
class Sphere(cqparts.Part):
    def make(self):
        return cadquery.Workplane('XY').sphere(1)
    def make_simple(self):
        return cadquery.Workplane('XY').box(2, 2, 2)

Assembly simplification
However, there is no method for doing the same with an assembly.
A perfect example of where this would be useful is the BallBearing.
A simplified model of a BallBearing instance would not be an Assembly but a single solid resembling the shape of the bearing.

Design
(placeholder)
The design at the lower level; the point of implementation requires a bigger discussion first... #48

Problem
each Assembly instance is expected to have components and constraints, however the simplified model of an Assembly is atomic, therefore more closely resembles a Part.
Potential Solution
If an Assembly normally contains a number of components, a simplified Assembly could contain a single component, and a single constraint fixing the simplified Part component to the assemblies origin.
So make_simple could be implemented in the same way as a Part, returning a cadquery.Workplane instance. When the Assembly is built with the _simple flag, it wraps the bound make_simple class into a Part class and instantiates it.


Dynamic input and output shape definition for object detection

I have this object detection model which works fine in the tf-serving and I am trying to deploy it on the tensorrt as well, but the issue I am facing- in tf-serving I do not need to define the input_shape and output_shape of the tensors which is considered directly from the model-file itself and works as well. But in tensorrt, I have to manually define those shapes and I can't escape that as the config-error is getting encountered.
This is the definition of my input tensor:

[{'name': 'input_1', 'shape': '(?, ?, ?, 3)', 'dtype': "<dtype: 'float32'>"},
{'name': 'pyramid_regression_0/kernel',
'shape': '(3, 3, 256, 256)',
'dtype': "<dtype: 'float32'>"}]

And this is the config.pbtxt
name: "objdtc_trt"
platform: "tensorflow_graphdef"
max_batch_size: 16
input [
  {
    name: "input_1"
    data_type: TYPE_FP32
    format: FORMAT_NHWC
    dims: [ 224, 224, 3 ]
  }
]
output [
  {
    name: "output_1_1"
    data_type: TYPE_FP32
    dims: [ 224 ]
  },
  {
    name: "output_1_2"
    data_type: TYPE_FP32
    dims: [ 224 ]
  },

]
instance_group [
  {
    gpus: [0]
  }
]

Is there a way, where I can define the input and output tensor dynamically- the dimensions.

@swarren @mkolod @lukeyeager

If you are referring to variable size tensor, then you can simply mark the variable size part to -1 (see detail). And you can actually provide no model config and set --strict-model-config=false on starting TRTIS, which will auto-fill the model config for you if the framework supports it (see detail).
If it is not variable size tensor that you are referring to, please provide more detail and the error message you encountered so that we can have a better understanding of the issue.