[Bug]: RSpack produces inflated bundle sizes as compared to other bundlers in a few cases

System Info
System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 102.23 MB / 32.00 GB
    Shell: 3.7.0 - /opt/homebrew/bin/fish
  Binaries:
    Node: 18.17.1 - ~/Library/Caches/fnm_multishells/38909_1717317465477/bin/node
    Yarn: 4.2.2 - ~/Library/Caches/fnm_multishells/38909_1717317465477/bin/yarn
    npm: 10.8.1 - ~/Library/Caches/fnm_multishells/38909_1717317465477/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Chrome: 125.0.6422.141
    Safari: 17.4.1
  npmPackages:
    @rspack/core: ^0.7.0 => 0.7.0

Details
Consider a blank project that only has one dependency — underscore.
import * as _  from 'underscore'

console.log(_)

Bundling the project using rspack or a fresh rsbuild project produces an output —
  dist/index.html                     0.32 kB    0.22 kB
  dist/static/js/index.919c8a90.js    1.4 kB     0.73 kB
  dist/static/js/718.655677ba.js      42.3 kB    11.2 kB

This is 42 KB minified.
However, an actual ESM minified build of rsbuild is only 25KB minified —
<EMAIL_ADDRESS>RSpack might be in-effecient at packing multiple small functions into a smaller space.
Reproduce link
https://github.com/pastelsky/rspack-repro-underscore
Reproduce Steps

Run yarn install
Run yarn build
Observe the built sizes

webpack/rspack generates runtime code for every small function. Maybe concatenateModules could help.

This build runs in production mode — so I assumed module concatenation would be on.

🤔 I manually set concatenateModules to true on top of your config and it did work.
  File                                        Size       Gzipped
  rsbuild-dist/index.html                     0.32 kB    0.22 kB
  rsbuild-dist/static/js/index.3a9fa5b5.js    1.4 kB     0.74 kB
  rsbuild-dist/static/js/712.32ef6620.js      26.4 kB    8.0 kB

  Total size:  28.1 kB
  Gzipped size:  8.9 kB

Interesting, I don't see such an option documented here —
https://www.rspack.dev/config/optimization
Where is this documented?
Also, would you know why this isn't enabled by default?

It's true that rspack document is not always up-to-date, which is a problem(I don't know how the team will improve this workflow in the future).
For now, we know rspack aims for alignment with webpack, so you may find some available configuration in webpack document such as https://webpack.js.org/configuration/optimization/#optimizationconcatenatemodules.
As for why it's not enabled by default in production mode, I guess this option was forgotten to be adjusted at the end of the experimental phase.
rspack: https://github.com/web-infra-dev/rspack/blob/4fcc42ce9bef0048901308205b5a188a1d7ddd50/packages/rspack/src/config/defaults.ts#L916
webpack: https://github.com/webpack/webpack/blob/8241da7f1e75c5581ba535d127fa66aeb9eb2ac8/lib/config/defaults.js#L1152
#6769
I'll make a simple pull request.

@CPunisher concatenateModules is not enabled in production because we're still testing it, we will enable it by default in production when we think it's production ready

@hardfist 😄 That's what I'm going to ask. After I changed the default configuration, there are some test cases with strange errors.

since concatedmodule is enabled by default in prouction now, I think this can be closed


What about the license?

Hallo anybody,
is this set free for commercial use?
great work, regards!

Refer to the repository LICENSE file, it's MIT, so to answer your question: Commercial use is permitted by the MIT license. :)


Explore pre-commit hooks

Explore if we can have a pre-commit hook to check the sorted order of modules in init.py

Hi! I have never contributed to an open-source project before, but need to for a school project. Do you think this would be a reasonable bug for me to handle? I have used Python before.
Thanks
Aliya

Yep! Go ahead. Let me know if you face any issues. 😄

@aliyareneekhan are you still working on this?

@suhasjrv Take a look at https://githooks.com/

I've created a pre-commit hook for this. Should I add a directory for hooks so client can manually link to the hook?

Yes. Try adding it at .git/hooks/pre-commit

@swapagarwal I have already added it there, but .git is not pushed to remote. I'm wondering how you would like it to be added to this repo.

Hi. May I know if the issue is still open. The tag indicates that it is so but I saw that a pull request has already been accepted. Thanks.

Hello, I am a new user to github. Can anyone tell me what exactly one can do to contribute to open source and how to solve an issue ?

Is the issue still open?

Hey if this issue is still open I'm happy to take a crack at it.

Checking the issue is still open, would love to contribute.

Checking the issue is still open, would love to contribute.

+1

Is this issue open then I would like to contribute

Is this issue still open? I'd like to contribute

Is the issue still open?

Is the issue still open?

Is this issue still open, if yes i would like to work on it


pod install faild

I use command pod install to install Loggerithm as Readme says, but the console logs Unable to find a pod with name matching Loggerithm'`. It seems something wrong happend to Loggerithm with cocoapods.

Can you show me your podfile? Or you can try pod try Loggerithm to see whether it works on your computer

My pod file as beblow:
source 'https://github.com/CocoaPods/Specs.git'
use_frameworks!

pod 'Loggerithm', '~> 1.2'

Maybe I need reinstall my cocoapods in my computer.

@dabing1022 Hey, your pod file looks good. Have you tried to install other pods? I'm not sure about your network accessibility to 'https://github.com/CocoaPods/Specs.git'. Anyway, let me know how it works now

Sorry to feed back. After I reinstall my cocoapods, everythings looks well. Many thanks.

@dabing1022 No problems


respect casing when comparing names of config files

// cc @mhegazy, @zhengbli

👍

Tested the fix works as expected.


feat: enhance detection by giving priorities to framework based component

it resolves #114

@kadel tried to implement the "multiple devfiles detection". First thing i noticed is that in some cases it could return several devfiles which is not nice. E.g for a maven/angular application i got
[
	{
		"Name": "java-maven",
		"Language": "Java",
		"ProjectType": "Maven",
		"Tags": [
			"Java",
			"Maven"
		]
	},
	{
		"Name": "java-openliberty",
		"Language": "Java",
		"ProjectType": "Open Liberty",
		"Tags": [
			"Java",
			"Maven"
		]
	},
	{
		"Name": "java-websphereliberty",
		"Language": "Java",
		"ProjectType": "WebSphere Liberty",
		"Tags": [
			"Java",
			"Maven",
			"WebSphere Liberty"
		]
	},
	{
		"Name": "java-wildfly-bootable-jar",
		"Language": "Java",
		"ProjectType": "WildFly",
		"Tags": [
			"RHEL8",
			"Java",
			"OpenJDK",
			"Maven",
			"WildFly",
			"Microprofile",
			"WildFly Bootable"
		]
	},
	{
		"Name": "nodejs-angular",
		"Language": "TypeScript",
		"ProjectType": "Angular",
		"Tags": [
			"Node.js",
			"Angular"
		]
	}
]

The outcome is correct as every java devfile contains the tag Maven in them but... does it look fine for you? My concern is that the final user would see this list and would start thinking "WTF is it asking me if i want to use a wildfly devfile?" ... :laughing:
So do you want me to restrict the list somehow? E.g i would pick the devfiles java-maven and java-openliberty because we cover 100% of their tags so it's most likely that they are more relevant to the project? WDYT?

@kadel now alizer provides the functions that return a list of devfiles. You just need to use the proper function (just add an s to devfile keyword.)
E.g
SelectDevFileFromRegistry -> SelectDevFilesFromRegistry
SelectDevFileUsingLanguagesFromTypes -> SelectDevFilesUsingLanguagesFromTypes
SelectDevFileFromTypes -> SelectDevFilesFromTypes


Unexpected error fetching DWD Weather Coordinator for amsterdam data: 'Nordholland'

Host-besturingssysteem | Home Assistant OS 5.13
Supervisor versie | supervisor-2021.04.0

v1.2.10; already present v1.2.9.

Describe the bug
Unexpected error fetching DWD Weather Coordinator for amsterdam data: 'Nordholland'
it happens at startup time and continuously creating errors (229 errors in about 24 hrs time)
To me, it looks like closed issue 26 - Saarland
Here is the problem "Nordholland'.

Logger: custom_components.dwd_weather
Source: custom_components/dwd_weather/connector.py:61
Integration: Deutscher Wetterdienst (DWD) (documentation, issues)
First occurred: 9:57:29 (229 occurrences)
Last logged: 13:45:29

Unexpected error fetching DWD Weather Coordinator for amsterdam data: 'Nordholland'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 173, in _async_refresh
    self.data = await self._async_update_data()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 143, in _async_update_data
    return await self.update_method()
  File "/config/custom_components/dwd_weather/connector.py", line 54, in async_update
    return await self._hass.async_add_executor_job(self._update)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/dwd_weather/connector.py", line 61, in _update
    self.dwd_weather.update()
  File "/usr/local/lib/python3.8/site-packages/simple_dwd_weatherforecast/dwdforecast.py", line 393, in update
    weather_report = download_weather_report(self.region_codes[self.region])
KeyError: 'Nordholland'

Yes, you are right. Thanks for the report! I'll fix this as soon as possible!

I released a new version, which fixes this error.

Thank you for your quick response and effort put in to fix the error!
Installed and DWD functions without the reported error.
I close this issue.


[9.x] Fixes rename method from File::isEqual() to File::hasSameHash()

See https://github.com/laravel/framework/pull/41586

Thanks!


Add record to panel and update confidence of record

For published records, these code changes provide functionalities to add record to panel and update confidence of record.
Code changes related to Jira tasks: G2P-85, G2P-238

PR to improve message: https://github.com/EBI-G2P/gene2phenotype_api/pull/38


[BUG] Instant disconnection from voice channel upon /play command

Describe the bug
The bot joins voice channel, adds music to its queue, then instantly disconnects.
To Reproduce
Steps to reproduce the behaviour:

Clone repository, install dependencies,
Copy config.example.yml to config.yml and add private tokens,
Run npm start,
Use /play to play any song in a voice channel.

