Adding sync_wait_with_variant

Any background context you want to provide?
P2300R5: sync_wait_with_variant
sync_wait_with_variant, which accepts any sender, but returns an optional whose value type is the variant of all the possible tuples sent by the input sender:
Checklist

[x] I have added a new feature and have added tests to go along with it.

Also, now that you have a working version that duplicates 95% of the sync_wait code, would you see a way to combine the implementations of sync_wait and sync_wait_with_variant to reduce the amount of code repetition?

Sure, will do.

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-07-18T15:18:52+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a9532c79274846535f93521e2f0c09f6e3c33857b77Datetime2022-05-31T15:13:01.357969+02:002022-07-18T17:31:13.698133+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid01193nid01193ClusternamedaintdaintEnvfileComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch=InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-07-18T15:18:52+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a9532c79274846535f93521e2f0c09f6e3c33857b77Datetime2022-05-31T15:13:18.026239+02:002022-07-18T17:31:30.343351+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid01193nid01193ClusternamedaintdaintEnvfileComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)-(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2021-11-10T19:14:21+00:002022-07-18T15:18:52+00:00HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a32c79274846535f93521e2f0c09f6e3c33857b77Datetime2021-11-10T20:28:18.266961+01:002022-07-18T17:31:45.627798+02:00Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid00120nid01193ClusternamedaintdaintEnvfileExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95258527e18e1e92797cfb624c87d248a63d461997HPX Datetime2022-05-31T12:57:29+00:002022-07-19T16:50:56+00:00Datetime2022-05-31T15:13:01.357969+02:002022-07-19T19:11:51.673858+02:00EnvfileClusternamedaintdaintHostnamenid01193nid01323Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95258527e18e1e92797cfb624c87d248a63d461997HPX Datetime2022-05-31T12:57:29+00:002022-07-19T16:50:56+00:00Datetime2022-05-31T15:13:18.026239+02:002022-07-19T19:12:08.672710+02:00EnvfileClusternamedaintdaintHostnamenid01193nid01323Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)-(=)Stream Benchmark - Scale(=)(=)=Stream Benchmark - Triad(=)-(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a258527e18e1e92797cfb624c87d248a63d461997HPX Datetime2021-11-10T19:14:21+00:002022-07-19T16:50:56+00:00Datetime2021-11-10T20:28:18.266961+01:002022-07-19T19:12:24.086403+02:00EnvfileClusternamedaintdaintHostnamenid00120nid01323Compiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)=(=)InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a958204a4e9486723f7c374924d209a682ae3409441HPX Datetime2022-05-31T12:57:29+00:002022-07-24T14:20:55+00:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid01193nid01260Datetime2022-05-31T15:13:01.357969+02:002022-07-24T16:36:41.128162+02:00EnvfileComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a958204a4e9486723f7c374924d209a682ae3409441HPX Datetime2022-05-31T12:57:29+00:002022-07-24T14:20:55+00:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid01193nid01260Datetime2022-05-31T15:13:18.026239+02:002022-07-24T16:36:58.275781+02:00EnvfileComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)-(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad-(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a8204a4e9486723f7c374924d209a682ae3409441HPX Datetime2021-11-10T19:14:21+00:002022-07-24T14:20:55+00:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Hostnamenid00120nid01260Datetime2021-11-10T20:28:18.266961+01:002022-07-24T16:37:13.706292+02:00EnvfileExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)=(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-07-31T14:05:57+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95a866cc4e205b418aae54a0e3323f8cc8eec40d03Hostnamenid01193nid01193Datetime2022-05-31T15:13:01.357969+02:002022-07-31T16:22:36.824275+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-07-31T14:05:57+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95a866cc4e205b418aae54a0e3323f8cc8eec40d03Hostnamenid01193nid01193Datetime2022-05-31T15:13:18.026239+02:002022-07-31T16:22:53.800696+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)-(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)-(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2021-11-10T19:14:21+00:002022-07-31T14:05:57+00:00HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01aa866cc4e205b418aae54a0e3323f8cc8eec40d03Hostnamenid00120nid01193Datetime2021-11-10T20:28:18.266961+01:002022-07-31T16:23:09.089371+02:00ClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-08-10T18:11:09+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95723fce0a95b8b5e4deb97521f5d5b6763eddc9ceCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileHostnamenid01193nid01159Datetime2022-05-31T15:13:01.357969+02:002022-08-10T20:26:22.517883+02:00ClusternamedaintdaintComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-08-10T18:11:09+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a95723fce0a95b8b5e4deb97521f5d5b6763eddc9ceCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileHostnamenid01193nid01159Datetime2022-05-31T15:13:18.026239+02:002022-08-10T20:26:39.667478+02:00ClusternamedaintdaintComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2021-11-10T19:14:21+00:002022-08-10T18:11:09+00:00HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a723fce0a95b8b5e4deb97521f5d5b6763eddc9ceCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0EnvfileHostnamenid00120nid01159Datetime2021-11-10T20:28:18.266961+01:002022-08-10T20:26:54.843070+02:00ClusternamedaintdaintExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

@hcq9102 things should be fine now. Please note that I also have rebased this branch onto master.

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-08-29T16:20:27+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a954830586c0a05305994e4faf0b1f5cb5a45ba1650Datetime2022-05-31T15:13:01.357969+02:002022-08-29T19:15:04.416369+02:00Hostnamenid01193nid00013EnvfileClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch-InfoPropertyBeforeAfterHPX Datetime2022-05-31T12:57:29+00:002022-08-29T16:20:27+00:00HPX Commitd5655f89770ba953e8de3ceb349567d5fcb74a954830586c0a05305994e4faf0b1f5cb5a45ba1650Datetime2022-05-31T15:13:18.026239+02:002022-08-29T19:15:21.732327+02:00Hostnamenid01193nid00013EnvfileClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2021-11-10T19:14:21+00:002022-08-29T16:20:27+00:00HPX Commit71d8dbe40b737be02ea734f9d98fcc0b1e21f01a4830586c0a05305994e4faf0b1f5cb5a45ba1650Datetime2021-11-10T20:28:18.266961+01:002022-08-29T19:15:37.032623+02:00Hostnamenid00120nid00013EnvfileClusternamedaintdaintCompiler/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0/apps/daint/SSL/HPX/packages/llvm-11.0.0/bin/clang++ 11.0.0Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

This can be closed as it is subsumed by #6015

#6015 got merged. We don't need this one anymore.(sync_wait_with_variant integrating with sync_wait)


Less compiling

Less files are not compiled into .css.
Instead they are handled by watchCopy.
Layout: src/css/app.less

Apparently it needs src/client/styles/app.less.
The read me says it's only src/client/

The import problem is gone now all imports work!
Also I am very happy a .less error doesn't stop id-builder!!!
id-project breaks down once that happens.
This is a lot better! =D


StatorRunner will refresh Config.system on each schedule_interval

Fixes a bug with System.load_user using wrong options_class.
Aysnc Config.aload_*
StatorRunner will freshen up Config.system on each scheduled interval

Appreciate the naming of the new methods here


@microsoft/teams-manifest requires an insecure axios version

Prerequisites
@microsoft/teams-manifest "axios": "^0.21.2",
see https://www.npmjs.com/package/@microsoft/teams-manifest?activeTab=code
An issue discovered in Axios 0.8.1 through 1.5.1 inadvertently reveals the confidential XSRF-TOKEN stored in cookies by including it in the HTTP header X-XSRF-TOKEN for every request made to any host allowing attackers to view sensitive information.

https://github.com/advisories/GHSA-wf5p-g6vw-rhxx
Expected behavior
Move to Version 1.6.0
Context
I shared this information with Microsoft Security Response Center but they closed the case with some bullshit bla bla about not having received enough information. Since I could not find a Github repo for @microsoft/teams-manifest and this package depends on it, and is also maintained by Microsoft, I am opening the ticket here.
However here is my attempt at tagging the NPM maintainers here on Github:
@zhyuer, @nintan
Since the security issue is public anyway and any hacker can easily determine vulnerable packages, I guess using a public channel for this isn't making matters worse.

fixed via https://github.com/OfficeDev/Office-Addin-Scripts/pull/825

@millerds from what I can tell there remains unsolved issues:
axios  0.8.1 - 1.5.1
Severity: moderate
Axios Cross-Site Request Forgery Vulnerability - https://github.com/advisories/GHSA-wf5p-g6vw-rhxx
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/@microsoft/teams-manifest/node_modules/axios
node_modules/@microsoft/teamsfx-api/node_modules/axios
node_modules/@microsoft/teamsfx-core/node_modules/axios
  @microsoft/teams-manifest  <=0.1.2
  Depends on vulnerable versions of axios
  node_modules/@microsoft/teams-manifest
    @microsoft/teamsfx-api  <=0.22.6
    Depends on vulnerable versions of @microsoft/teams-manifest
    Depends on vulnerable versions of axios
    node_modules/@microsoft/teamsfx-api
  @microsoft/teamsfx-core  <=2.0.6
  Depends on vulnerable versions of @microsoft/teamsfx-api
  Depends on vulnerable versions of axios
  node_modules/@microsoft/teamsfx-core
    @microsoft/teamsfx-cli  *
    Depends on vulnerable versions of @microsoft/teamsfx-api
    Depends on vulnerable versions of @microsoft/teamsfx-core
    node_modules/@microsoft/teamsfx-cli
      office-addin-dev-settings  >=1.11.0
      Depends on vulnerable versions of @microsoft/teamsfx-cli
      node_modules/office-addin-dev-settings
        office-addin-debugging  >=4.3.10
        Depends on vulnerable versions of office-addin-dev-settings
        node_modules/office-addin-debugging

7 moderate severity vulnerabilities

ie. https://www.npmjs.com/package/@microsoft/teamsfx-cli?activeTab=dependencies depends on https://www.npmjs.com/package/@microsoft/teamsfx-core?activeTab=dependencies which in turn depends vulnerable versions of axios
https://www.npmjs.com/package/@microsoft/teamsfx-core?activeTab=code

You are correct . . . only some of the vulnerabilities have been fixed . . . still working that that team to have their things fixed.

looks like the latest beta releases finally address this https://www.npmjs.com/package/@microsoft/teamsfx-core/v/2.0.7-beta.2024012307.0?activeTab=code

Looks promising . . . now we just need to see the core cli package take a dependency on a shipped 2.0.7 package.

sigh .. still no 2.0.7 release :-/

I believe it's fixed now.

well ...
# npm audit report

axios  0.8.1 - 0.27.2
Severity: moderate
Axios Cross-Site Request Forgery Vulnerability - https://github.com/advisories/GHSA-wf5p-g6vw-rhxx
fix available via `npm audit fix`
node_modules/@azure/identity/node_modules/axios
node_modules/@azure/msal-node/node_modules/axios
node_modules/@microsoft/teams-manifest/node_modules/axios
node_modules/@microsoft/teamsfx-api/node_modules/axios
node_modules/@microsoft/teamsfx-cli/node_modules/axios
node_modules/@microsoft/teamsfx-core/node_modules/axios
  @azure/identity  1.2.0-alpha.20200903.1 - 2.0.0-beta.6
  Depends on vulnerable versions of @azure/msal-node
  Depends on vulnerable versions of axios
  node_modules/@azure/identity
    @microsoft/teamsfx-cli  <=2.0.2-rc-hotfix.0 || >=2.0.3-alpha.313138833.0
    Depends on vulnerable versions of @azure/core-http
    Depends on vulnerable versions of @azure/identity
    Depends on vulnerable versions of @microsoft/teamsfx-api
    Depends on vulnerable versions of @microsoft/teamsfx-core
    Depends on vulnerable versions of axios
    node_modules/@microsoft/teamsfx-cli
      office-addin-dev-settings  1.11.0 - 2.0.4
      Depends on vulnerable versions of @microsoft/teamsfx-cli
      node_modules/office-addin-dev-settings
        office-addin-debugging  4.3.10 - 4.6.7
        Depends on vulnerable versions of office-addin-dev-settings
        node_modules/office-addin-debugging
  @azure/msal-node  <=1.14.5
  Depends on vulnerable versions of axios
  Depends on vulnerable versions of jsonwebtoken
  node_modules/@azure/msal-node
  @microsoft/teams-manifest  <=0.1.3-rc-hotfix.0
  Depends on vulnerable versions of axios
  node_modules/@microsoft/teams-manifest
    @microsoft/teamsfx-api  <=0.22.7-rc-hotfix.0
    Depends on vulnerable versions of @azure/core-http
    Depends on vulnerable versions of @microsoft/teams-manifest
    Depends on vulnerable versions of axios
    node_modules/@microsoft/teamsfx-api
  @microsoft/teamsfx-core  <=2.0.6
  Depends on vulnerable versions of axios
  Depends on vulnerable versions of xml2js
  node_modules/@microsoft/teamsfx-core

jsonwebtoken  <=8.5.1
Severity: moderate
jsonwebtoken unrestricted key type could lead to legacy keys usage  - https://github.com/advisories/GHSA-8cf7-32gw-wr33
jsonwebtoken's insecure implementation of key retrieval function could lead to Forgeable Public/Private Tokens from RSA to HMAC - https://github.com/advisories/GHSA-hjrf-2m68-5959
jsonwebtoken vulnerable to signature validation bypass due to insecure default algorithm in jwt.verify() - https://github.com/advisories/GHSA-qwph-4952-7xr6
fix available via `npm audit fix`
node_modules/jsonwebtoken

xml2js  <0.5.0
Severity: moderate
xml2js is vulnerable to prototype pollution - https://github.com/advisories/GHSA-776f-qx25-q3cc
fix available via `npm audit fix`
node_modules/xml2js
  @azure/core-http  <=2.3.1 || 3.0.0-alpha.20221110.1 - 3.0.0
  Depends on vulnerable versions of xml2js
  node_modules/@azure/core-http

ok with some help of Dependabot I have now managed to bump everything up sufficiently.
npm install --save
npm WARN deprecated<EMAIL_ADDRESS>A newer major version of this library is available. Please upgrade to the latest available version.

added 174 packages, removed 296 packages, changed 19 packages, and audited 1663 packages in 13s

240 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

I guess getting @azure/msal-node bumped would be good but at least isn't a security (known) issue at this point.


♻Add default auth for KF servers

Closes #52
Any sever with an IP address starting with 10.10.1 will require auth by default.

I have started working on #24 - I don't think it will take too long if we just want to wait for that

Closing in favor of #58
(Going to add another default server to the list with the new configuration)


Blueray always gets subtitle burned into video ínstead of activateable subtitle

[x] I have checked the wiki
[x] I have searched for similar issues

Describe the bug
Subtitles get burned into the MP4 file when ripping a Blue-ray, but not when ripping a DVD, the later one is desired. Subtitle for Blue-ray should be added as a optional to activate subtitle.
To Reproduce
Steps to reproduce the behavior:

Rip a Blueray with this handbrake settings.
HB_ARGS_BD:

--markers --audio-lang-list eng,deu,spa,fra --all-audio --audio-copy-mask aac,ac3,mp3,dts,dtshd --audio-fallback aac --subtitle-lang-list=eng,deu,spa,fra --first-subtitle --subtitle-burned=none --quality 24.0 --format av_mp4 --encoder-preset slower

I use almost the same settings for Handbrake in ARM when processing a DVD.
HB_ARGS_DVD:
--markers --audio-lang-list eng,deu,spa,fra --all-audio --audio-copy-mask aac,ac3,mp3,dts,dtshd --audio-fallback aac --subtitle-lang-list=eng,deu,spa,fra --first-subtitle --subtitle-burned=none --quality 24.0 --format av_mp4 --encoder-preset veryslow

In a DVD when I play the MP4 with VLC I can activate a subtitle in the menu, in a BL it is fixed text in the video, it's not possible to activate another subtitle or deactivate the subtitle at all.
It somehow looks like this Handbrake parameters are not working for the Blue-ray rip. I would be very happy if you can point me in the right direction to troubleshoot this.
I also checked that /opt/arm is up-to-date and also the system is up to date.
Environment
[OS Distribution and version (run "cat /etc/lsb-release")]
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.2 LTS"

[ARM Release Version or if cloning from git branch/commit (run "git branch" and "git log -1" to get this)]
v2_devel

Log file
[Run the rip in DEBUG and drag and drop the log file onto this comment box]
[>>>>> DRAG AND DROP LOG FILE HERE <<<<<]
log.log

OK, I did not find this info in the docs, it reads

Bluray PGS Subtitles
With MP4, you can burn ONLY 1 subtitle track into the video.
You can not pass-through PGS into MP4 as this file format does not support it.
With MKV, you can pass-through multiple PGS tracks. These are not burned into the video unless you choose to do so however you can only burn 1 subtitle track into the file. The rest must be passed through.

So I only have to switch the container format to mkv I guess and that would solve it? The MKV container would still enclose the same H.264 with all the audio tracks and would additionally contain the subtitles?

But it would still be H.264 in the container.

Yes
Did this solve your issue ?

Did this solve your issue ?

Yes, thanks for your patience and sorry for self solving my issue. I got some reaction to it in the Fediverse, which actually steered me in the right direction and now I get .mp4 with mkv container and H.264 I guess.


golang can't set properties

the version of golang  can't set properties in message

try this:
msg.GetProperties()["key"] = "value"

the version of golang can't set properties in message

https://github.com/apache/rocketmq-clients/pull/432


The terminal button doesn't restore the terminal pane on windows

After closing the terminal using the X button clicking on the terminal icon does not restore the closed pane, search and problems does and then you can switch back to the terminal.

Same issue on macOS

I can't reproduce this

Thanks for reporting this issue. I'm not able to reproduce it, and I would like to know if the issue is still present in the current master branch. Thanks!

Treating as resolved


x_nucleo_iks01a3 sample is not working anymore after #20560 has been merged

x_nucleo_iks01a3 sample does not print any message out on the console. Seems stuck somewhere. This happened after #20560. It seems to me the #20560 is not the source of the problem, but simply highlighted an issue.
To Reproduce
I used a nucleo F401re board with a x_nucleo_iks01a3 shield.
Then:
cd samples/shields/x_nucleo_iks01a3/standard/
mkdir build && cd build
cmake -DBOARD=nucleo_f401re ..
make
No messages on console.
System is stuck.
I debugged a bit and it seems to me that, maybe, a  "k_sem_give(&data->device_sync_sem);" is missing in i2c v1 implementation.

Hello @FRASTM can you take a look to this issue?
To recreate it you would need a x_nucleo_iks01a3.

As I did not find any X-NUCLEO-IKS01A3 here, I tried with a X-NUCLEO-IKS01A1 shield  and get the boot msg:

X-NUCLEO-IKS01A1 sensor dashboard
HTS221: Temperature: 23.8 C
HTS221: Relative Humidity: 41.5%
LPS25HB: Pressure:99.0 kpa
LIS3MDL: Magnetic field (gauss): x: 0.4, y: -0.3, z: 0.2
LSM6DS0: Acceleration (m.s-2): x: -0.1, y: 0.1, z: 9.9
30:: lis3mdl trig 0

Is the pb you report specific to a x_nucleo_iks01a3 ?

As I did not find any X-NUCLEO-IKS01A3 here, I tried with a X-NUCLEO-IKS01A1 shield and get the boot msg:

X-NUCLEO-IKS01A1 sensor dashboard
HTS221: Temperature: 23.8 C
HTS221: Relative Humidity: 41.5%
LPS25HB: Pressure:99.0 kpa
LIS3MDL: Magnetic field (gauss): x: 0.4, y: -0.3, z: 0.2
LSM6DS0: Acceleration (m.s-2): x: -0.1, y: 0.1, z: 9.9
30:: lis3mdl trig 0

Is the pb you report specific to a x_nucleo_iks01a3 ?

Yes.
I think the issue is related to the number of sensor requests to the I2C bus.
x_nucleo_iks01a1 has too few sensors

@FRASTM
Is it possible to get one x_nucleo_iks01a3?

Yes, one shield x_nucleo_iks01a3 has been purchased and should arrive within the week

FWIW I can confirm this is specific to ST, as the demo and i2c scanner work with the IKA01A3 on Nordic.

FWIW I can confirm this is specific to ST, as the demo and i2c scanner work with the IKA01A3 on Nordic.

@pabigot
Which hash id are you using?

It works on current master at 667886d63ddb1a75e5c34b4dd407093b4dfc254d

It works on current master at 667886d
west build -b nrf52_pca10040 -- -DSHIELD=x_nucleo_iks01a3

Can you quickly try with 732b815f ?
To be honest the issue disappeared also in my case with latest master even if there are no changes to i2c driver. It is a tricky issue.

Test command was:
rm -rf build && west build -b nucleo_l476rg -- -DSHIELD=x_nucleo_iks01a3 && west flash

AFAICT the sample works, from just before 732b815f4be46a0e0c1b540e8274a38b345e7276 through current master 16ba258052552a65d4041b8b0bf1db02643b2b56, but only with a power-down reset.  A running example will fail if the reset button on the board is pressed.
The output suggests the LSM6DSO initialization doesn't work with soft restart.
I can reproduce the soft-reset failure on the nrf52_pca10040 as well, and there I get an I2C transfer error, so it's not an ST I2C problem but probably an issue with communicating with an LSM6DSO that had already been configured and started.
I think it'd be fair to close this issue, but whichever of us touches the lsm6dso next should confirm and fix the soft-reset problem.  It'll be on my task list as part of the GPIO API rework if nobody else gets to it first.

Test command was:
rm -rf build && west build -b nucleo_l476rg -- -DSHIELD=x_nucleo_iks01a3 && west flash

AFAICT the sample works, from just before 732b815 through current master 16ba258, but only with a power-down reset. A running example will fail if the reset button on the board is pressed.
The output suggests the LSM6DSO initialization doesn't work with soft restart.

No, this is known. The issue is related to the fact that INT1 pin is used to output the drdy interrupt, but after reset in input to enter in I3C hotjoin mode. So, since there is a level shifter on INT1 the '1' state is preserved when interrupt is generated. There are possible solutions like using either polling mode or INT2 for interrupt (instead of INT1).

I can reproduce the soft-reset failure on the nrf52_pca10040 as well, and there I get an I2C transfer error, so it's not an ST I2C problem but probably an issue with communicating with an LSM6DSO that had already been configured and started.
I think it'd be fair to close this issue, but whichever of us touches the lsm6dso next should confirm and fix the soft-reset problem. It'll be on my task list as part of the GPIO API rework if nobody else gets to it first.

OK, I will close this one. I can try to fix it right now with above considerations.

No, this is known. The issue is related to the fact that INT1 pin is used to output the drdy interrupt, but after reset in input to enter in I3C hotjoin mode. So, since there is a level shifter on INT1 the '1' state is preserved when interrupt is generated. There are possible solutions like using either polling mode or INT2 for interrupt (instead of INT1).

Oh, good.  Actually that's common, and the idiom I've been using in updating the sensors for the new GPIO API accommodates it.  Now that I have an A3 I can add that sensor to the set of example conversions.


Update incidents.md

Attempt to further simplify our instructions for responding to suspicious emails.
URL of RTD document: https://civicactions-handbook.readthedocs.io/en/alanna-casey-phishing-updates/

@alanna-casey I like that it's even simpler, but two suggestions:

Remove item 1 from "If you got phished" and say "this must be reported as an incident (see below, next section)  -- note that the next section needs a bit of cleanup to make it more generally applicable
move item 2 from first section to under "If you receive a suspicious email:" as we want people to alert others if they receive a suspected phishing email.

Replaced with #675


New Updated curl to 7.65.1

Updated curl to latest version 7.65.1

@shubham769 can you please check build failures ?


msal-browser with Electron/React

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
So I am trying to currently get this working with Electron and have followed the following example on here by getting it listening to a custom protocol and it works... kind of. I can call PublicClientApplication.acquireTokenRedirect() and get a token fine but then I need to pass it to another page with hash, as I am using react-router-dom, I can't actually do this. Because of this I pass it as a search parameter and then pass it manually to PublicClientApplication.handleRedirectProimise().
Once I do this, it fails on the handleRedirectPromise() with the following message: "ClientAuthError: state_not_found: State not found:  Server State"
Anyoen have any ideas what might be causing this?
This is the following issue I have been going from: https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/324
Source

[ ] Internal (Microsoft)
[ ] Customer request

@Xecuter34 msal-browser is for browser based SPA apps and does not support Electron. To authenticate an Electron app you should use msal-node instead and can find our Electron sample app here


Bumping client-go to v0.25.1

Bumping client-go to v0.25.1

LGTM

Closing this as CVE-2022-3294 suggests to Bump on K8s v1.25.4 or later that means client-go v0.25.4 or later.


data-disable and data-disable-with with Turbolinks redirects

When Rails responds to an XHR request with redirect_to http://localhost:3000/, then it is converted to:
Turbolinks.clearCache()
Turbolinks.visit("http://localhost:3000", {"action":"replace"})

If the request was created by rails-ujs, then it is also processed by rails-ujs and the generated JavaScript is evaluated.
This is great, because this is expected. The issue is that it doesn’t work so great with data-disable and data-disable-with. Once the response is processed then the disabled element is enabled again for a little while until the page is replaced by Turbolinks.visit().
You can see it on the following example. I’ve added an artificial delay to the controller actions to make it more obvious:

Unfortunately this isn’t a made up issue and this is happening regularly in our real world application. From time to time users tend to click on a re-enabled element which is causing server side issues. I think that it would be the best to not re-enable disabled elements if the response is containing a Turbolinks redirect.
I know that this is not a rails-ujs issue and I am not sure if this should be solved in Turbolinks or rails-ujs (or somewhere else). Neither of them looks like a good candidate, because these libraries are independent and they shouldn't know about each other. But I would like to open a discussion and find the best solution. Also, I will be happy to submit a patch once we agree on a way how to solve this.

Thank you for the issue. I believe this fix make sense to be in turbolinks-rails. What we could do is change the enable hook to be triggered only when Turbolinks.visit finish, only if turblinks is enabled.
Turbolinks-rails knowing that rails-ujs exists if fine since rails-ujs is basic infrastructure of Rails fraamework


Support for Python 3

This pull requests adds support for Python 3.3 and 3.4. Thus prov now runs on Python 2.6, 2.7, 3.3, 3.4, and pypy. I did not test on 3.2 and 3.1 (and it will likely not work) as I consider those versions to be deprecated by now.
I used the six (https://pythonhosted.org/six/) library which is not even an additional dependency as python-dateutil already requires it. The whole process was fairly painless and I split up the various steps in separate commits so it should be easy to review.
I furthermore removed pydot from the requirements.txt as its not a hard dependency and pydot on Python 3 only works with this branch: https://bitbucket.org/prologic/pydot
I have no clue why this is not part of the main pydot. I also removed the upper version number limit on pyparsing. What was the rationale behind that?
Sorry for the travis fixing commits...but at least travis now tests all five supported Python versions.

Coverage increased (+0.37%) to 89.38% when pulling 7b738f9c1f05f181931d15c78d9341550aadc159 on krischer:python3 into 5204f5800ca4bb35da78b9c71a1fa00e25666eaa on trungdong:master.

Awesome!!! Thanks a ton for this, @krischer. Python 3 support is right on top of my TODO list. (I started on this before Christmas but went on a trip before I could make significant changes)
I really appreciate your efforts to make it easier to review the changes.
Re the pydot, I might need to look for alternatives. Its development seems to stall for a while now.
BTW, I limited the pyparsing's version since pydot is not compatible with 2.0.0 and above. In practice, it works fine, but I was just fed up with its warnings.
There are a few minor issues I may need to discuss with you later, but I can't wait to merge this!

Great :-) The one thing that is not particularly clean is the __hash__() method of the ProvBundle class. Hashes on mutable objects are in general not particularly advantageous and kind of defeat the purpose of hashes...Nontheless it is currently needed for this line here:
https://github.com/trungdong/prov/blob/27c79ccade9f3d4fa73a46adbbb07b8e9fed5530/prov/model.py#L1324
I am not entirely sure why the bundles property returns a frozenset but I didn't want to mess with it and it potentially should be dealt with in a separate issue.

I agree. frozenset is not needed for that property. I'll remove them and the __hash__ method for ProvBundle as well.


Fix Makefile typo

Description of changes:
Fixed typo in Makefile
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I have a couple other PRs that I am waiting for builds to pass before merging. Once those are through, I will update this branch, let travis rebuild, and then merge. No action is needed by you.


[opentitantool]: Adapt heuristics for Ti50 images

As part of the "rescue" protocol for Ti50 updates, OpenTitan tool needs
to recognize the "RW" part of the given image file.  This is done
through heuristics that recognize the header of each section, and
relying on the order in which sections appear.
Recent images happen to have the header signature in more places in the
image, causing mis-identification of the RW section.  This CL improves
the heuristics.
Change-Id: I3f9e0003ecf71b3967d25af88db962b87d041b0f

Chris, could you please take a look at this modification to the Ti50/H1D2-specific bootstrapping protocol that we call "rescue".


Allow your own primary key to be used with the Model class

Description
This PR adds a new option to the Model class: $useAutoIncrement which when set to false allows us to provide our own primary key for each record in the table. This is really handy e.g. when we want to implement 1:1 relation or use UUIDs in our table.
This new feature doesn't introduce any BC changes.
Ref: #3134
Closes #3211
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

Thank you :)

Let me think about this, and/or have @lonnieezell take a look. While it doesn't break anything from the framework, some of these changes (like insertID() can now return a string) could break people's integrations. For example, a module that assumes numeric indexes and attempts arithmetic operations or sorting.

I don't think it is a breaking change sort of issue. I'm wondering how many projects and modules have been made on the assumption that models and entities use int values for primary keys, and if we allow non-auto-incrementing strings are we responsible for that philosophy shift in a minor patch.
My guess is that my own answer is that this is an improvement and since there is no BC we move ahead. But I want to make sure that seems right to all. I currently have a Firebase project that use's Google's UIDs as primary key, so I get the interest for this but I don't want to be hasty about it.

Well... philosophy doesn't really change - until you disable auto-increment functionality. If this happens then it's the developer's responsibility to adjust their code. We don't change anything by default.
But I get it - I appreciate your concern and we can postpone this to 4.1.

I have no problems with the way this is implemented. I think we can add this now. Unless you're using a strict version of CI4 I think type conversion would handle 99% of any cases I can think of. I'm fine to include it now.


Deps: bump 'discord.py' to 1.5.1

Discord.py 1.5 has had its first bugfix release today, we should bump it in the Pipfile and relock soon.

We're using 1.5.1 after 9d91aa0 so this can be closed.


Updated docs/install.rst. Fixes issue #894

Summary
The file "install.rst" in docs explained creating and activating a anaconda environment. It showed the following command.
source activate name_of_my_env
However, for new anaconda version (>4.6), this command is depreciated and instead following command is used.
conda activate name_of_my_env
This has been updated in the file.
Details and comments
Working on qiskit using anaconda is very important. More than virtual environments, anaconda is preferred by many people. source activate or activate is a command for older versions of anaconda. New versions are depreciating this command and for following versions, this command might not even be recognized. Hence, this pull request updates the docs by giving instructions for new version.
While installing, anaconda still gets installed with older versions and many people may not update it. Therefore, the older command is not deleted. Just new command is added for current and future version of anacoda.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I am getting this error when i'm trying to run tox -elint.
I first installed tox using pip install tox with cmd. and I installed pycodestyle. But its still showing this error.

I don't think the change made in the commit https://github.com/Qiskit/qiskit/pull/957/commits/bf329ee19c9a5b16128ce681b35f6a237dd65c00 is necessary. Please revert it and you can also check here - https://travis-ci.com/github/Qiskit/qiskit/jobs/355186912#L470 to see the lines which are failing lint tests.

Hey, yes. I will revert back to jupyter executive. @nonhermitian  Actually that line kept showing as an error in VS code. So I tried installing Sphinx and then there were more errors. I will revert back all the changes and then run the code again in a conda env now as @SooluThomas guided..

I reverted back jupyter-executive and removed whitespaces from 36 and 37 as told in this link @SooluThomas
But this error shown above is being given by VS code. This is why I removed in the first place. Any idea why? 😐

Done. But that VS code is still showing error in this line.

I'm not sure why your VSCode is showing the error. But I think this PR is good to be merged.


test(pageserver): quantify compaction outcome

Problem
A simple API to collect some statistics after compaction to easily understand the result.
The tool reads the layer map, and analyze range by range instead of doing single-key operations, which is more efficient than doing a benchmark to collect the result. It currently computes two key metrics:

Latest data access efficiency, which finds how many delta layers / image layers the system needs to iterate before returning any key in a key range.
PiTR efficiency, as in https://github.com/neondatabase/neon/issues/7770, which is simply the number of delta files in the range. The reason behind that is, assume no image layer is created, PiTR efficiency is simply the cost of collect records from the delta layers, and the replay time. Number of delta files (or in the future, estimated size of reads) is a simple yet efficient way of estimating how much effort the page server needs to reconstruct a page.

Summary of changes
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Updated the tool to become an HTTP interface so that the Python tests can read it.

I believe CI will pass this time and ready for review :) Thanks! @arpad-m @problame

