I want to add no.of forks,issues,stars and other in readme file.

I want to add no.of forks , issues, stars in readme file .
Example:

I want to work on this . Please assign me this issue @muthuannamalai12 @Swarnimashukla .

Can I be assigned for the issue if its not yet done @Swarnimashukla

@Srishti013 Hello thanks for your interest the pr is ready.


expand cluster

Is it within the current capabilities of the library to expand a cluster? For example, if I have a cluster of pins really tight together, such that no matter how much you zoom it would be inappropriate to show, you can click on the cluster marker to expand the cluster and display the pins a set distance away from the cluster marker point. I believe mapbox/leaflet has this functionality

Didn't see that the Callout component could be used for this


here-wallet: upgrade to 1.4

Fix serialise binary arguments for function calls.
Fix multiple function calls
Add new UI for HERE Connector (Iframe widget)

Hey, @AZbang thank you for submitting this PR.
This looks good to me 👍 , but could you switch the base branch to dev?
In the future could you please follow the instructions which are in the description during submitting a new PR and choose the "General template (most common)" option because if we do not select it some of GH-Actions might not get triggered.


Default audio download isnt highest quality

Checklist

[ x] I've verified that I'm running youtube-dl version 2019.06.21
[x ] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar bug reports including closed ones
[ x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

youtube-dl -v -x https://www.youtube.com/watch?v=o_zD1YCj2fk
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-x', 'https://www.youtube.com/watch?v=o_zD1YC
j2fk']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.06.21
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] o_zD1YCj2fk: Downloading webpage
[youtube] o_zD1YCj2fk: Downloading video info webpage
[youtube] {18} signature length 104, html5 player vfl49f_g4
[youtube] {137} signature length 104, html5 player vfl49f_g4
[youtube] {248} signature length 104, html5 player vfl49f_g4
[youtube] {136} signature length 104, html5 player vfl49f_g4
[youtube] {247} signature length 100, html5 player vfl49f_g4
[youtube] {135} signature length 104, html5 player vfl49f_g4
[youtube] {244} signature length 104, html5 player vfl49f_g4
[youtube] {134} signature length 100, html5 player vfl49f_g4
[youtube] {243} signature length 104, html5 player vfl49f_g4
[youtube] {133} signature length 100, html5 player vfl49f_g4
[youtube] {242} signature length 104, html5 player vfl49f_g4
[youtube] {160} signature length 104, html5 player vfl49f_g4
[youtube] {278} signature length 104, html5 player vfl49f_g4
[youtube] {140} signature length 100, html5 player vfl49f_g4
[youtube] {171} signature length 104, html5 player vfl49f_g4
[youtube] {249} signature length 104, html5 player vfl49f_g4
[youtube] {250} signature length 100, html5 player vfl49f_g4
[youtube] {251} signature length 104, html5 player vfl49f_g4
[debug] Invoking downloader on 'https://r2---sn-fpqxc5oq-hxae.googlevideo.com/vi
deoplayback?expire=1561370023&ei=R0kQXaWqEcHRz7sPiNiW4AE&ip=<IP_ADDRESS>&id=o-AO
t-8uBf-YuA5JvjB5QXrjkCAfUveEur8qvUfnsBGmVj&itag=171&source=youtube&requiressl=ye
s&mm=31%2C29&mn=sn-fpqxc5oq-hxae%2Csn-ntqe6n7k&ms=au%2Crdu&mv=m&pl=24&initcwndbp
s=2488750&mime=audio%2Fwebm&gir=yes&clen=2657363&dur=157.731&lmt=156085012590574
8&mt=1561348324&fvip=5&keepalive=yes&beids=9466585&c=WEB&txp=2301222&sparams=exp
ire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&
lsparams=mm%2Cmn%2Cms%2Cmv%2Cpl%2Cinitcwndbps&lsig=AHylml4wRAIgATSS0FtQcD2dJxvKV
on2KhhO2dqPFJOgR7F2W2qtEyQCIEQGZ2NRUCLxBI5gLLx41g3rju70oxB5G_drapRsunQ5&sig=ALgx
I2wwRQIhANQmBr1wbyJzwp_8_I1VyFuFKA_a61BR_vVcLXtJeK8sAiA1UUwcHMOn-Vx9jc2ZnACTpJ6x
8yxcj3gzDoQsViL9xg==&ratebypass=yes'
[download] blink 182 - What's My Age Again for Brass Quintet and Drums with shee
t music-o_zD1YCj2fk.webm has already been downloaded
[download] 100% of 2.53MiB
ERROR: ffprobe/avprobe and ffmpeg/avconv not found. Please install one.
 Please install one.
Description

-x chooses the largest file, not the highest bitrate.
Youtube has many different audio qualities for this song
https://www.youtube.com/watch?v=o_zD1YCj2fk
as seen in this -F
format code  extension  resolution note
249          webm       audio only DASH audio   66k , opus @ 50k, 1.04MiB
250          webm       audio only DASH audio   87k , opus @ 70k, 1.36MiB
140          m4a        audio only DASH audio  130k , m4a_dash container, mp4a.40.2@128k, 2.44MiB
251          webm       audio only DASH audio  160k , opus @160k, 2.62MiB
171          webm       audio only DASH audio  162k , vorbis@128k, 2.53MiB
file chosen when you don't specify is the 128k vorbis, not the 160k opus. this punishes opus for being more efficient.
also, if youtube is being so kind as to wrap things up in .opus for us, we dont really need ffmpeg?

youtube-dl's quality selection always picks largest filesize over bitrate, if you don't like it, specify manually.
It's not wrapped in .opus, it's wrapped in .webm, the codec is opus.

It selects format with max total bitrate as reported by metadata that is 171.


Code implemented to handle/disable auto-shutdown of idle connections code can crash cloud nodes

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
Work has previously been done to disable auto-shutdown of idle connections here: https://github.com/ulixee/hero/pull/199 and the method introduced to check for idle timeout ms fails in the edge cases when the hero connection has already disconnected due to an error.
I observed the following in the logs:
[0] 2023-11-09T07:24:56.132Z ERROR [hero-core/index] UnhandledError(fatal) {
--
[0]   origin: 'uncaughtException',
[0]   context: {},
[0]   sessionId: null,
[0]   sessionName: undefined
[0] } TypeError: Cannot read properties of null (reading 'clearIdleConnectionsAfterMillis')
[0]     at ConnectionToHeroClient.get autoShutdownMillis [as autoShutdownMillis] (/app/ulixee/node_modules/core/connections/ConnectionToHeroClient.ts:37:22)
[0]     at ConnectionToHeroClient.disconnectIfInactive (/app/ulixee/node_modules/core/connections/ConnectionToHeroClient.ts:262:33)
[0]     at listOnTimeout (node:internal/timers:569:17)
[0]     at process.processTimers (node:internal/timers:512:7)
[0] 2023-11-09T07:24:56.132Z STATS [commons/lib/ShutdownHandler] ShutdownHandler.onSignal { signal: 'exit', context: {} }
[0] 2023-11-09T07:24:56.133Z STATS [commons/lib/ShutdownHandler] ShutdownHandler.execute {
[0]   signal: 'exit',
[0]   fn: 'function () { [native code] }',
[0]   callsite: 'at new CloudNode (/app/ulixee/cloud/main/lib/CloudNode.ts:148:21)',
[0]   context: {}
[0] }

