Why is my server up and running, but there is no response to my message to the bot?

logs
websocket connecting to ws://<IP_ADDRESS>/api/v4/websocket

Closing this with no further input, feel free to reopen once you have more information. It works for everyone else, I guess, and without knowing the difference in the setup there is no way we can help without further information.

Thanks, I already know what's going on because my robot's name isn't 'chatgpt'!


gdu: 3.0.0 -> 4.2.0

https://github.com/dundee/gdu/releases/tag/v4.0.0
https://github.com/dundee/gdu/releases/tag/v4.1.0
https://github.com/dundee/gdu/releases/tag/v4.2.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

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

1 package built:

gdu

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

1 package built:

gdu

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110227 run on x86_64-linux 1

1 package built:

gdu

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110227 run on x86_64-linux 1

1 package built:

gdu


feat: changed button hover effect

closes #1134
👷🏻 Changes made

Add a background color to show the differentiation between the buttons which is hovered by the user.
📸 Screenshots

Before:

After:

Hi @dhananjay-Byte , the changes look good however I would recommend a small change.
In cases where there are two buttons side by side, the solid button can have a hover effect (which you applied), and the other button can be as it it (the previous hover effect).

got it

Let's catch up 🙋🏻♂
Hi there, it has been a while since we last had a conversation so I wanted to know if you are still working on this. Please drop a reply shortly so that we know you are active, feel free to convey any problem you face in the comments, or mail me at<EMAIL_ADDRESS>

sorry for being inactive @tamalCodes  ill patch up this work ASAP from now.


Documentation: GPG stanza parameters

gpg.md should document how to get the key_id
A command line example should be inserted here - unfortunately, I've never used GPG verification so I have no idea how to do so. Pinging @fanquake here
Related to: https://github.com/caskroom/homebrew-cask/pull/16090

I'll take a look at this.

When we say we want the key ID, don't we actually mean the fingerprint?
i.e if you look at the TorBrowser cask, we have:
  gpg "#{url}.asc",
      key_id: 'ef6e286dda85ea2a4ba7de684e2c6e8793298290'

When you look at the Tor Browser Developers key:
gpg --fingerprint 0x4E2C6E8793298290
pub   4096R/0x4E2C6E8793298290 2014-12-15 [expires: 2020-08-24]
      Key fingerprint = EF6E 286D DA85 EA2A 4BA7  DE68 4E2C 6E87 9329 8290
