[Bug]: swa start --data-api-location causing Bad CPU type in executable

What happened?
OS: [macOS]
Version [Ventura 13.4]
Architecture: [arm64 (Apple M2 Pro)]
When running the command to start the SWA using
swa start --data-api-location swa-db-connections

The following error occurred - Bad CPU type in executable
[dataApi] /bin/sh: /Users/tianyili/.swa/dataApiBuilder/0.8.49/Microsoft.DataApiBuilder: Bad CPU type in executable
[dataApi] cd "/Users/tianyili/Development/freshfarmpicking-react/swa-db-connections" && "/Users/tianyili/.swa/dataApiBuilder/0.8.49/Microsoft.DataApiBuilder" start -c staticwebapp.database.config.json --no-https-redirect exited with code 126
--> Sending SIGTERM to other processes..
[swa] node<EMAIL_ADDRESS>exited with code SIGTERM
--> Sending SIGTERM to other processes..
[run] cd "/Users/tianyili/Development/freshfarmpicking-react/src/web" && yarn run dev  exited with code SIGTERM
✖ SWA emulator stoped because the --run command exited with code SIGTERM.

--data-api-location sets the swa-db-connections and declares where to run DataApiBuilder.exe to connect to the database.
If I change the options flag to --api-locaiton without the db connections, it works fine and no error is shown.
Is there any way to support the new Apple silicon chip plz?
Version
0.8.49
What database are you using?
Azure SQL
What hosting model are you using?
Static Web Apps (SWA)
Which API approach are you accessing DAB through?
REST
Relevant log output
No response
Code of Conduct

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

Working on this.

Hi @bianbianzhu, which version of SWA CLI are you using?
There was a possible similar issue #1560 recently which required a bugfix to be deployed on SWA CLI: https://github.com/Azure/static-web-apps-cli/pull/718
New version of SWA CLI: https://github.com/Azure/static-web-apps-cli/releases/tag/v1.1.4

You can get additional logging by using the following SWA CLI option per https://azure.github.io/static-web-apps-cli/docs/cli/swa#options
swa start --verbose silly [args]

Hi @seantleonard, thanks for the help.
The version of SWA CLI is 1.1.4
and the full log from swa start --verbose=silly is
Welcome to Azure Static Web Apps CLI (1.1.4)

Getting config file options from swa-cli.config.json...
Changed directory to /Users/tianyili/Development/freshfarmpicking-react
Using configuration "freshfarmpicking-react" from file:
  /Users/tianyili/Development/freshfarmpicking-react/swa-cli.config.json

***********************************************************************
* WARNING: This emulator may not match the cloud environment exactly. *
* Always deploy and test your app in Azure.                           *
***********************************************************************

Checking if localhost:4280 is accepting TCP connections...
Port 4280 is available. Use it.
Resolved port number: 4280
appDevserverUrl provided, we will try connect to dev server at dist
Api Dev Server found: http://<IP_ADDRESS>:7071
Trying to read workflow config with values:
 - appLocation: /Users/tianyili/Development/freshfarmpicking-react/src/web
 - outputLocation: http://localhost:3000
 - apiLocation: http://<IP_ADDRESS>:7071
Detected dev server configuration
User workflow config:
 - appLocation: /Users/tianyili/Development/freshfarmpicking-react/src/web
 - outputLocation: http://localhost:3000
 - apiLocation: http://<IP_ADDRESS>:7071
Running echo 'No Data API found'. Skipping
Starting the SWA emulator with the following configuration:
- ssl:
  - 0: false
  - 1: <undefined>
  - 2: <undefined>
- env:
  - SWA_RUNTIME_CONFIG_LOCATION: /Users/tianyili/Development/freshfarmpicking-react/src/web
  - SWA_RUNTIME_WORKFLOW_LOCATION: <undefined>
  - SWA_CLI_DEBUG: silly
  - SWA_CLI_API_PORT: 7071
  - SWA_CLI_APP_LOCATION: /Users/tianyili/Development/freshfarmpicking-react/src/web
  - SWA_CLI_OUTPUT_LOCATION: http://localhost:3000
  - SWA_CLI_API_LOCATION: http://<IP_ADDRESS>:7071
  - SWA_CLI_DATA_API_LOCATION: <undefined>
  - SWA_CLI_DATA_API_PORT: undefined
  - SWA_CLI_HOST: localhost
  - SWA_CLI_PORT: 4280
  - SWA_CLI_APP_SSL: false
  - SWA_CLI_APP_SSL_CERT: <undefined>
  - SWA_CLI_APP_SSL_KEY: <undefined>
  - SWA_CLI_STARTUP_COMMAND: yarn run dev
  - SWA_CLI_VERSION: 1.1.4
  - SWA_CLI_SERVER_TIMEOUT: 60
  - SWA_CLI_OPEN_BROWSER: false
- commands:
  - swa: node<EMAIL_ADDRESS>  - api: <undefined>
  - dataApi: <undefined>
  - run: cd "/Users/tianyili/Development/freshfarmpicking-react/src/web" && yarn run dev