I traced the cause to this line where this.core is set to null on disconnect: https://github.com/ulixee/hero/blob/main/core/connections/ConnectionToHeroClient.ts#L167
The checkForAutoshutdown is bound to the close event here https://github.com/ulixee/hero/blob/main/core/connections/ConnectionToHeroClient.ts#L202
Which then blows up when trying to retrieve the timeout ms from the core object here https://github.com/ulixee/hero/blob/main/core/connections/ConnectionToHeroClient.ts#L202
I'm mitigating this issue by adding the clearIdleConnectionsAfterMillis property to the connectionToHero object that is copied from the core instance during the constructor, that way the value is still accessible after this.core is cleared out on disconnect.
Here is the diff that solved my problem:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 54d1611..d67c16a 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -20,6 +20,7 @@ class ConnectionToHeroClient extends eventUtils_1.TypedEventEmitter {
         super();
         this.transport = transport;
         this.core = core;
+        this.clearIdleConnectionsAfterMillis = core.clearIdleConnectionsAfterMillis;
         this.sessionIdToRemoteEvents = new Map();
         this.activeCommandMessageIds = new Set();
         transport.on('message', message => this.handleRequest(message));
@@ -29,7 +30,7 @@ class ConnectionToHeroClient extends eventUtils_1.TypedEventEmitter {
         this.disconnectIfInactive = this.disconnectIfInactive.bind(this);
     }
     get autoShutdownMillis() {
-        return this.core.clearIdleConnectionsAfterMillis;
+        return this.clearIdleConnectionsAfterMillis;
     }
     ///////  CORE CONNECTION  /////////////////////////////////////////////////////////////////////////////////////
     async handleRequest(payload) {

This issue body was partially generated by patch-package.

Thanks! Will apply a fix on next release.


Decide on initial architecture

Create an initial architecture that can satisfy all requirements.

The main component to be developed is the authorization server. It will rely on an "authorization database" containing companies, users and scopes. It will also contain authorization codes and authorization tokens.
Companies will only have a company name identifying them.
Scopes will store a scope token following the format specified in the RFC. They will belong to a specific company and have a maximum span they can be assigned for.
Users will belong to a specific company and have a username and password, stored securely. They can also have roles assigned via a many-to-many relationship.
The authorization server will act as defined in the OAuth 2.0 protocol (only the authorization grant code flow).
On the client registration endpoint, clients may register. Note that "client" here means the program the user is interacting with, not a client of AstronAuth's services.
On the authorization endpoint, it will display an HTML page asking the user to log in. After that, it will check that the user has all scopes requested. If they do, it will inform the user of the scopes requested. Finally, if the user consents, it will redirect the user-agent to the URI given, giving an authorization code.
On the token endpoint, it will give the authorization token token corresponding to the authorization code if it is valid. It will never issue or accept refresh tokens for simplicity.
On top of the standard, it may have a token validation endpoint which verifies the validity of the token. I'll decide if this is necessary later as I read up on what encryption and formats tokens should have.

The over-all architecture of the system will be the following:
   +---------------+                           +---------------+
   | Authorization |                           | Authorization |
   |   Database    |<------------+             |   Database    |
   |    (Users)    |             |             |   (Clients)   |
   +---------------+         configures        +---------------+
           ^                   users                   ^
           |                     |                     |
   +-------|-------+     +-------|-------+     +-------|-------+
   | Authorization |     | Configuration |     | Authorization |
   |    Server     |     |    Server     |     |    Server     |
   |    (Users)    |     |   (Clients)   |     |   (Clients)   |
   +---------------+     +---------------+     +---------------+
           ^                     ^                     ^
           |                     |                     |
        log in                access              authenticate
         using                   +---------------------+
           |                     |
   +-------|-------+     +-------|-------+
   |               |     |               |
   |     Users     |     |    Clients    |
   |               |     |               |
   +---------------+     +---------------+

The two authentication servers and databases will be identical according to the previous comment.
The user authentication server will authorize user tokens based on the configuration specified in the user authorization database by the clients through the configuration server. For that configuration, they will have to authenticate through the client authentication server which uses the configuration on the client authentication database, controlled by AstronAuth.
Not represented will be two simple registration server, one for clients and one for users, which will allow every user and client to register for demonstration purposes.

This forces the administrators of AstronAuth to control the client database directly which is not great. However a copy of the configuration server could be launched for that purpose. I may decide on implementing it or not depending on if the effort is warranted.

The architecture needs to change. See Decide authorization database structure.


The hud view is not centered

I'm trying to add hud as a subview but it's not centered and it's showing at the top left corner.
What I tried:

Using constraints.
Adding it as a subview of the superview as stated in #343

Fixed using hud.center = view.center


Stop modification of br tag

When I pass
 through the sanitizeHtml() is there any way to keep it from changing to
 ?
Or for other tags that may be similiar..
Thanks!

 is not valid whereas
 is. What is your use case?

 is perfectly valid HTML,  valid XHTML. ;-)
http://w3c.github.io/html-reference/syntax.html#void-element

Optionally, a "/" character, which may be present only if the element is a void element.

As long as htmlparser2 doesn't offer a way to sniff out whether the original tag had a / or not, sanitize-html will continue to use its internal list of void elements to decide whether to use /.
But even if it did, I'm not sure I would change this.
 may be acceptable in HTML, but
 is always 100% valid as both HTML and XHTML. It's just cleaner and it does not change the meaning of the code in any way.

For large deployments that store a lot of user generated html, it is favorable to keep the data size as low as possible.

@boutell I am having a similar issue with an image tag. It is totally breaking my functionality since I am working with innerHTML attribute which return the html like it is in the browser's DOM. Chrome strips out self closing tags, which causes html that I pass through html-sanitize to be inconsistent with my innerHTML. Consistency is critical for me, since I am changing the html by selection indexes, and if the sanitize html changes it, indexes are offset.

I have same issue here
Please, Stop modification of br tag

I am hitting this issue with quill. I don't think quill likes

Hello, Thank you for this library.
I'm also a user of Quill and I'd find it helpful to have an option to disable the addition of a slash for
 tags.
I wouldn't say "it's just cleaner", but "this is one of the possible correct solutions".
For those interested in a solution to this particular problem, I've made it possible in my fork:
To achieve this, define the selfClosing option as an object. The keys of this object should be the tag names, and the value should be a truthy or falsy value. If the value of a specific tag has the property voidTag set to true, the library will not insert the space and the slash.

These are really two separate features:

"Preserve it the way it was" ("stop modification," as the title says)

We can't easily do this because of the way the upstream module gives us the information.

"An option to not use the /"

Sure, that would be fine as long as the default doesn't change unexpectedly. @gastonyte would you like to create a PR for your feature?

Hi @boutell, I didn't know if this feature was wanted or not, so I just made the modification in a fork to be able to use it in my app, but if it's something you would like to see integrated into your library, it is with pleasure that I will create the PR =)

Yes, supporting this via a flag sounds great! My objection was to the original request in which the original presence or absence of a / would be preserved, simply because I don't think we have access to that information.


Can't get to cocoapod 5

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Added "pod 'Alamofire' to my podfile and used  both pod install and pod update
What did you expect to happen?
Expected to receive Alamofire 5.0.2
What happened instead?
Received Alamofire 4.9.1
Alamofire Environment
Alamofire version: 4.9.1
Xcode version: 11.3
**Swift version:**5.1
Platform(s) running Alamofire: iOS
macOS version running Xcode: 10.14.6
Demo Project
Works fine on a new project.  There must be some target version somewhere I need to set.

Run pod update.

I did run pod update.  Still 4.9.1.  What are the settings needed to get to 5?

On Mar 13, 2020, at 3:19 PM, Jon Shier<EMAIL_ADDRESS>wrote:
Run pod update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Alamofire/Alamofire/issues/3109#issuecomment-598953603, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCKGU56FFOCZAFGKN2ENC3RHKWPVANCNFSM4LHPCI3A.

There are none. Do you have other dependencies on Alamofire which may require v4? I suggest you investigate your dependency tree and local CocoaPods setup to see.

You are correct.  There is a line in the Podfile: platform :iOS, :deployment_target => '8.0'
When I changed the '8.0' to '11.1', it worked.
Thank you!

On Mar 13, 2020, at 3:19 PM, Jon Shier<EMAIL_ADDRESS>wrote:
Run pod update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Alamofire/Alamofire/issues/3109#issuecomment-598953603, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCKGU56FFOCZAFGKN2ENC3RHKWPVANCNFSM4LHPCI3A.

Ah, interesting, I didn't realize that setting could affect installed versions. In any case, Alamofire 5 now requires at least iOS 10, FYI.


Various changelog updates

@axelson do you plan to cut a release anytime soon? I'd like to see elixir_sense updated before that happens - there's only one small bugfix there.

@lukaszsamson yes, I think we're well overdue for a release. I'll try to cut a release later this week.


React Group Project: Setup Kanban Board 6-3-23

https://github.com/eric2umeh/space-travellers-hub
There are two people on the team @lugard1 and @eric2umeh

Pull Request Approved ✅ 🎊 🎉
Hello Team! 👋
Things to Appreciate

[ ] Correct Flow ✅
[ ] Descriptive Pull Request and Summary 👌

Your project is complete!  There is nothing else to say other than it's time to merge it.
Congratulations!!! 🎉 🥳 👏 🙌 🎊 ☑

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Conditional tags are converted to conditional comments (nuget version 0.9.9)

Hello,
when conditional tags are parsed, they are converted to conditional comments.
var source = "<![if IE]> if-IE comment revealed <![endif]>";
var doc = parser.Parse(source);
var out = doc.ToHtml();
out == " if-IE comment revealed ";

see About conditional comments
I do not think this should happen.
Alexander.

Hi Florian,
okay now I understand the source of my problem better. I did not expect that AngleSharp actually evaluate these tags. When I look in the DOM of such a parsed page in IE I can see the observed changes there, too.
My problem is that I use AngleSharp to prepate HTML for editing in an email client.
And especially when replaying to outlook mails there is a lot of these crappy code in the source mail...
When these comments are pre-processed by AngleSharp, the wpf webbrowser component and my wpf html editor control actually displays the modified comment tags (converts  to )
Now I solved this by "escaping" <![if]> before parsing with AngleSharp as  which will not be touched by AngleSharp at all. And simply remove --~~~~ and ~~~~-- after .ToHtml().

Hi Alexander, potentially a more reliable and faster way would be to let AngleSharp parse it and use a tree walker to remove these nodes. Alternatively, if you care most about the ToHtml method you could provide our own formatter, which could omit these tags / information. It's quite easy to detect as it is always startin with a comment tag and ending with a comment tag. The order is fixed making things straight forward. Big advantage of this process is that you could also easily adjust for other conditionals (e.g., certain versions of IE being matched, or Outlook specific comments such as  [if gte mso 11]). As a basis you would use the standard HTML formatter - you would just have your own comment serializer (to identify a start / end conditional) and your own tag function (to surpress serialization if a certain flag was set from the comment function).

Hi, I'm running into this same issue. Unfortunately, if the document forces IE compatibility to an old version you actually get incorrect display in IE.
The code below illustrates the problem (you need to save the result in a file and open in IE). I'll try the recommendation to remove the nodes through the tree walker, and will post the code for that:
            var html = "<meta http-equiv=\"X-UA-Compatible\" content=\"IE=5\"><![if !supportLists]>Hello World<![endif]>";             var parser = new HtmlParser();             var document = parser.Parse(html);             document.DocumentElement.OuterHtml.Should().Be(html);

Sharing the code that I used to delete the conditional comments... thanks
document.Descendents<IComment>().Where(x => x.NodeValue.StartsWith("[")).ForEach(x => x.Remove());


Add nativeTypeAttributes option

Introduce a new nativeTypeAttributes option that automatically converts numbers and booleans in attribute values (similar to the existing nativeType option for text content).

@nashwaan Can you please have a look?

see also https://github.com/nashwaan/xml-js/issues/64


not working

Hi,
Was very happy that I found your utility, but it does not work on YII2...

Hi. I will take a look and sort it out.
I'll contact you when updated.
Thanks,
Jon
On Sun, May 10, 2015 at 5:19 AM, Ton<EMAIL_ADDRESS>wrote:

Hi,
Was very happy that I found your utility, but it does not work on YII2...
—
Reply to this email directly or view it on GitHub
https://github.com/c006/yii2-migration-utility/issues/9.

My apologies for the delay.
This was written for Yii2 specifically.
I have tested the migration utility and it's working for me, can you please explain where you are having issues.


Wireframe & Low-fi

[ ] Landing page
[ ] Login & sign up page
[ ] upload history
[ ] Dashboard, Profile, setting, gallery, upload history
[ ] Help center
[ ] Usage Metrics
[ ] FAQ


Seek-Preview

I want to add something like YouTube has for quite some time now, I want you to add a Seek-Preview for me like on YouTube, because I am recently quite interested in dtube project but the player pisses me so... here is my thing to ask if it is possible:
We have this Preview Image, Prerendered: https://ipfs.io/ipfs/QmaJNsdbT2jARQYSmNJuNhCQucLiTmWGookzCFjjSpzpLq
and this is the thing I want:

thanks in advance.
Sorry for my English.

1up for this , without the preview , every alternative youtube player is automatically worthless , sorry but thats the truth

I’ll still sleep at night don’t worry. Feel free to get a pull request in...

looks like it would be possible to add something into controls.js in updateSeekTooltip(event) to also look up a thumbnail image for the relevant timestamp and show it while scrubbing on the seek bar but it's not as trivial/easy as just 'add some hover-over iamges':

