Pr/named client

Dear All,
this is an alternative implemention of #181 . Instead of @CXFClient standard qualifier @Named is used to select a particular client configuration:
import javax.inject.Inject;
import javax.inject.Named;
import com.example.FruitWebService;

@Inject
@Named("my-featured-fruitservice-client")
FruitWebService clientService;   // ok, inject featured client

It also contains updated client documentation as well as many unit and integration tests.

I'm getting more and more convinced that using @Named is not a good idea and rather introduce dedicated annotations CXFClient and CXFImpl.
One mayor problem is this one:
@Inject @Named("foo") FruitWebService foo;        // A
@Inject @Named("bar") FruitWebService bar;        // B

@Inject @Default FruitWebService foobar;             // C

The injection at point (C) is ambigious and appears unresolvable using qualifier @Named. Usually qualifier @Default gets added to a producer method if no other qualifier is present. Hence this would allow for the creation of a "default" producer method applicable at injection point (C) to resolve that ambigiouty (by selecting a default configuration for that service).
Unfortunatly, @Named is an exception to the rule above: @Default gets added even in the presence of qualifier @Named. Hence the code above reads:
@Inject @Default  @Named("foo") FruitWebService foo;        // A
@Inject @Default  @Named("bar") FruitWebService bar;        // B

@Inject @Default FruitWebService foobar;                               // C

and this also applies for producer methods, here producer "foo":
@Produces @Named @Default                // CDI automatically added @Default (!!)
FruitWebService foo() { return .. }

Now it's obvious that producer foo() can be applied to (C).
Another problem with @Named is that it's annotation value is bound. Hence if we have @Named("foo") then a bean or producer named foo must exist. To overcome this, every injection point must be investigated, filtered and appropriate producers must be created (=> this is partially done in this PR where only field injection points are handled).
The problem with this approach is this one - assume for the moment that we have (in a very large and complex webapp) only a named "foo"injection point:
@Inject @Named("foo") FruitWebService foo;        // A
@Inject FruitWebService foobar;                              // C

Now there is only one producer and created and hence injection at (C) works without problem. Well, one problem is that you get configuration "foo" injected at (C) without further indication. Assume that someone changes (A)  to use @Named("bar") - now you also get "bar" injected at (C). This is rather non-intuitive and I guess very difficult to discover.
Then, again some time later, another team member introduces @Named("foo") again at a difference code location creating an ambigiouty at (C) - hence the application does not even startup now.
All this makes me believe that @Named is an inappropriate choice.
Better to have something like @CXFClient and @CXFImpl as qualifiers. This names tell exactly what is injected, making the code readable. The annotation property of  @CXFClient is also unbound (see #181) meaning that only one producer method is required and no static injection point scanning is necessary. This simplfies coding and especially testing.
It also avoids the problem described above:
@Inject @CXFClient("foo") FruitWebService foo;                  // A
@Inject @CXFClient            FruitWebService foobar;             // C
..
@Inject @CXFClient("bar") FruitWebService bar;                  //B

Here always the very same configured client is injected at (C) regardless of whether (B) is added later. What is injected at (C) is the default client where the default is a pure configuration issue. There is also no ambigiouty introduced by (B) later on cause there is only one producer method anyway.

I wonder how to continue with this PR and PR #181 ?


ADXT-164 Retry could'nt find resource flake

What does this PR do?
Add a retry for a flake that happens sometime with e2e tests

Motivation
Avoid flakiness

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge

/merge


[DISCUSSION] Automatic ensure_type()

look at this boilerplate. each type has to specify what other custom types it depends on, forming a kind of a tree that devs need to maintain. this leads to code duplication (I have this line at least 4 times), maybe we could solve this.
this is also confusing to beginners and adds extra burden...
proposed ideas:

maybe #[template_child] macro can check if the item has IsA<Object> and if so, ensure_type() on it automatically. is that possible? what would it take?
maybe have a list of type in some top level module (like window or application or whatever) with a list of all types we provide, then ensure_type them all whenever any type in the module is loaded?
other ideas?

we have most of the information to do it automatically. I think it should be done.

It is already called automatically for types that are used in TemplateChild

my bad! I was able to remove all but the last one! thanks!


username/password authentication can't be supported by etcdv3 backend

When I use etcd role based authentication at version3, I can access etcd with ectdctl like: ./etcdctl get --user root:rootpwd foo
but in confd when I start like: ./confd -onetime -backend etcdv3 -node http://<IP_ADDRESS>:2379 -username root -password rootpwd
there is a error below:
INFO Backend set to etcdv3
INFO Starting confd
ERROR etcdserver: user name is empty
FATAL etcdserver: user name is empty
Seems the username not take affect,
So my question is does confd support the etcdv3 user authentication? my confd version is confd-0.15.0

Looks like you must also specify -basic-auth, or the username and password do not take effect. Weird but that is how it works...

It works for me...  What is the exact configuration you are using (confd version, etcd version, options)?
If you are using TLS to connect to etcdv3, it is not possible to use username and password for authentication. You must put the username in Common Name of your client certification.

Thanks, I use the latest verison confd and add -basic-auth, now it works. I will close this issue

It works for me... What is the exact configuration you are using (confd version, etcd version, options)?
If you are using TLS to connect to etcdv3, it is not possible to use username and password for authentication. You must put the username in Common Name of your client certification.

This works fine. But why would one want such a strange behaviour and not give the user bei the username parameter?


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2127 ms

Resolved: Bofrost.at performance has improved in f690ff1.


Support for shebang

rustc already can ignore the shebang. And cargo-script RFC was accepted (https://github.com/rust-lang/cargo/issues/12207).
This is a valid rust:
#!/usr/bin/env -S cargo +nightly -Zscript

fn main() {
    println!("Hello from cargo script")
}

But it isn't parsed

It is also documented in https://doc.rust-lang.org/reference/crates-and-source-files.html#shebang


Invalid URL also if not true

TOOL: FAE 2.0
RULESET: HTML5/ARIA Techniques
DESCRIPTION: The url https://www.fineco.it/it/public is not redirected but FAE says that is an invalid URL and skip its processing.
URL: https://www.fineco.it/it/public

It looks like there is Javascript code on the page that HTMLUnit is not able to process, typically due to many code exceptions during execution.
You can use AInspector Sidebar (e.g. uses same OpenAjax Evaluation Library) to analyze the pages:
http://ainspector.github.io/


[RFC] Defer vim-state.coffee loading

I was wondering is it acceptable to defer vim-state.coffee loading until the first text editor is created?
On my machine with this change vim-mode loading time goes from ~100ms down to ~15ms.
On the other hand, this change will slow down first text editor opening a bit.
Change was proposed by @lee-dohm in atom/atom#2654.

In my opinion, since this won't help the common case of starting Atom with text editors open, it isn't worth lazy-loading in this case. I'm open to arguments in favor of doing it though.

Things have gotten a fair bit faster since this PR was filed.  I figured I'd run this test again...
  I ran the following commands alternately so each startup was very warm.

git checkout lazy-loading; sleep 0.5; atom .
git checkout master; sleep 0.5; atom .

Results

Lazy
Master

19ms
30ms

18ms
30ms

18ms
31ms

Stable and definitive: on my machine, this technique saves 10ms / ~30%.
Conclusion
18ms is worthy of a gold highlight-warning and 30ms is the red highlight-error.  This PR might get vim-mode out of the status-bar and symbols-tree-view doghouse.
That said, I agree with @maxbrunsfeld: every time I open my editor, I'm going straight into a Vim window.  For me, all this patch does is reorder the load process.  It doesn't actually save any real-world time.  Doesn't seem worth it.

Thanks @bronson. I think we'll need to use some different tactics for improving the package's load time. Closing this out.


Implement support for a deadman's switch.

Deadman's switches are alerts that are always firing. This way, we can check that the alerting stack is functional. We never want to receive an alert for a deadman's switch, but we want to receive an alert when it is missing.
This commit adds these configuration options:

deadman_switch_name: the name of the deadman's switch alert.
deadman_switch_interval: how often the alert is expected to arrive. Set this a bit higher than how often the alert is sent to prevent false positives.
deadman_switch_alert_interval: how often you want the bot to alert you of prolonged missing deadman's switch alerts.
deadman_switch_alert_chat_id: to what chat you want the alert to be sent.

Alerts with the deadman_switch_name are removed from incoming posts.  If the alert is the only one firing in a group, the overall status of the group is overwritten to "resolved".

hey @sgielen any update on this?

Sorry, not yet, I still intend to finish this but haven't found the time yet..

@hryamzik PR is updated with your suggestion to do the error handling inside HandleIncomingAlerts. I've added a return err inside the function, so the first Telegram Send error is returned, so that deadmanSwitchChecker knows whether the lastDeadmanSwitchAlert was sent successfully.

Closing, because it hasn't been picked up and I don't want to keep rebasing.

My bad, I've missed your previous comment.


Protocol specification issue

Hi, can you please correct the protocol definition in table 2-5,7 for http and https?
Do you really use UDP for http(s) ?

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

ID: ab256742-c202-ac5b-de83-3ce94e461923
Version Independent ID: efeaf566-85af-7957-8063-7045d69ce054
Content: Hybrid Identity required ports and protocols - Azure
Content Source: articles/active-directory/connect/active-directory-aadconnect-ports.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

Already addresses by my coworker. https://github.com/MicrosoftDocs/azure-docs/pull/5964

@KarstenKleinschmidt Thanks for reporting and also thanks to your coworker for addressing the issue by contributing directly !


Add nodejs20.x

AWS added support for Node.js 20 in the past few days.

aws-sdk release
PR for aws-cdk

Related comment from AWS.

Hey @o-alexandrov!
thank you so much for your contribution!
Would you be able to create the associated folder under s3-uploader/runtimes ?
Also please note that it's not yet available as a supported runtime but you're right it seems that it's coming soon so as soon as it will be available I'll merge :)

Thank you for pointing out the s3-uploader/runtimes and the great source to check actual cold starts across runtimes.
Updated PR w/ s3-uploader/runtimes

@maxday node.js 20 is available. It was available in SDK prior, so should be good (the ECR image is also marked as GA)

CDK has released Node.Js 20 support as well.

I'm interested in seeing this merged :)

force-pushed to update a commit signature, due to my email update on GitHub

Its official now :) https://aws.amazon.com/blogs/compute/node-js-20-x-runtime-now-available-in-aws-lambda/

Ha, I created #758 before seeing this PR exactly because it's official now. Would love to see this!

Thanks!


Fix sql-mapper eq/neq type declaration

Fix eq/neq type declaration in sql-mapper WhereCondition interface
Reference issue #2989

@mcollina finally I was able to solve the DCO error
sorry for the new PR


