Fix the configuration system

The configuration was built with the idea of running the application from a library.
It supported local configuration and access to different databases.
This doesn't really work when running the application in docker.

This has been slightly improved by #17 but it should be better.


[Outlook] (manifest) Update ordering of SupportsSharedFolders element

Addresses issue https://github.com/OfficeDev/office-js-docs-pr/issues/863.

Docs Build status updates of commit 8896d05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/manifest/supportssharedfolders.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit a3fde19:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/manifest/supportssharedfolders.md
:white_check_mark:Succeeded

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


[RESTEASY-2201] update jackson in jose-jwt

WF module.xml needs to be updated as well

WF module.xml needs to be updated as well
https://github.com/wildfly/wildfly/blob/master/feature-pack/src/main/resources/modules/system/layers/base/org/jboss/resteasy/jose-jwt/main/module.xml

That would be done when upgrading the resteasy version in wfly, yep

Having heard no real objections, I'm merging this (cc @marekkopecky )


[MLO Ytyp] huge lag with portal filter

🔍 Issue Description
Hello,
i have a MLO with 15 portals. If i clic on the mlo archetype => Entites => Filter by portal  : i have huge lags.
Everything is ok if i use "Room" filter or "All" filter.
If i do the same thing with a mlo ytyp with 5 portals everything is ok too. It seems to be due to the number of portal.
PS : i had the same issue with sollumz 2.4 & blender 4.1, i updated to sollumz 2.6 to see if it can solve the issue.
Also, note that this project has been made with sollumz 2.4 & blender 4.1.
Let me know if you need more information.
Thanks a lot for your help !
🛠 Sollumz Version(s)
2.6
🖥 Blender Version(s)
4.2.4
🧩 Area
Archetypes
🔄 Reproduction steps

🎯 Expected Result
No response
⚠ Current Result
Huge lags
📎 Attachments
No response

Yes, same issue as reported here https://github.com/Sollumz/Sollumz/issues/801.


Code Fix.

Why would u want to change Math.random().toString(36).substring(7) by new randomstring package? 🤷♂

Why would u want to change Math.random().toString(36).substring(7) by new randomstring package? 🤷♂

It was easy to just do with randomstring npm, but i will change it back to that.


Update token in Headers

Best practice for updating the token parameter in the swiftStomp headers?

If you want to update the header during the upgrade and establish the connection, please consider, it is not possible to update the headers, when the connection is established. You need to reconnect with the new authorization tokens.


Add version: Baidu.SwanIDE version 4.8.2

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?


[Tooling] Tag for the TH SHA

Reproduction steps / Feature
We need a tag or branch for the TH SHA both so we have an official record of where this was taken and so devs can easily repro.
Platform
core (please add to version below)
Platform Version(s)
No response
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

what is the SHA we need to tag? 19771ed7 or something else?

Yeah , [19771ed7101321d68b87d05201d42d00adf5368f] is the SHA we made the final release for TH

@andy31415 can we tag the above SHA ?

what should be the tag name? what is the description for it?

I understand that the tag should be created for previous Matter releases Test Harness (Matter 1.0, Matter 1.1),

@raju-apple - can you answer above about tag names?

@andy31415 Lets do TH-Matter-1.2 ? ( Current Matter Test Harness-Fall 2023 version is based of this SDK SHA )

https://github.com/project-chip/connectedhomeip/releases/tag/TH-Matter-1.2 created


Update admin boundaries layers in GeoNode

Remove GADM admin boundaries from GeoNode and add UN OCHA ones. This is because the UN OCHA is up to date with the changes made in provincial, districts and municipal borders in 2015.

Done


Update cli commands in documentation

Update the doc and cli command to make it clear which id needs to be given the Monitoring Metrics Publisher role

Docs Build status updates of commit 3bb4034:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-monitor/containers/container-insights-update-metrics.md
:white_check_mark:Succeeded

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

#sign-off


Two SVG bugs

Newlines \n in svg specification seems to break in some browsers.
Images that are too large and then scaled down cause Affine to think transformation are degenerate.

Small temporary fixes.

Hey! Thanks for the fixes!
The image fix works only for the SVG backend, right? I wonder if it would be more flexible (backend agnostic) to apply the scaling only when we create the image primitive:
 def image(local_path: str, url_path: Optional[str]) -> Diagram:
     from chalk.core import Primitive

-    return Primitive.from_shape(Image(local_path, url_path))
+    return Primitive.from_shape(Image(local_path, url_path)).scale(0.05)

Somewhat unrelated, but what should the url_path argument from the image function be set to (I see that's used  only in the SVG backend)?

I think we need a couple long term fixes.

figure out why large scaling ops break the degenerate check in affine.
make image backend specific. Diagrams also has some backend specific code (svgtext)
pull out and document scaling constants. I.e default images to local width 1? Not sure what is right to do here.

Okay! So I guess you are suggesting to merge these temporary fixes for the time being and create separate issues for the long-term problems, right? I'm fine with that, but I just wanted to point out that the current changes will affect the image rendering with cairo (since the Image's width and height do not match the true width and height of the PIL image data); I don't have a satisfactory solution—maybe we could try resizing the image by the 0.05 factor in the from_pil function?
     format: cairo.Format = cairo.FORMAT_ARGB32
+    im = im.resize((int(im.width * 0.05), int(im.height * 0.05)))
     if "A" not in im.getbands():

We don't need to merge. I'll fix Cairo for real here.
Mostly just sent this to remind myself since I needed it in my code.

Ah, cool! Sorry for misunderstanding! I'll take a look at #113 then if you haven't started already working on it.


Samples tests failing on master

Jiggity jig.
https://circleci.com/gh/googleapis/nodejs-speech/4857

Related to #173


Nothing works

I get errors like this for like everything https://mclo.gs/lg9MRZX

can you please try this version


Incorrect grammar for SERVICE_OBJECTIVE on CREATE DATABASE page

On the CREATE DATABASE page for SQL Database single database/elastic pool, the grammar for SERVICE_OBJECTIVE is not correct: every line end has | and subsequently every new line has | at the beginning. This prevents any grammar parser from understanding the alternatives.
For instance:
{ 'basic' | 'S0' | 'S1' | 'S2' | 'S3' | 'S4'| 'S6'| 'S7'| 'S9'| 'S12' |
      | 'P1' | 'P2' | 'P4'| 'P6' | 'P11' | 'P15'
      | 'GP_GEN4_1' | 'GP_GEN4_2' | 'GP_GEN4_3' | 'GP_GEN4_4' | 'GP_GEN4_5' | 'GP_GEN4_6' |
      | 'GP_Gen4_7' | 'GP_Gen4_8' | 'GP_Gen4_9' | 'GP_Gen4_10' | 'GP_Gen4_16' | 'GP_Gen4_24' |
      | 'GP_Gen5_2' | 'GP_Gen5_4' | 'GP_Gen5_6' | 'GP_Gen5_8' | 'GP_Gen5_10' | 'GP_Gen5_12' | 'GP_Gen5_14' |
      | 'GP_Gen5_16' | 'GP_Gen5_18' | 'GP_Gen5_20' | 'GP_Gen5_24' | 'GP_Gen5_32' | 'GP_Gen5_40' | 'GP_Gen5_80' |
      | 'BC_Gen4_1' | 'BC_Gen4_2' | 'BC_Gen4_3' | 'BC_Gen4_4' | 'BC_Gen4_5' | 'BC_Gen4_6' |
      | 'BC_Gen4_7' | 'BC_Gen4_8' | 'BC_Gen4_9' | 'BC_Gen4_10' | 'BC_Gen4_16' | 'BC_Gen4_24' |
      | 'BC_Gen5_2' | 'BC_Gen5_4' | 'BC_Gen5_6' | 'BC_Gen5_8' | 'BC_Gen5_10' | 'BC_Gen5_12' | 'BC_Gen5_14' |
      | 'BC_Gen5_16' | 'BC_Gen5_18' | 'BC_Gen5_20' | 'BC_Gen5_24' | 'BC_Gen5_32' | 'BC_Gen5_40' | 'BC_Gen5_80' |
      | 'HS_GEN4_1' | 'HS_GEN4_2' | 'HS_GEN4_4' | 'HS_GEN4_8' | 'HS_GEN4_16' | 'HS_GEN4_24' |
      | 'HS_GEN5_2' | 'HS_GEN5_4' | 'HS_GEN5_8' | 'HS_GEN5_16' | 'HS_GEN5_24' | 'HS_GEN5_32' | 'HS_GEN5_48' | 'HS_GEN5_80' |
      | { ELASTIC_POOL(name = <elastic_pool_name>) } })

Should be
{ 'basic' | 'S0' | 'S1' | 'S2' | 'S3' | 'S4'| 'S6'| 'S7'| 'S9'| 'S12'
      | 'P1' | 'P2' | 'P4'| 'P6' | 'P11' | 'P15'
      | 'GP_GEN4_1' | 'GP_GEN4_2' | 'GP_GEN4_3' | 'GP_GEN4_4' | 'GP_GEN4_5' | 'GP_GEN4_6'
      | 'GP_Gen4_7' | 'GP_Gen4_8' | 'GP_Gen4_9' | 'GP_Gen4_10' | 'GP_Gen4_16' | 'GP_Gen4_24'
      | 'GP_Gen5_2' | 'GP_Gen5_4' | 'GP_Gen5_6' | 'GP_Gen5_8' | 'GP_Gen5_10' | 'GP_Gen5_12' | 'GP_Gen5_14'
      | 'GP_Gen5_16' | 'GP_Gen5_18' | 'GP_Gen5_20' | 'GP_Gen5_24' | 'GP_Gen5_32' | 'GP_Gen5_40' | 'GP_Gen5_80'
      | 'BC_Gen4_1' | 'BC_Gen4_2' | 'BC_Gen4_3' | 'BC_Gen4_4' | 'BC_Gen4_5' | 'BC_Gen4_6'
      | 'BC_Gen4_7' | 'BC_Gen4_8' | 'BC_Gen4_9' | 'BC_Gen4_10' | 'BC_Gen4_16' | 'BC_Gen4_24'
      | 'BC_Gen5_2' | 'BC_Gen5_4' | 'BC_Gen5_6' | 'BC_Gen5_8' | 'BC_Gen5_10' | 'BC_Gen5_12' 'BC_Gen5_14'
      | 'BC_Gen5_16' | 'BC_Gen5_18' | 'BC_Gen5_20' | 'BC_Gen5_24' | 'BC_Gen5_32' | 'BC_Gen5_40' | 'BC_Gen5_80'
      | 'HS_GEN4_1' | 'HS_GEN4_2' | 'HS_GEN4_4' | 'HS_GEN4_8' | 'HS_GEN4_16' | 'HS_GEN4_24'
      | 'HS_GEN5_2' | 'HS_GEN5_4' | 'HS_GEN5_8' | 'HS_GEN5_16' | 'HS_GEN5_24' | 'HS_GEN5_32' | 'HS_GEN5_48' | 'HS_GEN5_80'
      | { ELASTIC_POOL(name = <elastic_pool_name>) } })

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