Expected behaviour
The bot joins my voice channel and starts playing music.
Actual behaviour
The bot joins my voice channel and it instantly disconnects, returning the following message:
The music was stopped because I was disconnected from the channel.
Screenshots

Log File
Log file,
Config file.
Additional context
None.

Hey! Thanks for reporting this issue. Unfortunately, I'm unable to replicate this issue at the moment. Can you please provide some additional information:

What operating system you're running Melody on (Windows, Mac, Ubuntu, etc.)
Whether you're running Melody within a Docker container, or are hosting the bot with a hosting company

Thanks :)

Hey there! I'm running Melody on an Arch Linux server in my VPS (no docker).

i have the same issue

I've just pushed an update that allows you to disable autocomplete if you're experiencing Unknown Interaction errors.
As for the original issue, the OP has confirmed to me through Discord that the bot is working flawlessly on Windows 10, inferring that it's most likely an issue with some Linux distros. It's likely that it's an issue with ffmpeg or some of its static binaries, but the issue seems to be out of my control. I'll close this for now, but if there's any updates I find I'll try and remember to post them here.


Optimize the buffer size in inverse_transform_add

The zeroing of this buffer was a hotspot within the native implementation of inverse_transform_add. Changed it to only zero a buffer equivalent to the size that will be used. This is significantly more efficient for smaller-than-64x64 transforms and equivalent for 64x64.
This results in an overall 2% encoding time improvement on HBD content at speed 5.
> hyperfine -w 1 "~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10" "~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10"
Benchmark #1: ~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10
  Time (mean ± σ):     15.188 s ±  0.079 s    [User: 15.169 s, System: 0.072 s]
  Range (min … max):   15.109 s … 15.383 s    10 runs

Benchmark #2: ~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10
  Time (mean ± σ):     14.953 s ±  0.061 s    [User: 14.939 s, System: 0.065 s]
  Range (min … max):   14.877 s … 15.053 s    10 runs

Summary
  '~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10' ran
    1.02 ± 0.01 times faster than '~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/blue_sky_360p_60f.y4m --limit 10'

> hyperfine -w 1 "~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10" "~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10"
Benchmark #1: ~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10
  Time (mean ± σ):     48.159 s ±  0.115 s    [User: 48.106 s, System: 0.146 s]
  Range (min … max):   47.951 s … 48.273 s    10 runs

Benchmark #2: ~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10
  Time (mean ± σ):     47.200 s ±  0.182 s    [User: 47.157 s, System: 0.147 s]
  Range (min … max):   47.025 s … 47.678 s    10 runs

Summary
  '~/Downloads/rav1e-try1 -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10' ran
    1.02 ± 0.00 times faster than '~/Downloads/rav1e-master -s 5 --quantizer 42 -o /dev/null ~/data/objective-1-fast-10bit/KristenAndSara_1280x720_60f.y4m --limit 10'

Coverage remained the same at 65.187% when pulling ec4d6e7487a844edddbe16ac618174c888a5f7ca on shssoichiro:ivtx-add-optim-hbd into 3649a2cee102523481b228b8d2c860aebcd4b28a on xiph:master.


Support custom hint text size

Customizable with ms_hintTextSize.
This is usefull when you use custom spinner_item with smaller text size.

Thanks for it


How to generate openapi specification without running the app?

Need help/suggestion here.
Use case - I am trying to use openapi.json file during CI pipeline execution. I need to upload it somewhere for further use.
openapi.json file is generated at run time and served at http://<IP_ADDRESS>:8090/docs/openapi.json
I can't run the app during CI pipeline execution due to other dependencies and it seems like a security issue as well.
Can someone please help here on how can I generate openapi.json file without actually running the app?
@ahopkins

Wow, I am not sure this would be easy to do. Sure, it's Python so everything should be possible. But it will take quite a bit of hacking. You would need to instantiate your Sanic app, then attach the blueprint using the blueprint factory which has some startup time methods to build the spec, and ultimate calling:
SpecificationBuilder().build(app).serialize()

Sanic is not really meant for offline usage, and the spec builder certainly not.
You are probably much better off running your app in CI with something like this:
from sanic_ext.extensions.openapi.builders import SpecificationBuilder
from ujson import dumps
from pathlib import Path

@app.after_server_start
async def build_spec(app):
    spec = SpecificationBuilder().build(app).serialize()
    Path("/some/location/spec.json").write_text(dumps(spec))
    app.stop()

Thanks @ahopkins.. Life saver :)


Helped me

Very helpful descriptions

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

ID: 8cfb6a8b-c7e5-1bf6-0c24-2757871e9d07
Version Independent ID: da5ffb60-d0c8-7117-02fe-7a3fd33169ac
Content: Change a Master Boot Record (MBR) into a GUID partition table (GPT) disk
Content Source: WindowsServerDocs/storage/disk-management/change-an-mbr-disk-into-a-gpt-disk.md
Product: windows-server-threshold
Technology: storage
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:IngridAtMicrosoft


Update the box-shadow implementation

Update the implementation of the box-shadow CSS property and make it close/similar to how it is currently implemented in Clay. This is a breaking change.
Resolves #87

Rebased, ready for CR

ah, there is missing None instance

ok, added support for global keywords and none, rebased, ready for review


docs: improvements

Some typo, grammar, and sentence structure updates to clarify (I think) the content.
This looks like a neat project!

Thanks!


Query Parameters in the redirectURI aren't handled correctly

If the redirectUri contains a query string parameter and you pass an $options array into AbstractProvider::getAuthorizationUrl($options), the resulting URL contains two question-marks, because the AbstractProvider::appendQuery($url, $query) always "glues" the $url and $query together with a '?'.
I suggest that appendQuery() be updated to choose an appropriate separator as follows:
            $separator = strpos($url, "?") === false ? "?" : "&";
            return $url . $separator . $query;

Happy to submit a PR if you agree - unless there's a reason it should not be changed?
(I appreciate that the extra query param(s) could be passed into the getAuthorizationUrl() method rather than being included in the redirectURI, but when offering multiple providers for login and a single callback url, a ?Provider=xyz parameter included in the redirectUri supplied to the constructor is more "obvious")

My apologies - just noticed there's a PR open for this already (#532).  Closing this issue.


validation not working in inventory page

Bagisto Version

[ ] master branch

Steps To Reproduce

go to admin panel

then go to inventory sources

Actual Result

@singhgagan03 I have installed a fresh setup of Bagisto (master branch) and could not reproduce this.
These fields are not required so no validation is set.

Latitude
Longitude
Priority

The contact number does have a validation set

@ghermans  character should not be insert  in longitute, latitue , priority and also in contact no fields

@devansh-webkul
Thanks, now this issue is fixed. And I have attached a screenshot for future reference.


Memory leak on Travis in evaluate.py

https://travis-ci.com/github/singlecellopenproblems/SingleCellOpenProblems/jobs/382491851#L970

@LuckyMD it looks like scBI's scran normalization is the problem here. See https://github.com/singlecellopenproblems/SingleCellOpenProblems/blob/feature/method-metadata/results.md

(results here are for test data -- I can't run the full evaluation on my machine due to memory issues.)

I can just see ModuleNotFoundError: No module named 'memory_profiler'
Where is the memory leak shown?

If you take a look at the .md file I linked you'll see it. I've also given a reproducible example in the linked issue.

Ah, I didn't scroll right...

Looks like this is resolved. https://github.com/singlecellopenproblems/SingleCellOpenProblems/blob/master/results.json

Hmm... I hadn't merged the scIB PRs yet... did you install the PR code?

Ah yes I did, this is running locally.
On Thu, 24 Sep 2020 at 13:40, MalteDLuecken<EMAIL_ADDRESS>wrote:

Hmm... I hadn't merged the scIB PRs yet... did you install the PR code?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/singlecellopenproblems/SingleCellOpenProblems/issues/54#issuecomment-698488211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA3DX7CRK6FZ4D7QL5VNDLSHOACHANCNFSM4RCXCT2Q
.

okay.. i guess no need to close the R session then. I will merge then :)

Merged... now you can use the master branch as well ;)

Excellent, thanks!
On Fri, 25 Sep 2020 at 09:52, MalteDLuecken<EMAIL_ADDRESS>wrote:

Merged... now you can use the master branch as well ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/singlecellopenproblems/SingleCellOpenProblems/issues/54#issuecomment-698941676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA3DX5H2IODXSL6QU3E45TSHSODTANCNFSM4RCXCT2Q
.


Staging and ActionGroups problems

Hello, some small problems comes up with the last Version

it's not possible to set the Fairings on an ActionKey
right click on the Staging Symbol and chose 'jetisson' release only the first of n: fairing-parts
while the Fairing is not jetissoned, MechJeb can't display/calculate the Stages

all other is great!

Just wanted to open an issue about this - after the latest build from today this is still an issue.


[race] - Added race function, refactored timeout to use it - TT

This PR adds a new public function called Race which allows consumers to race 2 tasks and rely on the first result. This is obviously useful in timeout type situations, but should allow for a much more sane refactor of the throttle operator and further allow for test stabilization in Throttle
Example usage:
try await Race {
    try await clock.sleep(until: clock.now.advanced(by: duration), tolerance: tolerance)
    throw CancellationError()
} against: {
    try await someOperation()
}

Love it! I wonder if we should support any number of potential "race" candidates?
E.g.
Race({
  // task 1
}, against: {
 // task 2
}, {
 // task 3
})

Testing out a variadic func signature locally, the trailing closure syntax still works fine for the 2 candidate case.

Love it! I wonder if we should support any number of potential "race" candidates?
E.g.
Race({
  // task 1
}, against: {
 // task 2
}, {
 // task 3
})

Testing out a variadic func signature locally, the trailing closure syntax still works fine for the 2 candidate case.

This is a good idea, I just pushed that change.


shield_max, energy_max, and weapon_cooldown are always zero (for replays)

I am looking a replay as observer (player_id = 0). During onStep() is called, I read the following values of all units:

health
health_max
shield
shield_max
energy
energy_max
weapon_cooldown

