Reject many skipped blocks

Motivation

Description

Closes #3988
Steps to test or reproduce

@dapplion responded to your recent comments...if you are fine by them...is it possible to help merge?


add support for crack servers

there would be quite a lot of users if this support was added, since playing without an xbox is quite a good feature in Minecraft.
So my question is, will there be support for logging in without xbox in a future update? I understand that this is risky in the sense that the server does not have authorization

since it is very stable than other economy plugins

#127
It will probably be expensive to rewrite the full plugin and replace xuid with a regular nickname?

I read that you asked the person who created the issues the reasons for not using an xbox account to play with the BedrockEconomy plugin. Some players are simply too lazy to register an xbox account, especially when the player has an Android platform
Related to this, I would be very happy with this update. @cooldogedev

This will be implemented in BedrockEconomy v4.1.0


Install fails if bower not installed globally

mcp-standalone/installer/../ui/install.sh: line 16: bower: command not found"
Looks like bower needs to be installed globally, or probably better still added as a dev-dep, and run via node_modules/bin/bower or something.

Pull request: https://github.com/feedhenry/mcp-standalone/pull/110


Clean up and add tests for extra and serialization

Selected Reviewer: @dmontagu

please review

Hold until https://github.com/pydantic/pydantic-core/pull/537

Hold until pydantic/pydantic-core#537

For what it's worth, I'm good with merging this now if you wanted, I recognize that will enable you to fix some more issues.

Let’s do that then

please review


[FEATURE] - Pagination in findAll for API FOOD

Problem:

The time to request a 'findAll' depends on the download capacity of the server that is consuming the information.

Solution

My suggestion is to add a pagination.

Thanks for raising this.
I'm currently working on a new version where it relies on GraphQL and in a proper database instead .json files.
Now it'll be easier to implement such features

On v2, now you can request X amount of data:


Profile not advancing through Vouching Phase

Describe the Bug
I have been vouched twice more than 4 days ago and still haven´t gonne through to the next phase
My profile: https://app.proofofhumanity.id/profile/0x3976CA473cc683EfEE877ca3d7E5f39452C62A0E
Expected Behavior
Go to the next pahse
Screenshots

Device and Browser Information
I used Chrome with Methamask extension, on Windows 10, although I dont think my browser info has anything to do with the problem, feels like its more of an internal issue

Your profile moved to Pending Registration a few hours ago. Probably the vouches were being used on other profiles or it simply took a while for your profile to move to the next phase.


Exam: Do we have to register with you as well or just appear in the room at that day?

You can just come directly at the exam room the same day. I will then submit the results after correction to the Uni admin, and thus confirm your registration.


Support Neovim "Remote" Plugins

Per Neovim's documentation, (e.g. remote-manifest at https://github.com/neovim/neovim/blob/master/runtime/doc/remote_plugin.txt), neovim-style "remote" plugins will require one extra step after being dropped into the appropriate directories.
Namely, after install/removal/update of a "remote" plugin, the user must execute the command :UpdateRemotePlugins (once, anywhere) to update neovim's manifest of updated plugins.
I can probably get around to submitting a patch myself pretty soon, but if not, to ease the transition for people into neovim (without breaking anything for vim users), I think there should be a default post-install/update/clean hook of
if has('neovim')
  UpdateRemotePlugins
endif

FWIW, this should now read if has('nvim')


[ray.so] Add support for solidity

Extension
https://www.raycast.com/garrett/ray-so
Description
Please add support for Solidity
Should be rather straightforward
Who will benefit from this feature?
No response
Anything else?
No response

For feature requests for ray.so, please use the Send Feedback link in the footer of the page. This repository holds the ray.so extension, which just forwards requests to ray.so, and therefor cannot add support for new languages.


Code runner not able to run python file because it can't find it

This was wierd because i could run the file manually by going to the directory and running python , i have a hotkey for code-runner to run python files and that was the first time i had this issue:

You see, it's right there, hope this is fixable

Pleace use python = "python -u '$dir/$fileName'" in your config


Simplify protocol with regards to optional fields vs nullable fields.

From my understanding, in the protocol, a field marked as "?" is optional. If it's not marked as optional, the key must be present, however, the value can still be null.  Let's take InitializeParams as an example:
interface InitializeParams {
    /**
     * The process Id of the parent process that started
     * the server.
     */
    processId: number;

    /**
     * The rootPath of the workspace. Is null
     * if no folder is open.
     */
    rootPath: string;

    /**
     * User provided initialization options.
     */
    initializationOptions?: any;

    /**
     * The capabilities provided by the client (editor)
     */
    capabilities: ClientCapabilities;
}

We can clearly see that the the initializationOptions key is optional, but rootPath and processId are not. However, rootPath can be null, as the documentation specifies. But what about processId? Can it be null? It's not clear from the spec. I'd imagine it shouldn't be null, if it's not explicitly mentioned in the documentation. Yet I've tried out LSP4J and it sends processId as null... Who is right?
I propose that any field that can be null should be marked as optional (with "?"). Such that effectively, LSP states there is no semantic difference between a field that is missing (no key at all), and a field with key present, but with null value. This would eliminate ambiguities such as the example above.
This semantic distinction is even more important for languages such as Rust which don't have nullable types. (I'm writing an LSP SDK for Rust, and just realized this ambiguity will be a major pain)

@bruno-medeiros  agree with you, the protocol should not make use of undefined values but stick to null.
I won't use LSP4J but VS Code to answer questions about the protocol since it is kind of a reference implementation.
Concerning processId, it can be null, consider cases when the language server is used locally or deployed as a web app. If you have a look at VS Code they also assume that. Another thing why do you actually need processId? Looking at the code you get an answer: the server should track whether the parent process is alive and exit, more to it if shutdownhas been called before it should exit with success otherwise with error.
Unfortunately currently the specification does not reflect effects of input parameters as well as legality and effects of previous calls on a current call's result. The best one can do is to test the server against VS Code and the client against let say the TS server, since the protocol was initially defined for them.

Yeah, I just made processId optional in my implementation, not really fussed about it. It was just the first field I stumbled upon with this ambiguity. The problem is now I'm looking at all the the other fields in the protocol that are not marked as optional, and wondering if they can actually be null or not. 😑
I could test it out with VS Code, but as far as I know there is no simple way to test out a custom LSP language server, at least for someone who is not familiar with Typescript and VS Code extension development  (and I'm not familiar at all with any of those). And even if I test with VS Code - the official LSP implementation - there is the risk of some other LSP implementation interpreting things in a different way.

@bruno-medeiros I think we just have to improve the spec for such properties.
Regarding to testing with VS Code. You can have a look at this simple extension. It is probably a bit outdated but it should be straightforward to upgrade it. It starts a java process with Xtext LS: https://github.com/akosyakov/ride-vsclient/blob/master/src/extension.ts#L18. You can change this line to start a process with your LS.

I will keep it open since there is indeed a mismatch between properties being null and undefined ones. We should make this better. In general I tend now to not make any properties optional (undefined) since undefined is very JS specific and the protocol is for more than one implementation language

Yeah, another example where it's not clear a property is optional or not is TextDocumentItem's languageId (or even version). There are sure to be others.
At the moment, I'm erroring on the cautional side, and try only to require a property if is strictly necessary for the underlying operation.

Somewhat related to this: all fields of ServerCapabilities are optional. But the ServerCapabilities key for InitializeResult.capabilities is non-optional. What is the value then of a ServerCapabilities with no fields that the protocol allows me to construct InitializeResult.capabilities?
And although it may be tempting to try fix this problem by allowing null, e.g.,
interface InitializeResult {
	/**
	 * The capabilities the language server provides.
	 */
	capabilities: ServerCapabilities | null;
}

this still implies that a ServerCapabilities with no fields is allowed, even though we can't construct such a value in json.
On a practical level, it appears desirable to be able to send a response where none of the ServerCapabilities are set (e.g., if the server only ever publishes diagnostic messages). Perhaps permitting a null value is a workaround.

Think about this again I am not sure if making all optional properties null is the right thing to do. We use JSON RPC and having a missing property in there is OK. The corresponding clients and server implemented in a different programming can then decide how they best map a missing attribute to their programing language.
There is on exception to this: error and result values send in the JSON-RPC responses can not be missing by JSON RPC definition. I made a pass a while ago to clearly specify where null is a valid result or error value. I used the TS or type to make this clear even for properties (e.g string | null).
I will close the issue since I think what we have in terms of the specification is clear enough right now (Please note that some properties instead of being optional are declared using  | null. This was done for properties that we required in 1.0 but it did make sense to relax this).


ParamReturnAndVarTagMalformsFixer not working as expected within method closures

Currently I got this
19) src/Security/ApiKeyAuthenticator.php

    ---------- begin diff ----------
