LightmapGI creating a new LightmapGIData results in a wall of errors (Vulcan)

Tested versions

Reproducible in v4.2.2.stable.official
Reproducible in v4.3.beta1.official

System information
Godot v4.3.beta1.official - Windows 10 - Vulcan Forward+ / Mobile
Issue description
Info
Creating a new LightmapGIData results in a wall of errors. Only on Vulcan!

Forward+ (Vulcan)
Set light_data
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
  servers/rendering/renderer_rd/uniform_set_cache_rd.h:130 - Condition "rid.is_null()" is true. Returning: rid
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
[...]

Mobile (Vulcan)
Set light_data
  Texture (binding: 6) is an array of (2) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 1).
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
  Texture (binding: 6) is an array of (2) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 1).
  servers/rendering/rendering_device.cpp:3633 - Parameter "uniform_set" is null.
  Uniforms supplied for set (1):
  Set: 1 Binding: 0 Type: UniformBuffer Writable: N Length: 128
  Set: 1 Binding: 1 Type: Texture Writable: N Length: 1
  are not the same format as required by the pipeline shader.

Compatibility
Set light_data

(no errors)
Steps to reproduce

new 3d scene (Node3D)
add LightmapGI
click Data - Light Data  - New LightmapGIData

Minimal reproduction project (MRP)
not needed

debug build v4.3.beta.custom_build [5f1184e93] Forward+
ERROR: Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
   at: (servers/rendering/rendering_device.cpp:2931)
ERROR: Condition "rid.is_null()" is true. Returning: rid
   at: _allocate_from_uniforms (servers/rendering/renderer_rd/uniform_set_cache_rd.h:130)
ERROR: Parameter "uniform_set" is null.
   at: draw_list_bind_uniform_set (servers/rendering/rendering_device.cpp:3943)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Texture (binding: 7) is an array of (8) textures, so it should be provided equal number of texture IDs to satisfy it (IDs provided: 7).
   at: (servers/rendering/rendering_device.cpp:2931)
ERROR: Condition "rid.is_null()" is true. Returning: rid
   at: _allocate_from_uniforms (servers/rendering/renderer_rd/uniform_set_cache_rd.h:130)
ERROR: Parameter "uniform_set" is null.
   at: draw_list_bind_uniform_set (servers/rendering/rendering_device.cpp:3943)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
ERROR: Uniforms were never supplied for set (1) at the time of drawing, which are required by the pipeline.
   at: (servers/rendering/rendering_device.cpp:4096)
[...]

Creating this resource manually doesn't achieve anything (it doesn't bake lightmaps), but we should prevent it from causing errors still.


SegFault using multi workers in Kubernetes

Describe the bug
We use a sanic in a container on kubernetes. Number of workers = 5
And this segmentation fault append very often :
Fatal Python error: Segmentation fault

Thread 0x00007fabd10ed740 (most recent call first):
File "/usr/local/lib/python3.8/site-packages/sanic/server.py", line 907 in serve
File "/usr/local/lib/python3.8/multiprocessing/process.py", line 108 in run
File "/usr/local/lib/python3.8/multiprocessing/process.py", line 315 in _bootstrap
File "/usr/local/lib/python3.8/multiprocessing/popen_fork.py", line 75 in _launch
File "/usr/local/lib/python3.8/multiprocessing/popen_fork.py", line 19 in __init__
File "/usr/local/lib/python3.8/multiprocessing/context.py", line 277 in _Popen
File "/usr/local/lib/python3.8/multiprocessing/process.py", line 121 in start
File "/usr/local/lib/python3.8/site-packages/sanic/server.py", line 1071 in serve_multiple
File "/usr/local/lib/python3.8/site-packages/sanic/app.py", line 1130 in run
File "app.py", line 1038 in <module>

Environment (please complete the following information):

Docker image: python:latest
Version 20.6.3

I would not suggest running multiple workers in this case. Let Kubernetes do the horizontal scaling. That is what it is meant for.

What version of k8s? What are the specs of your nodes?

Thank you for your answer.
I run my container on GKE with a version 1.15.12-gke.9
Machine types are n1-standard-4 for GCP. So it's :

4 cores
15 Go of memory

And my container has these limits and requests:

resources:
limits:
cpu: "2500m"
memory: "8000Mi"
requests:
cpu: "2000m"
memory: "7500Mi"

I didn't have this issue with gunicorn and the same number of workers.
Do you think it should be better to use 1 worker and more replicas ? (with less ressources per pods)

There is a bit of a difference in how gunicorn handles multiprocessing and how Sanic does it. I would not suggest exceeding the number of CPUs you have available.

I face a similar issue (segfault), even if I define 2 workers on 2 CPUs node

k8s replicas, or multiple Sanic workers in a single container on a single pod?
Also, what kind of executor are you running it in? Once you start doing that you need to be careful more closely with your memory usage.
My recommendation is keep 1 instance to 1 pod, and let k8s handle the scaling.

k8s replicas, or multiple Sanic workers in a single container on a single pod?
Also, what kind of executor are you running it in? Once you start doing that you need to be careful more closely with your memory usage.
My recommendation is keep 1 instance to 1 pod, and let k8s handle the scaling.

Having multiples workers in a single pod that define only one container.
ThreadPoolExecutor triggers the SegFault


tensorflow/tensorflow:1.4.0-devel-gpu uses libcuda.so stub at runtime

$ nvidia-docker run -ti tensorflow/tensorflow:1.4.0-devel-gpu /bin/bash
root@2ad65cce2269:~# ldd /usr/local/lib/python2.7/dist-packages/tensorflow/python/_pywrap_tensorflow_internal.so
[...]
        libcuda.so.1 => /usr/local/cuda/lib64/stubs/libcuda.so.1 (0x00007f9de7215000)

As a result, you can't use CUDA in this image:
2017-11-03 17:57:11.181620: E tensorflow/stream_executor/cuda/cuda_driver.cc:406] failed call to cuInit: CUresult(-1)
2017-11-03 17:57:11.181660: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:158] retrieving CUDA diagnostic information for host: 939b58c94187
2017-11-03 17:57:11.181677: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:165] hostname: 939b58c94187
2017-11-03 17:57:11.181848: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:189] libcuda reported version is: Not found: was unable to find libcuda.so DSO loaded into this program

Issue was caused by: https://github.com/tensorflow/tensorflow/pull/13399
Please backport https://github.com/tensorflow/tensorflow/pull/13456 into r1.4

@gunan

@case540 can we patch #13456 into the release branch and rebuild our docker images?

Ok, will get started on this. #14225

@gunan this seems to be fixed, right?

Patch was cherry-picked to 1.4. In process of rebuilding/uploading docker images for 1.4

New dockers built and uploaded. Unsure how to verify issue is fixed.

gunan@dragonite:~$ nvidia-docker run -ti tensorflow/tensorflow:1.4.0-devel-gpu /bin/bash
1.4.0-devel-gpu: Pulling from tensorflow/tensorflow
...
Status: Downloaded newer image for tensorflow/tensorflow:1.4.0-devel-gpu
root@d39ef3933d75:~# python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
>>> tf.test.iss_gpu_available()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'module' object has no attribute 'iss_gpu_available'
>>> tf.test.is_gpu_available()
.....
True
>>>

Looks like GPU is now accessible on 1.4 devel docker images.


Import New Database Project - Unable to save in root directory

Azure Data Studio Version:
Version: 1.20.0-insider (user setup)
Commit: 710d0462d8d81373aa2bc0f36f88955c902cb173
Date: 2020-06-25T05:31:00.820Z
VS Code: 1.46.0
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

Steps to Reproduce:

Right click on SQL Server 2019 database and choose Import New Database Project.
Use default name.
Choose SchemaObjectType.
Save on an empty drive.

An error like this would appear

Check if import is also creating a new folder with proj name - if not we may want to follow the same path


Refactor QuantumOperator and add ExternalVertices check

Codecov Report

Merging #59 (a22b477) into dev (e0f93ae) will decrease coverage by 2.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev      #59      +/-   ##
==========================================
- Coverage   49.69%   47.30%   -2.39%
==========================================
  Files          39       39
  Lines        2451     2488      +37
==========================================
- Hits         1218     1177      -41
- Misses       1233     1311      +78

Impacted Files
Coverage Δ

src/FeynmanDiagram.jl
25.00% <ø> (-16.67%)
:arrow_down:

src/computational_graph/ComputationalGraph.jl
0.00% <ø> (ø)

src/computational_graph/diagram.jl
0.00% <0.00%> (ø)

src/expression_tree/build.jl
72.97% <0.00%> (-18.92%)
:arrow_down:

src/diagram_tree/traits.jl
30.23% <0.00%> (-7.27%)
:arrow_down:

src/parquet_builder/polarization.jl
91.30% <0.00%> (-5.80%)
:arrow_down:

src/diagram_tree/optimize.jl
83.75% <0.00%> (-5.00%)
:arrow_down:

src/parquet_builder/sigma.jl
91.07% <0.00%> (-3.58%)
:arrow_down:

src/parquet_builder/vertex3.jl
94.82% <0.00%> (-3.45%)
:arrow_down:

... and 7 more

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


