Issues with loading and viewing openstreetmap tiles : Failed to load resource: the server responded with a status of 403 ()

As I described in this StackOverflow question : https://stackoverflow.com/questions/71861153/leaflet-suddenly-im-getting-this-error-message-failed-to-load-resource-the?noredirect=1#comment126988321_71861153
I'm experiencing issues with loading and viewing openstreetmap tiles  :
Failed to load resource: the server responded with a status of 403 ()
Based on the info here : https://operations.osmfoundation.org/policies/tiles/ the following are the requirements for the openstreetmap tiles usage:
Requirements

    Heavy use (e.g. distributing an app that uses tiles from openstreetmap.org) is forbidden without prior permission from the [Operations Working Group](https://wiki.osmfoundation.org/wiki/Operations_Working_Group). See below for alternatives.
    Clearly display [license](https://wiki.openstreetmap.org/wiki/License) attribution.
    Do not actively or passively encourage copyright infringement.
    Calls to /cgi-bin/export may only be triggered by direct end-user action. (For example: “click here to export”.) The export call is an expensive (CPU+RAM) function to run and will frequently reject when server is under high load.
    Recommended: Do not hardcode any URL at tile.openstreetmap.org as doing so will limit your ability to react quickly if the service is disrupted or blocked.
    Recommended: add a link to https://www.openstreetmap.org/fixthemap to allow your users to report and fix problems in our data.

Technical Usage Requirements

    Valid [HTTP User-Agent](http://en.wikipedia.org/wiki/en:User_agent) identifying application. Faking another app’s User-Agent WILL get you blocked.
    If known, a valid [HTTP Referer](http://en.wikipedia.org/wiki/en:HTTP_Referer).
    DO NOT send no-cache headers. (“Cache-Control: no-cache”, “Pragma: no-cache” etc.)
    Cache Tile downloads locally according to HTTP Expiry Header, alternatively a minimum of 7 days.
    Maximum of 2 download threads. (Unmodified web browsers’ download thread limits are acceptable.)

Note: modern web browsers in standard configuration generally pass all the above technical requirements.

As far as I understand, I do respect all of them
Here you can find a reproducible live test case:  https://github.com/raphael10-collab/LeafletElectronPlaying
Steps to reproduce:

git clone https://github.com/raphael10-collab/LeafletElectronPlaying.git
cd LeafletElectronPlaying
yarn
in one screen: yarn type-check
in another screen: yarn start

An Electron Window will open
What perplex me is that this happens only for the Basic Map Layer:

With the Topo Map Layer the visualization seems fine :

How to solve the issue?
Looking forward to your kind help

Hi, and thanks for taking the time to open a bug report in Leaflet.
However, in this repository we only handle bugs in "vanilla" Leaflet. This means that we do not handle bugs which are specific to frameworks such as:

React
Ractive
AngularJS
Bootstrap
Leaflet for R
Joomla
Wordpress
Ionic
Cordova
Vue
Electron
Polymer
TypeScript
D3
Folium
NativeScript
...etc

Please understand that we only have the time and energy to test Leaflet in plain web browsers.
Please try to either reproduce the bug without using any frameworks, or submit a bug to the appropriate repo.

Hi @IvanSanchez ! I'm going to ask clues about this specific issue in thereact-leaflet github repo.
If they say that it's a problem related to leaflet I will have to come back here


utils: fix bug when decoding

utils._decode should return an instance of 'str' instead of 'bytes' in
Python 3.
Fixes: #24

Codecov Report

Merging #25 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   87.26%   87.26%
=======================================
  Files           7        7
  Lines         314      314
  Branches       33       33
=======================================
  Hits          274      274
  Misses         25       25
  Partials       15       15

Impacted Files
Coverage Δ

etcd3gw/utils.py
78.37% <100%> (ø)
:arrow_up:

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 8fccb67...ab64cff. Read the comment docs.

Thanks @huntxu


Request: Text Rotation Option

Please, if possible, add this option, which would greatly simplify our work.

I just implemented rotation of text items. Or do you mean something else?


Featured communities component

Create a slideshow component for featured communities, similar to the one in Zenodo:

Related to:

https://github.com/inveniosoftware/invenio-app-rdm/issues/1932 and
https://github.com/zenodo/zenodo-rdm/issues/47

@jennur see comment from Alex here: does this follow his solution?


Fix all the kubernetes manifests for nexodus

agreed that remaining comments can be addressed as a follow-up


README not complete?

In section "Cache"

The nativescript-fresco {N} plugin has built-in cache mechanism which handles managing the images in the memory. There are two types of cache mechanisms memory and disk, you can manually manage both of them with the following functionality.

Which following functionality?

Hi @anki247,
The following sections describe the different ways you can affect the build-in fresco cache mechanism:

'Refresh' the 'imageUri'
Clear everything from the cache
Evict all images with a specific URI from the cache

[close]


Error on launch.

Got an error:
Traceback (most recent call last):   File "/usr/local/bin/proxybroker", line 11, in <module>     load_entry_point('proxybroker==0.2.0b3', 'console_scripts', 'proxybroker')()   File "/usr/local/lib/python3.4/dist-packages/pkg_resources/__init__.py", line 561, in load_entry_point     return get_distribution(dist).load_entry_point(group, name)   File "/usr/local/lib/python3.4/dist-packages/pkg_resources/__init__.py", line 2631, in load_entry_point     return ep.load()   File "/usr/local/lib/python3.4/dist-packages/pkg_resources/__init__.py", line 2291, in load     return self.resolve()   File "/usr/local/lib/python3.4/dist-packages/pkg_resources/__init__.py", line 2297, in resolve     module = __import__(self.module_name, fromlist=['__name__'], level=0)   File "/usr/local/lib/python3.4/dist-packages/proxybroker/__init__.py", line 28, in <module>     from .proxy import Proxy   File "/usr/local/lib/python3.4/dist-packages/proxybroker/proxy.py", line 35     async def create(cls, host, *args, **kwargs):         ^ SyntaxError: invalid syntax
What might be the reason?

README.rst =
Requirements
Python 3.5 or higher


Submit Tellurium 2.2.8

Simulation tool

id: tellurium
version: 2.2.8
specificationsUrl: https://raw.githubusercontent.com/biosimulators/Biosimulators_tellurium/139ab61be949ca54f52103293771d083ad8020bc/biosimulators.json
specificationsPatch:
version: 2.2.8
image:
url: ghcr.io/biosimulators/biosimulators_tellurium/tellurium:2.2.8
digest: "sha256:e739d499c143d8fe7455e981c088dd76b660af09bae9b1cf3ad824f671d43c44"
validateImage: true
commitSimulator: true

Support for BioSimulators' conventions

[X] The simulation tool is provided as a Docker image
[X] The ENTRYPOINT of the image is a BioSimulators-compliant command-line interface
[X] This command-line interface supports the COMBINE/OMEX archive format
[X] This command-line interface supports the Simulation Experiment Description Markup Language (SED-ML) format
[X] This command-line interface supports the Hierarchical Data Format (HDF5)
[X] The capabilities of the tool are described using BioSimulators' format
[X] The modeling frameworks supported by the tool are described using Systems Biology Ontology (SBO) terms
[X] The algorithms supported by the tool are described using Kinetic Simulation Algorithm Ontology (KiSAO) ontology terms
[X] The model formats supported by the tool are described using EMBRACE Data and Methods (EDAM) ontology terms
[X] The licenses supported by the tool are described using Software Package Data Exchange (SPDX) identifiers
[ ] The source code for the simulation tool is publicly available
[ ] The simulation tool is available through a software repository such as CRAN or PyPI
[ ] The simulation tool is registered with bio.tools
[ ] The Docker image provides Open Containers Initiative-compliant metadata
[ ] The Docker image provides BioContainers-compliant metadata
[ ] The Docker image is registered with BioContainers

Trying to resubmit this, since the original automatic submission failed.


Cardano-rosetta Docker Restart

Due to node reboot, after restart the node and restarted cardano-rosetta container.
The cardano-node seems won't sync anymore.
using /network/status and check the {"current_block_identifier": {"index" the block height won't increase anymore.
Is it this is a bug or anyway to solve this? Thank you.

Just found out need to wait around 20 mins for delete a certain old blocks and start continue sync again.


Ruff linter

This adds configuration of ruff linter to pyproject.toml and .pre-commit-config.yaml
This depends on #81, because ruff configuration is supposed to work with black. And it also requires #109 for pre-commit support
Fixes #82

I need to redo it to remove dependency on black (rejected PR #110) and remove all autofixes

The team doesn't like it


drawsvg:

Hello, I am going through the tutorials, and I keep getting an error when trying to create an SVG image from a SMILES string (this example is from ":gettingStarted.ipynb").
/home/brennan/.julia/artifacts/3f97eaf13c6af212b4374abe67b1f0a327305434/lib/libcoordgen.so: undefined symbol: getSketcherMinimizer

Stacktrace:
 [1] coordgen!(::GraphMol{SmilesAtom,SmilesBond}) at /home/brennan/.julia/packages/MolecularGraph/uylBq/src/coordgen.jl:13
 [2] draw2d!(::SvgCanvas, ::GraphMol{SmilesAtom,SmilesBond}; setting::Dict{Symbol,Any}, recalculatecoords::Bool) at /home/brennan/.julia/packages/MolecularGraph/uylBq/src/draw/draw2d.jl:193
 [3] draw2d! at /home/brennan/.julia/packages/MolecularGraph/uylBq/src/draw/draw2d.jl:192 [inlined]
 [4] #drawsvg#139 at /home/brennan/.julia/packages/MolecularGraph/uylBq/src/draw/svg.jl:107 [inlined]
 [5] drawsvg(::GraphMol{SmilesAtom,SmilesBond}, ::Int64, ::Int64) at /home/brennan/.julia/packages/MolecularGraph/uylBq/src/draw/svg.jl:106
 [6] top-level scope at In[6]:4```

Thank you for reaching out.
The error may be due to linking error of coordgenlib binary package.
Can you provide more information about your environment?

OS version
Julia version
MolecularGraph.jl version

Please check if necessary packages are correctly installed. Open Julia REPL, move to Pkg mode and type  status to show installed packages.
If you have upgraded from previous versions, rm MolecularGraph and add MolecularGraph will work.

I'm sorry that notebook tutorials are not up-to-date, and some expressions in the tutorial may no longer work.
I'll work on it soon.

Thanks for getting back to me. I tried opening up jupyter notebook in ~/Documents/MolecularGraph.jl_notebook, then changed Pkg.activate("..") to Pkg.activate("."), and that seemed to make everything work. I am using Julia 1.4.1, and MolecularGraph version 0.6.0. Ubuntu 20.04 LTS.


Updated to Ranch 2.0

@savonarola This seems to be a serious change, possibly breaking (for projects using other libraries with Ranch as dependency). Please consider versioning as a major release?
Call to all SmppEx users: your feedback about SmppEx with Range 2 in production (or in high traffic tests) would be very useful :-)

@savonarola This seems to be a serious change, possibly breaking (for projects using other libraries with Ranch as dependency). Please consider versioning as a major release?

Hello!
Thank you for the interest. It will definitely be a major release.


how to print the intermediate variables

Since i want to do some adjustments in RPNHead, but i can't even print strings(mush less saving Tensor) after i add print("###") in rpn_head.py or even in apis/train.py when i training faster_rcnn using the example_configs.

But got nothing "##" out in the log when the losses appeared

Great, i need to pip reinstall mmdet
Thx~


Add SSO session config support

At the very least it needs to support searching the SSO sessions in the config for instance information. https://github.com/boto/botocore/blob/5958cc82edee760308425fbe3e09bc73466b1766/botocore/tokens.py#L173
Also aws-sso-util configure should have support for writing sessions and using them in profiles

At the very least it needs to support searching the SSO sessions in the config for instance information.
I'm still waiting for this.

@ikeyan just just submitted a PR to read the config in #122.
@khuongduybui , would you be able to test @ikeyan's branch?


Site Admin can create Users and are assigned to their Sites

@mahdibenromdhane there are too many isXXX I don't like which should be resolved by the new auth framework.
We need to discuss this and first implement the new Auth properly in the backend and next in the front end.


Application Insights Search does not show debug session telemetry without ApplicationInsights.config

I've noticed that if the project does not contain a file called ApplicationInsights.config, the Application Insights Search does not show any data from “Debug session telemetry”.
Adding the file ApplicationInsights.config (even an empty one) solves the problem - but obviously it’s counter intuitive to users.
Bug submitted here https://developercommunity.visualstudio.com/content/problem/740343/application-insights-search-does-not-show-debug-se.html

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

ID: 09c3a89a-f9bc-1684-c007-376a9ef46bcd
Version Independent ID: cdc06a4f-c8fc-69a6-86c7-136cd9229f48
Content: Debug applications with Azure Application Insights in Visual Studio
Content Source: articles/azure-monitor/app/visual-studio.md
Service: application-insights
GitHub Login: @NumberByColors
Microsoft Alias: daviste

@ACsharp Thank you for bringing this to our attention and sharing the feedback, will get back to you shortly on this. Thanks

@ACsharp Thanks again for reaching out, can you please help to elaborate your scenario which is resulting into this issue.
Do you have the SDK installed already with in the project and you are your are not seeing the "Debug session telemetry" , do you have appsettings.json with the application insights instrumentation key in the project , instead of the applicatoninsights.config file.

@BharathNimmala-MSFT the project references the latest AppInsights NuGet package and includes instrumentation code to send traces to AppInsights.
In the example I'm using, the instrumentation key is hardcoded but in a real customer scenario, it would be either in a custom configuration file or in an environment variable.
Please note that the code sends telemetry successfully to the live AppInsights service.
The issue only happens when using the Visual Studio AppInsights Log Search window AND using "debug session telemetry" as a source. In this configuration the Log Search window is supposed to intercept AppInsights telemetry from the running process rather than the live AppInsights service. Unfortunately that happens only if the project contains ApplicationInsight.config (even an empty one).
The customer reports that the file wasn't required before (although they weren't able to specify the specific version where this was working).

@ACsharp, could you please share how you configure ApplicationInsights in the code?

@lmolkova That's the example code:
            TelemetryConfiguration config = new TelemetryConfiguration()
            {
                InstrumentationKey = "my key"
            };

            var tc = new TelemetryClient(config);
            tc.TrackTrace("my trace text");

In the real customer scenario the key wouldn't be hard coded but it would instead be either in a custom configuration file or in an environment variable.
I'd like to stress that the way ApplicationInsights is configured  in the code should be irrelevant in this case. When configured to get telemetry from debug session, VS AppInsight Search taps into the running process directly. Evidence of that is that if we add an empty ApplicationInsight.config, everything works fine.

@AndreaC-MSFT thanks for the repro steps. This is a VS AppInsights search issue, I have forwarded it to the responsible team.

@AndreaC-MSFT  Thank you again for bringing this to our notice and this issue has been acknowledged by the product team and being worked upon. Since this issue is also tracked in Visual Studio Dev Community forum , closing the Github Issue. Please feel free to revert back if you have any further queries. Thank you

A valid scenario where you wouldn't have an ApplicationInsights.config file is when you are developing a Function app. All documentation indicates you just need to set APPINSIGHTS_INSTRUMENTATIONKEY in your your host.json (or local.settings.json) file. While this should work for actual App Insights, no data is available in Visual Studio. Adding an empty ApplicationInsights.config file does resolve the issue.

Confirmed that is not fixed. Adding an empty file is what I needed to do in order to see telemetry events for my function app.

The workaround (adding an empty ApplicationInsights.config file) did not work for me in VS 2022. Adding the following to the .csproj file of the project being debugged
  <PropertyGroup>
    <!-- Needed for Visual Studio to "see" local App Insights data in debug output. -->
    <ApplicationInsightsResourceId>dummy-value</ApplicationInsightsResourceId>
  </PropertyGroup>

is the only thing that has worked to get App Insights data to show up from the debugger.

This SO post shows others having the same problem and what has worked:
https://stackoverflow.com/questions/54772078/application-insights-from-last-debug-session/72063038#72063038

Still broken by default.


ContextFlyout throw exception in a GridView with Template in v1.4

Describe the bug
i am using a ContextFlyout in my DataTemplate:
 <GridView.ItemTemplate>
     <DataTemplate>
         <Grid>
             <Grid.ContextFlyout>
                 <MenuFlyout>
                     <MenuFlyoutItem Text="Open Image (Photo App)" />

Right clicking on the item to open the menu will result in an error.
The app works without issues in version 1.3.3, after updating to v1.4-preview2 causes crashes
Steps to reproduce the bug
create app with ContextFlyout in datatemplate
Expected behavior
No response
Screenshots
No response
NuGet package version
Windows App SDK 1.4 Preview 2: 1.4.230811000-preview2
Packaging type
Packaged (MSIX), Unpackaged
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

I currently can't repro this issue. Is there more required to hit this? Can you share a callstack of the exception?

Hi @codendone
sorry for late response.
I removed the following option from csproj and it seems to have solved the problem.
<AccelerateBuildsInVisualStudio>True</AccelerateBuildsInVisualStudio>


Change vibratoSpeed to be a frequency

The idea is to change the vibratoSpeed to be a frequency instead of a float between 0 and 1.
Originally posted by @Tiedye in https://github.com/wintergatan-community/vmmx-schema/pull/17

As it stands with the machine, it's tied to tempo, not an absolute frequency. I agree that 0-1 without any unit doesn't make much sense, but I think we still want it to be relative.
Maybe a ratio between the vibrato shaft and the main crankshaft (which is one revolution per quarter note) would do.


numbers: thousands separator

Hey thanks for the nice cheat sheet!
You're missing the optional thousands separator for numbers
'{:,}'.format(1000000)
Output: 1,000,000

That was an intentional omission since that is unfortunately not locale aware. However mentioning it with a disclaimer of that fact could be useful.


Make arrow keys work on photo browser zoom

https://forum.inaturalist.org/t/arrow-keys-to-move-between-images-in-browse-photos/42258
On both taxon pages and taxon photo browser pages (eg https://www.inaturalist.org/taxa/55048-Trichodes-apiarius and https://www.inaturalist.org/taxa/55048-Trichodes-apiarius/browse_photos), you can click on a photo to zoom in. On each side of the photo are arrows you can click on to go to the next photo:

We should make it so the right and left arrow keys can also move between photos.

Just to copy over what's on the forum -- the arrow keys used to work (implemented in 092007d), I'm guessing they broke with 543b39f


[BEAM-4145] Populate the worker_id metadata in the Java SDK Harness

This is a take-over of #5456 since Thomas is currently unavailable.
I took the PR verbatim, rebased it / resolved conflicts, and addressed the review comments:

Added a comment about lifetime of logging service.
To "We are loosing the original list of interceptors here." - I think it's conventional to have a withFoo(List) method overwrite the list rather than append, so no action on this one.

R: @angoenka

retest this please

Run Java PreCommit

Test passed (but didn't propagate status to Github), PTAL.

Thanks!


fuzzing: create structured tar bytes in archive fuzzer

TarBytes has undergone some work with a few more improvements which makes it more suitable for the archive fuzzer.
@kzys
Signed-off-by: AdamKorcz<EMAIL_ADDRESS>

We may want to merge this after https://github.com/containerd/containerd/pull/7474 since the new Go version should contain fixes to archive/tar.

They also included a test-tar in their repository (src/archive/tar/testdata/pax-bad-hdr-large.tar.bz2); https://github.com/golang/go/commit/4fa773cdefd20be093c84f731be7d4febf5536fa


Cannot deploy Stack when using custom registry

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
When deploying a Stack on a standalone Docker instance (I haven't tested swarm), the deployment fails if one of the containers in the docker compose file uses an image that is in a private authenticated registry, even if that registry has been added to Portainer.
Expected Behavior
I should be able to deploy a Stack whose Compose file includes a container which image is in a private, authenticated registry.
Actual Behavior
When attempting to deploy the Stack, I get the error:
pull access denied for <registry host>:<registry port><registry path> (ex. example.com:5050/group/project), repository does not exist or may require 'docker login': denied: requested access to the resource is denied

I can however, pull the image in the Images area of Portainer.
Steps to Reproduce

Navigate to Settings > Registries.
Add a registry, either GitLab or Custom (that's all I've tested with).  Use credentials that have access to the registry of the project you want to deploy.
In a standalone Docker instance, go the Images tab and pull in the image (just to prove that the image exists and Portainer should have access to it).
Add a new stack to a standalone Docker instance.  Using either the web editor or a Repository, define a Compose file like the following:

services:
  web:
    image: example.com:5050/group/project

Try to deploy the stack.  It will fail.

Portainer logs or screenshots
No response
Portainer version
2.19.5
Portainer Edition
Community Edition (CE)
Platform and Version
Docker 24.0.7
OS and Architecture
Ubuntu 22.04.3 LTS x86_64
Browser
Firefox 128.0.3
What command did you use to deploy Portainer?
No response
Additional Information
No response

Hello @JordanBarnartt The FQDN of the registry you are using in your Stack deployment must match exactly to the FQDN you have configured for the registry in Portainer. Can you confirm this please

Yes, it does.  For example, if the Stack deployment has a docker-compose file with:
services:
  web:
    image: example.com:5050/group/project

And in the Registries area, we have a GitLab and a Custom registry, both of which are using the same example.com:5050 (as displayed in the URL column).

@JordanBarnartt in the environment where you are deploying the Stack from, under 'Hosts' and then registries, do you see you registry there? Just check it's there and under manage access you have permission to use it.

(I work with Jordan) @Nick-Portainer The registry shows in the environment, but the create access page shows "No users or teams available", but we are both admins .  We are using the community edition.
Actually, I just assigned a team to the environment and then was able to add the team to the registry access.  But our users don't actually show up in the team or being able to add them to the team.  We are using OIDC for authentication.

I also tried creating the stack as the internal admin user, but receive the same error.

@JordanBarnartt Can you share the compose file?

The compose file is:
services:
  web:
    image: git.uwaterloo.ca:5050/ist-soc/eolos:latest
    command: uwsgi --ini /code/config/uwsgi/uwsgi.ini
    restart: always
    volumes:
      - .:/code
    environment:
      - POSTGRES_DB=${POSTGRES_DB}
      - POSTGRES_USER=${POSTGRES_USER}
      - POSTGRES_PASSWORD=${POSTGRES_PASSWORD}
      - POSTGRES_HOST=${POSTGRES_HOST}
    networks:
      - internal
  nginx:
    image: nginx:1.27
    restart: always
    volumes:
      - ./config/nginx:/etc/nginx/conf.d
      - .:/code
    networks:
      - traefik-public
      - internal
    labels:
      - traefik.enable=true
      - traefik.docker.network=traefik-public
      - traefik.http.routers.web.rule=eolos.private.uwaterloo.ca
      - traefik.http.routers.portainer-http.middlewares=https-redirect
      - traefik.http.routers.portainer-https.rule=eolos.private.uwaterloo.ca
      - traefik.http.routers.portainer-https.entrypoints=https
      - traefik.http.routers.portainer-https.tls=true
      - traefik.http.services.portainer.loadbalancer.server.port=80

networks:
  traefik-public:
    external: true
  internal:

Here's a screenshot of the error:

We tested this internally and had no issues. Do you get the same error if you deploy just the git.uwaterloo.ca:5050/ist-soc/eolos:latest without the nginx?

I just tried without nginx there, still the same issue.  The error is slightly different, it doesn't have the line about "tvwt" variable is not set

@Goggin Do you have multiple gitlab or other registries defined in Portainer?

@Nick-Portainer There were multiple, but I removed the second one just not and tried again, still throwing an error.

@Goggin Can you remove all registries and try? If you still have no luck please join our Slack channel and if possible we can jump on a call or message there as it's easier to troubleshoot - https://join.slack.com/t/portainer/shared_invite/zt-2osa3lxsn-QZRAa3IAj~NPeRoP_YcqiA

This issue has been solved.  There was a typo in the registry path in the docker-compose file :(
For what it's worth, this helped us to discover an issue with our proxy configuration, so we likely would have run into problems anyway.
Thank you for your support on this!


Update puzzles table asynchronously (without page reload)

Changes:

Added new /hunts/<hunt_id>/puzzles endpoint used by AJAX to fetch JSON array of puzzle data
Made puzzles table refresh asynchronously every 10 seconds
Removed puzzles_extra.py and puzzles_table.html (combined with all_puzzles.html)

For #115.

Thanks for all your reviews, @daiweili! Much appreciated.


kernel() missing 1 required positional argument: 'XBLOCK'

Getting this issue when trying to optimize a model with torchdynamo - triton
  File "/home/nouamane/projects/torchdynamo/torchdynamo/eval_frame.py", line 166, in _fn
    return fn(*args, **kwargs)
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/functorch/_src/aot_autograd.py", line 957, in forward
    return compiled_f(
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/functorch/_src/aot_autograd.py", line 948, in new_func
    return compiled_fn(args)
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/functorch/_src/aot_autograd.py", line 287, in new_fn
    fw_outs = call_func_with_args(compiled_fw, args)
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/functorch/_src/aot_autograd.py", line 252, in call_func_with_args
    out = normalize_as_list(f(args))
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/functorch/_src/aot_autograd.py", line 230, in g
    return f(*args)
  File "/home/nouamane/projects/torchdynamo/torchinductor/compile_fx.py", line 125, in run
    compiled_fn = cudagraphify_impl(model, new_inputs, static_input_idxs)
  File "/home/nouamane/projects/torchdynamo/torchinductor/compile_fx.py", line 165, in cudagraphify_impl
    model(*inputs)
  File "/tmp/torchinductor_nouamane/h2/ch2xzeq3kuk5pvo2ygx4akvkyewu7w3kzrx6qbjjtunu4attvci4.py", line 7743, in call
    kernel0[grid(320)](arg687_1, buf0, buf1, 320)
  File "/home/nouamane/miniconda3/envs/namo/lib/python3.8/site-packages/triton/runtime/jit.py", line 106, in launcher
    return self.run(*args, grid=grid, **kwargs)
TypeError: kernel() missing 1 required positional argument: 'XBLOCK'

Any idea where could that come from?

Probably a bug in the new runtime. Can you share a minimal reproducible example?

import torch
import torchdynamo
from diffusers import UNet2DConditionModel
torchdynamo.config.verbose = True

mod = UNet2DConditionModel()
optimized_mod = torchdynamo.optimize("inductor")(mod)

sample = torch.randn(2, 4, 64, 64).half().cuda()
timestep = torch.rand(1).half().cuda() * 999
encoder_hidden_states = torch.randn(2, 77, 768).half().cuda()

optimized_mod(sample, timestep, encoder_hidden_states)
print("done")

Fixed for now by installing pip install triton==2.0.0.dev20220830
So it does seem to be related to the new runtime

Would it be possible to get a more minimal repro than that, i.e. just the Triton kernel, like e.g. #599  ? I can help but I don't have the bandwidth to bisect the faulty kernel myself

Can't seem to reproduce this issue anymore, so I'm gonna mark as closed for now. Thanks for your help


Add PR/MR issue templates to pull requests/merge requests automatically

Read Pull Request templates and Merge Request templates and put them into the body of the PR/MR when creating new PRs and MRs.
Grab all files that match the naming pattern set by GitHub/GitLab, and then ask a user which template they'd like to use (or none at all). And then put that template into the body of the PR/MR.
Pull Request templates
I'm pretty sure all pull request templates will fit into one of these options, according to the documentation. They'll all either be in the form .github/PULL_REQUEST_TEMPLATE/*.md, docs/PULL_REQUEST_TEMPLATE/*.md, docs/PULL_REQUEST_TEMPLATE.md, docs/pull_request_template.md, .github/PULL_REQUEST_TEMPLATE.md, .github/pull_request_template.md, pull_request_template.md, or PULL_REQUEST_TEMPLATE.md. Uffda. The following doesn't care about capitalization.
nested_templates = Dir.glob(File.join("**/pull_request_template", "*.md"), File::FNM_DOTMATCH)
non_nested_templates = Dir.glob(File.join("**", "pull_request_template.md"), File::FNM_DOTMATCH)
nested_templates.concat(non_nested_templates)
=> [".github/pull_request_template.md"]

Merge Request templates
I'm pretty sure all GitLab merge request templates must live in the .github/merge_request_templates/*.md directory, according to the documentation.
standard_templates = Dir.glob(File.join("**/merge_request_templates", "*.md"), File::FNM_DOTMATCH)
standard_templates
=> [".gitlab/merge_request_templates/default.md"]

I'd also like to investigate whether the .github/PULL_REQUEST_TEMPLATE/ directories can be lowercased (.github/pull_request_template/), and/or pluralized (.github/pull_request_templates/). On GitLab's side, I'd love to know if I can move the .gitlab-ci.yml file into the .gitlab/ directory.

Okay.... so GitHub and GitLab are both pretty limiting in different ways.
GitHub
Issues
Issue templates need to be defined in .github/ISSUE_TEMPLATE/*. You cannot make the directory lowercase (.github/issue_template/*), and you cannot pluralize it (.github/ISSUE_TEMPLATES/*).
Pull Requests
The use of multiple PR templates in GitHub is pretty sucky (the only way to utilize multiple PR templates is to use the query parameters, so there's no nice choice of template buttons like with Issues. For me, this makes multiple PR templates pretty much impossible. Furthermore, to create a default PR template, the naming scheme must be .github/pull_request_template.md (although you can capitalize this). You can't place the default template in .github/PULL_REQUEST_TEMPLATE/default.md, which would be my preferred method.
Ending solution
I shall use .github/issue_template.md to create a single issue template which will be applied on all issues made through the UI. I can optionally create a script that will create a GitHub issue that will apply any template found in that file or in nested files. This way, the script would work with other peoples' repos, but my layout will just have the single template.
nested_templates = Dir.glob(File.join("**/ISSUE_TEMPLATE", "*.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
non_nested_templates = Dir.glob(File.join("**", "issue_template.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
nested_templates.concat(non_nested_templates)
=> [".github/issue_template.md", ".github/ISSUE_TEMPLATE/bug_report.md"]

The GitHub UI gives me this annoying message:

But I can put up with that.
Use .github/pull_request_template.md to create a single pull request template which will be applied on all PRs made through the UI. Change the pull request script to read that (and nested templates), and offer options to apply to PRs made through the scripts. This way, the script would work with other peoples' multiple layouts, but my repos would just have the single template.
nested_templates = Dir.glob(File.join("**/PULL_REQUEST_TEMPLATE", "*.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
non_nested_templates = Dir.glob(File.join("**", "pull_request_template.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
nested_templates.concat(non_nested_templates)
=> [".github/pull_request_template.md", ".github/pull_request_template/other_template.md", "docs/pull_request_template.md"]

GitLab
Issues
Issue templates need to be defined in .gitlab/issue_templates/*. You cannot uppercase the directory (.gitlab/ISSUE_TEMPLATES/*), and you cannot singularize the directory, although my preference would be to pluralize anyway.
So essentially, the two systems require complete opposite naming patterns.
Similarly to GitHub, issues don't need to follow a template (they can be blank), and a user has to specifically choose to use any of these templates. A true issue default must be set in the project settings.
Merge Requests
I've created .gitlab/merge_request_templates/default.md, which works properly, but you have to explicitly choose for the MR to use that template, versus it just being a default. A true default must be set in the settings of the project. However, I think the use of multiple templates would be better with GitLab's naming scheme than with GitHub's.
Ending solution
For me personally, remove the directories .gitlab/merge_request_templates and .gitlab/issue_templates. Make single default templates and put them in .gitlab/merge_request_template.md and .gitlab/issue_template.md, and then update the gitlab settings to treat those as defaults. Then, make scripts that will work with my repos and with others' repos and look in nested templates:
nested_templates = Dir.glob(File.join("**/merge_request_templates", "*.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
non_nested_templates = Dir.glob(File.join("**", "merge_request_template.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
nested_templates.concat(non_nested_templates)
=> [".gitlab/merge_request_template.md", ".gitlab/merge_request_templates/other_template.md"]

# AND

nested_templates = Dir.glob(File.join("**/issue_templates", "*.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
non_nested_templates = Dir.glob(File.join("**", "issue_template.md"), File::FNM_DOTMATCH | File::FNM_CASEFOLD)
nested_templates.concat(non_nested_templates)
=> [".gitlab/issue_template.md", ".gitlab/issue_templates/bug_report.md"]

Since I'm not sure yet if I want to create scripts to create issues, I'll just close this Issue. I've done the necessary work on the pull/merge request templates.


TypeError: Cannot read properties of undefined (reading 'filter')

posthog e6ecdb60ccdb6a1122abbd07b2bf6d1059f82bc1

This issue happens due to these conditions :

When we want to check the error details from the list we and the error does not exist in the database and we return not found to the UI
When some exception happens during communication between the plug-in and UI it will also lead to returning empty object to UI and crashing it

https://github.com/user-attachments/assets/d11691e1-988e-47d5-9789-e198736cc7df


pull request for e2e test at 01/07/2017 07:45:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


add team and team admin to vars in dbt project

I am working with a client to use Fivetran to integrate with Intercom. It appears that for their instance fivetran does not pull team or team admin from the Intercom data model, presumably this is because they do not have team or team_admin defined as it is included in the Fivetran Sync Table. I was making these changes on their local package to stop dbt run failing and decided to raise this as a PR instead as I saw the release of this repo is recent.
In this PR:

add team and team admin as vars in the dbt_project.yml file
update README.md to mention these new variables
update tmp and stg models which reference team and team admin to link to the var in the project file

I plan to raise a similar change in the Intercom repo too. Let me know if this is helpful. Happy to close the PR and apply this to the company instance instead. 👍

Hi @robertpknight I actually have some time now to dive into these. I will approve and merge your changes! Thanks again for opening this PR and contributing!

Hi @robertpknight I actually have some time now to dive into these. I will approve and merge your changes! Thanks again for opening this PR and contributing!


[Bug][k8s]when recreate or upgrade helm release, the host ip changed

Describe the bug
When DS was deployed to k8s by helm, if you upgrade the release or recreate the release, the ip of worker server and other server will be changed, so when you try to query the log of history task, you will get empty log.
To Reproduce
Steps to reproduce the behavior, for example:

deploy the cluster use helm
recreate the deployed release or just upgrade
query a task which runned before upgrade/recreate
no log will show

Expected behavior
log should be show
Screenshots

Which version of Dolphin Scheduler:
-[1.3.1-release]
Additional context
Requirement or improvement

There are 2 solutions to this problem:

At the end of the task, the log is sent to distributed storage such as S3/HDFS and stored according to taskInstance id. A new LoggerService is added to obtain logs of completed tasks from distributed storage.
Added worker.id configuration in worker.properties, and added workerId field in taskInstance. When getting the log, get the corresponding worker according to the workerId and then get the log.

Which solution is better?

这个问题有2种解决方案：

任务结束时将日志发送至S3/HDFS等分布式存储，按taskInstance id存储。新增一个LoggerService实现从分布式存储获取已完成任务的日志。
worker.properties增加worker.id配置，taskInstance新增workerId字段。获取日志时根据workerId得到对应的worker再获取日志。

哪一种方案更好？


Upgrade to latest CSS 3 spec: CR-css-syntax-3-20140220/

See http://www.w3.org/TR/2014/CR-css-syntax-3-20140220/
This spec is likely to move to PR status by later this year. Some considerations:

implement tests for new sections: 4.1 Token Railroad diagrams, 4.3 Tokenizer Algorithms
also implements test for 5.1 Parser Railroad Diagrams and 5.4 Parser Algorithms
implement An+B microsyntax.
allow floating point numbers: 7.297e−3

CSS::Module::CSS3::Selectors is also impacted:

the nth-functor rules are moved to core css3 syntax
attribute selectors ^=, $= and *= are also moving to core syntax

Work is underway on branch css-syntax-3-20140220

I've merged the branch. This includes a full set of attribute selectors & AnB syntax.
Tests added for 4.1 Token Railroad diagrams.
A lot of the material in the latest spec is a basic simplistic (and incomplete) description for constructing procedural hand-built parsers.


Pass some control limits as part of Signal DataKey

PVA and CA give differently named fields for some common concepts of device limits.

CA
PVA

lower_alarm_limit
valueAlarm.lowAlarmLimit

upper_alarm_limit
valueAlarm.highAlarmLimit

lower_ctrl_limit
control.limitLow

upper_ctrl_limit
control.limitHigh

lower_disp_limit
display.limitLow

upper_disp_limit
display.limitHigh

lower_warning_limit
valueAlarm.lowWarningLimit

upper_warning_limit
valueAlarm.highWarningLimit

There should be a decision on which are passed downstream with the DataKeys of signals, and how they should be called. Or else it should be configurable.
Some of these could find use in e.g. adaptive scanning: knowing the limits of a signal prevents attempting to send it outside of those limits. Others are useful metadata for e.g. NeXus (NXPositioner has soft_limits_{min|max}: but this is Device level metadata, not signal).

Closed by #367


[Documentation] Triplet docs should mention that none or all of VCPKG_lang_FLAGS have to be defined

Describe the bug
Exactly what it says in the title. I am doing bunch of work with custom triplets, and my build failed about an hour in with error message that if either of VCPKG_CXX_FLAGS or VCPKG_C_FLAGS are defined, then the other flags have to be defined as well.
I would prefer to find this out from the documentation :smiley:

This issue is from 2021!


Dynamically loaded html pages

Make it so the documents are loaded asynchronously into the webpage rather than actually navigating to a whole new webpage everytime a link is clicked.

This is done in the latest update


Revise down the nondeterminism tutorial

Description of changes:
The existing nondeterminism tutorial consisted of:

Mentioning how any is safe
Using any_raw which is just asking for undefined behavior.
Using Invariant which... would be undefined behavior on most types.

Ugh. I gave up. I just removed the second two sections and added a little bit more to the first. We have some nasty problems to deal with here that go way beyond writing a tutorial section.
Resolved issues:
Towards #1320
Call-outs:

Testing:

How is this change tested?

Is this a refactor change?

Checklist

[ ] Each commit message has a non-empty body, explaining why the change was made
[ ] Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[ ] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

The is ready for review again. I've revised down the content still, but I also opened this tracking issue:

#1394

I think eventually we should add more here, but there are things in the issue that I think need fixing first. This at least revises this chapter and makes it usable today.


Derive Reflect + FromReflect for input types

Objective
Adds support for reflecting many more of the input types. This allows those types to be used via scripting, bevy-inspector-egui, etc. These types are registered by the InputPlugin so that they're automatically available to anyone who wants to use them
Closes #6223
Solution
Many types now have #[derive(Reflect, FromReflect)] added to them in bevy_input. Additionally, #[reflect(traits...)] has been added for applicable traits to the types.
This PR does not add reflection support for types which have private fields. Notably, Touch and Touches don't implement Reflect/FromReflect.
This adds the "glam" feature to the bevy_reflect dependency for package bevy_input. Since bevy_input transitively depends on glam already, all this brings in are the reflection impls.
Migration Guide

Input<T> now implements Reflect via #[reflect] instead of #[reflect_value]. This means it now exposes its private fields via the Reflect trait rather than being treated as a value type. For code that relies on the Input<T> struct being treated as a value type by reflection, it is still possible to wrap the Input<T> type with a wrapper struct and apply #[reflect_value] to it.

As a reminder, private fields exposed via reflection are not subject to any stability guarantees.

Changelog
Added

Implemented Reflect + FromReflect for many input-related types. These types are automatically registered when adding the InputPlugin.

Note: do not merge this yet. Currently, the #[reflect(...)] lists conflict with each other when the "serialize" feature is enabled. I left a comment about it in the Discord server, but basically only the last reflect list is used and the previous ones are discarded. We can mark this as a draft for the time being if anyone wants.

@TehPers Could you rebase this PR?

bors r+


Classic formatter adds new line at beginning of file

Description
The classic formatter adds a blank new line at the beginning of the file.
Formatter Implementation
classic
XML Tools Version
2.1.1
VS Code Version
1.23.0
Operating System
Windows 10 Build 1709

Thanks for the report! I do not intend to maintain the classic formatter moving forward. It has been included in v2.x as a "rollback" option for anyone broken by the v2 formatter.


fixes api tests for ds_requests where route params were removed in fa…

…vor of query params

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1375/
Test PASSed.


libsForQt5.phonon: 4.10.1 -> 4.10.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/phonon-qt5/versions.
meta.description for libsForQt5.phonon is: '"Multimedia API for Qt"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 4.10.2 with grep in /nix/store/mya9lcw28mhrjwxskv4jp5wf67sa3br9-phonon-qt5-4.10.2
directory tree listing: https://gist.github.com/f3c18cbd994e729ddc1945e3359ff235
du listing: https://gist.github.com/be26d8fcffff36ace0bdbcffd3378de6

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

1085 total rebuild path(s)
374 package rebuild(s)
374 x86_64-linux rebuild(s)
355 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
355 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
akonadi
akregator
amarok
amarok-kf5
ark
bluedevil
breeze-qt5
calamares
calligra
colord-kde
digikam
dolphin
dragon
elisa
falkon
ffmpegthumbs
filelight
frameworkintegration
gmic_krita_qt
gwenview
heaptrack
k3b
k9copy
kactivitymanagerd
kaddressbook
kate
kcachegrind
kcalc
kde-cli-tools
kde-gtk-config
kdeApplications.akonadi
kdeApplications.akonadi-calendar
kdeApplications.akonadi-contacts
kdeApplications.akonadi-import-wizard
kdeApplications.akonadi-mime
kdeApplications.akonadi-notes
kdeApplications.akonadi-search
kdeApplications.akonadiconsole
kdeApplications.akregator
kdeApplications.ark
kdeApplications.baloo-widgets
kdeApplications.calendarsupport
kdeApplications.dolphin
kdeApplications.dolphin-plugins
kdeApplications.dragon
kdeApplications.eventviews
kdeApplications.ffmpegthumbs
kdeApplications.filelight
kdeApplications.grantleetheme
kdeApplications.gwenview

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mya9lcw28mhrjwxskv4jp5wf67sa3br9-phonon-qt5-4.10.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A libsForQt5.phonon https://github.com/r-ryantm/nixpkgs/archive/93b7070a2f7f267d95a46cec694a5ad23ee51fbb.tar.gz

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

cc @ttuegel for testing.

duplicate of #54157


The measure on examples is infinite, is this not misleading?

When demonstrating the example should we not fix the maximum width? it's important to demonstrate that we understand the measure shouldn't be infinite. See screen grab

 Issue #162: The measure on examples is infinite, is this not misleading?

I think allowing examples to be full width is good for components such as a header and footer so maybe we should constrain the width of the example component by default and have an option to override to full width for certain components.

We should probably start using the grid in the patten library for content and the examples template.  Are we clear on which components should be never full width?

I think it's pretty much everything except headers, footers, timeout, and page template examples.


DEV_ENVIRONMENT.md procedure error

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Build and launch error following the DEV_ENVIRONMENT.md procedure
Reproduction
I followed the DEV_ENVIRONMENT.md procedure to launch the dev-app.
# Developer guide: getting your environment set up
1. Make sure you have both `node` and `yarn` installed.
   We recommend using `nvm` to manage your node versions.
2. angular/components uses Bazel which requires certain Bash and UNIX tools.
   - On Windows: Follow the [instructions](https://docs.bazel.build/versions/master/install-windows.html#installing-compilers-and-language-runtimes)
   to install [`MSYS2`](https://www.msys2.org/) and the listed "Common MSYS2 packages".
   Afterwards add `C:\msys64\usr\bin` to the `PATH` environment variable.
3. Fork the `angular/components` repo on GitHub.
4. Clone your fork to your machine with `git clone`.
   Recommendation: name your git remotes `upstream` for `angular/components`
   and `<your-username>` for your fork. Also see the [team git shortcuts](https://github.com/angular/components/wiki/Team-git----bash-shortcuts).
5. From the root of the project, run `yarn` to install the dependencies.

To build angular/components in release mode, run `yarn build`. The output can be found under `dist/releases`.

To bring up a local server, run `yarn dev-app`. This will automatically watch for changes
and rebuild. The browser should refresh automatically when changes are made.

I deleted %userprofile%/bazel%username%, but I have the same error.
Expected Behavior
dev-app launch successful
Actual Behavior
I have the following error:
[0 / 1] [Prepa] BazelWorkspaceStatusAction stable-status.txt
ERROR: H:/projets/github/angular/components/tools/angular/BUILD.bazel:14:30: Bundling Javascript<EMAIL_ADDRESS>[esbuild] failed: (Exit 127): _router_linked_bundle_esbuild_launcher.bat failed: error executing command bazel-out\x64_windows-opt-exec-2B5CBBC6\bin\tools\angular\_router_linked_bundle_esbuild_launcher.bat --esbuild_args=bazel-out/x64_windows-fastbuild/bin/tools/angular/router_linked_bundle.args.json ... (remaining 2 arguments skipped)
/bin/bash: line 1: C:/users/antho/_bazel_antho/qrvuvg3m/execroot/angular_material/bazel-out/x64_windows-opt-exec-2B5CBBC6/bin/tools/angular/_router_linked_bundle_esbuild_launcher.sh: No such file or directory

Environment
node --version
v18.16.0
yarn --version
1.22.17
iBazel - Version v0.16.2
get-command ibazel
H:\Projets\GitHub\angular\components\node_modules.bin\ibazel.cmd
bash --version
GNU bash, version 5.1.16(1)-release (x86_64-pc-linux-gnu)
get-command bash
Application     bash.exe                                           10.0.22... C:\Windows\system32\bash.exe

You might need to set an environment variable called BAZEL_SH to C:\msys64\usr\bin\bash.exe. Are you able to run any bazel commands?

That was it, thank you !
To complete the issue, that would be good to complete the DEV_ENVIRONMENT.md file as it is likely to have bash in c:\windows\system32 directory.

You might need to set an environment variable called BAZEL_SH to C:\msys64\usr\bin\bash.exe. Are you able to run any bazel commands?

this actually worked, thank you ❤

Fixed in https://github.com/angular/components/pull/27643.


[TensorAlgebra] Refactor contract

This refactors the contract function in TensorAlgebra based on refactored fusedims and splitdims functions for matricizing and unmatricizing.
It is based around a BlockedPermutation object which represents a permutation such as (2, 1, 4, 3) which is blocked or partitioned into parts, such as ((2, 1), (4, 3)). That is used to specify which dimensions of an array should get grouped together when performing a matricization of an n-dimensional array.
Basically ready, I still need to update definitions of fusedims and splitdims for NamedDimsArray where dimension names are used to specify matricization/dimension fusion.

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

Comparison is base (a00ef70) 84.05% compared to head (5b9a7ae) 53.83%.
 Consider uploading reports for the commit f1099c4 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1290       +/-   ##
===========================================
- Coverage   84.05%   53.83%   -30.23%
===========================================
  Files         100       99        -1
  Lines        8542     8489       -53
===========================================
- Hits         7180     4570     -2610
- Misses       1362     3919     +2557

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

[test ITensors mps]


Pressing ESC a second time after deactivate() eats up ESC without doing anything

In the Multiselect Mixin code, deactivate() is called when ESC key is pressed (see Multiselect.vue) the first time, but not the second time.

The behaviour encountered: once the Multiselect is deactivated (by pressing ESC once), the ESC key is no longer listened to, the ESC key event cannot be listened to by parent elements.
The expected behaviour: once Multiselect is deactivated (by a single ESC key press), any subsequent ESC key presses should be propagated without affecting the Multiselect.

@bryanluu Please provide reproduction link

@bryanluu Ok . I will try to replicate it. Thanks.

Not able to replicate this.


Demonstrate exclusiveFilter removing paragraph containing an image

This PR is adding a failing test to showcase the following issue that we are currently having with the exclusiveFilter option.
NOTE: I am not the original author of the issue.
Consider the following case in which a user enters some text and an image in a markdown editor, adding one extra newline between the text and the image:
Here, have this beautiful image:

![alt text](http://foo.net/bar.png)

This is the result of the conversion to HTML (a paragraph is wrapping the image due to the 2 newlines):

Here, have this beautiful image:

Of course, before displaying this on a page, we are going to sanitize it. So we are using sanitizeHtml with the exclusiveFilter option to filter out unwanted, empty paragraphs.
const sanitized = sanitizeHtml(raw, {
  exclusiveFilter: (frame) => frame.tag === 'p' && !frame.text.trim()
})

Unfortunately, this also strips away the paragraph containing the image.

Here, have this beautiful image:

We are stripping paragraphs without text, which makes sense. But we would also love to have a way to tell whether a
 contains something else meaningful, such as an image, to avoid stripping it.
The included test recreates the problematic situation.
Thanks a lot for this wonderful library, we use it extensively.

Thanks for this great message and the good detail on what you're asking for
with this feature. We'll see what we're able to do, but historically the
answer has been that if you want to really get structural like this it
makes more sense to use "cheerio" or a combination of the two modules.
On Wed, May 6, 2020 at 12:10 PM Marco Arduini<EMAIL_ADDRESS>wrote:

This PR is adding a failing test to showcase the following issue
https://github.com/apostrophecms/sanitize-html/issues/313 that we are
currently having with the exclusiveFilter option.
NOTE: I am not the original author of the issue.
Consider the following case in which a user enters some text and an image
in a markdown editor, adding one extra newline between the text and the
image:
Here, have this beautiful image:

This is the result of the conversion to HTML (a paragraph is wrapping
the image due to the 2 newlines):
Here, have this beautiful image:

Of course, before displaying this on a page, we are going to sanitize it.
So we are using sanitizeHtml with the exclusiveFilter option to filter
out unwanted, empty paragraphs.
const sanitized = sanitizeHtml(raw, {
exclusiveFilter: (frame) => frame.tag === 'p' && !frame.text.trim()
})
Unfortunately, this also strips away the paragraph containing the image.
Here, have this beautiful image:
We are stripping paragraphs without text, which makes sense. But we would
also love to have a way to tell whether a  contains something else
meaningful, such as an image, to avoid stripping it.
The included test recreates the problematic situation.
Thanks a lot for this wonderful library, we use it extensively.
You can view, comment on, or merge this pull request online at:
https://github.com/apostrophecms/sanitize-html/pull/349
Commit Summary

Add failing test where a p containing only an img tag is removed

File Changes

M test/test.js
https://github.com/apostrophecms/sanitize-html/pull/349/files#diff-c1129c8b045390789fa8ff62f2c6b4a9
(10)

Patch Links:

https://github.com/apostrophecms/sanitize-html/pull/349.patch
https://github.com/apostrophecms/sanitize-html/pull/349.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/pull/349, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27PKIPOMMGYXARSWQQDRQGDWRANCNFSM4M2SODPQ
.

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

Thanks for your work in documenting this. As mentioned in #313, the frame.text property is doing what it's designed to here. There's a PR that targets media elements, like img specifically for this use case (#356).


Increasing the backoff factor to reduce failures

Codecov Report

Merging #33 (96b85d0) into develop (16f77ad) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop      #33   +/-   ##
========================================
  Coverage    89.57%   89.57%
========================================
  Files           88       88
  Lines         3548     3548
========================================
  Hits          3178     3178
  Misses         370      370

Impacted Files
Coverage Δ

setup.py
0.00% <ø> (ø)

nbaspa/__init__.py
100.00% <100.00%> (ø)

nbaspa/data/endpoints/base.py
97.46% <100.00%> (ø)

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 16f77ad...96b85d0. Read the comment docs.


Support other decoding methods

In the paper, only two contrasting decoding methods were shown: AR (one character at a time) and non-AR (all characters at once). But in theory, PARSeq supports semi-autoregressive decoding and anything in between.
Some ideas:

Ensembled AR outputs - use random AR masks (similar to training), say 3 different ones, then greedily choose the character for each position.
Masked cloze refinement - based on the current prediction (can be from AR or NAR), the characters with low confidence are masked out, i.e. final mask = cloze mask & confidence mask. This way, low-confidence characters would not be considered as context for the other characters. In other words, only the low-confidence predictions will be refined further.
SAR decoding - start with... decoding two characters at a time?

Hello~@baudm, Thanks for you great work.  I have some questuions about the decoing methods.

Is the second idea "Masked cloze refinement " which you mentioned  smilar to the cloze mask in ABINet?
If I want to try the cloze mask / confidence mask, do I need to re-train the model with corresponding decoding methods.
I notice that PARSeq is with internal LM, while ABINet is with external LM, if the permuted language model can be appleyed in the  ABINet's  language model part?
Look forward to your favourable reply！

Hi @FaltingsA

The second idea is supposed to be an improvement over my current implementation of bidirectional refinement. IMO, the current refinement model, $P(y_t | \mathbf{y}{\neq t}, \mathbf{x})$, is already an improvement over ABINet's, $P(y_t | \mathbf{y}{\neq{t}})$ (no dependency on the image $\mathbf{x}$). I think that the refinement process could be restricted to characters that have low confidence, preventing erroneous rectification of already correct characters.

The cloze mask is already being used when you run iterative refinement. Please see Sec 3.3 of the paper.

It can't be because in PARSeq, like in similar prior work with internal LMs, there is no clear separation between the "vision" and the "language" parts of the model.


Roll Flutter Engine from d726345bf230 to 2db85cbe6a01 (4 revisions)

https://github.com/flutter/engine/compare/d726345bf230...2db85cbe6a01
2023-04-20<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from suSuT9F8zuP-pBg-E... to OcPCdaE17MAihaCrD... (flutter/engine#41352)
2023-04-19<EMAIL_ADDRESS>Roll Dart SDK from fe8bb0565a30 to 88a3b66b50d6 (2 revisions) (flutter/engine#41349)
2023-04-19<EMAIL_ADDRESS>Add missing header guards (flutter/engine#41322)
2023-04-19<EMAIL_ADDRESS>[web] Fix trivial type issue. (flutter/engine#40956)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from suSuT9F8zuP- to OcPCdaE17MAi
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update yubico-authenticator to 2.3.0

[x] Commit message includes cask’s name (and new version, if applicable).

yubico-authenticator updated to 2.3.0

[x] brew cask audit --download {{cask_file}} is error-free.

brew cask audit --download https://raw.githubusercontent.com/visualphoenix/homebrew-cask/update-yubi/Casks/yubico-authenticator.rb
==> Downloading https://raw.githubusercontent.com/visualphoenix/homebrew-cask/update-yubi/Casks/yubico-authenticator.rb
######################################################################## 100.0%
==> Downloading https://developers.yubico.com/yubioath-desktop/Releases/yubioath-desktop-2.3.0-mac.pkg
Already downloaded: /Library/Caches/Homebrew/yubico-authenticator-2.3.0.pkg
==> Verifying checksum for Cask yubico-authenticator
audit for yubico-authenticator: passed
==> Downloading https://raw.githubusercontent.com/visualphoenix/homebrew-cask/update-yubi/Casks/yubico-authenticator.rb
######################################################################## 100.0%

[x] brew cask style --fix {{cask_file}} left no offenses.

brew cask style --fix Casks/yubico-authenticator.rb

1 file inspected, no offenses detected

Thank you for the detailed reply, but no need to go through all that trouble, it’s enough to just tick the boxes


Support formatter

This extension replaces the C/C++ extension.
One thing missing is the ability to format the code. 🌷

My guess is that this is not possible because GNU Global is a tagging system?


add local docker setup + add timeout and logging to ensureDbConnected

Now devs can run postgres and minio with docker-compose up.
Updated README with docker instructions
Also added a timeout error if checking of db connection exceeds 5 seconds (makes it easier to determine if db connection is causing issues, before the request could hang indefinitely)

Thank you so much for the awesome contribution :pray:  This will ease development a lot !
Can you make a pull request to the unstable branch instead?


Package republished as fast-diff-match by another maintainer?

Heads up that this package has been republished by another PyPI user at https://pypi.org/project/fast-diff-match/ , though it still links to this Github repo and Author contact and so on. (And still imports as diff_match_patch, so I assume isn't related to fixing the name collision in #4.)
Also for what it's worth that republished version only includes Mac packages, so doesn't install on other OSes.
Not sure what's going on with that, just thought I'd flag it since it's a little confusing for someone looking for a python DMP library.

Thanks for the heads up. This isn't the first time one of my packages was re-posted, and to me this is a security/impersonation issue, but pypi didn't get back to me on the other one so I assume there is nothing to be done here.


Logical Model: prescriptionNumber

#128
PrescriptionId	prescriptionNumber	1 .. 1	0 .. 1	Must be "1.. 1" in FIHR Model. To be changed (id of prescription, always existing)

WG: check with the backend team.

WG: we check with the architects if we use a mandatory UUID to enable idempotent creation.

WG: UHMEP will provide the UUID. It is up to the clients to check whether they have (accidentally) created a double prescription.

WG: the proposal of the backend team is to make the FE responsible for checking whether the request is a duplicate or not in case of a network error (@MatonAnthony). The workgroup does not agree with this approach, and insists on a technical solution that does not burden the physician.

Solved in #214


Increase the shard and replica count for the benchmark-metrics-* indices in the data store

Description
This change will increase the number of primary shards in the monthly index "benchmark-metrics-yyyy-mm" to accomodate more documents and avoid reaching the shard limit of 2B on the metrics data store.
Issues Resolved
[List any issues this PR will resolve]
https://github.com/opensearch-project/opensearch-benchmark/issues/188

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please also add in the description why you have specifically chosen 10 shards for the primaries and 1 for the replicas. This will help future developers understand why this was changed in the past. Also, It might be good to add a brief explanation that clarifies that the 2 billion documents per shard limit is imposed by lucene.

On second thought, we should revert this change. This change potentially impacts other users since this enforces every user's datastore to use a 10 primary and 1 replicas index regardless of cluster configuration.
I encountered an issue with my personal datastore after this PR was merged in. Prior to the change, my OSB runs were writing metrics to an external OpenSearch datastore. After this change was incorporated, it encountered an issue where  the number of shards (10) was not a multiple of the number of awareness attributes (3). For more details, see issue https://github.com/opensearch-project/opensearch-benchmark/issues/330. I could change the cluster settings to get around this issue but users should not have to change their clusters to meet the requirements made by OSB.
Instead of changing the default values as done in this PR, users can override the default number of primaries and replicas with datastore.number_of_replicas and datastore.number_of_shards in the datastore config (~/.benchmark/benchmark.ini). This is easy to change and prevents us from messing with the default benchmark-metrics settings that can potentially interfere with other users' setups.


check if there is a comma in the input City, Country

https://github.com/b0bdN/prayer-times-menubar-app/blob/15074588857dacaefe949cbc30705014df91b53c/main.js#L234
We need to check if there is a comma (,) in the input for the City, Country when the user enter manually the information.
I think of two options:

the comma is mandatory, so we need to create a dialog box to show the error.
it's ok to have just a space between CITY and COUNTRY

Hello, can you assign this to me?

Hi!, thanks for your interest!
Of course you can work on this :)
What do you think will be best, should the comma be mandatory?

Some cities can be multiple words right? We should keep the comma mandatory

I am new to electron. Could you show me where in your code to create a dialog box in your to show the error? I have the logic that handles if the input is valid.

No worries ;)
I think it will be better to check the input in the renderer process when the user click on the Apply button before sending all the values to the Main process.
https://github.com/b0bdN/prayer-times-menubar-app/blob/06db1671277492d48a40d719682de747b8de5d77/renderer.js#L228
dialog work only in the Main process, so you need to create an Inter-process communication (IPC) between the Renderer and the Main.
You can use window.api.send('custom-channel', arg), add custom-channel in preload.js and listen in main.js with ipcMain.on('custom-channel', () => {dialog...}


Fixes Field::to_bits_le gadget.

This PR:

Fixes the Field::to_bits_le gadget by adding a check that enforces that the resulting bits represent a number less than the base field modulus.
Fixes the Scalar::to_bits_le gadget by adding a check that enforces that the resulting bits represent a number less than the scalar field modulus.
Cleans up some related circuits.
Updates constraint counts.

NOTE: CI fails because parameters need to be resampled.

I'm guessing CI is failing since the circuit changed.

This PR needs resampling.

Resampled parameters


Thorium's SystemWebView has a lower score than Google Play's SystemWebView

For speedometer testing, Thorium's windows version runs well on my device, outperforming Chrome by 15%. However, the Android SystemWebView version is 30% slower than the SystemWebView updated via Google Play, and the Public.apk is 10% slower.
The complete ranking is
Google Play's SystemWebView with browsers based on 110
Public.apk 97
chrome 86
Thorium's SystemWebView and browsers based on it 77
This could be related to Google Play's WebView utilizing the TrichromeLibrary and apks encapsulation? expect your optimization for Thorium's SystemWebView,thanks

It's likely due to the fact that there's no Thorium webview arm64 distributed build in /releases.


Alexa Fatal Error: Unable to connect to github.com

Alexa Build Github Error
I have followed all the instructions that amazon have given (https://developer.amazon.com/en-US/docs/alexa/alexa-voice-service/build-the-avs-device-sdk.html) me for downloading it, and I keep on getting errors when Im doing this step that say:
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
github.com[1: 2a02:8801:200:1::1]: errno=Network is unreachable
When it starts to say Cloning SDK, I really badly need help as this issue has been persisting for two days! Please Respond to this ASAP, I really want to get this completed by the next few days.

[ ] Desktop / Laptop

[Yes ] Raspberry Pi

[ ] Other - tell us more:

[ ] Linux

[ ] MacOS

[yes ] Raspbian Stretch

[ ] Raspbian Jessy

[ ] Other - tell us more:

Hi @MrVRishi , can you check if you can access the internet from your Pi?

Yeah I can,the internet is working
Get Outlook for iOShttps://aka.ms/o0ukef

From: Kenneth Chan<EMAIL_ADDRESS>Sent: Thursday, April 9, 2020 3:19:01 AM
To: alexa/avs-device-sdk<EMAIL_ADDRESS>Cc: MrVRishi<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [alexa/avs-device-sdk] Alexa Fatal Error: Unable to connect to github.com (#1667)
Hi @MrVRishihttps://github.com/MrVRishi , can you check if you can access the internet from your Pi?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/alexa/avs-device-sdk/issues/1667#issuecomment-611290132, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANQ5CHO7EVQQMI6OK6LZ3TTRLUWBLANCNFSM4MEJQ7OQ.

Are you able to access your own github repos from the command line on the raspberry pi?

Closing this due to inactivity. Please re-open if I've closed this in error.


leak in coda-definition-parse.c:parse_entry

Use of uninitialized variable found, whcih can be reproduced using the test case below:
==45769==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x557820a68700 in normal_scanPoundName third_party/expat/lib/xmltok_impl.c:953:13
    #1 0x557820a5eb3d in normal_prologTok third_party/expat/lib/xmltok_impl.c:1132:12
    #2 0x557820a813e6 in initScan third_party/expat/lib/xmltok.c:1640:10
    #3 0x557820a5e266 in initScanPrologNS third_party/expat/lib/xmltok_ns.c:63:10
    #4 0x557820a3055c in prologProcessor third_party/expat/lib/xmlparse.c:4093:13
    #5 0x557820a30280 in prologInitProcessor third_party/expat/lib/xmlparse.c:3920:10
    #6 0x557820a2e0b5 in XML_ParseBuffer third_party/expat/lib/xmlparse.c:1893:25
    #7 0x557820a2ccb1 in XML_Parse third_party/expat/lib/xmlparse.c:1857:14
    #8 0x5578208e5755 in parse_entry third_party/stcorp_coda/libcoda/coda-definition-parse.c:4218:14
    #9 0x5578208e6957 in read_definition_file third_party/stcorp_coda/libcoda/coda-definition-parse.c:4263:9
    #10 0x5578208e6633 in coda_read_definitions third_party/stcorp_coda/libcoda/coda-definition-parse.c:4471:21
    #11 0x5578208a198d in LLVMFuzzerTestOneInput third_party/stcorp_coda/fuzz/coda_read_definitions_fuzzer.cc:19:3
    #12 0x5578214451b9 in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, unsigned long) third_party/llvm/llvm-project/compiler-rt/lib/fuzzer/FuzzerLoop.cpp:559:15
    #13 0x55782140b494 in fuzzer::RunOneTest(fuzzer::Fuzzer*, char const*, unsigned long) third_party/llvm/llvm-project/compiler-rt/lib/fuzzer/FuzzerDriver.cpp:301:6
    #14 0x557821419367 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) third_party/llvm/llvm-project/compiler-rt/lib/fuzzer/FuzzerDriver.cpp:801:9
    #15 0x5578213ef5bd in main third_party/llvm/llvm-project/compiler-rt/lib/fuzzer/FuzzerMain.cpp:20:10
    #16 0x7f4b44823bbc in __libc_start_main (/usr/grte/v4/lib64/libc.so.6+0x38bbc)
    #17 0x5578208269e8 in _start /usr/grte/v4/debug-src/src/csu/../sysdeps/x86_64/start.S:108
SUMMARY: MemorySanitizer: use-of-uninitialized-value third_party/expat/lib/xmltok_impl.c:953:13 in normal_scanPoundName
Exiting

    parser_info_init(&info);
    info.zf = zf;
    info.entry_base_name = name;
    info.product_class = current_product_class;
    info.product_definition = current_product_definition;

    filesize = za_get_entry_size(entry);
    // The following allocates memory without initializing it. Using uninitialized memory can be potentially pose a security issue.
    info.buffer = malloc(filesize);
    if (info.buffer == NULL)
    {
        coda_set_error(CODA_ERROR_OUT_OF_MEMORY, "out of memory (could not allocate %lu bytes) (%s:%u)",
                       (long)filesize, __FILE__, __LINE__);
        parser_info_delete(&info);
        return -1;
    }

Test case:
0000000: 5020613d0231223e 504b010237ffffff  P a=.1">PK..7...
0000010: ff8f080000f900fb 00cb00ff0a000000  ................
0000020: 010900002b00183c 214154544c494c54  ....+..<!ATTLILT
0000030: ff2885000000696e 6465782e786d6c00  .(....index.xml.
0000040: 00ff00000f291400<PHONE_NUMBER>000000  .....)..........
0000050: 43ef2e3c21444e54 4954414e59ba00f1  C..<!DNTITANY...
0000060: 0c0c0c1f00000024 0001073f000100f6  .......$...?....
0000070: b20000504b1f0000 000000873f000070  ...PK.......?..p
0000080: 00004e02fb504b03 043334368f080000  ..N..PK..346....
0000090: f900fb00cb00ff0a<PHONE_NUMBER>00002b  ...............+
00000a0: 00080000f9252525 275b0000ff00002b  .....%%%'[.....+
00000b0:<PHONE_NUMBER>e5e5f5 e52e27504b0506bf  1579Q.....'PK...
00000c0:<PHONE_NUMBER>003300<PHONE_NUMBER>000000  ..7...3.........
00000d0: fa0021d3d149532e<PHONE_NUMBER>0039e5  ..!..IS.......9.
00000e0: 504b0506bf000037<PHONE_NUMBER>000016  PK.....7...3....
00000f0:<PHONE_NUMBER>fa                       ......

testcase-5633184541966336.zip

I can't replicate this problem. If I use your test case then I get:
codadd -D testcase-5633184541966336
ERROR: could not read data from definition file: could not open testcase-5633184541966336 (not a zip file)

Which means it already bails out at za_open in read_definition_file. I don't understand how you get to parse_entry.

 Have you forgot to add .zip to the file name?
For the record, this is the fuzz driver:
int coda_read_definitions(const char *definition_path);
}
extern "C" int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) {
  FuzzerTemporaryFile temp_file(data, size);
  coda_init();
  auto done = absl::MakeCleanup([] { coda_done(); });
  coda_read_definitions(temp_file.filename());
  return 0;
}```

I don't know how the fuzzer works, but CODA will check if the file starts with 'PK' (ziparchive.c:420) and otherwise bail out.
Your test file does not start with 'PK' (although there is a section 8 bytes in that starts with that).
A .zip extension is not needed (codadef files normally also have a .codadef extension)

Actually, looking at it. Your test case is an xxd dump. Please provide the binary file.

Can you please try with the file attached. Thanks.
testcase-5633184541966336.zip

Fixed in 22eb9197ab973055ce856f40b02157eb5833a958
Also improved check for proper zip file in 1d19ad61fd2011a54780335c896ceab4a7393cd3

Thanks, @svniemeijer !


Betting and client's coins

If a client's number of coins drops below 10, just kick them off as if they've lost. They can't bet anything anyway in the next round.

This should already happen.

This already happens

Well. Um. Not exactly. But sure!

Fixed by @pcjl .


chore: rebase merge branch migrate-metrics-dev into master

Compilation error for Go collector would  be fixed in:
https://github.com/apache/incubator-pegasus/issues/1761

I've reviewed these patches in branch incubator-pegasus:migrate-metrics-dev, the new branch empiredan:rebase-migrate-metrics-dev-onto-master is forked from there, and I've reviewed the new added patches appended to the branch, I this PR looks good to me.


Replace Heroku link with link to example app code

Closes #442

Could you please mention in the commit message that this fixes #442?

yep, done in both the commit message and PR comment/message

Thanks a lot.


Implement the HL7 data feed from PHIN MS into DEX

@ssk2cdcgov we need to bring all the meta data from Phinms. The payloads that will be pulled from the inqueue table should be processed in DEX, in order in which the message was received.

I able to connect to Databricks.  I have also had someone explain how to decrypt the PHINMS message from the 'DEX_inq' table. I have applied for the 'Service Account' to access this table and the encryption process. Contacted Boris and he will arrange access to the "rest of DEX's' resources.
I have access to "tfedemessagestoragedev" but cant see the 'Storage Browser'.

I spoke emailed John Stevens about the Service Account and it should be available soon.

The ADF environment is now working for 'edav-prd-dex-factory '. The blob storage, 'DEX_STG_ADLS_LS', is working and I have access to it. The ADF environment needs to have permissions added, but this has been communicated.

I have begun the process of moving over the initial efforts from the other ADF environment. I have also had a discussion with Marcelo regarding the final format the records will be added to the storage, including a method to name the file for each record pulled. There will be two parts to the pull, one will be to pull the HL7 record and persist it to a file. The other will be to pull the 'Metadata' around each record and persist this as well.

There was a right's issue to the Blob storage, which was fixed on Friday. There is still one last issue, I cannot save an ADF. But I can work for now in debug mode.

The rights issue was worked out yesterday afternoon. I can now persist the data to files and see them in the final 'landing space' for the blob storage. I am stil waiting for final permissions to place the decrypted files in the blob storage..

Holding for ATO.

The path for the data is now down to 'one step' as opposed to the 'two steps' we had earlier. I have rewritten the ADF I have to accommadate this and am waiting for a meeting today to give a final review for the encryption.

Made a first pass at documenting the process and, cleaned up and renamed the subordinate processes of the ADF. As well as restructuring some parts for the Prod version.

done not closed.

This should be done and running. There are other modifications such as moving part of the process to Databricks. These are tracked in other tickets.


Missing required "platform-project-id" in setters.yaml

This issue appears in two different blueprints:
https://github.com/GoogleCloudPlatform/blueprints/blob/main/catalog/acm/setters.yaml
https://github.com/GoogleCloudPlatform/blueprints/blob/main/catalog/anthos-cluster/setters.yaml
In both cases, we have "platform-project-id" missing in setters.yaml. As a result if customers try to apply these two blueprints they may see failure because this value is not set.

@kaariger can you PTAL? Lets remove platform-project-id if not needed as a setter.

Fixing this. This breaks the attribution mainly and does not impact the blueprint itself.


Issue with Update value from outside the plugin

Update value from outside the plugin is not working when using
List<DropdownMenuItem<VehicleMaster>> vehicleList
`class VehicleMaster {
String id = '0';
String title = '';
String sizeId = '0';
String size = '';
VehicleMaster();
VehicleMaster.add(this.id, this.title);
VehicleMaster.addModel(this.id, this.title, this.sizeId, this.size);
VehicleMaster.fromJSON(Map<String, dynamic> jsonMap) {
try {
id = jsonMap['Id'] ?? '0';
title = jsonMap['Title'] ?? '';
sizeId = jsonMap['VehicleSizeId'] ?? '';
size = jsonMap['VehicleSize'] ?? '';
} catch (e) {
print(CustomTrace(StackTrace.current, message: e.toString()));
}
}
}`
SearchChoices.single(                     items: _con.vehicleList,                     hint: "Select Vehicle*",                     displayClearIcon: false,                     value: _con.tempState,                     autofocus: false,                     style: Theme.of(context).textTheme.bodyText1,                     selectedValueWidgetFn: (value) {                       value as VehicleMaster;                       return Align(                           alignment: Alignment.centerLeft,                           child: (Text(                             value.title,                             overflow: TextOverflow.clip,                             maxLines: 1,                           )));                     },                     onChanged: (value) {                       value as VehicleMaster;                       _con.tempVehicle = value;                     },                   ),
Using List<DropdownMenuItem<String>> vehicleList it's working fine
Please provide me the solution with custom Type.

Hi @vivekfameitc ,
Thanks for your question.
Since you are dealing with custom objects (VehicleMaster), allow me to refer you to this explanation:
https://github.com/lcuis/search_choices/issues/66#issuecomment-1034656174
Please let me know whether or not this solves your issue.

I hope your issue is solved.
I am closing this issue for inactivity. Feel free to reopen if needed.


Refresh (in already running application instances) does not reload or crash when referenced Key Vault secret is deleted

If you delete a referenced Key Vault secret without removing the reference Key from App Configuration, running applications instances do not crash or reload other items, (probably due to the local caching) This behavior is misleading. Imagine if someone deletes a secret and forgets to delete the related key from the App Configuration, it might put applications in an invalid state.
When you restart the application instance you get the following exception which is good and expected.
Microsoft.Extensions.Configuration.AzureAppConfiguration.KeyVaultReferenceException
HResult=0x80131500
Message=Key vault error. ErrorCode:, Key:MyApp1:Connection, Label:Development, Etag:VTVSei4raljcWNNSRWiydwpbNrx, SecretIdentifier:https://vault.azure.net/secrets/testsecret
Source=Microsoft.Extensions.Configuration.AzureAppConfiguration
StackTrace:
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureKeyVault.AzureKeyVaultKeyValueAdapter.d__3.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.d__18.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.d__17.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.d__13.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Extensions.Configuration.AzureAppConfiguration.AzureAppConfigurationProvider.Load()
at Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList`1 providers)
at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()
at Microsoft.Extensions.Hosting.HostBuilder.BuildAppConfiguration()
at Microsoft.Extensions.Hosting.HostBuilder.Build()
at AppConfigInAspnetcore.Program.Main(String[] args) in

Thanks for the feedback, @msajid. Just to make sure I understand, here is what you suggest
For an invalid Key Vault reference (eg. the actual secret is deleted),

Throw at the app start-up time (ie. the first time load of configuration)
Do not throw at configuration refresh (eg. triggered by sentinel key change)

This sounds reasonable to me. In the second case, however, the old value from Key Vault will be removed from the configuration instead of keeping a cached value. Does this work for you?

It's by design.
The secret from Azure Keyvault may have changed, but the Keyvault app config reference itself isn't. It's more or less like a shortcut to a file. One can delete the file, but the shortcut isn't affected.
AppConfiguration doesn't cache/store Keyvault secrets. When it's asked to refresh, it will always try to fetch from Keyvault.
The issue here is that Keyvault didn't offer a mechanism to monitor secrets state - noting that integration with EventGrid (preview) was announced last week. Currently monitoring and refresh on Keyvault reference alone doesn't work. Using Sentinel Key may be a decent workaround, but it still  requites explicit action.

Throw at the app start-up time (ie. the first time load of configuration) Yes this is the expected behavior in my opinion and it works as expected.
Do not throw at configuration refresh (eg. triggered by sentinel key change) Yes this is exactly what is happening, I think client provider should throw exception at the refresh when secret doesn't exists or invalid. Here is how I experienced it.

Create a secret in Key Vault
Reference the secret in App Configuration
Start the application and it works perfectly and loads all the items including the one in Key Vault
Delete the secret from Key Vault
Modify sentinel and wait for the refresh to happen. Refresh never happens.
Stop the application and Start again. Exception thrown

I think client provider should throw exception while refreshing if secret is invalid.
I hope I was able to give better description this time.

@drago-draganov Thanks for the clarification, I am referring to a different issue.
I think refresh does not happen when there is problem fetching a value for the Key Vault secret.

I realized I was referring to different scenario by reading your last message. Sorry about that.
There are 2 different workflows to consider.
###1. Loading configuration at startup:
Any error is propagated to the caller.
###2. Runtime Refresh:
Refresh is atomic - succeeds when all requested config settings are successfully materialized.
In ASP.NET Core app, if you call UseAzureAppConfiguration then the default refresh behavior is set up. That means all Refresh failures are ignored and the app keeps using last known good configuration.
If you prefer to employ different behavior (ex. crashing the app), don't call UseAzureAppConfiguration and add own middleware instead:
class MyAppConfigurationRefreshMiddleware
    {
        private readonly RequestDelegate _next;
        public readonly IConfigurationRefresher _refresher;

        public AzureAppConfigurationRefreshMiddleware(RequestDelegate next, IConfiguration configuration)
        {
            _next = next;

            //
            // Get IConfigurationRefresher
            _refresher = ((IConfigurationRoot) configuration).Providers.FirstOrDefault(p=> p is IConfigurationRefresher) as IConfigurationRefresher;
        }

        public async Task InvokeAsync(HttpContext context)
        {
            try
            {
                await _refresher.Refresh();
            }
            catch (KeyVaultReferenceException)
            {
                //
                // Keyvalue reference error
            }
            catch (Exception)
            {
                //
                // Ignore unknown errors for now (not the best approach).
                // We are working to provide spec for transitive errors that can safely be ignored here.
            }

            await _next(context);
        }

Thanks @drago-draganov this is exactly what I was looking for 👍.


dlib compile error with CUDA 10.2 (identifier "dlib::sqrt_2" is undefined in device code)

Expected Behavior

dlib should compile with CUDA
Current Behavior

It does not compile
running install
running bdist_egg
running egg_info
writing tools/python\dlib.egg-info\PKG-INFO
writing dependency_links to tools/python\dlib.egg-info\dependency_links.txt
writing top-level names to tools/python\dlib.egg-info\top_level.txt
package init file 'tools\python\dlib\__init__.py' not found (or not a regular file)
reading manifest file 'tools/python\dlib.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
no previously-included directories found matching 'tools\python\build*'
writing manifest file 'tools/python\dlib.egg-info\SOURCES.txt'
installing library code to build\bdist.win-amd64\egg
running install_lib
running build_py
running build_ext
Building extension for Python 3.8.5 (default, Sep  3 2020, 21:29:08) [MSC v.1916 64 bit (AMD64)]
Invoking CMake setup: 'cmake C:\Users\Vayun Mathur\OneDrive\Documents\code\python\dlib\tools\python -DCMAKE_LIBRARY_OUTPUT_DIRECTORY=C:\Users\Vayun Mathur\OneDrive\Documents\code\python\dlib\build\lib.win-amd64-3.8 -DPYTHON_EXECUTABLE=C:\Users\Vayun Mathur\anaconda3\envs\AI\python.exe -DCMAKE_CXX_FLAGS=-DCUDA_HOST_COMPILER=/usr/bin/gcc-6 -DCMAKE_LIBRARY_OUTPUT_DIRECTORY_RELEASE=C:\Users\Vayun Mathur\OneDrive\Documents\code\python\dlib\build\lib.win-amd64-3.8 -A x64'
-- Building for: Visual Studio 16 2019
-- Selecting Windows SDK version 10.0.18362.0 to target Windows 10.0.19041.
-- The C compiler identification is MSVC 19.27.29111.0
-- The CXX compiler identification is MSVC 19.27.29111.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.27.29110/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.27.29110/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found PythonInterp: C:/Users/Vayun Mathur/anaconda3/envs/AI/python.exe (found version "3.8.5")
-- Found PythonLibs: C:/Users/Vayun Mathur/anaconda3/envs/AI/libs/Python38.lib
-- pybind11 v2.2.2
-- Using CMake version: 3.18.2
-- Compiling dlib version: 19.21.99
-- SSE4 instructions can be executed by the host processor.
-- AVX instructions can be executed by the host processor.
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of void*
-- Check size of void* - done
-- Enabling AVX instructions
-- Searching for BLAS and LAPACK
-- Searching for BLAS and LAPACK
-- Looking for pthread.h
-- Looking for pthread.h - not found
-- Found Threads: TRUE
-- Found CUDA: C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v10.2 (found suitable version "10.2", minimum required is "7.5")
-- Looking for cuDNN install...
-- Found cuDNN: C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v10.2/lib/x64/cudnn.lib
-- Building a CUDA test project to see if your compiler is compatible with CUDA...
-- Building a cuDNN test project to check if you have the right version of cuDNN installed...
-- Enabling CUDA support for dlib.  DLIB WILL USE CUDA
-- C++11 activated.
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Users/Vayun Mathur/OneDrive/Documents/code/python/dlib/build/temp.win-amd64-3.8/Release
Invoking CMake build: 'cmake --build . --config Release -- /m'
Microsoft (R) Build Engine version 16.7.0+b89cb5fde for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

Steps to Reproduce

Install CUDA 10.2
Install cuDNN 7.6.5
git clone https://github.com/davisking/dlib.git
python setup.py install

Version:  19.21.99
Where did you get dlib:  This github repo
Platform:  Windows 10 64 bit 10.0.19041 Build 19041
Compiler:  CMake 3.18.2

I am guessing it's (partially?) my fault... It sqrt_2 in cuda code was added in #2204.
I find it surprising, because the same line uses both pi and sqrt_2:
https://github.com/davisking/dlib/blob/3ba004f875d02c4c326f8cec5d9ec46b5708ea5a/dlib/cuda/cuda_dlib.cu#L1504
Both defined here:
https://github.com/davisking/dlib/blob/3ba004f875d02c4c326f8cec5d9ec46b5708ea5a/dlib/numeric_constants.h#L9-L10
https://github.com/davisking/dlib/blob/3ba004f875d02c4c326f8cec5d9ec46b5708ea5a/dlib/numeric_constants.h#L15-L16
Maybe it's grabbing pi from somewhere else... However, the CI tests passed...
Anyway, can you try with #2210?

I tried #2210, and the sqrt_2 issue went away, but the pi issue was still there.
Then I tried to replace pi in the code with a floating point literal, and then it worked.
If I remember right, I think that in CUDA you have to declare variables to be device to use them in a device method?

Thnk you for trying it out. It appears that CUDA already provides the constant I need to use CUDART_SQRT_2PI. I've just pushed a fix for that.


Stream info shows first and last timestamps incorrectly

Observed behavior
nats stream info <stream> shows the first sequence and last sequence timestamps with UTC, however, they are displayed in the local timezone.
      First Sequence: 1 @ 2024-03-28 07:22:07 UTC
       Last Sequence: 1,697,011 @ 2024-03-28 07:25:12 UTC

With --json, the time is correctly UTC.
    "first_ts": "2024-03-28T13:22:07.110043Z",
    "last_ts": "2024-03-28T13:25:12.970217Z",

Expected behavior
The time for the first and last sequences should either be in UTC format or the UTC should be removed to indicate it is the local timezone.
Server and client version
nats-server 2.10.12
natcli 0.1.4
Host environment
No response
Steps to reproduce
On a system that is not configured with UTC as the timezone:

Run nats stream info <stream>
Run nats stream info <stream> --json
Compare the first and last sequence times

yeah should just drop the UTC now, all times are local

Fixed in https://github.com/nats-io/natscli/pull/1032


Splash Screen Not available

Pls assign me this issue

@alok27a  read the readme and contributing.md before contributing.
Happy coding!🥳

#18 fixes this.


[ENH] Implemented pipeline_version and pipeline_name query fields

Closes #341

Checklist
This section is for the PR reviewer

[ ] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
[ ] PR has a label for the release changelog or skip-release (to be applied by maintainers only)
[ ] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[ ] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

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

35 of 37 (94.59%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 95.974%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/api/utility.py
4
6
66.67%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.07%

Covered Lines:
739

Relevant Lines:
770

💛  - Coveralls