[Table of contents]: Mobile highlight for anchor link

Description
On Mobile Safari, when clicking a selection from the ToC dropdown, the user is sent to the selected header. However, the area covered by the highlight is awkward, and covers the preceding content.

Component(s) impacted
Table of contents
Browser
Safari
Carbon for IBM.com version
v1.31.0-rc.1
Severity
Severity 4 = The problem is not visible to or noticeable to an average user. Affects minor functionality, no workaround needed.
Application/website
C4IBM.com
Package
@carbon/ibmdotcom-react
CodeSandbox example
https://codesandbox.io/s/github/carbon-design-system/carbon-for-ibm-dotcom/tree/main/packages/web-components/examples/codesandbox/components-react/table-of-contents
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Duplicate! Fixed in #8659


Add aggregate function to library API

This change will provide access to the aggregate operation through the library API.
This will allow aggregation of FHIR datasets using FHIRPath from Python, Java and Scala.

Fixed in #1309.


Change over to quality to determine picked "default" content types

In case of ties, tiebreaker is order of appearance in header, first wins

The main UX issue here is that the default quality is 1 and the range is 0 to 1, so making a particular content type a default involves modifying all the others. That said... this isn't a super common use case.


Lines display strangely

After I publish strip lines to rviz, the lines sometimes are blurry until the cursor slide over them. Is that a bug of rendering? There is a video in the zip. It's obviously at the beginning and end.
Looking forward to any replies. Thanks a lot.
slide over.zip

Could you provide a bag file and rviz config to easily reproduce this issue?
I never observed something like this.

I got the sample problem.
Please unzip the attached file and run the following commands:
rosbag play rviz_issue_1287.bag -k
rviz -d rviz_issue_1287.rviz

rviz_issue_1287.zip
LINE_STRIP marker is published and displayed in Rviz. In the "Move Camera" mode in Rviz, the marker disappears when changing the viewpoint. In the "Interact" mode, the marker disappears but it reappear if the mouse cursor goes close to the marker.

My environment: Ubuntu16.04.1 / ROS Kinetic. Rviz is built from source and its version is 1.13.12. I also install ogre from source to fix this issue and its version is 1.9.1.

Thanks for providing the bag file. I cannot reproduce this issue with a recent rviz (latest Melodic release).
Could you try to build the latest melodic-devel release and check with it?
Note, that you also need to build the melodic-devel branches of TF1 and TF2 within your workspace to do so.