Hm, and one more thought: The max_num_of_deltas_above_image is only for @latest LSN, but what we (also) care about is worst-case max_num_of_deltas_above_image at random LSN.

but what we (also) care about is worst-case max_num_of_deltas_above_image at random LSN.

Yep, that makes sense. I will submit a separate pull request for that.

Ready for review again :) Hopefully I've resolved all the concerns and we have quite some future works for this analysis code to be very useful.

but what we (also) care about is worst-case max_num_of_deltas_above_image at random LSN.

Yep, that makes sense. I will submit a separate pull request for that.

I'll continue to work on the quantification efforts, implementing my asks above, while @skyzh will work on #7948 and follow-ups.


alert message

Issue #659
Improvement in alert message.

Correction of placeholder values
Error message
Handled all the cases ✔

I think below the text field will look bit odd.
Yes I can make the field red.
By onChange you mean ?

https://materializecss.com/text-inputs.html
refer this fo help something kind if this just try finding about helper text
by if I write a single word it should start displaying the error until I finish the proper format given.


Add partition retrieval

This PR adds the ability to retrieve partition information for a given topic.  It simply forwards the request to the sarama.Client.
I find this useful for tools that wish to create consumer groups with n consumers where n is the number of partitions for a given topic.  Currently, as far as I can tell, the only way to do this (because the client on the returned cluster.Consumer is private) is to create sarama.Consumer solely to call the Partitions function on it and then Close.