For the replayes I watched the values shield_max, energy_max, and weapon_cooldown are always zero. Although shield and energy are not always zero. Is this a bug related to replays or observers? Can you fix it?

I just tested this and I'm not seeing the same issue.
Could you possible provide code to reproduce the issue?

Here is a minimal example. Maybe the problem is specific to the replay I used...
#include <sc2api/sc2_api.h>
#include <iostream>

class MyObserver : public sc2::ReplayObserver {
public:
	bool firstShield = true;
	bool firstEnergy = true;

	void OnStep() final {
		auto obs = Observation();
		for (auto it : obs->GetUnits()) {
			if (firstShield && (it->shield > 0) && (it->shield_max == 0)) {
				std::cout << obs->GetGameLoop() << ": Found invalid shield values!" << std::endl;
				firstShield = false;
			}
			if (firstEnergy && (it->energy > 0) && (it->energy_max == 0)) {
				std::cout << obs->GetGameLoop() << ": Found invalid energy values!" << std::endl;
				firstEnergy = false;
			}
		}
	}

	bool IgnoreReplay(const sc2::ReplayInfo& replay_info, uint32_t& player_id) final {
		player_id = 0;
		return false;
	}
};

int main(int argc, char* argv[]) {
	sc2::Coordinator coordinator;
	coordinator.LoadSettings(argc, argv);

	const char* kReplayFolder = R"(D:\path\to\replaypack1\0b5f25278d983edbe980a6ea723de75d281b90b21a2ac60286091ffcec9a5fb3.SC2Replay)";
	if (!coordinator.SetReplayPath(kReplayFolder)) {
		std::cout << "Unable to find replay." << std::endl;
		return 1;
	}

	MyObserver myObserver;
	coordinator.AddReplayObserver(&myObserver);

	while (coordinator.Update());
	return 0;
}

If I compile and run the program, I get the following output:

...
1: Found invalid shield values!
2985: Found invalid energy values!

Thanks for your help. I start to have the bad feeling that the problem is in front of my computer ;)


Deck name sanitization improvements, better intro box.

Hey @bakert,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 2c6f47a0-07ca-11ea-a7d8-2f45f8a3b8c0


chore(master): release 2.1.47

:robot: I have created a release beep boop
2.1.47 (2024-04-11)
Bug Fixes

