Suffix webhook nicknames with (IRC)

This needs to make sure not to break the avatar matching in the current setup.
It might also not be necessary. I think messages sent through webhooks in this manner have the BOT tag suffixed on the username, which might make this clear enough.

I don't feel too strongly about it either way, but I do think it could be a bit confusing for users that aren't aware of the IRC bridge. With that being said, I'm not sure if (IRC) will help that much at all in that case though, so let's leave it for now.


ColumnView contains a child which type does not exist in the binding

I try to access to the ColumnView's header:
let column_view = gtk::ColumnView::new(Option::<gtk::NoSelection>::None);
let header = column_view.first_child().unwrap();
dbg!(header.type_());

This code has next output:
header.type_() = GtkColumnViewRowWidget

Type GtkColumnViewRowWidget is similar to GtkColumnViewRow, but it's different types.
Solution
Generate code for GtkColumnViewRow.

It's not public API from what I can see. Why do you need bindings for it, what are you trying to do?

It's not public API from what I can see. Why do you need bindings for it, what are you trying to do?

I need to make the column header fixed (header stays on top when scrolling). I thought manipulating the ColumnView child object would help me.

That is expected, widgets and any library actually, can have internals that are not part of the public API.
The thing you are trying to achieve is probably something better discussed in the issues tracker of the GTK project itself. The bindings can't do anything here.


Request of adding a webhook in appinventor-sources repo

Hello App Inventor Develop Team
I'm Colin and you may know that I'm from China. In China, we have a unique network environment which causes some people in China may not able to visit Github or it is very slow to load data from it, so to pull appinventor sources from Github is almost impossible for them.
To solve this, I had wrote a bot: https://github.com/ColinTree-bot/webhook . One task of this bot is to keep code mirror of App Inventor on https://gitee.com/mit-cml/appinventor-sources up to date, and currently it syncs codes once an hour. (Gitee is a service provider in China and able to speed up local cloning.)
So if the team allow to, please add a webhook http://bot.colintree.cn:6474/mit-cml/appinventor-sources with push hooks only in appinventor-sources repo.
Thanks for your time!
Best wishes,
Colin Yang

@ColinTree I understand your concern about being able to keep a reliable mirror in China. However, I'd be a bit concerned about having a webhook we can't validate being responsible for the synching. If possible, would you be willing to share the source so that we could deploy the webhook at MIT?

@ewpatton Fully understandable sir, it's right here, https://github.com/ColinTree-bot/webhook/blob/master/webhook.js#L112
P.S. I will possibly rewrite this with koa in the future

Oh sorry did you mean share the organization on gitee? And then yes of course I can even transfer the ownership to MIT team.

@ColinTree It is set up. I've sent you email with an SSH public key you'll need to add to allow the server to push to the Gitee repo.

I had just deployed the key, it should now works.
Thanks you and the whole team!


Question: Is there an option to generate icon only for one platform?

I would like to have different icons for Android and iOS (just for the background). Thanks. Great Work!

Hi @vieraleonel great idea! I was thinking that maybe just adding a 'platform' flag would work, so it would be two calls like this:
app-icon -i android.png --platform android
app-icon -i ios.png --platform ios

This means we can add one new flag, instead of potentially two (like --icon-ios or --icon-android). Would that work for you?

That would be excellent!

Try installing version 0.5, just published:
app-icon generate -i android.png --platforms android
app-icon generate -i ios.png --platforms ios

Working great! Just to make it perfect you should update README and --help option. I can make a PR this weekend if you like.

Sorry I just see --help option has the documentation. So only a README update

@vieraleonel good point, I totally forgot the README! I'll update it 👍

I think we're covered with #35 now 😄


release-20.1: cli/zip: misc fixes

PR: https://github.com/cockroachdb/cockroach/pull/48094
From release notes:

cockroach debug zip now tries multiple times to retrieve data using SQL if it encounters retry errors and skips over fully decommissioned nodes. It also supports two command-line parameters --nodes and --exclude-nodes. When specified, they control which nodes are inspected when gathering the data. This makes it possible to focus on a group of nodes of interest in a large cluster, or to exclude nodes that cockroach debug zip would have trouble reaching otherwise. Both flags accept a list of individual node IDs or ranges of node IDs, e.g. --nodes=1,10,13-15. [#48094][#48094] {% comment %}doc{% endcomment %}
The copy of system and crdb_internal tables extracted by cockroach debug zip is now written using the TSV format (inside the .zip file), instead of an ASCII-art table. [#48094][#48094] {% comment %}doc{% endcomment %}

Closing, as --nodes and --exclude-nodes have already been documented, and I think the rest don't require documentation.

Reopening because I realized this is for a v20.1 backport!


how to query messages with rocketmq-v4.8.0

producer.queryMessage has been deprecated, then how to query messages in rocketmq. I found an API called  MQAdmin.queryMessage, but i don't know how to use it. could you give an example.

org.apache.rocketmq.tools.command.message.QueryMsgByUniqueKeySubCommand

org.apache.rocketmq.tools.command.message.QueryMsgByUniqueKeySubCommand
org.apache.rocketmq.tools.command.message.QueryMsgByKeySubCommand
You can look at these two class

Ok,thank you.  Finally,  I found an API called DefaultMQAdminExt.queryMessage which is just what i want , and i know  the API is based on tools.command.


Gitleaks/OSV/OWASP CLI arg examples STO-7278-v2

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

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

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

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


Support for target: 'experimental-serverless-trace'

Can you also add for target: 'experimental-serverless-trace'.
I'm using firebase, but the build returns an error:

Build error occurred
Error: ENOENT: no such file or directory, open 'google/protobuf/api.proto'

Previously used, target: 'serverless' is going to be deprecated.
Reference: https://github.com/zeit/next.js/issues/11860

Hi @theindiansoftwarecompany,
Thank you for using next-on-netlify and for posting an issue here! :slightly_smiling_face:
I had not heard of target: 'experimental-serverless-trace' before, but I just started some research and some experimentation.
I'm not super familiar with Firebase. Do you have a minimal repository that reproduces the issue? That would be super helpful!
We'll find a solution! Either by adding support for this new NextJS target or by manually including the google/protobuf/api.proto files. I found an interesting thread here: https://github.com/netlify/netlify-lambda/issues/112

I just tried spinning up a demo using target: 'experimental-serverless-trace'. It works!

Repo: https://github.com/FinnWoelm/next-on-netlify-demo-serverless-trace
Demo: https://next-on-netlify-demo-serverless-trace.netlify.app/

I basically just had to manually install three packages, that NextJS does not seem to bundle when using experimental-serverless-trace: node-sass, sass, and fibers:
npm install node-sass sass fibers

I'm not using Firebase, so we'll need to see how it works with that. If you can send a minimal repo, that would be awesome!

Hey @theindiansoftwarecompany,
Are you still needing support for this? Did you get a chance to set up a minimal repo where I can reproduce the error? I'm happy to look into it.
Otherwise, I'll be closing this issue in a few days. We can always reopen it if the need arises.
Hope you're doing well!

Hi @theindiansoftwarecompany,
I'm going to close this issue for now. Feel free to reopen it if you are still interested in resolving this issue! I'm here and happy to help :)
Happy hacking :fire:
Finn


docs: Update guidelines to URL-encode stream name or topic name.

Updated guidelines to URL-encode stream name and topic name. It will especially help
them who use emoji in the stream name and topic name
Fixes #16430.

Merged as 1abae8baaa94f8fe437bd129ddbb714a78a19cad, after a bit of editing and making sure we updated the non-indented copy as well.
Thanks @akshatdalton!


I added a Chinese version

I added a Chinese version,
and I hope more people will see it.
If you can。 Thank you！

@BolexLiu Thanks for this, please keep updating the Chinese version. 👍

Of course ！ ^-^


Fix: Rename method

This PR

[x] renames Authentication::check() to Authentication::isAuthenticated()

💁♂ I believe something like isAuthenticated() or isSignedIn() or isLoggedIn() reveals the intent better than something as generic as check(). It also aligns nicely with the Authentication::authenticate().

Thank you, @chartjes!


Recycle processors

Hey,
Just to share a (rather dirty) test I've made to drop processors outside the render thread. Very probably not something to merge as is but can be a basis for discussion
Several notes:

it implies adding a method on the AudioProcessor trait
I chose to send back the processors to the control thread rather than in the garbage collector thread, because it opens the room for some node pooling system I guess
I just abused the event system to make it work without much modifications, but probably a dedicated channel would be better.

Hey, thanks for getting the conversation started!
What's the main motivation for processor reuse? Do the browser implementations use this as well?
I would be very surprised if it actually brings a performance improvement. Most nodes are extremely lightweight. But I would love to be proven wrong. Perhaps we can think of a benchmark to validate this?
Some other remarks:

if the main motivation is to prevent deallocations on the render thread, I think we can come up with easier solutions than full AudioProcessor shipments.
why recycle back to the control thread? We could also stash them in the render thread which may be safer? Although this might put some administrative overhead on the render thread
I wonder how the actual recycling will work. Will you recycle the control side nodes as well?
resetting the state of the processor will be quite a lot of code in total I think, and it is probably error prone

In any case, please proceed. Interesting stuff

Hey,
Actually, the main goal is rather to find a solution to not deallocate the processors in the render thread, sorry for the confusing title. I really think it creates issues that get more apparent with the javascript GC.
For example, if we take the many_oscillator_with_env example (which is far from being the worse case we can imagine), we span 5 processors (2 nodes and 3 params) every 50ms, which means 100 processors per second. It may not seems that much, but I checked the GC calls in Node with this example and they appear every 4-5 seconds on my Mac (not very stable and probably different on other platforms, but that gives an order of magnitude). Then, if I understand well, that means we just drop ~500 processors at once when the javascript release ownership of the audio nodes, and not 5 of them every 50ms as in raw Rust. That's very probably not the only problem, but I think this is part of the perf issues I observe on Raspberry Pi.
The idea of sending the renderers back to the control thread was more to not close the door to an eventual future renderer pooling system, but I have no clear idea of how this could work nor of any eventual perf improvement. I just remember having this conversation w/ Paul saying he implemented such thing in Firefox, but maybe it was a bit late so let's not take this for granted :).

if the main motivation is to prevent deallocations on the render thread, I think we can come up with easier solutions than full AudioProcessor shipments.

Cool, let me know your ideas, I can try to prototype something
Side note:
Just an idea but maybe we could also refine Node:can_drop to also drop renderer of sources (i.e. AudioBufferSource, Oscillator) once they are stopped, even if the related control node as not been dropped itself, as they cannot be restarted in any case?

Sorry for the delay in response!
Perhaps a good experiment would be to replace can_drop with false. Effectively creating a resource leak because AudioProcessors are not deallocated. Does this solve stuttering? If not, there may be other issues at play.
Unfortunately I'm totally not familiar with napi-rs so I don't know how rust deallocation works together with the NodeJs GC. But I presume we only need to take care of heap-allocated objects (such as AudioBuffer) and for example not a pure stack-object such as the GainRenderer.
Looking at the AudioBufferSourceRenderer, only the buffer: Option<AudioBuffer> field is heap allocated. To prevent deallocation of it on the render thread, we can use the pattern that @uklotzde introduced for deallocating control messages in a separate thread. In short, the control thread supplies an llq::Node in advance, which the renderer holds on to, but when it drops it uses the link to send the heap allocated object to the deallocation thread.
I could whip up a prototype if you are unsure what I mean

Hey,
No problem I was in holidays anyway.

Perhaps a good experiment would be to replace can_drop with false. Effectively creating a resource leak because AudioProcessors are not deallocated. Does this solve stuttering? If not, there may be other issues at play.

Indeed, good idea I will have a try

Looking at the AudioBufferSourceRenderer, only the buffer: Option field is heap allocated. To prevent deallocation of it on the render thread, we can use the pattern that @uklotzde introduced for deallocating control messages in a separate thread. In short, the control thread supplies an llq::Node in advance, which the renderer holds on to, but when it drops it uses the link to send the heap allocated object to the deallocation thread.

I could whip up a prototype if you are unsure what I mean

Hum right, I see what you mean. I will try to have a shot, that seems like a good exercice for me :)
Let's close that then


