Use luaL_ref references instead of custom hashtable for objects stored in C++ data structures of Mewa

Check if
int r = luaL_ref(L, LUA_REGISTRYINDEX);
can be used to store Lua objects (like constructors) in C structures of Mewa instead of storing them in hashtables.
This can be especially useful in the case of set_instance.

Use
lua_pushvalue( ls, -1);
 = luaL_ref( ls, LUA_REGISTRYINDEX);
and
lua_rawgeti( ls, LUA_REGISTRYINDEX, );
instead of Lua table with custom counters as keys for object references stored as integers in the C++ structures of Mewa.


Add ics manager command line app

$ ics-manager
Traceback (most recent call last):
  File "/home/naoya/.local/bin/ics-manager", line 5, in <module>
    from rcb4.apps.ics_manager import main
  File "/home/naoya/ros/kxr/src/rcb4/rcb4/apps/ics_manager.py", line 8, in <module>
    from rcb4.ics import ICSServoController
  File "/home/naoya/ros/kxr/src/rcb4/rcb4/ics.py", line 6, in <module>
    import readchar
ModuleNotFoundError: No module named 'readchar'

--- Servo Status ---
>> Current Servo ID: 17 -> Next Servo ID: [0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 ]
   Angle: 7504
   Baud Rate: 1.25Mbps
   Rotation Mode: Disabled
   Slave Mode: Disabled
   Reverse Mode: Disabled
   Serial Mode: Disabled
   Free: Disabled
----------------------

Use ↑↓ to navigate, ←→ to adjust Current Servo ID or Servo Angles
Press 'Enter' when Current Servo ID is selected to save the currently highlighted ID in green.
Press 'z' to reset servo position
Press 'r' to toggle rotation mode (enables continuous wheel-like rotation)
Press 'f' to set free mode

'q' to quit.


How to add image at the tip of bar in barchart in chart.js

HI ,
I was wondering if it is possible to add images at the tip of the particular bars in barchart like below image

As you can see there is E image on the top of two bars. Is it possible in chart.js?
If not is there any way to put a tag above particular bars . I just need a way to notify user.
Thank you

Please ask questions on StackOverflow: https://stackoverflow.com/questions/tagged/chart.js


⚠ ClamAV has degraded performance

In 4577f92, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 200 ms

Resolved: ClamAV performance has improved in 18f30f7.


Add licence

I would like to port the code from rename-file.ts to a Lua snippet to include in my config, but without installing the whole plugin.
I can't do this at the moment because this project has no licence and therefore automatically falls under "all rights reserved".
(In fact, even users of this plugin are doing so in contravention of copyright law, since there is no automatic right of usage.)

Added in 1add655c7ecf1f61df9a40b43973b0394d9ae9c4.


Can't upgrade to @nuxt/scripts ^0.9.0 because peerDependencies from @nuxtjs/turnstile is: "@nuxt/scripts": "^0.6.0 || ^0.7.0 || ^0.8.0"

🐛 The bug
basically the successor of: https://github.com/nuxt-modules/turnstile/issues/327
 npx nuxi@latest module add turnstile
ℹ Installing @nuxtjs/turnstile@latest development dependency                                      2:43:36 PM
npm error code ERESOLVE
npm error ERESOLVE unable to resolve dependency tree
npm error
npm error While resolving: outletcars-frontend-app@undefined
npm error Found<EMAIL_ADDRESS>npm error node_modules/@nuxt/scripts
npm error   dev<EMAIL_ADDRESS>from the root project
npm error
npm error Could not resolve dependency:
npm error peer<EMAIL_ADDRESS>|| ^0.7.0 || ^0.8.0" from<EMAIL_ADDRESS>npm error node_modules/@nuxtjs/turnstile
npm error   dev<EMAIL_ADDRESS>from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.
npm error
npm error
npm error For a full report see:
npm error /home/beb/.npm/_logs/2024-09-16T12_43_36_147Z-eresolve-report.txt
npm error A complete log of this run can be found in: /home/beb/.npm/_logs/2024-09-16T12_43_36_147Z-debug-0.log

 ERROR  Command failed with exit code 1: npm install -D @nuxtjs/turnstile@latest                   2:43:37 PM

  at makeError (node_modules/nuxi/dist/chunks/index7.mjs:578:11)
  at handlePromise (node_modules/nuxi/dist/chunks/index7.mjs:1504:26)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  at async executeCommand (node_modules/nuxi/dist/shared/nuxi.e6ef3ea5.mjs:43:3)
  at async addDependency (node_modules/nuxi/dist/shared/nuxi.e6ef3ea5.mjs:208:3)
  at async Object.setup (node_modules/nuxi/dist/chunks/add2.mjs:145:19)
  at async runCommand$1 (node_modules/nuxi/dist/shared/nuxi.b8b195e1.mjs:1620:5)
  at async runCommand$1 (node_modules/nuxi/dist/shared/nuxi.b8b195e1.mjs:1639:11)
  at async runCommand$1 (node_modules/nuxi/dist/shared/nuxi.b8b195e1.mjs:1639:11)
  at async runMain$1 (node_modules/nuxi/dist/shared/nuxi.b8b195e1.mjs:1777:7)

✔ Install failed for @nuxtjs/turnstile@latest.

@danielroe Do we really need that strict peerDep restrictions now that Nuxt Scripts is released?  I'm asking you because you made the last PR to restrict it to 0.6-0.8 here https://github.com/nuxt-modules/turnstile/pull/328
🛠 To reproduce
https://stackblitz.com/github/nuxt-modules/turnstile/tree/main/playground
🌈 Expected behaviour
I expect not being stuck on an outdated version of a peerDep if there's no need
ℹ Additional context
No response

I've published a path extending the supported version range once more, but I keep this issue open for Daniel to answer your question.

Yes, it is still needed. It’s possible there will be breaking changes until it released stably as v1.
We’ll do our best to release quickly when it bumps version.


Use a more restrictive Java regexp to match class name.