deps: update dependency stripe/stripe-php to v14.1.0 (65b174d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v2.1.47 :sunflower:


Executing the suggested code blocks

Thank you for the awesome package!
What would be the best way to execute the code provided in chatGPTs response?

Cool idea! I'm not sure how we'd accomplish that but it would be cool. The tricky thing (I think) would be identifying part of the response is Julia code. Sometimes ChatGPT puts code snippets in fences but sometimes it just emits them without markdown too. Also, the user might sometimes ask for code in a different language, and we'd have to figure out whether or not it's code Julia should try to run (although this might be doable with Julia's reflection and metaprogramming stuff!).
Right now I'm just using the Markdown package to render terminal outputs, so we'd have to switch away from that or modify it to have a little "Run" button... not quite sure how to do that either but feels approachable.
Let's keep mulling on this and see if any good ideas come up!


Advanced worksheet creation

The current projects system on makecode.microbit.org just shows the whole program at each step ("modify your code so it looks like this"). This breaks down for larger programs. Even for an adult, it's going to become hard to see how we got from A to B.
A great worksheet makes it really clear what has changed at each step. So a great worksheet system would provide an elegant way to highlight the changes.
The two main questions:

How to represent this for editing.
What sort of visual appearance actually works best.

It would be quite a big project, and I'm not sure if it might fit better on the Blockly side, but any thoughts?

So far, I've been editing screenshots in an image editor, which is not ideal!
Thoughts so far regarding the appearance:
When a worksheet zooms in on the area to be changed, it seems a lot of children have difficulty understanding when they need to add blocks to what they made already. I've often seen them build a second copy of the code segment. (This isn't just with the micro:bit - I've seen it when they're working from published Scratch textbooks too.)
I tried drawing boxes and arrows saying "add these blocks", but this didn't seem to be good enough. Most recently, I tried making the previous blocks grey (in addition to the boxes/arrows) - the two children who tried it so far both understood, but that's too small a sample to draw any real conclusions.
I haven't yet needed to show the removal of a block...


10.7.3 Prebuilt Carthage XCFramework Lacks Info.plist And Xcode Won't Compile.

https://github.com/realm/realm-cocoa/releases/tag/v10.7.3
The Info.plist is used by Xcode to identify the framework from xcframework. Without it, Xcode could find the specify framework for specific architecture and target.
The full package "realm-swift-10.7.3.zip" contains the Info.plist that xcframework lacks. Anyone has to download the full package to get the whole xcframework.
Also, it would be welcomed to provide the prebuilt xcframework for Xcode 12.5 RC. RC is the name for GM for Apple's products.

Hi @owenzhao,
Thank you for reporting the issue. Will be fixed asap


Molecule Lint (Flake8) cannot import name ConverterMapping

Issue Type

Bug report

Molecule and Ansible details
oscar@infinitum:~/code/devops/ansible-roles/groups$ ansible --version
ansible 2.7.6
  config file = None
  configured module search path = [u'/home/oscar/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/oscar/.local/lib/python2.7/site-packages/ansible
  executable location = /home/oscar/.local/bin/ansible
  python version = 2.7.15+ (default, Oct  2 2018, 22:12:08) [GCC 8.2.0]

oscar@infinitum:~/code/devops/ansible-roles/groups$ molecule --version
molecule, version 2.19.0

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
Desired Behavior
I run molecule lint or molecule test and the lint process runs without software issue.
Actual Behaviour
Please give some details of what is actually happening.
Include a minimum complete verifiable example with
output of running molecule --debug.
oscar@infinitum:~/code/devops/ansible-roles/groups$ molecule lint
--> Validating schema /home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml.
Validation completed successfully.
--> Test matrix

└── default
    └── lint

--> Scenario: 'default'
--> Action: 'lint'
--> Executing Yamllint on files found in /home/oscar/code/devops/ansible-roles/groups/...
Lint completed successfully.
--> Executing Flake8 on files found in /home/oscar/code/devops/ansible-roles/groups/molecule/default/tests/...
Traceback (most recent call last):
  File "/home/oscar/.local/bin/flake8", line 7, in <module>
    from flake8.main.cli import main
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/cli.py", line 2, in <module>
    from flake8.main import application
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/application.py", line 14, in <module>
    from flake8.main import options
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/options.py", line 4, in <module>
    from flake8.main import vcs
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/vcs.py", line 4, in <module>
    from flake8.main import mercurial
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/mercurial.py", line 7, in <module>
    import configparser
  File "/home/oscar/.local/lib/python2.7/site-packages/configparser.py", line 12, in <module>
    from backports.configparser import (
ImportError: cannot import name ConverterMapping

Debug on:
oscar@infinitum:~/code/devops/ansible-roles/groups$ molecule --debug lint
--> Validating schema /home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml.
Validation completed successfully.
--> Test matrix

└── default
    └── lint

--> Scenario: 'default'
--> Action: 'lint'
--> Executing Yamllint on files found in /home/oscar/code/devops/ansible-roles/groups/...
DEBUG: ANSIBLE ENVIRONMENT
--- {}

DEBUG: MOLECULE ENVIRONMENT
---
MOLECULE_DEBUG: 'True'
MOLECULE_DEPENDENCY_NAME: galaxy
MOLECULE_DRIVER_NAME: docker
MOLECULE_ENV_FILE: /home/oscar/code/devops/ansible-roles/groups/.env.yml
MOLECULE_EPHEMERAL_DIRECTORY: /tmp/molecule/groups/default
MOLECULE_FILE: /home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml
MOLECULE_INSTANCE_CONFIG: /tmp/molecule/groups/default/instance_config.yml
MOLECULE_INVENTORY_FILE: /tmp/molecule/groups/default/ansible_inventory.yml
MOLECULE_LINT_NAME: yamllint
MOLECULE_PROJECT_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups
MOLECULE_PROVISIONER_LINT_NAME: ansible-lint
MOLECULE_PROVISIONER_NAME: ansible
MOLECULE_SCENARIO_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups/molecule/default
MOLECULE_SCENARIO_NAME: default
MOLECULE_VERIFIER_LINT_NAME: flake8
MOLECULE_VERIFIER_NAME: testinfra
MOLECULE_VERIFIER_TEST_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups/molecule/default/tests

DEBUG: SHELL REPLAY
MOLECULE_DEBUG=True MOLECULE_DEPENDENCY_NAME=galaxy MOLECULE_DRIVER_NAME=docker MOLECULE_ENV_FILE=/home/oscar/code/devops/ansible-roles/groups/.env.yml MOLECULE_EPHEMERAL_DIRECTORY=/tmp/molecule/groups/default MOLECULE_FILE=/home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml MOLECULE_INSTANCE_CONFIG=/tmp/molecule/groups/default/instance_config.yml MOLECULE_INVENTORY_FILE=/tmp/molecule/groups/default/ansible_inventory.yml MOLECULE_LINT_NAME=yamllint MOLECULE_PROJECT_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups MOLECULE_PROVISIONER_LINT_NAME=ansible-lint MOLECULE_PROVISIONER_NAME=ansible MOLECULE_SCENARIO_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups/molecule/default MOLECULE_SCENARIO_NAME=default MOLECULE_VERIFIER_LINT_NAME=flake8 MOLECULE_VERIFIER_NAME=testinfra MOLECULE_VERIFIER_TEST_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups/molecule/default/tests

DEBUG: COMMAND
/home/oscar/.local/bin/yamllint -s /home/oscar/code/devops/ansible-roles/groups/handlers/main.yml /home/oscar/code/devops/ansible-roles/groups/molecule/default/playbook.yml /home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml /home/oscar/code/devops/ansible-roles/groups/vars/main.yml /home/oscar/code/devops/ansible-roles/groups/defaults/main.yml /home/oscar/code/devops/ansible-roles/groups/meta/main.yml /home/oscar/code/devops/ansible-roles/groups/tasks/main.yml

Lint completed successfully.
--> Executing Flake8 on files found in /home/oscar/code/devops/ansible-roles/groups/molecule/default/tests/...
DEBUG: ANSIBLE ENVIRONMENT
--- {}

DEBUG: MOLECULE ENVIRONMENT
---
MOLECULE_DEBUG: 'True'
MOLECULE_DEPENDENCY_NAME: galaxy
MOLECULE_DRIVER_NAME: docker
MOLECULE_ENV_FILE: /home/oscar/code/devops/ansible-roles/groups/.env.yml
MOLECULE_EPHEMERAL_DIRECTORY: /tmp/molecule/groups/default
MOLECULE_FILE: /home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml
MOLECULE_INSTANCE_CONFIG: /tmp/molecule/groups/default/instance_config.yml
MOLECULE_INVENTORY_FILE: /tmp/molecule/groups/default/ansible_inventory.yml
MOLECULE_LINT_NAME: yamllint
MOLECULE_PROJECT_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups
MOLECULE_PROVISIONER_LINT_NAME: ansible-lint
MOLECULE_PROVISIONER_NAME: ansible
MOLECULE_SCENARIO_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups/molecule/default
MOLECULE_SCENARIO_NAME: default
MOLECULE_VERIFIER_LINT_NAME: flake8
MOLECULE_VERIFIER_NAME: testinfra
MOLECULE_VERIFIER_TEST_DIRECTORY: /home/oscar/code/devops/ansible-roles/groups/molecule/default/tests

DEBUG: SHELL REPLAY
MOLECULE_DEBUG=True MOLECULE_DEPENDENCY_NAME=galaxy MOLECULE_DRIVER_NAME=docker MOLECULE_ENV_FILE=/home/oscar/code/devops/ansible-roles/groups/.env.yml MOLECULE_EPHEMERAL_DIRECTORY=/tmp/molecule/groups/default MOLECULE_FILE=/home/oscar/code/devops/ansible-roles/groups/molecule/default/molecule.yml MOLECULE_INSTANCE_CONFIG=/tmp/molecule/groups/default/instance_config.yml MOLECULE_INVENTORY_FILE=/tmp/molecule/groups/default/ansible_inventory.yml MOLECULE_LINT_NAME=yamllint MOLECULE_PROJECT_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups MOLECULE_PROVISIONER_LINT_NAME=ansible-lint MOLECULE_PROVISIONER_NAME=ansible MOLECULE_SCENARIO_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups/molecule/default MOLECULE_SCENARIO_NAME=default MOLECULE_VERIFIER_LINT_NAME=flake8 MOLECULE_VERIFIER_NAME=testinfra MOLECULE_VERIFIER_TEST_DIRECTORY=/home/oscar/code/devops/ansible-roles/groups/molecule/default/tests

DEBUG: COMMAND
/home/oscar/.local/bin/flake8 /home/oscar/code/devops/ansible-roles/groups/molecule/default/tests/test_groups.py

Traceback (most recent call last):
  File "/home/oscar/.local/bin/flake8", line 7, in <module>
    from flake8.main.cli import main
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/cli.py", line 2, in <module>
    from flake8.main import application
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/application.py", line 14, in <module>
    from flake8.main import options
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/options.py", line 4, in <module>
    from flake8.main import vcs
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/vcs.py", line 4, in <module>
    from flake8.main import mercurial
  File "/home/oscar/.local/lib/python2.7/site-packages/flake8/main/mercurial.py", line 7, in <module>
    import configparser
  File "/home/oscar/.local/lib/python2.7/site-packages/configparser.py", line 12, in <module>
    from backports.configparser import (

Attempted pip uninstall
pip-purge pip tool
apt purge python*
reinstall Ubuntu 18.10 LTS

Please help :)

It was very clearly a local config issue.
I was using python2.7/pip to install.
I did the following:

sudo rm -rf /usr/lib/python2.7/dist-packages
pip install pip-purge && pip-purge
sudo apt purge python-pip
sudo apt isntall python3-pip
pip3 install ansible molecule docker --no-cache-dir

Working just fine with pip3!


ebus.0.cmd - Syntax definition?

Hi there,
I have the problem that my command which I put into ebus.0.cmd does not work. I assume that I am not writing the correct format.
My command "ebusctl write -c 700 z1OpMode night" works fine if I send it via propt over putty.
The mode changes correctely in my heating system.
My ebusd instance in ioBroker works fine too. All data is shown and updated all 5 minutes as set up.
I tried "write -c 700 z1OpMode night" in ebus.0.cmd aus described.
I see that the command is send but the mode doesn't change.
And I do not get any feedback in ebus.0.cmdResult - it's allways blank.
I tried some other syntax - no luck.
Can someone shortly help me understanding how the ebus.0.cmd has to be used correctely?

Looks like that I have a communication problem.
iobroker and ebusd are running on the same host. So ebusd.iobroker is communication with "localhost"
On processing a write command - I receive in the log an error:
"exception from tcp socket[Error: connect ECONNREFUSED <IP_ADDRESS>:8890]"

OK - solved.
Listening port for incomming commands on ebusd config file was missing.
"/etc/ebusd/default" added  --port=8890 (Listen for command line connections connections on PORT [8888])
After that "write -c 700 z1OpMode night" immediately worked and was acknowledged with "done"


Add rooms for bw coin rooms

co-authored-by: @artiwr

I can squash the commits as needed

Did some play testing and we’re missing configs for

7b d-02 right hole
9a h-06b top hole


space in path makes flutter build/flutter run fail (ld: cannot find <rest of path after space)

Steps to reproduce

have path in space
run flutter run
fails

Expected results
builds fine
Actual results
Get the (ld: cannot find <rest of path after space) error.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
Changing current working directory to: /home/stereomato/Documents/Software
Development/Repositories/Personal/spm_gui/spm_gui
Connected devices:
Linux (desktop) • linux  • linux-x64      • NixOS 24.05 (Uakari) 6.7.4-zen1
Chrome (web)    • chrome • web-javascript • Google Chrome 121.0.6167.184
[1]: Linux (linux)
[2]: Chrome (chrome)
Please choose one (or "q" to quit): 1
Launching lib/main.dart on Linux in debug mode...
CMake Error at /nix/store/5h0akwq4cwlc3yp92i84nfgcxpv5xv79-cmake-3.26.4/share/cmake-3.26/Modules/CMakeTestCXXCompiler.cmake:60 (message):
  The C++ compiler

    "/nix/store/i80k2s7lczrgr7s0dqlh1g9hhbc7gknj-clang-wrapper-16.0.6/bin/clang++"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/stereomato/Documents/Software Development/Repositories/Personal/spm_gui/spm_gui/build/linux/x64/debug/CMakeFiles/CMakeScratch/TryCompile-hNTp77

    Run Build Command(s):/nix/store/jyk3j4632ldxsikjq0syjlj16i12jfsx-ninja-1.11.1/bin/ninja -v cmTC_b9996 && [1/2] /nix/store/i80k2s7lczrgr7s0dqlh1g9hhbc7gknj-clang-wrapper-16.0.6/bin/clang++   -isystem /nix/store/3ggcnykapn5ks74b4f7470n79444fwyr-libX11-1.8.7-dev/include -isystem /nix/store/a8wxmz6hcvlpb44z3kf19mzcgak5zqlm-xorgproto-2023.2/include -isystem /nix/store/686lhcz4bwg3wk09pi1xxjgzbxv7ys5q-zlib-1.3-dev/include -MD -MT CMakeFiles/cmTC_b9996.dir/testCXXCompiler.cxx.o -MF CMakeFiles/cmTC_b9996.dir/testCXXCompiler.cxx.o.d -o CMakeFiles/cmTC_b9996.dir/testCXXCompiler.cxx.o -c '/home/stereomato/Documents/Software Development/Repositories/Personal/spm_gui/spm_gui/build/linux/x64/debug/CMakeFiles/CMakeScratch/TryCompile-hNTp77/testCXXCompiler.cxx'
    [2/2] : && /nix/store/i80k2s7lczrgr7s0dqlh1g9hhbc7gknj-clang-wrapper-16.0.6/bin/clang++ -isystem /nix/store/3ggcnykapn5ks74b4f7470n79444fwyr-libX11-1.8.7-dev/include -isystem /nix/store/a8wxmz6hcvlpb44z3kf19mzcgak5zqlm-xorgproto-2023.2/include -isystem /nix/store/686lhcz4bwg3wk09pi1xxjgzbxv7ys5q-zlib-1.3-dev/include -Wl,-rpath,/nix/store/k5j6vpppzrvk810k2w8wlc580prc5vi4-at-spi2-core-2.48.3/lib -Wl,-rpath,/nix/store/2dn1nkq2j96673y94ja1n325n1xqzwbh-cairo-1.16.0/lib -Wl,-rpath,/nix/store/01826v7q0a5l9ppbacg3lw137c43ancj-gdk-pixbuf-2.42.10/lib -Wl,-rpath,/nix/store/zrq9b75k3sv0l45n66rz72c4h03i1dbw-glib-2.76.4/lib -Wl,-rpath,/nix/store/6qp789r70ka5ahxfzrr9l3h6szclsgsr-gtk+3-3.24.38/lib -Wl,-rpath,/nix/store/xn97j65ccmd0gk6n05psdbd05x237k2l-harfbuzz-7.3.0/lib -Wl,-rpath,/nix/store/jkfhz54j6l2937ykyhslamlqrnpv3kzy-libepoxy-1.5.10/lib -Wl,-rpath,/nix/store/0xy54x90g2661i3qpjaipsjjjlvpmc85-pango-1.50.14/lib -Wl,-rpath,/nix/store/b6yspa0filw7fvfabbjfpi2z5k4igv1a-libX11-1.8.7/lib -Wl,-rpath,/nix/store/lsx6fhyyyrym3wg9xln2lhaj9lnsii5h-libdeflate-1.18/lib CMakeFiles/cmTC_b9996.dir/testCXXCompiler.cxx.o -o cmTC_b9996   && :
    FAILED: cmTC_b9996
    : && /nix/store/i80k2s7lczrgr7s0dqlh1g9hhbc7gknj-clang-wrapper-16.0.6/bin/clang++ -isystem /nix/store/3ggcnykapn5ks74b4f7470n79444fwyr-libX11-1.8.7-dev/include -isystem /nix/store/a8wxmz6hcvlpb44z3kf19mzcgak5zqlm-xorgproto-2023.2/include -isystem /nix/store/686lhcz4bwg3wk09pi1xxjgzbxv7ys5q-zlib-1.3-dev/include -Wl,-rpath,/nix/store/k5j6vpppzrvk810k2w8wlc580prc5vi4-at-spi2-core-2.48.3/lib -Wl,-rpath,/nix/store/2dn1nkq2j96673y94ja1n325n1xqzwbh-cairo-1.16.0/lib -Wl,-rpath,/nix/store/01826v7q0a5l9ppbacg3lw137c43ancj-gdk-pixbuf-2.42.10/lib -Wl,-rpath,/nix/store/zrq9b75k3sv0l45n66rz72c4h03i1dbw-glib-2.76.4/lib -Wl,-rpath,/nix/store/6qp789r70ka5ahxfzrr9l3h6szclsgsr-gtk+3-3.24.38/lib -Wl,-rpath,/nix/store/xn97j65ccmd0gk6n05psdbd05x237k2l-harfbuzz-7.3.0/lib -Wl,-rpath,/nix/store/jkfhz54j6l2937ykyhslamlqrnpv3kzy-libepoxy-1.5.10/lib -Wl,-rpath,/nix/store/0xy54x90g2661i3qpjaipsjjjlvpmc85-pango-1.50.14/lib -Wl,-rpath,/nix/store/b6yspa0filw7fvfabbjfpi2z5k4igv1a-libX11-1.8.7/lib -Wl,-rpath,/nix/store/lsx6fhyyyrym3wg9xln2lhaj9lnsii5h-libdeflate-1.18/lib CMakeFiles/cmTC_b9996.dir/testCXXCompiler.cxx.o -o cmTC_b9996   && :
    /nix/store/p58l5qmzifl20qmjs3xfpl01f0mqlza2-binutils-2.40/bin/ld: cannot find Development/Repositories/Personal/spm_gui/spm_gui/outputs/out/lib: No such file or directory
    clang-16: error: linker command failed with exit code 1 (use -v to see invocation)
    ninja: build stopped: subcommand failed.

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:3 (project)

Building Linux application...

Exception: Unable to generate build files

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.8, on NixOS 24.05 (Uakari) 6.7.4-zen1, locale en_US.UTF-8)
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (not installed)
[✓] Connected device (2 available)
[✓] Network resources

! Doctor found issues in 2 categories.

might be direnv/direnv#1403

This looks similar to https://github.com/flutter/flutter/issues/65514 which may be an OS-specific issue. This seems also to appear with a simple Qt app as well, see https://discourse.nixos.org/t/nix-shell-cmake-gcc-qt-creator-not-working-together/16703.
Could you try building a simple Qt app to check this?

I checked this on Ubuntu 22.04.3 with a project path containing a space but the issue doesn't appear.

Could you try building a simple Qt app to check this?

Could you try checking this?
Also, I see you are using an old Flutter version, please try upgrading your Flutter SDK to the latest stable 3.19.1 and retry to see if there is any difference or not. Thanks!


Help deploying a web app version?

Hello. I'd love some help deploying a web app of https://github.com/XavierXiao/Dreambooth-Stable-Diffusion on an AWS cloud server with a basic command terminal to enter prompts and generate images, and a place to upload training images. Help?
Reach out if you have avails to help<EMAIL_ADDRESS>

https://github.com/x-CK-x/Dreambooth-WebUI


Fix onHostHealthUpdate got called before the cluster is resolved.

Signed-off-by: Henry Yang<EMAIL_ADDRESS>Description: onHostHealthUpdate got called before the cluster is resolved, in our particular case this cluster was never created so it's an empty cluster.  Here's the call stack:

#2  Envoy::Extensions::Clusters::Redis::RedisClusterLoadBalancerFactory::onHostHealthUpdate (this=) at external/envoy/source/extensions/clusters/redis/redis_cluster_lb.cc:80
#3  0x00000000004ae8cf in Envoy::Extensions::Clusters::Redis::RedisCluster::reloadHealthyHostsHelper (this=0x30e1510, host=...) at external/envoy/source/extensions/clusters/redis/redis_cluster.cc:119
#4  0x00000000007e9d61 in reloadHealthyHosts (this=, host=...) at external/envoy/source/common/upstream/upstream_impl.cc:919
#5  Envoy::Upstream::ClusterImplBase::finishInitialization (this=0x30e1510) at external/envoy/source/common/upstream/upstream_impl.cc:877
#6  0x00000000007e9c62 in Envoy::Upstream::ClusterImplBase::onInitDone (this=0x30e1510) at external/envoy/source/common/upstream/upstream_impl.cc:863

Risk Level: Low
Testing: Test updated to reflect this case.
Docs Changes: N/A
Release Notes: N/A

:+1:


Please invite me to the GitHub Community Organization

Name
RainbowGenie
Discord Username (if applicable)
supergalaxy119
Additional Context
I want to contribute my skills to this organization.

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


Hello Kaoto component sql-stored How to use? Any examples?

Want to configure sql stored procedure through Kaoto, found SQL-stored but will not use, would anyone use it? Can you give me an example? Thank you

Hello @longhai-he , how to use Stored Procedures with Camel is described at https://camel.apache.org/components/4.8.x/sql-stored-component.html. In Kaoto you can configure the "template" and the "templateOptions" then. "template" should point to a sql file containing your stored procedure call (alternatively you can also put it in the message body if I understand the description correctly but you then need to toggle on the "Use Message Body For Template" checkbox. The templateOptions are imo used to specify the parameters for the stored procedure call.
I put on a rough example code here but keep in mind it is not tested and you should also not put database credentials as text into your yaml file.
- route:
    id: route-2657
    from:
      id: from-2641
      uri: timer
      parameters:
        period: "1000"
        timerName: template
      steps:
        - log:
            id: log-3799
            message: ${body}
        - to:
            id: to-3293
            description: call stored procedure
            uri: sql-stored
            parameters:
              dataSource: "#postgres"
              template: classpath:sql/myStoredProcedure.sql
              template.param1: ${header.param1}
              template.param2: ${header.param2}
              template.param3: ${header.param3}
- beans:
    - name: postgres
      properties:
        databaseName: postgres
        password: postgres
        portNumber: "5432"
        serverName: localhost
        user: postgres
      type: org.postgresql.ds.PGSimpleDataSource

I am closing this issue as this is more of a discussion. Please use our discussion forums at https://github.com/orgs/KaotoIO/discussions for this purpose. Thank you.

@lhein  Thank you for your help


[Bug]: Import a large XLSX file , simplexml_load_string() Huge input lookup

Is the bug applicable and reproducable to the latest version of the package and hasn't it been reported before?

[X] Yes, it's still reproducable

What version of Laravel Excel are you using?
3.1
What version of Laravel are you using?
5.8
What version of PHP are you using?
7.2
Describe your issue
simplexml_load_string() Huge input lookup
$import = new MyImport();
\PhpOffice\PhpSpreadsheet\Settings::setLibXmlLoaderOptions(LIBXML_COMPACT | LIBXML_PARSEHUGE);
\Excel::import($import, $file);
Solve the problem as above
How can the issue be reproduced?
20M XLSX file, 50000 rows of data
What should be the expected behaviour?
simplexml_load_string() Huge input lookup
$import = new MyImport();
\PhpOffice\PhpSpreadsheet\Settings::setLibXmlLoaderOptions(LIBXML_COMPACT | LIBXML_PARSEHUGE);
\Excel::import($import, $file);
Solve the problem as above

Please report to PhpSpreadsheet, we don't handle the loading of xml.


Add quarkus feature

Changes

Add support to Quarkus without using the extension

@dearrudam it works!
So, the solution works.
It would be amazing to check how to keep the processor as an extension.


Hosting bundle link update

Update hosting bundle installer link

Hi @GuardRex, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Fixes #2190

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnetcore/hosting/aspnet-core-module.md
:white_check_mark:Succeeded

Details

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.


m) optimize new conn logic

iquic on_new_conn only be called after handshake_ok, keep same with gquic.
on_conn_closed only be called by ci_destroy

optimize commit: https://github.com/litespeedtech/lsquic/commit/850b0a3d100b2abe89cf054a9f8d13054fac34a3

@litespeedtech Please check this PR, thank you!

Agree it!

We also did something like ACL and SNI integration in on_new_conn hook, but I cannot understand why ietf quic on_new_conn should be called before handshake_ok, while gquic on_new_conn can be called delayed?

I do not know why either. :-)
But I think the ietf quic way is better for ACL, as handshake has high CPU cost, we want to drop it as early as possible, especially when under attack.

