cmake: remove fixCmakeFiles

Rewriting /usr and /opt to /var/empty is no longer necessary since the sandbox was introduced. It also introduced unexpected side effects and changes paths like $out/etc/opt/ to $out/etc/var/empty/
Closes #24215
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Nice, this should fix https://github.com/NixOS/nixpkgs/issues/24215.

I'm not a darwin person, but AFAIK sandbox is still not enabled in there by default and /usr and /opt paths seem in use on those systems (for non-nix stuff).

On *-linux I wouldn't expect significant fallout from this, at least.

Actually, on *-linux I see risk for use cases like nix-shell on non-NixOS.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/making-a-development-flake-for-bespoke-synth/29988/19

Actually, on *-linux I see risk for use cases like nix-shell on non-NixOS.

hmmm, not sure if we can just chance that and hope for the best?

I'm not a darwin person, but AFAIK sandbox is still not enabled in there by default and /usr and /opt paths seem in use on those systems (for non-nix stuff).

Can someone more firm in darwin comment on that?

MacPorts and Homebrew (aarch64-darwin) both put stuff in /opt. Homebrew (x86_64-darwin) puts stuff in /usr/local. The sandbox is not enabled by default on Darwin, so my concern would be that those things could get picked up accidentally.

Actually, on *-linux I see risk for use cases like nix-shell on non-NixOS.

I think this is acceptable as this was creating lots of subtle bugs and sometimes evening installing files into void.

The sandbox is not enabled by default on Darwin, so my concern would be that those things could get picked up accidentally without it.

but that's also true for any other PATH and those lines of code caused probably more breakages than they fixed and didn't work like expected in the first place anyway.

I really just want to kill it. Can we please kill it.