ARROW-626: [Python] Replace PyBytesBuffer with zero-copy, memoryview-based PyBuffer

WIP, but tests all pass

Realized we need to maintain reference count regardless of whether we're operating on a memoryview object or not. This should be complete.

@wesm not sure how to test this explicitly as the functionality is already well-covered by existing unit tests in test_io.py

@wesm not sure how to test this explicitly as the functionality is already well-covered by existing unit tests in test_io.py

all of the tests for this code in test_io.py use bytes objects, e.g. https://github.com/apache/arrow/blob/master/python/pyarrow/tests/test_io.py#L83. There are other objects with implement the buffer protocol, like bytearray or ndarray

Looks like my comment about multidimensional memory got clobbered. a Py_buffer can have shape and strides, we can just ignore these attributes for now but this should at least be documented

OK, will add comment about GIL as well as note that shape and strides are ignored as we assume a one-dimensional byte-sized buffer. Will also fix constructor's use of Py_Bytes* macros.


Compare foreign keys using pk, not id

Trying to compare revisions of a page that includes changes to a foreign key field of a related model that declared a custom primary key failed with an uncaught exception.
The root cause was ForeignObjectComparison filtering by the id field, which is not present in models that declare a custom primary key.
The solution is simply to filter by pk instead of id, which always maps to the primary key of the corresponding model.
Include a regression unit test.
Fixes #5678.

Merged in b9f585a6c9ba157710838325ea5d7a5689975f36 - thanks @haplo!

@kaedroho @gasman I'm sorry but I forgot to include a changelog entry.

@haplo No problem - it's actually easier for us if you don't include the changelog entry, as that way we don't have to fix a merge conflict when other entries are added in the meantime :-)


Prepare for async

Add unique ID migration for entities to be able to switch back from the async version to this (current) threaded version.
When upgrading to the async version some some the unique ids of entities are renamed. This PR is the reverse the actual rename at the async version.

Perhaps due to the entity migration a One Way ticket warning should be added to the release


Cleanup error handling/docs in TaskList

/cc @aozarov

Coverage increased (+0.01%) to 21.17% when pulling 1ec11abb06f1843ec6f4949dfcad5fec5476ffce on cleanup-tasklist into c4439f61b23b9e46596b212028cad6340b593d65 on master.

Looks good. Few nits. Feel free to merge without them.

Coverage increased (+0.01%) to 21.17% when pulling 41dbfd9d2f2b700c5dc6a01f674c38ba07f67c83 on cleanup-tasklist into c4439f61b23b9e46596b212028cad6340b593d65 on master.

Fixed comments


Filled Shopping Cart Links and Eccentric Spacer dimensions

This would be a link to the full openbuildstore cart for the diyson express (including the  end caps): https://saveyourcart.io/cart-url/qZya83ZUsDFTxaBH/db371481-3787-42a3-bb20-5ef49feb6bf3
I was not sure if the eccentric spacer is the 6mm variant or the 1/4“ version. In the cart above it is the 6mm version.
I am not sure however if the referral link gets saved in the saved cart.

Thanks for sharing the saved cart. I noticed it was missing the 40 mm spacer to connect to the desk mount. Here is an updated cart: https://saveyourcart.io/cart-url/qZya83ZUsDFTxaBH/88853f4a-3ad2-4483-a033-878baa2b9b02.
I'm also assuming the 6 mm version of the eccentric spacer based on the step files. Please correct me if I'm wrong.

@momonunu @tobias-holmes Thanks for this! Yeah not sure if this will work with my referral link but this is just too convenient to pass up.
Best of both worlds; could this be set up using my site-wide referral link? https://openbuildspartstore.com/?ref=_2E7m4SXRIy8t
Not a huge deal if these things are incompatible, but I was hoping the referral commissions could help offset the project costs.

@momonunu @tobias-holmes And yes it's the 6mm eccentric spacer

Probably best to use the cart link first and then the sitewide referral in order to support you.


A bug against myself

Core

[x] support for local files/directories in Bucket

LOC

[x] run MARC tag & flavor stats job
[x] upload files to S3

OTL

[x] config wiring

Catalog cleanups

[x] split author into first name, surname fields
[ ] UNIMARC support
[ ] extract journal citation fields from MARC, see http://dublincore.org/usage/meetings/2002/05/citdcsv.html

Crossref

[x] unpack tarball & upload to S3
[x] MANIFEST support in FilesModel
[x] extraction & schema

VIAF next steps

[ ] script to split giant XML.gz & put on S3
[x] run MARC tag & flavor stats job
[ ] decent tests for VIAFWorkCluster

VIAF phase 2

[ ] bindings & tests for VIAFPersonal
[ ] bindings & tests for VIAFCorporate
[ ] do we need VIAFExpression?

More MARC21 fields:

issn
journal citation
separate first & surname?


Please add konduits chambers of xeric features:)

auto-tagging Vanguards NPCs with their respective color from the combat triangle

Vanguards displays all of their HP percentages close together

targets of various projectiles like vasa stones and shaman poison

-crystal bombs timer
Please add these, would make the awesome client even better!

targets of various projectiles like vasa stones and shaman poison

https://github.com/runelite/runelite/wiki/Rejected-or-Rolled-Back-Features


Change names of travis-to generators

Update the donejs generators that do CD to: donejs-travis-deploy-to-heroku and donejs-travis-deploy-to-firebase
Update the guides where appropriate.

@m-mujica don't forget this one :D

Fixed by https://github.com/donejs/donejs/pull/1082


Please add support for https://play.tv2.dk/

Checklist

[ ] I'm reporting a new site support request
[ ] I've verified that I'm running youtube-dl version 2021.06.06
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description

WRITE DESCRIPTION HERE
Please add support for https://play.tv2.dk/
H:\YoutubeDL>youtube-dl.exe --verbose -u, --username<EMAIL_ADDRESS>-p, --password Sille2sen https://play.tv2.dk/programmer/serier/komedie/minkavlerne/momsfri-mandag-243931
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '-u,', '--username', 'PRIVATE', '-p,', '--password', 'PRIVATE', 'https://play.tv2.dk/programmer/serier/komedie/minkavlerne/momsfri-mandag-243931']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.21390
[debug] exe versions: none
[debug] Proxy map: {}
[generic] momsfri-mandag-243931: Requesting header
WARNING: Falling back on generic information extractor.
[generic] momsfri-mandag-243931: Downloading webpage
[generic] momsfri-mandag-243931: Extracting information
ERROR: Unsupported URL: https://play.tv2.dk/programmer/serier/komedie/minkavlerne/momsfri-mandag-243931
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpkqxnwl31\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpkqxnwl31\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpkqxnwl31\build\youtube_dl\extractor\common.py", line 534, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpkqxnwl31\build\youtube_dl\extractor\generic.py", line 3520, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://play.tv2.dk/programmer/serier/komedie/minkavlerne/momsfri-mandag-243931

@Porkeld please edit your issue and remove your login credentials.

Even if they remove it, the old revision will still be there; this issue should be deleted entirely by a maintainer or someone else with the right permissions. In the worst case OP might have to contact Github to get it done. OP can re-create it then.
@Porkeld , change your password if you haven't done so already.


If you fast on the same object multiple times notify, container posit…

If you fast on the same object multiple times notify, container position dislocation occurs.

Hi, @herong
I've forked TipTip to maintain it and use in all web projects of my community.
Please clone your pull request to: https://github.com/Software-js/TipTip
Thanks!

Hi, @herong
This repo seems to be abbandoned with no activity since later 2015.
I've forked Notify.js to maintain it.
Please clone your pull request to: https://github.com/WS-js/Notify.js
Thanks!


Wildcard in the middle not working

Wildcards that are in the middle of a permission string are not working.
Example: Giving a player nucleus.*.exempt.cost and then testing if they have the permission nucleus.rtp.exempt.cost yields an undefined result.
Running LuckPerms Sponge v3.1.26 on SpongeForge 1.11.2-2282-6.0.0-BETA-2287

You can use this as a temporary workaround : https://github.com/lucko/LuckPerms/wiki/Advanced-Setup#regex

Thanks. I know about regex (and am using it already) but I'd still expect it to work.

This is expected behaviour for wildcards. You can only use them at the end of a permission.
If you want to include them in the middle, use regex permissions.


Added version command

Description of the change:
Adding version command.
Note: Leaves intact --version behavior in case that would affect somebody.
Motivation for the change:
Bringing operator-sdk into sync with other k8s binaries in look and feel
Closes #1165

It looks like https://github.com/operator-framework/operator-sdk/pull/1171 has gotten further along and they're basically the same PR in spirit, so I'll close this one. Nice work @Avni-Sharma !


bump AD Kibana plugin version to 1.11.0

Issue #, if available:
Description of changes:
Bump AD kiban plugin version to 1.11.0.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for bumping. I will handle a follow up release notes PR

Thanks for taking care of that!


external temperature sensor

Hey Dominik / @DomiStyle ,
Its been some time since we spoke. My ac's run like a charm for quite some time now with your code. I remember scrolling through your files and excel sheet back in the day when i started using this to see which bits and bites of the long code do stuff. I also remember that in those files there were some bits of code which did nothing or atleast you couldnt figure out what they were for.
Running the ac for some time now i notice that the measured indoor temperature of the AC unit is pretty off from the real temperature. I was wondering if maybe some of these bits of code are there to communicate with the ac for hooking up an external temp sensor.
Im not even sure if this is an option for these AC's because i havent found any online. But it would be awsome to let HA post the temperature to the AC from an external sensor. Though im not sure if you are still working on this project and are willing to do some digging into this. I dont believe there are bits of code the AC is communicating that dont mean anything. Every part of the code the ac's is posting back is for something i guess...
At the moment i use node red with a pid node. i have a tempsensor and a desired temperature that is set to a specific temperature. THe PID node calculates the difference in wished temp and the temp measured in the room and if its to high the AC temp is set accordingly to the PID output calculated in a temperature between 16 and 30 degrees celsius. this lets me control the room temperature up to 0.1 degree accuracy but the temp send to the AC goes all over the place and i wonder if this will shorten the life of the AC. It would be better if the internal PID of the ac itself did the temp monitoring correctly and adjusted the power accordingly
Also one more thing. I noticed that since updating the AC's esp to the latest esphome version the logs dont show the communication of the AC and the long bitcode anymore. Do you know a way to get this communication back?
Just wanted to say that this piece of code is by far the best work i have ever seen someone do in home assistant. This integration of the AC locally is GOLD!

Running the ac for some time now i notice that the measured indoor temperature of the AC unit is pretty off from the real temperature. I was wondering if maybe some of these bits of code are there to communicate with the ac for hooking up an external temp sensor.

I doubt there's such a feature in there since Panasonic would most likely advertise it.

