How to read minidumps - Extras

Include extra stuff such as: !devstack, !kb, !dpcwatchdog, !sysinfo smbios - And other advanced troubleshooting techniques to quickly identify the faulting driver from minidumps

Quote:
"if I recall, with bugcheck 0x9f and parameter 1 being 0x3, just running !devstack on parameter 2 is sufficient
or was it parameter 3"


Please add new waterfowl and nest.

This is a very good mod. Do you plan to join other waterfowls? I hope you can make a nest for waterfowls, like animania farm, to let waterfowls lay eggs in it and hatch their babies.thank you！

There is another mod in the works which adds swans(not mine though and probably not for fabric) and nests are already kinda covered in #7. I don't have the time to add new waterfowl at the moment but maybe in a future update there will be some.


STY: Fix PEP8 - E712 linting error (fixes tests)

Was causing some build failures on the linting step. This is one easy way to fix it. This assumes the comparison to True as opposed to using __nonzero__ is important.

Tests pass.

Thanks :)

No problem.


Support TRL's RLOO

As discussed in #515 , this is an RLOO implementation that follows TRL. Compared to TRL, OpenRLHF features a modern vllm inference engine, making it a more efficient reinforcement learning implementation. Therefore, this Pull Request aims to provide users who wish to use the TRL RLOO algorithm with a more efficient option.

openrlhf_train_ppo
It should be noted that, since RLOO treats an entire response as a single sentence, the shape of its rewards and advantages is [batch size, 1], which differs from the [batch size, seq len] shape of advantages in PPO. To maintain compatibility with the framework, in this implementation, after computing the advantages, they are placed into a zero tensor of shape [batch size, seq len], with the insertion point being the end of the sentence (determined by the action mask).
Another difference is that RLOO's advantages do not require normalization across the batch. When calculating advantages, RLOO inherently performs a certain degree of normalization.
Finally, when computing RLOO's loss, it is necessary to use floats and apply clipping in advance to prevent numerical overflow.

Thank you for the suggestion. The rollout batch size is set relatively small due to limited computing resources. I will try to scale up soon.

@hijkzzz Hi, here come the results:

openrlhf_train_ppo
The return of rloo with per token KL(orl_rloo) looks different, which makes me curious.
Additionally, previously setting init_kl_coef=0.001 https://github.com/OpenRLHF/OpenRLHF/pull/515#issuecomment-2514353697 caused the KL to spike, rendering the experimental results meaningless, so I used 0.01 instead.


✅ make tests pass with Puppet 6

Coverage remained the same at 98.684% when pulling 8fcf31b5d3f2db52127983a144d81712fd29bdb2 on raphink:puppet6 into f993160df4c4a73a96c1b2dd33deb5cb66ef1666 on hercules-team:master.

Coverage remained the same at 98.684% when pulling 8fcf31b5d3f2db52127983a144d81712fd29bdb2 on raphink:puppet6 into f993160df4c4a73a96c1b2dd33deb5cb66ef1666 on hercules-team:master.

Coverage remained the same at 98.684% when pulling 8fcf31b5d3f2db52127983a144d81712fd29bdb2 on raphink:puppet6 into f993160df4c4a73a96c1b2dd33deb5cb66ef1666 on hercules-team:master.


local_planner_stereo.launch fails to run

