Convert filters and search text into query params

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Fixes #3672
Type of change
Please delete any options that are not relevant.

User interface (UI)
New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas (including JSDoc for methods)
[x] My changes generates no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

The current behaviour with incorrect query params is:
Single invalid status like http://localhost:3000/dashboard?status=40

Multiple invalid status like http://localhost:3000/dashboard?status=40&status=23

Single invalid "active" filter like http://localhost:3000/dashboard?active=null applies filter for Paused

Multiple invalid "active" filters will apply filter for Paused but shows "Active" in selected filter instead of "Paused"

The behaviour is same for any number of invalid "tag" filters

=> could you change the links to the parts of the UI that deal with editing,cloning, adding monitors, .. which keep you on the same page visually?

Done. But I really don't like how this is implemented. Let me know if you have any suggestions.

Is it possible to retrigger the tests? It seems unrelated to this PR.

@CommanderStorm I have addressed all requested changes.

@CommanderStorm Would you add hacktoberfest-accepted label to this PR?

Done. But I really don't like how this is implemented. Let me know if you have any suggestions.

Using router hooks, it can be easilly managed! Here is a link to a Stackoverflow answer about this: https://stackoverflow.com/a/47195471/7826809. Don't hesitate if you need help!

Using router hooks, it can be easilly managed!

That is the current approach. I meant I couldn't figure out without using global router hook. Had it worked, using in-component guards would be clearer since all updates would be handled by same component. With current approach, there are two places that update the same filter query params.