can we query those images from youTube?
can we query those images from Vimeo?
in all cases what granularity of thumbnails do we need (eg is it one every second, or one every n% of the length of the content)
how would they be generated/named (ffmpeg for the first, number the naming convention is going to tie back to how they're used/referenced)?

There's already a PR: https://github.com/sampotts/plyr/pull/1253
I was hoping for a little testing/feedback from anyone before putting the finishing touches on it
In answer to your questions:

Thumb file naming: I've gone with the VTT spec for thumbs, same as a few other implementations out there, so that people can use any interval + naming convention that they want
Granularity: One every n seconds. Different intervals depending on video length. My personal preference is:

1hr: 10 second interval

20m: 5 second interval

5m: 2 second interval

< 5m: 1 second interval

@jamesoflol - quick look at the implementation and it's really nice! using vtt makes it more flexible (I've done an implementation in the past that embed the tc into the filename which was messy!)
quick play on macOS browsers (current Chrome, Firefox, Safari) I didn't see any issues.
Agree sprite images would be nice (though initial load might be slower, so would need to preload/pre-cache the image.
Is there any downside/issue to only supplying one vtt file have having it used for all quality sources?

Thanks for that. I've actually implemented jpeg sprites today, will commit new code soon. It's much more ideal with sprites. Better with high latency, safer with http1-only browsers, etc.
Sprites shouldn't hurt load time in any significant way? No images are downloaded until the user hovers seek bar. It means that implementers will want to choose a tile size that keeps each sprite down to under a few hundred KB. (There's also a CPU performance ramification in going too large with a sprite sheet, as the browser has to print the entire thing, not just the individual frame.)

How do you propose putting more than one quality source into a single VTT file?

How do you propose putting more than one quality source into a single VTT file?

was thinking rather than a vtt for each quality, just allow a single one for if implementor just supplies
src: [
'https://dev.videohop.com.au/s3/v/bkxvj83ae3k-hls/thumbs/thumbs.vtt',
]
it would use the same thumbs for 90p, 240p, and 1080p (some implementors might want to minimise thumbnail generation and reuse the same for each) ...
another random thought... does this work with yT and Vimeo thumbs as well, or just ones for a locally hosted asset?

Oh I see. Yeah, one single thumb quality is fine, and I guess may actually be the most common use - somewhere between 80 and 160px.
I'm not writing this to work with Youtube or Vimeo. They don't provide a supported interface that I know of, and as such I expect that their image infrastructure/locations/specifications will change without notice. I imagine that you could have Youtube hosting your video, and host the thumbnails yourself. Though I guess that kinda defeats the purpose of using youtube...

I imagine that you could have Youtube hosting your video, and host the thumbnails yourself. Though I guess that kinda defeats the purpose of using youtube...

had a bit of a dig around and can't find an API or official way that'll get the info (looks like they create storyboards which are image sprites and lazy load them, but doesn't seem to have any timing info etc ... and as you say, likely they'd change format at any time. I'd say not worry about yT and Vimeo for now (sorry I muddied the waters!)

Any chance that https://github.com/sampotts/plyr/pull/1253 gets merged?

Yep, I just need time to review.

Probably good that you haven't had time yet! The changes for jpeg sprites were a bit more destructive than I thought... Just adding neighbouring-image preloading now.

OK, preloading is implemented, demo updated. It seems to be working very nicely now, very fast. Think I'm happy with it, just need a few more hands testing now.

quick test, looks good in Chrome, Safari, Firefox and Tor Browser on macOS (as expected Tor has some latency but once caught up behaves as expected)
Edge (Win10) thumbnails appear, but ... seeking seems blocked (clicking on the seek bar did not re-position the video as expected) - error in console: SCRIPT5022: Exception thrown and not caught
is.js (1,1). Chrome on Win10 works. Also Chrome on Deepin Linux. Will try and test on R-Pi B+/Chrome later today (but have to find where I put it!)

Good catch! Right you are. I'll look into that. Ergh debugging Edge is awful :)
Thanks!

Ergh debugging Edge is awful

Isn't it?! Same for IE. There's a reason Microsoft have stopped trying to make browsers! 😂

I have confidence in their ability to screw up even a re-skin ;)

it'll be interesting to see how the move to Chromium takes things with Edge (and Brave also giving up on their skin, BTW just also testing in Brave on macOS, works fine).
FWIW the base plyr.io, seek is working in Edge

That's fixed. Seems Edge didn't like me using the array spread operator to convert an array. Replaced with Array.from()

@sampotts
Thanks a lot!

This has been in since 3.5.0 thanks to @jamesoflol 👍

Question about this old topic: What is the appropriate time interval between sprites best suited for Plyr?
Thanks.

Question about this old topic: What is the appropriate time interval between sprites best suited for Plyr?
Thanks.
UPDATE:
Just found it.... I guess I'll leave it here if somebody runs into the same issue
1 sec interval between thumbnails (sprite sheet)
SAMPLE_VTT_THUMBNAIL_SPRITE_FILE

Depends on the length of your video.
Under 5 minutes: 1 second. Under 20 minutes: 2 seconds. Under 1 hour: 5 seconds. Over 1 hour: 10 seconds.
If you set it too short for a long video, you'll have several unwanted effects: The code efficiently precaches the spritesheets before and after the currently previewed time, but if the interval is too short, the user's mouse will likely move faster than the cached images. And itll be wasted bandwidth, because in e.g. a 1-hour video, moving the mouse cursor 1 pixel to the right will usually actually be more than 10 seconds


Bad/redundant methods for Blob

Right now there's a whole bunch of methods for creating a new Blob:
new_with_buffer_source_sequence
new_with_u8_array_sequence
new_with_blob_sequence
new_with_str_sequence
new_with_buffer_source_sequence_and_options
new_with_u8_array_sequence_and_options
new_with_blob_sequence_and_options
new_with_str_sequence_and_options

These multiple methods exist because new Blob accepts multiple different types, and so web-sys generates one method per type.
However, these multiple methods are completely useless, because they all compile down to new Blob on the JS side, and they have the same type on the Rust side. This has caused confusion for users.
I see two options:

Remove them and just have two methods: Blob::new_with_array and Blob::new_with_array_and_options which accept &Array

Keep them but change their type to be more useful. For example, new_with_u8_array_sequence could accept a &[u8], new_with_blob_sequence could accept a &Blob, etc.
Because new Blob accepts 14 different Rust types, that would mean we would need 28 different methods with this approach.

I personally lean toward option 1.

It'd be great if we could automatically fixup methods in the WebIDL generator, it's true that we don't really need quite so many methods all doing the same thing!

Keep them but change their type to be more useful. For example, new_with_u8_array_sequence could accept a &[u8], new_with_blob_sequence could accept a &Blob, etc.

Note that this is not really an option, because all of these still need to be wrapped into an Array - new Blob doesn't accept byte slice / other blob / etc. directly.

@RReverser They could automatically create the Array internally, but yeah I prefer option 1.

I ran into this too. Perhaps at the very least it could be mentioned in the documentation that all the methods are identical.

How does instantiating a blob from u8  array (bytes) work?
I tried doing something similar to this
https://github.com/rustwasm/wasm-bindgen/blob/master/crates/web-sys/tests/wasm/blob.rs
but the new_with_u8... method seems do something weird where bytes over 15 increase the size by 2, so this will fail:
    let bytes = Array::new();
    bytes.push(&1.into());
    bytes.push(&2.into());
    bytes.push(&16.into()); //99 also gives 4 but 100 gives 5.

    let blob = Blob::new_with_u8_array_sequence(&bytes.into()).unwrap();
    assert_eq!(blob.size(), 3.0); //size is actually 4

giving a size of 4 instead...
I also tried using _with_options variant and passing some MIME types but same problem occurs. Using File::new_from_u8... has similar result, tho converting between the two (blob: Blob = some_file.into()) works fine.
As a concrete example, what I'm attempting is to do is convert an image file to bytes vec (no problems here) then back to an image but the blobs come out with ~2.5 times the size and obviously broken.

@foopub You have to convert the bytes into a Uint8Array:
let bytes = Array::new();
bytes.push(&Uint8Array::from(&[...]));

let blob = Blob::new_with_u8_array_sequence(&bytes).unwrap();

This is because new Blob only accepts ArrayBuffer, ArrayBufferView, Blob, or String.
You're trying to push numbers into the Array, which won't work. But Uint8Array is an ArrayBufferView, so it will work.
However, an even better option is to use the gloo_file crate, which has a Blob::new method, so you can just do Blob::new(&[...])

@Pauan thank you for the explanation!


top toast hides the appbar

can i manually lower slightly its position , i can't put on bottom or center cause i have a slider

@thorizer I will add a padding attribute , thanks for your feedback

Good morning  @thorizer sorry for the delay here's the new version with the padding attribute
https://pub.dev/packages/motion_toast

thanks a lot @koukibadr


FEATURE: Support for .env.js file with local environment dependent variables

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature: Add support for .env files which contains local environment variables (different for dev, test, production environments) like e.g. google_analytics_tracking_id or sentry_io_DSN, google_api_key etc. This is very useful feature for instance for docker image build
This approach (usage .env files) is popular in backend frameworks e.g. PHP Laravel. The main idea is to put all local environment dependent variables into single file to easy manage them.
Because index.html file is generated by webpack before typescript is set up the .env file must be written in pure js (so the name is .env.js) - but because body of this file is very simple it is no problem.
Last thing: in repository exist only .env.example.js file and during instalation this file should be copy as .env.js file (which is local dependent so excluded from repository) and edit.

What is the current behavior? (You can also link to an open issue here)

Curently there is no single simple config file where we can put all local environment dependent variables to manage them is not easy especially when the project is large and has many that variables.

What is the new behavior (if this is a feature change)?

To compile project .env.js file must exist (by copy .env.example.js). As default google_analytics_track_id variable inside this file is NULL so in index.html file section with GA script will not appear. In Home component I introduce (as example) how to get access to variables from .env.js file.

Other information:

I tested it for dev, prod and AoT build on MacOs

https://laravel.com/docs/5.4/configuration#environment-configuration

I also want to say that in .env.js file you can also keep API main url (different for production, dev, testing) and urls for other envirnment dependent services used by app.

Hi @SherlockStd @colinskow @nickppa can I know your opinion about this feature? I will gratefull for every comment.

Really nice ! Waiting for the merge ! :)

Hey @gdi2290 , I upgrade a little my solution:

At first running npm start/build if .env.js not exist then is created from .env.example.js (in webpack.common.js) - so project always run - (in previous version build fail if user not create by chand .env.js file). So project is compliable after git clone.

I remove non portable 'cp' command from README.md - now it is no needed any more.

I add 'API_URL' to .env.example.js to suggest user where he should put this kind of environment dependent variable.

I need you to answer 3 questions for more clarity:

So is this env now available on Angular ?
How can we access it ?
How can we modify it's content after prod compilation ?

Thanks in advance

I don't know that I understand the question - currently this is available only on this pull request :P

In src/app/home/home.component.ts and html is example how to get access to env inside controllers. And also in src/index.html is example (on google_analytics) how to get access in "webpack html templates"

The idea is NOT to modify value of this variables AFTER compilation. The idea is to set proper values to each environment BEFORE compliation - and .env.js  will be compiled into dist. If you want to change some environment values you should rebuild application. But if you wanna something more dynamicaly changed you should use other approach (like for example may be read variables from dedicated backend API)

I was talking about our ANgular app on this repo, but the questions 1 & 2 was stupid, I removed them :smile:
It will be nice to be able to, because suppose you want to ship a compiled bundle, it's impossible to reuse it because we can't change env vars

ad 2 Maybe in future I will think about this kind of solution where .env.js file is copy into 'dist' and main.bundle.js file read it. I created current solution because i needed this 'before compilation' .env.js file in many projects that I create using docker. So current solution is also helpful but of course in future we can improve it.

Hi @gdi2290 I would like to add that this pull request is in fact continuation of this one(dockerfile). This feature will allow following flow for deployment process:
git clone/pull ...
cp /path/to/env/file/.env.js ./.env.js
docker build -t angular-starter .
docker run --name angular-starter ...

Where in .env.js file which contains all environment dependent variables like backend_api_url etc.
This is very handy and allow to automate deployment process. Of course may be exist better solution to achieve this goal but for this moment this solution is enough.

@gdi2290 Storing configuration in the environment separate from code is based on The Twelve-Factor App methodology

Reading this and also this I realized that to achieve goal of .env which can be change without recompiling application is to remove webpack index.html generation part because this not allow to achieve this goal.  So I plan to work on that approach and in fact this pull request can be closed @gdi2290 - but please give me somme comment do you are open to support .env.js files which can be changed without app recompilation?

sorry I didn't get to this in time. I'll be adding better env support with something similar


[Service Bus] Message and error message handlers are not registered correctly for session receivers

Uncovered issue where the handlers on session receivers are not instantiated correctly.
This would involve same fix used as part of https://github.com/Azure/azure-sdk-for-js/issues/5541

@ramya0820 Does this issue cover any scenario that is not covered in #5541? If so, please provide details for the same.

@ramya0820 I was not able to replicate the problem of #5541 with session receivers. Please respond to the question in the previous comment as to whether this issue is tracking the same scenario as in #5541 or if there is more to it

I see - the message handlers are to be assigned as soon as the receiver object is created. Is there a reason why we wouldn't want to do that?
Can take a closer look if session receivers do not use lazy instantiation or there's something else about the implementation that isn't surfacing a problem - but ideally the user provided handlers should be assigned/made available on receiver instance immediately?

The discussion at https://github.com/Azure/azure-sdk-for-js/pull/6322#discussion_r350408260 goes over why we need not assign the handlers until the receive() call in the StreamingReceiver class is made.
When it comes to MessageSession class, the same class can be used for batching and streaming cases. So, handlers have to be assigned only when the corresponding receive related call is made.
Also, I cannot replicate the problem reported in #5541 when using session receivers.
Therefore, I am trying to understand what scenario is this issue tracking.

Discussed offline:

In case of #5541, the background retries get triggered when the receiver encounters an error during the receiver setup step. With changes in https://github.com/Azure/azure-sdk-for-js/pull/6322 we concluded that the background retries shouldn't happen during create phase.
Since background retries via onDetached handling is not in for sessions at all - the issue wouldn't exist. Though the setup model is similar in that the init() operation could encounter retryable errors (such as temporary claim negotiation errors).

Closing as registering the handlers for after receiver setup completes is as intended


Does this have any VIRUS?

Avira detects the Client as a virus, and ik this is meant to be a RAT and it should be detected as a virus, but just checking.

There are some suspicious functions included, that's why it's detect by antivirus.


training result cannot be loaded on another machine

I don't have GPU on my local machine, so I rent a google cloud vm with GPU.
I ran a couple of games there, e.g. cartpole.
I copied the result to my local and try to load the model and then do "4. Play against MuZero" on my local. But I got this error when loading the model.
Enter a number to choose a model to load: 0
Traceback (most recent call last):
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 187, in nti
    n = int(s.strip() or "0", 8)
ValueError: invalid literal for int() with base 8: 'rebuild_'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 2289, in next
    tarinfo = self.tarinfo.fromtarfile(self)
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 1095, in fromtarfile
    obj = cls.frombuf(buf, tarfile.encoding, tarfile.errors)
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 1037, in frombuf
    chksum = nti(buf[148:156])
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 189, in nti
    raise InvalidHeaderError("invalid header")
tarfile.InvalidHeaderError: invalid header

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/yao/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 595, in _load
    return legacy_load(f)
  File "/home/yao/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 506, in legacy_load
    with closing(tarfile.open(fileobj=f, mode='r:', format=tarfile.PAX_FORMAT)) as tar, \
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 1591, in open
    return func(name, filemode, fileobj, **kwargs)
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 1621, in taropen
    return cls(name, mode, fileobj, **kwargs)
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 1484, in __init__
    self.firstmember = self.next()
  File "/home/yao/anaconda3/lib/python3.7/tarfile.py", line 2301, in next
    raise ReadError(str(e))
tarfile.ReadError: invalid header

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "muzero.py", line 691, in <module>
    load_model_menu(muzero, game_name)
  File "muzero.py", line 597, in load_model_menu
    checkpoint_path=checkpoint_path, replay_buffer_path=replay_buffer_path,
  File "muzero.py", line 417, in load_model
    self.checkpoint = torch.load(checkpoint_path)
  File "/home/yao/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 426, in load
    return _load(f, map_location, pickle_module, **pickle_load_args)
  File "/home/yao/anaconda3/lib/python3.7/site-packages/torch/serialization.py", line 599, in _load
    raise RuntimeError("{} is a zip archive (did you mean to use torch.jit.load()?)".format(f.name))
RuntimeError: results/cartpole/2021-12-30--00-05-02/model.checkpoint is a zip archive (did you mean to use torch.jit.load()?)

Thanks for any help!

Hello @hairinwind ,
I have successfully trained a model on a machine with gpu and loaded it on a machine without gpu so I don't think this is the problem. However, you should make sure that you have the same versions of pytorch on the two machines used for training and loading.
Hope this helps !


about chinese language

Add language select Refer to awesome I AM.
Add contributing.
Update chinese language readme link.
BTW, in https://github.com/kdeldycke/awesome-iam chinese language was translated into 汉语. I think 中文 is better.
So I update it to 中文 in awesome-falsehood.

Oh that's perfect! Let's merge!

Just made the same changes in awesome-iam: https://github.com/kdeldycke/awesome-iam/commit/5fea6342f09383e7dbd30a238ebdf91e98661a0c


Upstream updates

Coverage increased (+0.05%) to 99.04% when pulling 0b30259bcc785368da7aa8c1addd9e07cdd2d4ec on upstream-updates into fba8d740e85c0246a89e2d08117d57c943099f0e on master.


Create home directory for cassandra user

Create the /home/cassandra directory and make it writable for the
cassandra user.
This will allow files, including the cqlsh history, to be written for
the user without warnings.
Resolves #88

@careykevin Thank you for the contribution. All I need you to do now is to sign our CLA policy here:
https://cla.datastax.com/
and ping me back on this PR when you have.

Thanks @emerkle826 - I have signed the agreement.


Update :uri_query

I had to update ecto to make the project compile for me.

Hi! Thanks for submitting PR. Could you please create an issue and provide more details about encountered problem?

Hi! Thanks for submitting PR. Could you please create an issue and provide more details about encountered problem?

I can not remember why I had to do this. I created the PR weeks ago and my brain RAM is not that large.
I do see here https://hex.pm/packages/uri_query that the current version is over 7 years old so this seems like a good change based on that alone.

Thank you @peaceful-james. I updated few dependencies (including ecto and uri_query) in #80.

Already done in #80


Discussion - typed struct instantiation DSL

The Goal
Say we have a typed struct:
class Foo(TypedStruct):
    bar = int
    bar.default = 1

    baz = str
    baz.default = 'two'

    qux = float
    qux.default = 3.4

We want a class-like syntax for instantiating Foo. Currently we have:
class MyFoo(Foo):
    bar.default = 10
    baz.default = 'twenty'
    qux.default = 30.45

We want to somehow get rid of the .default:
class MyFoo(???):
    bar = 10
    baz = 'twenty'
    qux = 30.45

Primary objectives:

We want to be able to "subclass" the instantiations:class MyOtherFoo(MyFoo):  # or MyFoo.??? or whatever
    bar = 100

Somehow trick the IDEs into suggesting completions of our fields. We want completion of instances, and specifically in PyCharm we want completion for field overrides:

Option 1 - default DSL
This is already implemented in pull request #82 (although it needs some fixing). The idea is that once we define a field, setting it again will be sugar for setting its .default. Then we'll just do:
class MyFoo(Foo):
    bar = 10
    baz = 'twenty'
    qux = 30.45

And the new defaults will be 10, 'twenty' and 30.45.
Drawbacks:

The other Easypeers think this is too much magic.
MyFoo will be a class - so we'll need to use MyFoo() to access its fields.
The type will be MyFoo and not Foo.

Option 2 - the .Instantiate property
class MyFoo(Foo.Instantiate):
    bar = 10
    baz = 'twenty'
    qux = 30.45

Foo.Instantiate will create a new class (TypedStructInstantiation?) which will act as both an object and a type - so we could access the fields and we can subclass it.
Drawbacks:

Field override in PyCharm doesn't work (although field access completion works... go figure)

Option 3 - metaclass kwargs:
class MyFoo(Foo, instantiate=True):
    bar = 10
    baz = 'twenty'
    qux = 30.45

Passing instantiate=True will make MyFoo an instance of Foo instead of a new class.
Drawbacks:

Looks weird and unintuitive.

Option 4 - mediate with another class
class FooInstance(Foo):
    _INSTANTIATE_()