yarn run v1.22.19
warning ../../../../package.json: No license field
$ vite
[swa] Loading staticwebapp.config.json schema...
[run] 2023-09-02T12:54:20.827Z vite:config bundled config file loaded in 138.89ms
[run] 2023-09-02T12:54:20.834Z vite:config using resolved config: {
[run]   plugins: [
[run]     'vite:optimized-deps',
[run]     'vite:watch-package-data',
[run]     'vite:pre-alias',
[run]     'alias',
[run]     'vite:react-babel',
[run]     'vite:react-refresh',
[run]     'vite:modulepreload-polyfill',
[run]     'vite:resolve',
[run]     'vite:html-inline-proxy',
[run]     'vite:css',
[run]     'vite:esbuild',
[run]     'vite:json',
[run]     'vite:wasm-helper',
[run]     'vite:worker',
[run]     'vite:asset',
[run]     'vite:wasm-fallback',
[run]     'vite:define',
[run]     'vite:css-post',
[run]     'vite:worker-import-meta-url',
[run]     'vite:asset-import-meta-url',
[run]     'vite:dynamic-import-vars',
[run]     'vite:import-glob',
[run]     'vite:client-inject',
[run]     'vite:import-analysis'
[run]   ],
[run]   server: {
[run]     preTransformRequests: true,
[run]     port: 3000,
[run]     sourcemapIgnoreList: [Function: isInNodeModules],
[run]     middlewareMode: false,
[run]     fs: { strict: true, allow: [Array], deny: [Array] }
[run]   },
[run]   resolve: {
[run]     mainFields: [ 'module', 'jsnext:main', 'jsnext' ],
[run]     browserField: true,
[run]     conditions: [],
[run]     extensions: [
[run]       '.mjs',  '.js',
[run]       '.mts',  '.ts',
[run]       '.jsx',  '.tsx',
[run]       '.json'
[run]     ],
[run]     dedupe: [ 'react', 'react-dom' ],
[run]     preserveSymlinks: false,
[run]     alias: [ [Object], [Object], [Object] ]
[run]   },
[run]   optimizeDeps: {
[run]     disabled: 'build',
[run]     force: undefined,
[run]     include: [ 'react', 'react/jsx-dev-runtime' ],
[run]     esbuildOptions: { preserveSymlinks: false }
[run]   },
[run]   esbuild: { jsxDev: true, jsx: 'automatic', jsxImportSource: undefined },
[run]   build: {
[run]     target: [ 'es2020', 'edge88', 'firefox78', 'chrome87', 'safari14' ],
[run]     cssTarget: [ 'es2020', 'edge88', 'firefox78', 'chrome87', 'safari14' ],
[run]     outDir: 'dist',
[run]     assetsDir: 'assets',
[run]     assetsInlineLimit: 4096,
[run]     cssCodeSplit: true,
[run]     sourcemap: false,
[run]     rollupOptions: { onwarn: [Function: onwarn] },
[run]     minify: 'esbuild',
[run]     terserOptions: {},
[run]     write: true,
[run]     emptyOutDir: null,
[run]     copyPublicDir: true,
[run]     manifest: false,
[run]     lib: false,
[run]     ssr: false,
[run]     ssrManifest: false,
[run]     ssrEmitAssets: false,
[run]     reportCompressedSize: true,
[run]     chunkSizeWarningLimit: 500,
[run]     watch: null,
[run]     commonjsOptions: { include: [Array], extensions: [Array] },
[run]     dynamicImportVarsOptions: { warnOnError: true, exclude: [Array] },
[run]     modulePreload: { polyfill: true },
[run]     cssMinify: true
[run]   },
[run]   configFile: '/Users/tianyili/Development/freshfarmpicking-react/src/web/vite.config.ts',
[run]   configFileDependencies: [
[run]     '/Users/tianyili/Development/freshfarmpicking-react/src/web/vite.config.ts'
[run]   ],
[run]   inlineConfig: {
[run]     root: undefined,
[run]     base: undefined,
[run]     mode: undefined,
[run]     configFile: undefined,
[run]     logLevel: undefined,
[run]     clearScreen: undefined,
[run]     optimizeDeps: { force: undefined },
[run]     server: {}
[run]   },
[run]   root: '/Users/tianyili/Development/freshfarmpicking-react/src/web',
[run]   base: '/',
[run]   rawBase: '/',
[run]   publicDir: '/Users/tianyili/Development/freshfarmpicking-react/src/web/public',
[run]   cacheDir: '/Users/tianyili/Development/freshfarmpicking-react/src/web/node_modules/.vite',
[run]   command: 'serve',
[run]   mode: 'development',
[run]   ssr: {
[run]     format: 'esm',
[run]     target: 'node',
[run]     optimizeDeps: { disabled: true, esbuildOptions: [Object] }
[run]   },
[run]   isWorker: false,
[run]   mainConfig: null,
[run]   isProduction: false,
[run]   preview: {
[run]     port: undefined,
[run]     strictPort: undefined,
[run]     host: undefined,
[run]     https: undefined,
[run]     open: undefined,
[run]     proxy: undefined,
[run]     cors: undefined,
[run]     headers: undefined
[run]   },
[run]   envDir: '/Users/tianyili/Development/freshfarmpicking-react/src/web',
[run]   env: {
[run]     VITE_BLOB_SAS_TOKEN: '',
[run]     VITE_APP_INSIGHTS_CONNECTION_STRING: 'InstrumentationKey=xxx;IngestionEndpoint=http://xxx;LiveEndpoint=https://xxx',
[run]     VITE_AZURE_SEARCH_CLIENT_ENDPOINT: 'https://xxx',
[run]     VITE_AZURE_SEARCH_KEY_CREDENTIAL: 'xxx',
[run]     VITE_GOOGLEMAP_API_KEY: 'xxx',
[run]     BASE_URL: '/',
[run]     MODE: 'development',
[run]     DEV: true,
[run]     PROD: false
[run]   },
[run]   assetsInclude: [Function: assetsInclude],
[run]   logger: {
[run]     hasWarned: false,
[run]     info: [Function: info],
[run]     warn: [Function: warn],
[run]     warnOnce: [Function: warnOnce],
[run]     error: [Function: error],
[run]     clearScreen: [Function: clearScreen],
[run]     hasErrorLogged: [Function: hasErrorLogged]
[run]   },
[run]   packageCache: Map(1) {
[run]     'fnpd_/Users/tianyili/Development/freshfarmpicking-react/src/web' => {
[run]       dir: '/Users/tianyili/Development/freshfarmpicking-react/src/web',
[run]       data: [Object],
[run]       hasSideEffects: [Function: hasSideEffects],
[run]       webResolvedImports: {},
[run]       nodeResolvedImports: {},
[run]       setResolvedCache: [Function: setResolvedCache],
[run]       getResolvedCache: [Function: getResolvedCache]
[run]     },
[run]     set: [Function (anonymous)]
[run]   },
[run]   createResolver: [Function: createResolver],
[run]   worker: {
[run]     format: 'iife',
[run]     plugins: [
[run]       'vite:optimized-deps',
[run]       'vite:watch-package-data',
[run]       'vite:pre-alias',
[run]       'alias',
[run]       'vite:modulepreload-polyfill',
[run]       'vite:resolve',
[run]       'vite:html-inline-proxy',
[run]       'vite:css',
[run]       'vite:esbuild',
[run]       'vite:json',
[run]       'vite:wasm-helper',
[run]       'vite:worker',
[run]       'vite:asset',
[run]       'vite:wasm-fallback',
[run]       'vite:define',
[run]       'vite:css-post',
[run]       'vite:worker-import-meta-url',
[run]       'vite:asset-import-meta-url',
[run]       'vite:dynamic-import-vars',
[run]       'vite:import-glob',
[run]       'vite:client-inject',
[run]       'vite:import-analysis'
[run]     ],
[run]     rollupOptions: {},
[run]     getSortedPlugins: [Function: getSortedPlugins],
[run]     getSortedPluginHooks: [Function: getSortedPluginHooks]
[run]   },
[run]   appType: 'spa',
[run]   experimental: { importGlobRestoreExtension: false, hmrPartialAccept: false },
[run]   getSortedPlugins: [Function: getSortedPlugins],
[run]   getSortedPluginHooks: [Function: getSortedPluginHooks]
[run] }
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteTimeMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / corsMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use /__open-in-editor launchEditorMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteHMRPingMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteServePublicMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteTransformMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteServeRawFsMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteServeStaticMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteHtmlFallbackMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteIndexHtmlMiddleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / vite404Middleware
[run] Sat, 02 Sep 2023 12:54:20 GMT connect:dispatcher use / viteErrorMiddleware
[run] 2023-09-02T12:54:20.848Z vite:deps Hash is consistent. Skipping. Use --force to override.
[run] 2023-09-02T12:54:20.851Z vite:esbuild 20.09ms tsconfck init /Users/tianyili/Development/freshfarmpicking-react/src/web
[run]
[run]   VITE v4.3.9  ready in 234 ms
[run]
[run]   ➜  Local:   http://localhost:3000/
[run]   ➜  Network: use --host to expose
[swa] Schema loaded successfully from https://json.schemastore.org/staticwebapp.config.json
[swa] Compiling schema...
[swa] Reading content from staticwebapp.config.json...
[swa] Parsing staticwebapp.config.json...
[swa] Validating staticwebapp.config.json...
[swa] File validated successfully. Continuing with configuration!
[swa] Content parsed successfully
[swa]
[swa] Found configuration file:
[swa]   /Users/tianyili/Development/freshfarmpicking-react/src/web/public/staticwebapp.config.json
[swa]
[swa] Validating dev server config:
[swa]  - url: http://localhost:3000
[swa]  - timeout: 60
[swa] Checking if localhost:3000 is accepting TCP connections...
[swa] Validating dev server config:
[swa]  - url: http://<IP_ADDRESS>:7071
[swa]  - timeout: 60
[swa] Checking if <IP_ADDRESS>:7071 is accepting TCP connections...
[swa]
[swa] Using dev server for static content:
[swa]   http://localhost:3000
[swa]
[swa] Using dev server for API:
[swa]   http://<IP_ADDRESS>:7071
[swa]
[swa] Azure Static Web Apps emulator started at http://localhost:4280. Press CTRL+C to exit.
[swa]
[swa]

I was affected by the unzip and node 18 issue previously and thought as SWA 1.1.4 no longer uses unzipper, it could be a good solution for me.
Thanks!

hi @bianbianzhu, in order to see the error, could you please reshare the verbose logs when --data-api-location argument is also given to swa start?
Something like:
swa start --data-api-location swa-db-connections --verbose=silly
Thanks!

Hi @Aniruddh25, the following is the the complete log from
swa start --data-api-location swa-db-connections --verbose=silly
OS: [macOS]
Version [Ventura 13.4]
Architecture: [arm64 (Apple M2 Pro)]
SWA CLI 1.14
Data-api-builder 0.8.50
Welcome to Azure Static Web Apps CLI (1.1.4)
Getting config file options from swa-cli.config.json...
Changed directory to /Users/tianyili/Development/freshfarmpicking-react
Using configuration "freshfarmpicking-react" from file:
/Users/tianyili/Development/freshfarmpicking-react/swa-cli.config.json

WARNING: This emulator may not match the cloud environment exactly. *
Always deploy and test your app in Azure.                           *

Checking if localhost:4280 is accepting TCP connections...
Port 4280 is available. Use it.
Resolved port number: 4280
appDevserverUrl provided, we will try connect to dev server at dist
Api Dev Server found: http://<IP_ADDRESS>:7071
Data Api Folder found: /Users/tianyili/Development/freshfarmpicking-react/swa-db-connections
Trying to read workflow config with values:

appLocation: /Users/tianyili/Development/freshfarmpicking-react/src/web
outputLocation: http://localhost:3000
apiLocation: http://<IP_ADDRESS>:7071
Detected dev server configuration
User workflow config:
appLocation: /Users/tianyili/Development/freshfarmpicking-react/src/web
outputLocation: http://localhost:3000
apiLocation: http://<IP_ADDRESS>:7071
Running dab --version
dab is not installed.
Downloading the version 0.8.50
[swa] ✔ Downloading<EMAIL_ADDRESS>Checksum match: 0496fb0cc47b9bd9cdddb86f4cb8bdbeb4dd5b0a7ac521ad1a5f01416cb6feb2
Saved binary to /Users/tianyili/.swa/dataApiBuilder/0.8.50/dab_osx-x64-0.8.50.zip
Saved metadata to /Users/tianyili/.swa/dataApiBuilder/DataApiBuilder.json
Setting executable permissions for data-api-builder binary
DataApiBuilder found: /Users/tianyili/.swa/dataApiBuilder/0.8.50/Microsoft.DataApiBuilder. Using this to start data-api server
Running cd "/Users/tianyili/Development/freshfarmpicking-react/swa-db-connections" && "/Users/tianyili/.swa/dataApiBuilder/0.8.50/Microsoft.DataApiBuilder" start -c staticwebapp.database.config.json --no-https-redirect
Starting the SWA emulator with the following configuration:
ssl:

0: false
1:
2:

env:

SWA_RUNTIME_CONFIG_LOCATION: /Users/tianyili/Development/freshfarmpicking-react/src/web
SWA_RUNTIME_WORKFLOW_LOCATION:
SWA_CLI_DEBUG: silly
SWA_CLI_API_PORT: 7071
SWA_CLI_APP_LOCATION: /Users/tianyili/Development/freshfarmpicking-react/src/web
SWA_CLI_OUTPUT_LOCATION: http://localhost:3000
SWA_CLI_API_LOCATION: http://<IP_ADDRESS>:7071
SWA_CLI_DATA_API_LOCATION: /Users/tianyili/Development/freshfarmpicking-react/swa-db-connections
SWA_CLI_DATA_API_PORT: 5000
SWA_CLI_HOST: localhost
SWA_CLI_PORT: 4280
SWA_CLI_APP_SSL: false
SWA_CLI_APP_SSL_CERT:
SWA_CLI_APP_SSL_KEY:
SWA_CLI_STARTUP_COMMAND: yarn run dev
SWA_CLI_VERSION: 1.1.4
SWA_CLI_SERVER_TIMEOUT: 60
SWA_CLI_OPEN_BROWSER: false

commands:

swa: node<EMAIL_ADDRESS>api:
dataApi: cd "/Users/tianyili/Development/freshfarmpicking-react/swa-db-connections" && "/Users/tianyili/.swa/dataApiBuilder/0.8.50/Microsoft.DataApiBuilder" start -c staticwebapp.database.config.json --no-https-redirect
run: cd "/Users/tianyili/Development/freshfarmpicking-react/src/web" && yarn run dev
[dataApi] /bin/sh: /Users/tianyili/.swa/dataApiBuilder/0.8.50/Microsoft.DataApiBuilder: Bad CPU type in executable
[dataApi] cd "/Users/tianyili/Development/freshfarmpicking-react/swa-db-connections" && "/Users/tianyili/.swa/dataApiBuilder/0.8.50/Microsoft.DataApiBuilder" start -c staticwebapp.database.config.json --no-https-redirect exited with code 126
--> Sending SIGTERM to other processes..
[swa] node<EMAIL_ADDRESS>exited with code SIGTERM
--> Sending SIGTERM to other processes..
[run] cd "/Users/tianyili/Development/freshfarmpicking-react/src/web" && yarn run dev  exited with code SIGTERM
✖ SWA emulator stoped because the --run command exited with code SIGTERM.

