Update Vitest to v1

Description

Updates Starlight’s test dependencies to use the latest version of Vitest, updating from a pre-release to v1.

A couple of test snapshots changed to a nicer string escaping algorithm — I inspected them visually and I don’t think anything important changed, mostly just stuff like:
- "<el attr=\\"val\\">"
+ `<el attr="val">`

Updated Vitest coverage thresholds config to match new format as changed in https://github.com/vitest-dev/vitest/pull/4442

size-limit report 📦

Path
Size

/index.html
5.22 KB (0%)

/_astro/*.js
21.54 KB (0%)

/_astro/*.css
13.14 KB (0%)

Locally (and CI too actually), I get a few [ERROR] [vite] WebSocket server error: Port is already in use errors but I don't think they're relevant to us.

Yes, I noticed these too. I think it’s something to do with things happening in parallel? @hippotastic reported the same errors in the parallel builds of the example projects and added a --workspace-concurrency 1 option to that package script as part of #1395. I can try that for the tests too, to see if it helps.

Yes, I noticed these too. I think it’s something to do with things happening in parallel? @hippotastic reported the same errors in the parallel builds of the example projects and added a --workspace-concurrency 1 option to that package script as part of #1395. I can try that for the tests too, to see if it helps.

Interesting, to add more details, I got them when running only the starlight package tests from its directory so not sure this would help (as I think it's a pnpm workspace flag).

Ah, good point — I think I saw that too. We do have multiple Vitest workspaces — perhaps would need those to be run sequentially.

I wonder if it could be https://github.com/vitejs/vite/issues/14328 and we should just ignore in this specific context of testing.

I agree, and it looks like Vitest is supposed to have this workaround but it's not working for some reasons.

Oh yeah… Well, overriding a global like that is always a bit flaky if some other code somewhere also does something to it, so I guess not totally shocking it’s not working. Still, think I’m happy to merge this for now.


[Version] Remove all Legacy 7.x Compatibility in 3.0

This is a meta issue to track all tasks for removing legacy compatibility in 3.0

[ ] #2746
[x] #2747

@nknize do we need to provide a deprecation message in the next minor release? or this should be the right expectation and no need for a notice.

Note - this task is now OBE/obsolete since we need to retain legacy compatibility for opensearch-migrations


Enhancement: Change VR/XR Player Settings based on Distribution

Along the same lines as Issue #6 and #27, it would be great if you could also set VR/XR Player settings according to the existing Filter system.
For example, we could re-arrange the preferred SDK order based on the distribution platform, or even leave  them all out except for "None" for a non-VR build.

@robinnorth This is perfect, thank you!


Paul atlanta

Added more useful info

Very interesting

Thanks for the review :-)

Any further changes?

Not today

🥇


TableFormat 03 Datastore Information - Colors not displaying properly

We've recently upgraded to the newest release of vCheck from the 6.20 version that we were using previously and noticed that the TableFormat no longer seems to work properly for 'critical' items. Here is the example that I have found: We have a datastore that matches the 'critical' criteria therefore it also matches that 'warning' criteria and it is being colored as a 'warning'. My guess was that once it determined it met the 'warning' criteria it applied those settings and moved on.
I've tested moving the 'critical' criteria above the 'warning' criteria and running the script again and indeed it does now display the item as red.
Just wanted to put this out there to see if others were having a similar issue and see if there is an appropriate fix.

This is an interesting one actually. It turns out that adding a class attribute changes the reference to and XmlElement instead of a string. I'll push an update to the dev branch soon.

Upon testing, this fix didnt seem to work in my case. I previously fixed the issue by flipping the critical and warning pieces in the Datastore Information plugin formatting. Would this affect the fix implemented here?

Yeah, it will apply the rules in order, with the last matching rule being the resukt. So since you flipped them it won't work for you. Flip them back and it should be OK (worked for my testing)


Clear does not reset to minimum assigned value, it resets to 0.

When "clearing" a selected value, the star system does not reset to the minimum value assigned. The clear button should reset the value to the minimum value designated by the user.
i.e. - If you have a rating system of 1 star - 5 stars. Select 3 stars, then clear the selection. The stars reset to 0, but the minimum value allowed is 1.

Thanks for reporting the bug, it's fixed on version 0.3.1


Error Reporting sample is not working.

I realized today that the following Error Reporting sample:
$msg = array(
        'message' => $e->getMessage(),
        'serviceContext' => array('service' => 'myapp'),
        // ... add more metadata
    );

Does not produce log messages that are processed by Stackdriver Error Reporting.
While waiting for the Stackdriver error parser to be updated, we must update our code sample to log errors in the appropriate format:
This code produces a payload that is correctly processed, I tested it and another person confirmed:
$msg = array(
        'message' => sprintf("PHP Notice: %s", (string)$e),
        'serviceContext' => array('service' => 'myapp'),
        // ... add more metadata
    );

I wanted to submit a PR, but realize that there are tests associated. Please update the sample as soon as possible to fix the Stackdriver Error Reporting compute engine instructions: https://cloud.google.com/error-reporting/docs/setup/compute-engine

@steren Thanks for the report, I have few questions.
What happens with the first code? Is the log lost? Is it stored, but in the wrong format?

appropriate format

Where is it defined? The sample you showed seems weird, because it always says PHP Notice.
Why is the string "PHP Notice: " required?
If you have a plan for fixing this, can you share the details of your plan? ETA, how will you fix, etc?
Thanks!

This sample is used for Error Reporting documentation, not for Logging.
In the first code, the error is just stored in Logs, but not processed by Error Reporting because it is not in the expected format.
While waiting for Error Reporting to be more flexible in its error parsing (no ETA yet), we should adapt our samples to what Error Reporting is able to parse today. "PHP Notice" is required to fit the current parser.
The exception format is not documented, the idea is that Error Reporting is supposed to process default exception format.
Keeping a broken sample doesn't make sense and is very misleading. We will update it again later once the parser is updated.

@steren
Then how about turn down the sample? Isn't it also confusing we keep changing our samples?

GCE support is in Beta, it is OK to change our sample over time.
The fix I suggest has unblocked a customer who used PHP. I do not understand this push back and why we would not document how to use Error Reporting with PHP, knowing we have a workaround to the parser limitation

The reason I'm hesitant is because the string "PHP Notice:" looks really weird.
@bshaffer WDYT?

I agree it looks weird, we can add a comment explaining that it is due to a limitation of the Error Reporting service and will change in the future.

Yeah, this is a pretty unfortunate solution. What if Exception is a fatal error? We are logging it as a Notice, which is considered a lower level than Warning.
If anything, these errors should be logged as Emergency, as an exception handler indicates a 500 error. @steren Is there another option here, or is Notice the only way?

Here are the current prefixes that will be identified by the parser: PHP (Notice|Parse error|Fatal error|Warning):.  We can pick any of this list at the moment, maybe Warning is better suited?
But note that the log entry severity is decorrelated from this. The severity can be set with the severity attribute.
So I suggest to update to:
$msg = array(
        'message' => sprintf("PHP Warning: %s", (string)$e),
        'serviceContext' => array('service' => 'myapp'),
        'severity'=>'ERROR'
        // ... add more metadata
    );

If everything goes well, the parser fix should be deployed by the end of the year or early next year. But in the meantime, I still think we should fix our broken sample.

@steren
I don't disagree that we should fix broken samples, but at the same time, we have finite resource. Considering the timing, the fix will be very short lived. We don't want to spend our time to write soon-to-be-thrown-away and looking-very-weird samples. So I suggested that we just turn down the sample and revive it when it just works.
However, what's the plan for the fix? Can we just pass the $e-getMessage() then will it work? Or will we still need to add some prefix?
That's why I wanted to know the future plans, specs etc.
The new sample you suggested look much nicer, so I'm not worried about the weirdness any more. We will fix it if time allows.

After the fix, you will need to pass 'message' => (string)$e,.
$e->getMessage() does not print a stack trace (and a stack trace is required for error grouping).

I think users should add severity if it matters, right?

Yes, adding a severity in the payload probably is better for users. But is not a hard requirement for errors to be processed.
So after the parser fix, the sample should be:
$msg = array(
        'message' => (string)$e,
        'serviceContext' => array('service' => 'myapp'),
        'severity'=>'ERROR'
        // ... add more metadata
    );

(ಠ_ಠ)
 self::fluentLogger()->post('errors', [
   'serviceContext' => ['service' => 'balance-checker'],
-  'message' => $e->getMessage() . PHP_EOL . "Stack trace:" . PHP_EOL . $e->getTraceAsString(),
+  'message' => str_replace('warning', 'Warning', $e->getMessage()) . PHP_EOL . "Stack trace:" . PHP_EOL . $e->getTraceAsString(),
 ]);

After this patch it finally worked.

Thanks. The second sample I provided in my original report should work too since the parser still expects this prefix: PHP (Notice|Parse error|Fatal error|Warning)


Build Error (OneShot)

This is a new one to me, and I don't even know if this is the right repo to report this issue on?
 Build Error (OneShot)

components/collapsible/index.ts: /home/runner/work/emberclear/emberclear/client/web/emberclear/components/collapsible/index.ts: unable to use this non-serializable babel config in this node process
        in /tmp/broccoli-3020EI1K66U07kNx/out-02-funnel
        at broccoli-persistent-filter:Babel > [Babel: @emberclear/ui] (Babel: @emberclear/ui)

from ci log: https://github.com/NullVoxPopuli/emberclear/pull/922/checks?check_run_id=746347841
on this branch: https://github.com/NullVoxPopuli/emberclear/pull/922
(using v4-alpha ember-cli-typescript and embroider 0.19.0)
maybe cc @chriskrycho and @ef4?
maaaybe related? https://github.com/emberjs/ember-test-waiters/issues/158

This looks like an embroider issue. Not related to broccoli-persistent-filter. This issue can be closed.
Most likely you can work around the problem by setting the environment variable JOBS=1. But please file an embroider bug.


fix togglePromptRemove

#2699 - this issue is caused by some changes to promptRemove that were made and then reverted in workload sidecars...I think this actually was fixed in the sidecar PR, but I must have clobbered the fix while correcting merge conflicts or something.

Think the root cause is that I pushed commits directly to your pr's branch. I'm not sure this is an ok process, will bring it up with the team and find consensus.

I think this would have been completely fine if I had exercised a greater degree of care in pulling your changes.

There was only one thing that's left to do, (action-menu.js onConfirm in state needs removing), once that's done this

Done


Port: Revert to SQLite 3.12.3

@bricelam @divega

Hi @smitpatel, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Handling of style-oriented ascii espace sequences.

Motivation
I wanted my gooey-powered programs to be able to display rich text (bold, colors, etc.) the same way that they do when running in a normal terminal emulator.
Description of the change
Using the RichTextCtrl widget we are able to use advanced formatting in the console panel. The formatting information is extracted from the text stream by searching for terminal escape sequences. The sequence values are taken from the colored package, which becomes a lazy dependency (imported only when the feature is activated).
The previous implementation (using the TextCtrl widget) has been extracted into a separate class.
The client code chooses between the basic or the rich class based on the richtext_controls configuration parameter. The default is to use the previous implementation based on TextCtrl.
Note to reviewer
This change is much bigger that any other that I submitted to the Gooey repo. It may conflict with your vision of the project or some project guideline I wasn't aware of. If it does, I will be happy to have your feedback and improve this PR.

👍


Feature: Address collection (billing/shipping) + tests

This should fix the last thing in #4 as it is the last thing that needed to be tested. If there are any other parts of redirectToCheckout that need to be tested let me know 👍

@ChrisBrownie55 Wow! I was just going to start on this today!
I literally just merged a PR from @doingandlearning before you submitted this, can you resolve the conflicts real quick?
I want to try to fix all-contributors to make sure you're getting credit haha

Oh of course lol, I'll resolve the conflicts in a minute. Just gotta get some food first

Rebased it 👍  all tests are passing

Yes!! Added you to contributors too haha


Compilation under Xcode 13

I'm getting loads of warnings when trying to compile this in the latest Xcode 13 RC (with Swift5) - mostly all errors are related to this sort of thing : "Argument type 'CBService?' does not conform to expected type 'CBUUIDConvertible'"
Any thoughts or suggestions? I'm having trouble working it out
EDIT: maybe helpful : If I open the same project into my saved copy of 12.5.1, it compiles the first try

Hello, I did a somewhat dirty fix (explicit unwrappings...) but I think it shouldn't cause any issue.
If you want, try with this branch https://github.com/jordanebelanger/SwiftyBluetooth/tree/swift5.5 until I finish merging the changes, need to make them backward compatible with Swift older than 5.5, that will take some time.

Thanks! The 5.5 branch will help in the mean time, at least it compiles again ;)

Thanks @jordanebelanger 👏 🍻


Re-enable iOS on CI

This started failing with a recent xcode update from Travis and had to be disabled in https://github.com/rust-lang/libc/pull/1051 due to errors like this -- https://travis-ci.org/rust-lang/libc/jobs/410547612
Looks like the simulated program is no longer printing out anything :(

We are seeing these same errors in mach and packed_simd https://github.com/fitzgen/mach/issues/39, https://github.com/rust-lang-nursery/packed_simd/issues/26
cc @kali

With xcode10, https://github.com/rust-lang/rust/issues/55477, x86_64-apple-ios and i386-apple-ios actually fail to link, so libc does not compile anymore for these targets apparently.


Added reference to CLI commands

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

@bwren Could you review this pull request from June?
Thank you!


Add big-clipboard extension

Description
Big Clipboard shows the current text in your clipboard, large in the Raycast window and indexed. Inspired by the 'Large Type' option in 1Password and Alfred.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I'm not sure if this is worthwhile being a standalone extension, or if I should add the features to the existing extension Large type as a separate command

I believe it makes sense to add it to the existing extension as a separate command, yes :)


Leetcode questions

I want to add leetcode questions

This repository is excluded from hacktoberfest if want to contribute you can contribute in https://github.com/Mridul-1-Sharma/data_structures_algos


Time picker behaves unintuitively

I understand that the time picker uses modulo logic when parsing values larger than 12/24/60 respecitvely. However this, combined with the fact that you cannot really delete the current time in the input field makes the usage very unintuitive for users who don't know how the time picker works.
A few examples, all assuming AM/PM format (all of these can be easily tried on the examples page):

User types 31 into the minutes field, tries to change it to 32. After hitting backspace the input changes to 03. While hitting 2 will result in the correct value, the intermediate 03 is pretty unexpected
User types 01 (or any other single-digit hour) into the hours field then tries to correct it. Once he hits backspace the input changes to 12
User types 50 into the minutes field, tries to input a new time (let's say 10): after pressing 1 the input changes to (501 % 60) = 21, after pressing 0 it changes to (210 % 60) = 30.
When typing 17 into the hours field the actual value will be 05 but it will not change the AM/PM setting

Expected (or at least more intuitive) behavior:

Don't pad the values to two digits. You can still interpret the value the same way, but this would avoid the issue described in the first example. If the picker closes without a valid time it defaults to some sane default value (e.g. defaultHour/defaultMinute or the time portion of minDate/maxDate, whichever is applicable)
Interpret user input in chunks of two. Given the third example, the 1 in the 10 would basically clear the input field and set the minutes to 1. This would result in a much less surprising behavior than the current one
Finally, only use the modulo logic when the input is actually 2 digits long, or the time picker closes with an invalid value. This means we can actually correct single digit hours using the AM/PM format.
Inputting a 24 hour value into the 12 hour input should set the AM/PM setting correctly (this is the tweak I care the least about, it probably won't matter to 90% of users)

Hello,
Thank you for the constructive feedback and bringing light to the issue.
It is my goal of making the calendar as easy to use as possible, down to every detail.
I'll work on incorporating some of your ideas into the next release.
Stay tuned :+1:

+1
The behaviour of inputting time is very strange.

+1

+1

I agree with @Gustorn and want to add that being able to add e.g. "h" and "m" for hours and mintues respectively to the fields would help the UX even more.

Is this fixed? When we can expect this release? Thanks


Update DID module with query params

Description
Add query params to the grpc_query functions to allow the resolution of fields in the DID document

https://github.com/allinbits/cosmos-cash/blob/main/proto/identifier/query.proto#L17-L22

Query params needed
TBD

superseeded by #156


Not Loading On Chrome OS

Whenever I Run It It Always Stays On The Screen "Connecting To eternalmc.xyz"  I Am Running It on ChromeOS Version 76.0.3809.136

why are u on chrome os 76 update to 120 and that's why I think

This is just PoC, we haven't tested it in all browsers, it should work on some normal windows or sth ;)


createNumberMask caret positioning bug

The bug is reproducible on the website: https://text-mask.github.io/text-mask/
If user selects "US dollar amount" then puts the input field in the following state...
$12,3|45

...then deletes the 3, the correct state should be...
$1,2|45

...instead, it goes to this state...
$1|,245

Any news on fixing this issue? I have experienced that today, it makes the behaviour strange when deleting chars before the thousands separator.

I'm afraid there's no progress on this yet.

+1

I'm working on this bug now since it affects me. I've found out that the problem appears here:
https://github.com/text-mask/text-mask/blob/master/core/src/adjustCaretPosition.js#L72
but the problem is that in this case, using normalizedConformedValue yields the right result because the number separator (,) has been moved.
I'd appreciate if someone has time to help out by explaining the rationale here (and also assign me to this bug).

What I can see is that targetChar is incorrectly set to ,. It should be 2.

The current recovery strategy is assuming that it can move the cursor to the left to look for the first available placeholder, but this is wrong if the mask is adaptive in the sense that the mask characters change position if the string changes length.
My best take on this is that we have to provide some flag to adjustCaretPosition that makes it search differently. It is an ugly solution because the flag has to be provided by the developer who writes the mask. Also, I am not sure it'll cover all cases. I don't think that there is a generic algorithm for adjustCaretPosition that will always work.

It seems that I've found a way to fix the bug without having to expose any external parameters. Please take a look at the PR above - extra unit test is included.


[manager/allocator] Logging for allocation workflows

In the allocator rewrite, everything will be returned as errors from the lower level, and logged at the very top level.


Android example without root user

Briefly summarize your issue:
https://developer.amazon.com/en-US/docs/alexa/avs-device-sdk/android.html
I am following this example here. But I can not run bash startsample.sh as I do not have root
What is the expected behavior?
Successfully run the sample app using  bash startsample.sh
What behavior are you observing?
I am unable to set executable permissions on anything in the /sdcard directory.  i.e. chmod +x
This is preventing me from running this app.
Also /sdcard is the only directory I can write to using the adb shell.
When trying to connect adb as root I get the following error message:
adbd cannot run as root in production builds
Tell us about your environment:
Android version 9
Suggestion
Is there a way to make just a normal APK of the sample app? So I can easily run it?
This won't require adb root
Please help
Many thanks :)

Please get back to me when you can.
My company are rolling out our smart devices to multiple care homes soon. We are deciding between: Alexa, Google and Mozilla's DeepSpeech.
Many thanks

Any update on this please?
Thanks

Hi,
We would like to further continue this conversation on another channel. Do you currently have an Amazon business representative you can contact? If not, we will follow up with how you can best reach out to us.

@dewijones92,
Have you been able to make contact through an Amazon business representative?
Best regards,
-SWH

@scotthea-amazon
Hi can you please give me your email address?
I have a query about integrating with amazon custom skills.
Here is my email address
<EMAIL_ADDRESS>Thanks

Hi there, can you please email our team at<EMAIL_ADDRESS>?
Thanks. I'll close this issue and we can continue the conversation over email.


chore(sdk): Protocol-ize Import API

Fixes

Part 4 of a series of PRs for landing W&B -> W&B Import API

https://github.com/wandb/wandb/pull/5780
https://github.com/wandb/wandb/pull/5781
https://github.com/wandb/wandb/pull/5782
https://github.com/wandb/wandb/pull/5783
https://github.com/wandb/wandb/pull/5785
https://github.com/wandb/wandb/pull/5777

Description
What does the PR do?

🤖 Generated by Copilot at 9e3e6af
Refactored the code for importing runs and reports from other sources into wandb, using new abstract protocols and helper classes. Improved the mlflow importer module to use the new code and to handle different mlflow versions better.
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at 9e3e6af

Sing, O Muse, of the skillful refactorings
That the wandb developers wrought with code,
Shaping new protocols and base classes
For importing runs and reports of old.

/fix-title

/fix-title


Make Hours and Rooms scrollable

Idea Description
It would be nice if Hours composable and Rooms composable were scrollable.

Hours composable should be able to scroll vertically.
Rooms composable should be able to scroll horizontally.

Reference images and links

I think It's a great idea!👍


google-cloud-sdk.withExtraComponents: fails due to missing libcrypt

Describe the bug
google-cloud-sdk.withExtraComponents fails even with an empty array passed as an argument.
Steps To Reproduce
$ cd $NIXPKGS_CHECKOUT
$ nix build --impure --expr "with builtins.getFlake \"$PWD\"; legacyPackages.\${builtins.currentSystem}.google-cloud-sdk.withExtraComponents []"

Expected behavior
That it would build successfully.
Screenshots
$ nix build --impure --expr 'with builtins.getFlake "/home/john/code/repos/mine/JohnOS"; (builtins.elemAt nixosConfigurations.framework-laptop.config.home-manager.users.john.home.packages 30)'
error: builder for '/nix/store/q00anv3nnrdvs3fbvd199nppsfvn5p9w-bundled-python3-unix-linux-x86_64-3.9.12.drv' failed with exit code 1;
       last 10 log lines:
       > auto-patchelf: 4 dependencies could not be satisfied
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/is05j1lzydh3i0i32c838hnq4vvvr8f4-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/bin/python3
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/is05j1lzydh3i0i32c838hnq4vvvr8f4-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/bin/python3.9
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/is05j1lzydh3i0i32c838hnq4vvvr8f4-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/lib/libpython3.9.so.1.0
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/is05j1lzydh3i0i32c838hnq4vvvr8f4-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/lib/libpython3.9.so
       > auto-patchelf failed to find all the required dependencies.
       > Add the missing dependencies to --libs or use `--ignore-missing="foo.so.1 bar.so etc.so"`.
       > /nix/store/gfjpk3574r2vzi04r25122mzyrb67h3j-stdenv-linux/setup: line 79: pop_var_context: head of shell_variables not a function context
       > /nix/store/gfjpk3574r2vzi04r25122mzyrb67h3j-stdenv-linux/setup: line 1457: pop_var_context: head of shell_variables not a function context
       > /nix/store/gfjpk3574r2vzi04r25122mzyrb67h3j-stdenv-linux/setup: line 1594: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/q00anv3nnrdvs3fbvd199nppsfvn5p9w-bundled-python3-unix-linux-x86_64-3.9.12.drv'.
error (ignored): error: cannot unlink '/tmp/nix-build-rustc-bootstrap-1.66.1.drv-5/rust-1.66.1-x86_64-unknown-linux-gnu/rust-docs/share/doc/rust/html/core/arch/x86_64': Directory not empty
error: 1 dependencies of derivation '/nix/store/a5491sbd2lvil9frmqdlgc27lrly8f33-google-cloud-sdk-408.0.1.drv' failed to build

Additional contextf
I've bisected the problem commit down to 578fb7fd1ff1bee335fbcfe6df745c6cab914e6d. But, I don't know why this would require a new dependency in google-cloud-sdk (and only when it's built withExtraComponents). But, I do have an open PR which solves this problem by adding libxcrypt to buildInputs of the derivation. Any commentary on why this is happening is more than welcome.
My best guess is that it's because libxcrypt now only exposes libcrypt.so.2 (avoiding backward compatibility with glibc). libxcrypt-legacy has been added in the interim which exposes libcrypt.so.1. But, I'm still not sure why this only applies to the gcloud components.
Notify maintainers

@iAmMrinal0 @pradyuman @stephenmw @zimbatm
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.2.6-JohnOS, NixOS, 23.05 (Stoat), 23.05.20230316.ae65ff2`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.0pre20230315_eb56cb7`
 - nixpkgs: `/nix/store/4miq0lkk9yfgdn2cb27wy4y44vc9kxl0-source`

This was just merged which should fix the issue: https://github.com/NixOS/nixpkgs/pull/223615

@jboyens That's so annoying. I had opened this 7 hours prior to that: https://github.com/NixOS/nixpkgs/pull/223806

This can't be prior to https://github.com/NixOS/nixpkgs/pull/223615, you can see it numerically.

True... I must have been comparing his merge time with my opening time.

If a PR has exactly the same patch as yours, and it does appear later, you can just close it and leave a link to your PR.

It requires manual tracking? I wish GH would flag you on opening a PR if you're changing lines on the same base branch as other open PRs.

Anyway, the discussion is off-topic and deserves another issue or even an RFC.

Meh, sounds like the process is manual, for now.

I'm still seeing this error. Any idea what I should do?
Package I'm trying to install:
packages = [
    pkgs.git
    pkgs.jq
    pkgs.yq
    (pkgs.google-cloud-sdk.withExtraComponents [pkgs.google-cloud-sdk.components.gke-gcloud-auth-plugin])
    ...

Error log:
$ direnv reload
direnv: loading ~/strike/zap-infrastructure/.envrc
direnv: loading https://raw.githubusercontent.com/cachix/devenv/d1f7b48e35e6dee421cfd0f51481d17f77586997/direnvrc (sha256-YBzqskFZxmNb3kYVoKD9ZixoPXJh1C9ZvTLGFRkauZ0=)
direnv: using devenv
direnv: .envrc changed, reloading
Building shell ...
error:
       error: builder for '/nix/store/59kj94x8mw61h22797q5iyjbd9vw3cms-bundled-python3-unix-linux-x86_64-3.9.12.drv' failed with exit code 1;
       last 10 log lines:
       > auto-patchelf: 4 dependencies could not be satisfied
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/bq6a2rvpqw2jh40ryjvghi64rzl0mjr5-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/lib/libpython3.9.so
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/bq6a2rvpqw2jh40ryjvghi64rzl0mjr5-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/lib/libpython3.9.so.1.0
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/bq6a2rvpqw2jh40ryjvghi64rzl0mjr5-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/bin/python3
       > error: auto-patchelf could not satisfy dependency libcrypt.so.1 wanted by /nix/store/bq6a2rvpqw2jh40ryjvghi64rzl0mjr5-bundled-python3-unix-linux-x86_64-3.9.12/google-cloud-sdk/platform/bundledpythonunix/bin/python3.9
       > auto-patchelf failed to find all the required dependencies.
       > Add the missing dependencies to --libs or use `--ignore-missing="foo.so.1 bar.so etc.so"`.
       > /nix/store/sw36plhp82916wwg6i6097rkzza7d950-stdenv-linux/setup: line 79: pop_var_context: head of shell_variables not a function context
       > /nix/store/sw36plhp82916wwg6i6097rkzza7d950-stdenv-linux/setup: line 1457: pop_var_context: head of shell_variables not a function context
       > /nix/store/sw36plhp82916wwg6i6097rkzza7d950-stdenv-linux/setup: line 1594: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/59kj94x8mw61h22797q5iyjbd9vw3cms-bundled-python3-unix-linux-x86_64-3.9.12.drv'.
error:
       error: 1 dependencies of derivation '/nix/store/ziwjz69s3bi4kny1x8rw883ij8h78xp1-google-cloud-sdk-408.0.1.drv' failed to build
error:
       error: 1 dependencies of derivation '/nix/store/svjcijkcwhvwskllfzjpwrhk3q3favhi-devenv-profile.drv' failed to build
error:
       error: 1 dependencies of derivation '/nix/store/z1cd93r1pry987b3fd12mihhan4dmy7z-devenv-shell-env.drv' failed to build
direnv: loading the environment failed

$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux <IP_ADDRESS>-microsoft-standard-WSL2, Ubuntu, 22.04.2 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.15.0`
 - nixpkgs: `/home/kevin/.nix-defexpr/channels/nixpkgs`

Which commit is your nixpkgs channel pinned to?


Implement the LT pipeline in XGA tools

I've created this playing around in notebooks, but need to implement it properly now.
The user should be able to decide the starting radius, pass in the scaling relation to derive $R_{500}$, starting temperature, etc.

User should also be able to decide on which overdensity radius to measure. Will develop just for $R_{500}$ initially, simply because I want to make things are working.

I think I will also implement another skew of this pipeline where it sets up array jobs for workload managers for HPCs like Slurm and SGE. As some clusters might take much longer to generate/fit spectra than others (maybe they have more observations or the fits don't converge as well), we might end up having the sample repeatedly waiting for one to finish when others could be going onto the next stage.
As this is a pipeline, and there isn't much to gain by running it interactively, a function that sets up a run where all the sources are run individually would be a good idea (no sample involved, just individual sources).

Ran a test on the SDSSRM-XCS Tx_err < 25% sample (150 clusters), comparing the XGA LT pipeline R500 measurements to the original XCS3P LT pipeline R500 values:

Looks pretty great given the differing approaches to the starting region (I use a fixed aperture, XCS3P uses a circle based on the semi-major axis of the detection ellipse).
9 of the 150 clusters didn't get a result, but I suspect that is because I didn't use adjusted regions for this run, and I know there are some that need it in this sample.

Now need to tidy up the function (add some more user-configurable options etc), add some documentation, and make sure that it measures temperatures and luminosities for the final, accepted, radii.
Maybe should let the user choose between returning a sample object, or just a pandas dataframe (of writing it to disk even).

I am going to try and determine the particular overdensity from the passed scaling relation y-axis name. That way I will put the values in the correct argument on sample declaration - it might matter slightly as it will alter the behaviour of sources in the middle of clusters being removed.


docs: create LICENSE

Description
Add MIT License

:tada: This issue has been resolved in version 0.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add range to willHandleNewline hook

I find out I have to access current section context to determine to disable newline or not. Seems add the editor and range into current hook callback is the easiest way.
The reason we also need editor, is to allow user to modify post in willHandleNewline callback.

@eguitarz Sorry for the delay here. Is there a reason you can't use the existing editor and read its range in the callback?
e.g.:
editor.willHandleNewline(event => {
  let range = editor.range;
});

@bantic I forgot why I added range, seems we can access editor.range directly... I'm going to change the commit a bit.

@bantic Ready for you

@bantic Thanks, now I see what you mean. Since there's no other changes, please close this PR. I agree with you that we don't  pass the editor in.

@eguitarz Thanks, yes, I'll give this a close. And I would like to apologize for all of the back-and-forth here. Somehow I mixed this up with your initial PR to add the willHandleNewline hook  and failed to realize that this wasn't the same — for some reason I thought this was continued discussion about the synthetic event that we had discussed in that PR. I could have saved you a lot of time by recognizing that and then pointing out that the editor and range are already accessible without being passed...my apologies! 🙇 😿


[mapping template] [request property type check] Create UNI add order and create Eline ADD order

a/c:

property type was added to mapping template
property type validation is checked after receiving request

need to add exist check for request validation

@xuelianhan007

eline name

paymenttype


Error while launching .exe file through PowerToys Run

Microsoft PowerToys version
0.87.1
Installation method
GitHub
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
I launched PowerToys Run using Alt+Space, then simply looked up a .exe file stored on my C: drive.
In my case, the file path was c:\WinaeroTweaker\WinaeroTweaker.exe.
✔ Expected Behavior
It should launch the app without having me look it up inside the File Explorer.
❌ Actual Behavior
2024-12-27.txt
It did launch the app, but also produced the log file above, and suggested me to report the bug.
Other Software
No response

/dup #36451


style(Form): convert component to stateless

#1367 removed all handlers, so Form can be stateless now :smile:

Codecov Report

Merging #1433 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1433      +/-   ##
==========================================
+ Coverage   99.74%   99.74%   +<.01%
==========================================
  Files         141      141
  Lines        2354     2365      +11
==========================================
+ Hits         2348     2359      +11
  Misses          6        6

Impacted Files
Coverage Δ

src/collections/Form/Form.js
100% <100%> (ø)
:white_check_mark:

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 206c777...06fb105. Read the comment docs.

🎉

What is the actual reason (or change request) to convert it to stateless and how do we handle submit or serialize the form from now on?

Please see the linked PR for before/after code examples for upgrading and more context on why the change.

Thx Levi I didn't see the reference to #1367 .

No worries, let me know if there are any further questions we could have answered.

Released in<EMAIL_ADDRESS>


Longtitudinal tuning of E9x

Car
E90 325i, MY2008
Tuning

Experimental mode ON
Acceptable performance matching to my driving style: more active acceleration, closer following distance, and less unnecessary braking.

need to figure out what values in toml causes:
openpilot/selfdrive/car/tests/test_lateral_limits.py::TestLateralLimits_7_BMW_E90::test_jerk_limits failed: self = <selfdrive.car.tests.test_lateral_limits.TestLateralLimits_7_BMW_E90 object at 0x7f95b2dd5340>

    def test_jerk_limits(self):
      up_jerk, down_jerk = self.calculate_0_5s_jerk(self.control_params, self.torque_params)
      assert up_jerk <= MAX_LAT_JERK_UP + MAX_LAT_JERK_UP_TOLERANCE
>     assert down_jerk <= MAX_LAT_JERK_DOWN
E     assert 6.0 <= 5.0

@OxygenLiu  any idea why it was constantly requesting max torque to the right and being undrivable? Is it related to lateralTuning.torque.latAccelOffset = -0.255 ? - I left it at default (0).

this is the torque params for cycloidal reducer 11:1, probably not suitable for planetary gearbox.
I had similar issue due to wrong stepper direction. :(

For 5:1 shouldn't be that much different.  Strange that it always go full to the right, not just opposite.
I will test longitudinal for now.


Can't build from source - meson: error unrecognized arguments: --enable-libmpv-shared

Trying to build from source but install fails at the 6th step:
./rebuild -j4
With this error:
make: Leaving directory '/home/user/Downloads/mpv-build/ffmpeg_build'
Using mpv options: --enable-libmpv-shared
usage: meson [-h]
{setup,configure,dist,install,introspect,init,test,wrap,subprojects,rewrite,compile,devenv,env2mfile,help}
...
meson: error: unrecognized arguments: --enable-libmpv-shared
Found a mention that meson stopped accepting unknown arguments in v0.61
https://www.reddit.com/r/Gentoo/comments/ssxk5x/meson_build_failed_due_to_unknown_options/hxf0z28/

You have to use this now:
-Dlibmpv=true

Sorry to bump my old issue, used your suggestion and got plex-mpv-shim to build successfully this time. Can the instructions on the main project page be updated to this?
You can build mpv from source to get better codec support. Execute the following:
1). sudo pip3 install --upgrade python-mpv
2). sudo apt install autoconf automake libtool libharfbuzz-dev libfreetype6-dev libfontconfig1-dev libx11-dev libxrandr-dev libvdpau-dev libva-dev mesa-common-dev libegl1-mesa-dev yasm libasound2-dev libpulse-dev libuchardet-dev zlib1g-dev libfribidi-dev git libgnutls28-dev libgl1-mesa-dev libsdl2-dev cmake wget python g++ libluajit-5.1-dev
3). git clone https://github.com/mpv-player/mpv-build.git
4). cd mpv-build
5). echo -Dlibmpv=true > mpv_options
6). ./rebuild -j4
7). sudo ./install
8). sudo ldconfig


Implement intervention

Add a method to Simulation which can modify reachable_states according to a function

Implemented in #19 and 8e5bb50


Why not use direct_uploads endpoint?

This is more of a question than an issue, but we currently have a github release in which the artifacts for a certain platform have the same name across versions, for example: ytt-darwin-amd64. This is causing errors when trying to automate the release creation with this resource because of duplicate filenames, although this is permitted via the UI. It looks like the browser uses the  https://network.pivotal.io/rails/active_storage/direct_uploads endpoint which provides a key and upload url whereas this resource computes a key locally and constructs the upload URL itself. Is there a specific reason this resource doesn't allow the same behavior that the UI does?
Thanks!
Eli
Note: we are more than willing to update our releases to include version numbers to distinguish them.

I see its not part of their API docs and is probably meant only for consumption in browser


makes it possible to run a GlowServer as part of another application

Example of minimal external application :
package com.acme;

import net.glowstone.GlowServer;

public class AcmeApplication {
    public static void main(String[] args) {
        GlowServer server = GlowServer.createFromArguments(args);

        // we don't want to run a server when called with --version
        if (server == null) {
            return;
        }

        // we could set some additional properties here

        try {
            glowServer.run();
        } catch (BindException e) {
            e.printStackTrace();
        }
    }
}

Please let me know if such a pull request would be acceptable, or if I can do anything to make it more pleasant :)

I don't think this is a patch suitable for Glowstone++, as it is intended to be its own server software. This patch should be for projects that need it.

I don't see how this is a bad patch. This makes sense- make it easier for users to embed a server in their application, without changing normal functionality.

It could be used as part of your test suite too. For exemple a test class could run a GlowServer to check that the server is listening on tcp port, and replying to some basic commands like "help". This is called smoke testing.

Open source software often find many use cases that were not intended or expected. I don't see the point of intentionally making it difficult to use by third parties.

I think this could be useful.

I guess since it doesn't affect us, we can merge it.

Thank you @twizmwazin @jimmikaelkael @mastercoms :)


Jupyter notebook: Option to dock the variable window on the righthand side

In jupyter notebooks the window with the variables from the kernel is positioned above the notebook. It would be nice to have the option to dock the window on the right side.

Thanks for the suggestion.
We've been considering creating a 'Jupyter' panel on the left (where the debugger/file/extensions panels are). This would likely have stuff like

TOC
Variables
Notebooks in a workspace

Any feedback on that idea?

Good idea, I‘d love to see that in a future version!


IMPORT ERROR

I am trying to pip install pretrainedmodels. I am getting an error as
ImportError: The 'packaging.version' package is required; normally this is bundled with this package so if you get this warning, consult the packager of your distribution.
Please help me solve.

Did you solve your problem? pip install pretrainedmodels works well on my side.


Support parsing LLVM's new JSON streaming API

LLVM added a JSON steaming output API in
https://reviews.llvm.org/D60804 and updated the TimeProfiler to use it:
https://github.com/llvm-mirror/llvm/blob/master/lib/Support/TimeProfiler.cpp#L164
This adds "prettifies" the JSON with whitespace and causes the "clang" metadata element to output as:
    {       "args": {         "name": "clang"       },       "cat": "",       "name": "process_name",       "ph": "M",       "pid": 1,       "tid": 0,       "ts": 0     }
This fails the quick check when scanning for files. The output from the
JSON formatter appears to be deterministic; this change adds the new
output format.

Hmm I'm testing with current LLVM/Clang 10.0 RC2 builds, and everything "just works" without this PR even. The JSON emitted by clang has the marker like {"cat":"","pid":1,"tid":0,"ts":0,"ph":"M","name":"process_name","args":{"name":"clang-cl.exe"}} and the current detection code handles it just fine.
I guess I'll close the PR; reopen it if you see different behavior.


Request: 'All' function in Remove Quarantine

Workflow version
Latest
Alfred version
Latest
macOS version
Latest
Debugger output
N/A

More details
Apologize for using bug report, but there was no way to request a feature.
@vitorgalvao  Is there any chance (pretty please with 🍒 on top) you could add a function to include all quarantined apps in the remove-quarantine workflow?  I realize one can use the Alfred buffer function but it would be so much more helpful and user-friendly to have a function to be able to dequarantine all of the apps in the list of quarantined apps without having to buffer each one and then apply actions to all.
I really hope you consider this as the extension would be perfection. Thank you for considering! Love your extensions. Appreciate your dedication to us in the  Alfred community.

Thank you for the kind words and the request, but this is not something I intend to add. Having to remove quarantine from multiple apps at once is a rare occurrence and is still quite fast to do via the File Buffer (hold ⌥↓). On the other hand, an option to remove from all could be triggered by mistake, which is a potentially undesirable outcome and is not easily reversible. The workflow allows you to remove quarantine, it does not encourage it. Quarantining is a security feature and thus should only be disabled on a case-by-case basis when necessary, not as a broad command.

That is very fair vitor. Thank you for taking the time to comment and explain.


TextBox SelectedForegroundColor

Is your feature request related to a problem? Please describe.
When text is selected in a TextBox, it can be difficult to read because of poor contrast.

On the above example, I would like white text color for the selected text.
Describe the solution you'd like
Would it be possible to add a property to set an alternate text color when text is selected?
Such property exists in WPF.
Thanks in advance if you could consider that improvement request!

https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls/TextBox.cs#L86

https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls/TextBox.cs#L86

Thanks a lot @Gillibald and sorry... how did I overlook that!? :(


Throne of miscellania

Quest helper is locked into marrying the princess with a male character, despite the fact that you can choose which one you want to wed, causes some issues because of the different process depending on which one you want to wed

This should be resolved with https://github.com/Zoinkwiz/quest-helper/commit/b689ec35e4190fbaea10abe78adff59e3659c0ae#diff-67fa8fbb61c8f974a94825b7cde1b4fcc63c6fd58b0dee473d463080026edcb3.


'Namespace' object has no attribute 'minidump'

Hi,
I just followed all the installation instructions through pip3, including all necessary modules.
All is installed in an ubuntu 18.04 server virtualbox.
I just provide SAM and SYSTEM files and need to have the hashes dumped.
I do:
pypykatz dpapi --system ./SYSTEM --sam ./SAM credential
And I get:
'Namespace' object has no attribute 'minidump'
I also do:
pypykatz dpapi --system ./SYSTEM --sam ./SAM vault
And I get:
'Namespace' object has no attribute 'vpol'
What is going on? Do you know of another method to do the same?
Thanks!

Ughh, you are right, fixing this mess as soon as I get my machine back.
Alternatively you can use the dpapi script from impacket here
Will get back to you when it's fixed.

Did a major rewrite on the dpapi command line helper, and added some fixes.
Please check the "devel" branch and let me know if it helped.
Please also note that the command have changed significantly :)


MAINT: fix incorrect noexcept usage in Cython functions

With Cython 3.1.0a0 (master), a lot of warnings like these are visible:
warning: scipy/linalg/_matfuncs_expm.pyx:90:26: noexcept clause is ignored for function returning Python object
warning: scipy/linalg/_matfuncs_expm.pyx:202:26: noexcept clause is ignored for function returning Python object

There can be several causes for this warning. The one given in the warning (returning a Python object), or the function does raise exceptions (in these cases MemoryError or ValueError), or it seems also functions/methods that don't return anything but use Python functionality internally (this is the least obvious, separated out in the second commit).
There's only one file with warnings that I left alone. That's _highs_wrapper.pyx; we're getting rid of that one and I don't want to create a merge conflict for gh-19255.

thanks Ralf


[release/1.7] Add cri-api v1alpha2 usage warning to all api calls

Previous https://github.com/containerd/containerd/pull/9336 added warnings for the usage of cri-api Status and Version.
There are some use cases where clients might use cri-apis without calling those two apis. For example,

client uses crictl to pre-pull container images,
client uses ContainerStatus to find the container for a certain pid for monitoring purposes.

Therefore, this PR adds warning to all cri-api v1alpha2 apis.

/cherrypick release/1.6


Third table in section 9.3.1 list four elements but related text and other table mention three elements

Note: Please ignore the header lines in the tables. GitHub flavored Markdown does not seem to support tables without headers.
Title
Third table in Section 9.3.1 list four elements but related text and other table mention three elements
Moderator
@user
Requirement Summary
Consistency of examples/tables/descriptions.
Technical Proposal Summary
Remove last line of third table of Section 9.3.1 to get this table:

time(o1)

time(o2)

time(o3)

Benefits
Consistency with text and other table.
Status Quo
Third table looks as follows:

time(o1)

time(o2)

time(o3)

time(o4)

Associated pull request
#289
Detailed Proposal
refers to Section 9.3.1 Orthogonal multidimensional array representation
Third table looks as follws:

time(o1)

time(o2)

time(o3)

time(o4)

However, the text states:

and the element coordinate axis will be one-dimensional with size 3 (for example, the time [...] coordinates that are shared by all of the timeSeries).

and the first table looks as follows:

(i1, o1)
(i2, o1)
(i3, o1)
(i4, o1)

(i1, o2)
(i2, o2)
(i3, o2)
(i4, o2)

(i1, o3)
(i2, o3)
(i3, o3)
(i4, o3)

Hence, the third table should look as follows:

time(o1)

time(o2)

time(o3)

Thanks @neumannd .This defect is clear, fixed by the PR, and more than three have passed with no comment. Therefore we can merge the fix and close the issue.


update travis to use Mac OS 10.14

Looks like the font rendering in Mac OS changed so we get
diffs when running locally. See if this causes travis to
fail so we can update to get consistent rendering again.

The font rendering issue actually seems to be some difference between OpenJDK 11.0.2 (11.0.2-open) and Zulu 11.0.4 (11.0.4-zulu).

Most likely the change was caused by: https://bugs.openjdk.java.net/browse/JDK-8218854


zygomaticomaxillary suture

abnormal zygomaticomaxillary suture morphology (this is a facial suture)
DEF: any structural anomaly of the suture between the zygomatic process of the maxilla and the maxillary margin of the zygomatic bone
SOURCE: https://radiopaedia.org/articles/zygomaticomaxillary-suture
Annotation in J:158773

abnormal zygomaticomaxillary suture morphology MP:0030231


Getting a few peer-dependency warnings on installing packages

Thanks for the plugin - it works great!
However I can see a few "incorrect peer dependency" warnings when installing packages which would be nice to get rid of, e.g.
sanity-plugin-asset-source-remove-bg > @sanity/ui > @reach/auto-id ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 17.x".

Hi, I have not touched this plugin in 2 years, but I can see if I can bump the versions. Are you running any specific version of the studio?


[SMALLFIX] Add options to methods in lineage client for consistency

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/5596/
Test PASSed.

+1 LGTM


Convert an enterprise organization to an enterprise account

Summary
GitHub Enterprise Accounts provide the latest enterprise administrative features, such as audit log streaming, and the ability to manage multiple organizations under one enterprise governance umbrella.
Today, organizations on an enterprise plan can convert to an Enterprise Account by contacting GitHub Support or a customer's account team. We are aiming to enable this as a self-serve experience.
Intended Outcome
Enterprise plan organizations paid by invoice will be able to convert their account to an Enterprise Account via the organization settings.
How will it work?
An invoice-based GitHub Enterprise Organization will see a Upgrade to Enterprise Account button on their Organization's Billing Settings page, which will then trigger the workflow to create the Enterprise Account.

https://github.blog/changelog/2021-11-10-github-enterprise-cloud-self-service-organization-upgrade-to-an-enterprise-account/


registerDefaultExceptionMappers: false still maps ValidationExceptionMapper

We've configured Dropwizard not to register default exception mappers by setting registerDefaultExceptionMappers: false in the yml file. However it's still mapping org.glassfish.jersey.server.validation.internal.ValidationExceptionMapper and is messing with our catch-all exception handler.
Why is this not excluded and is there a way to remove it?

registerDefaultExceptionMappers is for toggling if dropwizard should register its own exception mappers, it doesn't make reference to any of Jersey's internal exception mappers (eg: ValidationExceptionMapper).
I looked into it a little bit, and I'm not sure if using standard dropwizard code if ValidationExceptionMapper will be registered in the first place. Looking at the Jersey source code, it appears it is only registered if ValidationFeature is registered (which dropwizard doesn't). So I'd check if you're are registering either ValidationFeature or ValidationBinder (and if so, remove them)

Thanks! I did some more digging around and it appears it's registered by Jersey. Here's an open issue against Jersey on the issue: https://github.com/jersey/jersey/issues/3425
I was hoping there might be a more elegant way but looks like there isn't :(

@andrewoh531 did you ever fix this issue?


Where to find an exhaustive list of exposed vulnerabilities?

Is there a place where we can find an exhaustive list of all the vulnerabilities exposed by firing-range?
I wanted to compare the result found by some tools with the reality and opening the vulnerabilities pages one by one is quite time consuming.
Thanks,

Hi,
I understand the desire: we started maintaining a comprehensive list, but the rate of increase of new vulnerabilities while we were building the firing range was simply too fast to make it effective: we decided the code was enough documentation. It's clearly not a great decision, in hindsight, but I don't think the core team will have the time to build the list.
If you want to try to build one though, I think we can maintain it up to date with future changes.


Support running python images with a random UID

@bparees @soltysh ptal

lgtm but would like @soltysh opinion.

LGTM, just one question, I'll wait with merging to get the answer for that.


Configuration.timeout setting is invalid in open () method

I am using the following.
selenide: 4.14.0
geckodriver: 0.23.0
firefox: 60.3.0 ESR

In @ Before of the test, we specify the following.
Configuration.timeout = 1200000;

However, the value of Configuration.timeout = 1200000; is invalid when a timeout occurs in the open () method.
org.openqa.selenium.TimeoutException: Timeout loading page after 300000 ms
Build info: version: '3.14.0', revision: 'aacccce 0', time: '2018-08-02 T 20: 19: 58.91 Z'
System info: host: 'XXXX', ip: 'XXXX', os.name: 'Linux', os.arch: 'amd64', os.version: '3.10.0-693.21.1.el7.x86 _ 64', java .version: '11 .0.2 '
Driver info: org.openqa.selenium.firefox.FirefoxDriver
selenide.url: http://<IP_ADDRESS>:8082/XXXX
Moz: headless: false, moz: processID: 19333, moz: profile: capabilities {acceptInsecureCerts: true, browserName: firefox, browserVersion: 60.3.0, javascriptEnabled: true, moz: accessibilityChecks: false, moz: geckodriverVersion: /tmp/rust_mozprofile.c7blpK ..., moz: useNonSpecCompliantPointerOrigin: false, moz: webdriverClick: true, pageLoadStrategy: normal, platform: LINUX, platformName: LINUX, platformVersion: 3.10.0-693.21.1.el7.x86_64, rotatable: false, timeouts: {implicit: 0, pageLoad: 300000, script: 30000}}
selenide.baseUrl: http: // localhost: 8080

I think that pageLoad: 300000 is the cause.
Is the timeout setting of the open () method different from that specified by Configuration.timeout?

we are probably not polling often for checking page is loaded with open and I really don't know if we should or want to be.  I will look at this tonight to see.

@btkukinom @symonk Configuration.timeout is used when finding elements and checking conditions. For page load timeout, there is a separate setting. You can set it via webdriver api: getWebDriver().manage().timeouts().pageLoadTimeout(30, SECONDS);

Just in case anyone would know: getWebDriver() is available after you add this import:
import static com.codeborne.selenide.WebDriverRunner.getWebDriver;

@chipsenter Did you read the javadoc of this method? The same place where you see it's deprecated.
    /**
     * @param time The timeout value.
     * @param unit The unit of time.
     * @return A Timeouts interface.
     * @see W3C WebDriver
     * @see W3C WebDriver
     * @deprecated Use {@link #pageLoadTimeout(Duration)}
     *
     * Sets the amount of time to wait for a page load to complete before throwing an error.
     * If the timeout is negative, not null, or greater than 2e16 - 1, an error code with
     * invalid argument will be returned.
     */
    @Deprecated
    Timeouts pageLoadTimeout(long time, TimeUnit unit);