credslayer: remove failing tests, add missing runtime dependency

Motivation for this change
ZHF: https://github.com/NixOS/nixpkgs/issues/144627
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 144655 run on x86_64-linux 1

1 package built:

credslayer


Parsing microdata strips spaces

Given the following HTML:

Foo Bar

I would expect the library to extract a Product with the name of Foo Bar, but it extracts FooBar omitting the space.
Do you think this would be an easy fix?

@Vasanth-Indix @addnab Do you think the above is a valid expectation? Do you think you'd be able to address it or point me the right direction? Thanks!

Yes @hkdobrev. It's a valid expectation. We will look into it.


Exception thrown during report_package on v1.11.1

When running report_package on a project of mine I run into this error being thrown:
ERROR: TypeError: in typeassert, expected DataType, got Type{Tuple{Type{StaticArraysCore.SizedArray{Tuple{3}, T, 1, 1, Vector{T}}}, Tuple{Tuple{Any, Any, Any}}} where T}

I'm not sure how to dig deeper into this issue to produce a an minimal working example, I've enabled additional logging. If there is additionnal instrumentation or analysis that can be done I'm happy to dig a bit deeper into this issue.
Analysis
Running the same workload against two different julia versions shows this is a regression due to a julia version upgrade.

A private project, I can't share the code for it. It's also pretty large so it's not clear to me where things are going wrong. If you have some ideas on how I can narrow down the problem I'm happy to go and create a representative example.

It works, thanks for fixing this.


support for geyser

once geyser finishes its API for custom models you should support bedrock entities

Geyser seems to be compatible with custom models if they use armorstands assuming you manually provide the bedrock equivalent resource pack.
Thus this issue "should" be possible to achieve by making a bedrock resource pack generator as well. The tricky part, besides making said generator would be to move the resource pack into the geyser folder.
Personally, I'd prefer bedrock's native custom entity system since that would be more optimized for the bedrock client but it seems like geysermc does not support custom entities through resource & datapacks sadly. At least not yet.
Model engine 3 seems to be working with geyser with at least some models while Model engine 4 is completely incompatible due to it's use of display entities which bedrock does not have.

Geyser seems to be compatible with custom models if they use armorstands assuming you manually provide the bedrock equivalent resource pack.
Thus this issue "should" be possible to achieve by making a bedrock resource pack generator as well. The tricky part, besides making said generator would be to move the resource pack into the geyser folder.
Personally, I'd prefer bedrock's native custom entity system since that would be more optimized for the bedrock client but it seems like geysermc does not support custom entities through resource & datapacks sadly. At least not yet.
Model engine 3 seems to be working with geyser with at least some models while Model engine 4 is completely incompatible due to it's use of display entities which bedrock does not have.

Yes but if this api used geyser api it would probably save so much time then having to maunally implement it but the geyser native entitys will only progress if people commit and help


Reorganize registry deployment guide

Proposed changes
Rewrites and reorganization
Also add information about pushing non-distributable layers
into a private registry
Fixes #3433
Unreleased project version (optional)
Docker 17.06
PTAL @stevvooe @dmcgowan @thaJeztah @cpuguy83

PTAL @friism

Addressed feedback.

@stevvooe the first two bits of feedback you had were for unrelated changes that just indicated this patch needed a rebase. Maybe @londoncalling can make note of them to be changed elsewhere.

Addressed feedback and added a service example that uses secrets to store the TLS certificate material.

LGTM, merging into vnext-engine.


Remove the "Sign Up" button from navbar

This is a regression due to the last discourse update. We only need one button for log in and registration, not two.

I guess we need to modify this from our theme, right?

Yes, should be easy to do. We already had that, but I think the button class changed with the last update.
Code is here:
https://github.com/godotengine/discourse-theme/blob/main/common/common.scss#L203


incorrect overload resolution

I try to resolve all signatures of overloaded functions. It works well except case when one of overloaded signatures has no arguments.
P.S. I know about limitations of conditional types over functions with multiple call signatures.
But this hack widely used to resolve multiple call signatures
TypeScript Version:  3.4.5
Code
type Overloads<F> =
    F extends {
          (...args: infer A1): infer R1
          (...args: infer A2): infer R2;
      } ? {rule: 2, variants: [A1, R1] | [A2, R2]} :
    F extends {
          (...args: infer A1): infer R1;
      } ? {rule: 1, variants: [A1, R1]} :
    never;

declare const ok1: Overloads<(x: number) => boolean>;
// {rule: 1, variants: [[number], boolean]}

declare const ok2: Overloads<{(): 1; (x: number): 2}>;
// {rule: 2, variants: [[], 1] | [[number], 2]}

declare const wrong1: Overloads<() => boolean>;
// {rule: 2, variants: [[], boolean] | [unknown[], {}]}
// expected {rule: 1, variants: [[], boolean]}

declare const wrong2: Overloads<(...args: unknown[]) => boolean>;
// {rule: 2, variants: [unknown[], boolean] | [unknown[], {}]}
// expected {rule: 1, variants: [unknown[], boolean]}

I think this is a duplicate of https://github.com/Microsoft/TypeScript/issues/28867

@dragomirtitian Looks like duplicate. But wider: problem not only in no-arg function:
declare const wrong3: Overloads<(x: unknown) => boolean>;
// {rule: 2, variants: [[unknown], boolean] | [unknown[], {}]}
// expected {rule: 1, variants: [[unknown], boolean]}

declare const wrong4: Overloads<(x: any) => boolean>;
// {rule: 2, variants: [[any], boolean] | [unknown[], {}]}
// expected {rule: 1, variants: [[any], boolean]}

I've moved code to  #28867. Closing this as duplicate


What's the concept to distribute NPKs among a set of targets

I would like to know if there is a concept in place to cover the following user stories:

As developer I can upload packaged containers (npk) to a central place (registry) to share it with different runtime instances.
As operator I can define a set of npks and use this definition to update the local registry so that these containers can be started by the instance (e.g. at runtime, or ondemand)

To put it into different words:
With docker we have central registries where we can push and pull container images from. This is very helpful for sharing containers across teams, workflows, and of course target devices with their respective runtime instances and local registries.
Additionally to being able to have a central place where to store containers, it would also be interesting to know, how to describe and make sure that a set of containers will be started. I would call this container orchestration on the target runtime instance. It kind of like a deployment manifest file, that also could contain information about the required environment variables, that might very well be instance-specific, e.g. ports.
Further thoughts:

If we wanted to use a ci-workflow to create npks and then store the results, we could use an existing file-oriented registry, like Artifactory OSS.

A central repository implementation in not in the scope of the runtime. Furthermore keep in mind that containers are not that generic as for example Docker ones are. Every Northstar container is created and configured dedicated for a specific target -> a central repository doesn't make sense.
Updates are specific to the integration of the runtime and can be done by a target specific application.
Regarding orchestration: Northstar targets embedded use cases where a "scripted" orchestration is barely possible or a good choice at all. If you need something like that it's quite easy to realise with a couple of start commands with args and env.

Thanks for answering the question and sharing your thoughts.


Introduce hvloader package

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Introduce hvloader package, the open-source loader component required to load the dom0 image over mshv.
Change Log

Introduce hvloader package
Move acpica-tools and patchutils into SPECS (required to build the edk2 environment)

Does this affect the toolchain?

NO
Test Methodology

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=343232&view=results

I updated the PR to address comments, will see how the buddy build goes: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=354084&view=results

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=354680&view=results


Rework: Enum typename considering name in enumeratedValues

~Marked as draft: this pull request depends on https://github.com/rust-embedded/svd2rust/pull/626, and should be merged after that pull request is merged.~ Ready for review as #626 is merged.
This pull request is a rework to https://github.com/rust-embedded/svd2rust/pull/612; commits in this pull request only considers enumeration name other than writer structure and reader structure. Names to those two proxy structures are yet to be discussed.
svd2rust has a good mechanism to parse enumeratedValues into Rust enums. However, current code would pick name of the first field the enum resides, and use pub type to re-export it into other names.
SVD files provide name field in enumeratedValues, this commit make use of them to name the Rust enums it generated. After this commit, more information of SVD file is considered in output pac crate. Users may also discover a significant drop in amount of types, which will speed up rustdoc generation and crate compilation.
After this pull request is merged, we can discuss on how the read proxy and write proxy should be named as is mentioned in comment at #612.
Before this pull request:

After this pull request:

SVD file for this example:

        <register>
          <name>interrupt</name>
          <description>Interrupt enables, states and masks</description>
          <addressOffset>0x04</addressOffset>
          <resetValue>0x3f003f00</resetValue>
          <resetMask>0xffffffff</resetMask>
          <fields>
            <field>
              <name>fifo_error_enable</name>
              <description>Transmit or receive FIFO error interrupt enable</description>
              <lsb>29</lsb>
              <msb>29</msb>
              <writeConstraint>
                <useEnumeratedValues>true</useEnumeratedValues>
              </writeConstraint>
              <enumeratedValues>
                <name>InterruptEnable</name>
                <enumeratedValue>
                  <name>enable</name>
                  <description>Enable interrupt</description>
                  <value>1</value>
                </enumeratedValue>
                <enumeratedValue>
                  <name>disable</name>
                  <description>Disable interrupt</description>
                  <value>0</value>
                </enumeratedValue>
              </enumeratedValues>
            </field>
            <field>
              <name>arbitrate_lost_enable</name>
              <description>Arbitration lost interrupt enable</description>
              <lsb>28</lsb>
              <msb>28</msb>
              <writeConstraint>
                <useEnumeratedValues>true</useEnumeratedValues>
              </writeConstraint>
              <enumeratedValues derivedFrom="InterruptEnable" />
            </field>
<!-- ...... -->

r? @ryankurte
(rust-highfive has picked a reviewer for you, use r? to override)

rebase, please.
You can use last commit from https://github.com/rust-embedded/svd2rust/commits/enum-names2

@burrbull: thanks, I'll rebase to adapt main branch changes.

@burrbull: Hello! I have rebased to master and pushed my commit.

bors r+


Update to latest manifestival 0.6.0+

Fixes #184, fixes #189
It also adds support for generated names -- they will always be
created when the manifest is applied.

/lgtm


Kdrive sync webdav

**Hi, sync is ok on Kdrive cloud, thx for your job, i use a payed **

Thanks for your feedback!


Speed Property lost

The speed property of the location gets lost after the filter process.
Is it possible to add it?

Same issue here...

Also faced with this problem. Is it possible to fix this problem, please?

Same issue here...

anyone?


#6536 #10184 macOS Horizontal Scroll Issue

The method scrollHorizontally() is called on event SWT.MouseHorizontalWheel. In my testing, it seems like this event is being raised on macOS, but not Windows. The method overrides normal horizontal scrolling and forces either the leftmost or rightmost visible column to be fully visible - depending on whether the user is scrolling to the left or right. This causes scrolling to feel "jumpy". If the user has columnScrolling disabled (i.e. Smooth Scrolling enabled), this method probably shouldn't be called.
The value for HORZ_SCROLL_INCREMENT is only used if columnScrolling is disabled (Smooth Scrolling enabled) and only appears impacts the UI when responding to a horizontal mouse wheel or if the user clicks the arrows on the scrollbar. The reason for increasing this value is that previously horizontal scroll was moving a column at a time. With this change, the scroll wheel is moving pixel by pixel, and that made scrolling extremely slow. I tested a range of values on a 2019 MacBook Pro and a Microsoft Surface Laptop 4 (in order to test with a Windows Precision Touchpad enabled device), and a value of 12 gave the closest approximation to scrolling in applications like Excel. Behavior was unchanged when using a basic wired mouse.

Updated my original post after some additional testing. SWT.MouseHorizontalWheel does fire on Windows, my mistake.  Also noticed that the change to HORZ_SCROLL_INCREMENT also impacts if a user clicks the arrows on the scrollbar. Neither of these are problems, I just wanted to have my facts straight 😁.

@serge-rider I've been using it on my Mac for over a week and it resolves the issue for me. I'm curious what configuration you tested with where this didn't fix the issue? Would it be helpful if I uploaded a video?

Well, if it fixes the problem in your env then we'll give it another chance.
We'll try to test it on different machines.
Thanks.

@serge-rider Here's a short video that shows what I'm seeing. This is on my 2019 MacBook Pro. I also tested on a 2021 MacBook Pro. Both are running macOS 12.3.1.
Screen Recording on Vimeo


Unmet peer dependencies

When installing via yarn install I get the following warnings. These should be added to the package.json so that they are properly installed and I don't get console warnings when installing this package.
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>

I was use official Add-on kit package.json which has it this way. Will investigate. Thanks for reporting.

Fixed in 0.1.15

Great thank you @morewings!


Compute aggregated stats

For the current search, show aggregated stats at the top for all results below.
If no filter is applied, you see worldwide stats. If you filter only US, then you see US-wide stats.

We probably cannot add up the numbers for each country. Most likely there is overlap in the metrics. So we will need another data source.

I think that now the stats from the CSSE source can be safely aggregated.
There isn't a definitive answer on this in the data source repo, but if you troll through the issues, it seems to be the case. For example: https://github.com/CSSEGISandData/COVID-19/issues/590
I think aggregated stats would be a very useful feature. I can work on this unless you want to @briehl. If you work on it, then I may start working on a second dashboard that sources data from covidtracking.com

I'm going to go ahead and do this in the simplest way I can think of. In normalize-data.ts (I think I'm going to rename this to transform-data.ts, normalize is probably not the best term), I will add additional region entries for "Worldwide", as well as each top-level country that has provinces.

This is done. Was a pain with the source data :P

PR was #31 and some fixes in #32


SAMZA-1295: Add a script to merge pull requests in the hello-samza pr…

…oject

This is basically the same merge script as the main samza repo, but it points to the hello-samza project and has some safe guards for committing to the non-default (latest) branch.

@vjagadish1989
@navina


Postman Schema tests for response body properties that are arrays with min and max items dont match

Given the following property schema in a response body:
grantTypes:
          description: |
            List of grant types supported for this client.
            As per RFC6749, the clientCredentials grant can only be used by confidential clients.
            Clients SHOULD always only allow Authorization Code with PKCE.
          type: array
          minItems: 1
          maxItems: 2
          uniqueItems: true
          items:
            type: string
            enum:
              - authorizationCodePKCE
              - clientCredentials
            minLength: 0
            maxLength: 100
          example: [authorizationCodePKCE, clientCredentials]

This results in a postman schema test where the min and max items are 1
"grantTypes":{"description":"List of grant types supported for this client.\nAs per RFC6749, the clientCredentials grant can only be used by confidential clients.\nClients SHOULD always only allow Authorization Code with PKCE.\n","type":"array","minItems":1,"maxItems":1,"uniqueItems":true,"items":{"type":"string","enum":["authorizationCodePKCE","clientCredentials"],"minLength":0,"maxLength":100},"example":["authorizationCodePKCE","clientCredentials"]}

We also noted from trying random numbers setting min items to 0 and maxItems to 2
The generated postman schema check dropped maxItems all together.
Happy to provide a full OpenAPI example for testing on request
Thanks

hi @chyde80 & @savage-alex
We did some work in the last version on keeping the original JSON schema.
Is this issue with min & max items still happening?

hi @chyde80 & @savage-alex
More news, we were able to streamline the OpenAPI schema to JSON schema conversion further in #644
Solving this long lasting minItems, maxItems behaviour in the JSON schema validation.
Once the PR lands in the next version, let us know if it solved your problems.

@savage-alex @chyde80
Another update:
we just released Portman 1.30.1, which contains the enhancements for the OpenAPI to JSON schema conversion, including the fix of the reported unwanted minItems & maxItems modification.
Let us know, if the new version solved this issue.

Getting the team who reported this to pull this version down and try thanks !

sorry for the delay they are looking at it.

@savage-alex
Really, there's no need to apologize, I'm just following up to see if the issue is gone or if we need to continue to spend time on it.

thanks both! job well done


iCubGenova04: arms, legs and torso joint offsets recalibration

These changes update the joint offsets calibration:

left and right arms
Recalibrated left and right legs after r_hip_yaw encoder replacement
Recalibrated the torso after ironbot tests

These offsets have been used for at least a couple of months on the Green iCub(Genova04), and run on all the demos.

Ciao @S-Dafarra , this is just the last branch that was being used for some time, not yet integrated in devel. After this PR is merged, devel and devel_iCubGenova04 won't differ, except for minor changes in the Current Control config not impacting the behavior of the controllers.
Then, from the resulting devel commit, I can create a merge back to devel_iCubGenova04, such that this devel commit will be the new "base" for future merges from or to devel_iCubGenova04.
CC @traversaro

Just a side comment. Why some calibrationZero changed?

the robot torso has been removed and when it has been remounted it has been mounted 90 degree away from before unmounting.

If I recall well it was more or less the same case for the hip yaw.

Got it! Thanks!


Redundant condition

there are redundant ternary operators in NearbySelectionConfig

I'm not an OptaPlanner developer so will wait for Geoffrey de Smet.. but the data-type is an Integer and there's no check for it being null prior to the lines you're changing so why is the null check a redundant term?

