Add RabbitMQ pub/sub

Description
Add RabbitMQ as pub/sub provider.
This is how the components file looks like:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: messagebus
spec:
  type: pubsub.rabbitmq
  metadata:
  - name: host
    value: amqp://guest:guest@localhost:5672/

Issue reference
#105
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation

Additional information

I plan to add e2e tests once there is guidelines on how to do it
I am learning Go, all feedback is welcome!
Once this is accepted I will submit the PR to dapr/dapr to register RabbitMQ as a pub/sub

thans @fbeltrao! Review will lag a bit as some of us are at Kubecon this week :)

I'll review this in the next few days.

@yaron2 being new to golang I don't know how to fix the check failure caused by my last commit. It seems to be something related to go modules. Any help would be appreciated.

no I'll take a look

@fbeltrao can you try to run go mod tidy locally and push any changes?
Also, are you sure you don't have any stashed or uncommitted changes?

Thanks for your contribution @fbeltrao!


List containing nulls behaving inconsistently

When serializing lists containing nulls, rpc returns different values.
List ["one", null, null] returns ["one", null, null]
List [null, null, "three"] returns [null, null, "three"]
List ["one"] returns ["one"]
But
List [null] returns [ ]
An empty list, as opposed to a list with a single null inside. I was having an error where I expected a number of lists to have the same number of elements inside each, so it broke.
(haven't tested List [null, null, null] - sorry)

Thanks for the report. I will take a look at this shortly.

Is this always a bug ? because i have try to reproduce the error but without success...


Namicomi?

Source name
Namicomi
Source link
https://namicomi.com/en
Source language
English and more
Other details
No response
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

That was 3 months ago i just nvm..

Sorry..


[Ubuntu] Unable to build TEV; uint64_t, nBytes, uint8_t, and uintptr_t not declared in scope.

I'm on a debian machine, and followed the steps in the installation guide for building TEV, and am unable to finish the build. I'm not very experienced, but I believe the issue might be my compiler, and would appreciate a little bit of explanation with regards to what I need to do if the problem is my machine.
My current package versions are:
➜  tev git:(master) gcc --version
gcc (Ubuntu 13.2.0-4ubuntu3) 13.2.0
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

➜  tev git:(master) cmake --version
cmake version 3.27.4

CMake suite maintained and supported by Kitware (kitware.com/cmake).

And the error message is:
➜  tev git:(master) cmake --build build --config Release -j
[  3%] Built target Iex
[  3%] Built target clip
[  3%] Built target Half
[ 10%] Built target glfw_objects
[ 10%] Built target IexMath
[ 17%] Built target IlmThread
[ 35%] Built target nanogui
[ 39%] Built target Imath
[ 41%] Building CXX object dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfHuf.cpp.o
[ 41%] Building CXX object dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfMisc.cpp.o
[ 41%] Building CXX object dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfDwaCompressor.cpp.o
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfHuf.cpp: In function ‘void Imf_2_5::hufUncompress(const char*, int, short unsigned int*, int)’:
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfHuf.cpp:1105:5: error: ‘uint64_t’ was not declared in this scope
 1105 |     uint64_t nBytes = (static_cast<uint64_t>(nBits)+7) / 8 ;
      |     ^~~~~~~~
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfHuf.cpp:56:1: note: ‘uint64_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
   55 | #include <algorithm>
  +++ |+#include <cstdint>
   56 |
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfHuf.cpp:1107:16: error: ‘nBytes’ was not declared in this scope; did you mean ‘nBits’?
 1107 |     if ( ptr + nBytes > compressed+nCompressed)
      |                ^~~~~~
      |                nBits
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfHuf.cpp:1124:31: error: ‘nBytes’ was not declared in this scope; did you mean ‘nBits’?
 1124 |         if( ptr-compressed  + nBytes > nCompressed)
      |                               ^~~~~~
      |                               nBits
gmake[2]: *** [dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/build.make:328: dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfHuf.cpp.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfMisc.cpp:1392:17: error: ‘uint8_t’ does not name a type
 1392 | struct FBytes { uint8_t b[4]; };
      |                 ^~~~~~~
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfMisc.cpp:55:1: note: ‘uint8_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
   54 | #include <ImfTileDescription.h>
  +++ |+#include <cstdint>
   55 | #include "ImfNamespace.h"
gmake[2]: *** [dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/build.make:468: dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfMisc.cpp.o] Error 1
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfDwaCompressor.cpp: In member function ‘void Imf_2_5::DwaCompressor::LossyDctDecoderBase::execute()’:
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfDwaCompressor.cpp:746:31: error: ‘uintptr_t’ does not name a type
  746 |         if ((reinterpret_cast<uintptr_t>(rowBlockHandle + i) & _SSE_ALIGNMENT_MASK) == 0)
      |                               ^~~~~~~~~
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfDwaCompressor.cpp:177:1: note: ‘uintptr_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
  176 | #include "dwaLookups.h"
  +++ |+#include <cstdint>
  177 |
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfDwaCompressor.cpp:1014:38: error: ‘uintptr_t’ does not name a type
 1014 |                 if (reinterpret_cast<uintptr_t>(_rowPtrs[comp][y]) & _SSE_ALIGNMENT_MASK)
      |                                      ^~~~~~~~~
/home/tilmand/repos/tev/dependencies/openexr/OpenEXR/IlmImf/ImfDwaCompressor.cpp:1014:38: note: ‘uintptr_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
gmake[2]: *** [dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/build.make:454: dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/ImfDwaCompressor.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:1743: dependencies/openexr/OpenEXR/IlmImf/CMakeFiles/IlmImf.dir/all] Error 2
gmake: *** [Makefile:156: all] Error 2
➜  tev git:(master)

Thanks for the help!

Hi, thanks for reporting this! Could you try re-cloning the project and compiling again? I suspect the issue was an incompatibility between the (old) version of OpenEXR tev used until now and the (very new) version of gcc you've got. I migrated to a newer OpenEXR version just now and am hopeful that it fixed the issue.

Works perfectly, thanks a lot!


Doesn't reset on form reset

When you reset the form (via reset button), the underlying <select> resets, but the control doesn't update to match.

try this
$('[type=reset]').click(function() {
$('#status').selectpicker('val','your_default_value');
});

The default value is different across several elements. This works for me:
$('form').on('reset', function(event){
  $('.selectpicker',this).each(function(index, element){
    var $this = $(this);
    setTimeout(function(){
      $this.selectpicker('val',$this.val());
    },0);
  })
});

Still, I believe there should be some sort of integration such that the control behaves as the control it overrides.

This should work:
$('form').on('reset', function() {
  setTimeout(function() {
    $('.selectpicker').selectpicker('refresh');
  });
});

My understanding is that you can have more than one form on a page and that reset buttons (or reset events) only reset the form they are associated with. Your code would reset selectpicker items outside of just those associated with the form the particular reset button (or reset event) is associated with.

Ah, I see... Slight modification to take multiple forms into account and so dropping the each as @caseyjhol suggests:
$('form').on('reset', function() {
  var _this = this;
  setTimeout(function() {
    $('.selectpicker',_this).selectpicker('refresh');
  });
});

I like this solution:
$('.selectpicker').selectpicker('refresh');
and I am using it successfully (thanks, caseyjhol).
But I agree with previous comments that the regular Reset button should work out of the box with bootstrap-select. It would be nice if bootstrap-select could be improved so that no additional fiddling is necessary.

+1

+1

Please release this. Its default behaviour, and a long standing issue...

This change will be a part of the v1.14.0 release. This issue will be updated after it's made available.

Released in v1.14.0-beta!

For me is still not working, when I call $('#myform')[0].reset(); all the standard elements inside the form are reseted but not the multiselect, it still show the values separated by comma.
I tried with $('.selectpicker').selectpicker('refresh'); but it stills shows the selected values.
I am on<EMAIL_ADDRESS>+<EMAIL_ADDRESS>

@soyxan Can you link me to an example?

In the following link you can check 2 issues I am facing:

mobile=true makes the dropdown unusable in desktop #2557
reset does not work as expected

https://codepen.io/soyxan/pen/JjbwzKR
Thanks!


Using external secret operator with organization proxy

Hi, I'm currently doing a poc on external secret operator to be used with our on premise cluster to grab secret from aws secret manager
When i was done with the setup, external-secret pod fails to connect to aws secret manager public endpoint to connect and pull the secret from there, due to our outbound traffic is blocked, unless it will be going to a proxy
I have tried to add http_proxy, no_proxy and https_proxy as env variable to deployment inside the helm chart generated, but after deployed, external-secret-cert-controller and external-secret-webhook failed to start due to cert-controller unable to grab certificate.
Is there any guide for setting up correctly for external-secret-operator to use our organization proxy for it to work properly?
Thanks in advance

It totally depends on the proxy set-up.
Is the kube-apiserver also behind a proxy?
You probably have to specify NO_PROXY to allow eso to access the control plane from within the cluster without using the proxy.


Support named parameters for PRAGMA statements

Support e.g. PRAGMA some_function(1, x=10) instead of PRAGMA some_function(1, 10) so certain parameters can have default values, and it is more clear which parameters you are setting.
TableFunctions already support this. Initially I tried to unify these, but ran into a bunch of problems, because of differences between how PragmaStatement and TableRef store their parameters.
Now they are different in the parser, but under function TableFunction and PragmaFunction are derived from the same base class, SimpleNamedParameterFunction.
There are no PRAGMA's that currently need this, but I have plans! Also, there are no tests, but I did define a useless PRAGMA to test.

I made a few oopsies when implementing the feedback, but everything works in the travis of my fork now. I see that 2 jobs fail in travis here, but I don't exactly understand what's wrong with them by looking at the output. Any ideas?

The failures look rather innocent, there is a problem with the cibuildwheel for Windows which causes the failure there. The other test sometimes fails randomly because Postgres doesn’t start up correctly; no clue about why but it has nothing to do with your branch. I will rerun the Postgres one and have another look at the code, after that I think it’s ready to merge.

All tests now pass, seems like the efforts to fix Travis have worked!

Great! Then I think this can be merged :)


Long training time?

Compared to training a LoRA via for example kohya scripts it takes a great deal longer to train a slider LoRA. Considering that the results are not always that great, because concepts got confused and retraining is needed with additional --attributes, would it be possible to get an idea of whether the training is going anywhere so we can adjust earlier or abort entirely?
Or could training speeds perhaps be increased somehow? Maybe LCM would be an option as it seems actual images are created during training, which is a lot faster with LCM.

Couple of options -

Currently for text-based training - we are loading tensors to gpu only when needed. If there is enough GPU memory, I believe one can bypass the switching to just load all tensors directly to gpu device.
Implementing xformers for sliders. This is an open task that we will be working on in the coming days.
For testing intermediate results, in config file there is an option for save intermediate results per_steps: 500. You can store much earlier and test.

Given our base code is based on kohya script - we should be able to reduce the time a bit (my hunch - with xformers and accelerator). But we do note that our training has some additional steps compared to traditional training. It is bound to be slightly slower than the traditional lora training.

I also updated the scripts to streamline computation and reduce the device switching. In my end, I am seeing improvements.


Fix constructor of AutoMTK

Checklist

[ ] Appropriate tests were added
[ ] Any code changes were done in a way that does not break public API
[ ] All documentation related to code changes were updated
[ ] The new code follows the
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
[ ] Any new documentation only uses public API

Additional context
I was overloading the constructor in OptimizationBase since the deprecation here wasn't sufficient, leading to re-precompilation. I couldn't find a way to use @deprecated for this so this'll have to do.

I'll review tomorrow, please wait for me

Thanks, I wasn't completely sure what you meant by it in the chat

I'll wait for @gdalle to take a look to make sure it doesn't affect DI

The AutoModelingToolkit struct in ADTypes v0.2.7 was as follows, so this is the full API we want to deprecate but with meaningful fallbacks:
https://github.com/SciML/ADTypes.jl/blob/9f17b8227afdb12cbeb0faa7edc18abd5ffcae0c/src/ADTypes.jl#L151-L164
@ChrisRackauckas is right that the deprecation logic wasn't correct, so my last commit fixes it. We still generate a deprecation warning, which will be visible to users if such warnings are enabled, but we fall back on the correct version of the Symbolics backend.
@Vaibhavdixit02 what do you think?

Why did you change it back to being incorrect? The point is that there was a version without kwargs before.

My fallbacks handle every version that there was before, with and without kwargs. Both will give the correct result, but both will be deprecated

I didn't see the other dispatch. This should work then.

Let's wait for Vaibhav to test it locally with OptimizationBase before merging


/Users/[UserId] results in Duplicate Key Errors on Postgres

What version of UAA are you running?
4.6.1
{"app":{"version":"4.6.1"},"links":{"uaa":"http://localhost:8080/uaa","passwd":"/forgot_password","login":"http://localhost:8080/uaa","register":"/create_account"},"zone_name":"uaa","entityID":"cloudfoundry-saml-login","commit_id":"04c8408","idpDefinitions":{},"prompts":{"username":["text","Email"],"password":["password","Password"]},"timestamp":"2017-10-02T00:01:33+0000"}
How are you deploying the UAA?
Docker Container containing the official JAR + Tomcat.
What did you do?
curl -v -H 'Authorization: Bearer X' http://uaa/Users/[UserId]
What did you expect to see? What goal are you trying to achieve with the UAA?
There should be no errors in postgres.
What did you see instead?
Errors like the following in postgres:
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:ERROR: duplicate key value violates unique constraint "groups_unique_key"
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:DETAIL: Key (displayname, identity_zone_id)=(openid, uaa) already exists.
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:STATEMENT: insert into groups ( id,displayName,description,created,lastModified,version,identity_zone_id ) values ($1,$2,$3,$4,$5,$6,$7)
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:ERROR: duplicate key value violates unique constraint "groups_unique_key"
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:DETAIL: Key (displayname, identity_zone_id)=(scim.me, uaa) already exists.
2017-12-07 15:59:30 UTC:<IP_ADDRESS>(34428):cloud_uaa@cloud_uaa:[823]:STATEMENT: insert into groups ( id,displayName,description,created,lastModified,version,identity_zone_id ) values ($1,$2,$3,$4,$5,$6,$7)

These errors only appear on the postgres log. The UAA behaves just fine and does everything as intended. The lines above repeat for every default group of the UAA (i.e. scim.me, cloud_controler.write). For some reason, the UAA tries to create the default groups every time I request an user.
I tried to track this down, I suspect the following happens:

JdbcScimGroupMembershipManager.getGroupsWithMember is called on the HTTP request.
JdbcScimGroupMembershipManager.getDefaultUserGroups() is called by this which iterates over all default groups and calls JdbcScimGroupProvisioning.createOrGet() for all of them.
This calls into createAndIgnoreDuplicate which executes an INSERT and throws away the exception.

I'm not sure if this is a bug in the UAA: The code works well, it just relies on the database to throw an error if a group already exists. What I suggest we could do is replacing createAndIgnoreDuplicate with a call which first gets the group and if that fails tries to create the group.
One thing which I think is a bug is that we are trying to create the default groups on every incoming request. This puts completely unnecessary pressure on the database. One could create the groups on startup (or on first fetch) and don't try to recreate them all the time.

This is actually fixed with UAA #704. We upgraded and are happy now. Sorry to bother you.


Error 403 with make prep_lite

Hi there!
I'm trying to run the first steps but when running make prep_lite -j10 on my Ubuntu 22.04 I get a HTTP 403 somewhere:
Cloning into '/home/user/gits/blackparrot/black-parrot-sim/sdk/yocto/meta-linux-mainline'...
fatal: unable to access 'https://git.sr.ht/~pbarker/meta-linux-mainline/': The requested URL returned error: 403
fatal: clone of 'https://git.sr.ht/~pbarker/meta-linux-mainline' into submodule path '/home/user/gits/blackparrot/black-parrot-sim/sdk/yocto/meta-linux-mainline' failed
Failed to clone 'meta-linux-mainline'. Retry scheduled
Cloning into '/home/user/gits/blackparrot/black-parrot-sim/sdk/yocto/meta-linux-mainline'...
fatal: unable to access 'https://git.sr.ht/~pbarker/meta-linux-mainline/': The requested URL returned error: 403
fatal: clone of 'https://git.sr.ht/~pbarker/meta-linux-mainline' into submodule path '/home/user/gits/blackparrot/black-parrot-sim/sdk/yocto/meta-linux-mainline' failed
Failed to clone 'meta-linux-mainline' a second time, aborting
fatal: Failed to recurse into submodule path 'sdk/yocto'
fatal: Failed to recurse into submodule path 'sdk'
make: *** [Makefile:12: checkout] Error 1

Is this a known error / do you know a workaround?
Thank you very much!
Flavien

Hi Flavien, thanks for the issue.
Yes, this is a known issue. There was an external repo we relied on that was moved, breaking the submodule. If you update the remote of the bp-yocto submodule to the one from this PR, it should resolve:
https://github.com/black-parrot-sdk/bp-yocto/pull/1
I'll make a PR when I get to a computer so no one else runs into this...

Cool thanks! Then I'll retry when it's merged!

Give this a shot: https://github.com/black-parrot/black-parrot-sim/pull/12

That works! I'm getting the same segmentation fault as #10 but it seems benign from my understanding.
Thank you for your fast help!
Flavien


UI - Backup/Snapshot as a drop down menu option in scheduling

We believe backup schedules are used more than a snapshot schedule in our case.
Clicking + New might be helpful to show a drop down menu, to select either a Backup or Snapshot schedule.
(Saves from having to move the mouse all the way to the other side of the screen)

We can just make Backup as the default option instead. Updating + New is not going to make big differences since the user need to update the schedule anyway.
@smallteeths Can we change the job type dropdown to use Backup as the default instead of Snapshot?

Validation: PASSED
Longhorn UI version: wangsiye/longhorn-ui:81d49f5
Steps to test:

Create a volume, attach it to a node.
In Volume Details page, in Recurring Snapshot and Backup Schedule section, click  + New
Backup should be the default option in drop down menu for creating a schedule. (PASS)


pair wise training and fully convolutional training

In the paper of fully convolutional neural network,
the authors mention both patch wise training and fully convolutional training.
I did not find the exact description of patch wise training.
My understanding is this:
Given an M*M image, extract sub-images with N*N, where (N<M). The selected sub-images are overlapped with eath other.
For each batch, it can include all the sub-images for a given image or multiple images.
Is my understanding correct? Then what are the difference between patch-wise and fully convolutional training?
In Keras, how can we ensure the sub-images belonging to the same image to included in a single batch?

I am having the same question, looking forward to the answer!

Those two terms refer to different things.
The term "Fully Convolutional Training" just means replacing fully-connected layer with convolutional layers so that the network contains just convolutional layer (and pooling layers).
The term patchwise training is a little more involved. I will try to explain it.
In semantic segmentation, given that you are classifying each pixel in the image, by using the whole image, you are adding a lot of redundancy in the input.
A standard approach to avoid this during training segmentation networks is to feed the network with batches of random patches (small image regions surrounding the objects of interest) from the training set instead of full images. This "patchwise sampling" ensures that the input has enough variance and is a valid representation of the training dataset (the mini-batch should have the same distribution as the training set). This technique also helps to converge faster and to balance the classes (there may be more objects of certain classes on the training set). In this paper, they claim that is it not necessary to use patch-wise training and if you want to balance the classes you can weight or sample the loss.
In a different perspective, the problem with whole image training (batch size of 1) in per-pixel segmentation is that the input image have a lot of spatial correlation and its distribution is different from the training distribution. To fix this you can either sample patches from the training set (patchwise training) or sample the loss from the whole image. That is why the subsection is called "Patchwise training is loss sampling".
So by "restricting the loss to a randomly sampled subset of its spatial terms excludes patches from the gradient computation." They tried this "loss sampling" by randomly ignoring cells from the last layer so the loss is not calculated over the whole image.

Hi Juan,
Thank you so much for the reply.
In terms of the problem that I am studying, the single image has size about 700800, the mask image has just two classes, a foreground area(covering about 10~15% areas) as class 1 and the background area as class 2.
I have attached an image ( the large rectangle) for demonstration purposes. The blue area reflects the area in the original image that corresponds to class 1 in the mask image.  With respect to this kind of scenario, how can I construct the training set to support patch-wise training.
Assume I am planning to training network with input patch size of 128128,
My questions are:

For the patches ( the red ones) surrounding the interested areas, should we enforce them to have no-overlap?
Generally, should I also extract a few patches (the green ones)  in the arears corresponding to the background class
Regarding the statement of “the mini-batch should have the same distribution as the training set”, does that mean the ratio of  blue areas in the selected patches (two red ones and two green ones) over the white areas should equal to this area in the whole image?
Moreover, how to efficiently implement this kind of patch sampling mechanism in Python?

Thanks.
wenouyang


Improve multiple-pouch handling, add dye potential

They are great storage options but being able to color code them for quick use would be wonderful.

At the moment, there are problems surrounding the fact that multiple apothecary pouches, while providing storage, will also cause the game confusion as to which pouch to consult (if there are multiples) when providing herbs.
By default it supplies from the one in the Baubles slot, but in addition to this we should probably come up with a system that iterates over all the available pouches looking for herbs, breaking when found, rather than just the first one and then saying "nothing here, you can't cast it!"

(There is of course the fact that perhaps we only want herbs to be drawn from one pouch, but it becomes confusing to the user if this is not specifically or clearly stated. Some sort of "activation" to mark a pouch as the one being drawn from (and limiting the player to only one pouch/one equipped) might go a way to help with that.)


Burn the batch context with fire

It can be used to save on mesh/program cloning (which have allocations!) when creating lots of batches. However, I'm convinced it brings more trouble than the benefit (#510, #464, #351). Besides, it can just be moved outside of gfx-rs crate.
The problem with the Context, outside of it just being another thing to consider or think about, is that it doesn't have a very safe interface. Thus, it could be fine to use inside another library as an implementation detail, but not fine to expose it to the user. I may end up moving it out and using it for gfx_scene, but I'll hesitate to do it until mesh/program cloning shows up in the profiler.
Let's burn this thing! We only really need a Batch trait and two kinds of batches: one with everything in it (batch::Full?), and another one with just the core components that have to be bound together (batch::Core). This is a breaking change supposedly for gfx-0.6
Objections? @csherratt @cmr @bjz

I guess then gfx-rs would be unopinionated about how to share a resource. How does this work at the moment? I understand that e.g. a mesh is only a handle to a buffer on the GPU. What are its ownership semantics and when does it free the underlying resource?

@aepsil0n resources are reference-counted now anyway (they were not when Context was introduced), so they are shared. The only thing Context was providing is allocation-free copying of batches.
Since the mesh just has attributes pointing to GPU buffers, these get logically freed when the mesh goes out of scope (assuming it's the unique holder). The actual release call is done upon Device::cleanup(), typically right after the buffers swap.

This sounds good to me.

Fixed in #757


Add support for other GOV.UK products and refine detection of Frontend

This PR does the following:

Refines the detection of GOV.UK Frontend as we were getting lots of false positives off copy / pasted cookie banners
Adds the detection of GOV.UK Frontend's version number if a service team is using the dist version of the CSS or JS
Adds detection of three other GOV.UK Products: GOV.UK Template, GOV.UK Elements, GOV.UK Toolkit
All products are now use the 'UI Frameworks' category, rather than doubling up in 'Miscellaneous'

Thanks! I removed the references to window. as they're redundant.

Fantastic. Thank you!


Notification about UI state change for long running tasks

I play with simple NLP application inside browser.
I want to add loading indicator. Right now I use following approach
this.loading = true;
this.StateHasChanged();
try {
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}");
} finally {
     this.loading = false;
}

Main trick is using StateHasChanged. My question is: Is this correct way to do that? How Blazor in general think such cases should be handled. I'm fine with calling StateHasChanged() but I prefer that it would be called automatically before await.
Example 1
this.loading = true;
try {
    // this.StateHasChanged(); called here by runtime
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}");
} finally {
     this.loading = false;
}