Also one more thing. I noticed that since updating the AC's esp to the latest esphome version the logs dont show the communication of the AC and the long bitcode anymore. Do you know a way to get this communication back?

It still does when verbose logging is enabled.

Just wanted to say that this piece of code is by far the best work i have ever seen someone do in home assistant. This integration of the AC locally is GOLD!

Good to hear it has been of use to people besides me.


New NPM Release v20190215.0.0

@blickly Please merge and publish new version to npm!

Sorry I missed that! I'm trying to both do a library release for the first time and better document the steps so that we're more consistent with the closure library releases; it looks like I missed the npm publism step in my instructions.
I think it's released now (https://www.npmjs.com/package/google-closure-library) and that it's not necessary to merge the http://go/gh/google/closure-library/tree/npm-v20190215.0.0 branch into master.  Let me know if any of the assumptions I've made aren't true.
Thanks for catching!

@blickly Thanks!

@blickly closure-deps is not updated
https://www.npmjs.com/package/google-closure-deps

@jplaisted, do you know what that package is and how it is released?

https://www.npmjs.com/package/google-closure-library
I think we have a script to release it internally. But should be as simple as go to the commit the release was made at an run npm publish

OK, I think this should be (finally) released.


Enable deleting a project without deleting its namespace

For the garden-setup project, I'd like to create a garden project linked to the garden namespace, in order to be able to use the shoot.garden.sapcloud.io/use-as-seed annotation and have the seed created automatically.
However, since deleting a project will also delete the corresponding namespace, this causes trouble when trying to remove the project again, as the Gardener itself is within the garden namespace.
Unfortunately, it seems impossible to undo the step of turning a namespace into a project namespace.
It would be nice to have an annotation (or something else) that changes the project deletion behaviour, so that instead of deleting the namespace, the Gardener will instead undo anything it did to the namespace when it was associated with the project (e.g. deleting ownerReference, etc.) and leave it in the state it was in before the project was created.

workaround:

delete ownerRef on namespace and project related annotations and labels on the namespace.
remove finalizer on project and set confirmation.garden.sapcloud.io/deletion annotation
delete project
cleanup other resources created for the project (clusterroles + bindings)

/assign
/remove lifecycle/rotten


Bug with user password reset

Need to fix bug in password reset system

Fixed in Version 2.0.0-beta.3


Ensure we use PST colours throughout

Closes https://github.com/pydata/pydata-sphinx-theme/issues/1925
Also, as I was fixing the above issue, I took the opportunity to make some needed changes regarding text colour.
And I realised there was perhaps a missing style from the banner button.

Great to see this being fixed, as I was also trying to change the hover color of the 'back-to-top' button via a custom css file! However I saw that the same goes for the hover color for tables (this is also hard coded to a certain violet value in _color.scss ) Maybe that can also be changed? I am a css rookie so maybe I am mistaken and it isn't hard coded, in that case could you point me a way to edit this? :)

You can already change the table hover, it will be the color of whatever you have defined as --pst-color-table-row-hover-bg. You can follow the steps in https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/styling.html#base-variables to have these variables defined to whatever value you want, this css file can serve as example.

You can already change the table hover, it will be the color of whatever you have defined as --pst-color-table-row-hover-bg. You can follow the steps in https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/styling.html#base-variables to have these variables defined to whatever value you want, this css file can serve as example.

Ah okay, thats great. I had been staring at the color userguide and trying different things for a few hours, but the "_bg" suffix requirement wasn't clear to me. Thank you for the quick response, the example and the help!

Ah okay, thats great. I had been staring at the color userguide and trying different things for a few hours, but the "-bg" suffix requirement wasn't clear to me. Thank you for the quick response, the example and the help!

Not so long ago, we changed our colour system, and therefore how we define colours, so it looks like this could be better documented/updated in the docs.
Thanks for the feedback!


Allow for multiple datasets

Add a new Dataset tab similar to the one introduced by DMPOPIDOR. The user can then define datasets based on the expected info from the RDA common metadata standard.
The narrative portion of the DMP will remain unchanged at this point in time.
Each plan can have n datasets.

Fields to include for each dataset:

Description
Format
Anticipated Volume
Release Timeline (Tooltip: "How long after acquisition will this data be available"
Intended Repository
License for Reuse


proposal: affected/package: image Add support for qoi image format.

Add support for qoi image format.
As qoi is a new image format with significant performance advatages over other lossless compresion such as png, I believe that it would be beneficial to implement it as part of the standard library.
This would be beneficial over using third party modules as it means there will be a consistent and stable api which will be consistent with jpeg and png images.

Example usecase
For uses such as rendering fractals png can be slow to compress and save the images. The qoi image format allows this to be done far quicker and has similar levels of compression

I would be willing to implement this.

Marking for x/image
Seems like something that can and should live outside of the standard library https://go.dev/doc/faq#x_in_std
Appears to be very new with not many users/importers https://pkg.go.dev/search?q=qoi
cc @nigeltao

I still think it would be beneficial but it is probably worth waiting to see if qoi gains wider adoption.


Add support for a storage container for many morphologies in one file

A while ago we added support for reading morphologies from groups in #163. This has led to experimental work by @1uc to see how well HDF5 files work to store many morphologies in a single file (presentation incoming). I think it would be good to consolidate the work done in our internal MorphoKit in MorphIO by

Specifying how many morphologies should be stored in a single HDF5 file

the most simplistic approach by copying the contents of morphology files into sub-groups of a larger file. This layout is already facilitated by the group access in #163 and would be quick to implement.
concatenating all datasets of the morphologies into larger datasets and storing the offset, count into said datasets per morphology. This layout would allow to later perform similar optimizations as has been done for BlueBrain/libsonata by reading the required data for several morphologies in one go from the larger datasets.

Adding a MorphologyStorage (or similar) class that takes either a directory or path to an HDF5 file and exposes a .get method to obtain morphologies stored

this can and should be expanded to provide optimized bulk access
at least a utility to create containers if not an interface to write should be included
this would be our new preferred way of reading morphologies for large circuits with a large number of unique morphologies

Adding any other functionality in the reader required to support the format

Everything above looks fine.  Please remember that this has to build under linux/mac and windows, since we produce wheels and such for all platforms.

a get method to obtain morphologies stored

What would be the argument to the get?  A number? A string?

The current solution in morpho-kit is essentially:
class MorphologyStorage {
public:
    virtual morphio::Morphology get(const std::string& morph_name) const = 0;
};

class DirectoryStorage {
public:
    DirectoryStorage(const std::string& dirname);
    morphio::Morphology get(const std::string& morph_name) const override;

public:
    const std::string& dirname;
};

class FileStorage {
public:
    FileStorage(const std::string& container_name);
    morphio::Morphology get(const std::string& morph_name) const override;

public:
    HighFive::File file;
};

The assumption is that the file name is {dirname}/{morph_name}.{ext} where there's some guessing an precedence order for picking the extension.

Everything above looks fine. Please remember that this has to build under linux/mac and windows, since we produce wheels and such for all platforms.

a get method to obtain morphologies stored

What would be the argument to the get? A number? A string?

As @1uc pointed out in the interface snippet, the morphology name as currently stored in SONATA. If we had integration with libsonata, one could actually use a node id. Not sure that this is desired, though.

Multiple circuits should be able to use the same morphology container. Hence, the node ID alone doesn't identify the morphology; conversely multiple nodes can map to the same morphology.


Unable to use functions-framework-python with pubsub beta emulator

Following the README.md, I am able to trigger functions but I am expecting 'data' as a part of the event but unfortunately the message being passed from google beta emulatar is in the following format:
{
      "subscription": "projects/test/subscriptions/test1",
      "message": {
        "data": "TWVzc2FnZSBudW1iZXIgNA==",
        "messageId": "4",
        "attributes": {}
      },
      "id": 3
    }

Is there something I am missing? Please guide!

Thanks for the report. This is a duplicate of https://github.com/GoogleCloudPlatform/functions-framework-python/issues/23.


Add markdown preview for home page edition

The home page is edited from conference tab in admin area and markdown is supported.
We should add a preview of the rendered page.

Done in 825a0111b46aa9973f826e4cfc0078199d74f4f3


DB cleaning

There are 97 users that don't have a CollectiveId (and therefore no user profile). It's breaking a few things in the code. I want to add a not null validation to the model but we first need to clean existing data. We should either remove those rows or create a user collective for them.
There are only 8 of those rows with an email address. So should be quick.

Just need an explanation on those 8 users. You can see them with this filter on the Users table:
"CollectiveId" IS NULL and "email" IS NOT NULL

@xdamman fixed the dot net one. We missed that when we created a host org for them the other day. The rest of them I have no idea why they don't have a host. Some of them are probably related to the github flow

this was done already. re-open if you find more cases.


Failed build-storybook commands

Descriptions

I want to use this action to deploy my storybook to chromatic
However build failed with error like below
Could I get help about this problem to resolve and complete upload storybook?
On my local machine, build-storybook does not throw any errors.

  {
"timestamp": "2020-07-31T02:12:53.357Z",
"sessionId": "597648fd-3812-4618-b8b3-7e5d4e5d59b5",
"gitVersion": "2.27.0",
##[error]Cannot destructure property `webUrl` of 'undefined' or 'null'.
"nodePlatform": "linux",
##[error]TypeError: Cannot destructure property `webUrl` of 'undefined' or 'null'.
    at runChromatic (/home/runner/work/_actions/chromaui/action/v1/lib/main.js:65:20)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async Promise.all (index 0)
    at async run (/home/runner/work/_actions/chromaui/action/v1/lib/main.js:120:33)
"nodeVersion": "12.13.1",
##[error]Cannot destructure property `webUrl` of 'undefined' or 'null'.
"packageName": "chromatic",
"packageVersion": "5.0.0",
"storybook": {
  "version": "5.3.19",
  "viewLayer": "react",
  "addons": [
    {
      "name": "actions",
      "packageName": "@storybook/addon-actions",
      "packageVersion": "5.3.19"
    },
    {
      "name": "links",
      "packageName": "@storybook/addon-links",
      "packageVersion": "5.3.19"
    }
  ]
},
"flags": {
  "projectToken": "***",
  "fromCI": true,
  "interactive": false,
  "exitZeroOnChanges": true,
  "exitOnceUploaded": false,
  "allowConsoleErrors": false
},
"errorType": "Error",
"errorMessage": "✖ Failed to build Storybook\nCommand failed with exit code 1: npm run --silent build-storybook -- --output-dir /tmp/chromatic-2693nz3iqGPzbF9e"
}

Error: ✖ Failed to build Storybook
Command failed with exit code 1: npm run --silent build-storybook -- --output-dir /tmp/chromatic-2693nz3iqGPzbF9e
  at makeError (/home/runner/work/_actions/chromaui/action/v1/node_modules/execa/lib/error.js:59:11)
  at handlePromise (/home/runner/work/_actions/chromaui/action/v1/node_modules/execa/index.js:114:26)
  at processTicksAndRejections (internal/process/task_queues.js:93:5)
  at async buildStorybook (/home/runner/work/_actions/chromaui/action/v1/node_modules/chromatic/bin/tasks/build.js:50:5)
  at async Task.task (/home/runner/work/_actions/chromaui/action/v1/node_modules/chromatic/bin/lib/tasks.js:13:7)

Duplicate of https://github.com/chromaui/action/issues/36

I found out a workaround. First build and use this action.
For example like below.
      - run: npm run build-storybook
      - uses: chromaui/action@v1
        with:
          projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }}
          token: ${{ secrets.GITHUB_TOKEN }}
          storybookBuildDir: storybook-static