Sorry, but please use consumer.Subscriptions() instead, it's safer

Thank you!  Sorry for not seeing this in the API.


Fix crash in splash with no apps

(@rnplus already has this fix locally and asked not to merge this one to prevent conflicts)


Possible regression on "multiple outbound transitions evaluated true"

I suspect that it could have been some regressions in the way multiple transitions are handled, in particular in presence of composite states. In the ThingML tests (which compiles to state.js), we see a few tests failing because multiple outbound transitions evaluated true, whereas only one of those multiple transitions should in principle evaluate to true at a given time (they are guarded). Has it been any changes there between versions 5.6.10 and 5.11.0 that could have caused a regression?

We also went through quite a big refactoring in ThingML the past few months, so the problem might be on ThingML side. I have to look into more details at the generated JS code, though in principle the generated code should be the same as before. It might as well be a regression in state.js. I will come back to you (either close the issue if it appears it was a problem with ThingML, or try to provide you with a simple example that fail if the problem is with state.js).

There are two forms of that exception:
Multiple outbound transition guards returned true at ${pseudoState} for ${message}

and
multiple outbound transitions evaluated true for message ${message}

Are you able to confirm which it is? The first one is when evaluating guards after a junction pseudo state and the second is the more general form for states.
Prior to this exception being thrown, the code simply evaluates all outbound transitions guards with this statement:
transitions = vertex.outgoing.filter(transition => transition.guard(message, instance));