ID: 9e8abfde-2523-5c70-bb95-b10101a726ff
Version Independent ID: 3b8152d2-b49e-df4b-0f05-3ec52aa5bb71
Content: CREATE DATABASE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/statements/create-database-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@nahk-ivanov thank you for the submission. We're looking at the problem you describe.

@nahk-ivanov - thanks for pointing this out!  A fix is in process.

@nahk-ivanov - the updated article is live.
Thanks again!


usdt,uprobe: resolve binary name from PATH

@danobi any special reason to use command -pv  instead of which  on #303? command -pv  didn't worked here to resolve node when using nvm.

@mmarchini yeah this was the reason: https://github.com/koalaman/shellcheck/wiki/SC2230 .
On the nvm README, there seems to be a note about it:
Note: On Linux, after running the install script, if you get nvm: command not found or see no feedback from your terminal after you type:

command -v nvm

simply close your current terminal, open a new terminal, and try verifying again.

command -v node works, command -pv node doesn't 🤔
Can we use command -v instead?

Yeah. Hmm I wonder why I added -p. Reading through the man page, it seems we would want the user's PATH and not just the default.

Thanks; although has conflicts now I merged the other one (at long last).
library path next? should be able to do u:libc:strlen or u:c:strlen (bcc's trace has a syntax like the latter where the "lib" is dropped, but I think I'd prefer to see it, so u:libc:strlen).


ERROR while trying to access the Normalizedlandmarks from face mesh

I have been trying to access the landmarks from the face mesh . I have followed the steps given in https://github.com/google/mediapipe/issues/200#issuecomment-546619563 .
I get the following error :
 Compiling mediapipe/framework/formats/landmark.pb.cc [for host] failed: undeclared inclusion(s) in rule '//mediapipe/framework/formats:landmark_cc_proto': this rule is missing dependency declarations for the following files included by 'mediapipe/framework/formats/landmark.pb.cc':   'mediapipe/framework/formats/landmark.pb.h' Target //mediapipe/examples/desktop/face_mesh:face_mesh_cpu failed to build