I tried with the melodic-devel (the following catkin workspace) and the same problem still happens..
$ wstool info -t src
workspace: /home/mmurooka/ros/ws_rviz_melodic/src

 Localname S SCM Version (Spec) UID  (Spec)  URI  (Spec) [http(s)://...]
 --------- - --- -------------- -----------  ---------------------------
 rviz        git melodic-devel  2b31dc4be241 github.com/ros-visualization/rviz.git
 geometry2   git melodic-devel  9b8022c30304 github.com/ros/geometry2.git
 geometry    git melodic-devel  0712a2cb808c github.com/ros/geometry.git

Ok. Need to find a machine with older Xenial + Kinetic to test this tomorrow. But, I'm afraid the problem is related to your OpenGL lib. Could you please provide info on that (e.g. from rviz cmdline output).

provide info on that (e.g. from rviz cmdline output).

$ rviz
[ INFO] [1597888033.177879502]: rviz version 1.13.13
[ INFO] [1597888033.177934022]: compiled against Qt version 5.5.1
[ INFO] [1597888033.177951527]: compiled against OGRE version 1.9.0 (Ghadamon)
[ INFO] [1597888033.183421668]: Forcing OpenGl version 0.
[ INFO] [1597888033.901239841]: Stereo is NOT SUPPORTED
[ INFO] [1597888033.901307701]: OpenGl version: 4.6 (GLSL 4.6).

I still cannot reproduce the issue, even on a Xenial/Kinetic setup. I guess, we have to blame a different version of OpenGL libs.
See also https://github.com/ros-visualization/rviz/issues/1508 and try rviz --opengl 300.

I tried --opengl 300 and --opengl 210, and the issue was reproduced in all cases..

Sorry, then I don't have any more ideas.

Any progress on this? or workarounds? I have a simple visualization with LINE_LIST Markers and I can only see the lines in Rviz from some viewpoints. From other viewpoints the lines are not rendered, despite they should be visible. I'm using Noetic.

I wasn't able to reproduce the effect so far. I you can in Noetic, please provide a yaml or rosbag with a corresponding marker message to reproduce the issue.

Another weird effect I've just realised, This is another drawing with multiple lines.  The lines don't render from certain view angles, but if I pass the mouse pointer over the unrendered lines while having the "interact" or "focus camera" tools the l lines are then redrawn and I can see them. This doesn't happen with "move camera" or "select" tools, ie. even if I move the cursor over the unrendered lines the line don't get drawn. This suggest that it may be an issue with the drawing event logic.

I see the LINE_LIST markers are drawn as flat tapes!! for a moment I thought that was because I was only setting scale.x and ignoring scale.y and scale.z. But after setting these nothing changed. In fact in the documentation says: "Line lists also have some special handling for scale: only scale.x is used and it controls the width of the line segments.". With width here I interpreted thickness of the line (as if the line was a cylindrical tube), but it's literally width of the tape, with thickness being apparently zero. I think the disappearing lines issue is because the visibility of the tape is fine when looking orthogonal to the width of the tape but when looking from the side the tape has zero thickness and it disappears.
Could the LINE_LIST be reimplemented as a cylindrical tube rather than a flat (zero thickness) tape?

Maybe related #987

I have the same issue. on some view angles the line is not rendered.

I am able to reproduce the issue. All line strips essentially employ Ogre::BillboardChain to show a line that should always face the camera. However, this mechanism doesn't work for some reason. When moving the camera, the new camera pose isn't considered when drawing the BillboardChain.
In the Interact mode the camera pose is updated (and the line correctly displayed again) as soon as the mouse gets close to the line strip (into the bounding box?). This is related to the SelectionManager rendering the scene into a RenderTexture:
https://github.com/ros-visualization/rviz/blob/d8df909abe87899f66d1bf9471cfeb27395c4186/src/rviz/selection/selection_manager.cpp#L749

The bug stems from OGRE 1.9. Updating to OGRE 1.12 resolves the issue.

Trying to reproduce the issue once again, I first failed. Looks like the issue only occurs if the marker is not published regularly.
Or, phrased the other way around, a regular message updates the billboard line.

This issue was fixed in OGRE 13.4.1: https://github.com/OGRECave/ogre/pull/2493
It won't fix for ROS1 anymore, as we are stuck with OGRE 1.9.0.
One can work around the issue by regularly updating the line strip marker.

experiencing this issue as well with ros noetic


[rcore] No suitable DRM connector found on Raspberry Pi with composite output

Issue description
When using composite output only on Raspberry Pi Zero W, when trying to initialise raylib the output is as follows:
INFO: DISPLAY: No graphic card set, trying platform-gpu-card
INFO: DISPLAY: Failed to open platform-gpu-card, trying card1
INFO: DISPLAY: Failed to open graphic card1, trying card0
TRACE: DISPLAY: Connectors found: 1
TRACE: DISPLAY: Connector index 0
TRACE: DISPLAY: Connector modes detected: 4
TRACE: DISPLAY: DRM mode NOT connected (deleting)
WARNING: DISPLAY: No suitable DRM connector found
WARNING: TEXTURE: Failed to load texture
WARNING: TEXTURE: Failed to load default texture

Environment
Raylib 5.0 compiled from source on Raspberry Pi Zero w Linux 6.6.31+rpt-rpi-v6 #1 Raspbian 1:6.6.31-1+rpt1 (2024-05-29) armv6l GNU/Linux
Code Example
Used core_basic_window.c from examples.
Investigation
The issue seems to be related to the code in rcore_drm.c
if ((con->connection == DRM_MODE_CONNECTED) && (con->encoder_id))
        {
            TRACELOG(LOG_TRACE, "DISPLAY: DRM mode connected");
            platform.connector = con;
            break;
        }
        else
        {
            TRACELOG(LOG_TRACE, "DISPLAY: DRM mode NOT connected (deleting)");
            drmModeFreeConnector(con);
        }

because in my case the connector status is unknown as can bee seen on modetest -c output:
trying to open device 'vc4'...done
Connectors:
id	encoder	status		name		size (mm)	modes	encoders
45	44	unknown	Composite-1    	0x0		4	44
  modes:
	index name refresh (Hz) hdisp hss hse htot vdisp vss vse vtot
  #0 720x480i 29.97 720 736 800 858 480 486<PHONE_NUMBER>0 flags: nhsync, nvsync, interlace; type: preferred, driver
  #1 720x576i 25.00 720 732 796 864 576 581<PHONE_NUMBER>0 flags: nhsync, nvsync, interlace; type: driver
  #2 720x288 50.08 720 740 804 864 288 290<PHONE_NUMBER>0 flags: ; type: driver
  #3 720x240 60.05 720 734 798 858 240 243<PHONE_NUMBER>0 flags: ; type: driver
  props:

Dirty solution
While this seems like the OS / hardware? issue which reports unknown status of the connector (or maybe it's not supported at all in certain hardware configurations), I was wondering if this could be a valid case to inlcude as an option in the library, to overcome this limitation.
Doing a cheap and dirty fix like this
        if (
            ((con->connection == DRM_MODE_CONNECTED) || (con->connection == DRM_MODE_UNKNOWNCONNECTION))
            && (con->encoder_id)
           )
        {
            TRACELOG(LOG_TRACE, "DISPLAY: DRM mode connected");
            platform.connector = con;
            break;
        }
        else
        {
            TRACELOG(LOG_TRACE, "DISPLAY: DRM mode NOT connected (deleting)");
            drmModeFreeConnector(con);
        }

solves the problem. Not sure what would be other implications of such a change, but anyway reporting it for anyone interested.

@emdzej The "cheap and dirty" fix looks good enough to me, feelf ree to send a PR!

Will do shortly. Thanks.


Add openPage option.

See https://webpack.js.org/configuration/dev-server/#devserver-openpage.

Looks good to me 👍


Exclusion support in SourceFiles.

My thought is to combine WalkDir and ignore to use gitignore patterns as a whitelist for files to copy.  This will then give users a means to exclude files with !file.

Maybe globwalk would want this upstreamed?


Jax-RS - Reused Subresources not picked up by JaxRsApiListingResource / Reader

In my project, I use an extensive hierarchy of nested Sub-Resources.
For implementing "leaf" Sub-Resources I reuse the same Classes / Sub-Resources on multiple, different paths.
0 Top Level Resource
01 Subresource A
021 Reusable SubResource X
022 SubSubResource S
0221 Reusable SubResource X
02 Subresource B
021 Reusable SubResource X

My issue is that the Default Reader-class stops descending the Resource tree when it encounters a Sub-Resources it has encountered before, thus not picking up / rendering the complete API.
I have a workaround - a custom Reader class with this change
//if (subResource != null && !scannedResources.contains(subResource)) {
if (subResource != null) {

and a corresponding CustomJaxRsApiListingResource class that calls it.
However, I wonder if there is a way to fix this in the default implementation.

Yes, I believe you are right, we are not recursing into subresources over and over.

Then I believe there is no point in using Sub-Resources at all if I can't reuse them under a Root / Top Level Resource.

I think @elakito did something related to that? My memory could be playing tricks on me.

The change that I made for 1.5.4 intended to prevent an infinite recursion by checking for the visited resources while scanning down the path to determine when to stop. So, you can have sub resources recursed along the different paths but you can't visit the same resource along the path more than once.
@webron @fehguy
I think, to support what @osx2000 wishes (i.e., to be able to expand from path 0221 down), we will need a new notation for a resource-link in swagger-api, so that the swagger document is bounded but the UI can expand the path down on demand to represent an infinite recursion.
regards, aki

just a moment and correction.
@osx2000 you mean this X is a leaf resource? We could exclude the leaf resources from this recursive check. In that way, without adding a new notation, we can support such cases.

i just looked into it and i think a single liner should probably fix this issue.
i'll try it out with a test case and see if this is indeed the case...


failing to deploy when it tries to initialize fernet key authentication

When the deploy playbook tries to initialize the fernet key authentication, it fails, because the docker container is not ready.
This is because the container tries to restart and cant find python3.6, then crashes and restarts again.
We have to add python3.6 to the docker images or containers.

Might be resolved by issue #26.

Might be resolved by issue #26.


gameMode refactoring

Gamemode is redundant, complete removal. Additionally, conditions and checking them are too spread out, making a new file to do this.


Update openweathermap.js

Fixes the onecall API path to the current path at openweather.com.
Until a new version is available, you can manually correct this in your installation file to make it work again. But, as mentioned above, make as well sure you upgrade to the ONECALL product which comes with 1000 calls/day for free and can be capped at 1000 in the product settings to prevent unwanted payments.

Any plans to publish this npm?

Done in v3.2.7


Update README.md

Preview at https://github.com/msabramo/airfield/blob/patch-1/README.md

Thanks

Thanks


[processor/resourcedetection] support http detector and periodically do detect action

Description:

Link to tracking Issue:
Testing:
Documentation:

waiting upstream reply issue


[frigate] Support disabling probes and secrets for env vars

Special notes for your reviewer:
I don't mind moving this to the frigate repo if you would prefer.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [radarr])

Did the GH action ever succeed for frigate?

Hi @blakeblackshear!  If you're willing to host the chart in the frigate repo, I would happy to contribute a PR to start that process.  The github actions for linting/testing, and releasing charts works really well IMO.  And the repo could be added to helm hub for better discoverability.  Let me know if you're interested!
Now, with all that said, the installation test for this chart failed I suspect due to the fact that there isn't a real rtsp stream for frigate to connect to. Unless that can be stubbed-out somehow, it may not be possible to properly 'test' Frigate in a sanitized environment like github actions.

I propose we merge this given that it did pass linting and may not be possible to 'test' a full installation.

I'm definitely willing to host it given that is the primary way I run frigate myself.

Merging with failed test because it is not possible to currently test frigate without a real rtsp device available.


Replace print with logging

refs #79

Prints now include timestamps, useful to check processing time.
$ python examples/clock.py -d gifanim --max-frames 10
2016-12-08 23:29:12,151 - Recording frame: 1
2016-12-08 23:29:12,252 - Recording frame: 2
2016-12-08 23:29:12,354 - Recording frame: 3
2016-12-08 23:29:12,455 - Recording frame: 4
2016-12-08 23:29:12,556 - Recording frame: 5
2016-12-08 23:29:12,657 - Recording frame: 6
2016-12-08 23:29:12,758 - Recording frame: 7
2016-12-08 23:29:12,859 - Recording frame: 8
2016-12-08 23:29:12,960 - Recording frame: 9
2016-12-08 23:29:13,061 - Recording frame: 10
2016-12-08 23:29:13,061 - Please wait... building animated GIF
2016-12-08 23:29:13,071 - Wrote 10 frames to file: oled_anim.gif (16475 bytes)

Coverage remained the same at 87.415% when pulling f38e097c2d66f6511529d85fdaec25bac8fe41bd on print-logging-79 into 4ea16591b50e6540d6f0aeb4db8da84fcd26ef33 on master.

Coverage remained the same at 87.415% when pulling f38e097c2d66f6511529d85fdaec25bac8fe41bd on print-logging-79 into 4ea16591b50e6540d6f0aeb4db8da84fcd26ef33 on master.


Instead of Preview making space open the file or another keybinding on OSx

I was hoping to open in an external program instead of previewing some files in the dired window (for example a .jpg,.mp4, or .pdf. Can you set a keybinding to run the command  open filename here or another way if there is a better generic way to open any file on OSx.
Is this possible to add a keybinding to do this?

It is possible.
Since I cannot test it without OSX, I post diff. Can you test it? does it work?
--- dired_misc.py Tue Apr 05 12:42:12 2016
+++ dired_misc.py Wed Apr 06 13:57:51 2016
@@ -105,20 +105,22 @@
     """
     quick look current file in mac or open in default app on other OSs
     """
-    def run(self, edit):
+    def run(self, edit, preview=True):
         self.index = self.get_all()
         files = self.get_marked() or self.get_selected(parent=False)
         if not files:
             return sublime.status_message('Nothing chosen')
-        if OSX:
+        if OSX and preview:
             cmd = ["qlmanage", "-p"]
             for filename in files:
                 fqn = join(self.path, filename)
                 cmd.append(fqn)
             subprocess.call(cmd)
         else:
-            if NT:
-                launch = lambda f: os.startfile(f)
+            if OSX:
+                launch = lambda f: subprocess.call(['open', f])
+            elif NT:
+                launch = lambda f: subprocess.call(['start', f], shell=True)
             else:
                 launch = lambda f: subprocess.call(['xdg-open', f])
             for filename in files:

  {
    "keys": [" "],
    "command": "dired_quick_look", "args": { "preview": false},
    "context": [
      { "key": "selector", "operator": "equal", "operand": "text.dired" },
      { "key": "setting.dired_rename_mode", "operand": false }
    ]
  }

I tried to git apply the patch and it says corrupt patch at line 29.
Anyways, I manually entered the lines and it works great so thanks for that.
Also I think a good feature would be if you can open the current folder in the terminal. I know the command that needs to be run on mac but not sure how to get the current folder or the best place to put this in the code.

the best place to put this in the code.

dired_misc.py is exact place for that kind of stuff, if you’re going to send pull request.

how to get the current folder

If you inherit new command from BaseCommand (just like all others), then it is simply self.path.
There are several fixes pending to be committed and released, I’ve already promised to do that on prev weekend, but yea, this weekend—for sure.


fix(inline-list): add listitem role to items

General summary

Since each element of the InlineList component has a display: contents css attribute, it removes the semantic value of these items.
Added back a listitem role on each items to reset the semantics.
Screenshots

Test OK

stories showing list/listitem roles :heavy_check_mark:
demos showing text overflow properly :heavy_check_mark:


Number of characters deleted with CMD+Z depends on initial typing speed

Precheck

[x] Do a quick search and make sure the bug has not been reported yet.
[x] Please disclose security vulnerabilities privately at<EMAIL_ADDRESS>[x] Finally, be nice and have fun!.

Description
For typed text, the number of characters deleted each time the CMD+Z key combination is pressed depends upon the speed/cadence with which the original text was typed. This is due to the currently unchangeable 200ms debounce used in the observer.
Environment

Editor.js version: 2.23.2
Plugins you use with their versions: editorjs-undo
Device, Browser, OS: MacOS, Chrome

Current behaviour
The number of characters deleted each time CMD+Z is pressed depends upon the rate of speed with which the original text was typed. To reproduce this behaviour:

Type some text into the editor, varying your typing speed.
Press CMD+Z a few times.
Observe that sometimes a single character is deleted, and other times multiple characters are deleted.

https://user-images.githubusercontent.com/19937217/156613709-303b1dcd-24dc-4a02-bbb3-5252950613d3.mov
Expected behaviour
It should be possible to have pressing CMD+Z delete one character at a time, regardless of how quickly the original text was entered.
https://user-images.githubusercontent.com/19937217/156613741-44381c45-696a-4049-bd62-329edd1d09c3.mov
Additional context
N/A

@MarioRodriguezS @juanhurtado10 - Since my PR for fixing this issue has been approved and merged and will be available in the next release, I'm closing this issue.

@MarioRodriguezS Just leaving a note to let you know that the install command on npm is currently still using the rc.2 version instead of rc.3.


Dependency updates

Will raise a separate issue for okhttp-eventsource, as it is a major version change.
Signed-off-by: Paul Harris<EMAIL_ADDRESS>Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

com.fasterxml.jackson.dataformat.yaml.JacksonYAMLParseException: The incoming YAML document exceeds the limit: 3145728 code points.

Well I did not expect that failure...


lgb.dump fails for larger models

@guolinke In the latest version, the lgb.dump R function produces an empty character vector for larger models.  The largest model that worked successfully for me had a dump file of around 2^20 characters, which can be hit pretty easily (600 rounds, 7 num_leaves).
If it helps, I can post reproducible example when I get home.

@rgranvil it is caused by the latest commit ?

@guolinke I'm not sure exactly which commit caused it.  I was comparing v2.0.10 to the current master branch, so it was something in that range.

@rgranvil can you check it before and after this commit: 619c06d8f85149cd437df96cf9de9f22f7d48e8d ?

@rgranvil
can you also try set these two lines:
https://github.com/Microsoft/LightGBM/blob/master/src/lightgbm_R.cpp#L607
https://github.com/Microsoft/LightGBM/blob/master/src/lightgbm_R.cpp#L629
to if (out_len <= R_AS_INT(buffer_len)) {

@guolinke I confirmed that commit was causing the issue and modifying those two lines seems to fix the problem.

@rgranvil thanks, I just fix it in the latest commit

@rgranvil close. feel free to reopen if it doesn't work.

@guolinke I get the following errors when trying to install the R-package of the latest commit:
/tmp/RtmpZNfk4j/R.INSTALL64c02f214618/lightgbm/src/src/lightgbm_R.cpp: In function ‘LGBM_SER* EncodeChar(LGBM_SE, const char*, LGBM_SE, LGBM_SE)’:
/tmp/RtmpZNfk4j/R.INSTALL64c02f214618/lightgbm/src/src/lightgbm_R.cpp:41:27: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (R_AS_INT(buf_len) < str_len) { return dest; }
^
At global scope:
cc1plus: warning: unrecognized command line option "-Wno-ignored-attributes" [enabled by default]

@rgranvil sorry, it is a warning. I just fix it.

@guolinke The warning is gone, but I run into the original problem with lgb.dump of large models.

@rgranvil really? I don't think that commit will break the large model...

@guolinke Yes, I just pulled from the master branch and tested again. Here is a reproducible example:
Code:
library(lightgbm)
data(agaricus.train, package = "lightgbm")
train <- agaricus.train
dtrain <- lgb.Dataset(train$data, label = train$label)
data(agaricus.test, package = "lightgbm")
test <- agaricus.test
dtest <- lgb.Dataset.create.valid(dtrain, test$data, label = test$label)
params <- list(objective = "regression", metric = "l2")
valids <- list(test = dtest)
model <- lgb.train(params,
                   dtrain,
                   min_data = 1,
                   nrounds = 600,
                   num_leaves = 8,
                   learning_rate = .01)
lgb.dump(model)

Output:
> lgb.dump(model)
[1] ""

@rgranvil can you try it again ? I believe it is fixed by this commit: 63b60825ba90b348e8100d0f9bb23c14e5bb857d

@guolinke I just tried it again and get the same problem.

@rgranvil I just try it. and the result is fine:

can you check your installation ?

@guolinke Strange.  I still get the same blank output:
> lgb.dump(model)
[1] ""

I tried installing from the current master and from 63b60825ba90b348e8100d0f9bb23c14e5bb857d.  I tested it on both Linux and Windows machines.    I also tried completely removing the lightgbm R package prior to installation.  Everything gives the same result.  I don't see any errors/warnings in the installation.  Anything else I should try?

@Laurae2 can you help to test this as well?

@guolinke @rgranvil Installed latest LightGBM (c2191aa), same issue as OP but this is corrected by doing this:
> library(lightgbm)
Loading required package: R6
> data(agaricus.train, package = "lightgbm")
> train <- agaricus.train
> dtrain <- lgb.Dataset(train$data, label = train$label)
> data(agaricus.test, package = "lightgbm")
> test <- agaricus.test
> dtest <- lgb.Dataset.create.valid(dtrain, test$data, label = test$label)
> params <- list(objective = "regression", metric = "l2")
> valids <- list(test = dtest)
> model <- lgb.train(params,
+                    dtrain,
+                    min_data = 1,
+                    nrounds = 600,
+                    num_leaves = 8,
+                    learning_rate = .01)
Loading required package: Matrix
> lgb.dump(model)
[1] ""
> lgb.dump(model, num_iteration = 1)
(truncated long output...)
> model$dump_model(num_iteration = 1)
(truncated long output...)

@rgranvil please try the latest code

@guolinke Works well now, thanks!


feat!: stellar-config crate

This adds an new crate stellar-config which contains all logic around configuration. Currently this just a refactor but in the future it could use the config crate to more formally describe the config.
BREAKING CHANGE:
This also removes the deprecated config subcommand since we had split it up already into keys and network.

config work waiting for the alias PR to go in.

blocked by the alias work

closed in favor of #1479


Node.js build requirement breaks building on 32-bit PowerPC/big-endian 64-bit and other architectures

****************************************************************************
*** Duktape python tooling is obsolete, migrate to JS-based tooling!     ***
*** This tool now internally invokes the JS-based tooling.               ***
*** Minimum Node.js version is 14.x.                                     ***
****************************************************************************

node.js does not support 32-bit PowerPC on account of V8 not having a bytecode generator for it. At least on Linux. The last version of Node that did apparently work was in the 10.x series.
IBM's nodejs/v8/blink port to PPC only targets PPC64/POWER little-endian, if I remember correctly. So 64-bit big-endian systems are also now no longer able to build it. If you decide to test this, you can get an installer for 32-bit PPC from https://cdimage.debian.org/cdimage/ports/current/ - I believe it should run under QEMU if you (understandably) don't have appropriate PPC32 hardware. I am using a Powerbook G4 for testing.
Node.js requirements also break compatibility with a lot of more obscure/older CPU architectures (e.g., alpha, PA-RISC, even m68k) that it worked on before.
The bottom line: the move from python to a node-based build system makes it nearly impossible to use on PowerPC, even though once compiled and linked it can work fine.
I suppose it is possible to cross-compile, still - but I don't really consider that to be anything but a workaround.
Practically speaking, this isn't a major issue based on the sheer age of PPC32 and most of the other impacted architectures, but it still gets used in places and seems a bad fit for a project that claims to prioritize portability to the point of not having file or stream I/O.

The short answer is that in the current state:

You could try to use an existing release, and its default configuration in which case a configure step (and thus Python/Node.js) is not needed.
Failing that, you should run the dist/configure step on an x64 host and then use the result source/header files in the target build. The compilation itself doesn't require Node.js, obviously. Unless your configuration changes all the time, this may be quite workable.

I switched from Python to Node.js for a few reasons:

Python2 -> Python3 transition is painful, and I don't want to come to that transition again at some future point in time.
Using Javascript is more natural since this is after all a Javascript project.
Node.js is faster than Python, so configure/compile cycles are a bit faster (nice, but not critical).
And finally, having an option to self host the configuration process on Duktape itself.

Self hosting would make Duktape configure process portable to almost all targets running Duktape itself. It's actually functional in practice (though not available readily in master), and I hope to preserve and polish the self hosting capability into 3.x release.

From my point of view nodejs isn't just an issue specific to unsupported platforms but in general.
Just to built duktape now another build dependency is needed which takes at least about an hour to build. We have to build a Javascript engine to built another..
That python is slower does not impact the build process enough to matter.
duktape self hosting would only work if  it can bootstrap itself through another language that is
likely already installed on the system and doesn't require hour(s) of building.
Javascript for building software doesn't sound like a good idea. Instead of rolling a build system reusing one such as CMake would be better IMHO.
Not just for building but also for development to integrate with other tooling such as language-servers.


Some help with hyperparameters please!

Hey, hope you're doing well.
I've been experimenting with your codebase using my own captures, and converting them to the Dynerf structure.
It seems to reconstruct the static scene well (same as standard 3DGS).
However, the dynamic part is often ignored, maybe the movement is too large?
What parameters are the most relevant to mitigate this?
Attached is an example frame: The dynamic part (cyclist in this case) is almost invisible.
I'm guessing time_smoothness_weight should play a part in this, not sure about the rest.
Thanks.

you can enlarge the training iteration such as iterations = 60000, densify_until_iter = 30_000 or more.
Actually, 4DGS is hard to learn the large motion such as bicycle, you can check the appendix of my paper.
try to use the dense point clouds to initialize the 4D Gaussians.

I've been stuck on (and still am) avoiding overtraining scenes with sparse views. Through this there's a couple things I think I understand better -> Namely that the regularisers, except from the Plane TV, won't have a significant impact on the final result (unless the weights are really small/large). Essentially, if you are not getting "okay" results without the temporal regularisers, then using them/tuning them wont make a big difference.
As @guanjunwu alludes to, plane based methods are a lot more sensitive to the initialisation. So you may want to consider other methods (such as dust3R) to improve point-cloud initialisation, or perhaps increasing the bounding region and number of initial PCs. Additionally, for cases such as mine, lower learning rate and longer training time can help avoid overfitting (even if it takes very long to train).

I've been stuck on (and still am) avoiding overtraining scenes with sparse views. Through this there's a couple things I think I understand better -> Namely that the regularisers, except from the Plane TV, won't have a significant impact on the final result (unless the weights are really small/large). Essentially, if you are not getting "okay" results without the temporal regularisers, then using them/tuning them wont make a big difference.
As @guanjunwu alludes to, plane based methods are a lot more sensitive to the initialisation. So you may want to consider other methods (such as dust3R) to improve point-cloud initialisation, or perhaps increasing the bounding region and number of initial PCs. Additionally, for cases such as mine, lower learning rate and longer training time can help avoid overfitting (even if it takes very long to train).

I'm encountering a similar issue with dense pointclouds, namely PCLs coming from Flowmap. Which learning rates did you lower? Was it all of them in general or just the ones related to 4DGS (i.e. deformation_lr / grid_lr)?


Corrected PowerShell Syntax

Now aligns with https://docs.microsoft.com/en-us/powershell/module/az.network/add-azapplicationgatewayhttplistener?view=azps-6.3.0#example-3--add-a-https-listener-with-ssl-and-hostnames

@isjwuk : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


digitalocean.Droplet image is invalid

Problem description
The image name in pulumi registry digitalocean droplet is not working. https://www.pulumi.com/registry/packages/digitalocean/api-docs/droplet/ .
Its giving  You specified an invalid image for Droplet creation error
Suggestions for a fix
Please change the image from ubuntu-18-04-x64 to ubuntu-20-04-x64

Fixed via https://github.com/digitalocean/terraform-provider-digitalocean/commit/1124e8761e8257521981786a63c1eff18de6e29d.


Use a separate pool of small instances for long-running Buildkite jobs that don't need lots of CPU

e.g. when we're waiting for weco-deploy, we don't need a large instances to be running.
This is described in the Buildkite documentation: https://buildkite.com/docs/agent/v3/elastic-ci-aws#multiple-instances-of-the-stack

Here's a summary of what I've done: I've created a new pool of "cheap" Buildkite agents (https://github.com/wellcomecollection/platform-infrastructure/pull/244). These agents:

Run on t3.nano instances instead of r5.large
Only read jobs from the nano queue
Have less local disk, because they shouldn't be pulling lots of jars/dependencies
Shut down faster when not in use

You can opt a job into using one of these agents by adding the following lines to a Buildkite step:
   - command: "buildkite-agent pipeline upload .buildkite/pipeline.yml"
     label: ":pipeline:"
+    agents:
+      queue: nano

I've already added them to the "pipeline upload" step on a bunch of existing pipelines, and there are also PRs on the various repos (linked above).
That's all I'm going to do today, and I'll revisit it after next month's bill to see if it's making a difference. We might flip more stuff over to them as we go along, but that's plenty for now.

Done for now.


Trim repeating alerts which flooding log files and outputs triggered by same action

Hi,
Motivation
In some cases, one action repeating will generate same alerts continuously, and in contents only difference is the timestamp, then alerts will flood log files or other outputs locations, so other alerts may be hidden inside the flood.
e.g. use event-gernerater to simulate a "Write below /etc"
/bin/event-generator run --loop syscall.WriteBelowEtc
by default it will happen every 250ms, and in output, it will show same number of alerts there.
Feature
Probably we need a feature to trim such logs, e.g. it will only show the first log, then transfer other repeating logs to something like " happen xx times in last xx mins" or some json keys/values to indicate this
Thanks

A rate-limit mechanism for alert notifications is already in place. Though sometimes it's not a good thing to have, for example, see https://github.com/falcosecurity/falco/issues/1333.
But your suggestion is going further and makes sense. Moreover, aggregating notifications could be a problem in some use cases. For example, how would one reconstruct the history of a security incident if they don't have the complete list of all separate notifications with precise timestamps?
It would be a good topic for our community call btw.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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.


Will no longer be updated to the Raku ecosystem

Hello,
Thank you for the time and effort you have put in what is now the Raku Programming Language.  The language and its ecosystem are moving forward.
This means that these distributions:

https://github.com/skaji/App-P6Ghq
https://github.com/skaji/Data-Section-Simple
https://github.com/skaji/Frinfon
https://github.com/skaji/perl6-MetaCPAN-Favorite
https://github.com/skaji/perl6-tail
https://github.com/skaji/perl6-WaitGroup

will no longer be checked for updates by the Raku ecosystem.
This means that you will need to move your module to the zef ecosystem on the next update you make to these distributions.  Otherwise your update will simply not be seen by the ecosystem.
If you are no longer interested in maintaining these distributions, you can have these distributions moved to Raku Community Modules Adoption Center, where members of the Raku Community will take care of them.
This can be as easy as confirming that in a response to this issue.
Again, thank you for your time and efforts!
On behalf of the Raku Steering Council,
Elizabeth Mattijsen

Thanks for reaching out.
I think it's fine to simply remove them.
https://github.com/Raku/ecosystem/pull/631

So done  :-)


fix mocking of readers and writers

also add add option to call fake readers and writers in test

Hi jabdoa2, thank you for the contribution!
I just read the code, and I can't figure out what your changes do. This is probably because I don't have full understanding of Asyncio at this low level.
I think that we should first add some test that shows what doesn't work for you, and then check if the changes you made in this commit fix it. Can you add a test that shows what fails for you?

we maintain a local fork in MPF now


README

README
User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines
Acceptance Criteria
GIVEN a webpage meets accessibility standards
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

README updated


DEV-44: search result pagination

Backend PR : https://github.com/uCredit-Dev/uCredit-API/pull/69
Search result pagination

migrate performSmartSearch logic to backend
integrate frontend with new /search route and pagination
remove unnecessary filtering of courses (backend does all filtering)
cache courses once with /api/coursesByPlan route instead of multiple /search calls

!! work in progress !!

note: course searches for prereq checking and fine requirements still uses same search logic now /cartSearch. Better done as part of the distributions migration project...
Implementations
See under files changed tab
Testing
Tested search locally by course title, number, with various applicable filters. Confirmed that prereq checking and fine requirements searching still works.
todo: try importing plan / reviewing plan

not sure how to fix the prettier error; is successful locally but failing in CI :(((


Extensions can add context value (metadata) to editors

The basic idea is to let the extensions be able to register some metadata to the editors/documents, and can be consumed by others.
Example
In Java, a source file may have the following metadata:

hasMain: indicates if the source file as a main() method
isTest: indicates if the source file has test cases

Then, based on the metadata, the shortcuts in the editor toolbar can be executed with different behavior:

If hasMain is true, run the Java Application
If isTest is true, run the test cases
If both are true, ask user which kind of execution he want to launch.

You can use the setContext command to set a context keys for open text documents. For example, you can set a context key hasMainMethods to an array of the open document URIs that have a main method, and then have the "when" clause be something like resource in hasMainMethods. Let us know if that doesn't work for you.

Hi @connor4312, that does help to some extent. But we still have problems on consuming the context value.
Since currently there is no way to get the context value in the code, in order to handle with the different combination of the two booleans, we have to register different commands which represent each combination.

Alternately, you could have a "tracker" class internally that sets the context and provides a way to read the different states when a single command is called

Yes it's feasible in just one extension, considering the collaboration with multiple extensions, things will be much complicated. Different extensions need to contribute different metadata to the "tracker". And the "tracker" extension needs to expose the commands/apis for register context and consume context.
It will be great if the VS Code itself can support these commands/apis. Just like the context value in the tree view nodes.

/duplicate of https://github.com/microsoft/vscode/issues/10471


d-eisenga/update rules

Motivation
Converting the remaining rule functions so we have a single style of defining rules.
Changes

Move the file, people, and PR body-related rule functions into the tests
Rewrite the rules in a more functional style
Update the rule tests to match the structure of the commit rule tests

To do

Rewrite authorMatchesBranchPrefix in a functional style somehow
Split the all-prs file into parts like the tests?

I thought I'd take a stab at this. I'm not very used to FP so any feedback would be appreciated, as would any advice on how to go about implementing the authorMatchesBranchPrefix function in a more functional way


feat: Support may_act claim

The Token Exchange RFC https://www.rfc-editor.org/rfc/rfc8693.html#name-may_act-authorized-actor-cl defines the may_act claim, this claim identifies who is authorized to become the actor and act on behalf of another party.
The main goal of this issue add the claim may_act to the framework.

Closing.
The project has been archived.
#18


Better formatting for multi-line field comments in API reference documentation

This is a Feature Request

What would you like to be added
Some API fields have multi-line comments, some with formatting. For example the field fsGroup in corev1.PodSecurityContext has this comment in the Go source:
// A special supplemental group that applies to all containers in a pod.
// Some volume types allow the Kubelet to change the ownership of that volume
// to be owned by the pod:
//
// 1. The owning GID will be the FSGroup
// 2. The setgid bit is set (new files created in the volume will be owned by FSGroup)
// 3. The permission bits are OR'd with rw-rw----
//
// If unset, the Kubelet will not modify the ownership and permissions of any volume.
// +optional
FSGroup *int64 `json:"fsGroup,omitempty" protobuf:"varint,5,opt,name=fsGroup"`

In the HTML reference documentation, this is rendered as a single-line:

    fsGroup

    integer

  A special supplemental group that applies to all containers in a pod. Some volume types allow the Kubelet to change the ownership of that volume to be owned by the pod:  1. The owning GID will be the FSGroup 2. The setgid bit is set (new files created in the volume will be owned by FSGroup) 3. The permission bits are OR'd with rw-rw----  If unset, the Kubelet will not modify the ownership and permissions of any volume.

I would like to change the formatting of multi-line comments like this in the rendered documentation.
Why is this needed
Much of the author's original intent in this formatting is obfuscated in the HTML reference, and the resulting single line does not "scan" very well. Resulting in harder to understand documentation. Especially where a feature or side effect is complex enough to warrant a longer description.
Comments

It was suggested on Slack by Qiming Teng that newlines could be translated in HTML to
 tags to preserve the new lines.
In cases like fsGroup in particular, the author has used an ordered list. Ideally this could be translated into a HTML ordered list for the best formatting. However I'm unsure of the implications of attempting to do this on text comments.

Does Golang support Markdown comments (like in Rust)?

@sftim There are places where the comments are using inline markdown syntax such as links and backtiqs.

Sure; really I'm asking if that's a de facto thing or documented and de jure. (If it's documented like in Rust then it's easy to put it through a machine and expect ~consistent~ documented results).

I don't believe there is any particular Go language policy about using Markdown in comments, although I could well be wrong. The Go wiki is silent on it, and GoDoc doesn't seem to render field comments at all.

I looked for other examples. It seems that where there is formatting it is, by accident or design, mostly Markdown. But I haven't searched exhaustively so I wouldn't be surprised if you could find counter-examples.

maxSkew in corev1.TopologySpreadConstraint uses an unordered list and a table. Which renders about as well as you might expect:
// MaxSkew describes the degree to which pods may be unevenly distributed.
// It's the maximum permitted difference between the number of matching pods in
// any two topology domains of a given topology type.
// For example, in a 3-zone cluster, MaxSkew is set to 1, and pods with the same
// labelSelector spread as 1/1/0:
// +-------+-------+-------+
// | zone1 | zone2 | zone3 |
// +-------+-------+-------+
// |   P   |   P   |       |
// +-------+-------+-------+
// - if MaxSkew is 1, incoming pod can only be scheduled to zone3 to become 1/1/1;
// scheduling it onto zone1(zone2) would make the ActualSkew(2-0) on zone1(zone2)
// violate MaxSkew(1).
// - if MaxSkew is 2, incoming pod can be scheduled onto any zone.
// It's a required field. Default value is 1 and 0 is not allowed.
MaxSkew int32 `json:"maxSkew" protobuf:"varint,1,opt,name=maxSkew"`

MaxSkew describes the degree to which pods may be unevenly distributed. It's the maximum permitted difference between the number of matching pods in any two topology domains of a given topology type. For example, in a 3-zone cluster, MaxSkew is set to 1, and pods with the same labelSelector spread as 1/1/0: | zone1 | zone2 | zone3 | |   P   |   P   |       | - if MaxSkew is 1, incoming pod can only be scheduled to zone3 to become 1/1/1; scheduling it onto zone1(zone2) would make the ActualSkew(2-0) on zone1(zone2) violate MaxSkew(1). - if MaxSkew is 2, incoming pod can be scheduled onto any zone. It's a required field. Default value is 1 and 0 is not allowed.

matchPolicy in admissionregistrationv1.MutatingWebhook uses a definition list
// matchPolicy defines how the "rules" list is used to match incoming requests.
  // Allowed values are "Exact" or "Equivalent".
  //
  // - Exact: match a request only if it exactly matches a specified rule.
  // For example, if deployments can be modified via apps/v1, apps/v1beta1, and extensions/v1beta1,
  // but "rules" only included `apiGroups:["apps"], apiVersions:["v1"], resources: ["deployments"]`,
  // a request to apps/v1beta1 or extensions/v1beta1 would not be sent to the webhook.
  //
  // - Equivalent: match a request if modifies a resource listed in rules, even via another API group or version.
  // For example, if deployments can be modified via apps/v1, apps/v1beta1, and extensions/v1beta1,
  // and "rules" only included `apiGroups:["apps"], apiVersions:["v1"], resources: ["deployments"]`,
  // a request to apps/v1beta1 or extensions/v1beta1 would be converted to apps/v1 and sent to the webhook.
  //
  // Defaults to "Equivalent"
  // +optional
  MatchPolicy *MatchPolicyType `json:"matchPolicy,omitempty" protobuf:"bytes,9,opt,name=matchPolicy,casttype=MatchPolicyType"`

namespaceSelector, also in admissionregistrationv1.MutatingWebhook uses a couple of code bloks for example JSON. Interestingly it doesn't use the triple-backtick code block syntax.
// NamespaceSelector decides whether to run the webhook on an object based
  // on whether the namespace for that object matches the selector. If the
  // object itself is a namespace, the matching is performed on
  // object.metadata.labels. If the object is another cluster scoped resource,
  // it never skips the webhook.
  //
  // For example, to run the webhook on any objects whose namespace is not
  // associated with "runlevel" of "0" or "1";  you will set the selector as
  // follows:
  // "namespaceSelector": {
  //   "matchExpressions": [
  //     {
  //       "key": "runlevel",
  //       "operator": "NotIn",
  //       "values": [
  //         "0",
  //         "1"
  //       ]
  //     }
  //   ]
  // }
  //
  // If instead you want to only run the webhook on any objects whose
  // namespace is associated with the "environment" of "prod" or "staging";
  // you will set the selector as follows:
  // "namespaceSelector": {
  //   "matchExpressions": [
  //     {
  //       "key": "environment",
  //       "operator": "In",
  //       "values": [
  //         "prod",
  //         "staging"
  //       ]
  //     }
  //   ]
  // }
  //
  // See
  // https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/
  // for more examples of label selectors.
  //
  // Default to the empty LabelSelector, which matches everything.
  // +optional
  NamespaceSelector *metav1.LabelSelector `json:"namespaceSelector,omitempty" protobuf:"bytes,5,opt,name=namespaceSelector"`

Various types use single backticks for defined names (e.g., various fields in admissionregistrationv1.ServiceReference.

@JamesLaverack Thanks for the detailed examples. I have filed an issue for this. However, I don't think there is a smarter parser which can detect the code snippets in the sample comments above or the sample  table.
Yes, the comment looks like markdown syntax. But when you feed them into a markdown parser, you will not get the desired output, because the table is not good. It should be:
MaxSkew describes the degree to which pods may be unevenly distributed.
It's the maximum permitted difference between the number of matching pods in
any two topology domains of a given topology type.
For example, in a 3-zone cluster, MaxSkew is set to 1, and pods with the same
labelSelector spread as 1/1/0:

| zone1 | zone2 | zone3 |
|-------|-------|-------|
|   P   |   P   |       |

- if MaxSkew is 1, incoming pod can only be scheduled to zone3 to become 1/1/1;
  scheduling it onto zone1(zone2) would make the ActualSkew(2-0) on zone1(zone2)
  violate MaxSkew(1).
- if MaxSkew is 2, incoming pod can be scheduled onto any zone.
  It's a required field. Default value is 1 and 0 is not allowed.

Note how an empty line is inserted before the table and another line inserted after the table. Also note that the unordered list are properly indented to avoid breaking markdown.
For the code snippet to be "properly" rendered, the comment must be changed as well. E.g.
  // If instead you want to only run the webhook on any objects whose
  // namespace is associated with the "environment" of "prod" or "staging";
  // you will set the selector as follows:
  //
  // ```
  // "namespaceSelector": {
  //   "matchExpressions": [
  //     {
  //       "key": "environment",
  //       "operator": "In",
  //       "values": [
  //         "prod",
  //         "staging"
  //       ]
  //     }
  //   ]
  // }
  // ```

Yes, the comment looks like markdown syntax. But when you feed them into a markdown parser, you will not get the desired output […]

Ah I see. So it's kind of Markdown.
Looks like that "just" applying a Markdown parser isn't going to cut it. I took a look at the issue you raised and at least parsing single-backtick content into a  block seems reasonable.
Instead of trying to figure out how to parse the not-quite-markdown, could we change the comments to be valid Markdown? There's a whole host of questions I can think of as a result of doing that though:

Do we expect that future authors will use Markdown, and keep their comments, where they do use 'advanced' formatting, as valid Markdown?
Is such a change big enough in scope it would require a wider community discussion?
As @sftim mentioned above, is there any Go standard here?

Of course, for now just single backticks into  and preserving paragraphs is probably an improvement. Even if further thought about tables and multi-line code blocks is needed.

@JamesLaverack I'd assume a change like we suggested is big enough that warrants a community wide discussion, but I'm not aware of any Go standard on this.

Some further investigation into this ... The comment format has been lost when we got the API definition JSON file. Take the maxSkew in corev1.TopologySpreadConstraint as an example, the resulted OpenAPI JSON for it is something like this:
"maxSkew": {
          "description": "MaxSkew describes the degree to which pods may be unevenly distributed. It's the maximum permitted difference between the number of matching pods in any two topology domains of a given topology type. For example, in a 3-zone cluster, MaxSkew is set to 1, and pods with the same labelSelector spread as 1/1/0: | zone1 | zone2 | zone3 | |   P   |   P   |       | - if MaxSkew is 1, incoming pod can only be scheduled to zone3 to become 1/1/1; scheduling it onto zone1(zone2) would make the ActualSkew(2-0) on zone1(zone2) violate MaxSkew(1). - if MaxSkew is 2, incoming pod can be scheduled onto any zone. It's a required field. Default value is 1 and 0 is not allowed.",
          "format": "int32",
          "type": "integer"
        }

The multi-lined comment has been been merged in the API spec (https://raw.githubusercontent.com/kubernetes/kubernetes/master/api/openapi-spec/swagger.json).
Sigh ...

So the problem is in the OpenAPI generation code? To make sure I understand fully, the current process is:

An OpenAPI JSON file is generated from the Go source code
The website HTML is generated from the OpenAPI JSON file.

Are there other steps in there?

@JamesLaverack No other steps.

There is nothing left to do from website's perspective. Related enhancement request has been logged into the reference-tools project.
/close


added value scopedNamespaces to allow multiple namespaces to be manage…

…d by ESO

this will also satisfy wants requested by https://github.com/external-secrets/external-secrets/issues/1050

Hey @jmmclean ! Thank you for your contribution!
I particularly am not sure if this would be the right direction to go to (i.e. scope multiple namespaces by deploying multiple ESO instances within our helm charts). AFAIK users can already do that with some simple tooling like helm charts + terraform as is. I would also suggest you to look at #163 as it was the way we were thinking to tackle multiple namespace scoping (via ClusterSecretStore).

hey @gusfcarvalho , thank you for the suggestions and review of the code; id be happy to do multiple service accounts per operator deployment as well, but the gain there is basically more security granularity.
My big motivation for this PR is bc I leverage Argo to manage my applications within a K8s cluster. I have a single ops cluster that manages multiple environments, so i use namespacing as a mechanism to create isolation. Argo CD will not manage cluster resources (ie ClusterRole, ClusterRoleBinding, CRDs) in --namespaced mode. The above PR satisfies deploying applications to a environment specific namespaces (ex: external-secrets-dev) that can manage secrets per bounded namespace (think argo-workflows-dev or kyverno-dev or argo-cd-dev)

hey @gusfcarvalho , thank you for the suggestions and review of the code; id be happy to do multiple service accounts per operator deployment as well, but the gain there is basically more security granularity.
If we do not have extra service accounts, the Roles and RoleBindings we create for the additional external-secrets deployments would be bound to the same service account, leading to these external-secrets effectively managing more than they should. If we have a single service account this solution wouldn't be compliant to your requirement.

My big motivation for this PR is bc I leverage Argo to manage my applications within a K8s cluster. I have a single ops cluster that manages multiple environments, so i use namespacing as a mechanism to create isolation. Argo CD will not manage cluster resources (ie ClusterRole, ClusterRoleBinding, CRDs) in --namespaced mode. The above PR satisfies deploying applications to a environment specific namespaces (ex: external-secrets-dev) that can manage secrets per bounded namespace (think argo-workflows-dev or kyverno-dev or argo-cd-dev)

I think I got the idea here, but I'm wondering why not just deploy multiple external-secrets from the current helm charts (i.e. multiple Argo applications bound to the same external-secrets helm charts)? We are not having any benefits as one ESO is not managing multiple namespaces, it is just multiple ESO, one per namespace. You would have better configuration option and actually be able to scale ESO per namespace, configuring leader election properly (AFAIK this proposed way wouldn't allow this feature to be used).

hey @gusfcarvalho I implemented what you suggested by using the nameOverride helm value. this works PERFECTLY and mitigates the need for the helm to auto-deploy multiple ESO. I will be opting to use your recommendations! Closing PR and issue. Thanks for all your help!

Glad I could be of help! :smile: Thank you for your interest in the project, and for your will to contribute to it!

We are not having any benefits as one ESO is not managing multiple namespaces, it is just multiple ESO, one per namespace.

Wouldn't it lead to waste of resources?
I'm trying to manage around ~30 namespaces with External Secrets Operator, but I don't have full Cluster level access. So I can't deploy ESO with Cluster level access, but at the same time I don't want to deploy to ~30 namespaces. Is there any recommendation for me? :)

hi @shinebayar-g ! the reason to have external-secrets controllers namespaced-bound is where there is a requirement for a given namespace to only have secrets reconciled by one single operator. It is a security requirement, not an operational one!
For your use case, you'll be fine in having one single ESO deployment. The only extra step that you would need to do manually is create Roles and Role Bindings (since cluster roles and cluster role bindings would most likely be disabled) to allow external-secrets operator in the namespaces that you want it to work. I don't think you really need controller isolation here, and if you do need it, multiple namespaces would most likely violate your use case as well.

Yep, after looking at resources and comparing what values are being changed between namespace vs cluster scoped I think that should work for us. Thanks.


Create SideMenu with ItemSource

I try to us ItemSource to dynamicly create Sidemenu under MVVM pattern, But it doesn't work.
Does sidemenu support it?
If yes, how to achive it?

Have you completed modifying this issue now？


Handling file downloads (CSV, content files, etc)

Helpful Notes:

https://stackoverflow.com/a/31128811

For now, we've hidden these buttons, which should be enough for 0.1.0 -- later we can reenable.


Cannot install tensorflow-gpu==2.3.0rc0

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04.5 LTS (Google Colab)
TensorFlow version: tensorflow-gpu 2.3.0rc0
Python version: Python 3.7.12
Installed using virtualenv? pip? conda?: pip
CUDA/cuDNN version: CUDA Version: 11.2. Cuda compilation tools, release 11.1, V11.1.105
GPU model and memory: NVIDIA Tesla K80

Describe the problem
I get the following error when trying to install tensorflow-gpu in version 2.3.0rc0:
ERROR: Could not find a version that satisfies the requirement tensorflow-gpu==2.3.0rc0 (from versions: 1.13.1, 1.13.2, 1.14.0, 1.15.0, 1.15.2, 1.15.3, 1.15.4, 1.15.5, 2.0.0, 2.0.1, 2.0.2, 2.0.3, 2.0.4, 2.1.0, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.4.0, 2.4.1, 2.4.2, 2.4.3, 2.4.4, 2.5.0, 2.5.1, 2.5.2, 2.6.0, 2.6.1, 2.6.2, 2.7.0rc0, 2.7.0rc1, 2.7.0)
ERROR: No matching distribution found for tensorflow-gpu==2.3.0rc0

Provide the exact sequence of commands / steps that you executed before running into the problem
pip install tensorflow-gpu==2.3.0rc0

@jmudy sir, you might have to make sure the tensorflow version you are trying to install is a valid version. The  reason for the above error is that the distribution you are  tryingto install does not exist. I would suggest you take a look at the current gpu distributions that are available here and install the version that you feel is appropriate for you. Also I would suggest taking a look at the tensorflow installation section for a smoother and easier installation experience.Feel free to let the community know about the progress on your tensorflow  issue here.Thanks !


Posting image or video using "data" returns 401

I have been using this lib for quite sometime, recently I updated and I am not able to post videos and images using data='local filename' param.
I checked the commit history but did not find anything particular.
I get 401 not authorized for both image and video data post.

Blank tags array got me too.. :( A non 401 would be more useful.


Original always gets saved when using -U with --from_file

Describe your environment

GPU: RTX 3090
VRAM: 24 GB
CPU arch: x86_64
OS: Windows 10
Python: micromamba
Branch: refactoring-simplet2i
Commit: a515af5c349a085d969dd1e514133212f0984bc0

Describe the bug
Option --save_original is not used when using -U with --from_file, the original is always saved and this can't be disabled.
To Reproduce
Steps to reproduce the behavior:

Run prompts from a file with upscaling enabled
See that 1 iteration produces 2 images

Expected behavior
It should not save the original images (unless --save_original is used)

Thanks. I’ll look into it.

I can't reproduce the bug in the current development branch, so I think it was fixed with the recent rewrite. This will be pulled into main in a couple of days. Please feel free to file a new issue if the bug appears again.


EXC_BAD_ACCESS randomly after the app launches

We've been seeing some crashes come in from TrustKit.
We're using the latest version of TrustKit. So far we've seen in iOS 13.4.1, 14.4.1, 14.4.2, and 14.5. It seems to only happen shortly after the app launches so it might be caused by the first network request. All the crashes we've gotten have been within the first 5-10 seconds of the app being launched.
The exception is
EXC_BAD_ACCESS
containsObject: > hashSubjectPublicKeyInfoFromCertificate: >
Attempted to dereference garbage pointer 0x20.

And here's the related part of the stack trace
0   libobjc.A.dylib                 0x348153c90         objc_retain
1   CoreFoundation                  0x31f9578f0         __NSSingleObjectArrayI_new
2   Security                        0x32da65278         SecTrustCreateWithCertificates
3   TrustKit                        0x1068c0710         -[TSKSPKIHashCache copyPublicKeyFromCertificate:] (TSKSPKIHashCache.m:260)
4   TrustKit                        0x1068bfd50         -[TSKSPKIHashCache hashSubjectPublicKeyInfoFromCertificate:] (TSKSPKIHashCache.m:175)
5   TrustKit                        0x1068bf9f0         verifyPublicKeyPin (ssl_pin_verifier.m:71)
6   TrustKit                        0x1068c6d10         -[TSKPinningValidator evaluateTrust:forHostname:] (TSKPinningValidator.m:126)
7   TrustKit                        0x1068c70e0         -[TSKPinningValidator handleChallenge:completionHandler:] (TSKPinningValidator.m:202)

Hello and thanks for the bug report. It could be due to unsupported certificates/algorithms that these users are seeing on the network (because of some kind of network device/router).
When we get to https://github.com/datatheorem/TrustKit/issues/236 , it would probably fix this crash (but the app still wouldn't work if the certificates are unsupported).


Upgrading "adjust" variable doesn't work

"Custom periods" in the adjust variable are not upgrading correctly. The current version upgrade script on the version-upgrade-updates branch has a debugging msgbox showing the issue. There seems to be an issue switching between different matrix types, [ [x,y,z] ] and [ [x], [y], [z] ] for the adjust_periods variable.
Using the attached .sam file (created in SAM 2022.11.21), see that the "custom periods" loss is not upgraded correctly.
upgrade.zip
Once this is fixed, code needs to be updated in the version upgrader script in 3 places, currently lines 71, 104, and 114, which is all the places that the adjust_periods variable is set.

adjust_periods is not converted correctly for some files. To replicate:

Open test .sam file: adjust-test-2022-11-21.zip
Run simulation.
See error

adjust_periods:fail(adjust_periods, cols=3): 1
Simulation pvsamv1 failed :adjust_periods:fail(adjust_periods, cols=3): 1
gen not assigned
Simulation grid failed :gen not assigned


Bencode structure

Created bencode structure
@eliaperantoni
@z1ko

Would it be ok to return const (?) references from the get_value method of the various BencodeValue subclasses instead of concrete values? I find myself wanting to read a parsed bencode tree but I'd rather not have a value be copied each time I take a peek inside. This is especially critical for dicts and lists.

I think that we should include all furrent code inside a project-wide namespace, for example:
namespace fur { namespace bencode {      code  } }
also written as namespace fur::bencode { code }

I think that we should include all furrent code inside a project-wide namespace, for example:
namespace fur { namespace bencode { code } }
also written as namespace fur::bencode { code }

That's a good idea. I'll update my WIP code with that.

Thanks for the suggestions, @z1ko I also approve the namespace I'll update as soon as possibile


testutil.h: Remove duplicate test macros

A block of six TEST_int_xy() macro definitions was duplicated.

Travis fault unrelated

Merged. e433ac3c4f6dda1c77bc245ca0abb56970f67d5a


Class DaggerCqlEvaluatorComponent missing

I am trying to run some test and not able to find - org.opencds.cqf.cql.evaluator.dagger.DaggerCqlEvaluatorComponent.

Found the classes in the Target folder


Consider a release bot on new changes

See https://github.com/cloudposse/terraform-aws-ecr/blob/master/.github/auto-release.yml and https://github.com/cloudposse/terraform-aws-ecr/blob/master/.github/workflows/auto-release.yml for inspiration

Release bot has been created in #47


Upgrade to Botbuilder V4

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: tomstarstuck sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


New link for offline docs (I'm Paul Kögel now)

I recently got married and changed my last name.

Woah, congratulations!

hehe, thank you :)


[Feature-Request] Offload Accessibility Service onListen processing to a separate thread and timeout

Its a common problem that accessibility service often leads to ANRs because of timeouts in onListen/locks(ANR triggered by main thread waiting for too long).
Do you think it would be better to offload the processing inside onListen to a separate thread with a timeout? If the Thread doesn't finish processing within a set time, timeout it and exit the onListen.
What does the author think?
Something like this
 @Override
    public void onAccessibilityEvent(AccessibilityEvent event) {

        Callable<Void> task = () -> {
            handler()
        };

        Future<Void> future = executor.submit(task);
        try {
            future.get(5, TimeUnit.SECONDS);
        } catch (TimeoutException e) {
            future.cancel(true);
            handleTimeout(event);
        } catch (Exception e) {

            e.printStackTrace();
        }
    }

Thank you for the suggestion and for highlighting the common issue . Yes offloading this to a separate thread with a timeout mechanism is a thoughtful approach to enhance the robustness of the application.
i will try this otherwise if u open a PL if you can


Commend Line Fix

New files update the cli.py and test file, update the readme file for clearness.

Why are you changing the binaries?
Also, this PR has conflicting files, so they cannot be merged.


using RPM macro scriptlets / BuildRequires

I'm trying and failing to follow the instructions on https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#Systemd .
Here's what I have:
post_install_script_flags = 0b011 # enable macros
post_install_script = "%systemd_post foo.service"

which results in:
/var/tmp/rpm-tmp.G9cgiK: line 3: syntax error near unexpected token `&&'
/var/tmp/rpm-tmp.G9cgiK: line 3: `if  && ; then '

Perhaps the problem is that I'm missing BuildRequires: systemd-rpm-macros, but there doesn't appear to be a way to add this with cargo-generate-rpm?

@bryanlarsen  I believe that korbel's answer resolved the issue. Could you please try it?


Release 1.0.9 to benefit from fix in Flutter-WebRTC

@cloudwebrtc  can you please take a look?
thanks

Closes https://github.com/flutter-webrtc/flutter-webrtc/issues/1102

lgtm


CSRF verification failed. Request aborted.

Hello developers, I encountered the following problems while deploying your program:
Forbidden (403)
CSRF verification failed. Request aborted.

More information is available with DEBUG=True.

This is what appeared after my successful deployment. The deployment method I used is docker-compose.
Here are my relevant documents:
.env:
# PostgreSQL 数据库连接信息
DB_NAME=shynetdb
DB_USER=shynet
DB_PASSWORD=password
DB_HOST=<IP_ADDRESS>
DB_PORT=5432

# 如果不需要邮件功能的话可以直接注释掉
# EMAIL_HOST_USER=example
# EMAIL_HOST_PASSWORD=example_password
# EMAIL_HOST=smtp.example.com
# EMAIL_PORT=465
# EMAIL_USE_SSL=True

# 默认为 *，建议修改为你准备部署的域名
ALLOWED_HOSTS=my.site.example

# 时区设置，可以修改为 Asia/Shanghai
TIME_ZONE=Asia/Shanghai

nginx.conf:
server {
    server_name my.site.example;
    access_log /var/log/nginx/bin.access.log;
    error_log /var/log/nginx/bin.error.log error;

    location / {
        proxy_pass http://shynet:8080;
        proxy_redirect off;
        proxy_set_header Host $http_host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header X-Forwarded-Protocol $scheme;
        proxy_set_header X-Url-Scheme $scheme;
    }
    listen 80;

}

docker-compase:
version: '3'
services:
  shynet:
    container_name: shynet_main
    image: milesmcc/shynet:latest
    restart: unless-stopped
    expose:
      - 8080
    env_file:
      # Create a file called '.env' if it doesn't already exist.
      # You can use `TEMPLATE.env` as a guide.
      - .env
    environment:
      - DB_HOST=db
    networks:
      - internal
    depends_on:
      - db
  db:
    container_name: shynet_database
    image: postgres
    restart: always
    environment:
      - "POSTGRES_USER=${DB_USER}"
      - "POSTGRES_PASSWORD=${DB_PASSWORD}"
      - "POSTGRES_DB=${DB_NAME}"
    volumes:
      - shynet_db:/var/lib/postgresql/data
    networks:
      - internal
  webserver:
    container_name: shynet_webserver
    image: nginx
    restart: always
    volumes:
      - ./nginx.conf:/etc/nginx/conf.d/default.conf
    ports:
      - 3000:80
    depends_on:
      - shynet
    networks:
      - internal
volumes:
  shynet_db:
networks:
  internal:

here is some related picture.The same is true on the computer.

Looking forward to your reply, I would appreciate it. think you very much！

Hi. I am facing the same problem. I think it happened when I upgraded to the latest version.

I have solved it. I did not configure the parameter CSRF_TRUSTED_ORIGINS during the installation process. After configuring it, this problem did not occur again.

Can confirm that it worked, I was missing CSRF_TRUSTED_ORIGINS.

Great; glad that you were all able to resolve the issue.


Not working well with Doors on Diagonal walls

None of these players can open the door (it's unlocked).

Foundry VTT Version: 9 build 245
Windows
Chrome
dnd5e

I can't solve it, a temporary solution is to make the tokens move by holding the "Shift" button to disable the "snap to grid" functionality and move the token those 2px necessary when opening the door

I faced this issue and sorta "hacked" a solution. I changed the setting to allow steps of 1 instead of 5. Then changed interaction distance to 9. This way it "catches" diagonal doors, but won't work two squares away.
Not sure if I will face any errors for fiddling with the files but it works for now. :)


Stable

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 6 committers have signed the CLA.:white_check_mark: keenondrums:white_check_mark: dhh1128:x: rajeshkalaria80:x: mzk-vct:x: devin-fisher:x: MRJCrunchYou have signed the CLA already but the status is still pending? Let us recheck it.


Example Dockerfile usage

I'm trying to use a Dockerfile with GenericContainer, but can't seem to get it to work. The unit test shows it using .withFileFromClasspath(), but my Dockerfile is not on the classpath. I tried .withFileFromFile(), but it errors out with something like:
{"message":"No such image: testcontainers/b3ewhisadljlsccu:latest"}

Are the other .withFileFromXXX()'s supported?

Hi @miketwo
All .withFileFromXXX() methods should work. Since they are coming from the parent Java lib, If they don't, most probably the problem is in upstream library.
Could you please share more detailed code snippet with the paths you pass to  .withFileFromFile()


Running eval multiple times to reduce LLM randomness

Not sure where to put this as it's related to both LangChain and LangSmith, but...
I want to run evals (runOnDataset) multiple times over my dataset, and see all of those samples as part of one eval (aka I don't want X different eval runs). Ideally the scores in LangSmith would be averaged. For example, I have an eval that passes 3/10 times. If I don't sample, my overall eval result is highly unreliable. Ideally I'd see this as {evaluator key}: 0.3 in LangSmith

@jacoblee93 I found myself needing this again today. What do you think directionally of this solution?

RunOnDatasetParams: add option n or similar for how many times to run the eval
Around https://github.com/langchain-ai/langchainjs/blob/ddc44821b6c1e81c827332cf20468a9a9b19e719/langchain/src/smith/runner_utils.ts#L706, wrap wrappedRunnable in RunnableEach to run n times per sample
I don't really get how runExtractors works or how it gets its data, but I'm guessing from step 2 it would now get runs with an array of results rather than the result directly
Around https://github.com/langchain-ai/langchainjs/blob/ddc44821b6c1e81c827332cf20468a9a9b19e719/langchain/src/smith/runner_utils.ts#L546-L547 make the evaluator run n times for each result, and then average the score across those evals

Not sure how much time I'll have, but if you can guide me and point me to how you'd build this, I might be able to take a stab at it

CC @hinthornw would this make sense?

Ok, I have been told:
That's how we first implemented it, but it makes comparison view weird.

I think it makes sense to add if we can unblock the backend /UI for it.

FYI this is how we currently do it 🙈


[VL] Fix shuffle spill not reported to spark metric

Will have Spill (Memory) and Spill (Disk) shown on UI

cc: @zhztheplayer Seems like spill from Velox pipeline is not reported to the TaskMetrics either.

cc: @zhztheplayer Seems like spill from Velox pipeline is not reported to the TaskMetrics either.

Hi @marin-ma, would you like to file a issue for that? It was a known issue on which we are lack of tracking. Thanks.

Hi @marin-ma, would you like to help file a issue for that? It was a known issue on which we are lack of tracking. Thanks.

@zhztheplayer Filed issue here https://github.com/apache/incubator-gluten/issues/6763


Better constructors

I think the image initializing functions should be more readable and explicit:
    //  image.create() or regular constructor creates a new image and fills it with zeros.
    //  image.createUninitialized() or `setSize` create a new uninitialized image
    //  image.createWithNoData(); create an image with no data
    //  image.createViewFromData(); creates a view from existing data (would need a clone to own)

'kay


Update User Stories Backlog for Sprint 3

To do:
-Update acceptance criteria
-S2 markers
-Building Poly Color
-Add visual trajectory

All these items were added to zenhub, make sure they're reflected on the doc. @waynet022 @kajanthyS


 as command prefix as core and all other packages do.

Thanks


Removing last item from the last page of Grid does not navigate back to the new last page

Describe the bug
User experience is incomplete if user manipulates the contents of a Grid.
To Reproduce
Steps to reproduce the behavior:

Click "Add Employee" button
Navigate to the 2nd page
Click the "Remove Employee" button
Pagination updates to have only one page but does not navigate back to it. Numbering of items is also wrong.

Expected behavior
When the last element of the last page gets removed, user should be navigated back to the "new" last page and page numbering should be updated.
Screenshots

Versions (please complete the following information):

.NET Version: .NET 8
BlazorBootstrap: 2.2.1
Blazor Interactive Render Mode: Auto
Blazor Interactivity Location: Global

Sample code
<Button Type="ButtonType.Button" Color="ButtonColor.Primary" @onclick="AddEmployee"> Add Employee </Button>
<Button Type="ButtonType.Button" Color="ButtonColor.Danger" @onclick="RemoveEmployee"> Remove Employee </Button>

<Grid @ref="grid" TItem="Employee1" AllowPaging="true" PageSize="5" class="table table-hover table-bordered table-striped mt-3" Data="employees">
    <GridColumn TItem="Employee1" HeaderText="Id" PropertyName="Id">
        @context.Id
    </GridColumn>
    <GridColumn TItem="Employee1" HeaderText="Employee Name" PropertyName="Name">
        @context.Name
    </GridColumn>
    <GridColumn TItem="Employee1" HeaderText="Designation" PropertyName="Designation">
        @context.Designation
    </GridColumn>
    <GridColumn TItem="Employee1" HeaderText="DOJ" PropertyName="DOJ">
        @context.DOJ
    </GridColumn>
    <GridColumn TItem="Employee1" HeaderText="Active" PropertyName="IsActive">
        @context.IsActive
    </GridColumn>
</Grid>

@code {
    Grid<Employee1> grid = default!;
    private List<Employee1>? employees;

    protected override void OnInitialized()
    {
        employees = new List<Employee1> {
            new Employee1 { Id = 107, Name = "Alice", Designation = "AI Engineer", DOJ = new DateOnly(1998, 11, 17), IsActive = true },
            new Employee1 { Id = 103, Name = "Bob", Designation = "Senior DevOps Engineer", DOJ = new DateOnly(1985, 1, 5), IsActive = true },
            new Employee1 { Id = 106, Name = "John", Designation = "Data Engineer", DOJ = new DateOnly(1995, 4, 17), IsActive = true },
            new Employee1 { Id = 104, Name = "Pop", Designation = "Associate Architect", DOJ = new DateOnly(1985, 6, 8), IsActive = false },
            new Employee1 { Id = 105, Name = "Ronald", Designation = "Senior Data Engineer", DOJ = new DateOnly(1991, 8, 23), IsActive = true }
        };
    }

    private async Task AddEmployee()
    {
        // for the same employees collection, we are adding an object
        // explicit grid refresh required
        employees!.Add(CreateEmployee());
        await grid.RefreshDataAsync();
    }

    private async Task RemoveEmployee()
    {
        employees!.Remove(employees.Last());

        await grid.RefreshDataAsync();
    }

    private Employee1 CreateEmployee()
    {
        var emp = new Employee1();
        emp.Id = employees!.Max(x => x.Id) + 1;
        emp.Name = $"Employee {emp.Id}";
        emp.Designation = $"QA Engineer {emp.Id}";
        emp.DOJ = new DateOnly(new Random().Next(1970, 2000), new Random().Next(1, 12), new Random().Next(1, 25));
        emp.IsActive = true;
        return emp;
    }

    public record class Employee1
    {
        public int Id { get; set; }
        public string? Name { get; set; }
        public string? Designation { get; set; }
        public DateOnly DOJ { get; set; }
        public bool IsActive { get; set; }
    }
}

Desktop (please complete the following information):

OS: Windows 11
Browser: Edge
Version: 124.0.2478.51 (Official build) (64-bit)

@laurentiustamate94 Thank you for using BlazorBootstrap. We will investigate your scenario.


Visual Studio functions not created

The Visual Studio Functions (vs, vsadmin, and Install-VSExtension) are not registered when running PoSh.

Fixed in 2f95f1865ddda8f50109c24aeafec513abf8bcf4


WasapiLoopbackCapture for ASIO ?

Hi
I have been testing WasapiLoopbackCapture  and it works really well - however is there anyway to capture the output of another process when its using ASIO rather thatn Waspi?
I suspect there is not but wanted to check, thanks!

Were you able to capture audio of a single process using wasapi to begin with? To my knowledge that isn't possible with NAudio currently.


X-Axis grid line align left

G2 Version: ^4.0.15

Platform: macos

Mini Showcase(like screenshots):

CodePen Link: NA

How do I align the vertical x-axis to the left.
chart.axis("xAxis", {
      tickLine: null,
      line: null,
      grid: {
        line: {
          type: "line",
          style: {
            stroke: "#E1E1E1",
          },
        },
      },
    });

For the funnel chart
if I do alignTick: false, then the grid line goes to right, but is there a way to make it aligned to left?
Any help is appreciated. TIA

{
  label: {
    style: { textAlign: 'center' }
  }
}


refactor: netout property into a LinkList

We may want to use Hashable instead of Linkable - the original Packet source extended Hashable which extends Linkable
During Packet.release() -> .removeHead() affects the .next/.prev from Linkable due to it calling unlink. Linkable can only be part of 1 LinkList if I understand correctly? So if we want a LinkList on the player and a LinkList to pool allocations there's a good chance we won't be reusing memory as often.
Double check me - if I'm right then it needs to extend Hashable and we need a different container - either a 2nd LinkList that uses Hashable or maybe this Stack class? https://github.com/2004scape/Client2/blob/main/src/js/jagex2/datastruct/Stack.ts

Hashable change was reverted, good. If we want this PR to get in then the container still needs to be changed from LinkList to something i.e. Stack for reasons above.


Android Basics: Fragments Navigation component

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-fragments-navigation-component?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-fragments-navigation-component#3
In which task and step of the codelab can this issue be found?
4. Create Fragment and layout Files
Describe the problem
"tool:context" for "fragment_letter_list.xml" is ".WordListFragment" rather than ".LetterListFragment".
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

@M-Phuykong
Thank you for reaching out to us.
This should be fixed now.


weight prediction baselines

paper
Prediction of Links and Weights in Networks by Reliable Routes
Introduction
For a weighted undirected graph where links with higher weights represent stronger relations between nodes (e.g., interactions between proteins, genes, authors, neurons), weight prediction is an important task.
problem

given: the weights of links in a training set
task: predict the weights of links in the test set to minimize RMSE

solution

for each node pair in the entire data set, calculate the similarity of the pair (9 variants of similarity evaluated in experiments)
let a linear regression model weight = F(similarity) = k * similarity learn on the training set
let model F predict on the test set

comment

The description of the learning (or testing) process in the paper (the paragraphs around equation 3) seems wrong, because the model learns on the test set. This is a not a correct learning process or testing process. I didn't find a correct description in the paper so I made one up.
Each of the 9 variants of the pair similarity is a sum of "strengths" of all 2-link paths connecting the pair. The "strength" of a path is a sum or product of weights of links in the path, normalized by some factor. However, it is unclear why the paper only considers 2-link paths, but not 1-link, 3-link, 4-link ones.
The paper states this approach does not work well for graphs where higher weights represent weaker relations. Is this because dissimilarity is not additive?

I followed the paper's references to related work on weighted graph, but findings were little. This one paper is on weight prediction: http://comnet.oxfordjournals.org/content/early/2014/06/26/comnet.cnu026.short
but unfortunately our school doesn't have subscription for it. All the other papers are not on weight prediction. Will expand my search later.

Looks like an early version of the paper is available at http://arxiv.org/abs/1404.0431.

paper
Learning Latent Block Structure in Weighted Networks
Introduction
A weighted directed heterogeneous graph can have a 2-level hierarchy:

upper level: the heterogeneous graph is a set of homogeneous blocks connected by bundles
lower level: a homogeneous block is a set of nodes with similar connections connected by links

Solution

the model contains parameters including membership (the block each node is in) and bundles' weights
input: a pair of nodes
output: the weight of the link connecting the pair of nodes

Comments

too much statistics, didn't read all
this solution has a side effect of clustering nodes into blocks by assigning membership, a useful thing by itsefl

found the implementation of WSBM
need to figure out how to run it

paper
Weight prediction in complex networks based on neighbor set
problem
Weight link prediction, same as above.
solution
Not a machine learning approach, very hard to understand. The assumption it is based on is Our method relies on the assumption that the formation of link weights is regulated by local clusterings in which homogenous links tend to have similar weights. also very hard to understand.


Is it possible to change this for multiple clients?

Hi!
This project is really awesome.
Code is very clean, nice and working very well 😀
For now, this project allows only 2 clients.
Is it possible to change this for multiple clients?

Hi, thank you for your awesome comment and question :)
Actually, I'm working on webrtc full-mesh connection library named Peers. Now, Peers is done for server and web client with vue.js but not public repository. I'm ready for documents. I'm planning to publish Peers in early August.
After I publish Peers, I will make example of Peers implementation in this project.
If you want to know about WebRTC connection topology, refer here
After I created example fo Peers implementation, I will try to create SFU server and example client of that server. However, in this case, I'm not sure when I will work on it.
Thanks.

Oh, I See.
Actually, I'm also working on WebRTC connection for multiple peers.
As an iOS developer, it's kind of difficult but I'm having fun with it.
I'd love to contribute if you publish Peers!
Thank you so much for your comment ✋


Rewrite yet again of Discover methods

Follow-up on pr #130

@Naliath see this. Follow-up on #130

Looks good!


Never do work on the JXcore thread in iOS

We have to audit our code to make sure that when we get called on callNative we never do any work on the JXcore thread but instead schedule our work on a different thread, return the callNative call immediately and then later call the registered callback from the other thread.

@yaronyg, @tobybrad: Not sure if this is relevant to this issue, but when I start the unit test app on iOS, I noticed something like this gets printed:
2016-02-23 09:12:20.528 ThaliTest[661:892834] THREAD WARNING: ['JXcore'] took '10247.144043' ms. Plugin should use a background thread.

A longer debug output with more context at https://gist.github.com/vjrantal/d939a743b693eeacbf63.

One wonders if we initialize JXcore on the main thread.


Added Spanish Translations

Improved: Added "es" and "es-MX" values and Mexican Federal holiday schedule
Implemented:
Documented:
Completed:
Reverted:
Fixed: Corrected Abbreviations of Mexican States
(OFBIZ-)
Explanation
Added multiple "es" and "es-MX" values to framework/common/config/CommonEntityLabels.xml  and framework/common/config/CommonUiLabels.xml
Corrected Abbreviations of States on framework/common/data/GeoData_MX.xml according to the national usage.
Appended Mexican Federal holiday schedule to framework/service/data/ServiceDemoData.xml under the TemporalExpression "MX_FED_HOLIDAYS"
Atentamente: / Best Regards: / Atenciosamente: / Cordialement:
Zottacko WallyMan

This appears good enough.

This pull request seems already present on trunk, committed by Jacques (https://github.com/apache/ofbiz-framework/commit/17de4f6e0780c842b5e2114137015afeada0c56d), I think we can close it

Thanks Zottacko WallyMan!


Don't add external module completions in a pure commonjs file

Fixes #22574

I know @RyanCavanaugh has thoughts on this.

I'm fine with a bugfix for not including them since the current behavior is wrong, but we could also consider synthesizing
const { readFile } = require('fs');

as a stretch goal at a later date.

Filed #22602


Support "do not populate" and variants

The Interactive HTML BOM  plugin is very nice. But I have noticed some features missing, so I kindly ask for this wishlist item.
I would like to see the Interactive HTML BOM tool to be able to hide entries marked as "do not populate" in the schematic.
I am using KiCost and here one can mark it with DNP something. See the doc https://kicost.readthedocs.io/en/latest/usage.html#do-not-populate-parts
On top of this it could be useful if it would support the variant scheme used by KiCost as well. But this might be too much for this issue. See https://kicost.readthedocs.io/en/latest/usage.html#schematic-variants
I have not investigated other methods used to mark this information from other BOM tools.
I am currently on 401bc2831ac717a061de255e2ca18d76f7365263.

Both of these are already on the wish list, see #15 and #51.
KiCost and KiBom treat variant a bit differently and (of course) have different fields. I will add something that should work for most cases for users of both KiCost and KiBom but it won't be identical implementation.

Ahh, ok. I did try to look through the issues, but I didn't catch their terminology. Good to see you intend to support it in the future. Thanks.


feat: prewired added for es->kms

Description
Prewired rule added for es->kms.
Release required?
https://github.com/terraform-ibm-modules/terraform-ibm-cbr/issues/404

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

I suggest - Messagehub (Event Streams) should be the other way Event Streams (Messagehub), because the external name of the offering is Event Streams. Hence the variable name, rule name, readme should contain event streams instead of messagehub for better understanding.

I suggest - Messagehub (Event Streams) should be the other way Event Streams (Messagehub), because the external name of the offering is Event Streams. Hence the variable name, rule name, readme should contain event streams instead of messagehub for better understanding.

We have been following the same for other services as well such as IS (VPC Infrastructure Services) https://github.com/terraform-ibm-modules/terraform-ibm-cbr/blob/e13f05d129fa743e01ba4b99b0632e64c6d9486e/modules/fscloud/main.tf#L239 and considering this module is being used by customers, changing the variable names now may have cascading effect.

/run pipeline

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


Remove New Relic support from oc-id

We do not use New Relic at this point and we still have to package and update this gem continuously. There's no reason to include the gem:
https://github.com/chef/chef-server/blob/main/src/oc-id/Gemfile#L21

@saghoshprogress this should be fixed in https://github.com/chef/chef-server/pull/2993

This is done


feat: switch from pycryptodomex to cryptography

This should be functionally complete and the tests pass, but I don't have a way to test the change in production


Fix race condition when removing series from a shard

cc @robskillington
Currently there is a race condition in the shard.Tick() method. Basically it first iterates over all the series, finds out all the ones that are empty, acquires the shard lock, and removes them from the shard. AFAICT there are two issues that could cause a series with recent data to be removed from the shard:

If a series is written to after it's determined to be empty when we iterate over all series in Tick
Consider the following sequence of actions:

all datapoints for series "foo" have expired
a new datapoint for "foo" comes in, we find the corresponding series by calling series := shard.series(id), but haven't called series.Write(...) yet.
In the meantime, shard.Tick() starts, finds "foo" to be an empty series, and removes it from the shard.
series.Write(...) now runs
The end result is that "foo" has been removed from the shard, even though it has recent datapoints.

This diff attempts to fix the issues by:

keeping track of the number of active writers to a series so we don't remove a series if it's being written to
checking if the series is empty inside the shard lock, instead of relying on the possibly outdated information (i.e, the expired slice in Tick) from when iterating over the series.

Also added test cases to capture those cases.

Coverage increased (+1.2%) to 76.619% when pulling 183d5762456a928a0cf2b4e436d215a1e6e7ec13 on xichen-fix-race into 38df5898d91d3cfdee80562a6440198ef364b747 on master.

LGTM, yeah it was kind of "lossy" before


Update README with latest modifications

Some modifications has been made in order to clean the html output to be less ugly and include style to highlight code as in Jupyter nb.
Include these modifications in the readme.

Updated.