Cheers!

hmm, no body else has the same issue as me? I thought M2 Pro was not so different from M1

Close and waiting for future demand.


RHOAIENG-15105: Add offline support for LMEvalJobs

Refers to RHOAIENG-15105.
Add support for offline flags in the LMEvalJob container and mount PVC containing models and datasets.
If the optional offline field is set:
 offline:
    storage:
      pvcName: pvc-with-my-models

offline mode will be enabled. If it is missing, offline mode will be disable.

@yhwang Opened https://github.com/trustyai-explainability/trustyai-service-operator/issues/353


Add Support for Optimism Goerli

Website
https://community.optimism.io/docs/useful-tools/networks/
https://oplabs.notion.site/EXTERNAL-Optimism-Kovan-Optimism-Goerli-Migration-Progress-ced63e666fa34b148668581cb58809ee
ChainId
420
RPC Endpoint
https://goerli.optimism.io/
Thanks a bunch! :-)

Just a note: for the Safe contract deployment this is not strictly necessary as the default deployment factory is supported


Dragonrage: Watch cricketagony.com

Dragonrage requests the watch of the watch_keyword cricketagony\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Mast in Charcoal HQ

cricketagony\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Fix error on community detection with non pinned dependencies

Fixed the error described in https://github.com/aws-samples/amazon-braket-community-detection/issues/1

Hello, I've done all the check, even executing the quantum jobs and all seems working correctly :smile:


Explain what tf.nn.softplus does to integers

tf.nn.softplus computes log(1 + exp(x)).  Naively, I wouldn't expect this to work for integers, but it does.  On integers, it seems to degenerate to a poorly named version of tf.relu: it computes max(0, x).
We probably can't eliminate the integer versions for backward compatibility reasons, but we should at least explain what they do.

/CC @wolffg for documentation.

Hmm, there seems to be more of this in core/ops/nn_ops.cc.  E.g., softsign is the same.
I added it to the doc fixit spreadsheet.

tf.nn.softsign on ints is rather less useful: it's the constant zero function.
@martinwicke: Maybe we should drop these?  Arguably they're just bugs.

I'm a little reluctant to remove the functionality since that will break graphs (and code) that worked before. But a warning (deprecation?) would be nice.


Failed when installing using pip

Dear Ben,
It seems to be an issue with the dependecies when installing your module using pip:
Collecting spacy-to-hf
Using cached spacy_to_hf-0.0.3-py3-none-any.whl (10 kB)
Collecting spacy<3 (from spacy-to-hf)
Using cached spacy-2.3.9-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (4.9 MB)
Collecting pytokenizations (from spacy-to-hf)
Using cached pytokenizations-0.8.4.tar.gz (3.8 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
error: subprocess-exited-with-error
× Preparing metadata (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
Preparing metadata (pyproject.toml) ... error
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
Best regards,
Balazs

@bbanfai what version of python are you running?

I fixed it sir.

It's the same issue as here:
https://github.com/explosion/tokenizations/issues/92
I forgot but somewhere the author wrote he moved it to:
https://github.com/explosion/spacy-alignments?tab=readme-ov-file

@bbanfai just released 0.0.4, this should be addressed :)


Supporting data on another page

Note: for at least one class, we know lots and lots about the propulsion for a platform. So, it's in document of its own. In another case, a group of classes all share the same general propulsion details.   So these link to a shared propulsion document.  I wish to reproduce this in our mock data, so we know what to push the legacy content to.
Ian to generate DITA page containing some sections of data, with IDs.
A mock class should refer to this page, instead of it's own Propulsion section.  We need this to work in a way that the author can maintain the content.
I Guess the propulsion could either be a propulsion element or a propulsionRef element that is an XREF to either another document, or a named section in another document.  Aah, the propulsionRef should take an optional title field. This is what is shown in the in-page navigation block.
The publish process will create an html page for the class, and the in-page nav will show the title (if present), and link to the target content.

Hello @brahnavank - does the above strategy seem achievable?

Something like this?

Ok.  The PR linked above contains an XML structure that meets this requirement.
I'll modify the branch to include suitable content.


Fix race in QueryScheduler

Ensure the scheduler doesn't run a query if the previous run hasn't completed.

Fixed MaxListenersExceededWarning seen while running QueryScheduler.spec.js.

💃


New transaction format

environment:
plasma-contracts: 835d10c
elixir-omg: 40f899b
193
177

I'm curious why we're moving away from bignumber.js to bn.js? 🤔


fail to run test in maven

I encountered an issue that description is "org.springframework.jdbc.UncategorizedSQLException: PreparedStatementCallback; uncategorized SQLException for SQL [SELECT id as visit_id, visit_date, description FROM visits WHERE pet_id=?]; SQL state [S0022]; error code [0]; Column 'visits.id' not found.; nested exception is java.sql.SQLException: Column 'visits.id' not found." when running test in maven.
I use MySQL as database.

I think the root cause is that the sql in method "findByPetId" of class JdbcVisitRepositoryImpl are not mapped with the method "mapRow" of class JdbcVisitRowMapper.

Hi MikeZheng, could you please pull the last commit and execute your test again?


Provide a React component

Description
Provide a npm module that exposes a Player component for React.
Idea: import Player from 'indigo-player/react' (currently not available, this is merely an idea).
Code sample
import React, { useRef, useEffect } from 'react';
import PropTypes from 'prop-types';
import IndigoPlayer from 'indigo-player';

const CHUNKS_PATH = `https://cdn.jsdelivr.net/npm/indigo-player@${
  IndigoPlayer.VERSION
}/lib/`;

export default function Player({ config, chunksPath }) {
  const container = useRef(null);

  useEffect(() => {
    IndigoPlayer.setChunksPath(chunksPath);
  }, [chunksPath]);

  useEffect(() => {
    const player = IndigoPlayer.init(container.current, config);
    return () => player.destroy();
  }, [config]);

  return <div ref={container} />;
}

Player.propTypes = {
  config: PropTypes.shape({}).isRequired,
  chunksPath: PropTypes.string,
};

Player.defaultProps = {
  config: undefined,
  chunksPath: CHUNKS_PATH,
};

[x] Proper cleanup when player.destroy() is called.
[ ] Find a proper way to expose the player API which is returned by IndigoPlayer.init.
I'm not convinced about the IndigoPlayer.setChunksPath implementation for libraries. Chunks never really worked out-of-the-box when including an npm lib.

Find a proper way to expose the player API which is returned by IndigoPlayer.init

You could have a prop called onPlayerDraw or some such that fires after a player is ready and sends back the player along with anything else that is needed, like if (typeof onPlayerDraw === 'function') onPlayerDraw({player}); It would allow you to get the player API to the user if they want it (some may not need it) and allow them to run their own functionality to deal with the player draw (like autoplay checks and the like).

I've had to use the same thing on a React project to notify a parent component that a child component is now hidden.

Thought about it too but using a callback on a sync instruction (IndigoPlayer.init is sync) feels a bit counter-intuitive. One thing I'd like to try is using the useRef hook to create a reference to the player in combination with the useImperativeHandle hook to provide strictly the player API in the ref.
Eg:
const player = useRef();

// Access controller

// eg: player.play(), player.pause(), ...

// Access state
const [playerState, setPlayerState] = useState(null);

useEffect(() => {
  if (player.current) {
    player.current.addListener(Events.STATE_CHANGE, state => setPlayerState(state));
    return () => player.current.removeListener(Events.STATE_CHANGE);
  }
}, [player.current]);

// eg: access state like playerState.playing, playerState.paused, ...

<Player ref={player} config={config} />

@FreddyJD not at the moment, I've recently merged in https://github.com/matvp91/indigo-player/pull/53 which is a first step in a better module structure. Once I've cleared out the other roadblocks, you'll be able to install indigo-player as a module. This would make it a whole lot easier to wrap it around a React component.

@matvp91 Thanks for the sample.
Unfortunately I'm new to both React and Typescript and thought I'd found a holy-grail here, but alas I can't get it to work as is.

Property 'init' does not exist on type 'typeof import("/home/jon/source/myapp/node_modules/indigo-player/src/index")'.

I did manage to fix an issue with the useRef usage by making the following change:
const ref = useRef<HTMLDivElement>(null);


Tests + CI

I'm not a TDD fan, but we must keep tests in mind.
@syxolk Would be great if you could implement a couple of very simple tests for the backend, I'd take a look at them and try to write my own.

[ ] CodeClimate
[ ] Codacy

Deployment:

[ ] Dockerfile
[ ] Dply

Any chance we can deploy to the LRZ compute cloud? I read the manual but didn't get whether it is a good platform for a web server.
I'll contact LRZ one of these days.

BTW,

[ ] David

Oops, didn't see your last commit, sorry.

Codacy won't allow me to add web-quiz as a project.
CodeClimate checked the dev branch but doesn't server any badges for it (only master)

I've authorised Codacy and it seems like it doesn't like dev too.
I'd suggest we move the development over to master, I'm tired of changing branches every time I open the repo.

Oh, Codacy does allow you to select the branch to work with.

Any chance we can deploy to the LRZ compute cloud? I read the manual but didn't get whether it is a good platform for a web server.

Looks promising. Would be great, if they provide a server similar to a small Amazon EC2 instance.

How about Appveyor?

Honestly, I don't want to support development on Windows. So I say nay.


C# export typed enum's are corrupted when in a collection or array form.

Version:
v3.2.3.stable.mono.official
OS/Device
Windows 10 PC. High end.
Issue Description:
When exporting a collection of enumerators as a List or Array ([]), the properties inspector gives the ability to add and populate a list of of said enumeration. You can save and load the class, and the properties remain in tact. However if you attempt to access said variable at any point (even in its original property setter) , the results (value) are corrupted. The LENGTH of the collection or array is correct, but the data is not correct. I am seeing this issue with ANY exported collection of enumerations right now. Note that AFAIK enumerations work, in general. Collections work, in general. But collections and enumerations, together, bring back faulty data.
Steps to reproduce:
Create a script that subclasses Node. Create an enum type. Create a property or public exported variable of a collection type or array type on that node. Drag the node into the project. Set up a list of the enumerations. Save it. Write any code at any location that accesses said variable. Observe said variables contents. (or just see image)

Note the image above image is from a right as soon as the remote debugger attached and it was setting the value from the serialized scene.

I was able to do a bit more diagnostics on this, and I have a workaround and a hint as to whats happening.
If you export an array, the node inspector will represent it as an intpool.
If you export as a list, the node inspector represent it as an Array. (what looks right)
In both cases you can save and load these lists in the inspector, they serialize fine. they LOOK fine from the editor.
But when the scene consumes the values you have initialized, it does not correctly marshal them to the appropriate exported types. It reeks of an unmanaged cast to a generic collection interface, and my bets (working on testing this) is its really a Godot.Collections.Array being improperly cast through the CLR as a generic List interface.
A workaround for this is to only use the Godot.Collections.Array type for the lists being exported, at least so far as enumerations are concerned.

Yeah. Enums seem to be pretty buggy in the inspector for C# and GDScript

@Calinou This is still not fixed in Godot V3.4.stable for GDScript like mentioned in #52140

@Calinou This is still not fixed in Godot V3.4.stable for GDScript like mentioned in #52140

This issue will only be fixed in 3.5 as https://github.com/godotengine/godot/pull/53942 was merged too late for 3.4 (it was in feature freeze).

That PR doesn't fix this issue, this issue should be fixed in 3.4, I think #52140 is actually not a duplicate of this one and probably should not have been closed.


Globbing not working with PhysicalFileProvider

As title says, globbing is not working with PhysicalFileProvider. This was already reported in related GitHub repository but got rejected: https://github.com/dotnet/extensions/issues/486.
I'm opening the issue here because the documentation says the opposite (even if the doc is generic but not only for PhysicalFileProvider).

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

ID: 8d8787c1-1a02-ae68-aa59-7c0e2e823dde
Version Independent ID: 3e799e5d-a89c-cd03-999f-92ae431f4657
Content: File Providers in ASP.NET Core
Content Source: aspnetcore/fundamentals/file-providers.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@ydogus  thanks for pointing this out.


Update title and use it in alert dialog

Update https://github.com/google/horologist/blob/main/base-ui/src/main/java/com/google/android/horologist/base/ui/components/Title.kt to match redlines
And use it in the newly created AlertDialog

@laiyichin Closing as stale, reopen if still useful.


Automated Package Upgrade

:tada: This PR is included in version 6.0.22 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issue with mod interactions and not all chests being lootr

WIP-1.0.zip

I'm sorry, but this isn't an adequate.
You need to write a list of specific mods that aren't working and the specific structures associated with them.
There's no way I'm going to randomly download a zip file from an issue report.

it wouldn't let me use html file...

A list of mods in your pack isn't going to suffice either.
I need a list of the specific mods whose structures do not have Lootr chests.


UI: Go-To-Top Icon not visible

The  Go-To-Top icon is not visible when not in the active state. I would like to fix it.

@Spandan-Bhattacharya please assign this issue to me under JWOC'24.

please Assign it to me

@AryanPahuja21 Go for it!


Add a new host name (domain) in created wildcard

In my Windows Server 2008 r2 (IIS 7.5), I have created a wildcard cert adding four of existing site in IIS using Let’s Encrypt windows tool.
Then I have renamed my cert, introducing the “*.” before the name in order to binding wildcard to all the four sites in IIS. This workaround worked fine, so now I have an SSL certificate for all the sites.
Now the problem is: I have created a new site in IIS, how can I add it to the wildcard certifcate just created to assign the https to it?
I need to create a NEW wildcard that include the previous four and the new domain? Or is it possible to “add” the new one in the wildcard? What I need to do…?
Best regards and thanks a lot for any help.

A wildcard certificate (e.g. *.example.com) doesn't care for how many subdomains you use it. You shouldn't have to request a new certificate if all of your sites fit the pattern, e.g. a.example.com, b.example.com etc.
Somehow I suspect that you may have a SAN certificate, which is something different. A SAN certificate can be valid for foo.org and bar.net. Then if you want to add example.com to that, you will have to request a new one. That can be done by simply requesting a new certificate like you did before and cancelling renewal of the older one.

Thank you for your answer.

No problem, thanks for using win-acme :)