I do not know why either. :-) But I think the ietf quic way is better for ACL, as handshake has high CPU cost, we want to drop it as early as possible, especially when under attack.

"as handshake has high CPU cost"
Before entering function handshake_ok, handshake near the end, function handshake_ok only do some checking about transport parameters.

Then it does not matter much, will apply the pull request. :-)

Then it does not matter much, will apply the pull request. :-)

I have thought about it when apply the bug fix.
The situation is that we use on_new_conn hook for ACL and SNI integration, we can do something before the handshake.

I think what you are focusing is the ea_lookup_cert callback, which can be used to ACL and SNI integration before on_new_conn and it can improve performance when under attack. @litespeedtech


Fallback for invalid responses

I found case to responpse invalid for oEmbed format.
http https://www.reddit.com/oembed \
  url=='https://www.reddit.com/r/Python/comments/vdopqj/sphinxrevealjs_html_presentation_builder_for/' \
  format==json \
|jq
{
  "author_name": "attakei",
  "html": "
\nsphinx-revealjs: HTML presentation builder for Pythonista
 by\nu/attakei in\nPython\n
\n",
  "provider_name": "reddit",
  "provider_url": "https://www.reddit.com",
  "title": "sphinx-revealjs: HTML presentation builder for Pythonista",
  "type": "rich",
  "height": 316
}