Directly attempting to run this launch file resulted in an error in looking up the px4 launch file, which makes sense because there is no px4.launch file in the PX4 firmware folder.
So I figure the launch files are probably just a little out of date and I need to update some directories.  I change it to look for px4.launch in mavros, where it does find it, but it then complains about multiple instances of mavros running.  Indeed, px4.launch does start its own mavros node, so I tried to consolidate all of the launch files into a single file to try running the software
local_planner_stereo.txt.
I believe this has the correct configuration, but I now run into strange errors from the parser:
Error [parser.cc:689] Unable to find uri[model://iris_depth_camera]

<gravity>0 0 -9.8</gravity>

<magnetic_field>6e-06 2.3e-05 -4.2e-05</magnetic_field>

<atmosphere type='adiabatic'/>

<physics name='default_physics' default='0' type='ode'>

  <max_step_size>0.001</max_step_size>

  <real_time_factor>1</real_time_factor>

  <real_time_update_rate>1000</real_time_update_rate>

</physics>

<scene>

  <ambient>0.4 0.4 0.4 1</ambient>

  <background>0.7 0.7 0.7 1</background>

  <shadows>1</shadows>

</scene>

[INFO] [1536011813.196405, 10.104000]: Spawn status: SpawnModel: Entity pushed to spawn queue, but spawn service timed out waiting for entity to appear in simulation under the name iris
[spawn_model-11] process has finished cleanly
log file: /home/chuples/.ros/log/1c59b90e-afc4-11e8-a497-dcfe07d69cab/spawn_model-11*.log
^C[gazebo_gui-10] killing on exit
[gazebo-9] killing on exit
[local_planner_node-6] killing on exit
[rqt_reconfigure-7] killing on exit
[rviz-4] killing on exit
[mavros-8] killing on exit
[stereo/stereo_image_proc-5] killing on exit
[tf_90_deg-2] killing on exit
[tf_depth_camera-3] killing on exit
Warning: Invalid argument passed to canTransform argument source_frame in tf2 frame_ids cannot be empty
     at line 126 in /tmp/binarydeb/ros-kinetic-tf2-0.5.17/src/buffer_core.cpp

[local_planner_node-6] escalating to SIGTERM
[gazebo-9] escalating to SIGTERM
[rosout-1] killing on exit
[master] killing on exit
shutting down processing monitor...
... shutting down processing monitor complete

Please update the PX4Firmware to latest master, the px4.launch file will be there.

You probably didn't follow the readme setup instructions to add the avoidance repo to GAZEBO_MODEL_PATH.

Okay, thanks mhkabir, I did miss those steps.  Everything seems to run now.


Swapping left alt with ctr, broke right alt

Microsoft PowerToys version
0.69.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
I use a mac at work and become frustrated with different cmd position on Windows. I tried to use Power Toys to remap buttons and remapping left control to alt works great, but when I remap left alt to left control, my right alt stop working. I'm unable to use language-specific characters now. What is weird, key works correctly in WhatsUp application.
✔ Expected Behavior
Right alt should work as before
❌ Actual Behavior
Right alt don't work.
Other Software
No response

https://user-images.githubusercontent.com/12560152/235381256-39454715-05dd-4e55-b8f3-3dd3ba6f2b7d.mp4

Without seeing a the on-screen keyboard, bit hard to understand what is being typed.  But at the 30 second marker, you didn't actually click "OK".
With the following, this works on my computer.  Alt and Ctrl are flipped on my left hand.  Delete them and click ok, and it works.
The scenario you outlined was one of the big ones we had.  Make sure mac people can feel at home typing.

/needinfo


⚠ Fortnite has degraded performance

In 9fe1fd2, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1505 ms

Resolved: Fortnite performance has improved in 6300956.


Created IncludeAll method

Created an "IncludeAll" method to include all references to other tables.

Hi, I don't get it why change GetField method and do not use FieldName from member mapper.
Also, will be great an unit test included.
I'm do not write any code-style rule or setting editor config yet, but I recommend to follow code style rules, like:

always use var
any public method must have, at least, <summary> (sometimes in private method I add too)
inside code, comments must be single line and always use lowercase (Capitalize only in <summary>)
lambda expression with single char variable like:  items.Query(x => x.Name) (i always use x first)
no extra spaces in (, )


V3 Role Management

Here's a list of things this has planned, and the status of each

[X] Role criteria
[X] Role searches
[X] Self Role assignment by command
[X] Self Role assignment by reactions
[X] Mass Role assignment with criteria
[x] Per user forbidden role settings (prevent specific users from gaining a role)

With #87 finished, The remaining tasks here are going to be significantly easier to finish work on without introducing errors.
Reaching out to various parties on what they would want to see for automatic role assignment criteria along with a few things of my own based on time in server + activity levels in server.

I've not found a clean way to handle the hidden role part yet. I have a couple more things I can explore with that later tonight, but that particular part might be something handled later.
I'm starting work on the conditional automatic role grant tonight.

@HelmicNewciv
Here's the current plan for handling automatic role grant:
Upon reaching any combination of the below criteria set:
Number of qualifying messages*
Amount of time in server
Amount of time spent in voice chat (niche, but easy enough to add, and has been asked for)
Qualifying messages can have a minimum length, channel restrictions, as well as a maximum amount of them that can qualify in a period (to prevent spam -> role).
I'm looking to ensure this gets cleaned up and finished prior to a stable release of version 3 of Red, so sooner than later, but since I've been helping with that push as well, I have spent less time than I originally intended on this.

That looks good!  Just to be sure, will there be a way to blacklist individual users before they meet any of those requirements?  Being able to spot and mark problematic users ahead of time would keep them from accessing sensitive channels without outright banning them altogether.

I suppose the other feature that would be nice is a way to DM the user first and have them agree to rules before gaining access.  But then I start thinking about just wanting a general way to use these sorts of filters for other moderation-related or memey tasks.  I believe there were concerns about performance issues for making this sort of functionality part of core?

1daea75 Just ticked off the ability to forbid a user from self-assigning a role (or from gaining it via earned role criteria) I'm considering adding an additional stricter setting which would also prevent mods from adding that role to that user, but I'm trying to keep this cog from clogging bot events too, so I'l;l have to give that a bit more thought.

@HelmicNewciv I have a working implementation of the criteria roles, but it's going to need rewriting some parts to avoid some really draining performance considerations. likely won't be available for a bit, as I'm going to prioritize a few other cogs over implementing this feature.

I'm marking this as done (for now).
The remaining features can all be added in the future without the need to call this a beta.


Update working group page with content about submissions

What
Add content about submissions to Community page about the working group.
Why
Our Drive contains a doc about what should, and should not, go to the working group. It's only had a few viewers since a prior team-member created it. If this info were visible publicly, it could tell more users about the scope of the working group.
Who needs to know about this
Community Manager, Content Designer, Delivery Manager, Developers, Senior Product Manager, Technical Writer
Done when

[ ] Content drafts update
[ ] Team reviews update
[ ] Update gets 2i, if needed
[ ] Update published

Also, the second last sentence in the 'Join the Working Group' section capitalises the initials in 'working group' - but we display this text in lowercase everywhere else on this page.


Added animationDuration parameter to showMenu and PopupMenuButton.

List which issues are fixed by this PR. You must list at least one issue.
Fixes: #135638
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

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

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

After this PR and similar request for other widgets such as ExpansionTile
We're adding an AnimationOverride class to override such animation properities for widgets including `PopupMenuButton in https://github.com/flutter/flutter/pull/137945
Closing this as a result.


Increase upper bound to fix TestController_RotateCertificates unit test

The TestController_RotateCertificates fails frequently in Windows OS, The delta is usually around 11s, so increase the upper bound to 12s to fix the flaky test.

After sync with @xliuxu , he proposed an alternative way to fix this flaky test, I will keep this PR until he verify and submit a new PR.


"Not Enough Memory" at startup on Raspberry-Pi Pico

Hi,
Just for your information.
I did install MWS2 on a Raspberry-Pi Pico and getting Memory issue when starting the main.py example.
This error is always happening almost every time (the other time, it starts).
I will have a look to the previous version.
Additional information:

I'm running last nightly build of MicroPython (currently 1.19.1 205)
Free Mem @ startup : 153296 bytes
Free Mem after error : 52544 bytes
Running gc.collect() hangs everything

Here the error:
>>> import main_test

       ---------------------------
       - Python pkg MicroWebSrv2 -
       -      version 2.0.6      -
       -     by JC`zic & HC2     -
       ---------------------------

 + [@WebRoute] GET /test-redir
 + [@WebRoute] GET /test-post (TestPost1/2)
 + [@WebRoute] POST /test-post (TestPost2/2)

MWS2-INFO> Server listening on <IP_ADDRESS>:80.
MWS2-INFO> Starts the managed pool to wait for I/O events.
MWS2-INFO> Server <IP_ADDRESS>:80 closed.
MWS2-INFO> Stops the managed pool.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "main_test.py", line 170, in <module>
  File "/lib/MicroWebSrv2/microWebSrv2.py", line 223, in StartManaged
  File "/lib/MicroWebSrv2/microWebSrv2.py", line 220, in StartManaged
MicroWebSrv2Exception: Not enough memory to start 1 parallel processes.

Just work pretty well with the MWS version 1 available at
https://github.com/jczic/MicroWebSrv

Build it as a module in your own micropython firmware then only it may work

In my experience, uPy is a memory hog. I've had to either compile frozen modules (include in custom firmware) or use the mpy-cross tool to make .mpy files.
https://forum.micropython.org/viewtopic.php?t=4510

@BetterAutomations 👍🏻
That's right.


fix typos, spelling and grammar

Commit Message: fix typos, spelling and grammar
Additional Description: mklein is bad at spelling
Risk Level: low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@billgallagher-okta could you fix DCO please

/retest

/wait

@billgallagher-okta this PR will now need a main merge to resolve conflicts - it would be great to land this - only a few tweaks required

kindly ping @billgallagher-okta


[BUG] creating Anomaly Detector using the API does not work

Describe the bug
The anomaly detection panel in opensearch dashboards  fails after creating a detector via the API("Unable to get all detectors").
And it is also impossible to get/update/delete a detector created using the API.("Unable to get user information from detector")
To Reproduce
Steps to reproduce the behavior:

Create a detector using the create detector API, not via the plugin UI
Open Dashboards and navigate to the anomaly detection dashboards plugin UI

OpenSearch Dashboards logs:
Anomaly detector - Unable to search detectors { Error: [illegal_state_exception] Can't get text on a VALUE_NULL at 1:51
    at respond (/usr/share/opensearch-dashboards/node_modules/elasticsearch/src/lib/transport.js:349:15)
    at checkRespForFailure (/usr/share/opensearch-dashboards/node_modules/elasticsearch/src/lib/transport.js:306:7)
    at HttpConnector.<anonymous> (/usr/share/opensearch-dashboards/node_modules/elasticsearch/src/lib/connectors/http.js:173:7)
    at IncomingMessage.wrapper (/usr/share/opensearch-dashboards/node_modules/lodash/lodash.js:4991:19)
    at IncomingMessage.emit (events.js:203:15)
    at endReadableNT (_stream_readable.js:1145:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)
  status: 500,
  displayName: 'InternalServerError',
  message:
   '[illegal_state_exception] Can\'t get text on a VALUE_NULL at 1:51',
  path: '/_plugins/_anomaly_detection/detectors/_search',
  query: {},
  body:
   { error:
      { root_cause: [Array],
        type: 'illegal_state_exception',
        reason: 'Can\'t get text on a VALUE_NULL at 1:51' },
     status: 500 },
  statusCode: 500,
  response:
   '{"error":{"root_cause":[{"type":"illegal_state_exception","reason":"Can\'t get text on a VALUE_NULL at 1:51"}],"type":"illegal_state_exception","reason":"Can\'t get text on a VALUE_NULL at 1:51"},"status":500}',
  toString: [Function],
  toJSON: [Function] }

OpenSearch logs(when trying to delete the detector):
[2022-01-17T16:14:43,040][ERROR][o.o.a.u.RestHandlerUtils ] [opensearch-node] Wrap exception before sending back to user
org.opensearch.ad.common.exception.AnomalyDetectionException: Unable to get user information from detector QXycaH4BI3aocvnW0ms2
	at org.opensearch.ad.util.ParseUtils.onGetAdResponse(ParseUtils.java:561) [opensearch-anomaly-detection-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
	at org.opensearch.ad.util.ParseUtils.lambda$getDetector$0(ParseUtils.java:517) [opensearch-anomaly-detection-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
	at org.opensearch.action.ActionListener$1.onResponse(ActionListener.java:78) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.TransportAction$1.onResponse(TransportAction.java:103) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.TransportAction$1.onResponse(TransportAction.java:97) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.single.shard.TransportSingleShardAction$AsyncSingleAction$2.handleResponse(TransportSingleShardAction.java:287) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.single.shard.TransportSingleShardAction$AsyncSingleAction$2.handleResponse(TransportSingleShardAction.java:273) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.security.transport.SecurityInterceptor$RestoringTransportResponseHandler.handleResponse(SecurityInterceptor.java:302) [opensearch-security-<IP_ADDRESS>.jar:<IP_ADDRESS>]
	at org.opensearch.transport.TransportService$ContextRestoreResponseHandler.handleResponse(TransportService.java:1347) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TransportService$DirectResponseChannel.processResponse(TransportService.java:1425) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TransportService$DirectResponseChannel.sendResponse(TransportService.java:1405) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TaskTransportChannel.sendResponse(TaskTransportChannel.java:65) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:57) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:40) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.ActionRunnable.lambda$supply$0(ActionRunnable.java:71) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.ActionRunnable$2.doRun(ActionRunnable.java:86) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:792) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50) [opensearch-1.2.0.jar:1.2.0]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]
[2022-01-17T16:14:43,042][WARN ][r.suppressed             ] [opensearch-node] path: /_plugins/_anomaly_detection/detectors/QXycaH4BI3aocvnW0ms2, params: {pretty=true, detectorID=QXycaH4BI3aocvnW0ms2}
org.opensearch.OpenSearchStatusException: Unable to get user information from detector QXycaH4BI3aocvnW0ms2
	at org.opensearch.ad.util.RestHandlerUtils.lambda$wrapRestActionListener$2(RestHandlerUtils.java:205) [opensearch-anomaly-detection-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
	at org.opensearch.action.ActionListener$1.onFailure(ActionListener.java:86) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.ad.util.ParseUtils.onGetAdResponse(ParseUtils.java:561) [opensearch-anomaly-detection-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
	at org.opensearch.ad.util.ParseUtils.lambda$getDetector$0(ParseUtils.java:517) [opensearch-anomaly-detection-<IP_ADDRESS>-SNAPSHOT.jar:<IP_ADDRESS>-SNAPSHOT]
	at org.opensearch.action.ActionListener$1.onResponse(ActionListener.java:78) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.TransportAction$1.onResponse(TransportAction.java:103) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.TransportAction$1.onResponse(TransportAction.java:97) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.single.shard.TransportSingleShardAction$AsyncSingleAction$2.handleResponse(TransportSingleShardAction.java:287) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.single.shard.TransportSingleShardAction$AsyncSingleAction$2.handleResponse(TransportSingleShardAction.java:273) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.security.transport.SecurityInterceptor$RestoringTransportResponseHandler.handleResponse(SecurityInterceptor.java:302) [opensearch-security-<IP_ADDRESS>.jar:<IP_ADDRESS>]
	at org.opensearch.transport.TransportService$ContextRestoreResponseHandler.handleResponse(TransportService.java:1347) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TransportService$DirectResponseChannel.processResponse(TransportService.java:1425) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TransportService$DirectResponseChannel.sendResponse(TransportService.java:1405) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.transport.TaskTransportChannel.sendResponse(TaskTransportChannel.java:65) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:57) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:40) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.ActionRunnable.lambda$supply$0(ActionRunnable.java:71) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.action.ActionRunnable$2.doRun(ActionRunnable.java:86) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:792) [opensearch-1.2.0.jar:1.2.0]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50) [opensearch-1.2.0.jar:1.2.0]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
	at java.lang.Thread.run(Thread.java:832) [?:?]

Expected behavior
Detectors that were successfully created and exist should be displayed in the UI.
Detectors are removed successfully using the API.
Plugins
OpenSearch Dashboards:
$ /usr/share/opensearch-dashboards/bin/opensearch-dashboards-plugin list
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
OpenSearch:
$ /usr/share/opensearch/bin/opensearch-plugin list
opensearch-alerting
opensearch-anomaly-detection
opensearch-asynchronous-search
opensearch-cross-cluster-replication
opensearch-index-management
opensearch-job-scheduler
opensearch-knn
opensearch-observability
opensearch-performance-analyzer
opensearch-reports-scheduler
opensearch-security
opensearch-sql

Host/Environment (please complete the following information):

OS: Docker based on official opensearchproject/opensearch:1.2.0 and opensearchproject/opensearch-dashboards:1.2.0 images, Centos7
Version 1.2.0

Additional context
Installing the AD plugin <IP_ADDRESS> did not help

Hi @zil1, thank you for taking the time to report this issue to us! However, I haven’t been able to replicate the issue you specify here. I followed the steps you outlined with the same environment (also trying to replicate it with AD <IP_ADDRESS>). I was wondering if you could possibly provide additional insight to the request payload you have for creating the detector. Also, any additional network response information could be helpful along with the logs you provided above. Another thing that could possibly help as I try to replicate this issue, is if you have the exact docker commands, docker-compose file or any other steps you followed to initially set up the environment. Thank you!

we didn't have any important detectors and historical data, so we just deleted the system indexes .opendistro-anomaly*. After that, we do not observe any problems. Thanks

I think I figured out how to repeat the bug. If I don't specify a "description" in the request body, then the detector is created, but the AD plugin breaks and gives an error: "Unable to get user information from detector"
OpenSearch version 1.2.4

@zil1 thanks for providing more details, and apologize for the late response. I was able to reproduce this bug by not including a description field as well. But, from the API documentation, it does state that description is required.
However, the API should reject the request if the description field is missing, rather than creating a broken configuration. Let me open a separate issue to track this, and thanks for bringing this up.

Closing in favor of the mentioned issue.


v6: Significant jsonb performance issue when used with eager: true

Describe the bug
When an entity uses a jsonb column in addition to circular and eagerly loaded properties, the post-query processing has an exponentially growing performance issue
e.g.,
Book
  - jsonb property
  - Eager: Author
    - Eager: Publisher
      - Eager: Books
        - ...

Stack trace
N/A
To Reproduce
See repro below. Important notes:
// ~~~~~ Benchmark ~~~~~
// 50 book records populated

// MacBook Pro
// M1 Max
// 64GB Ram

// SQLite: ~5 seconds
// PostgreSQL: ~65 seconds

// NOTES: Removing either of the two `eager` loads within relevant entities will resolve the issue

While those performance times don't seem like much, the amount grows exponentially as the data set increases... a 100-record query came back in about 50ms and was still processing after 15 minutes of waiting.
Repro
Expected behavior
The query and the post-query processing will be relatively quick and will beO(n log(n)) or better as the record set increases
Additional context
Add any other context about the problem here.
Versions
| Dependency | Version |
| mikro-orm* | 6.0.0-dev.54 |
| node | 16 |
| typescript | 4.9.5 |
| mikro-orm | ? |
| your-driver | PostgreSQL or SQLite |

@B4nan were you able to validate the issue? I just had to limit a query of ours to 5 records because at 15 it causes the server to exceed CPU limits and the whole thing crashes 🙃

I don't think this is about the JSON property, maybe it makes things more visible as the hydration requires parsing, but it looks like its super slow without it too. Also with select-in strategy it gets into some infinite loop apparenlty.

Only saw the stackblitz, its valid for sure, but no insights yet, need to finish with other WIP first.

Cool, yea no rush I was mostly asking to make sure you thought it was something that should be addressed.
The details from your last comment all make sense; I figured it was something near-infinite-loop-y. If I can help at all, let me know!

It's funny how such a simple test case can uncover so many issues. It all boils down to the mergeData call which is indeed overused, and while I identified ~4 problematic parts in that method, it looks like the culprit is actually the initial result mapping - the first factory.create call gets data where the publisher has a books collection with 2500 items, so for each book there are 50 copies in there. It all gets deduplicated during hydration, but this results in huge overprocessing and recursion.

Oh yea, tests are invaluable and I've learned that more and more as I've grown over the years. However, it's still the first thing to get cut when time is tight 🥲
Ah alright that makes sense... and yea I almost figured you'd come back with "Yea but don't do that" as your first reply, so I was certainly expecting the note about it not being ideal. It felt a little dirty as I wrote it, but a lot had to do with compensation for types, which I no longer have to do. I'll remove all the eager uses and just swap to populate 😬 thanks for the update!

After a lot of struggle, I think I found a way to fix it. I had to add another meta property to keep a list of populated relations, that way I can skip them and escape the cycle when eager loading. It works for both loading strategies, the tests are now running super fast, when I run a single one locally it takes just ~12ms.

Oh snap that's incredible! So should I just leave our queries the way they are or would you still recommend we avoid eager?

I would still discourage eager loading of to-many relations, unless you are absolutely sure the collection will be small.

Ah ok that's a good clarification; yea they're all very small collections (1-5). I'll go update the docs to make that recommendation clear.

Well, you should drop the joined strategy in this case - it will always explode this way due to how sql works. So any to-many relation you want to eager load should have select-in strategy, that should help.

Just tried overriding it for those properties and it actually makes a second, redundant query for each...
A -> B -> C
turns into
A
  B
    C
      A
        B
          C

From the very beginning I was hesitant about using eager; I'm just gonna remove it... I think it's helpful but it's way too brittle in terms of ways it can blow up 😬


How to make crawljax stop listening port 8080?

Ive downloaded crawljax-web-3.6-distribution.zip and started it from command line. Now how can i stop it? I  tried to kill it by PID, but it still can be opened by <IP_ADDRESS>:8080.

If you killed the process running Crawljax, that should be it. What can be opened at <IP_ADDRESS>:8080? Not sure if I understand the question.

Thank you very much for the reply. It seems the problem was in my browser. It was solved after I cleaned the cache.


[ENH, MRG] Refactor ContainsMixin and MontageMixin

Fixes issue for https://github.com/mne-tools/mne-bids/pull/983.
This seems a lot more natural to me. It would be nice if an Info object could get_montage and allow why not __contains__ and get_channel_types while we're at it, those both operate on the info.

Thanks @alexrockhill !


Was failing HTML5 validation...

... because there wasn't a space between class="menu-home dropdown" and data-dropdown="dropdown".

thanks!


Latex Issue

Card in VsCode
What are the classical approaches to Density Estimation? Briefly describe their procedures and important differences.
Density Estimation: Learn an approximation $\hat{\mathcal{D}} \approx \mathcal{D}$ using $\mathcal{X}=\left{\mathbf{x}{1}, \ldots, \mathbf{x}{n}\right}$
... Apparently github rendering doesn't like it either when I copy paste. I have provided a pic of the raw latex.

Card in Anki

Anyone know what this is happening? It makes it difficult to tell which equations render incorrectly before sending the cards to deck.

I have now experience many issue when sending cards with Latex. My solution to this problem is to copy paste the Latex instance that failed directly in to the Anki GUI Mathjax inline or block editor. In every one of my instances this has worked perfectly. It seems that the Latex is getting jumbled up in an attempt to translate it to Mathjax, which might not be necessary. I can provide all of the failed instances if it is helpful.

Latex is not the same as Mathjax; therefore the syntax will be different.
You can test your Latex with overleaf.com before you send it to Anki
The following code may help to get you started.
VSCode Latex
# latex

## Latex

$\hat{\mathcal{D}} \approx \mathcal{D}$

using

$\mathcal{X} = \{ {\mathbf{x}_1, \ldots, \mathbf{x}_n} \}$

Anki card

Note:  It would help others to understand your issue if  you indicate Issue and Expected in your comments.

Following will also work
# Mathjax on back

## Mathjax on back

$ \hat{\mathcal{D}} \approx \mathcal{D} $

using

$ \mathcal{X} = \left \{  {\mathbf{\frac{x}{y}}_1, \ldots, \mathbf{x}_n} \right \} $

Testing Suggestions
@rleyvasal I think you made a typo, but I get the point.
screenshot

VsCode Markdown Example 1
I'll try to clarify my issue and my fix.
Input (Example 1)
Goal: Learn an approximation $\hat{\mathcal{D}} \approx \mathcal{D}$ using $\mathcal{X}=\left\{\mathbf{x}_{1}, \dots, \mathbf{x}_{n}\right\}$
Expected Output (Example 1)
screenshot

Anki After Sending to Deck (Example 1)
screenshot

So for some reason the _ is being translated to <em> and </em>.

My fix (Example 1)

Copy paste the exact Latex that didn't work from VsCode into the MathJax inline text box.

$\hat{\mathcal{D}} \approx \mathcal{D}$ using $\mathcal{X}=\left\{\mathbf{x}_{1}, \dots, \mathbf{x}_{n}\right\}$

Note: This is the portion of the Latex from Input above that failed.

screenshot

VsCode Markdown Example 2
Input (Example 2)
$$
\hat{\theta}_{\mathrm{MAP}}=\operatorname{argmax}_{\theta} p\left(\theta \mid \mathbf{x}_{1: n}\right)
$$

Expected Output (Example 2)
screenshot

Anki After Sending to Deck (Example 2)
screenshot

My Fix (Example 2)

Same as Example 1

Note too, this is still a MathJax block.

screenshot

This works, and I have around 5 different instances where this solution works. I can provide them if there is more interest in this.

@Mjvolk3
The problem is not with the Anki for VSCode extension. The issue is with the way the Latex is beign entered into vscode.
As a rule of thumb for subscripts if there is only one character {} are not needed, but when there are two characters as  {1+m} , then  the {} should be used such as the case below.

# test

## test

Goal: Learn an approximation $ \hat{\mathcal{D}} \approx \mathcal{D} $

using

Your example `\mathbf{x}_{1}`

$ \mathcal{X} = \left \{ \mathbf{x}_{1}, \dots, \mathbf{x}_{n} \right \}$

My example  `\mathbf{x}_{1+m}` adnd  `\mathbf{x}_n`

$ \mathcal{X} = \left \{ \mathbf{x}_{1+m}, \ldots, \mathbf{x}_n \right \} $

@rleyvasal How do you explain that things work when I copy the latex that contains _{x} into the MathJax inline/block in Anki? This can be seen in the first example I provided. This is the point I don't get.
Thanks for showing work arounds. Ideally, I could find all of the necessary work arounds and implement them in my Latex, but for some of my examples, like example 2 shown, I haven't been able to find a proper work around. If you have the time, would you mind taking a look at that example?

This is very likely a bug.
Latex below does not show correctly when sending from VSCode to Anki, but entering the same code without in the Anki app (without ``$$ ..$$`)  as a Mathjax block gives the correct math symbols.
$$
\hat{\theta}_{\mathrm{MAP}}=\operatorname{argmax}_{\theta} p\left(\theta \mid \mathbf{x}_{1: n}\right)
$$