--- Original
+++ New
@@ -126,13 +126,7 @@
      */
     private function getProvider(ChainUserProvider $userProvider): ApiKeyUserProvider
     {
-        /**
-         * Lambda function to filter user providers
-         *
-         * @param UserProviderInterface $userProvider
-         *
-         * @return bool
-         */
+        /** Lambda function to filter user providers * * @param UserProviderInterface $userProvider * * @return bool */
         $filter = static function (UserProviderInterface $userProvider): bool {
             return $userProvider instanceof ApiKeyUserProvider;
         };

    ----------- end diff -----------

Applied checkers:

 - Symplify\CodingStandard\Fixer\Commenting\ParamReturnAndVarTagMalformsFixer

And I don't really see the point of this change.

Looked in which version this is found;
5.2.8 = ok
5.2.9 = error
and all after that also error

That's definitelly a bug. Thanks for reporting!

Looked in which version this is found;

Excelent, that helps a lot :+1:


remove --nomigrations from setup, add gem test base

@codiebeulaine what does removing --nomigrations do for the tests?

@nathanbegbie it runs the test without migrations.


iOS 10 support?

I've got the same code running on a ios10 and ios9 device and notifications don't show up in the notification area in iOS 10.  Is this not supported or is it possible a bug in iOS 10?

This plugin comes from a fork, and a PR has been sent to the initial version to fix this issue.
You can check it out here : https://github.com/katzer/cordova-plugin-local-notifications/pull/1093

Nevermind... Not possible with Telerik, as they do not accept "10.0" as iOS target version. So it will not compile.

Any progress on getting iOS 10 support added?  This has broken an important feature for iOS 10 users.
0.8.4 installs fine on iOS 10 and accepts schedule requests, but never delivers any notifications.  The katzer ios 10 branch mentioned above will not build, so we are stuck for the moment.

That fork is for adding the new iOS 10 way of adding notifications to the plugin, however the current way should work just fine as it's only deprecated and will probably work fine for a few major iOS releases to come (as with all things Apple deprecated, luckily).
So there must be something else going on and I suspect it to be the repeat interval logic as that's what I had to fix on my NativeScript plugin recently as well.

Fixed in <IP_ADDRESS>.

Eddy - Thanks for the quick response - that did the trick!


feat: [CDS-80530]: Document rollback behaviour for traditional deployments

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cd479b017c59209e1e16a7--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda0dcecf95f50f3347055--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda0f51d0ba64ed904c39f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda0f57769f2667f0d12d1--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda0f7fbdf026c3c583b40--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda105555a086960a57803--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cda10cb0b0eb6ad44fab1f--harness-developer.netlify.app


Update README.md

Corrects totally erroneous lack of basic grammar.

I am deeply relieved that someone has caught this issue.

I don't believe that piper is a proper noun here. If you can provide some research, maybe look into the history of the story and find out if piper was the individual's name, it can be merged in.

When most people think of fairy tales, they typically conjure up images of idealized Prince Charmings and magical hair-braiding birds. However, if you've ever indulged a passing fancy of childhood nostalgia by picking up a copy of the Grimm brothers' original tales, you might have been startled by the contents inside. These stories are far from the bright and shiny, happily-ever-after ones your parents read to you as a child or that you saw in classic Disney movies. They can be exceptionally dark -- and as grim as their namesakes.
The Grimm brothers started collecting centuries-old, German folklore in the 19th century. Their interests in philology and linguistics drew them to write down the fascinating and culturally revealing stories that, up to that time, spread chiefly through oral storytelling. In that way, you could say that the Grimm brothers didn't set these tales down to paper for the particular purpose of entertaining children. And, although the tales themselves revolve around magical events and carry morals, many weren't geared specifically toward children as much as they were for the Germanic culture as a whole. However, the stories became popular among children, so they were toned down for that audience.
But the original, unbowdlerized Grimm stories still carry relevance. Not only are they fascinating peeks into the psyche of the German people of the Middle Ages, but certain stories actually help us make sense of mysterious and bizarre historical events.
This thought might be unsettling. Indeed, the Grimm stories are so dark that the suggestion one might carry a grain of truth is horrifying. However, in the case of the "Pied Piper of Hamelin," this seems to be the case.
Before we get into the theories and historical evidence behind the story, let's go over the story in detail. And we wouldn't suggest snuggling into bed and expecting to be lulled to sleep.
For those who thought the simplistic morals of Grimm tales were just for kids, "The Pied Piper of Hamelin" will probably convince you otherwise. In addition to its dark content, this is a cautionary tale about governance as well as taking responsibility for financial agreements.
The story takes place in 1284 in the Germanic town of Hamelin (now known as Hameln). At this time, the town was suffering from a severe rat infestation. The town and its mayor were at a loss when a stranger came into town. He wore slightly bizarre, pied (multicolored) clothing and carried a musical pipe. He told the people he'd heard about their rat problem and that he could rid the rats from the town -- that is, in exchange for a fee.
After agreeing to the sum, the piper took out his instrument and started playing a ditty. His song magically lured the rats to him: They came running and congregated around him. After he was convinced that he'd attracted all of the town's rats, he walked toward the nearby Weser River. There, with the rats following him all the way, he walked right into the water. The entranced rats continued blindly and dove right in, where they all drowned.
The townspeople celebrated, but they refused to pay the piper what they promised. He left, furious at the villagers and planning retribution. He got his revenge on the fateful date of June 26, which was both Saint John Day and Saint Paul Day. Dressed in hunter's attire and a red hat, the piper returned to the town, playing another song on his pipe. When the town's children heard the tune, they all flocked to him, in a similar trance as the rats. The children followed the piper out of the town and to a mountain cave, where they were never heard from again. Interestingly, the Grimm story makes a note that the mayor's grown daughter was among the group of children, which readers might infer was an act of revenge aimed directly at the mayor.
The Grimm brothers go on to say that the townspeople put an inscription on the town hall to commemorate what happened.
Evidence of the Real Pied Piper
We'd all love to dismiss this story as mere fairy tale and pure fiction. However, scholars generally agree that something horrible happened in the town of Hamelin to spawn the story. Though there might not have been a piper with magical musical talents, we can safely assume that some tragedy ensued. And, as you might expect, the scholars are basing this on more evidence than just the Grimm brothers' story itself.
Supposedly, the people put up a stained glass window in their church around 1300 that depicted a group of children and a motley-clad fellow. On the glass was an inscription that read, "On the day of John and Paul 130 children in Hamelin went to Calvary and were brought through all kinds of danger to the Koppen mountain and lost" [source: Zipes]. Interestingly, this inscription didn't mention anything about a piper. Accounts of the window have been found, but the window itself seems to have been destroyed.
In what's known as the Lüneburg Manuscript, which was written more than a century after the window is thought to have been constructed, a monk by the name of Heinrich of Herferd gives an account of what happened. He wrote that a man who was about 30 years old came to the town playing a flute and led the children out [source: Zipes]. By 1603, the town erected the façade of what's known now as the Pied Piper House. On the façade, there's an inscription similar to the one that was reputedly etched on the window -- but this one explicitly mentions a pied piper.
But the suggestion of rats having anything to do with the tragedy didn't come until later. It's not too far-fetched that rats would be connected to the incident; rat infestations were certainly a problem around the 13th century and ratcatcher was a common profession.
So what do historians really think could have happened on that fateful day?
Theories About the Origin of the Pied Piper
The townspeople of Hamelin -- and particularly the children -- were probably stricken by a horrible event in the 13th century. One of the most common theories about what happened was that the town's youth suffered from some epidemic. Historians suggest that if a widespread epidemic did occur, the townspeople would have buried the children in a large common grave, which the story depicts as the site of their disappearance [source: Dirckx].
The nature of this disease is disputed. Because rats are featured in the story, scholars think it might have been a disease that spread from rodents, like the bubonic plague. Some even suggest an early version of the Black Death infected them. The piper's pied clothing might have evolved from the idea that the visitor had splotchy skin lesions brought on by the disease. Another theory states that the dancing children in the story may have been exhibiting symptoms similar to those of Huntington's disease.
But if the young people were all infected, one can reasonably question why no adults were. One theory states that the children weren't afflicted with disease but rather struck out on an ill-fated children's crusade. If this is true, it probably happened a few decades earlier when groups of young people around Europe were known to participate in the Crusades. They would leave to follow one child who claimed to have a vision from God that ordered them to march to the Holy Land and win it back for Christendom.
And some people interpret the Grimm story quite differently. They claim that the children lived and came out the other end of the cave in Transylvania. This points to the idea that the children went east to form their own colony. Fairy tale scholar Jack David Zipes substantiates this notion with documents that show evidence that someone came to Hamelin around that time looking for recruits to colonize areas of eastern Europe [source: Zipes].
Whatever happened to the children, it seems to have been traumatizing enough for the people of Hamelin to produce this tale. Even if scholars never uncover the truth, perhaps we'd do well to simply heed the lessons in this cautionary tale.

I'd like to submit a pull request to fix typos in Mark's comment.

We will concede that when speaking of the Pied Piper himself, in so many words, we must treat his full name as a proper noun, and capitalize it as Mr. Mark "mosse" Macdonald has suggested; however, given the context of the situation and the intent to refer to Fantasyland in the most oblique way possible, we feel that when not explicitly Pied, he is no Piper but rather a piper, c.f. Led Zeppelin's allusion in the lyrics to "Stairway to Heaven", as reprinted at:
https://play.google.com/music/preview/Thhet5jtvzdxkti7vc62o2fiajq?lyrics=1&utm_source=google&utm_medium=search&utm_campaign=lyrics&pcampaignid=kp-lyrics
http://www.azlyrics.com/lyrics/ledzeppelin/stairwaytoheaven.html
http://www.songlyrics.com/led-zeppelin/stairway-to-heaven-lyrics/

:+1: @sdwebster

Excuse me???


FLUME-2752 Fix AvroSource startup resource leaks

Cleanup after Netty initialisation failes (call this.stop())
Make sure this.stop() releases the resources and end up the component in
a LifecycleAware.STOPPED state
Added junit tests to cover the invalid host and used port scenarios

Tests executed (and passed):
mvn test -DfailIfNoTests=false -fae -Dtest=TestAvroSource,TestAvroSink,TestDefaultSourceFactory,TestLoadBalancingLog4jAppender,TestLog4jAppender,TestLog4jAppenderWithAvro
(all tests where AvroSource was referenced)

TravisCI failed since it incorrectly used jdk1.7.0_80 for the build. Java target version was bumped to 1.8 some time ago. Please merge #142 first for fixing that.

Thank you for the patch @simonati. I have committed the #142 pull request, could you please push a new commit for this PR to retrigger the Travis CI build? Thanks.

Thanks for updating the patch, LGTM 👍


[Maintenance] Support for lunar

I had to drop lunar builds because of issues with python packages installation.
Let's try it again.
The idea is to use a venv at container build time (that was OK) and use PYTHON_PATH appropriately when running the container

ended up using --break-system-packages on pip, which was the first this I did days ago. But this time I also upgraded pip and hopefully it works for both jammy and lunar.


[Snyk] Upgrade html-snapshots from 2.10.5 to 2.11.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.10.5 to 2.11.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 2 days ago, on 2023-10-21.

Release notes

Package name: html-snapshots

2.11.0 -<EMAIL_ADDRESS>

2.10.5 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

6dedc41 Merge pull request #528 from localnerve/rc-2.11.0
02b97dc @ 2.11.0 -<EMAIL_ADDRESS>ea19f06 Merge pull request #526 from localnerve/doc-2.10.5
4327dba Merge pull request #527 from localnerve/snyk-upgrade-a21fd6b9932adb5bb7f43e596c59e105
8286e02 fix: upgrade html-snapshots from 2.10.4 to 2.10.5
ff7b8f2 fix: upgrade html-snapshots from 2.10.4 to 2.10.5
c4387c2 Merge pull request #525 from localnerve/snyk-upgrade-17ac7d6c13b8a94079a358c72d54507e
caf106f fix: upgrade html-snapshots from 2.10.4 to 2.10.5
46edef5 Merge pull request #524 from localnerve/snyk-upgrade-0a7b65892ef70416c8cc1fc4e6fa0fcc
3e23186 fix: upgrade html-snapshots from 2.10.4 to 2.10.5
b04149a Merge pull request #523 from localnerve/snyk-upgrade-1fdec93bad2c13f63bc448d753afcfa4
16ed4ce fix: upgrade html-snapshots from 2.10.4 to 2.10.5
81461d4 Merge pull request #522 from localnerve/snyk-upgrade-43adb84fd5a11e50ce6384cd14e3a925
b50ea7a fix: upgrade html-snapshots from 2.10.4 to 2.10.5
6706576 Merge pull request #521 from localnerve/snyk-upgrade-79b600170ac23041296f18db7d4d4e5b
21b6f78 fix: upgrade html-snapshots from 2.10.4 to 2.10.5

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same when pulling b96bd4cee1f6a6cb8899926a52efbd62e8bd1225 on snyk-upgrade-21d4ef18d20b7b31fdeee9e45c34b5e1 into d43a2851d531a8c0a3b94775597a184013621e67 on master.


Modify VPX scripts to automatically update VPX when run

Currently when you run the vpx-builder- scripts, you have to then manually configure the VPX from the output. This issue will address that by updating the scripts to remove any existing containers and add the ones output by the script.

Done


fix(@angular-devkit/build-angular): normalize Vite dev-server Windows asset paths

When using the esbuild-based browser application builder with the Vite-based development server on Windows, source asset paths were previously not normalized prior to being included in request URLs. This could result in invalid asset request URLs due to invalid path segment separators.

Does this solve https://github.com/angular/angular-cli/issues/25198?


Unexpected token error in JS files

I am getting this error [see image] in my console. I've added all the dependencies.

Have you actually clicked on those errors to see what they mean? Looks like the path to the script files is wrong.

Yes you are right the paths were wrong but still getting  '$injector:modulerr' error.  We are using angular version 1.4.4.

Click the error in the link and it will give you more info.


[BUG] pip install fails for decord on Mac M1

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
pip install fails on M1 Mac since there's no arm version for decord yet. See this issue.
ERROR: Could not find a version that satisfies the requirement decord (from versions: none)
ERROR: No matching distribution found for decord

期望行为 | Expected Behavior
Installation should proceed successfully.
复现方法 | Steps To Reproduce

Create a new conda environment on M1 Mac
Try to install dependencies using pip install -r requirements.txt
Get the error

运行环境 | Environment
- OS: MacOS Sonoma 14.5
- Python: 3.10
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
This can be resolved by using eva-decord which was forked from the official decord project to specifically support arm macs. See this comment for more context

You can use cv2 from opencv-python to decode video.
cv2.VideoCapture(video_path)


Update examples on CodeSandbox for React

Hello,
First of all, thank you for your excellent work ! This is the best carousel I've ever used. 🔥
Since the 4.0.0 update the previous code doesn't work anymore for arrows & dots for React.
If you could update the CodeSandbox examples it would be awesome!

Hello @RectoVersoDev,
Thank you for the awesome feedback =}. Are you sure? I updated the React CodeSandboxes to use the new ref approach yesterday. Maybe cache in your browser is causing this?
You mean this CodeSandbox right?
https://codesandbox.io/embed/embla-carousel-arrows-dots-react-z5fbs
Kindly,
David