https://github.com/snurkabill/optaplanner/blob/redundancy2/optaplanner-core/src/main/java/org/optaplanner/core/config/heuristic/selector/common/nearby/NearbySelectionConfig.java#L197
on this line, we determine if linear distribution is enabled. and then we have condition:
} else if (linearDistributionEnabled) {
int sizeMaximum = linearDistributionSizeMaximum == null ? Integer.MAX_VALUE : linearDistributionSizeMaximum;
where "linearDistributionSizeMaximum == null" is redundant, because the information is already stored in linearDistributionEnabled boolean value.

Agreed. Good found (there might be more valid ones in findbugs or IntelliJ inspection, keep any you think are valid coming).
But I need to add support for nearbySelectionType (which I will do now on master so we're not stuck in a limbo state), which will make that null check required. Once that's pushed, I 'll reference that commit here and I 'll close this PR without merging.

Fixed alternatively, by adding NearbySelectionDistributionType:
https://github.com/droolsjbpm/optaplanner/commit/cb288f817f048f90ad8037f3d93503873605c758


Question: How to predict on unknown query

Sorry for raising such question as I'm new to MatchZoo... I'm trying to use DRMM for a QA ranking problem and followed the preprocessing and training steps to train a model.
Wondering how can I do prediction for unknown data?
According to my understanding, in the preprocessing steps, all queries, docs and words are converted by ID. So if I'm given a new unseen query not in the training corpus, what ID can I assign to the query and what ID can I assign to the unseen words (if any) in the query?
I find the examples you provide generate train, val and test dataset at the same time for a given corpus beforehand, do I miss any example that illustrates the usage in the inference case I described above?
Really appreciate your help and thanks in advance!

checkout #236 .

@bwanglzu Thanks for your help. I have figured out how to do prediction :). I close this issue.

@hkvision Could you help me,I have the same question,give one query,and return top k documents,Could you show your code for me,Thanks very much


Incompatible with Crashlytics SDK on Android

After installation on Android, I am unable to build. I get a com.android.dex.DexException because both io.branch.external.answersshim (which is a dependency of the Android SDK) and com.crashlytics.sdk.android import the same classes (specifically in the error com/crashlytics/android/answers/shim/KitEventLogger). The Android SDK docs specify how to exclude that module, but I cannot determine how to accomplish this for the Cordova plugin. I know you guys are now affiliated with Fabric, so I'm hoping this is a quick fix and perhaps I've just missed something in the Cordova docs.
Any assistance would be appreciated!

I was indeed able to get the project to build by doing the following:

Commenting out the following line in the plugin.xml

<!--<framework src="io.branch.sdk.android:library:1.+" />-->

Adding the following to platforms/android/build.gradle:

compile ('io.branch.sdk.android:library:1.+') {
    exclude module: 'answers-shim'
}

The question now is...is this possible to do without editing the plugin? Unfortunately, I can't find any way to do it in the Cordova docs.

Ugh that's annoying. Let me look at how we could do an exclude from the plugin.xml or from within the project config. I'm sure Cordova supports it.
Thanks for the heads up! Glad you found the temporary work around.

Hey man - What do you think about this? https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking/pull/209 Then, when you encounter the conflict, you could just add
    exclude module: 'answers-shim'
}``` to the gradle extension file.

@aaustin I think it looks good. I did the same thing myself after I posted the comment above.
Honestly, it always feels off to me to have to do these kind of things, because I'd really love to leave the plugins directory out of my repos in the same way that I leave out node_modules. At the same time, I can't find a better way to do it short of adding it as part of the plugin install process. Basically, using a variable during the install command to do that configuration automatically. Of course, the drawback then is that if you want to change it, you have to remove and then add the plugin back which has it's own set of issues. It also seems like a lot of work for what is likely an edge case.
All of that nonsense to say... 👍 !

Travis - thanks for the feedback. The functionality was merged in, and I'll get the docs merged in next update.


[CALCITE-607] Change visibility of constructor in metadata providers for...

... size, memory, parallelism

Note that you can chain metadata providers. So, you could create say HiveRelMdMemory, a metadata provider that implements methods for Hive relational expressions, and chain it with RelMdMemory, the metadata provider that implements the core relational expressions.
However, sub-classing is also reasonable, so I'll accept this pull request.


Linking doesn't work correctly when keyboard used to select text in firefox

If you select text using the keyboard in firefox (e.g. shift+left) then add a link (either with the button or cmd+k) then the link itself gets pasted in instead of the text being linked. This doesn't happen in chrome.

https://github.com/guardian/scribe-plugin-link-prompt-command/issues/12

From Lydia: It's not the most urgent issue, but it's consistently affecting one of our users, so it would be good to get it fixed."

I've had a look into this and the occurrences are pretty inconsistent. This doesn't happen at all if you have more than one line of text. Also, if this has already happened once, it doesn't happen again. Those aside, it doesn't happen every time either. Perplexing!


Cannot export package in local repo

Xmake Version
2.7.6
Operating System Version and Architecture
Windows 11
Describe Bug
-- create pakage(base on target) to local repo(`add_repositories "local-repo build"` in my xmake.lua)
xmake package

-- query package is ok
xmake require --info ccexample

-- but export failed
xmake require --export ccexample

Expected Behavior
local package export successfully
Project Configuration
add_repositories "local-repo build"
target "ccexample"
add_files "src/*.cpp"

Additional Information and Error Logs
D:\shaq\Projects\ccexample>xmake require --export ccexample
packages(ccexample) not found, maybe they don’t exactly match the configuration
please attempt to export them with `--extra=` option, e.g.
    - xmake require --export --extra="{configs={...}}" package
    - xmake require --export --extra="{debug=true,configs={shared=true}}" package

try it again. xmake update -s dev

xrepo export ccexample not work.
xmake require --export ccexmple can find ccexample now, but dependencies not exported.

but dependencies not exported.

it works for me.
package("tbox")
    add_deps("tbox2")

ruki-2:testbox ruki$ xmake require --export tbox
exporting tbox2-latest [from:tbox]
  -> packages/t/tbox2
exporting tbox-latest
  -> packages/t/tbox

you can debug https://github.com/xmake-io/xmake/blob/43908d7c753b6502cf1f2f589cbc52b508738347/xmake/actions/package/local/main.lua#L34

It would be much easier, if debugger can be used to debug lua code. :)

you can add print log to debug it. it's easy too.

I know print can help debug on simple issues.
I care about dedicated debugger, as I can walk through the whole exection step by step.
It could be a greate help on understand the code structure & highlevel function flow.
Please kindly reconside on it. :)


Are batch sizes fixed in ngraph?

Hello, is there a way to use ngraph/plaidml with dynamic batch size?
If I torch.onnx.export with a dynamic batch size:
torch.onnx.export(..., {"input": {0: "batch_size"}, "output": {0: "batch_size"}})

When I try to load it to ngraph , I get a RuntimeError:
import onnx
from ngraph_onnx.onnx_importer.importer import import_onnx_model
onnx_protobuf = onnx.load("model.onnx")
ng_function = import_onnx_model(onnx_protobuf)
Traceback (most recent call last):
  File "foo.py", line 9, in <module>
    ng_function = import_onnx_model(onnx_protobuf)
  File "/Users/grapaport/ng/lib/python3.6/site-packages/ngraph_onnx/onnx_importer/importer.py", line 36, in import_onnx_model
    return onnx_import.import_onnx_model(onnx_protobuf.SerializeToString())
RuntimeError: Check '(batch_size.is_dynamic() || static_cast<size_t>(batch_size) > 0)' failed at /jenkins/workspace/ngraph-core-mac-wheel-generation/ng/src/ngraph/validation_util.cpp:316:
While validating node 'Convolution[Convolution_523](Parameter_522: float{0,3,224,224}, Constant_4: float{64,3,7,7}) -> (??)':
Batch size is zero.

Is this not supported by ngraph? If not - is there a way to bypass this?
EDIT: original PyTorch model can accept any batch size that does not OOM the machine.

ngraph has been moved into OpenVINO: https://github.com/openvinotoolkit/openvino

How come the main page of ngraph doesn't mention this?
https://github.com/NervanaSystems/ngraph
I got ngraph working in 10 minutes - OpenVINO's Python API isn't nearly as clear :(
Good luck in your new home!

#4952

🙇
Thanks!!!

🙇 Thanks!!!

hello ， would you like to share the ngroph-core wheel ？i try to install like pip or cmake the wheel ， but failed 。  think you！ plz

Everything migrated into OpenVINO with a somewhat different organization.


[Design] Added PushWriter and PopWriter to RazorPageBase

Removed WriteXYZTo methods

All functional tests pass.
Issue - https://github.com/aspnet/Razor/issues/1184
Related Razor PR - https://github.com/aspnet/Razor/pull/1291
@rynowak @NTaylorMullen

Ignore my add unit tests bit. Design looks good (didn't see the Design label)

🆙 📅 No longer in design

🆙 📅


More descriptive names for Crail parameters

The default config that is generated in assembly/target contains crail-site.conf, which has a list of parameters:
crail.blocksize				1048576
crail.buffersize			1048576
crail.regionsize<PHONE_NUMBER>
crail.cachelimit<PHONE_NUMBER>
crail.cachepath				<path, should be huge page mountpoint>
crail.singleton				true
crail.statistics 			true
crail.namenode.address			crail://<hostname>:9060
..

It would be nice if we can qualify all parameters that are listed there ... for example
crail.blocksize can be crail.fs.blocksize, and crail.buffersize can be crail.client.buffersize. This modification explicitly shows which parameter is relevant for which Crail component.

Agree. New code has params structured in client, storage and namenode.


Rendering issue on SDK homepage

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/149
Originally created by @aaronsteers on 2021-06-23 18:12:54

After merging !102, formatting is corrupted for links. Will need to correct.
https://sdk.meltano.com/en/latest/

View 2 previous comments from the original issue on GitLab


Move sqlalchemy and alembic to -server only

From Slack:

sqlalchemy and alembic are dependencies on the keylime package. This was because this package also provided registrar and verifier, whereas in the *** cloud they never do; they only install keylime as a package into the places where it runs as an agent.

And now that've we done the splitting for agent and the server, it's easy for us to move this in the right place. We should do this prior to the 6.2.0 release.

IIUC the target of this PR, If we don't care about what agents needs, because it needs the -lib package:
root@bionic-keylime:~/Code/Builds# apt depends python3-keylime-agent
python3-keylime-agent
  Depends: adduser
  Depends: python3
  Depends: python3-keylime-lib
  Depends: <python3:any>
    python3
  Recommends: python3-keylime-doc
  Suggests: python3-keylime-defaultconf

I suppose we can remove them from there. If not, the lib package would need to be split (one for server, one for agent, I guess...).


Fix Zygote accum for CA in GPU broadcasting

Continuing my saga of upstreaming pirated patches from Lux. Zygote + AMDGPU + CAs.jl don't play too well. See https://buildkite.com/julialang/lux-dot-jl/builds/634#018ab9ff-bbcb-47bf-aa05-079929053253/6-4602.
This is mostly a hacky patch, but it fixes it.

Codecov Report

Merging #221 (b269890) into main (140b899) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #221      +/-   ##
==========================================
- Coverage   73.16%   72.96%   -0.21%
==========================================
  Files          22       23       +1
  Lines         723      725       +2
==========================================
  Hits          529      529
- Misses        194      196       +2

Files Changed
Coverage Δ

ext/ComponentArraysZygoteExt.jl
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Related to #186, I think

Related to https://github.com/jonniedie/ComponentArrays.jl/issues/186, I think

Not really. I think it is mostly on AMDGPU side not being very robust to broadcasting. I have faced it in places completely detached from Zygote and CAs


Add legend after data has loaded, and only include dots that were used

Per https://github.com/LukePrior/nbn-upgrade-map/issues/353

eg Banks, ACT

Looks great, very nice qol feature


Trainer.load can use GPU's other than cuda:0

Hi! When I tried to load a checkpoint while using GPU 1 (because I'm sharing a computer with my other labmates), it decided to take up VRAM on both GPU 0 and GPU 1. This is because by default torch.load uses VRAM on cuda:0. I've added a fix; now this code will only use VRAM on the device set by the user.
#Code from https://github.com/lucidrains/denoising-diffusion-pytorch

import torch
from denoising_diffusion_pytorch import Unet, GaussianDiffusion, Trainer

dataset_path='/my/image/folder'

device=torch.device('cuda:1')
torch.cuda.set_device(device)

model = Unet(
    dim = 64,
    dim_mults = (1, 2, 4, 8)
).to(device)

diffusion = GaussianDiffusion(
    model,
    image_size = 128,
    timesteps = 1000,           # number of steps
    sampling_timesteps = 250,   # number of sampling timesteps (using ddim for faster inference [see citation for ddim paper])
    loss_type = 'l1'            # L1 or L2
).to(device)

trainer = Trainer(
    diffusion,
    dataset_path,
    train_batch_size = 32
    train_lr = 8e-5,
    train_num_steps = 700000,         # total training steps
    gradient_accumulate_every = 2,    # gradient accumulation steps
    ema_decay = 0.995,                # exponential moving average decay
    amp = False,                        # turn on mixed precision
    results_folder = './results',
)

trainer.load(11)

trainer.train()

oh, thanks for this!


Loosen checks in Dart2JS in a Web Worker

This is forked from an internal thread with @sigmundch and others.

As part of removing Isolates on web, I wanted to create a (very) simple example of using web workers with JS interop, enough to help inspire (some) confidence so we don't need to drop everything and add it to dart:html. Here's my WIP:
https://github.com/matanlurey/web_worker_channel
It almost works... except that any Dart2JS code that detects it is running in a web worker listens to incoming messages as part of isolate_helper.dart (patch), and then crashes because the incoming messages don't fit the old isolate messaging scheme.
To reproduce, git clone, make sure you on the latest Dart 2.0-dev, then:

pub upgrade
pub run build_runner serve
Navigate to localhost:8080

This loads a DDC application (main.dart), that tries to spawn a Dart2JS* web worker (worker.dart). It loads, and then the worker crashes as soon as main.dart sends a message:
Uncaught Error: Invalid argument (index): "command"
    at Object.wrapException (js_helper.dart:1773)
    at Interceptor.$index (js_string.dart:472)
    at IsolateNatives__processWorkerMessage (isolate_helper.dart:844)
    at isolate_helper.dart:257

... because I'm sending a String, and isolate_helper assumes all messages are a JSArray with a specific encoding. Since we are removing isolates anyway, is it possible to just no-op/remove this code, or perhaps just add a 1-liner of:
if (is_not_array) {
  return;
}

... to get around this until we have time for a proper cleanup.
I can file an external bug, I just wanted to give a chance for discussion first.
*This is because of https://github.com/dart-lang/build/issues/1059 - but we need this to work eventually anyway, hence while I'm following up here.

Here is the offending code:
https://github.com/dart-lang/sdk/blob/a1cacec5fc3482366b15fb39ef38a62ce7bd30fc/sdk/lib/_internal/js_runtime/lib/isolate_helper.dart#L843-L844
I'm going to add a tiny "if-this-is-not-a-map-or-jsobject-return":
  if (msg is! JSObject && msg is! Map) {
    // See github.com/dart-lang/issues/<this issue> for details.
    return;
  }

... which should unblock us until isolates are totally removed.

I think this will ALL go away when dart:isolate is removed from DDC/dart2js
DDC PL: https://dart-review.googlesource.com/c/sdk/+/45754/3/pkg/dev_compiler/tool/input_sdk/private/isolate_helper.dart#b1134
Related issue: https://github.com/dart-lang/sdk/issues/32684

I think we need to fix this issue first. We need to provide users a way to start using web workers and to start migrating before we delete the dart:isolate support.

@kevmoo thank you for your answer yesterday [1], I can not reply there because it got locked.
I love the fact that obsolete and unwanted stuff got completely removed, but my point is to have a documented supported alternative for the web and if there are none then to document that.
So, is [2] the official "supported" alternative for isolates on the web?
1: https://github.com/dart-lang/sdk/issues/32684#issuecomment-726385823
2: https://github.com/matanlurey/web_worker_channel


Get-AzRoleAssignment : The Principal ID<EMAIL_ADDRESS>is not valid. Principal ID must be a GUID

Hmm, getting a different error though:
Get-AzRoleAssignment : The Principal ID<EMAIL_ADDRESS>is not valid. Principal ID must be a GUID
Originally posted by @halfluke in https://github.com/hausec/PowerZure/issues/6#issuecomment-696450095

I have the same issue right now.. Any updates on this?

The above problem can be solved when changing $userid=$user.userprincipalname on line 157 to $userid=$user.id.
However, when working with guest users i stumbled over another problem when running Get-AzureTargets:
Get-AzADUser : Cannot validate argument on parameter 'UserPrincipalName'. The argument is null or empty.
I think the problem when working with guest users is that the user Id from Get-AzContext (e.g<EMAIL_ADDRESS>does not correspond with the actual userPrincipalName in the AzureAD (e.g. cyberfreaq_auditor.onmicrosoft.com#EXT@customer.onmicrosoft.com).
Therefore when the following code is running (line 1512 onwards), $user is empty and therefore $userid is also empty:
$Context = Get-AzContext
$user = Get-AzADUser -UserPrincipalName $Context.Account
$userid=$user.id

Empty output:
PS C:\> $user
PS C:\> $userid
PS C:\> $Context

Name                                     Account                   SubscriptionName          Environment              TenantId
----                                     -------                   ----------------          -----------              --------
XXXXXXX                                  XXXXXX                    XXXXXXXXXX                XXXXXXXXXX               XXXXXXXXX

A workaroung could be to use the same method as used in line 138 onwards, but I didn't have time yet to test this:
$APSUser = Get-AzContext
$Headers = Get-AzureGraphToken
$username = $APSUser.Account
$user = Invoke-RestMethod -Headers $Headers -Uri 'https://graph.microsoft.com/beta/me'
$userid=$user.id # The one I changed from userprincipalname to id

Yeah, it looks like we are swinging between https://github.com/hausec/PowerZure/issues/6 and this https://github.com/hausec/PowerZure/pull/9

I've changed the fetching of $user and $userid the Invoke-RestMethod approach in the following functions:
function Show-AzureCurrentUser
function Get-AzureTargets

Have not had any errors so far.

I've changed the assignment of $user and $userid to the Invoke-RestMethod approach in the following functions:
function Show-AzureCurrentUser
function Get-AzureTargets

Have not had any errors so far.

Any chance you can open a Pull request if your fix works?

I've actually tried to do this directly via the browser but Github tells me "We’ve detected the file has mixed line endings. When you commit changes we will normalize them to Windows-style (CRLF)." So it seems like I made hundreds of changes although I just changed 3 lines - could imagine that this is would be more a burden for the maintainer rather than helpful.
But you asked for it so here we go :)

Pull request: https://github.com/hausec/PowerZure/pull/10
https://github.com/cyberfreaq/PowerZure/tree/patch-1

Thanks a lot.

Fixed in #10


Lime needs a mailing list

Many of the issues are currently questions

I've thought about this, but questions as issues seems better than asking people to subscribe to a list at the moment. Our volume is low enough that it doesn't matter.
Thanks for the suggestion though.

The prblm is anyone that dont know this will only see an small project with
A lot of issues
On mié., 13 de sept. de 2017 19:29 Marco Tulio Correia Ribeiro <
<EMAIL_ADDRESS>wrote:

Closed #104 https://github.com/marcotcr/lime/issues/104.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marcotcr/lime/issues/104#event-1248193112, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASQAnQpFV_7XnnQ0n0XZ-sae1QDBoeoks5siFdGgaJpZM4PTtZB
.

--
Juan BC (from phone)

And also you made a community around the answers.
On mié., 13 de sept. de 2017 19:58 Juan BC<EMAIL_ADDRESS>wrote:

The prblm is anyone that dont know this will only see an small project
with A lot of issues
On mié., 13 de sept. de 2017 19:29 Marco Tulio Correia Ribeiro <
<EMAIL_ADDRESS>wrote:

Closed #104 https://github.com/marcotcr/lime/issues/104.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marcotcr/lime/issues/104#event-1248193112, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASQAnQpFV_7XnnQ0n0XZ-sae1QDBoeoks5siFdGgaJpZM4PTtZB
.

--
Juan BC (from phone)

--
Juan BC (from phone)


Data Augmentation

Hi,
It doesn't appear to be mentioned in your paper. Do you use any data augmentation during training? And if so, what is your procedure?

Hi, thanks for your interest in our work. We do not use any data augmentation during training, and all the details are provided in the code. Just tell me if you have any other questions.

Thanks @jiaxi-jiang that's great. Very impressed with your project.


add zalando tokens as dependency

Either mark it as prequisite or add it as a dependency please :)

@Eloberus
what do mean exactly? It's a dependency in the 'sense of maven'. Do you mean something different?

hmmm ok then i might just be wrong. I just took a look into this pom file and did not find the maven dependency but that it's just my fault. Sorry for opening an issue :)


Feature request:Model download/load refactor.

I save my version of colab notebook to my drive to save defaults.
So I propose to divide it into three different parts.
1)Download default models.1.5,<IP_ADDRESS>, inpainting.
2)Load models from certain folder.When you already have downloaded models and don't want to download anything.Just want to run.
3)Download model.Also this part can be enhanced by an ability to select into which folder to download the file.Lora,hypernetwork,model,e.t.c.Default ones.And maybe select file name.Civitai often gives Lora without extension.In my experience.
Current behavior doesn't allow to download model when path to model is defined.

if you specify a local path, the cell will use that option as a priority one, so you have to keep unneeded input boxes empty.
As for the Loras, I'll see what I can do.


Fix issues to allow compatibility with 0.6.0 and add an ignore parsing error option

Thanks for this!


06-pytorch-transfer-learning: got some error in [3.2 Setting up a pretrained model] (torch:2.1.0+cu121, torchvision:0.16.0+cu121)

I followed the video("Setting Up a Pretrained Model with Torchvision") and tried to run the code :
model = torchvision.models.efficientnet_b0(weights=weights)
model

then got the error message:
invalid hash value (expected "3dd342df", got "7f5810bc96def8f7552d5b7e68d53c4786f81167d28291b21c0d90e1fca14934")
It seems like a new bug, and it could be fixed temporarily by overriding  torchvision.models._api.WeightsEnum.get_state_dict()
source:https://github.com/pytorch/vision/issues/7744#issuecomment-1757321451
I hope this could be helpful to those who got the same error.

thanks

Thank you for the heads up!
A few other people are getting that too.
See the following issue/comment for further code/solution: https://github.com/mrdbourke/pytorch-deep-learning/issues/696#issuecomment-1776124098

If you still face the same issue after running the first code cell to install the packages, try to Restart Session on Google Colab. Then, run the imports again.
The issue seems to be with torchvision v0.16.0 specifically. Other higher versions work seamlessly. https://github.com/pytorch/vision/issues/7744#issuecomment-1925556903


Css and related to it selectors fails

I'm update master branch of capybara and run my requests specs. And fails on everyone, where used has_selector? and related find_link, find, all and etc. The error was:
can't convert Array into String
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css/tokenizer.rb:19:in `initialize'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css/tokenizer.rb:19:in `new'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css/tokenizer.rb:19:in `scan_setup'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css/parser_extras.rb:61:in `parse'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css/parser_extras.rb:79:in `xpath_for'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/nokogiri-1.5.0/lib/nokogiri/css.rb:23:in `xpath_for'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/gems/xpath-0.1.4/lib/xpath.rb:59:in `css'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/selector.rb:34:in `css'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/selector.rb:51:in `call'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/selector.rb:51:in `call'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/query.rb:23:in `initialize'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/node/finders.rb:112:in `new'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/node/finders.rb:112:in `all'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/node/matchers.rb:38:in `has_selector?'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/node/base.rb:46:in `wait_until'
/Users/sergeypchelincev/.rvm/gems/ree-1.8.7-2011.12@fotoshkola/bundler/gems/capybara-363b43acb738/lib/capybara/node/matchers.rb:37:in `has_selector?'