The  issue may be happening in one of the following:

It could be the Markdown parser is not parsing the underscore correctly e.g. CommonMark, or Pandoc(not sure what is being used)
or
The problem maybe happening if the parsed output is not being correctly sent as a Mathjax Block to Anki (Inline $...$ has the same issue) .

@jasonwilliams  may be in a better position to diagnose the issue.


[Bug]: Calling brower.newPage() from within a loop can result in silent OOM

Bug description
Hello, I had some code that was calling brower.newPage() in a recursive loop. The code would sometimes freeze without any warnings or core dumps. I realized that puppeteer would silently crash if the process OOMs, as I observe the memory usage by node.js increase incrementally until it uses all available system memory.
import puppeteer, {executablePath} from "puppeteer";

(async (url:string) => {
    const browser = await puppeteer.launch({
        headless: true,
        executablePath:executablePath(),
    });

    while (true) {
        // Using an infinite loop as an extreme example, the same OOM happens for large iterations of recursive.
        let page = await browser.newPage();
        await page.goto(url);
        await page.$('#some-element'); // An example of having the page do something
    };

    await browser.close();

})('www.example.com');

The code fails silently.
import puppeteer, {executablePath} from "puppeteer";

(async (url:string) => {
    const browser = await puppeteer.launch({
        headless: true,
        executablePath:executablePath(),
    });

    while (true) {
        // Using an infinite loop as an extreme example, the same OOM happens for large iterations of recursive.
        let page = await browser.newPage();
        await page.goto(url);
        await page.$('#some-element'); // An example of having the page do something
        await page.close();
    };

    await browser.close();

})('www.example.com');