Add fallback flow to use html content

Support as v0.7.0


keep Authorized/Unauthorized status at top of page, but not screen

Currently for goldchain wallets the authorization status is shown at the right top of the screen. I think this is ok, but I do not think it is ok that this floats forever. Best to keep it just at the top of the page I would say. Either that, or make it an actual banner with background, because now it becomes unreadable if shown on top of other text.

Sorry, seems like I posted too quickly. This is not an issue at all in fact as it seems that the status stays on top nicely on top of the white separator.


Re-enable deprecation warnings

Warnings were temporarily disabled in https://github.com/python/docs-community/pull/69. They should be re-enabled.

Yep, we can re-enable when this Sphinx issue is fixed: https://github.com/sphinx-doc/sphinx/issues/10440.

We could probably edit the title to reflect the conditional...

@encukou, if you edit the first message and add the link to https://github.com/sphinx-doc/sphinx/issues/10440  in a check/task list, it will be tracked automatically and once the issue is solved it will show as "1 of 1 task completed" (or something similar).


Testing whole B2B story with multiple sub organizations

Description
$Subject using IS-7.1.0-m5 federated IDP
Version
No response

Update
Observed below issues

Logout function is not properly working in suborgs
User groups in a sub org doesn't populate to APIM for login sub org user (This needs if we enable application sharing with org support)
When the users are residing on several sub orgs and trying login in to APIM, noticed some conflicts and several errors

Currently communicate with IS and figuring out these issues.


Cloud-container shouldn't handle SSH_KEY?

cloud-container adds the SSH_KEY env var into ~/.ssh/authorized_keys without a new line. In most flists, we do echo $SSH_KEY >> ~/.ssh/authorized_keys. Which causes the key ssh-rsa something to appear in the file as ssh-rsa somethingssh-rsa something which is invalid and prevents ssh access. When SSH_KEY is with the email after it, it appears as ssh-rsa something emailssh-rsa something email and it's recognized correctly.
But why does the cloud-container does this? it seems something the image should be handling. And if it does it would be convenient to add a new line so that existing images doesn't break.

@muhamadazmy please look into it

The newline was added in https://github.com/threefoldtech/cloud-container/commit/b9ac00257460af920925e4dcc22c8e82bcffe9da.


New screenshot and code width

Few aesthetic changes so the code is completely linted.
Also updated the screenshot to match the new features -- specially the post count -- and the pypi badge (it works under 3.5 without a hitch).

Looks good :+1: I appreciate that you kept the style of the screenshot as close to the original as possible. I generally don't encourage PEP8 changes but these ones look sane and don't hurt the readability.


Colour Blind Friendly SPELL page 7

Blue and Green boxes (both on right side of page when in landscape) and
Pink and Green boxes (both on left side of page when in landscape) need to be swapped, as image below:

Contents of all boxes: both characters in outer two columns (when viewed in landscape) need to swap sides, all character colours should remain the same, just positions of characters should change, as image above:

Think this largely got fixed in https://github.com/AceCentre/SpeakBook/commit/9472bf2b340c24977cd355ed896c70de18314c8d - @michaelritson - can you just check https://speakbook.netlify.app/speakbook/colour.html - and see if that's right?

Looks good to me! 👍

So just leaving this for @hosseinamin as now need a way of replicating this across all other languages

@willwade Got it


Upgrade Kured to v1.15.1

[!warning]
This is a public repository, ensure not to disclose:

[x] personal data beyond what is necessary for interacting with this pull request, nor
[x] business confidential information, such as customer names.

What kind of PR is this?
Required: Mark one of the following that is applicable:

[ ] kind/feature
[x] kind/improvement
[ ] kind/deprecation
[ ] kind/documentation
[ ] kind/clean-up
[ ] kind/bug
[ ] kind/other

Optional: Mark one or more of the following that are applicable:

[!important]
Breaking changes should be marked kind/admin-change or kind/dev-change depending on type
Critical security fixes should be marked with kind/security

[ ] kind/admin-change
[ ] kind/dev-change
[ ] kind/security
[ ] kind/adr

What does this PR do / why do we need this PR?
Upgraded kured to Chart Version 5.4.5 and App Version 1.15.1

Fixes #1900

Information to reviewers
Two security-related default-values were improved:

hostNetwork is set to false by default now.
readOnlyRootFilesystem is set to true by default now.

Also kured now wants to mount a hostPath for /var/run, for the reboot-sentinel-file, by default. Any opinions on this?
Checklist

[x] Proper commit message prefix on all commits

Change checks:

[ ] The change is transparent
[ ] The change is disruptive
[ ] The change requires no migration steps
[ ] The change requires migration steps
[ ] The change upgrades CRDs
[ ] The change updates the config and the schema

Metrics checks:

[x] The metrics are still exposed and present in Grafana after the change
[ ] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[ ] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[x] The logs do not show any errors after the change

Pod Security Policy checks:

[x] Any changed pod is covered by Pod Security Admission
[ ] Any changed pod is covered by Gatekeeper Pod Security Policies
[ ] The change does not cause any pods to be blocked by Pod Security Admission or Policies

Network Policy checks:

[ ] Any changed pod is covered by Network Policies
[ ] The change does not cause any dropped packets in the NetworkPolicy Dashboard

Audit checks:

[ ] The change does not cause any unnecessary Kubernetes audit events
[ ] The change requires changes to Kubernetes audit policy

Falco checks:

[ ] The change does not cause any alerts to be generated by Falco

Bug checks:

[ ] The bug fix is covered by regression tests

Also kured now wants to mount a hostPath for /var/run, for the reboot-sentinel-file, by default.

The conclusion here was that the default behavior changed from using nsenter to read the file (still uses nsenter to execute the reboot command as far as I understand) on the host to mounting it as a hostPath instead.
It would be interesting to look at using the signal-reboot mode as that seem to remove the need for Kured to run in privileged mode!

Tested out the signal method with the config changes from the latest commit.
Noteworthy is that I had to add the annotation container.apparmor.security.beta.kubernetes.io/kured: unconfined as mentioned here, as kured was getting:
time="2024-06-05T07:11:07Z" level=info msg="Emit reboot-signal for node: simonl-dev-mc-worker-nova-4018765f-9m8kl"
time="2024-06-05T07:11:07Z" level=fatal msg="Signal of SIGRTMIN+5 failed: permission denied"

Thoughts?

Tested out the signal method with the config changes from the latest commit. Noteworthy is that I had to add the annotation container.apparmor.security.beta.kubernetes.io/kured: unconfined as mentioned here, as kured was getting:
time="2024-06-05T07:11:07Z" level=info msg="Emit reboot-signal for node: simonl-dev-mc-worker-nova-4018765f-9m8kl"
time="2024-06-05T07:11:07Z" level=fatal msg="Signal of SIGRTMIN+5 failed: permission denied"

Thoughts?

I unfortunately can't say what implications it has to disable apparmor vs running it as privileged, but my initial hunch is that it is more secure to run the container without privileged and apparmor than it is to run it privileged.

I moved the signal method out of this branch for now, pending further investigation into the apparmor issue.


fix: Use a FQDN for metadata.google.internal

Minimize lookups by not expanding search path when resolving metadata.google.internal.
A customer found that about 85 DNS requests were made at init of packages relying on gcp-metadata. Upon investigation the issue was we were forcing path expansion for DNS resolution. After this change about 21 requests are made.

It does result in slightly odd looking requests, but I think it's the correct behavior, otherwise, the resolver will first try metadata.google.internal prepended to every domain in the search path, e.g. (from tcpdump):
22:02:20.530118 IP <IP_ADDRESS>.49662 > <IP_ADDRESS>.53: 62251+ AAAA? metadata.google.internal.default.svc.cluster.local. (68)
22:02:20.530633 IP <IP_ADDRESS>.52955 > <IP_ADDRESS>.53: 49598+ A? metadata.google.internal.svc.cluster.local. (60)
22:02:20.530657 IP <IP_ADDRESS>.52955 > <IP_ADDRESS>.53: 49950+ AAAA? metadata.google.internal.svc.cluster.local. (60)
22:02:20.530957 IP <IP_ADDRESS>.40507 > <IP_ADDRESS>.53: 45306+ A? metadata.google.internal.cluster.local. (56)
22:02:20.530984 IP <IP_ADDRESS>.40507 > <IP_ADDRESS>.53: 45635+ AAAA? metadata.google.internal.cluster.local. (56)
22:02:20.531301 IP <IP_ADDRESS>.50796 > <IP_ADDRESS>.53: 62724+ A? metadata.google.internal.c.project-name.internal. (67)
22:02:20.531330 IP <IP_ADDRESS>.50796 > <IP_ADDRESS>.53: 63148+ AAAA? metadata.google.internal.c.project-name.internal. (67)
22:02:20.535246 IP <IP_ADDRESS>.53737 > <IP_ADDRESS>.53: 1695+ A? metadata.google.internal.google.internal. (58)
22:02:20.535284 IP <IP_ADDRESS>.53737 > <IP_ADDRESS>.53: 2075+ AAAA? metadata.google.internal.google.internal. (58)

The quad-A lookups also multiply the # of total lookups. Ideally, the result could be read once and then cached in-memory, but this should be a big improvement.
In normal cases, this might not be so bad, but lots of apps on lots of pods running on GKE with lots of domains in the (default, not trivially changeable) search path results in a pretty heavy load on kube-dns / dnsmasq, which as of now, has no caching layer.

@crwilcox @ofrobots the system test did exactly what we wanted it to! 💃  I've never been so excited to see a failing test.