After spending some time with source code I find a problem. It was in lib/capybara/selector.rb 34 line
31   def css(&block)
33      if block
34        @xpath = xpath { |*args| XPath.css(block.call(*args)) }
35      end
36      @xpath
37    end

Instead of putting all params, I'm passed only first and error gone.
Or maybe better remove xpath_options below in method "call"
def call(locator, xpath_options={})
    @xpath.call(locator, xpath_options)
end

I didn't find where them applied, but them also create warning

multiple values for a block parameter (2 for 1)

Thanks for the great report! A fix is in the queue -- I'll push it within the next hour.

Okay, pushed the fix. Thanks again!


mongoDB url with authentication information is not working

I'm a newbie to transporter, following all the possible issues and solutions, I finally being able to save local mongo db data to elastic search with config.yaml look like this: ( notice that I use mongodb for type because I'm using master branch)
nodes:
  localmongo:
    type: mongodb
    uri: mongodb://localhost:27017/foo
    namespace: foo.bar
    tail: false

but when I want to add authentication info in order to access my real remote mongo db, it just kept throw the familiar adapter error:
adaptor 'mongodb' not found in registry
and the modified config.yaml now looks like this:
nodes:
  localmongo:
    type: mongodb
    uri: mongodb://username:password@host:port/dbname
    namespace: dbname.collection
    tail: false