class MyFoo(FooInstance):
    bar = 10
    baz = 'twenty'
    qux = 30.45

_INSTANTIATE_() will be a magic method that makes FooInstance our hybrid object&type.
Drawbacks:

Magic hidden in FooInstance.

Well, I don't think that any syntax that is not compatible with base type system is viable.
There is subclassing method in python, which is option 1, so I don't think that it should be anything but first option (and not supporting it is my primary complaint)
In general, I'd say it worth to take step back, and review(again?) what we need to support python 3.7
As there is built in support for data classes
And enforcing of values probably can be implemented on top of it.
And I don't understand why drawbacks of option 1 are drawbacks
Both are exactly how python classes work. As long as isinstance(Foo, MyFoo()) is True

im with @tigrawap on this. although I dont think the move to 3.7 will happen in the near future.

All options seem awkward and artificial to me. Ultimately TypedStruct is a type, and should be used to create struct objects that are populated with appropriate values. Using the .default is abuse. Either we use it as intended (CONFIG = Foo(p1=1, p2="bla")), or we use something else.

It is possible (@koreno's sugestion) to manually add an __init__ to these classes:
class Foo(TypedStruct):
    bar = int
    bar.default = 1

    baz = str
    baz.default = 'two'

    qux = float
    qux.default = 3.4

    @some_magic_decorator_so_we_wont_have_to_repeat_the_var_names_three_times
    def __init__(self, bar, baz, qux):
        pass

That way PyCharm could provide completion for constructors.

And then it would be
BIG_FOO = Foo(bar='three') LITTLE_FOO = Foo(bar='one', qux=0.1)
I like it.

ill just put this here: https://jsonnet.org/

ill just put this here: https://jsonnet.org/

How is it going to help? It looks like a templating engine, not something that can define a schema...

How about using JSON schema?

Builtin support in PyCharm.
Other editors (Sublime/Vim/whatever) can support it with LSP.
Can be used with YAML files.
Can autocomplete both field names and values from a closed list.

The downsides are that we'll need some manual intervention:

Generate the schema file every time we change the configuration.
Each developer will need to configure their editor to link the schema file to the directory of the iogen config YAMLs.


Production build of default create-react-app displays blank white page

Describe the bug
npx create-react-app test
cd test
yarn build

Opening the output (build/index.html) results in a blank white page, as the App.css (and possibly more) is missing from the bundled CSS.
I'm serving it at http://localhost.test:3000/graphiql, and using a setting of "homepage": "http://localhost.test:3000/graphiql", in the package.json.
Environment
Environment Info:
current version of create-react-app: 3.4.1
running from /Users/jcw/.nvm/versions/node/v12.18.1/lib/node_modules/create-react-app
System:
OS: macOS 10.15.6
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Binaries:
Node: 12.18.1 - ~/.nvm/versions/node/v12.18.1/bin/node
Yarn: 1.22.4 - ~/.nvm/versions/node/v12.18.1/bin/yarn
npm: 6.14.5 - ~/.nvm/versions/node/v12.18.1/bin/npm
Browsers:
Chrome: 85.0.4183.102
Firefox: Not Found
Safari: 13.1.2
npmPackages:
react: ^16.13.1 => 16.13.1
react-dom: ^16.13.1 => 16.13.1
react-scripts: 3.4.3 => 3.4.3
npmGlobalPackages:
create-react-app: 3.4.1
Steps to reproduce

npx create-react-app test
cd test
Add "homepage": "http://localhost.test:3000/graphiql", to the package.json
yarn build
Host at http://localhost.test:3000/graphiql and view in Chrome.

Expected behavior
I expect to see the same thing as yarn start, the spinning logo.
Actual behavior
A blank white page is displayed.

zomg - it was my nginx hosting of the static files, it was serving css files as text/plain, I needed to add include /etc/nginx/mime.types; to my config... nevermind!


chore(deps): update renovate/renovate docker tag to v37.59.8

This PR contains the following updates:

Package
Update
Change

renovate/renovate
minor
37.56.1 -> 37.59.8

Release Notes

renovatebot/renovate (renovate/renovate)
v37.59.8
Compare Source
Bug Fixes

github-actions: allow subdirectories by default (#25813) (da0a7e1)

v37.59.7
Compare Source
Documentation

reading-list: add custom datasources to the advanced reading list (#25803) (43fcfe3)

Build System

deps: update dependency @cdktf/hcl2json to v0.19.1 (#25806) (2f64809)

v37.59.6
Compare Source
Build System

deps: update dependency toml-eslint-parser to v0.9.3 (#25802) (64c694c)

v37.59.4
Compare Source
Bug Fixes

galaxy-collection: return sourceUrl of highest_version in getReleases() (#25769) (786f4e8)

Miscellaneous Chores

deps: update dependency @types/traverse to v0.6.35 (#25783) (8cc610a)
deps: update dependency @types/validate-npm-package-name to v4.0.2 (#25784) (55a2619)

v37.59.2
Compare Source
Miscellaneous Chores

deps: update dependency @types/linkify-markdown to v1.0.3 (#25763) (c02320c)
deps: update dependency @types/luxon to v3.3.4 (#25764) (8c2cc2f)

Build System

deps: update dependency @types/ms to v0.7.34 (#25762) (771a7e5)

v37.59.0
Compare Source
Features

pipenv: add support for auth (#24581) (8401943)

Miscellaneous Chores

deps: update dependency @types/ini to v1.3.33 (#25756) (4dbb17e)
deps: update dependency @types/json-dup-key-validator to v1.0.2 (#25757) (07b9405)

v37.57.2
Compare Source
Bug Fixes

template/category: Replaces not working templating field category with categories (#25735) (a9e50fd)

Documentation

categories: fix links in category manager summary block (#25724) (1467e94)

Miscellaneous Chores

deps: update actions/dependency-review-action action to v3.1.3 (#25733) (510fb5f)

v37.57.1
Compare Source
Bug Fixes

pre-commit: Gitlab Custom RegistryUrl not honored (#25682) (c8c0e1b)

v37.57.0
Compare Source
Features

datasource/custom: allow to fetch from YAML data endpoint (#25695) (53401eb)

Documentation

azure: simplify Azure Pipeline example (#25719) (2ce4fc5)

v37.56.4
Compare Source
Bug Fixes

datasource/galaxy: loosen schema (#25712) (e0ade37)

Documentation

update references to renovate/renovate to v37.56.3 (#25715) (0846c1f)

Miscellaneous Chores

deps: lock file maintenance (#25716) (4c409f1)
deps: update dependency @swc/core to v1.3.96 (#25707) (deeab52)
deps: update dependency corepack to v0.23.0 (#25709) (0373661)
rename CommitSha type to LongCommitSha (#25698) (242e278)

Code Refactoring

http: Extract GotTask type (#25713) (19e9e8e)

Tests

http: Disable http retries for tests inside constructor (#25711) (75df23b)

v37.56.3
Compare Source
Bug Fixes

azure: exclude disabled repositories when discovering (#25104) (552dba5)

Documentation

add information about permissions for azure pat (#25534) (fbf8a15)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


KMS Key not found error during dataset import

Describe the bug
On dataset import If a KMS key cannot be read due to permissions error there should be a very clear error saying that datalll does not have permissions to access the KEY and must provide clear instructions how the key policy must be altered. Currently it just says “key not found” which is confusing and misleading. We would like to fix this as soon as possible because this will be a source of problems for us and everyone will keep running into this issue because their KMS key will not be open to the pivotRole.
How to Reproduce
*P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.*

Expected behavior
Provide options such as : 1) Check if the KMS key for the bucket has a key policy defined and if pivot role has permissions to all kms resources in the key policy,
Your project
No response
Screenshots
No response
OS
All
Python version
3.1
AWS data.all version
all
Additional context
No response

Probably ties into https://github.com/awslabs/aws-dataall/issues/614

Yes, Agree with @blitzmohit. We can think of how the keys are imported.

@dlpzx would be good to have some kind of brainstorm/review session how we would like to implement this

Merged and released with v2.1.0 🚀 Although more enhancements are expected for the updates on KMS policies in sharing requests


CI revamp: base everything on Makefile

Haskell test suite instead of an arbitrary Main, which now runs on CI

also adds the Haskell files in the generated website for completeness, c.f. the PR's source branch Github page for the resulting website.

[fix] LaTeX issue with lmroman fonts
Upload PDFs in a .zip artifact on CI builds of PRs (for better reviewing workflow)

c.f. the new item in the Actions page that lets us download the generated PDFs uploaded from this PR's CI build

NB: The more principled approach introduced in this PR will lay out a better foundation for the upcoming conformance testing pipeline.

ACTION REQUIRED
After merge, for the Github page to be deployed correctly, the gh-pages folder must be set to docs/ (in contrast to the previous root folder /) in Settings -> Pages -> Branch.

This looks fine to me, but I don't know enough about CI to review this properly, so I'll defer this to @teodanciu

@WhatisRT there are still a lot of changes unrelated to CI which you might want to take a look at though.
Specifically, anything other than build.yml has to do with building the project and, in particular, how the Haskell package is built and tested.

I just noticed that the section Test the spec using nix-shell of the README will be outdated by this PR, could you update that as well?


php artisan serve can't bind to any port

Laravel Version: 8.0.1
PHP Version: 7.4.10
OS: Windows 10
Database Driver & Version: no database needed

Description:
After upgrading to laravel 8, and running the command php artisan serve, it responds with an error recursively, like so:

This behavior does not happen if i change php version to 7.4.5 or switch to laravel 7.x.
Steps To Reproduce:

have a laravel project started up with version 8.x
make sure you have the most recent stable php version installed (at time of writing 7.4.10)
run php artisan serve
see that the command can't bind itself to any port

As additional note, i debugged the generated command (running php artisan serve), and copy paste it directly in the command line and it worked, without returning any error.
If you need any more info, just ask.
Thanks

We don't use windows ourselves so feel free to send in a PR which fixes this on Windows.

Can't reproduce this. Might be something on your machine interfering.

Hi, thanks for your feedback.
I managed to resolve this issue. Not quite sure what as the cause, but it was related with something not right in php.ini.
So what i did was use the standard php.ini-development (that comes with a pre built php), and add whats missing in the file and should work. In any doubt try only renaming php.ini-development to php.ini without modifying its contents.

The one line I needed to change in my php.ini to fix this problem was
variables_order = "EGPCS"
to
variables_order = "GPCS"

Something to do with the Env variable. No ideia how this affects the core functionality of php built in server to this degree.

It's in the 8^ laravel upgrade. Becaouse the server if one runn it directly works fine:
php -S <IP_ADDRESS>:8000  -t public/ server.php

and in Laravel 7 artisan serve works without problems (windows 10, php 7.4.6).

For me, it was silly mistake. I have installed php in new machine but php.ini was missing. So, I have created php.ini file from php.ini-production file and then php artisan serve command worked fine as expected.

php.ini was missing .Check whether your composer has installed.

After upgrading php7 to php8 for XAMPP,I receive this error.re-creating php.ini from php.ini-production works for me.

Hi guys, I created a merge request https://github.com/laravel/framework/pull/40819 to fix the issue.  Could you try it out to see if it's working?

The one line I needed to change in my php.ini to fix this problem was variables_order = "EGPCS" to variables_order = "GPCS"

i have been with this problem for days and your solution worked thank

injimjin ihkhh uhihkiuhhhu

The one line I needed to change in my php.ini to fix this problem was variables_order = "EGPCS" to variables_order = "GPCS"

Thanks！This is a very good

Yes, it was php.ini
change below
variables_order = "EGPCS"
to
variables_order = "GPCS"

variables_order = "EGPCS"
to
variables_order = "GPCS"

If you are using laravel herd and facing same issue and can't find the php.ini just right click the herd icon from the taskbar and open the  configuration files you will see all your php version folders, click onto the one of the folder as per your configuration of php and search for php.ini and make the above changes.

In case anyone using PVM on windows comes here looking: I installed a new version of PHP using PVM, but it didn't come with a php.ini file. That was causing the issue. To fix it, I just had to add a php.ini file.

if you're using windows open your powershell and type
php --ini
that comand will provide the path to your php.ini file, then go to your laravel app  click on  vendor => laravel => sail => runtime then choose which php version instaed on your machine  copy the php.ini file and post it to the initial php.ini file that will be able to solve the issue on windows OS

For windows PowerShell,
PS D:\xampp\htdocs\testing-project-1> php artisan serve
Failed to listen on <IP_ADDRESS>:8000 (reason: ?)
Failed to listen on <IP_ADDRESS>:8001 (reason: ?)
Failed to listen on <IP_ADDRESS>:8002 (reason: ?)
Failed to listen on <IP_ADDRESS>:8003 (reason: ?)
Failed to listen on <IP_ADDRESS>:8004 (reason: ?)
Failed to listen on <IP_ADDRESS>:8005 (reason: ?)
Failed to listen on <IP_ADDRESS>:8006 (reason: ?)
Failed to listen on <IP_ADDRESS>:8007 (reason: ?)
Failed to listen on <IP_ADDRESS>:8008 (reason: ?)
Failed to listen on <IP_ADDRESS>:8009 (reason: ?)
Failed to listen on <IP_ADDRESS>:8010 (reason: ?)
PS D:\xampp\htdocs\testing-project-1> php --ini
Configuration File (php.ini) Path:
Loaded Configuration File:         C:\Users\User.config\herd\bin\php83\php.ini
Scan for additional .ini files in: (none)
Additional .ini files parsed:      (none)
Step 1:   Copy this = C:\Users\User.config\herd\bin\php83\php.ini
Step 2: Go to File Explorer on your local hardware
Step 3: Paste the copied path
Step 4: Search "variables_order"
Step 5: Change "EGPCS" to "GPCS"
Step 6: php artisan serve
This worked for me in 2024 with php version 8.3.11

si señor, esta es la solucion, muy descriptivo, casi que no encuentro algo funcional, hay varios archivos php.ini en el proyecto Thank you so much.


Condense ProxyStubs to WindowsAppRuntime.ProxyStub.dll

Condensed DynamicDependencyDataStore/DynamicDependencyLifetimeManager.ProxyStub.dll into WindowsAppRuntime.ProxyStub.dll. Other features that need to define their com interfaces for their ProxyStubs will define them here.
TODO: Change the dll paths in the appxmanifest of DataStore/DDLM packages.

I don't understand how this is intended to work. You're coming "all COM proxy/stub" into 1 dll, but we have packaged COM in Main + Singleton + DDLM packages. That's different versioning models, as well as different processes. For instance, the DDLM processes are tiny - they shouldn't be loading Push or other support.
PackagedCOM requires <Application> scope <Extension>s today so that means main packages. Where does 'the' single WAR.ps.dll come into play?
BadMojo(TM).
If you want to advocate for 1 proxy/stub dll per MSIX package we could discuss further. But that's not what's going on here.


⚠ Cioos National CKAN Sitemap has degraded performance

In 1c42a86, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 15179 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 35bd0a1 after 5 minutes.


feat: Accepting IPv4 and IPv6 parameters as command line input arguments

Made changes to params.go to label all parameters with prefix ipaddr as type TypeString
Made changes to ebpf.go operator to convert all parameters with prefix ipaddr_v4 from string to uint32 and
all parameters with prefix ipaddr_v6 from string to struct uint128
Defined uint128 as a struct with 2 members of uint64 and wrote a helper function IPStringToUint128() in helpers.go

Accepting IPv4 and IPv6 arguments as a input parameter
This commit allows us to now specify IPv4 addresses as an input parameter in the cli.
For example:
Let's assume we wanted to have a parameter of ipv4 and ipv6 address. We declare it as a u32 int and u128 in the kernel space as shown below:
volatile const __u32 ipaddr_v4 =<PHONE_NUMBER>;
volatile const u128 ipaddr_v6 =  {0x123456789ABCDEF0, 0x0FEDCBA987654321};

GADGET_PARAM(ipaddr_v4);
GADGET_PARAM(ipaddr_v6);

Then after building the gadget we run it using the following
sudo ig run bit_flip:exp --verify-image=false --ipaddr_v4="<IP_ADDRESS>"  --ipaddr_v6="2001:db8::1" -v

Here the ipaddr_v4 is be converted into a 32 bit integer using the pre-defined helper function gadgets.IPStringToUint32()
And the ipaddr_v6 is converted into uint128 using a custom defined helper function gadgets.IPStringToU128()
There values are logged when we use -v flag:
DEBU[0007] setting param value "ipaddr_v4" = 16777343
DEBU[0007] setting param value "ipaddr_v6" = {2306139568115548160 1}

How to use
The parameters to be accepted as IPv4 must begin with prefix ipaddr_v4
So it can be ipaddr_v4_1 or ipaddr_v4_anything but should have prefix ipaddr_v4
Similar for IPv6 parameters, they need to have prefix ipaddr_v6
Testing done
Locally tested it.

If we really need, we can extend gadget_ip_addr_t with alternative definitions for the addresses, like u64[2], u32[4], etc. for the v6 one and u8[4] for the v4 one.

Thank you so much for the help provided with the approach.
The suggested method of taking IP parameters does seem better. I have made the changes as suggested .
I have also added the default value of the parameter to be "<IP_ADDRESS>" in case the parameter is not specified else by default if would be set to nil resulting in the invalid IP error.
I did test this with basic filtering. The endianess is by default the big endian (network endian)I believe as we don't make any casting anywhere and it works while filtering packets. So we can directly read the address without changing it from network endian to host endian etc...

yup, I have updated the code, please do check again

yup
I'll do that

@MeherRushi I updated this PR to align it with 3c90ec83ae4b017c94eb8e37657547dfbfa15a52. I'll merge it once the CI is green.


⚠ Bofrost.at has degraded performance

In d1227dc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1578 ms

Resolved: Bofrost.at performance has improved in a076c18 after 46 minutes.


Lack of support for notebooks

Kobo Sage and Kobo Elipsa have a new feature: Notebooks.
Currently there is no way to open a notebook or list of notebooks via NickelMenu. There is also no way of adding an item to the notes app menu as far as I can see.
Is that something that is in the radar? Any plans for bringing that functionality in?

Yes, that's planned in #109. I just haven't had enough time to implement it yet.

duplicate of #109

I am willing to help try implementing this though I don't have any experience and cannot reverse engineer myself.

I did a quick read of the source and extracted the latest kobo firmware and found:
_ZN15LibraryNavMixin13showNotebooksE14MainNavSection
I think this may be the relevant section. I can make a pr and try to test

I have tested and that string does indeed launch the notepad. However, notes created cannot be opened for me...

Also, I was doing some more testing and realized that my kobo Libra 2 has full support for my laptop stylus. Pressure support and everything (eraser and highlighter buttons). This may mean that full notebook support can be unlocked for this device as well.

https://github.com/pgaskin/NickelMenu/pull/144


error too many values to unpack

https://github.com/primepake/wav2lip_288x288/blob/12b90f2b77a2aa0fa4019bf96e796fdac3ad0f95/hq_wav2lip_sam_train.py#L310C43-L310C43
Line 310: running_sync_loss, running_l1_loss, running_perceptual_loss = 0., 0., 0., 0.
Hi, is this hq_wav2lip_sam_train.py premature to use? should I try the "not sam" hq version first? -oyiyi

Replace line running_sync_loss, running_l1_loss, running_perceptual_loss = 0., 0., 0., 0.
with
running_sync_loss, running_l1_loss, running_perceptual_loss = 0., 0., 0.
will resolve the issue.


What's the true memory cost of a signal through arena allocation?

@gbj It would be useful to have documented some perspective of the cost of an individual signal against memory (RAM/CPU caches).
Since it seems that arena allocation is a bit like 'heap'?
Ex.  a semi-complex web app might have 500 Read signals and 500 write signals, what might be the memory cost formula of their lifetime through arena allocation?
I can certainly see that performance gains are driven by doing much of this in memory, and even perhaps CPU caches themselves where a lot of signal state is actually happening through L1, L2, L3 ?
Thanks in advance.

Thanks very much. really helpful.


Fix List Equality

The old recursive definition, while clever, was wasteful.. This definition is not only far, far, far, quicker, but short-circuiting and handles infinite lists.

:boat:


Request body of API-Request is limited in size

I'm currently working on an alternative runner implementation using python and docker. During testing I experienced 403 Forbidden errors when the request body exceeded ~128k. This is easily reached while updating a build with a large trace. Steps to reproduce:

Get a runner token.
RUNNER_TOKEN="..."

Grab a build
curl -X POST -H "Content-Type: application/json" -d "{\"token\": \"$RUNNER_TOKEN\"}" http://localhost:1234/gitlab-ci/api/v1/builds/register.json
{"id":812,"commands":"ls -la\r\npwd","ref":"master","sha":"...","project_id":2,"repo_url":"...","before_sha":"..."}

I got build 812, you number will vary.

PUT a valid update (optional, just an example how it should work)
curl -X PUT -H "Content-Type: application/json" -d "{\"token\": \"$RUNNER_TOKEN\", \"state\": \"running\", \"trace\": \"foobar\n\"}" http://localhost:1234/gitlab-ci/api/v1/builds/812.json

PUT a big update
echo -n "{\"token\": \"$RUNNER_TOKEN\", \"state\": \"running\", \"trace\": \"" > long_request
for i in {1..20000}; do echo -n "aaaaaaaaaa" >> long_request; done
echo -n "\n\"}" >> long_request
curl -X PUT -H "Expect:" -H "Content-Type: application/json" -d @long_request http://localhost:1234/gitlab-ci/api/v1/builds/812.json
{"message":"403 Forbidden"}

The Expect header suppresses a Expect: 100-continue header that rails / rack cannot handle.

I did a bit of debugging and found that in lib/api/helper.rb line 17 params seems to be empty, therefore it cannot load the current runner and fails. I have no clue why the request body is not deserialized.
That's all I found out so far, and would really appreciate ideas and help finding a solution other than shorter traces.

Thanks for the issue report but we only support issues for the latest stable version of GitLab. I'm closing this issue but if you still experience this problem in the latest stable version, please open a new issue (but also reference the old issue(s)). Make sure to also include the necessary debugging information conforming to the issue tracker guidelines found in our contributing guidelines.


ViT image classification example broken with error: cannot import name 'is_fairscale_available' from 'transformers.integrations'

System Info
optimum                       1.16.2
optimum-neuron                0.0.13
transformers                  4.37.2
transformers-neuronx          0.8.268
accelerate                    0.23.0
aws-neuronx-runtime-discovery 2.9
libneuronxla                  0.5.538
neuronx-cc                    <IP_ADDRESS>+c5231f848
neuronx-distributed           0.5.0
neuronx-hwm                   <IP_ADDRESS>+e34678757
optimum-neuron                0.0.13
torch-neuronx                 <IP_ADDRESS>.12.0
torch-xla                     1.13.1+torchneuronc
transformers-neuronx          0.8.268
*Note: Neuron version is 2.16.1 public release

Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)

Launch Inf2.8xl EC2 with U22.04.
Install Neuron SDK 2.16.1 and PT1.13 per instruction.
Install public version transformers, accelerate, diffusers, scikit-learn, datasets, evaluate
Run script found in this repo with the following command:

torchrun --nproc_per_node 2 run_image_classification.py \
--dataset_name food101 \
--output_dir ./food101_output/ \
--remove_unused_columns False \
--do_train \
--do_eval \
--learning_rate 2e-5 \
--num_train_epochs 1 \
--per_device_train_batch_size 16 \
--seed 100 \
--overwrite_output_dir \
--logging_steps 10

Observe output:

*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1364, in _get_module
Traceback (most recent call last):
  File "/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1364, in _get_module
    return importlib.import_module("." + module_name, self.__name__)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return importlib.import_module("." + module_name, self.__name__)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
        return _bootstrap._gcd_import(name[level:], package, level)return _bootstrap._gcd_import(name[level:], package, level)

  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/optimum/neuron/trainers.py", line 31, in <module>
  File "/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/optimum/neuron/trainers.py", line 31, in <module>
        from transformers.integrations import is_fairscale_availablefrom transformers.integrations import is_fairscale_available

ImportErrorImportError: : cannot import name 'is_fairscale_available' from 'transformers.integrations' (/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/transformers/integrations/__init__.py)cannot import name 'is_fairscale_available' from 'transformers.integrations' (/home/ubuntu/aws_neuron_venv_pytorch/lib/python3.10/site-packages/transformers/integrations/__init__.py)

Expected behavior
Expected this sample script to run without issue.

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Thank you!


Data Grid: Make pinned columns accessible

Area
UI Components
The problem
?
The solution
?
Alternatives and examples
?
Are you a JPMorgan Chase & Co. employee?

[X] I am an employee of JPMorgan Chase & Co.

@maj-stella can you outline the accessibility issues with our current pinned columns implementation? If you don't remember from the top of your head (we removed the story), I can publish a branch with the pinned columns story so you can take a look at it again.


remove all-error-codes.md

Fixes #2333.
Part of #2329
Fixes #2186.
Fixes #891
This PR includes PRs #2330 and #2332 and #2384 and should be merged after them (or instead of them).
In this PR I remove all-error-codes.md. These codes are already documented in netcdf.h.
@WardF you probably want to get these documentation fixes in before the 4.9.0 release.
(Also adding netcdf_filter.h to doxygen build.)

I will take down the pre-cursor PRs, since this PR contains all my documentation changes for 4.9.0...


Blob container context menu Refresh item fails to refresh the view

Storage Explorer Version: 1.13.1
Build Number: 20200512.1
Platform/OS: Windows 10
Architecture: x64
Regression From:  Unknown
Bug Description
The Refresh menu item on the blob container context menu fails to refresh the view. However, clicking the 'More' ellipsis on the ToolBar and selecting Refresh does refresh the view.
Steps to Reproduce

Right-click on a blob container
Left-click the Refresh context menu item
Note that the view fails to refresh
Left-click to select a blob container
Left-click the 'More' ellipsis in the ToolBar
Left-click the Refresh menu item
Note that the view refreshes as expected

Expected Experience
The view representing the blob objects should be refreshed
Actual Experience
The view representing the blob objects is not refreshed
Additional Context
No other context

@InteXX the refresh in the tree view is for refreshing that node. If you want to refresh the view on the right side, you should use the refresh button in the toolbar (as you discovered).

Very good, thank you.


fix specifying nz only

fixes #56

Codecov Report

Merging #57 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
+ Coverage   91.96%   92.03%   +0.06%
==========================================
  Files           4        4
  Lines         610      615       +5
  Branches      134      135       +1
==========================================
+ Hits          561      566       +5
  Misses         32       32
  Partials       17       17

Impacted Files
Coverage Δ

xmitgcm/mds_store.py
92.14% <100%> (+0.1%)
:arrow_up:

Continue to review full report at Codecov.

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


feat(Scaler): Adds CouchDB Scaler

Provide a description of what has been changed

Adds CouchDB Scaler for autoscaling based on Couchdb queries

Checklist

[x] When introducing a new scaler, I agree with the scaling governance policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Relates to #3746
Relates to kedacore/keda-docs#967

/run-e2e couch*

@26tanishabanik any update on this please? we plan to do a release in ~2 weeks.

@26tanishabanik any update on this please? we plan to do a release in ~2 weeks.

Yes, @zroubalik , I will complete it by this weekend, have been busy with work

@26tanishabanik any update on this please? we plan to do a release in ~2 weeks.

Yes, @zroubalik , I will complete it by this weekend, have been busy with work

cool, and no worries, if you are not able to make it, we will ship this in the next release.

We are going to release KEDA v2.9 on Thursday. Do you think you can complete the open work by Wednesday @26tanishabanik?

We are going to release KEDA v2.9 on Thursday. Do you think you can complete the open work by Wednesday @26tanishabanik?

Yes @tomkerkhove , I can complete it by weekend itself

Great, thank you!

@26tanishabanik could you please fix the problems in Static Checks? and also the unit test failed:
=== RUN   TestCouchDBGetMetricSpecForScaling
    couchdb_scaler_test.go:96: Wrong External metric source name: s0-s0-coucdb-animals
    couchdb_scaler_test.go:96: Wrong External metric source name: s1-s1-coucdb-animals

/run-e2e couch*

@26tanishabanik could you please fix the problems in Static Checks? and also the unit test failed:
=== RUN   TestCouchDBGetMetricSpecForScaling
    couchdb_scaler_test.go:96: Wrong External metric source name: s0-s0-coucdb-animals
    couchdb_scaler_test.go:96: Wrong External metric source name: s1-s1-coucdb-animals

The problems in static checks were related to keda version unit test and duplicate lines of code
The unit test cases are failing because of the metric name change, I am checking them

Details

What should I do about the duplicate lines of code?

@26tanishabanik there is still error in unit tests:
 RUN   TestParseCouchDBMetadata
    couchdb_scaler_test.go:80: Expected error but got success
--- FAIL: TestParseCouchDBMetadata (0.00s)

@26tanishabanik also, could you please fix the PR to contain only your commits?

@26tanishabanik also, could you please fix the PR to contain only your commits?

How can I do that @zroubalik

@26tanishabanik to fix the Static Check, please add similar rule for this scaler: https://github.com/kedacore/keda/blob/cbdb405dc3f9936d100f39ee0a1c6c1f98c1c2dd/.golangci.yml

@26tanishabanik to fix the Static Check, please add similar rule for this scaler: https://github.com/kedacore/keda/blob/cbdb405dc3f9936d100f39ee0a1c6c1f98c1c2dd/.golangci.yml

Like this: https://github.com/kedacore/keda/blob/cbdb405dc3f9936d100f39ee0a1c6c1f98c1c2dd/.golangci.yml#L78-L84

@26tanishabanik also, could you please fix the PR to contain only your commits?

How can I do that @zroubalik

@26tanishabanik this could be tricky, sometimes might be better to use this approach:

backup the changes you did
reset your branch: git reset --hard couchdb-scaler This removes all your changes!
update branch: git pull --rebase upstrea main (in case upstream poinst to this repo)
move back your changes to from the backup
commit, push

Details

Let me try that @zroubalik , thank you for the suggestion

@26tanishabanik also, could you please fix the PR to contain only your commits?

How can I do that @zroubalik

@26tanishabanik this could be tricky, sometimes might be better to use this approach:

backup the changes you did
reset your branch: git reset --hard couchdb-scaler This removes all your changes!
update branch: git pull --rebase upstream main (in case upstream points to this repo)
move back your changes from the backup
commit
force push: git push -f origin couchdb-scaler

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git reset --hard couchdb-scaler
HEAD is now at 56cbc982 added dupl fix for mongodb

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git pull --rebase upstream main
fatal: 'upstream' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@26tanishabanik also, could you please fix the PR to contain only your commits?

How can I do that @zroubalik

@26tanishabanik this could be tricky, sometimes might be better to use this approach:

backup the changes you did
reset your branch: git reset --hard couchdb-scaler This removes all your changes!
update branch: git pull --rebase upstream main (in case upstream points to this repo)
move back your changes from the backup
commit
force push: git push -f origin couchdb-scaler

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git reset --hard couchdb-scaler
HEAD is now at 56cbc982 added dupl fix for mongodb

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git pull --rebase upstream main
fatal: 'upstream' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

it depends how you have named your git remotes, see that in:
git remote -v

@26tanishabanik also, could you please fix the PR to contain only your commits?

How can I do that @zroubalik

@26tanishabanik this could be tricky, sometimes might be better to use this approach:

backup the changes you did
reset your branch: git reset --hard couchdb-scaler This removes all your changes!
update branch: git pull --rebase upstream main (in case upstream points to this repo)
move back your changes from the backup
commit
force push: git push -f origin couchdb-scaler

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git reset --hard couchdb-scaler
HEAD is now at 56cbc982 added dupl fix for mongodb

keda on  couchdb-scaler via 🐹 v1.19.1
❯ git pull --rebase upstream main
fatal: 'upstream' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

it depends how you have named your git remotes, see that in: git remote -v

@26tanishabanik feel free to ping me on slack about this

/run-e2e couch*


remove /files directory from app-sync

exclude files to save volume space and massive duplication on DfM.
performance offset testing with front end files/images only is worth it for the extra space on the limited 64GB HYVE VM.

fixed in UNISON container and compose files to set /files directory from host. minimal performance  deficit expected.


Main Monitor Bar Overlapping Side Portrait Monitor

I have two monitors and one of them is in portrait mode. I am running BSPWM. I been having this problem where applications on my side monitor would leave a lot of empty space on the top half of the monitor. I figured out that it was because of polybar. I read that it can be due to the bar overlapping to the side which is why the apps start halfway through the monitor. Only problem is that I can't tell since visually it seems fine.
Both monitors are 2560x1440
Example
Polybar Config

Easy fix by adding override-redirect=true into my music and tray bars.


Silence Wandb visualisation at inference

Wandb visualisation options displayed to users should be switched off.
Also, allow passing options through config file.

Temporary solution can be running:
$ cd catena/local_shape_descriptors
$ wandb off

Based on info here

Also, silence Wandb when running on cluster.


MM-42210 Fix import of private playbooks.

Summary
Fix import not working for unlicenced servers because licencing doesn't allow private playbooks.
Ticket Link
https://mattermost.atlassian.net/browse/MM-42210

@lieut-data Public / Private is the only one checked at creation time. Everything else is checked elsewhere.


cancelTriggerNotifications returns before it's fully finished

Hi,
I noticed that cancelTriggerNotifications sometimes returns before the scheduled notifications are removed from the database. So far I only noticed it on Android.
In the test below, the number of scheduled notification is not always zero after notifee.cancelTriggerNotifications returned.
await notifee.cancelTriggerNotifications()
console.log('number of scheduled notification:', (await notifee.getTriggerNotifications()).length) // not always 0

In our app, we cancel all notifications before scheduling a new one.  The issue cases a race condition. Sometimes the first few notifications we scheduled are removed unexpectedly by the ongoing cancelTriggerNotifications. The example below shows what we are doing.
await notifee.cancelTriggerNotifications()
await notifee.createTriggerNotification(...)  // this notification could be removed

I looked into the native code, do you think this might be the issue that the deletion occurs in the onSuccess callback?
https://github.com/invertase/notifee/blob/685cdc3b068fde5a969ee7387245b6b0f9a2608d/android/src/main/java/app/notifee/core/NotificationManager.java#L477
Many thanks,
Yupeng

This is on 5.7.0 by the way, we can't move to 6+ yet due to the targetSdkVersion 33 requirement.

An aside: compileSdkVersion should pose no difficulty whatsoever, it's literally just pulling in new symbols and that is the only requirement for v6+. You are not required to change targetSdkVersion to 33 yet, unless you want to use the new symbols.
Ah, my mistake, I read "And, to support this feature, the targetSdkVersion must also be increased to 33." as it saying we needed to increase it anyway, got you.

unfortunately it appears to be an async process inside android and there isn't anything we can do about the API not actually meeting it's contract

Ah, that's annoying :/

The "real workaround" (which is a silly phrase, but please allow it) is for Notifee to cover up this platform API deficiency itself by doing something like this:...

Sounds good, the only problem I can potentially see here is that Notifee retrieving the scheduled notifications would potentially trigger https://github.com/invertase/notifee/issues/515 again, yes?
For now, since we're due to have a release go out, we're going to handle it in the JS layer:

iOS:

clear
schedule

Android <33:

clear
poll getTriggerNotificationIds until it's empty
schedule

Android >=33:

clear
dumb wait for x milliseconds (we're going to play with a slow device to work out what might be sufficient)
schedule

The reason for the last one is so we don't retrigger #515 for the people who have upgraded from 12 to 13 (I assume getTriggerNotificationIds needs to decode the parcel in the same way getTriggerNotifications does?).

Could trigger #515 again however since this is all native code, we can correctly handle the exception vs blowing up, via the implementation sketch I posted up on that one. The problem right now is that it is blowing up in the react-native bridge, but it's doing so in a way that could be probed before sending it to the bridge.
So I suppose it would mean two workarounds at once in order to avoid #515. Mobile programming is ridiculously hard sometimes

Hi Mike,
Do you know whether createTriggerNotification has the same issue on android or not?
We check the number of scheduled notifications after creating a new notification. But sometimes the number didn't increase. This morning, we got a sentry message saying 5 were scheduled out of 50.
The sample code is as below.
const notificationsToSchedule  = [...]  // Array of notifications to schedule

for (const notification of notificationsToSchedule) {
     await notifee.createTriggerNotification(...)
}

const scheduledNotifications = await notifee.getTriggerNotifications()
if (scheduledNotifications.length < notificationsToSchedule.length) {
     // report to sentry.
}

Thank you

I am not sure if createTriggerNotification has the same "semi async" behavior or not @Yupeng-li, sorry. In my experience the cancel version of the issue can be detected fairly easily with local reproductions, which leads me to think that if create suffers from it, then you could probably create a local reproduction that stressed the system with a lot of creates, and see the behavior as well.

@mikehardy Thank you for the reply :)  I'll try and see if I can reproduce.

Sorry for the late reply. I did the stress testing, but I couldn't reproduce the issue with createTriggerNotification on my devices.
I tested on

iPhone mini 12 (iOS 16.0)
Samsung galaxy A12 (Android 11)


Draft Partition Failover: Adds the a new handler to failover a write specific partition

Pull Request Template
Description
This is a DRAFT PR. Waiting for backend changes to full end to end validation.
This PR adds a new handler which will allow a single partition to failover for a write operation on a strong consistency account.
Type of change
Please delete options that are not relevant.

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Closing issues
To automatically close an issue: closes #IssueNumber

Closing this PR because of the design change made. Opening a new PR to avoid any confusion around open comments.


Does not work with yarn

What version of starlight are you using?
0.25.0
What version of astro are you using?
4.11.5
What package manager are you using?
yarn
What operating system are you using?
Mac, Arm
What browser are you using?
Chrome
Describe the Bug
Clean install fails with yarn
yarn 4.3.1
yarn create astro --template starlight

With defaults (Typescript, Strict, Install dependencies, Init git) fails with

▲  error Error
▲  error Dependencies failed to install, please run yarn install to install them manually after setup.

yarn install

➤ YN0000: · Yarn 4.3.1
➤ YN0000: ┌ Resolution step
➤ YN0001: │ Error<EMAIL_ADDRESS>isn't supported by any available resolver

Before using the clean project...
I was able to create the project but building it failed when using page find.
This issue seems now have gone.
But the basic problem was the same: It works with npm but not with yarn.
Switching to npm works
rm yarn.lock
npm install

Link to Minimal Reproducible Example
see above - just run yarn create astro.
Participation

[ ] I am willing to submit a pull request for this issue.

This should be fixed in the last version of @astrojs/check. Sorry for the inconvenience. I ultimately think this is a bug in Yarn 4, as previous versions of Yarn were able to install it just fine, and so does npm and pnpm.
I would suggest not using Yarn

Thanks for investigating and fixing @Princesseuh!


Integer sqrt and nth roots

Initiated in  forum: https://forum.crystal-lang.org/t/integer-sqrt-in-stdlib/1839/8
Using Math.sqrt(n).to_i provides inaccurate results once n reaches a certain large size,
rendering this operation useless for numerical algorithms/processing using large numbers.
Including methods for Integer sqrt, and nth roots makes Crystal much more useable for
numerical applications  (which Julia, et al, promote as its claim-to-fame).
require "big"

module IntRoots

  def isqrt(n = self)   # Newton's method for integer sqrt
    raise "ivalid negative integer" if n < 0
    return n if n < 2
    b = n.to_s(2).size
    one = typeof(self).new(1)  # value 1 of type self
    x = one << ((b-1) >> 1) | n >> ((b >> 1) + 1)
    while (t = n // x) < x; x = (x + t) >> 1 end
    x
  end

  def irootn(n)        # Newton's method for integer nth root
    return nil if self < 0 && n.even?
    raise "root n not an Integer >= 2" unless n.is_a?(Int) && n > 1
    return self if self == 0 || (self == -1 && n.odd?)
    num = self.abs
    one = typeof(self).new(1)  # value 1 of type self
    e, x = n-1, one << (num.to_s(2).size-1) // n + 1
    while (t = (e * x + num // x ** e) // n) < x
      x = t
    end
    x *= self < 0 ? -1 : 1
  end
end

struct Int; include IntRoots end

There are multiple implementation details that need to be agreed upon (and documented) to present a consistent name and use API to users, but other than that, the code above provides working implementations that can be a starting point for optimization.

Partially resolved by #10549.


PHOENIX-7428 Add usable error message in BackwardCompatibilityIT

This works, I was able to find the BackwardCompatibilityIT error messages.


remove 0x from pvt keys

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (4e1fc60) 21.57% compared to head (399da59) 21.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #739      +/-   ##
==========================================
- Coverage   21.57%   21.43%   -0.15%
==========================================
  Files          31       31
  Lines        4167     4167
==========================================
- Hits          899      893       -6
- Misses       3177     3182       +5
- Partials       91       92       +1

see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Vulnerability roundup 79: libtiff-4.0.10: 1 advisory

search, files

[ ] CVE-2019-17546 CVSSv3=8.8 (nixos-19.03)

Scanned versions: nixos-19.03: d1dff0bcd9f. May contain false positives.

See also: #73663


allow non-latex intermediaries for pdf generation

fixes #380

Hi @ashi-agrawal
Some CI is failing. Would you be able to take a look at that