The logic then checks for the filtered collection having 0, 1 or many entries and acts accordingly.
If you have an example of a machine that isolates I can look in more detail.

It is definitely the more general form multiple outbound transitions evaluated true for message ${message}. I will try to find the most simple ThingML test that highlight the issue. Again, thanks for the rapid feedback 👍

OK, here is the state machine:

We send the following sequence of message: n n i (look for those values in the guards)
Basically, we start in State I, then

n moves the current state to C1S1, and produces the trace 01
another n moves the current state to C2S1, and produces the trace 23. So far, everything goes well...
i should move the current state back to I, as no one can consume this event, except the top most composite state C1. This should produce the trace 20, basically as C2S1 and then C1 exits. But this does not happen and produces: TestCompositeStates.default.C1: multiple outbound transitions evaluated true for message [object Object]

See the whole execution trace:
$ node main.js
[Expected]  012320 [Test] 0123
.....\TestCompositeStates_2_Cfg\node_modules\state.js\lib\node\state.js:1124 throw message;
TestCompositeStates.default.C1: multiple outbound transitions evaluated true for message [object Object]

See below for the whole source code, which to me, looks OK and aligned with the UML diagrams (though we should not exclude a mistake their neither...)
TestCompositeStates_2_Cfg.zip

Do you confirm the bug?

I am just in the process of stripping down your example to leave just the FSM code... then I will confirm.
In parallel are you able to build against the earlier version of state.js?

I tried several versions down to 5.3.0 and they behave the same, and raise the exception. I could not directly go to previous version, as it seems it has been some changes in the API between 5.2.x and 5.3.x. We had the ThingML tests off for quite a while, so I do not have any idea when the issue appeared. Also, it might still be some issues in our code (though it pretty much look like it should do what it is supposed to do...) that is not generated the very same way than before. I will try to investigate more.

I think there may be an issue with you code generation I'm afraid as the transition from C1 to I seems to be defined twice; this is an excerpt from TestCompositeStates.js (lines 67-72)
 TestCompositeStates_TestCompositeStates_C1.to(TestCompositeStates_TestCompositeStates_I).when((testIn) => {
    return testIn._port === 'harnessIn' && testIn._msg === 'testIn' && testIn.c === 'i';
  });
 TestCompositeStates_TestCompositeStates_C1.to(TestCompositeStates_TestCompositeStates_I).when((testIn) => {
    return testIn._port === 'harnessIn' && testIn._msg === 'testIn' && testIn.c === 'i';
  });