When I did that it still doesn't work.


Bug: (Exception occurs in markdown/no-missing-label-refs when links from the second onwards have undefined labels)

Environment
ESLint version: 9.12.0
@eslint/markdown version: 9.12.0
Node version: 20.9.0
npm version: 10.8.3
Operating System: Windows 11
Which language are you using?
commonmark
What did you do?

Configuration
import markdown from "@eslint/markdown";

export default [
    {
        files: ["**/*.md"],
        plugins: {
            markdown
        },
        language: "markdown/commonmark",
        rules: {
            "markdown/no-missing-label-refs": "error",
        },
    }
];

Create a Markdown file with the following content:[Foo][foo]
[Bar][]

Enable the @eslint/markdown plugin and the markdown/no-missing-label-refs rule in your ESLint configuration.
Run ESLint on the Markdown file.

What did you expect to happen?
  1:7  error  Label reference 'foo' not found  markdown/no-missing-label-refs
  2:2  error  Label reference 'Bar' not found  markdown/no-missing-label-refs

What actually happened?
ESLint throws the following exception and stops execution:
Oops! Something went wrong! :(

ESLint: 9.12.0

TypeError: Cannot read properties of undefined (reading 'length')
Occurred while linting ...\eslint-markdown-sample\docs\sample.md:1
Rule: "markdown/no-missing-label-refs"
    at findMissingReferences (.../eslint-markdown-sample/node_modules/@eslint/markdown/dist/esm/index.js:1509:25)
    at text (.../eslint-markdown-sample/node_modules/@eslint/markdown/dist/esm/index.js:1580:9)
    at ruleErrorHandler (...\eslint-markdown-sample\node_modules\eslint\lib\linter\linter.js:1084:48)
    at ...\eslint-markdown-sample\node_modules\eslint\lib\linter\safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
    at Object.emit (...\eslint-markdown-sample\node_modules\eslint\lib\linter\safe-emitter.js:45:38)
    at NodeEventGenerator.applySelector (...\eslint-markdown-sample\node_modules\eslint\lib\linter\node-event-generator.js:297:26)
    at NodeEventGenerator.applySelectors (...\eslint-markdown-sample\node_modules\eslint\lib\linter\node-event-generator.js:326:22)
    at NodeEventGenerator.enterNode (...\eslint-markdown-sample\node_modules\eslint\lib\linter\node-event-generator.js:337:14)
    at runRules (...\eslint-markdown-sample\node_modules\eslint\lib\linter\linter.js:1128:40)

Link to Minimal Reproducible Example
https://stackblitz.com/edit/stackblitz-starters-bhmkmm?file=package.json
Participation

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

Additional comments
Failing Test Case
The following test case results in the exception being thrown:
// tests/rules/no-missing-label-refs.test.js

invalid: [
    {
        code: "[Foo][foo]\n[Bar][]",
        errors: [
            {
                messageId: "notFound",
                data: { label: "foo" },
                line: 1,
                column: 6,
                endLine: 1,
                endColumn: 9,
            },
            {
                messageId: "notFound",
                data: { label: "Bar" },
                line: 2,
                column: 2,
                endLine: 2,
                endColumn: 5,
            },
        ],
    },
],

Other Cases Where the Error Occurs

Case 1[Foo][]
[Bar][]

Case 2[Foo][foo]
[Bar][bar]
[Hoge][]

Case 3[Foo][foo]
[Bar][]
[Hoge][hoge]

Cases Where No Error Occurs

Case 1[Foo][]
[Bar][bar]

Case 2[Foo][foo]
[Bar][bar]

Case 3[Foo][]
[Bar][bar]
[Hoge][hoge]

I am not sure if it's related but a similar error happens when using this plugin alongside other plugins like @eslint/js
This works perfectly
import markdown from '@eslint/markdown'

export default [
  ...markdown.configs.recommended,
]

but this throws an error message
import jsEslint from '@eslint/js'
import markdown from '@eslint/markdown'

export default [
  ...markdown.configs.recommended,
  jsEslint.configs.recommended,
]

Oops! Something went wrong! :(

ESLint: 9.12.0

TypeError: Error while loading rule 'no-irregular-whitespace': sourceCode.getAllComments is not a function
Occurred while linting C:\Users\slzr\Dev\Personal\three-app\packages\core\readme.md
    at Object.create (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\rules\no-irregular-whitespace.js:87:41)
    at createRuleListeners (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:943:21)
    at C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:1068:84
    at Array.forEach (<anonymous>)
    at runRules (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:999:34)
    at #flatVerifyWithoutProcessors (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:1914:31)
    at Linter._verifyWithFlatConfigArrayAndWithoutProcessors (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:1995:49)
    at Linter._verifyWithFlatConfigArray (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:2084:21)
    at Linter.verify (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:1528:61)
    at Linter.verifyAndFix (C:\Users\slzr\Dev\Personal\three-app\node_modules\eslint\lib\linter\linter.js:2322:29)

@Ymmy833y thanks for the report and all the details. I can reproduce this.

@salazarr-js it isn't related. jsEslint.configs.recommended doesn't specify files, so in your configuration JS recommended rules apply to all files. You can fix this in your config file by adding files like this:
import jsEslint from '@eslint/js'
import markdown from '@eslint/markdown'

export default [
  ...markdown.configs.recommended,
  {
    ...jsEslint.configs.recommended,
    files: ["**/*.js", "**/*.mjs", "**/*.cjs"]
  }
]

I'm working on this.

@mdjermanovic thanks for that tip, it helped me a lot and made me understand a little more how eslint and markdown plugins work, one thing that has bothered me is that @eslint/js can work perfectly alongside other plugins like typescript-eslint inclusive with eslint-plugin-vue, but when I add the @eslint/markdown it starts to throws those errors unless I made the change that you showed me before
This is the eslint-plugin-vue recommended config setup
import js from '@eslint/js'
import ts from 'typescript-eslint'
import vue from 'eslint-plugin-vue'

export default ts.config(
  js.configs.recommended,
  ...ts.configs.recommended,
  ...vue.configs['flat/recommended'],
  {
    files: ['*.vue', '**/*.vue'],
    languageOptions: {
      parserOptions: {
        parser: '@typescript-eslint/parser'
      }
    }
  }
)

but to be able to use @eslint/markdown the only way to make everything work is with this workaround
import md from '@eslint/markdown'
import js from '@eslint/js'
import ts from 'typescript-eslint'
import vue from 'eslint-plugin-vue'

export default ts.config(
  { ignores: ['**/.vitepress'] },

  {
    ...md.configs.recommended[0],
    language: 'markdown/gfm'
  },
  {
    files: ['**/*.ts'],
    extends: [
      js.configs.recommended,
      ...ts.configs.recommended,
    ],
  },
  {
    files: ['*.vue', '**/*.vue'],
    extends: [
      ...vue.configs['flat/recommended'],
    ],
  },
  {
    files: ['*.vue', '**/*.vue'],
    languageOptions: {
      parserOptions: {
        parser: '@typescript-eslint/parser'
      }
    }
  },
)

one thing that has bothered me is that @eslint/js can work perfectly alongside other plugins like typescript-eslint inclusive with eslint-plugin-vue, but when I add the @eslint/markdown it starts to throws those errors unless I made the change that you showed me before

Those plugins provide parsers that produce ASTs compatible with the AST specification expected by the core JS rules (ESTree) and/or processors that extract JS (or JS-like) code from the files. So, in these cases, core JS rules get a JS-like source code and a JS-like AST to work with. In most cases, that works perfectly fine.
Markdown source code is very different from JS source code, and their ASTs are very different, so core JS rules cannot work with it. When you add jsEslint.configs.recommended without files, that means core JS rules are enabled on all files, including .md files (for which the linting is enabled by including markdown.configs.recommended). Then, a core JS rule gets a markdown file to work with and crashes due to an unexpected source code structure or AST.

@mdjermanovic that was a masterclass! Thanks for taking the time to explain it and share it with us 🙋🏽♂


v2.4.9 doesn't contain any pages

Is there a reason why 2.4.9 release (I mean downloadable ZIP file - https://github.com/almasaeed2010/AdminLTE/archive/v2.4.9.zip) doesn't contain any of HTML pages, documentation, and other stuff, while all other releases contained these pages?
In other words, 2.4.9 only contains several CSS/JS files, and this is it.
P.S. All JS files in 2.4.9 release say 2.4.8 for some reason too.

Yep, there is no any HTML code....

It doesn't have folder plugins.
How to use any plugin, for example icheck?

@sihar, you need to download v2.4.8, then download v2.4.9 and manually copy files from 2.4.9 to 2.4.8, so you will have all the HTML/plugins stuff + some JS files from latest (2.4.9) version. Just not sure why @almasaeed2010 didn't include HTML stuff in 2.4.9 by default.

@toxpal, thank you for the answer. I am just want to know, how if we develop in laptop and deliver in server, should I do same task again? Usually I do composer update in server that done

Yes, just upload all the new files to server, and you should be done.

Fixed on 2.4.10.
Thanks!


[Feature Request] Add a way for a user to cancel an image submission from makubot's message

Have there be a reaction on the message makubot sends to indicate receipt of an image add request. There should be a line in the message indicating what that reaction does. Have there be a confirm-to-delete interaction (and a timeout for response).

Pressing the X will trigger the confirm-to-delete interaction, which should be a simple "Are you sure you wish to cancel this request? [y/n]" that accepts one of ["y","n","yes","no"] (.lower() the input).
The Result of the cancellation should be similar to when an image is not approved but automated and with a different resulting message to the user.

Oooo I love this!

Meant to mention it for being explicit about it, but only the user who made the request should be allowed to press the X and have anything happen. If another user reacts to it, nothing happens (or an error message depending on what you want).

If the user adds multiple images at a time, what should happen? Since at the moment, there's just one response message.
I personally think we should have the X cancel all the image requests.

Actually, I might not do this... I add a reaction to images to approve/deny them in my DMs, and there was previously a problem where makubot gets super slow when it's watching all those messages. Because of that, I had to only watch the 3 most recent image approval messages, so that Makubot wouldn't get super slow. If we're watching a message for every image addition request, even divided by the number of URLs per message, it'll slow down a lot.

It wouldn't have to watch them indefinitely. I think it's safe to assume that after say, 60 seconds, if the user hasn't cancelled it then they aren't going to. You could have the reaction remove itself after that period of time and stop watching then.


autorefresh callback from python client

Is your feature request related to a problem? Please describe.
I am often editing datasets and would like to refresh open browsers automatically.
Describe the solution you'd like
Some callback after operations that might require a refresh.
Describe alternatives you've considered
Manually refreshing
Additional context
Something similar to the fastapi/uvicorn --reload param

@frascuchon @dvsrepo @keithCuniah @rhoitjadhav

@keithCuniah webhook-master


[Table Storage] Azurite filters not matching production filters for null values

Which service(blob, file, queue, table) does this issue concern?
Table
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
Visual Studio Professional 2022 (v 17.8.4)
What problem was encountered?
When trying to query for properties with null values or without them the query is different when using azurite and when using the real azure storage.
The following will filter out null values in azure storage but not in azurite
[PROPERTY_NAME] lt ''
While the following will filter out null values in azurite but not in azure storage
[PROPERTY_NAME] gt ''
Steps to reproduce the issue?
Create a table, add two entities with a few properties, leave one property as NULL  in one of the entities and execute a query using that property with the filters described above.

@pizerg
Thanks for raising this issue!
Would you please give the detail repro steps, better with Azurite debug log?
Like how to create such a entity with null properties.

@pizerg
Thanks for raising this issue!
Would you please give the detail repro steps, better with Azurite debug log? Like how to create such an entity with null properties.

@blueww
You can use the following entity (C#), just set Foo to null in some of the entities you insert for the test and query for it using Foo lt '' and Foo gt '' to see the different results obtained in Azurite vs Azure Table Storage:
public class TestEntity : ITableEntity
{
    public string PartitionKey { get; set; }
    public string RowKey { get; set; }
    public ETag ETag { get; set; }
    public DateTimeOffset? Timestamp { get; set; }

    public double? Foo { get; set; }
    public double? Bar { get; set; }
}

@pizerg
I tried to repro your issue, but I can't repro it.
In my test, both Azurite and product Azure storage server will filter out null values with foo gt ''.
And foo lt '' not work for both Azurite and product Azure storage server, the result is always empty with this filter.
Following is the network trace of my testing with real product Azure storage server. You can see foo gt '' works, which is different from your description.
list all table entities, you can see 2 of them has foo as null
GET https://accountname.table.core.windows.net/testtable?$top=20&$select=RowKey%2Cfoo%2Cfoo2%2Cuserid%2CPartitionKey%2CTimestamp HTTP/1.1
Accept-Charset: UTF-8
MaxDataServiceVersion: 3.0;NetFx
Accept: application/json; odata=minimalmetadata
DataServiceVersion: 3.0;
x-ms-client-request-id: 3efc1636-2d7b-47ea-87b4-f7fd43035809
User-Agent: Azure-Cosmos-Table/1.0.8 (.NET CLR 4.0.30319.42000; Win32NT 10.0.22631.0)
x-ms-version: 2017-07-29
x-ms-date: Thu, 18 Jan 2024 08:00:25 GMT
Authorization: SharedKey accountname:[hidden]
Host: accountname.table.core.windows.net

HTTP/1.1 200 OK
Cache-Control: no-cache
Transfer-Encoding: chunked
Content-Type: application/json;odata=minimalmetadata;streaming=true;charset=utf-8
Server: Windows-Azure-Table/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: f1da65bd-4002-0068-1be4-49bc52000000
x-ms-version: 2017-07-29
X-Content-Type-Options: nosniff
Date: Thu, 18 Jan 2024 08:00:59 GMT

2C4
{"odata.metadata":"https://accountname.table.core.windows.net/$metadata#testtable&$select=RowKey,foo,foo2,userid,PartitionKey,Timestamp","value":[{"odata.etag":"W/\"datetime'2024-01-18T07%3A42%3A04.5721829Z'\"","PartitionKey":"partition1","RowKey":"1","Timestamp":"2024-01-18T07:42:04.5721829Z","foo":null,"foo2":null,"userid":null},{"odata.etag":"W/\"datetime'2024-01-18T07%3A41%3A15.9849281Z'\"","PartitionKey":"partition1","RowKey":"2","Timestamp":"2024-01-18T07:41:15.9849281Z","foo":"TestVal","foo2":null,"userid":null},{"odata.etag":"W/\"datetime'2024-01-18T07%3A49%3A52.3712301Z'\"","PartitionKey":"partition1","RowKey":"3","Timestamp":"2024-01-18T07:49:52.3712301Z","foo":null,"foo2":null,"userid":null}]}
0

list with filter foo gt '' , the entity with foo not equal to null will return.

------------------------------------------------------------------
GET https://accountname.table.core.windows.net/testtable?$filter=foo%20gt%20%27%27&$top=20&$select=RowKey%2Cfoo%2Cfoo2%2Cuserid%2CPartitionKey%2CTimestamp HTTP/1.1
Accept-Charset: UTF-8
MaxDataServiceVersion: 3.0;NetFx
Accept: application/json; odata=minimalmetadata
DataServiceVersion: 3.0;
x-ms-client-request-id: d750ac7d-1086-4dfe-9d35-4bbae9c79d60
User-Agent: Azure-Cosmos-Table/1.0.8 (.NET CLR 4.0.30319.42000; Win32NT 10.0.22631.0)
x-ms-version: 2017-07-29
x-ms-date: Thu, 18 Jan 2024 08:00:29 GMT
Authorization: SharedKey accountname:[hidden]
Host: accountname.table.core.windows.net

HTTP/1.1 200 OK
Cache-Control: no-cache
Transfer-Encoding: chunked
Content-Type: application/json;odata=minimalmetadata;streaming=true;charset=utf-8
Server: Windows-Azure-Table/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: f1da6644-4002-0068-0ae4-49bc52000000
x-ms-version: 2017-07-29
X-Content-Type-Options: nosniff
Date: Thu, 18 Jan 2024 08:01:03 GMT

14E
{"odata.metadata":"https://accountname.table.core.windows.net/$metadata#testtable&$select=RowKey,foo,foo2,userid,PartitionKey,Timestamp","value":[{"odata.etag":"W/\"datetime'2024-01-18T07%3A41%3A15.9849281Z'\"","PartitionKey":"partition1","RowKey":"2","Timestamp":"2024-01-18T07:41:15.9849281Z","foo":"TestVal","foo2":null,"userid":null}]}
0

list with filter foo lt '' , no entity will return.

------------------------------------------------------------------
GET https://accountname.table.core.windows.net/testtable?$filter=foo%20lt%20%27%27&$top=20&$select=RowKey%2Cfoo%2Cfoo2%2Cuserid%2CPartitionKey%2CTimestamp HTTP/1.1
Accept-Charset: UTF-8
MaxDataServiceVersion: 3.0;NetFx
Accept: application/json; odata=minimalmetadata
DataServiceVersion: 3.0;
x-ms-client-request-id: b9d7bf88-45a4-49d0-9cb0-e3cc9c579940
User-Agent: Azure-Cosmos-Table/1.0.8 (.NET CLR 4.0.30319.42000; Win32NT 10.0.22631.0)
x-ms-version: 2017-07-29
x-ms-date: Thu, 18 Jan 2024 08:01:51 GMT
Authorization: SharedKey accountname:[hidden]
Host: accountname.table.core.windows.net

HTTP/1.1 200 OK
Cache-Control: no-cache
Transfer-Encoding: chunked
Content-Type: application/json;odata=minimalmetadata;streaming=true;charset=utf-8
Server: Windows-Azure-Table/1.0 Microsoft-HTTPAPI/2.0
x-ms-request-id: a2e45885-b002-0053-75e4-49f9f6000000
x-ms-version: 2017-07-29
X-Content-Type-Options: nosniff
Date: Thu, 18 Jan 2024 08:02:26 GMT

8F
{"odata.metadata":"https://accountname.table.core.windows.net/$metadata#testtable&$select=RowKey,foo,foo2,userid,PartitionKey,Timestamp","value":[]}
0

@blueww
Thanks for your tests, it seems that with String data fields, it works as expected on both ends (azurite + azure product), however if you use Double or DateTime you'll see that the issue arises as described in my initial message presenting differences in azurite vs azure product.

@pizerg
I can repro this issue with Double or DateTime type entity.
It looks the reason is in JS (Azurite based on JS), a none null Double or DateTime value is taken as greater than ''.
But on Azure server, from the query result it looks a none null Double or DateTime value is taken as less than ''.
I have tried  foo ne '', it looks can filter out null value, and only return none null value on both Azure server and Azurite. Not sure if it can meet your requirement.
If we want to make Azurite works same as Table on the specific value query. We need change the compare code like: https://github.com/Azure/Azurite/blob/d544d16f910e490fdd9db5565459df701895308f/src/table/persistence/QueryInterpreter/QueryNodes/GreaterThanNode.ts#L29
https://github.com/Azure/Azurite/blob/d544d16f910e490fdd9db5565459df701895308f/src/table/persistence/QueryInterpreter/QueryNodes/LessThanNode.ts#L29
We need add specific code for different data type and value to simulate Azure Server behavior, instead of just compare the values. It will make the code not easy to maintain and has regression risk.  So   if the workaround can unblock you, we might won't take this issue fix as high priority. Thanks for your understanding!

@blueww
I've updated my queries to filter by ne '' and now I can get consistent results on both platforms, so for my use case this workaround is enough, thanks!

@pizerg
Thanks for your confirmation!
Good to know the workaround works for you.

Different behaviour between Azurite and real Azure Table storage also exists when comparing directly to null. The filter query myCol eq null works just fine against Azurite, but fails against real Azure Table storage with error:
Exception: {"odata.error":{"code":"InvalidInput","message":{"lang":"en-US","value":"One of the request inputs is not valid.\nRequestId:10d6a929-e002-007e-4cd3-110d71000000\nTime:2024-09-28T18:25:57.3856306Z"}}}
Stack: RestError: {"odata.error":{"code":"InvalidInput","message":{"lang":"en-US","value":"One of the request inputs is not valid.\nRequestId:10d6a929-e002-007e-4cd3-110d71000000\nTime:2024-09-28T18:25:57.3856306Z"}}}
    at handleErrorResponse (C:\Users\nej\source\repos\azf-samples-ts\node_modules\@azure\core-client\dist\commonjs\deserializationPolicy.js:148:19)
    at deserializeResponseBody (C:\Users\nej\source\repos\azf-samples-ts\node_modules\@azure\core-client\dist\commonjs\deserializationPolicy.js:83:45)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5).

Different behaviour between Azurite and real Azure Table storage also exists when comparing directly to null. The filter query myCol eq null works just fine against Azurite, but fails against real Azure Table storage with error:
Exception: {"odata.error":{"code":"InvalidInput","message":{"lang":"en-US","value":"One of the request inputs is not valid.\nRequestId:10d6a929-e002-007e-4cd3-110d71000000\nTime:2024-09-28T18:25:57.3856306Z"}}}
Stack: RestError: {"odata.error":{"code":"InvalidInput","message":{"lang":"en-US","value":"One of the request inputs is not valid.\nRequestId:10d6a929-e002-007e-4cd3-110d71000000\nTime:2024-09-28T18:25:57.3856306Z"}}}
    at handleErrorResponse (C:\Users\nej\source\repos\azf-samples-ts\node_modules\@azure\core-client\dist\commonjs\deserializationPolicy.js:148:19)
    at deserializeResponseBody (C:\Users\nej\source\repos\azf-samples-ts\node_modules\@azure\core-client\dist\commonjs\deserializationPolicy.js:83:45)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5).

Hi @yohny
This is a different case than the original issue (one compare to ' ', one compare to null).
Would you please open a new issue, and follow the issue template and attach the full debug log  of Azurite. Then we will look into it.


Article date is wrong

This page received important updates after the listed 10/17/2018 date https://github.com/MicrosoftDocs/windows-uwp/commits/docs/windows-apps-src/publish/payment-thresholds-methods-and-timeframes.md

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

ID: 801e8a2c-1bc7-1abc-3c36-de1da74eb77b
Version Independent ID: 2482d516-3b32-3068-a038-ccdc3f94d7fb
Content: Payment thresholds, methods, and timeframes - UWP applications
Content Source: windows-apps-src/publish/payment-thresholds-methods-and-timeframes.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

Thanks. The changes you refer to appear in that article, but you're correct that the date was never updated. I've just merged in a change to the correct date, it should be published shortly.


Can't start frontend and backend projects at the same time

Hi!
I have a problem with the tutorial specified in this issue. It's my first time using Visual Studio 2022 (not Visual Studio Code) to create a solution consisting of a .NET backend and an Angular frontend.
I've followed all the steps until "Start the project" (including it). When you hit the Start button, two command prompts (one for the backend and the other for the frontend) should open but this is what happens to me:

I hit the Start button in VS
Only one command prompt, for the frontend, appears. Inside, it executes npm prestart, and npm start and at the end it shows the message Angular Live Development Server is listening on localhost:4200, open your browser on https://localhost:4200/.
I open the frontend correctly in the browser but there is no backend connection.
I close the command prompt for the frontend.
I get a dialog window saying "There were deployment errors. Continue?".
a. If I click "No", all processes that might be running terminate and nothing happens. But I notice a new message in the Output box in VS saying An exception happened at deploy, the port 4200 configured on launch.json is unavailable.. Then I ask myself: "But... I only want to build and start the project, not deploying it...". Is this message telling me something?
b. If I click "Yes" the command prompt for the backend starts correctly, and two URL's open in the browser: https://localhost:7237/swagger/index.html and https://localhost:4200/ (the URL for the frontend, but the frontend is down at this point!).

What can I be doing wrong?
Edit:
I tried starting backend and frontend manually and it works perfect. The problem seems to be related to VS and how it starts the projects when I hit the Start button.

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

ID: 4286e883-bc14-0f1e-1262-08e8432b1b46
Version Independent ID: 09df456f-db19-ad41-9d7f-fa81ddb33c92
Content: Create an ASP.NET Core app with Angular - Visual Studio (Windows)
Content Source: docs/javascript/tutorial-asp-net-core-with-angular.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@minicatsCB -- thank you for your detailed feedback.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

If you feel this is a product issue, please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

@Mikejo5000 -- please look into this issue.

Thank you, I posted the question on Microsoft Q&A. I'm receving some help but still not solving the problem.
By the way, I was about to post it also in StackOverflow when noticed that someone has already asked the same question: https://stackoverflow.com/q/73066531
The person mentions "Script Execution permissions" (this?), but I don't know how it relates to the problem, since I can execute both backend and front running them manually in sequence (right-click on the project > Debug > Start New Instance, instead of just clicking Visual Studio "Play" button).

@minicatsCB We just updated the Publish section. You might test that out, though I'm not sure it will help with this issue:
https://learn.microsoft.com/en-us/visualstudio/javascript/tutorial-asp-net-core-with-angular?view=vs-2022#publish-the-project
Otherwise, we retested and can't repro the issue (you may want to recheck your steps).  To make sure that this is logged in the right place, please use the Report a Problem tool to report the issue. When you do this, your report becomes part of the work items that are listed in the in the Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.

Just wanted to share the solution I found in case it is useful for others.
The problem was the Node.js version: I downgraded from v18.12.1 (LTS at the time) to v16.18.1. For more details you can check here or here.


Implement case-insensitive kebab-case comparison

Implement case-insensitive comparisons for kebab-case names as described in the component-model thread:
https://github.com/WebAssembly/component-model/issues/118
This also adds previously missing validation for duplicate names in some contexts.

This doesn't yet implement case-insensitive comparisons for wit-component's decoder, though that could be added using the same KebabNamespace utility introduced here.

Would you be ok retargeting this at the wasm-tools repository?
Additionally for the changes to decoding.rs in wit-component, would it be ok to leave those out for now? Eventually these changes need to make their way into wasmparser's validation where after that it shouldn't be necessary to have the changes in decoding.rs, and for now it seems unlikely a bug will crop up due to that.

Just an FYI, but I also have a PR nearly ready for wasm-tools that brings the tooling up to date with kebab case validation in wasmparser

@peterhuene Cool, I'll wait for your changes then. Closing this PR as the code is no longer in this repo.


Add WIP Readme

Add an initial README explaining how this plugin works. Closes #5

@kyazdani42 I've added some docs to the readme and a brief intro didn't really do a super detailed project info since I'm short on time. Hopefully this helps for now


Cannot bind argument to parameter 'command' because it is an empty string.

I am getting this error when trying to use this package "Cannot bind argument to parameter 'command' because it is an empty string." all of the parameters are filled

Can you tell me which task you are using please?

I am trying to use RunSQLCommand

just let me know what information you need and I will get it to you

I've tried and failed to replicate this issue. Can you tell me what sort of SQL command you are trying to run? I've tried tried running multi line selects/updates and stored procs without any issue.

"ALTER DATABASE $(SourceDatabase) SET SINGLE_USER WITH ROLLBACK IMMEDIATE"
I have taken your powershell script out of github modified it to take parameters instead of using the vsbinding  and run it with the powershell tool and it has worked.

I've tried your command, created a variable for my build and it worked OK. Even setting the variable value to empty does not give the same error message.
Have you tried creating a SQL script and using the Run Script task? I was wondering if you get the same error message. If you do, then its starting to look like an environment problem.


chore: remove repetitive words

We could use grep -P '\b(\w+)\s+\1\b' on all files (won't find duplicates across lines, but for that we'd need comment syntax-aware tool imo.


Evaluate TREs

We would like to evaluate the

Microsoft AzureTRE
TREEHOOSE
Data Safe Haven

What are we looking for?

Control over data ingress and egress (including an approval process)
MFA on login
Environments isolated from one another
Ability to forbid incoming/outgoing network connections
Ability to forbid copy-and-paste
Ability to install (some) packages from PyPI/CRAN/others
Custom compute image and/or easy to customise default image
Logging of user actions inside the TRE
Logging of network connection attempts
Logging of infrastructure actions
Delegation of control to specified user roles (eg. permissions to turn VMs on and off)
Automatic updates
Anti-virus
Automatic backup
Ability to add/remove compute as needed
Ability to use GPU VMs
Git server (eg. GitLab/Gitea)
Collaborative document writing (eg. CodiMD)
Collaborative Jupyter notebooks
Compute beyond VMs (eg. clusters, ML workbench, databricks)
Ease of deployment
Ease of use by end-users

TODO:

[x] #8

For future reference, I have a login to deployed TREEHOOSE at https://treehoose.dev.hic.dundee.ac.uk/

Falls under SATRE WP1


full screen dashboard and creating multiple bords

I would like to propose an idea:
to make a full-screen view of the dashboard so that you can conveniently observe all servers,
even if there are 100+ of them
like this concept (that's metricbeat from elastik)

We have over 300 monitors, and a feature like this would be really helpful.

@iamkhris Approach looks great! it would be even more beneficial if it had the counts at the top, like the current dashboard does.


Add panzoom disclaimers to all panzoom files

Since all this code will be used in our exams, we should make it expressly clear when we are using code that we did not write ourselves. Therefore we should add disclaimer comments to all files imported from the panzoom package, which we did not write.

Thank you gentlemen


Unable to extract audio from an output

Hi - I'm afraid I'm very new to rust, so this problem is almost certainly with my understanding rather than the library.   But just in case I have identified a genuine issue, here's a ticket.
I'm attempting to pass a midi note to an instrument VST, and get the audio out of that VST (with the hopes of eventually playing it ;))

extern crate vst2;

use std::sync::{Arc, Mutex};
use std::path::Path;

use vst2::host::{Host, PluginLoader};
use vst2::plugin::Plugin;
use vst2::event::Event;
use vst2::buffer::AudioBuffer;

struct SampleHost;

impl Host for SampleHost {
    fn automate(&mut self, index: i32, value: f32) {
        println!("Parameter {} had its value changed to {}", index, value);
    }
}

fn main() {
    let host = Arc::new(Mutex::new(SampleHost));
    let path = Path::new("/Library/Audio/Plug-Ins/VST/helm.vst/Contents/MacOS/helm");

    let mut loader = PluginLoader::load(path, host.clone()).unwrap();
    let mut instance = loader.instance().unwrap();

    instance.init();
    let note = Event::Midi {
        data: [0, 60, 127],
        delta_frames: 0,
        detune: 0,
        live: false,
        note_length: Some(100),
        note_off_velocity: 10,
        note_offset: None
    };

    instance.process_events(vec![note]);

    let mut in1 = vec![0.0; 1024]; // just to make the frame length right
    let mut out1 = in1.clone();
    let mut out2 = in1.clone();

    let buffer = AudioBuffer::new(vec![&mut in1], vec![&mut out1, &mut out2]);
    instance.process(buffer);

    // ...
}

The way I think rust-vst2 wants me to do this is to process the buffer using the plugin instance, and then at some later point to inspect the AudioBuffer I sent into it.  However, if I attempt to access the AudioBuffer (buffer) or the one mutable vectors (in1, out1 or out2) that make it up after instance.process(buffer);, I get a borrower check error.
I can understand from the rust docs that this is expected behaviour, but does that then render it impossible to get audio data out of the VST?

What's the error?
I think you have to drop buffer to be able to access out1 and out2 afterwards.
Because it holds references to them.
Maybe do this instead:
    let mut in1 = vec![0.0; 1024]; // just to make the frame length right
    let mut out1 = in1.clone();
    let mut out2 = in1.clone();
    {
        let buffer = AudioBuffer::new(vec![&mut in1], vec![&mut out1, &mut out2]);
        instance.process(buffer);
    }

And I think you should tell the plugin the sample rate and block size before you process(). (And everytime you change the buffer size).
https://github.com/overdrivenpotato/rust-vst2/blob/master/src/plugin.rs#L81
https://github.com/overdrivenpotato/rust-vst2/blob/master/src/plugin.rs#L83

@Boscop : thanks a lot for that.  I was getting errors like:
error[E0499]: cannot borrow `out1` as mutable more than once at a time
  --> src/main.rs:85:28
   |
80 |     let buffer = AudioBuffer::new(vec![&mut in1], vec![&mut out1, &mut out2]);
   |                                                             ---- first mutable borrow occurs here
...
85 |     println!("{:#?}", &mut out1);
   |                            ^^^^ second mutable borrow occurs here
...
92 | }
   | - first borrow ends here

As you explained, it was because the buffer had not yet been dropped.  Once I limited the scope of the buffer, and only accessed out1, out2 or in1 after that, the borrower was happy again :)
I've also taken your advice about setting up the plugin with the sample rate and block size - will take a look at loading a 'debugging-based' plugin for use in other hosts.  I think that will help me also know what midi events I need to send to my current plugin to get it to work :)


invite: Change setup tips banner from warning to info and remove redundant code.

This PR contains the following changes,

Change banner type from warning to info.
Rename "setup_tips_warning" classname to "setup_tips_banner". (part of the same commit as 1)
Remove redundant param "banner_html" which was introduced in eecb611789aa61ee84c6ffc8d2f5bafb333da4c7

CZO: link

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @sayamsamal!


[Gif Search] Define Download location

Extension
https://raycast.com/josephschmitt/gif-search
Description
Please add the option to define a custom download location instead of using the default download folder.
Who will benefit from this feature?
This will ensure that the downloads folder remains uncluttered by GIFs. By having a single location to review already downloaded GIFs, everything stays organized.
Anything else?
No response

@raycastbot assign me

Thank you for picking this up - @ridemountainpig
I know it's a niche ask - but makes big QOL for me.


ci(release): change ubuntu-18 to latest (#147)

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


No Documentation for "data" fields in Ansible modules

Scenario/Intent
Use the documentation provided here to see options for data fields in all modules.
Environment Details

Module Version: hewlettpackardenterprise/oneview-ansible-debian:latest Docker image
Ansible Version: 2.8.0 (from Docker container)
OneView Appliance Version: 4.20
OneView Client API Version: 900
Python Version: 2.7.16 (from Docker container)
python-hpOneView SDK Version: 5.0.0b0 (from Docker container)
Platform: Docker running on CentOS 7. Using the debian-based Docker container

Steps to Reproduce
Go to the documentation provided here and look at the data field for any module.
Expected Result
Each possible option for each data field of each Ansible module is listed with a description.
Actual Result
The only information about the data fields for each module are given in the examples. The data fields do not list all possible options for each module, which makes it difficult to use any options besides those provided in the examples.
For example, looking at the data field for the oneview_logical_enclosure, this is all we see

List with Logical Enclosure properties and its associated states.

If you wanted to update the firmware, you might look at the example

data:
name: "Encl1"
firmware:
firmwareBaselineUri: "/rest/firmware-drivers/SPPGen9Snap3_2015_0221_71"
firmwareUpdateOn: "EnclosureOnly"
forceInstallFirmware: "false"

However, none of these options have any explanation. For instance, what are the other options besides EnclosureOnly? What would be the option to specify ALL hardware in the enclosure?

@ikegentz, Adding explanation for all the data fields here is not possible because of the number of fields available for each resource. Please refer the OneView API documentation to get a detailed explanation of all the fields available for each resource.
For "data" fields, refer request body of each endpoint.


WRITEPOD: Implement writePod() to push Solid Pod datafile from app

Depends

[x] https://github.com/anusii/solidpod/issues/77

Description
A Solid Pod app will want to write (encrypted) datafiles (ttl) to the data folder on the Pod. We also want to hide any complexity from the app developer.
This task is to implement writePod() in solidpod and demonstrate it's use in keypod.

[x] Check style guidelines - is this writePod() or WritePod().

writePod() needs the following parameters:
TODO

[x] Get master password from local secure storage, otherwise ask user to provide the master password
[x] Verify master password using the verification key stored in PODs
[x] Use a popup window for user to provide the master password if verification failed
[x] Encrypt data and format encrypted data in TTL format (if data file already exists, reuse its individual key and IV)
[x]  Save encrypted data to PODs (delete the file first if it already exists)
[x] Update ind-key.ttl with the individual key and IV for the encrypted data (if applicable)
[x] Add an editable table for user to input key/value pairs in keypod and write data in TTL format (e.g. using rdflib)

Why
So that a Solid Pod based app developer can very simply push data to the pod.
Closing Criteria

[x] KeyPod demonstrates the use of writePod() by saving a sample text to a file in the pod, then retrieving it to then display and confirm.

A keypod demo showing:

Write one key-value pair to POD
Read that key-value pair from POD and display
Add one more key-value pair and then write to POD

https://github.com/anusii/solidpod/assets/13044715/7e2d0b68-9019-485c-93d1-dd9dfd67c2bb


Can not load global module 'eslint-plugin-html'.

StandardJS version?
10.0.2
VSCode version?
1.12.1
StandardJS configuration?

"standard.enable": true,
"standard.validate": [
  "javascript",
  "javascriptreact",
  "vue",
  "html"
],
"standard.options": {
  "plugins": ["html"],
  "parser": "babel-eslint",
  "ignore": [
    "node_modules/**"
  ],
  "globals": ["$", "React", "Vue"],
  "parserOptions": {
    "ecmaVersion": 7,
    "sourceType": "module",
    "ecmaFeatures": {
      "jsx": true
    }
  },
  "env": {
    "commonjs": true,
    "node": true,
    "browser": true
  }
}

Problem:
VScode pop the following error continuously, when I'm editing JS files.

Failed to load plugin html: Cannot find module 'eslint-plugin-html'

Unless installed eslint-plugin-html module at local project.
Additional information

Looks like you have standard installed locally under your project, then you should have those plugins installed locally too. Or it will fail to resolve.

Can we fallback to global module, when local node_modules doesn't have it?

I'm not sure if it will work, you can check the standardjs doc https://github.com/feross/standard#how-do-i-use-experimental-javascript-es-next-features:

If standard is installed globally (i.e. npm install standard --global), then be sure to install babel-eslint globally as well, with npm install babel-eslint --global.

Maybe you could raise an issue there :)

Oh...it's not working. 😞


Fixed bug in the EPG license detection logic. We now also check for NOKEY.

This is a real problem for people using XMLTV plugins because they keep getting error messages and the guide data is not being imported. Not surprisingly, this does not prevent the plugin itself from loading, but since we check for the license key before we try to use the plugin, it fails immediately.

Thanks!


Ensure triggers are properly set up e.g. doc should not trigger integration

Fixed by https://github.com/konveyor/pelorus/pull/818


Fix the JDBC timeout error in release mode

Moving the variable old_db_name out of try block.
In debug mode, "old_db_name" inside try block was not getting free and hence at the runtime error we had the correct old_db_name.
But in release mode, "old_db_name" inside try block was getting free and it was leading to segmentation fault in the case of runtime error.
Task: BABEL-3077
Signed-off-by: Shalini Lohia<EMAIL_ADDRESS>

Updated the commit message.


Can I add .env file to my project ?

Hi!
I want to add a .env file because I have to store variables depend on the environment, but I can't use config/dev.env.js because my var depend on the developer environment. How I can do that?
I want to use https://github.com/mrsteele/dotenv-webpack in my project based on webpack template, I know I have to add the plugin somewhere but where ? And the webpack.DefinePlugin overwrite process.env
Thanks.

you can add the dotenv-webpack plugins in the webpack.base.conf.js, like this:
first, you may need to import the package:
const Dotenv = require('dotenv-webpack');

then, config the path for it:
 module: {
   ...
 },
 node: {
   ...
 },
 plugins:[
    new Dotenv({
      // set the path for the dot-env, more config information you can visit the official documents
      path: path.resolve(__dirname, `../.env.${process.env.NODE_ENV}`),
    })
  ],

next, you need to disable the
 new webpack.DefinePlugin({
  'process.env': env
 }),

in  webpack.prod.conf.js, it would overwrite the dot-env.
One more step, you need to add the basic environmental flag for the build file, because the process.env.NODE_ENV have been set in the build.js file, so you don't need to set it for the production stage, but you need to add the NODE_ENV flag for the development stage in the package.json file, like this:
...
"scripts": {
"dev": "NODE_ENV=development webpack-dev-server  --inline --progress --config build/webpack.dev.conf.js",
"start": "npm run dev",
...
}
...

ok, but I have to disable also the DefinePlugin in webpack.dev.conf.js ?

@lefuturiste yes

@lefuturiste yes, sorry for missed it, and you'll got the same situation at the webpack.test.conf.js file.

@lefuturiste I was using dotenv-cli, hope that helps with you:
npm i -D dotenv-cli

   "scripts": {
-    "dev": "webpack-dev-server --inline --progress --config build/webpack.dev.conf.js",
+    "dev": "dotenv webpack-dev-server --inline --progress --config build/webpack.dev.conf.js",
-    "build": "node build/build.js"
+    "build": "dotenv node build/build.js"
   },

why not simply use https://www.npmjs.com/package/dotenv-webpack ?


Misleading error message on keras.Model initialization

System information

Ubuntu linux 20.04
From pip, tf version 2.3.0
Describe the current behavior

Consider this simple code from this tutorial
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers

assert tf.__version__ == '2.3.0'

initial_model = keras.Sequential(
    [
        keras.Input(shape=(250, 250, 3)),
        layers.Conv2D(32, 5, strides=2, activation="relu"),
        layers.Conv2D(32, 3, activation="relu"),
        layers.Conv2D(32, 3, activation="relu"),
    ]
)

feature_extractor = keras.Model(
    inputs=initial_model.inputs,
    #Note the error!
    blah=[layer.output for layer in initial_model.layers],
)

And the error is:

TypeError: ('Keyword argument not understood:', 'inputs')

Describe the expected behavior

I expect the error to contain a meaningful message, and at least to mention the incorrectly specified keyword blah. Just mentioning the one argument that is correctly written does not help. Maybe the error could be like

TypeError: ('Keyword argument not understood:', 'blah')

or

TypeError: Invalid keyword arguments

I tried running the code you have mentioned and got this error message: TypeError: ('Keyword argument not understood:', 'blah')

I suggest replace from tensorflow.keras import layers with from keras import layers

@AnimeshMaheshwari22 Could you explain why? I have other keras backends and importing just keras would risk importing the wrong keras type.

@tornikeo,
I was able to reproduce the error with TF v2.3. However, the issue seems to be fixed with the latest TF-nightly. Please find the attached gist. Thanks!

That resolves this issue. Thanks @amahendrakar!


Improve type inference of (n)one_of(_bytes).

I was playing with nom earlier today, and found that the following doesn't compile:
many1!(one_of_bytes!(b"0123456789ABCDEFabcdef"))

I tracked this down to some unusual behavior in the type inferencer, which I don't really understand. Basically, adding the _ type parameter causes the inferencer to correctly determine the type, so that the above (and similar variants) compile.
You can see an extracted version of the same problem here: http://is.gd/fuSl3z
Since I don't really understand the fix, I'm not sure you should merge this. But I assume a pull request is more useful than a bug report.

Hi!
From what I see, it would work, but I have to test it a bit before merging.  As an example, I could factor take! and take_s! in the same macro.
So, your fix could be very useful ;)

Hi!
From what I see, it would work, but I have to test it a bit before merging.  As an example, I could factor take! and take_s! in the same macro.
So, your fix could be very useful ;)