Example 2
or better, if I could control that using for example .ConfigureAwait(false)
this.loading = true;
try {
    // this.StateHasChanged(); called here by runtime
    this.result = await httpClient.GetJsonAsync<SearchResult[]>($"api/search/sentence?content={content}")
             .ConfigureAwait(false);
} finally {
     this.loading = false;
}

As far as I can see, this is working as expected. You do need to call StateHasChanged after each await (except for the last one), because Blazor has no way to know about your "awaits" except for your last one.


How to use schema.gql instead of schema string

Hi, how to use schema.gql with graphql-yoga? Currently, I'm using schema.js
# ./schema.js
const schema = `
  type userData {
      _id: ID!
      username: String!
      age: Int!
      blood_group: String!
      nationality: String!
      contact_no: Int!
  }
`
module.exports = schema

#./app.js
const { GraphQLServer } = require('graphql-yoga');
const schema = require('./schema')
const server = new GraphQLServer({
    typeDefs: schema
    resolvers
});

If I use schema.graphql, how should I pass the arg to GraphQLServer constructor?

Duplicate https://github.com/dotansimha/graphql-yoga/discussions/1101 lets discuss there :)


after form submission i want to load form values from current QUERY_STRING

I included the form in a php file
form id="myForm" name="input" method="get"
for some reasons i don't want to post with ajax .
I need get the form values back from the QUERY_STRING