No just read the code :)
getWebDriver().manage().timeouts().pageLoadTimeout(10, TimeUnit.MINUTES);

would this work instead >
getWebDriver().manage().timeouts().scriptTimeout(Duration.ofMinutes(10));

yes, it should work.

@chipsenter Why you think this wait is not working?
I think it works. I see the clear text "Timeout: 45 s." in your error message. It means Selenide had been waiting for 45 seconds before throwing this error. And the element hadn't became visible in 45 seconds. Probably you just need a bigger timeout?

I see that too and that part is OK,  only thing I don't understand is why
it's not checking the element even though it's visible in dom?
What I understand from this call,  selenide should wait until the element
is rendered, if it takes 1 sec or 30sec it will wait and click.?
Wish I could show you when it runs.?
On Wed, Aug 17, 2022, 10:54 AM Andrei Solntsev @.***>
wrote:

@chipsenter https://github.com/chipsenter Why you think this wait is
not working?
I think it works. I see the clear text "Timeout: 45 s." in your error
message. It means Selenide had been waiting for 45 seconds before throwing
this error. And the element hadn't became visible in 45 seconds. Probably
you just need a bigger timeout?
—
Reply to this email directly, view it on GitHub
https://github.com/selenide/selenide/issues/906#issuecomment-1218270548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFKMYXAUKU232EYS5ABCBDDVZUKLRANCNFSM4GULEU7Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