Thanks!


Rev CI/CD docs

Updates the documentation for how to use Pulumi in a CI/CD setup. I pretty much just rewrote the entire Travis doc, as the "example script" was far more complicated than it needed to be. And I think it's much more important to just explain some of the core mechanics, e.g. push jobs should run  pulumi update and pull_request jobs should run pulumi preview.
Fixes #511.

Any chance we can add one more concrete example (Jenkins or AWS Code*)

Just want to give this a big :+1:
As part of https://github.com/pulumi/docs/issues/508, can we take an afternoon to document the integration with additional CI products? E.g., (in my priority order) AWS CodePipeline, Jenkins, VSTS, CircleCI, Shippable, ...


Cannot change attributes of configuration ':composeApp:debugFrameworkIosX64' after it has been locked for mutation

Describe the bug
I'm trying to add ksp and room in my CMP project, but I always get the error from title when trying to add "dependencies { ... }" in build.gradle.kts
Affected platforms

Desktop (Windows, Linux, macOS)
iOS

Versions

Libraries:

Compose Multiplatform version:1.6.10
ksp version: 2.0.0-1.0.21
room version: 2.7.0-alpha02

Kotlin version: 2.0.0

To Reproduce
Steps to reproduce the behavior:

Add room and ksp configuration in libs.version.toml:

[versions]
...
room = "2.7.0-alpha02"
ksp = "2.0.0-1.0.21"

[libraries]
...
room-compiler = { group = "androidx.room", name = "room-compiler", version.ref = "room"}
room-runtimme = { group = "androidx.room", name = "room-runtime", version.ref = "room"}

[plugins]
...
ksp = { id = "com.google.devtools.ksp", version.ref = "ksp" }
room = { id = "androidx.room", version.ref = "room" }

Add room and ksp in build.gradle.kts

plugins {
...
alias(libs.plugins.ksp)
alias(libs.plugins.room)
}

kotlin {
...
 sourceSets {
  ...
  commonMain.dependencies {
   ...
   implementation(libs.room.runtime)
  }
 }
}

add room schema directory in build.gradle.kts

room {
    schemaDirectory("$projectDir/schemas")
}

add the dependencies block with ksp reference

    dependencies {
        add("kspAndroid", libs.room.compiler)
        add("kspIosSimulatorArm64", libs.room.compiler)
        add("kspIosX64", libs.room.compiler)
        add("kspIosArm64", libs.room.compiler)
    }

Expected behavior
Sync gradle without failure
Additional context
I could not find a proper setup tutorial for CMP, so I took Google's Frutties KMP Sample as reference, so I guess it's just a miss configuration from my part.
I have tried to place the "dependencies" block outside and inside "kotlin" block in build.gradle.kts, but it didn't work.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