uid                 [ unknown] Tor Browser Developers (signing key<EMAIL_ADDRESS>sub   4096R/0x7017ADCEF65C2036 2014-12-15 [expires: 2017-08-25]
      Key fingerprint = 5242 013F 02AF C851 B1C7  36B8 7017 ADCE F65C 2036
sub   4096R/0x2E1AC68ED40814E0 2014-12-15 [expires: 2017-08-25]
      Key fingerprint = BA1E E421 BBB4 5263 180E  1FC7 2E1A C68E D408 14E0

0x4E2C6E8793298290 or 93298290 is the key ID.
EF6E 286D DA85 EA2A 4BA7  DE68 4E2C 6E87 9329 8290 is the key fingerprint.

@fanquake I would like to help you with this, as I have some knowledge of gpg. Is there anything you would like me to do?

What we're calling the key_id is known by GPG as the fingerprint.
One way to find fingerprints (known here as key_id) is as follows:
gpg --search-keys <search terms>

Then select the correct match to import the key. For example, $gpg --search-keys libreoffice code signing finds the correct key for Libre Office. Next, type `$gpg --signature
Next, download and verify the signature available on publisher's website. Note that the installation file must be in the same folder as the signature file.
gpg --verify /path/to/signature

Verification going well, you can add that fingerprint and link to signature file to the cask file.
However, I think it should be noted also that it would be up to the community here to certify that the key_id specified within the cask does in fact belong to the developer or publisher, just like it is up to the contributor to certify that the URLs are owned by the developer, and those are basically trusted by the users. We can see the key_id present within the cask file as a sort of separate key certification that we are providing here.
It would also be worth saying something about whether it is any better to include a gpg check than a hash check. Since both the hash and the key_id are provided by whoever is contributing the cask, I'm not sure how much more secure a gpg check would be.
I can volunteer to write all this into gpg.md, if others agree.

@tgdnt go ahead.

An important question: has this functionality been implemented yet?

No, not yet. Open issues: https://github.com/caskroom/homebrew-cask/issues/164 and https://github.com/caskroom/homebrew-cask/issues/5971


First Draft

Hi guys, as you know I'm struggling to push my code up to this repository (will ask Greg tomorrow). For now, I've attached a pdf copy of the first draft for someone to look at and maybe start thinking about comments and docstrings.
draftcodeRN_10112022a.pdf

Update: find the file in my branch, rubabadrafts, and @sksamkumar pull it down from there

I'm on it! :)


Update Screen component insets

Due to the wrong order of styling prop inside the Screen component
we were facing with the unwanted behaviour of bottom tab (iOS) and
header (android)

Thanks, @emadbaqeri , I checked your PR out, and there are these issues:

The space between the bottom tap bar labels with the bottom of screen is too much, please find the attachment, you can compare similar app screenshot
Dake mode does not work correctly for bottom tab bar
Header is not adjust in dark mode
Status bar color is not the same as header in dark mode

Thanks, @emadbaqeri , I checked your PR out, and there are these issues:

The space between the bottom tap bar labels with the bottom of screen is too much, please find the attachment, you can compare similar app screenshot
Dake mode does not work correctly for bottom tab bar
Header is not adjust in dark mode
Status bar color is not the same as header in dark mode

Thanks for your comments @ghorbani-m, I will be fixing these issues today and will put them in the highest priority

Closed by #283


feat!: replace Margin and Padding with Gaps

This is breaking for sure and will very likely generate a lot of churn.
Fixes #931
Also increases code readability as arguments always keep their intention in the method name. If that's not enough, the Struct pattern Gaps {left: …, … } should be used to keep the readability. I didn't create a new method for exactly this reason in order to keep readability of code up.
I intentionally Padding::proportional as characters on terminals are not 2x1 and it's only a very broad approximation. It's likely a better idea to use something custom for the given use case than some magic n*2.

Consider whether this could be implemented with an Into<Gaps> approach that is (ignoring type inference) non-breaking on the public API. I'd be inclined to leave the deprecation alone and treat this as a way to add features without breaking existing ones.
I'm not sure that Gaps seems like the right name for this. Consider looking to CSS / Taffy / other GUI-ish libs and crates for inspiration in a name.
Is it possible that Padding is the right name to use here and the problem then becomes just convert Margin to Padding in all places?

Consider whether this could be implemented with an Into<Gaps> approach that is (ignoring type inference) non-breaking on the public API. I'd be inclined to leave the deprecation alone and treat this as a way to add features without breaking existing ones.

I did. Not without removing the const possibilities of this and I think having const for something that is called often is quite useful.
As it creates a lot of churn I think it might be a good idea to either do a clean cut or just drop it and ignore these inconsistencies to exist (which would be annoying and smelly but well…)

I'm not sure that Gaps seems like the right name for this. Consider looking to CSS / Taffy / other GUI-ish libs and crates for inspiration in a name.
Is it possible that Padding is the right name to use here and the problem then becomes just convert Margin to Padding in all places?

Not sure about the naming of Gaps either, but neither Padding nor Margin are a good name for it.
CSS: padding and margin are two opposites. They take the same type of arguments (CSS doesn't seem to give it a name), but their concepts are completely opposite.
At first, I implemented both Margin and Padding with a macro as their implementations are nearly identical, but this seemed wrong too. There should be a single struct being used for both concepts and the actual place that uses them should name it.

Semantically / Linguistically, a margin is a form of padding, but padding is not a form of a margin, so Padding kinda makes sense still (even if it's not used in the same sense as CSS does). What we're really referring to is the amount of space to allocate on each direction of a rect.

I did. Not without removing the const possibilities of this and I think having const for something that is called often is quite useful.

What are these concerns?

As it creates a lot of churn I think it might be a good idea to either do a clean cut or just drop it and ignore these inconsistencies to exist (which would be annoying and smelly but well…)

I'm leaning towards dropping preferentially instead of Gaps (unless there's a really good name that makes sense), but would like to explore Into a bit more before dropping this PR fully.

I did. Not without removing the const possibilities of this and I think having const for something that is called often is quite useful.

What are these concerns?

const allows to be run at compile time rather than runtime. When layouting stuff is a constant it can be done ahead of time as much as possible to remove that from runtime. Doesn't work as soon as the current detected area is used but for stuff with fixes sized for example it will not require anything to be computed at runtime at all.
Ideally I would like to have the complete layout module to be const.

Semantically / Linguistically, a margin is a form of padding, but padding is not a form of a margin, so Padding kinda makes sense still (even if it's not used in the same sense as CSS does). What we're really referring to is the amount of space to allocate on each direction of a rect.

Hm… still would like to avoid the naming confusion with CSS.

I meant, which parts of this can no longer be const that are performance sensitive?
There's probably some fundamental ratatui design things (stemming from the builder-lite pattern) that are causing this.

I meant, which parts of this can no longer be const

Everything with Into

I currently don't see this getting merged as its breaking and not well fitting


Look into signing commits

Signed commits would be nice to have

We can do that by configuring gpg in the github SCM. Docs https://www.updatecli.io/docs/plugins/scm/github/.


docs: remove review capacity notice

The Cargo project is in a better state than it was when maintainers added this notice. I believe it's time to take it down, as maintainers have got more time on it, and more people are willing to contribute to the project constantly.
Note that this doesn't mean Cargo start accepting arbitrary pull requests or features. The reviewers are still a small handful of people. The guideline never change — issue, discuss, then pull request.

@bors r+

:pushpin: Commit 6f3e30b773fdd3cc8a7f771509dda8dca616bf0d has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 6f3e30b773fdd3cc8a7f771509dda8dca616bf0d with merge 9b994157b35285b34aeec5110b8ec4c6b0292aee...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 9b994157b35285b34aeec5110b8ec4c6b0292aee to master...


Name change from World edit to scarpet edit

Closes #47
Also helps disassociate from regular WE mod, to avoid possible problems.

Looks good. I'll test it soon, but I don't expect it to break much. Either that, or break catastrophically.

ok, it appears to work fine now.

I crash when I load the world with the app. I had the same happen to me when developing some other thing, it happened when the command system was badly defined, somehow. Running latest carpet and fabric.

It was a broken softlink from the namechange. I'll leave this here in case the same happens to someone else. Works fine!


Goes haywire on both LD2410B and LD2410C

I have both an LD2410B and C, connected via USB UART adapter to two Macs running Chrome.
On both laptops, with both devices, I get this behavior:

It flashed intensely, as the numbers switch quickly between small (hundred) and large (tens of thousands).
What might be going on? It's almost like some endianness is being misinterpreted?

Thanks for the report. That's seriously strange. I'm unable to reproduce this on a LD2410C. I wonder if there is a new protocol version, firmware, variant or something like that.

Could you please send an excerpt from the serial console? I'm particularly interested in how the device replies to the payload sent by the configurator when you click "Reload all".
When you click "Reload all", does any information ever get populated in the "Firmware version" and "Protocol version" fields up top?
Roughly when did you purchase each of your sensors?

A few weeks ago, I bought one "LD2410B kit" and one "LD2410C kit" from https://www.aliexpress.us/item/3256805945328831.html
LD2410C
Here's an excerpt form the serial console for the LD2410C: https://gist.github.com/andorardo/ae19b1756d24d2e8902be451986621d1
I tried repeatedly pressing "Reload all", but unfortunately nothing changed.
LD2410B
Same phenomenon when trying to connect at 256000.
I was able to connect to the LD2410B via BT via the iPad app, and configured the serial port to a slower speed 230400. Now the details show up: Firmware version 2.68.36074916 Protocol version 1.
Here is the serial console from the slow LD2410B: https://gist.github.com/andorardo/7a15873084cc371bfcc76c42f14f1d54 -- Is that helpful?
Serial adapter
The kit comes with a CH340, which is what I've been using. I don't know if that makes any difference.

UPDATE: The FT232 arrived this afternoon, sooner than I expected. The LD2410C now works. 👍  Also: Firmware version 2.68.36074916 Protocol version 1
I'm glad that's sorted. :) Needless to say, I'm rather disappointed in the UART that came with the kit.  :(

Thanks for the additional info, and the confirmation that things are working now with a new serial-to-USB adapter. I'll close the issue but please re-open it if you think there's something broken in the configurator which contributes to the haywire behaviour you saw previously.
Also keep an eye on #13 which implements Bluetooth in the configurator. I finally got around to reviewing and testing the PR recently and it's a great alternative to using Serial over UART and USB.


Add support for batches

Closes #39.
Support and tests for analyzer calls on input batches.
Analyzers still return Explanations. However, if the batch dimension (=last dim.) of the explanation is not equal to one, heatmap will output a Vector of images.
As a default, this vector is unpacked for "singleton batches". This can be turned off by setting the heatmap kwarg unpack_singleton=false.
This required refactoring of the neuron selection, calls to gradient_wrt_input and an additional Tullio Einsum index for LRP rules on Dense layers.

Codecov Report

Merging #46 (f8db22a) into master (715c5a8) will decrease coverage by 1.60%.
The diff coverage is 94.64%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
- Coverage   94.52%   92.92%   -1.61%
==========================================
  Files           9        9
  Lines         201      226      +25
==========================================
+ Hits          190      210      +20
- Misses         11       16       +5

Impacted Files
Coverage Δ

src/lrp_checks.jl
96.29% <ø> (ø)

src/analyze_api.jl
85.71% <81.81%> (-14.29%)
:arrow_down:

src/heatmap.jl
93.75% <93.33%> (-2.81%)
:arrow_down:

src/gradient.jl
100.00% <100.00%> (ø)

src/lrp.jl
96.29% <100.00%> (+0.14%)
:arrow_up:

src/lrp_rules.jl
100.00% <100.00%> (ø)

src/neuron_selection.jl
100.00% <100.00%> (ø)

src/utils.jl
50.00% <100.00%> (-6.25%)
:arrow_down:

Continue to review full report at Codecov.

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


Cannot resolve "pino-pretty" nuxt 3 cloudflare-pages preset

Check existing issues

[X] I checked there isn't already an issue for the bug I encountered.

Describe the bug
On build ssr nuxt 3 project with nitro preset cloudflare-pages
ERROR  Cannot resolve "pino-pretty" from<EMAIL_ADDRESS>and externals are not allowed!
Link to Minimal Reproducible Example
https://stackblitz.com/edit/nuxt-starter-mdeyveim?file=nuxt.config.ts
Steps To Reproduce
No response
What Wagmi package(s) are you using?
@wagmi/vue
Wagmi Package(s) Version(s)
latest
Viem Version
latest
TypeScript Version
latest
Anything else?
Nuxt 3 ssr with nitro preset cloudflare-pages. Please check reproduction

https://github.com/WalletConnect/walletconnect-monorepo/issues/4466


Support deep param for $.extend

jQuery.extend API supports passing a boolean argument as first param and it will recursively copy properties of the source object.
Would this library consider support that as well?

If someone is willing to push a PR implementing it, that doesn't add many bytes to the minified and gzipped bundle then yes, I'll probably accept it.
What is your use case for this exactly?

Aligning cash more with jQuery is important (https://github.com/kenwheeler/cash/issues/193), but that is more about ensuring the things we implement work the same way as they do in jQuery, we are not trying to reimplement the whole of jQuery itself.

Awesome - thanks.
I've added the following in this commit:

cash.inArray
cash.merge
cash.map (array only currently, like cash.each)
cash.isPlainObject
cash.trim
Deep copying for cash.extend

Those were all needed for DataTables to run. I've also committed a couple of changes to my v2 branch of DataTables (I'm fairly sure most of the same changes could be applied to 1.10) to address one or two points that wouldn't be suitable for cash (e.g. using :eq() in a selector).
I think it takes the size up from 14.4KB to around 15.3KB - so roughly a K more. Happy to send a pull request with that diff if you think its suitable?

@DataTables We are trying to keep cash as small as possible, ideally the minified and gzipped code should be under 5kb total.
As a rule of thumb those sorts of utilities are not provided by cash, unless we also need them internally and jQuery exposes them too.
So I would avoid including the following methods:

cash.inArray
cash.merge
cash.isPlainObject
cash.trim

cash.map (array only currently, like cash.each)

This one is useless to us, since we support only relatively modern browsers, so the Array.prototype.map method is available to us.

Deep copying for cash.extend

Feel free to submit a PR about this. Try to keep the code as short as possible.

I think it takes the size up from 14.4KB to around 15.3KB - so roughly a K more.

That unminified and ungzipped size doesn't really matter for us, we care about the minified and gzipped size.
Since we don't ship some of the utilities DataTables needs how do you plan to add compatibility with cash? Will you add a cash-compat library that includes those missing utilities maybe?
IMHO ideally cash should be a drop-in replacement for jQuery, maybe you could consider avoiding using those missing methods entirely.

The gzip version comes out at 5331 bytes.

This one is useless to us, since we support only relatively modern browsers, so the Array.prototype.map method is available to us.

Its more as a proxy so libraries like DataTables don't need to change their code (and jQuery can be dropped in for support for older browsers).

Since we don't ship some of the utilities DataTables needs how do you plan to add compatibility with cash?

I'm not sure I will add compatibility with Cash if it doesn't provide a drop in replacement for jQuery. I don't mind changing some things in DataTables (such as one or two of the selectors I used not being compatible with qsa) but if there are utility methods missing such as isPlainObject I'd just need to implement them in DataTables, so there would be no size benefit to the end user (indeed, it would result in a larger size for those using jQuery).

I'm not sure I will add compatibility with Cash if it doesn't provide a drop in replacement for jQuery.

Its more as a proxy so libraries like DataTables don't need to change their code (and jQuery can be dropped in for support for older browsers).

I understand that, but the only way to make cash a drop-in replacement for jQuery on our end is to re-implement jQuery, which we are not interested in doing.
Library authors on the other end can use only the commonly available APIs to make their libraries more compatible. Also keep in mind that cash's goal is to be "absurdly small", implementing things like $.map, which basically nobody needs except people supporting very old browsers (and cash itself is "only" IE10+ compatible), goes against that.

so there would be no size benefit to the end user (indeed, it would result in a larger size for those using jQuery).

It really depends, let's say the hypothetical cash-compact shim that implements a few missing methods weighs .5kb minified and gzipped, then:

DataTables + cash + cash-compact = 28kb + 5kb + .5kb = 33.5kb
DataTables + jQuery 3 = 28kb + 30.3kb = 58.3kb
DataTables + jQuery 3 + cash-compact = 28kb + 30.3kb + .5kb = 58.8kb

If you implement those methods in a separate bundle there's no extra cost for jQuery users, and a potential 24.8kb of savings for cash users.
If you implement those methods inside DataTables then users can more easily switch to cash and save bytes, at the cost of 1% in the library size increase, which IMHO it's probably an overly pessimistic estimate, considering that I'm happy to add deep extend support to cash.
If all the extra methods end-up weighting .1kb once minified and gzipped, which seems a more reasonable estimate to me, then we are talking about a potential 0.2% increase in the bundle size for jQuery users for a potential 43% savings when switching to cash, and we are not even accounting for the extra time needed for parsing, compiling and executing all that extra JS that comes with jQuery.
IMHO if I add deep extend support here, and you add the other methods on your end, the value proposition is pretty good, what do you think?

The deep extend needs isPlainObject to be added unfortunately, so I've exposed that along with the other similarly exposed jQuery methods.
I guess my issue as a library author is that I don't really want to write and support the missing methods of jQuery. I could, and it would be a way out of this, but it would also mean that any other library authors that wanted to add support for cash would also need to fill any gaps in the api. To my mind, utility functions such as this need to be in a utility library such as cash or jQuery as a core point.
I absolutely take the point about not reimplementing jQuery - that has always been my reply when folks ask me if there is going to be a DataTables version without the requirement of jQuery. The problem is I used a lot of the jQuery utility methods to avoid browser issues in the old days (and I still intend to support older browsers) - so only having a select subset makes it difficult for me and I think would limit other jQuery plug-ins from using cash.

The deep extend needs isPlainObject to be added unfortunately

We could add $.isPlainObject too in that case.

I could, and it would be a way out of this, but it would also mean that any other library authors that wanted to add support for cash would also need to fill any gaps in the api.

No way around that unfortunately, only jQuery implements everything that jQuery provides.

To my mind, utility functions such as this need to be in a utility library such as cash or jQuery as a core point.

IMHO they made a mistake exposing some of those methods. Maybe it made sense in the old days, when browsers didn't implement some core methods like map or trim and when things like WebPack weren't a thing, but now we are left with these kinds of problems.

so only having a select subset makes it difficult for me and I think would limit other jQuery plug-ins from using cash.

Hopefully not too difficult, if with a few changes and less than 50 lines of code you can get something as big as DataTables to use cash. Unfortunately supporting some of those utility methods just don't make sense for cash, we aren't trying to be maximally compatible with all existing plugins.

I think its remarkable what you've been able to do with cash. As you say, getting something like DataTables to run with it, with just a few hours of effort on my part is a huge testament to the capability of cash and its jQuery interoperability.
The deep clone is available in the fork I linked to above with the diff if you want to pull it in that way or I can clean my fork and add just that in.
For the moment I don't think I'll be adding the extras needed to DataTables to make it compatible but I'll keep cash in mind for future development.

Implemented in #347


My users does not have an email

I have 2 question:

Before I start even learning I need to know if it would work on a existing database.

I need my users to be able to log in with an user not an email, what this means is that a user doesn't have an email so it is not necesary the @example.com. To make clear some users could be: jhon or george, this package could work so they can log in with that users?

@jiac0911 this package will create just a scaffold for you. so even if it is generating scaffold with an email, you can change your fields, form or controllers at any time the way you want.
It is not using any static package inbuilt class, so you are free to do whatever you want. And as it's also utilizing laravel auth scaffold only, so you can customize anything that is supported by laravel.
And this package can generate forms from your database as well, check here, but it's not fully working, you have to do the modification by yourself to make it fully working.


Options for IProfileService?

I have code that will save Clients / Scopes to a database similar to the IS4 Host Example. However, I need to think about how to handle IProfileService. (builder.AddInMemoryUsers(Users.Get())). What comes to mind is something along the lines of ASP.NET Identity. Salt / Hash, etc. Perhaps I could grab the latest code on github and copy it for my own. Do you have any advice or know of any good projects that store user names / passwords securely and / or that are built for IS3 / 4?

Use the aspnet core web app template with individual auth. This has all the aspnet identity pieces in it. Learn it and understand it. Then apply to idsrv's login pages.

I can see now how a 'UserStore' could be quite varied across use-cases. I am trying to take a simple approach and have one place users log into for many sites. Basically a user interface with my logo that extends the IS4 Host project somehow. User management screens would be nice as well. Maybe this is the wrong approach and i should instead use two sites for this, one using asp.net identity pretty much out of the box. Maybe I am confused. That being said, I also want to ensure the passwords are stored encrypted. I can build code to do this. The headache of forced lockout, etc is another concern of mine. Anyhow, my followup question is, if I created a User mimicking the example and a simple IProfileService would one User be associated with many Client / ClientClaims or should it be a strictly one to one. (one user name / password) = one IdentityServer4.Core.Models.Client or should it be one to many? Also, more interfaces for IS4 models might be handy for extensibility purposes.


NIFI-13072: Fix MonitorActivity problems with cluster scope flow monitoring

Summary
NIFI-13072
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

Reviewing...

@takraj Could you please rebase you PR onto the latest main? The module structure has been refactored since you opened the PR.
I also noticed that MonitorActivity's logic has been changed a bit:

original version: processor restart resets the state
new version: inactive state preserved upon restart

I suggest introducing a new property: Reset State on Restart (or similar) with default value true in order to keep the original behaviour.

@turcsanyip
I've made it optional, with the original behavior as default. Thanks for the suggestion. I also had to fix a bug, that prevented ths feature to work in case NiFi is not clustererd.
Could you please revisit?


WIP: Index anonymous structs

In #2863 I tried to use a fixed name for anonymous structs. That was no good so it was suggested to generate unique names for them. This does basically fix my original use case, but it doesn't seem to have the intended effect. I can still reproduce the crash in some of the test cases though I'm not sure why yet.
Also, some types that I don't think should be anonymous are identified as such:
$ sudo ./src/bpftrace -e 'uprobe:libc:getaddrinfo {printf("%d\n", args.hints->ai_protocol);}'
stdin:1:41-64: ERROR: Struct/union of type 'struct __anon_1' does not contain a field named 'ai_protocol'
uprobe:libc:getaddrinfo {printf("%d\n", args.hints->ai_protocol);}

but hints should have type struct addrinfo * [1].
Intended for #2867
[1] https://debuginfod.archlinux.org/buildid/8bfe03f6bf9b6a6e2591babd0bbc266837d8f658/debuginfo

closing due to inactivity


axios calls don't return

I have a suite of integration tests that use axios - which in turn uses the newly mocked out XMLHttpRequest. These calls don't return anymore.
I do stub out the API requests for the rest of my test suite, but I'd still like to be able to run integration tests to verify the contracts for some of the APIs - verifying the integration doesn't break.
Is there a way to replace the mock or unmock it? This used to work before 0.3.0.

This is likely because the mock for XMLHttpRequest never returns anything, which is fine. Generally you shouldnt actually need to mock things this far down, and should never actually call the network.
In this case, the best solution would be to use libraries like Mockery and Sinon to mock out axios, so you dont need to touch XMLHttpRequest at all.
Alternatively, a better solution would be writing an inteface for XMLHttpRequest which allows you to stub out individual urls so it can actually resolve, in the same way Superagent-mock does for superagent.

Yep right now the majority of the tests are using a mock for axios so we don't actually make the network calls.
The issue I want to solve is how to still allow for a subset of integration tests to verify that those contracts haven't been broken either by changes in the client or the API code.

I fear then you can't use react-native-mock for integration testing, as we mock out the network-calling parts of the app, so you couldn't do any real API calls with it.

That's unfortunate. It was working before 3.0.0. Just curious, what was the main driver for adding the mocks for the networking calls?
Maybe for now I can separate out the integration tests into a separate test run.

The driver was someone submitted a PR (#119). I've never done integration tests, so i'd consider network requests unnecessary for react-native-mock, but clearly there's a market for it. I believe these mocks also normalize the API between version of node and react-native, so this was still the right thing to do.
I think one of the nicest and simplest solution would be adding an environment variable (eg RNMOCK_ENABLE_NETWORKING) that wouldnt mock out network stuff, so people that need network can still have it when needed. This should be really simple to do, and not be a breaking change!

Yeah, even better would be a way to enable/disable in the code so we can turn it on or off for specific tests. I went down the route of separating out the test suites but now I need to combine the coverage reports so it would remove that need if I could do it within a single test run.


use electron-builder

https://github.com/electron-userland/electron-builder
Should simplify a lot of things

related to https://github.com/juliangruber/dat-desktop/issues/36

done! :)


Revert "Add debug info support for inlined and specialized generic va…

…riables."
There is an assertion failure building the source compatibility suite that
needs to be investigated.
This reverts commit 91f6f34119d74e2a729f2794040417708b77b4d5.

@swift-ci smoke test and merge


chore: update react-native-onyx with the latest version

@marcaaron
Details
This is just a follow up PR on #3423 to update to Onyx with the latest bugfix
Fixed Issues

Fixes #3482

Another PR was already open for this: https://github.com/Expensify/Expensify.cash/pull/3483/files


Azure WebApp  and inprocess

How to enable inprocess mode on azure web apps?
Trying to deploy I still see two process, include the dotnet.exe one.

Thanks


Source directory and Dockerfile in different locations

I have my source code in ~/projects/repo, and I have a Dockerfile inside ~/dockerfiles.
I'm trying to write a script where I can pass in the source code path, like " ~/projects/repo", and then it will run the Dockerfile against that. So I thought I could do someting like:
import Docker from 'dockerode';
import fs from 'fs';
import path from 'path';
import url from 'url';
import { nanoid } from 'nanoid';
import { LogStream } from './utils/log-stream.mjs'

const docker = new Docker();

const __dirname = path.dirname(url.fileURLToPath(import.meta.url))
const dockerFilePath = path.join(__dirname, '../dockerfiles/Dockerfile')

export async function buildImage(sourcePath, imageName, deploymentId) {
  buildId = nanoid()
  const absoluteSourcePath = path.resolve(sourcePath);

  if (!fs.existsSync(absoluteSourcePath)) {
    throw new Error(`Source path '${absoluteSourcePath}' does not exist`);
  }

  await docker.buildImage({
    context: absoluteSourcePath,
    src: [dockerFilePath]
  }, {
    t: imageName
  });

  console.log(`[${buildId}] Docker image '${imageName}' built successfully`);

}

So then I can run buildImage("~/projects/my-repo", "my-repo", "repo-id").
Although I get the error
Error: ENOENT: no such file or directory, lstat '~/projects/my-repo/Users/bitttttten/dockerfiles/Dockerfile'
}```

In theory it looks like it should work, however it seems to be putting the Dockerfile onto the context, rather than looking for it relatively.

Am I doing something wrong here? I've tried many different combinations and so far I'm stuck 😅

I'm also getting the same error

I've tried every combination as well. This seems to be a limitation of dockerode. A possible workaround is moving the Dockerfile to the top directory and having it named "Dockerfile" but this isn't ideal for every project.


Revert enabling "warnings as errors" in the default build

Following up and reverting part of https://github.com/SFML/imgui-sfml/pull/246

This is overkill. Remove -Werror if you're worried about that. Better yet add a new option like SFML does that lets users disable warnings as errors. It's a very bad idea to entirely remove compiler warnings because it's increasing the chances of bugs or other code smells getting in.

https://github.com/SFML/imgui-sfml/commit/7da7dfe10ee27c48878195940a5355eed60e3104
Here's the fix you actually want. Our list of warnings is only going to grow so it's good to have this option for those who for whatever reason are seeing a warning that isn't already caught in CI.

I want to make that release happen, and take no risks in it.

See https://github.com/SFML/imgui-sfml/pull/251


Request to add feature to set default alarm ringtone in settings

Is your feature request related to a problem? Please describe.
Given in title. It should be similar to how there is such an option for timers.
Describe the solution you'd like
An option for the same.
Use case
To avoid effort.

That's a good idea, this is the updated reposity https://github.com/BlackyHawky/Clock
Open the same issue there. We are already working on adding the ability to choose music on the device, but it will take some time

@qw123wh You can close this as it is implemented in my repository.

Completed


Added DBMS page

I have added dbms page.
Closes #253

fix your code website has errors

updated @yatulearn

@yatulearn can u check once please

make it responsive for mobile phones


Typo Correction and Clarity Improvement in Vector Search Explanation

Usage of "using or an embeddings API": A  typo or grammar error in this part. Can be corrected it as follows:
Original: "by using or an embeddings API"
Suggested: "by using an embeddings API"
Rephrasing for Clarity: The following sentence can be changed to enhance readability
Original: "It works by taking the vector representations (lists of numbers) of your data that you created by using a machine learning model by using or an embeddings API."
Suggested: "This process involves taking the vector representations (lists of numbers) of your data, which you've generated using a machine learning model or an embeddings API.

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

ID: 5e53e4aa-c504-448b-9598-419eda127192
Version Independent ID: 79fdea2a-5317-4996-a2b1-57662c2b9aa0
Content: Vector - Azure Cosmos DB for MongoDB vCore
Content Source: articles/cosmos-db/mongodb/vcore/vector-search.md
Service: cosmos-db
Sub-service: mongodb-vcore
GitHub Login: @gahl-levy
Microsoft Alias: gahllevy

@ymkale
Thanks for your feedback! We will investigate and update as appropriate.

Hi @gahl-levy,
Could you please look into this request, thanks.


[Event Request] Table 7002 - OnValidateSalesCodeOnAfterGetCustomerPriceGroup

Please add a new integration event OnValidateSalesCodeOnAfterGetCustomerPriceGroup in Table 7002.
    { 2   ;   ;Sales Code          ;Code20        ;TableRelation=IF (Sales Type=CONST(Customer Price Group)) "Customer Price Group" ELSE IF (Sales Type=CONST(Customer)) Customer
ELSE IF (Sales Type=CONST(Campaign)) Campaign;
    OnValidate=BEGIN
      IF "Sales Code" <> '' THEN
        CASE "Sales Type" OF
          "Sales Type"::"All Customers":
            ERROR(Text001,FIELDCAPTION("Sales Code"));
          "Sales Type"::"Customer Price Group":
            BEGIN
              CustPriceGr.GET("Sales Code");
+             OnValidateSalesCodeOnAfterGetCustomerPriceGroup(Rec,CustPriceGr);
              "Price Includes VAT" := CustPriceGr."Price Includes VAT";
              "VAT Bus. Posting Gr. (Price)" := CustPriceGr."VAT Bus. Posting Gr. (Price)";
              "Allow Line Disc." := CustPriceGr."Allow Line Disc.";
              "Allow Invoice Disc." := CustPriceGr."Allow Invoice Disc.";
            END;
            [...]

New integration event
[Integration]
LOCAL PROCEDURE OnValidateSalesCodeOnAfterGetCustomerPriceGroup@100000000(VAR SalesPrice@100000000 : Record 7002;CustomerPriceGroup@100000001 : Record 6);
BEGIN
END;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Rename abstract log member in EventsourcedActor

Eventsourced#log name collides with ActorLogging#log.

Done with https://github.com/RBMHTechnology/eventuate/commit/5ee20bf84bc0f33e977fd6ddc43527c456ac35b9


[WIP]: Proposal for a LigandNetwork class in gufe in stead of OpenFE

I think we should like with ligand atom mapping move the ligandNetwork to gufe. (more consistent)

I think this got done elsewhere at some point


Implement holds and freezes in orml-tokens.

The latest Polkadot release (v0.9.42) added two new traits to the Fungibles family: hold and freeze (which has been added in https://github.com/paritytech/substrate/pull/12951).
It would be very helpful if the tokens pallet also implement this additional Fungibles traits. At least for the hold trait it should be possible to use the underlying reserve mechanism.

Regarding holds and freezes: Will anything change for the interface of orml_currencies? Or is it completely abstracted away and we can use reserve_named and unreserve_named and set_lock, extend_lock and remove_lock in the future?

fungibles traits are implemented
Current interface is not impacted.

fungibles traits are implemented

@xlc This issue is specifically about Fungibles::InspectFreeze, Fungibles::InspectHold, etc. (three more, see here).
To my knowledge they are not implemented yet.

I'd like to add that the fn reducible_balance impl of orml-tokens seems to diverge from frame (both for fungible, i.e. pallet-balances as well as fungibles, i.e. pallet-assets:
For orml-tokens, the only case considering subtracting ED from the liquid funds is Preservation::Protect whereas in frame this can also occur for other preservations with Preservation::Preserve being the most protected one:
orml-tokens
https://github.com/open-web3-stack/open-runtime-module-library/blob/83a76c2bf66c0b1236c31077e6fb24bb760a3535/tokens/src/lib.rs#L1809
pallet-balances
https://github.com/paritytech/polkadot-sdk/blob/3ae86ae075ac5eb9b80f89f09bd7f4a63f97c582/substrate/frame/balances/src/impl_fungible.rs#L57-L67
pallet-assets
https://github.com/paritytech/polkadot-sdk/blob/3ae86ae075ac5eb9b80f89f09bd7f4a63f97c582/substrate/frame/assets/src/impl_fungibles.rs#L58-L59


Changes to allow tlspretense to be run during CI

Exit with code 1 if any test is failing, that allows scripts to detect failure easily
Allow it to run on non-tty stdin

Merged and released it as part of 0.7.0 (along with a few maintenance updates for rake/and unit tests)

Great! Thanks!


nginx1.21.4 is not supported

Can the version be supported?

@wuwuwl : can you please provide the name of the module you are using?


Create/re-publish post about data layers

Joel wrote a nice blog post about data layers (I remember his awesome presentation about this in 2021) which we put on Towards Data Science: https://towardsdatascience.com/the-importance-of-layered-thinking-in-data-engineering-a09f685edc71
We have a little about it in the FAQ too, but I'm removing it because, it doesn't work as Kedro documentation. It is useful knowledge for someone in the field but it doesn't tell you about Kedro (so much as advocate for Kedro's opinionated data storage).
I think we should take the blog post we published and maybe repurpose it into a post for the new Kedro blog, then point to that when we need to demonstrate the reason for layering.

Bruce Philp and Guilherme Braccialli are the
brains behind a layered data-engineering convention as a model of managing data.
Refer to the following table below for a high level guide to each layer's purpose

The data layers don’t have to exist locally in the data folder within your project, but we recommend that you structure your S3 buckets or other data stores in a similar way.

Folder in data
Description

Raw
Initial start of the pipeline, containing the sourced data model(s) that should never be changed, it forms your single source of truth to work from. These data models are typically un-typed in most cases e.g. csv, but this will vary from case to case

Intermediate
Optional data model(s), which are introduced to type your :code:raw data model(s), e.g. converting string based values into their current typed representation

Primary
Domain specific data model(s) containing cleansed, transformed and wrangled data from either raw or intermediate, which forms your layer that you input into your feature engineering

Feature
Analytics specific data model(s) containing a set of features defined against the primary data, which are grouped by feature area of analysis and stored against a common dimension

Model input
Analytics specific data model(s) containing all :code:feature data against a common dimension and in the case of live projects against an analytics run date to ensure that you track the historical changes of the features over time

Models
Stored, serialised pre-trained machine learning models

Model output
Analytics specific data model(s) containing the results generated by the model based on the model input data

Reporting
Reporting data model(s) that are used to combine a set of primary, feature, model input and model output data used to drive the dashboard and the views constructed. It encapsulates and removes the need to define any blending or joining of data, improve performance and replacement of presentation layer without having to redefine the data models

With the rise in popularity of the Medallion Architecture (Bronze + Silver + Gold) I'm not very excited about producing content of our own convention. Plus, our layering mixes data stages with model stages.

@datajoely @stichbury what do you think about shelving this one?

Yes - may as well

Closing for now


Bump v1.8.0

Track changes for Docker 1.8.0.
Release scheduled for Tuesday, August 11th.

Are RC builds tagged?  I can't find a tag anywhere.

https://github.com/docker/docker/releases/tag/v1.8.0-rc1 :+1:

We have just pushed 1.8.0-rc2 to https://test.docker.com. You can download it with the following:
RPMs/DEBs: curl -sSL https://test.docker.com | sh
Linux 64bit binary: https://test.docker.com/builds/Linux/x86_64/docker-1.8.0-rc2
Darwin/OSX 64bit client binary: https://test.docker.com/builds/Darwin/x86_64/docker-1.8.0-rc2
Darwin/OSX 32bit client binary: https://test.docker.com/builds/Darwin/i386/docker-1.8.0-rc2
Linux 64bit tgz: https://test.docker.com/builds/Linux/x86_64/docker-1.8.0-rc2.tgz
Windows 64bit client binary: https://test.docker.com/builds/Windows/x86_64/docker-1.8.0-rc2.exe
Windows 32bit client binary: https://test.docker.com/builds/Windows/i386/docker-1.8.0-rc2.exe
Please test and make sure all links are working. You can see a list of the changes in the project changelog:
https://github.com/calavera/docker/blob/8a9d4eaa6e1645c66cd8047e87b8df323a6e7082/CHANGELOG.md

We've been running rc2 on some build nodes for a few days and everything is looking good :+1:

Thanks @burke!

I just tagged RC3 :scream_cat: We've started to use GitHub releases to keep track of changes and links. Feel free to download the binaries from the urls in https://github.com/docker/docker/releases/tag/v1.8.0-rc3.

We've started to use GitHub releases to keep track of changes and links

:+1: :100: thanks! Much easier to point users to the right downloads
@calavera should we add a link to this PR on the releases page?

LGTM

LGTM

Congrats all! And thanks @calavera for being release captain! Have one on me 🍺🍻🍷🍸🍼☕


ReduceSum onnx->torch->linalg fails

Trying to match "(anonymous namespace)::ConvertReductionOp"
** Failure : dim argument must be a constant int list or None
https://github.com/llvm/torch-mlir/blob/3c252cdd44f411ef67e3a759319be53b46396d44/lib/Conversion/TorchToLinalg/Reduction.cpp#L453

Looks like the static shape requirement goes pretty deep through Reduction.cpp and Utils.cpp. Probably not worth it at the moment to try to support dynamic shapes. Will make the shapes static in the test cases.


Global sample ratio overrides route-specific

Hello, guys,
Title says it all pretty much. Not sure if this issue is with the plugin (since the config is calculated by kong), but other plugins seem to prioritize configuration values correctly.
So, having config like this
_format_version: "2.1"
_transform: true

services:
- name: echo
  url: 'https://httpbin.org'
  routes:
  - name: echo
    paths:
    - /echo
    plugins:
    - name: zipkin
      config:
        "http_endpoint": "http://jaeger-collector:9411/api/v2/spans"
        "sample_ratio": 1

plugins:
- name: zipkin
  config:
    "http_endpoint": "http://jaeger-collector:9411/api/v2/spans"
    "sample_ratio": 0

will leave without any traces from requests to  /echo.

Thanks for writing such a detailed issue, we'll try to investigate it as soon as possible.

Hello @burdikov,
I programmed a fix for this problem and just released version 1.4.1 of the plugin with it on it.

Thanks a lot!


Add cli command modal

Summary
Add a modal to show most important cli commands. The commands are grouped into two tabs domain & workflow
Changes

Create a reusable component for copying text button
Fix the styles of page tabs to add the spacing around the whole element including the endEnhancer
Create configs for cli commands
Create modal and button components for showing cli commands

Screenshots

How do we want to make these commands configurable in the future?
If we want to show the commands on a different page, would it be possible to configure these commands to show there too?

1- These commands are already configurable. We have configuration files for tabs and commands that can be replaced.
2- If we are using the same modal/button components we can move them to shared directory.
Otherwise we can create components reading from the same commands configs

These commands are already configurable. We have configuration files for tabs and commands that can be replaced
Sorry, my bad. My question was: how can we prepopulate params in these commands in the future?

We can use string interpolation.
One option is to add placeholder for values that can be replaced with the values in the page context.
This approach can be even extended to show supported values for each flag on hover.
But keeping this new ideas to the improvement cycle after the Parity with v3


Support Linux ARM64

Qt 6.7, currently in beta, will include ARM64 binaries. ARM64 should be added as a target to support small form factor Linux devices.
This should be able to be worked once aqtinstall supports the 6.7 ARM64 binaries.
Primarily, this is expected to involve setting up an additional target in .github/workflows/ci.yml for ARM64 builds, using a cross compiler.

This should be implementable now.
https://github.com/miurahr/aqtinstall/pull/766

Cant wait for this!

Just tried, latest aqtinstall with v6.7.1 still only offers x86_64

I haven't gone through the install yet, but it appears to show up with the latest version (3.1.15):
$ aqt list-qt linux_arm64 desktop
6.7.0 6.7.1

$ aqt list-qt linux desktop
5.9.0 5.9.1 5.9.2 5.9.3 5.9.4 5.9.5 5.9.6 5.9.7 5.9.8 5.9.9
5.10.0 5.10.1
5.11.0 5.11.1 5.11.2 5.11.3
5.12.0 5.12.1 5.12.2 5.12.3 5.12.4 5.12.5 5.12.6 5.12.7 5.12.8 5.12.9 5.12.10 5.12.11 5.12.12
5.13.0 5.13.1 5.13.2
5.14.0 5.14.1 5.14.2
5.15.0 5.15.1 5.15.2
6.0.0 6.0.1 6.0.2 6.0.3 6.0.4
6.1.0 6.1.1 6.1.2 6.1.3
6.2.0 6.2.1 6.2.2 6.2.3 6.2.4
6.3.0 6.3.1 6.3.2
6.4.0 6.4.1 6.4.2 6.4.3
6.5.0 6.5.1 6.5.2 6.5.3
6.6.0 6.6.1 6.6.2 6.6.3
6.7.0 6.7.1

As a side note, I have some minor rendering artifacts with Qt 6.7.x (at least on Windows) that I want to resolve before updating CI. However, the develop branch at least builds with 6.7 (x86_64).  This behavior isn't present in Qt 6.6.2.  I'm concerned I may have to file a bug report with Qt and wait for a later release.  It's most visible in the Debug > ImGui Debug window:
https://github.com/dpaulat/supercell-wx/assets/2886605/63aef7c7-6ba6-4b3f-a073-5d5aa392eab7

Armv8 would be great for this program. I hope you get it figured out on linux soon.

The hard part is finding QT for any raspberry pi 4 distro. If we was looking at QT5 it wouldn't be an issue at all.  Debian experimental has QT6 and its the correct version, but giving me errors on the PI trying to install it. I might play around with the Odroid some tomorrow and see if I can get something out of it.

It'll only help you if you're developing, because the rest of the software still needs built for Arm64, or if you're checking it out for another reason. I do not rely on the OS-provided Qt packages, but rather ship with my own. Qt has supported Arm for awhile now, but only recently has provided binaries themselves. These are the same ones that you can grab from the installer:
https://www.qt.io/download-qt-installer-oss

Qt 6.7.x issues were fixed a week or 2 ago, removing that blocker. To make this happen, one of two things needs to be done:

Add cross compile job to ci.yml (or another .yml)
Wait for Arm64 runner: https://github.blog/2024-06-03-arm64-on-github-actions-powering-faster-more-efficient-build-systems/

I expect a manual compile of arm64 would work at this point if anyone wants to start wringing out issues. I have a Raspberry Pi I am able to test with after working alert enhancements.

I have successfully compiled on a Raspberry Pi 3B and 4B (4GB) on Raspberry Pi OS. The only big issue is that Raspberry Pis only support older versions of OpenGL. I believe hardware wise they only go up to OpenGL 2.1, but software wise they can be made to run OpenGL 3.3, but it is a bit slow and not enabled by default. I had to use MESA_GL_VERSION_OVERRIDE=3.3 environmental variable to get it to work.
It is also complaining about not having audio codecs. I have ffmpeg installed and it can play the ogg file, so I am not sure what codec it is talking about.
For anyone who is wanting to replicate, here are some recommendations.
You need to install Qt for arm64. This can be done using aqt, as explained on the developer setup page. You will also need to modify the location of the Qt install in the setup-*.sh scripts.
During compilation, the Pis are usually limited by their RAM. I found that only running one job at a time was the most stable, but running multiple jobs could work depending how much RAM you have or if you do not mind watching it. There is at least one file which used around 1.3GB of RAM in MapLibre. I would recommend using a Pi with at least 2GB of memory. 1GB mostly works, but you need a lot of swap space  (around 1-2GB), and it is very slow. With an 8GB Pi, you could probably do 3-4 jobs.
I moved my home directory and swap space to an external USB3.0 HDD. SD cards can be quite slow, so this may or may not help with compilation speed.
It took about 2-3 hours to compile supercell-wx on my Raspberry Pi 4 (4GB). (not including conan packages)


Inherit benchmark simulators

Implemented example for using an inherited simulator
Added OOP design to two_moons.py

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 25 lines in your changes missing coverage. Please review.

Project coverage is 57.14%. Comparing base (3855cb2) to head (6033a67).

Files
Patch %
Lines

bayesflow/benchmarks/two_moons.py
0.00%
14 Missing :warning:

bayesflow/benchmarks/benchmark.py
0.00%
10 Missing :warning:

bayesflow/benchmarks/__init__.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                   Coverage Diff                   @@
##           streamlined-backend     #180      +/-   ##
=======================================================
- Coverage                57.34%   57.14%   -0.20%
=======================================================
  Files                      104      104
  Lines                     2070     2077       +7
=======================================================
  Hits                      1187     1187
- Misses                     883      890       +7

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

Looking good!


Change "Turn on Preferences Sync..." item on Manage menu while transitioning from Off to On

After turning on sync, if I revisit the Manage menu (from the gearwheel) there's no indication that something is happening.
If I wait long enough before revisiting that menu it will have changed to "Preferences Sync is On".
Compare this with how the "Check for Updates..." entry behaves during download of an update.
VS Code version: Code - Insiders 1.44.0-insider (d7d1147d6b764a48b789000c03088ee9afabcd8a, 2020-03-31T08:23:56.446Z)
OS version: Windows_NT x64 10.0.18363

Following is shown while sync is being turned on...


Fix: Issue: CarbonLink ignore CARBONLINK_HOSTS = []

when configured with CARBONLINK_HOSTS = []

select_host fails with webapp/graphite/render/hashing.py, line 88, in get_nodes ZeroDivisionError: integer division or modulo by zero
trying to get_connection on empty host list is waste of time (and fails similarly as select_host)

0.9.x has similar patch already applied btw - https://github.com/graphite-project/graphite-web/pull/1270


Improvements for assessment annotation user-friendliness

Is there an existing issue for this?

[ ] I have searched the existing issues

New feature
There are a few aspects of the assessment tool annotation that I think we could make more user-friendly!

User first identifies all columns in their TSV about any assessment, and then matches them to specific assessments from CogAtlas. However, if assessments are not found in CogAtlas, user has to go back up to the initial column categorization and remember & de-select all of the ones that they couldn't annotate

Maybe we could prompt them to remove the un-assigned ones automatically? Or alternatively, make them assign columns to assessment terms right away after designating a column as an assessment column

Being able to search by instrument abbreviations as well as full names
The UI for the assessment-to-column matching is a little "bulky", in that when verifying columns for each assessment, I have to scroll through the entire list including the ones I have annotated as being about other assessments

Maybe we could use a two-column layout for the column names, such that the ones annotated with the currently selected assessment are all grouped together? or otherwise visually group them somehow

Unclear documentation
No response

Hello, I tried to solve the issue.
This is what I did:
Modified the category-toolgroup component to implement the following improvements:

Automatically prompt users to remove unassigned columns
Enable searching by instrument abbreviations
Improve UI layout for assessment-to-column matching

You can review changes in this commit: https://github.com/jjyutm4532/neurobagel-annotation_tool-815/commit/da0c75af2bb8587f5951d964fce5617014e2b669.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


[SuperTextField][Mobile] Expose TextInputAction

In order to modify the virtual keyboard action on Mobile, TextInputAction needs to be exposed and configurable.
At Superlist mobile we have a multiline SuperTextField, but pressing enter on the virtual keyboard should not add a line break but be done instead. This is not possible because TextInputAction currently depends only on maxLines.

Hi @jvelezos can you please add appropriate tests for this behavior?
@angelosilvestre can you do a first pass review of this PR once the tests are added?

Hi @jvelezos can you please add appropriate tests for this behavior?
@angelosilvestre can you do a first pass review of this PR once the tests are added?

Sure, I've added tests 👍

PS - I couldn't find an issue ticket for this. In the future, please be sure to start with an issue ticket so that we can mark the issue as resolved by a given PR. Thanks.


release 3.5.3

Motivation/Description of the PR
Release 3.5.3

Thank you @kobenguyent
Changelog looks wonderful!


Change py-solc to py-solc-x

Fixes #142. This PR is a reloading of #147. Thank you @posix4e for the help!

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


Mole.search_shell_id freeze

Currently the search_shell_id method on Mole looks like this:

def search_shell_id(self, atm_id, l):
return search_shell_id(self, atm_id, l)

Where as it looks like it should point to the function with the same name in the same file.

search_shell_id = search_shell_id

This is how all the other features are implemented for Mole.
I'll open a PR with this fix and attach it .

rereading this now it doesn't look like it's recursive...
Whatever problem my machine was having seems to have stopped now so this is a non-issue.


Simple tutorials....

Feature Request or Enhancement
I'm wondering if there are intro tutorials to someone without C experience. I've created the interface but doing something as easy as changing text for a clock alludes me.I'm coming from a phthon/PGP/Swift/Kotlin......
Additional context
Been trying to do simplle text updates to a default text area E_TEXT_1 and no idea where to start...

@bostonmacosx You could begin by reading the FAQ.pdf available at:
Builder Releases
Then looking over the examples, ex04_bld_ctrls does what you want.
Paul--

There is a good amount of documentation for GUISlice/Builder although it tends to be spread around on various platforms.

For starters, here is a Builder Walkthrough with examples.
The Builder User Guide on this page is quite detailed with several code examples.
There's a GUISlice User Guide on this page.
There are several documentation links on this page. The GUISlice API  documentation is included. It's extremely detailed. There is an online version which might be somewhat more up-to-date, but I tend to prefer the pdf version because it has a much better search capability.

Hope this helps,
Don

thanks for all the info


handle 'once' mode

@iantocristian Do we need to get this one updated? Is it still applicable?

@iantocristian If this is still good go ahead an reopen. Cheers.


Dashboard / API Capabilities

This is more of an epic and has to be stripped down to some working packages.
Once we are going in the direction of #25 (state-management) it might make sense to offer a dashboard that allows some more monitoring capabilities.
Initially, that can be read-only, but we might want to allow adding instant jobs via the dashboard (that triggers an API endpoint we have to build anyway).
I am leaning towards an extension package rather than pushing more and more features into the core library itself.

Yes!
This feature is indeed an epic and should be approached in modular packages. I propose two potential strategies:

Embedded UI for Monitoring:
Introduce an extension that enables developers to add a UI dashboard for monitoring all the jobs within their application. This would be similar to what Hangfire does. Developers can easily integrate this by adding the optional middleware app.UseJobsUI(); to their application pipeline:

using LinkDotNet.NCronJob;

var builder = WebApplication.CreateBuilder(args);

// Add NCronJob to the service container.
builder.Services.AddNCronJob(n => n
    .AddJob<PrintHelloWorldJob>(p =>
        p.WithCronExpression("*/2 * * * *"))
);

var app = builder.Build();

// Integrate NCronJob UI middleware.
app.UseJobsDashboard();
...
app.Run();

Centralized Jobs Control Center:
Develop a separate, centralized control center/hub that aggregates events/streams from all nodes running jobs. This control center would provide a dedicated dashboard for comprehensive monitoring and control. Nodes would communicate with the control center via a configuration setup in IConfiguration. Here’s a preliminary implementation idea from the client/node perspective:

using LinkDotNet.NCronJob;

var builder = WebApplication.CreateBuilder(args);

// Register NCronJob.
builder.Services
    .AddNCronJob(n =>
        n.AddJob<PrintHelloWorldJob>(p =>
            p.WithCronExpression("*/2 * * * *"))
        )
    .TryJoinJobsControlCenter(builder.Configuration);

var app = builder.Build();

app.Run();

The necessary configuration in appsettings.json might look like this:
{
    "Logging": {
        "LogLevel": {
            "Default": "Information",
            "Microsoft.AspNetCore": "Warning"
        }
    },

    "NCronJob": {
        "ControlCenter": {
            "Url": "http://localhost:7177/hub",
            "ApiKey": "1234567890"
        }
    }
}

Obviously the second approach requires more effort, especially in terms of communication strategy and infrastructure. We can implement either option or both options (centralized hub more of a long term goal).

I am leaning towards an extension package rather than pushing more and more features into the core library itself.

Absolutely, we need to start breaking the project into a core and then the separate extension components that rely on core. Keeping the core library lean makes it easier to maintain.

I do like the second approach as it gives a USP for this library.
Let's keep that open and tackle it once we have a definitive idea on how to tackle it and how to tackle #25

https://github.com/NCronJob-Dev/NCronJob/discussions/25#discussioncomment-9813281


[14.0.x] ISPN-15323 Upgrade os-maven-plugin to 1.7.1.Final

https://issues.redhat.com/browse/ISPN-15323

merged! thanks @ryanemerson !


[Proposal] Remove add props

remove additionalProperties from existing artifacts and add pre-commit to prevent this in the future.
please tell me if I'm misunderstanding the need for this as part of: https://github.com/massdriver-cloud/massdriver-cli/issues/64 @coryodaniel

additionalProperties: false prevents users from adding additional properties to our objects. This is a good thing because API consumers could at some point in the future add giant objects of random garbage and wreak havoc on our system. We want to keep additionalProperties: false. Any parameter we take via JSON schema should be accounted for and validated.

so is the idea to only remove these in the publish workflow here: https://github.com/massdriver-cloud/artifact-definitions/blob/main/.github/workflows/publish-stable.yaml#L25 ?

That CLI issue has to be addressed in the CLI code. We can sync tomorrow on this but we don’t want it removed on the definitions.


feat: 🎸 add ability to export all icons in root and folders

all icons can be exported from index.ts file in root, or from
group/index.ts file.
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

I had a case where I needed to export all icons from one file. Ex: In content management system, I want to give user ability to choose any icon.
I have another case where I have entry points in my project and to make icons work, I have to export every icon from public-api.ts file.
Issue Number: N/A
What is the new behavior?
With this change, no index.ts file will be generated in root directory, with every icons exported. Also export statement will be added to group folders index.ts file.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

This is done in v3 - https://github.com/ngneat/svg-icon/commit/0a0f8bfcfe233fb94fb1fd1f9b649420b9074962#diff-142ec6eabfb5776aeca0c476aafb5b36fba7e37efafa13179f5578b469a0ff90R17
Notice that we update svgo to v2.


Archaic Wisp cannot attack

Whenever I me or any of my friends uses legendary mask the wisp spawns and does an attack animation, but not actual attack happens. This could very well be caused by one of the other mods we are playing with and if that is the problem then I will close this issue. I just didn't want to leave it unreported in case this is an actual issue with the mod.

Turns out it's caused by Moisture Upset, which isn't shocking. If support between the two mods ever get implemented that would be nice, but I imagine it's a weird technical issue that's more work than it's worth.


chore: add code owners

Add CODEOWNERS file
https://github.blog/2017-07-06-introducing-code-owners/

It didn't seem to do it's thing


HSEARCH-3755 Configuration property checking displays the wrong name for the property controlling its behavior

https://hibernate.atlassian.net/browse/HSEARCH-3755

Merged thanks


Embeddable can't have embedded relation

Let's say we have a parent entity:
<?php

declare(strict_types=1);

namespace App\Database;

use Cycle\Annotated\Annotation;

/** @Annotation\Entity() */
class Parent
{
    /** @Annotation\Column(type="primary") */
    public $id;
    /** @Annotation\Relation\Embedded(target="Embed") */
    public Embed $embed;
}

An embeddable entity:
<?php

declare(strict_types=1);

namespace App\Database;

use Cycle\Annotated\Annotation;

/** @Annotation\Embeddable() */
class Embed
{
    /** @Annotation\Column(type="text") */
    public $data;
    /** @Annotation\Relation\Embedded(target="Embed2") */
    public Embed2 $embed;
}

And another one embeddable entity:
<?php

declare(strict_types=1);

namespace App\Database;

use Cycle\Annotated\Annotation;

/** @Annotation\Embeddable() */
class Embed2
{
    /** @Annotation\Column(type="text") */
    public $data;
}

When I try to add an embedded object to an embeddable object, I got an error:
[Cycle\Annotated\Exception\AnnotationException]
 Relations are not allowed within embeddable entities in ...

Is this going to be improved sometime? With this constraint, developers can't design entities more than 1 nested level without using hasOne relation or similar

Interesting use-case. This is something I would def review in future releases.

I have the same issue. Currently, doctrine supports this functionality.

We will take a look at this issue after 2.0 milestone.


Update README

Current requirements stated in README is "Unity 2018.4.20f1" which is very outdated.

you could just make a PR tho ^^

Resolved with #56

you could just make a PR tho ^^

I would if I knew how to do that lol

you could just make a PR tho ^^

I would if I knew how to do that lol

I did it allready, but heres the how-to for next time!
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request


Notification for speaking while muted

What is the feature that you would like to see in the app? Please describe.
We all forget to unmute ourselves sometimes. It would be great to receive a toast style notification of when your microphone detects input while on mute.
Additional context
Something similar to this, but with the listener only enabled while on mute: https://www.twilio.com/docs/video/build-js-video-application-recommendations-and-best-practices#testing-the-microphone-and-camera

Hey @bryan-palmer - thanks for the feature request!
We discussed this feature as a team, and we agree that it would be useful to have in this app, so we added it to the roadmap. There's no estimate on when the feature will arrive in the app, but I'll keep you posted!


UsbDkController -n  Command not recognized.

I installed it, but I have no idea where it installed, and why it isn't being recognized?

Hi,
Should be under Program File directory. Search the disk.
If you cannot find it, check that you ran the installer as "Administrator"
Best regards,
Yan.


Splitting @keyframes leads to wrong behaviour

The default behaviour when the splitting point is within an at-rule, as illustrated by the example in the readme, is to duplicate the at-rule. However, when this happens within an @keyframes rule, the result is invalid:
/* input - style.css */
@keyframes myAnimation {
  0% {
    background-color: red;
  }
  100% {
    background-color:transparent
  }
}

The result is:
/* style.css */
@keyframes myAnimation {
  0% {
    background-color: red;
  }
}

/* style-0.css */
@keyframes myAnimation {
  100% {
    background-color:transparent
  }
}

which is no longer valid.
In this case, the whole @keyframes block should be copied to the new file.

@nicolashoby Hi! Thanks for finding this one, I've fixed it in v0.1.2. From now, this at-rule block wouldn't be separated.

Hi Vitaliy
Thanks very much for the quick fix, much appreciated!
Greetings
Nico

Am 05.04.2017 um 13:49 schrieb Vitaliy Ribachenko<EMAIL_ADDRESS>@nicolashoby Hi! Thanks for finding this one, I've fixed it in v0.1.2. From now, this at-rule block wouldn't be separated.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Scale tests time to create addresses

Type of change

Enhancement

Description

Scale test to measure time to create new addresses
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests profile=scale testcase=ScaleTest#testTimeToCreateNewAddresses test_only

:x: Test Summary :x:
TEST_PROFILE: scale
TEST_CASE: ScaleTest#testTimeToCreateNewAddresses
TOTAL: 1
PASS: 0
FAIL: 1
SKIP: 0
BUILD_NUMBER: 94
:heavy_exclamation_mark: Test Failures :heavy_exclamation_mark:

testTimeToCreateNewAddresses in io.enmasse.systemtest.scale.ScaleTest


feat: .2dja support for poster.lua

This pull request adds support for .2dja files, a file format used by the community to simplify the printing of large signs.
While maintaining compatibility, two optional numbers can now be appended to the syntax to indicate the interval of pages to be used if a .2dja file is provided. Or a single specific page.
Additionally, the status text has been updated.
This commit also ensures that the program is now compatible with normal PCs that don't support all colors.
Finally, comments and style changes have been included for improved readability.
I tried to ensure that the program was as robust and secure as possible.

Why does this PR include rewriting variable names to use Hungarian notation? This is not consistent with our other ROM programs. (Also FYI, CC-Tweaked ROM is migrating away from it in general)
Also, for these changes:

Additionally, the status text has been updated.
This commit also ensures that the program is now compatible with normal PCs that don't support all colors.

We should make the same changes in the print3d program for consistency.

Could you maybe explain what exactly you mean with Wrong formatting codes? Is it "only" a reference to wrong comma placing?

I applied all requested changes. However, I left the ones open where I'm not sure if I have fully fulfilled them, which is why I request small feedback please.

Could you maybe explain what exactly you mean with Wrong formatting codes? Is it "only" a reference to wrong comma placing?

string.format uses the same format string rules as printf in C. The short version is that you should use %s for string parameters, %d for integers and %f for decimal values.
It also allows you more control over the way your numbers are presented, as an example using %5.1f would format a number as 5 characters long with one number after the decimal point (3 digits, a dot then another digit makes 5 characters).
The following snippet could show the following percentages as an example
printRow("Ink:", ("%5.f%%   "):format(math.floor(100/max*cur)))

Ink: 100.0%
Ink:  52.5%
Ink:  12.3%
Ink:   0.0%

Look at how nice they align

Ah I see! Thanks for taking the time explaining me this. I will overview this again.


Thread safety

Hi,
I've been experimenting with rustbox and wanted to use it to across threads.
According to this commit it looks like I can: https://github.com/gchp/rustbox/pull/8
But then I get this error when I try to use it in a thread:
the trait `core::marker::Send` is not implemented for the type `*mut ()`

Which I think is coming from here:
https://github.com/gchp/rustbox/blob/7e76a04e7c78a96e00b0435ef5065ce2381c7492/src/rustbox.rs#L221
Which indicated that termbox isn't thread safe.
Can rustbox be used in threads?
If so, any tips on how? I've tried wrapping it in Arc::new(Mutex::new(Rustbox::init()) but without luck.
Cheers

Retouching this one to show I'm still interested. :-)

+1 about this issue.

Sorry for the delay getting back to you, I was on vacation without my laptop :)
The short answer is, I'm not sure. Which I realise is not a very satisfying answer! The whole thread-safety side of things is a bit of a gray area to me at the moment unfortunately. I'll do some digging and see if I can figure it out!

I've experimented using it across threads by disabling the PhantomData (https://github.com/sebglazebrook/rustbox) and while it compiled it did hang during execution.
Haven't had a chance to investigate whether it's my code or something else.
Will let you know if I find anything.

To get back to you about the above. The issue I was seeing was due to my code.
Now that I've fixed that I've found rustbox to be fine across threads aonce wrapped in an Arc and Mutex.
I've created a PR (https://github.com/gchp/rustbox/pull/44) to remove PhantomData unless there is another reason why it should be there.

@gchp I think we can safely close this, as it's addressed by #81.


[SMALLFIX] Fixed documentation of the allocateBlockWithView method in the Allocator interface.

The allocateBlockWithView method in the Allocator interfaces had a reference to wrong signature of the anyDirInTier method of the BlockStoreLocation class.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6840/
Test PASSed.

LGTM, thanks for keeping the javadoc up to date.


Replace lazy statics with consts when supported in std

There are places in the codebase where we use static Lazy variables just because it's not possible to define constants of that type.
For example:
pub static CAN_TRANSFER_USER_ASSETS_TOKEN: Lazy<Name>;

can be replaced with
pub const CAN_TRANSFER_USER_ASSETS_TOKEN: Name;

once Option::expect or Option::unwrap is made a const function. Tracking issue

Duplicate of #1978


ExecutionPolicy and File options are not order agnostic

Powershells -ExecutionPolicy and -File options are order sensitive. Almost all of the command line tools I use do not have functional differences in the order of their command line options. This has lead me to build a mental model of arguments as an unordered dictionary or set of tuples and flags. In fact it is more like an ordered table.
Steps to reproduce
# run with admin privileges
Set-ExecutionPolicy Restricted
# restart the shell, dont need admin
Write-Output "echo 'I'm running!'" | Out-File "./hello.ps1"

powershell.exe -ExecutionPolicy Bypass -File ./hello.ps1 #successfully prints "I'm running!"
powershell.exe -File ./hello.ps1 -ExecutionPolicy Bypass # fails: scripting is disabled by execution policy

Expected
Argument order should not matter. Where it does matter it should be validated. It should be documented as such.

-File and -ExecutionPolicy can be written in any order
OR: calling powershell -File asdf -ExecutionPolicy Bypass results in an error "ExecutionPolicy must come  before the -File option"
OR: powershell.exe -Help should mention that the ExecutionPolicy parameter is order sensitive.

Actual
Argument order does matter, and when that order is not used correctly powershell fails silently

the -File and -ExecutionPolicy arguments are order sensitive.
-ExecutionPolicy appears to simply "not work" in some circumstances, and with the apparent security implications of such a variable I was willing to write it off as windows security black magic when infact PEBCAK

Environment data
PSVersion                      5.1.17134.407
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17134.407
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

I'm loathe to admit that this has contributed to a 2-year quirk in my software. Years ago I tried the -ExecutionPolicy Bypass option, had no impact, so I simply moved past it requiring that our users enabled scripting on their systems. This was a way bigger deal to deployment that I thought it would and contributed to a couple dozen man-hours of support time. More recently I tried -ExecutionPolicy Bypass again and everything worked as its supposed to because, by chance, I wrote it first. Perhaps I'm dumb or impatient, but my mental model of command line as being an "option dictionary" did not lead me to think about argument order.

more specifically, a session:
PS C:\Users\geoff> Get-ExecutionPolicy
Restricted
PS C:\Users\geoff> .\hello.ps1
.\hello.ps1 : File C:\Users\geoff\hello.ps1 cannot be loaded because running scripts is disabled on this system. For
more information, see about_Execution_Policies at https:/go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:1
+ .\hello.ps1
+ ~~~~~~~~~~~
    + CategoryInfo          : SecurityError: (:) [], PSSecurityException
    + FullyQualifiedErrorId : UnauthorizedAccess
PS C:\Users\geoff> powershell.exe -ExecutionPolicy Bypass -File .\hello.ps1
hello!
PS C:\Users\geoff> powershell.exe -File .\hello.ps1 -ExecutionPolicy Bypass
File C:\Users\geoff\hello.ps1 cannot be loaded because running scripts is disabled on this system. For more
information, see about_Execution_Policies at https:/go.microsoft.com/fwlink/?LinkID=135170.
    + CategoryInfo          : SecurityError: (:) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : UnauthorizedAccess

As I recall, both the -File and -Command parameters must always come last in a command line call to powershell.
When using -File, any further arguments after the file name are interpreted as arguments to be passed to the script itself. See here.
When using -Command everything that follows is interpreted as part of the command string when called from the command line. If you're executing an instance of powershell from PowerShell itself, then you can use an actual script block to properly delineate the command you're passing in. See here.
So, yeah, it's not really possible for -File or -Command to be order-agnostic; in general those two need to be the last in any command sequence, although apart from that you can put the other parameters in whichever order you like. 😄


rvc keyerror:config

I trained rvc on the original github with version 1 i think, I cant seem to make it work on tts gen webui, all models i have downloaded work through, its like all keys dont match succesfully or something similar

Does this model work with another RVC client?
I did test v1 and v2 models before and they worked, but it might be that something has broken in the background.


Add FAQ section

Add A FAQ page
link in footer (next to Impressum).
should contain cards per theme / topic. maybe link to training videos.

topics to add:
for Roles:

regionadmin

adding new users (self register, approve, invite)
setup a region (list formats, league phase dates)

leagueadmin:

manage a league
assign a club

clubadmins

register a team
pick a character
download reports


Small vector optimization for operands.

We replace the std::vector in the Operand class by a new class that does
a small size optimization.  This helps improve compile time on Windows.
Tested on three sets of shaders.  Trying various values for the small
vector.  The optimal value for the operand class was 2.  However, for
the Instruction class, using an std::vector was optimal.  Size of "0"
means that an std::vector was used.
            Instruction size
        0      4      8

Operand Size
0               489    544    684
1               593    487
2               469    570
4               473
8               505
This is a single thread run of ~120 shaders.  For the multithreaded run
the results were the similar.  The basline time was ~62sec.  The
optimal configuration was an 2 for the OperandData and an
std::vector for the OperandList with a compile time of ~38sec.  Similar
expiriments were done with other sets of shaders.  The compile time still
improved, but not as much.
Contributes to https://github.com/KhronosGroup/SPIRV-Tools/issues/1609.

Out of curiosity, did you try calling .reserve() on the vector to set the number of operands it would need to store or are they just added to the back?

The unique_ptr to a std::vector seems like a waste (double indirection), checking against empty is just as fast as checking a pointer so you may as well keep it inline if you don't mind the extra space needed. Also std::aligned_storage is a better solution than a char array.
You are likely to get better performance for indexing and push_back by having a iterator begin; as a field and use that for the operator[]. You can even use that field to check whether it is pointing to the small buffer as the test to see whether the large or small buffer is being used.
This lets you recycle the storage space:
T* begin;//always points to &sb or bb.data()
size_t length;
union {
    std::aligned_storage<sizeof(T)*small_size, alignof(T)> sb;
    std::vector<T> bb;
}

This does require more lifetime finagling but moveToBig is then a std::move of sb to a local std::vector, destruct of the sb elements and then a placement move construct of local to bb.

@ratchetfreak I implemented you suggestion (or a least part of it) in https://github.com/s-perron/SPIRV-Tools/tree/small_vector2.  It did not give any improvments.  In general I agree it would be better to reuse the same data; however, in this case I do not believe the amount of data being allocated is the bottleneck.  Even large shaders are not that large.  We just want to avoid a lot of calls to malloc and free.
For now, I will go with the simpler implementation.  I do not like unions all that much because the make it harder for the compiler to optimize, and easier for the developer to make a mistake.
I also tried Abseil, and it does give a small improvement over my implementation, but it is not very much.  So there is room for improvement.  We can also look into this a little more later, but I want to get the majority of the improvement first, then we can work to get the rest.

Overall the approach seems fine and the code looks good.
However, bots fail compilation:
/home/travis/build/KhronosGroup/SPIRV-Tools/source/util/small_vector.h:336:12: error: variable has incomplete type 'void'
auto ret = large_data_->insert(new_pos, first, last);

At one point I wondered if the creation of operand lists is sufficiently constrained that in the vast majority of cases we have all the operand values in hand at one time, so we can allocate their storage in bulk.  But that's a more intrusive change to this one.


Typo in Documentation addResponse_

WebhookClient.addResponse_(response) is not a function.
I can use add, but not addResponse_ .
What is the correct function name for sending only a single response?

docs/WebClient.md

add is the correct function to use to add a single response and addResponse_ is a private function for the library.

Maybe the method should be removed or marked as private.


yotta run <programname> subcommand

Should run the program either natively, or after loading it onto the target. This will need support defined by target descriptions.
Related: the yotta debug subcommand should accept program names without the full path, and find them for you.

+1 on this. For executable modules,  could be optional, defaulting to the main program.
Icing on the cake would be an option to also start a serial terminal connects to the device, maybe using pySerial (I think mbed 2's make.py had an option for that).

Added by #741 (released in yotta 0.15.0)


Enhance error message of failed decoders/encoders

Version: (e.g. 1.4.0)
#869, #773 - Implicits with the same name could shadow each other.
We need to either add 3rd condition in #840, or actually print the actual error message which caused encoders/decoders failure (if possible)
@getquill/maintainers

@mentegy We should clearly state in this error message that there can be conflicts by type and by name, for example:
1. ensure that implicit `Decoder[java.time.LocalDate]` is provided and there are no other conflicting implicits with the same type or the same name.

I also meet this error when using quill-async-mysql 2.3.1


[GPOs] The end user docs have a list of GPOs that is out of date

Provide a description of requested docs changes
We should link to the documentation on GitHub instead of maintaining two lists:

https://learn.microsoft.com/en-us/windows/powertoys/grouppolicy
https://github.com/microsoft/PowerToys/blob/main/doc/gpo/README.md

@crutkas
Can you please create an aka link?

actually, i think we should do the opposite.  Move the GPO docs to docs

actually, i think we should do the opposite.  Move the GPO docs to docs

Sounds good too. Especially as the one on docs.msft have wording mistakes. (I have the plan to rework things after my vacation.)

@crutkas
Please don't wait on # because I can't work on the other issue the next three weeks.


fix issue #559: to json -r serializes datetime without spaces

Prior to this fix we were removing whitespace from inside the keys and values
as well as outside the keys and values....
Now we NO longer remove whitespace from inside the keys and values,
but instead are leaving them intact....

This should fix #559

nice