Attaching file with long name truncates to 40 characters (and doesn't indicate so)

When attaching files whose filenames exceed 40 characters, the uploaded file is truncated to include just the first 40 characters of the filename. This is all well and good, however the .NET app continues to show the original, un-truncated version of the filename until the next app session is launched and the attachments are retrieved from the server.
Expectation: truncate the file name in the active app session to indicate this is the case.

Verified this message is now displaying:


fix(button): stroked button crops applied badges

Since we hide overflow for stroked buttons in order to properly handle the border-radius for the child ripple container and focus overlay, badges which are applied on the button will be clipped.
In order to fix this bug, we just remove the overflow: hidden and go back to the initial implemenation where the border radius is inherited for the child elements. Just to keep https://github.com/angular/material2/issues/13738 resolved, we ensure that the child elements fully cover the actual button element for the correct border radius.
Fixes #13909

We added the overflow: hidden with https://github.com/angular/material2/commit/4cdf5bafd6af0085d3fe93c755a92c92bfd3e483, so I guess it's kind of like a revert for the next patch version.

I think it should be fine for this to be in patch

Looks like this actually fixes a regression caused in anchor buttons by #13745.
The vertical spacing was messed up by overflow: hidden on <a mat-stroked-button> elements.


Replace "Hudson" by "Jenkins" in the codebase

There is some old Jenkinsfile Runner code which was originally a part of Jenkins test frameworks. This code uses the "Hudson" name though Jenkins was renamed to Hudson in 2011. It would be great to cleanup this code. Some occurences: https://github.com/jenkinsci/jenkinsfile-runner/search?q=Hudson
Acceptance criteria:

"Hudson" occurrences in the documentation, Javadoc and comments are replaced with Jenkins
"Hudson" occurrences in class and method names are replaced by "Jenkins". In this repository binary compatibility is not a concern even for public classes

I can work on it.

Should only the occurence in the search result be replaced?
I am new to the open source world and want to start.
An assistance would be very helpful.

@darkphoenix17 GitHub search is not ideal. If you see occurrences in other places, please feel free to update them as well. Same can be done for other Jenkins components, though you need to be more careful about API changes in them.

Okay, I am making a PR with the necessary changes. Guide me if I am wrong somewhere, please.


Please invite me to the Recode-Hive GitHub Community Organization

Name
Ashutosh Shukla | Contributor
Discord Username (if applicable)
ashutoshshukla831888
Additional Context
Helping others and get saved by others were always my type of things. This community aligns with my core values to encourage , inspire and collabrate with amazing minds out there . and that's the reason why i am here .

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


WIP: cpptraj recipe

@hainm, this successfully builds for me on linux (linking against system BLAS), and OS X using https://github.com/Amber-MD/cpptraj/pull/106 (linking against Accelerate for BLAS). Getting it to build on Windows using mingw-w64 will require more work.

cool. +100.

@rmcgibbo have you tested OSX on travis yet? I don't see your build.

I've actually never used OS X travis.

I just wrote and debugged the recipe locally on my OS X and linux machines. After this is merged, this this repository is tied to a dedicated build machine running Jenkins that @jchodera's group administers, so all the recipes get built and deployed to anaconda.org for OS X and linux through that infrastructure (and we use Appveyor for windows).

I think travis only recently support osx. add os tag (work for parmed)
os:
    #- linux
    - osx

I think travis only recently support osx. add os tag (work for ParmEd)

Wow, does that really work for everybody now?

Wow, does that really work for everybody now?

@hainm just added a test env for ParmEd on OS X (see the apple symbol at the end of my most recent build: https://travis-ci.org/ParmEd/ParmEd

Ideally we'd like to get this deployed through the amber-md conda channel at some point.
Pinging @drroe in case he's curious/interested about this at all.

@hainm just added a test env for ParmEd on OS X (see the apple symbol at the end of my most recent build: https://travis-ci.org/ParmEd/ParmEd

Awesome!  We should start adding these to our other projects.
I wonder if this also means we can start to eliminate Jenkins completely by migrating the conda package builds over to travis for linux/osx.

I wonder if this also means we can start to eliminate Jenkins completely by migrating the conda package builds over to travis for linux/osx.

Would this work for the OpenCL and CUDA platforms in OpenMM?  My understanding was 'no' (unless Travis has an easy way of adding the CUDA toolkit version we want).

I think it is actually straightforward to install the toolkits (in the
root environment), just not test with them.

I think it is actually straightforward to install the toolkits (in the
root environment)

It's apparently been whitelisted: https://github.com/travis-ci/apt-package-whitelist/pull/596
But some caveats here:

We would have no control over the CUDA version that gets installed if we go through apt
It would add a ~1.5-2GB download every time conda packages were built (at minimum, if we didn't add it to the standard Travis test builds)

We might be able to alleviate the first point by downloading the toolkit from nVidia and installing it, assuming that was automatable somehow (keeping in mind that we have no root privileges in the container infrastructure).  And maybe we can cache the toolkit somewhere, but that might violate licensing terms in addition to any other issues with storing a very large file somewhere in/around Travis.

Is there a reason we couldn't use the same rpm installation mechanism for CUDA we use in the current vagrant provisioning script?  Similarly, I have cached the AMD APP SDK privately, and use a mechanism to auto-install that.

Travis uses Ubuntu, so are rpm/yum even available?
If it could work, that'd be great.  Do you think it will be just as easy to get the OS X VM properly provisioned?

Do you think it will be just as easy to get the OS X VM properly provisioned?

For Jenkins?  Or for travis?  I don't think there's anything that needs to be installed for travis, actually---OpenCL is a built-in part of the OS, isn't it?

For Jenkins? Or for travis? I don't think there's anything that needs to be installed for travis, actually---OpenCL is a built-in part of the OS, isn't it?

Travis.
First step would be getting an OpenMM build working on OSX for Travis.  I'll see if I can port what @hainm did for ParmEd over there.
If we can get Travis building conda packages and uploading them, then we may be able to get rid of the Jenkins machines.

@swails I am interested, thanks for the ping. I haven't made use of conda myself but it seems extremely useful.

I tried tweaking the Travis build script for OpenMM to run on the Mac.  Didn't have much luck, and I don't have time to try too hard, so I'm giving up.

I tried tweaking the Travis build script for OpenMM to run on the Mac. Didn't have much luck, and I don't have time to try too hard, so I'm giving up.

Me neither with cpptraj osx on travis.


Unroll

This would go on top of the macc4 PR.
@tcal-x wdyt? There's only ~0.5M cycles of savings here. How does that balance against the impact on readability of the C code?
Maybe not now, but come back to it later when we need the last few tenths of a percent of performance?

hmm... this is failing timing for me now. Will refactor.


Bump python versioning to [3.8, 3.11]

This PR removes support for Python 3.7, as napari no longer supports it, and begins testing Python 3.11, as it has been released!
Note that the validity of the build process was checked as follows:
git checkout 18a6bba;
python -m build;
cd dist;
tar xzvf napari-imagej-0.1.0.dev0.tar.gz;
mv napari-imagej-0.1.0.dev0 ../old;
cd ..;
git checkout 147-bump-minimum-python-version-to-38;
python -m build;
cd dist;
tar xzvf napari-imagej-0.1.0.dev0.tar.gz;
mv napari-imagej-0.1.0.dev0 ../new;
cd ..;
diff old new;
rm -rf old new;

One remaining concern: the 3.11 builds take a lot longer than the 3.8 builds, due to the generation of the scikit-image wheel. Is this an issue?

About the 3.11 builds: unfortunate that they take so long. Presumably the problem would be fixed once scikit-image makes a new release that includes pre-built wheels for 3.11, right?

Yeah, let's downgrade to 3.10 for right now, and file an issue to bump up to 3.11 once https://github.com/scikit-image/scikit-image/pull/6581 can be depended upon.

Codecov Report
Base: 90.41% // Head: 90.41% // No change to project coverage :thumbsup:

Coverage data is based on head (214fd61) compared to base (06fec2a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #149   +/-   ##
=======================================
  Coverage   90.41%   90.41%
=======================================
  Files          41       41
  Lines        3984     3984
=======================================
  Hits         3602     3602
  Misses        382      382

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[lldb] Capture error messages from parseASTSection to log from caller

This helps fix a lldb console output mixup between the lldb logging and the llvm::dbgs() messages from parseASTSection.

CC @compnerd
This is paired with https://github.com/apple/llvm-project/pull/7198

CC: @xymus

PTAL.

Please test with following PRs:
https://github.com/apple/llvm-project/pull/7198
@swift-ci please test

Please test with following PRs:
https://github.com/apple/llvm-project/pull/7198
@swift-ci please test Linux platform


When cache exist font not included into build with nuxt

Expected Behavior
fonts included.
Actual Behavior
fonts not included. On first build when there no cache fonts included as they should be.
On second build -> font folder not created and no fonts copied.
Is an error being thrown?
No.
Steps to Reproduce
I have a nuxtjs project.  And it have a reference to a @mdi/font
relevant section from nuxt.config
  css: [
    '~/assets/style/app.styl',
<EMAIL_ADDRESS>  ],
  ....
   build: {
    extend(config, ctx) {
      const HardSourceWebpackPlugin = require('hard-source-webpack-plugin')
      config.plugins.push(new HardSourceWebpackPlugin())
    }
  }

Operating System, Node, and NPM dependency versions
Windows 10
Node 10.0.2

"dependencies": {
    "@mdi/font": "2.4.85",
    "lodash-webpack-plugin": "0.11.5",
    "nuxt-edge": "2.0.0-25452542.7f9d16c",
    "vue": "2.5.16",
  },
  "devDependencies": {
    "@sentry/webpack-plugin": "1.5.2",
    "babel-eslint": "8.2.3",
    "babel-plugin-lodash": "3.3.2",
    "chance": "1.0.16",
    "copyfiles": "2.0.0",
    "cross-env": "5.1.6",
    "cssnano": "3.10.0",
    "cypress": "2.1.0",
    "eslint": "4.19.1",
    "eslint-config-prettier": "2.9.0",
    "eslint-config-standard": "11.0.0",
    "eslint-loader": "2.0.0",
    "eslint-plugin-chai-friendly": "0.4.1",
    "eslint-plugin-cypress": "2.0.1",
    "eslint-plugin-import": "2.12.0",
    "eslint-plugin-node": "6.0.1",
    "eslint-plugin-prettier": "2.6.0",
    "eslint-plugin-promise": "3.8.0",
    "eslint-plugin-standard": "3.1.0",
    "eslint-plugin-vue": "4.5.0",
    "eslint-plugin-you-dont-need-lodash-underscore": "6.3.1",
    "glob-all": "3.1.0",
    "hard-source-webpack-plugin": "0.7.2",
    "prettier": "1.13.0",
    "purgecss-webpack-plugin": "1.1.0",
    "stylus": "0.54.5",
    "stylus-loader": "3.0.2",
    "vue-eslint-parser": "2.0.3"
  }

Did you find a solution?

Nope, not using it for now because of this...

Okay I will just use cache-loader with happyPackMode,!
I will continue using hard-source for my dll generation!
Thank you for fast reply :)


Goto definition displays nonexistent file when path contains spaces

Lapce nightly, commit 5bc1e95c67dbed2bd0ea353e7a3d3a55db92de0b
I have a repository placed in path \JA2 1.13 SVN\JA2-xml-editor
Upon clicking goto definition, Lapce attempts to display file in location JA%201.13%20SVN\JA2-xml-editor and shows an empty file with a couple of graphical artifacts.
Opening the actual file from the sidebar explorer, modifying and saving it works fine.

I am not able to reproduce on my Macbook.
Would you be able to provided the OS you are using? From the \ in the file path, I'm guessing you are likely on windows

Windows 10 Pro
Version	10.0.19045 Build 19045

Coming from #1989
Lapce is pretty unusable due to this bug on Windows. I'd be happy to test the fix out once it is available.

I've started working on the fix sometime ago but it's a larger issue of us not translating the paths and just shoving full paths everywhere.


bump ansible collections community.kubernetes to kubernetes.core

The ansible collection community.kubernetes is now being released as kubernetes.core
This ticket is to update:

scaffolding
tests
docs

/assign @VenkatRamaraju


bug: melta.no does not support HTTPS

Meltano Version
N/A
Python Version
NA
Bug scope
Other
Operating System
N/A
Description
http://melta.no works.
https://melta.no does not work:

From @tayloramurphy in https://github.com/meltano/meltano/pull/6748#issuecomment-1248406910:

I've emailed the domain provider to see about getting an SSL Certificate. http://melta.no/ does work and forwards appropriately from what I see. FYI @therebbie I put an inquiry through the Marcaria contact form to go to the hello meltano address.

Code
No response

@WillDaSilva I thought we ran into this a couple of weeks ago and via a different issue (not the one listed above).
@tayloramurphy the hello address forwards (I believe) to Zendesk for such things. In this case, Marcaria responded thusly:

To add HTTPS to a domain, you need hosting. The SSL issuer needs to verify that the domain's host is valid. Maybe check with your current hosting provider about the option to host melta.no, but with just URL forward is not possible even with frame-forward; it will show a not secure connection for melta.no and meltano.com's website.

fwiw @therebbie  Google Domains auto sets up an SSL cert for you an will forward your subdomain (or domains) to whatever pages you want. Maybe others do this too, but might be easier then setting up something on aws and managing it 🤷

To add HTTPS to a domain, you need hosting. The SSL issuer needs to verify that the domain's host is valid. Maybe check with your current hosting provider about the option to host melta.no, but with just URL forward is not possible even with frame-forward; it will show a not secure connection for melta.no and meltano.com's website.

@therebbie @tayloramurphy So if I understand the situation correctly, we are currently using our domain registrar Marcaria to provide URL forwarding instead of managing the DNS directly with our regular DNS provider?
Seems like we should switch away from this URL forwarding service, and onto our regular DNS server. Then since CNAME is not an option because we are pointing it at a specific path under docs.meltano.com, we should set up a 301 redirect with our webserver (whatever that may be) with TLS support. We'll either need to get new TLS certs for melta.no, or get a multi-domain cert that supports both docs.meltano.com and melta.no.

I believe you are correct above, @WillDaSilva ... FWIW, I don't manage any of the actual IT infrastructure but I am always a fan of keeping DNS and registrar registrations with different providers.

Seems like we should switch away from this URL forwarding service, and onto our regular DNS server. Then since CNAME is not an option because we are pointing it at a specific path under docs.meltano.com, we should set up a 301 redirect with our webserver (whatever that may be) with TLS support. We'll either need to get new TLS certs for melta.no, or get a multi-domain cert that supports both docs.meltano.com and melta.no.

docs.meltano.com and friends already use Netlify , and I think' we've already got one or two domain aliases there.  So we can just mirror what we've done in the past and configure melta.no in Route53 and point it to Netlify. Netlify will take care of the SSL coverage and allow us to configure the redirect:

https://docs.netlify.com/domains-https/https-ssl/#domain-aliases
https://docs.netlify.com/domains-https/custom-domains/multiple-domains/#domain-redirects

@WillDaSilva @pandemicsyn so it sounds like we have a resolution here with Route53? Can this get a weight and we can get it prioritized? It's not clear what we'd have to do on the Marcaria side, but it looks like set up nameservers and dns records as needed. Happy to do a quick pair on this so we can get it across the finish line.

I can't find a way to lower the TTL with Marcaria. It's not essential - the current TTL is 21600 seconds (6 hours), so if something goes wrong during the transfer to Route53 melta.no may go down for up to 6 hours. Because of that, I'll do this during the upcoming weekend. That way there'll be minimal disruption is something goes wrong.

Current DNS records:

More useful links:

https://docs.netlify.com/routing/redirects/redirect-options/#domain-level-redirects
https://docs.aws.amazon.com/Route53/latest/DeveloperGuide/migrate-dns-domain-in-use.html

@pandemicsyn @tayloramurphy I'll note every step I take here.
I created a new hosted zone for melta.no in Route53. It comes with default NS and SOA records.
Route53 does not support ALIAS/ANAME records, or CNAME flattening, so we cannot place a CNAME record at the root domain melta.no that points to meltano.netlify.app as we have done with docs.meltano.com and discovery.meltano.com.
I tried creating a new subdomain docs.melta.no which has a CNAME record pointing at meltano.netlify.app, and then made melta.no an alias for docs.melta.no, but Route53 only allows record aliases between records of the same type.
I set an A record that points melta.no at Netlify's load balancer at <IP_ADDRESS>. Unfortunately Netlify does not provide an IPv6 load balancer: https://answers.netlify.com/t/aaaa-ipv6-record-for-apex-domains/11872/9
The nameservers used by Marcaria have been updated to the ones provided by Route53 (from the default NS record):
ns-1581.awsdns-05.co.uk (<IP_ADDRESS>)
ns-141.awsdns-17.com    (<IP_ADDRESS>)
ns-1443.awsdns-52.org   (<IP_ADDRESS>)
ns-552.awsdns-05.net    (<IP_ADDRESS>)

I added a domain alias for melta.no to our Netlify account. I will wait until http://melta.no is using Route53 (hopefully within 6 hours) to update the TLS certificate managed by Netlify to include melta.no in addition to docs.meltano.com and discovery.meltano.com.
The PR https://github.com/meltano/meltano/pull/6865 has been opened to configure the Netlify redirects from melta.no to docs.meltano.com/guide/troubleshooting. If this PR is not merged in before the DNS updates take effect, we should find melta.no routing to docs.meltano.com.

I have renewed the Netlify TLS certificate, and HTTPS is now working for https://melta.no/
At the moment it points to the root of the docs site, but that should be fixed once #6865 has been merged.

#6865 has been merged, and all seems to be working as intended.


File rename

Could you change eda.py to maybe "correlation_eda.py"

I think its fine as eda

it looks too exposed as just eda 🤣

Thanks @thayeylolu
If you guys are all ok with the file rename , can I make changes in the make file for these ?

I think its fine as eda

I mean I would like to get @liannah 's opinion on this, I just think that the whole point of this exercise was to have more distinct file names for everything, and so it might be confusing to have the name "eda" for both ipynb and the script. Could be confusing also because of the fact that the script and the ipynb hold completely different plots.

I think its fine as eda

I mean I would like to get @liannah 's opinion on this, I just think that the whole point of this exercise was to have more distinct file names for everything, and so it might be confusing to have the name "eda" for both ipynb and the script. Could be confusing also because of the fact that the script and the ipynb hold completely different plots.

I agree with @Arushi282. I think eda should be only .ipynb file, and other name for .py file, as they represent two different things.  correlation_eda.py will work @thayeylolu @karanpreetkaur

the

I think its fine as eda

I mean I would like to get @liannah 's opinion on this, I just think that the whole point of this exercise was to have more distinct file names for everything, and so it might be confusing to have the name "eda" for both ipynb and the script. Could be confusing also because of the fact that the script and the ipynb hold completely different plots.

I agree with @Arushi282. I think eda should be only .ipynb file, and other name for .py file, as they represent two different things. correlation_eda.py will work @thayeylolu @karanpreetkaur

The difference between the two files is their file extension name. I.py  and .ipynb naming .ipynb correlation dea will imply we only did a correlation analysis in the notebook. Which we did not


Centralise the chain sync and block fetch codec schemas

Adjust the chain sync and block fetch codec formats slightly. Put the header and block types into a CBOR bytestring wrapper. This has a number of advantages:

It makes it straightforward to pass on the data without decoding it, or delay decodoing.
It allows a single CDDL spec, for many instantiations to different header and block types.
It allows decoders access to the raw input bytes, without any further wrapper. This is needed for decoding the cardano-ledger headers and blocks where we need to annotate with the original bytes for verification purposes.
This has to be done anyway for cardano-ledger, which is a change in CBOR schema. This just centralises that so the schema can be specified in one place.

Note that this PR does not yet adjust the byron proxy correspondingly. That should be done.

It makes it straightforward to pass on the data without decoding it, or delay decodoing.

How so?

It allows a single CDDL spec, for many instantiations to different header and block types.

Right, but the CDDL spec says basically nothing.

It allows decoders access to the raw input bytes, without any further wrapper. This is needed for decoding the cardano-ledger headers and blocks where we need to annotate with the original bytes for verification purposes.

This is possible even without this patch. byron-proxy's ChainSync codec already does it.

In byron-proxy's ChainSync, we use the block type
type Block = Binary.Annotated (Cardano.ABlockOrBoundary ByteString) ByteString

The annotations are needed, because a boundary block doesn't carry all of the content of an old Byron-era EBB. By keeping the annotation of the bytes from which it was decoded, it's possible to store it in the database, and to serve it up to a Byron peer who actually needs it.
Currently, the codec for this type is defined as such in Ouroboros.Byron.Proxy.ChainSync.Types
-- | A `Block` has a `ByteString` annotation which is assumed to be its
-- CBOR encoding, so we drop that into a "CBOR-in-CBOR" (tag 24).
encodeBlock :: Block -> CBOR.Encoding
encodeBlock = Binary.encodeUnknownCborDataItem . Lazy.fromStrict . Binary.annotation

decodeBlock :: EpochSlots -> CBOR.Decoder s Block
decodeBlock epochSlots = do
  bytes <- Binary.decodeUnknownCborDataItem
  case Binary.decodeFullAnnotatedBytes "Block or boundary" internalDecoder (Lazy.fromStrict bytes) of
    Right it  -> pure $ Binary.Annotated it bytes
    Left  err -> fail (show err)
  where
  internalDecoder :: Binary.Decoder s (Cardano.ABlockOrBoundary Binary.ByteSpan)
  internalDecoder = Cardano.fromCBORABlockOrBoundary epochSlots

Changing decodeBlock to fit the new model in this patch for the ChainSync codec is no problem:
-- Note the new ByteString parameter: it's supplied by the ChainSync codec
-- scheme, got from tag 24 CBOR-in-CBOR as above in decodeBlock
decodeBlockNew :: EpochSlots -> ByteString -> CBOR.Decoder s Block
decodeBlockNew epochSlots bytes =
  case Binary.decodeFullAnnotatedBytes "Block or boundary" internalDecoder (Lazy.fromStrict bytes) of
    Right it  -> pure $ Binary.Annotated it bytes
    Left  err -> fail (show err)
  where
  internalDecoder :: Binary.Decoder s (Cardano.ABlockOrBoundary Binary.ByteSpan)
  internalDecoder = Cardano.fromCBORABlockOrBoundary epochSlots

Fine. But what about encodeBlock?
encodeBlockNew :: Block -> CBOR.Encoding
encodeBlockNew = ???

I'm stuck here. If it's a boundary block, the only thing I can legitimately do is encode tag 24 CBOR-in-CBOR because I just don't have enough information to get a proper CBOR.Encoding (I only have the bytes).

This has to be done anyway for cardano-ledger, which is a change in CBOR schema. This just centralises that so the schema can be specified in one place.

AFAICT nothing in this patch is actually necessary to do what we need. Another viable solution is to leave the ChainSync codec scheme as it is, and make it easy (as I believe it already is) to define a CBOR-in-CBOR codec for the particular choice of header type (as we already do in byron-proxy).


Three node cluster doesn't have all peers set correctly?

Hi.  I was fiddling with a 3 node cluster and saw some odd behavior on commit 097fcb2dedd575ee738a9c3cbd4e4e03492ae066.

For all nodes, rm -rf ./data, just in case.
For all nodes, start Erlang with short node names p0@sbb4, p1@sbb4, and p2@sbb4. (Your hostname will vary, of course.)  E.g. erl -sname p0 -pz ebin _build/*/*/*/ebin
For all nodes, run plumtree:start().
On node p0, run: [{goo, X} ! hello || X <- [p1@sbb4, p2@sbb4] ].
On node p0, run, [plumtree_peer_service:join(X) || X <- nodes()].

Then, if I run the following:
f(All).
All = lists:sort([node()|nodes()]).
[{X, Y, catch plumtree_broadcast:debug_get_peers(X, Y)} || X <- All, Y <- All].

I see the following oddity on node p0:
[{p0@sbb4,p0@sbb4,{[],[]}},
{p0@sbb4,p1@sbb4,{[],[]}},
{p0@sbb4,p2@sbb4,{[],[]}},
{p1@sbb4,p0@sbb4,{[p2@sbb4],[p0@sbb4]}},
{p1@sbb4,p1@sbb4,{[p2@sbb4],[p0@sbb4]}},
{p1@sbb4,p2@sbb4,{[p2@sbb4],[p0@sbb4]}},
{p2@sbb4,p0@sbb4,{[p0@sbb4],[p1@sbb4]}},
{p2@sbb4,p1@sbb4,{[p0@sbb4],[p1@sbb4]}},
{p2@sbb4,p2@sbb4,{[p0@sbb4],[p1@sbb4]}}]
... which means that replication of any metadata put() on node p0 is needlessly (?) delayed by 10s of seconds before appearing on p1 and p2.
If I stop & start the VM of node p0 and restart Plumtree, the peer sets are non-empty as expected.

Was on vacation past few days. Looking into this tonight. Thanks for opening the issue!

what's the status of this issue?


docker composition not working on M1 arm64 architecture

Error is coming up with the following:
❯ docker compose up -d
image with reference postgres:14.3 was found but does not match the specified platform: wanted linux/amd64, actual: linux/arm64/v8

Apparently RedHat is building only linux/arm64arch so we need to build our own images. We can follow official instructions https://www.keycloak.org/server/containerson running Keycloak in a container

@krishnaglodha can you check with that variable if then it works?

Thanks, will check and revert

yes! It works

I believe we should remove platform from docker-compose all together. thoughts ? @francbartoli

I don't see it in develop


Adding base domain and region to secrets

Summary | Résumé
Required for the helmfile work - the base domain and region are set as secrets.
Related Issues | Cartes liées

https://app.zenhub.com/workspaces/notify-planning-core-6411dfb7c95fb80014e0cab0/issues/gh/cds-snc/notification-planning-core/339

Test instructions | Instructions pour tester la modification

TF Apply Works
Notify Helm Chart PR works

Release Instructions | Instructions pour le déploiement
None.
Reviewer checklist | Liste de vérification du réviseur

[ ] This PR does not break existing functionality.
[ ] This PR does not violate GCNotify's privacy policies.
[ ] This PR does not raise new security concerns. Refer to our GC Notify Risk Register document on our Google drive.
[ ] This PR does not significantly alter performance.
[ ] Additional required documentation resulting of these changes is covered (such as the README, setup instructions, a related ADR or the technical documentation).

⚠ If boxes cannot be checked off before merging the PR, they should be moved to the "Release Instructions" section with appropriate steps required to verify before release. For example, changes to celery code may require tests on staging to verify that performance has not been affected.

This will fix our helm issues in the workflow?

Yes - the workflow is complaining that these secrets don't exist


Build improvements

Add makefile target for running k8sinfra-gen.
Allow user to specify output directory in configuration file, and configuration file on cmdline

This fixes #144.

Improved configuration yaml file property casing.
Add new dependency because golang filepath.glob doesn't support recursive matching.

Pull Request Test Coverage Report for Build 133636153

-4 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
190 unchanged lines in 4 files lost coverage.
Overall coverage increased (+5.9%) to 39.416%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hack/generator/pkg/config/configuration.go
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/zips/env.go
3
0%

pkg/zips/types.go
25
54.76%

pkg/zips/client.go
75
3.96%

pkg/zips/template.go
87
26.32%

Totals

Change from base Build 131581462:
5.9%

Covered Lines:
769

Relevant Lines:
1951

💛  - Coveralls


Replace custom css for form elements on apply with Tailwind

Fixes: #3439

Nice work!
The old radio and checkbox styles have a blue border. That is in this PR replaced with a more standard grey and they are slightly smaller as well.
I do not have a preference for one over the other but I prefer not to change things to much, unless we have a good reason.

It could be the missing accent color on the check box, let me take a stab at it. Definitely not intensional, i get you point.

Updated in 4eff69c4d


PowerToys run dark mode needs contrasting outline

When running in dark mode, if PowerToys Run is opened over a dark window (black or other dark color), it can get lost against the background.  A contrasting outline around the Run prompt would be helpful.

We're tracking this in #2428


Dynamically overriding rate limit with data from headers and dynamic metadata

Title: Dynamically overriding rate limit with headers and dynamic metadata
Description:
Hello!
Apologies in advance if this is something trivial. We are spiking use of Envoy to replace our in-house api-management solution.
We're trying to make use of ExtAuthz filter for api key validation and ratelimit filter for limiting. The ratelimit should make use of information returned by our ExtAuthz filter in headers to set different rate limits per api key. The ratelimit makes use of https://github.com/envoyproxy/ratelimit as the rate_limit_service
At first stage of our spike we achieved a global rate limiting, where each api key got their own quota tracked. This was a good starting point. But this meant that all api keys get the same rate limit (e.g. all got 10 requests per minute as an example)
The next step was to make use of the quota information retrieved from the key validation API. Our use case allows different clients represented by api keys to have different quotas.
It was to our delight to find that we can make use of ratelimit override functionality (https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/rate_limit_filter#rate-limit-override). So we tried to set it up.
We understand the ratelimit filter can't make use of override data in headers, but it can get it from dynamic metadata.
The simplest way I can describe what we try to do is this:

A client makes a request to an API with an api key attached in a header.
The request reaches envoy
External auth filter type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.ExtAuthz configured to make a call to our "sidecar" application, where the api key is extracted from headers and passed down to our Key Validation API
The Key Validation API responds with status of the api key (success or failure). On success is attaches information about quota information - quota (Integer) and time unit (enum containing MINUTE, HOUR, DAY etc)
We grab the information from headers using header_to_metadata filter
Then this metadata is passed onto ratelimit override.
Ratelimit service makes retrieves the override
Ratelimit filter makes decision to let the request through or not
The request reaches backend (or not)
Response is sent back (either 429 code or another response code from backend)

Unfortunately the rate limits are not overridden at the service level and we still get the global rate limit which was set in the https://github.com/envoyproxy/ratelimit service. So all api keys (clients) get the same rate.
We have added a simple lua filter to confirm the dynamic metadata is attached, and in fact it is. But the Rate Limit Override does not seem to kick in. Reading through the docs it looks like Rate Limit Override expects the metadata to come in as a struct containing requests_per_unit and unit fields - I'm thinking the problem with our config is around this - i.e. we're just setting primitive values on dynamic metadata and not struct.
Is our scenario possible to achieve with rate limit override?  Any help would be appreciated 😌
We were also thinking of having another external filter which could read the quota headers already attached and make the call to ratelimit service for us, but we would obviously want to avoid this as it requires more code and therefore maintenance in future.
Below is our config:
static_resources:

  listeners:
  - name: listener_ingress
    address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 10000
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          stat_prefix: ingress_http
          http_filters:
          - name: envoy.filters.http.ext_authz
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.ExtAuthz
              http_service:
                authorization_request:
                  allowed_headers:
                    patterns:
                    - exact: "x-api-key"
                server_uri:
                  uri: <IP_ADDRESS>:3000
                  cluster: ext-authz
                  timeout: 1s
              # Default is 200ms; override if your server needs e.g. warmup time.
              include_peer_certificate: true
          - name: envoy.filters.http.header_to_metadata
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.http.header_to_metadata.v3.Config
              request_rules:
                - header: product-quota
                  on_header_present:
                    # metadata_namespace: envoy.lb
                    key: rate.requests_per_unit
                    type: STRING
                  on_header_missing:
                    # metadata_namespace: envoy.lb
                    key: rate.requests_per_unit
                    value: '10'
                    type: STRING
                  remove: false
                - header: product-quota-time-unit
                  on_header_present:
                    key: rate.unit
                    type: STRING
                  on_header_missing:
                    # metadata_namespace: envoy.lb
                    key: rate.unit
                    value: 'MINUTE'
                    type: STRING
                  remove: false

          - name: envoy.filters.http.lua
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.http.lua.v3.Lua
              inline_code: |
                -- Called on the request path.
                function envoy_on_request(request_handle)
                  local requests_per_unit = request_handle:streamInfo():dynamicMetadata():get("envoy.filters.http.header_to_metadata")["rate.requests_per_unit"]
                  local unit = request_handle:streamInfo():dynamicMetadata():get("envoy.filters.http.header_to_metadata")["rate.unit"]
                  request_handle:logInfo("requests_per_unit value: " .. requests_per_unit)
                  request_handle:logInfo("unit value: " .. unit)
                end

          - name: envoy.filters.http.ratelimit
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.http.ratelimit.v3.RateLimit
              domain: apis
              stage: 0
              request_type: external
              failure_mode_deny: true
              rate_limit_service:
                grpc_service:
                  envoy_grpc:
                    cluster_name: ratelimit
          - name: envoy.filters.http.router
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains: ["*"]
              routes:
              - match:
                  safe_regex:
                    google_re2: {}
                    regex: "^/(.*)$"
                route:
                  regex_rewrite:
                    pattern:
                      google_re2: {}
                      regex: "^/httpbin(.*)$"
                    substitution: \1
                  host_rewrite_literal: httpbin.org
                  cluster: egress-httpbin
              rate_limits:
                - stage: 0
                  actions:
                    - header_value_match:
                        descriptor_value: api-key-limit
                        headers:
                          - name: ":path"
                            prefix_match: "/"
                    - request_headers:
                        header_name: "X-Api-Key"
                        descriptor_key: api-key
                  limit:
                    dynamic_metadata:
                      metadata_key:
                        key: envoy.filters.http.ratelimit
                        path:
                          - key: rate

          access_log:
            - name: envoy.access_loggers.file
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
                path: "/dev/stdout"

Thanks !

@RafalJachimczyk I'm trying to achieve the same thing here. We're you able to figure this out?

@RafalJachimczyk I'm trying to achieve the same thing here. We're you able to figure this out?

I wrote an article about something similar. It’s not a global rate limit. It is a local override. Hope it might be helpful https://medium.com/@chrislam67/rate-limiting-on-envoy-by-api-key-e623ea7267fa


Help Button is not directing to right URL in properties menu of circuit element.

Describe the bug
help button is not directing to right URL in properties menu of  circuit element in simulator.
To Reproduce
Steps to reproduce the behavior:

Select an circuit element
Click on help from properties menu
See error

Expected behavior
It should be directing to circuitverse docs
Screenshots

Are you working on this issue? (Yes/No)
yes

@CircuitVerse/circuitverse-core-team What should be the ideal path for help link?

Can I contribute to it

circuitverse docs are in the process of refactoring. if refactoring is finished you can start to change the links
you can see the difference, so each  doc is present in particular chapter folder and links are different now
https://docs.circuitverse.org/#/chapter4/3output?id=output

@pavanjoshi914  is this issue assigned to me?

circuitverse docs are in the process of refactoring. if refactoring is finished you can start to change the links
you can see the difference, so each doc is present in particular chapter folder and links are different now
https://docs.circuitverse.org/#/chapter4/3output?id=output

should I wait for the refactoring to be done?

@pavanjoshi914 is this issue assigned to me?

@kushalShukla-web I think  @aashimawadhwa  already mentioned working on this issue
Are you working on this issue? (Yes/No)
yes

I think a great place to start is going with issues labelled good first issue
if someone is not working on that issue, or if issue is pending for a long time you can start working on such issue without waiting for issue being assigned
hope this helps!

circuitverse docs are in the process of refactoring. if refactoring is finished you can start to change the links
you can see the difference, so each doc is present in particular chapter folder and links are different now
https://docs.circuitverse.org/#/chapter4/3output?id=output

should I wait for the refactoring to be done?

cc @Shivansh2407

circuitverse docs are in the process of refactoring. if refactoring is finished you can start to change the links

Already deployed updated docs, so we can start fixing links

@aashimawadhwa  you can start working on this

Fixed in #3215


test-branch/execute-job/wrong-commit-sha-on-pr/0b549df8-d5e9-4634-8f3a-2ba9df2c872a

PR for testing 323cd6605b0598ca586d697ebdc0a0105d0deef9

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


developerPayload for subscriptions?

I noticed that normal purchases have the ability to pass in the developerPayload, although subscriptions do not. Is there a way to currently do this?
Use case would be server to server authentication

Currently, developerPayload is affected on consumePurchase and acknowledgePurchase. Could you describe more specifically?

The issue that I am having currently, is that I am trying to implement server side authentication, without having to make a handshake with the client to confirm the purchase. The issue I have currently, is that my real time notification sent from pub sub is being sent before the developerpayload is set in const test = await RNIap.acknowledgePurchaseAndroid(purchase.purchaseToken, tokenHeader);.
Either I would have to do setTimeout on the server, for like 10 seconds, or somehow get the developerpayload as the purchase is being sent out


[Feature Request]: Suggestions for Implementing Traceable Query Features

Do you need to file an issue?

[x] I have searched the existing issues and this feature is not already filed.
[ ] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[x] I believe this is a legitimate feature request, not just a question. If this is a question, please use the Discussions area.

Is your feature request related to a problem? Please describe.
Introduction and Gratitude:
Graphrag has demonstrated remarkable adaptability, robustness, and efficiency in handling multi-text queries. Through this project, I have significantly enhanced the efficiency of our department's software development, receiving high praise during customer deliveries. Over time, as project files have accumulated to a substantial level of tens of millions of words (approximately 13,900 books), the advantages of Graphrag have become even more apparent, offering many novel insights and perspectives, which have greatly impressed us.
Everyone knows that innovation is extremely challenging, and we hope to capture these shining moments. We hope the official team can introduce a feature for tracing responses based on Graphrag. For each detailed response (especially when using OpenAI GPT-o1 for discussions), if it could be traced back to the original text, chapter, or other related original content on a sentence-by-sentence basis, it would potentially be more helpful for usage and verification.
Describe the solution you'd like
Setting up traceable indexes during the construction process? Or adding more information in summaries to enable cross-referencing within the community? I haven't thought of more ideas yet.
Additional context
No response

Hi @shaoqing404, I've also been on the lookout for something similar.
In the meantime (admittedly on a much smaller scale than you describe) I have used post-processing scripts to create a mapping from the responses back to the original sources (via entities, relationships and text snippets) using the response objects. It's far from perfect but has been very useful for corroborating outputs for research. You can check out the scripts at this repository, in case they are helpful in any way.


Table format is wrong

@GeorgeH-MS : Thanks for your contribution! The author, @brenduns, has been notified to review your proposed change.


Missing a file for the llama3-8b-instruct-mnn model

Hello, I think one file is missing for the llama3-8b-instruct-mnn model

not missing file, just process info has some inaccurate

OK, clear.
Why does the model always answer every question the same way?

Q: who is the best football player in the world?
A: It seems like you started to ask a question, but didn't complete it.
If you could finish your question, I'd be happy to help you out!


Equal sum partition in Java

💥 Proposal
I want to add the code for equal sum partition in Java
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


6/23 PM Publish

Docs Build status updates of commit 79322e3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azps-7.5.0/Az.Purview/New-AzPurviewTrigger.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Purview/New-AzPurviewTriggerObject.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-8.0.0/Az.Purview/New-AzPurviewTrigger.md
:bulb:Suggestion
View (azps-8.0.0)
Details

azps-8.0.0/Az.Purview/New-AzPurviewTriggerObject.md
:bulb:Suggestion
View (azps-8.0.0)
Details

azps-0.10.0/Az.Compute/New-AzVmss.md
:white_check_mark:Succeeded
View (azps-0.10.0)

azps-7.5.0/Az.ADDomainServices/Get-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ADDomainServices/Remove-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ADDomainServices/Update-AzADDomainService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.Aks/Get-AzAksNodePoolUpgradeProfile.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.Aks/Get-AzAksUpgradeProfile.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.Aks/Start-AzAksCluster.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.Aks/Stop-AzAksCluster.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.AppConfiguration/Get-AzAppConfigurationStore.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.AppConfiguration/New-AzAppConfigurationStoreKey.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.AppConfiguration/Remove-AzAppConfigurationStore.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.AppConfiguration/Update-AzAppConfigurationStore.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ApplicationInsights/Get-AzApplicationInsightsWebTest.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ApplicationInsights/Remove-AzApplicationInsightsWebTest.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.ApplicationInsights/Update-AzApplicationInsightsWebTestTag.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.Automation/Start-AzAutomationDscCompilationJob.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.BareMetal/Update-AzBareMetal.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.BotService/Get-AzBotService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.BotService/Remove-AzBotService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.BotService/Update-AzBotService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

azps-7.5.0/Az.CloudService/Get-AzCloudService.md
:white_check_mark:Succeeded
View (azps-7.5.0)

This comment lists only the first 25 files in the pull request.

azps-7.5.0/Az.Purview/New-AzPurviewTrigger.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Purview/New-AzPurviewTriggerObject.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-8.0.0/Az.Purview/New-AzPurviewTrigger.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-8.0.0/Az.Purview/New-AzPurviewTriggerObject.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'any' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


bug: ion-menu is not visible after page transition on iOS

Bug Report
Ionic version:

[x] 4.10
Current behavior:

When you open ion-menu and navigate to a page and then go back the menu is no longer visible but if you click anywhere on the screen it then reappears and closes immediately.  This only occurs on iOS.
Expected behavior:

The menu should be visible after opening it and navigating to a page and back.
Steps to reproduce:

Create an Ionic app with the iOS platform and add an ion-menu that includes an ion-item that uses routerLink to navigate to another page.  Then click an ion-back-button.
Other information:

This only occurs on iOS and I believe this began with with version 4.8 or 4.9.
Ionic info:

Ionic:

   Ionic CLI                     : 5.4.1 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.10.0
   @angular-devkit/build-angular : 0.802.1
   @angular-devkit/schematics    : 8.2.1
   @angular/cli                  : 8.2.1
   @ionic/angular-toolkit        : 2.0.0

Capacitor:

   Capacitor CLI   : 1.2.1
   @capacitor/core : 1.2.1

Cordova:

   Cordova CLI       : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms : not available
   Cordova Plugins   : not available

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   ios-deploy : 2.0.0
   NodeJS     : v10.15.0 (/usr/local/bin/node)
   npm        : 6.11.3
   OS         : macOS Mojave
   Xcode      : Xcode 11.0 Build version 11A420a

Thanks for the issue. Can you provide a repo with the code required to reproduce this issue?

the same for me, 2 ion-menus are super buggy on iOS, Ionic 4.10.0

I tested the side menu on an iphone 11 and was unable to reproduce the issue.  So far this only appears on my iphone X.  Not sure if that helps but wanted to point it out.

Even on PWA it works wrong on any iPhone devices.
Try yourself on iOS Safari: https://adam.app.andrzej-ondra.botyourself.net

The very last Ionic version where our apps are not affected is 4.8.1

Can you provide the code required to reproduce this issue? I am unable to reproduce the issue using the demo site you provided.

Sure, I will try to prepare some MVP.

We found the issue. It was on our side, but would be great to secure such case with a default fallback:
<ion-menu type="undefined"

Ah ok glad that issue is resolved. It looks like you're passing in "undefined" as a string which may be why the default fallback is not being applied.
@kyleabens Are you still able to reproduce this issue?

@liamdebeasi  I can only reproduce the issue surrounding ion-menu on an iPhone X.  Every other device works fine so I’m not going to waste anymore time on it.  Thanks!

I managed to finally find to cause of that: it happens when you have 2 menus with default types. If one of them is set as type="push" (like in docs example https://ionicframework.com/docs/api/menu) it works properly.
Reproduction steps: iOS device (webapp or native) or iOS simulator

swipe open left menu
navigate to different page
try to swipe open right menu, its impossible to open

Here you go https://github.com/kleeb/ionic-issue-19500
Video: https://drive.google.com/open?id=1K8ufJulCBTqxsgWo7FK5_t5FyFuSGc_8

@kyleabens @liamdebeasi
are you planning to check this out ?

Hi there,
I am not able to reproduce the behavior you are describing on an iOS device using the provided repo. Additionally, I do not have permission to view the video you have posted. Can you verify the reproduction as well as the video permissions?

@liamdebeasi sorry, now its public
pls check

@liamdebeasi sorry, now its public
pls check
@kleeb
it looks like problem with swipe  is resolved  by you i cannot reproduce problem after your fix

@kleeb Thanks for the follow up. I still cannot reproduce this issue. What version of iOS are you seeing this on?

definitely on the iOS 12.1, but I think our QA also reproduced in 13

currently its not a showstopper anymore, as we started using push type for the right menu, but if its like that would be nice to add such info in the docs, so that other devs are not wasting time on trying to fix such a hard-to-guess-what bug

Yeah I definitely understand that. Unfortunately we're not really able to document/fix much if we can't reproduce it. If you manage to find a more consistent way to reproduce the issue, feel free to open a new issue and I can take a look. 🙂

I think I did everything I could - created a repo based on your sample project, added a video, reproduced on simulator, and independently our QA also reproduced that on a device. Have you followed exactly what I did in the sample video?

I used repository provided by kleeb, run project and opened it externally (iphone6, ios 12.3.1) and Im experiencing the same issue. Steps to reproduce:

Open left side menu with button
Select list
try to swipe open right side menu
menus start to behave strange

Sometimes it takes longer to reproduce it - please try to open items from left menu, then right menu and left menu again. After a few attempts you will surely see the buggy behaviour.


[SMALL] RevEng: Remove unused usings from EntityType template

Fixes #3229.


GopherCon 2018: Matt Layher - Implementing a Network Protocol in Go

This is a great talk https://www.youtube.com/watch?v=pUaFW98V1Sc. The topic is very similar to what we're building. He even talks about go-fuzz.
One thing that came to mind. Our interface names are Decode and Serialize. He's using Marshal and Unmarshal like the json package https://golang.org/pkg/encoding/json/ and the proto package https://godoc.org/github.com/golang/protobuf/proto.
Therefore I'd like to propose changing our interface before releasing version 1.

Nice proposal. I saw it before and it made me more eager to work on this project :smiley:
Actually, the words Decode and Serialize are inspired by google/gopacket, and for me it was more familiar English word for this purpose.
However, I also think following the standard libraries and the leaders like @mdlayher is better, so let me see some examples around to consider more about it.

FYI, I created GitHub project to collect items to be done before first release, not to miss something :100:
https://github.com/wmnsk/gopcua/projects/2

Glad you all enjoyed the talk!  Ultimately I think it's just important to keep your naming consistent where possible.  And definitely run go-fuzz on your code!  Best of luck on the project.

Re-considered in the future, as the PR#124 gives a whole new paradigm.


Use controller-gen to generate CRDs instead of writing them by hand

Currently, we write CRDs by ourselves, it's tedious and error-prone. We should use tools like controller-gen to generate them so that we can focus on the changes of go data structures, and let it do the rest jobs.

@gxthrj @tao12345666333 @lingsamuel

To do so, we need to remove usage of:
any type (interface{}), float64 (or enable allowDangerousTypes=true flag).
And add markers like this.

To do so, we need to remove usage of:
any type (interface{}), float64 (or enable allowDangerousTypes=true flag).
And add markers like this.

Got it.

xref: https://github.com/apache/apisix-ingress-controller/pull/689#issuecomment-927212290

If any CRD validation rules cannot be represented by markers, we should ensure that they are migrated to our code.


Perfomance

Your script is great, but it's rather heavy.
One select with only 5 options can add even 50 watchers.
Maybe there is something to optimize? Using ng-if instead of ng-show? One time bindings?

What version of the library are you using?

v0.9.6

+1

+1 one time binding is the key, meanwhile you can use https://github.com/tadeuszwojcik/angular-once

One time bindings are available in angular 1.3 {{::oneTime}}

there is no where to use the one time bindings with ui-select. I love the way it functions but it is very heavy with all the watchers

The number of watchers is a big killer with ui-select. I have a page with about 10 ui-select directives each with about 5 items in each. This is contributing to a watcher count in the thousands and the application performance is very poor.
The ability to do one-time bindings to reduce this count, as well as a straightforward way to disable animations would make a huge difference to ui-select.

+1

+1

+1

See #1096 / #1108

Thanks for the refs to that issues

+1

Before and after opening the UI select:

OMG!


fix: Linkerd uses helm template commands to generate certs. This happ…

…ens on every sync, so it goes into a loop. This commit adds the necessary ignore differences specification to avoid an out of sync loop.

:tada: This PR is included in version 1.0.0-alpha.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


"this app does not provide any candidate for installation"

I'm trying to install Bottles via Zap as a possible alternative to Flatpak.
I don't prefer how Flatpak keeps everything in a sandbox as it makes it much harder to manage everything for me, and disk usage is much higher than using standard system libraries. Zap being closest to a user-level non-sandboxed package manager makes it easier for me in general.
However, whenever I put in the command
zap install bottles
as it tells me to both after searching the package with Zap, or on the official Bottles website, I get the error
[FATAL] 2022/04/04 16:22:28 main.main:main.go:163 this app does not provide any candidate for installation
Is there something I'm doing wrong with Zap?
Thanks in advance.

@cmdrneon maybe this issue is related https://github.com/bottlesdevs/Bottles/issues/1272


[Bug]: Two addresses provided on the official website cannot be opened in iOS 16

Attach (recommended) or Link to PDF file
https://mozilla.github.io/pdf.js/legacy/web/viewer.html
https://mozilla.github.io/pdf.js/web/viewer.html
Web browser and its version
latest
Operating system and its version
ios 15.5
PDF.js version
15.5
Is the bug present in the latest PDF.js version?
Yes
Is a browser extension
No
Steps to reproduce the problem
1.open  https://mozilla.github.io/pdf.js/legacy/web/viewer.html
2.
What is the expected behavior?
Can open correctly.
What went wrong?

Link to a viewer
No response
Additional context
No response

https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support, which the issue template explicitly asks you to check before reporting a bug.


Add dsari as software which directly provides Prometheus metrics

FYI, capitalization (or lack thereof) is correct.

This appears to be an exporter rather than software directly exposing metrics, which goes in a different section.
I'm not sure about dsari_run_count, the _count suffix is for summaries not counters. With this sort of system this is also usually a gauge, as jobs get GCed over time.

@rfinnie Could you move this into the exporters list and rebase?

Thank you for your contribution. Closing this pull request due to lack of recent activity, but we'd be happy if you want to pick up the work again. In that case please make the requested changes, ensure all tests pass, and re-open the PR.


Cleaned Up List Item Text Update Logic

Changes Made:

Fixed an example that actually included codeblocks to no longer include them
Refactored and reworked update list item text logic to let unist do the work of finding the list items and then getting the paragraph kids of them as the actual text for them instead of all text inside a list item

I need to check to see why the text of the task itself is including the task list indicator. It should not be included, which implies that I need to add the column and offset together to get the position, but I am not certain of this yet. I will test somethings out and see what happens.


move adapters to home repo

addressing #489

@GrahamCampbell StyleCI has a bug. The headers were duplicated because they are different.
Please let me know, if not i can run the gush command to format the headers since i remember it does it correctly. Also i had to download by hand the patch and apply it, i remember we have a fabpot command to do the apply patch automatically. Maybe you want to send a command to do that with SyleCI, i did not see easily how to send a PR.

@sstok not sure why scrutinizer crashes. Any hint?
Other than this we can merge this :+1:

also since i believe it is a waste of time to maintain the other repos let's remove them from gush main organization and also deprecate them on packagist. I think that is all that is left. Tests and further refactoring can wait for follow up PRs.

IMO the 'ThirdParty' namespace doesn't really make sense, since the adapters is not part of a 3rd party, but part of Gush core and now just lives in the main repo.

I agree with @pierredup on the namespace part.
But we can't delete repositories just yet as some still use Gush with Composer (global).
We should keep them for about a year and then remove them.

we can change to any other name you want without problems. This is more so that we get the repos in.

I think there was a ticket in which we discouraged the global composer for gush. Recommended ways are clone and symlink AND phar.

hmm :) what is the common name used in other situations?
We can use the Adapter sub namespace and change it once we have better name.
Just make sure they are marked @internal so we can easily change them (if anyone would use Gush as a library ;) )

I think there was a ticket in which we discouraged the global composer for gush.
Hmm, OK :)

Adapter conflicts with Adapter folder already existing. That is why I choose the non nice Third Party.

we can definitely not put the generic Adapter classes together with the third party adapters implementations.

looks like scrutinizer passed


getting value coordinate error everytime also inpainting is broken

https://github.com/AUTOMATIC1111/stable-diffusion-webui/
this is the repo where you should ask the question.

This is weird I got the same error. And I run the same repo on my machine (updated both to the current repo). But on my machine it doesn't produce this error.
Traceback (most recent call last):   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 45, in f     res = list(func(*args, **kwargs))   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/call_queue.py", line 28, in f     res = func(*args, **kwargs)   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/img2img.py", line 152, in img2img     processed = process_images(p)   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 464, in process_images     res = process_images_inner(p)   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 532, in process_images_inner     p.init(p.all_prompts, p.all_seeds, p.all_subseeds)   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/processing.py", line 814, in init     mask = mask.crop(crop_region)   File "/usr/local/lib/python3.8/dist-packages/PIL/Image.py", line 1207, in crop     raise ValueError("Coordinate 'right' is less than 'left'") ValueError: Coordinate 'right' is less than 'left'

I'll check it out

I saw on Automatic repo that it's still a bug tho. here

if you use the latest colab it should work without issue, i'm testing inpainting right now and it works

Still broken


Remove custom routing from API base controller

Purpose or Intent
Now that all API controllers respond to #show, #update and
#destroy which all do the appropriate thing for their corresponding
resource, we don't need the custom routing in the base controller.
@miq-bot assign @abellotti
@miq-bot add-label api, refactoring
/cc @jrafanie :fire: :scissors: :fire:

🎶  to my 👂  love those 👶 steps ...

😍


Add graphiql.app v0.3.1

FYI @kevinold, @skevy, @KyleAMathews

Thank you for the contribution. It needed some fixes, so they were made in a78edc13b600abb102526455e1846bb53d645887. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Thanks!

@tribou nice!


Fixed delta compression

This modifies the integer bitpacking compression to make use of a fixed offset when values are all large, and if it will reduce the number of bits required to store each value.
The offset is stored in the ColumnChunk metadata (increasing the size of that by 8 bytes), and removed from values when compressing and added back during decompression.
Decompression is done in-place, but does require an extra pass over the decompressed data when the offset is non-zero. Compression when the offset is non-zero is done using a small temporary buffer.
This should be effective at reducing the size needed for data such as timestamps, which usually fall into a relatively small range of values, but with a relatively high minimum value, and will still have very good random read/write performance. Large numeric IDs with a fixed number of digits should also benefit from this.
If also functionally implements constant compression for integers, as if all values are the same the data can be stored in 0 bits per value with the actual value stored as the offset.
I'll try and put together a quick benchmark with a generated dataset tailored to show the performance difference when this is used.

I'm informed that this is Frame of Reference encoding (without delta coding): https://lemire.me/blog/2012/02/08/effective-compression-using-frame-of-reference-and-delta-coding/
I've updated some of the terminology in this PR to reflect that.


Repo description url is broken

The repo description url at the top of the page is broken.
It should be: http://piotrmurach.github.io/finite_machine/
:smile:

Hi Matteo,
Today I've changed my GitHub username and broke couple of things in the process :smile:
This is fixed now.


Extend PyG dataloader support

It would be great if closer integration with PyTorch Geometric could be provided for the various TDC datasets. While it is straightforward to convert e.g. SMILES into the graph format for single instances (not sure however how to interpret the node features or include edge features and labels?) via tdc.chem_utils.MolConvert, it is not clear to me how this can be turned into a PyTorch Geometric compatible dataset/dataloader including the desired features. E.g. the OGB repository provides support as follows:
from ogb.graphproppred import PygGraphPropPredDataset

dataset = PygGraphPropPredDataset(name='ogbg-molhiv' )
split_idx = dataset.get_idx_split()
batch_size = 32
dl_tr = DataLoader(dataset[split_idx['train']], batch_size=batch_size, shuffle=True)
dl_val = DataLoader(dataset[split_idx['valid']], batch_size=batch_size, shuffle=False)
dl_te = DataLoader(dataset[split_idx['test']], batch_size=batch_size, shuffle=False)

Maybe I am missing something here and the above is already possible via the otherwise great TDC library?

hey, thanks for the suggestion! I believe we indeed have some basic node/edge attributes (atom/bond index, atom charge/chirality, etc) supported in the MolConvert class (https://github.com/mims-harvard/TDC/blob/afe08ca727cda11b3a65af5caadf16f8352878a9/tdc/chem_utils.py#L3449). To clarify, are you saying we should provide more options for the atom/bond features?

Note that you can also do something like this:
from tdc.single_pred import ADME
data = ADME(name = 'Caco2_Wang', convert_format = 'ECFP4')

It will create a separate column in the dataframe with the converted format.
However, as our current dataloader in default returns pandas dataframe, some of the convert format wouldn't work, for example, the DGL graph object would return error when we put it in a dataframe. So, in these cases, it would be ideal to simply return the DGL/PyG objects. We would work to integrate this feature in the future.


Implement a SHA256 based brain wallet lookup

Provide an implementation which reads rows of a file and create sha256 hash which is used as private key.
See:

https://brainwalletx.github.io/
https://allprivatekeys.com/brain-wallet-cracker

Word-Lists:

https://github.com/dwyl/english-words

It's implemented. For example, refer to: https://github.com/bernardladenthin/BitcoinAddressFinder/blob/ad9f09343659a75c4444480225be63a091f4fc00/examples/config_Find_SecretsFile.js. The mode for pre-hashed SHA256 values is set to "SHA256".


Remove interface MatchBuilder

The interface MatchBuilder is unused and should be removed.

Done: https://github.com/kotlintest/kotlintest/commit/adf6aa5975ddd59b55bfceddc2b4914019c6f4a2


[webview_flutter] There will be a black screen during the transition of plugin loading Url

Hey, brother, I used the plugin [webview_flutter] to load the target Url, and found that there will be a brief black screen situation, passing by.
On the Github plugin [webview_flutter], I did not find a place to submit my question, I can only submit it here, sorry.
My local Flutter environment:
heliquan@Mac  ~  flutter doctor -v
[✓] Flutter (Channel stable, v1.17.0, on Mac OS X 10.15.5 19F101, locale
    zh-Hans-CN)
    • Flutter version 1.17.0 at /Users/HLQ/flutter
    • Framework revision e6b34c2b5c (5 周前), 2020-05-02 11:39:18 -0700
    • Engine revision 540786dd51
    • Dart version 2.8.1

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/HLQ/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/HLQ/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
    • PCT AL10 • xxx • android-arm64 • Android 10 (API 29)

Use webview_flutter version
webview_flutter: ^0.3.22+1
My Coding
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:flutter_easyloading/flutter_easyloading.dart';
import 'package:xxx/base/constants.dart';
import 'package:xxx/theme/colors.dart';
import 'package:xxx/utils/log_util.dart';
import 'package:xxx/utils/navigator_util.dart';
import 'package:xxx/utils/sharepreferences_util.dart';
import 'package:webview_flutter/webview_flutter.dart';

/// @date 2020-06-02
/// @author HLQ_Struggle
/// @desc 奖励规则
class RewardRulesPage extends StatefulWidget {
  @override
  State<StatefulWidget> createState() => _RewardRulesState();
}

class _RewardRulesState extends State<RewardRulesPage> {
  String _loadUrl = '';

  @override
  void initState() {
    getCacheValue(rewardRules).then((loadUrl) {
      setState(() {
        _loadUrl = loadUrl;
      });
      LogUtil.e(' 准备加载：$_loadUrl');
    });
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    Widget appBarSection = new AppBar(
      backgroundColor: ThemeColors.colorMain,
      centerTitle: true,
      elevation: 1,
      title: new Text(
        '收徒奖励说明',
        style: new TextStyle(
          color: Colors.black,
          fontWeight: FontWeight.normal,
          fontSize: 18,
        ),
      ),
      leading: BackButton(
        color: Colors.black,
        onPressed: () => NavigatorUtil.goBack(context),
      ),
    );
    return Scaffold(
      appBar: appBarSection,
      body: Container(
        child: (_loadUrl.length > 0)
            ? WebView(
                initialUrl: _loadUrl,
                javascriptMode: JavascriptMode.unrestricted,
                onPageStarted: (url) {
                  EasyLoading.show();
                },
                onPageFinished: (url) {
                  EasyLoading.dismiss();
                },
              )
            : Center(
                child: Text(
                  '页面正在加载中',
                  style: TextStyle(
                    fontSize: 16,
                    color: Colors.grey,
                  ),
                ),
              ),
      ),
    );
  }
}

Display of results

Hi @HLQ-Struggle
could you please provide  a minimal reproducible code sample that does not use 3rd party plugins
Thank you

Hi，@iapicca
I'm sorry, but overtime work has been more serious recently. I just realized that what do you mean by not using third-party plug-in loading?

@HLQ-Struggle
You can provide code sample with just webview_flutter in the coming days, no worries
Thank you

Hi，@TahaTesser
Do you mean let me wait for the next version of this plugin?
Novice feels helpless. Almost crying.

Hi @HLQ-Struggle
this platform is meant to provide improvements to flutter
with bug reports and feature request;
if you are in need of assistance please see https://flutter.dev/community for resources
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
If instead you believe the issue doesn't depend from your implementation,
but from flutter itself please provide a minimal reproducible code sample
that does not use 3rd party plugins
Thank you

Thanks


while install ms mpi.exe,i found i don't have include or lib file,what should i do

now my pc is this

my program files/Microsoft MPI is those file.

and my microsoft sdks file is those

I guess you forgot to install .msi AND .exe files from the link.
msmpisetup.exe installs files you have
and msmpisdk.msi installs the SDK files you need


Device#install_app

@umtrey
So now it seems like MokiRuby::Device instance should unravel the response instead of anything inside of MokiRuby class methods.

Remember to use a TenantManagedApp for all of the goodies you need for the call.
Client name is device name, item name is app name, set managed flag to 0 if itunes store and 1 if enterprise/has url, notify should be true.

love what you've done.


Login View Requing Authenticated User

I have seen this been discussed in
https://github.com/James1345/django-rest-knox/issues/7
I would like to share my point of view and an alternative solution here :) :).
The LoginView we have right now requires the user to be authenticated in the first place. This means, first the user has to pass a valid Token (somehow obtained earlier) in the HTTP Authorization Header or have a Valid Session, etc in order to pass permission classes in  https://github.com/James1345/django-rest-knox/blob/master/knox/views.py#L16.
Is this the desired behavior for a Login View or API? Shouldn't we allow users, who are not authenticated yet, to pass credentials, authentic the credentials and finally login the user. So the LoginView will take in credentials like username, password etc, authenticate the credentials, and finally create an AuthToken and return response.

True! I have overriden the view for my application and removed the permission.

No.
I must emphasize, that this is a Stateless system. It is designed for use with single page applications and mobile apps.
The LoginView is the POST endpoint for the user's credentials (e.g. username and password). The page that the user enters these into can be accessed without any form of login, as it should be static content.
When these credentials are authenticated by the auth system (e.g. BasicAuth), the LoginView returns a token that can be used as the auth from that device for all other views.
Removing the permission from LoginView would allow anyone to aquire an AuthToken whether they had authenticated or not.
If this is still unclear let me know. Please do not remove authentication from loginview in your apps though, I'd hate for security problems to happen due to a misunderstanding.

@James1345  Ok I just analyzed how you implemented this, now I get it :)

@James1345  In my case I had a CAS SSO login, so I just verify a service ticket and then generate the token like you have done in the view and return the token in response. Mine is a custom implementation.

It's not clear to my mind.
Could you provide a sequencial exemple fallowing this scheme or a sequence graph ?
(Sequance graph exemple : https://upload.wikimedia.org/wikipedia/commons/9/9b/CheckEmail.svg)
UnAuthed User --> POST user/pass -->  Which endPoint should be used?
Returns token if auth ok <--
Thank you for your help.


c doesn't work on strings

Vyxal, 14 bytes
`Hello` `lo` c

Try it Online!

It works with individual characters, so it's just not doing substring matching. I think we probably want that, but it's not a bug because as specified currently it looks like it's only supposed to do individual elements. ('lo' is not an element of 'H', 'e', 'l', 'l', 'o')


LilyPond Update to 2.22.0 (January 10th, 2021)

LilyPond Update to 2.22.0 (January 10th, 2021)
They just uploaded LilyPond, so I just put this here to ask if this extension is going to be updated. (Ignore the date in the URL)
https://lilypond.org/news.es.html#LilyPond-2_002e21_005e82-released-December-15_002c-2020

Hi! Thanks for this! The version dependent extension is actually https://github.com/lhl2617/VSLilyPond-snippets--I'll update this tomorrow!

Hi! Thanks for this! The version dependent extension is actually https://github.com/lhl2617/VSLilyPond-snippets--I'll update this tomorrow!

https://github.com/lhl2617/VSLilyPond-snippets bumped to 0.0.5 supporting 2.22. Thanks for flagging this :D

https://github.com/lhl2617/VSLilyPond-snippets bumped to 0.0.5 supporting 2.22. Thanks for flagging this :D


Add Set and Histogram

This PR adds the Set and Histogram metrics for the datadog sink.
Similar to Emit, I added these metrics to the MetricSink interface and made them noop for other sinks as per this slack thread.

Hello!
Is there any update regarding this PR, aside from the need to rebase it?

This PR is useful, but the prometheus package is missing implementation.