Ah yes... I see the new code now! Sorry for the inconvenience and thank you for the quick answer! I close the issue

Nice to hear that. Thanks and enjoy 🙂!


Keyword is_bc_policy

Hi!
I was trying to go through the training process with SawyerPushObstacle-v0. I met the keyword error when the following line is executed
https://github.com/clvrai/mopa-pd/blob/ac55f568149d8e79c28326bcd9b63336ed065a61/rl/behavioral_cloning_visual.py#L367
where the step function in BaseEnv has no is_bc_policy keyword.
https://github.com/clvrai/mopa-pd/blob/ac55f568149d8e79c28326bcd9b63336ed065a61/env/base.py#L264
Or, the _step method should be called here?
https://github.com/clvrai/mopa-pd/blob/ac55f568149d8e79c28326bcd9b63336ed065a61/env/sawyer/sawyer_push_obstacle.py#L192
Looking forward to your reply, thanks.

Hello!
I don't remember having to change the step function to _step, but after looking at the code, I think replacing that step function call with _step makes sense. Could you try it and see if everything works as intended?
Thanks!

Yes, please change it to the seed function.


Update kl10 density calc to handle nonlinear EOS (#647)

What changes does this PR introduce?
The current implementation of the KL10 mixing parameterisation (which switches on vertical diffusivity when there are density overturns) only calculates density overturns correctly with a linear equation of state, as in-situ density is used. This change instead uses the gradient of iso-neutral density to find overturns in the more general case of a nonlinear equation of state. Closes #647.
What is the current behaviour?
When a linear equation of state is used (as in the internal_wave verification example), the behaviour is as expected. If used with a nonlinear equation of state (tested with JMD95Z), there are unexpected large patches of high diffusivity where stratification is stable.
What is the new behaviour
Overturns are calculated correctly for a nonlinear equation of state (tested with JMD95Z). For a linear equation of state, the results should be the same as the original. In testing on verification/internal_wave.kl10, it was found that the output fields were initially the same as before the change, but after several hundred timesteps the outputs diverged to a different but qualitatively similar state. Differences in the output originated from initially very small (1e-13 kg/m^3) differences in the calculation of density that led to large differences as the small errors propagated through the simulation.
Does this PR introduce a breaking change?
Yes, as the output of the internal_wave.kl10 verification experiment is different. I think that this difference is just due to small numerical differences in the calculation of density that get propagated through the solutions, but I'd be glad if someone could check the change and that this type of difference from the original output is expected.
Running testreport showed differences in the internal_wave.kl10 experiment only, and running do_tst_2+2 produced no differences - the relevant outputs are attached.
Other information:
tr_out_master.txt
tr_out_KL10_bugfix.txt
tst_2+2_out_master.txt
tst_2+2_out_KL10_bugfix.txt
diff_tr_out_master_KL10_bugfix.txt
diff_tst_2+2_out_master_KL10_bugfix.txt
Suggested addition to tag-index

Just a comment regarding size of local array "rhoS" that I change to (0:Nr):
When running the executable compiled with -devel, with a slightly set of params, the model was stopping for an "index below lower bound" in the "DO WHILE" evaluation as JJ reach 0.
So I agree that rhoS(0) is never used, but it's still accessed when evaluating the "DO WHILE" expression (it's value does not matter) and this was preventing to run with -devel, as least in some cases.  It should be fixed now.
I also updated the reference output so I thing this PR is now ready to be merged.
@loisbaker and @jklymak do you agree ?

Looks great, thanks @jm-c


ValidationError should be more expressive than "error.pattern"

We should fill the JSON constraints with more context. E.g. the min length constraint just gives error.minLenght but not the actual value.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4173. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Prevent having a null value on from and to property.

The reason why to prevent this from is because this will give an error. This doesn't actually make sense for GDScript programmer because you don't actually had to put null on the value, but on those who is making a wrapper on other language to use this addon, this gets annoying, especially when working with relative values.

Makes sense 💯


feat(tags): support inject default tags from env

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.郑力升 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Empty strings for model root directory

Hello!
at the moment it’s not possible to set an empty directory for the module root in modules.php.
Let’s say that I would want to keep models within the root directory as is within a basic Laravel application (modules/moduleName/ModelName.php).
I’ve tried with empty string and a “/“, but when doing any of that an error is being thrown.
Is it possible to do that or is it limited to live within an actual subdirectory in the module?
I see that the default is Entities, but I would like to keep my modules as much like the app-directory as possible.
If it’s not possible I might be able to throw in an PR when I get the time, but I wanted to check with you first.
Thanks for an awesome package!

Hello,
If I understand it correctly you would like to have your module in the root of laravel so :

app/
config/
Module1
Module 2

If this is correct, it isn't possible