I'm closing this issue since it is probably outdated — Please reopen if you still need help on this. Thanks


ONNX export of decoder/decoder with past has unwanted output nodes

System Info
Transformers commit: 3ee958207a9830da6afcf2c20772ef9a2159ca30
Optimum commit: 9c71dd52f71f397bd97da2228691a3c7049f4452

Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
python3 -m optimum.exporters.onnx --model t5-small --task seq2seq-lm-with-past --for-ort t5_onnx

Expected behavior
No unwanted output node

cc @JingyaHuang I'll look into it

For decoder with/without past it's the same:

@mht-sharma looks similar to the issue you had no?

This is a new issue, have observed this a little back in a few models.
I have a saved checkpoint of t5 with me but I do not see the node. Some option in the modelling code might be returning the encoder_hidden_states with hidden_states?

yes exactly Mohit!


Remove Types

This is more of a stylistic suggestion, so feel free to just close if you're confident in your approach.
I recommend against the use of records unless you need one of the specific things they provide (i.e. Java interop or type-based polymorphism.) They provide no type safety or validation like a class does in a statically typed language.
The most idiomatic Clojure approach is to have all internal data be just primitives, maps, seqs and sets.
If you know data needs to conform to a particular structure you can use spec, but even that is strongly optional. If you're going to use specs, I usually recommend adding them after you've got the code stable. If you write your specs first, you lose the benefits of dynamic typing and quick iteration as you code, since you have to keep going back and changing the specs every time you change your mind about structure, which can easily double up the amount of work.

Hey Luke, thanks a lot for that suggestion! My idea was to somehow generate "types" programmatically from the lexicon, in the same way as a typescript app would generate types from Swagger, but maybe that's not the right approach here.
I'm happy to accept tis PR.


Allow BorderRadius.circular to be a const factory

The class BorderRadius has:
BorderRadius.circular(double radius) : this.all(
    Radius.circular(radius),
);

And this all factory is already a const factory, but the parent one (circular) is not, so Im changing it to this:
const BorderRadius.circular(double radius) : this.all(
    Radius.circular(radius),
);

This will declare the factory as const and help in the usage of this factory in a more performant way
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Looks like this isn't valid in Dart:
  error â�¢ Invalid constant value â�¢ packages/flutter/lib/src/painting/border_radius.dart:302:5 â�¢ invalid_constant

See also https://github.com/flutter/flutter/issues/28535 for previous discussion.

Closing this for now, this is not currently possible without dart language changes


npm ERR! node-pre-gyp