Write a Guide for Deploying the App to Digital Ocean

Please name it Digital_Ocean.MD
Make and place it inside the Guides/ directory of the master branch
Add Snapshots of the process as and where required
Add Useful Tips section for certain optimizations, bug fixing advice and cite corresponding references

CAN I WORK ON THIS ?

@mohit200008 go ahead. You can work on this issue.

please assign me this task (if you thinks it will enhance your project) under the Lgm-Soc'21 and add labels if possible.

@Astitva7717 the issue is already assigned to someone else, so you are requested to go for other issues.


Add code coverage for types/

Codecov Report

Merging #60 into master will decrease coverage by 0.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   86.29%   85.92%   -0.37%
==========================================
  Files          27       29       +2
  Lines         744      874     +130
==========================================
+ Hits          642      751     +109
- Misses        102      123      +21

Impacted Files
Coverage Δ

types/list.js
69.56% <0%> (ø)

types/struct.js
91.66% <0%> (ø)

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 b7bc92a...8dd54c7. Read the comment docs.


Weirdness in canvas

I like this plugin a lot but I also use canvas sometimes so I disable this plugin whenever I want to edit text in canvas. If this plugin didn't effect how editing in canvas works then that would be a viable hotfix.

There is an option in settings that lets you disable the plugin in canvas. Have you tried this? If the problem persists, please reach out to me again.

Edit: Since the plugin causes issues with several plugins (e.g. canvas, kanban, lineage) and it is out of scope to make it compatible with all plugins, I decided to only enable it in markdown files from the next version on. This means that it should now be disabled in canvas by default.

I hadn't tried that, sorry for not checking plugin settings. XD

Edit: Since the plugin causes issues with several plugins (e.g. canvas, kanban, lineage) and it is out of scope to make it compatible with all plugins, I decided to only enable it in markdown files from the next version on. This means that it should now be disabled in canvas by default.

Yeah this seems like a smart move, thanks for the great work!


bug: package opentrons 7.1.1 is not published to pypi

Overview
The latest non --pre release 7.1.0
The 7.1.1a0 and 7.2.0a2 are both pre-releases. When the 7.1.1 release of the robot stack came out and got installed onto the Flexes, our 7.1.0 protocols stopped working.
This is related to #14208 where the opentrons-hardware package dependency also isn't getting published to pypi
Our current workaround is to point our dependencies to the git repo rather than pipy.
Steps to reproduce
No response
Current behavior
No response
Expected behavior
No response
Operating system
None
System and robot setup or anything else?
7.1.1 app ad Flex robot stack

Thanks for the report, and sorry for the delayed reply. Indeed, it looks like our v7.1.1 deployment failed because of a CI error. This got fixed in #14464.
It might be a little annoying to go back and fix this for v7.1.1, so I suspect we'll leave this alone for v7.1.1 and just fix it going forward for v7.2.0.


Use trajectory id as namespace for trajectory marker

Continuation of #452.

@jihoonl I assume this is OK with you since you closed #452 and referred to this one.

yeah no problem.


authenticate method not named authenticate_user!

An app using devise can sometimes have it's authenticate method to be something other than authenticate_user!, for example it could be authenticate_employee!. An easy work around for now is just to add a method in your application_controller names authenticate_user! that calls the proper method.

Closing, as this will be moving to closed source.


Game screen size is way to large for many people

1920 x 1080 pixels??? Come on! I do not have a so old monitor, but it is smaller than that.  My computer has a screen of 1366 x 768 pixels. So, setting up and playing the game in your site, today (2021/10/20) was basically a BIG challenge, and i downloaded the source, to start, just to change this "detail".

Hey, this game isn't really optimized. We made this over a weekend as a game jam, and you'll have to expect it to be sub-par on features that wasn't important at the time. We're picking up the development of this game asap, and we'll add responsive resolution to the short list.


Update uploader() to slow down initial message doubling

This PR works around the Safari bug where bufferedAmount isn't updated immediately after a sock.send() and thus causes:

the send loop to spin extremely fast at the beginning (when messages are small)
too many sock.send() in a very short time
the test itself crashing with a Failed to send WebSocket frame error because it tried sending more data than the WS buffer can contain.

By removing the loop in the uploader function no more than one send() can happen in a single call to uploader(), and this slightly slows down the beginning of the upload. It also changes the logic from:

On each call to uploader(), completely fill the buffer

to:

On each call to uploader(), queue one more message if the buffer is not full already

Additionally, by looking at WebKit's source code I noticed that the maximum size for the WebSocket buffer is 100MB. By reducing maxMessageSize from 16M to 8M we keep the buffer size below that value and also have some overhead for when bufferedAmount is updated late. The (new) maximum buffer size is 8 * 8MB - 1 byte ~= 64M.
The combination of these two changes seems to be enough for Safari not to crash. This has been tested under different simulated network speed and latency conditions.
Please note that this is just a workaround. I believe a "real" solution would involve changing WebKit so that bufferedAmount is always reported correctly, likely by synchronously incrementing it after a sock.send() and before returning to the JS engine. This also seems to be how Chrome's and Firefox's WebSocket implementations work since this issue could not be replicated there -- bufferedAmount immediately increases at the beginning of the upload, right after the first sock.send().
Closes https://github.com/m-lab/ndt7-js/issues/44.