No, you misunderstood. Sorry if I was being vague.
What I want is for my models to live within the modules  root directory.
So let's say that I have a module called MyModule.
Then the path to that module would be Modules/MyModule - right.
Now I want to create a model for MyModule called SomeModel, but I don't want it to be placed in Modules/MyModule/Enteties/ but rather directly in the module root directory (Modules/MyModule).
So when I'm generating it, the generator looks to config('modules.generator.model') to know where it should be placed.
If I set the path in the model to be an empty string I thought it would create the model in Modules/MyModule, but instead it throws an error.
Is there a way to configure so that scaffolded models will be placed within the root directory of the module or do they have to live within a subdirectory (such as Enteties)?
I've also seen that if I delete the model config all together it will still set the namespace to be \Modules\MyModule\Enteties (I guess it's falling back on the configuration of the package if it's not defined at all?)
Example:
config/modules.php
    'config' => ['path' => 'Config', 'generate' => true],
    'model' =>  ['path' => '/', 'generate' => true],
]```

Ah ok I see.
Hm I wouldn't recommend doing this, but I agree it should be possible. Though it might require a lot of changes which might not be worth it for the value.

I'll close this as there hasn't been a reply in a while, feel free to comment if needed.


Button to save changes in the "My_controller" section

0.0.3.Alpha2-SNAPSHOT
When I want to edit the configuration "My_controller", the "Save" button is disabled.

@aldepas It is because I made Widget image file disk location as mandatory field. I will make this field as optional or add default location.
Thank you!


Make default values of set_defaults consistent.

The "factory default" of write_timeout, set at import time, is None. In the signature of set_defaults(...) the default value of write_timeout is, arbitrarily, 2. This means that calling
EpicsSignalBase.set_defaults()

with no arguments changes the default value of write_timeout from None to 2. This doesn't feel right. I have changed the signature to set_defaults(..., write_timeout=__default_write_timeout) and so on for the other parameters. That way, if we change the factory defaults someday, everything will stay in sync.
Note that, as usual, the docstring shows the concrete values, so the user can easily see what the default values actually are:
In [4]: EpicsSignalBase.set_defaults?
Signature:
EpicsSignalBase.set_defaults(
    *,
    timeout=2.0,
    connection_timeout=1.0,
    write_timeout=None,
    auto_monitor=False,
)
Docstring:
Set class-wide defaults for EPICS CA communications
...

Waiting for CI on this.


Saving changes never completes

Environment
Windows build number: [run "ver" at a command prompt]
Version 2004 (OS Build 19041.264)

PowerToys version: 0.17.0
PowerToy module for which you are reporting the bug (if applicable): FancyZones

Steps to reproduce

Start PowerToys Settings
Click to turn off FancyZones
Click to turn on FancyZones
Click on Save
Expected behavior

Change would be saved and
Actual behavior
Saving with a circle spinning in upper right, never stops.

Not much
Screenshots

this should be fixed with 0.18

The fix is now available in 0.18 https://github.com/microsoft/PowerToys/releases


[google_sign_in] I don't want to use Firebase in IOS

When I set up google-sign-in, Android could be set up without a firebase. However, IOS had to sign up for Firebase to download GoogleServie-Info.plist.
I want to set it up without signing up for Firebase.
So I am personally modifying the code and using it.
The question is whether "clientId" is used in "method_channel_google_sign_in.dart".
I am currently using IOS "clientId" by assigning it to "clientId".
  @override
  Future<void> init(
      {@required String hostedDomain,
      List<String> scopes = const <String>[],
      SignInOption signInOption = SignInOption.standard,
      String clientId}) {
    return channel.invokeMethod<void>('init', <String, dynamic>{
      'signInOption': signInOption.toString(),
      'scopes': scopes,
      'hostedDomain': hostedDomain,
      'clientId': clientId, // I added.
    });
  }

Duplicate of #36673
please follow the issue there


[Bug?] Can't get it to work on Unity 2021.3.16f1...

So I actually tested this out today, but somehow I can't get it to work?
It seems like it should allow C#10 Compilation etc... as the project says but it just won't work!
There was a section about Unity 2021.1 and later...
mob-sakai/CSharpCompilerSettingsForUnity: Change the C# compiler (csc) used on your Unity project, as you like!
I followed exactly what was said, used the CUSTOM_COMPILE compiler directive and had also an #else flow...
Why isn't it working? Even when building the project into an executable the defaut compiler for Unity is still used...
Help!

So the UI Object doesn't become red neither on build nor in the Unity Player...
Thanks in advance

This project is unmaintaned ig...


Add custom dockerfile and nginx config

Deploy hac-core as image
Since we want to adjust to the new pipeline of releasing frontend trough images we have to create custom dockerfile build with nginx. This PR adds both of them and utilizes previously used quay.io/redhat-cloud-services/releaser next step is to use PROW to deploy the new image to quay so frontend operator can use it.

I'll look to Blake to review this proper.


[Live] Investigate using @alpinejs/morph

Suggested by @kbond after talking with @calebporzio (Livewire lead dev): https://www.npmjs.com/package/@alpinejs/morph
Livewire also uses morphdom, but it seems they're switching to this new lib. We haven't had specific problems with morphdom. However, it's not super-well maintained and this new library is less than half the size!! It also looks quite simplified & streamlined. And it would be great to continue sharing as many tools a possible with a similar library like Livewire.

We're now using Idiomorph.


you cannot build this "safely"

the build instructions make no sense, you made the app "open-source" but didn't include the "smali patcher", what if the sussy stuff is in it? you're making your users do something you used to disallow and ban people for (decompilation) just to get a patcher you don't want to release. by "safely" i mean you cannot fully audit this app.

but didn't include the "smali patcher"

There isn't one in extistence, it is manually done. Feel free to submit one if you would like to see it faster.


New member cannot log into the system

Invite a team member<EMAIL_ADDRESS>A verification link is sent successfully and he verifies the link.
He is taken to a Login page where he is asked to enter a password but he doesn't have any password so far.
Probable Solution: Create another page where he can set his "new" password.

Yep, this is the stupidest item in my todo list! Right now, you need to reset your password after being invited to a team, but I am going to change that so you get an email verification/account creation/join organization link in your email where you also set your password. 😄

Thanks for the response. I think an answer is to redirect him to a new screen/view named Set Password screen and display an alert message that your verification is successful.
btw great work from your side!
PS: I am seeing that there are few more issues related to team. For instance, a team member who is currently an Admin is working in his own browser window and at the same time, the Owner updates his role to Basic. What should be  the correct answer, I am wondering.

That's a great suggestion, thanks @coooolers!
For the other point — Because the authorization helpers check the user's role, if the Owner updates another Admin to Basic before the Admin/Basic member makes any changes, they will see an error message saying that they don't have permission to edit any data. When they navigate to any other page, the same will happen.
Staart UI currently does not have any member distinction, but in the next week I'm shipping that feature, so the frontend app will reflect the role as well, not just the APIs.

Hello Anand @AnandChowdhary ]
Okay I almost got you. Kindly explain this when you said: "Only when there is another interaction with the app will the API tell us that the USER is unauthorized."
Here the USER is Owner or Admin/Basic?
I have a suggestion regarding this ... When both the users namely Admin and Owner are logged in at the same time and the Owner attempts to change the Admin's role to Basic OR attempts to delete that user, do not let the Owner to change his role or delete him by keeping a flag in the User table to identify whether a particular user is logged in or not. Flash an alert to the Owner "Can't do this operation because the Admin is logged in. Ask him to logout first" The owner can obviously ask the admin personally to logout so that he can alter his role or delete him.
Here one more point arises, namely, what should happen to the records/data created (owned) by the Admin? Probably, the answer could be that we should transfer the ownership of those records to the Owner role. It is basically a Cascade Delete issue.. obviously we cannot those records but who is going to own those reords is a question.
Thanks and Regards

@AnandChowdhary If you have time, pl. reply to this.
by the way, I guess that which role can do what kind of work (permissions) will be hard-coded. Or, is it the Admin who is going to decide which role/person will do which work?

Hi @coooolers, for now, the roles will be hardcoded in Staart logic for consistency between organizations:

Owner: Can delete a team
Admin: Can edit team settings but not delete team, can CRUD members
Manager: Cannot view/edit team settings, only CRUD members
Member: Cannot view/edit team settings or other members
Basic: Cannot do anything related to team/members, only view app (can be use-case specific)

Hi Anand @AnandChowdhary
I agree with you that these roles should be hardcoded. But what I am seeing is that they are more (in number) than they should be .. making things little more complex. So what I suggest is that you should reduce the number of roles, probably only 3: Owner, Admin, and Basic. Rest is up to you :)
I am thinking of one more feature here. Consider that a user creates a document and his senior (another team member) is supposed to approve it before it is forwarded to other people in an app. So the Approver role should be there. This is basically a workflow functionality.
Another instance could be that one team member should be able to do CRUD operations while another member can do only 'R' (read/retrieve) operation.
A Manager of a company can view Dashboards while others cannot. From a company point of view, this is what the Manager role should be able to do. But according to what is implemented right now, a manager of a team is doing some other task i.e. CRUD team members. This leads to ambiguity.
Possibilities are endless.. we cannot implement all.
With Regards
Raman


OAK-10875 - unmerged bc on root can make it appear merged

If an oak instance crashes during a branch commit, which in turns leaves the bc unmerged - and that branch contained changes on root - it can later result in that unmerged bc to be considered as merged. If that values is then cached before other changes of the branch are read, those other changes are then considered merged even though they never have.

Also it might be good to modify the original test to have 2 kinds of tests:

one that tests normally (basically the current test)
one that tests when the repository was brought into the corrupted state, without any fix applied yet. Worst case the repository state would need to be created "hard coded" - eg by going through the steps as 1, but then explicitly force the state where _bc is removed but _commitRoot is not. This would be used to simulate suggested part 2. above.

(perhaps we can come up with a test that ensures that a property created as unmerged bc on root is never treated as committed - even though it is passed sweep rev and no branch commit info is remaining)

For the first part I removed the unmerged revision from _commitRoot(I think this is the proper fix to solve the bug).
For the second part, I will create another ticket to restore the state of repository if it is needed.

What about splitting the two parts, as you suggested, into two different PRs? I would tend to prefer if OAK-10875 would stay open as I believe both parts are required? But they could certainly be split into two PRs to simplify things. Wdyt?
Btw: there's something odd going on with this PR as it seems to duplicate the original : eg unmergedBCOnRoot is already there in trunk, but this PR adds it again, same with the changes of FailingDocumentStore. The CI currently complains with
[ERROR] /home/jenkins/jenkins-agent/workspace/Jackrabbit_oak-trunk-pr_PR-1605/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/BranchTest.java:[317,18] method invalidateCommitValueResolverCache(org.apache.jackrabbit.oak.plugins.document.DocumentNodeStore) is already defined in class org.apache.jackrabbit.oak.plugins.document.BranchTest

I think originally the PR was complaining it can't merge due to these duplications - now it doesn't do that anymore. No idea why...

Yes. We can have 2 PR for this ticket. The first part can be reviewed. I added a test method for the case where we have unmergedBC but no property set on root in the same BC(it didn't failed).

looks like there's some remaining merge conflict

It is fine now.

waiting for tests to succeed (or only have unrelated flakyness) before approving/merging next

failure seems unrelated :
[ERROR] Process Exit Code: 143
[ERROR] Crashed tests:
[ERROR] org.apache.jackrabbit.oak.jcr.RepositoryTest
[ERROR] org.apache.maven.surefire.booter.SurefireBooterForkException: The forked VM terminated without properly saying goodbye. VM crash or System.exit called?
[ERROR] Command was /bin/sh -c cd /home/jenkins/workspace/Jackrabbit_oak-trunk-pr_PR-1605/oak-jcr && /usr/local/asfpackages/java/adoptium-jdk-<IP_ADDRESS>+1/bin/java -javaagent:/home/jenkins/maven-repositories/1/org/jacoco/org.jacoco.agent/0.8.12/org.jacoco.agent-0.8.12-runtime.jar=destfile=/home/jenkins/workspace/Jackrabbit_oak-trunk-pr_PR-1605/oak-jcr/target/coverage-reports/jacoco-ut.exec -Xmx512m -XX:+HeapDumpOnOutOfMemoryError -Dupdate.limit=100 -Djava.awt.headless=true -jar /home/jenkins/workspace/Jackrabbit_oak-trunk-pr_PR-1605/oak-jcr/target/surefire/surefirebooter9881064834527679463.jar /home/jenkins/workspace/Jackrabbit_oak-trunk-pr_PR-1605/oak-jcr/target/surefire 2024-08-21T14-23-22_802-jvmRun1 surefire1194720512709980725tmp surefire_011467686415861265474tmp


Make the default action for file like providers to send to quck list if there are several items selected

This way mimics the fzf.vim behavior and reduces the need for having a shortcut only to do that.

https://github.com/ibhagwan/fzf-lua/commit/64ceffcd0a307e63837f41a63e21c55d3b1d4278


[FEAT] Re-defining a collection object is not possible

What is the problem? Please describe.
Once the SH collection, such a byoc collection, is defined, it cannot be redefined, this can be annoying, and I was wondering if there are special reasons for this
E.g.
import numpy as np
from sentinelhub import Band, DataCollection

col = DataCollection.define_byoc('some-byoc-id')

If I later decide to define also bands and do
# update the collection
col = DataCollection.define_byoc('some-byoc-id', bands = [Band(name='B01', units=('DN'), output_types=(np.int16))])

This fails with
ValueError: Data collection name `BYOC_some-byoc-id` is already taken by another data collection

While it could just drop the old definition and redefine it with the bands. Or perhaps give some warning or something. Right now we have to restart the notebook kernel, which is annoying.

The issue is that we never know which collection is already online, and in those cases you really cannot and should not redefine it.

The issue is that we never know which collection is already online, and in those cases you really cannot and should not redefine it.

But the official collections are already pre-defined and on collection definition a new collection is checked against existing ones, and then the new one is added. What if we never extend the enum, but always just check against the official ones?


Kivy references deprecated "renpy"

Software Versions

Python:
OS:
Kivy:  master
Kivy installation method:

Describe the bug
Kivy.metrics has 2 instances of autoclass('org.renpy.android.PythonActivity')
These should be autoclass('org.kivy.android.PythonActivity') as "renpy" is deprecated.
https://github.com/kivy/kivy/blob/master/kivy/metrics.py#L195
https://github.com/kivy/kivy/blob/master/kivy/metrics.py#L257
These result in run time errors on Android.
Code and Logs and screenshots
08-28 17:03:31.600 29390 29706 I python  :    File "/home/feluk6174/python/llista/.buildozer/android/platform/build-arm64-v8a_armeabi-v7a/build/python-installs/llista/armeabi-v7a/kivy/metrics.py", line 195, in get_dpi
08-28 17:03:31.602 29390 29706 I python  :    File "/home/feluk6174/python/llista/.buildozer/android/platform/build-arm64-v8a_armeabi-v7a/build/python-installs/llista/armeabi-v7a/jnius/reflect.py", line 209, in autoclass
08-28 17:03:31.603 29390 29706 I python  :    File "jnius/jnius_export_func.pxi", line 24, in jnius.jnius.find_javaclass
08-28 17:03:31.605 29390 29706 I python  :    File "jnius/jnius_export_class.pxi", line 266, in jnius.jnius.JavaClass.__init__
08-28 17:03:31.607 29390 29706 I python  :  AttributeError: 'Class' object has no attribute '_JavaClass__cls_storage'
08-28 17:03:31.607 29390 29706 I python  : Python for android ended.

"I have encountered this  issue that prevents me from using my app, and I'm looking for a solution."

Hardware is still accessed through renpy (it's not PythonActivity on those mentioned lines), it's not fully depreciated - something else is going on here.

Could be.
Since this issue I have seen more cases of this message, and it is not a message I usually see.
Don't know what the common denominator is.
These are the other cases:
https://groups.google.com/g/kivy-users/c/5JHi0lCoAlI/m/v49UD507AwAJ
https://discord.com/channels/423249981340778496/712344698559397895/1146462240195756205
https://discord.com/channels/423249981340778496/423250272316293120/1146358878318100541

I don't yet know what it is.....
Btw, elsewhere in the same file renpy is escaped
https://github.com/kivy/kivy/blob/master/kivy/metrics.py#L292-L295

Hi Robert Flatt, I'm would like to ask you something about Camera4Kivy, I know that's not the place to do this, so where can I write to you one question about c4k ? Thanks a lot

@javixav I don't engage in personal conversations, emails, dms, etc. If you want to ask a question do so on Discord or Google Groups.

@HyTurtle
FYI one user found a cause of this, documented here
https://github.com/Android-for-Python/Android-for-Python-Users#no-attribute-_javaclass__cls_storage


⚠ Bofrost.be has degraded performance

In c0fb67e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1405 ms

Resolved: Bofrost.be performance has improved in 5dceb18.


Prettier2

The main two changes are:

Change default value for arrowParens to always
Always add a space after the function keyword

We could configure arrowParens back to avoid but I am quite convinced by the new default
I left out:

Change default value for trailingComma to es5

I am relatively convinced by https://medium.com/@nikgraf/why-you-should-enforce-dangling-commas-for-multiline-statements-d034c98e36f8 but if we decide to do it we can do it later.
The main benefit of upgrading to prettier 2 is that it means that we are ready for TypeScript 3.8 and private fields !

It will take some getting used to the space in function ( but the benefits of prettier far outweigh the costs. I understand the trailing comma arguments but am still put off by the appearance. But I am adaptable.

The main benefit of upgrading to prettier 2 is that it means that we are ready for TypeScript 3.8 and private fields !

Yes!


Callback for interact button is wrong

Related to the new traitlet API.  Running
interact(slow_function,i=FloatSlider(min=1e5, max=1e7, step=1e5),__manual=True)

yields
<function __main__.slow_function>
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/home/jon/github.com/ipython/ipywidgets/ipywidgets/widgets/interaction.py in call_f(change)
    249     # Build the callback
    250     def call_f(change):
--> 251         name, old, new = change['name'], change['old'], change['new']
    252         container.kwargs = {}
    253         for widget in kwargs_widgets:

TypeError: 'Button' object is not subscriptable

Hmmm, I'm having trouble reproducing this in an isolated environment...


Bjorncs/improve user auth with athenz filter

FYI @frodelu
Fixes some issues with the combined user auth + Athenz filter

👍


Typo in refresh_token_lifetime_secs

The value of refresh_token_lifetime_secs had a comma in the wrong place. it was throwing me off.

#sign-off


💥FIX : Broken images on Blog page and Back to Top button

Title of the Pull Request

[x] Have you renamed the PR Title in a meaningful way?

Related Issue

Closes: #1250
Description
Broken images on the Blog page and Back to the Top button fixed.
How Has This Been Tested? ⚙

Screenshots 📷

Type of change
What sort of changes have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)

Checklist

[x] My code follows the code style of this project.
[x] I have followed the contribution guidelines
[x] I have performed a self-review of my own code.
[x] I have ensured my changes don't generate any new warnings or errors.
[x] I have updated the documentation (if necessary).
[x] I have resolved all merge conflicts.

@vishanurag Please review and merge the PR. Please add a level 2 label. The code was a little messy.


Only check the value for a relationship when the value is a string.

$faker->datetime returns a DateTime object, this causes an error to be thrown when it is passed to preg_match() in the hasRelationshipAttribute() method.

Sorry for delay! But thanks!


Fix #104 Toggling renderInPlace

After the change in #101, the wormhole cannot toggle from render in place to render elsewhere, because destinationElement CP is never read and therefore the observer will never fire when renderInPlace has changed. (Issue #104).
I'm little unclear why we need the change from #101, but presumably we want to specify an explicit element to be used as a wormhole rather than selector. In this case, we will need to a new computed property (private) to capture all possible parameters, which ember-wormhole component can receive and observe them for changes.
If there are other reasons, we can discuss a different fix for the issue.

@dnachev I think you can fix travis with a patch like this: https://github.com/ember-cli/ember-fetch/commit/0a817ff9f80f78a635f16d398e138739308ec52f

Thanks @nlfurniss . Trying it out.

@lukemelia apologies for the direct ping; do you think you or another yapper have some cycles to review+release this?

@fivetanley For you, anything :D -- released as 0.5.5

@lukemelia <3 quickest merge in the west. thanks!


Update single route definition

closing this in favor of https://github.com/cloudflare/cloudflare-docs/pull/40


The artifact authoring experience is terrible.

I've spent quite a bit of time authoring Azure RM templates, VSO build tasks and DevTest Lab artifacts. When I compare and contrast the experiences, the DTL artifact authoring is the worst of the three. Here is some (hopefully actionable) feedback:
Artifact manifest
Support more parameter types: Looks like we only support string and securestring types for parameters. Can we please add support for other types (int, bool, object, array etc)?
Support static dropdown lists for parameters: Self explanatory.
Support dynamic dropdown lists for parameters: Similar to  VSO tasks, we should provide dynamic dropdown lists for parameters. VSO dynamic dropdowns allow you to call REST endpoints and bind the result to the underlying list. This functionality would be tremendously useful for DevTest Lab artifacts (E.g. You can query the Chocolatey/WebPI catalog and populate your dropdown with list of available packages, versions, skus etc). See example of a VSO dynamic dropdown here.
Support regex or custom validators for string parameters: Currently, when using string parameters we can only avail of "allowEmpty" and "defaultValue" fields. Would be great if we can specify regex and other custom validators (e.g. string length must meet some conditions etc) in the artifactfile.json.
Built-in helper functions: Similar to Azure RM template functions (here) we should provide some built-in helper functions that could be used in artifactfile.json (e.g. function to url-encode a string, find-replace a substring etc).
Artifact scripting
Powershell named parameters: It is not possible to call named parameters in a powershell script, if the parameter values (usually supplied by the lab user) contains a quote or a whitespace.
Running as elevated admin: Artifact authors should not have to write code to run their artifacts in the context of an elevated admin (this is what most chocolatey based artifacts are doing. See example here). Would be great if the artifactfile.json exposed some field (say 'RunElevated'), which when used, causes the artifact to automatically run elevated.
Artifact execution
Indicating "critical" errors: As of today, artifact execution errors are treated as non-critical. If one artifact fails, it does not stop execution of other artifacts in the list. Artifact authors (and/or lab users) need a mechanism to mark an artifact failure "Critical". E.g. If installation of VS or SQL fails, then don't bother installing the other artifacts in the list (abort execution of all subsequent artifacts). This is tremendously useful for long-running machine prep scenarios (e.g. dev machine prep involves installation of VS, cloning repos, installing editors & other tools etc).
Reboot after execution: Some installers exit with 3010 code, which indicates a successful installation, but a reboot is required. There should be a field (say 'RebootIfSuccessful'), which when used, forces the machine to reboot after the artifact executes.
Checkin process(es)
Bot/Auto validation on checkins and submitted PRs: We should auto-validate all PRs and checkin to the artifact repo master (license agreement check, schema checks for artifactfile.json, running unit tests etc). This is standard practice for many azure repos (E.g. The Azure quickstart gallery runs a travis/jenkins bot on every PR and checkin. See example here). I accidentally checked in a regression (wrong casing for artifactfile.json). Issues like these are hard to diagnose and should be caught by the auto-validation process.
Need mandatory readme.md for artifacts: All artifacts should have a mandatory readme.md (see example here). On the portal's 'Add artifact' blade, it would be useful to have a hyperlink to the artifact’s readme file. E.g. http://imgur.com/lt3f5pX
Miscellaneous
Documentation: Where does the most up to date documentation on artifact authoring reside? On MSDN? Or on github? Both these set of docs seem to be a bit out of sync.
Custom log files: What is the "recommended" location for generating custom log files (via the artifact's powershell script)? I currently log to this location: %SystemDrive%\Packages\Plugins\Microsoft.Compute.CustomScriptExtension[version]\Downloads.

Thanks for reporting the issues. Some of them are feature requests. Would you mind creating tickets at our UserVoice forum (where we collect votes from the community on feature requests)?
https://feedback.azure.com/forums/320373-azure-devtest-labs
Thanks,
Xiaoying Guo
Senior Program Manager, Azure DevTest Labs

Anything actionable has been updated in the backlog.

I created custom artifact and uploaded in git hub public repository ,given git hub credentials in azure configuration policies.The application which i want to apply artifact on azure virtual machine is uploaded from my own organization HTTP server,here the scenario is artifact.json file and powershell.ps1 file and application which i want to install is downloading from my organization server.Upto here the file and required downloads are downloading correctly,while execution to install application its starts the application but the application is not installed,when i saw in task manager, application status showing running but process % is 0.Later when i saw on azure artifact details its showing succeded.Kindly help me from this issue.

Hi yashg7cr,
please open a new issue and share your script and the logs you get.
Thanks,
Jonathan


Create new chat room and new stream is created

@yj-soh Refactored the codes. You can check this again.


Fix ndarray broadcasting example and description.

Fixes after this PR is resolved. https://github.com/dmlc/mxnet/pull/3085

it looks like you deleted all outputs. is there any particular reason for that. otherwise i suggest to keep these outputs for people to just view these notebooks online.

Oh, my fault. I don't have gpu on my laptop so it will give error. Let me fix that.


[Question] Can you access preprocessor errors programatically?

I'm running karma programatically through the Server API and was wondering if there was a good way catch preprocessor errors when they occur.
Right now, if a preprocessor (babel for example) fails, I'll just see a message logged from Karma, but the server will still start up.
I'd like to be able to cancel the whole process, or at least catch the error when it occurs. I tried tracing my way through the source, and couldn't exactly understand what happens when a preprocessor invokes it's callback with an error: https://github.com/karma-runner/karma/blob/master/lib/preprocessor.js#L26
Any advice?

Still curious about this one

You should be able to catch an event with the error. It is emitted here:
https://github.com/karma-runner/karma/blob/master/lib/preprocessor.js#L64

I did a little more digging today, and from my understanding, that line will emit errors due to a preprocessor failing to start up, but will not emit an error that the preprocessor itself reports. For example when a there is a syntax error in one of the files babel is trying to preprocess and that error is thrown in on this line:
https://github.com/babel/karma-babel-preprocessor/blob/master/lib/index.js#L24
If I'm reading the code well enough, when tracing this example the error would turn into a rejected promise on this line:
https://github.com/karma-runner/karma/blob/master/lib/file-list.js#L186
Falling through to the next rejection handler, which occurs back in the Server on this line:
https://github.com/karma-runner/karma/blob/master/lib/server.js#L192
However, since afterPreprocess takes no error argument, that error is effectively lost. I was thinking it would be nice if the server had the option to abort if a preprocessor error like this occurred. Perhaps, something like the following:
  var afterPreprocess = function () {
    if (config.autoWatch) {
      self._injector.invoke(watcher.watch)
    }

    webServer.listen(config.port, function () {
      self.log.info('Karma v%s server started at %s//%s:%s%s', constant.VERSION,
        config.protocol, config.hostname, config.port, config.urlRoot)

      self.emit('listening', config.port)
      if (config.browsers && config.browsers.length) {
        self._injector.invoke(launcher.launch, launcher).forEach(function (browserLauncher) {
          singleRunDoneBrowsers[browserLauncher.id] = false
        })
      }
      var noLoadErrors = self.loadErrors.length
      if (noLoadErrors > 0) {
        self.log.error('Found %d load error%s', noLoadErrors, noLoadErrors === 1 ? '' : 's')
        process.kill(process.pid, 'SIGINT')
      }
    })
  }

  /*
   * Propsal
   */
  var handlePreprocessError = function(err) {
    if (config.failOnPreprocessError) {
      throw err; // or handle more gracefully
    } else {
      afterPreprocess()
    }

  fileList.refresh().then(afterPreprocess, handlePreprocessError)

I also understand that my current use-case causing me to run into this problem is somewhat non-traditional, so I'm not sure if this would be worth implementing.
Thanks for your time!

Thank you for the digging. I like your proposal, but I would make the default to log the error as a warning and then call afterPrepocess so it is always surfaced and not swallowed like it is right now.

Is throwing the error the right move, or would you propose a more graceful tactic?

I just realised, that you could simply add it to self.loadErrors.
The current behaviour is to exit the process if any occur, so we don't actually need to make this configurable and just exit.

Oh excellent! I'll open a PR soon

@dignifiedquire - it looks like self.loadErrors is designed to capture a specific type of initialization error:
// lib/server.js
self.on('load_error', function (type, name) {
self.log.debug('Registered a load error of type %s with name %s', type, name)
self.loadErrors.push([type, name])

// lib/launcher.js
try {
  var browser = injector.createChild([locals], ['launcher:' + name]).get('launcher:' + name)
} catch (e) {
  // ...
  emitter.emit('load_error', 'launcher', name)
}

// lib/plugin.js
try {
  modules.push(require(name))
} catch (e) {
  // ...
  emitter.emit('load_error', 'plug_in', name) // load_error
}

// lib/preprocessor.js
try {
  p = injector.get('preprocessor:' + name)
} catch (e) {
  // ...
  emitter.emit('load_error', 'preprocessor', name) // load_error
}

Knowing this, it feels like a hack to use self.loadErrors for preprocess-specific errors, especially when I would ideally like to be able to access the error programmatically. I'm still a fan of either the proposal above, or creating another event-type for preprocessor-specific errors:
// lib/preprocessor.js
var createNextProcessor = function (preprocessors, file, done) {
  var emitter = injector.get('emitter'); // proposed
  return function nextPreprocessor (error, content) {
    // ...
    if (error) {
      file.content = null
      file.contentPath = null
      emitter.emit('preprocess_error', error) // proposed
      return done(error)
    }
   // ...
}

This event could be listened to anywhere, including the Server.
Thoughts?

Bump @dignifiedquire


[Radiko] ERROR: 'int' object has no attribute 'decode'

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.10.07 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[x] I've read about sharing account credentials and I'm willing to share it if required

Region
Japan
Provide a description that is worded well enough to be understood
yt-dlp breaks when trying to download from Radiko with the error: 'int' object has no attribute 'decode'
The page is only accessible from Japan.
Command used: yt-dlp 'https://radiko.jp/#!/ts/LFR/20231010022853' -v --update
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['https://radiko.jp/#!/ts/LFR/20231010022853', '-v', '--update']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[377e85a17] (zip)
[debug] Python 3.11.5 (CPython x86_64 64bit) - Linux-6.5.6-arch2-1-x86_64-with-glibc2.38 (OpenSSL 3.1.3 19 Sep 2023, glibc 2.38)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0, rtmpdump 2.4
[debug] Optional libraries: pycrypto-3.19.0, sqlite3-3.43.1
[debug] Proxy map: {}
[debug] Loaded 1886 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 47743cde1f66f16f8ef8440b78171877122b1277f906dbebfb7d49fb4c64ab0b
yt-dlp is up to date<EMAIL_ADDRESS>[Radiko] Extracting URL: https://radiko.jp/#!/ts/LFR/20231010022853
[Radiko]<PHONE_NUMBER>2853: Downloading radio program for LFR station
[debug] Loading radiko.auth_data from cache
[Radiko] Checking cached token
[Radiko] Downloading authentication page
[Radiko] Downloading player js code
[Radiko] Authenticating
[debug] Saving radiko.auth_data to cache
[Radiko]<PHONE_NUMBER>2853: Downloading stream information
[Radiko]<PHONE_NUMBER>2853: Downloading m3u8 information from tf-f-rpaa-radiko.smartstream.ne.jp
[Radiko]<PHONE_NUMBER>2853: Downloading m3u8 information from tf-c-rpaa-radiko.smartstream.ne.jp
[Radiko]<PHONE_NUMBER>2853: Downloading m3u8 information from rd-wowza-radiko.radiko-cf.com
[Radiko]<PHONE_NUMBER>2853: Downloading m3u8 information from radiko.jp
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: best/bestvideo+bestaudio
[info]<PHONE_NUMBER>2853: Downloading 1 format(s): rd-wowza-radiko.radiko-cf.com
[debug] Invoking ffmpeg downloader on "https://rd-wowza-radiko.radiko-cf.com/medialist?session=7saahb5uTGe6cYFRxDT436&station_id=LFR"
[download] Destination: Adoのオールナイトニッポン 2023-10-12 18_20 [20231010022853].m4a
ERROR: 'int' object has no attribute 'decode'
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1723, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 2921, in process_video_result
    self.process_info(new_info)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3387, in process_info
    success, real_download = self.dl(temp_filename, info_dict)
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3108, in dl
    return fd.download(name, new_info, subtitle)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/common.py", line 455, in download
    ret = self.real_download(filename, info_dict)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/external.py", line 50, in real_download
    retval = self._call_downloader(tmpfilename, info_dict)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/external.py", line 620, in _call_downloader
    args = [encodeArgument(opt) for opt in args]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/external.py", line 620, in <listcomp>
    args = [encodeArgument(opt) for opt in args]
            ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/utils/_utils.py", line 878, in encodeArgument
AttributeError: 'int' object has no attribute 'decode'

Looks like a bug that's been present since the extractor was added (1931a55ee8412ee385357f33128996cc3d07560e)
diff --git a/yt_dlp/extractor/radiko.py b/yt_dlp/extractor/radiko.py
index 8c8fb1a8f..c363d9ba5 100644
--- a/yt_dlp/extractor/radiko.py
+++ b/yt_dlp/extractor/radiko.py
@@ -154,7 +154,7 @@ def _extract_formats(self, video_id, station, is_onair, ft, cursor, auth_token,
                     sf['preference'] = -100
                     sf['format_note'] = 'not preferred'
                 if not is_onair and timefree_int == 1 and time_to_skip:
-                    sf['downloader_options'] = {'ffmpeg_args': ['-ss', time_to_skip]}
+                    sf['downloader_options'] = {'ffmpeg_args': ['-ss', str(time_to_skip)]}
             formats.extend(subformats)

         return formats

CC: @garret1317

tested, can confirm the fix works
but im curious why this behaviour is something you'd want
surely you should download the whole programme then seek to the bit you want? why start right in the middle of it?

either way it doesnt look like it's the ffmpeg arg that's doing the seeking forward, it's the seek query that gets added in RadikoIE._real_extract
(download still starts at the same time with the ffmpeg bit commented out)
(also, the seeking, with or without the ffmpeg arg, only works for certain formats: works on wowza, doesnt work on radiko.jp, other formats dont work anyway)


UI to edit video metadata

Editable fields from MediaItem object from the UI and MediaItems ACL
Publish/Unpublish
Delete MediaItem

Duplicated from #148


document gcs

why is your commit getting attributed to github.com/ifx?  Is that an alt GitHub account of yours?

It is yes. I'm not sure why it does that, it's been doing it for years
On Feb 1, 2018 22:30, "Will Norris"<EMAIL_ADDRESS>wrote:

why is your commit getting attributed to github.com/ifx? Is that an alt
GitHub account of yours?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/willnorris/imageproxy/pull/137#issuecomment-362498964,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAanP2p4SBKNLsE6XYh-FJ_X3EjaJkTxks5tQquRgaJpZM4R2uwx
.

I went ahead and rewrote the gcscache implementation last night (7bf3645c1006f60c40775c385a81befdf5b7cb04, 0c20cbe5b5828cfd74d1fa5c4ea6c15d2b80d6be) to use application default credentials, so this is pr is no longer needed.
(As for the account issue, it's because your allenday.com email address is attached to that GitHub account.  You'll need to remove it from there and add it to your main GitHub account.  You should probably go ahead and do that, otherwise you're going to continue to run into CLA problems like this for any Google projects you contribute to, and possibly other places as well.)


Initial Draft

Merging from my personal private repo.
Added:

openstream (core)
openstream plugins:

github actions
argocd
argocd application

@basicthinker merging recent work into the public repo.
More documentation, GitHub Issues template, and PR template come later in the next PR.


Add Python 3 support

It's just silly to have a modern library that only supports Python 2.7.

There is no ported version of m2crypto to python3. I found 2 approach:

https://github.com/mcepl/m2crypto/tree/python3
https://gitorious.org/chandler/m2crypto/source/31b07d2668d8132fec90fb94417d3c331521283b:
but both have errors on the x509 test, so we can't use it.
Also dm.xmlsec.binding keep supporting only python2.

I googled and found some libraries compatible with python3

https://github.com/mehcode/python-saml  (Uses lxml and xmlsec library)
https://github.com/rohe/pysaml2-3 (Uses pycrypto and its own xmldsig and xmlsec classes)

Feel free to use them meanwhile we support python3 (maybe we wait till m2crypto/dm.xmlsec.binding ported to python3, maybe we replace those libraries by others supported in python3).

There is a fork that support python3 and we plan to support it soon:
https://github.com/bgaifullin/python3-saml

Is this ready for use? Would love to use OneLogin SAML from a Python3 app.

We plan to publish it at OneLogin soon. Meanwhile clone and use https://github.com/pitbulk/python3-saml


driver and robot to take screenshot by default, since a very common ask

the afterScenario() approach has limits, especially in called features: https://stackoverflow.com/q/65088505/143475
thinking a configure driver setting to disable is needed, and this is enabled by default
* configure driver = { type: 'chrome', screenshotOnFailure: false }

1.0 released


Update sbt-bloop to 1.4.7

Updates ch.epfl.scala:sbt-bloop from 1.4.0-RC1-190-ef7d8dba to 1.4.7.
GitHub Release Notes
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-bloop" } ]

labels: sbt-plugin-update, semver-pre-release

Superseded by #58.


translation 381~786

Improved inaccurate translations.
Changed incorrect Listing and Section numbers.
There is one line that doesn't need to be changed, so I left it as it is.

LGTM!


Add container security scanning + badge

Hi,
In order to keep the docker images up to date and secure, could the maintainers please add this repository to a service like Snyk that monitors and alerts when there are vulnerabilities or insecure configurations in the container image? Ideally a badge could be added to the README for quick visibility of the status.
Thanks!

Duplicate

Please see also my comment on https://github.com/ivx/yet-another-cloudwatch-exporter/issues/306#issuecomment-792291847


google scholar pagination skips result 20

When retrieving results from Google Scholar using the pagination() method, the first article on the second page of google scholar is always missing.
I think this is caused by the following snippet in the update() method of google-search-results-python/serpapi/pagination.py:
def update(self):
        self.client.params_dict["start"] = self.start
        self.client.params_dict["num"] = self.num
        if self.start > 0:
            self.client.params_dict["start"] += 1

This seems to mean that for all pages except the first, paginate increases start by 1. So while for the first page it requests results starting at 0 and ending at 19 (if page_size=20). For the second page it requests results starting at 21 and ending at 40, skipping result 20.
If I delete the if statement, the code seems to work as intended and I get result 19 back.

@jvmvik Can you take a look?

@samuelhaysom Currently, the best approach would be to use serpapi_pagination instead. When #30 is merged, the pagination() method would be the preferred one. Let us know if you need additional help and sorry for such a long reply.
if "next" in results.get("serpapi_pagination", {}):
    search.params_dict.update(dict(parse_qsl(urlsplit(results.get("serpapi_pagination").get("next")).query)))
else:
    break

For example:
# Google Scholar Search API

from serpapi import GoogleSearch
from urllib.parse import (parse_qsl, urlsplit)

params = {
  "api_key": "...",             # serpapi api key
  "engine": "google_scholar",   # search engine
  "q": "minecraft redstone",    # language
  "hl": "en"                    # search query
}

search = GoogleSearch(params)   # where data extraction happens

# to show page number
page_num = 0

# iterate over all pages
results_is_present = True
while results_is_present:
    results = search.get_dict()  # JSON -> Python dict

    if "error" in results:
        print(results["error"])
        break

    page_num += 1
    print(f"Current page: {page_num}")

    # iterate over organic results and extract the data
    for result in results.get("organic_results", []):
        print(result.get("position"), result.get("title"), sep="\n")

    # check if the next page key is present in the JSON
    # if present -> split URL in parts and update to the next page
    if "next" in results.get("serpapi_pagination", {}):
        search.params_dict.update(dict(parse_qsl(urlsplit(results.get("serpapi_pagination").get("next")).query)))
    else:
        break


Open in integrated terminal not working

Issue Type: Bug
When i right click on a directory/file and select open in integrated terminal, instead the terminal opens in my computers root directory. I've ensured that there is no terminal.integrated:cwd, it is blank. I even tried downloading the terminal here extension, but every terminal only opens at my root directory. There are no vscode updates avail. I run on windows 11.
VS Code version: Code 1.64.2 (f80445acd5a3dadef24aa209168452a3d97cc326, 2022-02-09T22:02:28.252Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11370H @ 3.30GHz (8 x 2995)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.74GB (9.40GB free)

Process Argv
--crash-reporter-id 6881ff56-3a9f-4b19-aa5b-f712b8f275ce

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.2.2

gitlens
eam
11.7.0

heroku-command
pko
0.0.8

LiveServer
rit
5.7.4

html5-boilerplate
sid
1.1.1

vscode-zipexplorer
sle
0.3.1

open-in-browser
tec
2.0.0

pdf
tom
1.2.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30425749
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30442554
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30433507
vsclayoutctrc:30437038
vsrem710:30416614
dsvsc008:30440022
pythonvspyt640:30442238
vsbas813:30436447
vscscmwlcmt:30438805
helix:30440343

Please use the Start Extension Bisect command to investigate whether the problem is being caused by one of your extensions.

This is working for me.

I changed terminal > integrated > default profile : windows setting to powershell instead of git bash and now I can open in integrated terminal. Is there a way I can use git bash instead and have it open in integrated terminal? Or am I misunderstanding something about the nature of the terminal?

Thanks for the info. We'll see if we can repro the issue on windows. It should work for git bash

Are there any errors in the dev tools console when you try to do that with git bash as the default?

Thank you for your prompt help. I am an amateur coder, so I apologize if this isn't what you're looking for, but I searched developer tools in the command palette and found the console, I believe. When I open vs code with powershell as the terminal.integrated.default profile.windows option, I get this in the console:

When I right click a file and attempt to open with integrated terminal, I get the following picture, which I believe matches the first. Again this is using powershell and the terminal will correctly open at the file location:

Then I changed the terminal.integrated.default profile.windows option to git bash, closed vscode, reopened vscode, opened the dev tools and I get this picture:

Then I right click a file and attempt to open with integrated terminal, I get the following picture:

The only difference I spot is another instance of the message "WARN Settings pattern "keyboard.*" doesn't match any settings" log.ts.307 ( the same line at the top when I started vscode), and the terminal will not open at the correct file location, only the root directory.

Git bash works fine for me, could you try updating git?

Without re-installing bash, I found the solution:
My .bashrc file had one line of code "cd C/users/gener". We were doing a git demo to show how to run shell scripts and I hashed this line out. Now my git bash terminal correctly opens to the project from which I select 'Open in integrated terminal."


Errors and warnings page needs description for more errors

Currently we have the following errors that are not described in the errors and warning help page -
  - "NU3009"
  - "NU3010"
  - "NU3011"
  - "NU3012"
  - "NU3013"
  - "NU3014"
  - "NU3015"
  - "NU3016"
  - "NU3017"
  - "NU3019"
  - "NU3020"
  - "NU3021"
  - "NU3022"
  - "NU3023"
  - "NU3024"
  - "NU3025"
  - "NU3026"
  - "NU3027"
  - "NU3029"
  - "NU3030"
  - "NU3031"
  - "NU3032"
  - "NU3033"
  - "NU5000"
  - "NU5001"
  - "NU5002"
  - "NU5003"
  - "NU5004"
  - "NU5005"
  - "NU5006"
  - "NU5007"
  - "NU5008"
  - "NU5500"

cc @kraigb @rido-min

Appears to be addressed.


Do not allow for spaces in local instance paths

As of AEM startup script is failing when there is any space in current working directory.
Consider disallowing spaces in all / other paths.

released in 12.0.0


[improve][io] The JDBC connector supports JSON substructure schema

Motivation
Currently, when JDBC to convert value that has substructure schema, will get error:
java.lang.Exception: Not supported value type, need to add it. class org.apache.pulsar.client.impl.schema.generic.GenericJsonRecord

I know handling substructure types is complex, and JDBC databases may not always support defining substructure types.
However, it is possible to write the substructure type as a JSON string into the table. This can be very useful for databases that support JSON parsing.
For examples:

JSON data

{
    "state": "ts",
    "zipCode": 1000,
    "pcList": [
        {
            "brand": "b1",
            "model": "m1"
        },
        {
            "brand": "b2",
            "model": "m2"
        }
    ]
}

We can insert it to table like:

state
zipCode
pcList

ts
1000
[{"brand":"b1","model":"m1"},{"brand":"b2","model":"m2"}]

Modifications

When value instanceof GenericJsonRecord, use jsonNode string.

Verifying this change

Add testSubFieldJsonArray to cover it.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report
Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 73.43%. Comparing base (bbc6224) to head (f860ca8).
Report is 453 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23043      +/-   ##
============================================
- Coverage     73.57%   73.43%   -0.15%
- Complexity    32624    33101     +477
============================================
  Files          1877     1914      +37
  Lines        139502   143627    +4125
  Branches      15299    15672     +373
============================================
+ Hits         102638   105469    +2831
- Misses        28908    30085    +1177
- Partials       7956     8073     +117

Flag
Coverage Δ

inttests
27.61% <0.00%> (+3.02%)
:arrow_up:

systests
24.66% <0.00%> (+0.34%)
:arrow_up:

unittests
72.50% <50.00%> (-0.35%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

.../apache/pulsar/io/jdbc/BaseJdbcAutoSchemaSink.java
74.48% <50.00%> (+1.62%)
:arrow_up:

... and 500 files with indirect coverage changes


imgui: fix sdl config

https://github.com/xmake-io/xmake/issues/4859

I noticed this PR reverts the C++ revision to C++11, is there a reason for this? I had changed it to C++14 in #3545 to fix compilation issues when using Clang on Windows.


Add BiuBiubiub1u's file

created a branch, created a file and made a commit, and opened a pull request


Add Code of Conduct.

There you go - this is the code of conduct from avrgirl-arduino just copied across to this project.
Hope this helps.

No idea why the build's failed, @noopkat. I'll try to look into it.

There seems to be a SyntaxError at node_modules/testdouble/lib/replace/module/require-actual.js. I'll . continue looking into it.

I think it's an issue with one of the node packages, you might need to fix it - not sure though, not very experienced with node package debugging :smile:


A system for book sections enabling section labelling and distinct page numbering

This is one take on how to handle the notion of a brew having multiple, distinctly numbered sections in a way that both the CSS and snippets can track the page number resetting.
This system works by adding a single word parameter to the \page token. This value is added to the  class list. This allows both a predefined list and the users to create their own.
These classes should hold no CSS properties and exist to act as triggers to reset the page counter ( phb-page-numbers - which should probably be renamed ) and reset the Section Label and the numbering type for the page counter (Roman numerals, decimal, etc  - based on the LI numbering types)
This system uses .(sectionname):nth-of-type(1) for counter resets and .(sectionname) to override two global CSS variables where (sectionname) current defaults to one of index, appendix, chapter, frontmatter, and index.
Current design weaknesses that aren't addressed

There is no way to track multiple indexes or appendixes and label accordingly ( Appendix A, pg 7 )
Assumes each section type is used once
Does not enforce section order
Behaves the same as the present scheme if the first line of the brew does not set a section style
Still requires the user to insert the {{pageNumber }} bug.

Strengths:

SHOULD work on both the CSS side of the document and the snippets side by binding the counter resets to a token
Requires nothing of users not wanting to use it.
Easy extension via themes
Allows user extension via the brew styles tab

Solves:
Issue #2826

The deployment doesn't seem to work at this point.  It doesn't load the stylesheet and some other error.  Crashes if I type in the editor as well.
I haven't taken a look at the code yet so don't have much more to say, just a heads up.

The deployment doesn't seem to work at this point. It doesn't load the stylesheet and some other error. Crashes if I type in the editor as well.
I haven't taken a look at the code yet so don't have much more to say, just a heads up.

I haven't updated this one in a bit, I'm not surprised it has some issues. I'm going to move it back to draft until I can catch it back up.


Update caffeine to 3.0.0

Updates com.github.ben-manes.caffeine:caffeine from 2.9.0 to 3.0.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.ben-manes.caffeine", artifactId = "caffeine" } ]

labels: library-update, semver-major

Superseded by #53.