We did this elsewhere: https://github.com/googleapis/google-cloud-node/pull/2214, and we had to revert it because of this: https://github.com/googleapis/google-cloud-node/issues/2249.

After reviewing the above, I am forced to conclude that software is terrible. 😭
Let's just use an ip address and avoid DNS resolution altogether. I believe the python client libs already use ips.

The solution will need to achieve these attributes:

We cannot use a trailing dot because software is terrible (we get Java IllegalArgumentExceptions on Android as per https://github.com/googleapis/google-cloud-node/issues/2249.)
Regardless of if we are running on GCP or not, we should be able to discover the presence (or otherwise) of the metadata service fast (~seconds, ideally milliseconds).
Should not make repeated requests to resolve addresses. (i.e. use DNS caching).

Concurrently:

Try to resolve metadata.google.internal (no trailing dot).
Open a TCP socket to <IP_ADDRESS>. If the socket hasn't connected in N seconds, treat this as a failure to connect.  N is a small number, say 5. (Is there an SLA for the metadata service response time?)

Whichever operation finishes first, cache the result as the hostname of the metadata service and use that going forward.
When running on GCP, step 2 will finish in milliseconds. When running off GCP, step 2 will fail in N (say 5) seconds, so the operation latency will be dependent on the first check passing. This adds no additional latency to our current behavior in the off-GCP environment.
WDYT?

s there an SLA for the metadata service response time?

The metadata service has absolutely no SLA.

Actually as worded, my strategy doesn't depend upon the SLA for metadata service at all (any more than today). The concurrent direct connection to the IP is merely a fast path.
@wyardley

Resolving it without the trailing dot will result in Nx # of DNS requests for each entry in the OS’s resolver library search path before trying it as a FQDN

In GCP environments the IP connect will finish first which means that these M DNS queries will be initiated but the result will be inconsequential.

@ofrobots in GKE (where there are ~ 6 domains in the search path and the resolver makes both A and quad-A lookups), this will hit kube-dns heavily which is the whole thing this is trying to fix, so I wouldn’t say it’s consequential. Also, wouldn’t deciding too quickly that we’re not in GCP potentially could cause auth failures, no?

Also, wouldn’t deciding too quickly that we’re not in GCP potentially could cause auth failures, no?

We don't decide that we are not GCP until both IP-connect and DNS-lookups fail, so no, from auth point of view my proposal is same as current behavior.

in GKE (where there are ~ 6 domains in the search path and the resolver makes both A and quad-A lookups), this will hit kube-dns heavily which is the whole thing this is trying to fix, so I wouldn’t say it’s inconsequential.

To make sure I understand, is the 6 lookups the problem or the fact that there was no cache of the result?

My suspicion is that resolving it should work on all platforms; it’s actually using it in the Host: header that’s probably causing a problem.

I think we need more definitive information here. Any one with Java chops willing to try OkHttp (possibly on Android)?

I think I t’ll be 12-14 lookups for each request or so, because there’s both the A and quad-A lookups. @crwilcox has a POC I threw together that generates about 80 or so DNS lookups by connecting to a handful of Google services in GKE using the official libs.
Yes, once GKE rolls out better caching in front of dnsmasq / kube-dns, this should be less of a problem. I don’t think that will be rolled out til next year sometime?

I'm also proposing we cache the resolution result locally in this module – that is this library will try to resolve metadata.google.internal only once (which will generate M DNS requests based on the DNS search path). However, once resolved we would not see any more lookups regarless of whether kube-dns has caching. Would that address your concerns?

As a further optimization perhaps we can limit to A lookups only.

Yes, resolving it once should definitely be a massive improvement, and would also be cleaner than hard-coding the IP.
I’m pretty confident, though, that if you’re using that approach, it should be safe (and most correct) to use the trailing dot as well.

My suspicion is that resolving it should work on all platforms; it’s actually using it in the Host: header that’s probably causing a problem.

Staring at the Java exception a bit more, I think you're right. The error is a SSL handshake error which implies that the DNS lookup was already done. It would be good to get something more definitive though.

Some really interesting points made in this link pointed to me by Google support:
https://github.com/nodejs/node/issues/15780#issuecomment-425930255
which then links to:
https://github.com/gliderlabs/docker-alpine/blob/master/docs/caveats.md
https://pracucci.com/kubernetes-dns-resolution-ndots-options-and-why-it-may-affect-application-performances.html
These discuss some of the finer points of why this can cause such headaches with GKE, especially with Alpine base images. I'm not sure if there's been any discussion about changing options ndots:5 to 2 or 3 in the GKE nodes' /etc/resolv.conf, but my quick read is that this would tend to mitigate some of these problems (admittedly, I don't know if that would break anything else).

It looks like this actually got fixed in GCF!  @ofrobots mind taking another look?

We need another fast way to detect if an environment is GCP before attempting the TCP connection.

FWIW, Here is how go handles it:
https://github.com/googleapis/google-cloud-go/blob/master/compute/metadata/metadata.go#L119


walletconnect doesn't work for trust wallet ios app

Copy of https://github.com/trustwallet/wallet-core/issues/2197, which I could not reopen. Provided more context, including a link to a demo where you can observe the issue happening.
Describe the bug
Using the WalletConnect npm package, we are not able to our Trust Wallet app to properly connect with our dapp. A new WalletConnect connection seems to be prematurely added to the WC seciton of the app, but our site is still waiting for the user to read the ETH message and sign it
To Reproduce
You can see it here: https://wallet-kit-e2e-react-pr-299.onrender.com/
Steps:

click on "connect my wallet"

scroll down to "trust" and click on it

scan the QR code with ios app (my app version is 7.9 (650) )

app hangs with:

Expected behavior
A sign-in view, with the message, should be displayed and there's a way for the user to confirm the message. For example, we get the following with rainbow using the exact same code in the previous section:

Additional context
In case it might help, the client object from client.signPersonalMessage above is the following (after JSON.stringify):
{
  "protocol":"wc",
  "version":1,
  "_bridge":"https://f.bridge.walletconnect.org",
  "_key":{ },
  "_clientId":"68a2c54b-0245-4526-9a25-79d77d71a9d1",
  "_clientMeta":{
    "description":"",
    "url":"http://localhost:4200",
    "icons":[
      "http://localhost:4200/assets/apple-touch-icon.png",
      "http://localhost:4200/assets/favicon-32x32.png",
      "http://localhost:4200/assets/favicon-16x16.png",
      "http://localhost:4200/favicon.ico"
    ],
    "name":"Dashboard"
  },
  "_peerId":"a938900b-6831-4740-b41c-59a2af9448af",
  "_peerMeta":{
    "description":"",
    "icons":[ ],
    "name":"Trust Wallet",
    "url":"https://trustwallet.com"
  },
  "_handshakeId":1651769694012533,
  "_handshakeTopic":"a56c05ed-bc12-47bc-af06-646c1c71c423",
  "_connected":true,
  "_accounts":[ "0xADe186d1607fC324196aF0d37aFF64e975ea4378" ],
  "_chainId":1,
  "_networkId":0,
  "_rpcUrl":"",
  "_eventManager":{
    "_eventEmitters":[
      { "event":"display_uri" },
      { "event":"connect" },
      { "event":"call_request_sent" },
      { "event":"wc_sessionRequest" },
      { "event":"wc_sessionUpdate" },
      { "event":"response:1651769694012533" },
      { "event":"connect" },
      { "event":"disconnect" },
      { "event":"response:1651769706341401" }
    ]
  },
  "_cryptoLib":{ },
  "_sessionStorage":{ "storageId":"walletconnect" },
  "_signingMethods":[
    "eth_sendTransaction",
    "eth_signTransaction",
    "eth_sign",
    "eth_signTypedData",
    "eth_signTypedData_v1",
    "eth_signTypedData_v2",
    "eth_signTypedData_v3",
    "eth_signTypedData_v4",
    "personal_sign",
    "wallet_addEthereumChain",
    "wallet_switchEthereumChain",
    "wallet_getPermissions",
    "wallet_requestPermissions",
    "wallet_registerOnboarding",
    "wallet_watchAsset",
    "wallet_scanQRCode"
  ],
  "_transport":{
    "opts":{
      "protocol":"wc",
      "version":1,
      "url":"https://f.bridge.walletconnect.org",
      "subscriptions":[ "68a2c54b-0245-4526-9a25-79d77d71a9d1" ]
    },
    "_queue":[ ],
    "_events":[
      { "event":"message" },
      { "event":"open" },
      { "event":"close" },
      { "event":"error" }
    ],
    "_subscriptions":[ "68a2c54b-0245-4526-9a25-79d77d71a9d1" ],
    "_protocol":"wc",
    "_version":1,
    "_url":"https://f.bridge.walletconnect.org",
    "_netMonitor":{
      "_eventEmitters":[
        { "event":"online" }
      ]
    },
    "_socket":{ },
    "_nextSocket":null
  }
}

could you please try to update iOS app to latest version (7.9 650)? I tried and worked

This wallet kit looks good for testing, can we get a beta invite?

@hewigovens thanks for quickly taking a look at that. i made sure my app was updated, but still see the same issue on my end:
https://user-images.githubusercontent.com/85070/175329853-6ae24b38-ee83-4955-b8fd-b8f0569e28e8.MP4
Is there anything I can provide that would help? Thanks!

@paololim  On your side, do you see connection screen or sign request does not work?

@paololim On your side, do you see connection screen or sign request does not work?

Just seems to hang on "connecting may take a few seconds" on my ios app. I deleted it and re-installed too, just in case if it makes a difference (it did not).

Update: I actually waited for it and it says "Failed to connect" after about 1 full minute.

Looks like network issue, try to remove all sessions and kill the app then try again

@paololim try to do it with VPN, please

Looks like network issue, try to remove all sessions and kill the app then try again

By removing all sessions, what exactly do you mean?

Remove all WalletConnect sessions on your app, Settings -> WalletConnect -> Remove each

Remove all WalletConnect sessions on your app, Settings -> WalletConnect -> Remove each