@chipsenter Yes, Selenide waits (for 45 seconds) until the elements becomes visible.
But the problem is that the element doesn't get visible. It's invisible. Look at the error message:
Element should be be visible
Element: '<input displayed:false>'
Actual value: hidden
For some reason, Selenium webdriver thinks this element is not visible.

Sorry I created duplicates for this, but why does it work when I remove the Duration call and use a hard sleep instead ??

@chipsenter How can I answer your question?
If you provide some instructions how I could reproduce the issue, I would be able to investigate it. But I cannot just answer all types of questions like "why my code isn't working?" without seeing the code...
Could you at least show your code with "hard sleep"?

Thought I posted code above ?.
this is the method that will check if there is any elements in the table.
line 1-5 is the most important one, when we enter this method and use selenide DurationOfSeconds its basically just waiting for 45sec and wont jump to the line 3 to give me the total checkboxes seen.
scenario 2.  if I comment out line 2 (hard sleep) and comment line 1 test will execute fine and work. So to me the 'shouldBe visible param is not working in this case.
public void checkFirstCheckboxInTableAndValidate() throws InterruptedException {

   $$(By.className("ag-checkbox-input")).get(0).shouldBe(visible, Duration.ofSeconds(45));

//        HelperMethods.wait(20000);  // HARD SLEEP

  int checkboxes = $$(By.className("ag-checkbox-input")).size();

  System.out.println("Selenide can see: " + checkboxes + " in the table");

   for (int i = 12; i < checkboxes; i++){

      $$(By.className("ag-checkbox-input")).get(i).click();
     HelperMethods.wait(1000);
     if($(By.id("my-tasks-actions-mark-complete-btn")).isDisplayed()){
         System.out.println("Mark as completed button is visible and enabled");
         $(By.id("my-tasks-actions-mark-complete-btn")).shouldBe(visible).shouldBe(enabled);
         break;
     }
 }

}

