Question: How to get currency location of another device

@malcommac
Question, How could I use SwiftLocation for this scenario (I will user Uber as an example)
Driver is delegated to capture a charge (Charge Rider's credit card) at the end of a trip.
The action can only happen if Rider is within Room distance (Or an arbitrary distance i set).
Else ERROR: Rider is not within distance
How can use swift location to so Driver can get Rider's current location when Rider's device is either in Foreground or Background

This has nothing to do with SwiftLocation. You need a server for that. Check Firebase for a mobileBaaS (or any other mBaaS).


feature(apps/dev/prow/prow/release): bump release to 0.9.4

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


blank page on certain browser plugins injecting stylesheets

Describe the bug
Loading DSpace with certain plugins leads to problems and the site will not load correctly resulting in some white page after the initial load.
Screenshot of some  white demo page - not really valuable for debugging, i must admit :)

As far as we know some dark mode plugin
as well as some of our users reported this with the Ghostery Plugin.
We could reproduce the problem locally on dev mode/localhost as well as on the demo page with the dark mode plugin on Firefox 116.0.3. On my settings we could not reproduce the problem with the ghostery plugin
To Reproduce
Steps to reproduce the behavior:

Install the dark mode plugin or any similiar plugin (you must not  if it concerns your security settings or you are unsure if this impacts your system.)
Call the page https://demo7.dspace.org/
Wait for the Loading - at the end some white page should occur
The Browser Console logs the following errors (won't paste the whole stack trace, but the relevant parts):

ERROR DOMException: CSSStyleSheet.cssRules getter: Not allowed to access cross-origin stylesheet
    h https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1
    getCSSVariablesFromStylesheets https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1
    storeCSSVariables https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1

Unhandled Promise rejection: CSSStyleSheet.cssRules getter: Not allowed to access cross-origin stylesheet ; Zone: <root> ; Task: Promise.then ; Value: DOMException: CSSStyleSheet.cssRules getter: Not allowed to access cross-origin stylesheet
    h https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1
    getCSSVariablesFromStylesheets https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1
    storeCSSVariables https://demo7.dspace.org/main.135b9dd6bfbbbfb9.js:1

Expected behavior
We except the page to be loaded correctly.
Investigation
Our investigation come to the conclusion, that the stylesheets injected by external plugins (into the dom) can lead to problems when the stylesheet contains no cssRules and thus the stylesheetobject has no css rules.
The line where the error occurs is this
https://github.com/DSpace/dspace-angular/blob/77e994dcd63ce38b90d7172bfd07d14f2e795d71/src/app/shared/sass-helper/css-variable.service.ts#L102
Possible Solution
One possible solution could be to add some additional filter in  https://github.com/DSpace/dspace-angular/blob/77e994dcd63ce38b90d7172bfd07d14f2e795d71/src/app/shared/sass-helper/css-variable.service.ts#L97
which checks whether the stylesheet contains some cssRule property before processing the variables
Related work
This was introduced in 7.5 with commit https://github.com/DSpace/dspace-angular/commit/89ba24fc1c1df13cfecdf12ebe59883901980d28 (https://github.com/DSpace/dspace-angular/pull/1853) and should occur on 7.5 and 7.6 instances.

@floriangantner : Thanks for the detailed analysis.  I agree this would be nice to fix if possible, to avoid odd behaviors if a user happens to have these browser plugins installed.
If you wanted to create a PR with your "Possible Solution" we could have others try it out and see if it works for them as well.
Moving this to our 7.6.x maintenance board and flagging as needs a volunteer in the meantime.  PRs welcome though.

I've been seeing this in my Firefox as well, even with plugins disabled.
It only seems to happen with some themes; like the dspace theme on https://demo.dspace.org/

Reopening, as #2454 had to be reverted since it caused the bugs noted in #2557 .
Unfortunately, this would need more work to resolve.  Perhaps an updated version of #2454 will provide better behavior overall.
For now, we may have to continue to ask users to disable problematic Browser Plugins (like Ghostery, etc), which is alkso sometimes the recommendation on StackOverflow: https://stackoverflow.com/a/62475347/3750035

NOTE: another possible option to solving this problem might be to see if there's a way to no longer use cross-origin stylesheets?  I'm surprised we have any, as I thought all our stylesheets should be served from DSpace itself?

Further thoughts today.  This problem might be related to this external Google font which is loaded into only the dspace theme: https://github.com/DSpace/dspace-angular/blob/main/src/themes/dspace/styles/_theme_sass_variable_overrides.scss#L6
That's the only place where I've found a possible cross-origin stylesheet.  Maybe it could be fixed if we switched the dspace theme to using a font that doesn't require import?  (Or alternatively distribute this Nunito font in the dspace theme directly instead of having to import it via URL... as all Google fonts are allowed to released in other products.)

Sorry about the circumstances and the late reply, i had time to further test it and reproduce it locally (also in dev mode) which leads to an more detailed error log (also see https://github.com/DSpace/dspace-angular/issues/2557 )
 NG03505: @slideSidebar has failed due to:
 NG03003: Please provide a value for the animation param sidebarWidth

The error after logging in (and hovering the admin sidebar https://github.com/DSpace/dspace-angular/blob/81e3b36d7d5bd909c9bd4bde607dd946ef6b3f6c/src/app/admin/admin-sidebar/admin-sidebar.component.ts#L72 ) should occur because some variable --ds-sidebar-items-width is missing. The variables from _exposed_variables.scss https://github.com/DSpace/dspace-angular/blob/main/src/styles/_exposed_variables.scss seems to be unset after the stylesheets are filtered, so it should come from our own code.
Further investigating which stylesheets are being filtered by my proposed solution there were also some where (at least at the point of the request in the browser console) some cssRules were mentioned, but these were filtered.

It seems they have no own property "cssRules" but the cssRules are accessible/processed anyway. so some exception to catch the Unhandled Promise rejection: CSSStyleSheet.cssRules getter: Not allowed to access cross-origin stylesheet ; Zone: <root> ; Task: Promise.then ; Value: DOMException: CSSStyleSheet.cssRules getter: Not allowed to access cross-origin stylesheet error for external cross-origin stylesheets could solve the problem, although this looks a little bit odd for me.
  hasCssRules = (styleSheet) => {
    // Injected (cross-origin) styles might have no css rules value and throw some exception
    try {
      return styleSheet.cssRules;
    } catch (e) {
      return false;
    }
  };

Fixed (finally) in #2580


id property of typeAnnotation of NullableTypeAnnotation is not always defined

STDERR
     ../eslint-plugin-relay/eslint-plugin-relay.js:330
          propTypeProperty.value.typeAnnotation.id.name === type

TypeError: Cannot read property 'name' of undefined
at validateObjectTypeAnnotation (../eslint-plugin-relay/eslint-plugin-relay.js:330:45)
at Program:exit.expectedTypes.forEach.type (../eslint-plugin-relay/eslint-plugin-relay.js:741:23)
at Array.forEach (native)
at EventEmitter.Program:exit (../eslint-plugin-relay/eslint-plugin-relay.js:687:25)
at emitOne (events.js:120:20)
at EventEmitter.emit (events.js:210:7)
at NodeEventGenerator.leaveNode (../eslint/lib/util/node-event-generator.js:49:22)

Concerning this bit of code here:
if (
    propTypeProperty.value.type === 'NullableTypeAnnotation' &&
    propTypeProperty.value.typeAnnotation.id.name === type
  ) {
    return true;
  }

It probably needs a null-safe drilldown into this member expression chain to get name.


Install error

Hello, sir I just want to install nanopolish to use with my dataset. However, its go an error in install process
here is a error message from terminal
jirachote@ubuntu:~/nanopolish$ make if [ ! -e hdf5-1.8.14.tar.gz ]; then \ 	version_major_minor=echo "1.8.14" | sed -E 's/.[0-9]+$//'`;
wget https://support.hdfgroup.org/ftp/HDF5/releases/hdf5-${version_major_minor}/hdf5-1.8.14/src/hdf5-1.8.14.tar.gz;
fi
--2021-02-01 00:51:39--  https://support.hdfgroup.org/ftp/HDF5/releases/hdf5-1.8/hdf5-1.8.14/src/hdf5-1.8.14.tar.gz
Resolving support.hdfgroup.org (support.hdfgroup.org)... <IP_ADDRESS>
Connecting to support.hdfgroup.org (support.hdfgroup.org)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 11761702 (11M) [application/x-gzip]
Saving to: ‘hdf5-1.8.14.tar.gz’
hdf5-1.8.14.tar.gz                 100%[================================================================>]  11.22M   216KB/s    in 52s
2021-02-01 00:52:32 (222 KB/s) - ‘hdf5-1.8.14.tar.gz’ saved [11761702/11761702]
tar -xzf hdf5-1.8.14.tar.gz || exit 255
cd hdf5-1.8.14 &&
./configure --enable-threadsafe --disable-hl --libdir=pwd/../lib --includedir=pwd/../include --prefix=pwd/.. || exit 255
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
checking for gawk... no
checking for mawk... mawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether make supports nested variables... (cached) yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking shell variables initial values... done
checking if basename works... yes
checking if xargs works... yes
checking for cached host... none
checking for config x86_64-unknown-linux-gnu... no
checking for config x86_64-unknown-linux-gnu... no
checking for config unknown-linux-gnu... no
checking for config unknown-linux-gnu... no
checking for config x86_64-linux-gnu... no
checking for config x86_64-linux-gnu... no
checking for config x86_64-unknown... no
checking for config linux-gnu... found
checking for config ./config/site-specific/host-ubuntu... no
checking for gcc... gcc
checking whether the C compiler works... no
configure: error: in /home/jirachote/nanopolish/hdf5-1.8.14': configure: error: C compiler cannot create executables See config.log' for more details
make: *** [Makefile:115: lib/libhdf5.a] Error 255
`
How to fix this. Thanks in advance.
Best
Jirachote

I'm afraid this looks like an issue with your development environment and I'll be unable to help with this. I suggest installing nanopolish through conda instead.


[Gremlin support] Guid for Vertex id is not working

My query was this, but I got error.
g.addV('node').property('id',3b28ce3e-7863-475e-a674-4661d648a374)
Microsoft.Azure.Graphs.GraphException: 'Failed to compile query:
(1,97): error CS1525: Unexpected symbol `b28ce3e'
(1,115): error CS0589: Internal compiler error during parsingSystem.FormatException: Input string was not in a correct format.
I assume it should just work.

It works:
g.addV('node').property('id','3b28ce3e-7863-475e-a674-4661d648a374')
Same here #265


Fix user guide pages 4-8

Have also added more general error message for starting with symbols

Resolves #322, Resolves #315, Resolves #308, Resolves #307, Resolves #306, Resolves #304, Resolves #299, Resolves #286


Won't complete - browser zoom set to 100% on Windows - Chrome

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

What was the error? I dont seem to be able to progress?
:-)

don't put . in front of p. when you put . then it is intrepreted as a class and not the element.


Add DB_MAX_SAFE_INTEGER constant

Summary
Context: https://github.com/safe-global/safe-client-gateway/pull/1773#discussion_r1684538079
Changes

Adds a DB_MAX_SAFE_INTEGER constant, with value (2^31)-1 (max Postgres value for an integer column).

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

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
4332

Relevant Lines:
7207

💛  - Coveralls


Need same diagnostic logging for both optional and required config data import.

Type: Feature
Component:

Secrets Manager
Parameter Store

Is your feature request related to a problem? Please describe.
When using the config import feature for both AWS Secrets Manager and Parameter Store, if the secret or parameter is required (not prefixed with "optional:") and  cannot be found, diagnostic messaging is missing. When using "optional:" import, the diagnostic messaging is logged.
First example config:
spring:
  config:
    import:
      - "aws-secretsmanager:demo-dev"
      - "aws-parameterstore:/demo-dev/"

When using the config import feature for both AWS Secrets Manager and Parameter Store, if the secret or parameter is required (not prefixed with "optional:") and  cannot be found, only the following message is printed:
22:25:38.162 [restartedMain] ERROR org.springframework.boot.diagnostics.LoggingFailureAnalysisReporter --

***************************
APPLICATION FAILED TO START
***************************

Description:

Config data resource '[SecretsManagerConfigDataResource@1678b7be context = 'demo-dev', optional = false]' via location 'aws-secretsmanager:demo-dev' does not exist

Action:

Check that the value 'aws-secretsmanager:demo-dev' at class path resource [application.yaml] - 7:9 is correct, or prefix it with 'optional:'

The problem is that the reason for the secret not being found is not included in the output.
Second example config (optional import):
spring:
  config:
    import:
      - "optional:aws-secretsmanager:demo-dev"
      - "optional:aws-parameterstore:/demo-dev/"

However, if I prefix the config import with "optional:", I get a WARN log message that includes the precise reason why it could not be found (e.g., wrong region, wrong IAM policy, or wrong name). For example:
2023-08-09T22:33:42.750Z  INFO 8534 --- [  restartedMain] .a.c.a.c.s.SecretsManagerPropertySources : Loading secrets from AWS Secret Manager secret with name: demo-dev, optional: true
2023-08-09T22:33:42.750Z  WARN 8534 --- [  restartedMain] .a.c.a.c.s.SecretsManagerPropertySources : Unable to load AWS secret from demo-dev. User<EMAIL_ADDRESS>is not authorized to perform: secretsmanager:GetSecretValue on resource: demo-dev with an explicit deny in a service control policy (Service: SecretsManager, Status Code: 400, Request ID: d56e7b7f-c589-4ae8-8550-e9ff8a9fa8eb)

Describe the solution you'd like
It would be nice in the fail-fast scenario when requiring the import to exist if the failure output would include the same diagnostic information or if this information could at least be logged before the failure output.
Describe alternatives you've considered
Currently, I'm instructing my engineers to put "optional:" on their required config imports and run the application again to get the diagnostic logging. This is an acceptable stop-gap solution, but it's inefficient time-wise to wait for an automated build & deploy process to complete.
Additional context
I'm willing to submit a PR if that will help get this done more quickly.

@thorntonrp thanks for reporting. I'll add same log statement, just with INFO level when config is set as optional.


Refactor documentation

Merge once https://github.com/pydantic/pydantic/pull/7417 is merged.

Ups. wrong thing.


sam build can't resolve dependencies - tensorflow

Description
Error when building projects containing tensorflow on a Mac:
Error: PythonPipBuilder:ResolveDependencies - {tensorflow==2.2.0(wheel), grpcio==1.29.0(wheel)}
This could be the same as Issue #1840 but I'm not using PyCharm - just the command line.
Steps to reproduce
(Assumes python 3.6 is already installed)

sam init (Project sam-bld-err, choose python 3.6 runtime and Hello World template)
Create a python virtual environment within the project
cd sam-bld-err
virtualenv -p /Library/Frameworks/Python.framework/Versions/3.6/bin/python3.6  venv_be
(Adjust path to python 3.6 as needed)
Activate virtual env and install tensorflow
source ./venv_be/bin/activate
pip install tensorflow==2.2.0
Update project requirements
pip freeze > ./hello_world/requirements.txt
Build the project
sam build or sam build --use-container   (Same results for each)

Observed result
(Sorry - Lots of debug output!)
sam build --debug
Telemetry endpoint configured to be https://aws-serverless-tools-telemetry.us-west-2.amazonaws.com/metrics
'build' command is called
No Parameters detected in the template
2 resources found in the template
Found Serverless function with name='HelloWorldFunction' and CodeUri='hello_world/'
No Parameters detected in the template
Building function 'HelloWorldFunction'
Loading workflow module 'aws_lambda_builders.workflows'
Registering workflow 'PythonPipBuilder' with capability 'Capability(language='python', dependency_manager='pip', application_framework=None)'
Registering workflow 'NodejsNpmBuilder' with capability 'Capability(language='nodejs', dependency_manager='npm', application_framework=None)'
Registering workflow 'RubyBundlerBuilder' with capability 'Capability(language='ruby', dependency_manager='bundler', application_framework=None)'
Registering workflow 'GoDepBuilder' with capability 'Capability(language='go', dependency_manager='dep', application_framework=None)'
Registering workflow 'GoModulesBuilder' with capability 'Capability(language='go', dependency_manager='modules', application_framework=None)'
Registering workflow 'JavaGradleWorkflow' with capability 'Capability(language='java', dependency_manager='gradle', application_framework=None)'
Registering workflow 'JavaMavenWorkflow' with capability 'Capability(language='java', dependency_manager='maven', application_framework=None)'
Registering workflow 'DotnetCliPackageBuilder' with capability 'Capability(language='dotnet', dependency_manager='cli-package', application_framework=None)'
Registering workflow 'CustomMakeBuilder' with capability 'Capability(language='provided', dependency_manager=None, application_framework=None)'
Found workflow 'PythonPipBuilder' to support capabilities 'Capability(language='python', dependency_manager='pip', application_framework=None)'
Running workflow 'PythonPipBuilder'
Running PythonPipBuilder:ResolveDependencies
calling pip download -r /Users/philsweeney/Documents/AWSProjects/sam-bld-error/hello_world/requirements.txt --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z
Full dependency closure: {tensorflow==2.2.0(wheel), markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), scipy==1.4.1(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), h5py==2.10.0(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), protobuf==3.12.2(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), termcolor==1.1.0(sdist), gast==0.3.3(wheel), wrapt==1.12.1(sdist), cachetools==4.1.0(wheel), grpcio==1.29.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), absl-py==0.9.0(sdist), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), numpy==1.18.4(wheel), werkzeug==1.0.1(wheel)}
initial compatible: {markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), gast==0.3.3(wheel), cachetools==4.1.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), werkzeug==1.0.1(wheel)}
initial incompatible: {protobuf==3.12.2(wheel), tensorflow==2.2.0(wheel), termcolor==1.1.0(sdist), wrapt==1.12.1(sdist), numpy==1.18.4(wheel), grpcio==1.29.0(wheel), absl-py==0.9.0(sdist), scipy==1.4.1(wheel), h5py==2.10.0(wheel)}
Downloading missing wheels: {tensorflow==2.2.0(wheel), termcolor==1.1.0(sdist), absl-py==0.9.0(sdist), h5py==2.10.0(wheel), protobuf==3.12.2(wheel), wrapt==1.12.1(sdist), numpy==1.18.4(wheel), grpcio==1.29.0(wheel), scipy==1.4.1(wheel)}
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z tensorflow==2.2.0
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z termcolor==1.1.0
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z absl-py==0.9.0
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z h5py==2.10.0
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z protobuf==3.12.2
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z wrapt==1.12.1
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z numpy==1.18.4
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z grpcio==1.29.0
calling pip download --only-binary=:all: --no-deps --platform manylinux1_x86_64 --implementation cp --abi cp36m --dest /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z scipy==1.4.1
compatible wheels after second download pass: {markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), scipy==1.4.1(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), h5py==2.10.0(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), protobuf==3.12.2(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), gast==0.3.3(wheel), cachetools==4.1.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), numpy==1.18.4(wheel), werkzeug==1.0.1(wheel)}
Build missing wheels from sdists (C compiling True): {termcolor==1.1.0(sdist), wrapt==1.12.1(sdist), absl-py==0.9.0(sdist)}
calling pip wheel --no-deps --wheel-dir /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z/termcolor-1.1.0.tar.gz
calling pip wheel --no-deps --wheel-dir /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z/wrapt-1.12.1.tar.gz
calling pip wheel --no-deps --wheel-dir /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z/absl-py-0.9.0.tar.gz
compatible after building wheels (no C compiling): {markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), scipy==1.4.1(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), h5py==2.10.0(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), protobuf==3.12.2(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), termcolor==1.1.0(wheel), gast==0.3.3(wheel), cachetools==4.1.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), absl-py==0.9.0(wheel), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), numpy==1.18.4(wheel), werkzeug==1.0.1(wheel)}
Build missing wheels from sdists (C compiling False): {wrapt==1.12.1(sdist)}
calling pip wheel --no-deps --wheel-dir /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z /var/folders/s5/_787jljn6l14fhg2w4wn4d_w0000gn/T/tmplzw2zx5z/wrapt-1.12.1.tar.gz
compatible after building wheels (C compiling): {markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), scipy==1.4.1(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), h5py==2.10.0(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), protobuf==3.12.2(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), termcolor==1.1.0(wheel), gast==0.3.3(wheel), wrapt==1.12.1(wheel), cachetools==4.1.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), absl-py==0.9.0(wheel), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), numpy==1.18.4(wheel), werkzeug==1.0.1(wheel)}
Final compatible: {markdown==3.2.2(wheel), setuptools==47.1.1(wheel), tensorboard==2.2.2(wheel), tensorboard-plugin-wit==1.6.0.post3(wheel), scipy==1.4.1(wheel), chardet==3.0.4(wheel), google-auth-oauthlib==0.4.1(wheel), h5py==2.10.0(wheel), opt-einsum==3.2.1(wheel), certifi==2<IP_ADDRESS>(wheel), oauthlib==3.1.0(wheel), protobuf==3.12.2(wheel), google-auth==1.16.0(wheel), zipp==3.1.0(wheel), six==1.15.0(wheel), tensorflow-estimator==2.2.0(wheel), importlib-metadata==1.6.0(wheel), termcolor==1.1.0(wheel), gast==0.3.3(wheel), wrapt==1.12.1(wheel), cachetools==4.1.0(wheel), pyasn1==0.4.8(wheel), google-pasta==0.2.0(wheel), requests-oauthlib==1.3.0(wheel), urllib3==1.25.9(wheel), pyasn1-modules==0.2.8(wheel), absl-py==0.9.0(wheel), astunparse==1.6.3(wheel), keras-preprocessing==1.1.2(wheel), idna==2.9(wheel), requests==2.23.0(wheel), rsa==4.0(wheel), wheel==0.34.2(wheel), numpy==1.18.4(wheel), werkzeug==1.0.1(wheel)}
Final incompatible: {tensorflow==2.2.0(wheel), h5py==2.10.0(wheel), protobuf==3.12.2(wheel), wrapt==1.12.1(wheel), numpy==1.18.4(wheel), grpcio==1.29.0(wheel), scipy==1.4.1(wheel)}
Final missing wheels: {tensorflow==2.2.0(wheel), grpcio==1.29.0(wheel)}
PythonPipBuilder:ResolveDependencies failed
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/aws_lambda_builders/workflows/python_pip/actions.py", line 42, in execute
requirements_path=self.manifest_path,
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/aws_lambda_builders/workflows/python_pip/packager.py", line 137, in build_dependencies
self._dependency_builder.build_site_packages(requirements_path, artifacts_dir_path, scratch_dir_path)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/aws_lambda_builders/workflows/python_pip/packager.py", line 201, in build_site_packages
raise MissingDependencyError(packages_without_wheels)
aws_lambda_builders.workflows.python_pip.packager.MissingDependencyError: {tensorflow==2.2.0(wheel), grpcio==1.29.0(wheel)}
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/aws_lambda_builders/workflow.py", line 269, in run
action.execute()
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/aws_lambda_builders/workflows/python_pip/actions.py", line 45, in execute
raise ActionFailedError(str(ex))
aws_lambda_builders.actions.ActionFailedError: {tensorflow==2.2.0(wheel), grpcio==1.29.0(wheel)}
Build Failed
Sending Telemetry: {'metrics': [{'commandRun': {'awsProfileProvided': False, 'debugFlagProvided': True, 'region': '', 'commandName': 'sam build', 'duration': 50308, 'exitReason': 'WorkflowFailedError', 'exitCode': 1, 'requestId': '199413b8-0cd6-41f2-8750-b2189bccb2fa', 'installationId': '27166426-9c3e-4c34-93bc-1f3346ed5909', 'sessionId': '08d96ae3-9148-4850-ad9c-87e77e262c0d', 'executionEnvironment': 'CLI', 'pyversion': '3.6.8', 'samcliVersion': '0.52.0'}}]}
HTTPSConnectionPool(host='aws-serverless-tools-telemetry.us-west-2.amazonaws.com', port=443): Read timed out. (read timeout=0.1)
Error: PythonPipBuilder:ResolveDependencies - {tensorflow==2.2.0(wheel), grpcio==1.29.0(wheel)}
Expected result
I expected a successful build.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: MacOS 10.14.4
sam --version: SAM CLI, version 0.52.0

I am currently having the exact same issue with those libraries and SAM.  I thought I would check in to see if you had managed to discover the cause, and identified a solution.  If not, I will persist and if I manage to find the solution myself - I will be sure to post it here.

@divnrio I haven't been able to find a work around.  Please let me know if you do!

Also having this issue, following!

Submitted 17 days ago, and no activity that I can see.  Not being able to build a project is a real drawback to using the SAM platform.

@pasppro You will need to build tensorflow separately and either vend in your function code or within a layer.
sam build tried to build something that is known to be compatible with Lambda. We first search for a whl. If one is not available, we attempt to build from source. These failures are commonly seen when the package has c-bindings or dependencies that require c-bindings (can be seen by other "can't resolve dependencies" issues in the repo.

@jfuss Thanks!  I think I see.  sam build needs a wheel for tensorflow 2.2.0, but there isn't one.  So, I'll need to create one(?)
Can you point me to an example of how to include my local build in the project? I've looked at the Layer examples here, but it's not exactly clear.

@pasppro Sorry this took me a while to respond. I was digging into some of what sam build does for python (more details below).
To answer the "how to include to include it": The easiest way I can think of is doing something like pip download --only-binary=:all: --platform manylinux2014_x86_64 --implementation cp --abi cp36m --dest <directory> tensorflow should work (though I have not tested but this is what sam build does). The  can be in your function code directory. This should make in import-able then (that is more or less what we are doing under the hood). You can also put this into a directory outside of the function, say myproject/tensorflow-layer/ and then create a Layer resouce that points the ContentUri to that directory, and Ref that resource within the Layers property of the Function.
So over the weeks, we have seen a number of "we can't build this X dependency". Nearly all of these have been related to python. Our builder logic claims Lambda only supports the following platforms (whl specifically). So build generally works if a package creates a manylinux1_x86_64 wheel, but packages like Tensorflow are using a newer platform wheel, manylinux2010, and some only package manylinux2014. Reading through PEP 513 (manylinux1) and PEP 599 (manylinux2014), it seems the part we really care about is the glibc versions. PEP 599 states that if the platform supports manylinux2014, then it will support earlier versions as they are meant to be upper bounds. I did some testing on the Python3.8 and Python2.7 runtimes and looks like we meet the min glibc versions of 2.17 or greater of manylinux2014.
So what does this mean?
The tldr: We and should expand scope to manylinux2014.
I have a patch locally that will download the manylinux2014 whls (or if manylinux2014 is not available to download either manylinux2010 or manylinux1). This will require an AWS Lambda Builders release and then an update in SAM CLI. I want to do a little more testing but going to try and get a patch out to that repo.

@jfuss Thanks!  You're the BEST!

Thanks for looking into this issue, it's been causing us some headaches with another dependency!
It seems glibc-2.17 is available in Amazon Linux as well as Amazon Linux 2, so presumably Python 3.8 will also be fine after this update?
I saw there's a PR out for the lambda builder already. Is there an ETA for when this might be fixed in both packages @jfuss ?

There is a  PR #2116 that will incorporate the updated Lambda Builder release into SAM

This was released in v1.0.0.
Note: You will need to update pip to at least 19.3 to support manylinux2014 downloading, otherwise sam build will still fail.
Closing

@jfuss This issue still persist with SAM CLI v1.0.0 when building with the --use-container option, because the SAM CLI built into amazon/aws-sam-cli-build-image-pythonXX docker images is still 0.53.0.
Steps to reproduce:
docker pull amazon/aws-sam-cli-build-image-python3.7:latest
docker run -it --rm amazon/aws-sam-cli-build-image-python3.7:latest
sam --version # returns "SAM CLI, version 0.53.0"

@tianchu thanks for reporting this, we're going to track it in a separate issue:
https://github.com/awslabs/aws-sam-cli/issues/2134


Feature/pytest

Could someone please review this? Thanks.

Please confirm test coverage and success status screen shot, and I'll approve and merge!


version 1.8.0 is breaking angular

I'm getting issues with 'reflect-metadata' not found, or Zone js not found, all kinds of import errors during runtime and tracked it down to the tslib update. Reverting package.json to 1.7.1 fixes the issue.
SHould this be reported here or angular?

The only change between the two versions is an addition of a new __makeTemplateObject. does deletion this function in the file address your issue?

Thanks for the response. It seems that it may be related to my version of Angular (4.3.1). When I updated my package.json to angular 4.4.4 (latest stable version), everything worked normally. I am having trouble reproducing the issue now, so I will close this.


[Feature Request] Able to define different responses for errors

Used to be able to define errorResponses in previous versions of this library - Seem to now be deprecated, with no obvious replacements in place.
Is there anyway to define different error responses based on status code or some other way to achieve something similar?
If not, consider this a feature request ❤

@skela you can use method .response(statusCode: ..., body: .type(of: example))

ugh, sorry, i did not realise u could chain it with .response :'(
thx!


Azure: rewrite locking mechanism to use channels

Currently the Azure locking system to enforce concurrency, while functional, is a bit of a mess.
When testing out, we ran into problems where prometheus would scrape the metrics endpoint after the machine population interval had expired, and if it scraped it again within the window where the machines were being queried from the API, we'd run into a deadlocking situation, exacerbated by the readiness checks trying the locks.
We should re-write it to use channels as they're much easier to work through the concurrent problems.
The structure of the pricing map and machine map seems to be sound, but may need to be touched as well.

closing as we have sufficiently solved this problem with the current locking implementation.  Channels won't get us anything except slightly more idiomatic go code, in my opinion


Limit message payload size, or not?

Currently the JsonData payload in NewStreamMessage is unbounded. Or rather, it is bounded to the limits the underlying store can support. For MsSQL server (nvarchar(max), text) is rather large. This allows other patterns such as storing "documents" in the streams. However, EventStore has a limit of 64KB per message. Not having this limit further reduces portability.
While full portability with GES is not a goal, I thought that mentioning this decision here would be a good idea. :)
At this moment my preference is not to limit the size, but leave that to the user, if they so need it.

👍 for no limits - PGSQL has jsonb for efficient binary-encoded JSON.

Vote for having 64KB as an overridable default limit. That way you make GES portability more explicit to consumers, and also put a boundary forcing people to think. You might want to deal with bigger files in a different way.

GES limit is actually 4MB or or similar. So no, not going to limit the size; consumers will have to make appropriate decisions for themselves. Will re-open later if it really appears to need it in practice.


Add Python packaging and deploy to PyPI

In order to make FENRIR available to Python users, it would be great to make it available on PyPI, the Python Package Index. This would allow users to get the package via pip as
pip install fenrir

It's a good idea, yeah. Priorities right now are : 1. Migrate to Python3, 2. Improve autoconf module, 3. Everything else


VMC connection

When I initially launch a VM from Hyper-V manager by double-clicking the VM, VMC connection settings are applied (full screen, mapped drive).
Once logged in, there isn’t an option to switch to another user account (from the start menu, user icon)
So I must log out to switch users.
When I log out, the VM exits full screen mode.
Log in  (as a different user or the same user) and the VM stays in windowed mode.
Try to go full screen mode, and the VM is letterboxed (wont stretch). Also, no mapped drives. Its like its not applying VMC connection settings. But I can switch users from the start menu.
The only way to go full screen (and access mapped drives) is to close the connection and reconnect.
Once reconnected, full screen works and mapped drives appear, but I can no longer switch users from the start menu.
Link to doc:

Posted in wrong location


Option to encode to H.264 or HTML5 compatible format

Hi thank you for this great tool! I love it, I was wandering if it was possible to convert the video to H.264 or other HTML5 supported format with this. I didn't find anything on the web or in the issue tab. And I was wandering if was actualy an option.
I would like to encode in some HTML5 format in order to display it in a webpage, but when I try to play the "altered" video it gives me only the audio.  Looking at the info with MediaInfo the original video Stream was AVC and  was working but the "altered" video was MPEG-4 Visual.
I don't know if it could help,
Thank you for your time

You can use the --video_codec option to change the output file to the h264
Example:
auto-editor example.mp4 --video_codec h264

And you can verify the output using info:
auto-editor info example_ALTERED.mp4

file: example_ALTERED.mp4
 - fps: 30.0
   - mode: VFR:0.000000 (0/526)
 - duration: 17.566667
 - resolution: 1280x720 (16:9)
 - video codec: h264
 - video bitrate: 207k
 - audio tracks: 1
   - Track #0
     - codec: aac
     - samplerate: 48.0 kHz
     - bitrate: 127k


For the second point, when I see the description, I will put 'searchString', but it passed even if I have not finished typing 'searchString.

For the second point, when I see the description, I will put 'searchString', but it passed even if I have not finished typing 'searchString.
Author: Thomas Chen
Slide: Local,Public

It indeed passes with any param name.


Use temporary file for tarball instead of tmp directory (fixes #387)

Description
Our current code relies on the /tmp/ directory, which is not guaranteed to exist on non-Linux operating systems, such as Windows. This PR uses the temporary file functionality in urllib.request.urlretrieve to avoid explicitly assuming anything about the local file structure.
References
N/A
Blocked by
N/A

Looks, great thanks!


Cursor colour enhancement?

I've been using this theme for over a month now and it's great except that the cursor can appear too well camouflaged on the dark background. Perhaps setting it to a brighter colour might be called for? I've manually set it to white (#ffffff) for now.
Happy to share screenshots of this if necessary.

Thanks for reporting this! I use Emacs inside iTerm 2 which by default set my cursor to white so I never noticed this issue. I was able to reproduce the dark gray cursor when opening Emacs directly in a window. I've released v0.3.1 that changes the cursor to white from gray. Melpa could take upwards of ~24 hours to rebuild the package and pull in this update.
I'd suggest updating this theme tomorrow and you should see the corrected cursor then.


Serialize Class from an external module

What would be the best way to serialize a Class imported from an external module 'library'. BabylonJS for instance.
Would it be possible to create a custom function that creates a new Class instance on deserialization.

This feature is now implemented and will be released shortly. https://github.com/marcj/marshal.ts/commit/dadf0ec3a6567f8a211cb1041e895af966e4c0b2


Stop Autoscrolling upon hovering the page's content

Stop Autoscrolling upon hovering the page's content.
Also refactors some of the files' messy code.
demo link
related issue: #5
mention for reviewer: @pinkienort

@pinkienort Thanks for the review, I've addressed your comments.
Seems like https://github.com/pinkienort/AutoScrolling/issues/18 is still happening even after changing to setInterval. : - | I'll be looking to fix that in a separate branch after this.

@pinkienort I've changed the code around your comments. You can also directly commit to this branch with your own code if you find anything else you dislike.


Build PIP package

Anyone want to contribute building PIP package?

I tried to create it here is it ok?

https://github.com/DevconX/Malaya/pull/4

I updated setup.py in master, can you let me contribute for malaya pypi, thanks!

could you remove your malaya in pypi, I want to update the description, thanks!

or add as maintainer pun no problem

add me as maintainer / owner pun no problem. username husein.zol05

I add as owner


Expert parallelism / MoE example would be awesome :)

I loved seeing the blog post with a simple, standalone implementation of many techniques used in production to speed up LLMs. Would love to see this extended to MoE like Mixtral, which at the moment seem fairly annoying to use and hack on. Curious how torch.compile can help with these, and possible issues that might arise like graph breaks due to gating.

@andersonbcdefg We have added the support of Mixtral-8x7B MoE, please check https://github.com/pytorch-labs/gpt-fast/pull/71. Feel free to try and share feedback.


Support for OpenSSL 1.1.1d <1.10.x> [8224]

This should fix #1087 for branch 1.10.x
It has all the changes on #1162 applied on 1.10.x instead of master.

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows


build: include the lock file

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Playgrond] Add StacksOverTabs example

StacksOverTabs is exactly the same as StacksInTabs, except that when navigating, the routes get pushed on top of the tabs instead of inside of them. This is a common paradigm.
Also fixes StacksInTabs so it navigates to SettingsTab instead of pushing a new Settings screen to the stack.

Looks good, thanks for the contribution!


Is it possible to run c++ code (instead of Python ) in the 24 lesson course against CounterFit ?

Thank you

In theory yes, but the Arduino libraries haven't been written to work with it.


Ask Jean about Adopt-a-Street Program

http://sfdpw.org/index.aspx?page=1229

Jean is finding out more information from DPW.

@hackajesse any update on this? Is this issue still relevant?

cc/ @jasonlally

Bump @jasonlally @hackajesse

hey hey, we added Adopt a Street language to the 2nd email, and Public Works added (or will add) info on Adopt a Drain to their page, so this issue should be resolved.

oops, they haven't added the language yet so i'm re-opening this. i'll bug them about it.

Assigning to @jeanwalshie, let me know if you need anything from our end!

nothing needed on your end so i'm closing this, thanks!!


Update upgrading-to-0-21-0.md

Description & motivation
I added a link to an explainer for DML statements. I had never heard of DML statements, so I was intimidated when I saw the language. I thought a simple link to a widely accepted Stack Overflow answer could help future readers quickly recognize that while the acronym is unfamiliar, they already know plenty about DML statements.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Thanks @joellabes! Glad you agree this made sense! I wasn't sure if a link to SO would be accepted, but I figured it was better than Oracle documentation 😅

I tend to link to Wikipedia, but its article wasn't very clear so SO got the link after all

@joellabes yup, it was the best one I could find. Thanks again!


an output file format

What should our output file have in it? How will we represent haplotypes within that file? What information will we want to include about the haplotypes?
I'm using this issue to track my thoughts on these questions.
One thing that comes to mind: I might want to look again at the output files for SuSiE and FINEMAP. There may be aspects of those files that we can share.

ok, the plan is to output a TSV file <filename>.haps containing columns:

haplotype ID
tree ID
variant ID (or POS?) <-- has to match something from the genotypes file
p-value for inclusion of variant
beta value for inclusion of variant

and we might also want another file with the following columns:

haplotype ID
tree ID
effect size (from leaves of tree)
p value
PIP (from running susie)

we can also consider combining the two files using something similar to the GFA spec

ok, new plan is to combine the .vars and .haps files into one file format using a strategy similar to the GFA spec
we're going to prefix each line with a letter, denoting the type of line, something like "V" for variants and "H" for haplotypes, for example
and the variants lines will just have columns 3-5 as identified here but they will follow the haplotype they belong to

TODO:

[ ] write up the output file format in the docs


Implement Interceptors

A more formal proposal will follow but creating a new issue here to allow [https://github.com/jsr107/jsr107spec/issues/32](Issue 32) and [https://github.com/jsr107/jsr107spec/issues/43](Issue 43) to be closed.
Instead of listeners the proposal is for Interceptors. Conceptually these will be registrable at well defined points in the call cycle of Cache operations. They will occur in the line of execution of blocking calls. If registered before the "low level" key operation (e.g get/put) from the cache they will be able to block the operation (e.g. by throwing an exception), or to modify any value being submitted. If after the key operation they may modify the return value. Interceptors may be chained. The proposal replaces Listeners but will provide clear semantics and potentially far greater power in that some cache topologies may provide multiple points where interceptors may be registered. For the purpose of 107 which does not define topologies the 2 points where interception will be defined is immediately before and immediately after the "low level: operation.

A preliminary +1 from me. We added something similar to the ehcache-spring-annotations library. I'd also be interested with the more formal proposal of looking at annotations specific interceptors (or extensions to the interceptors) to expose things like the invocation context which a specific to an intercepted method invocation.

Agreed - interceptors would make for a lot of excellent possibilities. With a good spec for interceptors, listeners are not needed.

Interceptors seem fine in principle.  From what I can tell from your description an interceptor is just a listener under a different name along with two additional constraints:

Interceptors are listeners that are fired synchronously
Interceptors can "listen" both before and after a "low level" operation.

It seems however that almost every issue that troubled us about spec'ing the behavior of listeners just becomes a question regarding the behavior of interceptors : https://docs.google.com/document/d/1gJ2XTO8ERne3tokLMI-FsOCQAmryjeDbS889YPJgcH0/edit .  If we had just said listeners are synchronous we would actually have a smaller set of behaviors to pin down than we now have with interceptors (as we wouldn't have the before/after complication).  It also seems slightly confusing to me because under the original listener discussion there seemed to be a leaning in favor of asynchronous listeners not synchronous ones.
Maybe someone can enlighten me about the advantages of interceptors over synchronous listeners?

Okay, I understand the motivation now.  I reserve judgement until the specification of them is more complete (and I'll hold off on pushing any comments or review of the related features that may be subsumed by interceptors).  My two main concerns at the moment (which may be assauged by a more complete spec) are:

Listeners, Statistics, Readers and Writers will all have different requirements from invokers they will need to be sensibly reconciled - two examples:

writers may (will?) need to maintain ordering of calls for a given key across all accessing threads
statistics don't really need any guarantees outside of eventual execution

You've already (off the cuff) mentioned the concept of interceptor invocation within a locking context, this is an implementation detail (caches may not use locking).  We'll likely have to be careful to define the behavior of interceptors in terms of the guarantees they provide to users, instead of their interaction with an implementation specific feature (e.g. locks).

I can't get support for using interceptors instead of Statistics, Read Through, Write Behind so am closing this

Was there some additional discussion that lead you to close this issue Yannis?
As a cache user, I'd rather have interceptors than listeners.
Even though they might not replace the statistics in the API, having interceptors in the API would allow an implementor to easily use an interceptor to implement statistics, read through and write behind. For instance - implementing statistics, read-through, and write-through via interceptors in a reusable library like the common-ri layer you started working on would make implementing the spec quite easy for many vendors.

I (obviously) prefer interceptors. We had discussion with Greg and other Software eg people and I could not get support for interceptors instead of (though enabling), listeners, statistics, write-behind.

That is too bad, I would rather of had interceptors be the basis for those
other features as well. Aspect style development is significantly more
powerful and flexible in the long term.
-Eric
On Thu, Jan 5, 2012 at 2:59 PM, Yannis Cosmadopoulos <
<EMAIL_ADDRESS>
wrote:

I (obviously) prefer interceptors. We had discussion with Greg and other
Software eg people and I could not get support for interceptors instead of
(though enabling), listeners, statistics, write-behind.

Reply to this email directly or view it on GitHub:
https://github.com/jsr107/jsr107spec/issues/48#issuecomment-3376239


Fixes for canonicalization, and stereochemistry

Reference Issue
Fixes a bug in canonicalization and one in stereochemistry.
What does this implement/fix? Explain your changes.
In canonicalization, a choice was made too early. When there were 3 or more possible branches, a choice was made by comparing the first two branches and not considering the third or subsequent branches.
for stereochemistry, a procedure was added to NOT generate stereo bonds where they should not be.   Also, some stereo/chirality clean up.
Any other comments?

@tadhurst-cdd I'm still digging out from the UGM, but I will definitely start reviewing this one this week.

@tadhurst-cdd is it possible to provide an example showing the canonicalization bug that this fixes? You've added tests (thanks!), but I'm not sure which test is for which problem.

@greglandrum what do you think about adding canonicalRank directly to the Atom class.  If we are requiring it for chirality and other algorithms, it might make sense to make this a direct member and not a property.
(I would have both canonicalRank and symmetryClass, but that might be a bridge too far!)

@tadhurst-cdd is it possible to provide an example showing the canonicalization bug that this fixes? You've added tests (thanks!), but I'm not sure which test is for which problem.

There were a couple of tests that rely on the new canonicalization fix, but the best one was omitted (It was next to an atropisomer test, and I missed it.  That test is not included and is in
cxsmiles_test.cpp -> TEST_CASE("SMILES CANONICALIZATION")
in this test:
Without the fix:
CC@@H[C@]12C[C@@]3(F)CC@@(CC@(C3)C1)C2
canonicalizes to:
CC@@H[C@]12C[C@]3(F)CC@(CC@(C3)C1)C2
CC@@H[C@]12C[C@]3(F)CC@(CC@(C3)C1)C2
canonicalizes to:
CC@@H[C@]12C[C@@]3(F)CC@@(CC@(C3)C1)C2
with the fix in, all (incluiding randomizations) canonicalize to:
CC@@H[C@]12C[C@@]3(F)CC@@(CC@(C3)C1)C2
There are other examples in that test, and the testing includes multiple randomization, some of which give the correct result and some give an error.
To see the errors, you can change line 726 in new_canon.cpp to:
if (tooManySimilarNbrs && false) {

@greglandrum what do you think about adding canonicalRank directly to the Atom class. If we are requiring it for chirality and other algorithms, it might make sense to make this a direct member and not a property.
(I would have both canonicalRank and symmetryClass, but that might be a bridge too far!)

I am fine with that.  But be advised that the canonicalAtomRank   is only used with the new (not legacy) stereo.   I would prefer to make the new stereo the default BTW.  But that could only happen after it is tested and does not dramatically impact performance.  I will put it in the atom class if @greglandrum is OK with that.

@tadhurst-cdd moving to the atoms probably needs to be a separate PR, it might touch a bunch of places.

The checks all pass now Tad @.*** Nov 3, 2023, at 11:15 PM, Greg Landrum @.***> wrote:
@greglandrum commented on this pull request.
The changes look fine to me, but they did break the CI builds.
I'll merge this when that has been resolved
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

The checks all pass now Tad

Thanks @tadhurst-cdd!


Inaccurate path for location of yarn-site.xml file

The path shown for the location of yarn-site.xml was incorrect (missing backslashes)

@NathanMSFT : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@NathanMSFT Thank you for catching this. #sign-off


Warning: The set-output command is deprecated and will be disabled soon.

Found a bug? Maybe our Slack Community can help.

Describe the Bug

Warning: The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Expected Behavior
No warning.
Steps to Reproduce
Steps to reproduce the behavior:

Run the workflow

Screenshots
If applicable, add screenshots or logs to help explain your problem.
Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:

OS: [e.g. Linux, OSX, WSL, etc]
Version [e.g. 10.15]

Additional Context
Add any other context about the problem here.

@johncblandii it seem solved now


Defining fields

In the docs, there is a mix of:
fields: {
  phones: {
    ...
  }
}

and:
fields: {
  'phones': {
    ...
  }
}

Sometimes a field name is provided as a string literal. I assume this makes no practical difference?

Yes there is no difference. It's just how JS works.

Thank you! That's what I thought. Let me know if you would like me to submit a PR to standardise the readme examples one way or the other. Personally, I prefer without the quotes, especially as that forces the use of valid JavaScript identifiers.

I don't think that there is a need for that. Developers know that it's possible to define it in both ways so they can choose the one preferred by them.


Bumped upstream version to v0.9.9

Thanks for the PR. But you need to bump everything to use v0.9.9 and update the commit here https://github.com/open-web3-stack/open-runtime-module-library/blob/cae9290ad75463bb5a4a9599401bb8f3952b19b4/Cargo.dev.toml#L34

Closing in favor of #587


Feedback on TTNN testing (round # 1)

Hi Eyon and Akhmed, here is a brief summary of my findings:

It's a great tool and will enable lot of proof of concept and model bring-up efforts. I'll extensively used it in the coming weeks

I did my testing on a bare-metal in a customer-like environment as a typical use case for Project-GS

At the step of launching the jupyter-notebook, we need to hint to the user to ssh -L8888:localhost:8888<EMAIL_ADDRESS>from a second terminal to establish the connection

My log messages wasn't verbose like your tutorial, which is informative in understanding the steps undergone when running matmul for example

Need to add to_layout(ROW_MAJOR) before printing/slicing the output tensor. I believe it's missing in tutorial 1

The %%time is not that consistent from the notebook to compare the perf of different options (i.e. pre and post tilizing omem_config=L1)

Related to last point, it's good to have a normal .py example to compare the perf

Related to last point, it'll crucial to add a profiler() example

Can we add the GFLOPS metric beside the absolute runtime?

There was random hanging when I was moving up/downwards between the cells, as well as from one notebook to another, so I resolved that by re-launching the jupyter session from the server side

The cells are dependent on each other.  Added a comment in the documentation: "Hint: Be sure to always run the cells from top to bottom as the order of the cells are dependent."

In tutorial 1, I tried the bcast option for the ttnn.add(), but seems the only config working is on the last dimension, and not the others. So I couldn't make an example to add the number option instead of the tensor

Added: https://github.com/orgs/tenstorrent-metal/projects/59/views/1?pane=issue&itemId=44944938

ttnn.mul() is not working as we discussed

Fixed

In tutorial 2, Core-grid option did't work for me, and it said reverted to default settings

Fixed, no longer going to revert to default settings.  TODO: Add documentation.

For the multi-head attention example, I understand we'll have another optimized version (i.e. FUSED OPs) for the first things to do with METAL, correct? If so then when will it be ready please?

I feel like we can close this one for now given the update. Let's open a round 2 with the next round of documentation sprint

Thanks!


[19] Remove Nth Node From End of List, one pass

Codecov Report

Merging #21 (2b26f12) into master (b7588af) will increase coverage by 1.9%.
The diff coverage is 100.0%.

Impacted Files
Coverage Δ

.../problem/p0019_remove_nth_node_from_end_of_list.rs
100.0% <100.0%> (ø)

src/util/linked_list.rs
100.0% <100.0%> (ø)


installing package head versions is now broken

Describe the bug
latest vcpkg tool has broken downloading port head versions
Environment
all
To Reproduce
Steps to reproduce the behavior:

./vcpkg install darknet --head
See error

Expected behavior
port head version to be downloaded
Building package darknet[core,opencv-base]:x64-windows from HEAD...
-- Downloading https://api.github.com/repos/AlexeyAB/darknet/git/refs/heads/master -> AlexeyAB-darknet-master.tar.gz.version...
CMake Error at scripts/cmake/vcpkg_from_github.cmake:164 (message):
  Failed to parse API response from '':

  {"ref":"refs/heads/master","node_id":"MDM6UmVmNzUzODg5NjU6cmVmcy9oZWFkcy9tYXN0ZXI=","url":"https://api.github.com/repos/AlexeyAB/darknet/git/refs/heads/master","object":{"sha":"b8dceb7ed055b1ab2094bdbd0756b61473db3ef6","type":"commit","url":"https://api.github.com/repos/AlexeyAB/darknet/git/commits/b8dceb7ed055b1ab2094bdbd0756b61473db3ef6"}}

Call Stack (most recent call first):
  ports/darknet/portfile.cmake:1 (vcpkg_from_github)
  scripts/ports.cmake:140 (include)

Error: Building package darknet:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: darknet:x64-windows
  Vcpkg version: 2021-08-12-85ab112d5ee102bc6eac8cdbbfdd173a71374e04

Additional context
it fails on all machines I have tried, even on github actions, consistently since latest vcpkg tool update
https://github.com/AlexeyAB/darknet/runs/3366754905?check_suite_focus=true

This seems to be where it all happened:
99e06a64e [rollup:2021-08-09] Rollup PR (#19469)

99e06a64e^ builds fine, 99e06a64e fails like above


general command for setting config strings

I'm working on extending the HA Fully Kiosk to allow me to set the start URL within an automation.
Rather than adding a specific command for the start URL, I thought it'd make more sense to have a general function in the library and register a service in HA to be able to set any of the config values.

Thanks!


Error when choosing a certificate from Key Vault on App Gateway

I've created a certificate using Let's Encrypt and imported to my Key Vault, which it's a PEM file.

But when i try to choose it, to use on the Listener from my Application Gateway, i got this following error:

I've tried to choose a existing certificate on App Gateway and renew it, but i've got another error:

Despite the message say "See details below", there's none.

@brenovult -
Thank you for your question.
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft QnA or Stack Overflow. Thank you for your understanding.

@brenovult -
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft QnA or Stack Overflow. Thank you for your understanding.

@brenovult -
Since we have not heard back from you we will now proceed to close this thread.
However if you can share additional information regarding the documentation you are referring and circle back on this thread , we will gladly reopen the issue and continue the discussion. Thank you.


Separate header and content background color

It is not possible to give the header a different background color than then content.
The content background always be also visible on the header (seems like there is a padding?)

@all-contributors please add @mariomurrent-softwaresolutions  for code


Spy between executions

test('spy', () => {
  const play = { foo: () => {} }

  const spy1 = jest.spy(play, 'foo')
  play.foo()
  const spy2 = jest.spy(play, 'foo')

  expect(spy1).toHaveBeenCalled()
  expect(spy2).not.toHaveBeenCalled()
})

It fails as spy2 actually has been called.
I am wondering if this is the desired behavior,
as I thought that jest.spy() create a spy at the specified point in time.
I have some tests that use a global variable (for the suite), and I want to spy it in specific tests.

Try to restore the original implementation before assigning it to a new spy with spy1.mockRestore()
Please note this issue tracker is not a help forum (as stated in issue template)
We recommend using our discord channel or StackOverflow where there is an active jestjs tag for questions


Content Static - troubleshooting

content/information.blade.php
A page extrated fro the gudes pages to explain troubleshooting

[ ] make model


A1112 Exception in glyf test with Microsoft shipped fonts.

simsunb.ttf, win 8.1, Glyph index 12837 Test: ValidateSimpContInters
This happens with both FontVal 1.0 and 2.0 , slightly different detailed message with the latter.

Fixed in upcoming v2.1


Overestimates cardinality for bruteforce substrings when other types of characters present elsewhere

It seems that the cardinality used to calculate entropy of bruteforce substrings is calculated based on the entire string. For example:
frzplfqetuothv: cardinality 26, as expected
frzplfqetuothvpassword: cardinality 26, as expected
frzplfqetuothvpasswordA: cardinality 85.
The A present elsewhere in the string causes it to assume that the letters of frzplfqetuothv were sampled from the larger set of characters. This doesn't make sense, because users often pick passwords with e.g. a punctuation mark attached. Thus it is vastly overestimating the entropy of such passwords. The cardinality should be calculated per bruteforce substring.
Related: It parses passwords like frzplfqetuothvCOCIWDZOAZPVRL as one long bruteforce string. It should attempt to split such strings into multiple bruteforce runs with lower cardinality.

This is now fixed. Cardinality for bruteforce matches is now 10 across the board, independent of types of characters present.
One thing is for sure -- earlier cardinality estimates were way too high. A markov attack, for example, could significantly reduce the number of tries needed per character if the password is "english word-like" [common consonant-vowel sequences] or "password-like" [starts with capital letters, lowers in the middle, ends with numbers].
It would be a fun future research project to investigate better models for estimating resistance to markov attacks / other "smart" bruteforce attacks on a given password, perhaps trained on a large password corpus. For now this is a simple fix in the right direction.
One downside is this significantly underestimates the guessability of long random strings. A random alphanumeric string will get estimated as 10^n instead of 62^n, for example. Yet another case of underestimating to be safe.


Cell running should use progress notifications in status bar

Have a notebook with a lot of cells.
Scroll down to cell N#8 for example. Click on the Run all above cells button.
Now you are not aware of the progress of the above cells, you have to scroll up.
I suggest to use a progress notification in the status bar, that would say which cell is currently in progress. That way the user would be aware of what is going on without having to scroll down.
fyi @jrieken

You can have multiple cells in progress, do we list all of them?

I suggest if there is just one mention the name. If there are multiple say "n cells in progress..." to not take too much status bar space.

I can keep this open but I don't think that notebook users are often concerned with tracking cell-by-cell execution progress.

My painpoint is always hard to find which cell is executing.

There's https://github.com/microsoft/vscode/issues/107919 that tracks the running of the active cell when executing Run All Cells

IMO, I think adding an item in the status bar will work fine, similar to the running tasks.

What does the webview notebook do for this?

They show a sinner in the status bar with Executing Cell text

@digitarald also brought up the point of not being able to tell whether a collapsed cell is executing. Not sure I want to keep the cell status bar visible when a cell is collapsed, but we could have an item in the editor status bar (or notebook toolbar?) like @isidorn suggests, and if I click it, the top currently executing cell is focused/revealed.

Just ran into this issue where I have a notebook with 142 cells and when I hit "running all" I wasn't sure where the current cell was without scrolling:

Would love it if we showed a status bar item with the spinner icon after a certain amount of time (10s?)

https://github.com/microsoft/vscode/issues/130611


Show completed code in completed steps (courses - learn platform)

Is your feature request related to a problem? Please describe.
I am learning from the RWD course, and I wanted to revise what I had done previously.
I went to previous steps, but I found that the steps are shown incomplete when I open them. It tells you to do the step again.
This doesn't make sense because you have a Restart Step button at the top if you do want to do the step again.
Describe the solution you'd like
Completed code should be shown in completed challenge's code editor along with an indicator like  ✅ Step completed.
Describe alternatives you've considered
Additional context
What it currently looks like when you visit a previously completed step

What it should look like when you visit a previously completed step

It would be relatively easy to show the solution to a step, and the "Step completed" text, (rather than storing the user's code) if a user has already submitted that step.
If other maintainers are in favour of this change, I'd be happy to open this up for PRs.

My concern with that approach would be it completely destroys the UX if someone wants to go through a practise project again.

My concern with that approach would be it completely destroys the UX if someone wants to go through a practise project again.

I propose to add a button Restart Project under the summary of each project for this issue.

I'm not keen on adding a button to do that, given the potential for frustration if someone triggers that accidentally.

I think I'd be in favour of the existing flow, presonally.

At a minimum, we could probably add the checkmark next to "Step 22" like we do for all the other challenges after it has been completed. That seems to be missing here.

That's a good idea, IMO.

I think just adding that check mark is probably the way to go then. It keeps it consistent with everything else. I'm going to open it up for contributions. If anyone still has reservations, be sure to bring them up.

May I suggest in addition to indicating completed challenges in the challenge itself, to do that in the list of steps. Here is a crude representation of what I have in mind:

@raditotev I believe it already does that. Here's a screenshot of what the challenge looks like for me, after I've completed it (the steps are highlighted in blue instead of white).

Also, I would like to work on this issue, and I agree that adding a checkmark next to the step would be a good fix, but I noticed that the RWD courses don't have a step number or title at the top like the other courses do. For other courses, the checkmark for completion is displayed next to the name of the challenge, like you can see in the first screenshot (of the JS Algorithms & Data Structures courses). However, for the RWD course, there is nothing in this location, as you can see in the second screenshot.

How should we go about adding a checkmark to the RWD course? Should we just put it next to where it says Step 1 on the screenshot (inside the instructions block)? Or should we move the step number to the same place as where the challenge names are for the other courses and put the checkmark there, in order to be consistent with the other courses?

@raditotev I believe it already does that. Here's a screenshot of what the challenge looks like for me, after I've completed it (the steps are highlighted in blue instead of white).

Thanks @benjaminJohnson2204. I could not log in since yesterday and this is probably the reason I don't see indication for completed challenges in the list of steps.


Update the header, logo, and TOC to match preview UI in Figma

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2909  👈
#2901
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nickgraz and the rest of your teammates on  Graphite

Merge activity

Jul 29, 12:16 PM EDT: @nickgraz started a stack merge that includes this pull request via Graphite.


Model Configuration sliders info buttons accessibility improvement

Feature Request
Description
Improve accessibility and titles of the Model Temperature, Top P, Presence Penalty and Frequency Penalty sliders in the Model Configuration settings.
Screenshots

Before

Desired Outcome

Additional context

adjust the aria-label and title of the info buttons to include the name of the control
add aria-hidden to the info icons (the buttons already have labels, so the icons are purely decorative)

Acceptance criteria
GIVEN a screen reader user interacts with the Model Configuration
WHEN they focus on the Model Temperature, Top P, Presence Penalty, and Frequency Penalty info icons
THEN they know which setting the button will give them more info for
GIVEN a user is in the Model Configuration menu
WHEN when they hover over Model Temperature, Top P, Presence Penalty, and Frequency Penalty info icons
THEN the title (hover text) of the info icon tells them which setting the button will give them more info for
GIVEN a screen reader user interacts with the Model Configuration
WHEN they focus on the Model Temperature, Top P, Presence Penalty, and Frequency Penalty info icons
THEN they only see the button in the accessibility tree, not the graphics-document

Sounds good in NVDA
Title popups look good on mouse hover
can't see the graphic in the accessibility tree
💯


relationshipLabels ignored

I have the following yaml:
# Connection properties
connectionString: "postgres://postgres:postgres@localhost/postgres"

# Define what schemas should be used
useAllSchemas: true

selectedTables:
  - schema1.table1
  - schema1.table2
  - schema2.table1
  - schema3.table1

# Additional flags
showAllConstraints: true

showDescriptions:
  - enumValues
  - columnComments
  - notNull

showSchemaPrefix: true

relationshipLabels:
  - "schema3.table1 schema2.table1 : participates_in"
  - "schema2.table1 schema1.table2 : results_into"
  - "schema2.table1 schema1.table1 : results_into"

but when I run mermerd --runConfig ./config.yaml I don't see any labels generated. Everything else seems to be respected. What am I missing?

👋🏼 One thing to make sure of: you must list the table with the primary key first and the table with the FK reference second.
Perhaps we could add support for either direction?


Draft: Reusable workflow

@weaversa I'd like to get a version to work for saw, too, but I'm not sure if it's going to work yet

I'm not as happy with the saw version yet, but I think it's in a better place now to be merged, and I might make another branch to keep working on it


How to properly pass ffmpeg options?

I have a video file with audio codec that causes no audio on TV when casting through castnow. When I convert the audio with ffmpeg it works fine (both audio and video):
ffmpeg -i sample.mkv -vcodec copy -acodec vorbis -ac 2 -strict -2 -f mp4 -movflags frag_keyframe+faststart - | castnow -

But then the castnow shortcut keys don't work. How can I convert above command to castnow options? I tried:
castnow ./sample.mkv --ffmpeg-vcodec copy --ffmpeg-acodec vorbis --ffmpeg-ac 2 --ffmpeg-strict -2 --ffmpeg-f mp4 --ffmpeg-movflags frag_keyframe+faststart

And it plays but with no sound.

try with:
castnow ./sample.mkv --tomp4 --ffmpeg-vcodec copy --ffmpeg-acodec vorbis --ffmpeg-ac 2 --ffmpeg-strict experimental --ffmpeg-f mp4 --ffmpeg-movflags frag_keyframe+faststart

--tomp4 is needed to use the transcoding plugin.
note: seeking is disabled when transcoding
hope this helps

I can't even verify it because castnow stopped working (stuck on "Loading...."). Thank you anyway. I think I'll stick to chrome extension for casting my files to chromecast.
This project seems to be abandoned...

A simpler command for this problem of no audio with castnow would look something like.
castnow --tomp4 --ffmpeg-c:a ac3


registry-cli feat: add restart and start commands

@dileepbapat Could you please review and merge this? Thanks!


Documentation for "How do I install my operator"

/lgtm
I think we'll want additional instructions for "directly applying" without a subscription or a catalog, but that can come later.

@awgreene addressed your comments.


Bugfix: Check for Metrics Route in HttpContext.

This fix addresses issue #191, adding a quick dictionary check and returning the default value if the item does not exist in the dictionary (instead of throwing a KeyNotFoundException).
Details on the issue fix or feature implementation
Normally it appears AppMetrics endpoints append some custom items to the HttpContext dictionary items, which are then referred to later. If for some reason the dictionary items are blank, a default value is returned.
In the scenario where somehow the dictionary item is not appended in the first place, the code was throwing a KeyNotFoundException. With this bugfix, the code now checks for the existence of the dictionary item first, and returns the same default value if it is missing.
Confirm the following

[X] I have ensured that I have merged the latest changes from the dev branch
[X] I have successfully run a local build
[ ] I have included unit tests for the issue/feature
[X] I have included the github issue number in my commits

I did not include any unit tests for this fix because there are none for these internal extension methods.

I amended the commit; so now the method only queries the dictionary once, and only casts to a string once. Also, string interpolation (if that's acceptable).


Add additional properties for Fuel Gauge

Fuel Gauge mode, status, charge current, charge voltage, design capacity and design voltage are added.

The twister failures look real:
INFO    - 313/544 native_posix              tests/drivers/fuel_gauge/sbs_gauge/drivers.sbs_gauge_new_api FAILED Failed (native 0.551s)
INFO    - /__w/zephyr/zephyr/twister-out/native_posix/tests/drivers/fuel_gauge/sbs_gauge/drivers.sbs_gauge_new_api/handler.log
ERROR   - [00:00:00.000,000] <inf> emul: Registering 1 emulator(s) for i2c@100
[00:00:00.000,000] <inf> i2c_emul_ctlr: Register emulator 'sbs_gauge@b' at I2C addr 0b

*** Booting Zephyr OS build v3.3.0-rc2-162-ga827b65086b6 ***
Running TESTSUITE sbs_gauge_new_api
===================================================================
START - test_get_all_props_failed_returns_negative
 PASS - test_get_all_props_failed_returns_negative in 0.000 seconds
===================================================================
START - test_get_props__returns_ok
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x9 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0xa = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0xb = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x8 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0xe = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x13 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x11 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0xf = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x10 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x17 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x0 = 0x0
[00:00:00.000,000] <err> sbs_sbs_gauge: Unknown register 0x3 read
[00:00:00.000,000] <err> sbs_gauge: Unable to read register
[00:00:00.000,000] <err> sbs_sbs_gauge: Unknown register 0x14 read
[00:00:00.000,000] <err> sbs_gauge: Unable to read register
[00:00:00.000,000] <err> sbs_sbs_gauge: Unknown register 0x15 read
[00:00:00.000,000] <err> sbs_gauge: Unable to read register
[00:00:00.000,000] <err> sbs_sbs_gauge: Unknown register 0x16 read
[00:00:00.000,000] <err> sbs_gauge: Unable to read register
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x18 = 0x1
[00:00:00.000,000] <inf> sbs_sbs_gauge: read 0x19 = 0x1

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/fuel_gauge/sbs_gauge/src/test_sbs_gauge.c:225: sbs_gauge_new_api_test_get_props__returns_ok: (props[i].status is non-zero)
Property 11 getting 15 has a bad status.
 FAIL - test_get_props__returns_ok in 0.000 seconds

The twister failures look real:

Yes. They are. Hopefully just fixed them.

@albertofloyd


Display hydrogen in atom label for molecules in macromolecules mode

Background
Currently labels for molecules atoms do not contain hydrogens as it is done in molecules mode.
Molecules mode:

Macromolecules mode:

Solution
It needs to apply same rules for hydrogens in atom labels as in molecules mode.
Rules:
TBD

Tested.

Desktop:

OS: Windows 11
Browser Chrome
Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.27.0-rc.1]
Indigo version
[Version 1.26.0-rc.1]


Spotify segfaults because of gdk-pixbuf

Issue description
Spotify segfaults upon start since b9d77e9f518cb2986c093ee53250ec284d855c8a introduced in #45950, as determined by git bisect.
strace shows that it has to do something with gdk-pixbuf locales:
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en_US.UTF-8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en_US.utf8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en_US/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en.UTF-8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en.utf8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/zz5hx05f0kbjcrfx8w0yywyrmy49l4pr-gdk-pixbuf-2.38.0/share/locale/en/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)

cc: @jtojnar @edolstra @timokau @ftrvxmtrx @sheenobu @laMudri
Steps to reproduce
Run spotify.
Technical details

system: "x86_64-linux"
host os: Linux 4.18.0, NixOS, 18.09.2078.54e78e87b09 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.2078.54e78e87b09, nixos-unstable-19.03pre166987.bc41317e243"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Probably the same issue https://github.com/NixOS/nixpkgs/issues/54278. There has been some change in gdk-pixbuf loaders ABI, resulting in crashes when it picks up loader from the environment built for older gdk-pixbuf.
You can avoid it by adding wrapGAppsHook to the package’s nativeBuildInputs, which will set the path to loaders correctly.

Unfortunately adding wrapGAppsHookto nativeBuildInputs does not seem to fix the problem.

The gdk-pixbuf is used for loading icons by GTK.

Finally, it seems the spotify segfault could be unrelated, or at least, the strace should show it loading a 2.36 object, not failing to open locale files.

I do not think @hyperfekt choose the right segment of the strace. If there is no further output before the crash, it is probably due to output buffer not being flushed.

I'm not experienced with debugging stuff with strace, and while I don't think the problem is unflushed output, you are right that the cause seems to be related to loading 2.36.12. Here is a longer part of the trace:
stat("/nix/store/h7mxmd68k47xh52fskkyqi8cigfsjv0d-spotify-<IP_ADDRESS>.gf6bc1b6b-12/share/spotify/icons/spotify-linux-512.png", {st_mode=S_IFREG|0444, st_size=26124, ...}) = 0
openat(AT_FDCWD, "/nix/store/h7mxmd68k47xh52fskkyqi8cigfsjv0d-spotify-<IP_ADDRESS>.gf6bc1b6b-12/share/spotify/icons/spotify-linux-512.png", O_RDONLY) = 71
fcntl(71, F_SETFD, FD_CLOEXEC)          = 0
fstat(71, {st_mode=S_IFREG|0444, st_size=26124, ...}) = 0
read(71, "\211PNG\r\n\32\n\0\0\0\rIHDR\0\0\2\0\0\0\2\0\10\6\0\0\0\364x\324"..., 26124) = 26124
close(71)                               = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
openat(AT_FDCWD, "/nix/store/yv5wznpjrfb21dl8121cg90qijs1whhv-gdk-pixbuf-loaders.cache", O_RDONLY) = 71
fstat(71, {st_mode=S_IFREG|0444, st_size=4488, ...}) = 0
read(71, "# GdkPixbuf Image Loader Modules"..., 1024) = 1024
read(71, "aders/libpixbufloader-icns.so\"\n\""..., 1024) = 1024
read(71, "q3wylygi-gdk-pixbuf-2.36.12/lib/"..., 1024) = 1024
read(71, " \" 100\n\" \\001\\t\" \"x  \" 100\n\"  \\0"..., 1024) = 1024
read(71, "0/2.10.0/loaders\n#\n\"/nix/store/a"..., 1024) = 392
read(71, "", 1024)                      = 0
close(71)                               = 0
stat("/nix/store/nr0wgqv4wbh9x3sll5abhr1mq3wylygi-gdk-pixbuf-2.36.12/lib/gdk-pixbuf-2.0/2.10.0/loaders/libpixbufloader-png.so", {st_mode=S_IFREG|0555, st_size=32992, ...}) = 0
openat(AT_FDCWD, "/nix/store/nr0wgqv4wbh9x3sll5abhr1mq3wylygi-gdk-pixbuf-2.36.12/lib/gdk-pixbuf-2.0/2.10.0/loaders/libpixbufloader-png.so", O_RDONLY|O_CLOEXEC) = 71
read(71<EMAIL_ADDRESS>832) = 832
fstat(71, {st_mode=S_IFREG|0555, st_size=32992, ...}) = 0
mmap(NULL, 2121784, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 71, 0) = 0x7f93745f2000
mprotect(0x7f93745f7000, 2097152, PROT_NONE) = 0
mmap(0x7f93747f7000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 71, 0x5000) = 0x7f93747f7000
close(71)                               = 0
mprotect(0x7f93747f7000, 4096, PROT_READ) = 0
mmap(NULL, 1052672, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f93744f1000
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en_US.UTF-8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en_US.utf8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en_US/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en.UTF-8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en.utf8/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/nix/store/rk26pyb7qan47q27qikdhhi8p7jyswq2-gdk-pixbuf-2.38.0/share/locale/en/LC_MESSAGES/gdk-pixbuf.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
futex(0x7f93971c2f78, FUTEX_WAKE_PRIVATE,<PHONE_NUMBER>) = 0
--- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_MAPERR, si_addr=0x20} ---
+++ killed by SIGSEGV +++

I invite people who actually know what they're doing to do nix-shell -p spotify themselves and take a closer look.

Try coredumpctl gdb and then bt (if you have systemd.coredump.enable in your configuration.nix)

Looks like this made it to 19.03 stable, I just did a normal upgrade rebuild and it started crashing for me :( I went ahead and got the backtrace you asked for @jtojnar:
Backtrace
[nix-shell:~]$ coredumpctl gdb 4050
           PID: 4050 (.spotify-wrappe)
           UID: 1000 (riley)
           GID: 100 (users)
        Signal: 11 (SEGV)
     Timestamp: Mon 2019-08-05 00:28:15 EDT (30s ago)
  Command Line: /nix/store/k9s5qhafrjzhi57wbh7dni8xax96g2gd-spotify-<IP_ADDRESS>.gf6bc1b6b-12/share/spotify/.spotify-wrapped
    Executable: /nix/store/k9s5qhafrjzhi57wbh7dni8xax96g2gd-spotify-<IP_ADDRESS>.gf6bc1b6b-12/share/spotify/.spotify-wrapped
 Control Group: /user.slice/user-1000.slice/session-1.scope
          Unit: session-1.scope
         Slice: user-1000.slice
       Session: 1
     Owner UID: 1000 (riley)
       Boot ID: 72b70e56ccdb4e13a62e502eeed0c096
    Machine ID: 82240612c9de45ec8e28a61059221a64
      Hostname: carbide
       Storage: /var/lib/systemd/coredump/core.\x2espotify-wrappe.1000.72b70e56ccdb4e13a62e502eeed0c096.4050.1564979295000000.lz4
       Message: Process 4050 (.spotify-wrappe) of user 1000 dumped core.

GNU gdb (GDB) 8.2.1
Copyright (C) 2018 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-unknown-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
    <http://www.gnu.org/software/gdb/documentation/>.

For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from /nix/store/k9s5qhafrjzhi57wbh7dni8xax96g2gd-spotify-<IP_ADDRESS>.gf6bc1b6b-12/share/spotify/.spotify-wrapped...
warning: Loadable section ".dynstr" outside of ELF segments
(no debugging symbols found)...done.

warning: core file may not match specified executable file.
[New LWP 4070]
[New LWP 4052]
[New LWP 4078]
[New LWP 4056]
[New LWP 4058]
[New LWP 4060]
[New LWP 4059]
[New LWP 4065]
[New LWP 4068]
[New LWP 4127]
[New LWP 4062]
[New LWP 4076]
[New LWP 4066]
[New LWP 4128]
[New LWP 4075]
[New LWP 4063]
[New LWP 4073]
[New LWP 4077]
[New LWP 4130]
[New LWP 4069]
[New LWP 4074]
[New LWP 4123]
[New LWP 4079]
[New LWP 4064]
[New LWP 4071]
[New LWP 4122]
[New LWP 4131]
[New LWP 4129]
[New LWP 4072]
[New LWP 4055]
[New LWP 4061]
[New LWP 4050]
[New LWP 4057]
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/nix/store/681354n3k44r8z90m35hm8945vsp95h1-glibc-2.27/lib/libthread_db.so.1".
Core was generated by `/nix/store/k9s5qhafrjzhi57wbh7dni8xax96g2gd-spotify-<IP_ADDRESS>.gf6bc1b6b-12/sha'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x0000000000000000 in ?? ()
[Current thread is 1 (Thread 0x7f3304ff9700 (LWP 4070))]
warning: File "/nix/store/hlnxw4k6931bachvg5sv0cyaissimswb-gcc-7.4.0-lib/lib/libstdc++.so.6.0.24-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
To enable execution of this file add
	add-auto-load-safe-path /nix/store/hlnxw4k6931bachvg5sv0cyaissimswb-gcc-7.4.0-lib/lib/libstdc++.so.6.0.24-gdb.py
line to your configuration file "/home/riley/.gdbinit".
To completely disable this security protection add
	set auto-load safe-path /
line to your configuration file "/home/riley/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
	info "(gdb)Auto-loading safe path"
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x0000000001bcd198 in ?? ()
#2  0x0000000001bcd055 in ?? ()
#3  0x0000000001bcd20e in ?? ()
#4  0x0000000001bc9f07 in ?? ()
#5  0x0000000001b9a03c in ?? ()
#6  0x0000000001b9fb78 in ?? ()
#7  0x0000000001b7e43a in ?? ()
#8  0x0000000001b7e166 in ?? ()
#9  0x0000000001b5ce7d in ?? ()
#10 0x0000000001b5e963 in ?? ()
#11 0x0000000001a185ae in ?? ()
#12 0x0000000001a13479 in ?? ()
#13 0x0000000001a1a89b in ?? ()
#14 0x00000000010c5340 in ?? ()
#15 0x00000000010c5282 in ?? ()
#16 0x00000000010c516f in ?? ()
#17 0x00000000010c4e93 in ?? ()
#18 0x00000000010c4d15 in ?? ()
#19 0x0000000001c0c0eb in ?? ()
#20 0x0000000001c0c2fa in ?? ()
#21 0x0000000002074a53 in ?? ()
#22 0x00000000020718d4 in ?? ()
#23 0x000000000183887e in ?? ()
#24 0x000000000183f69b in ?? ()
#25 0x000000000183f4b4 in ?? ()
#26 0x000000000110ce30 in ?? ()
#27 0x000000000110cb65 in ?? ()
#28 0x000000000110caa6 in ?? ()
#29 0x000000000208756c in ?? ()
#30 0x00007f333d37eef7 in start_thread () from /nix/store/681354n3k44r8z90m35hm8945vsp95h1-glibc-2.27/lib/libpthread.so.0
#31 0x00007f333c71a22f in clone () from /nix/store/681354n3k44r8z90m35hm8945vsp95h1-glibc-2.27/lib/libc.so.6
(gdb)

@rileyinman Does unsetting GDK_PIXBUF_MODULE_FILE environment variable allow you to run Spotify?

@jtojnar That variable isn't set in my environment as far as I can tell, and running env GDK_PIXBUF_MODULE_FILE= spotify doesn't have any different effects

@rileyinman Then this is a different issue. Unfortunately, the backtrace is not very useful to tell what is causing your problem.

Darn :/ Is there any other kind of info I can provide that would help?

I believe this is no longer an issue.


Yarn 1338

https://issues.apache.org/jira/browse/YARN-1338

Can one of the admins verify this patch?

/tmp mount point should allow program executions, the mount flag 'noexec' should NOT be present in fstab


Avoid looping with two mutually exclusive desired state

Given a CRD that allows to set a value on a given key in a config map and an operator using informer as event source for the secondary resource (configmap)
When those two CRs
spec:
  configmap: my-config
  key: fruit
  value: banana

spec:
  configmap: my-config
  key: fruit
  value: apple

are accidentally deployed, the operator loops and reconciliations of each CR are fighting to set their value. This cause an heavy load on the k8s api.
The current version of my operator is based on an old version of the framework and basically use a timed reconciliation with withReschedule. The targetted config map was only flickering every minutes or so.
I would like to refactor and use informer, but need a circuit breaker when thing goes wrong. For eg, limiting the number of reconciliation after a burst is detected for a given period of time.
This should also ensure a reonciliation will always occured at the end of this "peace" period of time even if no new event happened, This is because if I removed one of the conflicting CR, the remaining one may not be the one that win the last fight.
While it should be possible to implement skiping in the reconcile method and have a reconciliationMaxInterval, I think that maybe https://github.com/java-operator-sdk/java-operator-sdk/issues/1281 could help, or maybe have a throttled event source wrapping the informer.
Any idea ?

Yes, I think that this scenario should be covered by #1281 as the SDK should never DoS the API server even in the presence of badly designed operators and/or user error.


Add CI to publish configurations

Codecov Report

Merging #4 (1c25506) into main (98bfb80) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           16        16
  Branches         2         2
=========================================
  Hits            16        16

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

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 98bfb80...1c25506. Read the comment docs.


example bag data is 404

Fixed! Thanks for letting me know. For some reason, the previous link was expired.