I cloned this repo, cd into the api folder and entered sudo npm install  but got a bunch of errors
npm ERR! node-pre-gyp WARN Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v93 ABI, unknown) (falling back to source compile with node-gyp)
npm ERR! node-pre-gyp http 404 status code downloading tarball https://github.com/kelektiv/node.bcrypt.js/releases/download/v3.0.6/bcrypt_lib-v3.0.6-node-v93-darwin-x64-unknown.tar.gz
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | x64
npm ERR! gyp info ok
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | x64
npm ERR! gyp info find Python using Python version 3.8.5 found at "/opt/anaconda3/bin/python3"
npm ERR! gyp WARN EACCES current user ("yuvrajvijayagarkar") does not have permission to access the dev dir "/Users/yuvrajvijayagarkar/Library/Caches/node-gyp/16.14.2"
npm ERR! gyp WARN EACCES attempting to reinstall using temporary dev dir "/tmp/.node-gyp"
npm ERR! gyp info spawn /opt/anaconda3/bin/python3
npm ERR! gyp info spawn args [
npm ERR! gyp info spawn args   '/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/gyp_main.py',
npm ERR! gyp info spawn args   'binding.gyp',
npm ERR! gyp info spawn args   '-f',
npm ERR! gyp info spawn args   'make',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/User
I updated my node to 16th version, also npm  to 8th version but still issue persists

Same for me... I also installed node-pre-gyp locally and used 14.15.3 node version, but it did not change anyting. Seems like many packages that was used here are outdated.
Solution: If you will remove bcrypt package from package.json and try once more it will work. You just have install bcrypt manually by usin "npm i bcrypt" it will download latest version without any errors.
Warning: But be carefull, i am not sure that using latest version of bcrypt good idea, cause some modules from older bcrypt could be removed/renamed in newer one and you will have problems in future.

I don't have any errors after npm install. Node: 16.13.2, Windows 10

the problem is with bcrypt package, which needs python installed on your pc
I just installed Python from Microsoft store and npm install worked without errors
Or we need to pick another bcrypt package from npm
https://stackoverflow.com/a/29320712/12654113

I got this to work for me. I needed to use node version 12.14.1, the current version as of the books publishing. I am using windows 11.
Try:
Uninstall node.js
Install nvm-windows: https://github.com/coreybutler/nvm-windows#installation--upgrades
In the terminal type nvm install 12.14.1 Followed by nvm use 12.41.1
then clone the repo and use npm install.

this repo's codes need  to update


'BrownPerspectiveCamera' object has no attribute 'focal'

I am using OpenSfM as part of OpenDroneMap - there, running the reconstruction fails for me when I try to use a Brown-Conrady camera model, with the following output:
  File "/code/SuperBuild/src/opensfm/bin/export_bundler", line 48, in <module>
    io.export_bundler(images, reconstructions, track_graph, bundle_file_path, list_file_path)
  File "/code/SuperBuild/src/opensfm/opensfm/io.py", line 703, in export_bundler
    focal = camera.focal * scale
AttributeError: 'BrownPerspectiveCamera' object has no attribute 'focal'

I have seen this error message mentioned before in a bug that was solved: https://github.com/mapillary/OpenSfM/issues/259
For reference, this is the cameras.json file that I am feeding to ODM. (Note: there are probably a few superfluous properties set, and ODM might also do some type of massaging before passing it on to OpenSfM.)
{
    "dji fc2204 4000 3000 perspective 0.6666": {
        "projection_type": "brown",
        "width": 4000,
        "height": 3000,
        "c_x": 0.492639,
        "c_x_prior": 0.492639,
        "c_y": 0.503773,
        "c_y_prior": 0.503773,
        "k1": -0.040661,
        "k1_prior": -0.040661,
        "k2": 0.104878,
        "k2_prior": 0.104878,
        "k3": -0.095506,
        "k3_prior": -0.095506,
        "p1": 0.000122,
        "p1_prior": 0.000122,
        "p2": 0.000276,
        "p2_prior": 0.000276,
        "focal": 0.721327,
        "focal_prior": 0.721327,
        "focal_x": 0.721327,
        "focal_x_prior": 0.721327,
        "focal_y": 0.721327,
        "focal_y_prior": 0.721327
    }
}

Hello,
Bundler export doesn't support full Brown-Conrady model (skew ratio + 3 radial + 2 tangential).
I pushed a fix though but beware that any further process that will use the Bundler export (I'm summonning @pierotofy here) will not be optimal reprojection-wise, as we can't export the full model.
Yann

Thanks @YanNoun ! Correct, that's a limitation of the bundler format (although some output is better than no output?). Thanks for the fix!


zq: Better error messages for single argument commands

I just tested this branch out at commit f361f112 using the repro example from #5046.
$ zq -version
Version: v1.14.0-7-gf361f112

$ zq -Z 'fork(=> file "./csv_1.csv" | yield email | collect(this), => file "./csv_2.csv" | yield this["Recommended Email"])'
zq: could not invoke zq with a single argument because:
 - the argument did not parse as a valid Zed query
 - a file could not be found with the name "fork(=> file \"./csv_..."

This seems like a great improvement to me so I'm functionally 👍 on this.
As @mattnibs and I have discussed offline, the user would surely appreciate being able to see the parse error in a situation like this. Two thoughts come to mind:

We could present it as part of this two-part message. If we do, I might suggest swapping the order since I expect the parse error will take up multiple lines.
We could implement the -query flag as mention in the docs and #5052 and make some reference to it in this error message (e.g., "consider -query").

I just re-tested the branch now at commit 2699090 once again using that example from #5046. It looks great with the parse error!
$ zq -version
Version: v1.14.0-8-g2699090d

$ zq -Z 'fork(=> file "./csv_1.csv" | yield email | collect(this), => file "./csv_2.csv" | yield this["Recommended Email"])'
zq: could not invoke zq with a single argument because:
 - a file could not be found with the name "fork(=> file \"./csv_..."
 - the argument could not be compiled as a valid Zed query due to parse error (column 58):
   fork(=> file "./csv_1.csv" | yield email | collect(this), => file "./csv_2.csv" | yield this["Recommended Email"])
                                                         === ^ ===

I then thought to see how it would look if I had some multi-line Zed.
$ zq -Z 'fork (
>   => file "./csv_1.csv" | yield email | collect(this),
>   => file "./csv_2.csv" | yield this["Recommended Email"]
> )'
zq: could not invoke zq with a single argument because:
 - a file could not be found with the name "fork (\n  => file \"./..."
 - the argument could not be compiled as a valid Zed query due to parse error (line 2, column 2):
     => file "./csv_2.csv" | yield this["Recommended Email"]
   = ^ ===

That's great since it mentions the line & column but shows just the line that had the error. Very nice! 👍
(I guess it's a partial bummer that the parser doesn't point directly at the , as being the problem, but that doesn't appear to be this PR's fault.)


Add a bundler tool

Add a script to embed binary Wasm module into a Wasm object file,
accessible via a C sybmol to the outside application.

Didn't want to create a separate repo for this, I think this is a good spot for this tool.
Currently, it only supports Clang, but I can extend it to support Emscripten as well.


[BEAM-4565] Fix hot key fanout in the face of combiner lifting.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.

R: @youngoli

Pre-existing issue:
  combine_per_key = combine_per_key.with_hot_key_fanout(fanout)

should be
  combine_per_key = combine_per_key.with_hot_key_fanout(self.fanout)

(Ie a "self" was missing).
Do we understand why tests are passing nonetheless?

The GlobalCombine path wasn't tested. Fixed and tested now.


New cop for .. operator precedence

The following code has a bug.
# test.rb
def foo(x = nil)
  x || 1..2
end

p foo
p foo(1..3)

$ ruby test.rb
1..2
test.rb:3:in `foo': bad value for range (ArgumentError)
	from test.rb:7:in `<main>'

The code is interpreted as (x || 1)..2, but it looks like x || (1..2) to me.
I think it is confusing, so, we need a cop for this problem. However we don't have the cop.
The cause is .. operator's precedence. https://ruby-doc.org/core-2.2.0/doc/syntax/precedence_rdoc.html

Expected behavior
x || 1..2

RuboCop adds an offence for the above code.
Actual behavior
The cop for this problem does not exist.
RuboCop version
$ rubocop -V
0.48.0 (using Parser <IP_ADDRESS>, running on ruby 2.4.1 x86_64-linux)

It can also be unexpected on the "other side" of the range:
1..2.to_a

will attempt to send #to_a to 2, rather than the range.

True. I've made this mistake like a million times. :-)
On Sun, Apr 2, 2017 at 08:53 Ted Johansson<EMAIL_ADDRESS>wrote:

It can also be unexpected on the "other side" of the range:
1..2.to_a
will attempt to send #to_a to 2, rather than the range.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bbatsov/rubocop/issues/4182#issuecomment-290966579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVypdJu6kYYnTQvP_UcpfGEeev0lfAks5rrzfcgaJpZM4MqOjD
.

@Drenmi is it actual issue?


Inconsistent commafication in precedence table

Commas got into the last two lines of some nice work.

Good catch, thanks.

Those commas have been there since the original version of the table, 11 years!


Use poetry to build / publish OpenPype wheel

Feature:
This adds ability to use poetry to build / publish OpenPype wheels.
poetry build

will creates OpenPype wheel unde /dist directory
poetry publish

will then publish just built wheel to pypi. You can add --username and --password to specify credentials for pypi. --dry-run will simulate process without publishing anything. For use with pypi API tokens should be used instead:
poetry config pypi-token.pypi pypi-api-token

OpenPype can be run after poetry install with poetry run openpype. Installing wheel will create executable script in target python openpype.

Works really well!


Add more integration tests for camel-hazelcast  extension

Hazelcast : add tests for :

query on HazelcastMapProducer
HazelcastInstanceComponent
HazelcastQueueComponent
HazelcastRingbufferComponent
HazelcastSedaComponent
Best effort : Repositories and RoutePolicy

To follow up with PR for https://github.com/apache/camel-quarkus/issues/1647

Working on this one. Please assign it to me.

query on HazelcastMapProducer will be done for GraalVm 21.0.0. It needs a Serializable object for the test. see note in camel-quarkus-2101

query on HazelcastMapProducer will be done for GraalVm 21.0.0. It needs a Serializable object for the test. see note in camel-quarkus-2101

HazelcastAggregationRepository and ReplicatedHazelcastAggregationRepository integration tests will be added in camel-quarkus-2101

HazelcastAggregationRepository and ReplicatedHazelcastAggregationRepository integration tests will be added in camel-quarkus-2101


EasingFunctions

It would be great if you could add more Easing Functions to Composition (similar to the ones available in XAML) with an option to specify whether they are EaseIn or EaseOut functions.
Also could you provide some more details about how CubicBezierEasingFunction should be used. How should the two Vector2 arguments be calculated?

@ratishphilip Yup, we're looking into providing easier access to different easing functions, particularly the Penner Equations. You can also generate the EaseIn, EaseOut, EaseInOut variations for a number of the Penner Equations using a Cubic Bezier with the exception of more complicated ones like Elastic and Bounce.
When constructing a Cubic Bezier, you provide two different control points that define the curve/form of the easing function. These control points are of type Vector2. I use this website to visualize what kind of curve my control points generate.

There was already a question about this - https://github.com/Microsoft/WindowsUIDevLabs/issues/36#issuecomment-200569775 although, I think this is feedback on the lack of proper documentation.

@tomzorz I must have missed that. Thanks for pointing it out.
@patsiu thanks for the link.
Yeah I agree, a strong and detailed documentation is what is missing most in Composition.


[OS X] Docker not able to pick the right path to ca cert

At the root of /Volumes I have 2 directories belonging to the 2 hard drives in my macbook pro, it seems that docker and/or docker-machine isn't able to handle directories with white-spaces in their names:

Does $(docker-machine env dev) work?
Looks like the cert path needs to be surrounded by double-quotes

How can I check if $(docker-machine env dev) works properly?

@smace - sorry - I meant to run the command in the terminal, then try running docker commands. I checked it myself though, and it doesn't work.
The PR I issued fixes the problem, though.
Until the PR is merged and you have a build with the fix, you can work around it by running docker-machine env dev, and copy-pasting the export commands, except for DOCKER_CERT_PATH, which you should change to something like:
export DOCKER_CERT_PATH="/Volumes/Macintosh HD/swann/.docker/machine/machines/dev"

Then, you can run docker ps and it'll pick up the environment variables for its configuration.

Thank you very much this quick fix!

@smace - just FYI, this was fixed, but @nathanleclaire and @ehazlett found a bug in my PR, so the fix had to be reverted... I'm working on a better fix now ;)

Ok, it actually was fixed, and the revert is being reverted in #814.
@smace you will need to do eval "$(docker-machine env dev)" to set the variables in future (though the $(docker-machine config dev) usage will also work).

@ehazlett can you close this again? :)

@hairyhenderson yes.  thanks for the clarification!


show watch options in modal view as well

current behaviour
The stopwatch shown in modal view (on clicking Notes) does not have option to pause refresh or remove.
Expected Behaviour
It would be great to have those options in modal view as well.

Hi @1995YogeshSharma, I have started working on this


jvalue has no field _data anymore

The crate jni_sys has renamed the field _data of struct jvalue to data, hence this crate does not compile anymore due to this line in jvalue.rs (line 54):
trace!("converted {:?} to jvalue {:?}", self, val._data);

This is the commit that renamed the field.
The field is no longer public either, I didn't see any function that expose the value. Maybe the solution is just to remove the log or the post an issue on rust-jni-sys repository for exposing the field again.

Will work again soon: https://github.com/sfackler/rust-jni-sys/pull/6

Fixed in https://github.com/prevoty/jni-rs/commit/29a786d627e8f55038ce1aa75ee1a9024f934bf4

Did not see there was a PR for this. I was looking for issues.
Great to know it is fixed :-)


fix(traits): use Comparable matches