I'm facing an issue: I can't select more than one filter. Here is vieo recordings:
Prod version (working):
https://github.com/user-attachments/assets/78bad66a-4ecc-437c-91f0-7067084c109c
This PR (not working) (I'm clicking on the different items I hover), started using docker run --rm -it -p 3000:3000 -p 3001:3001 --pull always -e 'UPTIME_KUMA_GH_REPO=Suven-p:feat-3672-use_query_params_for_filter' louislam/uptime-kuma:pr-test2:
https://github.com/user-attachments/assets/85f93335-6090-4591-93b9-41ed3715b60a

The latest commit should have fixed that.

Confirmed, it's working as the prod version!

Found another issue. Modifying filter parameters from add monitor page with partially formed fields causes the form to be rest.


[Bug] No Luscious extension settings

I acknowledge that:

I have updated to the latest version of the app (stable is v8.7.0)
I have updated all extensions
If this is an issue with an extension, that I should be opening an issue in https://github.com/inorichi/tachiyomi-extensions

Device information

Tachiyomi version: 8.7.4-az
Android version: 13
Device: Samsung s22 ultra

Steps to reproduce

Install Luscious
Go to extensions
Go to details of Luscious

Expected behavior
There should be an option to merge chapter
Actual behavior
No settings can be found
Other details
On the main tachiyomi app this is fixable and this issue is related to https://github.com/tachiyomiorg/tachiyomi-extensions/issues/9422#issuecomment-944928050

i think i accidentally fixed this at some point, closing


cigien: Watch sumitron.com

cigien requests the watch of the watch_keyword sumitron\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sumitron\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

sumitron\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ

Approved by Spevacus in Charcoal HQ


Adding multicursor support for inline completions

with @hediet

please use @ a bit more cautious.
i have nothing to do with this stuff.
On Thu 18. Jan 2024 at 16:17, Aiday Marlen Kyzy @.***>
wrote:

cc @henning https://github.com/henning Dieterichs
You can view, comment on, or merge this pull request online at:
https://github.com/microsoft/vscode/pull/202739
Commit Summary

be9e2fe
https://github.com/microsoft/vscode/pull/202739/commits/be9e2feb1422b270c3e2ee072d51253826dc2915
wip

File Changes
(3 files https://github.com/microsoft/vscode/pull/202739/files)

M src/vs/editor/contrib/inlineCompletions/browser/ghostText.ts
https://github.com/microsoft/vscode/pull/202739/files#diff-3c352d6f1b7dcf08cce153a7bca6ffd474aa27a83039aa8e41a950920d2f183a
(18)
M
src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsModel.ts
https://github.com/microsoft/vscode/pull/202739/files#diff-50ace5ee66f51a08197b7cf6320399eeb82730ef146cfa885b9d7c90c0cc56aa
(7)
M
src/vs/editor/contrib/inlineCompletions/test/browser/inlineCompletionsProvider.test.ts
https://github.com/microsoft/vscode/pull/202739/files#diff-5f3b947fa27f8b6d4422b63ef38aec90857d715507423f62149be1da85bb5e3a
(66)

Patch Links:

https://github.com/microsoft/vscode/pull/202739.patch
https://github.com/microsoft/vscode/pull/202739.diff

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/pull/202739, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAJ2UGUPFEEKAMJANPRAVDYPE4HLAVCNFSM6AAAAABCAOPSA6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DQNJQGY3DMMY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ah yes apologies, a member of my team has the same name, I made a mistake


mj-style removes !important

Describe the bug
As stated in the title, if you use !important for a specific value, it isn't added to the generated HTML.
To Reproduce
Steps to reproduce the behavior:
Inspect any text on this generated file https://mjml.io/try-it-live/components/head-style
Expected behavior
!important should be added to the inline styles.
MJML environment (please complete the following information):

OS: MacOS (Chrome)
MJML Version 4.3.0
MJML tool used MJML Live editor and MJML Node locally

Email sending environment(for rendering issues):

Tested on Emailonacid. Issue presents itself on the web versions of Outlook / Office 365.

Affected email clients (for rendering issues):

Same as above.

Hi,
cc @kmcb777 we should just add an option to extend juice config as we do on minify.


lyxal: Watch supremeperlite.com

lyxal requests the watch of the watch_keyword supremeperlite\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by double-beep in Charcoal HQ

supremeperlite\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


How to access from shell scripts

I'm migrating from react-native-config and trying to access some of the environment variables from a shell script after initiating the rnuc utility. I have this working like this which is an adjustment from how I had it working from react-native-config:
# Get current version from production env file
yarn rnuc .env.prod
set -o allexport; source ios/rnuc.xcconfig; set +o allexport
echo "Current version: ${APP_VERSION}"

But is there a better/simpler recommended way? I cant see it in the docs.
What does the rnuc utility write to the android project by the way? Whatever it is seems to disappear after running yarn so I've put a postinstall script in my package.json to run yarn rnuc .env - again if that's the best way perhaps it could be added to the readme.

Can you give an example of how it used to work with react-native-config?

What does the rnuc utility write to the android project by the way? Whatever it is seems to disappear after running yarn so I've put a postinstall script in my package.json to run yarn rnuc .env - again if that's the best way perhaps it could be added to the readme.

can you please elaborate on this a bit? can you suggest exact copy which could help to avoid this confusion? Feel free to submit pr too =)

Can you give an example of how it used to work with react-native-config?

react-native-config relied in setting an ENVFILE environment variable to point to whatever .env file should be the source of all congig variables, so to expose the variables from that file in a script what I had was:
set -o allexport; source $ENVFILE; set +o allexport
I don't recall where I got this from however, it's not in their README.

Ok. While I do not natively support this it is still possible to simply source your env file as long as this file is a dotenv one.
If you're using YAML then it's going to be tricky but using xcconfig maybe an acceptable workaround.

If your just trying to add secrets from yoour bash profile you could try this:
module.exports = {
  js_override: true,
  on_env: async function (env) {
    // append process.env
    return {
      ...env,
      API_KEY: process.env._API_KEY
    }
  }
};


MatTable should not assume that an Http client will be registered in the DI container

The MatTable component has an @inject statement requiring an HttpClient to be registered in the DI container.  In my instance, I am querying a DB context directly for items and so I don't need an HttpClient to retrieve any data.  Also, for anyone loading static data they would also not need an HttpClient.

@StevenRasmussen  I think I understand the issue you are addressing but you might want to clarify it a bit for me. Is it that your application doesn't use the HttpClient at all AND when you want to use MatTable it errors out if you do not include @using HttpClient? If that is the case please provide your code or examples.
I ran two tests using default Blazor applications.  From VS, install Web Applications "Blazor (ASP.NET Core Hosted)" and "Blazor (client-side)"  Then used the first basic example with the static data class cars and in both cases I did not get any errors or warnings for missing "HttpClient" dependency. So my application did not have to include "@using HttpClient"
If your issue is that the MatBlazor is bloated because it requires HttpClient then I would have to defer that to @SamProf.
Either way I will need more information on your specific situation to address it, Thanks.

@enkodellc - thanks for the reply!  Your first assumption was correct.  Currently I don't use an HttpClient at all.  I may need one down the future but for now I don't.  I just started this project and used the "Blazor (Server Side)" template found in Asp.Net Core 3.0 Preview 4.  To resolve the error I simply added an HttpClient to the DI container in the startup method like this:
services.AddScoped<HttpClient>();

@enkodellc I dont really check MatTable code and API.
I just think that MatTable should work with abstract Storage, that can be only client-side for example or server side or custom.... it;s just my opinion. I will check your MatTable later.

He is correct that if you have a "Blazer (server-side)" project it needs. services.AddScoped<HttpClient>();
Personally I think that most people will be doing client-side. Server side will take a fairly large toll on the server for an application with significant traffic.  It is a minor issue that should be noted in the MatTable page. Just my opinion. Of course if this is not satisfactory anyone can fork it and use the code on their own.
The API portion is tested fine but I have not tested the API paging, sorting and filtering portion.

Related of #166


Eyes do not track

Release: 1.2.0
VRC Version: Build 1033
DLLs loaded in MelonLoader?: Yes
I've got an Avatar 3.0 package with existing Parameter fields. To test, I've added LeftEyeX, LeftEyeY, RightEyeX, RightEyeY, and LeftEyeLid as Float. EyeLook has been disabled in the descriptor, and Additive shows "None (Runtime Animator Controller)".
Based on the provided documentation, it's unclear if anything else needs to be added to the avatar package. There's mention of simple blendstates, but it's unclear if the parameter items are all that needs to be added or not based on the docs.

Have you added animations for the parameters inside your fx controller? The mod only automates setting the parameters with information from the eye tracking sensors but you still need to define the animation layers and blend trees to move the eyes from the parameters. For example, my eye look blend tree looks like this:

Have you added animations for the parameters inside your fx controller? The mod only automates setting the parameters with information from the eye tracking sensors but you still need to define the animation layers and blend trees to move the eyes from the parameters. For example, my eye look blend tree looks like this:

This explains much. Thanks!

This explains much. Thanks!

Was this able to get it working?  VRCMods has the plugin listed as "Not working"

Was this able to get it working?  VRCMods has the plugin listed as "Not working"

Can confirm it works, released a build a couple hours ago that gets it working with the new update but it's not fully complete yet. Still need to figure out network priotization but other than that, should work fine!     https://github.com/benaclejames/VRCEyeTracking/releases/tag/v1.2.1

Can confirm it works, released a build a couple hours ago that gets it working with the new update but it's not fully complete yet. Still need to figure out network priotization but other than that, should work fine!     https://github.com/benaclejames/VRCEyeTracking/releases/tag/v1.2.1

Awesome. I'm also stuck on the blend trees, mostly because I'm not sure what to do with what. Would it be possible to add your .anim(ations) for the blinking and moving, as well as the FX controller? This would help a lot, even if I'd have to adjust them.

Awesome. I'm also stuck on the blend trees, mostly because I'm not sure what to do with what. Would it be possible to add your .anim(ations) for the blinking and moving, as well as the FX controller? This would help a lot, even if I'd have to adjust them.

Awesome. I'm also stuck on the blend trees, mostly because I'm not sure what to do with what. Would it be possible to add your .anim(ations) for the blinking and moving, as well as the FX controller? This would help a lot, even if I'd have to adjust them.

Aye, same here. Examples would be great for reference - Unity keeps throwing an error saying that rotational keyframes aren't supported on humanoid rigs. Even if there's extensive edits involved, it would be good to see how it works in at least one case.

Awesome. I'm also stuck on the blend trees, mostly because I'm not sure what to do with what. Would it be possible to add your .anim(ations) for the blinking and moving, as well as the FX controller? This would help a lot, even if I'd have to adjust them.

Aye, same here. Examples would be great for reference - Unity keeps throwing an error saying that rotational keyframes aren't supported on humanoid rigs. Even if there's extensive edits involved, it would be good to see how it works in at least one case.

Been thinking about making an example avatar for a while now but I'm still struggling to find a good model to use. Once I do find that, though, I'll include all the anim files and layers required. @AmaliaKalio To fix the issue you're having, remove the eyes from the avatar definition on the model

Been thinking about making an example avatar for a while now but I'm still struggling to find a good model to use. Once I do find that, though, I'll include all the anim files and layers required. @AmaliaKalio To fix the issue you're having, remove the eyes from the avatar definition on the model

Here's my avatar setup

Add the parameters you want to use to your VRCExpressionParameters Object.
In your FX Object, add the parameters and then make layers for the animation. I independently control the eyes and have left eye blinking control both eyes. So I created the layers "Left Eye", "Right Eye", "Left Eye Blink". I've also added "Eyes Dilation" and "Eyes Widen". Set the weights to 1

For blinking:

Create a blend tree as the default state in the layer. Inside the blend tree set the parameter to LeftEyeLid and add tow Motions, "anim_blink_both" and "anim_unblink_both". (Or similar.) The animations should blink and un-blink the eye(s).

Have anim_blink be set closer to 0 and anim_unblink be set closer to 1. I used threshold values of 0.4 and 0.9 respectively, which helped fully control the eyelid.

For eye movement

Create a blend tree in the left/right eye layer and set it to be 2D Freeform Directional. Set the two parameters to LeftEyeX and LeftEyeY (or RightEyeX/Y).
Add five animations with the following PosX and PosY values:

anim_left_eye_left  -0.7, 0, 1
anim_left_eye_right 0.7, 0, 1
anim_left_eye_up 0, 0.7, 1
anim_left_eye_down 0, -0.7, 1
anim_left_eye_default 0, 0, 1
Those animations should control the eye blendshape(s) to move the eye(s) in those directions. See the attached screenshots

Here's my avatar setup

Add the parameters you want to use to your VRCExpressionParameters Object.
In your FX Object, add the parameters and then make layers for the animation. I independently control the eyes and have left eye blinking control both eyes. So I created the layers "Left Eye", "Right Eye", "Left Eye Blink". I've also added "Eyes Dilation" and "Eyes Widen". Set the weights to 1

For blinking:

Create a blend tree as the default state in the layer. Inside the blend tree set the parameter to LeftEyeLid and add tow Motions, "anim_blink_both" and "anim_unblink_both". (Or similar.) The animations should blink and un-blink the eye(s).

Have anim_blink be set closer to 0 and anim_unblink be set closer to 1. I used threshold values of 0.4 and 0.9 respectively, which helped fully control the eyelid.

For eye movement

Create a blend tree in the left/right eye layer and set it to be 2D Freeform Directional. Set the two parameters to LeftEyeX and LeftEyeY (or RightEyeX/Y).
Add five animations with the following PosX and PosY values:

anim_left_eye_left  -0.7, 0, 1
anim_left_eye_right 0.7, 0, 1
anim_left_eye_up 0, 0.7, 1
anim_left_eye_down 0, -0.7, 1
anim_left_eye_default 0, 0, 1
Those animations should control the eye blendshape(s) to move the eye(s) in those directions. See the attached screenshots

@benaclejames I typically have been keeping "Eye Look" disabled, but went ahead and briefly re-enabled it to remove the eye references from the bone slots before disabling again. No dice, unfortunately.

Curiously enough, I can edit the .anim file via notepad and force the values in that way, but if I ever scrub through the time in the animation file within Unity, it reverts the rotation.x value back to 0. ¯_( ͡° ͜ʖ ͡°)_/¯ I guess I'll try doing this for the other directionals on one eye and see what happens.

@benaclejames I typically have been keeping "Eye Look" disabled, but went ahead and briefly re-enabled it to remove the eye references from the bone slots before disabling again. No dice, unfortunately.

Curiously enough, I can edit the .anim file via notepad and force the values in that way, but if I ever scrub through the time in the animation file within Unity, it reverts the rotation.x value back to 0. ¯_( ͡° ͜ʖ ͡°)_/¯ I guess I'll try doing this for the other directionals on one eye and see what happens.

Thanks for the better explaination. Would it still be possible to get those .anim files uploaded? That would help a lot.

Thanks for the better explaination. Would it still be possible to get those .anim files uploaded? That would help a lot.

The anim files have to be customized for each avatar, and you might have to add blendshapes to your avatar for the eye deflection. Here's an example video on how to create animation files: https://www.youtube.com/watch?v=iYDDVbx5Wpk
You'll have to either create them yourself for your avatar (and assign them in Blend Trees) or commission someone to do it for you. I wrote a more complete guide on setting it up here which may help: https://github.com/InconsolableCellist/VRCEyeTracking/blob/docs_improvement/EXAMPLE.md
If you're looking for someone to set it up for you, you could view my profile and contact me. But without seeing your avatar setup it's not possible to tell you exactly what settings to change, if the above info and guides don't help. I'd like to make a video at some point on how to do it

The anim files have to be customized for each avatar, and you might have to add blendshapes to your avatar for the eye deflection. Here's an example video on how to create animation files: https://www.youtube.com/watch?v=iYDDVbx5Wpk
You'll have to either create them yourself for your avatar (and assign them in Blend Trees) or commission someone to do it for you. I wrote a more complete guide on setting it up here which may help: https://github.com/InconsolableCellist/VRCEyeTracking/blob/docs_improvement/EXAMPLE.md
If you're looking for someone to set it up for you, you could view my profile and contact me. But without seeing your avatar setup it's not possible to tell you exactly what settings to change, if the above info and guides don't help. I'd like to make a video at some point on how to do it

I've had no problem getting blinking to work, but when I go to set up eye movement I'm trying to use a rotation transform to move the eyes and it doesn't seem to be working...debugger shows that they're tracking.
Is the best way to create blendshapes then animate like I did the blinking?
Thanks!

I've had no problem getting blinking to work, but when I go to set up eye movement I'm trying to use a rotation transform to move the eyes and it doesn't seem to be working...debugger shows that they're tracking.
Is the best way to create blendshapes then animate like I did the blinking?
Thanks!

I can confirm that even with blendshapes, it doesn't seem to trigger anything. This also includes blinking, in my case. I have tried where the animation contains two keyframes of different values (IE: Center to position in the case of look animations, or Open to closed/visa-versa for blinking), or where there are two keyframes of the target value, as well as just using a single keyframe. I can confirm in the Animation editor that these should be setting the proper items and values.
The closest I've seen was when the avatar didn't load in right and I couldn't even use standard toggles. At this point, the left eye was stuck looking up. Reloading the avatar resolved both of these, but there's still no tracking. At the very least, it tells me that the parameters are plugged in for the animations.

I can confirm that even with blendshapes, it doesn't seem to trigger anything. This also includes blinking, in my case. I have tried where the animation contains two keyframes of different values (IE: Center to position in the case of look animations, or Open to closed/visa-versa for blinking), or where there are two keyframes of the target value, as well as just using a single keyframe. I can confirm in the Animation editor that these should be setting the proper items and values.
The closest I've seen was when the avatar didn't load in right and I couldn't even use standard toggles. At this point, the left eye was stuck looking up. Reloading the avatar resolved both of these, but there's still no tracking. At the very least, it tells me that the parameters are plugged in for the animations.

@AmaliaKalio Do you see the param values changing in the debug menu?
@ehefk You may need to create an avatar mask and remove the eye bones from your avatar definition for the bones to rotate

@AmaliaKalio Do you see the param values changing in the debug menu?
@ehefk You may need to create an avatar mask and remove the eye bones from your avatar definition for the bones to rotate

@benaclejames I do, yes.

@benaclejames I do, yes.

Thanks Ben!
I watched a quick video on how to make an avatar mask, selected just the eyes and removed the bones from the avatar's config screen and now have tracking eyes!

Thanks Ben!
I watched a quick video on how to make an avatar mask, selected just the eyes and removed the bones from the avatar's config screen and now have tracking eyes!

After setting up everything, the blinking animations as well as the eyes movement work. Unfortunately the eyes barely move, even with adjusted thresholds... Am I missing anything? They do move, but just about nothing, even though the maximum of the animation has been placed further.

After setting up everything, the blinking animations as well as the eyes movement work. Unfortunately the eyes barely move, even with adjusted thresholds... Am I missing anything? They do move, but just about nothing, even though the maximum of the animation has been placed further.

@benaclejames Found the issue (ish). The short of it is that it was on my end. Animations that I had created in Unity checked out, but there was something built into the avatar preventing them from triggering properly (likely some sort of animation conflict, though I still haven't been able to track down where/what). Got that sorted by going about it a different way, and can confirm it works as-intended!

@benaclejames Found the issue (ish). The short of it is that it was on my end. Animations that I had created in Unity checked out, but there was something built into the avatar preventing them from triggering properly (likely some sort of animation conflict, though I still haven't been able to track down where/what). Got that sorted by going about it a different way, and can confirm it works as-intended!

Doesn't work parameter change in VRChat build 1040

@Kieaer Are you sure everything is setup correctly on your end? Nobody else has reported any issues and I can't see any issues locally. Are you on the newest release and not in any VRC betas?

Closing due to inactivity

Solved in last release. thanks!


sendMediaGroup fails when sending multiple videos without audio track

POST request to https://api.telegram.org/bot<token>/sendMediaGroup:
{
	"chat_id": "<id>",
	"media": "[{\"type\":\"video\",\"media\":\"https://softether.download/test/clock_1280x720_120fps.mp4\"},{\"type\":\"video\",\"media\":\"https://softether.download/test/clock_1280x720_120fps.mp4\"}]"
}

Response:
{
	"ok": false,
	"error_code": 400,
	"description": "Bad Request: failed to send message #1 with the error message \"Wrong file identifier/HTTP URL specified\""
}

No problems at all when sending a single video (which shows up as a GIF in clients). On that note, according to the method documentation:

A JSON-serialized array describing messages to be sent, must include 2-10 items

Which doesn't seem to be the case as a single item is happily accepted.
The issue may be related to the animation type not being supported for media groups:
{
	"ok": false,
	"error_code": 400,
	"description": "Bad Request: type \"animation\" can't be used in sendMediaGroup"
}

Ideally videos should always be treated as such though, the inconsistency is especially evident when sending WEBP files: if without an audio track they're treated as a GIF, otherwise as a document.

See https://github.com/tdlib/telegram-bot-api/issues/301.

Sorry, missed that.
I agree with @Poolitzer: the root issue is the Telegram backend treating videos without an audio track as animation, even when not explicitly asked to do so (sendAnimation).

One URL = one file. There is no way the file can be treated differently, no matter how you try to send it. You have an issue, because you are trying to use a URL to an animation instead of a video URL.

It's the distinction between animation and video based on the presence of an audio track that doesn't make sense, in my opinion.
I understand the usefulness behind the animation type, but I believe the choice whether to use it (instead of video) should be up to the API user.

There is a URL. There are no users involved.

I will try to explain it since I was mentioned anyway. Telegram does not care which API method sends it an URL. It queries/downloads the URL, decides which type it is (in this context important: Animation or Video), and then passes this on to the API. So the download happens independent from the invocation method.
Media groups can not consist of animations, so if an URL links to an animation, you can not put it into a media group, or rather, the API will return an error that an invalid file type was put in the media group.
There is no way of letting the "API user" decide the file type without making huge changes to the back end, first allowing the telegram API to override the file type detection of the URL, then also implementing this into the bot API. While it would be convenient for us, it would be a lot of work for telegram, so I can very much understand them not implementing it.

Thank you for the detailed explanation, I understand the reasoning behind the behavior now.
At this point the question would be: does it actually make sense to consider MP4 videos without an audio track animation rather than video?

At this point the question would be: does it actually make sense to consider MP4 videos without an audio track animation rather than video?

Otherwise, you wouldn't be able to upload MP4 animations via URLs at all.

Correct, is there a need for that though? I mean, true "animations" are usually in WEBP or GIF format.

GIF is a highly ineffective format with bad image quality. Most modern animations are in video formats like MP4 or WEBP.

Right. Considering that WEBP is quite common (and still spreading), I think we should default to video for MP4.
This would be a feature request at this point, as we agreed the behavior I encountered is not a bug.


Auto fill pass

Fix #65
Simple solution is not using <form> tag in submit form. I've replaced all <form> with <div> tag, and browsers do not remember password / auto complete anymore.

Yeah, this makes sense. Well done 👍  🍺

I will still merge this PR because it fixes the problem on other browsers. I can still see Safari Mac recommend password on passphrase field.


[DOCS] Consistency in function and method docstrings

Issue Description
Function and method parameters are documented using either Parameters, Arguments, or Args keywords. We should only use Arguments everywhere.
Code of Conduct

[x] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

@jrobinAV can i take up the issue ?

@jrobinAV would like to work on this , could you assign this to me?

@Tushar98644 @wizaye I have assigned this issue to both of you! Thank you for your contribution!

@FlorianJacta I've made a PR with the necessary changes. please check it ;)

Is this issue still open

@Harish-2003 its open but we're working on it kindly check other issues for time being

@Harish-2003 its open but we're working on it kindly check other issues for time being

Sure

@trgiangdo i had a issue while updating the previous pr, i've opened a new one with the requested changes, please look into it

I would like to work on this.

@wizaye is already working on it and created a PR on this issue. But I can also assign you if you want to collaborate.

How do I take up an issue?

@margaret-hu  someone from the team assigns it to you and then you work on it and make a valid PR

Can I assist with this issue as well?

@margaret-hu Thank you for your help. You are assigned as well.

How do I directly contribute to an issue?

i do like to take on this issue please assign it to me!

I would like to work on this project. Please assign it to me .

@Sushil2k4 and @margaret-hu, look up the PR created for this issue. You can either collaborate on it or create your own PR that you think improve it

@FlorianJacta Sorry for the delay, but I’ve submitted a PR with the required changes. Please take a look when you can! PR #2046

@jrobinAV can i take up this issue ?

Hi I want to make my first hacktoberfest contribution, can you help me?

This issue has been resolved by #2046


Title: Add a candy Crush Game

Is your feature request related to a problem? Please describe.
Candy Crush game has many defined set of rules and can be very interesting to play !!
It can be a good Javascript Training as it has too many rules to handle
Describe the solution you'd like.
Adding a UNO game as a part of Tutorial
Describe alternatives you've considered.
--
Add any other context or screenshots about the feature request here.
No response

@debangi29 Thank you for providing your insight but it is not required as of now


Compound_ Interest Calculator

Calculator Functions
It will find out the compound interest .I would like to work on it.
Please assign me!
Contribution Event
GSSOC22
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


add anonymous user-defined command and command object

define and call anonymous command like the follow
var cmd = (){
  echo $@
  return 34
}

$cmd.call(['a', 'b'])

also add command type for represent anonymous user-defined command
related to #460

also access upper scope variables


Keep Order Lines Always on Top in TradingView Charts Library

Is your feature request related to a problem? Please describe.
I'm working with the TradingView Charting Library, and I've created some custom order lines using the createOrderLine function. The problem I'm facing is that these lines sometimes get hidden behind the candles, especially when the mouse is hovering over the candles.
Describe the solution you'd like
I would like an additional function in createOrderLine API that will ensure that the order lines always stay on top (z-index-wise) and never get hidden by other chart elements like candles.
Additional context
The order line is created like below,
const line = chart.createOrderLine(...)
.onCancel(...)
.setPrice(...)
.setText(orderText)
.setTooltip(tooltip)
...
Here I would like to specify to keep the line allways on top or something akin.
.allwaysOnTop()

Hi @flipflopapp this repository is for the lightweight charts. To submit a ticket related to advanced charts, please use the following repository: https://github.com/tradingview/charting_library/issues

Hi @flipflopapp this repository is for the lightweight charts. To submit a ticket related to advanced charts, please use the following repository: https://github.com/tradingview/charting_library/issues

This link is broken. What is the renamed repo name? Ty

the link is not broken, you need to get an access https://www.tradingview.com/charting-library-docs/v26/getting_started/quick-start#getting-access


changes for including JSON instead of XML

@jesuspg @jframos
Still testing

LGTM

A general comment, I assume that it is necessary change the acceptance tests associate to this component too.

Other impression, maybe I am wrong it that this changes are related only to obtain floating IPs in JSON format, but the rest of nova operations are taking into account? I assume that now all should be in JSON format, don't?

These changes are related to all request which were using XML. You can have a look at ClaudiaClientOpenStackImpl.java

Acceptance tests are not affected, because changes are related the way PaaSManager sends/receives requests to Nova. Features and messages of PaaSManager have not been modified.

LGTM

LGTM


Update test.md

Docs Build status updates of commit d6bdf9f:
:clock10: Preparing: the average preparing time is not available, there is no data for this repository.

Docs Build status updates of commit d6bdf9f:
:clock10: Incremental building: the average incremental building time is not available, there is no data for this repository.

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

File
Status
Preview URL
Details
Dependent Files

ShayaIntRepo91953-02/test.md
:white_check_mark:Succeeded

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


(maint) Use 64bit Ruby for testing

Previously Appveyor testing was only using the 32bit runtime, however as bolt
is 64bit only on Windows, this should really be 64bit.  This commit updates the
RUBY_VERSION to express that it should be 64bit.

@Iristyle Light is green, trap is clean.

CLA signed by all contributors.


Replace "Download" with "Delete"

https://github.com/glotlabs/gdrive/blob/a25418c1357295ce3a56c58363f12d35cdcc8dee/src/main.rs#L132

Thanks, fixed in latest release.


Updates to functional specifications for patterns that include the ImageWithCaption or VideoPlayer

User Story

As a [user role below]:
IBM.com Library adopter

I need to:
see updated specifications for patterns that utilize the ImageWithCaption or VideoPlayer

so that I can:
see which patterns can use either media type

Additional information

Dev story: https://github.com/carbon-design-system/ibm-dotcom-library/issues/1672
Updated specifications for the following patterns identified:

Content Block Simple
Content Block Segmented
Content Group Simple
Content Item

Acceptance criteria

[ ] Updated functional specifications for the patterns affected

Content Group Simple - https://ibm.box.com/s/einladcnn9zrfbmd0asuoywppr9zx7j9

v1.2 uploaded to box with updates to reflect that Image with Caption can also be used (doc file)

Content Item - https://ibm.box.com/s/xfm17x7dwbe4z7kjk482b8hbpjasft7o

already reflects that Image with Caption and Video can be used

@photodow do you want to do these two since the specs are currently in pages files and it's a small update?
Content Block Simple - https://ibm.box.com/s/7f8mb1xyqdu5pimx9k1xtpl7oqrme0qf

needs update to reflect that Image with Caption can also be used

Content Block Segmented - https://ibm.box.com/s/pkascpokk7ae4t3ixis98hmw4e5y1v56

needs update to reflect that Image with Caption can also be used (pages file)


multi textured mesh

Minetest allows six textures per node, one for every side.
Meshes in this mod only allow one texture.
This is easy to fix but hard to make syntax compatible with the rest on Minetest.
Some meshes will not have all six sides, if a missing side happens to be a top or bottom a small dummy mesh should be added for that side.
Here is an example cylinder.
cylinder.obj.zip
Use cases for this would be for default:tree or default:cactus
BTW: a cylinder might make a nice addition to moreblocks. Cylinder.obj is by pithydon (CC0).
edit: on second thought it would be strange for a circular saw to make a cylinder, so forget this last part.

There is a PR for this now #53


Callback was already called error

Failed to download selenium and/or chromedriver: Error: Could not download https://github.com/mozilla/geckodriver/releases/download/v0.20.1/geckodriver-v0.20.1-win64.zip: Error: read ECONNRESET
    at Request.<anonymous> (Yarn\\Data\\global\\node_modules\\selenium-standalone\\lib\\install.js:386:12)
    at Object.onceWrapper (events.js:315:30)
    at emitOne (events.js:116:13)
    at Request.emit (events.js:211:7)
    at Request.onRequestError (Yarn\\Data\\global\\node_modules\\request\\request.js:877:8)
    at emitOne (events.js:116:13)
    at ClientRequest.emit (events.js:211:7)
    at TLSSocket.socketErrorListener (_http_client.js:387:9)
    at emitOne (events.js:116:13)
    at TLSSocket.emit (events.js:211:7)
selenium-standalone will attempt to re-download next time it is run.
Yarn\\Data\\global\\node_modules\\async\\dist\\async.js:966
        **if (fn === null) throw new Error(\"Callback was already called.\");**
                         ^

**Error: Callback was already called.**
    at Yarn\\Data\\global\\node_modules\\async\\dist\\async.js:966:32
    at Local\\Yarn\\Data\\global\\node_modules\\async\\dist\\async.js:3885:13
    at onOpenZipFile (Yarn\\Data\\global\\node_modules\\selenium-standalone\\lib\\install.js:401:9)
    at Yarn\\Data\\global\\node_modules\\yauzl\\index.js:36:7
    at Yarn\\Data\\global\\node_modules\\yauzl\\index.js:183:5

Cannot install polymer-cli, getting same error with Yarn and with npm as well.

Not getting this exact error, but when I updated to 1.7.4 I had to manually install selenium-standalone to be able to run polymer test. Not sure if related or not.

Cleared cache, uninstalled polymer-cli, then manually installed selenium-standalone and finally again polymer-cli with no errors.


Fetch Metadata action returns null update-type output for pull requests

Description
I am using the dependabot/fetch-metadata action (v1.3.6) in my workflow to handle Dependabot pull requests. However, I am encountering an issue where the update-type output is returning null, even though the pull request is created by Dependabot.
Repository Configuration
Here's the dependabot.yml configuration for the repository:
version: 2

updates:
  - package-ecosystem: pip
    directory: /dependabot/
    schedule:
      interval: "weekly"
      day: "sunday"
    commit-message:
      prefix: "build: "
      prefix-development: "build: "
      include: "scope"
    rebase-strategy: "auto"
    target-branch: "develop"
    labels:
      - "build"
      - "dependencies"
    versioning-strategy: auto
    allow:
      - dependency-type: "all"
    open-pull-requests-limit: 10

The pyproject.toml file is located in the /dependabot/ directory, which is specified in the dependabot.yml configuration
Workflow Configuration
The following is the configuration for the workflow that uses the dependabot/fetch-metadata action:
on:
  pull_request_target:
    types:
      - opened
      - synchronize
permissions:
    pull-requests: write
    contents: write
jobs:
  review-dependabot-pr:
    runs-on: ubuntu-latest
    if: ${{ github.event.pull_request.user.login == 'dependabot[bot]' }}
    outputs:
      approved: ${{ steps.set-output.outputs.approved }}
    steps:

        # ... (other steps)

        - name: 📛 Dependabot metadata
            id: dependabot-metadata
            uses<EMAIL_ADDRESS># https://github.com/dependabot/fetch-metadata
            with:
                github-token: "${{ steps.generate_token.outputs.token }}"

        # ... (subsequent steps using the outputs)

Issue Details
When the workflow runs for a pull request created by Dependabot, the action returns a null update-type output, which causes issues in the subsequent steps that rely on this output.
Here's an example of the outputs returned by the dependabot/fetch-metadata action:
##[group]Run<EMAIL_ADDRESS>with:
  github-token: ***
  skip-commit-verification: false
##[endgroup]
Parsing Dependabot metadata
##[group]Outputting metadata for 1 updated dependency
outputs.dependency-names: pytest
outputs.dependency-type: direct:development
outputs.update-type: null
outputs.directory: /dependabot/develop
outputs.package-ecosystem: pip
outputs.target-branch: develop
outputs.previous-version:
outputs.new-version:
outputs.compatibility-score: 0
outputs.alert-state:
outputs.ghsa-id:
outputs.cvss: 0
##[endgroup]

The issue persists even after verifying that the pull request is created by Dependabot and that the workflow is triggered by the correct event (pull_request_target with opened and synchronize types).
Request for Assistance
I would appreciate any help in diagnosing and resolving this issue. If there's any additional information or logs that would be useful, please let me know, and I will provide them.

Alternatively, you can extract the diff/patch as it will probably include everything we need
(add .patch or .diff suffixes to the pull request url)

The version bump dependabot is trying is actually fairly simple:
From d9ab05edcb36f0086ba97b187a13c30822efa735 Mon Sep 17 00:00:00 2001
From: "dependabot[bot]"<EMAIL_ADDRESS>Date: Sun, 16 Apr 2023 05:43:13 +0000
Subject: [PATCH] build: (deps-dev): update pytest requirement in /dependabot

Updates the requirements on [pytest](https://github.com/pytest-dev/pytest) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](https://github.com/pytest-dev/pytest/compare/7.2.0...7.3.1)

---
updated-dependencies:
- dependency-name: pytest
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>---
 dependabot/pyproject.toml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dependabot/pyproject.toml b/dependabot/pyproject.toml
index 0bad554..dde5ebc 100644
--- a/dependabot/pyproject.toml
+++ b/dependabot/pyproject.toml
@@ -28,7 +28,7 @@ commitizen = "2.39.1"
 coverage = { extras = ["toml"], version = "7.0.3" }
 mypy = "1.2.0"
 pre-commit = "2.21.0"
-pytest = "7.2.0"
+pytest = "7.3.1"
 pytest-clarity = "1.0.1"
 pytest-mock = "3.10.0"
 pytest-xdist = "3.1.0"

One thing I might have missed when creating the issue is that this version bump is attempted before there is a poetry.lock file. Could that be the issue?

Would it be possible to share the workflow logs as well?

Requested labels: ubuntu-latest
Job defined at: redacted/redacted-repo/.github/workflows/dependabot_reviewer.yml@refs/heads/develop
Waiting for a runner to pick up this job...
Job is waiting for a hosted runner to come online.
Job is about to start running on the hosted runner: GitHub Actions 2 (hosted)
Current runner version: '2.303.0'
##[group]Operating System
Ubuntu 22.04.2 LTS
##[endgroup]
##[group]Runner Image
Image: ubuntu-22.04
Version: 20230409.1
Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230409.1/images/linux/Ubuntu2204-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230409.1
##[endgroup]
##[group]Runner Image Provisioner
<IP_ADDRESS>
##[endgroup]
##[group]GITHUB_TOKEN Permissions
Contents: write
Metadata: read
PullRequests: write
##[endgroup]
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Download action repository 'tibdex/github-app-token@v1' (SHA:b62528385c34dbc9f38e5f4225ac829252d1ea92)
Download action repository 'actions/checkout@v3' (SHA:8e5e7e5ab8b370d6c329ec480221332ada57f0ab)
Download action repository<EMAIL_ADDRESS>(SHA:4de7a6c08ce727a42e0adbbdc345f761a01240ce)
Complete job name: review-dependabot-pr
##[group]Run tibdex/github-app-token@v1
with:
  app_id: ***
  private_key: ***
  github_api_url: https://api.github.com
  repository: redacted/redacted-repo
##[endgroup]
Token generated successfully!
##[group]Run actions/checkout@v3
with:
  repository: redacted/redacted-repo
  path: redacted-repo
  token: ***
  ssh-strict: true
  persist-credentials: true
  clean: true
  fetch-depth: 1
  lfs: false
  submodules: false
  set-safe-directory: true
##[endgroup]
Syncing repository: redacted/redacted-repo
##[group]Getting Git version info
Working directory is '/home/runner/work/redacted-repo/redacted-repo/redacted-repo'
[command]/usr/bin/git version
git version 2.40.0
##[endgroup]
Temporarily overriding HOME='/home/runner/work/_temp/9e4e5cf0-ca5b-4318-b354-78686c3f421c' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/redacted-repo/redacted-repo/redacted-repo
##[group]Initializing the repository
[command]/usr/bin/git init /home/runner/work/redacted-repo/redacted-repo/redacted-repo
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>
Initialized empty Git repository in /home/runner/work/redacted-repo/redacted-repo/redacted-repo/.git/
[command]/usr/bin/git remote add origin https://github.com/redacted/redacted-repo
##[endgroup]
##[group]Disabling automatic garbage collection
[command]/usr/bin/git config --local gc.auto 0
##[endgroup]
##[group]Setting up auth
[command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
[command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :"
[command]/usr/bin/git config --local http.https://github.com/.extraheader AUTHORIZATION: basic ***
##[endgroup]
##[group]Fetching the repository
[command]/usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +106a6da65802fcc960e36f40c062c652bc366db3:refs/remotes/origin/develop
remote: Enumerating objects: 74, done.
remote: Counting objects:   1% (1/74)
remote: Counting objects:   2% (2/74)
 ...[truncated]...

remote: Counting objects: 100% (74/74)
remote: Counting objects: 100% (74/74), done.
remote: Compressing objects:   1% (1/64)
 ...[truncated]...
remote: Compressing objects: 100% (64/64)
remote: Compressing objects: 100% (64/64), done.
remote: Total 74 (delta 11), reused 43 (delta 4), pack-reused 0
From https://github.com/redacted/redacted-repo
 * [new ref]         106a6da65802fcc960e36f40c062c652bc366db3 -> origin/develop
##[endgroup]
##[group]Determining the checkout info
##[endgroup]
##[group]Checking out the ref
[command]/usr/bin/git checkout --progress --force -B develop refs/remotes/origin/develop
Switched to a new branch 'develop'
branch 'develop' set up to track 'origin/develop'.
##[endgroup]
[command]/usr/bin/git log -1 --format='%H'
'106a6da65802fcc960e36f40c062c652bc366db3'
##[group]Run<EMAIL_ADDRESS>with:
  github-token: ***
  skip-commit-verification: false
##[endgroup]
Parsing Dependabot metadata
##[group]Outputting metadata for 1 updated dependency
outputs.dependency-names: actions/checkout
outputs.dependency-type: direct:production
outputs.update-type: version-update:semver-major
outputs.directory: /develop
outputs.package-ecosystem: github_actions
outputs.target-branch: develop
outputs.previous-version: 2
outputs.new-version: 3
outputs.compatibility-score: 0
outputs.alert-state:
outputs.ghsa-id:
outputs.cvss: 0
##[endgroup]
##[group]Run gh pr merge --auto --merge "$PR_URL"
"gh pr merge --auto --merge "$PR_URL"
shell: /usr/bin/bash -e {0}
env:
  PR_URL: https://github.com/redacted/redacted-repo/pull/30
  GH_TOKEN: ***
##[endgroup]
##[group]Run gh pr comment $PR_URL --body "I'm **not approving** this PR because **it includes a major update of a dependency used in production**"
"gh pr comment $PR_URL --body "I'm **not approving** this PR because **it includes a major update of a dependency used in production**"
"echo "APPROVED=false" >> $GITHUB_OUTPUT
shell: /usr/bin/bash -e {0}
env:
  PR_URL: https://github.com/redacted/redacted-repo/pull/30
  GH_TOKEN: ***
##[endgroup]
https://github.com/redacted/redacted-repo/pull/30#issuecomment-1510292219
##[group]Run echo "APPROVED=$APPROVED" >> $GITHUB_OUTPUT
"echo "APPROVED=$APPROVED" >> $GITHUB_OUTPUT
shell: /usr/bin/bash -e {0}
env:
  APPROVED:
##[endgroup]
Post job cleanup.
[command]/usr/bin/git version
git version 2.40.0
Temporarily overriding HOME='/home/runner/work/_temp/d54a6c5a-eb84-46b3-9de4-14913f3b16a5' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/redacted-repo/redacted-repo/redacted-repo
[command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
[command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
http.https://github.com/.extraheader
[command]/usr/bin/git config --local --unset-all http.https://github.com/.extraheader
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :"
Evaluate and set job outputs
Cleaning up orphan processes

Your logs ☝ are showing:
outputs.update-type: version-update:semver-major

This doesn't match the behavior you described?

I apologize I posted the wrong log file. The log below pertains to the same action, but with null metadata.
Requested labels: ubuntu-latest
Job defined at: redacted/redacted-repo/.github/workflows/dependabot_reviewer.yml@refs/heads/develop
Waiting for a runner to pick up this job...
Job is waiting for a hosted runner to come online.
Job is about to start running on the hosted runner: GitHub Actions 2 (hosted)
Current runner version: '2.303.0'
##[group]Operating System
Ubuntu 22.04.2 LTS
##[endgroup]
##[group]Runner Image
Image: ubuntu-22.04
Version: 20230409.1
Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230409.1/images/linux/Ubuntu2204-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230409.1
##[endgroup]
##[group]Runner Image Provisioner
<IP_ADDRESS>
##[endgroup]
##[group]GITHUB_TOKEN Permissions
Contents: write
Metadata: read
PullRequests: write
##[endgroup]
Secret source: Actions
Prepare workflow directory
Prepare all required actions
Getting action download info
Download action repository 'tibdex/github-app-token@v1' (SHA:b62528385c34dbc9f38e5f4225ac829252d1ea92)
Download action repository 'actions/checkout@v3' (SHA:8e5e7e5ab8b370d6c329ec480221332ada57f0ab)
Download action repository<EMAIL_ADDRESS>(SHA:4de7a6c08ce727a42e0adbbdc345f761a01240ce)
Complete job name: review-dependabot-pr
##[group]Run tibdex/github-app-token@v1
with:
  app_id: ***
  private_key: ***
  github_api_url: https://api.github.com
  repository: redacted/redacted-repo
##[endgroup]
Token generated successfully!
##[group]Run actions/checkout@v3
with:
  repository: redacted/redacted-repo
  path: redacted-repo
  token: ***
  ssh-strict: true
  persist-credentials: true
  clean: true
  fetch-depth: 1
  lfs: false
  submodules: false
  set-safe-directory: true
##[endgroup]
Syncing repository: redacted/redacted-repo
##[group]Getting Git version info
Working directory is '/home/runner/work/redacted-repo/redacted-repo/redacted-repo'
[command]/usr/bin/git version
git version 2.40.0
##[endgroup]
Temporarily overriding HOME='/home/runner/work/_temp/ec6851f5-0803-45fe-a587-8dc9a2779829' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/redacted-repo/redacted-repo/redacted-repo
##[group]Initializing the repository
[command]/usr/bin/git init /home/runner/work/redacted-repo/redacted-repo/redacted-repo
hint: Using 'master' as the name for the initial branch. This default branch name
hint: is subject to change. To configure the initial branch name to use in all
hint: of your new repositories, which will suppress this warning, call:
hint:
hint: 	git config --global init.defaultBranch <name>
hint:
hint: Names commonly chosen instead of 'master' are 'main', 'trunk' and
hint: 'development'. The just-created branch can be renamed via this command:
hint:
hint: 	git branch -m <name>
Initialized empty Git repository in /home/runner/work/redacted-repo/redacted-repo/redacted-repo/.git/
[command]/usr/bin/git remote add origin https://github.com/redacted/redacted-repo
##[endgroup]
##[group]Disabling automatic garbage collection
[command]/usr/bin/git config --local gc.auto 0
##[endgroup]
##[group]Setting up auth
[command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
[command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :"
[command]/usr/bin/git config --local http.https://github.com/.extraheader AUTHORIZATION: basic ***
##[endgroup]
##[group]Fetching the repository
[command]/usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +7f82a6ac524148ad1f0439128d6fdeadad4b58ee:refs/remotes/origin/develop
remote: Enumerating objects: 74, done.
remote: Counting objects:   1% (1/74)
 ...[truncated]...
remote: Counting objects: 100% (74/74), done.
remote: Compressing objects:   1% (1/64)
 ...[truncated]...
remote: Compressing objects: 100% (64/64), done.
remote: Total 74 (delta 11), reused 45 (delta 4), pack-reused 0
From https://github.com/redacted/redacted-repo
 * [new ref]         7f82a6ac524148ad1f0439128d6fdeadad4b58ee -> origin/develop
##[endgroup]
##[group]Determining the checkout info
##[endgroup]
##[group]Checking out the ref
[command]/usr/bin/git checkout --progress --force -B develop refs/remotes/origin/develop
Switched to a new branch 'develop'
branch 'develop' set up to track 'origin/develop'.
##[endgroup]
[command]/usr/bin/git log -1 --format='%H'
'7f82a6ac524148ad1f0439128d6fdeadad4b58ee'
##[group]Run<EMAIL_ADDRESS>with:
  github-token: ***
  skip-commit-verification: false
##[endgroup]
Parsing Dependabot metadata
##[group]Outputting metadata for 1 updated dependency
outputs.dependency-names: pytest
outputs.dependency-type: direct:development
outputs.update-type: null
outputs.directory: /dependabot/develop
outputs.package-ecosystem: pip
outputs.target-branch: develop
outputs.previous-version:
outputs.new-version:
outputs.compatibility-score: 0
outputs.alert-state:
outputs.ghsa-id:
outputs.cvss: 0
##[endgroup]
##[group]Run gh pr merge --auto --merge "$PR_URL"
"gh pr merge --auto --merge "$PR_URL""
shell: /usr/bin/bash -e {0}
env:
  PR_URL: https://github.com/redacted/redacted-repo/pull/29
  GH_TOKEN: ***
##[endgroup]
##[group]Run echo "APPROVED=$APPROVED" >> $GITHUB_OUTPUT
"echo "APPROVED=$APPROVED" >> $GITHUB_OUTPUT"
shell: /usr/bin/bash -e {0}
env:
  APPROVED:
##[endgroup]
Post job cleanup.
[command]/usr/bin/git version
git version 2.40.0
Temporarily overriding HOME='/home/runner/work/_temp/f72043ac-62de-49a1-b4c1-be04b289ebd9' before making global git config changes
Adding repository directory to the temporary git global config as a safe directory
[command]/usr/bin/git config --global --add safe.directory /home/runner/work/redacted-repo/redacted-repo/redacted-repo
[command]/usr/bin/git config --local --name-only --get-regexp core\.sshCommand
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'core\.sshCommand' && git config --local --unset-all 'core.sshCommand' || :"
[command]/usr/bin/git config --local --name-only --get-regexp http\.https\:\/\/github\.com\/\.extraheader
http.https://github.com/.extraheader
[command]/usr/bin/git config --local --unset-all http.https://github.com/.extraheader
[command]/usr/bin/git submodule foreach --recursive sh -c "git config --local --name-only --get-regexp 'http\.https\:\/\/github\.com\/\.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :"
Evaluate and set job outputs
Cleaning up orphan processes

Is there anything else I can supply to make the debugging process easier?

A Reproducer would be ideal, but I don't know if you'll be able to set that up
Where a Reproducer is a public repository that shows the same problem

We are seeing this as well for internal Docker image update tags (either incoming or outgoing) have a prefixing v then the outputs.update-type is null
We have tried every variant of this and we have found (using internal docker builds) the following:
v1.5.1 to v1.5.2 -> outputs.update-type: null
v1.5.1 to 1.5.2 -> outputs.update-type: null
1.5.2 to v1.5.3 -> outputs.update-type: null
1.5.2 to 1.5.3 -> outputs.update-type: patch
While it is true that this sort of tag is not strictly semver compliant, I would expect Dependabot to be able to infer this sort of thing with a non-complex tag.
I did just create an issue in Dependabot Core but I looked here to see this and thought it was out issue so I closed it.

This does not seem to be just an update-type problem, but a whole version extraction problem.
outputs.previous-version:
outputs.new-version:

This action extracts versions from the default commit message, but your custom configuration looks like it removes the versions or changes the format here.
    commit-message:
      prefix: "build: "
      prefix-development: "build: "
      include: "scope"

Subject: [PATCH] build: (deps-dev): update pytest requirement in /dependabot

Updates the requirements on [pytest](https://github.com/pytest-dev/pytest) to permit the latest version.

https://github.com/dependabot/fetch-metadata/blob/d1defa476969826684d277f851861155deb3dbe1/src/dependabot/update_metadata.ts#L30-L32
(But I also don't fully understand what https://github.com/dependabot/dependabot-core/blob/8fee559e211d8292aad095256e300d02d9ba451f/common/lib/dependabot/pull_request_creator/message_builder.rb#L151 does either.)
As I understand it, there is no perfect solution for deciding the versions from the commit message with providing the user customization feature.
But I think by providing fixed metadata fields in all commit messages with the dependabot-core, the parser could be made simpler and more stable.

This is a reproducible PR https://github.com/haoliangyu/dependabot-minor-version-update-demo/pull/1 for this problem. The workflow run is here. It was initially used to test a bug in fastify/github-action-merge-dependabot and that bug originates from this issue. If you check the workflow run raw log, you can see
2023-08-03T00:44:36.9920160Z ##[group]Run dependabot/fetch-metadata@v1
2023-08-03T00:44:36.9920532Z with:
2023-08-03T00:44:36.9920860Z   skip-commit-verification: false
2023-08-03T00:44:36.9921240Z   skip-verification: false
2023-08-03T00:44:36.9921820Z   github-token: ***
2023-08-03T00:44:36.9922112Z ##[endgroup]
2023-08-03T00:44:37.7113504Z Parsing Dependabot metadata
2023-08-03T00:44:37.7232030Z ##[group]Outputting metadata for 2 updated dependencies
2023-08-03T00:44:37.7233040Z outputs.dependency-names: webpack, @nestjs/cli
2023-08-03T00:44:37.7233576Z outputs.dependency-type: direct:production
2023-08-03T00:44:37.7234039Z outputs.update-type: null
2023-08-03T00:44:37.7234498Z outputs.directory: /webpack-and-nestjs
2023-08-03T00:44:37.7234998Z outputs.package-ecosystem: npm_and_yarn
2023-08-03T00:44:37.7235448Z outputs.target-branch: main
2023-08-03T00:44:37.7235906Z outputs.previous-version:
2023-08-03T00:44:37.7236352Z outputs.new-version:
2023-08-03T00:44:37.7236777Z outputs.compatibility-score: 0
2023-08-03T00:44:37.7237245Z outputs.maintainer-changes: true
2023-08-03T00:44:37.7237693Z outputs.dependency-group:
2023-08-03T00:44:37.7238079Z outputs.alert-state:
2023-08-03T00:44:37.7238441Z outputs.ghsa-id:
2023-08-03T00:44:37.7238764Z outputs.cvss: 0
2023-08-03T00:44:37.7239757Z ##[endgroup]

This is the initial state of the repo https://github.com/haoliangyu/dependabot-minor-version-update-demo/tree/4ceb6428814f98a70bc836537eebe21558fbdf5f if you want to setup your test.

I have added this test on my local, using the commit message from the above PR, and it doesn't set the update-type correctly:
test.only('it properly handles dependencies which contain slashes', async () => {
  const commitMessage =
    'Bumps [webpack](https://github.com/webpack/webpack) to 5.88.1 and updates ancestor dependency [@nestjs/cli](https://github.com/nestjs/nest-cli). These dependencies need to be updated together.\n' +
    '\n' +
    '\n' +
    'Updates `webpack` from 5.73.0 to 5.88.1\n' +
    '- [Release notes](https://github.com/webpack/webpack/releases)\n' +
    '- [Commits](webpack/webpack@v5.73.0...v5.88.1)\n' +
    '\n' +
    'Updates `@nestjs/cli` from 8.2.8 to 10.1.11\n' +
    '- [Release notes](https://github.com/nestjs/nest-cli/releases)\n' +
    '- [Changelog](https://github.com/nestjs/nest-cli/blob/master/.release-it.json)\n' +
    '- [Commits](nestjs/nest-cli@8.2.8...10.1.11)\n' +
    '\n' +
    '---\n' +
    'updated-dependencies:\n' +
    '- dependency-name: webpack\n' +
    '  dependency-type: indirect\n' +
    '  update-type: minor\n' +
    '- dependency-name: "@nestjs/cli"\n' +
    '  dependency-type: direct:production\n' +
    '  update-type: major\n' +
    '...\n' +
    '\n' +
    'Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>
  const getAlert = async () => Promise.resolve({ alertState: '', ghsaId: '', cvss: 0 })
  const getScore = async () => Promise.resolve(0)
  const updatedDependencies = await updateMetadata.parse(commitMessage, '', 'dependabot/nuget/api/rails/coffee', 'main', getAlert, getScore)

  console.error(updatedDependencies);
})

To fix it, we could change the updateMetadata.parse function to parse the commit message for multiple dependencies or fix dependabot-core to set the update-type on the yaml part of the commit. I think the latter is the best.

Also happening with latest dependabot/fetch-metadata@v2 for at least Python pip updates:
Run dependabot/fetch-metadata@v2
  with:
    github-token: ***
    skip-commit-verification: false
    skip-verification: false
  env:
    PYTHON_VERSION: 3.12
Parsing Dependabot metadata
Outputting metadata for 1 updated dependency
  outputs.dependency-names: pydantic-settings
  outputs.dependency-type: direct:production
  outputs.update-type: null
  outputs.directory: /
  outputs.package-ecosystem: pip
  outputs.target-branch: main
  outputs.previous-version:
  outputs.new-version:
  outputs.compatibility-score: 0
  outputs.maintainer-changes: false
  outputs.dependency-group:
  outputs.alert-state:
  outputs.ghsa-id:
  outputs.cvss: 0


VCO audio mixer switches different order

On the main VCO board, there is a 5 input audio mixer.
Each input can be turned ON/OFF with a toggle switch, and several of the inputs have 3 position switches that go:

MODE 1
OFF
MODE 2

For example, the suboscillators have switches that go:

the same octave as parent (-0)
OFF
minus one octave relative to parent (-1)

For performance reasons, Ox has requested that these switches instead go:

MODE 1
MODE 2
OFF

This way, you can quickly turn a signal OFF by jamming the switch downwards. This is easier to do quickly than setting it to the middle position.

Note that the FM mixer uses 3 position toggles too. If we change the audio mixer order, we should probably do the FM mixer to for consistency.

Another sticky point regarding this: The original switch arrangement of MODE 1 - OFF - MODE 2 allows you to enter any setting from an "off" state.
With the proposed MODE 1 - MODE 2 - OFF arrangement, you need to go through MODE 2 from OFF to get to MODE 1. If this is an audio or modulation source there will be some discontinuity as the middle position is temporarily engaged on the way to the top position.
IMO the original arrangement has some merit. I personally prefer the middle-off setup. There should be another meeting with Ox to talk about it before deciding anything final.

Yet another reason to prefer center off: The second suboctave generator gets its input from the last "on" state of the first one.
Meaning, if  SUBOCT1 is in -0 or moved to OFF from -0 mode, then SUBOCT2 will get its input from the undivided SUBOCT1. This means that SUBOCT2 can be either the same base octave or one octave lower than the main oscillator.
But if SUBOCT1 is in -1 or moved to OFF from -1, then SUBOCT2 will get its input from the already divided SUBOCT1. This means that SUBOCT2 can be either one octave lower or two octaves lower than the main
If we put the OFF position downwards, then SUBOCT2 will always get the divided SUBOCT1 output when SUBOCT1 is turned off. We can't make SUBOCT2 the same base frequency of the main oscillator when SUBOCT1 is turned off. This seems like a pretty bad situation to me.
That's a lot of confusing talk to say "if we put the OFF position downwards, some useful actions are impossible to do".

New twist: Ox has suggested small LEDs near each of the audio attenuators to show you which signals are on and off.
These would light up when a signal is on. For signals with two "ON" settings (RING MOD XX, RING MOD XY. etc) it could either be a two color red/green LED, or two individual LEDs.
This solves the problem of not being able to easily tell which signals are on while letting us keep the original switch order. Food for thought as we continue to hash out the ideal main VCO board layout.


Update Version Catalog updates version ref

./gradlew versionCatalogUpdate causes version.ref to be updated instead of version. I'm not sure if it's a bug or expected behaviour.
The issue happens with room in my case. Here is the git changelog

I would expect versions.room to be updated instead of replacing version.ref = room with version.ref = roomPaging

It's not entirely clear what's going on from your example, if you have a small project to reproduce that would help...Or if that's not possible could you share the report.json file and your toml file?
Did the room version also get removed? There are some heuristics to try to update the correct version for dependencies like Room that update everything in the group to the same version and it looks that is what going on, also removing any duplicates.

Here's the report.json file
report.json.txt

room wasn't removed. Just references to room version where updated with roomPaging in libraries section

OK looking at the report, the reason why room wasn't updated is probably because the room-compiler did not update versions while the other ones did. So what's happening here is that multiple dependencies in androidx.room updates to 2.5.0-alpha01 and you already had a 2.5.0-alpha01 reference for paging, it switched the affected dependencies to the roomPaging common version. The heuristic is that if there are multiple dependencies in the same group using the same version, the version ref must be for the whole group, which is where it breaks down a bit now.
I can understand this is maybe unexpected, but since the room version was also still valid updating the room version ref would also be incorrect (room-compiler is still on 2.4.2).
I guess multiple things could happen here:

Pick the most appropriate version reference that we have for a particular group and version (which is what it is doing now)
Update the room version reference and drop the version.ref on the libraries using that reference (also unexpected I suppose)
Create a new version reference (which would duplicate roomPaging, so not great)
Panic and remove the version ref all together 🤔
Maybe some other option I haven't thought about :)

Happy to hear your thoughts here too, I don't have an immediate solution other than renaming the version reference to something more appropriate because in this case it appears that some of the androidx.room artifacts are on 2.5.0-alpha01 while others aren't.

I am working on automating updates to our dependencies and from testing, I noticed I am running into this issue myself.
Simplified libs.versions.toml:
[versions]
airbnb-mavericks = "2.5.1"

[libraries]
airbnb-lottie = "com.airbnb.android:lottie:3.7.0"
airbnb-lottie-compose = "com.airbnb.android:lottie-compose:4.2.2"
airbnb-mavericks = { module = "com.airbnb.android:mavericks", version.ref = "airbnb-mavericks" }
airbnb-mavericks-mocking = { module = "com.airbnb.android:mavericks-mocking", version.ref = "airbnb-mavericks" }
airbnb-mavericks-testing = { module = "com.airbnb.android:mavericks-testing", version.ref = "airbnb-mavericks" }

After running versionCatalogUpdate this is what I see:
[versions]
airbnb-mavericks = "5.0.3"

[libraries]
airbnb-lottie = { module = "com.airbnb.android:lottie", version.ref = "airbnb-mavericks" }
airbnb-lottie-compose = { module = "com.airbnb.android:lottie-compose", version.ref = "airbnb-mavericks" }
airbnb-mavericks = { module = "com.airbnb.android:mavericks", version.ref = "airbnb-mavericks" }
airbnb-mavericks-mocking = { module = "com.airbnb.android:mavericks-mocking", version.ref = "airbnb-mavericks" }
airbnb-mavericks-testing = { module = "com.airbnb.android:mavericks-testing", version.ref = "airbnb-mavericks" }

It updated so that lottie is now referencing the version from airbnb-mavericks which isn't what I want or expected. It now  has a lottie version which won't work for mavericks since that mavicks version doesn't exist.
Ideally, I would like to keep the airbnb group. I have things setup where I would use the same naming from version as part of my library.
So with the example above, I have version airbnb-mavericks and then I have libraries airbnb-mavericks + airbnb-mavericks-*
Right now, my thoughts are that if I didn't specify a version ref, then I wouldn't expect the update to add one or to update a library to use a version ref.

Thanks for sharing that snippet!
I'm still not sure what the best fix is. The use case for adding version refs is simple: you can just dump in your dependencies and if they are part of a group, you get a common version that you can bump or have bumped by the plugin. Even if you start out with no version references and later add more dependencies from that group. That also prevents you from mixing and matching "wrong" versions too. And at the same time keep the amount of version references reasonable and optimised.
But as you point out, if the group is too common, like with airbnb here (IMHO Lottie and Mavericks shouldn't be in the same group) then things start to break down. I'm not sure if it works currently if you manually set up those refs. I guess if you manually correct it once and it would stick that could be OK (but not sure how to deal with libs without a version reference; there would be no way to tell if you want that to be without a version on purpose or not).
Switching the ref on mavericks for a non-existent version is wrong in any case, it shouldn't be doing that. So this might be a slightly different scenario vs the initial Room one. I think ideally it should add a version ref if it can (unique version for a set of libraries within the same group for example) or otherwise leave it.
Open to other suggestions and other edge cases, this probably will need some good tests around when working towards a fix.
If anyone else sees something like this, please share the "before" relevant TOML snippet and the report.json if you have it, that will help me to setup some tests...I'll try to spend some time on this soonish.


[DataGridPro] Documentation on customizing theme

I am trying to pass in an existing theme to the DataGridPro component, but can't figure out how from the Docs. It seems I need to override the panel components so that when I click to open a panel it still inherits my theme (as of now, clicking to open any filter or columns panel causes the whole app theme to change.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Unable to see how I can pass in a theme to the panels for filter and columns
Expected Behavior 🤔
documentation would be great, and happy to contribute.
Steps to Reproduce 🕹
Steps:

provide a custom theme to your app ThemeProvider
render DataGridPro
Click to view columns or filter panel
the whole app's theme will be overwritten

Context 🔦
I am trying to use the datagrid component with the theme I already have.
Your Environment 🌎

`npx @material-ui/envinfo`
Chrome
SC-MKEFFEL-OSX mkeffele$ npx @material-ui/envinfo
npx: installed 2 in 3.75s

  System:
    OS: macOS Mojave 10.14.6
  Binaries:
    Node: 14.5.0 - /usr/local/Cellar/node/13.2.0/bin/node
    Yarn: 1.19.2 - /usr/local/bin/yarn
    npm: 6.14.5 - /usr/local/bin/npm
  Browsers:
    Chrome: 92.0.4515.159
    Edge: Not Found
    Firefox: 88.0.1
    Safari: 14.1.2
  npmPackages:
    @material-ui/core: ^4.12.0 => 4.12.0
    @material-ui/icons: ^4.11.2 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.58
    @material-ui/pickers:  3.2.2
    @material-ui/styles:  4.11.4
    @material-ui/system:  4.12.1
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @material-ui/x-grid-data-generator: ^4.0.0-alpha.34 => 4.0.0-alpha.34
    @types/react:  17.0.11
    react: ^17.0.0 => 17.0.2
    react-dom: ^17.0.0 => 17.0.2

screenshots showing the difference attached. no code changes between screenshots, all the styling gets removed.

I love coding :)
Wrapping the  in a  resolved this.


Image Fix rotate_90/rotate_180 methods

Changes/fixes Image::rotate_90 so it doesn't use Image::resize as it's not needed (and was resulting in interpolating the pixels). Now rotation happens in-place by direct pixel manipulation (and swapping width and height after all).
Fixes #64048.
Fixes Image::rotate_180 which was not flipping the middle row for odd-height images. It's also ~2.5x faster now.
Also now these methods don't throw an error on width == 1 or height == 1.

4.0.alpha14
this PR

More details on rotate_90:
Pixels are being moved according to the cycles in the permutation the given 90 degrees rotation (clockwise/counterclockwise) defines (rotating just permutes 1D pixel buffer of the image).
Current implementation is based e.g. on this paper. But it's applying rotation-permutation, not transposition-permutation (rotation could be made by transposition+flip in 2 steps but in this PR it's done directly in 1 step). Nonetheless, this is a simple, naive implementation (the part determining which pixels were not yet moved).
It can be fast/slow depending on how permutation decomposes into cycles (it's quite random for rectangular matrices). It's correct and in some cases super fast but the worst case could be slower than the current incorrect implementation which uses resizing. Treat it as a bug-fix. I'll look into improving it performance-wise (the worst case) some time later in another PR.

Thanks!

Approved by PR meeting and the consul of Yuris

Heh, had to reverse image search to understand, so saving some folks the trouble ;)
https://villains.fandom.com/wiki/Yuri_(Command_%26_Conquer)


加载权重失败

=> loading HRNet pretrained model models/hrnet_w18_small_v2.pth
HRNet weight-loading fails
NLCDetection weight-loading fails
DetectionHead weight-loading fails
train的时候加载权重失败的原因是什么呢


Add getting amount keywords

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


AutoClosingTimerCallBack threads causing app suspension

Nlog version: 4.5.11
.Net Framework  7.7.2
On our production environment yesterday we experienced unresponsiveness. Looking at the memory dump we noticed an unusual amount of nlog functions in threads which appear to be locking our system.
Can someone please assist us in determining what went wrong or what could have done wrong.
We currently log out only errors to a file and a database table.

Nlog Config: (removed tables names)
`

<!--Database Setup-->
<target  xsi:type="Database" name="db" connectionString="${var:NlogConnectionString}">

  <commandText>
    INSERT INTO SCHEMA.TABLE (Column1, Column2, Column3, Column4, Column5, Column6, Column7, Column8, Column9)
    VALUES(@VALUE1, @VALUE2, @VALUE3, nullif(@VALUE4,''),  @msg, nullif(@VALUE5,''), nullif(@VALUE6,''), nullif(@VALUE7,''), nullif(@VALUE8,'') );
  </commandText>
  <parameter name="@logdate" layout="${longdate}" />
  <parameter name="@msg" layout="${message}" />
  <parameter name="@level1" layout="${level}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}"/>
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${threadid}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />

</target>

<!--
add your targets here.
See https://github.com/nlog/NLog/wiki/Targets for possible targets.
See https://github.com/nlog/NLog/wiki/Layout-Renderers for the possible layout renderers.
-->

<!--
Write events to a file with the date in the filename.
<target xsi:type="File" name="f" fileName="${basedir}/logs/${shortdate}.log"
        layout="${longdate} ${uppercase:${level}} ${message}" />
-->

`

Looks like you are using File-Targets from the screenshot, but those are not in your NLog.config?
Looks like you are initializing an NLog-target from the screenshot. Are you using NLog autoreload=true? Or are you perform explicit reload of NLog-config ? Or are you dynamically adding new NLog targets to the NLog-config? Or is the screenshot from the application hanging at startup?

@snakefoot  Thank you for your response.
Sorry it must have got cropped while pasting (please find it below).
Yes we are using autoreload = true in the config.
We do the following dynamically:
1- adding the connection string to the database
2- Specifying the log file name
We then call LogManager.ReconfigExistingLoggers(); , the rest is controlled through the config.
The screen shot is from a memory dump analysed after the app was suspended and inactive.
The app had been online for 5 days prior to the event, it wasn't on start up.

<!--Database Setup-->
<target  xsi:type="Database" name="db" connectionString="${var:NlogConnectionString}">

  <commandText>
    INSERT INTO SCHEMA.TABLE (Column1, Column2, Column3, Column4, Column5, Column6, Column7, Column8, Column9)
    VALUES(@VALUE1, @VALUE2, @VALUE3, nullif(@VALUE4,''),  @msg, nullif(@VALUE5,''), nullif(@VALUE6,''), nullif(@VALUE7,''), nullif(@VALUE8,'') );
  </commandText>
  <parameter name="@logdate" layout="${longdate}" />
  <parameter name="@msg" layout="${message}" />
  <parameter name="@level1" layout="${level}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}"/>
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />
  <parameter name="@PARAM" layout="${threadid}" />
  <parameter name="@PARAM" layout="${event-properties:item=PARAM}" />

</target>

<!--
add your targets here.
See https://github.com/nlog/NLog/wiki/Targets for possible targets.
See https://github.com/nlog/NLog/wiki/Layout-Renderers for the possible layout renderers.
-->

<!--
Write events to a file with the date in the filename.
<target xsi:type="File" name="f" fileName="${basedir}/logs/${shortdate}.log"
        layout="${longdate} ${uppercase:${level}} ${message}" />
-->

Are you writing to a network path using the FileTarget ? (Filename is blank in the attached Nlog.config)
Not sure I understand the meaning of suspended/inactive. Do you mean deadlocked/frozen ?

The file is written to the directory where our app is hosted , so not over the network (its configurable therefore we have made it dynamic , but in this instance its the same directory where the app which contains nlog is hosted) .
I mean locked not accepting connections due to it being suspended.

I guess you have to navigate through all the threads, and find the one that is holding the FileTarget.SyncRoot-lock but is stuck on something else (Ignore all other threads that are waiting for FileTarget.SyncRoot).

Thanks very much @snakefoot  for your response I really appreciate your time.
I am currently looking into the above.
May I ask what would be best practice if I wanted to change the log location on the fly based on company for example (different folders).  Its all shared code , but would having a nlog configuration be the best practice here or a shared instance (what we currently have and we are changing the filename dynamically)

If the same running instance of the application is used by different companies at the same time, then I would probably use a scope-name-variable (See ${mdlc}) or a logger-name-variable (See ${logger}) in the FileTarget.FileName that matches company.


Cannot build web app with pure Dart

Describe the bug
Building a web app not using Flutter fails due to the universal_io package that is currently lacking support for web.
To Reproduce
Steps to reproduce the behavior:

Create a project with dart create -t web.
Run dart pub add supabase and dart pub get to start using the supabase package.
Prepare to use Supabase by calling the SupabaseClient constructor.
Run the app by webdev serve.
See the warnings printed in the console.
Open localhost:8080 in a browser.

main.dart should look like this:
import 'dart:html';

import 'package:supabase/supabase.dart';

void main() {
  SupabaseClient(
    'Your Supabase URL',
    'Your Supabase Key',
  );

  querySelector('#output')?.text = 'Your Dart app is running.';
}

Expected behavior
There is no warning during a build, and "Your Dart app is running." is shown in the browser.
Screenshots
N/A
Version (please complete the following information):
└── supabase 1.11.5
    ├── functions_client 1.3.2
    ├── gotrue 1.12.2
    ├── postgrest 1.5.1
    ├── realtime_client 1.2.2
    ├── storage_client 1.5.2

Additional context
[WARNING] build_web_compilers:entrypoint on web/main.dart: Skipping compiling bug|web/main.dart with ddc because some of its
transitive libraries have sdk dependencies that not supported on this platform:

universal_io|lib/src/_exports_in_vm.dart
universal_io|lib/src/internet_address.dart
universal_io|lib/src/http_client.dart

How about removing the dependency on universal_io to be less subject to external changes if it is not heavily used?
As far as I see,

gotrue uses SocketException
storage_client uses SocketException and File

It may be possible to replace File with XFile.

Related issue: https://github.com/dint-dev/universal_io/issues/41

I'm unsure on how to replace the SocketException. Does everything work except that warning?

Does everything work except that warning?

main.dart.js is not created due to the dependency on universal_io. I cannot start the web app at all. If you have to see what actually happens, please try building an app, following the steps I wrote earlier.
I was thinking SocketException was reimplementation in universal_io. However, having looked at the source code, it looks like the SocketException itself from dart:io via universal_io if I'm not wrong. It means the package depends on dart:io, whether the app is for mobile / desktop or for web. It seems like the cause of the issue.
I was able to build my blog site made with Dart + Jaspr + Supabase when I was using supabase 1.4.0. I'm not sure about other versions between that version and the latest. Investigating how universal_io was handling SocketException in the past would probably be a way to find a solution.


Add slides related to communcation evolution

@AndreyKrel
add please benefits of bitcoin as a communication tool https://github.com/decentcult/blockchain-intro/blob/master/index.md#benefits-as-a-communication-tool


Multi GPU training

Hi，
how to train use mutil gpus in this project? try to set CUDA_VISIBLE_DEVICES=0,1, but it is wrong.
best

@WHQCHINA Thank you for the report !
We reproduced this issue and fixed it with this solution.
Please check the recent commit.
Thank you.

@ku21fan thanks，it is work!

@ku21fan mutil GPU training and workers number > 0 ,  it is GPUs stuck for long time.

@WHQCHINA In our environment (Intel Xeon(R) E5-2630 v4 2.20GHz CPU and NVIDIA TESLA P40 GPU),
we could not reproduce 'GPUs stuck' with worker number 4 x num_gpu.
Though we could not reproduce it, it could happen with a different environment, so we added a comment to here.
Thank you for report and solution!

Hi，
I want to known how long you to train a TPS+Resnet+BiLSTM+Attention model? I train 1000 iterations using one hour using two NVIDIA TESLA P40 GPUs.
I have train a model using  synthetic data, but test in real data is bad, can you give me some davise? I train model using real data now.

@WHQCHINA

how long you train a TPS+Resnet+BiLSTM+Attention model? with P40 GPU
The training time takes about 2days with 1 GPU and takes about 31 hours with 2 GPUs.
We are re-training the model to re-check training time for now.
I attached the training log of them.
log_train_1gpu.txt
log_train_2gpu.txt
It took 1500 seconds per 2000 iteration with 2 GPUs.
I am wondering if you have used cudnn.
In our experiment, cudnn.benchmark = True accelerate the training time greatly.

Combining synthetic data and real data and using it as a training data set would be a good plan.
try it with --select_data and --batch_ratio options.

best


fixing cloud watch integration in non-default regions

Fixes issues #9

Hi,
Thank you for providing this fix. We are reviewing your contributions(including the other few pull requests) this week and will update you shortly.

This fix has been verified. Please confirm this contribution is under the terms of the Apache 2.0 license, before we can merge it to master. Thanks

I can confirm that this and my other pull requests are under the license to the best of my knowledge.