Given that the sandbox is off by default on Darwin (which seems to be because it doesn't work well enough), this PR would introduce a huge amount of impurities on Darwin. This would be the case not only for users who have Homebrew and/or MacPorts installed alongside Nix, but for any Darwin user, since the system puts files in /usr as well.
To me, this seems like a change that would be a huge detriment to users of Nix on Darwin while providing comparatively little benefit to other Nix users

I think I know why the hook is there now. These are the results of applying it to cmake itself: https://gist.github.com/K900/1da3c1e5e1a46ae1dd5c413f9ec35b1d

@K900 Thank you. First we need to fix that particular problem you've mentioned, and then let's try to figure out the impact of a PR like this before we merge such a thing.

Given that the sandbox is off by default on Darwin (which seems to be because it doesn't work well enough), this PR would introduce a huge amount of impurities on Darwin. This would be the case not only for users who have Homebrew and/or MacPorts installed alongside Nix, but for any Darwin user, since the system puts files in /usr as well.
To me, this seems like a change that would be a huge detriment to users of Nix on Darwin while providing comparatively little benefit to other Nix users

If this seems to be only useful on darwin, why not make it a darwin only thing and on Linux we just rely on the sandbox?

Given that the sandbox is off by default on Darwin (which seems to be because it doesn't work well enough), this PR would introduce a huge amount of impurities on Darwin. This would be the case not only for users who have Homebrew and/or MacPorts installed alongside Nix, but for any Darwin user, since the system puts files in /usr as well.
To me, this seems like a change that would be a huge detriment to users of Nix on Darwin while providing comparatively little benefit to other Nix users

If this seems to be only useful on darwin, why not make it a darwin only thing and on Linux we just rely on the sandbox?

That seems reasonable to me. And if the Darwin sandbox does reach a point where it is effective and can be enabled by default, this PR could be revisited on Darwin.

How will this affect building inside nix-shell on non-NixOS?

It will not.

Aaaah so that's what was going on.

changes paths like $out/etc/opt/ to $out/etc/var/empty/

Ran into one of these myself during what would otherwise be a maintenance bump of a package because src/opt is a folder with source files there and paths got rewritten to src/var/empty. A lot of head-scratching later I find this issue and the dontFixCmake = true; fix.
Since this PR seems a ways from being merged, maybe a quick win would be making the sed expression more specific?
https://github.com/NixOS/nixpkgs/blob/a292058564022422d26461512a1fd18bdd4f87ec/pkgs/by-name/cm/cmake/setup-hook.sh#L10
I'm not sure exactly which characters are allowed in filepaths, but at the very least [a-zA-Z0-9]*/opt shouldn't be matched right?

FWIW I think the way to fix this is to proceed with the approach in https://github.com/NixOS/nixpkgs/pull/290170. I even got a Hydra jobset for it, but never got around to investigating the failures: https://hydra.nixos.org/jobset/nixpkgs/pr-290170-cmake-hook-shenanigans

The bills failures there are random failures/time outs.

Not all of them, though a lot of them are in fact garbage.

@K900 any insights from the above Hydra run?

Everything explodes really badly, notably cmake itself. Someone needs to go through cmake and fix it, and then we can continue.

@SuperSandro2000, should this then be marked draft or closed? Seems this proposed way forward has some serious thorns.

but that's also true for any other PATH and those lines of code caused probably more breakages than they fixed and didn't work like expected in the first place anyway.

It is, but the Darwin module bundled with CMake searches those locations by default. It would be nice if it could be made not to do that (even if it doesn’t address every possible problem).

Also aren't the builds run under a different user even outside the sandbox? If that can't write to there, we should be safe, otherwise this seems like a big glaring issue that affects darwin in general.

The concern wasn’t about writing to those locations. It’s that packages installed with other package managers could be picked up instead of those provided by nixpkgs (e.g., enabling unwanted/unexpected features or causing other problems).
This actually is causing problems for the Darwin refactor. I have a workaround unless this lands first, but I wouldn’t mind a patch or fixup to address the default search paths including other package managers on Darwin.

After working on https://github.com/NixOS/nixpkgs/pull/346043 and testing https://github.com/NixOS/nixpkgs/pull/349555, I would support undoing at least the /usr mangling.
On Darwin, /usr/include doesn’t exist, and /usr/lib barely contains anything. Using the SDK as a sysroot maps those locations to $SDKROOT/usr/include and $SDKROOT/usr/lib, but fixCmakeFiles interferes with that. The .NET build needs to link against Swift frameworks, so it attempts to add -L/usr/lib/swift to the library path, but that gets mangled to -L/var/empty/lib/swift, causing the build to fail to link libswiftFoundation.dylib after https://github.com/NixOS/nixpkgs/pull/349555.

I think that we could reasonably specialize it to /usr/(local|X11)  without losing anything and fixing a large portion of the problems here.

After some discussion on Matrix, we came up with the idea that, since -I/foo and -L/foo will look under the SDK root first on Darwin, we could simply ensure that our SDK packages (or a separate stub directory we layer on) contain empty {usr/,opt/}{,local/,X11/}{lib,include} hierarchies. That should ensure that the common compiler tools don’t access anything leaky even with the sandbox off, and allow us to massively reduce the blast radius of getting rid of this hack.


fix: Dupe detection - 'Keep this one' button appears when there is a paid expense.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/48366
PROPOSAL: https://github.com/Expensify/App/issues/48366#issuecomment-2322377408
Tests

Go to workspace chat.
Submit an expense and pay it.
Submit another two same expenses as Step 2.
Go to transaction thread (any expense).
Click Review duplicates.
Note that Keep this one button does not appear when there is a paid expense.
Click Keep all.
Submit another expense that is same as the previous expenses.
Go to transaction thread of the new expense.
Click Review duplicates.
Verify "Keep this one" button does not appear for any duplicate expense.

Submit an expense and pay it.
Submit another two same expenses as Step 2.
Go to transaction thread (any expense).
Click Review duplicates.
Note that Keep this one button does not appear when there is a paid expense.
Click Keep all.
Submit another expense that is same as the previous expenses.
Go to transaction thread of the new expense.
Click Review duplicates.
Verify "Keep this one" button does not appear for any duplicate expense.

QA Steps

Go to workspace chat.
Submit an expense and pay it.
Submit another two same expenses as Step 2.
Go to transaction thread (any expense).
Click Review duplicates.
Note that Keep this one button does not appear when there is a paid expense.
Click Keep all.
Submit another expense that is same as the previous expenses.
Go to transaction thread of the new expense.
Click Review duplicates.
Verify "Keep this one" button does not appear for any duplicate expense.

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

PR Author Checklist

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

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

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

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

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/4407556d-3a14-45bd-9e1a-fe3f8755492b

Android: mWeb Chrome
https://github.com/user-attachments/assets/d9c18387-3e42-4a38-8027-6cf6867822c2

iOS: Native
https://github.com/user-attachments/assets/073a31f9-82a2-4932-bed8-f0fab12f65f0

iOS: mWeb Safari
https://github.com/user-attachments/assets/9c2ce045-c870-4670-815d-eb926a7a8186

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/97844bd9-626f-4446-8095-2186e525886b

MacOS: Desktop
https://github.com/user-attachments/assets/85a44d74-2ab3-4968-9699-d8e76774bfd0

Reviewer Checklist

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

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

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

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

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop


updated NNX rnglib docstring

updated NNX rnglib docstring

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

Project coverage is 0.00%. Comparing base (abf36e5) to head (d8d4e5d).
Report is 27 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff           @@
##            main   #3980    +/-   ##
======================================
  Coverage   0.00%   0.00%
======================================
  Files        102     106     +4
  Lines      13208   13582   +374
======================================
- Misses     13208   13582   +374

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


[CPM IT]  Adding seed-taint-testrun-value property

How to categorize this PR?

/area control-plane-migration
/kind enhancement
What this PR does / why we need it:
This PR adds seed-taint-testrun-value property that will enable to set the value for the test.gardener.cloud/test-run. With the current implementation for that value the environment variable TM_TESTRUN_ID. This is limiting as it requires that the seed is also tainted the same way and with the same TM_TESTRUN_ID as value.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

Since it seems that the seed used for the test will only have the protected taint and will be "invisible", we probably don't even need an additional special taint. I would suggest to add the test-run taint's key as a parameter - e.g. if it is defined then add the test-run toleration with a value equal to the testrun id, if it is not defined then don't add any additional tolerations (apart from the protected toleration)

@plkokanov I am not sure we should change the property type to bool, because when the TM runs the MigrateShoot.yaml test definition without specifically set environment variable ADD_TEST_RUN_TAINT=true/false the test would fail, because the parameter -add-test-run-taint would be equal to "" and the error message would be invalid boolean value "" for -add-test-run-taint

@plkokanov I am not sure we should change the property type to bool, because when the TM runs the MigrateShoot.yaml test definition without specifically set environment variable ADD_TEST_RUN_TAINT=true/false the test would fail, because the parameter -add-test-run-taint would be equal to "" and the error message would be invalid boolean value "" for -add-test-run-taint. With this implementation we avoid setting the the ADD_TEST_RUN_TAINT explicitly and we can default every value to false.

Hmm when using
var v = flag.Bool("


RV10-lite robot vacuum creates error in Plugin on startup

Describe The Bug
I just got a TAPO RV10-lite vacuum and after adding the plugin and configuring the IP address the following error is thrown in the homebridge log:
 Failed to get info about: xxx.xxx.xxx.xxx | The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined

Expected behavior
The plugin to connect to the device and retrieve the information
Logs
[3/30/2023, 11:09:07 AM] Loaded plugin<EMAIL_ADDRESS>[3/30/2023, 11:09:07 AM] Registering platform 'homebridge-tp-link-tapo.HomebridgeTPLinkTapo'
[3/30/2023, 11:09:07 AM] ---
...
[3/30/2023, 11:09:07 AM] [Tapo] Initializing HomebridgeTPLinkTapo platform...
[3/30/2023, 11:09:07 AM] [Tapo] Initializing child bridge AA;AA:AA;AA
...
[3/30/2023, 11:09:08 AM] [Tapo] Error sending command: deviceInfo AxiosError: connect ECONNREFUSED xxx.xxx.xxx.xxx:80
    at Function.AxiosError.from (/usr/local/lib/node_modules/homebridge-tp-link-tapo/node_modules/axios/lib/core/AxiosError.js:89:14)
    at RedirectableRequest.handleRequestError (/usr/local/lib/node_modules/homebridge-tp-link-tapo/node_modules/axios/lib/adapters/http.js:577:25)
    at RedirectableRequest.emit (node:events:513:28)
    at ClientRequest.eventHandlers.<computed> (/usr/local/lib/node_modules/homebridge-tp-link-tapo/node_modules/follow-redirects/index.js:14:24)
    at ClientRequest.emit (node:events:513:28)
    at Socket.socketErrorListener (node:_http_client:502:9)
    at Socket.emit (node:events:513:28)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at processTicksAndRejections (node:internal/process/task_queues:82:21) {
  port: 80,
  address: 'xxx.xxx.xxx.xxx',
  syscall: 'connect',
  code: 'ECONNREFUSED',
  errno: -61,
  config: {
    transitional: {
      silentJSONParsing: true,
      forcedJSONParsing: true,
      clarifyTimeoutError: false
    },
    adapter: [ 'xhr', 'http' ],
    transformRequest: [ [Function: transformRequest] ],
    transformResponse: [ [Function: transformResponse] ],
    timeout: 0,
    xsrfCookieName: 'XSRF-TOKEN',
    xsrfHeaderName: 'X-XSRF-TOKEN',
    maxContentLength: -1,
    maxBodyLength: -1,
    env: { FormData: [Function], Blob: [class Blob] },
    validateStatus: [Function: validateStatus],
    headers: AxiosHeaders {
      Accept: 'application/json, text/plain, */*',
      'Content-Type': 'application/json',
      'User-Agent': 'axios/1.3.4',
      'Content-Length': '352',
      'Accept-Encoding': 'gzip, compress, deflate, br'
    },
    method: 'post',
    url: 'http://xxx.xxx.xxx.xxx/app',
    data: '{"method":"handshake","params":{"key":"-----BEGIN PUBLIC KEY-----\\nMIGfM.....\\n-----END PUBLIC KEY-----\\n"},"requestTimeMils":1680199748319}'
  },
  request: <ref *1> Writable {
    _writableState: WritableState {
      objectMode: false,
      highWaterMark: 16384,
      finalCalled: false,
      needDrain: false,
      ending: false,
      ended: false,
      finished: false,
      destroyed: false,
      decodeStrings: true,
      defaultEncoding: 'utf8',
      length: 0,
      writing: false,
      corked: 0,
      sync: true,
      bufferProcessing: false,
      onwrite: [Function: bound onwrite],
      writecb: null,
      writelen: 0,
      afterWriteTickInfo: null,
      buffered: [],
      bufferedIndex: 0,
      allBuffers: true,
      allNoop: true,
      pendingcb: 0,
      constructed: true,
      prefinished: false,
      errorEmitted: false,
      emitClose: true,
      autoDestroy: true,
      errored: null,
      closed: false,
      closeEmitted: false,
      [Symbol(kOnFinished)]: []
    },
    _events: [Object: null prototype] {
      response: [Function: handleResponse],
      error: [Function: handleRequestError],
      socket: [Function: handleRequestSocket]
    },
    _eventsCount: 3,
    _maxListeners: undefined,
    _options: {
      maxRedirects: 21,
      maxBodyLength: Infinity,
      protocol: 'http:',
      path: '/app',
      method: 'POST',
      headers: [Object: null prototype],
      agents: [Object],
      auth: undefined,
      beforeRedirect: [Function: dispatchBeforeRedirect],
      beforeRedirects: [Object],
      hostname: 'xxx.xxx.xxx.xxx',
      port: '',
      agent: undefined,
      nativeProtocols: [Object],
      pathname: '/app'
    },
    _ended: false,
    _ending: true,
    _redirectCount: 0,
    _redirects: [],
    _requestBodyLength: 352,
    _requestBodyBuffers: [ [Object] ],
    _onNativeResponse: [Function (anonymous)],
    _currentRequest: ClientRequest {
      _events: [Object: null prototype],
      _eventsCount: 7,
      _maxListeners: undefined,
      outputData: [],
      outputSize: 0,
      writable: true,
      destroyed: false,
      _last: true,
      chunkedEncoding: false,
      shouldKeepAlive: false,
      maxRequestsOnConnectionReached: false,
      _defaultKeepAlive: true,
      useChunkedEncodingByDefault: true,
      sendDate: false,
      _removedConnection: false,
      _removedContLen: false,
      _removedTE: false,
      strictContentLength: false,
      _contentLength: '352',
      _hasBody: true,
      _trailer: '',
      finished: false,
      _headerSent: true,
      _closed: false,
      socket: [Socket],
      _header: 'POST /app HTTP/1.1\r\n' +
        'Accept: application/json, text/plain, */*\r\n' +
        'Content-Type: application/json\r\n' +
        'User-Agent: axios/1.3.4\r\n' +
        'Content-Length: 352\r\n' +
        'Accept-Encoding: gzip, compress, deflate, br\r\n' +
        'Host: <IP_ADDRESS>\r\n' +
        'Connection: close\r\n' +
        '\r\n',
      _keepAliveTimeout: 0,
      _onPendingData: [Function: nop],
      agent: [Agent],
      socketPath: undefined,
      method: 'POST',
      maxHeaderSize: undefined,
      insecureHTTPParser: undefined,
      joinDuplicateHeaders: undefined,
      path: '/app',
      _ended: false,
      res: null,
      aborted: false,
      timeoutCb: null,
      upgradeOrConnect: false,
      parser: null,
      maxHeadersCount: null,
      reusedSocket: false,
      host: 'xxx.xxx.xxx.xxx',
      protocol: 'http:',
      _redirectable: [Circular *1],
      [Symbol(kCapture)]: false,
      [Symbol(kBytesWritten)]: 0,
      [Symbol(kEndCalled)]: false,
      [Symbol(kNeedDrain)]: false,
      [Symbol(corked)]: 0,
      [Symbol(kOutHeaders)]: [Object: null prototype],
      [Symbol(errored)]: null,
      [Symbol(kUniqueHeaders)]: null
    },
    _currentUrl: 'http://xxx.xxx.xxx.xxx/app',
    [Symbol(kCapture)]: false
  },
  cause: Error: connect ECONNREFUSED xxx.xxx.xxx.xxx:80
      at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1494:16) {
    errno: -61,
    code: 'ECONNREFUSED',
    syscall: 'connect',
    address: 'xxx.xxx.xxx.xxx',
    port: 80
  }
}
[3/30/2023, 11:09:08 AM] [Tapo] Failed to get info about: xxx.xxx.xxx.xxx | The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined
[3/30/2023, 11:09:08 AM] [Tapo] Retry to connect in 10s : xxx.xxx.xxx.xxx

Plugin Config
 {
            "addresses": [
                "xxx.xxx.xxx.xxx"
            ],
            "name": "Tapo",
            "email"<EMAIL_ADDRESS>            "password": "****************",
            "platform": "HomebridgeTPLinkTapo",
            "_bridge": {
                "username": "AA;AA;AA;AA",
                "port": 33747
            }
        }

Plugin Version
homebridge-tp-link-tapo v3.2.2
Homebridge Version
Homebridge v1.6.0
Node.js Version
Node v18.15.0
NPM Version
9.5.0
Operating System
MacOS Ventura

Only supported devices are Lights and Outlets

Some data. I have successfully integrated the robovac (RV10) into HUBITAT ENVIRONMENT. Does not help here except I developed protocol and command data as part of my errort. Location: https://github.com/DaveGut/TAPO-Robovac-Dev


Fields should not be required by default

I think this is divergent from any CMS I've used - fields should be optional unless configured as required. Adding this to the 1.0 milestone, but let's discuss.

@erquhart I'm with you on this - it's very unnatural and leads to use of a lot of placeholders, particularly since we currently enforce required fields even when saving a draft, so you can't save your work without filling out all the fields.

All fields in Webpop were required by default.
For all the actual integrations I've done of the CMS, I would say that the vast majority of fields tend to be required before a post is ready to go live.
The main issue is around drafts where it's really annoying not being able to save before all fields are filled out. The issue with allowing drafts without required fields, on the other hand, is that missing fields can often break the build.
That might still be an acceptable tradeoff though...

I think folks are more used to the big CMS's, where fields generally aren't required by default, eg. Wordpress. Allowing drafts to save without required fields would probably help, but I still think the expectation is that fields are only required if they're configured to be required.

Weird. I did not expect to be able to publish a WordPress post with no title, body, or anything. Had to test it for myself—confirmed!

Definitely feel they should not be required unless explicitly set

@biilmann I'd like to do this in 1.0, but with an additional fieldsRequiredByDefault flag in the config that can switch it back, both to ease migration and for folks who just want it that way. That should cover our bases pretty well.
(We'll also note it as a breaking change in the release notes.)

@erquhart I like that approach a lot. The option to switch it back is a good idea.

Actually, I need to update this after a productive discussion with @biilmann about why people want this.
writing...


[Accessibility][Status Message - Bot Responses-New Templete button]: Screen reader doesn’t provide any confirmation on activating 'New Template' button.

User Experience:
Screen reader users will get confused that whether the action is performed or not if user will not get any confirmation after executing that action.
Test Environment:
Environment: Chromium Edge (Anaheim) + Narrator
Build Version: 2004 (OS Build 19564.1005)
Microsoft Edge Version: 82.0.425.0 (Official build) canary (64-bit)
Test URL: http://localhost:3000/home
Repro Steps:

Open URL: http://localhost:3000/home
Select 'Bot Responses' link from left navigation.
Navigate within the table using tab key and navigate up to 'New template' button.
Press enter on it and observe the issue.

Actual Result:
Screen reader doesn’t provide any confirmation on activating 'New Template' button. New templates gets added but screen reader kept silent.
Expected Result:
Screen reader should provide the confirmation of newly added templates on activating 'New Template' button.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/sites/accessibility/CELA/Official_Microsoft_Accessibility_Standards/MAS 4.1.3 – Status Messages.docx?d=wc154fbe5702f42fbb2bc87fbb3a23d91&csf=1&e=UcFfpy
Attachment:
MAS4.1.3_New Template.zip

As checked issue still not fixed. Now on activating 'New Template" focus jumps to the first row of the the bot response table.

As checked the issue is resolved. So, HCL is closing the issue.


added a snake game

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #(issue_no)
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Hey @activohuman …Thanks for contributing.. But Can You Create a Issue for the PR...

@activohuman Are you working on this or should I close this?

how can i add readme and where to add it please tell @avinashkranjan

@activohuman Create a Readme along with with script..and Go to Issue tab and create a Script addition issue and describe you script.


Configurable figsize?

Hey,
i would like to ask if there is some reason against a configurable figsize in methods like display_activations
Thank you

@Litschi123 I don't think there's any reasons against a configurable figsize.

Done in 4.2.1


Code snippet error on Lifting up the state tutorial section.

After creating a handle click event in the board component, we replace this.state.value with this.props.onClick().

This leads to an error in the code due to the immediate invocation of the callback function this.props.onClick()

Here is the fix I made. #3839


Improve types export for lucide-react

Closes #1373.
Closes #1420.
Improves Typescript types
Improves Typescript types export by switching to rollup-plugin-dts for types generation.
Added JS Docs
When hovering over a Lucide component you get a JSDoc comment with useful information about the Lucide Icons component. It also shows a small preview of the icons you imported with the link to the lucide website.

Dynamic Imports Map

Experimental Feature
This is icon dynamic import map of all the lucide components. Useful for application that want to show icons dynamically, usecase for this using a CMS to dynamically imports each icon.

Example with NextJS:
import dynamic from 'next/dynamic'
import { dynamicIconImports, LucideProps } from 'lucide-react';

interface IconProps extends LucideProps {
  name: keyof dynamicIcons;
}

const Icon = ({ name, color, size }) => {
  const LucideIcon = dynamic(dynamicIconImports[name])

  return <LucideIcon color={color} size={size} />;
};

export default Icon;

Thank you for your hard work @ericfennis! I'm really looking forward to this pull request to be progressed as it just happens to be what I need.

@jsaunders92 Thank you very much!
I needed to test it before I merged this, and it worked really nicely. You probably don't need the dynamic import version for the latest NextJS since it is streaming everything from the server. But it is nice to have some flexibility. For example, if you want to do client-side rendering. I've added the examples to the documentation.


Follow merge_from_artifact interface update

Overview
Problem
https://github.com/AltimateAI/vscode-dbt-power-user/issues/1140
Solution
Describe the implemented solution. Add external references if needed.
Screenshot/Demo
A picture is worth a thousand words. Please highlight the changes if applicable.
How to test

Steps to be followed to verify the solution or code changes
Mention if there is any settings configuration added/changed/deleted

Checklist

[ ] I have run this code and it appears to resolve the stated issue
[ ] README.md updated and added information about my change

Feel free to review the code or test it in your environment and let us know if it works.

@mdesmet wasn't aware of that you were also working on that. Thanks!
Let me take a look.

@mdesmet #1147 perfectly fixes my problem reported at #1140 👍
looking forward to see that merged. thanks!

fixed in #1147


[AKS] Fix #22321: az aks get-credentials: Fix path separator for Windows when finding kubeconfig_path

Related command

az aks get-credentials
Description

Use os.pathsep instead of hardcoded ":" that breaks things on Windows. Fixes #22321
Testing Guide

History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

AKS

/azp run


Add drug and gene resolvers

[ ] Gene Resolver (kori)
[ ] Drug Resolver (dorian)

https://github.com/dgidb/dgidb/tree/master/app/filters

Taken care of in #29


Code refactoring of echarts module

Additional refactoring to deal with common options from different charts. I'll work on this after other higher priority issues have been closed.


Copy transfers early on

I am actually not sure if this is a good idea. :D

Fixed in the upcoming v0.2 branch!


Refactor test files that use msw

Deprecations
req.body is deprecated. Please use explicit request body reading methods: req.text(), req.json(), req.arrayBuffer() (other methods, like req.formData() are currently not supported).
rest.post('/user', async (req, res, ctx) => {
  const newUser = await req.json()
})

req.body is still present for compatibility reasons but will be removed in the next releases.

I can take a look at this :D

Is this just asking to replace all req.body calls due to deprecation?

Is this just asking to replace all req.body calls due to deprecation?

Yea haha

So I am not too familiar with promises. For some reason this snippet is expecting a promise to be returned if any of the new methods are used.
test("should display wrong password", async () => {     server.use(       rest.post("https://test.supabase.co/auth/v1/token", (req, res, ctx) => {         const { password } = req.text();         if (password !== "bones1234") {           return res(             ctx.status(401),             ctx.json({               message: "Wrong Password",             })           );

sorry for late reply i think u can just wrap it with async await
    server.use(
      rest.post(
        "https://test.supabase.co/auth/v1/token",
        async (req, res, ctx) => {
          const { password } = await req.json();
          if (password !== "bones1234") {
            return res(
              ctx.status(401),
              ctx.json({
                message: "Wrong Password",
              })
            );
          } else {
            return res(
              ctx.status(200),
              ctx.json({
                message: "Success",
              })
            );
          }
        }
      )
    );


RuntimeError: cublas runtime error : an internal operation failed at /pytorch/aten/src/THC/THCBlas.cu:258

I implemented my model referring to the implementation of the examples, when my model running several batches, the error shown in the title occurs.
The whole trace is as followed:
Traceback (most recent call last):
File "train.py", line 549, in
train()
File "/media/////.py", line 120, in forward
topic_representation = self.bertRepresentation(topic_ids,topic_type_ids,topic_mask)
File "/media/////.py", line 113, in bertRepresentation
_, pooled_output = self.bert(input_ids, token_type_ids, attention_mask, output_all_encoded_layers=False)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//Downloads/pytorch-pretrained-BERT-master/pytorch_pretrained_bert/modeling.py", line 736, in forward
output_all_encoded_layers=output_all_encoded_layers)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//Downloads/pytorch-pretrained-BERT-master/pytorch_pretrained_bert/modeling.py", line 409, in forward
hidden_states = layer_module(hidden_states, attention_mask)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//Downloads/pytorch-pretrained-BERT-master/pytorch_pretrained_bert/modeling.py", line 394, in forward
attention_output = self.attention(hidden_states, attention_mask)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//Downloads/pytorch-pretrained-BERT-master/pytorch_pretrained_bert/modeling.py", line 352, in forward
self_output = self.self(input_tensor, attention_mask)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//Downloads/pytorch-pretrained-BERT-master/pytorch_pretrained_bert/modeling.py", line 303, in forward
mixed_query_layer = self.query(hidden_states)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 489, in call
result = self.forward(input, kwargs)
File "/home//anaconda3/lib/python3.6/site-packages/torch/nn/modules/linear.py", line 67, in forward
return F.linear(input, self.weight, self.bias)
File "/home/***/anaconda3/lib/python3.6/site-packages/torch/nn/functional.py", line 1354, in linear
output = input.matmul(weight.t())
RuntimeError: cublas runtime error : an internal operation failed at /pytorch/aten/src/THC/THCBlas.cu:258
In the trace, self.bert is the function to call the BERT model, I am looking forward to all the suggestions from all of you. I have google this problem and there are no methods could solve this problem.
I also run my model using CPU, although it is very slow, no errors.

Python version is 3.6, the cuda version is 10.1.105, cudnn version is  7.51, and pytorch version is 1.0.1.
Thanks all of you!

I am running into almost exactly the same issue. python3.6 cuda 10.0 ubuntu 18.04 and a 1080ti as well. Not sure what it is coming from. I see lots of google results from 2080ti's but those looked like architecture issues.

I tried running with CUDA_LAUNCH_BLOCKING=1 because others have stated this will get us a more accurate error. This led to RuntimeError: Creating MTGP constants failed. at /pytorch/aten/src/THC/THCTensorRandom.cu:33
. Doing some searching as to why this might be. Some people said indexing error somewhere. seems to be happening in the dropout area though so I dont see how that is possible.

https://github.com/pytorch/pytorch/issues/20489
https://github.com/pytorch/pytorch/pull/20886
These look related

Python version is 3.6, the cuda version is 10.0 , cudnn version is 7.5, and pytorch version is 1.1.0, platform is ubuntu 16.04. And GPU is Titan GTX 2080Ti.
Solve this problem.


Add a healthcheck to the Dockerfile

Thanks for PR :) I see 2 issues with those changes:

Port in Reposilite is configurable, so we cannot hard code it to 8080 :/
I'd start after 15s, because budget ARM devices might be too slow to launch whole instance in 5s

Thanks for PR :) I see 2 issues with those changes:

Port in Reposilite is configurable, so we cannot hard code it to 8080 :/
Not sure how to handle this as there are multiple ways of configuring the port, the configuration file can likely be read and port pulled that way but the command line argument is more complex.
Needs more thinking to handle that. Although most docker users probably wont be changing the port since you can redirect it using docker.

I'd start after 15s, because budget ARM devices might be too slow to launch whole instance in 5s
Ajusted

I guess we could generate at startup some kind of a temp file with address pointing to the local instance of Reposilite 🤔

Sounds like a good idea, I might not get chance to do that until the weekend, feel free to edit this PR if you want to have a go. I can make the health check changes if you make just the address file dump.

Sorry, I forgot that there was an enhancement request in this PR. The address is available in .local/reposilite.address file now.

Thanks, let's see how it works :)


[spirv] Avoid extract/insert due to NCHW convolution transpose

This commit adjusts the unrolling configuration for convolution to tile the output innermost dimension by compute size. This allows to avoid generating innermost unit dimensions after unrolling NCHW convolution, which now has affine_map<(d0, d1, d2, d3) -> (d0, d2, d1)> after merging accumulator/result transpose ops. This cleans up the extract/insert ops generated from lowering transpose ops.
Towards https://github.com/iree-org/iree/issues/10536

Can we do the llvm cherry-pick separately?

Sure thing. Here it is https://github.com/iree-org/iree/pull/10713. :)

Can we do the llvm cherry-pick separately?

Sure thing. Here it is #10713. :)

Actually the cherry-pick itself is not standalone; it need fixes to these tests anyway. So I prefer to just land this combined one now.


docs: update hybrid search example to latest langchain

let's update the imports as well:
from langchain_community.document_loaders import PyPDFLoader
from langchain_openai import OpenAIEmbeddings
and remove
from lancedb.pydantic import Vector, LanceModel
also include langchain_openai from pip install


Clean up comments in scripts 02 and 03

@juliablanchard to do this.

I edited these files to decrease horizontal scrolling needed to read them. I also added some vertical spaces to make it easier to read. This means that the code will appear longer than the original version, but I did not change the code itself.
File paths will need to be updated in both scripts, and we also need to update 03_rungridbyLME.R to use terra instead of raster.
I almost done with #4 and I plan to work on these scripts from Monday, August 29. @juliablanchard, does that work for you or would you prefer that I leave this for later?

The get_lme_inputs from the LME_calibration.R script used in 02_runLMEcalibration.R is being updated because paths to do were from GEM.
I have also simplified it significantly and it can now be used with either LMEs or FAO regions. Tested up to line 233 and it produces the same outputs as original version. The code will be updated in the repository once testing is completed for this function.

Script tested and updated up to line 397. Encountered issues with the pbapply function used in line 444.

Encountered issues with the pbapply function used in line 444.

The issue is that pbapply extracts rows in a data frame as a named numeric vector. This vector has a single column with no names and rows with names that match the original data frame columns. For example, the first row of the data frame below:
         f.u       f.v  f.minu   f.minv search.vol
1 0.01558051 0.1735269 1.95498 0.658593  0.7518921

will become:
                 [,1]
f.u         0.1558051
f.v         1.7352690
f.minu     19.5498000
f.minv      6.5859300
search.vol  7.5189210

The run_model function has now been adapted to work with this structure. Information about data structure expected by this function has been added to the script.

Functions: get_lme_inputs, run_model, getError, and LHSsearch from the LME_calibration.R script have been updated to work with updated outputs from 01 script and streamlined. These functions are used in 02_runLMEcalibration.R.
They have been tested against original script and data stored in GEM and outputs are the same. Testing their use with pbsapply function in 02 script.

Since calculations for all LMEs (66 in total) take a long time, I ran out of computing time to process all LMEs. This is an issue because we need all LMEs to be processed before we can save the results. My work around this was to save results for the LHSsearch function every time an LME is processed. Saving can be done by providing a folder where results will be stored, but this is an optional parameter. This parameter can be especially useful in the "refine estimates" section that iterates 1000 times instead of 100 times per LME to estimates the best values for the fishing parameters.
The saved results will have a single row in a csv file, which can be read all together in a single data frame. The advantage of doing this is that we can process only a subset of LMEs when parallelising work. The disadvantage is that when saving results, processing takes longer.
Note that individual LME files can be removed after all results are merged.

Script 02_runLMEcalibration.R has been tidied up, but there are still some comments that I do not know if we should keep or not.  The first section of this script (up to line 34) is almost done running.
The rungridbyLME function in script 03_rungridbyLME.R has been tidied up too. But there are still lots of comments, which I do not know if they should be kept or recorded somewhere.

All plots in sections two of 02_runLMEcalibration.R have been created. This section and the one below have been parallelised. Creating plots for all LMEs take about 2-3 minutes now.
Tuning the fishing parameters values takes approx. 1.5 hours per LME (1000 iterations per LME) and there are 26 LMEs in total. Results are saved for each LME so we can pick up from any LME.

The last section of 02_runLMEcalibration.R, which tunes finishing parameters (iter = 1000) is still being processed (7 out of 26 LME sectors are now complete). I will leave it running overnight.
I divided the rungridbyLME function in script 03_rungridbyLME.R into smaller tasks. This way we can save intermediate outputs more often and can reuse them to run specific tasks (e.g., save input forcings vs getting input forcings + calculating gridded parameters + running gridded model).

The optimisation section of script 02_runLMEcalibration.R is still being completed. We have 14 out 26 LME sectors processed, and I am planning to complete this by tomorrow afternoon.
Work on script 03_rungridbyLME.R is ongoing, but testing is scheduled to start on Thursday 15/08.
@juliablanchard script 02_runLMEcalibration.R is to ready for you to review and decide if comments should be kept or moved somewhere else (e.g., README file). I removed comments that were testing different ways of parallelising work as they did not work as well as the option left uncommented.

Script 02_runLMEcalibration.R has successfully ran for all LME sectors.
Testing script 03_rungridbyLME.R with LME 4 because this is the smallest region.

All comments can be removed from scripts 02 and 03


Remove version: Jackett.Jackett version 0.20.2464

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2464/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


task: add highlight calculation logic for layouts.

Description

Add highlight calculation logic for all basic Layout Components.
Add unit tests to validate the functionality.

PR fixes following issue(s)
Fixes #27004
Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Testing
How Has This Been Tested?

[ ] Manual
[ ] JUnit
[x] Jest
[ ] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


<[REQUEST_DENIED] Google - Geocode [empty]>

when i have installed geocoder, run the examples in readme.md, it shows:

g = geocoder.google([-6.1166283, 106.88594], method='reverse')
g
<[REQUEST_DENIED] Google - Reverse [empty]>

I'm having the same error with normal geocoding
>>> g = geocoder.google('Mbezi, Tanzania')
>>> g
<[REQUEST_DENIED] Google - Geocode [empty]>

Not sure when this got implemented, but I noticed you now need to include the third parameter key=[YOUR_API_KEY]. For example, g = geocoder([-6.1166283, 106.88594], method='reverse', [YOUR_API_KEY])
If that still does not work, you likely need to enable Billing for the project: https://console.cloud.google.com/billing

I have the same problem. I have billing set up and the python googlemaps code works with the apikey, but the geocoder doesn't:
$ python
Python 3.9.5 (default, May 18 2021, 14:42:02) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import googlemaps
>>> import os
>>> gmaps = googlemaps.Client(key=os.getenv('GOOGLE_API_KEY'))
>>> tz = gmaps.timezone((45.15, -75.14))
>>> tz
{'dstOffset': 3600, 'rawOffset': -18000, 'status': 'OK', 'timeZoneId': 'America/Toronto', 'timeZoneName': 'Eastern Daylight Time'}
>>> quit()

$ geocode '45.15, -75.14' --provider google --method timezone
{"raw": {"errorMessage": "You must use an API key to authenticate each request to Google Maps Platform APIs. For additional information, please refer to http://g.co/dev/maps-no-account", "status": "REQUEST_DENIED"}, "status": "ERROR - No results found", "ok": false}


Self-signed SSL

Does the current package support environments with self-signed SSL certs? I can't figure out what I need to do to pass the "verify=False" argument in requests.post()

Hey,
We support it at the server object level.  There is an add_http_options method call that allows you to pass arguments to requests.
You can see an example of this in the sample here: https://github.com/tableau/server-client-python/blob/master/samples/set_http_options.py#L36
Line 36 is the interesting bit there.  Let me know if you run into any problems with setting it.

In addition you may get warnings, the workaround is to put this in your script:
import requests
from requests import InsecureRequestWarning
requests.packages.urllib3.disable_warnings(InsecureRequestWarning)

Great! Thanks for the help

Can we close this issue out, given that I updated the sign-in/sign-out doc to explicitly talk about the workaround for self-signed certificates? https://github.com/tableau/server-client-python/blob/development/docs/docs/sign-in-out.md


RFC: Mac: add some examples and best-practices for mutagen

Proposed changes
Describe the new Docker Desktop two-way file sync feature with some examples and notes on best-practices.
Unreleased project version (optional)
This is for an upcoming Docker Desktop edge release.

FYI @usha-mandya , let me know what you think!

Thank you @djs55. I'll take a look.


Remove dynamic allocations [8256]

This PR removes the following dynamic allocations:

std::function parameter on readParameterListfromCDRMsg

Implies moving ParameterList class to a private header

Unnecessary vector copy on WriterProxy::send
Unnecessary CDRMessage initialization when participant is not secure

Build status:

Linux
Mac
Windows

I cannot reproduce the warning on VS2015 😠 Will try again on Monday

It looks like std::function can throw std::bad_alloc on copy constructor. The old ParameterList::readParameterListfromCDRMsg took a std::function by value, and that may be the reason for the catch. Now that the method is templatized the compiler may not be able to detect a path to that exception.
I guess the try-catch can be removed, since we know the processor (defined right above the try) does not throw.

Build status:

Linux
Mac
Windows


Feature Request: rolling restart

Hi,
We want to implement a rolling restart of an application. An example use case is when a service which is used by the app has been modified and the security groups of the app need to be changed.
Its not going to be too tough to implement it ourselves via some sequential calls to 'cf restart-app-instance', however I'm guessing this must be a fairly common requirement. Are there plans to add it as part of the CLI?
Thanks
Dave

Perhaps have a look in https://plugins.cfapps.io for some CF plugin options
On Mon, Jul 27, 2015 at 3:56 AM, davehr<EMAIL_ADDRESS>wrote:

Hi,
We want to implement a rolling restart of an application. An example use case is when a service which is used by the app has been modified and the security groups of the app need to be changed.
Its not going to be too tough to implement it ourselves via some sequential calls to 'cf restart-app-instance', however I'm guessing this must be a fairly common requirement. Are there plans to add it as part of the CLI?
Thanks
Dave
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry/cli/issues/524

@davehr  Basically, you need to restart the app because of the service change and you're looking for a way to do this instance by instance so that you have no downtime, right?
As Dr.Nic suggested, this plugin for Blue/Green Deployments could work for your use case:  https://github.com/krujos/scaleover-plugin.  It is included on  https://plugins.cfapps.io

If i have a app with 5 instance ,due to some issue/any  other reason  i have to do rolling restart .... instead of below method ( running one after the other for 5 times)
cf app-restart instance 0
cf app-restart instance 1
cf app-restart instance 2
cf app-restart instance 3
cf app-restart instance 4
by using a command in single line how can i run a command for doing rolling restart?.


feat: allow config file by env var

Summary
Allow to set-up configuration file path with SUBSTRA_CLIENT_CONFIGURATION_FILE_PATH
Notes
Fixes FL-1063
Please check if the PR fulfills these requirements

[ ] If necessary, the changelog has been updated
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] The commit message follows the conventional commit specification
For any breaking changes, companion PRs have been opened on the following repositories:

[ ] substra-tests
[ ] substrafl
[ ] substra-documentation

/e2e

End to end tests: :hourglass_flowing_sand: STARTING


Rename 'anyone' account in tests to something else

When testing access control related functionality, we usually have an anyone account that represents an account with no special permissions, e.g.:
it('reverts when anyone attempts to self-destruct', async function () {
  await assertRevert(this.contract.destroy({ from: anyone });
});

anyone is probably not the best name we can find, we should find something that better conveys the meaning of 'anyone else', 'any random account', etc.

How about stranger?

what about guest?

Hmm, I like other better than the other proposals, I think it could work.

My vote is on other as well.

I'd like to try and help work on this!

I'll probably need a bit of guidance though - are we looking to change all references in anyone to "other" in the tests, or only in some specific instances, or only in the description?

I think just the account name: test descriptions to go 'tokens can be transferred to anyone' are actually correct (assuming anyone is no-none :stuck_out_tongue:).

:+1: got it. @nventuro, am I changing each instance of from: anyone to from: other? Or are some of the "anyone's" valid in the tests?
In other words, I'm only changing this:
  it('can be finalized by anyone after ending', async function () {
    await time.increaseTo(this.afterClosingTime);
    await this.crowdsale.finalize({ from: anyone });
  });

to
  it('can be finalized by anyone after ending', async function () {
    await time.increaseTo(this.afterClosingTime);
    await this.crowdsale.finalize({ from: other });
  });

Correct! I don't think we want to leave any instance of anyone: the core issue is that 'anyone' shouldn't be an account name (because it belongs to 'someone'). 'other' implies that that account isn't special (and represents 'anyone', which is why we leave it in the test descriptions :stuck_out_tongue:).

Got it. @nventuro - You can see my first attempt here: https://github.com/OpenZeppelin/openzeppelin-solidity/pull/1718
I changed all the references of "anyone" in the code to "other" and I'm pretty confident I didn't miss any. Just not sure if I might have done a few that weren't meant to have changed. I think I noticed one instance where "other" was declared but never actually used. I could remove that if you'd like.


migrate-monitors : Private minions are not handled correctly. Allow specifying target  location name for private minion.

Summary
Description
Private minion location name end up being different in different accounts and the migration script is unable to find a matching private minion.
Steps to Reproduce
Create two accounts with different private minion names. Try to move to source monitor to target account. It will fail as same private minion location name is not found in target account
Expected Behavior
Source monitor should migrate to target account without errors.
Possible Solution
Allow passing a targetLocationName which should be used when monitor is created in target account
Additional context
The locationName is not the same as displayed in UI. A unique locationName_SomeNumber id is created for the private location.

This is available in a separate branch. Not merged as only 1 customer has asked for this.

Hello Adiosspandit..I am running into an issue when attempitng this migration from one account to another for synthetics.
Probably a user error as I am new to python
I know this comment is not relevant here. Kindly let me know if you are willing to look into my issue so that I can provide more details.
Any help would be  very highly appreciated

Could you send me some more details.
Which script are you trying and what is the error message?
On Thu, Apr 7, 2022 at 12:19 PM GokulKNair @.***> wrote:

Hello Adiosspandit..I am running into an issue when attempitng this
migration from one account to another for synthetics.
Probably a user error as I am new to python
I know this comment is not relevant here. Kindly let me know if you are
willing to look into my issue so that I can provide more details.
Any help would be very highly appreciated
—
Reply to this email directly, view it on GitHub
https://github.com/newrelic-experimental/nr-account-migration/issues/31#issuecomment-1091948329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJMT2E7MRXX7FDYFUYOFTQLVD4DKLANCNFSM5ATZGMZA
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Fixed by https://github.com/newrelic-experimental/nr-account-migration/pull/39


Renovate: resolve dependency conflicts

Summary
The renovate patch PRs have several dependency conflicts and large version bumps that need to be resolved.
Related

#363
#364
#366
cds-snc/platform-core-services#118

After review, all currently failing PRs are related to large major version updates and missing support for peer dependencies.
As such, scan websites can be considered fully onboarded and now just needs some support making the large updates.


Improve documentation test in merkle_tree

Removes unnecessary usage of vector and adds a missing assert. Closes #3353.
Note: @Kixunil in the referred comment wondered about missing 'unused variable' warning. It seems that rustdoc adds allows to the documentation tests. However, these warnings can be enabled on module level. Do we want that? I tried to enable and there are already some occurrences of unused variables and imports. Caveat: it seems that we can't deny warnings on that, only print the warnings to stdout with --show-output.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
18938

Relevant Lines:
22880

💛  - Coveralls

The docs you link suggest #![doc(test(attr(warn(unused))))] ... I think if we changed the warn in there to deny it'd deny?
Anyway very interesting find. I vote yes, let's turn on these lints and fix all the instances.

I think if we changed the warn in there to deny it'd deny?

Right! I opened #3362 for it.


Slackbot unable to join private channel with conversations.join

Description
Hello I am running into a problem that is preventing my slackbot from joining private channels (AKA groups).
So I made sure to add the groups:write scope upon initializing the app, but for whatever reason when i call conversations.join I am faced with the following error:
method_not_supported_for_channel_type
I would have assumed that since I contain the proper scope, then my app should be able to join the private channel. Any suggestions on how to successfully join the private channel? Am I perhaps missing another scope?
Here's some of my code for reference:
const expressReceiver = new ExpressReceiver({
  processBeforeResponse: true,
  signingSecret: process.env.SLACK_SIGNING_SECRET,
  clientId: process.env.SLACK_CLIENT_ID,
  clientSecret: process.env.SLACK_CLIENT_SECRET,
  stateSecret: "my-secret",
  scopes: [
    "chat:write",
    "commands",
    "channels:join",
    "reactions:write",
    "groups:write",
  ],
  installationStore: {
    ...
  },
});

What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[x] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Reproducible in:
package version:
{
  "name": "party-poll-v2",
  "version": "1.0.0",
  "main": "index.js",
  "license": "MIT",
  "dependencies": {
    "@slack/bolt": "^3.12.1",
    "@slack/oauth": "^2.5.4",
    "@vendia/serverless-express": "^4.10.1",
    "axios": "^1.1.3",
    "querystring": "^0.2.1"
  },
  "devDependencies": {
    "serverless-dotenv-plugin": "^4.0.2",
    "serverless-offline": "^10.0.2"
  }
}

node version:
v14.20.1
OS version(s):
macOS Big Sur 11.6.8 (20G730)
Steps to reproduce:

To to slackbot website
Select "Add to Slack"
Authorize
Try running a party poll command, i.e. /partypoll "What is your favorite color?" red blue green in a private channel

Expected result:
User sees slackbot respond and post a poll as a message to the channel
Actual result:
User's command disappears. User sees message /partypoll failed with the error "dispatch_failed"
Express server fails with the following error:
{
  code: "slack_webapi_platform_error",
  data: {
    ok: false,
    error: "method_not_supported_for_channel_type",
    response_metadata: {
      scopes: [
        "chat:write",
        "commands",
        "channels:join",
        "reactions:write",
        "groups:write",
      ],
      acceptedScopes: ["channels:join"],
    },
  },
}

Hey @adrielklein - that's an intentional design decision here and the error message is pretty descriptive. For context, only existing members of private channels can add other members, and that goes for both regular slack users as well as apps with the permissions to groups:write. With that scope you can create a new private channel with your app and add other members to that, but existing private channels where the bot isn't the owner cannot be joined from the outside.
I'm not entirely sure what your use case would be, but perhaps creating a new private channel is an option here?

@adrielklein

So there's no way possible for my slackbot to post any type of message (or even error message) to display to the user that they must use the command in public channels only?

For slash command patterns, you can use ack() method to send a simple reply to the command. This method works everywhere including the DMs/private channels where your app is not allowed to post a message using chat.postMessage API.
app.command("/partypoll", async ({ ack, body }) => {
  if (isMemberOfChannel(body)) {
    // post a message in the channel
  } else {
    // tell the end-user what to do next
    await ack("/partypoll command cannot be used in a private channel or DM, please use the poll only in public channels.");
  }
});

@seratch that's a great idea, however I'm wondering how isMemberOfChannel would work because as far as I'm aware there is no information in body that specifies whether or not the slackbot is a member of that channel.
On top of that, I don't see any data in body that indicates whether or not the channel is private, or a DM.
The only solution I can see to solving this would be a bit more complex where the slackbot calls conversations.list earlier to get all the information about every channel in the workspace and create a map. On subsequent calls for users to create polls, we'd do a lookup in this map based on channel_id to determine if the channel is public, private, or a DM to determine how to proceed.
Clearly that would be a lot more effort and complexity that I'm trying to avoid. Do you know if there's any other data in body that can be used to determine either if the slackbot is already a member of the channel or if the channel is public?

ah just doing a bit more research on this and perhaps I can just use conversations.info after I get back the channelId to determine if it's public or not

ah just doing a bit more research on this and perhaps I can just use conversations.info after I get back the channelId to determine if it's public or not

Turns out I tried this but getting channel_not_found even though I verified that I successfully passed in the valid channelId of the private channel.
For the records I've also added the required scopes for conversations.info
    "channels:read",
    "groups:read",
    "mpim:read",
    "im:read",

So for the record, I was able to find a makeshift solution by using conversations.list to determine if the conversation is public or not:
app.command("/partypoll", async ({ ack, say, body, client }) => {
  const conversationListResponse = await client.conversations.list();
  const isPublicChannel =
    conversationListResponse.channels.filter(
      (channel) => channel.id === body.channel_id
    ).length > 0;
  if (isPublicChannel) {
    ack();
    await client.conversations.join({
      channel: body.channel_id,
    });
    await createPoll(body.text, say, client);
  } else {
    ack(
      "/partypoll cannot be used in a private channel or DM at this time. Please use the poll only in public channels."
    );
  }
});

I'm still curious though about the conversations.join method and limitations to joining the private channel in the first place to create the poll there. In fact, I was looking into other polling apps like Simple Poll and discovered that that app is able to join create polls in private channels seamlessly, so I'm just wondering how it's possible for them to do this given the road block I initially faced with my bot joining a private channel via method_not_supported_for_channel_type.
However, I'm going to table that for since thanks to both of your help I was able to successfully to create the poll to public channels and show private error messages (via ack) for private channels, DMs, and multiple DMs.
Thanks again for your help 🙏 . Will close now.


MAINT: stats: fix thread-safety issues under free-threaded CPython

Fixes unsafe caches for ansari and mannwhitneyu, and adds locks for stats._mvn. _mvn will be rewritten in not-Fortran in the near future, so no need to do more for it that a lock now.
This extracts the changes that need to be backported for 1.15.0rc2 from gh-22125.

Ah good, I was just trying to fix the test fails locally

Yeah I had just kept only the functional changes, but unfortunately the existing tests were digging into private implementation details, so I needed part of the test changes (pretty mechanical, should be fine).


Scoop updates/cleanups don't seem to work with certain app versions

Here are the steps to reproduce:
I'm running latest version of scoop:
Current Scoop version:
a9fa775d (HEAD -> master, origin/master, origin/HEAD) chore(buckets.json): Update scoop-nonportable URL (#3776)

When I run scoop status shows I have IntelliJ-IDEA v 2019.3-193.5233.102 installed.
But when I open IntelliJ it tells me there's an update.
So I run scoop update * and scoop reports:
IntelliJ-IDEA: 2019.3-193.5233.102 (latest version)
Latest versions for all apps are installed! For more information try 'scoop status'

So, based on the workaround mentioned in #3644, I run scoop update -f IntelliJ-IDEA. Scoop dutifully updates IntelliJ to version 2019.3.1-193.5662.53.
But I don't want the old version on my machine, so I run a cleanup scoop cleanup *, only to watch scoop uninstall the new version that it just installed!
Removing IntelliJ-IDEA: 2019.3.1-193.5662.53
Everything is shiny now!

No, scoop, everything is not shiny now.
So my question is, why is scoop not registering the new version as new? I thought that was supposed to be fixed with #3644? And until that is fixed should I just never run cleanup?
Thanks!

Duplicate/Related #2720
Being fixed in #3721

Obsolete.


#48041: WIP: migrate and refactor price element to price element and …

Ref: #41
WIP: migrating and refactoring price element.
However as my experience with vue is minimal I'm not sure about about my idea presented it this work in progress PR so waiting for feedback.

I think it would be better to split up to single "price" element with different variations like default, special/sale, old/regular as then it's more flexible to build different html structure.

Use in price element sth like https://sarahdayan.github.io/dinero.js/ to handle easily different types of showing the price (formatting, currencySymbol ($ at front, zł at the end) or currency (code like USD/PLN), showing FREE (instead of 0 if needed) etc.

If required had price-box could/should be refactored still.

@Igloczek @jszczech ?

@Januszpl easy, there is a queue full of tasks that needs some review or feedback.


Please add support for Docubay.com

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.docubay.com/last-shop-standing/

Description

This is a great on line indie documentaries source, so I believe it is going to  be great adding support for this site as well. Thank you.

Install HLS downloader in chrome, itll give you the m3u link which you can use with youtube-dl


Support Adding Annotations to the ServiceAccount

Enhancement.
We should expose the option for setting annotations on the service account, so we can associate an IAM role to a service account easily.

available in rancher-backup chart version 1.1.100+up1.1.1

can be tested on rancher-backup:2.0.0+up2.0.0-rc2 in rancher:master-head

On master-head commit id: 4d18400

Rancher backups version: 2.0.0+up2.0.0-rc2
Has in edit as yaml this:

serviceAccount:
  annotations: {}


Proposal: Support for docker ps filter to accept network name and list containers

Support for docker ps command to accept network as filter
docker ps --filter network=mynet
Output should contain all the containers inside the network "mynet"

@thaJeztah Can I submit a PR or should I wait for some acceptance?

You can open a pull request, and we can discuss the design during design review (but no guarantee it gets accepted, it may depend on the implementation / changes required)

#dibs

Was implemented.


sf::Font does not properly free freetype glyphs in the destructor.

It appears the glyph cache inside of sf::Font is never properly freed, leaking all used glyphs after the Font is destroyed.
If fonts are repeatedly created and destroyed during a program, the leaked memory can become quite large.
Below is a valgrind memcheck output from a small project I noticed the problem in. Notice that it leaks 14 blocks, which is exactly how many distinct glyphs I use (numbers 0-9, '.' and 'fps'). For unicode text and larger point sizes, each font could easily leak into the 100s of KBs of memory. The sf::Font object is properly destroyed in this example before termination.
==30433== 9,188 (1,120 direct, 8,068 indirect) bytes in 14 blocks are definitely lost in loss record 83 of 85
==30433==    at 0x4C2CE5F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==30433==    by 0x7882EEC: ??? (in /usr/lib/libfreetype.so.6.15.0)
==30433==    by 0x7882F36: ??? (in /usr/lib/libfreetype.so.6.15.0)
==30433==    by 0x788A368: ??? (in /usr/lib/libfreetype.so.6.15.0)
==30433==    by 0x788A6A9: FT_Get_Glyph (in /usr/lib/libfreetype.so.6.15.0)
==30433==    by 0x528165A: sf::Font::loadGlyph(unsigned int, unsigned int, bool, float) const (in /usr/lib/libsfml-graphics.so.2.4.2)
==30433==    by 0x5281CEB: sf::Font::getGlyph(unsigned int, unsigned int, bool, float) const (in /usr/lib/libsfml-graphics.so.2.4.2)
==30433==    by 0x52B222B: sf::Text::ensureGeometryUpdate() const (in /usr/lib/libsfml-graphics.so.2.4.2)
==30433==    by 0x52B277B: sf::Text::draw(sf::RenderTarget&, sf::RenderStates) const (in /usr/lib/libsfml-graphics.so.2.4.2)
==30433==    by 0x52A48A8: sf::RenderTarget::draw(sf::Drawable const&, sf::RenderStates const&) (in /usr/lib/libsfml-graphics.so.2.4.2)
==30433==    by 0x10ED1D: Game::draw_fps(GameTime const&) (Game.cpp:137)
==30433==    by 0x10E627: Game::draw(GameTime const&) (Game.cpp:123)

Are you using the public 2.4.2 release of SFML, or the master revision from git?

This was on 2.4.2.


Split the suppression-list-removal permissions out into their own role

💡 Summary
Consider splitting out the suppression list removal permissions into their own role.
Motivation and context
It would be nice to be able to give some or all CyHy folks the ability to remove addresses from the suppression list themselves, but such folks do not currently need access to send emails.  This would free @team-ois from this simple and quick task that still requires a context-switch.
I suggest starting with @climber-girl and adding other CyHy folk as needed.
Implementation notes
Note that if an entity is granted permission to assume the email-sending role, then that entity will likely also require permission to assume the newly created role to self-manage suppression list removals.  The converse is not true.
Acceptance criteria

[x] Email-sending role and suppression-list-removal role are separate roles.
[x] All entities (groups, users, and accounts) that currently have access to the email-sending role that also require access to the suppression-list-removal role are granted access to said role.
[x] Some (possibly not proper) subset of the CyHy team is granted permission to assume the suppression-list-removal role.

See pull request #46.

I gave @climber-girl the ability to manage the suppression list, and I am waiting for @climber-girl to verify her access before closing this issue.

@climber-girl verified her ability to manage the suppression some time ago.


Incorrect CIGAR string found. CIGAR string can only have I/D/H/S/X/=

./minimap2 -cx asm5 asm1.fa asm2.fa > aln.paf
fixchr -c aln.paf -r asm1.fa -q asm2.fa -F P

I got an error:

Hi!  @mnshgl0110 .
How to fix it?
Thank you for your reply!

Sorry for the missing documentation. Please use the --eqx parameter of minimap2 as described here.


Update Schnorr Multisig spec to use checkbits triggering

This way of triggering the new Schnorr mode seems to have far cleaner logic.
PR created for discussion and consideration of this approach.

wrong branch


Running tf from docker with --read-only mode

TensorFlow version: 2.2.0
I'd love to run my app with docker run --read-only option, but startup fails when tf runs tests that try to create a temporary folder.

  import tensorflow as tf  # noqa
File "/usr/local/lib/python3.6/dist-packages/tensorflow/__init__.py", line 41, in <module>
  from tensorflow.python.tools import module_util as _module_util
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/__init__.py", line 132, in <module>
  from tensorflow.python.platform import test
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/test.py", line 24, in <module>
  from tensorflow.python.framework import test_util as _test_util
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/test_util.py", line 36, in <module>
  from absl.testing import parameterized
File "/usr/local/lib/python3.6/dist-packages/absl/testing/parameterized.py", line 186, in <module>
  from absl.testing import absltest
File "/usr/local/lib/python3.6/dist-packages/absl/testing/absltest.py", line 244, in <module>
  flags.DEFINE_string('test_tmpdir', get_default_test_tmpdir(),
File "/usr/local/lib/python3.6/dist-packages/absl/testing/absltest.py", line 180, in get_default_test_tmpdir
  tmpdir = os.path.join(tempfile.gettempdir(), 'absl_testing')
File "/usr/lib/python3.6/tempfile.py", line 437, in gettempdir
  tempdir = _get_default_tempdir()
File "/usr/lib/python3.6/tempfile.py", line 372, in _get_default_tempdir
  dirlist)
FileNotFoundError: [Errno 2] No usable temporary directory found in ['/tmp', '/var/tmp', '/usr/tmp', `'/opt/proj']

I wonder if it's possible to turn off these tests?
Provide param to turn off tests on starting up?
Those who are willing to run tf from docker in a read-only mode which is safer will benefit from this possibility.
If it's already possible please share how.

This is not something we'll be able to support. Sorry.
You may have some success with the --tmpfs flag.


:wrench: chore(changeset): fix paths for windows

Summary
Changeset script has issues with windows systems
Additional Information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

/cib


Triple slash reference should include extension '.d.ts' or not?

/// <reference path='../../../typings/index.d.ts' />
or
/// <reference path='../../../typings/index' />
?
Thanks!
tsc recognizes the file without .d.ts extension, but vscode's ts validation has a different behavior.
This makes a headache communication between typescript coder.

Moving to the TS team since the error is generated by the tsserver. I have no idea what makes the difference between tsserver and tsc here.

This issue was moved to Microsoft/TypeScript#9116


Add codespaces config

The intent here is to add config for codespaces that does not break our existing docker-compose dev env setup.
Seems to work! :smile:

The entrypoint needs an update to the updated working directory.
https://github.com/OfficeMomsandDads/scheduler/blob/23388a0ac65f152949eab220bf3f307ae9cd6d0e/docker/rails-entrypoint#L4

Thanks!
Ready for re-review. I'm also building from scratch both locally & in codespaces, just to ensure everything is working in both contexts one last time.


Force Download Web App

Hi there!
I'm wondering if there's a way to download the web app like calling a function instead of waiting the pop up to appear. I'm planning to add a button somewhere on the website to install the system instead of having them to clear cache for the pop up to reappear.
Thank you.

Should have researched deeper than asking it directly here. My first impression is that this is a completely separate function but found this gem: https://developer.mozilla.org/en-US/docs/Web/Progressive_web_apps/Tutorials/js13kGames/Installable_PWAs


Allow simultaneous highlighting of defaulted and hidden stacks in index

Fixes a bug where the Highlight Defaulted setting would be ignored when Edit Mode was enabled.

My diff didn't show that I had changed all the tabs to spaces. Let me fix that

Had to reset the file and make the changes anew for it to recognize the changes properly.

Had to force-push to revert accidental conversion of all indentation tabs to spaces


[WIP] Add a react native loader to make it fully customizable

Issue
#14
Description:

Add a React native loader, without showing the basic html loader, in order to make it fully customizable

It's in WIP because I need to test it later (blind coded)

Unfortunately I didn't find out the time and I'm not using the module anymore, if someone wants to continue, feel free to cherry-pick or request me access...


octant: Update to version 0.5.0

Bump version
Signed-off-by: GuessWhoSamFoo<EMAIL_ADDRESS>

b50451711e04a12814a0abe10e94a42672b79db8


connecting to deluge on other servers

would it be possible for me to connect storm to deluge running on my vps as i just use ftp to get my files from the vps since that gives me faster speeds.

Same question. Can we set hostname to the URL for Deluge on a hosted seedbox?


[Bug]: Segment file first binpack strategy leads to inability to optimize SplitTasks

What happened?

OptimizeEvalutor found some partitions that need to optimize, but Planer didn't get any tasks.
Affects Versions
master
What engines are you seeing the problem on?
AMS
How to reproduce
No response
Relevant log output
No response
Anything else
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for feeding back.
Is there any more conditions to reproduce this situation?

Thanks for feeding back. Is there any more conditions to reproduce this situation?

I believe this case would happen if there are a certain number of segment data files in the table


ISPN-6306 Distributed Streams should also be used for Replication Cache

Fixed issue with filterBySegment / REPL cache and failover

https://issues.jboss.org/browse/ISPN-6306

Added changes

Integrated. Thanks @wburns !


Update - Translated to English

There were parts of answers that were in Russian. I translated them to English and edited the links to sections.

Great job, Dipeeka. Thank you!


Added tcp/hello_protocol acknowledgment commands, verbose logging, and more clear documentation

I added more commands to the communication between tcp.py and hello_protocol.sh. After the pi receives a command, it responds alerting piman that the command was successfully received. The pi will send another message after the instruction was finished executing. I also changed the tcp documentation to make it less misleading about the tcp.py/hello_protocol.sh exchange.

All absolute paths in the documentation are now relative, and I changed all if statements that resulted in only logging to a dictionary dispatch defined with lambda expressions. I left the other if statements where the command results in action by the server as is because I don't think taking those out into separate functions will make the code more clear.


Investigate possible race condition between HighlightJS and Mermaid

There is a known conflict between HighlightJS and Mermaid which Academic attempts to workaround.
Based on recent comments at the end of the #1298 thread, it appears that there may be a race condition present between Highlight's initHighlightingOnLoad and the Mermaid workaround. If initHighlightingOnLoad is async, then that could be the source of the issue.
Workarounds
There are two simple user-based workarounds if Mermaid does not display correctly:

Disable math math: false on the pages that have Mermaid diagrams
Use the built-in diagram shortcode

@Bertbk and @laughingrice: if possible, please can you attempt to update to the latest master version and test if you still experience any issues with diagram rendering whilst math is enabled on the page (when using the standard backtick approach rather than the 2 alternative approaches above).

Hello @gcushen ,
I upgraded Academic to the last version but it still failed on my computer. The following code (without the white spaces):
    The below diagram illustrates the workflow:
    ```mermaid
    graph LR
        A("Test") -->|Hello|B(Test)
    ```

lead to that:

On the Academic docs page, the same problem appears (not everytime but after some refresh!):

Just pulled the latest master. Still having problems.
Using the following code in the markdown, under Firefox on Mac it usually renders as
graph TD; A[test]-->B; A-->C; B-->D; C-->D;
Although if I reload a few times, every once in a while it renders correctly. With Safari it usually renders correctly, but if I reload a few times, every once in a while it fails to render. Will see if I can test under linux to compare when I have access.
graph TD;
  A[test]-->B;
  A-->C;
  B-->D;
  C-->D;

Thanks

On Nov 21, 2019, at 16:45, George<EMAIL_ADDRESS>wrote:
@Bertbk https://github.com/Bertbk and @laughingrice https://github.com/laughingrice: if possible, please can you attempt to update to the latest master version and test if you still experience any issues with diagram rendering whilst math is enabled on the page (when using the standard backtick approach rather than the 2 alternative approaches above).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/gcushen/hugo-academic/issues/1434?email_source=notifications&email_token=AAKTVMI7A5PRMWSGE5JRSUTQU36PHA5CNFSM4JPZRJTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE3YG2A#issuecomment-557286248, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKTVMOAD4UIKJKYKFLFLJTQU36PHANCNFSM4JPZRJTA.


Langfuse question

Request for Explanation Regarding Changes in Internal Callback Method Management Logic in Langflow Upgrade to v1.0 for Langfuse Pull Request
I'm requesting an explanation regarding changes in the internal callback method management logic in the Langflow upgrade to v1.0. It seems that records are not being correctly logged in Langfuse as a result. This request is to gather information for the purpose of creating a pull request.

Hi @YamonBot
Do you still need any support with this issue?

#2120


Reproduce the same reward graph as AWS Console

Useful for users to provide a familiar point of reference
Useful for testing to prove our data analysis is not completely wrong - i.e. our graph should look identical to the console, doh!

Not going to do more work on this - equivalent graph are possible - but won't ever look quite the same anyway since DRG uses two graphs to improve clarity anyway


Automatic CSV downloads

It would be great if you brought back CSV downloads. Or even if that's not possible, to enable automated downloads via server side scripting. I miss both from the old openaddresses.

Yep, I agree. I added this ticket to track that: https://github.com/openaddresses/batch/issues/79


Fix SILVerifier of open_existential_addr and opaque value loads

A 'load [copy]' of an 'open_existential_addr [immutable]' address is okay.

@swift-ci Please smoke test


Added favicon issue #44

Added favicon to the website, this will make the website more attractive

kindly mention your issue number at the pull request time.

any  update ?

any update ?
I made pull request right?
what I'm supposed to do next ???

thankyou for contribution


Update README.md

Adding a line that for local build an execution of "--init-issuer" is needed for the demo.

Thanks for pointing this out @xxLestadxx :+1:


Issue when compiling native images with PineconeVectorStore

seems to be a regression because this worked before. I sat with @joshlong and we managed to get it working again by adding the following hints:
  class Hints implements RuntimeHintsRegistrar {

    @Override
    public void registerHints(RuntimeHints hints, ClassLoader classLoader) {
        for (var t : Set.of(com.google.protobuf.Value.class,
                com.google.protobuf.Value.Builder.class,
                com.google.protobuf.Struct.class)) {
            hints.reflection().registerType(t, MemberCategory.values());
        }
    }
}

Fixed in https://github.com/spring-projects/spring-ai/pull/1266