Reverting #4512 which introduced a function diverging the match from the original design
This PR aims to fix a diverging design I introduced some times ago. With this one we move back to the original design making correct usage of Comparable interface in trait which is used by the operator to distinguish when to reuse or rebuild an Integration based only on certain IntegrationKit traits changes (ie, runtime version, builder properties).

Release Note
fix(traits): use Comparable matches

Result of Quarkus native integrations:
--- PASS: TestNativeBinding (549.82s)
    --- PASS: TestNativeBinding/binding_with_native_build (535.03s)
--- PASS: TestNativeIntegrations (1114.16s)
    --- PASS: TestNativeIntegrations/unsupported_integration_source_language (0.90s)
    --- PASS: TestNativeIntegrations/xml_native_support (632.23s)
    --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit (472.05s)
        --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit/yaml_native_should_not_rebuild (2.20s)

=== NAME  TestNativeHighMemoryIntegrations
    native_with_sources_test.go:60:
        Timed out after 60.000s.
        Expected
            <string>: 2024-03-07 17:04:07,602 INFO  [org.apa.cam.k.Runtime] (main) Apache Camel K Runtime 3.2.3
            2024-03-07 17:04:07,602 INFO  [org.apa.cam.qua.cor.CamelBootstrapRecorder] (main) Bootstrap runtime: org.apache.camel.quarkus.main.CamelMainRuntime
            2024-03-07 17:04:07,602 INFO  [org.apa.cam.mai.MainSupport] (main) Apache Camel (Main) 4.0.2 is starting
            2024-03-07 17:04:07,610 INFO  [org.apa.cam.imp.eng.AbstractCamelContext] (main) Apache Camel 4.0.2 (camel-1) is starting
            2024-03-07 17:04:07,610 INFO  [org.apa.cam.imp.eng.AbstractCamelContext] (main) Routes startup (started:0)
            2024-03-07 17:04:07,610 INFO  [org.apa.cam.imp.eng.AbstractCamelContext] (main) Apache Camel 4.0.2 (camel-1) started in 0ms (build:0ms init:0ms start:0ms)
            2024-03-07 17:04:07,612 INFO  [io.quarkus] (main) camel-k-integration 2.3.0-SNAPSHOT native (powered by Quarkus 3.2.9.Final) started in 0.052s. Listening on: http://<IP_ADDRESS>:8080
            2024-03-07 17:04:07,612 INFO  [io.quarkus] (main) Profile prod activated.
            2024-03-07 17:04:07,612 INFO  [io.quarkus] (main) Installed features: [camel-bean, camel-cloudevents, camel-core, camel-java-joor-dsl, camel-k-core, camel-k-runtime, camel-knative, camel-kubernetes, camel-timer, cdi, kubernetes-client, smallrye-context-propagation, vertx]

        to contain substring
            <string>: Java Magicstring!

Fixing this.

--- PASS: TestNativeHighMemoryIntegrations (1750.79s)
    --- PASS: TestNativeHighMemoryIntegrations/java_native_support (847.92s)
        --- PASS: TestNativeHighMemoryIntegrations/java_native_support/java_native_same_should_not_rebuild (2.22s)
        --- PASS: TestNativeHighMemoryIntegrations/java_native_support/java_native_should_rebuild (417.19s)
    --- PASS: TestNativeHighMemoryIntegrations/groovy_native_support (457.62s)
    --- PASS: TestNativeHighMemoryIntegrations/kotlin_native_support (434.65s)
PASS
ok  	github.com/apache/camel-k/v2/e2e/native	1750.826s
--- PASS: TestNativeBinding (435.95s)
    --- PASS: TestNativeBinding/binding_with_native_build (421.78s)
--- PASS: TestNativeIntegrations (846.09s)
    --- PASS: TestNativeIntegrations/unsupported_integration_source_language (0.80s)
    --- PASS: TestNativeIntegrations/xml_native_support (410.86s)
    --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit (424.50s)
        --- PASS: TestNativeIntegrations/automatic_rollout_deployment_from_jvm_to_native_kit/yaml_native_should_not_rebuild (2.76s)
PASS
ok  	github.com/apache/camel-k/v2/e2e/native	1282.082s
``


Noticed a typo while reading this last night

Maybe I should have waited until October to PR this... 🤔

i know some project dont like little PRs like this. but i think they are awesome. any more cosmetics fixes or typo prs will be greatly accepted


codemirror popup flickering when updating content

This problems occurs in all browsers I have tested (IE11, chrome, firefox, safari) all latest version.
At dartpad we have this issue that the popup will flicker when the content is updating.

This is probably because the content is coming from a server, and it takes around 200-300 ms for that request to complete.
Now in this situation it is a bit unfortunate that codemirror is removing the popup from the dom in the showWidget.activity function:
function activity() {
  clearDebounce();
  var pos = completion.cm.getCursor(),
    line = completion.cm.getLine(pos.line);
  if (pos.line != startPos.line || line.length - pos.ch != startLen - startPos.ch ||
    pos.ch < startPos.ch || completion.cm.somethingSelected() ||
    (pos.ch && closeOn.test(line.charAt(pos.ch - 1)))) {
    completion.close();
  } else {
    debounce = requestAnimationFrame(update);
    if (completion.widget) completion.widget.close(); //completion popup removed here
  }
}

In a situation where it takes a long time for the update function to complete, this seems to cause flickering. To solve this, I think it would be best to only  remove the popup from the dom just before it is updated again. So while analyzing it a bit more, it seems that in the finishUpdate function, does exactly that:
function finishUpdate(data_) {
  data = data_;
  if (finished) return;
  if (!data || !data.list.length) return done();
  if (completion.widget) completion.widget.close(); //completion widget also closed here
  completion.widget = new Widget(completion, data);
}

So either the line commented here, or the line commented above is redundant. To solve the flickering alltogether, I think it would be better to close the the widget in the finishUpdate function and remove the line in the showWidget.activity function. If I do this, the flickering disappears:

I think this line that closes the completion widget in activity may not be necessary, as the completion widget is already removed in the Widget.finishUpdate function:
      function finishUpdate(data_) {
        data = data_;
        if (finished) return;
        if (!data || !data.list.length) return done();
        if (completion.widget) completion.widget.close(); // see this line
        completion.widget = new Widget(completion, data);
      }

And also in the completion.close function
    close: function() {
      if (!this.active()) return;
      this.cm.state.completionActive = null;

      if (this.widget) this.widget.close(); //at this line
      if (this.onClose) this.onClose();
      CodeMirror.signal(this.cm, "endCompletion", this.cm);
    },

As far as I can see, you will either want to remove the widget from the dom to refresh it (while updating) or you want to remove the widget from the dom if you just want to close the completion alltogether. Both cases seem to be covered by those two function above, so therefore I don't think it is necessary to also remove it from the dom at Widget.activity.

The reason that call is there is that if the user keeps typing and then presses enter in the moment when the new results are being fetched, but the widget hasn't updated yet, they'll get a rather bogus result: the old text is replaced, and the text typed since the last refresh is left at the end. Similarly, if they delete text, the hint picking will overwrite unrelated text.
Concretely, create a slow (~500ms) async hinting function, and open the completion menu. Now type 'x' and quickly press enter. The result will be [completion]x.
I think this problem could be reasonably solved by disabling but not hiding the completion popup while waiting for an asynchronous completion. I.e. add a method to the widget object to remove its keymap, and call that instead of close.

@marijnh thanks for your response :)
I didn't think about that case. I've been trying to play around with your example, at dartpad, but this issue seems to also exist at dartpad without removing that line:

To reproduce, here is the screenshot taken: http://dev.dart-pad.appspot.com/9126d5d48ebabf5bf547

@marijnh I I change
if (completion.widget) completion.widget.close();
this line for
completion.cm.removeKeyMap(completion.widget.keyMap);
both problems seem to be solved

Only now you have that if type to fast and press enter, you really will press enter,

I think this is best handled by remembering the enter press, and if the completions come up with a single result, automatically pick that. I've implemented this in attached patch.


[bug]:  component autoComplete prop

What version of design-system are you using?

Strapi Design System - v1.14.1
Browser - Google Chrome v129.0.6668.91

What's Wrong?
When focusing on <DatePicker /> component browser's default auto complete feature is working and some options are being suggested from the browser browser.

To Reproduce

Click on input area of the <DatePicker /> component when the input is empty

Expected Behaviour
There should not be browser's default auto complete feature enabled.

As a solution I suggest to pass autoComplete="off" prop by default at packages/design-system/src/components/DatePicker/DatePicker.tsx file's <DatePickerTextInput /> component.
I will open a PR for this today.


New helper: palindrome test

Codecov Report

Merging #9 (b394d6d) into main (c5f8dde) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        15    +2
  Lines          156       173   +17
=========================================
+ Hits           156       173   +17

Impacted Files
Coverage Δ

src/helpers/palindrome.py
100.00% <100.00%> (ø)

src/helpers/palindrome_test.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


azurerm_subscription_policy_remediation issue

Issue Template
Prerequisites

[ ] I am running the latest version
[ ] I checked the documentation and found no answer
[ ] I checked to make sure that this issue has not already been filed

Context

Module Version:
master branch
Terraform Version:
terraform version
Terraform v1.3.1
AzureRM Provider Version:
version = ">=3.23.0"

 on ../modules/def_assignment/main.tf line 139, in resource "azurerm_subscription_policy_remediation" "rem"

139 resource azurerm_subscription_policy_remediation rem {
140   count                   = local.create_remediation + local.remediate.sub > 1 ? 1 : 0
141   name                    = lower("${var.definition.name}-${formatdate("DD-MM-YYYY-hh:mm:ss", timestamp())}")
142   subscription_id         = local.remediation_scope
143   policy_assignment_id    = local.assignment.id
144   resource_discovery_mode = var.resource_discovery_mode
145   location_filters        = var.location_filters
146   failure_percentage      = var.failure_percentage
147   parallel_deployments    = var.parallel_deployments
148   resource_count          = var.resource_count
149 }

Expected Behavior

try to perform a testing for example
Current Behavior

make example works for testing
Possible Solution

Failure Information (for bugs)

Steps to Reproduce

git clone https://github.com/gettek/
cd terraform-azurerm-policy-as-code/example
terraform init
terraform plan -out=./myplan
terraform apply "./myplan"

Failure Logs