This code will not use up all the memory.
Puppeteer version
19.4.1
Node.js version
v18.12.1
npm version
9.1.3
What operating system are you seeing the problem on?
Linux
Configuration file
No response
Relevant log output
No response

I would say this is expected:

you have to dispose element handles you receive from querying the page https://pptr.dev/api/puppeteer.jshandle/
creating an infinite number of pages without closing them will obviously consume all memory in the browser as well as in the Puppeteer's process.


Outdated dependency

I noticed during building an application that uses winit, that serde_json 0.9.10 (which is pre-1.0 and no longer supported) is still used by libloading 0.3.4 (currently used by this crate). libloading 0.4.0 no longer requires the now deprecated target_build_utils crate (the API hasn't changed AFAICT), so the old serde_json version is no longer used, but libloading 0.4 requires at least rustc 1.14. According to crates.io, the main users of this crate is the various Wayland-related crates and winit, so updating libloading means for most of the GUI related stuff rustc 1.14 (released in last december) is a new minimal requirement. Is it acceptable?

The wayland-rs libs already depend on features that are newer than (and thus already do not compile on 1.14). So I see no issue with that.
Though this is a good point, I should find out what is the minimal rust version they depend on a test this on travis.

Released 0.4. Will update wayland-client and wayland-server too.


Snapshot download shows api_password in url

Home Assistant release with the issue:

Home Assistant
0.73.2
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Operating environment: Docker
OS: Linux (Debian Stretch)
Component/platform:

Description of problem:
In the snapshot tab of Hass.io select a snapshot that has been created.
When hovering over the 'Download snapshot' icon, the full download path is shown.
At the end of the path, the api_password is shown:
https://hass.domain.tld/api/hassio/snapshots/1bdb0c06/download?api_password=api_password_here
The api_password shouldn't be shown here on the website and in the download link of the snapshot.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
-

Traceback (if applicable):
-

Additional information:

You may want to anonymise the URL on screen :-)

Could it be that this breaks the snapshot download with the latest version?

/move home-assistant/hassio


Please read the text and correct

[Enter feedback here]

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

ID: 35765cda-23fb-5b94-0856-7002e37dadd3
Version Independent ID: 82c39d58-a4e3-4562-53ab-131b831ec987
Content: Archive inventory transactions - Supply Chain Management | Dynamics 365
Content Source: articles/supply-chain/inventory/archive-inventory-transactions.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @sherry-zheng
Microsoft Alias: chuzheng

@jmertens1, thanks for the feedback. Can you provide additional information about what is incorrect in this topic.

For each archived record it must be
For each archive in the grid, the following information is provided:
Active – A check mark indicates that the archive is active.
From date – The date of the oldest transaction that can be included in the archive.
To date – The date of the latest transaction that can be included in the archive.
Scheduled by – The user account that created the archive.
Executed – The date when the archive was created.
Reverse – A check mark indicates that the archive has been reversed.
Stop current update – A check mark indicates that the archive is in progress, but it has been paused.
State – The processing status of the archive. The possible values are Waiting, In progress, and Finished.

@jmertens1 , I don't think those are archived records that are listed there. The rows don't indicate records that were archived--each row is itself an archive that contains a collection of archived records (each of which is a transaction that occurred during the dates shown for the archive). When you open an archive, then you see the list of archived records that belong to that archive. Please let me know if that sounds incorrect and I will look into it further.


ci: update vars description

Description
SSH private key not required to be entered in here doc string format anymore for IBM Cloud catalogs
Release required?

[x] No release
[ ] Patch release (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.

/run pipeline

:tada: This PR is included in version 5.1.0 :tada:
The release is available on:

GitHub release
v5.1.0

Your semantic-release bot :package::rocket:


compilation error

I am getting this compilation error. Please suggest how it can be resolved
golangci-lint run
make: *** [Makefile:9: build] Killed
Error: error building at STEP "RUN make clean && make build": error while running runtime: exit status 2

Hi @girisubhra, seemed to be an issue with the golangci-lint version.
I have created a PR #16 to fix this.
For now, you can update the version in the Makefile, until the PR is merged.

Hi @girisubhra, seemed to be an issue with the golangci-lint version. I have created a PR #16 to fix this. For now, you can update the version in the Makefile, until the PR is merged.

Thank you very much it is built now. I have another doubt is this mandatory "docker push localhost:5000/sriov-metrics-exporter"

Hi @girisubhra, typically you would build an image, and push it to a local registry for your pod to pull the image down.
If you are testing locally, you do not need to push to a registry.
If you are testing on a multi-node cluster, I'd recommend setting up a registry.


Fix nbdev_install_hooks references in the docs

Changed a couple of references to the old nbdev_install_git_hooks instead of nbdev_install_hooks.
Some issue still remains when previewing the documentation with nbdev_preview, all notebooks are stripped of the numbering prefix, except for 11_clean.ipynb. (See screenshots below), which breaks the autogenerated cross references in the docs (which point to clean.html instead of 11_clean.html)

I haven't been able to locate why that's going on yet.

Many thanks! I'll look into that odd 11_clean issue.

It seems to be working now, and the link points in the right direction in the docs. What was the issue?

That nb is the only one with a raw cell containing frontmatter, instead of using an H1 in the markdown. There was no filename in the frontmatter.

interesting, thanks!


Add Standard Layer5 GitHub Issue Templates

Description:
Layer5 GitHub repos all have similar GitHub issue templates to assist in providing contributors and users common formats for creating and working with issues.
Implementation
Copy the layer5io/meshery repository's issue templates into this repo

@leecalcote can I take this up?


Added ARG query for A01.40 in network_appdelivery_checklist.en.json

Added ARG query for A01.40 in network_appdelivery_checklist.en.json
Description
Please provide a detailed description of your changes.
Related Issue
Link to any related issues or discussions here. This helps reviewers understand the context and the need for your changes.

Checklist

[x] I've tested my changes to ensure they are ready for review.
[x] I've read the CONTRIBUTING.md guide.
[ ] I've updated the documentation (if applicable).
[x] Resource Graph queries have been included (and tested) for recommendations where ever possible[^note].
[ ] Resource Graph queries have NOT been included (please explain below).

Additional Information
Is there any additional context, screenshots, or considerations that might help in the review process? Please include them here.
Reviewer Notes
Is there a specific area you’d like feedback on? Please highlight it here. We're here to help and learn together! 💡
[^note]:
Details on how to add Azure Resource Graph queries to recommendations can be found here.

Updated the review comment Jose, sorry =~ was left in type == 'microsoft.network/frontdoorwebapplicationfirewallpolicies', updated it with the =~. Good to be merged now. :)

From: Jose Moreno @.>
Sent: Monday, October 7, 2024 6:36 AM
To: Azure/review-checklists @.>
Cc: Aarthi Sukumar @.>; Mention @.>
Subject: Re: [Azure/review-checklists] Added ARG query for A01.40 in network_appdelivery_checklist.en.json (PR #884)
@asukumar22https://github.com/asukumar22 could you please resolve the conflicts in this PR? Otherwise it can't be merged
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/review-checklists/pull/884#issuecomment-2396956558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ5NY37VN3F6MROJ653NZW3Z2KE4DAVCNFSM6AAAAABOKRWJXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJWHE2TMNJVHA.
You are receiving this because you were mentioned.

Hey @asukumar22 sorry to bother you again: there seems to be a conflict, possibly with another PR in parallel. It is only a single line one of the "more info" links

Updated Jose.
Thanks,
Aarthi

From: Jose Moreno @.>
Sent: Monday, October 21, 2024 2:15 PM
To: Azure/review-checklists @.>
Cc: Aarthi Sukumar @.>; Mention @.>
Subject: Re: [Azure/review-checklists] Added ARG query for A01.40 in network_appdelivery_checklist.en.json (PR #884)
Hey @asukumar22https://github.com/asukumar22 sorry to bother you again: there seems to be a conflict, possibly with another PR in parallel. It is only a single line one of the "more info" links
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/review-checklists/pull/884#issuecomment-2427732560, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ5NY377O6OLHEMHJIYDKBDZ4VVGVAVCNFSM6AAAAABOKRWJXSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRXG4ZTENJWGA.
You are receiving this because you were mentioned.Message ID: @.***>

@asukumar22 It looks like there are conflicts to be resolved?


Update strings.xml

Fix a typo in Slovak translation.

Translations are managed on Transifex: https://www.transifex.com/projects/p/k9mail/
Changes in the source files will be overwritten when pulling translations from Transifex.

Thanks, fixed the translation on Transfiex


Perfect line completion appears but is then replaced by wrong or shorter suggestion

It takes a millisecond or two before my brain has processed it and decided to accept it, and during those milliseconds, my fingers keep typing letters(*) that are identical to the starting letters of the completion (the suggestion is exactly what I want, after all!).
But sometimes/often during those milliseconds, the ingenious suggestion disappears and is replaced with something else(**) - nooooo! Whaaat? As said, the typed and suggested letters are identical, so why does it change its mind??
The logic behind the behavior might be

"Oh, he keeps hitting the keyboard after I suggested, he must not be happy!"
"My initial guess was based on local code, but the backend now says something else and the backend is always right"
Something else?

Possible fixes which should be fairly straight-forward to implement:
a) Set an (adjustable?) delay before Copilot "changes its mind"
b) Prioritize (initial) guesses based on local code
c) Copilot should absolutely never change its mind when the user keeps typing letters that are identical to the letters at the start of the suggestion!
Pleeeease upvote,  pleeease fix :)
*) The commands to type the desired key sequence have already been dispatched from the frontal cortex... too late to take them back!
**) Sometimes the initial suggestion is a full line completion (Yesss, accept!!), and the second suggestion is just shorter (a word or two). Frustrating, but at least it's helpful. But other times the suggestion is just plain wrong. Not just frustrating, it's ruining a few seconds of my day ;).

Correct. I have intellisense enabled too, though. But all of the stuff described above are in the "gray matter" ;-). I.e. just the Copilot suggestions.

Seems like you've gotten the point already, but I'm anyway sending two screenshots that illustrate the issue:
This is the first suggestion - great, I'll accept DISPLAY as a continuation!

Then, after typing a D, it changes to this:

(WTF? Why does it think I don't want DISPLAY, when I just typed the first letter!!)
BTW, when reconstructing this actual example, there was a long delay (several seconds) between the first and second image, so the issue seems not to have anything to do with "an improved response from the backend" arriving after a local guess.


lack of classification of acyl-CoA binding terms

We have several existing acyl-CoA binding terms, which don't classify under a CoA related grouping term, even though it seems like they should:

acetyl-CoA binding
fatty-acyl-CoA binding
long-chain fatty acyl-CoA binding

Looking at ChEBI, it looks like this is the relevant structure:
-- adenosine 3',5'-bisphosphate
--- acyl-CoA
---- acetyl-CoA
---- fatty acetyl-CoA
----- long-chain fatty acetyl-CoA
--- coenzyme A
Looking at the classification of terms like acyl-CoA metabolic process and acetyl-CoA metabolic process, I think that if I add a term for acyl-CoA binding then, all three of the terms I listed will automatically classify under the new acyl-CoA binding term.
Originally posted by @krchristie in https://github.com/geneontology/go-ontology/issues/19257#issuecomment-626064743

I think having a term for acyl-CoA binding will also answer this question:

In the extension is specified (3S)-3-hydroxy-3-methylglutaryl-CoA as the bindng partner.. How should that be captured?

@pgarmiri - Would it work for you to transfer your annotation to a term for acyl-CoA binding and then you can use the same extension as in the original annotation to coenzyme binding? If so, then we don't have to create a term for such a specific CoA deriviative.

@pgarmiri - Considering that we already have metabolism terms in GO for succinyl-CoA:
-- succinyl-CoA metabolic process
--- succinyl-CoA catabolic process
--- succinyl-CoA biosynthetic process
I think it would be reasonable to go ahead and make a term for "succinyl coA binding" to accomodate this annotation:

UniProtKB:P9WP21
2,3,4,5-tetrahydropyridine-2,6-dicarboxylate N-succinyltransferase
GO:0050662
MTBBASE
NCBITaxon:83332
IDA
PMID:19394346
20100505
coenzyme binding => succinyl-CoA binding

Alternatively, you could use the to be created term for "acyl-CoA binding" with an extension for "succinyl-CoA". Let me know your preference.

from @pgarmiri by email:

Hi Karen,
Thank you very much for your message. It was in my to do list but with the GOC meeting and the my reduced work hours, I could only look into this now.
I am very happy with the suggestions and I think they will work well.
So, I will add the (3S)-3-hydroxy-3-methylglutaryl-CoA as the binding partner in the extension  of the new term
"acyl-CoA binding" .
For the other case, I would prefer a new term succinyl-CoA binding since there are other terms present in the ontology that include it. If, however, there  is a scope of merging the other terms to the acyl-CoA related terms , I am happy to go with the second suggestion.
Thanks,
Penelope

I prefer the option of creating the specific term for succinyl-CoA binding at this time also since there are already metabolism terms specific to succinyl-CoA, so I’ll make both terms. I’ll do this today, so they’ll be available soon.


age-plugin-yubikey: init at 0.2.0

Motivation for this change
I want to use my yubikey with (r)age.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

You missed to update the hashes 🙂

My bad, fixed :)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/430


change the log file name to uninstaller.log to the name of the script

change the log file name to uninstaller.log to the name of the script

Good call, was thinking the same


Create Content on Community Section

Summary
Four new pages have been added to the community section of the documentation site.
Testing Guidance

Run bundle exec jekyll serve
Navigate to localhost:4000/index_new.html, then go to the Community page.
Confirm that the content in the community section is accurate

I think we should work in here somewhere a list of available public hosts.  That could either go in the 'Test Kit' section (simply because we reference the only known public host).  Perhaps adding a link to Inferno.healthit.gov qa server would be worthwhile, as SOMETIMEs we want people to go there for connectathons, etc... so we would have 2 public hosts i suppose :)

We also need to work in 'Additional Tools' or 'Inferno Add-ons' section.  Which basiscally will just be the 'inferno reference server' (which really, we should start calling the 'inferno us core server simulation'), the fhir-validator-wrapper, and the validator-app.

We also need to work in 'Additional Tools' or 'Inferno Add-ons' section. Which basiscally will just be the 'inferno reference server' (which really, we should start calling the 'inferno us core server simulation'), the fhir-validator-wrapper, and the validator-app.

I feel like Additional Tools would go under Documentation, rather than Community?


Fixed spelling and formatting.

@tseaver this addresses my own comments on PR #1932.

See also #1948.


got a question

What is your question?
there are some mods that move the camera a little bit when the opponent hits a note
what is that on the source code?

An edit of moveCamera function, i won't explain this.

why


A way to get data state in JSON format?

Hallo!
I'm trying to tie together Ractive with a Backbone Model.
Since the adaptor doesn't really work that well: https://github.com/ractivejs/ractive/issues/1877, I passed a JSON object to the Ractive instance instead (model.toJSON()).
I have a form on the page that updates some data. I need to get that changed data back to set on a model and call model.save()
If I simply call ractive.get() it returns entire template context with additionally set properties, helpers and everything else. Is there a way to get simple JSON without all the clutter?
As if this moment I have to do the following:
data = {}
    for input in @$inputs
      data[input.name] = input.value

    @fire('submit', data)

That's a bit tedious to do all the time and kind-of defeats the purpose of having ractive in the first place...

Use a key on the data object to isolate your model
data: {
    myModel: json
}

ractive.get('myModel')