OS : Windows 10
So while looking into BUILD file related to landmark_proto, I see that there are no dependencies mentioned (reference: https://github.com/google/mediapipe/blob/a92cff7a60031f5c3097b06e74416732d85b5011/mediapipe/framework/formats/BUILD#L250 ). Is there something that this file is missing in terms of mentioning the dependencies for landamrk_proto??
Could anyone help me with this?

I can successfully build it bybazel build -c opt --action_env PYTHON_BIN_PATH="C://Path//to//python.exe"  --define MEDIAPIPE_DISABLE_GPU=1 mediapipe/framework/formats:landmark_cc_proto. What's the issue? Can you list all your changes?

I have made the following changes :
Created a new file demo_run_graph_main_landmarks.cc inside mediapipe\mediapipe\examples\desktop. This file is a replication of demo_run_graph_main.cc only difference is that I have the code to get the landmarks .
I have included the followig headers:
#include "mediapipe/calculators/util/landmarks_to_render_data_calculator.pb.h" #include "mediapipe/framework/formats/landmark.pb.h"
And then added a variable to tap landmarks :
`constexpr char kDetectionsStream[] = "multi_face_landmarks";'
And then included the code to poll the output packet with landmarks as follows:
ASSIGN_OR_RETURN(mediapipe::OutputStreamPoller poller_detection,   graph.AddOutputStreamPoller(kDetectionsStream));
Got the landmarks from the packet using :
auto& output_landmarks = detection_packet.Get<std::vector<::mediapipe::NormalizedLandmarkList>>();
And then trying to print the landmarks :
for (const ::mediapipe::NormalizedLandmarkList& normalizedlandmarkList : output_landmarks) {         std::cout << normalizedlandmarkList.DebugString();     }
After which I added the following lines to BUILD file inside mediapipe\mediapipe\examples\desktop
`cc_library(
name = "demo_run_graph_main_out",
srcs = ["demo_run_graph_main_landmarks.cc"],
deps = [
"//mediapipe/calculators/util:landmarks_to_render_data_calculator",
"//mediapipe/framework:calculator_framework",
"//mediapipe/framework/formats:image_frame",
"//mediapipe/framework/formats:image_frame_opencv",
"//mediapipe/framework/formats:landmark_cc_proto",
"//mediapipe/framework/port:commandlineflags",
"//mediapipe/framework/port:file_helpers",
"//mediapipe/framework/port:opencv_highgui",
"//mediapipe/framework/port:opencv_imgproc",
"//mediapipe/framework/port:opencv_video",
"//mediapipe/framework/port:parse_text_proto",
"//mediapipe/framework/port:status",
],

)`
And also I have changed the line:30 in BUILD file inside mediapipe\mediapipe\examples\desktop\face_mesh to :
"//mediapipe/examples/desktop:demo_run_graph_main_out",
For the build I am using the following command :

`

I copied your code into demo_run_graph_main.cc and compiled mediapipe/examples/desktop/face_mesh:face_mesh_cpu. It went well on a surface pro:

The diff is the following. Note that I didn't check the correctness of the code and just copied your code into the files.
diff --git a/mediapipe/examples/desktop/BUILD b/mediapipe/examples/desktop/BUILD
index 7772e21..c1ec33e 100644
--- a/mediapipe/examples/desktop/BUILD
+++ b/mediapipe/examples/desktop/BUILD
@@ -48,6 +48,8 @@ cc_library(
+++ b/mediapipe/examples/desktop/BUILD
@@ -48,6 +48,8 @@ cc_library(
         "//mediapipe/framework/port:opencv_video",
         "//mediapipe/framework/port:parse_text_proto",
         "//mediapipe/framework/port:status",
+        "//mediapipe/calculators/util:landmarks_to_render_data_calculator",
+        "//mediapipe/framework/formats:landmark_cc_proto",
     ],
 )

diff --git a/mediapipe/examples/desktop/demo_run_graph_main.cc b/mediapipe/examples/desktop/demo_run_graph_main.cc
index e09d2ab..57ed708 100644
--- a/mediapipe/examples/desktop/demo_run_graph_main.cc
+++ b/mediapipe/examples/desktop/demo_run_graph_main.cc
@@ -25,6 +25,8 @@
 #include "mediapipe/framework/port/opencv_video_inc.h"
 #include "mediapipe/framework/port/parse_text_proto.h"
 #include "mediapipe/framework/port/status.h"
+#include "mediapipe/calculators/util/landmarks_to_render_data_calculator.pb.h"
+#include "mediapipe/framework/formats/landmark.pb.h"

 constexpr char kInputStream[] = "input_video";
 constexpr char kOutputStream[] = "output_video";
@@ -53,7 +55,7 @@ mediapipe::Status RunMPPGraph() {
   LOG(INFO) << "Initialize the calculator graph.";
   mediapipe::CalculatorGraph graph;
   MP_RETURN_IF_ERROR(graph.Initialize(config));
-
+
   LOG(INFO) << "Initialize the camera or load the video.";
   cv::VideoCapture capture;
   const bool load_video = !FLAGS_input_video_path.empty();
@@ -78,6 +80,8 @@ mediapipe::Status RunMPPGraph() {
   LOG(INFO) << "Start running the calculator graph.";
   ASSIGN_OR_RETURN(mediapipe::OutputStreamPoller poller,
                    graph.AddOutputStreamPoller(kOutputStream));
+  ASSIGN_OR_RETURN(mediapipe::OutputStreamPoller poller_detection,
+                   graph.AddOutputStreamPoller("multi_face_landmarks"));
   MP_RETURN_IF_ERROR(graph.StartRun({}));

   LOG(INFO) << "Start grabbing and processing frames.";
@@ -118,7 +122,14 @@ mediapipe::Status RunMPPGraph() {
     mediapipe::Packet packet;
     if (!poller.Next(&packet)) break;
     auto& output_frame = packet.Get<mediapipe::ImageFrame>();
-
+    mediapipe::Packet detection_packet;
+    if (!poller_detection.Next(&packet))
+      break;
+    auto &output_landmarks = detection_packet.Get<std::vector<::mediapipe::NormalizedLandmarkList>>();
+    for (const ::mediapipe::NormalizedLandmarkList &normalizedlandmarkList : output_landmarks)
+    {
+      std::cout << normalizedlandmarkList.DebugString();
+    }
     // Convert back to opencv for display or saving.
     cv::Mat output_frame_mat = mediapipe::formats::MatView(&output_frame);
     cv::cvtColor(output_frame_mat, output_frame_mat, cv::COLOR_RGB2BGR);

@jiuqiant  Yeah this works. the build is successful now I am not sure what's wrong with the new file though. Now when try to run the face_mesh_cpu.exe by passing the corresponding calculator graph config file I get the following error :
F20210312 16:41:07.562562 24428 packet.h:768] Packet::Get() failed: Expected a Packet of type: class std::vector<class mediapipe::NormalizedLandmarkList,class std::allocator<class mediapipe::NormalizedLandmarkList> >, but received an empty Packet.
Command used : "bazel-bin/mediapipe/examples/desktop/face_mesh/face_mesh_cpu.exe" --calculator_graph_config_file=mediapipe/graphs/face_mesh/face_mesh_desktop_live.pbtxt
In the pdtxt file I [assed, I can see that there is an output stream named "multi_face_landmarks" of type std::vector. I am not sure why it says it received empty packet . What can be the reasons for this ? Am I accessing it the wrong way ? Were you able to run the executable and print the landmarks?

Yeah, because there is a bug in the code. Should be poller_detection.Next(&detection_packet) rather than poller_detection.Next(&packet).
    mediapipe::Packet detection_packet;
    if (!poller_detection.Next(&detection_packet))
      break;
    auto &output_landmarks = detection_packet.Get<std::vector<::mediapipe::NormalizedLandmarkList>>();
    for (const ::mediapipe::NormalizedLandmarkList &normalizedlandmarkList : output_landmarks)
    {
      std::cout << "FaceLandmarks:";
      std::cout << normalizedlandmarkList.DebugString();
    }

Then, you will get something like the following in the terminal:

BTW, you can use the MediaPipe Python PyPI package if it's not necessary to be c++.

@jiuqiant  Yeah I just came here to mention that. I figured it out. Anyways thanks man for the help :)

Glad to help. Closing this issue now.

I am facing the issue

Yeah, because there is a bug in the code. Should be poller_detection.Next(&detection_packet) rather than poller_detection.Next(&packet).
    mediapipe::Packet detection_packet;
    if (!poller_detection.Next(&detection_packet))
      break;
    auto &output_landmarks = detection_packet.Get<std::vector<::mediapipe::NormalizedLandmarkList>>();
    for (const ::mediapipe::NormalizedLandmarkList &normalizedlandmarkList : output_landmarks)
    {
      std::cout << "FaceLandmarks:";
      std::cout << normalizedlandmarkList.DebugString();
    }

Then, you will get something like the following in the terminal:
BTW, you can use the MediaPipe Python PyPI package if it's not necessary to be c++.

I have been trying to get the iris_landmarks.
Even after resolving everything the way it is recommended ... I am facing the an issue. The console shows no error but screen just freezes with camera on but giving no response at all ... Please help me..

Yeah, because there is a bug in the code. Should be poller_detection.Next(&detection_packet) rather than poller_detection.Next(&packet).
    mediapipe::Packet detection_packet;
    if (!poller_detection.Next(&detection_packet))
      break;
    auto &output_landmarks = detection_packet.Get<std::vector<::mediapipe::NormalizedLandmarkList>>();
    for (const ::mediapipe::NormalizedLandmarkList &normalizedlandmarkList : output_landmarks)
    {
      std::cout << "FaceLandmarks:";
      std::cout << normalizedlandmarkList.DebugString();
    }

Then, you will get something like the following in the terminal:
BTW, you can use the MediaPipe Python PyPI package if it's not necessary to be c++.

I have been trying to get the iris_landmarks. Even after resolving everything the way it is recommended ... I am facing an issue. The console shows no error but screen just freezes with camera on but giving no response at all ... Please help me..

Hello! I have the same issue, as you do. I am also trying to figure out for iris. Have you managed to solve it?


RuntimeException

When processing the attached file a RuntimeException is thrown with the message to report the problem.
closure_rte.zip

ControlFlowAnalysis crashes on baz: break baz;
Repro:
https://closure-compiler-debugger.appspot.com/#input0%3Dbaz%253Abreak%2520baz%253B%26input1%26conformanceConfig%26externs%26refasterjs-template%26CHECK_SYMBOLS%3Dtrue%26MISSING_PROPERTIES%3Dtrue%26TRANSPILE%3Dtrue%26CHECK_TYPES%3Dtrue%26CLOSURE_PASS%3Dtrue%26PRESERVE_TYPE_ANNOTATIONS%3Dtrue%26PRETTY_PRINT%3Dtrue


Added render_mode world_vertex_coords in canvas

This PR brings the render mode world_vertex_coords to canvas item shaders. It also provides access to two new uniform matrices for converting to and from world coordinates named  GLOBAL_MATRIX and INV_GLOBAL_MATRIX respectively. I would prefer to name them WORLD_MATRIX (which is what it is internally) and INV_WORLD_MATRIX, however, for some reason the modelview transform is accessed using WORLD_MATRIX (which is incredibly confusing for users).
The behaviour is the same as in spatial shaders. It presents VERTEX to the user in world coordinates, and then transforms it into view and NDC coordinates after the user written code is run.
Many users have asked for access to this render mode and it directly addresses issue #19800. With the current system there is no way to acquire world position in the shader without passing in the global transform as a mat4 (which is currently broken, but fixed with #23976) or passing in the global position, scale, rotation and calculating the position manually (which is needlessly complicated/difficult).
Bugsquad edit: Fixes #19800.

Actually I'd like @reduz to have a glance at this one before moving to 3.2. It doesn't really impact the existing features, so it could still be merged.

I am not sure about this for a number of reasons:
-Goes against eventually adding batching (so maybe, you could check if this is used, and if not the geometry is batched)
-Will likely hit performance a tiny bit on mobile, though probably not so bad.
Would kick for after 3.1 is out to discuss properly.

@reduz do you think it would be a good idea to make a note in the documentation for now?

Might be time to revisit this PR?

Sure! I can resolve conflicts and we can see what reduz thinks.

Bump :)

@akien-mga Thanks for the reminder :)
The merge conflict led me to realize that there was a bug in nearby code that would have broken particles in GLES2 if uses used "skip_vertex_transform". So I fixed the bug and included it in this PR. If reduz decides not to merge this for 3.2 then I need to fix the bug in its own PR.

We discussed this on IRC, @reduz is not so keen on merging this as it has already been fixed properly in the vulkan branch for 4.0, and he prefers not to add a workaround in 3.2 in the meantime:
13:56 <Akien> #24002 (clayjohn)
13:56 <IssueBot> #24002: Added render_mode `world_vertex_coords` in canvas | https://git.io/fpzJP
13:57 <Akien> This one was "Would kick for after 3.1 is out to discuss properly." but the discussion didn't really happen :D
13:57 <reduz> I think I changed this in 4.0 branch
13:57 <reduz> or actually yes, its different in 4.0, so you dont need this
13:58 <Akien> Would it be worth merging for 3.2 only?
13:59 <reduz> i wouldnt change 3.2 rendering much tbh, but as you prefer
13:59 <reduz> this is more of a workaround for a limitation in 3.2 design which was fixed in 4.0
14:01 <Akien> Are there drawbacks to the workaround, or is it ok as band-aid until 4.0? Since people will still make and release games with 3.2 (likely many will stay on 3.2 for current games in production, since porting to 4.0 might appear as work they don't want to do)
14:03 <Akien> On the PR you mentioned that it goes against implementing batching, but that's unlikely to happen for 3.2 anyway.
14:03 <reduz> Akien: I dont even know how useful it is
14:03 <Akien> Well fracteed and a few other users seem to really want it
14:03 <Akien> #19800
14:04 <IssueBot> #19800: World_vertex_coords flag required in canvas_item shaders | https://git.io/fjxIm
14:04 <reduz> the problem is that you cant really have real world coordinates in canvas item shaders in 3.x
14:04 <reduz> I am not sure if this fixes that
14:04 <reduz> because canvas transform is preapplied to the transform
14:05 <Akien> I guess we should ask fracteed and the other interested user to test the PR and check if that does what they need?
14:05 <reduz> this solution is hacky, while in 4.0 it is like this by default
14:06 <reduz> i generally prefer to do things properly in general even if it takes longer
14:06 <Akien> Yeah but that's the engine dev perspective, not the engine user one :P
14:07 <reduz> if its up to me I would not merge it, but feel free to do as you wish
14:07 <Akien> I'll just ask the users to test it and confirm that it solves something they really need in 3.x -- otherwise we can skip it indeed.
14:08 <Akien> I think fracteed is already waiting for Vulkan for his game anyway, so he may not need this in 3.2 so badly.
14:08 <reduz> ask them if they will really use it, knowing its going to be different in 4.0 given it has world coords by default

@fracteed @bfishman Do you still need this feature, and if so, do you need it in 3.2 or can your projects wait for 4.0? And if you need it in 3.2, could you test this PR and check that it actually solves your use case?
@clayjohn I guess the bugfix you mentioned would be worth having in another PR.

@akien-mga sure I am happy to wait for 4.0 for this feature.

@akien-mga personally I can wait. I had to put all non essential projects on hold because of life and don't expect to revisit until 2020.  And honestly, once we have easy access to compute shaders I will end up taking a different approach for this particular use case anyhow.
Thank you all

Thanks for the update. Given @reduz's reluctance with merging this for 3.2, I guess we'll skip it then.
I set #19800 to the 4.0 milestone, we'll have to confirm that it's actually fixed as expected in the vulkan branch, when it's ready for broader testing.


(v4) customize input height not working for IE 9 to 11

Just notice that the height for  is commented out for the inputs:
https://github.com/twbs/bootstrap/blob/v4-dev/scss/_forms.scss#L11
  // // Make inputs at least the height of their button counterpart (base line-height + padding + border)
  // height: $input-height;

I want to customize my input height to 48 px:
I do the following change:
$input-line-height : 30px;
$input-padding-y: 8px:
$input-btn-border-width: 1px;
base line-height + padding + border = 48 px

It works perfectly for Chrome + Firefox+ IE edge, but it does not work for IE 9-11.
Plunker:
https://plnkr.co/edit/kJdhvFyAZmDVkeTATkiW?p=preview
Just open above link in Chrome and IE edge. You will find the difference.
Is there any workaround for this?

IE 9 isn't supported. Have you tested on IE 11?

@wolfy1339  IE 11 not working

line-height doesn't really have an affect on the computed height of <input>s in IE. The only way to do it right is to set a fixed height and apply a mix of line-height and padding alongside that. Alternatively, you can try your best to not care about slight inconsistencies across browsers.
I'll either be removing that commented out code, or redoing the input sizing a bit here soon.


Fix cases where failing to create a backend resource doesn't result in SG_RESOURCESTATE_FAILED.

See: https://github.com/floooh/sokol/pull/883

Fixed for Metal, but in GL or WebGPU this isn't really fixable for out-of-memory situations. In GL, calling glGetError() causes a full pipeline flush in WebGL, and with WebGPU's asynchronous error reporting it's also not possible to immediately return an out-of-memory error at object creation time.

Fixed.


Add support for Ready, Hostname and Nodename Endpoint attributes. And some other refactoring changes.

Issue #, if available:  #192
Description of changes:

Add support for Ready, Hostname and Nodename Endpoint attributes.
Refactor cluster properties changes - move to model package, simplify code.
Add controller watches for Service event. Filter out events if cluster properties is not defined.
Graduate from k8s.io/api/discovery/v1beta1 to k8s.io/api/discovery/v1 (v1beta1 will not be supported from k8s v1.25)
Improve kind integration test - add curl test, use dnsutils pod for dig, use ngnix app for deployment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #210 (2f090d4) into main (d6769d8) will decrease coverage by 1.26%.
The diff coverage is 50.66%.

@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
- Coverage   72.80%   71.53%   -1.27%
==========================================
  Files          15       16       +1
  Lines        1673     1746      +73
==========================================
+ Hits         1218     1249      +31
- Misses        365      410      +45
+ Partials       90       87       -3

Impacted Files
Coverage Δ

...ntrollers/multicluster/serviceexport_controller.go
38.96% <34.23%> (-2.15%)
:arrow_down:

pkg/cloudmap/client.go
72.56% <42.85%> (-1.46%)
:arrow_down:

pkg/model/types.go
65.24% <59.09%> (-0.51%)
:arrow_down:

pkg/model/cluster.go
60.00% <60.00%> (ø)

...kg/controllers/multicluster/cloudmap_controller.go
47.71% <66.66%> (+0.34%)
:arrow_up:

pkg/cloudmap/api.go
80.12% <70.00%> (-1.21%)
:arrow_down:

integration/janitor/janitor.go
87.14% <88.88%> (+1.20%)
:arrow_up:

integration/janitor/api.go
78.57% <100.00%> (ø)

pkg/controllers/multicluster/endpointslice_plan.go
95.95% <100.00%> (ø)

pkg/controllers/multicluster/utils.go
98.57% <100.00%> (+0.03%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6769d8...2f090d4. Read the comment docs.


Can't reproduce L1-score

Environment info
Operating System: Linux 64-bit
CPU/GPU model:
C++/Python/R version: 3.7.3
LightGBM version or commit hash: 2.2.1
Reproducible examples
When I run LGBM with early stopping, it gives me the scores corresponding to its best iteration.
When I try to reproduce these scores myself, I get different numbers.
import lightgbm as lgb
from sklearn.datasets import load_breast_cancer
import pandas as pd
from sklearn.metrics import mean_absolute_error
from sklearn.model_selection import KFold

data = load_breast_cancer()
X = pd.DataFrame(data.data)
y = pd.Series(data.target)

lgb_params =  {'boosting_type': 'dart', 'random_state': 42}

folds = KFold(5)

for train_idx, val_idx in folds.split(X):
    X_train, X_valid = X.iloc[train_idx], X.iloc[val_idx]
    y_train, y_valid = y.iloc[train_idx], y.iloc[val_idx]
    model = lgb.LGBMRegressor(**lgb_params, n_estimators=10000, n_jobs=-1)
    model.fit(X_train, y_train,
              eval_set=[(X_valid, y_valid)],
              eval_metric='mae', verbose=-1, early_stopping_rounds=200)
    y_pred_valid = model.predict(X_valid)
    print(mean_absolute_error(y_valid, y_pred_valid))

I was expecting that
valid_0's l1: 0.123608

would match my own calculation from mean_absolute_error, but it doesn't. Indeed, here is the top part of my output:
Training until validation scores don't improve for 200 rounds.
Early stopping, best iteration is:
[631]	valid_0's l2: 0.0515033	valid_0's l1: 0.123608
0.16287265537021847

dart cannot be used with early stopping.
refer to https://github.com/Microsoft/LightGBM/issues/1893


Unicode issue in petition summary

On home page, this shows up in hot petitions list, and we need to fix it there most immediately. But because the hot petitions list is dynamic, pointing to an example of a specific petition that demonstrates the issue on the sign page instead:
/sign/keep-hearst-elementary says â€œOUR YOUTH ARE OUR FUTUREâ€ and should instead say what https://petitions.moveon.org/sign/keep-hearst-elementary says, “OUR YOUTH ARE OUR FUTURE”.
This may end up being an issue on the API side, not sure.

This is indeed on the API side, so closing this issue to open another there.


SQLite automatically convert datetime data into UnixEpoch values

I have a table in MSACCESS which contain a DateTime column. when i export this table to a SQLite table, Dbeaver automatically convert the Datetime Data into Epoch values. Any way to control these conversion???

Technically SQLite doesn't support DateTime data type. SQLite driver converts all date/timestamp values into epoch time. It is the default behavior. As far as I know it is not possible to change it.
You can with with epoch time in SQLite as with dates, there are special conversion functions for that.
If you want to persist timestamps in form of strings then you need to convert datetime columns on MSAccess side (using some datetime conversion function). For example:
SELECT COL_DATETIME || '',COLUMN2,COLUMN3 FROM NEW_TABLE;

SELECT CStr(COL_DATETIME),COLUMN2,COLUMN3 FROM NEW_TABLE;


bug: App crashes right after launch

Plugin version: latest

After installing this plugin I now have an issue where the android app crashes right after launch. I can't debug because it won't show me any error. : S
Platform(s):

android
Pixel 4

Capacitor doctor:

💊   Capacitor Doctor  💊

Latest Dependencies:

  @capacitor/cli: 3.4.3
  @capacitor/core: 3.4.3
  @capacitor/android: 3.4.3
  @capacitor/ios: 3.4.3

Installed Dependencies:

  @capacitor/cli: 3.4.1
  @capacitor/core: 3.4.1
  @capacitor/android: 3.4.1
  @capacitor/ios: 3.4.1

[success] iOS looking great! 👌
[success] Android looking great! 👌

not sure if it's related to this or not:
https://github.com/robingenz/capacitor-plugin-demo/issues/727

now also tried

using android OPPO F1
updating capacitor libs to latest

   Capacitor Doctor  💊

Latest Dependencies:

  @capacitor/cli: 3.4.3
  @capacitor/core: 3.4.3
  @capacitor/android: 3.4.3
  @capacitor/ios: 3.4.3

Installed Dependencies:

  @capacitor/cli: 3.4.3
  @capacitor/ios: 3.4.3
  @capacitor/android: 3.4.3
  @capacitor/core: 3.4.3

[success] iOS looking great! 👌
[success] Android looking great! 👌

but the app still crashes upon launch : S

Hmmmm. maybe this plugin is not the cause, because now I still have it even after uninstalling @robingenz/capacitor-badge again.
Do you have any advice how to best debug this? 😭
https://user-images.githubusercontent.com/3253920/161440429-6eb88583-7b49-42ac-afb3-ccf644cbd960.mov

I found a crash log and it led me to firebase auth plugin, so will close this.


imprv: Use latest BuildKit feature and GHA for faster/lighter docker build

This PR adds the following improvements to Docker Build to Growi

Use GHA native cache in GHA including the build stage image layers (brings sugnificant build speed improvemetnt)

type=gha,mode=max

chmod and chown can be with COPY and it makes the image layer lighter

Planning future work

Add --link options to COPY ref. https://codezine.jp/article/detail/15745

I think you should add on.push.paths-ignore to reduce unnecessary CI runs 🤔
It doesn't make sense ro run node tests when only YAML and Dockerfiles are changed.

Error:  Missing Slack Incoming Webhooks URL.

🤔

Thank you👍


Namarupa/Wingpanel-Indicator-Ayatana

Something that I have always found lacking in elementaryOS - and by extension Pantheon - was Appindicators/Tray Icons
However, I haven't missed these because they look nice, or are a good solution, but rather because many applications relying on them have no better way to run in the background than this (Telegram, Nextcloud, Discord, I think even Slack, you name it)
Thus, I propose either including, or having a way to install Namarupa or Wingpanel-Indicator-Ayatana
They're practically identical, other than the fact that Namarupa puts them not directly on the panel, but rather in an indicator on the panel (a 3-dot indicator). This provides support for legacy tray icons for applications which need them, without looking out of place on the wingpanel, nor taking up too much space
Even though Namarupa has not been updated in a while, it still works just fine on my ElementaryOS installation (6.1), and Wingpanel-indicator-Ayatana also still works
(Note that Wingpanel-Indicator-Ayatana used to be the official implementation until 2017, and it has been community-maintained until 2021)

I'll dig into this again, but I seem to remember dropping it quite some time ago due to some missing dependencies in Fedora (since Wingpanel-Indicator-Ayatana used to be available in the elementary-staging repo).

Wingpanel-Indicator-Ayatana is now back once more in main and will be in the next release.

However, considering indicator-application is essentially abandoned, and Namarupa and Ayatana both don't seem to in active development, there's no real gurantee of how long it'll work for. But it seems to work properly for now!


Allow user-provided LU solver for radau and bdf solvers

The new ivp solvers in integrate look wonderful. However, it seems that the bottleneck of implicit methods would be the LU decomposition, that currently uses SPLU provided via scipy. In our experience, other sparse solvers, notably MUMPS may significantly faster (easily 2x). Therefore it might be advantageous to allow the user to provide pluggable LU solver.
Pinging @nmayorov as the author of the new ODE solvers.

Hey, Anton! Glad you liked it.
It sounds reasonable, I suppose it can be designed as additional argument lu=None which must have two methods factor and solve. But are you sure it is worth it to farther extend already long list of options for 2-x speed improvement? For example, using MUMPS --- can you call it from Python easily, how often it will be used? I mean scipy is not really for high performance computing (in my opinion), so maybe it's not worth it to consider such improvements, if they won't be used often.
If you think it is indeed an important improvement (and want to use it yourself perhaps), then send a pull request (it shouldn't take much time), then we'll ask someone else. I'm neutral on that.
Also please do report any issues or ideas for new solvers.


Article was written before server 2016 was released

Replaced the reference to "Server Technical Preview" with Windows Server 2016 and higher

@billmath Could you review this pull request? Thank you.

@daveba @billma Would you review this pull request? Thanks!


[Idea] Ability to increase the severity of specific CWEs to critical

Introduce the ability on the side of SecHub to increase or set the severity of specific CWEs to critical. For example, the ability to set the CWE-89 SQL Injection to critical.

We currently rely on the tool output

We currently rely on the tool output


Introduce a new regex for a special comment: howto?

I tried to introduce a special comment for CPP-directives like
#include "bla.lua"
in lua.tmLanguage.json
.vscode-oss/extensions/sumneko.lua-2.5.3/syntaxes
But now luck with
				{
						"begin": "^\\s*#[a-z]+",
						"beginCaptures": {
							"0": {
								"name": "punctuation.definition.comment.lua"
							}
						},
						"end": "\\n",
						"name": "comment.line.double-dash.lua"
					},

Maybe ralated too: https://github.com/sumneko/vscode-lua/issues/51

I can't understand what you want, please tell more details

In some lua-file, e.g.
#include "bla.lua"
#define VALUE 42
local function test()
#ifdef DEBUG
return 42;
#else
return 0;
#end
end

I want to use CPP-preprocessor directives like demonstrated above.
These should be

first not disturb the parsing of the lua code
be coloured in a special way.

I tried to achieve that with the above extension.

For 1. , please check https://github.com/sumneko/lua-language-server/wiki/Plugin

Ok, thanks.
The plugin gets found, but no diffed.lua is created.

{
    "Lua.misc.parameters": [
        "--develop=true"
      ],

    "Lua.diagnostics.disable": [
        "undefined-global"
    ],
    "Lua.completion.displayContext": 2,
    "files.associations": {
        "*.lpp": "lua",
        "*.lh": "lua",
        "makefile.include": "makefile"
    },
    "Lua.diagnostics.globals": [
        "__libI",
        "errorCode",
        "__libD",
        "__libP",
        "__stopWmSw2",
        "__libA"
    ],
    "Lua.workspace.library": ["lib"],
    "files.autoSave": "afterDelay",
    "Lua.completion.showWord": "Enable",
    "Lua.diagnostics.libraryFiles": "Enable",
    "Lua.hint.enable": true,
    "Lua.color.mode": "SemanticEnhanced",
    "Lua.runtime.plugin": "/home/lmeier/Projekte/lua/plugin.lua",
}

Ok, works.

What about 2) ?

You may try this: https://code.visualstudio.com/api/language-extensions/syntax-highlight-guide#injection-grammars

Wouldn't be sufficient to edit lua.tmLanguage.json?
Can you give me a hint to do that?

Wouldn't be sufficient to edit lua.tmLanguage.json? Can you give me a hint to do that?

It is not good to modify internal files directly, because you need to modify it again after the new version is released.
You could replace # to // by plugin, then add // to Lua.runtime.nonstandardSymbol and set Lua.color.mode to Semantic or SemanticEnhanced

It is not good to modify internal files directly, because you need to modify it again after the new version is released.

Yes, I would go an patch approach.

You could replace # to // by plugin, then add // to Lua.runtime.nonstandardSymbol and set Lua.color.mode to Semantic or SemanticEnhanced

Why replace by // instead of  -- as lua comment for the parser?

Color of -- is parsed by tmLanguage.json, so it can not handle plugin.
Color of // is parsed by semantic-tokens language feature, it can handle plugin.

Can I choose a speacial color for // then?

"editor.semanticTokenColorCustomizations": {
    "rules": {
        "comment": "#ff0000"
    }
}

Mmh, does not work:
diffed.lua:
//include "../../global.h"

//ifdef INCLUDE_ANIMATIONS
//endif

local function run(event)
lcd.clear();
local x = 20;
local y = 20;

lcd.drawText(x, y, "Press ENTER", MIDSIZE);

if (event == EVT_VIRTUAL_ENTER) then
return 1;
else
return 0;
end
end

return {
run = run
};

settings.json:
{
    "editor.semanticTokenColorCustomizations": {
        "rules": {
                "comment": "#FF0000"
        }
    },
    "Lua.misc.parameters": [
        "--develop=true"
      ],
    "Lua.diagnostics.disable": [
        "undefined-global"
    ],
    "Lua.completion.displayContext": 2,
    "files.associations": {
        "*.lpp": "lua",
        "*.lh": "lua",
        "makefile.include": "makefile"
    },
    "Lua.diagnostics.globals": [
        "__libI",
        "errorCode",
        "__libD",
        "__libP",
        "__stopWmSw2",
        "__libA"
    ],
    "Lua.workspace.library": ["lib"],
    "files.autoSave": "afterDelay",
    "Lua.completion.showWord": "Enable",
    "Lua.diagnostics.libraryFiles": "Enable",
    "Lua.hint.enable": true,
    "Lua.color.mode": "Semantic",
    "Lua.runtime.plugin": "/home/lmeier/Projekte/lua/plugin.lua",
    "Lua.runtime.nonstandardSymbol": ["//"]
}

Try to reopen/scoll this file, the client may not refresh at the first time

No luck

Try set editor.semanticHighlighting.enabled to true

As well no luck

I have no idea, but I can confirm that it works at my side

Can you share your example?

https://github.com/sumneko/sample-plugin-2

I'm using exaclty your example / plugin / setting ... no luck with color

Your example only works if main.lua has 5 lines. With 6 lines (blank) it does not work anymore

This is Linux, so LF only

Maybe you can only waiting for fixing this bug or editing lua.tmLanguage.json first.
Notice that the getlen symbol (print(#array)) has used #, you must inject before it.

Maybe you can only waiting for fixing this bug or editing lua.tmLanguage.json first. Notice that the getlen symbol (print(#array)) has used #, you must inject before it.

Well, I have no real idea how to do that.
I hope you can fix that soon


feat(expr): support sign and distinguish is_positive vs is_sign_positive

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Support PostgreSQL expression sign for decimal and double precision (Resolves #8900). Input numeral of other types would be implicitly casted to double precision as in PostgreSQL.
It does not map to rust signum directly:

value
sign
signum
is_positive
is_negative
is_sign_positive
is_sign_negative
num_traits is_positive
num_traits is_negative

-0.0
0
-1
x
x
false
true
false
true

0.0
0
1
x
x
true
false
true
false

0
0
0
false
false
x
x
false
false

Given that num_traits::Signed confuses between is_positive and is_sign_positive. We removed its usage.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[x] My PR contains user-facing changes.

Types of user-facing changes

SQL commands, functions, and operators

Release note
Support PostgreSQL expression sign for decimal and double precision.

abs('-0'::double precision) is supposed to remove the sign. Fixing.


[Feature/1538] Add 'diff_patch' notification body token

Add diff_patch notification body token, allowing the diff to be displayed in unified format. Based on the discussion from #1538.
Notes:

PR is being made against master, given that the dev branch is actually behind it.
Usual patch header with file/URL and date is not being added, given that this information is currently not present anyway.
Not sure if files like changedetectionio/tests/test_notification.py and changedetectionio/tests/test_group.py need to be changed, given that they have references for the other diff tokens (like diff_full). Tests haven't failed when running on GitHub Actions on my fork anyway.

Looks OK to me, thanks for adding to the tests also
Anyone else want to confirm this one?
final thoughts?

Thanks for the review.

final thoughts?

I'm just running the test suite after adding diff_patch to notification_form_data, like on test_notification.py. I'll update this PR accordingly if everything goes fine.

Done. Please let me know if there's anything missing.


how to give custom template in checkbox

where to write the custom html template in checkbox

how are you generating the checkbox?

@sanket0052, please use Official Yii forum to ask a questions - GitHub is a bug tracking system.


Can't sign an msig proposal with owner key

Describe the bug
A clear and concise description of what the bug is.
Telos Explorer will not allow me to sign an msig updateauth proposal with an owner key.   It defaults the key back to active which results in an assertion error.
To Reproduce
Steps to reproduce the behavior:

Go to https://explorer-test.telos.net/proposal/updateactivf
This transaction should be called updateowner,  its updates the owner keys of bennyfitokent1
Even thought I'm logged in as key1bennyfi1@owner,  when I go to execute the transactions, it defaults to key1bennyfi1@active
resulting in an assertion error.
I reported this to Anchor,  thinking it was their issue,  but here's their response.
"I spoke with our team regarding this and it turns out the issue resides with the block explorer. Regardless of the account/permission you sign in with to the block explorer, the explorer itself is forcing the msig “approval” to use the accounts active permission, so then when its routed to Anchor, it pops up the active permission.  You can checkout the code for the block explorer here - https://github.com/telosnetwork/open-block-explorer/blob/ac8ed50add1cf1557bce40b7596dd28d48c69539/src/pages/ProposalItem.vue#L355

Anyhow, this is not something that we can repair on our end, so i would suggest reaching out to the Telos support team for their block explorer and see what they recommend as a solution,  We'll be reaching out as well, but the more people bring up the issue, the more they are likely to fix it.
If there's anything else we can help you with, please let us know."
Expected behavior
A clear and concise description of what you expected to happen.
I log in as owner to approve a msig,  I need to be able to sign as owner.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]  windows
Browser [e.g. chrome, safari]  Brave
Version [e.g. 22]

Additional context
We're hardening our treasury security with Ledger wallets.   We need to be able to add and remove owner keys to our system accounts.   We are using a similar account structure to Telos's eosio account.   Can you suggest a work around for this that will allow us to still use HW wallets?  How do you handle this on the eosio account?

Any comment on this?

Any comment on this?

@cuvialabs Please see my comment on #753, I believe I've resolved the issue

Replied to #753,   same result.   Its defaulting to the active key,  when I'm logged in with the owner.

Sorry to be a pain on this,   but its blocking our workflow.    Any update?   Did you push the new version with the fix?


[Release 1.21] /var/lib/rancher/k3s/storage should not be world-readable

Backport changing local storage permissions from https://github.com/k3s-io/k3s/issues/2348

Validated on release-1.21 branch commit 267adf64dc96fe64e3b27149b1b0b69b4842ece9
Only changes were made to items within the storage directory.

/var/lib/rancher/k3s/storage directory has 0755 permissions
Items within storage directory correctly have 0700 permissions

$ ls -la /var/lib/rancher/k3s/storage/
total 16
drwxr-xr-x 4 root root 4096 Jul  9 22:10 .
drwxr-xr-x 6 root root 4096 Jul  9 21:57 ..
drwx------ 2 root root 4096 Jul  9 21:57 pvc-6401329b-e2f1-4c47-9e1d-cb86fe3abbd8_default_local-path-pvc
drwx------ 2 root root 4096 Jul  9 22:10 pvc-86c318f1-6c64-4908-9bc9-c5368fbca0e6_default_other-path-pvc

$ stat -c %a /var/lib/rancher/k3s/storage/
755

$ stat -c %a /var/lib/rancher/k3s/storage/pvc-6401329b-e2f1-4c47-9e1d-cb86fe3abbd8_default_local-path-pvc/
700

$ stat -c %a /var/lib/rancher/k3s/storage/pvc-86c318f1-6c64-4908-9bc9-c5368fbca0e6_default_other-path-pvc/
700


ReleaseSuite.TestReleaseImages: timed out waiting for new host

----------------------------------------------------------------------
FAIL: test_release.go:78: ReleaseSuite.TestReleaseImages

++ 18:05:30.211 adding 1 hosts
test_release.go:93:
    // boot a host to release components to a local blobstore, outputting the new version.json
    buildHost := s.addHost(t)
helper.go:192:
    t.Assert(err, c.IsNil)
... value *errors.errorString = &errors.errorString{s:"timed out waiting for new host"} ("timed out waiting for new host")

https://s3.amazonaws.com/flynn-ci-logs/20150226180025-2b79d3a6-build-a58fb7e32511670554f0f55eaad0e17b38477aca-2015-02-26-18-09-08.txt

(same ci job as #1096 and #1098)

https://ci.flynn.io/builds/20150228015315-d7e3b12c

This error hasn't occurred since 27687ef, closing.


Add SDL to migrations.

When doing migrations or migration rewrites, the stored schema::Migration objects will have their resulting SDL saved.
related #7621
Note: I made the sdl property required, however the issue mentions an upgrade path for older versions. Some feedback on how to reconcile that would be appreciated.

@msullivan In the end, figuring exactly how the migration was created is a bit too complex, so I've just left it as a always/never setting.


LineChart 1st data point does not show pointer label if initialSpacing={0}

Description
When I set initialSpacing={0} to LineChart, every other data point shows my pointerLabelComponent just fine, but not the first one.
I'm working around it by adding a width to yAxisLabelContainerStyle bigger than yAxisLabelWidth (to hide the initial spacing), shifting vertical lines, subtracting this offset from the chart width and rules length. It's a nightmare.
I wonder why I can't "hover" the first data point.
My code is now completely poisoned with my workaround, but I can try to clean it up and post it here if necessary.
Steps to reproduce

Create a LineChart with initialSpacing={0}
Set a custom pointerLabelComponent
Verify that you are unable to show the component on the 1st data point
Remove initialSpacing or set a high enough value and see that now everything works fine

Snack or a link to a repository
No response
version of react-native-gifted-charts
1.4.47
React Native version
0.76.5
Platforms
iOS
Workflow
Expo Go

Hi @pakerwreah Thanks for reporting this issue. This has been fixed from versions 1.4.49 onwards.
Please use the latest version of the library.


Add Cortex (CTXC)

http://www.cortexlabs.ai/
https://coinmarketcap.com/currencies/cortex/
We need help finding a vector version of their logo.

I have vectorized their logo: ctxc-logo.svg.zip


rbac: add cluster scoped CRD ClusterRbacConfig and deprecate RbacConfig.

Some notes for reviewers:

I changed the RbacConfig from cluster scope to namespace scope in pilot/pkg/model/config.go otherwise the test will fail due to duplicate message type: "istio.rbac.v1alpha1.RbacConfig". I tested this change by fresh install and upgrading Istio. This is okay as Pilot won't create a CRD if it finds it already exists in the k8s cluster.
I changed the sample config to use ClusterRbacConfig. This won't affect the current tasks on istio.io as the change is on master branch and the istio.io is using samples in release-1.0 branch.
@rshriram, @costinm: I removed a unit test (destination rule nil traffic policy) in route_test.go as it's depending on the outdated IstioConfigStore fake object. The DestinationRule() method has already been removed in #7635. Please let me know if you don't want to remove the unit test.

For more background, see #8825.

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

Needs approval from an approver in each of these files:

galley/OWNERS
install/OWNERS
pilot/OWNERS
samples/OWNERS
tests/e2e/tests/pilot/OWNERS

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

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
c577bbf48737de2e7c17457617585f4297cce7a3
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-simple

prow/istio-pilot-multicluster-e2e.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-v1alpha3.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test e2e-simple

prow/istio-pilot-multicluster-e2e.sh
b3aec245d10723b80e808843b23328d93afec31f
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
d1220000af80869bc2325b7665034538e0ed1f25
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
85ab5479e566e7a00eb4f39422813df0b5553a6a
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
85ab5479e566e7a00eb4f39422813df0b5553a6a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
85ab5479e566e7a00eb4f39422813df0b5553a6a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@diemtvu
I'm planning to either implement a tool in istioctl or put it in a simple bash script to help user migrate, any suggestions for this?

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
85ab5479e566e7a00eb4f39422813df0b5553a6a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-multicluster-e2e.sh
0ddac426aef91d053d1773e22866cb806a6689c9
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj I have changed to ClusterRbacConfig, PTAL, thank you.

What is the recommended name for Mesh level /cluster scoped object?  I think authentication policy has MeshPolicy. @ayj @diemtvu @geeknoid @ozevren

@yangminzhu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
0ddac426aef91d053d1773e22866cb806a6689c9
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth-mcp.sh
44da2802e8aebe30d64455fddf7a85bffa570f95
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

What is the recommended name for Mesh level /cluster scoped object? I think authentication policy has MeshPolicy.

We should work towards using Cluster for cluster-scoped resources. It's consistent with k8s naming conventions and accurately represents the scope of these resources. Multi-cluster (i.e. mesh) scoped resources is a layer on-top cluster scoped resources.  We'll need to encode this recommendation as part of Istio API governance / style.

@diemtvu
I'm planning to either implement a tool in istioctl or create a simple bash script to help user migrate, any suggestions for this?

sgtm

@rshriram @ayj @liminw  I have addressed all comments so far, PTAL. Thanks.


Is it .yml or .yaml?

I have noticed that the yaml files in this suite of projects have inconsistent extensions.
Sometimes it is .yaml and other times it is .yml
We need to pick one, so which is it?

I have done a few googles.
I am erring towards yaml becuase:

https://stackoverflow.com/a/21059164/8640984 - Currently, .yaml is the chosen answer on stackoverflow
The creators say use .yaml
Yaml is pronounced ya-ml
Laravel Homestead uses yaml file instead of yml. And I like Laravel, they seem to think hard about their shizzle.

But what does symfony say?
https://symfony.com/doc/current/components/yaml.html#writing-yaml-files
Even that used .yaml.

Ok. So, let's go through all files in the suite and make them consistent.

xml-authoring-project - changed and tagged new version
gsheet-to-xml - no change needed
xml-authoring-tools - no change needed
capture-lookups - changed and tagged new version
ping-drive - no change needed
xml-authoring-library - changed and tagged new version
xmlauthor-example-command  - changed and tagged new version

I have just deleted the screen-gsheet repo cos it never was built.

Your repository "forikal-uk/screen-gsheet" was successfully deleted.

I have ensured that all unit tests are passing on the projects I changed (except for xml-authoring-library which appeared to have a fail for some other reason - I added a comment about it).


⚠ Samsung has degraded performance

In b52d0bf, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2726 ms

Resolved: Samsung performance has improved in a97ae2e.


add device_type as an attribute for sensors and elements

This exposes the new device-type meta attribute as an attribute of Helium::Sensor and Helium::Element

Coverage remained the same at 100.0% when pulling f1b595df7607e4d2a66e13f8264893ea42900acb on feature/device-types into 3fa1d35ae3634a30aee927829f609d701ad68bb5 on master.

going to merge and release this


 Configured localnet for development purposes.
Could you help me with that?
Thanks

thanks for detailed answer!


Closed paren on README.md async/await section

Closed left parenthesis at line 189

This is not correct.


Missing FSS Callsigns in Activity bot.

Missing some FSS call signs that are missing

I have already worked on this and am including it in a bigger commit with some other changes for activity logging and reporting

Is a feature, not a bug.
On Sat, Feb 4, 2023 at 7:08 AM mikobeako @.***> wrote:

I have already worked on this and am including it in a bigger commit with
some other changes for activity logging and reporting
—
Reply to this email directly, view it on GitHub
https://github.com/vancouverfir/czvr-core/issues/5#issuecomment-1416777342,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWX5VN5LHSG7MOX6D5WT6TWVZWGJANCNFSM6AAAAAAURGX4GE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Issue resolved


WARNING!! The "modules" key in the @Module() decorator is deprecated and will be removed within next major release. Use the "imports" key instead.

Issue type:
[ ] question
[x] bug report
[ ] feature request
[ ] documentation issue
nestjs-config version
@latest
@nestjs/common+core or other package versions

@nestjs/common: 5.7.3
@nestjs/core: 5.7.3

Excepted behavior
No warnings
Actual behavior or outcome (for issue)
https://github.com/nestjs/nest/blob/master/packages/common/decorators/modules/module.decorator.ts#L39
WARNING!!
The "modules" key in the @Module() decorator is deprecated and will be removed within next major release. Use the "imports" key instead.

this is related to the upcoming nestjs 6.0.0 version..
See the "new" decorator here (currently rc1 state):
https://github.com/nestjs/nest/blob/00ddc0aeeb2b7c75203fa53826b1e3c86f3c0f00/packages/common/decorators/modules/module.decorator.ts#L5-L10
However, the official docs for the current (5.x) version of nestjs ( https://docs.nestjs.com/modules ) does not indicate to use the modules key.. it suggest to use imports
All the best

This doesn't appear to be an issue eith nestjs-config. Try searching your code base for where you might've used the modules key.
I'm going to close this however if you find it is the nestjs-config modules please provide code examples and I'll look into it :)


Add missing 'else' block in iob_cache_control

Add missing else block after the end of the if (USE_CTRL_CNT) begin block.
Without this, quartus would throw an error about multiple writes to  the rdata signal.
Fixed tab identation for ease of readability of Verilog blocks.

Why wqs this not happening on the master branch?

Why wqs this not happening on the master branch?
The master branch is correct. It has the 'else' block.


Report type when erroring on unsupported-type

When logging the value, it's not always clear the exact type that's causing failure (ArrayList vs vec for example). Adding the type makes this error message a little more informative.

Codecov Report

Merging #15 into master will decrease coverage by 1.8%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage     100%   98.19%   -1.81%
==========================================
  Files          13       13
  Lines         811      831      +20
  Branches        0        4       +4
==========================================
+ Hits          811      816       +5
- Misses          0       11      +11
- Partials        0        4       +4

Impacted Files
Coverage Δ

src/clj_cbor/codec.clj
100% <100%> (ø)
:arrow_up:

src/clj_cbor/data/simple.clj
28.57% <0%> (-71.43%)
:arrow_down:

src/clj_cbor/data/tagged.clj
50% <0%> (-50%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 875784b...b1af7fc. Read the comment docs.


Elaborate on metadata references

Right now we just add a sentence at the end of the description specifying how the fact may be looked up in the references. I regret this; we would have more flexibility in how this information is presented if we had put it in the metadata. (Probably the job of a student worker...)

Closed as stale


after uninstall calico node alreday trigger condition by calico

kubectl get node k8s-node-5
conditions:

lastHeartbeatTime: "2022-12-20T02:19:40Z"
lastTransitionTime: "2022-12-20T02:19:40Z"
message: Calico is shutting down on this node
reason: CalicoIsDown
status: "True"

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Calico version
Orchestrator version (e.g. kubernetes, mesos, rkt):
Operating System and version:
Link to your project (optional):

I'm sorry, but we can't action this issue without more information. Please fill out the entire issue template.

use kubectl patch remove this condition. it is works


Case insensitive chain name

Description

This PR makes the name of ChainConfig case insensitive.
Closes #526
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[ ] Wrote integration tests (simulation & CLI).
[ ] Updated the documentation.
[ ] Added an entry to the CHANGELOG.md file.
[ ] Re-reviewed Files changed in the Github PR explorer.

@RiccardoM why backslash? I don't think we will see chains where name differs only for some case letter.


CARRY: Configure Kueue Pod webhooks so they do not interfere with infra Pods

Issue Link
fixes https://issues.redhat.com/browse/RHOAIENG-2123
What this PR does / why we need it:
This pr updates to the Kueue deployment, specifically targeting the validation and mutating admission webhooks. By default, Kueue's webhooks operate on Pod resources, which can inadvertently affect critical infrastructure components, such as CNI plugins. To prevent potential interference the pr adds a patch,removing the webhooks.
This pr also re-adds the original patches in config/components/webhook/kustomizaiton.yaml to avoid future rebase conflicts from the kueue repository.
Adding a standard patch with and empty webhook field [], would not suffice, and therefor  $patch: delete was used.
Testing the changes:
Without the patch applied:
Ensure you deployment directory in the makefile is set to config/rhoai
run make manifests and  IMAGE_REGISTRY=<your-image-registry> make image-local-push deploy.
Here, some operations will present the interference, such as running 'kubectl apply -f examples/admin/single-clusterqueue-setup.yaml' will result in the following:
localqueue.kueue.x-k8s.io/user-queue created
Error from server (InternalError): error when creating "examples/admin/single-clusterqueue-setup.yaml": Internal error occurred: failed calling webhook "mresourceflavor.kb.io": failed to call webhook: Post "https://kueue-webhook-service.kueue-system.svc:443/mutate-kueue-x-k8s-io-v1beta1-resourceflavor?timeout=10s": no endpoints available for service "kueue-webhook-service"
Error from server (InternalError): error when creating "examples/admin/single-clusterqueue-setup.yaml": Internal error occurred: failed calling webhook "mclusterqueue.kb.io": failed to call webhook: Post "https://kueue-webhook-service.kueue-system.svc:443/mutate-kueue-x-k8s-io-v1beta1-clusterqueue?timeout=10s": no endpoints available for service "kueue-webhook-service"

Notice kueue-validating-webhook-configuraiton will have 16 webhooks, and `kueue-mutating-webhook-configuration will have 14.
Now reapply the patches, and follow the same steps ensuring conflicts no longer arise.
Check the kueue-validating-webhook-configuration and kueue-mutating-webhook-configuration and ensure they contain the updated configuration, without the  vpod.kb.io and mpod.kb.io webhooks.
kueue-validating-webhook-configuraiton will now have 15 webhooks whilst `kueue-mutating-webhook-configuration will have 13.

/lgtm

/approve
Thanks!


Fix conversion of ListObjectsV2Output::key_count to response

I discovered today that changing key_count has no effect on response and traced it to this code. I didn't find any tests to update.

Note that s3-server project is now unmaintained due to legacy design issues. There will be no more releases. https://github.com/datenlord/s3-server/issues/108#issuecomment-13849072
The maintained alternative is https://github.com/Nugine/s3s.

Thanks for notification, i'll switch. Do you want me to mirror the PR in that repo?

s3s uses a custom code generator. This bug does not exist there.


Upgrade to broadcast-channel 4.0

Fixes https://github.com/SvelteStack/svelte-query/issues/45. I hope. Note that I've never used svelte-query and haven't tested this. That being said, I got some changes into broadcast-channel that I believe should make it compatible with SvelteKit. You might have to define process.browser if Vite doesn't, but besides that I hope it should just work

Thanks for working on this Ben! I'm still seeing problems during Vite SSR phase here, where it tries to load node.js which uses require.
More details below, but I've also created a minimal reproduction here: https://github.com/dimfeld/sveltekit-with-svelte-query. This repo is set up with pnpm and so you can override the version of broadcast-channel used in the pnpm.overrides section of package.json. Hope it helps!
So first I see this error:
ReferenceError: require is not defined
    at /node_modules/.pnpm/broadcast-channel@4.0.0/node_modules/broadcast-channel/src/methods/node.js:6:14
    at instantiateModule (/home/dimfeld/projects/vite/packages/vite/dist/node/chunks/dep-ea6d097e.js:73433:166)

Notably, this is the version in the src directory. I see a comment in the source that it's faster to run the non-transpiled version here, but it does seem to cause issues. If I change the import to the transpiled version (./methods/node.js) then it gets past that, but actually runs into issues with in babel runtime's async transformation code.
ReferenceError: module is not defined
    at /node_modules/.pnpm/@babel+runtime@7.14.8/node_modules/@babel/runtime/helpers/asyncToGenerator.js?v=710ddb8b:35:1
    at instantiateModule (/home/dimfeld/projects/vite/packages/vite/dist/node/chunks/dep-ea6d097e.js:73433:166)

Thanks for the reproduction. That does help. I'll close this for now

I've never used svelte-query

@benmccann why? Can I ask you what are you using right now? Will you ever use it in the future?

It's nothing against svelte-query. There's just more libraries out there than I've had a chance to use :smile: I may use it in the future

Yes, that's what I meant. I just don't know them. Can you tell me what they are?


Why spaces instead of tabs?

Sorry, I know this is not an "issue", but I wonder why the "standard" should force me to use spaces instead of tabs for indentation.
When using spaces, I'm forcing every one else to see my code as I like. In certain monitors/resolutions 2 spaces could be a bad choice.
By using tabs, anyone can configure his text editor to display tabs with the desired visual width (for example, 2 or 4 spaces), and we all are happy.
I've been never told about a real advantage of using spaces instead of tabs.

Duplicate of #58

#58 is not just closed, but also locked (after just 6 comments). So no debate, right?
It is also so funny how #58 ends:

If you care a lot about tabs vs spaces, standard isn't for you.

I should not care about tabs vs spaces because you authors already did for me, right?
IMHO you should have chosen any other name for this library. This is not the "standard" of anything but the way you authors code, nothing else.

I'll just leave this here: https://www.youtube.com/watch?v=SsoOG6ZeyUI
You're right that tabs have some benefits. Spaces have some benefits too. The main issue with this topic (and why the previous issue was locked) is that this is not something that the maintainers of standard have any interest in debating. There are plenty of blog posts about why spaces are better than tabs if you want to go hunting for reasons.
You have the whole rest of the internet to debate the merits of spaces vs. tabs. This is not the place, sorry.

I already knew that video, so nice :)
OK, I just assume standardjs is not for me, and I respect your decision. Just:

Spaces have some benefits too

That's the only rationale I've found regarding spaces benefits: none (or undefined).

The reason that standard prefers spaces is simple: The most popular way of indenting JavaScript is with spaces. There are people who have written about why spaces may be better, if you're interested: http://paul-m-jones.com/archives/2312

The two arguments in favor of styles in that post are mainly wrong:
Inter-Line Alignment
Even when I use TAB for indentation, I do NOT use TAB for "inter-line alignment" but spaces. That's not "indentation" but "text decoration".
Line Length Recognition
Well, a problem if (as the post states) someone sets a virtual 8-space!! width for TAB...

As mentioned before, I'm not interested in debating spaces vs. tabs.


[WIP] Use Plain XHR instead of jQuery to avoid multiple request to endpoints

Refers #8923 and #9196

What's the status of this PR?

What if we use the fetch API and its polyfills for older browsers?

@bugron I seem to have problem reproducing this and the issue is highly weird. Could any of the @FreeCodeCamp/issue-moderators pitch in and test this PR as it is with the email verification scenario.
If it looks okay, then I'll clean up the fix, add it to night mode and update.
@alayek yeah we could, but I think @BerkeleyTrue and myself felt making a vanilla implement to be most clean way to address this, because even a solid library like jQuery seems to act weird over here.

@raisedadead I've already updated the ajax requests to use XHR on the react branch. I have not run into the issue again since.

@BerkeleyTrue in that case closing this in favor of react infrastructure. And awesome job resolving it!

Beta is using the the new ajax to update email. I still haven't run into the same issue

Cool. Could you try adding the verification prompt like we did and maybe I'll give it test to double check?


Flow fails to catch errors with bottomless types

Here is an example
/* @flow */

type Never =
  | Never

class Process <exit, message> {
  send(payload:message) {
  }
}

declare export function spawn <exit> ():Process<exit, Never>

spawn().send(5)

I would expect flow to error with message like:
15: spawn().send(5)
                 ^ number. This type is incompatible with
12: declare export function spawn <exit> ():Process<exit, Never>
                                                          ^ Never

But instead it reports no errors.

I probably define bottomless types incorrectly here, but regardless flow should definitely be failing here.

Seems like what you want is similar to Empty from this issue #2225
You can use null & void as Never and it should work.

We now have empty. We recommend that you use that 😊


useCalls randomly got left out

Describe the bug
Our project has a custom hooks that will go to different chain id and fetch the data to the frontend. We are using useCalls for each chainid.

The issue is sometimes, the useCalls will be left out for specific chain, and in usedapp dev tool, it shows it is not even got called:
correct numbers of call:

incorrect numbers of call:

In this case, one of three chains information is not getting called on.
those calls are never got executed(we have set refresh to 'never').
It happens randomly.
Does anyone knows why that is happening and how to fix it?
Software versions

useDapp version: 1.0.10

Hey @JasonHe-da , does the issue still persist in the latest version of useDApp (1.0.12)

@yivlad it is still not working: it turns out to be one of three useCalls return undefined for all the data randomly and not even trigger calls in the usedapp dev tool, but if we use useCalls for only one chain, it works just fine

Please read our docs:
https://usedapp-docs.netlify.app/docs/Guides/Transactions/Contract Functions

@JasonHe-da can you show us your config?

@mj426382 @yivlad

@JasonHe-da I have been trying to reproduce what you were talking about, but unfortunately with no luck. Can you provide minimal reproducible example or some other guidance how to reproduce the bug? (I'm assuming that in config you showed us you deleted readOnlyUrls and mutlicallAddresses)


Removing default value for skipUpgrade to fix flux errors

Description of changes:
Removing defaults for skipUpgrade to handle unpredictable behavior from Flux controller.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/cherrypick release-0.15


Update README.md

Adding a few more Repo's

Not going to add the WiFi grabber since it would fit better in a Duckyscript repo than here. I link hak5's repo because it's a good starting point. As for Amibo's, I'll probably merge that as-is.

Reopening so I can mark this as merged.


Netlify bug: Page preview gets undefined first jumplink

go to https://cfpb.github.io/design-system/components/expandables
Edit the page
Note in the page preview that the first item in the jump links is undefined. This doesn't appear in the published page, however.

On hold till we move to Decap

https://github.com/cfpb/design-system/pull/1980 has been merged.


SpookyJS still under active development?

I get an error just running spooky.start(); -- undefined function...

Does everything spooky does have to be in the new Spooky object?

yes and no. You need to have a spooky onstance to work with but you can pass result via the emit function outside the spooky object and process the results there.

@solick cool.  Can you give me an example on how to do that?

Because I copy and pasted my whole CasperJS script into new Spooky();.

Have you tried the example provided in the repository (file hello.js)? the spooky.on functions allow you to deal with results outside the new spooky object function.

Ok -- but you can't use spooky.start() outside of new Spook();?

Because I copy and pasted my whole CasperJS script into new Spooky();.

That definitely won't work. Please take a look at the example script.

@lawnsea the example script really doesn't have much functionality though... It's basically just getting a page title...

You cannot coppy-paste your casperjs script because spookyjs does not support all functions of casperjs. As pedro93 said, you have to emit inside your spooky instance and pass variables / objects through the event which can than be catched and processed outside.

@sawteeth a basic example could look like this:
var spooky = new spooky({...}, function(err) {
     ...
    /* do something */
    spooky.run(function() {
       this.emit("finalResult", yourFinalResult);
 });
});

spooky.on("finalResult", function(yourFinalResult) {
    /* do something with your final result */
});

@solick thanks.  Are there docs that flesh out the available methods?

Not a direct document but you can get all provided document out of this file: https://github.com/SpookyJS/SpookyJS/blob/master/lib/bootstrap/casper.js we are evluation at the moment which functions we could additionally provide and also work in the documentation.


Update scientific names to their accepted name?

The following taxa are considered synonyms by GBIF. For each: ask Hugo if it's OK to a) update in the xls our scientific name to the accepted species or b) keep our current species name

[ ] Aspius aspius (https://www.gbif.org/species/2360181) is considered a synonym of Leuciscus aspius (https://www.gbif.org/species/5851603)
[ ] Acipenser guldenstaedti... actually written as Acipenser güldenstaedti (https://www.gbif.org/species/2402131) is considered a synonym of Acipenser gueldenstaedtii (https://www.gbif.org/species/2402129)

I asked Hugo, he agrees to update the scientific names. Will adapt this asap.

These two synonyms are now handled in the verification of the unified checklist (accepting the proposed synonymy by GBIF). There is no longer a need to update these in this checklist. In fact, it might be better to keep the original names. Closing issue.


allow provider to update the list when search text changes

Is there a sample provider that updates the list when on_type changes?
In the below example, as use searches for npm packages I run curl after some debounce. I'm currently using (quickpick.vim](https://github.com/prabirshrestha/quickpick.vim) and I would like to know if vim-clap supports this scenario or if it is possible to add this scenario.

I'm mainly trying to expose api's vim-lsp for custom provider specially :LspWorkspaceSymbol. This way as the user is searching I can make requests to language server protocol and then update the results.
Some of my thoughts can be found here:
https://github.com/mattn/vim-fz/issues/4
https://github.com/mattn/gof/issues/18

It's not hard to add an API for updating the source list if the source of provider returns a List instead of String. l:raw_lines is what we need to set dynamically if you are interested.
https://github.com/liuchengxu/vim-clap/blob/41ebbfde1f6775ab4b4dfb9154cccec015dc0d20/autoload/clap/impl.vim#L87
But I'm still unsure about the interaction when the source List can be obtained in async. If this operation takes some time, when you enter the clap window, you start the async job but there is no content yet, what you typed right now does nothing until the async data comes.

Here is an example using quickpick: https://github.com/prabirshrestha/quickpick-npm.vim/blob/4f4d913a103f84b8844ef59bd06afd52e13fc07b/autoload/quickpick/pickers/npm/search_packages.vim
source list can change any time. Which means as a provider author I need to be able to reset the list, set the busy status, update the list and debounce as needed.
At high level I would like to achieve something like this. Think of like creating an autocomplete search results. Clap should care if it is async job or some other api.
function! s:on_type(query) abort
    if exists('s:timer')
        call timer_stop(s:timer)
        unlet s:timer
    endif
    let s:timer = timer_start(250, {->s:search(query)})
endfunction

function! s:search(query) abort
    call clap#set_busy(1)
    if exists('s:last_req_id')
        call lsp#cancel_request(s:last_req_id)
    endif
    let s:last_req_id = lsp#send_request({
        \ 'on_notification': {result->s:on_result(result)}
        \ })
endfunction

function! s:on_result(result, id) abort
    unlet s:last_req_id
    call clap#set_busy(0)
    call clap#set_items(a:result)
endfunction

Actually a provider author is able to this right now, all basic APIs are avaliable, see :Clap grep, the data is fetched in async. But that has to be a pure-async provider, which means the provider author must provide the on_typed implementation by himself.

Use g:clap.display related API to manipulate the display window, e.g., g:clap.display.set_lines() can be used for setting the source list at any time, see the complete list https://github.com/liuchengxu/vim-clap/blob/master/autoload/clap/api.vim#L69 . Closing as it's already supported, let me know if you have more questions.


Make Postgres surfacer writeChan configurable

In some cases it is easy to hit 1000 limit. This makes this limit configurable.
Also increasing to 10k which is consitent to Prometheus surfacer.

Thanks, I've started the process of importing it as per https://github.com/google/cloudprober/blob/master/CONTRIBUTING.md#source-of-truth-sot-and-commit-process.

Committed (https://github.com/google/cloudprober/commit/3d5080b5dd0ee6a23395e6cf42a24c3e10557c2d) through #509.
Thank you once again for the contribution, @benner. Much appreciated.


access to accelerometer X, Y, Z

to make "level" program

This is probably beyond the level of math of our target audience?

Maybe


Update Composer dependencies (2023-04-24-00-08)

Visual regression test passed!
View the visual regression test report


fragments in select

This query doesn't seem to work with SQLite3 https://github.com/elixir-sqlite/ecto_sqlite3/blob/6dcc0b2410ac2e2727d6c81ecb2e96c410eb8be4/test/ecto/adapters/sqlite3/connection_test.exs#L139-L144
sqlite> create table example(a integer);

sqlite> select e0 from "example" as e0;
Parse error: no such column: e0
  select e0 from "example" as e0;
         ^--- error here

I wonder if there is something we can do about it. Either raise or add .*.

Postgrex has a similar test case
https://github.com/elixir-ecto/ecto_sql/blob/c7d58a3f963cc605ef73c4eec2c03fc46c6db804/test/ecto/adapters/postgres_test.exs#L87-L88
I'll check if that query works with postgres and if not, I'll reopen this issue on ecto_sql.

Oh that's confusing. We should do what is right and just raise here honestly. Doing a naked select like that and appending .* seems risky to me. Thoughts?


Implement a Kitura-Credentials plugin for Google OAuth tokens

https://github.com/IBM-Swift/Kitura-CredentialsGoogleToken/commit/92ebdf62251f3d0f747c8328e8c416f152b0fbd3

Validated. Tested using Kitura-Bluepic from master as part of Issue #324


Adding instructions for linux

ZD: https://t3n.zendesk.com/agent/tickets/1672146

https://t3n.zendesk.com/agent/tickets/1672146


Checklist for going into August

Actions needed to get everything billing for August:

Deploy Alway-on stack ✅
Deploy Time-bound stack ✅
Size ec2 and rds instances (t2.small) in always-on and time-bound ✅
Re-run always-on Lambda builds and reinstate Code.fromBucket(...) ✅

Monitor costs daily at first! Then weekly and we may need to pull the plug if it's costing too much.

Checklist completed


Cards : Add link field

Could you add a field "lien" in cards please ?
This field should be not required.

Fixed in v1.0.0 (yet to be released)


Hi, is there an Export to PDF soon ?

thanks

@Jemison1 No. But any PR is welcome. This feature is minor to me currently.

Im having the same question about this. Its will be amazing!

@AllenFang I would like to work on this issue if not picked up yet, thanks. Is this still open?


Not possible to see the code from a runtime packages even "includeSourceInSymbolFile": true

1 .Describe the bug
When debugging OnPrem Customers is not possible to see code from runtime packages even "includeSourceInSymbolFile": true was used in function Get-BcContainerAppRuntimePackage.
2. To Reproduce
We are using the next configuration for resourceExposurePolicy
"resourceExposurePolicy": {
"allowDebugging": true,
"allowDownloadingSource": true,
"includeSourceInSymbolFile": true
}
I used this command to get the runtime package
Get-BcContainerAppRuntimePackage -containerName $ContainerName -appName $AppName -appFile "$Destination$PackageName" -IncludeSourceInPackageFile:$true
3. Expected behavior
Code should be included (exposed) in the symbols of a downloaded runtime package when the package was created with -IncludeSourceInPackageFile $true parameter
4. Actual behavior
After the runtime package is published, when downloading symbols the objects are missing the code, not possible to add brake points, see the code from functions in debug mode , etc.
5. Versions:
•	AL Language: v13.1.1065068
•	Business Central application:  "<IP_ADDRESS>",
•	List of Visual Studio Code extensions that you have installed:
AL Language

Hi @cosminahristofor,
Thanks for reporting the issue.
I just tried your code from the management client, and it works as expected, meaning it includes the code.
This suggests to me that the issue is with the BCContainerHelper, so I'm going to transfer it.