There are a few other transitions also defined twice scanning further down the code.

Ok, my bad. I should have looked more carefully. Anyway, that is a good news. Should not be too difficult to fix on our side. Sorry for that fake issue!
-----Message d'origine-----
De : "David Mesquita-Morris"<EMAIL_ADDRESS>Envoyé : 23/05/2017 20:04
À : "steelbreeze/state.js"<EMAIL_ADDRESS>Cc : "Brice Morin"<EMAIL_ADDRESS>"Author"<EMAIL_ADDRESS>Objet : Re: [steelbreeze/state.js] Possible regression on "multiple outboundtransitions evaluated true" (#32)
I think there may be an issue with you code generation I'm afraid as the transition from C1 to I seems to be defined twice; this is an excerpt from TestCompositeStates.js (lines 67-72)
TestCompositeStates_TestCompositeStates_C1.to(TestCompositeStates_TestCompositeStates_I).when((testIn) => {
return testIn._port === 'harnessIn' && testIn._msg === 'testIn' && testIn.c === 'i';
});
TestCompositeStates_TestCompositeStates_C1.to(TestCompositeStates_TestCompositeStates_I).when((testIn) => {
return testIn._port === 'harnessIn' && testIn._msg === 'testIn' && testIn.c === 'i';
});There are a few other transitions also defined twice scanning further down the code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


openAppSettings() opens an inconvenient page

🐛 Bug Report
Geolocator.openAppSettings() opens the app settings page. This forces users to navigate two levels down from app settings page, to the app location permission page itself.
Google provided a video with their most recent changes to the Location Permissions policy: youtu.be/b0I1Xq_iSK4?t=94. This shows how, instead of the app settings page, the app location permission page can be opened directly.
Expected behavior
Open the app location permission page instead of the app settings page.
Reproduction steps
Call Geolocator.openAppSettings() with the app's location permission set to denied.
Configuration
Google Pixel 2 XL, updated. Android 11, API 30. Flutter 1.22.5. Project target API 30.
Version: 1.1.13
Platform:

[ ] :iphone: iOS
[x] :robot: Android

Hi @averstuyf,
Unfortunately the page shown in the video cannot directly be called programmatically. This page is automatically shown when after you requested and was granted ACCESS_FINE_LOCATION permission, you request the ACCESS_BACKGROUND_LOCATION permission. In this case the Android system will not show the normal dialog but will redirect the user to the page shown in the video.
The closest we can get programmatically is the App settings page where the user needs to select the "Permissions" followed by the "Location" options first before he or she is redirected the page where location permissions can be updated.
P.S. since the geolocator doesn't support acquiring location updates in the background we also don't offer an explicit option to request the background permissions.

Hi @averstuyf,
Unfortunately the page shown in the video cannot directly be called programmatically. This page is automatically shown when after you requested and was granted ACCESS_FINE_LOCATION permission, you request the ACCESS_BACKGROUND_LOCATION permission. In this case the Android system will not show the normal dialog but will redirect the user to the page shown in the video.
The closest we can get programmatically is the App settings page where the user needs to select the "Permissions" followed by the "Location" options first before he or she is redirected the page where location permissions can be updated.
P.S. since the geolocator doesn't support acquiring location updates in the background we also don't offer an explicit option to request the background permissions.


git fetch remote fetches the remote into FETCH_HEAD not into origin…

…/main.
So the line that follows doesn't really do anything. This fixes it to do the checkout from the nearly fetched data. Yay git!

Ha, I can imagine your thoughts when you found that one :-D

Basically "I miss SVN".


Server/Client Data Syncing

Sync the Seed cache with the server on join. Client would dump it's list into a secondary list and pull in data from the server on join. When the client disconnects, cache the server's list locally (As SERVERIP-Cache.dat in saves/cache/) then bring it's original list back.
Would need to "versionize" server caches in case the server changes it's list.
Or something like that. I'unno.

Any progress on this? I'm seeing conflicts in the seed cache between server and client, and deleting the client cache isn't fixing it for some reason.

I think I'm having the same or a similar issue. When anyone connects to our server some of the seeds get switched around with another seed but not all do. For example Blaze and Slimey switch names and colors as well as the drops. The client and server configs are the default configs. Only happens when players connect to the server and does not happen in SSP.
Version: ResourcefulCrops-1.9.4-1.3.2-54.jar
Client json: http://pastebin.com/e9FFbBA3
Server json: http://pastebin.com/gdYcCS2p

Hmm, in 1.9+ the syncing is all handled by Forge using the same system as blocks and items. I'll take a look tonight.

@TehNut Let me know if I can provide anymore information that would help you find this bug.

Hrm, could you load up a single player world and check the meta of the problem seeds, then check them again on the server?
I believe the problem is that I register recipes and OreDict entries for specific meta's, then when connecting to a server, those meta's get shuffled a bit with the recipes/entries keeping reference to the old meta.

The problem disappears in single player and if you quit and then connect to a server, the problem returns. I had multiple people test it using our server.

@Mohron Could you try with this build? I moved recipe registration around a bit. http://tehnut.info/share/ResourcefulCrops-1.9.4-1.3.2-CUSTOM.jar

I put this build on my client and server. All recipes are missing JEI including the essence and Stones of creation.  I tested crafting with some of the shards and the crafting works but fails to show an output.

Odd... The recipes showed up in my testing.

I guess it could be a cross mod bug of some sort but I would ha no idea what could be causing an issue like this.

How about this one?
http://tehnut.info/share/ResourcefulCrops-1.9.4-1.3.2-CUSTOM.jar

That one on the other hand appears to work just fine!

Had that same issue just when i added a seed

Version: ResourcefulCrops-1.7.10-1.0.0-26

1.7.10 is unsupported.
Provide the same JSON to your clients as you do to your server.


ScannerWrapper renaming

See the commit messages for details.

ExperimentalScanner(Test).kt still has references to the regex (local|remote).?scanner. Could you please fix these to say "(path) scanner" or similar?

(local|remote).?scanner

I'm still seeing quite some references:

(local|remote).?scanner

I'm still seeing quite some references:

Fixed now as well, I forgot to check ExperimentalScanner.kt.


NLog failed to parse config file - Target Console not found.

Hi! Thanks for reporting this feature/bug/question!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
QUESTIONS are preferred on StackOverflow. You could expect a faster response there (but do include all the info below). Use the tag "nlog" https://stackoverflow.com/questions/ask
For .NET Core users, please check the platform support: https://github.com/NLog/NLog/wiki/platform-support
NLog version: (e.g. 4.4.13) 4.5.11
Platform:  .NET Core 2
Current NLog config (xml or C#, if relevant)
<?xml version="1.0" encoding="utf-8" ?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      autoReload="true"
      internalLogLevel="info"
      internalLogFile="internal-nlog.txt">

  <!-- enable asp.net core layout renderers -->
  <extensions>
    <add assembly="NLog.Web.AspNetCore"/>
  </extensions>

  <!-- the targets to write to -->
  <targets>

      <target xsi:type="Console"
          name="String"
          detectConsoleAvailable="false"
          layout="${longdate}|${event-properties:item=EventId_Id}|${uppercase:${level}}|${logger}|${message} ${exception:format=tostring}"/>
  </targets>

  <!-- rules to map from logger name to target -->
  <rules>
    <!--All logs, including from Microsoft-->
    <logger name="*" minlevel="Trace" writeTo="Console" />

    <!--Skip non-critical Microsoft logs and so log only own logs-->
    <logger name="Microsoft.*" maxLevel="Info" final="true" /> <!-- BlackHole without writeTo -->
    <logger name="*" minlevel="Trace" writeTo="Console" />
  </rules>
</nlog>

What is the current result?
App start, but no logs are written
What is the expected result?
Logs are written to console
Did you checked the Internal log?

2018-12-06 16:02:45.4642 Info Message Template Auto Format enabled
2018-12-06 16:02:45.4816 Info Loading assembly: NLog.Web.AspNetCore
2018-12-06 16:02:45.5330 Info Adding target Console Target[String]
2018-12-06 16:02:45.5639 Error Parsing configuration from <>/nlog.config failed. Exception: NLog.NLogConfigurationException: Exception when parsing <>/nlog.config.  ---> NLog.NLogConfigurationException: Target Console not found.
   at NLog.Config.XmlLoggingConfiguration.ParseLoggerElement(NLogXmlElement loggerElement, IList`1 rulesCollection)
   at NLog.Config.XmlLoggingConfiguration.ParseRulesElement(NLogXmlElement rulesElement, IList`1 rulesCollection)
   at NLog.Config.XmlLoggingConfiguration.ParseNLogElement(NLogXmlElement nlogElement, String filePath, Boolean autoReloadDefault)
   at NLog.Config.XmlLoggingConfiguration.Initialize(XmlReader reader, String fileName, Boolean ignoreErrors)
   --- End of inner exception stack trace ---
2018-12-06 16:02:45.5652 Warn Failed loading config from <>/nlog.config. Invalid XML?
2018-12-06 16:02:45.5734 Info Found 15 configuration items

Please post full exception details (message, stacktrace, inner exceptions)
Are there any workarounds? no
Is there a version in which it did work?
Can you help us by writing an unit test? yes

Could seem like you are using NLog for NetStandard1.3 (UWP).Console should be available with NLog NetStandard 1.5 + 2.0.  Could you verify the file-properties of the NLog.dll in use?

"NLog/4.5.11": {
        "runtime": {
          "lib/netstandard2.0/NLog.dll": {
            "assemblyVersion": "<IP_ADDRESS>",
            "fileVersion": "<IP_ADDRESS>45"
          }
        },
        "compile": {
          "lib/netstandard2.0/NLog.dll": {}
        }
      },

This one?

Looks good.

Not that I'm aware of. MacOS use regular dotnet core

What happens if you perform the configuration using code:
var config = new NLog.Config.LoggingConfiguration();
var logconsole = new NLog.Targets.ConsoleTarget("logconsole");
config.AddRule(LogLevel.Info, LogLevel.Fatal, logconsole);
NLog.LogManager.Configuration = config;

Maybe for some reason then it might use the iOS-assembly instead of the wanted NetStandard. Or maybe Xamarin-build is doing some linker-tree-shake that removes everything not used directly.
Not an experienced MacOS-developer. Can only say that NLog works just fine with console on NetCore2

Yep, applying configuration from code is working. So weird...

Yep, applying configuration from code is working. So weird...

Then your build-environment is doing ahead-of-time (AOT) compiling of code with linker-tree-shake. You need to modify your build to ignore NLog.dll when stripping away "unnecessary code" .

Can you help me understand how to do that?

Don't know what tools you are using. But maybe this can help:
https://docs.microsoft.com/en-us/xamarin/mac/deploy-test/linker
https://docs.microsoft.com/en-us/xamarin/mac/internals/aot

Created #3039 for NLog 4.6 that should fix the problem with Xamarin Linker removing everything from NLog.dll Allowing NLog.config-files to work without need to specify --linkskip=NLog

@omerlh If you know how to setup a local nuget-package-repository, then you are very welcome to try this pre-release build to see if it works as expected (without  using --linkskip=NLog):
https://ci.appveyor.com/project/nlog/nlog/builds/20883080/artifacts

I tried, but look like I can't use this package:
error: Package 'NLog.<IP_ADDRESS>79-PR3039.nupkg' is incompatible with 'all' frameworks in project <>.

@omerlh Try rename the package from NLog.<IP_ADDRESS>79-PR3039.nupkg to NLog.<IP_ADDRESS>79.nupkg
It might help as someone says here: https://github.com/NuGet/Home/issues/5127 (Have no issues myself using these package-names in VS2017)

#3039 has been merged, so assume this is fixed. Will reopen if not!

I'm facing the issue of config file not loading to the logmanager.configuration. due to this the the config object has not targets in it whereas i have added the rules n targets in the nlog.config file.
Package is Nlog 4.6.1. the nlog.config file is android asset. also the code is portable project

I'm facing the issue of config file not loading to the logmanager.configuration. due to this the the config object has not targets in it whereas i have added the rules n targets in the nlog.config file.
Package is Nlog 4.6.1. the nlog.config file is android asset. also the code is portable project

I assume this is https://github.com/NLog/NLog/issues/3283

NLog.config can be added as an Xamarin resource, which is automatically deployed but has to be loaded manually:
https://github.com/NLog/NLog/wiki/Explicit-NLog-configuration-loading#loading-nlog-configuration-from-xamarin-resource


Please update the jackson-databind version

It would be good to update the jackson version so this fix can be included in there:
https://github.com/FasterXML/jackson-databind/issues/2704

Hi  @NikolayMetchev ,
Thanks for opening the issue, and sorry for the late reply. The jackson-databind version was upgraded as part of #24 , so will close off this issue for now. Feel free to reopen if needed.
Thanks,
Dani


Spanish language not work

hello guys, in my project ionic 3 when call es.json show english
someone help me

follow this guide https://ionicframework.com/docs/developer-resources/ng2-translate/

follow but not work, only spanish not work

Sorry for not answering the issue for such a long time.
We try to do better in the future, promised.
I am closing the issue for now - you can re-open it if the issue still persists.


building script from html templates

As it now seems, HTML templates for each date will live in a individual folder. We will "build" each html page with the value of co2 on that date.
This means we probably need a simple build script, which:

reads the raw co2 data (json dict)
reads all html template files (glob)
writes co2 values from json dict to those html

It probably makes sense to have 2 branches in this repository, similar to static site builders. main branch is for the html templates and builder script (python) and the gh-pages branch is for the public html.
A github action can be configured to build the public html and push to gh-pages every time there is a change to the main branch.

Do you want to take a swing @ a skeleton build script? So that I can chop up the frontend into a template.
Sorry, I should have asked for this much earlier.

Yeah, I can write the build script. Point me to an html template file when ready.

I think this is done with https://github.com/co2birthdate/website/commit/9db60142b83587f8436a6ca52183a92951958b78


Implement missing options for RemoveGroupMembers API

This endpoint allows for two optional options: https://docs.gitlab.com/ee/api/members.html#remove-a-member-from-a-group-or-project
@svanharmelen would be super great to get a release after a merge 😇 🙏

Released v0.72.0


Added docker-compose

Hi! I've added a docker-compose.yml which is handy for those using docker with docker-compose. Further I've added a section about in the README. Would be great if you could merge. Any questions, please let me know.

added another section to the README about volumes


[release-0.41] move host device validation to happen on pod creation

What this PR does / why we need it:
This is a manual cherry-pick of https://github.com/kubevirt/kubevirt/pull/6161
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Remove HostDevice validation on VMI creation

/retest

/lgtm

/retest

Errors seems to be related to the GOPROXY value https://github.com/kubevirt/project-infra/blob/efd82844cad82370f2f10225eedf9890bc718a1a/github/ci/prow-deploy/kustom/base/configs/current/config/config.yaml#L577. The | separator was introduced in go 1.15 https://go.dev/doc/go1.15#go-command, while release 0.41 uses go 1.13 and hence the error parse [https://proxy.golang.org|direct](https://proxy.golang.org%7Cdirect/): invalid character "|" in host name

/retest

@enp0s3 PTAL

/lgtm


Add test for check 078

#1795

Ok... I'm not completely happy with the idea of compiling UFOs into font binaries as part of the test code. Looks a bit too complex, which could result in testsuite bugs. But I'll merge it anyway, since it works (and the testing code seems to be correct) and is better than having no test at all.
But I would advise that simpler testing code would be preferable.
Also, it is not clear yet if all this effort is really useful, since we still lack a proper rationale description for check/078, as discussed at #1795 Hopefully we get some insight from the original check author (@mekkablue) at that issue.

Yes, exactly why do you need to pull in the whole ufo2ft and compile a UFO just to check that the glyph names don’t exceed some (arbitrarily defined?) limit?

Basically, lazyness in case I reuse this for other checks.


Enhancements - x86 support (WoW64 + native), ability to change function prefix

Seems to work. Please check if 32 bit version asm is correct.
I was wondering if a separate .asm file should be generated for this. Now the generated .asm will build on both architectures using IFDEF RAX.
Some ideas worth mentioning to improve this even further:

Add -x86/-32bit argument to optionally enable 32-bit support

Custom function prefix only works with Nt functions. I don't know if it's needed for Rtl as well.
Closes #7

I tried since this morning to make the code compatible, and now I just saw this PR, I hate my life but thanks a lot @bugproof

I have a variation that generates separate ASM files fro x86 and x64 as well as some changes to shave off some redundant code. Should I make a new pull request?

I have a variation that generates separate ASM files fro x86 and x64 as well as some changes to shave off some redundant code. Should I make a new pull request?

I did the same thing on my side but not tested it, I just waited this PR but do not worry, I will let you create a new PR ;)


Fix crash on looking up databased attributes when none are present

In my test database this would result in a failure like this:
System.Collections.Generic.KeyNotFoundException: The given key '19' was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at osu.Game.Rulesets.Osu.Difficulty.OsuDifficultyAttributes.FromDatabaseAttributes(IReadOnlyDictionary`2 values, IBeatmapOnlineInfo onlineInfo)
   at osu.Server.Queues.ScoreStatisticsProcessor.Stores.BeatmapStore.GetDifficultyAttributesAsync(APIBeatmap beatmap, Ruleset ruleset, Mod[] mods, MySqlConnection connection, MySqlTransaction transaction) in /Users/dean/Projects/osu-queue-score-statistics/osu.Server.Queues.ScoreStatisticsProcessor/Stores/BeatmapStore.cs:line 83

Not 100% sure this is the right solution so please double-check.

Which beatmap is this for? Why does it have no attributes? This will pp as well, which makes it super scary.

This is in my sandbox database which doesn't have attributes for any beatmap 😅

Ah, okay.


Model validation

Thanks for your great job!
Did you use validation data to obtain the best checkpoint (model)?

@pencilzhang Hi, thank you for appreciating our work. We follow the practice of most BIQA papers to split the training/testing results by an 80%/20% ratio. So, there is no data reserved for validation. For a more strict evaluation without referring to the results on test sets, we can always use the latest checkpoints for evaluation, although we see no big difference between both strategies.

@zwx8981  Thanks for your response!


Copy file in create-home.sh not working

The following lines :
cp -r "$NETWORK_PATH"/app.toml "$GALACTICA_HOME/config/app.toml"
cp -r "$NETWORK_PATH"/client.toml "$GALACTICA_HOME/config/client.toml"
cp -r "$NETWORK_PATH"/config.toml "$GALACTICA_HOME/config/config.toml"
cp -r "$NETWORK_PATH"/genesis.json "$GALACTICA_HOME/config/genesis.json"
cp -r "$NETWORK_PATH"/gentx "$GALACTICA_HOME/config/gentx"
Are not working because the file are not existing in $NETWORK_PATH.

Seems to be the issue resolved by #15

@yakud , it's not the same issue as #15 , here the files are missing. This files have been deleted in #8 (when changing the folder structure).

same for update-home :-)

Thank you for your attention.
When I accepted the PR with deleting the configuration files, I did not pay attention to the fact that they were explicitly copied in the script.
Should be fixed in the main branch.


Segment from Lambda triggered by SQS is never sampled

Hey folks, I've been scratching my head against this one for a while, I created this issue hoping that someone can help figure out what's going on.
I have a Lambda function with an alias, and the alias can be triggered by SQS. Events are coming in from EventBridge and routed to SQS. The function has Active tracing enabled.
When the function gets triggered by SQS, I see the following in my logs (values from one specific run are shown below, but this is consistent behaviour on every attempt):

The SQS message itself contains the following in its attributes:"AWSTraceHeader": "Root=1-63e54f60-54c0dec03aa822cf0eb80f6b;Parent=7026c82d12761ec6;Sampled=0"

The X-Ray SDK ends up apparently ignoring that though, and uses this facade segment:Lambda trace data found: Root=1-63e54f76-613d90840ef5611617d19f78;Parent=5b3f323a471e441f;Sampled=0
Segment started:
{
    "root": "1-63e54f76-613d90840ef5611617d19f78",
    "parent": "5b3f323a471e441f",
    "sampled": "0",
    "data": {}
}

At the end of the function, the SDK ends up deciding not to flush any subsegments, because (as I showed), the segment is not sampled:Ignoring flush on subsegment 3b13dabf96025041. Associated segment is marked as not sampled.

Note how both of those segments are not sampled (the one from SQS and the one Lambda creates). I've tried recreating the whole infrastructure (SQS queues, EventBridge triggers, Lambda function, IAM roles) from scratch, and I tried disabling Active tracing and re-enabling it, but this behaviour keeps happening. The sample policy I'm using is the default one, and the Lambda was invoked only to test this, which means it was definitely within the policy of 1 sample per second.
However, if I manually invoke the Lambda function (with the same message it gets from SQS, no changes at all, including the unsampled AWSTraceHeader), then it creates a sampled segment (as expected!). Note that this is a manual invocation of the Lambda, it isn't being triggered by SQS this time:
Lambda trace data found: Root=1-63e5ac97-07c68b9b2d943eb56a158f57;Parent=e42fbe5a5a09d152;Sampled=1
Segment started:
{
    "root": "1-63e5ac97-07c68b9b2d943eb56a158f57",
    "parent": "e42fbe5a5a09d152",
    "sampled": "1",
    "data": {}
}

And it properly flushes the subsegments, and I can see them in the X-Ray console:
Subsegment sent: {"trace_id:"1-63e5ac97-07c68b9b2d943eb56a158f57","id":"b2f07bfe66ccf783"}
UDP message sent:
{
    "id": "b2f07bfe66ccf783",
    "name": "<redacted>",
    "start_time":<PHONE_NUMBER>.821,
    "namespace": "remote",
    "http": {
        <redacted>
    },
    "end_time":<PHONE_NUMBER>.035,
    "type": "subsegment",
    "parent_id": "e42fbe5a5a09d152",
    "trace_id": "1-63e5ac97-07c68b9b2d943eb56a158f57"
}

Does anyone know why apparently the Lambda function is never sampled when invoked through the SQS trigger? I noticed you folks have recently worked on an SQS->Lambda and SNS->SQS->Lambda trace continuation features (congrats btw! I'm definitely going to make use of them), so I'm wondering if something along the way changed and broke this scenario I'm describing?

Hi @DanielSidhion,

Events are coming in from EventBridge and routed to SQS. The function has Active tracing enabled.

Can you be more specific about how Events are generated in EB? The sampling decision is made at the root of your request, whatever that is, as long as that root has active tracing enabled. So if there is anything upstream of EB that may be injecting the unsampled decision. EB itself does not have active tracing, so it should not be making a sampling decision (neither should SQS).

Hi @willarmiros,
Thank you for the reply. In this particular case, I have the following request path:
 -> API Gateway -> Lambda 1 -> EventBridge -> SQS -> Lambda 2 (the one I was investigating)
Both API Gateway and Lambda 1 don't have X-Ray/Active tracing enabled. I just experimented with turning on Active tracing on Lambda 1 and that didn't work either, but enabling X-Ray on API Gateway AND enabling Active tracing on Lambda 1 finally makes Lambda 2 get a sampled segment.
This was a bit unexpected, since it looks like I'll have to enable X-Ray along the whole chain just so I can get insights on Lambda 2 (which is the thing I'm most interested in). Is there no way to work around this? I can enable X-Ray on everything, but I'm curious whether this is forcing a specific workflow or if there's a way to work around it using the X-Ray SDK.
An interesting behaviour is that the ADOT layer seems to basically ignore the unsampled segment and just generates its own in that case. I haven't tested the ADOT layer with the sampled segment yet, but I'll experiment to see how it goes.

I just experimented with turning on Active tracing on Lambda 1 and that didn't work either, but enabling X-Ray on API Gateway AND enabling Active tracing on Lambda 1 finally makes Lambda 2 get a sampled segment.

That is surprising that enabling it on Lambda 1 did not start to introduce sampled segments. For context, it is a known issue that Lambda functions will generate Sampled=0 if they do not have active tracing enabled. This is a bug, as they should not be adding any sampling decision, and we are working w/ Lambda to get it fixed. So getting Sampled=0 with Lambda1 active tracing disabled is expected, but if you enable active tracing on it I would expect it to go away. The approach with minimal tracing is you could enable active tracing on only APIGW and Lambda2, then the other services should just pass the trace context (with Sampled=1) through.

An interesting behaviour is that the ADOT layer seems to basically ignore the unsampled segment and just generates its own in that case. I haven't tested the ADOT layer with the sampled segment yet, but I'll experiment to see how it goes.

Can you clarify what you mean here? Are you saying if there is Sampled=0 in the header, you still see a segment from the ADOT Lambda layer, but no segments from the Lambda function itself? Are you using Node.js Lambda layer?
To summarize, it seems like there are 2 possible issues:

APIGW possibly injecting a Sampled=0 when tracing is disabled
ADOT lambda layer (for JS?) is not respecting the sampling decision when generating segments


revert #875

Description
Reverts the changes from #875.
The reason for #875 was https://github.com/scaffold-eth/create-eth/pull/55#issuecomment-2195411531. Since in create-eth we are no more using prettier from outside to format files and instead using SE-2 instance prettier to format files we don't require this changes anymore.
The reason for reverting this is #901.
To see if this solve #901 :
Copy paste the below in page.tsx insde Home component:

Test snippet :

  const [count, setCount] = useState(0);
  const [name, setName] = useState("shiv");
  // should get an eslint error
  const x = "shiv";

  useEffect(() => {
    setCount(count + 1);
    console.log(count);
    setName("shiv");
    console.log(name);
    // should get an eslint warning
  }, [count]);

And hopefully you should get eslint warning / error at mentioned places in the comment.

@technophile-04 you are my hero :heart_eyes:

Works great to me! And I agree that this solution is better because it's more stable! Thank you!

Thanks all! Merging this!


Perfect ug

Codecov Report

Merging #259 (9656db1) into master (2b35f24) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #259      +/-   ##
============================================
- Coverage     51.62%   51.56%   -0.06%
  Complexity      733      733
============================================
  Files           141      141
  Lines          3440     3444       +4
  Branches        478      481       +3
============================================
  Hits           1776     1776
- Misses         1538     1542       +4
  Partials        126      126

Impacted Files
Coverage Δ

...in/java/seedu/address/logic/parser/ParserUtil.java
41.83% <0.00%> (-1.13%)
:arrow_down:

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


white borders around some objects

I'm trying to draw 3D stick/lollipop plots with a combination of linesegments and scatter. I have some visual glitches where it looks like the lines are being drawn over the scatter dots, and they have a white line around them.

Here's how i made the plot:
julia> t = range(0, 10, length=40)
0.0:0.2564102564102564:10.0

julia> sig = exp.(im*t) + exp.(2im*t);

julia> function sticks(x, y, z)
           origy = zeros(length(y))
           origz = zeros(length(z))
           linesegments(
               reshape(permutedims([x x]), :),
               reshape(permutedims([origy y]), :),
               reshape(permutedims([origz z]), :))
       end
sticks (generic function with 1 method)

julia> sticks(t, real(sig), imag(sig))

julia> scatter!(t, real(sig), imag(sig))

Yeah, that's a problematic artifact right now... You can try to set transparency = true for the lineplot.

yeah, that looks much nicer. thanks!


Added Cloudflare + php7-bcmath support

Built and tested to be working.

Would you also be open to adding the Cloudflare plugin? As without it, you can't really use Caddy when your domains are also behind Cloudflare.

The challenge will bundling other plugins is primarily maintenance. I prefer to include plugins I can respond to its requests.

All right, are you able to just approve the bcmath addition then? Or do I need to create a new PL?

You can create an additional commit to revert the cloudfare addition and I'll merge it in.
Thanks.

By the way, I am open to ideas to make plugin inclusion dynamic if possible.
As a workaround, you can base your image off this as many other people have done.

Sweet, removed Cloudflare.
I've relatively new to Docker but from what I can see it looks like tags are the only way to include different plugins and keep them out of the main build... not really ideal.
And yeah thanks, I've already forked this project and running it locally with CloudFlare.
Would you be ok if I published it on hub.docker? (giving you credit of course)

Not a problem.


Add search command

Currently, it is impossible to find available packages from the CLI. You have to navigate to the web UI at https://sqlpkg.org and search/explore there. Then, it is difficult to copy the package identifier, as the identifier is a link to the GitHub repo. All in all, package discovery could be improved.
Adding a sqlpkg search {phrase} command would allow users of the CLI to search the registry's packages exactly like via the searchbar on the https://sqlpkg.org page, making it easier to discover packages available to install.

Yeah, I don't know about that.
I totally agree that a CLI-based search would be nice. At the same time, I'm pretty happy with the basic feature set right now.
And Python or Go do just fine without CLI search or registry (#14). So I think I'll pass for now.

Your choice. Thanks for the great lib


Alignment in a resource with a single parameter

Should this trigger a linter error:
class { 'some_class':
  some_param                             => 'some_value',
}

Note the extra spaces. Currently, there are no warnings in that situation.

Hi @gerases, sorry for the long delay in follow-up.
I have replicated the issue and can confirm this is still active. Given the behaviour of our arrow_alignment plugin, I would say this is probably a bug as adding any extra parameters to the class triggers the correct response (aligning '=>' symbols one space after the longest parameter name).
To answer the question: Yes, it should be throwing a lint warning. We have ticketed this and will be looking to fix it in the near future.


Sync PRIVAaaS in BIGSEA and its fork at ATMOSPHERE

Refactory was done at the original PRIVAaaS (BIGSEA). That is good, but new developments should occur at ATMOSPHERE fork.
Fork at ATMOSPHERE should be synchronized with BIGSEA.

Waiting for Nuno's positioning about Repo.

Yesterday we had a definition about this repo in favor of sync and then keep working on this fork only.


Update kb_item.html

Fixed a minor typo: space after colon in H2 heading

Codecov Report

Merging #855 into 0.2.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            0.2.x     #855   +/-   ##
=======================================
  Coverage   63.56%   63.56%
=======================================
  Files          40       40
  Lines        3722     3722
  Branches      574      574
=======================================
  Hits         2366     2366
  Misses       1201     1201
  Partials      155      155

Continue to review full report at Codecov.

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


cellxgene-schema CLI is not correctly enforcing obs['is_primary_data']

Context
Reported by @jahilton. See #sci-data-eng.

The issue here is that we currently accept both bool and numpy.bool types for 'is_single'. However, our validation check for is_primary_data checks against "is False" which only works as expected for bool.
These datasets used numpy.bool and thus unexpectedly passed by evaluating "numpy.bool(False) is False" as False instead of True.
The solution is to either update this check (and others) to account for numpy.bool OR enforce that we do not accept numpy.bool types. Probably easiest to just account for numpy._bool.


Fetch of BIPM PV 105 is not working

... and used to

@opoudjis now the correct reference is BIPM CIPM Decision 105

OK, will change my test file. Will also need to get editors to update BIPM brochure, if they haven't already.

I believe we haven't :). Have assigned myself as a remainder.

@opoudjis what else do we need to do here? Update BIPM brochure?

@andrew2net there is a doubt about the rendering of fetched BIPM references exposed in https://github.com/metanorma/bipm-si-brochure/issues/145#issuecomment-1114072781

@manuel489 the rendering of a reference is done in Metanorma, so we would need to ask @opoudjis

@manuel489 is the default rendering in Metanorma not correct? I think we need to ask @opoudjis about relaton-render.

Question answered.
From my understanding, there is nothing else to do here.
I think we can close this ticket.


[WEBSITE-798] Implemented filtering for blank nodes

Added check for blank nodes in filter method.
Added more testing around this.
Added yard documentation covering this.

Coverage remained the same at 100.0% when pulling ce209f1b0ba52f20fb8aadce28a82a0db800ffae on katylouise:katylouise/website-798_refactor-letter-queries into 40d21cb808eb70f13729b2c68e59e6055a0ce00a on ukparliament:master.


Add Terraform GitOps

Details
Describe the solution you'd like:
Several months ago I did a PoC implementing a Terraform driven flux cluster. Taking inspiration from that, I want to add that capability here.
Anything else you would like to add:
Ancillary goal is to add satellite nodes and/or edge tiers to bloopysphere. Including but not limited to the free tier oracle cloud, cloudflare workers and maybe something like netlify.
Additional Information:

#2472 is part of this issue


Restrict which hosts use acme

What version of Traefik are you using (traefik version)?
1.1.2
What is your environment & configuration (arguments, toml...)?
using docker-compose labels as well as toml config for main.
What did you do?
I am trying to use letsencrypt to serve one service, and also serve an admin console to localhost (later may also want to publish the admin console on a subdomain). For flexibility I am trying to use OnHostRule=true and use labels in the individual services.
I can get the certificate properly generated for the main service, but traefik also requests a certificate for the admin. Is there a tag I can include to exclude the admin service from certificate generation for the moment?
What did you ~expect~ hope to see?
Certificate for domain, no request for localhost.
What did you see instead?
Requests for both.

Are you serving the admin console via the same TLS entrypoint? If so, it will require a certificate. Unless you are providing your own, letsencrypt will request one due to the OnHostRule
If you are NOT using tls for the admin, it should NOT be requesting certs for non-TLS enabled entrypoints.

I'll close this issue, because I think the question is answered, but feel free to continue the conversation.


How to I "unregister" a credential from Fido2 ?

I am building an application using Python Flask and I would like to have the option for the user to "delete" the key, or "update" the usb key (register a new one).
How do I "unregister" the user's credentials from fido2 ?
Thank you.

Just delete from your server


Use .html instead of .text

Hello,
Unless I'm mistaken, as per https://stackoverflow.com/a/18418270, it's recommended to use .html instead of .text when amending the contents of a page using jQuery (it's certainly faster to use .html over .text).
This PR changes the relevant .text entries in the JavaScript to use .html instead.

Hi, thanks for the feedback. Although faster, the .text() is there on purpose: it avoids inserting by mistake HTML where HTML should not be. The interface mandates that those options are "text", and thus we use the jQuery entity filter to escape the text. Despite really unlikely, using .text() closes the door to XSS attacks - should the modal be evaluated with options coming from untrusted sources.


[Proposal] Opacity Output

Many Programs that deal with Procedural texturing ex. (Blender/Substance Designer) have support for Opacity maps/alpha transparency, this is really useful for things like a mesh fence material

That would be a piece of cake to implement, only requires work on the 3D preview and export targets.

should close this since it seems to be dealt with


Support single input file

Add support for a single input file.
Validation
$ tree crawl/
crawl/
├── 00001.html
├── 00002.html
├── 00003.html
└── MANIFEST.txt

1 directory, 4 files

$ cat crawl/MANIFEST.txt
00001.html
00002.html
00003.html

$ rm -rf output
madoop \
  -input crawl/MANIFEST.txt \
  -output output \
  -mapper /bin/cat \
  -reducer /bin/cat
cat output/*
INFO: Starting map stage
INFO: Finished map executions: 1
INFO: Starting group stage
INFO: Starting reduce stage
INFO: Finished reduce executions: 3
INFO: Output directory: output
00001.html
00002.html
00003.html

LGTM thanks for the addition


The Example app stops working after I updated the device to androd 12.

The Example app stopped working right after I updated the app to andriod 12.
I was using two google pixel 4A. I updated one of them to Andriod 12 and then the I needed to add the andriod:exported parameter in Manifest. and after installing the app new the browsing or the advertising is not working. The other phone can not discover the device anymore. The device would vibrate or notify everytime I clicked on brower or advertisement. But now it is not doing that either.
This could be a great help if someone could help me regarding this.

@sasikdar
update android 12
You must add these 2 permissions to your app

Manifest.permission.BLUETOOTH_SCAN, Manifest.permission.BLUETOOTH_CONNECT


Admin-sdk Error 400: Bad Request, badRequest

Expected Behavior
I'm trying to use this example:
https://developers.google.com/admin-sdk/directory/v1/quickstart/go
I expect to see a list of users names and email addresses from my domain
Actual Behavior
I get an error from this line:
https://github.com/gsuitedevs/go-samples/blob/master/admin_sdk/directory/quickstart.go#L104
Unable to retrieve users in domain: googleapi: Error 400: Bad Request, badRequest

I am using ... .Customer("<my-domain>") ..., there are several users in the org that have email addresses in that domain.
Steps to Reproduce the Problem

Followed the G Suite wide delegation guide to get client credentials/private key:
https://developers.google.com/admin-sdk/directory/v1/guides/delegation
Added the service requests to get domain users
Compile succeeds and run => error.

Specifications

go1.10.2
macOs 10.13.6

The quickstart uses user (3-legged) OAuth, not service accounts. Have you gotten that to work? When you use the special value "my_customer" for the customer ID, does that work?

Thanks! I also got an error when I used OAuth, I can't remember what it was now though, will try again tomorrow.
When I use the "my_customer" value it says something like that domain doesn't exist.

 Is the authorized user part of a G Suite domain, or is it a gmail.com account?

for future folks who may see this, i hit this when querying the directory API trying list users. however the list users api requires a domain string be set

verify this by trying the api

unfortunately the err doesn't tell you the field you're missing


bower install fails

When running mvn --settings [Path to your settings.xml] clean compile install package  I receive a build failure on com.github.eirslett:frontend-maven-plugin

Are you able to just run bower install by itself from cli?

can you run this command  npm install -g bower
and let us know the results

jdamick - I'm not able to run bower install
amitmawkin - Here are the results:
npm ERR! registry error parsing json
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs
\node_modules\npm\bin\npm-cli.js" "install" "-g" "bower"
npm ERR! node v0.12.4
npm ERR! npm  v2.10.1
npm ERR! Unexpected token C
npm ERR! Cannot GET http://registry.npmjs.org/bower
npm ERR!
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\admin\Documents\GitHub\Hygieia\npm-debug.log

@raposok are you running this behing a proxy ,  the error is in your logs
npm ERR! Cannot GET http://registry.npmjs.org/bower,
please make sure you can get to this endpoint