@hewigovens This occurs even when I delete the app. The video I posted above has no existing WC sessions

@paololim Does this still happen on latest iOS app? we couldn't reproduce it

@paololim Does this still happen on latest iOS app? we couldn't reproduce it

@hewigovens hello! Sorry, I just saw this now. We are still seeing this issue when testing internally - though some people have said it is working for them, while others have said it is not. We would really like to work with you on this to make sure the experience is great for both Trust's and Dynamic's users. Someone from our team is compiling videos/screenshots to recreate the issue and will post them here shortly.

This wallet kit looks good for testing, can we get a beta invite?

When you have a chance, please email<EMAIL_ADDRESS>directly and we will get your team set up with beta access directly to our SDK - again, we would love to work with your team to make this a great experience for both!

to add to paolo's comment - this what happened yesterday for me:

I was able to connect and sign with Opensea - then i signed out
I was able to connect but not sign with dynamic - then i signed out
Tried uniswap and it failed
Went back to Opensea and could no longer connect
Went back to Dynamic and could no longer connect
I cleared the cache in Trust app and in my Browser
I went back to Opensea, could not connect
I went back to Dynamic, could not connect

to add to paolo's comment - this what happened yesterday for me:

I was able to connect and sign with Opensea - then i signed out
I was able to connect but not sign with dynamic - then i signed out
Tried uniswap and it failed
Went back to Opensea and could no longer connect
Went back to Dynamic and could no longer connect
I cleared the cache in Trust app and in my Browser
I went back to Opensea, could not connect
I went back to Dynamic, could not connect

@eadonj this looks weird, usually you disconnect from the dapp + remove the session from the app, you should be able to connect (at most kill and relaunch the app), do you run your own bridge or somehow?

i don't use a bridge or anything like that. it was just standard connecting and disconnecting from the different sites.

should we setup a call? we can screenshare a show you what's happening

Can confirm this is a trustwallet issue. Random and hard to track as well

Hey there! Having the same issue when deep linking from a flutter app on IOS...

I also have this problem when trying to use trust walletconnect feature. It just hangs on that "wait a few seconds" message. I have the latest app version on IOS. Sometimes after trying to get a connection multiple times I get connected by chance after many attempts but this is really annoying. It has been like this for ever. It is not a matter of now or a month ago.


New customizations

Hello everyone!
My team is currently developing a new app over here and we would like to use your barcode reader. Unfortunately it is not a perfect fit. We need a few more options, among them:

Customization of the Cancel button (color, positioning, font and size)
Customization of the reader area: optimizing for barcode reading when horizontally oriented and QR code when vertically oriented
Addition of a new “dynamic” button: if a barcode line is present on copy/paste, this button shows-up, otherwise it stays hidden.

We would like to have those as native features, and not as a stand apart fork.
Given that we develop these new features, and proper documentation and tests be generated, would you be willing to accept this on the main repo?

@Leonardo-Ferreira did you find another solution for this problem? I'm struggling with the same situation and I don't want to use the google ML kit.


Ressource path not changed for content parameter in meta tags

Describe the bug
When using meta html tags for website preview such as <meta property="og:image" content="/path/to/icon.png">, the icon's path isn't correctly changed inside the html tag when the website is built.
Like the href field in html tags such as <link rel="icon" href="/path/to/icon.png" />, the content field of those meta tags should be changed as well.

Reproduction
Create a simple website with  tags for website preview
System Info
System:
    OS: Windows 10 10.0.22000
    CPU: (12) x64 Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
    Memory: 3.81 GB / 15.77 GB
Binaries:
    Node: 16.14.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.3.1 - C:\Program Files\nodejs\npm.CMD
Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (101.0.1210.47)
    Internet Explorer: 11.0.22000.120
npmPackages:
    @vitejs/plugin-vue: ^2.0.0 => 2.2.4
    vite: ^2.7.2 => 2.8.6

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of https://github.com/vitejs/vite/issues/7362, there is an active PR at https://github.com/vitejs/vite/pull/7643


Fixed, improved, and optimized stencil shader.

Fixed, refactored, and heavily commented the stencil shader.  The main purpose of doing this is to support slider controls to dynamically change the grid size in a future PR.  This is being submitted as a standalone PR because 1) it improves the code and makes it slightly faster and 2) it will be easier to review and understand.

Included in #161


[WIP] add gdpr form to artist page cta

This adds the gdpr-form to the artist page cta. I think we'd want to hold off until we close the artist page test before adding this but I wanted to get some eyes on in the event I missed something.

LGTM! Feel free to extract some of those methods out to share with the base AuthModal GPDR stuff.


[Feature] Remove Whitespace In Metrics Query

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Right now, there is a lot of whitespace produced in the metrics query. This is not helpful to those who want to see the compiled sql. Figure out a way to reduce the whitespace introduced by comments!
Describe alternatives you've considered
Not doing this?
Who will this benefit?
Anyone who wants to read the query and doesn't want to scroll down 60 empty lines.
Are you interested in contributing this feature?
You break it, you buy it. So.... yea.
Anything else?
No response

allow me to take a spin here my friend

All you @dave-connors-3! I recommend basing your changes on https://github.com/dbt-labs/dbt_metrics/pull/90 as that makes some pretty big changes to structure that will get merged in this week most likely


Add stub building guide that describes use of Dockerfile

Commit message says it all.

I don't have a contributing guide to document this but my build system requires that all commits be gpg signed by a pinned set of keys (currently the set of keys is exactly my key). I found some way of ammending your commit with my signature and that seems to be ok and will keep the build system happy. But it means that github didn't figure out that this was merged. I'm going to simply close the pull request and you can see in the commits section that your commit was pushed into the tree.
Thanks a bunch for figuring out how to use the docker file and providing this documentation.
Let me know if you have any questions about running ssh-cert-authority or feature requests or whatever else.

Yup that's fine, thanks for notifying. We're looking at using this in our deployment workflow so I'll probably be sending docs and requests as I figure things out. If you think of an alternative way to do pull requests let me know and I'll adjust.


GUVNOR-2557: Add keyboard navigation through the table

See https://issues.jboss.org/browse/GUVNOR-2557

Related to https://github.com/droolsjbpm/drools-wb/pull/243

@ederign Changes complete. Please take a second look.

thanks @manstis!!!! +1


allowing configuring shutdown hook

The shutdown hook is hooked into the JVM and is executed when the JVM stops.
This can be beneficial when the application is not managing pi4j's life cycle, but when an application wants to properly manage the life cycle, then this can be detrimental to the shutdown sequence of the application.
This change allows such an application to disable this feature.
The shutdown hook is still enabled by default, a future version might decide to disable it by default.

@FDelporte should we disable it by default, or leave it enabled as is currently the case?

Not sure, I started doubting myself based on your remark in the ticket...

Well in the end, it is quite simple for those who care. Change the following line:
Context pi4j = Pi4J.newAutoContext();

to
Context pi4j = Pi4J.newContextBuilder().autoDetect().disableShutdownHook().build();

or if it was default disabled, they can enable it again:
Context pi4j = Pi4J.newContextBuilder().autoDetect().enableShutdownHook().build();

I personally would prefer having it opt-in / disabled by default.

That calls for some extra documentation ;-)

I'm a little confused 😄
Which option most closely resembles the existing behaviour?
It sounds like this decision could introduce a breaking change for existing users, but maybe I'm misreading this thread

I'm a little confused 😄
Which option most closely resembles the existing behaviour?
It sounds like this decision could introduce a breaking change for existing users, but maybe I'm misreading this thread

Yes, this is a breaking change. Existing behaviour can be reactivated with:
Context pi4j = Pi4J.newContextBuilder().autoDetect().enableShutdownHook().build();

We will communicate this in the release notes, when we release.


Support Environment Variables

It would be nice to be able to set environment variables for the spawned process.

+1

+1

+1

Here a Pull Request for this feature

FIxed. @xsellier the pr i still open and we can discuss improvements there.


[fix] TypeError: Object.defineProperties called on non-object (issue #166)

The original code doesn't really fix the issue #166 .

Howdy All,
I'm not sure what's up with this PR?
Basically, if I install Winston for logging, and able the default CouchDB logger via winston-couchdb, which uses this library, I immediately get the error "TypeError: Object.defineProperties called on non-object" that is fixed by this PR.
Any chance of getting the basic fix for this issue merged?
Phillip.

Hi Phillip,
Maybe you can help to write some tests for this PR?

Maybe you can help to write some tests for this PR?

Looking at the diff, I don't see anything that would require a test (or a new test): from what I can see, you've simply re-factored the code to actually work with the response that's received. What would the test be trying to assert?


Feat/update repo 2

sync up with Microsoft repo

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


Return value of incorrect type from a method is not reported

Input
→ View on sorbet.run
# typed: true
extend T::Sig

sig { returns(String) }
def foo
  if cond
    10 # <<< Value of incorrect type
  else
    T.unsafe(nil)
  end
end

sig { returns(T::Boolean) }
def cond
  true
end

Observed output
No errors! Great job.

Expected behavior

Expected String but found Integer(10) for method result type https://srb.help/7005

It works correctly with explicit return :  → View on sorbet.run
It works correctly if else branch returns a typed value (not T.untyped):  → View on sorbet.run
It works correctly if else branch is omitted: → View on sorbet.run

Duplicate of #4129


Flutter won't detect device

Environment

OS: Windows 11
scrcpy version: 1.24
installation method: Windows release
device model: Oneplus 5T
Android version: 10

Problem
When I run adb devices, I can see my device. However, when I run flutter devices, the device doesn't appear. My phone is connected by usb. Scrcpy works as expected. I just don't know why the phone doesn't appear when running fluttere devices."

I don't know. You should ask/report to flutter :)
https://stackoverflow.com/questions/49045393/message-flutter-run-no-connected-devices

I don't know. You should ask/report to flutter :)
https://stackoverflow.com/questions/49045393/message-flutter-run-no-connected-devices

Sorry I thought it might be a problem with scrcpy.