any suggestions for change above ?

Ahhaa!
And what you see in the logs after "Selenide can see: " in scenario 2?

[image: Screen Shot 2022-08-17 at 11.20.13 AM.png]
It took 8 sec to load the table this run, but Selenide waited for 45sec and
failed.
On Wed, Aug 17, 2022 at 11:09 AM Johan Lindstrom @.***>
wrote:

I see that too and that part is OK,  only thing I don't understand is why
it's not checking the element even though it's visible in dom?
What I understand from this call,  selenide should wait until the element
is rendered, if it takes 1 sec or 30sec it will wait and click.?
Wish I could show you when it runs.?
On Wed, Aug 17, 2022, 10:54 AM Andrei Solntsev @.***>
wrote:

@chipsenter https://github.com/chipsenter Why you think this wait is
not working?
I think it works. I see the clear text "Timeout: 45 s." in your error
message. It means Selenide had been waiting for 45 seconds before throwing
this error. And the element hadn't became visible in 45 seconds. Probably
you just need a bigger timeout?
—
Reply to this email directly, view it on GitHub
https://github.com/selenide/selenide/issues/906#issuecomment-1218270548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFKMYXAUKU232EYS5ABCBDDVZUKLRANCNFSM4GULEU7Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


More trouble getting online

I know its literally two years since you finished this project, but I tried getting it working again (on windows 11) and got this message. I've never coded in java, and haven't worked with discord bots before so I don't know how to fix this, although it seems quite simple