The regexp found in the BaseAssertionGenerator tries to match the Java class name. However, it read too much thing:
public class SoftAssertions extends org.assertj.core.api.SoftAssertions {
-------------^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The change to the regexp will ensure that we read only a valid java identifier, compromised of two part (see Character.isJavaIdentifierStart.
The \b before is useless, as we are expecting space before.
Note:  this is required for the unit test of maven plugin to work (I'll do another PR about that).

thanks @glhez for the PR.

Is that good for you ? I also used a name pattern instead of relying on the first group. That would allow for more tweak in the future, eg: if you want to match class / interface.

I can do it, but you ought to remember that a developer should read the manual first :)
Someone who need to understand the regexp would have to read https://docs.oracle.com/javase/8/docs/api/java/util/regex/Pattern.html

That is true ... except for regex ;-)

Hello, Is my last comment good enough ?


feat(stable): add stable screen

resolves #1130
resolves #1132
resolves #1133
fixes  #1145
https://github.com/get10101/10101/assets/382048/effb7953-441c-49bb-9c82-6b1629927bc1

Next I'd love to see the USD balance in the account overview.

I've added it into iteration: https://github.com/get10101/10101/issues/1153

Preparation for https://github.com/get10101/meta/issues/216


Use max values instead of zeros in reset context

Description
Sets most values context values to non-zero by default.
Todo:

[ ] Similar treatment for the reentrancy guard.

Security consideration:
There is a danger that one of these values is read before it is set. This was the case with the ovmGasRefund value, so we need to ensure there are no other such cases.
Metadata
Fixes

Fixes https://github.com/ethereum-optimism/roadmap/issues/641

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Would be great if we could add tests that show the amount of gas saved with these optimizations

This is ready for final review, once #263 has been merged.

I think this is obsoleted by other work. Closing for now.


Change fee setting to two-way setter instead of two different function.

Pull Request Test Coverage Report for Build 2168

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.871%

Totals

Change from base Build 2165:
0.0%

Covered Lines:
563

Relevant Lines:
563

💛  - Coveralls


[13.0] ISPN-14103 Use supplied security provider to load keystore and init key/trust managers

https://issues.redhat.com/browse/ISPN-14103 Use supplied security provider to load keystore and init key/trust managers
https://issues.redhat.com/browse/ISPN-14114 Fix TLS 1.3 with cipher suites

@tristantarrant  there is a failure in CI.

Once https://github.com/infinispan/infinispan/pull/10281 is merged, I'll update this

CI is ok

Thanks @tristantarrant


Python debugger crashes instantly

VSCode Version: Code - Insiders 1.17.0-insider (b0c0634339c6f64a5c7f08b56fe732fb9154d3c4, 2017-09-15T05:24:23.107Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

python
don
0.7.0

cpptools
ms-
0.12.4

code-spell-checker
str
1.4.0

Steps to Reproduce:

open a folder.
create a python file, with these contents.x = int(input("Enter your x"))
y = int(input("Enter your y"))
z = x + y
if z < 5:
    print("Addition to less than 5")
else:
    print("More than 5.")
print(z)

open debug launch. Use autocomplete to complete the external debugger launch.
press f5 in the python file.
Expected: External terminal with printouts and such.
Actual: Terminal comes up very briefly, then boom, disappears. additionally, my default python file assosiation was at one point vs code, and a python file was launching in vs code as soon as I pressed f5 (Assumed to be used to spawn the debugger). I set python 3 as my default python handler, but still, not sure what's going on.

Reproduces without extensions: Yes/No Not applicable.

If I could get the thing to stay open I could get the traceback I think must be happening.

wo, that's some cool bot magic.

Please file this against the vscode python extension
https://github.com/DonJayamanne/pythonVSCode
Thank you

sweet, thanks.


Fix dependencies between fixie-sidekick-template and fixie SDK

Do we care about this template anymore?

We might not. Haven't looked at this in a while and it might just make sense to drop.

ok. going to move it out of this iteration accordingly.


get_verified_jwt requires a request to check query args

This fixes a bug in #80 where we couldn't actually check query args due to not passing in the request. (This time it's been tested with a live app. :smile: )

Codecov Report

Merging #85 into master will decrease coverage by <.01%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   84.15%   84.14%   -0.01%
==========================================
  Files          40       40
  Lines        5573     5577       +4
==========================================
+ Hits         4690     4693       +3
- Misses        883      884       +1

Impacted Files
Coverage Δ

endpoints/test/users_id_token_test.py
98.58% <100%> (ø)
:arrow_up:

endpoints/users_id_token.py
87.22% <66.66%> (-0.24%)
:arrow_down:

Continue to review full report at Codecov.

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


Move keydown close event to element

Keeps the listener off of the document for better UX.
Closes https://github.com/gregnb/mui-datatables/issues/1203.

Coverage decreased (-0.01%) to 76.426% when pulling bc77e64f503618b2e3d851d9486dfb992b3ed976 on enhancement/issue-1203-search-keydown-close into 4f836096a56be93ee1322f3c98cbaca485f0b064 on master.


Add devenv-up as a package (fix #711)

This PR adds devenv-up as a package so that it can be created from nix build, fixing #711

Could you explain how it works?

The original flake expression devShells.${pkgs.stdenv.system}.default.config.procfileScript does not exist. This PR instead put a package to packages.${pkgs.stdenv.system}.devenv-up that can be passed to nix build.

I was honestly surprised to see the use of devShells.${pkgs.stdenv.system}.default.config.procfileScript.
At least, it was new to me and I still can't decide if it's supposed to work (but it doesn't at the moment) or not.
It came up in #739 as a way to print devenv info as well.


Include full or no license-text in podspec

Currently the podspec directs to the README file:
s.license = { :type => 'MIT', :text => '@see README' }

however, if you want to use CocoaPods auto-generated acknowledgement files Pods-acknowledgements.markdown or Pods-acknowledgements.plist this is suboptimal because it only contains '@see README' which makes the whole acknowledgement file unusable.
Instead, simply remove the :textpart and add a dedicated LICENSE file.

I have changed it to a url to the README.

This isn't better as the previous solution. Now the acknowledgement plist will contain a URL but the MIT licence requires inclusion of the full licence text:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

So people using PromiseKit will not be able to use the auto-generated acknowledgement plist.

I wish I'd just gone with a license without restrictions.


fix issue with calling get_balanced in the wrong table

This fixes an issue where get_balanced would be called in a table that isn't [features] or a dependency table.
To reproduce place below in a Cargo.toml above [features] and try building the docs
[package.metadata.release]
pre-release-replacements = [
  {test="## "},
]

close #10

Thanks for the patch!


http://unirest.io/nodejs doesn't work

The URL in the repo (http://unirest.io/nodejs) does not work

I took a look at http://web.archive.org and noticed that the last entries looked like something hacked


frameit depends on newer version of Mini Magick

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Frameit is crashing for me with the new iPhone SE images.  Seems like it's depending on a Mini Magick call that was added in v4.2.0.  The gemspec only requires 4.0.2
Complete output when running fastlane, including the stack trace and command used
12:46 $ frameit silver
[12:49:55]: Converting PSD file '/Users/rod/.frameit/devices_frames/iPhone SE/iPhone-SE-Silver-horizontal.psd'
[12:49:55]: Removing white background 🚫 ⬜
Successfully submitted a crash report. If this is a problem with one of the tools specifically,
please submit an issue on GitHub and attach the following number to it: '64181bced9e89310f98e73caa0b71b92'
The crash report has been stored locally '/tmp/sentry_64181bced9e89310f98e73caa0b71b92.json'
/Users/rod/.rvm/gems/ruby-2.2.4/gems/mini_magick-4.0.4/lib/mini_magick/image.rb:382:in `block in method_missing': [!] undefined method `details' for #<MiniMagick::Image:0x007fd323d76df8> (NoMethodError)
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/mini_magick-4.0.4/lib/mini_magick/image.rb:499:in `block in mogrify'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/mini_magick-4.0.4/lib/mini_magick/tool.rb:52:in `new'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/mini_magick-4.0.4/lib/mini_magick/image.rb:492:in `mogrify'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/mini_magick-4.0.4/lib/mini_magick/image.rb:378:in `method_missing'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/frame_converter.rb:76:in `block (2 levels) in convert_frames'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/frame_converter.rb:75:in `reject'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/frame_converter.rb:75:in `block in convert_frames'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/frame_converter.rb:61:in `each'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/frame_converter.rb:61:in `convert_frames'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/lib/frameit/runner.rb:10:in `initialize'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/bin/frameit:42:in `new'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/bin/frameit:42:in `block (2 levels) in run'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/commander-4.3.5/lib/commander/command.rb:153:in `run'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/commander-4.3.5/lib/commander/runner.rb:428:in `run_active_command'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/fastlane_core-0.43.0/lib/fastlane_core/ui/fastlane_runner.rb:23:in `run!'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/commander-4.3.5/lib/commander/delegates.rb:15:in `run!'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/bin/frameit:57:in `run'
	from /Users/rod/.rvm/gems/ruby-2.2.4/gems/frameit-2.6.0/bin/frameit:72:in `<top (required)>'
	from /Users/rod/.rvm/gems/ruby-2.2.4/bin/frameit:23:in `load'
	from /Users/rod/.rvm/gems/ruby-2.2.4/bin/frameit:23:in `<main>'
	from /Users/rod/.rvm/gems/ruby-2.2.4/bin/ruby_executable_hooks:15:in `eval'
	from /Users/rod/.rvm/gems/ruby-2.2.4/bin/ruby_executable_hooks:15:in `<main>'

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
# INSERT CONTENT HERE

Environment
fastlane version (run fastlane -v):
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)?
no
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)?
rvm

Thanks for the report! This will be fixed in the next release with https://github.com/fastlane/fastlane/pull/4483

@rodbegbie this just went out with frameit 2.6.1, could you give it a try and let me know if it fixed it for you? Thanks!

Thanks!  Sadly, bundler is unhappy because it clashes with the mini_magick version requested by badge  Guess I'll have to file a bug on their repo.
Bundler could not find compatible versions for gem "mini_magick":
  In Gemfile:
    badge was resolved to 0.0.1, which depends on
      mini_magick (~> 4.0.2)

    frameit (~> 2.6.1) was resolved to 2.6.1, which depends on
      mini_magick (~> 4.5.1)

Hi @rodbegbie! Thanks for following up. Badge pushed a change this morning, can you check against that? Thanks!

Everything looks great now  Thanks for the help!

@rodbegbie Awesome! So glad to hear you are back up and running 👍


refresh controllers don't always refresh when comfy refreshes node definitions

to pick up new options in the loaders

I’d really like to make the refresh button redundant anyway, by picking up
changes as they happen. Some of the stuff I’ve been doing recently has been
towards making that possible.
On Mon, 4 Nov 2024 at 11:44 AM, JorgeR81 @.***> wrote:

Do you think this could replace the Controller's refresh button  ?
We could just click the ComfyUI's refresh button, instead.
AIso, I think using multiple controllers, in the whole canvas, is going to
be a major use case.
And, for that, we don't need a refresh button, because we would toggle the
Controller, to make workflow changes.
—
Reply to this email directly, view it on GitHub
https://github.com/chrisgoringe/cg-controller/issues/268#issuecomment-2453655730,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBLMA4IZMQWCV5TKBGRB33Z627PZAVCNFSM6AAAAABRCKOBLGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGY2TKNZTGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

done on dev
also - refresh button now gone. I think it's now redundant.

I'm on dev, and updated, but I don't have this.

do you still have the update button?

If you have Use Everywhere nodes installed, could you make sure they are updated as well? There might be a conflict.

do you still have the update button?

Yes, now it's gone.
The Comfy UI refresh also works in the Controller.

Ok, you must have been one update behind :).

I tested again and it was working, but it was a bit slow to update.
So I restarted ComfyUI, to see if it would improve.
But after restarting, it did not work again.
I have this log after refreshing the nodes, when it does not work.

If you still have the problem, could you next time scroll back up the log to see if there are any other messages mixed in with the 'replacing...' ones, and also to see what is immediately before they start?

could you next time scroll back up

Before clicking I cleared the console.
These are all the messages that show up when I refresh the nodes.
( You can see that there is no scroll bar in the console, for these messages )

what browser?
There definitely should be more messages off the top - one 'replacing node type' line gets generated for each node type when you refresh.. I don't have a lot of custom nodes and I get 450 'replacing' messages!

Just had an idea as to what might be going on. Could you update from dev and see how it behaves now?

I updaded all.
I tried in Chrome and Edge, and still have this issue.
Here is the log in Chrome, when I refresh the nodes.
It's much larger now ( see the scroll bar ).
Here is top an botton scroll:

( ... )

Ok, I don't know why it's happening, but at least now I know what is happening! The update request is there right at the top, even though it's after an await command. So it's refreshing before the nodes get updated.
I can do a hack solution, I think, by adding another update with (say) a two second delay. But I'd really like to work out what's going on!

Wait ! ... now it works. And its fast !
( I don't recall changing anything ?! )
Here is the log:

( ... )

I restarted ComfyUI and it's broken again !
I don't know what I did to make it work before ?!

Those on_change lines are what I get all the time, don't know why you sometimes don't get them. I think there must be some timing issue that determines whether or not it gets initialised properly.
That's at least a hint to follow...

I think there must be some timing issue

Yeah, yesterday, when it worked, it had about a 2 second delay, after clicking the button.
This last time today, when it worked, it has no noticeable delay.

hopefully now it will refresh after a delay if it doesn't initially....

Unfortunately, it is still not working.
Log:

( ... )

Could you set a breakpoint for me? To do this, in the dev tools window, find the "Source" tab, find controller.js, go to line 121, and click just to the left of the '121' (where the left red dot is in the image - the dot should appear). Then press the refresh button in Comfy.
What ought to happen is that the browser will pause with a message like "Paused in debugger", and that line will be highlighted in the dev tools window. You can then click the red dot again to make clear it, and press the 'play' button (or just reload the page)

Then, as another test, I'd like you to go to the console and type app.refreshComboInNodes and screenshot what it says. Like this:

(btw, what browser are you using?)

I will be using Chrome, to test this.
I can also try Edge

Test 1 - on Chrome.
I've found the line, click on it and a blue arrow shows up.
But when I refresh Comfy, nothing happens.
I've also ticked the 2 checkboxes in the breakpoints section, but still nothing happens.

jsut confirming - when you 'refresh comfy', you mean clicking the refresh button, yes?

jsut confirming - when you 'refresh comfy', you mean clicking the refresh button, yes?

Yes.
And I also tried test 1 on Wdge, and nothing happens.
I will try test 2

test 2

OK, getting close. I just need you to do a search... in Edge you do it like this: in the 'Sources' tab, right click on 'extensions' and select 'Search in Folder'

A search area will pop up at the bottom: type file:extensions refreshComboInNodes into it (file:extensions will probably already be there)

Screenshot the result (you might need to scroll to see everything)

There it is:
( test 2 as the same result in Edge, by the way )

Thanks. I'm fairly sure I now know what's happening. Do a git pull and see if it works for you

It works !

OK, it was a clash with ComfyUI-Custom-Scripts. They've hijacked the refreshComboOnNodes, which we were also doing, but in a way that wasn't safe.
But in the process I've discovered that there is actually an 'official' way to do it, which we are now using.

I just tried in the new Frontend and it also works.

I also just noticed that today they moved the refresh button to a drop down list !
https://github.com/Comfy-Org/ComfyUI_frontend/releases/tag/v1.3.36

Yeah, I saw that.
I've just offered a PR on ComfyUI-Custom-Scripts to make the same change as I did.


Remove unnecessary subscribeStores from cluster overview

Same stores are already loaded in app.tsx.

Though shouldn't additional subscribes not matter? Especially since these will be disposed of before any dispose within app.tsx?

Should we also move subscribing to namespaceStore from namespace-select.tsx to app.tsx?
      kubeWatchApi.subscribeStores([namespaceStore], {
        preload: true,
      })
``


Blynk Sketch LED Status of Button

The sketch is good and working.
I need a revised sketch with an additional button
Need sketch monitoring 2 buttons.
I tried a lot since i am new to all this IoT concept could not do it myself.

Please use https://community.blynk.cc/


Remove ISDLProxy protocol

Fixes #70, fixes #19. This is based on the new distribution branch #57 and can be merged after that one is

Recreate against new develop branch

Closed in favor of #104


0.2.19

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.556%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
2549

Relevant Lines:
2754

💛  - Coveralls


[Snyk] Fix for 2 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

packages/node/base/package.json
packages/node/base/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-SEMVER-3247795
  696

Regular Expression Denial of Service (ReDoS) SNYK-JS-MINIMATCH-3050818
  479

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Checkmarx One – Scan Summary & Details – dac3f7f6-483a-4813-a6fd-6294f4e44218
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2023-34092
Npm-vite-4.3.1
Vulnerable Package

CVE-2024-23331
Npm-vite-4.3.1
Vulnerable Package

CVE-2024-27351
Python-Django-1.2
Vulnerable Package

CVE-2024-29180
Npm-webpack-dev-middleware-6.0.2
Vulnerable Package

CVE-2024-4068
Npm-braces-2.3.2
Vulnerable Package

CVE-2024-4068
Npm-braces-3.0.2
Vulnerable Package

Cx37f28a21-b2a7
Npm-angular-1.8-1.8.2
Vulnerable Package

Cx94a5c81d-63b1
Npm-angular-1.8-1.8.2
Vulnerable Package

Cx9eb96c9b-eb45
Npm-angular-mocks-1.8-1.8.2
Vulnerable Package

Cxcf770dcb-fd21
Npm-angular-1.8-1.8.2
Vulnerable Package

Sensitive Port Is Exposed To Entire Network
/ec2.tf: [118](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/ec2.tf# L118)
A sensitive port, such as port 23 or port 110, is open for the whole network in either TCP or UDP protocol

Sensitive Port Is Exposed To Entire Network
/ec2.tf: [118](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/ec2.tf# L118)
A sensitive port, such as port 23 or port 110, is open for the whole network in either TCP or UDP protocol

CVE-2024-28849
Npm-follow-redirects-1.14.6
Vulnerable Package

CVE-2024-28863
Npm-tar-6.1.11
Vulnerable Package

CVE-2024-28863
Npm-tar-4.4.19
Vulnerable Package

CVE-2024-31207
Npm-vite-4.3.1
Vulnerable Package

CVE-2024-4067
Npm-micromatch-3.1.10
Vulnerable Package

CVE-2024-4067
Npm-micromatch-4.0.4
Vulnerable Package

IAM Access Analyzer Not Enabled
/rds.tf: [1](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/rds.tf# L1)
IAM Access Analyzer should be enabled and configured to continuously monitor resource permissions

IAM Access Analyzer Not Enabled
/s3.tf: [1](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/s3.tf# L1)
IAM Access Analyzer should be enabled and configured to continuously monitor resource permissions

IAM Access Analyzer Not Enabled
/ecr.tf: [1](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/ecr.tf# L1)
IAM Access Analyzer should be enabled and configured to continuously monitor resource permissions

IAM Access Analyzer Not Enabled
/iam.tf: [1](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/iam.tf# L1)
IAM Access Analyzer should be enabled and configured to continuously monitor resource permissions

Fixed Issues

Severity
Issue
Source File / Package

CVE-2021-3765
Npm-validator-8.2.0

CVE-2022-25858
Npm-terser-5.10.0

CVE-2022-25858
Npm-terser-5.9.0

CVE-2023-28154
Npm-webpack-5.64.1

Cx37f28a21-b2a7
Npm-angular-1.8-1.8.2

Cx4615fedd-34f3
Npm-validator-8.2.0

Cx94a5c81d-63b1
Npm-angular-1.8-1.8.2

Cx9eb96c9b-eb45
Npm-angular-mocks-1.8-1.8.2

Cxab55612e-3a56
Npm-braces-3.0.2

Cxab55612e-3a56
Npm-braces-2.3.2

Cxca84a1c2-1f12
Npm-micromatch-3.1.10

Cxca84a1c2-1f12
Npm-micromatch-4.0.4

Cxcf770dcb-fd21
Npm-angular-1.8-1.8.2

Cxd312845a-fedc
Npm-validator-8.2.0

S3 Bucket SSE Disabled
/ec2.tf: [306](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/ec2.tf# L306)

S3 Bucket SSE Disabled
/s3.tf: [42](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/s3.tf# L42)

S3 Bucket SSE Disabled
/s3.tf: [65](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/s3.tf# L65)

S3 Bucket SSE Disabled
/s3.tf: [89](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/s3.tf# L89)

S3 Bucket SSE Disabled
/s3.tf: [1](https://github.com/demolab1/TerraformWebgoat/blob/snyk-fix-894db202fa21c85f63c39fc38eab2200//terraform/aws/s3.tf# L1)

CVE-2007-2379
Npm-jquery-3.6.0

CVE-2014-6071
Npm-jquery-3.6.0

CVE-2021-23566
Npm-nanoid-3.1.30

Cx14b19a02-387a
Npm-body-parser-1.19.0


wrong example for SUSE distributions

The document describe that you need to install all samba packager (sudo zypper install samba*)
This is NOT necessary. ONLY the package cifs-utils is needed for mounting with SUSE Linux Enterprise Server (SLES) and openSUSE as the upper text describes.

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

ID: 8d60d83f-0094-6c5e-e105-712c24bf1acc
Version Independent ID: 594af294-fd5d-a8c7-d456-6d753b7b62f8
Content: Use Azure Files with Linux
Content Source: articles/storage/files/storage-how-to-use-files-linux.md
Service: storage
GitHub Login: @RenaShahMSFT
Microsoft Alias: renash

@MicahMcKittrick-MSFT: @RenaShahMSFT and I agree this is correct. Please approve this PR.
@petersatsuse Thanks for correcting our bug Peter! :)

@petersatsuse -- we will create a PR to implement the suggestions you've made in this Issue.
@wmgries or @RenaShahMSFT -- when you've created the PR, could you please link it here so @petersatsuse can follow the work?
@petersatsuse -- #please-close this Issue when you are satisfied that we're taking appropriate action.  Thank you!

@petersatsuse
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Leons weapon changes to Claires Handgun

Version
2.1.7
Seed
R521-73T8-9JNN5CCHLB3P00100W00000Z88Z
Scenario
RE 2 (Leon B)
What happened?
Even with randomizer turned off, it still changes Leons Handgun to Claires. Leon B

@ShannonCobannon what do you mean by, even with the randomizer off?

@ShannonCobannon what do you mean by, even with the randomizer off?

When Randomize Items is unchecked and it is just default RE2 with no randomization of the items or Weapons but in the 2nd scenario Leon uses Claire's handgun instead


[pre-sve] TC-TSTAT-1.1 Not avaliable mandatory command based of feaureMap value

Problem
all-cluster-app
nrf52840dk
revison: 425f1fb916ca154deeb75cd82cd5196778aba152
DUT has feature map 11 (001011) which means PICS_TSTAT_HEAT, PICS_TSTAT_COOL, PICS_TSTAT_SCH.
Based on that information accepted command list should support commands  0, 1,2,3  while supports only cmmand 0.
[1654257169.210131][68046:68051] CHIP:TOO:   AcceptedCommandList: 1 entries
[1654257169.210148][68046:68051] CHIP:TOO:     [1]: 0

[1654258099.390828][70421:70426] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0201 Attribute 0x0000_FFFC DataVersion:<PHONE_NUMBER>
[1654258099.390868][70421:70426] CHIP:TOO:   FeatureMap: 11

Cert Blocker Review: Marking cert blocker, given this is a failure in the all cluster app re: thermostat cluster.


Add types for all components and functions

Closes #8 🙂

Thank you very much!
Released with v1.1.0


partial naming

Right now partials are just named after the basename, including the _. This means we have to do {{> _pager }}. I wonder if we should strip the _? Furthermore, what if we allowed partials to be named based on their directory hierarchy like views are: shared_pager if it's in shared/_pager. Thoughts?

I'm open to anything because I'm not using them :)
Send a pull request and I'll merge it.

How's this one look?


Adjust PhpUnitDedicateAssertFixer priority on 2.x line

Fixer must be run after no_alias_functions
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1792#issuecomment-194758297

@SpacePossum please, could you take care of this ?

will do :)
(please tag the issue as 2.0 so it shows up for the 2.0 release)

done


[BUG]: errors while trying to generate MinGW makefile with CMake on windows

Describe you problem:
Windows users may need to use MinGW with CMake to compile the program. When generating a makefile for the sorting-visualizer using mingw32-make the following errors are thrown:
\sorting-visualizer\src\SortController.cpp: In member function 'void SortController::_startSort(int)':
\sorting-visualizer\src\SortController.cpp:144:71: error: no matching function for call to 'std::thread::thread(void (SortController::*)(), SortController*)'
  144 |         _animThread = std::thread(&SortController::checkSortAnim, this);
      |                                                                       ^
In file included from C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/thread:43,
                 from \sorting-visualizer\src\SortController.h:10,
                 from \sorting-visualizer\src\SortController.cpp:1:
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:156:5: note: candidate: 'std::thread::thread(std::thread&&)'
  156 |     thread(thread&& __t) noexcept
      |     ^~~~~~
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:156:5: note:   candidate expects 1 argument, 2 provided
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:120:5: note: candidate: 'std::thread::thread()'
  120 |     thread() noexcept = default;
      |     ^~~~~~
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:120:5: note:   candidate expects 0 arguments, 2 provided
\sorting-visualizer\src\SortController.cpp: In member function 'void SortController::startSort(int)':
\sorting-visualizer\src\SortController.cpp:165:81: error: no matching function for call to 'std::thread::thread(void (SortController::*)(int), SortController*, int&)'
  165 |         _sortingThread = std::thread(&SortController::_startSort, this, sortType);
      |                                                                                ^
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:156:5: note: candidate: 'std::thread::thread(std::thread&&)'
  156 |     thread(thread&& __t) noexcept
      |     ^~~~~~
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:156:5: note:   candidate expects 1 argument, 3 provided
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:120:5: note: candidate: 'std::thread::thread()'
  120 |     thread() noexcept = default;
      |     ^~~~~~
C:/mingw64/lib/gcc/x86_64-w64-mingw32/12.2.0/include/c++/bits/std_thread.h:120:5: note:   candidate expects 0 arguments, 3 provided
mingw32-make[2]: *** [CMakeFiles\sorting-visualizer.dir\build.make:121: CMakeFiles/sorting-visualizer.dir/src/SortController.cpp.obj] Error 1
mingw32-make[1]: *** [CMakeFiles\Makefile2:82: CMakeFiles/sorting-visualizer.dir/all] Error 2
mingw32-make: *** [Makefile:90: all] Error 2

Steps to reproduce (optional):

clone the directory on windows using git clone https://github.com/alesbe/sorting-visualizer && cd sorting-visualizer
install CMake version 3.25.1
navigate to the root directory of the above cloned repository on local machine
Set up gcc and g++ compilers using (installation path may vary):

CMake -DCMAKE_C_COMPILER="C:/mingw64/bin/gcc.exe"
CMake DCMAKE_CXX_COMPILER="C:/mingw64/bin/g++.exe"

run CMake -G "MinGW Makefiles"
Build files get written to /sorting-visualizer
run mingw32-make
errors above occur at this point

Additional info (optional):
Running Windows 11
gcc.exe (x86_64-win32-sjlj-rev0, Built by MinGW-W64 project) 12.2.0
g++.exe (x86_64-win32-sjlj-rev0, Built by MinGW-W64 project) 12.2.0
OS

[x] : Windows
[ ] : Mac OS
[ ] : Linux

@alesbe i'm fairly new to development in C++ so quite possible I'm just making some newbie mistake somewhere. Thought it might be good to create this issue anyway for any other users who may run into issues trying to get sorting-visualizer to work on windows. I'll keep looking into it.

Thanks for reporting the bug!
When I was developing this, I was using Linux with CMake, I've never used CMake in Windows so I don't know what is causing the error...
Anyways, I'll look into it right now. There should be an easier way to compile the project in Windows, like the install.sh for Linux.
Here is a guide from SFML to compile it using CMake. I will try to fix it and after that, I will make an installation script for Windows. I'll keep you updated!

Thanks for the link. I tried this morning following along with the tutorial. Was able to generate the makefile successfully but get the same errors when trying to build - it gets 66% of the way through. I suspect that maybe it has to do with some kind of compiler version mismatch?
Will try use a different version of mingw and see if that makes a difference.

@alesbe I was able to get it working on windows so I think this reported bug isn't actually a bug, just a configuration issue on my side. Once I installed MinGW-Builds and used their included version of g++ compiler  rather than the one included with MinGW I was able to build without errors.

That's so interesting! Thanks for your interest :)
I still can't compile the project on windows. Did you installed MinGW-Builds from here?
Also, a new algorithm was added to the project, you might want to pull the changes before implementing anything! :)


Tobin gh423 memory tests

More changes for #423
Some more memory tests and removing another invalid check.
Also updating doc validation to make un-implemented tests that don't have "TODO" in the document an error.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Force pushed a couple more updates and not planning any further development on this branch.
Have all the cases from #423 covered or killed and made a few updates to doc validation.

pushed an additional CL with a few clean-ups

:+1:

cherry-picked and pushed to master


LayoutAnimation + Keyboard bug?

Hi, I am having a little bug with my app using LayoutAnimation on Android (Samsung Galaxy S8).
Another user reported this error a while ago... (It is exactly the same)
https://github.com/facebook/react-native/issues/26551
Is this an incompatibility with react navigation 5?
Any solution?
It works good on my iOS (iPhone 6)
This is my react native version:
react-native-cli: 2.0.1
react-native: 0.61.4
Here I explain more widely the problem: https://stackoverflow.com/questions/62244339/react-native-layoutanimation-bug-with-react-navigation

For others encountering this problem - take a look at: https://github.com/facebook/react-native/issues/26551#issuecomment-686316279


python2 import error in SyntaxError

using pip install
error when  from torchsummary import summary
>>> from torchsummary import summary
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/qiaoran/python/python2/lib/python2.7/site-packages/torchsummary/__init__.py", line 1, in <module>
    from .torchsummary import summary
  File "/home/qiaoran/python/python2/lib/python2.7/site-packages/torchsummary/torchsummary.py", line 9
    def summary(model, *input_size, batch_size=-1, device="cuda"):
                                             ^
SyntaxError: invalid syntax

The code is fixed to run correctly on Python2 now. Thanks.


fix: add pointerEvents: ‘box-none’ to ItemLayout

Description

Add pointerEvents: ‘box-none’ to ItemLayout component to allow touches for children components outside item boundaries

Review

[X] I self-reviewed this PR

Added in another PR, credits added. Thank you! https://github.com/dohooo/react-native-reanimated-carousel/pull/733

Released https://github.com/dohooo/react-native-reanimated-carousel/pull/734


Connect a Paired Bluetooth Audio Device with Hotkeys

Please ensure you include/confirm the following information before submitting an issue.

[x] I have listed the Audio Switcher version
[x] I have listed my operating system, and version
[x] Include any details about the audio devices on the system
[ ] If you are using Nvidia or AMD for sound, include the driver versions
[ ] This is not an issue about setting more than one device as default from a hotkey, quick switch or other relating switching activity.

And do remember this application is developed and distributed FOR FREE, you should have no expectation that an issue will be fixed in a reasonable time-frame.
Version: <IP_ADDRESS>
OS: Windows 10
Would it be possible to connect a paired bluetooth audio device with hotkeys do you think?
Cheers!

It's not a feature out of the box, with the Windows Sound libraries.
Audio Switcher would have to integrate with the underlying System Bluetooth libraries - not something that is in the scope of the application at this stage.


Linked Users fail when initially logged in using redirect

There is an issue with linked users.  The first time a user is linked, that user gets rejected due to a duplicate email problem.  Subsequent logins work.

Fixed with release 3.2.15: https://github.com/auth0/wp-auth0/pull/275


Create Contributing.md issue #33

please let me know what to add or what to remove!

Please let me know any other changes!

Added the Link to the code_of_conduct.md already, please check, and will bold the phrased or important words in next commit
cc: @N3v1

Added the Link to the code_of_conduct.md already, please check, and will bold the phrased or important words in next commit
cc: @N3v1

Oh yes, you're right. My mistake 🙈 @Devansh-Singh-Parmar
Sorry

@N3v1  What do you mean by that table of content?
also, asking for a final review from @SID262000  before making a final PR

Maybe you could add a table of content, if you want to

Table of content is not needed as the contributors should go through each and every guideline. Hence i feel it's not necessary to add that

Asking again for a review from
cc : @SID262000

Already answered in preceding PR, Thanks


Running in Non gui mode in linux platform gives CannotResolveClassException

I am trying to run jmeter script with shhmo listener in linux vm using non gui mode command. Gives me the below error while starting up the test.
Error Details:
./jmeter.sh -n -t  XXXXX.jmx -l results.jtl -e -o XXXX/TestRun_"date +%d%m%y%s"
Error in NonGUIDriver java.lang.IllegalArgumentException: Problem loading XML from:'XXXX.jmx'.
Cause:
CannotResolveClassException: nz.co.breakpoint.jmeter.vizualizers.sshmon.SSHMonCollector
Detail:com.thoughtworks.xstream.converters.ConversionException:
---- Debugging information ----
cause-exception     : com.thoughtworks.xstream.converters.ConversionException
cause-message       :
first-jmeter-class  : org.apache.jmeter.save.converters.HashTreeConverter.unmarshal(HashTreeConverter.java:66)
class               : org.apache.jmeter.save.ScriptWrapper
required-type       : org.apache.jmeter.save.ScriptWrapper
converter-type      : org.apache.jmeter.save.ScriptWrapperConverter
path                : /jmeterTestPlan/hashTree/hashTree/nz.co.breakpoint.jmeter.vizualizers.sshmon.SSHMonCollector
line number         : 41
version             : 5.4.1

Hi @RatishMohan007
Please double check that the plugin's jar files are actually present in the JMeter lib/ext folder.

Hi @tilln ,
Plugin jar's are available. I have tested it first on gui mode and everything works fine there. This issue is coming only in non gui mode. May be a common issue in headless mode that plugin which use visual graphs do not work there?

One jar file was missing indeed. Closing the issue


Fixing servicing bug by always specifying LayerFolderPath

During the recent OCI changes, I mistakenly thought LayerFolderPath is only needed for Windows Server containers (isolation=process) and not for Hyper-V Containers, but it turns out it is also required for servicing containers used to finish installing updates.  Since the servicing containers need to reuse the container's create options, this change makes it so that LayerFolderPath is always filled in for all containers as part of constructing the create options.
@jhowardmsft @jstarks @brian-young

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "servicing_fix" git@github.com:swernli/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "servicing_fix" git@github.com:swernli/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Changes look good. Needs DCO.

@jhowardmsft DCO?

LGTM.
@swernli @jhowardmsft We have to reevaluate the place where containerd.Create on daemon exit event handler. This greatly makes libcontainerd unusable as a library and should be handled internally.

LGTM


Detailed requirements for Fleet supporting k8s 1.27 & 1.28

This issue is to implement the chart changes & perform the testing required to upgrade to k8s 1.27. Please perform below steps.

[x] Check whether the chart has any references to the below deprecated features of k8s 1.27. You can find information about deprecated APIs here on Kubernetes Deprecation guide.

storage.k8s.io/v1beta1 API version of CSIStorageCapacity

[ ] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.28.0-0'

Note: If this is an upstream chart and if the upstream chart doesn't support k8s 1.27 then do not update the kube-version annotation or update the k8s version accordingly.

[ ] Perform below tests

Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.27 provisioned on Rancher. Test the chart functionality.
(Please refer to the https://github.com/rancher/rancher/issues/41395 to know what testing was done for k8s 1.26 upgrade for your chart)

Provision rancher on k8s 1.26 or lesser

Install chart on this cluster

Upgrade this cluster in rancher to k8s 1.27

Make sure the existing chart and the app is functional after cluster upgrade

Needs at least a Fleet RC.

@manno , Are you still blocked?

Fleet is installed automatically with Rancher. The upstream cluster has the fleet-controller and fleet-agent, and all downstream clusters have a fleet-agent.
There is no helm chart for Rancher v2.8 yet, so I used the following command to install Rancher:
helm install rancher rancher-latest/rancher --create-namespace --devel \
  --namespace cattle-system \
  --set hostname=<hostname> \
  --set rancherImageTag=v2.8-head \
  --set bootstrapPassword=<pass>

Perform below tests

Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.27 provisioned on Rancher. Test the chart functionality.

Rancher installation failed with the following error message:
Error: INSTALLATION FAILED: chart requires kubeVersion: < 1.27.0-0 which is incompatible with Kubernetes v1.27.5+k3s1

Do we need to wait for the Rancher 2.8 alpha version?

Provision rancher on k8s 1.26 or lesser
Install chart on this cluster
Upgrade this cluster in rancher to k8s 1.27
Make sure the existing chart and the app is functional after cluster upgrade

Upgrade worked fine.

Installed Rancher in a k8s v1.26.8+k3s1 cluster, and added a downstream cluster with the same version.
Upgraded both upstream and downstream to v1.27.5+k3s1
Verified fleet-agent and fleet-controller were working fine.

Version of rancher: rancher/rancher:v2.8-head
Version of Fleet: rancher/fleet:v0.9.0-rc.1

System information:

Rancher version: 2.7.7
Fleet version: 0.8.0

Rancher Upgrade Scenario

Upgrade from Rancher 2.7.7 to 2.8.0-alpha2 works fine, (Upgrading k3s version v1.27.1+k3s1 is done prior to the Rancher upgrade).
Fleet upgraded to the 0.9.0-rc3 without any error.
GitRepo is also working fine.

Fresh Rancher installation

Fresh Rancher installation with the v1.27.1+k3s1 or beyond is throwing below error:

Error: INSTALLATION FAILED: chart requires kubeVersion: < 1.27.0-0 which is incompatible with Kubernetes v1.27.1+k3s1

@gunamata / @thardeck, shouldn't the patch be catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.29.0-0' according to the issue title, or you are doing each version step by step?

System information:

Rancher version: 2.7.7
Fleet version: 0.8.0

Rancher Upgrade Scenario

Upgrade from Rancher 2.7.7 to 2.8.0-alpha2 works fine, (Upgrading k3s version v1.27.1+k3s1 is done prior to the Rancher upgrade).
Fleet upgraded to the 0.9.0-rc3 without any error.
GitRepo is also working fine.

Fresh Rancher installation

Fresh Rancher installation with the v1.27.1+k3s1 or beyond is throwing below error:

Error: INSTALLATION FAILED: chart requires kubeVersion: < 1.27.0-0 which is incompatible with Kubernetes v1.27.1+k3s1

Fresh installation of Rancher version: v1.27.6+rke2r1 working without any error. Also, tested below points:

[ ] Cluster import
[ ] Installs fleet version 0.9 RC
[ ] Working of GitRepo.

/backport release/v2.7

helm install rancher rancher-latest/rancher
--namespace cattle-system
--set hostname=hostname
--set bootstrapPassword=****
--set replicas=3
Error: INSTALLATION FAILED: chart requires kubeVersion: < 1.28.0-0 which is incompatible with Kubernetes v1.28.7
Hey is there any chance of installing rancher on kubeverison 1.28.0


Converting Uint8Array.from() to new Uint8Array()

When using Etebase in a Typescript/React Native environment, I ran into this error on login/signup ... and maybe a few other operations.
TypeError: TypedArray.from requires its this argument subclass a TypedArray constructor
And found a similar issues reported here:
https://github.com/openpgpjs/openpgpjs/issues/1076
https://github.com/zloirock/core-js/issues/285
I'm not familiar with Uint8Array, so I don't know if there's a meaningful difference between the two constructor methods. If smarter people than me say that there's no difference, maybe it's worth changing these constructors to avoid issues?
FWIW, I've been using the alternate constructors in my own fork and haven't run into any issues for a week or so ...

Released a version with this fix. Sorry for the delay!


docs: add build and test docs for all mayastor repos

Thank you so much for putting this together, cleared up a lot of questions I've had.

Thank you so much for putting this together, cleared up a lot of questions I've had.

Thanks @maxwnewcomer, I'm glad this helps!
Let us know if there's anything else we could improve on
Something that could be interesting is how to cross-compile for ARM from x86.
The kubectl-mayastor plugin is currently cross-compiled for ARM.
It does take an awful lot of time though:  https://github.com/openebs/mayastor-extensions/actions/runs/12351790197/job/34467462887
Though I'm not sure why, I suspect caching may not be working quite correctly on the github actions.


[Bug] doris V2.1.5版本，1fe、3be节点，我执行一个sql，直接导致doris挂掉

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
V2.1.5
What's Wrong?
我执行了一个sql，导致doris集群的BE节点全部挂掉
What You Expected?
如何修复这个问题
How to Reproduce?
我的sql：
SELECT road_code,
road_section_code,
road_section_name,
toll_start_mileage_num,
SUBSTRING_INDEX('+', toll_start_mileage_num, 1)
FROM dwd_zsj_yljc_road_section_base_info
LIMIT 0, 200;
执行之后，doris就挂掉
这是doris-be节点的日志

Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

can you provide more information to let us reproduce this or try using jdk8

When will a stable version be released? From 1.0 to 2.1, just simple sql, just crashed. As a database product, no matter how many functions it has and how good its performance is, if there is no reliability support, no one will dare to go online. Don’t blindly pursue new functions and features, please pay more attention to reliability and stability.


Add assembler QC report

Description of feature
Assembly quality can vary based on a large number of parameters, such as data type and quality. Tools like TransRate (https://hibberdlab.com/transrate/) can quickly analyze fasta files and generate metrics. These can be included in the final MultiQC report.
Another, more advanced, QC step would be to align the input reads to the generated assembly. Trinity recommends about 80% of the reads show up in the assembly, but this is likely going to be lower for meta-transcriptomes.
I foresee the TransRate existing as a module, and the Bowtie assembly alignment would be a subworkflow.

@erikrikarddaniel feel free to assign me here!

FYI if you go down the transrate path, I had difficulties installing so thought I'd share:

I was able to get the pgcbioinfo docker container to run: https://hub.docker.com/r/pgcbioinfo/transrate/tags. It's built from this docker file: https://github.com/pgcbioinfo/containers/blob/5ebf3fc205501216c6f301c577f9ca5fcd029c77/transrate/Dockerfile-1.0.3. A lot of the other docker containers did not work (you can pull and run them, but they're missing deps).
In order for transrate to work, you do need unique fasta headers, and your headers cannot contain a pipe (|).
the conda package failed for me: https://anaconda.org/bioconda/transrate. It failed with missing dependencies:

[DEPRECATION] The trollop gem has been renamed to optimist and will no longer be supported. Please switch to optimist as soon as possible.
Dependencies are missing:
  - snap (1.0dev.96)
  - salmon (0.6.0)
To install only the reference-metrics dependencies:

To install all missing dependencies, run:
  transrate --install-deps all

I installed these deps both with conda, and with the transrate command, and neither removed the error. Others have suggested that they got it to work with conda following instructions like these but I was not successful with that either.

Thanks for the heads up! I had already set up my own custom module and found success with just the transrate biocontainer. Had you tried that container?
I'm also only using transrate for the simple statistics instead of trying to map reads back.
process TRANSRATE {
    tag "$meta.id"

    container "quay.io/biocontainers/transrate:1.0.3--h031d066_5"

    input:
    tuple val(meta), path(assembly)

    output:
    path("*assemblies.csv")      , emit: assembly_qc
    path "versions.yml"          , emit: versions

    when:
    task.ext.when == null || task.ext.when

    script:
    def args = task.ext.args   ?: ''
    prefix   = task.ext.prefix ?: "${meta.id}"
    // tolerate both gzipped and normal inputs
    def gzipped = assembly.toString().endsWith(".gz") ? true : false
    """
    if $gzipped; then
      gzip -d -c $assembly > assembly_unzipped.fa
    else
      mv $assembly assembly_unzipped.fa
    fi

    transrate \\
      --threads $task.cpus \\
      --assembly assembly_unzipped.fa \\
      --output ${prefix}_transrate \\
      $args

    mv ${prefix}_transrate/assemblies.csv ${prefix}_assemblies.csv
}

Done in #167


[DUOS-1764, DUOS-1765][risk=no] Create a single DAR for N Datasets on submission

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-1764
https://broadworkbench.atlassian.net/browse/DUOS-1765
Changes

Update to the DAR submission process so we only create a single DAR instead of N DARs per dataset requested.
Updates to resource class and tests to cover this change.

Note that we should immediately follow up on https://broadworkbench.atlassian.net/browse/DUOS-1961 as that story is blocked by this one and we will no longer need to archive a group of DARs (DARCollection) and recreate a new draft from them.

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

However, I think some changes will need to be made to the bucketing logic on the frontend to support this change. The collection I created contained 4 datause buckets, each containing all of the elections since they are processed by dar. The votes are then processed by elections, so voting shouldn't be accurate with these changes.

I did actually test this case, but the bucketing and votes looked correct to me when I did so. That could have been an edge case where it worked, and there are others where it doesn't ... I'll try to dig more into the front end and figure something out.


WEB Api Token

Every time I press on " Start Authentifikation" for a new Site to Pop up the Website give me the error "Localhost cannot access this Website.
What can I do?

Any log?


Not really an Issue but a simple question

Been looking around ( probably missed it knowing me) to see how can I add the music and not let it show up on the chat, for example not let it show a link of the youtube video who posted it and so on. Unless it doesn't show for anyone only me?

@UndeadDreamz https://github.com/Just-Some-Bots/MusicBot/blob/master/musicbot/bot.py#L398


[Documentation]-Type necUniversal doesn't exist

Describe the bug
On wiki page: https://github.com/PepperDash/Essentials/wiki/ConfigurationStructure
In section : Device Object Structure
type": Think of this as a way to identify what specific module you might associate with this device.  In Essentials, this value is used to determine what class will be instantiated for the device (ex. "necUniversal" or "samsungMdc" for two types of displays)
Expected behavior
Remove necUniversal as an example or replace it with necmpsx.

I've updated the page. Thanks for letting us know!


Don't call BreadcrumbsPicker.show inside IContextViewDelegate.getAnchor

Fixes #75263

Sorry, too late...


Make better button docs

https://shopify-dev.shopify-dev-minimal-pfjv.anthony-frehner.us.spin.dev/api/hydrogen-react/components/addtocartbutton

@wizardlyhel
The ShopPayButton is one where I think the current complex type does a better job of describing the either or that is actually happening than an interface can.
The current docs: https://shopify.dev/api/hydrogen/components/primitive/shoppaybutton#props, at a glance make it seem like you can use both a prop called variantIds and variantIdsAndQuantities. The text above does describes the actual situation but as a developer I think the typescript does it more elegantly.
Thoughts?

👍

Thanks all!


Remove duplicate error logging

I haven't been able to figure out how to shut off the default non JSON error logging, so that the error is only logged in the JSON format instead of the default + JSON formats.

Hi, you probably don't removed plug Plug.Logger from endpoint.ex in your project (3rd step in Installation).

The "Running  with Cowboy using " message comes from here: https://github.com/phoenixframework/phoenix/blob/master/lib/phoenix/endpoint/cowboy_handler.ex#L100

@johnkelly apart from @mizalewski's , check your log level. I had the same issue with regular logging still appearing in logs, only to realize I had :debug level logs enabled.
Make sure that:

default logger is set to log only :error or :debug level,
plug_logger_json is set to log on info level.


X.H.ManageDocks: Fix button events on decoration windows

Description
Decoration windows are created using XMonad.Util.XUtils.createNewWindow
which happens to set _NET_WM_WINDOW_TYPE to _NET_WM_WINDOW_TYPE_DESKTOP,
and ManageDocks considers such windows candidates for struts and
therefore requests property events, thus overriding the original event
mask requested by decorations.
The fix is to first obtain the current event mask, set the required bits
in it only then reset the mask.
Fixes: https://github.com/xmonad/xmonad-contrib/issues/517
Fixes: ec146171238b ("X.H.ManageDocks: React to strut updates of override_redirect docks")
Related: https://github.com/xmonad/X11/pull/77
Checklist

[X] I've read CONTRIBUTING.md

[X] I've considered how to best test these changes (property, unit,
manually, ...) and concluded:
I tested this manually and I've verified that dragging works now and
that https://github.com/xmonad/xmonad-contrib/pull/492 isn't
regressed either. We do not and will not have the infrastructure to
write an automated test for this, unfortunately.

n/a I updated the CHANGES.md file
This is a recent regression of git master, no need to clutter the
CHANGES.

n/a I updated the XMonad.Doc.Extending file (if appropriate)

Should createNewWindow even be doing that? It sounds wrong for most decorations.

CI is expected to fail because this bumps the X11 dependency, but we first need to merge https://github.com/xmonad/X11/pull/77, release X11 0.10, bump the upper bound in https://github.com/xmonad/xmonad (including adding a revision for old releases on Hackage), and only then will this build.

There's also a typo in the cabal file.

Should createNewWindow even be doing that? It sounds wrong for most decorations.

https://github.com/xmonad/xmonad-contrib/blob/246926911940ccff1f6fad13d82ceb5c91cd01dc/XMonad/Util/XUtils.hs#L66:
-- @@@ ugly hack to prevent compositing

There's also a typo in the cabal file.

Good catch! Force pushed (sorry).


bump minimum python versions

docs failing b/c AWS Landsat public dataset no longer exists
https://github.com/intake/intake-stac/blob/main/docs/source/tutorial.rst#loading-a-catalog ->
https://raw.githubusercontent.com/sat-utils/sat-stac/master/test/catalog ->
https://landsat-pds.s3.amazonaws.com/c1/L8/107/018/LC08_L1TP_107018_20181001_20181001_01_RT/LC08_L1TP_107018_20181001_20181001_01_RT_B1.TIF

upstream test failing with AssertionError: assert <MediaType.PDF: 'application/pdf'> in {'application/geo+json': 'geopandas', 'application/geopackage+sqlite3': 'geopandas', 'application/json': 'textfiles', 'application/netcdf': 'netcdf', ...}
So pystac has added the application/pdf media type, which looks like is used for things like product documentation links: https://github.com/stac-utils/pystac/blob/a8f50cba92e27c9e1630bb8332523e9d06ee9cc4/tests/data-files/examples/1.0.0-beta.2/extensions/sar/examples/envisat.json#L80-L82
    "links": [
        {
            "rel": "about",
            "href": "https://earth.esa.int/pub/ESA_DOC/ENVISAT/ASAR/asar.ProductHandbook.2_2.pdf",
            "title": "ASAR Product Handbook",
            "type": "application/pdf"
        },

I'm just going to add a special case to that existing test...


Demonstrate building a native Android project

In place of NDK CMake

I assume this is blocked by #169?

Not exactly, it's possible now, it's just nowhere near what we may want users to do. The basic idea of those demonstrations is to see where we should invest our engineering effort to bridge the gap instead of moving on features that have little to no impact on what users can do.

Any progress on this?

No, and if you want I can provide you some guidance for writing the sample. It should be possible to accomplish with what is already available.

It's greatly appreciated.
I have a small native library project used for Android and iOS, which only have 2 c files and 1 JNI c files.
I use standard Android Studio Gradle/CMake workflow, gradle assemble/build and output aar files(with multiple arch so files inside) for Android.
And use another cmake-toolchains workflow, build SHARED or STATIC library as Framework Bundle for iOS device/simulator.
I'm really looking forward to seeing the unified gradle workflow.

The other alternative is to wrap the CMake build in Gradle so you keep using the tools offered by Android NDK. It depends the end goal. We have a CMake wrapping sample already.


Check if package exists before installing minikube-required packages

Per discussion with @hklai here: https://github.com/istio/istio/pull/8248#discussion_r214234630
This PR checks if packages exist before installing minikube required packages - support both apt and yum(for centos support).
/assign @hklai @JimmyCYJ

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

Needs approval from an approver in each of these files:

tests/e2e/local/OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/e2e/local/OWNERS [hklai]

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

@hklai
Awesome! Thanks for your diligent review :+1:


Missing DoctrineTransportFactory (symfony/messenger)

I think this package needs symfony/doctrine-messenger since I can not use symfony/symfony ^5.4 and symfony/orm-pack ^2.2 together without that package. I'm not using Symfony Flex.
Running bin/console with just give me:

The autoloader expected class "Symfony\Component\Messenger\Transport\Doctrine\DoctrineTransportFactory" to be defined in file "/var/www/html/vendor/composer/../symfony/symfony/src/Symfony/Component/Messenger/Transport/Doctrine/DoctrineTransportFactory.php". The file was found but the class was not in it, the class name or namespace probably has a typo.

Well, true. This class is deprecated:
// Symfony\Component\Messenger\Transport\Doctrine\DoctrineTransportFactory.php

if (false) {
    /**
     * @deprecated since Symfony 5.1, to be removed in 6.0. Use symfony/doctrine-messenger instead.
     */
    class DoctrineTransportFactory
    {
    }
}

$> grep symfony\/ composer.json
        "symfony/monolog-bundle": "^3.8",
        "symfony/orm-pack": "^2.2",
        "symfony/symfony": "^5.4"

After requiring symfony/doctrine-messanger, the autoloading/-wiring works fine.

Using symfony/symfony directly is not supported.


Frequent relist all objects when edge workers disconnect to apiserver

Now for the edge computing  scenario, since the low quality of network, edge workers always disconnect to apiserver.
At this time, if the connection is disconnected for a while(enough for the watch cache in apiserver expired), it will cause relisting all the objects, which consumes a lot of bandwidth and the control plane.
And for K8s run in DC, nodes rarely disconnect or reconnects soon. I think this is different from the edge computing scenario.
Do we have any optimization for this edge computing  scenario?

cc @erikwilson @ibuildthecloud

For edge computing scenarios we usually envision many small clusters, managed through Rancher or Fleet. Kubernetes is not really built to handle the kubelets disconnecting and reconnecting frequently.


Please add/update Japanese version of this document.

Please add Japanese translation of the following FAQ if possible.
When creating a disk from a blob, is there any continually existing relationship with that source blob?
No, when the new disk is created it is a full standalone copy of that blob at that time and there is no connection between the two. If you like, once you've created the disk, the source blob may be deleted without affecting the newly created disk in any way.

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

ID: 263abdef-3478-8261-6c4c-5d608698ba4e
Version Independent ID: 61ef3094-fda7-d7b9-e19c-4e8f6a5b2f2d
Content: Frequently asked questions (FAQ) about Azure IaaS Windows VM disks
Content Source: articles/virtual-machines/windows/faq-for-disks.md
Service: virtual-machines-windows
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.


Fix List.compareWith case

List.comparewith => List.compareWith.
Ref: List.compareWith<'T> Function (F#)

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/conceptual/collections.list-module-[fsharp].md
:warning:Warning

Details

docs/conceptual/collections.list-module-[fsharp].md

Line 42: [Warning] Invalid file link:(~/list.chunkBySize['t%5D-function-%5Bfsharp%5D.md).
Line 44: [Warning] Invalid file link:(~/list.compareWith['t%5D-function-%5Bfsharp%5D.md).
Line 46: [Warning] Invalid file link:(~/list.contains['t%5D-function-%5Bfsharp%5D.md).
Line 47: [Warning] Invalid file link:(~/list.countby['t,'key%5D-function-%5Bfsharp%5D.md).
Line 48: [Warning] Invalid file link:(~/list.distinct['t%5D-function-%5Bfsharp%5D.md).

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


Mobile CSS enforces minimum 1200px width

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
The blog CSS enforces a minimum width of 1200px for blog posts, with an obvious negative impact for mobile users.
Proposed Solution:
Remove the attribute.

Fixed by #13417, closing.


How to send data from main to renderer process?

The documentation says
promiseIpc
  .send('getRendererData', webContentsForRenderer)
  .then((rendererData) => console.log(rendererData))
  .catch((e) => console.error(e));

However, this is actually a get, not a send.
How do you send data?  I want to send parameters.
My flow looks like this
Renderer window sends IPC request for information by ID number
Main gets this, sends again to another hidden Renderer window with the ID number
Second hidden renderer gets the ID, spends a lot of computational power to figure it out, then sends the result back to the main process.
Main gets this, forwards back to the first Renderer window.
How is this done?  In renderer, the second parameter of .send is the data you want to send.  But in main, the second parameter is the webcontents of the window you are targeting to send to.  So how do you send data as well?

looks like the answer is
promiseIpc
  .send('getRendererData', webContentsForRenderer, {data_to_send:'here'})
  .then((rendererData) => console.log(rendererData))
  .catch((e) => console.error(e));

And then in the renderer, the first parameter is no longer the event, but the value that you send as the third parameter.


Using MlpLstmPolicy with PPO2

Describe the bug
To start the training I need a vectorized environment so I use
make_vec_env(id, n_envs = 2)
But when the training finishes and I try
action, _states = model.predict(self.obs)
I get this error:

System Info
Describe the characteristic of your environment:

Describe how the library was installed (pip, docker, source, ...)
pip
GPU models and configuration
gtx 1080 Ti
Python version
3.7.7
Tensorflow version
1.15.0 -gpu
Versions of any other relevant libraries

Additional context
I'm new to this and don't really understand the error message, but looks to me that it doesn't accept the input as it has (2,12) because of the 2 environments,  but then again I need at least that to get training started. I've tried loading with one environment and training with n_envs=nminibatches without any luck. Thanks

Please include a minimal code to replicate the issue, and also check docs on how to use recurrent policies (e.g. here). It is hard to say with this information what might be wrong.

What type of code should I include, like my custom environment?
I see now that I didn't call model.predict with state=state, mask=done . I will try that

What type of code should I include, like my custom environment?

No need to include your full custom environment if you prefer not to share it, just a dummy one that manages to raise the error. The code should be complete in a sense that running it produces the error.

Hi, so I got it working by training with a dummy env with 2 environments and then creating a test environment with one env after finding #166 .  And also doing
new_obs, rewards, done, info = test_env.step(action)
self.xcoord.append(info[0]['x'])
To extract what I needed


[JavaScript/Node Js]: Added Authentication in Link unfurling, ME Action

Addition of Authentication for Message Extension Action, Link Unfurling to existing sample 52.teams-messaging-extensions-search-auth-config
This PR is raised for https://github.com/microsoft/BotBuilder-Samples/issues/2988
C# PR: #3007
Proposed Changes

Added Authentication for Action command in ME --
 Displaying user info once user sign in
Added Authentication for Link unfurling in ME --
Displaying Thumnail card for link unfurling authentication
Updated Manifest for all 3 types of command
Added SHOWPROFILE, messageHandler command

Testing

ME-Action Command Auth Show Profile:
Sign In Card:

Show Profile :

ME-Link Unfurling Authentication:
Sign In card:

Thumbnail Card once user login:

Sign out:

@EricDahlvang , @clearab , Could you please this PR and let me know your comments?


README link to playground does not work

Dear Author,
Your links to the playground under "Example command is provided [Here (https://github.com/JiangWeibeta/MLIC/tree/main/playground)." do not work.
That's because you have moved the scripts to MLIC++ folder in this commit: https://github.com/JiangWeibeta/MLIC/commit/51f22af6ca8af444a98a46cda849a3eeac7369af
This link should work :) https://github.com/JiangWeibeta/MLIC/tree/main/MLIC%2B%2B/playground

Thank you! I have updated the link to playground.


[dart2js] Unreachable code after return null

Not a big deal, but in the new DartPad frontend I noticed warnings about unreachable code. Something along the lines of:
return null
return new ...,

Maybe the return null was an early exit behind a condition that was determined to always occur in an earlier pass?
I don't have a super simple reproduction yet, but I was able to reduce it to:
import 'package:flutter/cupertino.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const CupertinoApp(
      title: 'dart2js reproduction',
      home: CupertinoPageScaffold(
        navigationBar: CupertinoNavigationBar(
          middle: Text('dart2js reproduction'),
        ),
        child: Text('Hi web compiler engineers'),
      ),
    );
  }
}

With flutter build web on main at https://github.com/dart-lang/sdk/commit/b6647fd9ab2d3308a5894cc0e70d7da20d413082.

Are you seeing warnings from the browser about unreachable code in the generated JavaScript?
If so, which browser?
We don't generally aim to satisfy 'lints' coming from the browser.
dart2js might occasionally generate this kind of code because the simplifier phase (which runs 'top down') is run several times, including after the last dead code elimination phase (which runs 'bottom up'). It would be unusual to see dead code like this since the earlier runs of the simplifier phase usually 'decide' the condition guarding first return.
It is beneficial to do dead code elimination as part of simplification (we do a limited amount here).
Perhaps we should be more aggressive about dead code elimination during simplification, or always run dead code elimination after the last simplification.

Are you seeing warnings from the browser about unreachable code in the generated JavaScript?
If so, which browser?
We don't generally aim to satisfy 'lints' coming from the browser.

Firefox warns about it: "unreachable code after return statement"
I agree it's not import to eliminate the warning, and really the code causing this shouldn't even be in the app (https://github.com/flutter/flutter/issues/142633).
I still thought I'd call it out in case its expected to be able to handle this case. Especially since users might find warnings like this surprising and not matching their understanding of the optimizations that dart2js provides.

It is beneficial to do dead code elimination as part of simplification... Perhaps we should be more aggressive about dead code elimination during simplification, or always run dead code elimination after the last simplification.

It'd be interesting to at least to see the impact of an extra DCE pass after the last simplification, both on size and compilation times for a large app. Slight size reductions and increased user confidence are always great!


EDM-788: Add init script for PostgreSQL to set master user as superuser

Added enable-extensions.sh script to initialize PostgreSQL with specified extensions.
The script checks the POSTGRESQL_EXTENSIONS environment variable and enables each extension dynamically.
Integrated the script into the flightctl-db ConfigMap and Deployment.

@siserafin, Could you please test this thoroughly?

Can we now remove https://github.com/flightctl/flightctl/blob/main/test/scripts/deploy_with_helm.sh#L87 ?

@siserafin, Could you please test this thoroughly?

I can test this in local, since I need a flightctl helm chart to install flightctl in ocp. Or do you know another way?

Could you please also add this change to the Quadlet deployment (https://github.com/flightctl/flightctl/blob/main/deploy/quadlets/flightctl-db/flightctl-db.container), so they don't get out-of-sync?

@siserafin, Could you please test this thoroughly?

I can test this in local, since I need a flightctl helm chart to install flightctl in ocp. Or do you know another way?

Locally I don't see the issue, I will test  in ocp when the PR is merged, and I have the updated helm chart

@siserafin, Could you please test this thoroughly?

I can test this in local, since I need a flightctl helm chart to install flightctl in ocp. Or do you know another way?

Locally I don't see the issue, I will test in ocp when the PR is merged, and I have the updated helm chart

you can test before merging using
helm upgrade --install --namespace <ns> --create-namespace flightctl ./deploy/helm/flightctl --values <path_to_values_file>

Tested in ocp, and after the installation, the api pod is running:
[kni@titan54 flightctl]$ oc get pods -n flightctl
NAME                                  READY   STATUS    RESTARTS        AGE

flightctl-api-65f5889b79-hh8cn        1/1     Running   2 (4m54s ago)   5m8s

LGTM

Please keep in mind that the Postgres database may not be the one we deploy by default, but an external one provided by the end-user. IOW, we cannot assume control over that external database.

I agree with your point. Do you have any suggestions on how we can best accomplish this?
We are currently creating tables and indexes, as well as enabling database extensions. This requires the end-user to either provide a database user for the API server with the necessary permissions or enable the extensions themselves.

As a first step, I believe we should separate the PG admin and user secrets like in the Keycloak sub-chart (and not pass via env). Then we could use an init-container to do the required operations before starting the API server least privileged.

@fzdarsky I agree with you, but that seems to be outside the scope of this fix?

@fzdarsky I agree with you, but that seems to be outside the scope of this fix?

Yeah, not saying this PR needs to do everything. Ok, let’s proceed and open a Jira for follow-up work.


build(normalizr): Add own reduced normalizr package

Thoughts
The normalizr package is making 22% of the size. TIme to change it and make a own package where only the functions are which are used!

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8b430f2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #190   +/-   ##
=========================================
  Coverage          ?   99.57%
=========================================
  Files             ?       54
  Lines             ?     4520
  Branches          ?      374
=========================================
  Hits              ?     4501
  Misses            ?       19
  Partials          ?        0

Impacted Files
Coverage Δ

packages/pinia-orm/src/composables/mapRepos.ts
100.00% <ø> (ø)

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


Silence 'URI.escape obsolete'

@rurounijones thank you for #71 👍
@bprotas thank you for reminding me of this 👍
Please review and comment this PR.
I am sorry you guys had to withstand these boring error messages on your projects for so long.
Please let me know if you find other things this project could improve on.

@thejamespinto - thanks for doing this, I can confirm that the PR you have put in here does silence the warnings.
I'm curious about the approach though - #71 used URI::Parser.new which did not get the deprecation warnings, and still allowed the reserved characters.  Do you think that's an option?  I'm happy to put together a PR as I might be missing something.

@bprotas actually these warnings have been haunting me since I came onboard to help with the project and I already wanted to do something about them. It was actually your email that made me want to do something about this one. Please check all my currently open PRs. I have silenced most warnings, but I will let them open for a while to give followers of the project a chance to voice their opinions.
Please, go check all my open PRs for this project.

@bprotas About this particular issue again.
If you think about it, all I'm doing is suppressing a warning introduced in 2009.
I think this is more clear than 71 because it simply takes the original code and removes the warning.
also, DEFAULT_PARSER = Parser.new
https://github.com/ruby/ruby/commit/238b979f1789f95262a267d8df6239806f2859cc
And the tests pass.
https://travis-ci.org/oauth-xx/oauth-ruby/builds/131959709
Everybody on the internet seems to agree to override URI.escape with CGI.escape, but the latter does not take a second argument.
If you could come up with a PR that actually moves from URI to CGI, I would personally favor it over my own code. 💯
Don't forget to check all my open PRs for silencing several other warnings.

makes sense,thank you for this and for your work on the other warnings.  the test output is much easier to read now!


Nested filter are ignored

The following path "$.store.book[?(@.authors[?(@.lastName1)])]" returns both books despite the fact none has lastName1 attribute.
Example JSON:
{
    "store": {
        "book": [
            {
                "authors": [
                    {
                        "firstName": "Nigel",
                        "lastName": "Rees"
                    },
                    {
                        "firstName": "Evelyn",
                        "lastName": "Waugh"
                    }
                ],
                "title": "Sayings of the Century"
            },
            {
                "authors": [
                    {
                        "firstName": "Herman",
                        "lastName": "Melville"
                    },
                    {
                        "firstName": "Somebody",
                        "lastName": "Else"
                    }
                ],
                "title": "Moby Dick"
            }
        ]
    }
}

ref https://github.com/RedisJSON/RedisJSON/issues/494

There are some alternative to get by nested array filter ?


Fix htmlreport python package install

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] ~Reset the build number to 0 (if the version changed)~
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@seanyen @timsnyder @wolfv The PR should be ready for review. A few things had to be fixed to make all builds work. I'll YOLO-merge this on Friday, September 6 (end of this week) unless you comment earlier. Thanks!

This should also fix #81 (if it was not fixed yet). The pygments files are not present in the Windows build, see end of build log: https://dev.azure.com/conda-forge/84710dde-1620-425b-80d0-4cf5baca359d/_apis/build/builds/1016259/logs/150

Looks good to me :)

@wolfv Thanks for checking. I'll wait until Friday, just in case someone else wants to comment.


support withknown.com and other indiewebcamp.com sites.

I was wondering if there was any interest in beginning work on federating Ghost with other blogs.  At an initial level it would just be a matter of adding microformats-2 structure to the themes  and maybe go on to being able to pull in comments from other blogs.

The best way to find an answer to this question (whether there is interest) is to suggest it on our wishlist. That page is accessible from inside the Ghost admin, and allows you to build traction for a feature request in a way that is not possible with a GitHub issue :wink:


Hidden directories are ignored in static/

Describe the bug
The build process does not copy over hidden directories from static/. This prevents adding a .well-known/ directory in a simple way.
Reproduction
Add a static/.well-known/ directory with some files in it.
Logs
No response
System Info
System:
    OS: Linux 6.1 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (4) x64 Intel(R) Core(TM) i5
    Memory: 1.25 GB / 7.46 GB
    Container: Yes
    Shell: 5.1.4 - /bin/bash
  Binaries:
    bun: 1.0.36 - /usr/local/bin/bun
  npmPackages:
    @sveltejs/adapter-static: ^3.0.1 => 3.0.1
    @sveltejs/enhanced-img: ^0.2.0 => 0.2.0
    @sveltejs/kit: ^2.5.0 => 2.5.5
    @sveltejs/vite-plugin-svelte: ^3.0.2 => 3.0.2
    svelte: next => 5.0.0-next.90
    vite: ^5.0.12 => 5.2.7

Severity
annoyance

I’ve had the opposite happen for me. Can you please provide a minimal reproduction?

Could this be an @sveltejs/adapter-static bug? Do the different adapters handle static assets differently?

I found the cause of the error. It was unrelated to SvelteKit. Sorry for the inconvenience.

@notramo what was the issue? I have the same issue

@francesco-gaglione, unfortunately, I don't remember. Probably something Git-related.


[devtools] refactor js debugging docs

Updates screenshots.
Moves all of the debugging features to a single task-based "reference".  Uses a fairly consistent and much more concise style to describe each feature.
Deprecates the old docs by putting warnings at top of each section, along with links to new content.
Adds "deprecation" status to old docs in ToC.

Thanks for the review, @jpmedley. I updated per your review.


Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

Pending a fix from the open issue on a dependency of this project we'll need to update this repo to pull in their changes.

https://github.com/highlightjs/highlight.js/issues/1990

I think this issue has been fixed by highlightjs version 9.15.6 moving the offending dependencies to devDependencies (see pull request here). Updating this package to depend on 9.15.6 or greater should fix the vulnerabilities being pulled in from their build system.

Since highlight.js is a floating range, no update should be needed in this repo

@felixfbecker Quite right, my mistake. I thought that package-lock.json would lock the version for downstream dependers but I was incorrect. I think this can be marked as closed, then.


Form validation does not have a rule to validate if field value differs from a string

Hello i am trying to validate an input that must have another string besides what i set
So for example
i have an input type text with value of "Test"
I want to validate this field that must be changed so it cannot have string Test inside the field
I have no rule for this
Something like
is_not_string[test]

You can probably use regex_match rule for this. Something like: regex_match[^(?!.*?Test).*] maybe?

I also can verify the post and give a custom message saying the string cannot be "Test"
With regex i get error "The Test input has not a valid format". (for this to correct i have to put a custom validation error message)
While a is_not_string[test] can give an error message like "The Test field cannot be Test".
I would really like this is_not_string in the validation form.

We have an in_list rule already, it would make sense to have the inverse of that, not_in_list.


MyObservations signedInUsers test error

Error when running test suite
 FAIL  tests/integration/MyObservations.test.js (7.019 s)
  ● MyObservations › when signed out › should not make a request to observations/updates

    expect(received).toEqual(expected) // deep equality

    Expected: 0
    Received: 1

      53 |       expect( apiMethod ).not.toHaveBeenCalled( );
      54 |       const signedInUsers = global.realm.objects( "User" ).filtered( "signedIn == true" );
    > 55 |       expect( signedInUsers.length ).toEqual( 0 );
         |                                      ^
      56 |       renderAppWithComponent( <MyObservationsContainer /> );
      57 |       const loginText = i18next.t( "Log-in-to-contribute-your-observations" );
      58 |       expect( await screen.findByText( loginText ) ).toBeTruthy( );

      at toEqual (tests/integration/MyObservations.test.js:55:38)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:22:9)
      at _next (node_modules/@babel/runtime/helpers/asyncToGenerator.js:27:7)
      at node_modules/@babel/runtime/helpers/asyncToGenerator.js:19:12
      at apply (tests/integration/MyObservations.test.js:51:42)
      at Object.testApiMethodNotCalled (tests/integration/MyObservations.test.js:69:13)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:3:24)
      at asyncGeneratorStep (node_modules/@babel/runtime/helpers/asyncToGenerator.js:22:9)
      at _next (node_modules/@babel/runtime/helpers/asyncToGenerator.js:27:7)
      at Object.<anonymous> (node_modules/@babel/runtime/helpers/asyncToGenerator.js:19:12)

I do not get this error on main.


[AutoRelease] t2-storage-2024-06-19-95149(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/5261
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=842
BuildTargetingString
azure-mgmt-storage
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/5261


GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets.

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-708

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Create a new DatasetDescriptor for JDBC sourced datasets.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Added Unit test in SqlDatasetDescriptorTest.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

+1


Concurrency ABI for global-actor-qualified function types

Explanation: Implements all of the ABI-impacting pieces of global-actor-qualified function types, including name mangling, type metadata, and reflection capabilities. As part of this, provides standard mangled names for entities in the _Concurrency library, which reduces the length of mangled names for programs using concurrency.
Main branch PRs: https://github.com/apple/swift/pull/37733, https://github.com/apple/swift/pull/37759, https://github.com/apple/swift/pull/37769
Radar: rdar://78269642
Risk: Moderate. This is a non-trivial change to IR generation, the Swift runtime, and reflection, and a significant ABI break.
Testing: normal compiler/runtime regression tests, including a number of new tests for round-tripping between the various forms type metadata can take.

@swift-ci please test

@swift-ci please nominate


[BUG] Footer button doesn't scroll to top

Description
Footer button for FAQ's, about us etc doesn't scroll to top and user has to do it manually .
I would like to work on this issue.
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@kunalkashyap-1 It is working from my side.

Screencast from 2023-05-25 20-48-43.webm
I changed the browser as well the issue consists i would like to write a section of code explicitly to scroll back to top when clicked.

@kunalkashyap-1 ok got ur issue. Go ahead.


Remove senseless spec.

A negative size makes no sense at all, -infinity even less so. It is indeed the case that Ruby currently returns -infinity (even if you take the product of two of these senseless enumerables), but that doesn't make it a valid spec. Ruby does this because it is simpler to test for ±infinity than it is to test for +infinity...
Other alternative implementations should not attempt to mimick such behaviour.
One could argue it is a bug (and nil should be returned maybe, but that would be the case for any negative size), or that the current result is acceptable because  "garbage in garbage out", but nobody in their right mind would rely on this behavior and clearly returning nil or raising an exception are better results than returning - infinity.
This PR leaves this uninteresting and ill-defined corner case as implementation dependent.
Ref: https://github.com/marcandre/backports/pull/192

Agree!
Wondering if it makes sense to have means to specify explicitly in specs that some case has undefined behaviour or implementation-specific one? In case a CRuby behaviour is considered as not accurate?


Data volume test - add tests ids

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test pull-kubevirt-e2e-okd-4.1

/approve


🎉 6.0.1

ℹ About this release

Version: 6.0.1
Type: patch
Last Release: 6.0.0 (8/26/2024, 6:48:31 PM) [?]
Commits to merge: 24 [?]

📦 Dependencies

Update express from ^4.19.2 to ^4.21.1
Update ical-generator from ^7.2.0 to ^8.0.0

📦 Development Dependencies

Update @eslint/js from ^9.9.0 to ^9.11.1
Update @sebbo2002/semantic-release-docker from ^4.0.3 to ^5.0.1-develop.4
Update @types/express from ^4.17.21 to ^5.0.0
Update @types/mocha from ^10.0.7 to ^10.0.9
Update @types/node from ^22.5.0 to ^22.7.5
Update eslint from ^9.9.0 to ^9.11.1
Update nyc from ^17.0.0 to ^17.1.0
Update tsup from ^8.1.0 to ^8.3.0
Update tsx from ^4.16.2 to ^4.19.1
Update typedoc from ^0.26.6 to ^0.26.8
Update typescript from ^5.5.4 to ^5.6.2
Update typescript-eslint from ^8.2.0 to ^8.8.1

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

npm package (@latest dist-tag)
GitHub release
Docker container (sebbo2002/fitness-first-ical-proxy:6.0.1)
v6.0.1

Your semantic-release bot :package::rocket:


cj-001-headerHTMLManinder

Add the creative juices png as the header background

The header portion of this website will use the image which has our creative juice logo on it. The image is attached to the starter code.

The header has been created. Currently working on the landing page