At first I though it's because my authentication info was wrong, but even if I add authentication info with localhost (which don't actually have authentication, but it should work, right?), I became unable to access the local db's data.
ubuntu@ip-10-0-7-39:~/go/src/github.com/compose/transporter$ transporter run --config ./test/config.yaml ./test/application.js
Mongo Config {URI:mongodb://username:password@localhost:27017/foo Namespace:foo.bar Ssl:<nil> Timeout: Debug:true Tail:false Wc:0 FSync:false Bulk:false}
adaptor 'mongodb' not found in registry

and the authentication-version of config.yaml for localhost look like this:
nodes:
  localmongo:
    type: mongodb
    uri: mongodb://username:password@localhost:27017/foo
    namespace: foo.bar
    tail: false

So I'm wondering  that the url with authentication is not working, please kindly tell me where did I do wrong and how to fix it, any advice is much appreciated.

hi @abc123634, please change the type attribute from mongodb to mongo and see if that helps.

@jipperinbham much thanks for the advice, but it didn't work either even if I changed to mongo like this:
nodes:
  localmongo:
    type: mongo
    uri: mongodb://localhost:27017/foo
    namespace: foo.bar
    tail: false

The error still exist
adaptor 'mongo' not found in registry
and you can notice that it didn't work even if without authentication from above config info.
I think my version of transporter is fine with mongodb and local mongodb is working as expected like below.
MongoDB shell version: 2.6.12
connecting to: test
> use foo
switched to db foo
> db.bar.find().pretty()
{
	"_id" : ObjectId("585a19ff3052ff841716102c"),
	"firstName" : "Robert",
	"lastName" : "Baratheon"
}
{
	"_id" : ObjectId("585a1a053052ff841716102d"),
	"fi
```rstName" : "John",
	"lastName" : "Snow"
}

Any further suggestion is appreciated.

@abc123634 can you try moving the namespace to your application.js file so it'd look as follows:
Source({name:"localmongo", debug: true, namespace: "foo.bar"})
.save({name:"foofile", namespace: "foo.bar"})

and the YAML would become
nodes:
  localmongo:
    type: mongo
    uri: mongodb://localhost:27017/foo
    namespace: foo.bar
    tail: false

Also you may have just excluded it from what you pasted previously, I do not see any reference to a foofile node defined in the YAML.

@abc123634 the latest release v0.1.2 covers all possible connection scenarios for mongodb so you shouldn't be having this issue. I'm going to close it out for now but please feel free to re-open with any additional info.


issue with Range when using f7 with react, router and AppBar

Framework7 version: 4.4.0
React.js version: 16.7.0
Platform and Target: Chrome
Live Link or JSFiddle: https://lucid-cori-c77ce7.netlify.com/

Describe the bug
Range is broken. Values cumming from range are NaN, and is showing wrong values
To Reproduce
Steps to reproduce the behavior:

Go to https://lucid-cori-c77ce7.netlify.com/
Click on AppBar lightning icon
You can see broken range both for the dual range and for the normal range

Expected behavior
Range should show values 30 to 60 but instead everything is at 0
Source code:
https://github.com/orhan-swe/f7_router_react_appbar_range_issue/blob/master/src/scenes/SearchPage.js
and the whole sample app source:
https://github.com/orhan-swe/f7_router_react_appbar_range_issue
Actual Behavior
Range should show between 30 and 60
Screenshots
See source code and https://lucid-cori-c77ce7.netlify.com/
Additional context
If the SearchPage is main page that is shown first the error does not happen.

Views are created in:
https://github.com/orhan-swe/f7_router_react_appbar_range_issue/blob/master/src/root/AppViewLoader.js

Issue is because you just setting tab-active class on view. It is not enough to switch the tab, it also need to trigger events that can tell to Range Slider that it became visible and needs to recalc its styles. Switch tabs with built-in logic using tab-link class on tab buttons, like in examples here https://framework7.io/react/tabs.html

@nolimits4web thanks for the info, I switched to routed tabs instead of views for this and the issue can not be seen anymore


i18next backend plugin loading resources after init

Hi guy, I'm using i18next backend plugin in a react app. The issue I'm having is the backend plugin always fetches aall resources on init. I can use the filter tags to filter the ones required on all pages.
const txBackend = new TransifexI18next({
  token: import.meta.env.TRANSIFEX_TOKEN,
  filterTags: "layout",
});

However, it seems like there's no easy way to load other tags after init. Something like loadTags(["home"])?

Maybe you can try using Lazy loading like this:
const txBackend = new TransifexI18next({
  token: import.meta.env.TRANSIFEX_TOKEN,
  filterTags: "layout",
});

...

txBackend.tx.fetchTranslations(txBackend.tx.getCurrentLocale(), {
  filterTags: "home",
});

Closing this, feel free to re-open it if needed.


Upgrading packages in base_install chef receipe

Upgrading packages during base_install receipe
execution. As we add upgrade here, we can now
remove packer command on upgrade
Signed-off-by: Mohan Gandhi<EMAIL_ADDRESS>Confirmed platform_family is correct for amazon linux too
Ref: https://github.com/awslabs/cfncluster-cookbook/blob/cf6c7c287b0effcd4d76f8a884d334086cb4bf89/attributes/default.rb#L59
And tested
kitchen test centos7-minimal - working
kitchen test amazon-linux is running successful ( need couple of minutes to confirm)

Ya receipe yum should update as per the link you referred should work but when we were testing centos ami build it was not working. we should definitely dive in to see what's happening, I am going to create an issue.
Pushing this change to move a step forward with kitchen test


Modulus operator does not create uniformly distributed random numbers

https://github.com/fetlife/rollout/blob/master/lib/rollout.rb#L91
https://stackoverflow.com/questions/10984974/why-do-people-say-there-is-modulo-bias-when-using-a-random-number-generator#10984975

There's already similar issue https://github.com/fetlife/rollout/issues/58
We know it's not perfect, do you have suggestion how to address this? :)


add account and sso to not_actions in DenyRegions and RestrictToSpecifiedRegions

what

Add account:* and sso:* as not_actions in the DenyRegions and RestrictToSpecifiedRegions SCPs.

why

Like the other not_actions resources in the SCPs, these resources are provisioned in a specific AWS region that may not necessarily be the one explicitly allow/deny listed.
sso:* being denied results in the entire AWS Identity Centre console being unusable.
account:* being denied results in, for example, GetAlternateContact requests failing.

@Nuru @aknysh  could you review this PR pls

@BFarand Thank you for this contribution.
I am going to close this in favor of PR #50, which incorporates some, but not all, of your changes.
SCPs are supposed to be restrictive, so I want to error on the side of them being overly restrictive. People are free to copy and modify our SCPs if they want to relax some restrictions; I am uncomfortable doing that for them. I did, however, add account and artifact as service exceptions for region restrictions.

sso can be provisioned in a region, so I did not add that.
Lambda region restrictions are tricky and I do not want to spring that on someone by changing the existing policies.
Many S3 actions are region-specific, including creating a bucket, so I did not relax those restrictions.
Access Analyzer and SSM are also region-specific and so I did not add those services to the list of exceptions.

I think, in general, it may be asking too much to try to prevent people from provisioning resources in us-east-1 since there are so many special cases of resources that must be provisioned there (e.g. ACM certificates, Lambda@Edge). So if the region restriction policy is too restrictive for those special cases, just add us-east-1 to the allow list. Otherwise we may need a whole new set of policies to deal with all the special cases.
After PR #50 merges, if you still want changes, I suggest you create a separate set of new policies where the global restrictions are relaxed and then individually tightened up, as you did here with Lambdas. Maybe just create a separate statement for the services that need us-east-1 that ensures they are enabled in that region, and then people can choose.


bug with display on custom saving example / documentation build never fails

The rendered documentation: https://deltarcm.org/pyDeltaRCM/examples/custom_saving.html
does not include the example source code. I think the directives under doctest are wrong (https://github.com/DeltaRCM/pyDeltaRCM/runs/2237426803?check_suite_focus=true#step:6:88).
Our documentation build is not correctly failing anymore, which is why this was not caught...

Good catch regarding that rendered example.
From what I can tell, in that PR the directive :context: given to :doctest:: is just invalid, and for whatever reason the doctest extension throws a warning in that case instead of an error. When I've altered and broken the contents of the doctest blocks locally I am getting a proper error and the docs do not finish building during make docs.
This seems to have happened because I was mixing the doctest and plot directives. Moving forward we should probably just stick to one for a given documentation file and that should help minimize these type of bugs.

Yeah, the :context: is a pyplot directive thing. doctest has context by default. It is messy using both, I agree, we should try to not mix them within a file.
To be clear though, your pr #182 does not fix the failing-ness of the docs build on CI, right? I can open a new issue with just that bug, since they are actually separate issues.

To be clear though, your pr #182 does not fix the failing-ness of the docs build on CI, right?

It does not.
But I think the build didn't fail because a warning as opposed to an error was raised by doctest. When an actual error is encountered the doctest call in the Makefile will break the build.


UI issue in display screen of each item under items module

Tax category is not displayed.Just : is given

issue fixed !


Mac: Enum property value text should not be editable

Proppy update has been merged.  Closing issue.


Metrics

https://grafana.com/blog/2018/08/02/the-red-method-how-to-instrument-your-services/ USE and RED

We have some initial tokio task metrics

Need to do:

RTF, processing time and media time. Do we want to abstract this in some way since all 3 are likely recorded together (some form of RAII guard?)


Fix allowUnmocked when last interceptor was removed.

Fixes #491.

Coverage increased (+0.01%) to 96.547% when pulling b9eaa75844a5a8158a87e21d724d839b32279c1d on djanowski:fix-allow-unmocked-when-interceptors-removed into 65373a96883a7a547100a3394e5aeed4ca70bfd5 on node-nock:master.

Coverage increased (+0.01%) to 96.547% when pulling b9eaa75844a5a8158a87e21d724d839b32279c1d on djanowski:fix-allow-unmocked-when-interceptors-removed into 65373a96883a7a547100a3394e5aeed4ca70bfd5 on node-nock:master.

Would be awesome if this could be merged. 👍

Hello, anything blocking this?

@pgte Can we get this one merged?

thank you all 💐


Project 1 feedback (Team Garuda)

Hi Team scapsulators
I would like to give my review of your project.
I found the documentation easy to read and setting up project was very simple.
I must say that that UI is very user friendly and cool.
I was tried out same query on quest generator twice and found that the response time was same for both the queries. Since the graph plot generated will be same both the time you can add a feature to cache the generated graph ? Suggestion to add a database/cache linked to Nexrad Stations Fetch module.
Also after requesting the query, I was not able to use the website until the response was received. Maybe a feature to load the request in background so that user can continue to use the website ?
Amazing work guys !
Review by Team Garuda

Hey @pranavacharya ,
Thank you for pointing out. We have planned caching in one of the future milestones.
Also thank you for the UI suggestion, we will add this into the list of feature additions to our UI in future milestones.
Thank you for the review.
Cheers!


[FEATURE] Use Consul KV data for vars

Summary
Enhance levant to use data from Consul KV as variables when rendering nomad jobs.
Background
When an organization uses Consul KV as a configuration repository for applications running within a particular environment and datacenter, the ability to access this data when rendering and deploying nomad jobs is very appealing.
The configuration data in Consul KV is often backed by a version controlled git repository, loaded using a tool like git2consul, and already accounts for per-environment and per-region / per-datacenter differences.
When similar data is also required to render and deploy nomad jobs, we are faced with either fetching the data from Consul prior to calling levant or replicating the required data locally to levant in the form of disparate vars and/or vars files structured by environment and/or region.
A very simple key keyword could support many of the more common use cases without embedding or attempting to emulate full consul-template functionality:
[[ key "config/serviceA/keyA" ]]

User feedback could help decide if support for just key is sufficient for the most common use cases. keyOrDefault is frequently useful as well.
Several new configuration items will be required to access the consul API whether running on the same host that is running levant or remotely. Following the patterns established by HashiCorp for consul environment variable and cli options should help users leverage their existing consul configuration settings.
The existing -var and -var-file should remain and can take precedence.
Examples
For each example, the syntax shown would be part of the nomad job file.
A dynamic job name for a service:
job "[[ key "config/serviceA/nomad_job_name" ]]" {
  ....
}

The nomad region is a required attribute and deploys to multiple datacenters each need to specify a value for it:
job "api" {
    type = "service"

    region = "[[ key "config/serviceA/nomad_region" ]]"
    datacenters = ["us1"]

Container registry and artifact paths are frequently dynamic and vary across projects and environments:
job "api" {
    config {
        image = "[[ key "config/serviceA/nomad_registry_url" ]]:${NOMAD_META_VERSION}"
     }

     artifact {
         source = "[[ key "config/serviceA/nomad_artifact_url" ]]/${NOMAD_META_VERSION}/foo.tar.gz"
     }

I certainly like this idea. I will look at trying to leverage Consul template for this if possible, however, if not I can add a template function which may be preferable as it opens up the possibility of adding addition functions (similar to Packer) in the future.

I just discovered your work, and internally we use consul-template to template our nomad job files so they are slightly different for the dev environment, staging and production. It works really well, but it doesn't have all the other features you have that integrates to nomad.
Perhaps you can make use of consul-template too and you get a templating engine with consul key-value access for free, and all the features of consul-template (when a consul key changes, the template is regenerated and you can hook into that to execute a command: deploy on nomad for example)


Add a section about reproducibility.

Thanks for the careful review, Peter!

Thanks for the careful review, Peter!


reactstrap Alert Component timeout leave is not work

transitionAppearTimeout: PropTypes.number,
transitionEnterTimeout: PropTypes.number,
transitionLeaveTimeout: PropTypes.number
this is my code
<Alert           color={color} isOpen={this.state.visible}           toggle={this.onDismiss} transitionAppearTimeout={150}           transitionLeaveTimeout={150} transitionEnterTimeout={150}         >           <strong>{message}</strong>         </Alert>
after 150ms , it does not leave

I am guessing Alert will not dismiss automatically for now, all *timeout props here should be derived from ReactCSSTransitionGroup which is only about animation when component mount/unmount.

@cunwenWang transitionLeaveTimeout dictates it how long the element should animate when its being removed from the DOM.  If you want to have the alert removed after a 150ms, you'll need to set a timeout for 150ms that calls this.setState({visible: false})

@cunwenWang did the replies from senseluo and mkalish answer your question? If not, can you clarify what you expect to happen and what is actually happening?

I'm closing this since the the issue is no longer active and it appear it was answered above. Please comment and it will be opened if you feel the issue has not been resolved / the question has not been answered

please install this npm i react-transition-group --save


docs: route ordering and utils docs

We want to order like:

Home
Documentation

Modifiers

Popper Offsets
... (based on default dependencies and phase, we can manually specify this?)

Utils

Detect Overflow

Typings

Tried using index in the frontmatter but the MdxRoutes plugin doesn't seem to pick it up.
docs/src/utils/processRoutes.js algorithm is so inefficient/brittle 😥, needs improvement, pls help @FezVrasta 😄

Can this be merged?


Unable to build a command line target

Accio 0.6 and Xcode 10.2.1 on macOS 10.14.4
Hi, this may just be user error, I've only just started trying to use Accio and don't have much experience with SPM/Carthage.
I want to add SwiftyJSON to a command line target for macOS, and have it be self-contained, so I expect the right approach is for the framework to be embedded like this https://iosdevcenters.blogspot.com/2015/12/realm-error-dyld-library-not-loaded.html
I'm seeing this when I try to build BIAdapter in Xcode after generating a Package.swift and running accio install
Environment variable not set: FRAMEWORKS_FOLDER_PATH
Command PhaseScriptExecution failed with a nonzero exit code

If I export FRAMEWORKS_FOLDER_PATH=$HOME/Library/Frameworks and xcodebuild -workspace X.xcworkspace -scheme BIAdapter it builds, but then the framework isn't embedded:
$ otool -L BIAdapter
BIAdapter:
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 400.9.4)
	@rpath/SwiftyJSON.framework/Versions/A/SwiftyJSON (compatibility version 1.0.0, current version 1.0.0)
        ...

$ ./BIAdapter
dyld: Library not loaded: @rpath/SwiftyJSON.framework/Versions/A/SwiftyJSON
  Referenced from: $HOME/Library/Developer/Xcode/DerivedData/Defendpoint-fgbftiezftkbbebxeuagglvuyrtr/Build/Products/Debug/./BIAdapter
  Reason: image not found

Here's my Package.swift
// swift-tools-version:5.0
import PackageDescription

let package = Package(
    name: "BIAdapter",
    products: [],
    dependencies: [
        .package(url: "https://github.com/SwiftyJSON/SwiftyJSON.git", .upToNextMajor(from: "5.0.0")),
    ],
    targets: [
        .target(
            name: "BIAdapter",
            dependencies: [
                "SwiftyJSON"
            ],
            path: "BIAdapter"
        ),
        .testTarget(
            name: "BIAdapterTests",
            dependencies: [
                "SwiftyJSON"
            ],
            path: "BIAdapterTests"
        ),
    ]
)

Yes, this sounds like a misunderstanding. When your goal is to write a command line tool, there‘s no reason to use Accio. You can use SwiftPM directly for that. Actually, Accio itself is a command line tool which uses SwiftPM to include dependencies, so you might just want to check out the project itself. There‘s instructions to get you started with development in the CONTRIBUTING.md file.
For more details on how to get started, checkout the likes of this or that article.

When your goal is to write a command line tool, there‘s no reason to use Accio. You can use SwiftPM directly for that.

Sorry, I didn't make it clear in my original post - I already have a fairly complex project file which I can't replicate using SwiftPM and a custom xcconfig. I don't think Swift is able to integrate well with existing Xcode projects, and the articles you linked are for creating a new project, which makes Accio seem like a good fit for my use case.

In that case, I will need further explanation of your goal here, especially including the following:

Your Package.swift file contents
The last 10-20 lines of Accios output


process_failed_first_last_failed → NotImplementedError

I often encounter this exception, especially when I previously ran pytest and a syntax error prevented it to run.
File "/usr/local/lib/python3.7/site-packages/random_order/plugin.py", line 60, in pytest_collection_modifyitems
INTERNALERROR>     process_failed_first_last_failed(session, config, items)
INTERNALERROR>   File "/usr/local/lib/python3.7/site-packages/random_order/cache.py", line 28, in process_failed_first_last_failed
INTERNALERROR>     raise NotImplementedError()
INTERNALERROR> NotImplementedError

Do you have a name of the test that caused this?
Is that a method in a test class or a plain test function?

I'm asking because the issue is to do with the parsing of the name of the failed test in the cache plugin's lastfailed.

Oh interesting. Let me dig that information for you.

Is it possible that it is the case when pytest is run on tests that don't include the file in question? I run pytest 3 time in a raw and I have the issue if pytest is ran without the erroneous file.

I just tried removing the test or test module that has last failed and then running with --last-failed and was not able to reproduce the error. If you could edit locally that raise NotImplementedError() line to include more information and that would help.

Today I found the same bug on a project but it happened only once, very weird.


docs(guides) update tree shaking guide

In last PR #2426 the mode was changed from production to development and it broke the tree shaking behavior because usedExports is false in development mode.

By the way @torifat , the PR you've linked wasn't changing the mode at all, only double quotes to single quotes for consistent formatting.

Thanks!


Remove homogeneous collection API from records

Records should not have the map/filter/reduce and other API methods useful for homogeneous collections like List and Map. Their operation does not do what is expected.
#220 #268

I just ran into this issue and spent a while trying to debug it. Is there any way we can add this to the docs so that future developers don't spend a while debugging this issue? I'd be more than happy to make that change

:+1:

I found all methods that called recordType.constroctor() has problem, includes merge (actully mergeIntoCollectionWith).
var R = Immutable.Record({});
var r = new R();
r.merge({});   // no problem, return "r" itself
r.merge({a: 1});  // return undefined

Why remove them altogether though ? Record.map seems very useful for instance.

Note when tackling this, ensure isImmutable() still returns true for record instances


Embedded HTML width

Hi,
I am tweaking Daux to build a course, and it is amazing. I am trying to embed HTML. However, width is always 100% even in sliced mode (menu on the left, text on the center, code on the left).
The embeded HTML takes all the space available. Here is a sceenshot :

Here, I thought the Fiddle would be only in the center column.
I am not sure whether this looks good or bad yet (the more I look at it the better it is^^), but anyway is this the expected behaviour ?

I'm closing this issue as the Daux project has been moved to https://github.com/dauxio/daux.io
If this is still an issue for you on the new project, please open an issue on the new repository.


Is this project not active any longer?

same question...  i assume it is dead... looking for a project like SEAN...
i take this as a starting-pont and look at all the errors...

Same for me... Cannot do anything except create a user

This one is active: https://github.com/seanjs-stack/seanjs But it's still under heavy development.


The first code file after vscode is opened cannot display language intelligent prompts correctly

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.93.0-insider (user setup)
OS Version: Windows_NT x64 10.0.26120

提交: 8245d426e009ec7864018ed05ae2a282af42f03b
日期: 2024-08-08T05:03:51.428Z
Electron: 30.1.2
ElectronBuildId: 9870757
Chromium: 124.0.6367.243
Node.js: 20.14.0
V8: <IP_ADDRESS>-electron.0
Steps to Reproduce:

open vscode
The first file cannot display the language's intelligent prompt correctly
Other files are  normal

Maybe a duplicate of #225183

/duplicate of #225134


maximize current editor window to full DBeaver window size

Is your feature request related to a problem? Please describe.
Dont think so.
Describe the solution you'd like
Ability to maximize current editor window to occupy all DBEaver window.
Describe alternatives you've considered
I think only option at the moment - which I don't really like - is to right click tab > Detach which makes specific editor window floatable, meaning I can resize and use physical monitor screen space I want, but this isn't the most user friendly as needs user to manually move/resize, while my suggestion is simply to maximize current editor window using and get constrained by full DBeaver window size.. is this possible ?
Additional context
current view with multiple editor open (grid 1 row x2 columns) - with each zone having 5 files

Desired functionability - maximize current editor (any of the 10 files on the 2 tabs on the 1x2 grid) to use full physical window size of DBeaver like this:

Does this option already exists or would need this to be a new feature ?
Thanks in advance ~
Emanuel

Does it the behaviour are you asking for?
(I'm double-clicking on the script or other tabs to maximize or collapse the tab content)
https://user-images.githubusercontent.com/28875055/199428079-56912694-104f-458d-bfa9-f30bca1ba717.mp4

Does it the behaviour are you asking for? (I'm double-clicking on the script or other tabs to maximize or collapse the tab content) https://user-images.githubusercontent.com/28875055/199428079-56912694-104f-458d-bfa9-f30bca1ba717.mp4

Thats exactly what I'm looking for, how is it done ? some keyboard shortcut or mouse interaction?

@emanueol mouse double-clicking on tab (cursor is a little bit higher than tab on the video, maybe it's my screenshot recorder issue)

Doesn't work. When I double-click any of the editor tabs nothing happens, see the animated gif:

Hello @emanueol,
You can detach the editor by clicking the Detach menu item in the context menu.

Hello @emanueol,
You can detach the editor by clicking the Detach menu item in the context menu. Perhaps this will help you.

I/we know, but that's not what I suggest to request as an improvement on this ticket, thanks.

Unfortunately, we can't do anything regarding this issue on our side.


Move restart-policy validation from client to daemon.

First crack at addressing  #23960
Moved the specific checking around restart-policies into the daemon. Because the client still needs to send a valid HostConfig, the parsing of Name and MaximumRetryCount have to minimally occur in the client. The same checking is done in compose.
Reason for doing this is to make a better experience for non-Docker clients (like compose) to avoid having to do complex policy checking in those clients.
Refactored the client tests to only check for structure, and added tests into the daemon to verify the new behaviour.
My first contribution so all / any feedback welcome.
Signed-off-by: John Harris<EMAIL_ADDRESS>

@johnharris85  Thank you for your contribution. reviewed and tested.
LGTM

Thanks @johnharris85! Left some suggestions
For reference; the output format changes a bit, because the error message now comes from the daemon;
Before:
docker run --restart always:12 -d nginx:alpine
docker: maximum restart count not valid with restart policy of "always".
See 'docker run --help'.

docker run --restart something -d nginx:alpine
docker: invalid restart policy something.
See 'docker run --help'.

docker run --restart on-failure:2:3 -d nginx:alpine
docker: restart count format is not valid, usage: 'on-failure:N' or 'on-failure'.
See 'docker run --help'.

After:
docker run --restart always:12 -d nginx:alpine
docker: Error response from daemon: maximum restart count not valid with restart policy of 'always'.
See 'docker run --help'.

docker run --restart something -d nginx:alpine
docker: Error response from daemon: invalid restart policy 'something'.
See 'docker run --help'.

docker run --restart on-failure:2:3 -d nginx:alpine
docker: invalid restart policy format.
See 'docker run --help'.

Also, this needs to be mentioned in the API changes. We temporarily removed the 1.25 from that document (https://github.com/docker/docker/pull/23777), but will add that back once Docker 1.12 is GA.

ping @johnharris85 could you update your PR?

Sorry @thaJeztah this one slipped under the radar. Will update this weekend.

No worries! We're lagging behind as well due to the 1.12 release :innocent:

@thaJeztah there is no API change here though right? Just a moving of where the logic checking occurs?

@johnharris85 it sounds like it's API change because now API returns new errors which previously were only in CLI.
LGTM

@johnharris85 I think the API change here is that values that were previously accepted, will now produce an error (if someone uses the API directly), so it may be worth a small mention in the API changes https://github.com/docker/docker/blob/master/docs/reference/api/docker_remote_api.md#v125-api-changes
(similar to the note we added for hostname being now validated in API 1.24)

LGTM
@thaJeztah i don't think it needs a mention, its a bug that it does not validate on the server side in the first place.

The docs/references/api/docker_remote_api_v1.25.md have to be update to talk about the validation I think.
/cc @thaJeztah

ping @johnharris85

I'll merge this and will do a follow-up for the documentation update


Unable to get the Last page working

Hi,
I'm trying to get the "Last page" logic working. But it keeps returning the same first page rows. I following of logic mentioned in the documentation to use "backward"
searchResultsQuery.KeysetPaginate(_keysetBuilderAction, KeysetPaginationDirection.Backward);
but it keeps returning the same first page results. My query uses multiple joins across multiple tables using EF Core lambda. As EF Core doesnt support DistinctBy so I'm using .GroupBy(z=>z.Id).Select(g=>g.First()) for getting the Distinct rows in my query, is this causing an issue finding the last page ?

Hi. A minimal repro would really help, as I won't be able to tell just from what you explained. Not sure how .GroupBy(z=>z.Id).Select(g=>g.First()) factors into this, would need to see the full code in a repro.
Would also help if you try to run the sample in this repo and double check there. If the sample works that means something's wrong with either how you're building this query or maybe there's an edge case in what you're using.

I'm closing this, please reopen if you still have issues.