I figured it out: fixing this is built into the node.js system
you can literally type " npm install discord.js "


Error: The specified GIMP installation is not supported.

What went wrong?
During installation, there's an error. The error message says "Error: The specified GIMP installation ('/Applications/GIMP.app/Contents/MacOS/GIMP') is not supported. Please install GIMP 2.8 or later or specify custom path to GIMP with the -g option and the plug-in installation path with the -i option."
I tried with -g and -i option. Still the same thing.
If you can, tell us the steps to reproduce the problem.
Not sure how to reproduce since it's an installtion problem
Configuration
Version of Export Layers: 3.2.1
Version of GIMP: 2.10.6
Operating system: macOS High Sierra 10.13.4

Try the manual install (the ZIP archive), that should work. Check the instructions if needed.
For the next version, I'm thinking of removing the installer scripts for Linux and macOS altogether due to how unreliable they are to use (different installation paths, using portable installations, etc.), when it's really about copying a file and a folder.


Updated versions of postgresRAW and postgresRAW-UI

Adaptation to schema v1.1 and bug fix

This change is

Why no tags in https://github.com/HBPMedical/PostgresRAW-docker and https://github.com/HBPMedical/PostgresRAW-UI-docker?

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add warning for long durnation of max CPU usage on powersave

Is your feature request related to a problem? Please describe.
Adds a warning (in monitor perhaps) for long durnation of max CPU usage for one of the cores when in powersave mode, so that it could notify the user when a process is stuck (like let's say a webpage that does not stop loading) and potentially draining the battery
Describe alternatives you've considered
Keeping an eye on CPU usage manually
Additional context
My battery died in a lab because canvas keeps loading forever and I didn't notice :(

We could also predict battery life. I could see these two features working well together.

Mb we can also add a graph for predicted battery life


Webpack source maps

I am struggling to get NYC working with a combination of webpack, mocha & mocha-webpack. Right now I am getting a test coverage report from NYC but that's just one compiled file. According to the documentation the source map should be inline so I did that with webpack but unfortunately I still see the compiled file in the test report.
Is this approach even possible, and if so what am I doing wrong here?

@JaKXz can you provide any feedback on this, I know you were attempting a similar setup.
@reauv we're having some luck hooking into source-maps using an alternate instrumenter, discussed here: https://github.com/bcoe/nyc/pull/268

Hi @reauv, I have indeed attempted a similar setup: ATM and AFAIK, webpack is not compatible with nyc because nyc looks at all the files that are required in your tests to instrument them, whereas webpack loads them into a bundle runs things differently. See #176 or #229.
If possible, I'd suggest running your tests directly with mocha and using ts-node. Here's an example from a project I'm working on: https://github.com/JaKXz/typescript-react-redux-starter/tree/749a0f22b7dce6781b9e52c3c95e089effbde810
The only thing that example is missing is the ability to instrument --all files, because there's an esprima error from trying to transpile the TS code for nyc to look at that gets thrown that I haven't had time to look further into. If you're just using ES2015+, you should just be able to do nyc --all --require babel-core/register mocha [assuming you have your ./test/mocha.opts file configured correctly]. Hope that helps!

Thanks for the responses. I am not using TS, so I am not sure if that's going to help me. I ended up switching to a setup with Karma & Isparta, because that works with webpack. I kinda need webpack because otherwise I have to rewrite all my imports because I extensively use webpack's resolve settings. Thanks anyway!


Cross-vault reference links are not rendered in V2 without dendron// prefix

Please select the area your Bug applies to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[x] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Describe the bug
A cross-vault reference link without the dendron// prefix does not render in V2 preview:
![[foo.bar]]

---

error reading file, c:\Users\vault\foo.bar.md

The full link (![[dendron://vault/foo.bar]]) does render correctly.
To Reproduce
Steps to reproduce the behavior:

Add a reference link to a note from a different vault.
Open V2 preview
See the error message

Expected behavior
Reference links should correctly render in preview without the dendron// prefix.
Desktop (please complete the following information)

Version 0.56

This issue has been fixed and shipped with v59 🥳
Please feel free to leave a comment if the issue still occurs. Otherwise, this issue will be automatically closed in 4 days.

Confirm this is fixed now. Thank you!


Automatic reboot after shutdown due to fatal error

When fatal error occurs, JVM kills itself (this is configurable but it shouldn't be disabled).
When akashic-storage is in production it shouldn't be stopped for a long time then it should restart automatically. Rebooting only needs binding to the port. It's quick.

The easiest way to do this is making a daemon watching service status and when it's not running start it. the daemon should run outside the JVM akashic-storage daemon is running on.
When user really want to stop the storage, he should first stop the watching daemon and then storage daemon.


Add an annotation when report is too large

This would at least give people a clue as to why there were no annotations but the tests failed.

This looks good! thanks @goodspark for your contributions!


CAMEL-17797: Test with FHIR version R4

Cherry pick over James' commit to make the camel-fhir-starter tests work.
Do we need to cherry-pick over
https://github.com/jboss-fuse/camel-spring-boot/commit/583eb3872345bd57f23edb4b8ab6d2c2beedee48
and the associated camel commit?

Looks like the commit is already present in the branch camel-spring-boot-3.14.5-branch so we should be good to go

Sweet!    Totally missed that that was there.


[✨Feature Request] Support CSS-in-js syntax for shortcuts

Clear and concise description of the problem
Currently when defining shortcuts, when you want to apply different styles when different shortcuts are combined, it's more or less a pain now. It's basically the concept of component in tailwind.
Giving an example, I want it to have different style when .btn-primary, .btn-secondary are combined with .btn-sm, .btn-md, .btn-lg.
And the current shortcuts does not support extracting common part of two classnames because the earlier one will be overwritten by the latter one.
UnoCSS shortcuts:
This is how I implement it in shortcuts, which is a little bit frustrating and very hard to read...
const shortcuts = {
  btn: 'inline-flex shrink-0 cursor-pointer items-center justify-center select-none [&.disabled,&:disabled]:cursor-not-allowed',

  'btn-primary': 'transition-color,background-color,border-color rounded-full nf-transition text-white bg-orange [&:not(:disabled,.disabled)]:@hover:bg-orange-700 [&.disabled,&:disabled]:bg-orange-200 [&.btn-sm]:(font-medium text-16/18 px-16 py-8) [&.btn-md]:(font-bold text-18/none px-30 py-16) [&.btn-lg]:(font-bold text-20/[0.9] px-40 py-16)',

  'btn-secondary': 'transition-color,background-color,border-color rounded-full nf-transition border text-orange border-orange [&:not(:disabled,.disabled)]:@hover:bg-orange-50 [&.disabled,&:disabled]:(border-orange-200 text-orange-200) [&.btn-special.disabled,&.btn-special:disabled]:(text-grey-300 border-grey-300) [&.btn-sm]:(font-medium text-16/18 px-15 py-7) [&.btn-md]:(font-bold text-18/none px-29 py-15) [&.btn-lg]:(font-bold text-20/[0.9] px-39 py-15)'
} satisfies UserShorcuts

Tailwind plugin
where in tailwind's css-in-js syntax, it's a lot easier and readable...
import plugin from 'tailwindcss/plugin';
export default plugin(({ addComponents }) => void addComponents({
  // button
  '.btn': {
    '@apply inline-flex shrink-0 cursor-pointer items-center justify-center select-none': {},

    '&.disabled, &:disabled': { '@apply cursor-not-allowed': {} }
  },

  '.btn-primary, .btn-secondary': {
    transitionProperty: 'background-color, border-color, color',
    '@apply rounded-full nf-transition': {},

    '&.btn-sm': { '@apply font-medium text-[16px]/[18px]': {} },
    '&.btn-md': { '@apply font-bold text-[18px]/none': {} },
    '&.btn-lg': { '@apply font-bold text-[20px]/[0.9]': {} }
  },

  '.btn-primary': {
    '@apply text-white bg-orange': {},

    '&:not(:disabled, .disabled)': { '@apply hover:bg-orange-700': {} },
    '&.disabled, &:disabled': { '@apply bg-orange-200': {} },

    '&.btn-sm': { '@apply px-4 py-2': {} },
    '&.btn-md': { '@apply py-4 px-7.5': {} },
    '&.btn-lg': { '@apply px-10 py-4': {} }
  },

  '.btn-secondary': {
    '@apply border text-orange border-orange': {},

    '&:not(:disabled, .disabled)': { '@apply hover:bg-orange-50': {} },
    '&.disabled, &:disabled': { '@apply border-orange-200 text-orange-200': {} },

    '&.btn-special': {
      '&:disabled, &.disabled': { '@apply text-grey-300 border-grey-300': {} }
    },

    '&.btn-sm': { '@apply px-3.75 py-1.75': {} },
    '&.btn-md': { '@apply px-7.25 py-3.75': {} },
    '&.btn-lg': { '@apply px-9.75 py-3.75': {} }
  }
}));

Suggested solution
Support CSS-in-js syntax for shortcuts, which will make this kind of use case a lot easier and maintainable.
Alternative
For apply, this is a limitation in tailwind, where you have to use syntax like { '@apply px-2': {} } to use @apply directive in css-in-js syntax. However, Windi provides another better way https://windicss.org/features/shortcuts.html#shortcuts
export default {
  shortcuts: {
    'btn': {
      '@apply': 'py-2 px-4 font-semibold rounded-lg'
    }
  },
}

which worth to be included in.
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the README.md of using the package.
[X] Already used the Interactive Docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

bump

bump

Since we support passing CSS object in shortcuts: https://github.com/unocss/unocss/blob/07a67f97f5c41f55ee47c0157c4335860577292a/test/shortcuts.test.ts#L32
I guess it should be possible for a community solution that converts this css-in-js format to Uno's shortcuts format.

Interesting point. I was looking for the same @apply-like thing.
I am not quite sure if I understand it right...
What is a CSS object?
This is working:
['shortcut-inline-body', ['p2', { margin: '3px' }]]

This is not:
['shortcut-inline-body', ['p2', { marginTop: '3px' }]]

But this is:
['shortcut-inline-body', ['p2', { 'margin-top': '3px' }]]

Is this a bug or is it meant to work this way?

CSS Object keys

Yes, it's designed to work that way. UnoCSS does not do case conversion for you to maximize the performance and align more with how CSS looks.

Thank you @antfu


Is AppSync available for Xamarin?

I was just wondering is AWS AppSync is available for Xamarin PCL or Xamarin.iOS/Xamarin.Android?
I kinda see the code but are they just placeholders for now?
Github : https://github.com/aws/aws-sdk-net/tree/master/sdk/src/Services/AppSync
Then how come there's Nuget project?
Nuget : https://www.nuget.org/packages/AWSSDK.AppSync
If this is meant to work, please confirm it for me. I really couldn't find any sample snippet or app on Google but hopefully I will be able to work it out myself as long as this is "meant" to work.
Thanks in advance!

Is there a sample project for AppSync in Xamarin?

Anyone make any progress with this? Looking for an update. Thanks!

Any updates on this? I would like to use this in a project I am working on, thank you!

I've done all my server-side stuff (schema, data sources... etc) only to find out there's nothing about appsync on xamarin 😞 .. I probably should have checked availability before embarking!
Would be nice to get an official word/update on it :)

Hello @johnchoicl,
I have added this as a feature request in our backlog and have submitted it to the team for prioritization. I do not have specific timeline that I can share at this point. We will let you know, either on this thread, or through a forum announcement when the feature is released in the SDK

@johnchoicl Thanks for adding this. Any progress so far?

@BryanGerre,
Unfortunately, there's no update to report yet.

Hey all, just letting you know that the AppSync client is currently in development, will have an update on dates for release soon.

Awesome!
Sent from my iPhone

On Feb 11, 2019, at 12:41 PM, Flush Arcade<EMAIL_ADDRESS>wrote:
Hey all, just letting you know that the AppSync client is currently in development, will have an update on dates for release soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@BryanGerre, who is Flush Arcade? Where did this email come from?

Not sure Git blanked out the email I assumed it was someone in your organization. When I reply it posts to the appropriate git
Sent from my iPhone

On Feb 11, 2019, at 12:41 PM, Flush Arcade<EMAIL_ADDRESS>wrote:
Hey all, just letting you know that the AppSync client is currently in development, will have an update on dates for release soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Not sure where this came from. I have no update on an AppSync client for Xamarin.

Is there any news about AppSync for Xamarin? So now, using AmazonAppSyncClient, we can work with appsync service(create, update, delete functions, tables, query), but no any way just to launch query?

Are there any news about AppSync for Xamarin? So now, using AmazonAppSyncClient, we can work with appsync service(create, update, delete functions, tables, query), but no any way just to launch query?

Is AppSync now working for Xamarin?

+1 for AppSync GraphQL client for .NET SDK.
For those trying to do this manually today, I have a gist available that shows using the built-in AWS4Signer to access an AppSync GraphQL endpoint using GraphQL .NET client protected by AWS_IAM authorization.

+1 I'm also really interested in seeing a sample xamarin or dotnetcore example application

+1

+1

+1

@muellerfr Hi, how can we find out whether a Xamarin version is in the pipeline, and an indication of when it might come out? If there are no such plans, how can we go about it to get it on the radar? I for one think that Xamarin support would have the potential to bring a significant user base over from MS Azure (Mobile Apps which also supports offline sync) to AWS AppSync. This is the only reason that we are still on Azure - all our UI code is done in Xamarin.

+1

As part of our platform consolidation project, Xamarin components are being archived.   Xamarin users hoping to utilize AWS AppSync should use the .NET SDK targeted to .NET Standard 2.0.

@klaytaybai does the  .NET Standard 2.0 AWS .NET SDK have examples for AppSync? Thanks

Ok but can you release a sample Xamarin app that uses it so that we can see how it should be done?

Any update on this? Could you provide a sample project for AppSync with Xamarin?

I have been in contact with aws support about this, this is what they sent me:
To sync the data between the client and server you can push data to your AppSync data source using mutations [1]. With AppSync subscriptions, you can get updates to the data back in real time whenever a mutation is performed [2].
There is currently no .NET SDK which can be used for interacting with your AppSync API, but calls to AppSync can be made over internet using a HTTP client. Therefore, you can leverage HTTP requests to make queries and mutations over internet. Subscriptions in AppSync need a Websocket connection and operate like a normal graphQL subscription [3].  I do not have a C# code example for subscriptions, but I have attached a great document (AppSyncwscat.docx) that shows how to connect to your AppSync real time endpoint using wscat, which I think you may find useful. You can try to use that example to change the logic and adapt it for whatever SDK you are using.
With regards to AWS .Net SDK with AppSync libraries, there is currently a feature request open for an AWS .Net SDK with AppSync libraries, but there is no timeline available for when this will be released. Although there is no timeline available for an AWS .Net SDK with AppSync libraries, please note that AppSync can be used from any SDK that has GraphQL capabilities.
===References===
[1] Mutations: https://docs.aws.amazon.com/appsync/latest/devguide/quickstart-write-queries.html#add-data-with-a-graphql-mutation
[2] Real-Time Data: https://docs.aws.amazon.com/appsync/latest/devguide/real-time-data.html
[3] Subscriptions: https://graphql-dotnet.github.io/docs/getting-started/subscriptions/
AppSyncwscat.docx

So why are there NuGet packages for both AppSync and Amplify, if neither are supported in C#/.NET?
The very existence of these packages, combined with the fact that they're actively maintained/new versions get released, it all just seems highly confusing, especially as the official documentation seems to be all over the place regarding this too.

Any Updates please,anytime line for AppSync in Xamarin

Similar issue tracking adding support AppSync/Amplify

We will address this request in the ticket mentioned by @ppittle


Valuation P/E and P/S should be unique.

https://github.com/oceanprotocol/tokenspice2/blob/aeca439b4beb1462e1f988f5d8eac60c2c5e8cbf/util/valuation.py#L28
These  functions are the same. I believe they should be slightly different given that one is P/E and  one is P/S. I don't know enough about KPI's to fix this confidently myself. Looking for help.
def firmValuationPE(annual_revenue: float, p_e_ratio: float) -> float:
    """Valuation by price-to-earnings (P/E) ratio. Earnings are
    sales minus costs.
    Some people prefer P/E over P/S as it captures profitability. However,
    companies plowing extra money into growth will have poor P/E. Eg Amazon.
    """
    return annual_revenue * p_e_ratio

def firmValuationPS(annual_revenue: float, p_e_ratio: float) -> float:
    """
    Valuation by price-to-sales (P/S) ratio.
    Valuation is simply a multiple of sales.
    Datapoint: most blockchain co's are 30x-50x sales, as are high-flying
    startups like Zoom.
    """
    return annual_revenue * p_e_ratio

The price-to-sales one currently uses incorrect variable naming, it should be p_s_ratio not p_e_ratio. Just created #7 to address. Therefore this issue is obsolete.

The price-to-sales one currently uses incorrect variable naming, it should be p_s_ratio not p_e_ratio. Just created #7 to address. Therefore this issue is obsolete.


notary can't start with postgres 14.1

We found that the notary can't start up with postgres 14.1 which is shipped as the default postgres version inside Photon 4.0.
This may require the gorm dependence to be upgrade.

@zyyw  does this come from the scam authentication issue ?


Consider using cargo-generate v0.6.0+

💡 cargo-generate v0.6.0+
Describe the feature
Currently wrangler generate ... calls out to cargo-generate v0.5.0 to template out any files.
From v0.6.0+ cargo-generate supports template defined placeholders (aka custom variables) that allows template authors to better customize templates based on user prompts.
Would it be possible to bump the dependency on cargo-generate to at least v0.6.0 (but preferably to the latest release)?
Describe the alternatives
My hack was to copy over a build of cargo-generate (v0.7.0) to $HOME/.cache/.wrangler/cargo-generate-0.5.0/cargo-generate. Not really a solution, but a good way to test out things without making any code changes.

@jashandeep-sohi - thanks for the suggestion! I'll try it out and see if there is any objection to doing this.

I think our blocker right now is that cargo-generate does not distribute already-built binaries through GitHub releases (like it use to)

I think our blocker right now is that cargo-generate does not distribute already-built binaries through GitHub releases (like it use to)

Hopefully since v0.7.2 it should :wink: https://github.com/cargo-generate/cargo-generate/releases/tag/v0.7.2

Thanks for adding that back @jashandeep-sohi! I'll update our tracking ticket to get this tested. Will update here with progress asap.

Thanks for reporting this issue! Wrangler v1 is now deprecated and support is only being provided for critical updates or security concerns. As such, we are closing this issue.
New versions of Wrangler are maintained in the workers-sdk repo. If you are running into a similar issue with wrangler v2, please report it in the workers-sdk repo. For more info about wrangler v1 deprecation, please check out our blog post.


When would process hook run after reducer?

Hi Jeff,
In your process hook docs you state that the process hook typically runs after the state has been updated (unless other logic delay's execution).
I'm not sure I fully understand the caveat of other logic delaying execution.  The only thing I could think of was other logic having asynchronous validate/transform hooks, but even that would seem to also delay the reducer execution as well.
I have a couple of logic modules that are relying on the state to be updated, so I want to fully understand the scenario.
Can you enumerate just a bit?
Thanks for your time, and thanks for a great product!

@KevinAst Thanks for the question and the kind words.
Yes, in most common use cases the state will be updated before the process hook runs. The original action that triggered the logic to run has passed through any validate/transform hook and it will have been passed onto the next logic (and any other middleware below this).
A few less common scenarios that could delay the state being updated:

logic with a validate/transform hook that runs after this logic and which performs an asynchronous task before continuing. If everything is synchronous in the validate/transform then there will be no delay. Or if you configure any logics that might do async work in the validate/transform hooks to run first in the logic stack then that could help.
middleware that is configured to run after the logicMiddleware which does async work before continuing. It depends on the middleware as to whether it will just pass through or do other work. If these are configured before logicMiddleware then they won't come into play.
If your validate/transform hook changes the action type to a different type, then it will be dispatched to the top of the redux pipeline (so everything has a chance to see it, rather than falling through using next). This by itself wouldn't necessarily delay it, but again steps 1 and 2 apply to that action.

So the main thing is if you are doing async validate/transform code then that could delay an update, but only if those are below your other logic. Most of the time validate/transform logic code is synchronous so it is not an issue.
The middleware async is not normally an issue, but you can configure it to run before redux-logic if you need to.
Hopefully that helps clarify.

Thanks @jeffbski for your quick response.
Stated a slightly different way (if I understand what you are saying):

State change (i.e. reducer execution) only occurs after ALL validate/transform hooks give the thumbs up (i.e. for ALL logic modules monitoring that action).  As a result any one of the logic module validate/transform hooks could delay state change, when async behavior is employed.

On the other hand, a process hook is executed solely on the determination of it's own validate/transform.  I think you typically invoke the reducer before the process hook, but if the reducer execution is being held up by a downstream asynchronous validate/transform, then the current process hook is invoked before the reducer.

If this is true, I guess it kinda makes sense.
Assuming my summary is correct, would it be advantageous to delay ALL process hooks till after ALL validate/transform hooks had resolved?  Regardless of whether the reducer was executed or not, you could have a "stake in the ground" that ALL process hooks would see the end-state result of the dispatched action.
In my mind, this seems more deterministic.
Thoughts?
In regard to other middleware, why would anyone ever want other middleware (with the advent of redux-logic)?  I'm kidding (in a way), but seriously: I previously used 3 or 4 middleware services, and after I started using redux-logic all other middleware just naturally disappeared (with very simple straight-forward logic modules).

@KevinAst That seems like a great idea and I think it would be relatively easy in the new design I am considering which manages each of those features separately. That does seem like that would be more deterministic.
I would also agree with you regarding not needing any other middleware after adding redux-logic. That's been the case for me too. Of course we do want to make it easy for people to co-exist and to transition to redux-logic over time.
Thanks for the wonderful discussion and ideas. I'll carefully consider this idea but it does seem like the right way to go.


Fix deadlock between RTPSender and TrackLocalStaticRTP

RTPSender.mu.Lock() and TrackLocalStaticRTP.mu.Lock() caused deadlock
when continuously create and close PeerConnection.

The deadlock was occurred in my custom interceptor. Closing.


Refactor external memory formats.

Save base_rowid.
Return write size.
Remove unused function.
Add tests.

Part of https://github.com/dmlc/xgboost/pull/7070 .

Codecov Report

Merging #7089 (9d723d2) into master (615ab2b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7089   +/-   ##
=======================================
  Coverage   81.61%   81.61%
=======================================
  Files          13       13
  Lines        3899     3899
=======================================
  Hits         3182     3182
  Misses        717      717

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 615ab2b...9d723d2. Read the comment docs.


SUBMARINE-1110. Add yaml linter or formatter

What is this PR for?

Use yamllint to enforce coding style of YAML files.
Note that since yamllint doesn't support helm syntax, test is NOT run on files under helm-charts directly.
Instead, those files are rendered with helm template before being tested.
See script dev-support/style-check/lint-yaml.sh for more detail.
What type of PR is it?
Improvement
Todos
What is the Jira issue?

https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-1110
How should this be tested?

A new test script is added: dev-support/style-check/lint-yaml.sh
Screenshots (if appropriate)
Questions:

Do the license files need updating? No
Are there breaking changes for older versions? No
Does this need new documentation? No

@KUAN-HSUN-LI thanks for the review.

Yes. The outputs are now properly formatted and can be seen at Actions > Check Style > View raw logs. Do ctrl+F for .github to see warnings regarding those files.
I think I just fixed the script lint-yaml.sh with commit 2fae698.

Right now submarine-cloud-v2/test/e2e/ is failing and I'm looking into it; any help is appreciated.
Also, should I mark this PR as a draft before autoformat and the corresponding docs are added?

Actually, I can't quite find a YAML formatter that supports helm syntax. I'm tempted to leave autoformat for a future issue and limit this one to linting.

Another issue I found is that Prettier, which is used in submarine-cloud-v2/, will always break linting for multiline flow sequences (the [] list). An example would be line 64 of submarine-cloud-v2/artifacts/submarine/submarine-mlflow.yaml
# output of prettier (4 spaces for items inside "[]", 2 spaces anywhere else)
Item:
  Item-1: [
      "item-a",
      "item-b",
    ]
  Item-2: item-a
  Item-3: item-a

# expected: consistent indentation (2 spaces)
Item:
  Item-1: [
    "item-a",
    "item-b",
  ]
  Item-2: item-a
  Item-3: item-a

# work-around: no flow styles, only block styles?
Item:
  Item-1:
    - "item-a",
    - "item-b",
  Item-2: item-a
  Item-3: item-a

We might need to give up Prettier if we want to use linting for indentation...

Can prettier with the command --no-bracket-spacing do well in helm template?
If it could, we can change the link by using prettier --check --no-bracket-spacing and write a autoformat script using prettier

The one thing that prettier (and yamllint) fails to parse is the {{- -}} syntax, so it won't work with helm-charts/submarine/charts/traefik/templates/deployment.yaml for example.
Other than that, we could switch yamllint for prettier.

The one thing that prettier (and yamllint) fails to parse is the {{- -}} syntax, so it won't work with helm-charts/submarine/charts/traefik/templates/deployment.yaml for example.
Other than that, we could switch yamllint for prettier.

Got it.
@joshvictor1024 Would you handle the yq or other formatter that can format helm syntax?
Or is this PR just for a YAML linter?

It seems that yq doesn't and won't support helm syntax either. In its issue #636 it's mentioned that support is impossible.
I'll limit this PR to just a YAML linter. I'll make changes to submarine/submarine-cloud-v2/docs/developer-guide.md accordingly.

Upon discussion, we concluded that helm syntax support and auto-formatting are still desired. Can we put this PR on hold for now?

@joshvictor1024 OK, I will merge this PR until the helm syntax and auto-formatting are provided.


.Order() lowercases the query on Postgres

What version of Go are you using (go version)?
go 1.11 (docker: golang:1.11-alpine)
Which database and its version are you using?
Postgres 11 and SQLite3
Background:
I have replaced the NamingStrategy to create the columns with the same names as the struct fields:
gorm.AddNamingStrategy(&gorm.NamingStrategy{
	Column: func(name string) string {
		return name
	},
})

Problem:
Executing db.Order("CreatedAt desc") will correctly query SQLite, but on Postgres fails with the error:
pq: column \"createdat\" does not exist

Seems that .Order() is applying a lowercase transform for Postgres.
Is there a workaround?

Solved by adding quotes around the field name:
db.Order(""CreatedAt" desc")


Use ConfigmapsLeases Multilock for controller's leader election

This is a follow-up work to #3926 where we changed how leader election is done for cainjector , see the comment here and #4013 .
This PR changes the lock used for controller's leader election from ConfigMaps to a  ConfigMapsLeasesResourceLock Multilock so that leader election is done in the same way for both controller and cainjector.
From user's perspective, the difference will be the new Leases object created in the leader election namespace. This is not a breaking change.
We already modified controller's RBAC to allow it to access Lease objects in #3926 see this commit.
I have manually tested this change with a cert-manager deployment with 3 controller instances. I tested upgrade from v1.3.1 -> this version as well as upgrade from to this version followed by a downgrade, followed by an upgrade again and checked that the leader identities in the ConfigMap and Lease are as expected.
We don't test with multiple replicas in CI (I think).
fixes #4013
cert-manager controller now uses ConfigMapsLeasesResourceLock for leader election.

Signed-off-by: irbekrm<EMAIL_ADDRESS>/kind cleanup

/milestone v1.4

Is primary: configmap and secondary: lease the correct way around for the interim period?

It's the same as what client-go creates for a new lock ConfigMapsLeasesResourceLock here which is what we use in cainjector.
We might want to refactor controller's leader election to re-use more of the exisiting functionality I guess.

Or I can refactor it to use that New func quickly might be less ugly..

Yeah, it's a nit that results in exactly the same result, but I would prefer that

Yeah, it's a nit that results in exactly the same result, but I would prefer that
done


Any plan to change some variants of enum Event to struct variants?

The enum was made long time ago when there were no struct variants for enum to use.
Now that today we have struct variants I think we could do better.
And such code:
match poll_event() {
	Quit(_) | KeyDown(_, _, KeyCode::Escape, _, _, _) => break,
	_ => {}
}

can then be written as
match poll_event() {
	Quit(_) | KeyDown{ key_code: KeyCode::Escape, .. } => break,
	_ => {}
}

I really think this looks better.
Although it will break current code I believe it is worth trying.
I would submit a PR if there's no objection.

Nice. I'd agree that struct variants look better; no more guessing the correct order of parameters or what they do!
For consistency's sake, I think this should be applied to all Event enums, even the ones with single parameters (though you probably already had that in mind).

I agree. Be bold and make a PR :)


Infer outputs in tf

Some tf tests have been adjusted accordingly so that they hit the targeted code changes.