This change is

Is the slack open to contributors outside of the organization?

Hi @RA80533, definitely. Thank you for submitting the PRs you have. We're working on refining our process for reviewing external contributions outside of the core contributors team, but in the mean time, if you're interested in contributing more, could you reach out to me at<EMAIL_ADDRESS>Thanks!


containerd: 1.4.4 -> 1.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/containerd/containerd/releases.
meta.description for containerd is: "A daemon to control runC"
meta.homepage for containerd is: "https://containerd.io/"
meta.changelog for containerd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/containerd passed the binary check.

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/containerd-shim passed the binary check.

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/containerd-shim-runc-v1 passed the binary check.

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/containerd-shim-runc-v2 passed the binary check.

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/containerd-stress passed the binary check.

/nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin/ctr passed the binary check.

6 of 6 passed binary check by having a zero exit code.

3 of 6 passed binary check by having the new version present in output.

found 1.5.0 with grep in /nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0

found 1.5.0 in filename of file in /nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0

directory tree listing: https://gist.github.com/c8ee05344b89c3f3c61145653362be21

du listing: https://gist.github.com/f97ac0c7423ef1f36d33e0ab911a9fc4

Rebuild report (if merged into master) (click to expand)

48 total rebuild path(s)

17 package rebuild(s)

17 x86_64-linux rebuild(s)
15 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
16 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
charliecloud
containerd
disnix
disnixos
docker
docker-edge
docker-gc
docker_20_10
dydisnix
dysnomia
grype
ignite
nvidia-docker
out-of-tree
pipework
python38Packages.jupyter-repo2docker
python39Packages.jupyter-repo2docker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A containerd https://github.com/r-ryantm/nixpkgs/archive/0192fb36a98230b173240011c0124b03610b3fff.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0
ls -la /nix/store/dwnadrn7l14s2kf33cx1nb5sbnz9kmpa-containerd-1.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @offlinehacker @vdemeester for testing.

Result of nixpkgs-review pr 121642 at 0192fb36 run on x86_64-linux 1

1 package failed to build:

dydisnix

14 packages built successfully:

charliecloud
containerd
disnix
disnixos
docker (docker-edge ,docker_20_10)
docker-gc
dysnomia
grype
ignite
nvidia-docker
out-of-tree
pipework
python38Packages.jupyter-repo2docker
python39Packages.jupyter-repo2docker

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/go-packages/generic/default.nix:216:5:
    |
216 |     installPhase = args.installPhase or ''
    |     ^

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild.
Near pkgs/development/go-packages/generic/default.nix:138:5:
    |
138 |     buildPhase = args.buildPhase or ''
    |     ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.


EF7 Class Inheritance

using EntityFramework 7.0.0.-beta1
I have class inheritance setup with the intent to implement Table per Hierarchy (TPH) database schema on SQL Server.  As I understand it EF7 should default to TPH but in my test application it is defaulting to (TPC).  Is there a HINT I can give EF7 to indicate TPH is my intent?
public class Book
{
public int BookId;
public string Name;
}
public class Paperback : Book{}
public class Notebook : Book{}
public class Hardcover: Book{}

I found where TPH support is a still a TODO item in the repo...

Here is the work item tracking inheritance support: https://github.com/aspnet/EntityFramework/issues/247


Issue compiling on Windows - error C2220 and warning C4204

Hi Tarek, thank you for making this project!
For anyone trying to compile this project on windows (Windows 10 Home for me), I ran into a slight problem. After running build.bat in a shell with cl setup, I got an error:
....
..\src\game\entities.c(135): error C2220: the following warning is treated as an error
..\src\game\entities.c(135): warning C4204: nonstandard extension used: non-constant aggregate initializer
...More of these warnings...

The way I solved this for now was to remove the /WX compile option from the build.bat script (basically replace line 8 with the line below):
@set shared_args=/std:c11 /W4 /wd5105 /wd4201 /wd4100 /D SOGL_MAJOR_VERSION=3 /D SOGL_MINOR_VERSION=3 /Fespace-shooter

/WX tells the compiler to treat all warnings as errors, so this probably isn't the best solution! But it's a workaround for now. Im not a C/C++ dev so not sure how to solve "non-constant aggregate initializer".  If I figure it out I will post here.

Thanks for reporting this @mawaldne. Can you tell me what version of MSVC you're using? I don't get those errors when I compile with MSVC 2019 Community Edition.
Warning C4204 is related to the fact that I initialize those "argument" structs with non-constant values, but it looks like that should only be emitted when the ANSI compatibility flag (/Za) is set. I believe this behavior is valid in C11, but I'll double check that.
If you explicitly enable MS extensions by adding the /Ze flag to the shared_args does it work?

I figured it out. I was on an older version of Visual Studio Community. I went from 16.4.2 -> 16.11.8 and that seemed to fix the problem. Compiles on Windows now with no issues.
Thanks you!

Great! Thanks for letting me know.


"live" directory always empty, no m3u8 file

Describe the bug
After activation of the start stream button "live" directory always empty, no m3u8 file
To Reproduce
Steps to reproduce the behavior:

Go to 'Objets'
Click on 'eufy-security.0.T8010XXXXXXXXXXX.cameras.T8114XXXXXXXXXXX.start_stream' button.start

Expected behavior
The presence of the eufy-security.0/T8010XXXXXXXXXXX/live/T8114XXXXXXXXXXX.m3u8 file.
Screenshots & Logfiles
attached logfiles
LogEufy18022021.txt
Versions:

Adapter version: 0.2.3
JS-Controller version: 3.1.6
Node version: v14.15.3
Operating system: Ubuntu 20.04.1 LTS

Hi @rmickael62 ,
can you please increase the debug mode to silly, reproduce the issue and send me the logs? With silly debug mode I get more information to understand the problem.
Thanks!

Hi @bropat,
Thank you for the interest of my problem, here is the requested file.
LogEufy20022021.txt

@rmickael62 try version 0.2.5 and let me know.