module.org_mg_configure_az_monitor_and_security_vm_initiative.azurerm_management_group_policy_remediation.rem["ASC_AMA_DefaultPipeline_Deploy"]: Creation complete after 4s [id=/providers/Microsoft.Management/managementGroups/policy_dev/providers/Microsoft.PolicyInsights/remediations/asc_ama_defaultpipeline_deploy-20-10-2022-01:32:03]
╷
│ Error: creating/updating /subscriptions/7996451c-728c-4b55-87f3-dfa8aeac980e/providers/Microsoft.PolicyInsights/remediations/inherit_resource_group_tags_modify-20-10-2022-01:32:00: remediations.RemediationsClient#RemediationsCreateOrUpdateAtSubscription: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="InvalidCreateRemediationRequest" Message="The policy assignment specified in remediation 'inherit_resource_group_tags_modify-20-10-2022-01:32:00' is out of scope. Policy assignments should be specified only at or above the remediation scope."
│
│   with module.team_a_mg_inherit_resource_group_tags_modify.azurerm_subscription_policy_remediation.rem[0],
│   on ../modules/def_assignment/main.tf line 139, in resource "azurerm_subscription_policy_remediation" "rem":
│  139: resource azurerm_subscription_policy_remediation rem {

I was caused by the assignment scope , I fixed it ,let me close this issue


Save predictions on test data if test is enabled

Hi SciBERT team, First of all, it is an awesome work. I have a question cum feature request.
Q1.
Is scibert predicting the entity for each token or for a sentence (bunch of tokens) from the test set?
Q2. Is the accuracy/f1 score calculated at the "span label" level (PER) or "IOB label + span label" level (B-PER)?
If we have a feature to save the predictions (in addition to outputting just the acc/f1 score) on the test data when if the test is enabled during training, we can figure out the above such questions ourselves.
Thank you

1- The NER model predicts an IOB label per token in the sentence, which can be used at decoding time to find spans of entities
2- We use span-based f1 (f1-measure-overall) (here https://github.com/allenai/scibert/blob/master/allennlp_config/ner.json) which is this allennlp metric https://github.com/allenai/allennlp/blob/master/allennlp/training/metrics/span_based_f1_measure.py
You need to implement an allennlp predictor to get predictions from the trained models


Webview2: filepicker causes locking mouse and reentrancy issue

Describe the bug
Custom web:
custom.zip
Open that web by webview2 -> click to open filepicker -> choose file -> sometimes the mouse is lock and cannot interact to webview2 until I move the cursor out of webview2 then re-enter in.
Another problem is sometime while filepicker is openning, I do some interact as click on the webview, the reentrancy issue show up in debug.
Steps to reproduce the bug
Open custom web then open filepicker
Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.3.3: 1.3.230724000
Windows version
No response
Additional context
No response

problem mouse is locked same with https://github.com/microsoft/microsoft-ui-xaml/issues/8668


AMBARI-26049: Fix type error in ambari port alert

What changes were proposed in this pull request?
This PR addresses a bug that was introduced after upgrading from Python 2 to Python 3. The original code was using byte strings for socket communication, which is the default string type in Python 2. However, in Python 3, strings are Unicode by default and need to be explicitly encoded to byte strings before being sent over a socket. Similarly, received data must be decoded back into a Unicode string. The changes in this PR ensure that the socket_command is properly encoded before sending and the received data is decoded back into a string for further processing. This fix restores proper socket communication functionality under Python 3.
(Please fill in changes proposed in this fix)
How was this patch tested?
unit test
manual test
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Ambari Contributing Guide before opening a pull request.

@JiaLiangC , I ran into this bug as well with the Zookeeper port alert in Bigtop. Any reason you closed this PR? I was planning on fixing this but saw you had this out here.

@Danori I haven't had time to continue working on this because I'm too busy. Could you please open a new PR to fix it?

@JiaLiangC , I also encountered the same problem and used this PR to fix it. Thank you.

Thanks @JiaLiangC, please feel free to merge the PR and close the Jira with the right fix version.

@virajjasani Do you mean that if my submitted PR has been reviewed and approved by other committers or PMC members, I can merge this PR myself and update the related issue?

Absolutely :)


[github-actions] Cert_9_2_04_ActiveDataset.py fails intermittently

Describe the bug
tests/scripts/thread-cert/Cert_9_2_04_ActiveDataset.py fails intermittently.
Console/log output

logs_47184.zip
packet-verification-pcaps.zip

In Step 3, should filter MGMT_ACTIVE_SET.rsp with Accept, but there is a LEAD_PET.rsp sent out by Leader with the same Message ID(packet index# 36) in Step 2 @librasungirl @simonlingoogle

@FiveDimensions @librasungirl
Seems wireshark matched PKT 36 as the ACK of PKT26 because they have same TKN.
It's strange that PKT 34 and PKT 26 use the same TKN: 89 03. CoAP should generate random token for each new request.

@FiveDimensions @librasungirl
Seems wireshark matched PKT 36 as the ACK of PKT26 because they have same TKN.
It's strange that PKT 34 and PKT 26 use the same TKN: 89 03. CoAP should generate random token for each new request.

There is no guarantee in existing implementation that the random token of 2 bytes would be unique as required by RFC7252.
Some possible options

increase the token length (valid token length [0, 8]) to reduce the probability of generating the same token - the disadvantage is the extra CoAP header cost
Update Wireshark to match request/response properly as Message ID should also be considered when matching the CON req and ACK for the piggybacked response, per RFC7252 Section 5.3.2

The exact rules for matching a response to a request are as follows:

The source endpoint of the response MUST be the same as the destination endpoint of the original request.
In a piggybacked response, the Message ID of the Confirmable request and the Acknowledgement MUST match, and the tokens of the response and original request MUST match. In a separate response, just the tokens of the response and original request MUST match.

Thoughts? @simonlingoogle @jwhui @FiveDimensions

@FiveDimensions @librasungirl
Seems wireshark matched PKT 36 as the ACK of PKT26 because they have same TKN.
It's strange that PKT 34 and PKT 26 use the same TKN: 89 03. CoAP should generate random token for each new request.

There is no guarantee in existing implementation that the random token of 2 bytes would be unique as required by RFC7252.
Some possible options

increase the token length (valid token length [0, 8]) to reduce the probability of generating the same token - the disadvantage is the extra CoAP header cost
Update Wireshark to match request/response properly as Message ID should also be considered when matching the CON req and ACK for the piggybacked response, per RFC7252 Section 5.3.2

The exact rules for matching a response to a request are as follows:

The source endpoint of the response MUST be the same as the destination endpoint of the original request.
In a piggybacked response, the Message ID of the Confirmable request and the Acknowledgement MUST match, and the tokens of the response and original request MUST match. In a separate response, just the tokens of the response and original request MUST match.

Thoughts? @simonlingoogle @jwhui @FiveDimensions

I think we should update the matching to conform with RFC 7252 Section 5.3.2.

I think we should update the matching to conform with RFC 7252 Section 5.3.2.

Have not seen this failure for a while. Closing.


fuzz: Add fuzzer for RawNetworkMessage.

Comes at @TheBlueMatt's request.

Note there is currently a crash if the message is too large. This fuzz test should be easy to make crash until that is fixed.

On May 21, 2019, at 11:34, Carl Dong<EMAIL_ADDRESS>wrote:
Comes at @TheBlueMatt's request.
You can view, comment on, or merge this pull request online at:
https://github.com/rust-bitcoin/rust-bitcoin/pull/261
Commit Summary
fuzz: Add fuzzer for RawNetworkMessage.
File Changes
M fuzz/Cargo.toml (4)
A fuzz/fuzz_targets/deserialize_raw_network_message.rs (52)
Patch Links:
https://github.com/rust-bitcoin/rust-bitcoin/pull/261.patch
https://github.com/rust-bitcoin/rust-bitcoin/pull/261.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Travis failed cause the fuzzer got stuck and hung. Seems this fuzz target is too slow, but its probably cause of our encode/decode code. See #264.

Taken via #272.


how to run this project?

I don't see any guides to run this project. I had to find by myself. But I run into many problems about that. Like some dependencies is not available at the time. I had some errors when I run start_ServiceAgentGenerator.bat file. How can I solve this problem?

Have you already seen the las2peer-template-project and its wiki?


Updating Resource with non-numeric primary key

@neomerx I am confused about updating resource that are not with numeric primary key.
for example "roles"
i could not get the primary id changed.

What do you mean by "get the primary id changed"? Change it in "update" operation?

If you try to change the primary key value in an update operation you may bump into an SQL issue. That's what I got
update roles set id_role = 'super' where id_role = 'admin';

Output

FOREIGN KEY constraint failed: update roles set id_role = 'super' where id_role = 'admin';

What do you mean by "get the primary id changed"? Change it in "update" operation?

yes, get primary id changed by "update" operation

FOREIGN KEY constraint failed: update roles set id_role = 'super' where id_role = 'admin';

i got exactly same foreign key constraint error in MySQL operation.
in general "PATCH" operation, i  have no place to update the id, says from "users" to "moderators".. etc.
is it possible to do the update of non-numeric primary id by patch?

If you tell me how to do it in SQL then I can tell what do to in the framework. Currently, I have a feeling that it would be an SQL limitation.
A generic answer would be that limoncello can call an SQL stored procedure on the update which does all the changes.

If you tell me how to do it in SQL then I can tell what do to in the framework. Currently, I have a feeling that it would be an SQL limitation.
agree. it is a very underlying problem. not limoncello. sorry for my stupid question :(

A generic answer would be that limoncello can call an SQL stored procedure on the update which does all the changes.

i think it is not easy to do it, isn't it?
i think i should think change the primary key from non-numeric to numeric for easier workaround

i think i should think change the primary key from non-numeric to numeric for easier workaround

If you can do it easily in a 'standard/typical' way then why not?

i think it is not easy to do it, isn't it?

Limoncello supports stored procedures on SQL migrations and API levels. If you already have a stored procedure it could be called from API like below
    public function updateXXX(int $input): bool
    {
        $spName    = 'spPutNameHere';
        $paramName = ':id';
        $slq       = "CALL $spName($paramName)";

        $result = $connection->executeQuery($slq, [$paramName => $input])->execute();

        return $result;
    }

Then a corresponding method should be added to Controller with a route in Routes. I think it's a rather typical approach.


Update proposal.yaml

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

@hifabienne Do you want to collect proposals as issues or as discussions?


Add imagePullSecrets to Helm chart

Fixes #192

Tested with imagePullSecrets: [{ name: mysecret }]
    spec:
      serviceAccountName: release-name-secrets-store-csi-driver-provider-aws
      hostNetwork: false
      containers:
        - name: provider-aws-installer
          image: public.ecr.aws/aws-secrets-manager/secrets-store-csi-driver-provider-aws:1.0.r2-42-ga2fd202-2<IP_ADDRESS>.03
          imagePullPolicy: IfNotPresent
          args:
            - --provider-volume=/etc/kubernetes/secrets-store-csi-providers
          resources:
            limits:
              cpu: 50m
              memory: 100Mi
            requests:
              cpu: 50m
              memory: 100Mi
          securityContext:
            allowPrivilegeEscalation: false
            privileged: false
          volumeMounts:
            - mountPath: /etc/kubernetes/secrets-store-csi-providers
              name: providervol
            - name: mountpoint-dir
              mountPath: /var/lib/kubelet/pods
              mountPropagation: HostToContainer
      volumes:
        - name: providervol
          hostPath:
            path: /etc/kubernetes/secrets-store-csi-providers
        - name: mountpoint-dir
          hostPath:
            path: /var/lib/kubelet/pods
            type: DirectoryOrCreate
      imagePullSecrets:
        - name: mysecret
      nodeSelector:
        kubernetes.io/os: linux

Bumping this as it is needed for many helm deployments and was previously supported by version 0.0.4

Codecov Report

Merging #200 (925cb42) into main (fdecd23) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #200   +/-   ##
=======================================
  Coverage   53.15%   53.15%
=======================================
  Files           8        8
  Lines         730      730
=======================================
  Hits          388      388
  Misses        332      332
  Partials       10       10

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

@lasred / @mandydas - either of you have a second for a quick review/approval?

When is this going to be released? We're still maintaining a fork, but would like to get back on track sooner rather than later


Unable to connect to Aurora MySql with Aws ECS fargate container

I have a .net core 3.1 application which is deployed as docker container on AWS Ecs with fargate. I'd created 2 new Aurora MySql dbs and tried to connect to one of the db cluster by using the writer endpoint. Now the issue is when I try to call the POST method through postman by calling load balancer DNS, I am getting the following exception:

"An exception has been raised that is likely due to a transient failure. Consider enabling transient error resiliency by adding 'EnableRetryOnFailure()' to the 'UseMySql' call."

I'd also tried adding retry pattern as mentioned in the exception but no luck.
I've allowed Aurora/MySql on port 3306 from anywhere in my security group which I created for RDS. I'd also added Aurora/MySql on port 3306 for source "My cluster service security group and also load balancer security group" but even then no luck. Finally I added All traffic on all ports from anywhere but even then the error was same. So I suspect it's something different.
I've then tried running the application through the docker desktop on my windows but I still got the same error there.

An error occurred using the connection to database '' on server 'db-cluster.cluster-cqb2jskhx78p.us-east-2.rds.amazonaws.com'.

My connection string is:
"DefaultConnection": "Data Source=db-cluster.cluster-cqb2jskhx78p.us-east-2.rds.amazonaws.com;Initial Catalog=demodb;User ID=admin;Password=MyPassword;port=3306"
Any help?

I have the same issue. Had any luck since you posted this?

 The Terraform "endpoint" property that I was using to build the connection string for the container had the port number.
Failed
server=dbcluster:**3306**;user=sqluser;password=mypw;database=dbname;AllowLoadLocalInfile=true
Obviously works:
server=dbcluster:**3306**;user=sqluser;password=mypw;database=dbname;AllowLoadLocalInfile=true
@junaid-ahmed92 Have you set the db server to be publicly accessible?


Add "start_output" argument to the launch() command to stream the output

📝 Description of the feature
When the user launches the System Coupling session, by default the console output is not streamed in the Python console. In order to do that, the start_output() method needs to be called and then end_output() method should be called to stop streaming the output. For example:
syc = pysyc.launch(version = "24.1")
syc.start_output()
...
syc.end_output()
syc.exit()

It would be good to add an argument to the launch() method to start the output right away, i.e.
syc = pysyc.launch(version = "24.1", start_output = True)
...
syc.exit()

💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

Completed this issue as part of the work done in PR #314


New version: DistributionsAD v0.1.1

Registering package: DistributionsAD
Repository: https://github.com/TuringLang/DistributionsAD.jl
Created by: @mohamed82008
Version: v0.1.1
Commit: a13a865aaab51fc68acc94367a1fce51e8704b84
Reviewed by: @mohamed82008
Reference: https://github.com/TuringLang/DistributionsAD.jl/issues/10

Please fix the compatibility bounds for the next release such that it will be automatically merged.


Add node: prefix in require for built-in modules

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/metavm/commit/be07aa5892606082f612bee45e41a824fb77b358 and https://github.com/metarhia/metavm/commit/d6d0aee8dcc6a3948d1fee304f4bd1421a80bc5e


Having issue downloading a large dataset

I am trying to pull the building feature layer for all of South Africa (roughly 20GB), and I keep on encountering this error:
OSError: AWS Error NETWORK_CONNECTION during GetObject operation: curlCode: 28, Timeout was reached
Error downloading full dataset: Command '['overturemaps', 'download', '--bbox', '16.2816999,-47.1313489,38.2216904,-22.1254239', '-f', 'geojson', '-t', 'building', '-o', 'buildings_South_Africa_full.geojson']' returned non-zero exit status 1.
I've downloaded large maps before like the whole of Ontario, Canada which is over 30GB and the download was successful . This problem has been occurring for the last three days, and is usually timing out once it has been running for 4500-5000+ seconds. I am running it using the python interface through a script I created to download the map and upload to our ArcGIS account. Like I said, I've been running into no issues until three days ago. Some support would be greatly appreciated. Thank you very much.

Hello,
I would like to follow up on this issue. It is still occurring and is happening to me at around the 500 second mark of runtime. I am trying to download the building layer for Guernsey, UK. This is the message I am getting:
OSError: AWS Error NETWORK_CONNECTION during GetObject operation: curlCode: 28, Timeout was reached
Error downloading data: Command '['overturemaps', 'download', '--bbox', '-8.8988999,34.5614,33.9165549,60.91569999999999', '-f', 'geojson', '-t', 'building', '-o', 'Guernsey,_UK_buildings.geojson']' returned non-zero exit status 1.
Support would be greatly appreciated. Thank you.

@ryan-routethis I've also encountered this exact same error over the last few days.
My steps can be replicated here: https://docs.overturemaps.org/examples/lonboard/#geopandas

@ryan-routethis @mtravis Could you share more info about your setup? These would be a good start:

python version
pip list | grep -E 'overturemaps|numpy|pyarrow|shapely|geopandas'
OS + version

I'll try reproducing it myself later today

@ryan-routethis Actually, it looks like this might be a memory issue - your bbox covers a huge portion of the planet which I assume is unintentional?


PGP verification of signature with SHA3 algorithm

Does anyone know when Bouncy Castle will be providing support for OpenPGP data that has been signed with a SHA3 algorithm? I can get the Bouncy Castle library to sign data using a SHA3 algorithm e.g. SHA3_256. The HashAlgorithmTags file includes the SHA3 algorithms i.e. https://github.com/bcgit/bc-java/blob/main/pg/src/main/java/org/bouncycastle/bcpg/HashAlgorithmTags.java .
But on a verify operation, it accesses the org.bouncycastle.openpgp.operator.jcajce.OperatorHelper#getDigestName method (https://github.com/bcgit/bc-java/blob/main/pg/src/main/java/org/bouncycastle/openpgp/operator/jcajce/OperatorHelper.java). This method doesn't have the SHA3 algorithms, so the operation fails with:
throw new PGPException("unknown hash algorithm tag in getDigestName: " + hashAlgorithm);
Strange that BC can sign but not verify.

Are you able to provide a (non-BC) SHA-3 signed document?


ccall with two parameters called in a dynamo

Here's an example:
julia> using IRTools: @dynamo, IR

julia> @dynamo function foo(a...)
           ir = IR(a...)
           return ir
       end

julia> mylog2(x) = ccall((:log2, Base.Math.libm), Float64, (Float64,), x)
mylog2 (generic function with 1 method)

julia> foo(mylog2, 3.3)
ERROR: TypeError: in ccall: first argument not a pointer or valid constant expression, expected Ptr, got Tuple{Symbol,String}
Stacktrace:
 [1] mylog2 at ./REPL[3]:1 [inlined]
 [2] foo(::typeof(mylog2), ::Float64) at /home/tshort/.julia/packages/IRTools/pgYVw/src/reflection/dynamo.jl:0
 [3] top-level scope at REPL[4]:1

This comes from IR conversion:
julia> @code_lowered mylog2(3.3)
CodeInfo(
1 ─ %1 = Base.cconvert(Main.Float64, x)
│   %2 = Base.unsafe_convert(Main.Float64, %1)
│   %3 = $(Expr(:foreigncall, :(Core.tuple(:log2, Base.Math.libm)), Float64, svec(Float64), :(:ccall), 1, :(%2), :(%1)))
└──      return %3
)

julia> ir = @code_ir mylog2(3.3)
1: (%1, %2)
  %3 = Base.cconvert(Main.Float64, %2)
  %4 = Base.unsafe_convert(Main.Float64, %3)
  %5 = Core.tuple(:log2, Base.Math.libm)
  %6 = $(Expr(:foreigncall, %5, Float64, svec(Float64), :(:ccall), 1, %4, %3))
  return %6

By default we split nested expressions for convenience (they usually just segfault the compiler), but clearly for foreigncalls the tuple expression needs to be inlined. The splitting happens here so that's probably where we need to check for special expression types.

Changing that line to the following seems to work. Not very pretty, though.
  if x.expr.head == :foreigncall
    e = x.expr
    x = Statement(x, expr = Expr(e.head, e.args[1], [x isa Expr ? push!(b, Statement(x, expr = a)) : a for a in e.args[2:end]]...))
  else
    x = applyex(a -> push!(b, Statement(x, expr = a)), x)
  end

To be both correct and as convenient as push! for normal expressions, we'd actually have to recursively push! all the inner expressions except for the one special one.
I don't think all that is necessary, though. We can sacrifice a tiny bit of convenience by just assuming foreigncalls are correct, and doing something like
x = isexpr(x.expr, :foreigncall) ? x :
  applyex(a -> push!(b, Statement(x, expr = a)), x)

cglobal requires the same calling convention, appearently:
julia> cglobal(:jl_n_threads, Cint)
Ptr{Int32} @0x00007fc7919a72f8

julia> f(s) = cglobal(Symbol(string(s)), Cint)
f (generic function with 1 method)

julia> f("jl_n_threads")
ERROR: TypeError: in cglobal: first argument not a pointer or valid constant expression, expected Ptr, got Symbol
Stacktrace:
 [1] f(::String) at ./REPL[2]:1
 [2] top-level scope at REPL[4]:1

But it doesn't lower to a :foreigncall expression:
julia> ir = @code_ir Threads.nthreads()
1: (%1)
  %2 = Base.Threads.cglobal(:jl_n_threads, Base.Threads.Cint)
  %3 = Base.Threads.unsafe_load(%2)
  %4 = Base.Threads.Int(%3)
  return %4

This hit me in code that I generated, an I soved it by a special case. But can it ever fail due to IRTools normal code transformations?

This is different from the invokelatest case as one of the arguments isn't required to be a nested expression; so I don't think any IRTools pass with mess with it.


fix 0-init of local ( stack ) buffer.

buffer_limit might be bigger than the MAX_PACKET_SIZE of the local buffer
usually the buffer_limit is ensured during receiving data and it should
not be possible to trigger this stack overwrite with an malicious packet.
the buffer_limit in the default config i at least twice the MAX_PACKET_SIZE
and other not connected in any way to the size of the local buffer.
for correctnes, use the local buffer size for full zero initialization.
if parital initialization is wanted then the code should read
memset(visited, 0, min(sizeof(visited), buffer_limit(buf)+7/8).
i went for the conservative full init approach.

Thanks, committed the min() approach you mentioned.  That should make sure the memset is not too long.


VS Code can't find HDF5 installation

I am using  hdf5 = "0.8.1" in my Cargo.toml in VS Code.
I went to the HDF5 website, downloaded hdf5-1.14.1-2-Std-win10_64-vs17.zip and hdf5-1.14.1-2-Std-win10_64-vs16.zip, and ran the installers to install HDF5.
I then restarted my PC for good measure.
However when I cargo build or cargo run my code, I get an error saying it can't finding any installations of HDF5, despite the fact I do have installations:
Compiling hdf5-sys v0.8.1
error: failed to run custom build command for `hdf5-sys v0.8.1`

Caused by:
process didn't exit successfully: `blahblahblah` (exit code: 101)
--- stdout
Searching for installed HDF5 (any version)...
Found no HDF5 installations.

--- stderr
thread 'main' panicked at 'Unable to locate HDF5 root directory and/or headers.', blahblahblah
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

(the blahblahblah is the file path)
How do I make hdf5 Rust in VS Code find the installations?

@themathsgirl Hi, I am in the same situation as you, how do you fix that?

I didn't - I ended up converting the HDF5 data into CSV (via h5py) and then ran my Rust code on the CSV data.


[RFE] Simple Permissions endpoint for UI

Is your feature request related to a problem? Please describe.
The UI needs a way for the logged in user to quickly and easily determine their permissions to an individual resource for the purposes of enabling/disabling UI elements based off a user's Global/Cluster/Project roles which may be directly assigned to a user or may be inherited via group membership, special care should be taken to account for both the "Admin" and "Restricted-Admin" roles, both of which adhere to special logic as it relates to cluster and project permissions.
Describe the solution you'd like
There's numerous ways to do this well from a FE perspective but we're looking for the following criteria:

dense: minimize bytes sent over the wire for rancher instances that include more clusters/projects
flattened: this endpoint shouldn't show individual permissions are bound to the user, just that it is bound to the user
contextual: this endpoint only returns data for the current user
easily filtered on the frontend: rather than making dozens of calls with individual filters we'd prefer to make a single call to the endpoint (or at most filtered to a single cluster) and the data should be structured such that keylookups by resource and project are quick (ideally without iterating over large arrays)
fast and watchable: ideally the initial call to this endpoint should complete in under 250ms or faster (assuming minimal latency) and the FE should be able to watch for changes to permissions via websocket

Describe alternatives you've considered
Performing this type of permission mapping on the frontend requires a minimum of 4 (and upwards of 6 in some cases) separate endpoints, iterating over potentially large lists to discard duplicates, duplicating application logic embedded into the backend and still seems to miss a few important use-cases as our prototypes have shown.

This ties in with https://jira.suse.com/browse/SURE-6164
Generally, the UI needs to understand it can do action x to resource y in context z. For example

Add a user as a member of a project
Remove a user as an owner from a cluster
Remove a user's restricted-admin's role
Edit a project (i don't think this is achievable via the resource's link.update value)
Add a configmap in a namespace

Historically this was done via either a global schema applying to all resources, or the contents of a resource's actions/links collection. We would receive updates to both over socket if anything changed.
We need to work together to understand how this can be done in a way that scales, automatically updates and integrates with the UI well

Ties into fixing a bug reported by QA in https://github.com/rancher/dashboard/issues/9159 -- immediate blocker was addressed.
Initial attempts to address this were done in https://github.com/rancher/dashboard/pull/9184 but that's blocked on this ticket.

Internal reference: SURE-6067

Also this blocks https://github.com/rancher/dashboard/issues/8544

Moving out of Q1 in order to do this instead: https://github.com/rancher/rancher/issues/41301

Closing in preferences of SURE-6164


Add LaTeX support

To make it possible to insert mathematical formula, I think LaTeX rendering functionality should be added.

What about KaTeX or something else that can be used to render LaTeX on the front-end?


Footer

Dear developer
How to change footer ?

Thanks for using this theme, you mean footer is link section at index page? You can edit this section in file _includes/sections/links.html, but data are written at _data/links.yml, hope this can help you.

Thank you so much..


chore(npm): update package.json and added circle config for npm publish step

Reverts Pocket/apollo-cursor-pagination #4
Goals

add circle ci jobs to publish npm package step

Implementation decisions:

circle ci config is in same format as that of terraform-modules's circle config. Removed terraform related configs
in package.json, took the format of types, publishConfig, release and repository from apollo-utils's package.json
removed the yarn.lock file as we don't use it in our circle ci config

Note:

the npm publish steps stopped, but looks similar to the steps in terraform-module PR: https://app.circleci.com/pipelines/github/Pocket/terraform-modules/2085/workflows/5de196e9-7fb6-4c5b-8861-f356d5456b08/jobs/1885
my guess is it might get triggered after a merge to main.

JIRA Ticket: INFRA-213
old PR is here: https://github.com/Pocket/apollo-cursor-pagination/pull/3

When this was private and just used by list-api we didn't update the readme, but I don't think those installation instructions are correct anymore.

When this was private and just used by list-api we didn't update the readme, but I don't think those installation instructions are correct anymore.

update readme installation instruction, description was earlier updated...everything below looks like how we use the pagination, and not sure about the test...do we have to update these yarn to npm?

update readme installation instruction, description was earlier updated...everything below looks like how we use the pagination, and not sure about the test...do we have to update these yarn to npm?

I think so, but double check by getting everything running locally.


Wire up passing LNK/EXE data from OpenCon to ITerminalHandoff

This PR by itself doesn't really change much. Technically, now the Terminal will respect the Title of a .lnk when started for defterm, but we don't do anything else yet. Primarily, the goal of this PR is to just wire up startup info in OpenConsole to the connected Terminal.

This required a bit of changes in srvinit.cpp:ConsoleEstablishHandoff to replicate other bits of startup, where we crack open the connect message to get the relevant bits of info.
We pack that all into a TERMINAL_STARTUP_INFO, which we pass along to the registered terminal application.
ConptyConnection accepts the handoff, and gathers that information out of the TERMINAL_STARTUP_INFO
Some other updates to the scratch sln were made to make it build again (related, but unimportant).
This is a precursor to:

#13111
#12154

Closes #9458
Tested manually
I work here

hey look, this made this better:

hey look, this made this better

helk yeah

Yo @DHowett I did run this with appverifier and this ran clean 😄

@msftbot make sure @DHowett-MSFT signs off

This will never be true. It will not clear this status, so you'll have to manually merge it. :)


Update react-native-web version

Hi, the used react-native-web version is very old. If i try to update it i get some errors. Could you try to update the react-native-web dependency. I would need FlatList to work, which is not right now.

Hey there,
Updated the base code containing a wip updated version of react-native-web library, feel free to send a PR in case something won't work as expected.

@agrcrobles thanks, in my personal project i am using now react-native-web "react-native-web": "0.8" and       "react": "16.3.1",   "react-dom": "16.3.1". If i ever start a new project i will let you know. I only know that the version of react-native-web and react and react-dom can cause some problems.

sure let me know, yeah I am aware of that, I updated it with react 16.5.0 react-dom ^16.8.6 and react-native-web 0.11.2 and it worked it's still in wip


[Bug]: The placement option on the button widget does not work correctly.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The placement option on the button widget does not work correctly. Start and Between both keep the button label on the left side. There is no option to align the button label to the right side.
Also, actions on the label are misleading. It should be Left, Center, and Right or it should be Start, Center, and End.
Widgets where this problem is occurring:-

Button Widget
Menu Button Widget
Code Scanner
Form widget button
JsonForm Widget Button

Also, check out this https://github.com/appsmithorg/appsmith/issues/10219
Steps To Reproduce

D&D Button widget or the above-mentioned widgets.
Select the Placement property under the Style pane

Public Sample App
No response
Environment
Release
Issue video log
https://www.loom.com/share/713af3a0036740979c163f64052ecd20
Version
Selft-Hosted

@pranavkanade I want your attention to this.

@somangshu Please take a look at this. I think this is something big.

@iSatVeerSingh thanks for reporting this!
There is more context to this, Ill go though them 1 by 1

The placement option on the button widget does not work correctly. Start and Between both keep the button label on the left side. There is no option to align the button label to the right side.

Check this out 👇 , there is actually a way to do it, however I feel the name between is actually space-between flex property and is kinda misleading right now, cc @ichik @KelvinOm @jsartisan
https://user-images.githubusercontent.com/11089579/230360352-e75ef1f9-3413-4e04-b267-847f7510749e.mov

actions on the label are misleading. It should be Left, Center, and Right

Did not understand what you mean by this?

What if a user wants to place both icon and text to the end(most right)?

actions on the label are misleading. It should be Left, Center, and Right.
I mean Position of text and icon at most left part of button or at center or at most right part of button.

What if a user wants to place both icon and text to the end(most right)?

I agree that this option is missing and should be present.

It should be Left, Center, and Right.

We are using the flex box model, that means space-between which is currently between is a valid option, We will think more around the naming convention. cc @dilippitchika

I think the Between or Space-Between option should only be present if there are two or more items in the button, for example, Text and Icon. If user want to add only text or only icon, then Between option does not make any sense.

Closing this in favour of #10219