But then I'll have to prefix everything in the view with myModel.
Oh :(

But then I'll have to prefix everything in the view with myModel.

{{#with myModel}}

<!-- no prefixes needed here -->

{{/with}}

legit! thanks @martypdx


Add scroll-behavior: smooth to the root element

html {
  /* … */
  scroll-behavior: smooth;
}

Decided that this is too opinionated and would be better applying on a case-by-case basis.


Remove duplicate notification on windows 7

@RocketChat/core

Closes #456

The latest version of electron now includes notifications for windows 7, so removing the custom implementation to remove duplicates.

Ah sweet 1.6.7
https://github.com/electron/electron/pull/9168


brew cask install  - Staged installation

Hi there! :)
I'm kinda reopening @dschinkel's issue https://github.com/caskroom/homebrew-cask/issues/9974 about the staged installation.
I've been having this issue for the last 10 casks I've been trying to install. Most notorious are Mailbox App, Skype, Sublime-Text and Spotify.
Here after are screenshots of the Mailbox installation. For every app it's the same, the message says the installation stagged and when I cd to the app folder, I find a symlink of the Applications folder.

This happens on a brand new Macbook Pro, brew doctor says that everything is fine.
And here is the output of brew cask doctor: https://gist.github.com/ladislas/097d87cbe98b7b33dd19

Everything appears to be working normally, but there may be some confusion about what “staging” and “installation” mean in homebrew-cask.
The staging process merely extracts the contents of a package as downloaded. Many dmg packages include an /Applications symlink for ease of manual installation, causing it to appear in the staging location.
Afterward, brew cask install symlinks the desired artifacts (in this case, Mailbox (Beta).app) to your application folder, /Applications.

so if I understand correctly, the the last message with the beer means that everything went well?

the last message with the beer means that everything went well?

Yes.
Just a clarification, though. The symlinks will, by default, be placed in ~/Applications. To set them in /Applications you need to set the —appdir option. This will be reversed in the future.
Closing as solved.

Okay thank you very much! :)
Yes, I know about the --appdir option and I'm using it. Great to see it will become the default behavior.


Add integrated vectorization as an option vs. push mechanism (including vectorizers) and keep, document both approaches

Chat with your Data currently uses a push model. It reads a message from the doc_processing queue and then uses /workspaces/chat-with-your-data-solution-accelerator/code/backend/batch/utilities/helpers/DocumentProcessorHelper.py to Load, Chunk and add them to a Vector Store.
Integrated Vectorization adds a Pull model that will do that step for us: - https://techcommunity.microsoft.com/t5/ai-azure-ai-services-blog/announcing-the-public-preview-of-integrated-vectorization-in/ba-p/3960809
We still want to keep both methods so that the end user can chose which mechanism to use.

The accelerator today just supports to index using a function and push the data in the index using the SDK. All the chunking and everything else works by calling langchain libraries to take care of the chunking and such.
Today, we have support to do fixed-chunking and have integrated vectorization using pull indexers via: https://learn.microsoft.com/en-us/azure/search/vector-search-integrated-vectorization, so the request is to add an option for customers to use the current option (using Push API) or pull option (using built-in indexers to do the cracking, chunking, vectorize and index). This option removes the need to use AI doc intelligence for customers who have simple text or other files not supported by AI doc intelligence.
There are a few examples of this implementation, here for the demo: https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/scripts/prepdocs.py
Here for Python scripts as a sample: https://github.com/Azure/azure-search-vector-samples/tree/main/demo-python/code/integrated-vectorization
I hope this helps with the requirement definition.

HI @gmndrg , wanted to confirm your thoughts on when to provide user an option to choose between push or pull (integrated vectorization)

When the user is deploying the Solution accelerator, at the very beginning, via  .env file. So with this option, bicep will create a setup that will only work with only 1 of the 2 options.
OR, User should be able to decide between push or pull while adding documents via admin portal. Here both the push and pull will coexist and user can make a decision at document level. (this solution will have higher complexity, we may have to check if data added in index via push api is not wiped off when the indexer runs.)

Hi @prasbhargava1 it should be at the very beginning since the indexer is created for scheduled approach and with the fixed options and it would keep running. However, customer can also use push to the same index, both can coexist. So the current method of uploading data can remain there for uploading URLs and such but an indexer picking data from a container if needed. Please make sure to enable deletion policy by default: https://learn.microsoft.com/en-us/azure/search/search-howto-index-changed-deleted-blobs?tabs=portal.
Thank you so much

@prasbhargava1 Thanks for your work on this. I wanted to ask if you could please document the design decisions made under this issue (there seem to be a few based on the comments above) in an ADR document like the ones here.
It can be lightweight, we just think it will be beneficial to keep a record of what decisions were made as part of this work and why. Thanks!

Currently the data ingestion into Azure Search takes place through Push API where the chunking, vectorization is all under the control of the developer. We need to provide a demo of how Pull approach (Integrated Vectorization) of ingesting data into azure index works. Pull approach handles all the internals of data ingestion by itself.
Requirements
User needs to opt of either of the approach at the time of deployment.
Tasks

[ ] ADR
[ ] For URL embeddings, download the url content & upload it to the blob so that it works with Pull approach.
[ ] Task 2
[ ] Task 3


depends resolves unbound parameter instance in unexpected ways

ALL software version info
Python 3.9.13
macos m1 - 12.4 (native python)
safari
panel 0.14.0
param 1.12.2
Description of expected behavior and the observed behavior
I like to use classes for a lot of my panel apps because it gives a self contained app and it gives an init for grabbing data from an external data source. It can be hard though to find mappings for all of the param parameters to panel widgets, and some don't have mappings, plus most of the docs for panel don't use param. So I have been playing around with trying to use pn.depends and pn.bind in classes but encountering some rough edges, which led me to using them in parameterized classes, which led me to this. If you use pn.depends on a panel widget inside a parameterized class, it fires the method twice, with some weird behavior.
Complete, minimal, self-contained example code that reproduces the issue
This is me changing the selector twice
import param
import panel as pn

class Test(param.Parameterized):
    w = pn.widgets.Select(options=["a", "b", "c"])

    @pn.depends(w, watch=True)
    def print_(self):
        print("hello world")
        try:
            self.w
        except AttributeError:
            pass
        print("self: ", repr(self), end="\n\n")

t = Test()
pn.Row(t.w)

So if we change the selector then it prints "hello world" twice. But weirder, self is different each time. Here is the output
hello world
self: Test(name='Test08207')

hello world
self: 'c'

hello world
self: Test(name='Test08207')

hello world
self: 'b'

So the first time it is being handed self, but the second time it is being handed w.param.value. Also if we make it a staticmethod, then it only fires one, and it is only handed w.param.value.
Screenshots or screencasts of the bug in action
I used a button in this example for simplicity, but a selector above since the button only real param.Parameter internally is value, so using Selector and Button at least means it more than just boolean like ones.
This is me clicking the button twice

Sorry, I completely misread your example. Declaring widgets as class-variables is strongly discouraged precisely because it can cause problems like this. You should either instantiate the widget in the constructor of the class or declare a Parameter which you then convert to a widget, i.e. either:
class Test(param.Parameterized):
    w = param.Selector(objects=["a", "b", "c"])

    @pn.depends('w', watch=True)
    def print_(self):
        print("hello world")
        try:
            self.w
        except AttributeError:
            pass
        print("self: ", repr(self), end="\n\n")

or:
class Test(param.Parameterized):
def __init__(self, **params):
    super().__init__(**params)
    self._w = pn.widgets.Select(options=["a", "b", "c"])
    self._w.param.watch(self._print, 'value')

def print_(self, *events):
    print("hello world")
    try:
        self.w
    except AttributeError:
        pass
    print("self: ", repr(self), end="\n\n")

@philippjfr That is fair enough and makes sense, I am assuming that param inside a parameterized class's init is doing a similar thing as dataclass or attrs do in their new to turn class variables into instance variables, but also param is then watching for events and other things like that.
I just also wanted to mention that the documentation doesn't give that feeling. The impression I and another user have gotten is that most of the examples are around pn.bind, pn.depends, etc. and there isn't nearly as much about parameterized classes and neither of us found another about what happens when you can't figure out how to map a certain Parameter to the panel widget you actually want, which is what partially what led to this.
The differences between between those two different api styles feels like a bit of a deadend instead of a shortcut, https://holoviz.org/background.html#shortcuts-not-dead-ends, especially if you don't find the ways to do it. You start out with simpler apps using bind and pn.depends, then it gets larger and switch to a Parameterized class and those don't work any more, and you now need to figure out the which Parameters to use and class mappings between Parameters and widgets if you don't like the default. The second option you specified would potentially be good as well for people looking for self contained apps not relying on global variables, but I didn't see any examples or anything like that anywhere.
One final thought, would it be possible to write in panel (since param doesn't have external dependencies at all, let alone to panel) a pn.ParamWidget that could be used on a class level on widgets to convert them to Parameters (inside Parameterized classes), kind of like param.ClassSelector, but probably only listening to value since I assume a bunch of other things might get changed internally. I don't know if this is possible, but it would stream line things for people and making the transition between one api style to the other a lot easier. If you think this is possible, you could try and point me in the right direction and I can try and see if I can get a start on it.

Thanks for your thoughtful comments! Unfortunately I'm about to log off and go on vacation so I can't reply in detail.

One final thought, would it be possible to write in panel (since param doesn't have external dependencies at all, let alone to panel) a pn.ParamWidget that could be used on a class level on widgets to convert them to Parameters (inside Parameterized classes), kind of like param.ClassSelector, but probably only listening to value since I assume a bunch of other things might get changed internally.

So turns out you can already do this:
class Test(param.Parameterized):

    widget = param.ClassSelector(
        class_=pn.widgets.Widget, default=pn.widgets.Select(options=['a', 'b'])
    )

    @param.depends('widget.value', watch=True)
    def print_(self):
        print(self.widget.value)

Give it a try and see if this is enough for you needs. Wrapping this up in a pn.PanelWidget parameter might still be useful though.

Great! Thanks, that is helpful. I think wrapping this up in a pn.ParamWidget or something might be nice. So you could could do something like
class Test(param.Parameterized):

    widget = param.ParamWidget(pn.widgets.Select(options=['a', 'b']))

    @param.depends('widget.value', watch=True)
    def print_(self):
        print(self.widget.value)

Though, I don't know if it could streamline all of the .values. But if so, maybe it could be as simple as
class Test(param.Parameterized):

    widget = param.ParamWidget(pn.widgets.Select(options=['a', 'b']))

    @param.depends('widget', watch=True)
    def print_(self):
        print(self.widget)

Anyways, enjoy your vacation! I can tinker with this a bit in my spare time when I am not working on my research.


Documentation Needed

Need Documentation :D

yes you are right. I'm on it... maybe you can help me :D

hey what you think of the brief summary in readme.md?

hey @ifobos can you migrate this code to android? if you do it, I'd appreciate you


Derive constants c1 and c2

By Abadi et al. theorem 1 (pg. 4) we can choose constants c1 and c2 such that sigma will guarantee differential privacy

From Theorem 1 (pg. 14) we have a DP guarantee provided that

T * q^2 * lambda^2 / sigma^2 <= lamba * epsilon / 2
exp(-lambda * epsilon / 2) <= delta

So given epsilon and delta from the inputs to the function, we need to find the optimal sigma (noise scale) to satisfy (1) and also verify that (2) is satisfied.

I believe this can be merged in with #313


NetCDF data processing

Parse NetCDF data in preparation for loading into database

I have written code that:

Loads the "Copy of measured channel xlsx" file and reads the meter name and measurement type
Goes through all the folders in the data and extracts .nc files
Determines the meter name and the measurement type for each .nc file.
Combines all measurements for each meter into a single data dataframe.

I implemented the above for a single day. Eventually, we will want to load the data into a database but I put in in a dataframe for now to test the functionality of the code.

To read the .nc file, I found a package called xarray It was easier to use and has more features than the Netcdfhandler.py file.

The NetCDF files are being read in the extract_data() function as seen here


Allow using LiveSplitOne through WebSockets

While autosplitting for LiveSplitOne is still in development it seems to be possible to perform basic tasks through a WebSocket although in the long run autosplitting should be done as wasm.
Given that this already uses a server connection to the legacy LiveSplit it seems somewhat easily to adapt.
There's a proof of concept also in python: https://github.com/mgolisch/lso-remote

Hopefully implemented by c639c316d9b001817e04baee451a3a1e771d89f9


Flow with many new data points results in overlapping x axis tick labels

Hi,
First off, fantastic library! Thank you!
I'm currently working on a graph display, with incremental updates to the data by polling. It's working really well, with only one minor issue: the tick labels. I first load() the graph with an initial data set, usually about 60 timepoints. Then, after a fixed time, I poll the server for any new data, typically returning 10 new timepoints, which I push onto the graph using flow(). The graph lines transition beautifully, but for every new timepoint a new tick is added.
This means that the original graph may have had 6 or so axis tick labels, and then the first flow() call adds another 10, after the second flow() another 10 and so, resulting after a while in a continuous blur of tick labels at the bottom.
I found I could mask this a little using the axis.x.tick.count option, but when I use this, all the ticks are redrawn after each flow, disrupting the otherwise smooth transition of the graph.
Using version 0.4.4.

Hi
When all ticks available then its not showing tick text for all tick points.

@nitin0708 That sounds like a different issue; this issue is about overlapping labels when using chart.flow. Could you please:

Research whether there's an existing issue that describes the behaviour you're encountering?
If not, open a new one?

Many thanks!


WEB9JA-27 uploaded images to s3 and re-configured FE reuest

Hi, I completed the uploading of pictures to S3 and also modified how FE should send Data using multer library.
Please, kindly go into ads and user controller...I left a comment for @binidowu.
I need either @binidowu  or @Abdul4real  to work on a route that will allow me get user information by a userID...i will attached the userId in the param...I will like to have user profile picture, username/ name, email and createdAt.
Thank you guys

Nice work! I will see to the route.


select2-ng dropdown cause scrolling when it's positioning

When the body perfectly fit the viewport height, the dropdown causes page scrolling before it's position is calculated.
Source: https://github.com/select2/select2/blob/select2-ng/src/js/select2/dropdown/attachBody.js#L81
Demo: http://plnkr.co/edit/dbLkCuhly9291I1Yop2s?p=preview
Scrollbar moves the select2 container before calculation of dropdown position, and then returns the container to normal location.

It appears to be an issue only when the dropdown is opened for the first time.


Improve random tip UI

After playing the game for a while I've noticed that the random tips that were added by #13509 tend to just get lost in the chat log. This PR slightly tweaks the way the tips are presented and makes them play some sounds so that they stand out a bit more, which should help newer players. I actually started working on this shortly after the tip system was merged but only now found the motivation to finish the PR. This PR is bigger and took longer than it has any right to, and is still quite janky.
This PR also adds a new command that allows admins to trigger a random tip to be sent, as well allowing them to send custom tips to specific players

[X] I have added screenshots/videos to this PR showcasing its changes ingame

https://user-images.githubusercontent.com/60421075/228731344-5ebb0da3-e911-44b8-aec2-ab5e6c241c36.mp4
:cl:

tweak: Improved the random-tip UI. Tips should be easier to see and will no longer get lost in the chat log.

If this is gonna be permanent there should be an option to disable non-admin ones in the client, also making them go away after clicking the control would be nice

(It is not meant to be permanent... unless people actually want this for some reason)

(It is not meant to be permanent... unless people actually want this for some reason)

i cant lie to myself, i dig it

Make it NOT april fools if this gets removed I will be sad

I think it would be pretty silly if it were permanent, it's intentionally very intrusive.


[on hold] Multi-environments module format

prebuild an UMD bundle on publish, in order to keep providing browser
and common.js assets
convert an original source code to ES Module which is now a
primary entry point, so it can be used without a friction for consumers
which are written in ES Modules format, or use module bundlers like
Webpack or rollup

this is supposed to be a part of  the https://github.com/san650/ember-cli-page-object/pull/556#discussion_r775649852
Let's wait a bit with merge, until all the details are settled in there. Though let's discuss if there are questions/concerns.


Fix TypeError when working with nested maps

Fixes #401 (CLIENTS-349)

@nicorevin Could you please add a test case for your change?  That would help me to verify.  Thanks!

@javajolt I've added. See riak/tests/test_datatypes.py

Sorry, I missed it.  Thanks!

:+1: 4dca64c

Looks like make lint failed.  I'm guessing this is a newer version of pep8.  I'll create a new PR to address that.

@borshop: merge

By the way, is it ok that modified attribute does not correspond with _updates and to_op() is None?
Even without this PR we have the following picture:
In [1]: from riak import RiakClient
In [2]: c = RiakClient()
In [3]: b = c.bucket_type('maps').bucket('wow')
In [4]: m = b.get('key')
In [5]: m.__dict__
Out[5]:
{'_context': None,
 '_removes': set(),
 '_updates': {},
 '_value': {},
 'bucket': <RiakBucket 'maps'/'wow'>,
 'key': 'key'}

In [6]: m.maps['a']
Out[6]: <riak.datatypes.map.Map at 0x7fad9a294c50>
In [7]: m.flags['b']
Out[7]: <riak.datatypes.flag.Flag at 0x7fad99be8668>
In [8]: m.registers['c']
Out[8]: <riak.datatypes.register.Register at 0x7fad99be8278>
In [9]: m.counters['d']
Out[9]: <riak.datatypes.counter.Counter at 0x7fad99be8630>

In [10]: m._updates
Out[10]:
{('a', 'map'): <riak.datatypes.map.Map at 0x7fad9a294c50>,
 ('b', 'flag'): <riak.datatypes.flag.Flag at 0x7fad99be8668>,
 ('c', 'register'): <riak.datatypes.register.Register at 0x7fad99be8278>,
 ('d', 'counter'): <riak.datatypes.counter.Counter at 0x7fad99be8630>}

In [11]: m.to_op()
# None

In [12]: m.modified
Out[12]: True

In [13]: m.context
# None

Of course if we call m.update() we catch No operation to send exception.
It means we know we have nothing to commit but we set the modified flag to true and keep the stack of local changes that will never ever be commited. Why?

Updated pull request regarding my last comment.

Hi! Any updates?

Hmmm... Looks like it tried to get merged on April 16th, but I see no successful merge messages.  Looks like our merge tool does not work on other repos.  Let me look at your subsequent changes now, too.

We've been successfully using this patch for more than half a year.
Any chances for this to be merged into upstream?


Update sendTransfer to iterate over an array of coins

The approach taken in the PoC is again to only handle the case where we have a slice of sdk.Coin, if we are receiving a message that has the old token field populated, we create a slice of one item and execute the same code path.
implementation here

Closed by https://github.com/cosmos/ibc-go/pull/6189


Failed to find cgroup v2 RPi (alt)

Environmental Info:
K3s Version:
k3s version v1.24.6+k3s1 (a8e0c66d)
go version go1.18.
Node(s) CPU architecture, OS, and Version:
Linux node1 4.9.241-BPI-M5 #2 SMP PREEMPT Mon Jun 21 16:29:40 HKT 2021 aarch64 GNU/Linux
Cluster Configuration:
1 server. 4 agents. (4 RPI, 1 Banana Pi) The RPI's are running. FYI - The banana pi is running onhttps://wiki.banana-pi.org/Getting_Started_with_M5/M2Pro#Install_Image_to_EMMC with a 32GB microSD card as additional storage.
Describe the bug:
Working with a raspberry pi alternative (banana pi) when installing k3s with the k3s agent installs on the system and then fails to run with the following exit code:
Oct 09 15:47:34 node1 k3s[2888]: time="2022-10-09T15:47:34+01:00" level=fatal msg="failed to find cpu cgroup (v2)" Oct 09 15:47:34 node1 systemd[1]: k3s-agent.service: Main process exited, code=exited, status=1/FAILURE
Steps To Reproduce:

All commands run as root via sudo su -.

Pre-installation:

nano /boot/cmdline.txt added: cgroup_enable=cpuset cgroup_memory=1 cgroup_enable=memory ip=<IP_ADDRESS>::<IP_ADDRESS>:<IP_ADDRESS>:node1:eth0:off

nano /boot/config.txt added: arm_64bit=1

Installed K3s agent:
Command run as root via sudo su -: curl -sfL https://get.k3s.io | K3S_TOKEN="MyTokenFromServer" K3S_URL="https://[Server IP]:6443" K3S_NODE_NAME="node1" sh -

Expected behavior:

Expected to install and automatically start the k3s service
Actual behavior:

k3s service failed to load due to cgroup v2 not foud
Additional context / logs:

Full errors from systemctl status k3s-agent.service
● k3s-agent.service - Lightweight Kubernetes
Loaded: loaded (/etc/systemd/system/k3s-agent.service; enabled; vendor preset: enabled)
Active: activating (start) since Sun 2022-10-09 15:47:33 BST; 122ms ago
Docs: https://k3s.io
Process: 2872 ExecStartPre=/bin/sh -xc ! /usr/bin/systemctl is-enabled --quiet nm-cloud-setup.service (code=exited, status=0/SUCCESS)
Process: 2886 ExecStartPre=/sbin/modprobe br_netfilter (code=exited, status=0/SUCCESS)
Process: 2887 ExecStartPre=/sbin/modprobe overlay (code=exited, status=0/SUCCESS)
Main PID: 2888 (k3s-agent)
Tasks: 6
Memory: 10.3M
CGroup: /system.slice/k3s-agent.service
└─2888 /usr/local/bin/k3s
Oct 09 15:47:33 node1 systemd[1]: Starting Lightweight Kubernetes...
Oct 09 15:47:33 node1 sh[2872]: + /usr/bin/systemctl is-enabled --quiet nm-cloud-setup.service
Oct 09 15:47:33 node1 sh[2881]: Failed to get unit file state for nm-cloud-setup.service: No such file or directory
Oct 09 15:47:34 node1 k3s[2888]: time="2022-10-09T15:47:34+01:00" level=info msg="Starting k3s agent v1.24.6+k3s1 (a8e0c66d)"
Oct 09 15:47:34 node1 k3s[2888]: time="2022-10-09T15:47:34+01:00" level=fatal msg="failed to find cpu cgroup (v2)"
Oct 09 15:47:34 node1 systemd[1]: k3s-agent.service: Main process exited, code=exited, status=1/FAILURE
Oct 09 15:47:34 node1 systemd[1]: k3s-agent.service: Failed with result 'exit-code'.
Oct 09 15:47:34 node1 systemd[1]: Failed to start Lightweight Kubernetes.
journalctl -xe
`Oct 09 16:04:26 node1 systemd[1]: Starting Lightweight Kubernetes...
░░ Subject: A start job for unit k3s-agent.service has begun execution
░░ Defined-By: systemd
░░ Support: https://www.debian.org/support
░░
░░ A start job for unit k3s-agent.service has begun execution.
░░
░░ The job identifier is 13939.
Oct 09 16:04:26 node1 sh[7640]: + /usr/bin/systemctl is-enabled --quiet nm-cloud-setup.service
Oct 09 16:04:26 node1 sh[7641]: Failed to get unit file state for nm-cloud-setup.service: No such file or directory
Oct 09 16:04:26 node1 k3s[7645]: time="2022-10-09T16:04:26+01:00" level=info msg="Starting k3s agent v1.24.6+k3s1 (a8e0c66d)"
Oct 09 16:04:26 node1 k3s[7645]: time="2022-10-09T16:04:26+01:00" level=fatal msg="failed to find cpu cgroup (v2)"
Oct 09 16:04:26 node1 systemd[1]: k3s-agent.service: Main process exited, code=exited, status=1/FAILURE
░░ Subject: Unit process exited
░░ Defined-By: systemd
░░ Support: https://www.debian.org/support
░░
░░ An ExecStart= process belonging to unit k3s-agent.service has exited.
░░
░░ The process' exit code is 'exited' and its exit status is 1.
Oct 09 16:04:26 node1 systemd[1]: k3s-agent.service: Failed with result 'exit-code'.
░░ Subject: Unit failed
░░ Defined-By: systemd
░░ Support: https://www.debian.org/support
░░
░░ The unit k3s-agent.service has entered the 'failed' state with result 'exit-code'.
Oct 09 16:04:26 node1 systemd[1]: Failed to start Lightweight Kubernetes.
I've ensured that cgroup2 is mounted via mount : cgroup2 on /sys/fs/cgroup type cgroup2 (rw,nosuid,nodev,noexec,relatime)
I attempted to apt-get install cgroupfs-mount and ran cgroupfs-mount the output is:
mount: /sys/fs/cgroup/memory: cgroup already mounted on /sys/fs/cgroup/cpuset. mount: /sys/fs/cgroup/pids: cgroup already mounted on /sys/fs/cgroup/cpuset.
Checked to see the cgroup.controllers with  cat /sys/fs/cgroup/cgroup.controllers resulting in:
memory pids (I expected to see cpuset here, but that could be because I don't know any better)

failed to find cpu cgroup

It sounds like your system is using hybrid (mixed cgroup v1/v2) and the cpu controller is still on v1. You should disable the hybrid configuration, and go for pure v1 or v2.

I'm chasing down some similar behavior on my Jetson Nano's @BrandWest I'll let you know what I find - I think it's related to the cgroup kernel params

@BrandWest Seems like this is what ended up sorting out my installations - https://kubernetes.io/docs/concepts/architecture/cgroups/#linux-distribution-cgroup-v2-support
I'm still playing with any potential additional cgroup cmdline flags but this took care of the error you reported and I experienced.

failed to find cpu cgroup

It sounds like your system is using hybrid (mixed cgroup v1/v2) and the cpu controller is still on v1. You should disable the hybrid configuration, and go for pure v1 or v2.

I will attempt this and see where I get.
I will also follow up with @VestigeJ 's suggestion

@brandond according to my stat -fc %T /sys/fs/cgroup/ and [ $(stat -fc %T /sys/fs/cgroup/) = "cgroup2fs" ] && echo "unified" || ( [ -e /sys/fs/cgroup/unified/ ] && echo "hybrid" || echo "legacy") I am running cgroupsfs and unified respectively.
I've attempted to add other flags requiring cgroup v2s including: systemd.unified_cgroup_hierarchy=1 cgroup_no_v1=all cgroup_enable=cpu cgroup_enable=cpuset cgroup_cpu=1 cgroup_cpuset=1 cgroup_enable=memory cgroup_memory=1
From my research hierarchy=1 forces cgroup v2 and cgroup_no_v1=all should stop cgroup 1's from being used.
However, my exit code continues to be "Failed to find CPU Cgroup (V2)".
Additionally I followed @VestigeJ 's suggestion and after getting all the proper packages installed I still have the same issue.

What platform is this on? I have K3s running on several Pi4bs with Ubuntu with no problems or special configuration necessary. Is this another Jetson or off-brand Pi clone board? I wonder if there's something odd in their kernel configuration?

It is an off brand Pi clone, a Banana Pi.

Yeah, I suspect that there are features missing from their kernel build.

After a bunch of issues I was able to install their version of Ubuntu and install k3s.

After a bunch of issues I was able to install their version of Ubuntu and install k3s.

Were they issues with the manufacturer? Or were you able to resolve it on your own?


feat: add chainflip

Hey @morelazers, could you please contact me on Element? Thanks in advance!

@morelazers please merge master


WAP in C++ to delete a node in binary search tree considering all cases!!!

💥 Proposal
I would like to contribute to this issue .Please assign it to me under GSSOC'21.
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
YES!

/assign


Unboxing fails without 2nd unboxing

I have a bizarre unboxing issue in my test cases. If I make a call to wtf() below, everything compiles and the test runs correctly. If I comment out wtf() I get a Type 'any Unit' (aka 'any Unit') cannot conform to 'Unit'.
static var testValues: [(unit: any Unit, abbreviated: String, narrow: String, wide: String)] {
	[
		(EnergyUnit.kilowattHours, "kW·hs", "kWh", "kilowatt-hours"),
	]
}

func test_formatted__Unit_FormatStyle() {
	func test(unit: some Unit, abbreviated: String, narrow: String, wide: String) {
		XCTAssertEqual(unit.formatted(.unit(width: .abbreviated, locale: locale)), abbreviated)
		XCTAssertEqual(unit.formatted(.unit(width: .narrow, locale: locale)), narrow)
		XCTAssertEqual(unit.formatted(.unit(width: .wide, locale: locale)), wide)
	}

	func wtf(unit: some Unit) {
	}

	// Returns a properly formatted string
	Self.testValues.forEach {
		wtf(unit: $0.unit) // comment this out and the code fails to compile

		test(unit: $0.unit, abbreviated: $0.abbreviated, narrow: $0.narrow, wide: $0.wide)
	}
}

Environment
swift-driver version: 1.75.2 Apple Swift version 5.8 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>.100)
Xcode 14.3
Build version 14E222b

wtf() can also be called after test() and everything will still compile.

Reduced:
protocol P {}

func unbox(_: some P) {}

let _: ((any P, Void)) -> Void = { tuple in
//  unbox(tuple.0)
  unbox(tuple.0)
}

Swift version 5.9-dev (LLVM 91080e65df791d6, Swift c1d5118c21da49a)

This still exists in Xcode Version 15.0 beta (15A5160n)

This is still an issue in Xcode Version 16.0 beta 4 (16A5211f)


Convert readthedocs links for their .org -> .io migration for hosted projects

As per their blog post of the 27th April ‘Securing subdomains’:

Starting today, Read the Docs will start hosting projects from subdomains on the domain readthedocs.io, instead of on readthedocs.org. This change addresses some security concerns around site cookies while hosting user generated data on the same domain as our dashboard.

Test Plan: Manually visited all the links I’ve modified.

More :cake: Thanks @adamchainz.

🍽


AWS account without default VPC

Hello @Young-ook,
Your module is great but I have small problem on account where i dont have default VPC. I have working 2 VPCs on this account but without default one and i get this error after terraform plan: Error: no matching VPC found. On second account where we have defualt vpc configured everything is working fine. The problem is that we cant create additional VPC and make it default (routings, etc. ...).
Thanks
Kamil

Hi Kamil,
FYI, This module uses the default vpc for fail-safe if the user does not define the subnets to install eks on, thanks!
https://github.com/Young-ook/terraform-aws-eks/blob/64abacdc341d47dfbbd3842fee5380b59eac5772/network.tf#L13

This issues was fixed in version 1.7.3.
https://github.com/Young-ook/terraform-aws-eks/releases/tag/1.7.3


readme --check example appears broken

Running the exact command within the provided examples
$ rustup update
$ rustup add component rustfmt
$ cargo fmt --all -- --check

Results in an error message where cargo fmt will display its possible flags instead of an error concerning formatting.
Is there a documented work around?

Hmm, strange...would you tell us the output of rustfmt --version? rustup somehow may have failed to overwrite the old rustfmt and cargo-fmt. Or could you try removing rustfmt and cargo-fmt manually, then re-run rustup update please?


Allow for wildcard of the namespace in the IAM condition for namespaces with a pattern

What would you like to be added:
Reference to this issue: https://github.com/aws/amazon-eks-pod-identity-webhook/issues/58
We'd like to have wildcard implemented for incomplete namespaces, for example:
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "",
            "Effect": "Allow",
            "Principal": {
                "Federated": "arn:aws:iam::xxx:oidc-provider/oidc.eks.ap-southeast-1.amazonaws.com/id/xxx"
            },
            "Action": "sts:AssumeRoleWithWebIdentity",
            "Condition": {
                "StringEquals": {
                    "oidc.eks.ap-southeast-1.amazonaws.com/id/xxx:aud": "sts.amazonaws.com"
                },
                "StringLike": {
                    "oidc.eks.ap-southeast-1.amazonaws.com/id/xxx:sub": [
                        "system:serviceaccount:some-namespace-*:my-sa"
                    ]
                }
            }
        }
    ]
}

In the current state of things, implementing it the way shown above will result in a failure to assume the role, with an error message "An unknown error occurred" reported via Cloudtrails.
Why is this needed:
Our namespace model follows a pattern with which such a feature would allow us to specifically grant permissions on SAs in namespaces following this pattern.

Up for this functionality also

+1 We have a similar scenario where we launch ephemeral envs with unique namespaces.

Just want to add that we also tried system:serviceaccount:*:my-sa and it's not working for us it seems either. We are using KOPS to create the role, which looks correct (Fixed in https://github.com/kubernetes/kops/issues/16027).

I could also really use this functionality. Not being able to use wildcard patterns in the namespace has completely prevented my organization from moving forward with transition to this feature from the standard IRSA method. With multiple teams deploying to the cluster, often to feature branches that live in their own namespaces, expecting them to first update their EKS Pod Identities with the new namespace first is a massive inconvenience.

We need that either. +1!

+1 This would be great for preview environments

Needs this feature for ephemeral environments

+1 need this feature for ephemeral environments


OnNavigatedToAsync with x:Bind

I have just faced a problem with the INavigable.OnNavigatedToAsync method. If I do multiple long running tasks in the ViewModelBase.OnNavigatedToAsync override the UI is not updated after the first one because the second awaited task blocks. For example:
public override async Task OnNavigatedToAsync(object parameter, NavigationMode mode, IDictionary<string, object> suspensionState)
{
    Something1 = await _someService.FirstLongRunningTask();

    await SecondLongRunningTask();

    await OtherLongRunningTask();

    await base.OnNavigatedToAsync(parameter, mode, suspensionState);
}

I want to update the UI with the Something1 property's changes. Also the SecondLongRunningTask() update some property whose updates I want to see in the UI before the method is fully awaited.
This problem is caused by the NavigationService's implementation (https://github.com/Windows-XAML/Template10/blob/337047cd1d66b083bf74c4087e2a2366ef997324/Template10 (Library)/Services/NavigationService/NavigationService.cs#L180-L184)
The NavigationService initializes the compiled bindings after the OnNavigatedToAsync call.
Please consider to provide a way to initialize compiled bindings before the OnNavigatedToAsync.
Maybe it's also related with the #1261 discussion.

I think the error is that it should NOT Initialize(). What do you think?

I took further investigation and I found out that this bug only appears when we using dependency injection so when we do not set the Page.DataContext explicit in the xaml.cs. I will dive into the code for the reason,
ps.: I'm using x:Bind and DI together with this helper property:
public sealed partial class DetailPage : Page
{
    public DetailPageViewModel ViewModel => (DetailPageViewModel) DataContext;

    public DetailPage()
    {
        InitializeComponent();
    }
}

What do you think? Do I create a PR for the 10.2 codebase or wait until the stabilization?


Add Fedora dependency install instructions.

Adds Fedora instructions for building from source.

Spelling fix

Alright, looks good with dnf. Thanks for compiling the list of deps :)

Thanks for the info regarding yum 🙂


Send command to device

Assuming that the matterHub has been added to the ACL of your target device by the commissioner, send a command that modifies the devices state.

Wo wollen wir den Code dafür einbinden?

Yeah, we should look into code organization some more. I found this when looking for examples on how to include a static library target in the final binary.
I also don't quite understand where the gn calls happens right now.
I've always worked exclusively with targets, as this is the recommondation for modern CMake however since we are not using CMake directly but ESP-IDF, we might have to compromise on that.


Anki needing an uninstall first was a one-time migration

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/109579&drop=dogfoodAlpha

Won't other users still need to uninstall the previous version when upgrading?


Auto-generated brief summary for the open-source code repository.

Dear repository maintainers,
I am developing a model designed to automatically generate concise and accurate summaries for open-source repositories. This model is intended to produce a one-sentence summary that briefly describes the main functions and purposes of the project. To validate the effectiveness of this summary and to further refine my model, I propose to trial my autogenerated summary in summary.md file.
The purpose of adding this summary is to quickly convey the core value of the repository in a manner that is genuinely useful to readers. Please evaluate the accuracy and practicality of this summary by scoring it on a scale from 1 to 10(1 is the worst and 10 is the best), assessing how well it succinctly communicates the project's essence. Your feedback will be crucial in helping me improve the accuracy and relevance of the model.
The submission attached includes the summary generated by my model. Please review this description and provide your scoring and any specific feedback. If you have any questions or require additional information, feel free to contact me at any time.
Thank you very much for your valuable time and support!
Best regards,
Zheng Guan

Summary properly describes the purpose of the Confluence Publisher, but it does not add the designed and documented constraint that it aims to focus on technical documentation rather than supporting each and every Asciidoc or Confluence feature. 8/10

Oh, thank you very much. I will finetune my model.

Oh, thank you very much. I will finetune my model.