log with version 0.2.5.
I delete the instance and put a new one and leave the stream for a few seconds for a rather small log.
still no m3u8 file created  :-(
LogEufy20022021_2.txt

@rmickael62 can you try version 0.3.0 and let me know?

I don't think I'm lucky because the problem always seems the same.
LogEufy07032021.txt

@rmickael62 - i had the same issue. In the end I reinstalled everything (including iobroker). I run iobroker just for eufy so not a big deal for me. Not sure what your setup is but to replicate my solution (and rule out other issues) perhaps initiate a new docker container and test with just a webserver and eufy adapter installed?

@ronaldt80 I had a glimmer of hope when I saw your massage because I did not try to reinstall iobroker in full but unfortunately after creating a new iobroker instance under docker the problem is still present.

@rmickael62
I think the best solution is to organize a google meet and troubleshoot the problem right at your end.
Let me know.

@bropat It's a good idea but I only master oral French


topK in kernel benchmark script utilises <15% GPU

🐛 Bug
I was running the kernel benchmarks to evaluate the performances of the models. I started by running only topK by removing all the model name expect for topk in main.py. While running the script I found that topK is utilising only <15 % of the GPU (GTX 1080 Ti). It is taking 9 min for last config of hyperparam on MUTAG and >40 mins on last hyperparam on PROTEINS dataset. Is something wrong with topK implementation or is it normal for it to have such a low GPU utilisation?
To Reproduce
Steps to reproduce the behavior:
1.Remove all the names of models except topK here.
2. Run main.py

Environment

OS: Ubuntu 16.04.5
Python version: 3.7.3
PyTorch version: 1.1.0
CUDA/cuDNN version: 10.0
GCC version: 5.4.0
Any other relevant information:

I guess this depends on the batch_size, num_features and dataset which one uses, e.g. the MUTAG dataset is really small, so I think it is quite normal to expect low GPU utilisation?

Yes, that could be MUTAG. But for PROTEINS, the GPU util is not exceeding 15% and it takes total 40 min to run on all hyperparameters. In this case it will take around 2 days to run all 11 models in the list on 1 1080 Ti. Is the benchmark timing similar to what you must have done on your system?

Yes, it takes that long as well on my system. But in the end, we train around 8800 models (with 10-fold cross validation).

@rusty1s The benchmark is using stratified K-fold technique however I cannot see any mentions of it in  the papers of diffpool, gPool or set2set. Wouldn't using stratified k fold increase the performance as there is a data leak from the test data to train and val due to the fact that we are considering the test distribution before splitting to allow all the splits to have nearly equal proportion of classes?

Well, technically you are right, but personally I find this evaluation much more reasonable than hoping for some lucky splits. In the end all those datasets are quite balanced so this shouldn't affect the performance at all. I experienced this in my tests myself.

Thank you for the clarifications and help! I consider this as resolved and will now close this issue.


Corrected the path of the dataset to be 1 level up instead of 2

../../data/cleaned_cuisines.csv-> ../data/cleaned_cuisines.csv

ah yes, this reflects the path from /solutions - better to have it at level 2 rather than 3.

ah yes, this reflects the path from /solutions - better to have it at level 2 rather than 3.


prevent panic on build error

Describe the bug
If conducting an ocictl build, occasionally, if there is a build error, the error channel go routine will panic and output a stack trace.
Expected behavior
Only ocibuilder defined errors should be output to the tool users
Screenshots

If applicable, add screenshots to help explain your problem.

closed in #127


Stacktrace is not printed by failure signal handler when built with Bazel

When glog is built with Bazel, failure signal handler is unable to print stack trace.
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1720166741" if you are using GNU date ***
PC: @                0x0 (unknown)
*** SIGSEGV (@0x3e8000070c7) received by PID 28701 (TID 0x7ffff478aa40) from PID 28871; stack trace: ***
This problem has occurred before and has been fixed, but now it has reappeared. https://github.com/google/glog/issues/346

According to #851 this should be working. What glog version are using?

My mistake. The stacktrace is indeed printed, but it's not interpreted as symbols (function names)，LOG(FATAL) doesn't interpret either.


contract register error - rgc1... invalid

Classify the bug
Wrong validation
Describe the problem
The 'rgb-cli' return invalid format of the contract source generated by 'rgb20'
error: Invalid value "rgbc1...." for '<CONTRACT>': payload data are not strictly encoded
- An unsupported value `TypeRef` for enum `TypeRef` encountered during decode operation

To Reproduce
Steps to reproduce the behavior:

Generate the contract using rgb20 (version: 0.8.0-rc.2)
Try register the contract using rgb-cli (version: 0.8.0-rc.1)

Other information
I tried to run these commands using signet and testnet.

Please try rgb20 version 0.8.0-rc.3

That's works!
Thanks!


Change cSpell.enabledLanguageIds yml to yaml

Per VSC Language Identifiers, the official one for YAML is yaml. However, client/package.json uses yml. So, getting spell check to work for YAML in VSC 1.33.0 requires a workaround. Suggestion: change yml to yaml

Thank you. I'll release a version that has it on by default.


Document both the low-level API and the SPARQL API

SPARQL API is via ARQ, already documented. Low-level API is now more minimal.


[DOCS]: Enhance Readme

Issue Description
The README file is the first thing users and contributors see when they visit your repository. It serves several key purposes:
Introduction and Overview: It provides a clear and concise introduction to your project, explaining what it does, its purpose, and why it’s valuable.
Installation Instructions: It guides users on how to set up and install the project on their local machines, ensuring they can quickly get started.
Usage Guidelines: It offers detailed instructions on how to use the project, including examples and common use cases.
Contribution Information: It informs potential contributors on how they can contribute to the project, including any guidelines or requirements they should follow.
License Information: It specifies the licensing terms under which the project is distributed, ensuring users and contributors understand their rights and obligations.
Kindly assign this issue to me under GSSoc'24.
Screenshots or Examples (if applicable)
No response
Proposed Solution (optional)
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines
[X] I have read the Code Of Conduct
[X] I joined onboarding group for knowledge flow and direct conversation and support from admins  Curious Developers Community - Curious Connect
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done
[X] If you need any help please join the default  (Google Meet Link)

You can start working on it @aasthakourav20

Instructions
You can start working on it by taking latest pull from main branch

[ ] Please raise pull request to main
[ ] Please read the contributors guidelines Contributing Guidelines.
[ ] Please follow the code of conduct Code Of Conduct
[ ] Don't Forget to join onboarding group for knowledge flow and direct conversation and support from admins
[ ] Link Curious Developers Community - Curious Connect
[ ] Please attach the screenshots and video of the development work once it is done
[ ] Contributions more than 3 will be awarded a certificate of appreciation from curious ecosystem
[ ] please mail to<EMAIL_ADDRESS>for claiming certificate and please attach your issue number

if any help required reach-out to admins or mentors in the onboarding group default meeting link and approach to admins - GOOGLE MEET Issue Discussion Google Meet
Respect Co Developers And Please Reach-out to admins if any issues

you can start working on this @aasthakourav20

Can I do this

Can you assign me this issue @thecuriousteam


Use easy scrypt in tests (N=2 vs N=8192)

Motivation
Unit tests that initialize POST used very expensive scrypt params (N=8192). Tests were initializing POST for the most time.
Changes
Change Scrypt N 8192 -> 2 in unit tests
Reduced testing time for activation pkg 1m35s -> 15s.

bors merge


Dynamic filtering on Zuul SimpleRouteLocator.getRoutes()?

I have a need for dynamically filtering out zuul routes when SimpleRouteLocator/DiscoveryClientRouteLocator assembles its routes in the getRoutes() and getMatchingRoute() methods.
The need is security based in that I don't think all routes should be advertised, if, say, a particular user (Principal) doesn't have access to the given routes/paths.  For instance, I want to allow fine-grained route access to users via a custom JWT claim that specifies what routes/paths he/she has access to.  I could prune routes in my application after calling routeLocator.getRoutes(), but I'd have to do that in multiple places potentially.  And I'm sure I'd forget completely about such post-filtering after a week or so and end up leaking unauthorized routes elsewhere in my code.  I'd much rather specify a filter at configuration time that SimpleRouteLocator can use so that I'm guaranteed that all calls to routeLocator.getRoutes() returns only the routes the user is allowed to see.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e4e29e1). Click here to learn what that means.
The diff coverage is 71.42%.

@@            Coverage Diff            @@
##             master    #2190   +/-   ##
=========================================
  Coverage          ?   70.67%
=========================================
  Files             ?      230
  Lines             ?     7638
  Branches          ?      944
=========================================
  Hits              ?     5398
  Misses            ?     1859
  Partials          ?      381

Impacted Files
Coverage Δ

...cloud/netflix/zuul/filters/SimpleRouteLocator.java
80.61% <71.42%> (ø)

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 e4e29e1...4ec0d78. Read the comment docs.

I'd prefer to not add another level of abstraction here. Can subclassing SimpleRouteLocator or having another RouteLocator that delegates and filters work?

@spencergibb yeah, I understand, no worries.  Can I propose an even simpler change to SimpleRouteLocator that will allow me to subclass it and have the control over the route assembly that I'm seeking?
FROM THIS:
public List<Route> getRoutes() {
	if (this.routes.get() == null) {
		this.routes.set(locateRoutes());
	}
	List<Route> values = new ArrayList<>();
	for (String url : this.routes.get().keySet()) {
		ZuulRoute route = this.routes.get().get(url);
		String path = route.getPath();
		values.add(getRoute(route, path));
	}
	return values;
}

TO THIS:
public List<Route> getRoutes() {
	if (this.routes.get() == null) {
		this.routes.set(locateRoutes());
	}
	return assembleRoutes(Collections.unmodifiableMap(this.routes.get()));
}

protected List<Route> assembleRoutes(Map<String, ZuulRoute> routes) {
	List<Route> values = new ArrayList<>();
	for (String url : routes.keySet()) {
		ZuulRoute route = routes.get(url);
		String path = route.getPath();
		values.add(getRoute(route, path));
	}
	return values;
}

Sample Subclass:
private static class FilteringRouteLocator extends SimpleRouteLocator {
	public FilteringRouteLocator(String servletPath, ZuulProperties properties) {
		super(servletPath, properties);
	}

	@Override
	protected List<Route> assembleRoutes(Map<String, ZuulRoute> routes) {

		List<Route> values = new ArrayList<>();
		for (String url : routes.keySet()) {
			ZuulRoute route = routes.get(url);
			if (acceptRoute(route)) {
				String path = route.getPath();
				values.add(getRoute(route, path));
			}
		}
		return values;
	}

	private boolean acceptRoute(ZuulRoute route) {
		return route != null && !(route.getId().equals("foo"));
	}
}

Hi Spencer,
I absolutely could, but it isn't the most desirable or efficient solution.  Let me explain.
The big sticking point is that SimpleRouteLocator controls the cache of the routes in a privately held AtomicReference that subclasses don't have access to.  That's appropriate.  However, if I were to override SimpleRouteLocator.getRoutes(), I'd have to call super.getRoutes() in my subclass, which will assemble all the Routes and then in my subclass I'd have to re-iterate over all the Routes and remove the ones that I want to filter.  That's why I say it is inefficient.
In my PR I'm introducing a callback method, assembleRoutes(), that allows subclasses the opportunity to build the route list directly from the cache, without having access to the cache, and iterating over the routes only once.
Did I clarify, or were you thinking of something entirely different?

There's two other protected methods that access this.routes.get(). Maybe we make a protected getRoutesMap() so we could satisfy overriding the other methods as well, rather than a special purpose assembleRoutes() method.


Update documentation for HolyokeUrbanRenewal

Update the documentation for the HolyokeUrbanRenewal dataset once wrangling is finished and we have decided which variables to keep in the dataset.

Done via https://github.com/STAT325-S24/HolyokeUrbanRenewal/commit/321d96575c64af89a0441c552169a35ed46115fe


TestTerminalStringWidth

With:
go1.10 linux/amd64
Hello,
I try to build and test your project and it fails with :

Testing: "/builddir/build/BUILD/markdownfmt-10aae0a270abfb5d929ae6ca59c4b0ac0fa8f237/_build/src/github.com/shurcooL/markdownfmt/markdown"

GOPATH=/builddir/build/BUILD/markdownfmt-10aae0a270abfb5d929ae6ca59c4b0ac0fa8f237/_build:/usr/share/gocode
go test -buildmode pie -compiler gc -ldflags '-extldflags '''-Wl,-z,relro  ''''
--- FAIL: TestTerminalStringWidth (0.00s)
stringwidth_test.go:35: got 10, want 8
FAIL
exit status 1
FAIL    github.com/shurcooL/markdownfmt/markdown

Could you help me please ?
Thanks

Hello,
I can't reproduce the failure. It works okay for me:
$ go test github.com/shurcooL/markdownfmt/...
?   	github.com/shurcooL/markdownfmt	[no test files]
ok  	github.com/shurcooL/markdownfmt/markdown	0.036s

I suspect it could be failing for you for one of 2 reasons:

Perhaps one of your dependencies (most notably, go-runewidth is out of date. Try updating your dependencies with:
go get -u github.com/shurcooL/markdownfmt/...

Then try again.

Perhaps your environment is different. I know that go-runewidth reads some environment variables to determine its behavior. Check if any of https://github.com/mattn/go-runewidth/search?q=getenv are set.


Fixing build/test/deploy workflows

Removing old heroku deployment and improving build/test/deploy scripts.

@thombergs there's no way to change that?  :(


[RFC] test: Clean test_bitcoin-qt output

All of expected QDEBUG : ... and QINFO  : ... messages could be silenced with QTest::ignoreMessage.
It will improve test_bitcoin-qt output quality.
But some cases require QRegularExpression which is unavailable since https://github.com/bitcoin/bitcoin/pull/16386.
Is it worth to drop -no-feature-regularexpression for such a use case?

cc @fanquake @luke-jr @practicalswift


SNAPSHOT of telegram-api RuntimeException Unknown dispatcherId 'connection'

I managed to figure out how to build the telegram-api  shadowJar snapshot (see here for how to fix the gradle build files... https://github.com/ex3ndr/telegram-api/issues/26)
But if I replace 1.1.127 shadow jar with the snapshot build I see these errors...
Exception in thread "Connection#757386994" java.lang.RuntimeException: Unknown dispatcherId 'connection'
	at com.droidkit.actors.ActorSystem.actorOf(ActorSystem.java:102)
	at com.droidkit.actors.ActorSystem.actorOf(ActorSystem.java:63)
	at org.telegram.mtproto.transport.TransportTcpPool.<init>(TransportTcpPool.java:62)
	at org.telegram.mtproto.MTProto.<init>(MTProto.java:116)
	at org.telegram.api.engine.TelegramApi$ConnectionThread.waitForDc(TelegramApi.java:895)
	at org.telegram.api.engine.TelegramApi$ConnectionThread.run(TelegramApi.java:946)

And none of my (previously working) code will work.   What is the status of this project?  Please can you let us know if it is expected to work (for example with the telegram-bot example) and replaces the 1.1.127 release or is it unstable and expected to be broken like this?
Thanks - I'd really like to have a stable Java API to send messages into Telegram.
One final thing - if I call TelegramAPI.close()  with the 1.1.127 release it fails with NPE (so I can never cleanly shutdown) however if it do it with the SNAPSHOT it seems to work (but catch-22 I can't send message ;-)

I think the status of this project is almost dead. The developer has left and we have his code working with Telegram API version #12.
Anyway I would use it if I had the certainty it's not going to stop working in the close future and with no way of upgrading in order to get work it again.
Conclusion: I don't really know if it's worthful to begin a project based in this java libraries.


Less spacing in lists of new theme

Is your feature request related to a problem? Please describe.
Portainer's new theme does a massive update on how it uses screen space. In some areas like the dashboard this is a HUGE improvement. The list view became slightly less usable however.
As a list view, it's fundamental nature is that of a spreadsheet. I understand we want to make spreadsheets beautiful, but they still need to provide functionality- this is one of the main views of portainer and one users spend a lot in it- it needs to communicate a lot of data in that view.
Describe the solution you'd like
Adjust horizontal spacing of elements down slightly. The vertical spacing should be adjusted more significantly.
Describe alternatives you've considered
I adjusted my screen zoom but that made all the icons and text size smaller & more difficult to read. I considered a css adjustment browser extension but I frequently access portainer from a variety of devices and feel this change would benefit the community as a whole.
Additional context
❤ Portainer.

Thanks for the feedback! This is something we're aware of already, and we're looking at refining in an upcoming release.


Are there any named colors?

What are the possible color formats I can specify? Is the only way to specify a color is to use notation like [255, 56, 254]?
I'd like to use a Web color name, eg
setComparmentFillColor ('vol', 'ForestGreen')
or maybe
setComparmentFillColor ('vol', colors.ForestGreen)
Color depends methods should also probably access hex, such as #ff3456

I tried to specify the name and it seems not to complain, can I use code like this:
la.setCompartmentFillColor('default_compartment', 'coral')

yes, see below also see the updated documentation.
There are three ways to set colors and opacity is optional:
1) list-decimal_rgb 1*3 matrix, i.e. [255, 255, 255];
2) str-html_name, i.e. "white";
3) str-hex_string (6-digit), i.e. "#000000";


Documentation:  Mention templates in NEWS.md

Please select a category the issue is focused on?
No response
Let us know where something needs a refresh or put your idea here!
The fact that we added templates as well as vignettes should be specifically mentioned in a new section of NEWS.md

Duplicate of #61


Update versions.gradle

related: https://github.com/AzureAD/android-complete/pull/278

Marking as breaking change because version.gradle of ADAL/MSAL/Broker repo isn't yet updated, so the build will always fail.


angular scope variable not bound inside bootstrap/jquery modal

I'm working with bootstrap (v3.2.0), angular (v1.3.13) and jquery (v2.1.3) and this issue is appearing when I'm trying to upgrade the bootstrap version to 3.3.2. We've a bootstrap modal which is toggled with angular route. The modal has $scope bound variables.
In v3.2.0, the $scope bound variables are appearing fine in the modal, but in v3.3.2 angular compiler is throwing an error TypeError: Cannot read property 'childNodes' of undefined
I've created two examples also:
With bootstrap v3.2.0 ($scope vars working) : http://plnkr.co/edit/cGf3C2gHR0GOK5JBido1?p=preview
With bootstrap v3.3.2 ($scope vars not working) : http://plnkr.co/edit/kj66Gal5s5Yczyom1PHC?p=preview
Is there something wrong with angular/jquery/bootstrap combination or I'm doing something wrong here?

I assume there's a race condition between the angular compiler and the bootstrap modal function. If you wrap your call to $('#myModal') into a setTimeout, it works. You'll have to check the bootstrap changelogs etc. to find out why this happens after you upgrade. I don't think this is an issue with angular.
Another tip: it's not a good idea to do DOM manipulation inside controllers - this is what directives are for. See this example, which also doesn't cause the error: http://plnkr.co/edit/A9bmqR5cMa4tVguM6hLB?p=preview

Thank you. We're in the process of converting bootstrap/jquery related code to angular-ui-bootstrap so that we can use directives as you recommended.


chore: release v1.407.0

feat: time estimate UI for trade quotes (#5118)
chore: remove market data from trade execution (#5117)
chore: add error handling to thorchain swapper route generation (#5115)
chore: remove dead code from thorchain swapper (#5114)
chore: remove redundant lifi tools map (#5113)
feat: thor streaming swaps quote (#5106)
feat: upgrade swappers to support returning multiple quotes (#5095)
feat: make oneinch great again (#5105)
chore: update app translations (#5104)
feat: use market data from bridged assets if base asset data is unavailable (#5103)
feat: fetch market data at quote time (#5102)
chore: remove manual memo calcs (#5094)
fix: handle positive price impact (#5087)
feat: slippage improvements (#5093)
feat: handle minimums using error monads (#5081)

Closing, will re-cut.


[state sync] prune outdated LIs from PendingLedgerInfo

Motivation
PendingLedgerInfo prunes out LIs that we can potentially set to as a target LI in chunk requests based on committed version, not synced version. This causes problems in the following FN long-poll scenario:

A FN sends an upstream node a chunk request with known version X and target version Y (X < Y)
The upstream that receives this is behind version X when it receives the FN's request, so it caches it for long-poll subscription.
Within the long-poll time window, the upstream reaches a version Z > X, and it fulfills the cached long-poll request with a target version of Z and a chunk of transactions X .. W, where W > Y
FN processes that response successfully and the resulting local synced version is W, but because of the way PendingLedgerInfos is pruned, the outdated LI Y is still set as the target LI.
The next chunk request the FN sends is for known_version W and target version still Y. Since W > Y, the response it gets for such a request is empty, so the FN does not make any state sync progress.

This PR fixes this issue by making sure no target LI behind the local synced version is a candidate for a target LI (s.t. target version > known_version).
Will land this PR first and put out a follow-up cherry-pick PR
Test Plan
Added test to simulate above scenario, fails without fix, passes with fix

:exclamation: Invalid command

Bors help and documentation

Bors is a Github App used to manage merging of PRs in order to ensure that CI is always green. It does so by maintaining a Merge Queue. Once a PR reaches the head of the Merge Queue it is rebased on top of the latest version of the PR's base-branch (generally master) and then triggers CI. If CI comes back green the PR is then merged into the base-branch. Regardless of the outcome, the next PR is the queue is then processed.
General

This project's Merge Queue can be found here.
This project requires PRs to be Reviewed and Approved before they can be queued for merging.
Before PR's can be merged they must be configured with "Allow edits from maintainers" enabled. This is needed for Bors to be able to update PR's in-place so that Github can properly recognize and mark them as "merged" once they've been merged into the upstream branch.

Commands
Bors actions can be triggered by posting a comment which includes a line of the form /<action>.

Command
Action
Description

Land
land, merge
attempt to land or merge a PR

Canary
canary, try
canary a PR by performing all checks without merging

Cancel
cancel, stop
stop an in-progress land

Cherry Pick
cherry-pick <target>
cherry-pick a PR into <target> branch

Priority
priority
set the priority level for a PR (high, normal, low)

Help
help, h
show this help message

Options
Options for Pull Requests are configured through the application of labels.

          Option
Description

Indicates that the PR is high-priority. When queued the PR will be placed at the head of the merge queue.

Indicates that the PR is low-priority. When queued the PR will be placed at the back of the merge queue.

Before merging the PR will be squashed down to a single commit, only retaining the commit message of the first commit in the PR.

is it able to commit the LI with empty chunks?

LI only gets committed if all txns leading up to it are synced, so would expect no

or you're saying if the txns are already there and we try to commit the LI afterwards separately? not sure

yeah, the empty chunk sent back by a honest validator means the txns are already there and should be able to commit the LI with empty chunks, not sure if the API supports it

:exclamation: Invalid command

Bors help and documentation

Bors is a Github App used to manage merging of PRs in order to ensure that CI is always green. It does so by maintaining a Merge Queue. Once a PR reaches the head of the Merge Queue it is rebased on top of the latest version of the PR's base-branch (generally master) and then triggers CI. If CI comes back green the PR is then merged into the base-branch. Regardless of the outcome, the next PR is the queue is then processed.
General

This project's Merge Queue can be found here.
This project requires PRs to be Reviewed and Approved before they can be queued for merging.
Before PR's can be merged they must be configured with "Allow edits from maintainers" enabled. This is needed for Bors to be able to update PR's in-place so that Github can properly recognize and mark them as "merged" once they've been merged into the upstream branch.

Commands
Bors actions can be triggered by posting a comment which includes a line of the form /<action>.

Command
Action
Description

Land
land, merge
attempt to land or merge a PR

Canary
canary, try
canary a PR by performing all checks without merging

Cancel
cancel, stop
stop an in-progress land

Cherry Pick
cherry-pick <target>
cherry-pick a PR into <target> branch

Priority
priority
set the priority level for a PR (high, normal, low)

Help
help, h
show this help message

Options
Options for Pull Requests are configured through the application of labels.

          Option
Description

Indicates that the PR is high-priority. When queued the PR will be placed at the head of the merge queue.

Indicates that the PR is low-priority. When queued the PR will be placed at the back of the merge queue.

Before merging the PR will be squashed down to a single commit, only retaining the commit message of the first commit in the PR.

yeah, the empty chunk sent back by a honest validator means the txns are already there and should be able to commit the LI with empty chunks, not sure if the API supports it

It might be worth trying because I don't see anything in the executor that would prevent this, but there are a lot of proofs that would have to verify correctly even on an empty proof.
Though I am okay even if we just move forward. I do like the attempt to optimize.

I'd prefer we tackle that issue in a separate future PR - for now keep the scope of this PR as is

/land

:exclamation: Invalid command

Bors help and documentation

Bors is a Github App used to manage merging of PRs in order to ensure that CI is always green. It does so by maintaining a Merge Queue. Once a PR reaches the head of the Merge Queue it is rebased on top of the latest version of the PR's base-branch (generally master) and then triggers CI. If CI comes back green the PR is then merged into the base-branch. Regardless of the outcome, the next PR is the queue is then processed.
General

This project's Merge Queue can be found here.
This project requires PRs to be Reviewed and Approved before they can be queued for merging.
Before PR's can be merged they must be configured with "Allow edits from maintainers" enabled. This is needed for Bors to be able to update PR's in-place so that Github can properly recognize and mark them as "merged" once they've been merged into the upstream branch.

Commands
Bors actions can be triggered by posting a comment which includes a line of the form /<action>.

Command
Action
Description

Land
land, merge
attempt to land or merge a PR

Canary
canary, try
canary a PR by performing all checks without merging

Cancel
cancel, stop
stop an in-progress land

Cherry Pick
cherry-pick <target>
cherry-pick a PR into <target> branch

Priority
priority
set the priority level for a PR (high, normal, low)

Help
help, h
show this help message

Options
Options for Pull Requests are configured through the application of labels.

          Option
Description

Indicates that the PR is high-priority. When queued the PR will be placed at the head of the merge queue.

Indicates that the PR is low-priority. When queued the PR will be placed at the back of the merge queue.

Before merging the PR will be squashed down to a single commit, only retaining the commit message of the first commit in the PR.

:exclamation: Invalid command

Bors help and documentation

Bors is a Github App used to manage merging of PRs in order to ensure that CI is always green. It does so by maintaining a Merge Queue. Once a PR reaches the head of the Merge Queue it is rebased on top of the latest version of the PR's base-branch (generally master) and then triggers CI. If CI comes back green the PR is then merged into the base-branch. Regardless of the outcome, the next PR is the queue is then processed.
General

This project's Merge Queue can be found here.
This project requires PRs to be Reviewed and Approved before they can be queued for merging.
Before PR's can be merged they must be configured with "Allow edits from maintainers" enabled. This is needed for Bors to be able to update PR's in-place so that Github can properly recognize and mark them as "merged" once they've been merged into the upstream branch.

Commands
Bors actions can be triggered by posting a comment which includes a line of the form /<action>.

Command
Action
Description

Land
land, merge
attempt to land or merge a PR

Canary
canary, try
canary a PR by performing all checks without merging

Cancel
cancel, stop
stop an in-progress land

Cherry Pick
cherry-pick <target>
cherry-pick a PR into <target> branch

Priority
priority
set the priority level for a PR (high, normal, low)

Help
help, h
show this help message

Options
Options for Pull Requests are configured through the application of labels.

          Option
Description

Indicates that the PR is high-priority. When queued the PR will be placed at the head of the merge queue.

Indicates that the PR is low-priority. When queued the PR will be placed at the back of the merge queue.

Before merging the PR will be squashed down to a single commit, only retaining the commit message of the first commit in the PR.


GraphModel Samples

Anybody have samples for the GraphModel?  Any docs on how to use the related APIs will be appreciated.
I can't get passed the following as it has error stating no arguments for GraphNode constructor.
Graph graph = new Graph();
graph.Nodes.Add(new GraphNode());

Look at my CodeTours repo


Basic code example doesn't work

Hello,
I am kind of surprised that even your main example on your git readme doesn't work. Even though you haven't work in years for this library, I guess it worked at some moment, but someone probably crashed it some time ago and pushed it. It would be nice, if you change it to "deprecated and not working" instead of keeping it up without any comments...
Thanks

@Shryne Can you be more specify which part does not work?


Implement Bifurcated Resolve Handling

Avoid the bug spelled out in https://github.com/python-poetry/poetry/issues/4381.

Depends on #1402 which is where the bug will appear.

It's not clear that this needs to be addressed since its only an issue when attempting to export a lock to requirements.txt format for direct ingestion by Pip. Pex will be handling resolving from lock files natively and avoiding the Pip resolver altogether. Pip will only be called in to turn sdists, VCS projects and local projects into wheels and install wheels in chroots if needed. As such, I'm going to close this bug as obsoleted by not needing to export any longer, and #1645 tracks removing that subcommand.


DCOS tests for Marathon are failing

JIRA Key: MARATHON-697
Reporter: Sunil Shah
Priority: Medium
Description:
The tests that are posted to the #dev-agility channel are part of the DCOS tests repo and test installation of the Marathon package onto master branch of DCOS image.
TeamCity output here: https://teamcity.mesosphere.io/viewLog.html?buildId=103274
Currently they're failing - perhaps due to the changes in the Marathon package in the Universe.
We should fix these!

@gkleiman can you look into those tests. Thanks.


Sweep Doc modifications for defaults and descriptions

Modified the description for max_total_trials, max_concurrent_trials, and default value of timeout to 60days which was previously set as 7days : https://learn.microsoft.com/en-us/azure/machine-learning/reference-yaml-job-sweep
Modified the note : https://learn.microsoft.com/en-us/azure/machine-learning/how-to-tune-hyperparameters.

@amibp

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 526 ms

Resolved: ClamAV performance has improved in 57beb63.


Fix deceleration value bug in following state

Status
DEVELOPMENT
Description
inv_time will be negatived when forward object too close, but deceleration_critical will be smaller than max_deceleration, that makes vehiche hard to stop .

@MoeLang Is this PR still valid?

@MoeLang Is this PR still valid?

Yes, we used fixed code for months and it works well


How to update / set geofirestore and non-geofirestore db entries in a single GeoTransaction

Hey - great library, really useful. I have a use case with transactions which I'm not sure is supported. I am using a GeoTransaction to create a new geofirestore entry in my database. But within the same transaction, I need to update other parts of my db, which are non-geofirestore entries. Here's a minimal example:
import * as geofunctions from 'geofirestore'
import * as admin from 'firebase-admin'

export const db = admin.firestore()
export const geoDb = new geofunctions.GeoFirestore(db)

export async function geoTransactionExample() {

    const geoPoint = new admin.firestore.GeoPoint(5, 5)

    const geoDocRef = db.collection('geoDataCollection').doc()

    const geoDataEntry = {
        coordinates: geoPoint,
        extra: "extra",
        data: "data"
    }

    const normalDocRef = db.collection('normalDataCollection').doc()

    const normalEntry = {
        normal: "normal",
        data: "data"
    }

    await geoDb.runTransaction(async (t) => {

        const geotransaction = new geofunctions.GeoTransaction(t)

        // This succeeds
        geotransaction.set(geoDocRef, geoDataEntry)

        // This fails
        geotransaction.set(normalDocRef, normalEntry)

    })

}

This fails on the second non-geo set operation with Error: Invalid GeoFirestore document: could not find GeoPoint - which makes sense. But is there e.g. a flag I can pass to the set operation to allow a normal non-geo set? Or is there a different approach to achieve this within a single transaction? Or is this not currently supported?

Something not currently exposed by GeoTransactions in 3.3.1 (which you will see in the next release of GeoFirestore) is the native transaction. To access the native transaction object to run a set on you could do something like this:
    await geoDb.runTransaction(async (t) => {

        const geotransaction = new geofunctions.GeoTransaction(t)

        // This succeeds
        geotransaction.set(geoDocRef, geoDataEntry)

        // This should succeed
        geotransaction['_transaction'].set(normalDocRef, normalEntry)

    })

Note that in the next version you'll be able to access the native transaction object not like this: geotransaction['_transaction'] but like this geotransaction.native.

@MichaelSolati - tested, works. Fantastic, thank you.


chore: bdd-nh-1583651379 to 0.0.1

chore: Promote bdd-nh-1583651379 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Question] After updating to 0.5.1 multi-gpu support have not been working

Hi! Thank you for the great lib!
After updating from 0.2.4 to 0.5.1 only one gpu out of two is used.
The code almost the same:
hyper_parameters = BLSTMModel.get_default_hyper_parameters()
hyper_parameters['layer_bi_lstm']['units'] = 1024

model = BLSTMModel(embedding, hyper_parameters=hyper_parameters)
model.build_model(X_train, y_train)
model.build_multi_gpu_model(gpus=2, x_train=X_train, y_train=y_train, x_validate=X_valid, y_validate=y_valid)
model.fit(X_train, y_train, epochs=15, batch_size=512, x_validate=X_valid, y_validate=y_valid, callbacks=[tf_board_callback, checkpoint_callback])

NVtop and nvidia-smi showing that only one gpu is working.
In the previous version I have both of my gpus been used.

@HaoyuHu Maybe you could help me out, I don't have multi-gpu environment.

I am also facing the same issue. Would be great if someone could help.

Could you please provide a runnable code to reproduce this problem, thx!

@jeshuren So if fixed by not calling the build_model  function?

@jeshuren So if fixed by not calling the build_model function?

Yes @BrikerMan

@sld Maybe you can try it without calling build_model function.

Thanks! This https://github.com/BrikerMan/Kashgari/issues/170#issuecomment-513875786 worked!