fix: Semantic release setup in the monorepo

Description
Fixes semantic release setup in the new monorepo structure of the library repository. Upgrades expo and improves caching in CI workflows.

:tada: This PR is included in version 3.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Request to Salt-API does not implicitly set a value for body

When an array is passed in as part of the body, the Request library does not correctly submit the data to the Salt-API.

Not an issue, I was looking at the NPM version of the lib instead of the Git version. Any way you can publish the latest to NPM? That would resolve this issue

Sorry, I thought I already published it. Thanks for the info and contribution.
NPM version is not at 1.1.0.


Fix broken link

Filename for the "PickSaveFileAsync" page was misspelled.

@mcooley : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add kubectl shp for release

Changes

Change prefix cli to shp for release

Add kubectl plugin of shp for release

Change replacements to name_template for archive files

Because replacements  will be deleted and it's not correctly working for multiple archives.
Ref from goreleaser:
https://goreleaser.com/deprecations/?h=replacements#archivesreplacements
The replacements will be removed soon from the archives section, as it was never handled correctly when multiple archives were being used, and it also causes confusion in other places.

You can still get the same features by abusing the name_template property.

Fixes #179
Related:

#102

Tested
gorelease logs:
• writing effective config file
    • writing                                        config=dist/config.yaml
  • building binaries
    • building                                       binary=dist/shp_linux_arm64/shp
    • building                                       binary=dist/shp_linux_amd64_v1/shp
    • building                                       binary=dist/shp_darwin_amd64_v1/shp
    • building                                       binary=dist/shp_darwin_arm64/shp
    • building                                       binary=dist/shp_windows_amd64_v1/shp.exe
    • building                                       binary=dist/shp_windows_arm64/shp.exe
    • building                                       binary=dist/kubectl-shp_linux_amd64_v1/kubectl-shp
    • building                                       binary=dist/kubectl-shp_linux_arm64/kubectl-shp
    • building                                       binary=dist/kubectl-shp_darwin_amd64_v1/kubectl-shp
    • building                                       binary=dist/kubectl-shp_darwin_arm64/kubectl-shp
    • building                                       binary=dist/kubectl-shp_windows_amd64_v1/kubectl-shp.exe
    • building                                       binary=dist/kubectl-shp_windows_arm64/kubectl-shp.exe
    • took: 11m27s
  • generating changelog
  • archives
    • creating                                       archive=dist/shp_v0.99.0_linux_arm64.tar.gz
    • creating                                       archive=dist/shp_v0.99.0_linux_x86_64.tar.gz
    • creating                                       archive=dist/shp_v0.99.0_macOS_x86_64.tar.gz
    • creating                                       archive=dist/shp_v0.99.0_macOS_arm64.tar.gz
    • creating                                       archive=dist/shp_v0.99.0_windows_arm64.tar.gz
    • creating                                       archive=dist/shp_v0.99.0_windows_x86_64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_windows_arm64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_linux_x86_64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_linux_arm64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_macOS_x86_64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_macOS_arm64.tar.gz
    • creating                                       archive=dist/kubectl-shp_v0.99.0_windows_x86_64.tar.gz
    • took: 39s
  • calculating checksums
  • publishing
    • scm releases
      • creating or updating release                 repo=liangyuanpeng/shipwright-cli tag=v0.99.0
      • release updated                              name=v0.99.0 release-id=99490864 request-id=2[34](https://github.com/liangyuanpeng/shipwright-cli/actions/runs/4696180341/jobs/8326032835#step:5:35)0:450B:2DB4B57:2EEA3BB:6438D0D6
      • uploading to release                         file=dist/shp_v0.99.0_linux_arm64.tar.gz name=shp_v0.99.0_linux_arm64.tar.gz
      • uploading to release                         file=dist/shp_v0.99.0_linux_x86_64.tar.gz name=shp_v0.99.0_linux_x86_64.tar.gz
      • uploading to release                         file=dist/shp_v0.99.0_macOS_arm64.tar.gz name=shp_v0.99.0_macOS_arm64.tar.gz
      • uploading to release                         file=dist/shp_v0.99.0_macOS_x86_64.tar.gz name=shp_v0.99.0_macOS_x86_64.tar.gz
      • uploading to release                         file=dist/shp_v0.99.0_windows_arm64.tar.gz name=shp_v0.99.0_windows_arm64.tar.gz
      • uploading to release                         file=dist/shp_v0.99.0_windows_x86_64.tar.gz name=shp_v0.99.0_windows_x86_64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_windows_arm64.tar.gz name=kubectl-shp_v0.99.0_windows_arm64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_linux_x86_64.tar.gz name=kubectl-shp_v0.99.0_linux_x86_64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_linux_arm64.tar.gz name=kubectl-shp_v0.99.0_linux_arm64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_macOS_x86_64.tar.gz name=kubectl-shp_v0.99.0_macOS_x86_64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_macOS_arm64.tar.gz name=kubectl-shp_v0.99.0_macOS_arm64.tar.gz
      • uploading to release                         file=dist/kubectl-shp_v0.99.0_windows_x86_64.tar.gz name=kubectl-shp_v0.99.0_windows_x86_64.tar.gz
      • uploading to release                         file=dist/checksums.txt name=checksums.txt
      • published                                    url=https://github.com/liangyuanpeng/shipwright-cli/releases/tag/v0.99.0
      • took: 11s

More info at https://github.com/liangyuanpeng/shipwright-cli/actions/runs/4696180341/jobs/8326032835

Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[ ] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
Maybe should be add release note after finished for krew?
Release kubectl shp binary

@qu1queee Thanks for remind , I think it's ready for review.

/lgtm
Thanks for the contribution, @liangyuanpeng! 🙇♀
/assign @SaschaSchwarze0
/cc @akram

/retest

@liangyuanpeng from the Shipwright Refinement meeting, can we help you here?

/retest

Tests have no logs anymore and I cannot re-run them from the GitHub Actions UI. Will try another retry here and if this does not work, I am afraid I have to do a close and reopen.
/honk

/retest

@liangyuanpeng Could you have a look into rebasing this PR? The feature branch is behind some 15 commits. Locally, on the latest commit, I do not get the golangci-lint error that is showing up here in the Verify failure.

@HeavyWombat checking on the status of this. Should another community member rebase and re-submit this PR?

@HeavyWombat checking on the status of this. Should another community member rebase and re-submit this PR?

I'll have a look tomorrow.