[AutoPR network/resource-manager] Adding VirtualWan Swagger

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3204

(message created by the CI based on PR content)

Codecov Report

Merging #2756 into restapi_auto_Network-2018-06-01 will decrease coverage by 0.03%.
The diff coverage is 0%.

@@                         Coverage Diff                         @@
##           restapi_auto_Network-2018-06-01    #2756      +/-   ##
===================================================================
- Coverage                            55.85%   55.82%   -0.04%
===================================================================
  Files                                 7293     7293
  Lines                               153449   153533      +84
===================================================================
- Hits                                 85707    85706       -1
- Misses                               67742    67827      +85

Impacted Files
Coverage Δ

...ork/azure/mgmt/network/v2018_02_01/models/probe.py
100% <ø> (ø)
:arrow_up:

...18_02_01/models/network_management_client_enums.py
100% <ø> (ø)
:arrow_up:

...azure/mgmt/network/v2018_02_01/models/probe_py3.py
100% <ø> (ø)
:arrow_up:

...rk/azure/mgmt/network/network_management_client.py
29.33% <0%> (-2.66%)
:arrow_down:

Continue to review full report at Codecov.

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


Credit for the idea of Assigning Node Metadata to Pods

Added credit for the idea of using kyverno for "Assigning Node Metadata to Pods"
Following my idea to use kyverno to add node metadata to pods using kyverno policy in the kyverno slack thread, including implementing the first POC

I'm not the author of this blog post, deferring it to @chipzoller .

@chipzoller Can you approve and merge?


feat(build): add shell script to invoke Discord webhook

What this PR changes/adds
Adds a script, that is executed by a dedicated Jenkins Job called Send Discord Webhook.
Why it does that
Notifications in Discord about succeeding/failing Jenkins Jobs
Further notes

The Job in question is in turn triggered by upstream SNAPSHOT builds, e.g. the EDC-Snapshot
I put this file in the EDC repo, because the "Send Discord Webhok" jenkins job can then checkout the repo and use the script.
I opted against dedicated jenkins plugins because having a script using the Discord API is likely quicker, more controllable and could be re-used, e.g. from Github Actions

Currently this is only enabled for snapshot builds
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and Etiquette for pull requests for details)

Codecov Report
Base: 64.08% // Head: 64.08% // No change to project coverage :thumbsup:

Coverage data is based on head (0b046e4) compared to base (9b20768).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2230   +/-   ##
=======================================
  Coverage   64.08%   64.08%
=======================================
  Files         822      822
  Lines       17233    17233
  Branches     1073     1073
=======================================
  Hits        11043    11043
  Misses       5750     5750
  Partials      440      440

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Speed-up Front-end part

Currently the Front-end assets doesn't have any structure. I want to propose some points.

In the master folder we will have a folder named assets/
assets/ folder will contain 2 folder named dist/ and production/
production folder will contain all the necessary assets needed during production time. Like: unminified css, js and images
dist/ folder will contain only distributed assets. Every asstes will be minified and concatenated.
These minification, concatenation and compression will be done by automated system like Gulp.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Hey @choyan. I use gulp daily in dev and production for @system76, but it's probably overkill for this project at this point. A huge reason we're structuring the site as we are is to significantly lower the barrier of entry. Adding preprocessing raises that barrier extremely high for little tradeoff of a project this size.

That said, if you notice assets that could be better optimized, feel free to file bugs.


Cache the output of open source models

We currently cache the results of requests to the OpenAI API. We should do the same thing with OSS models. The cache would only be effective if the same rng is passed at each call.

Hey @rlouf,
I was looking around in the codebase for this, and had question in terms of what do we want to cache for OSS model? Do we want to cache the sampled output or just the logits that are outputted by the model? The logits would be quite large so ideally we shouldn't be doing that.
Also, just to confirm the if cache behaviour should be similar to openAI API call where we cache an entire generation at once instead of caching tokens (which I thought would be the way given current impl)?

Hi @rlouf ,
I am almost done with this. Will open a draft PR in a couple of days.


Is it possible to render in SVG?

This is basically a question,
I have seen PDF js render to SVG instead of Canvas. Is this feasible with vue-pdf as well? If so, how can I do it?
Thanks in advance!!!

I don't know the status of PDF.js SVG backend. I will integrate it as soon as it is ready.

Have you looked into this??
svgviewer.


Css linters errors

make sure to not use capital letters when attributing a color to an element

plz change the color code from capital letters to lowercase to avoid linter errors


windows 11 network config

Describe the issue
install windows 11 , network config step "let's connect to network" doesn't have network option or show input network（look at the picture below） , did i config something wrong , thanks!
Configuration

UTM Version: 3.1.5
OS Version: macOS Monterey 12.2.1 (21D62)
Intel or Apple Silicon?
Apple Silicon

Try this

I added the VM tools ISO, booted the VM up, used Shift+F10 to navigate to the D: drive and run the installer, and then installed it. Rebooted again and the installer was able to connect to the internet using the drivers that were installed.

@

Try this

thank you, i finnaly success by your reply!

Try this

Those instructions are for an older version of the Windows installer and not applicable to more recent technical previews of of Windows 11 ARM being released by Microsoft.
Specifically, as of March 28, 2022, it's near impossible to get past the show stopping "lack of network" error page when installing e.g. Windows11_InsiderPreview_Client_ARM64_en-us_22579.VHDX. The Windows installer doesn't seem to offer an offline option anymore - thus the official UTM Windows 11 ARM installation instructions on Apple Silicon are incomplete and will fail, because networking does not work in UTM during the installation stage.

I added the VM tools ISO, booted the VM up, used Shift+F10 to navigate to the D: drive and run the installer, and then installed it. Rebooted again and the installer was able to connect to the internet using the drivers that were installed.

Like @computerwizjared, I figured out the same way to enable networking during the installation. The steps in more detail are

Follow the official UTM Windows 11 ARM installation instructions but before running the vm, do the following
Download the spice guest tools iso
In UTM vm settings add a drive, browse to the spice ISO, change the drive type to CDROM.
Run the vm and begin the installation process
Optionally hold ESC as you boot the vm to get into the BIOS.  Then select Device Manager → OVMF Platform Configuration, Change Preferred Resolution for Next Boot to be 1024x768, save and exit out to the top level bios menu and hit CONTINUE
When stuck at no networking available, hit SHIFT F10 to get a windows command prompt, type D: then run spice…exe which installs the networking drivers - do not reboot. ALT-Tab to get back to the installer - networking will come alive and you will be able to proceed with Windows 11 installation.

Arguably the official instructions should be updated with the above steps, unless someones has a better solution.

How do i press SHIFT F10 from a Mac Keybaord?
fn + SHIFT + F10

Try this

Those instructions are for an older version of the Windows installer and not applicable to more recent technical previews of of Windows 11 ARM being released by Microsoft.
Specifically, as of March 28, 2022, it's near impossible to get past the show stopping "lack of network" error page when installing e.g. Windows11_InsiderPreview_Client_ARM64_en-us_22579.VHDX. The Windows installer doesn't seem to offer an offline option anymore - thus the official UTM Windows 11 ARM installation instructions on Apple Silicon are incomplete and will fail, because networking does not work in UTM during the installation stage.

I added the VM tools ISO, booted the VM up, used Shift+F10 to navigate to the D: drive and run the installer, and then installed it. Rebooted again and the installer was able to connect to the internet using the drivers that were installed.

Like @computerwizjared, I figured out the same way to enable networking during the installation. The steps in more detail are

Follow the official UTM Windows 11 ARM installation instructions but before running the vm, do the following
Download the spice guest tools iso
In UTM vm settings add a drive, browse to the spice ISO, change the drive type to CDROM.
Run the vm and begin the installation process
Optionally hold ESC as you boot the vm to get into the BIOS.  Then select Device Manager → OVMF Platform Configuration, Change Preferred Resolution for Next Boot to be 1024x768, save and exit out to the top level bios menu and hit CONTINUE
When stuck at no networking available, hit SHIFT F10 to get a windows command prompt, type D: then run spice…exe which installs the networking drivers - do not reboot. ALT-Tab to get back to the installer - networking will come alive and you will be able to proceed with Windows 11 installation.

Arguably the official instructions should be updated with the above steps, unless someones has a better solution.

This works great Abulka, just one small hint for those that are pressing Shift+10 and nothing is happening. The prompt window is behind the installation, so you need to use Alt+TAB to show it and install the spice tools as the instructions above.


ci: fix unix slashes in run command

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


SUBMARINE-763. Support uploading a conda config in environment page

What is this PR for?
Support uploading conda config file interface in environment page
What type of PR is it?
[Feature]
Todos

[ ] - Support the URL to upload the file.
[ ] - Improve preview appearance.

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-763
How should this be tested?
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@KUAN-HSUN-LI @pingsutw Maybe we can move the upload and preview area to the next page?

@pingsutw @kobe860219 The "Kernel Spec" which is below will be removed.
Or, maybe just put the preview inside the uploading area, and limit the height with a scroll bar.

@KUAN-HSUN-LI The current PR is okay with me.
No need to change. Thanks.


Instruction delegation

Currently, we have no way to "invoke" an instruction from another instruction. This is especially important when an instruction uses a permutation, since we cannot just duplicate the called instruction's code in the calling instruction.
To solve this, the following syntax is proposed (the memory example is bad, because the memory machine can probably not be implemented like that, but also see the binary example below):
instr madd X, Y -> Z: mop, WVAL <== wrap(RVAL1 + RVAL2) {
    RADDR1 = X,
    RADDR2 = Y,
    WADDR = Z,
}

instr mop {
    { RADDR1, RADDR2, WADDR, STEP, RVAL1, RVAL2, WVAL } is 1 { m_raddr1, m_raddr2, m_waddr, m_step, m_rval1, m_rval1, m_wval }
}

Here, the madd instruction calls both the mop and the wrap instruction. In doing this, we can read two values from memory, add them (including wrapping the result) and write the result to memory again in the same step. This allows us to even use the same memory address for the written value since the memory machine knows that in one step, the write always comes last.
Since the mop instruction does not have parameters, it is invoked as is. The wrap instruction has parameter and thus also needs two additional assignment registers different from X, Y and Z. In assembly-to-pil conversion, whenever we set the flag for madd, we also set the flags for map and wrap (note that this could create problems with the "else" condition in the register update constraint).
As an alternative, we could just allow the instructions in the body of the instruction definition, but I'm not sure how this would later work with translation via PIL.
Binary example:
instr add X, Y -> Z: binary(1, X, Y) {
}

instr mul X, Y -> Z: binary(2, X, Y) {
}

instr binary op: integer, X, Y -> Z {
    { op, X, Y, Z } in { bin.op, bin.A, bin.B, bin.C }
}

What would the binary example compile to? I can think of 3 ways:
1:
instr_add { 1, X, Y, Z } in bin.LATCH { bin.op, bin.A, bin.B, bin.C }
instr_mul { 2, X, Y, Z } in bin.LATCH { bin.op, bin.A, bin.B, bin.C }

(Like now, just a different way to write it)
2:
(instr_add + instr_mul) { instr_add * 1 + instr_mul * 2, X, Y, Z } in bin.LATCH { bin.op, bin.A, bin.B, bin.C }

(Saves one lookup)
3:
instr_binary { binary_op, X, Y, Z } in bin.LATCH { bin.op, bin.A, bin.B, bin.C }

(Where instr_binary & instr_op come from the instruction lookup, reduces the number of witness columns in the main machine (for more than 2 operations), this is also what Polygon does).
I think we should aim for the 3rd translation! But is this easy to infer? If not, it might be better to have a more direct syntax?

I had the last option in mind and don't think it's complicated.

the memory example is bad, because the memory machine can probably not be implemented like that

Let me try to fix it:
instr madd addr_arg1, addr_arg2 -> addr_res: mload_arg1, mload_arg2, mstore_res, val_res <== wrap(val_arg1 + val_arg2) {}

instr wrap wrap_Y -> wrap_X { wrap_Y = wrap_X + wrap_bit * 2**32, wrap_X = X_b1 + X_b2 * 0x100 + X_b3 * 0x10000 + X_b4 * 0x1000000 }
instr mload_arg1 addr_arg1 -> val_arg1 ~ memory.mload addr_arg1, STEP -> val_arg1;
instr mload_arg2 addr_arg2 -> val_arg2 ~ memory.mload addr_arg2, STEP -> val_arg2;
instr mstore_res addr_res, val_res -> ~ memory.mstore addr_res, STEP, val_res ->;

This would compile to 4 permutations (1 for the wrap call + the 3 memory permutations). The memory operations need to be separate permutations, because they happen at the same time, but they can be re-used by other instructions. This would work with the current memory machine in STD, as long as the input and output addresses are different.
Did I understand this right?
One question I have about this: Does that mean that we allow for assignments between assignment registers? I this case, we'd have to set wrap_Y = val_arg1 + val_arg2, all of which are assignment registers. Is this feasible? What if there is a chain of 5 instructions that are executed, wouldn't the number of necessary assignment registers explode?

One question I have about this: Does that mean that we allow for assignments between assignment registers? I this case, we'd have to set wrap_Y = val_arg1 + val_arg2, all of which are assignment registers. Is this feasible? What if there is a chain of 5 instructions that are executed, wouldn't the number of necessary assignment registers explode?

How about this:
instr madd addr_arg1, addr_arg2, addr_res ->: mload_arg1, mload_arg2, mstore_res {
    val_res = val_arg1 + val_arg2
}
instr mwrap addr_arg1, addr_res ->: mload_arg1, mstore_res, wrap {}

instr wrap val_arg1 -> val_res { val_arg1 = val_res + wrap_bit * 2**32, val_res = X_b1 + X_b2 * 0x100 + X_b3 * 0x10000 + X_b4 * 0x1000000 }
instr mload_arg1 addr_arg1 -> val_arg1 ~ memory.mload addr_arg1, STEP -> val_arg1;
instr mload_arg2 addr_arg2 -> val_arg2 ~ memory.mload addr_arg2, STEP -> val_arg2;
instr mstore_res addr_res, val_res -> ~ memory.mstore addr_res, STEP, val_res ->;

So the program has to run two instructions instead of one, but there are a lot fewer columns. I think allowing something like val_res <== wrap(val_arg1 + val_arg2) introduces complexity we might not need (i.e., allowing to assign from one assignment register to the other).

As discussed with @pacheco and @chriseth, as an alternative to instruction delegation, we can support links inside instruction definitions. To rewrite Chris' example:
instr add X, Y -> Z {
    link 1 ~> bin.add X, Y -> Z
}
instr sub X, Y -> Z {
    link 1 ~> bin.sub X, Y -> Z
}

// Step 1: Make all constraints conditional on the instruction being active
link instr_add ~> bin.add X, Y -> Z
link instr_sub ~> bin.sub X, Y -> Z

// Step 2: Expand links
instr_add {0, X, Y, Z } is <sel> {bin.operation_id, bin.arg1, bin.arg2, bin.res};
instr_sub {1, X, Y, Z } is <sel> {bin.operation_id, bin.arg1, bin.arg2, bin.res};

// Step 3: Combine permutations / lookups if RHS is the same
(instr_add + instr_sub) {0 * instr_add + 1 * instr_sub, X, Y, Z } is <sel> {bin.operation_id, bin.arg1, bin.arg2, bin.res};

Implemented in https://github.com/powdr-labs/powdr/pull/1439


Preferences should be scraped and copied

Potentially use a different script since the name of this is "resub" but in the use-case of copying subscribed subreddits to a new account I can't see why one would not want their permissions copied too.

This isn't really viable.


npm start error after update vs code and node last versions

Hi, with older versions of node and VScode, no problem, but when i updated these versions and  try to run the app, breaks with this error:
[dev] There might be a problem with the project dependency tree.
[dev] It is likely not a bug in Create React App, but something you need to fix locally.
[dev]
[dev] The react-scripts package provided by Create React App requires a dependency:
[dev]
[dev]   "jest": "26.6.0"
[dev]
[dev] Don't try to install it manually: your package manager does it automatically.
[dev] However, a different version of jest was detected higher up in the tree:
[dev]
[dev]   C:\video3\node_modules\jest (version: 26.6.3)

after updates, i remove node_module folder and package-lock.json and  run npm install, sends me:
PS C:\video3> npm install
npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition. dotenv provides its own type definitions, so you do not need this installed.
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>flatten is deprecated in favor of utility frameworks such as lodash.
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features added
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/address'
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Switch to 'npm install joi'
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact Support at https://www.npmjs.com/support for more info.
npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/system. See the guide: https://mui.com/guides/migration-v4/
npm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.
npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/styles. See the guide: https://mui.com/guides/migration-v4/
npm WARN deprecated<EMAIL_ADDRESS>Version no longer supported. Upgrade to @latest
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.
npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/material. See the guide: https://mui.com/guides/migration-v4/
npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/icons. See the guide: https://mui.com/guides/migration-v4/

><EMAIL_ADDRESS>postinstall
> rimraf public/virtualbackground && copyfiles -f node_modules/@twilio/video-processors/dist/build/* public/virtualbackground

added 2761 packages, and audited 2762 packages in 8m

  run `npm fund` for details

64 vulnerabilities (4 low, 45 moderate, 13 high, 2 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

What i can do?

Sorry, i have not used corrrectly packacge.json,
Thanks


Set error level psalm to 1.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues

closed in favor https://github.com/yiisoft/db/pull/294


Loosen version restrictions

lower minimum Python supported to 3.8
removed poetry.lock

prevented installation of library in existing envs without serious corruption
see https://python-poetry.org/docs/basic-usage/#as-a-library-developer

Verified all tests and pre-commit hooks pass
Tested on various Python versions on Windows, Ubuntu, macOS

See also: bubblesub/ass_parser#1, bubblesub/ass_renderer#3


Update CodePen examples to use latest React

IIRC currently they don’t. We should probably leave them using latest since if it breaks, we have to update them anyway.
Edit This issue is likely made unnecessary by the outcome #122

Auto-assigning to Dan since he owns most of the Codepens in question.

how can people who dont own these codepen's help? i am happy to reconstruct, just assign stuff to me

There are lots of issues tagged "good first issue". If you're interested in helping with the website, I recommend grabbing one of them! 😄 This particular issue is easier for Dan to handle which is why I've pre-assigned it to him.

This can be done as part of the Codepen-to-embedded migration now that we can link to embedded examples so I'll take this one on.

Hello @bvaughn!
I believe this issue is already outdated and can be safely closed, as now the Sandpack is being successfully used and the versioning matter is solved.
Please correct me if I'm somehow wrong, otherwise we can close this.

Yeah https://github.com/reactjs/react.dev/issues/122 closes this

hello friends happy 2024

Hiiiii :)


Allow direct access to ICacheEntry and make it thread safe

Currently all methods that return a value from the cache (most importantly TryGetValue) directly return the underlying object. This is convenient but does not allow for a scenario of updating an entry's expiration after it has been added. Even if I retain a reference to the original ICacheEntry returned from CreateEntry it won't be good enough as no memory barriers are raised when accessing its properties (so I might change the expiration time in my thread, but your StartScanForExpiredItems thread won't observe them).

Which expiration are you specifically trying to change and why? AbsoluteExpiration, AbsoluteExpirationRelativeToNow, SlidingExpiration, or ExpirationTokens?

I'm proposing to allow all types of expirations to be set after the value has been added to the cache.
As for why, the entry's expiration date might be a function of the value. I want to be able to do this:
public Task<T> GetOrAddAsync<T>(
    string key,
    Func<Task<T>> factory,
    Action<T, ICacheEntry> expirationSetter,
    LazyThreadSafetyMode mode)
{
    var lazy = (Lazy<Task<T>>) _cache.GetOrCreate(key, ce => new Lazy<Task<T>>(async () =>
        {
            var value = await factory();
            expirationSetter(value, ce);
            return value;
        }, mode));
    return lazy.Value;
}

For this to work well, I'd need https://github.com/aspnet/Caching/issues/240 too.

I think something like this could be broken too:
public Task<T> GetOrAddAsync<T>(
     string key, Func<Task<T>> factory, Action<ICacheEntry, T> expirationSetter)
{
    return _cache.GetOrCreateAsync(key, async cacheEntry =>
    {
        var value = await factory().ConfigureAwait(false);
        expirationSetter(cacheEntry, value);
        return value;
    });
}

Because after the await you might be in a different thread whose changes are not observed by the GetOrCreateAsync thread that created the cache entry (and would later put it in the cache dictionary).

Have you tried using a custom IChangeToken that wrapped your value and managed its timeout internally?
https://github.com/aspnet/Common/blob/52c69ec4d9c0e3de40da092e3721ace0ce0ac489/src/Microsoft.Extensions.Primitives/IChangeToken.cs

Interesting, this could work:
public Task<T> GetOrAddAsync<T>(
    string key, Func<Task<T>> factory, Func<T, TimeSpan> expirationCalculator)
{
    return _cache.GetOrCreateAsync(key, async cacheEntry =>
    {
        var cts = new CancellationTokenSource();
        cacheEntry.AddExpirationToken(new CancellationChangeToken(cts.Token));
        var value = await factory().ConfigureAwait(false);
        Task.Delay(expirationCalculator(value)).ContinueWith(t => cts.Cancel());
        return value;
    });
}

However this won't allow for sliding expirations. Also, https://github.com/aspnet/Caching/issues/240 would still be required to guarantee the factory is only called once.

Oh, I was thinking more along the lines of:
    return _cache.GetOrCreateAsync(key, async cacheEntry =>
    {
        var value = await factory().ConfigureAwait(false);
        var token = new CustomChangeToken(value);
        cacheEntry.AddExpirationToken(token);
        return value;
    });

public class CustomChangeToken : IChangeToken
{
  public CustomChangeToken(MyType value) { }
  public bool HasChanged { get => value.HasExpired }
  ...
}

Sliding expiration would only work if you kept pinging something on the value each time you used it. E.g. value.ExtendLifetime().

Yes, if you need custom expiration logic that might make more sense, but I reckon using the existing CancellationChangeToken rather than rolling your own would be simpler in most cases.
Regardless, I am a bit concerned about the thread safety of the code you posted. You operate on cacheEntry from a different thread than the one it was created in, so I don't believe changes made on it after the await are guaranteed to be observed by the creating thread without any memory barriers raised. Specifically you call AddExpirationToken which boils down to List<t>.Add which does not raise memory barriers as far as I can tell.

Fair point, I was assuming behavior of the existing synchronous extensions like GetOrAdd, not the async/lazy versions you've defined.


Change from creating a PR to creating an issue when there is a close match in a license submittal

PR #128 implements a feature where if there is a close match, the user can submit a pull request to update the XML to allow for the match.
After trying this out on my local machine, it occurred to me that many users may not have the technical skills or interest in creating the XML with the proper <optional or <alt... tags.  We could instead create an issue with a title "Update [licenseID] to allow matching to the included text".  The text of the license and/or the diff could added as a comment or as an attachment to the issue.
@jlovejoy @swinslow - let me know what you think.

Sounds great to me :)


[SP-6283] - Backport of PPP-4815 - Vulnerable Component: Ini4j 0.5.1 (9.4 Suite)

@andreramos89
@bcostahitachivantara
@smmribeiro
Original PR:

https://github.com/pentaho/pentaho-kettle/pull/8769

:x: Build failed in 1m 17s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

:x: Build failed in 2h 15m 13s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message


replies to replies not sent from my blog to the fediverse

So I produce an article on my blog, and it does to the fediverse.
Someone replies to my article on the fediverse, and the reply also goes on my blog. Yay!
Now, when I reply to this person on my blog, I see the reply on my blog, but nothing is sent to the fediverse. Bummer!
I'd don't see my last reply and neither the person I replied to does. (Yes, I asked. I wanted to make sure nothing unexpected was happening.)
Version of the plugin 0.17.0
Version of WP 6.3
I can add more if needed.

Sorry, but this is not implemented yet!

Yikes... and I guess there's no way to prevent people from replying to the post that my blog puts out every time I publish a new article.

@YourAutisticLife There isn't any agreed ActivityPub method yet to prevent or control who can reply to a post.
The choice to include those replies and display them as comments on your site is yours though.

Is there any time frame on when this feature might be added?

We are actually working on it https://github.com/Automattic/wordpress-activitypub/pull/142 :)

It's one of the focus topics for the next iteration!

Awesome!  Thank you!

Is there a way to reply to replies, from your blog? Like, if comments don't federate ... can we post a new post and mention the other (Mastodon) account and maybe even link to the status (maybe using the (u-)in-reply-to microformat or something, just thinking out loud) and have the person receive a notification?

give us some more days and it will work out of the box!

You can try the latest version here on GitHub!

It does work in the latest version.... great!!!
However, the behavior is confusing. I replied to a post of mine. I have two accounts: one tied to my WordPress instance, and one tied to a Mastodon instance of mine. The scenario is as follows:

An article is posted to the fediverse by my WordPress instance.
I commented on it with my Mastodon account.
Then I went into the comment page of my article (in number 1) and replied to the reply in number 2.

I saw the reply but I was confused that I did NOT get a notification for it in Mastodon. I looked at the post closer and I noticed that the reply does not automatically @-mentions the person that you are replying to.
If I add the @-mention manually, then I get a notification.
Is it possible to pre-populate the reply field with the handle of the person I'm replying to (and actually all the other user handles used in that post, like Mastodon does)?

This is a fair expectation for the vast majority of ActivityPub software users.

Should work now!


[feature] functionality to execute the task as django-celery-beat does:

Suppose we schedule the task to execute for every 1 hour and we created the schedule task at 2PM

Now  Next Run would be Feb. 24, 2020, 3:00 p.m. , Let's imagine we kept the Scheduler
redisbeat.RedisSchedule off for 4 hours.

And our time would be now "7 PM" , As soon as we start our redisbeat.RedisSchedule again , i believe it should execute the missed task immediately for only one time,since our schedule task was in the past and has missed the intervals and set the next run time automatically scheduled.

It seems like a good case, i will try to do it in these days.

I am doing a patch to add serval init policy for redis beat, however it may need some times to test.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
xiaohui, @leezhenghui, @yzwdroid, @smilelikeshit, @deanhigh, @mfykmn, @TakatoHano, @JackTroy, @SeananXu, @raakasf, @bnour-corrae, @weibaohui, @cnych, @arusso, @cnutshell, @ZeroExistence, @kmdkuk, @guozenghuang, @xiazemin, @frittentheke


Working with React 16 docs don't match the Readme

Heyo! First, thanks for all you do. I ❤ Enzyme, and I appreciate you maintaining it. 😄
Overview
The docs here on AirBnB do not seem to match the docs in the README here. I assume they just need to be pushed up.
Instead of enzyme-adapter-react-16 it says enzyme-react-adapter-16.
Your error message:
          Enzyme Internal Error: Enzyme expects an adapter to be configured, but found none. To
          configure an adapter, you should call `Enzyme.configure({ adapter: new Adapter() })`
          before using any of Enzyme's top level APIs, where `Adapter` is the adapter
          corresponding to the library currently being tested. For example:

          import Adapter from 'enzyme-adapter-react-15';

          To find out more about this, see http://airbnb.io/enzyme/docs/installation/index.html

pointed me to this doc, which is how I found it.
Screenshots
README

AirBnB Site Docs

It's a pretty small thing and should be a quick fix. 😄 Thanks again for all you do!

+1

The docs are fixed in #1142.
@Rob-Leggett no, import * as is both incorrect here, and never necessary. The adapters have a single default export; import * as provides a ModuleRecord object with a ".default" property.

Correct, the docs are fixed in the README. But the site docs are not (as of now).
http://airbnb.io/enzyme/docs/installation/index.html#
This is still there:
npm i --save-dev enzyme enzyme-react-adapter-16

Thanks again for working on this.

That won't be fixed until the next release.

Sounds good. 👍 Appreciate your time!

There is a typo on the docs, this is the correct name for the package https://www.npmjs.com/package/enzyme-adapter-react-16


Traefik seems to watch too many rules files with file based config and watch=true

Do you want to request a feature or report a bug?
Possible bug, or undocumented feature as I was unable to find anything related to this case...
What did you do?
Setup an extremely basic traefik instance with file based and watched config.
# Some defaults, stripped...

traefikLogsFile = "/var/log/traefik/traefik.log"
accessLogsFile = "/var/log/traefik/access.log"

# The usual entry points, stripped...

[file]
filename = "/etc/traefik/rules.toml"
watch = true

Make a backup of /etc/traefik/rules.toml
$ cd /etc/traefik/
$ cp rules.toml rules.toml-bak.20170724-132846

Check the logs:
$ cat /var/log/traefik/traefik.log

time="2017-07-24T15:30:02+02:00" level=debug msg="File event:\"/etc/traefik/rules.toml-bak.20170724-132846\": CREATE"
time="2017-07-24T15:30:02+02:00" level=debug msg="File event:\"/etc/traefik/rules.toml-bak.20170724-132846\": WRITE"

What did you expect to see?
Nothing at all in the logs as the file watched should be the one as set in the config...
What did you see instead?
It appears that traefik is watching any file in /etc/traefik that starts with rules.toml?
Ubuntu 16.04, TOML config:
graceTimeOut = 10

debug = true

checkNewVersion = false

traefikLogsFile = "/var/log/traefik/traefik.log"
accessLogsFile = "/var/log/traefik/access.log"
logLevel = "ERROR"

ProvidersThrottleDuration = 5
MaxIdleConnsPerHost = 200
InsecureSkipVerify = true

defaultEntryPoints = ["http", "https"]

[entryPoints]
  [entryPoints.http]
  address = ":80"
  # The containers handle this!
  compress = false

[healthcheck]
interval = 30

[web]
address = ":1936"
#CertFile = "traefik.crt"
#KeyFile = "traefik.key"
ReadOnly = true
[web.statistics]
  RecentErrors = 10
[web.auth.basic]
  users = ["test:$apr1$H6uskkkW$IgXLP6ewTrSuBkTrqE8wj/", "test2:$apr1$d9hr9HBB$4HxwgUir3HP4EsggP/QNo0"]
#  usersFile = "/path/to/.htpasswd"

[file]
filename = "/etc/traefik/rules.toml"
watch = true

If applicable, please paste the log output in debug mode (--debug switch)
time="2017-07-24T15:30:02+02:00" level=debug msg="File event:\"/etc/traefik/rules.toml-bak.20170724-132846\": CREATE"
time="2017-07-24T15:30:02+02:00" level=debug msg="File event:\"/etc/traefik/rules.toml-bak.20170724-132846\": WRITE"

In provider/file/file.go I can see this here:
// Provider holds configurations of the provider.
type Provider struct {
	provider.BaseProvider `mapstructure:",squash"`
	Directory             string `description:"Load configuration from one or more .toml files in a directory"`
}

// Provide allows the file provider to provide configurations to traefik
// using the given configuration channel.
func (p *Provider) Provide(configurationChan chan<- types.ConfigMessage, pool *safe.Pool, constraints types.Constraints) error {
	configuration, err := p.loadConfig()

	if err != nil {
		return err
	}

	if p.Watch {
		var watchItem string

		if p.Directory != "" {
			watchItem = p.Directory
		} else {
			watchItem = p.Filename
		}

		if err := p.addWatcher(pool, watchItem, configurationChan, p.watcherCallback); err != nil {
			return err
		}
	}

	sendConfigToChannel(configurationChan, configuration)
	return nil
}

But I have setup a file, not a directory... So is this some kind of default that can be disabled perhaps...? Am I missing documentation concerning this behavior?
Or is there a problem with 'extension handling' as the new file does not end in .toml and therefore should not be watched?
Watching an entire path is great for the situation I'm in, but the behavior is not at all clear :-s

The code you quote makes a difference between a directory and a file.
This code is the next version of Traefik (1.4.0).
I advise you to test with version 1.3.3.

In the previous versions (before the next version 1.4.0), we watch all files which contains /etc/traefik/rules.toml, e.g. /etc/traefik/rules.toml-bak.20170724-132846 contains  /etc/traefik/rules.toml (+ -bak.20170724-132846).
workaround: add a prefix instead of a suffix.
This is  fixed in the next version (1.4.0).

Thanks for this clarification! Just make sure to mention this behaviour somewhere in the docs ;-)


Add a page for example projects

It would be nice to have a page where some example projects of various types are displayed. For example, it would be handy to have examples for web apps, CLI utils, etc.
I usually find this quite useful when I'm learning a new language observing "well designed" projects to see how they're implemented.

I found a somewhat decent source here.

We can use this page as a starting point: https://www.lri.fr/~filliatr/software.en.html

Great idea. Probably makes sense to put it under Learning.

@kalouantonis Go for it!

I'm on it, PR coming soon...


Added backers and sponsors from OpenCollective

So they automatically update on your repo. :)

Thanks! Closing in favor of #87 by Xavier


isTitleCase checking snippet

Hello, 30 Seconds code.
I think there is no snippet available for isTitleCase checking. Can I add a snippet for this?

No, thanks. This wouldn't be very valuable to the average reader and you can always compare the result of toTitleCase() with the original string to get the same result.


feat: local scene dev landscape

Issue: https://github.com/decentraland/unity-explorer/issues/1656

Re-enables the landscape for Local Scene Development.
Modify worlds dynamic terrain generation to also run during local scene development (by reading the new 'local scene parcels' collection introduced in the /about response from the sdk runtime for local scenes
Updated ServerAbout + ServerConfiguration to be able to read the new collection field from the /about response.
Updated IRealmData and RealmData to transport the new local scene parcels collection for its usage during dynamic terrain landscape generation
Updated FixedScenePointers + LoadFixedPointersSystem to support "non URN fixed scenes", as the local scenes don't have a urn identifier, we just get their occupied parcels.
Updated IGlobalRealmController, RealmController, RealmNavigator so that the fixed scenes async await returns a SceneEntityDefinition directly, instead of a promise. Since promises to get the SceneEntityDefinition for "urn scenes" and for "pointer scenes" are different and are handled internally.

https://github.com/user-attachments/assets/5e4c0b0f-43d9-495a-9299-87a33cc21869

Related PRs

https://github.com/decentraland/js-sdk-toolchain/pull/1013

TEST INSTRUCTIONS

Clone/Download any scene; For example the sdk7 template scene
Install the SDK's next package running npm i @dcl/sdk@next in the root folder of the scene.
Run npm run start -- --explorer-alpha to start the scene
Kill the launcher/explorer that will auto-open, but leave the server running in the console
Download the build from this PR
Open the build from this PR connecting it to the locally running scene (WIKI steps for that, remember to put the correct scene position in the console command)
Finally: confirm that the landscape is enabled and it is limited around the scene (with a couple of parcels margin) as if it were a world

Closing this PR in favor of https://github.com/decentraland/unity-explorer/pull/2316, since having the local scene parcels in the staticLoadPositions makes everything way easier for the setup of the landscape dynamic generation


Add workflow to synchronize go mod changes with submodules

Summary
This PR attempt to improve the workflow of managing Dependabot updates to this repo. The problem is that when Dependabot updates a dependency in the main module, those updates are not reflected in the "submodule" (https://github.com/sigstore/sigstore-go/tree/main/examples/oci-image-verification) which includes a reference to the main module. That causes the build to fail, and one maintainer must manually run go mod tidy in the submodule and push those changes, and another maintainer must approve and merge them (as the last committer is disallowed from merging a PR). (example)
This PR automatically runs go mod tidy in both modules and commits/pushes them whenever Dependabot creates a new PR.
There is a caveat with this, which is that the actions do not run when the last committer used GITHUB_TOKEN to authenticate to the Git server, which is intentional to avoid accidentally creating recursively invoked workflows. This can be worked around by using a PAT in this workflow to authenticate to the Git server, or maintainers can push an empty commit to trigger the Actions workflows.
If this seems like a good approach to fellow maintainers, I'd recommend we create a PAT for use within this repo and modify this workflow to use it.
This workflow could also be added to sigstore/sigstore for use with the KMS providers (example).
Release Note

Documentation

I think it may even work to just re-run the failing workflows from the UI instead of pushing an empty commit?

I don't think there's a way to trigger the workflows from the UI if they are not run due to the last commit originating from Actions -- they are not actually "failed" workflows, they just aren't created at all. Maybe I'm missing something, but I think it would require an empty commit to trigger the build, unless we add a PAT to a secret and use it to commit the go.mod changes.
@haydentherapper (or maybe @cpanato?), what do you think of incorporating this workflow and using a PAT to authenticate with git? As explained in the description, we need to use a PAT or the actions-generated commit will not trigger actions to run again, therefore the automation would be incomplete. We can create an org-level PAT secret and also use it to keep the kms providers in sigstore/sigstore up to date using the same workflow. Somebody who is org admin would need to create the PAT and upload it as an org-level secret.

I really like this workflow, this is great! This has definitely been a nuisance here and in Sigstore/sigstore.
PATs are a little annoying to keep up to date, but we’ve already got a number that we manage, one more is fine. In the future, we can look at creating a GitHub app to replace PATs.
what are the permissions needed for the PAT?

PATs are a little annoying to keep up to date, but we’ve already got a number that we manage, one more is fine. In the future, we can look at creating a GitHub app to replace PATs.

Yeah an app is ideal, just requires a little more setup. PATs are indeed annoying.

what are the permissions needed for the PAT?

It can be a fine-grained permission PAT with support to the specific repos we need (sigstore, sigstore-go), with "Contents" set to "Read and write".

@cpanato  can you create the PAT?

Coming back to this, I can create the PAT...but with that said, keeping these PATs up to date is a bit annoying as they expire. Is there any way to do this without another PAT?

We had also looked into https://github.com/apps/octo-sts awhile ago which Chainguard made.

sorry for the delay here
we can create a PAT, but i would vote to use the octo-sts, because we dont need to manage the PAT and that will be valid only for the job execution

and this looks soo cool!!

octo-sts is awesome, I just tried it on a test repo and it works great for this purpose. I'll update the PR

We'll need to install it in the org as well. Should mention this to the TSC before we progress.

I updated the workflow to use octo-sts, so if TSC approves, we can proceed to merging this.

@codysoyland Can you file an issue on https://github.com/sigstore/tsc asking for approval?

@codysoyland Can you file an issue on https://github.com/sigstore/tsc asking for approval?

https://github.com/sigstore/TSC/issues/51


What's the difference between Test-Agnostic LT and Out-of-Distribution Generalization?

Hi,
As a researcher in the long-tailed recognition field, recently, when I started to work on OOD generalization, I found that the long-tailed problem seems to be a special case of OOD generalization, whose definition is quite similar to your test-agnostic LT task. Could you explain their differences? because I'm quite curious about the performance of the OOD algorithms in LT tasks.
Best,

Hi, Kaihua,
Thanks for your comments. I think your method in NeurIPS'20 is interesting and insightful, so I also use it as a baseline in my work.
In my view, OOD generalization is very important for DNNs, which is a crucial factor for the deployment of DNNs to real-world applications. From the perspective of concept, OOD generalization is not a specific task. Instead, in my mind, it particularly indicates the situations that training distributions and test distribution are inconsistent, including inconsistent data marginal distributions (e.g., domain adaptation, domain generalization), inconsistent class distributions (e.g., long-tailed recognition, open-set classification), and the combination of the previous two situations (e.g., VQA for sometimes, automatic drive). In comparison, test-agnostic long-tail (LT) in this work is inspired by the conventional LT (typically assuming test class distribution is uniform) and real-world scenarios (test class distributions may not be uniform, e.g. in the automatic drive). Exploring test-agnostic LT can further enhance the practicability of LT methods.
In addition, I think OOD generalization methods may help to handle test-agnostic LT, e.g., better de-biasing the issue of class imbalance. Nevertheless, OOD generalization methods also require further improvement to handle this task, since the test class distributions are unknown and can be any kind of distribution, and simply using OOD generalization methods may be unable to handle different distributions. From the perspective of de-biasing, the long-tailed and inversely long-tailed test class distribution may also be regarded as bias to some degree, but (in my view) it is real scenarios and is not wrong. Meanwhile, in my view, test-agnostic LT may also contribute to the field of OOD generalization,  e.g., providing a specific evaluation task by using our provided test-agnostic class distributions (txt) on different LT datasets. In summary, test-agnostic LT (without the prior of test class distributions) not only broadens the research of LT, but is also beneficial to the research of OOD generalization. Therefore, I believe this is an important task and worth further exploring in the future.
BTW, I am also quite interested in causal inference for deep learning, and I think it may be a feasible way to improve the (OOD) generalization of DNNs. Maybe we can add wechat and discuss more.
Best,
Yifan

Thank you for answering my questions. You did a great job. I think your work highlighted a crucial drawback of current LT settings.

Best,
Kaihua


Tag should be inside SELECT statement

Hi,
Thanks for implementing this for EF6!
I was the one who originally proposed the EFCore TagWith API.  After it was released, we discovered that the SQL Server Query Store eliminates comments outside the query text, but keeps comments inside the query.
Thus:
-- Get top 10 older friends with country

SELECT TOP(@__p_0) [friend].[Name] AS [FriendName], [friend].[Age],
                   [friend.Country].[Name] AS [CountryName]
FROM [Friends] AS [friend]
LEFT JOIN [Countries] AS [friend.Country] ON [friend].[CountryId] = [friend.Country].[Id]
ORDER BY [friend].[Age] DESC

Should be:
SELECT TOP(@__p_0) [friend].[Name] AS [FriendName], [friend].[Age], /* Get top 10 older friends with country */
                   [friend.Country].[Name] AS [CountryName]
FROM [Friends] AS [friend]
LEFT JOIN [Countries] AS [friend.Country] ON [friend].[CountryId] = [friend.Country].[Id]
ORDER BY [friend].[Age] DESC

or even
SELECT /* Get top 10 older friends with country */
TOP(@__p_0) [friend].[Name] AS [FriendName], [friend].[Age],
                   [friend.Country].[Name] AS [CountryName]
FROM [Friends] AS [friend]
LEFT JOIN [Countries] AS [friend.Country] ON [friend].[CountryId] = [friend.Country].[Id]
ORDER BY [friend].[Age] DESC

First of all, thanks for contributing and congrats for such great idea for the product :)
I didn't know that about the query store, so thanks for pointing it out. At a first sight, it doesn't seem to be too complex to implement in the component.
I'll take a look at it and study one way to enable this "query store" mode, just for not breaking the current behavior for people that could be using it.
Thank you very much!

@jmaguilar Great! I also opened #7 .  If you do not have time to maintain, consider asking Jonathan Magnan at ZZZProjects.  He sells support for this kind of stuff, and he built EF6 temporal table support for us!  Highly recommended.

Thank you. I'll try to work on the component next Christmas holidays, so I hope to release a new version soon :)

Feature included in version 1.2.0. Thanks!

Feature included in version 1.2.0. Thanks!


Cached export for incremental

See https://github.com/typst/typst/issues/2393

The memoize annotations on render and svg functions still need to be removed (but you probably know that).

Damn, I did it then I messed up my git and didn't migrate it 😠

Thank you again!


Doc Review and Update

We've noticed some (minor) inconsistencies in the docs while going through the Getting Started and updating / launching the demo services. There needs to be a quick scan of the docs to ensure they're all updated to current information.

I've added myself as a second reviewer once @kflynn does a first pass.

I reviewed things and I think we're OK.


input directories:  hard to understand error when forgetting .contents

I forgot to use .contents on my input directory (source: inputs.directories.testdata instead of source: inputs.directories.testdata.contents):
	inputs: {
		directories: {
			testdata: path: "./testdata"
		}
	}

	actions: {
		TestReact: {
			pkg: yarn.#Build & {
				source: inputs.directories.testdata
			}
	}

This is the error I'm getting back, which is very hard to understand:
4:48PM FTL system | failed to load plan: actions.TestReact.pkg.source.$dagger: field not allowed: fs

Any chance we can make this better?
/cc @talentedmrjones @jlongtine

I'm closing this, since we are no longer adding features to the CUE API - new features should be added to the GraphQL API instead.


feat(pixels): add my new pixel (51, 8)

Checklist

[ ] I ran npm test locally and it passed without errors.

[ ] I only edited the _data/pixels.json file.

[ ] I entered the username in the pixels.json that I'm also using to create this pull request.
[ ] I acknowledge that all my contributions will be made under the project's license.

learning...


Stan.jl warning

From Eric Chameroy on Discourse:
Hi everyone,
When I load Stan.jl, I receive the following message:
WARNING: using Stan.CMDSTAN_HOME in module Main conflicts with an existing identifier.
I had created a .juliarc.jl file containing the statement:
CMDSTAN_HOME = “C:\Users\USERNAME\CmdStan” (I’m using Windows 10)
as stated in the Stan.jl documentation for setting up CmdStan.
What does this warning mean and is there a way to fix it? Considering it’s a warning and not an error, do I need to be concerned with it at all?

From Robin Deits on Discourse:
Looking at the Stan.jl source code, I think this probably isn’t something you need to worry about. But I think you can make the error go away by setting an environment variable rather than creating a global Julia variable. That would mean changing your .juliarc.jl command to:
ENV["CMDSTAN_HOME"] = "C:\Users\USERNAME\CmdStan"
does that work? And does it get rid of the warning?
BTW, the relevant lines are here: https://github.com/goedman/Stan.jl/blob/ef6b07f43f575cad5b8d40e0953e013ab7def64c/src/Stan.jl#L12

From Rob Goedman on Discourse:
Thanks for jumping in Robin!
Hi Eric,
We’ve gone through several iterations on how to set this up in such a way that it works across platforms. Clearly it still isn’t correct.
Before changing anything, let’s see if Robin’s suggestion works. If so, maybe I should just update the documentation.
Does Pkg.test(“Stan”) work ok for you?
Regards,
Rob

From Eric on Discourse:
Thanks for the help!
Robin - you’re suggestion did remove the warning message, thanks.
Rob - I got errors when I ran Pkg.test(“Stan”). When that command is executed, is it running the same model as used in the walkthrough example provided in the documentation? I was trying to run that code in a Jupyter notebook and was receiving errors - I think the same as I had gotten with Pkg.test(“Stan”).
I can post the output files if that would help.

From Rob on Discourse:
Hi Eric,
would it be possible to switch to https://github.com/goedman/Stan.jl/issues/47 ? I have copied the thread so far.
Yes, I definitely need to see the error messages and versions you are using (I assuming Julia 0.6 and Stan.jl 2.0.0).
The first step is to see if CmdStan works.
Regards,
Rob

Hi Rob,
Here are the output files. If you need anything else, let me know.
I am using Julia 0.6 and Stan.jl 2.0.0.
bernoulli.exe_build.log.txt
bernoulli.stan.txt
Best,
Eric

This looks like CmdStan is not installed properly and it attempts to install/build it. Have you been able to follow the installation instructions from the Stan website, i.e. where the Stan compiler (stand) and some other utilities are build?
We need to get to the point where we can run Bernoulli just using CmdStan.

I believe I followed the instructions correctly. I created my .juliarc.jl file specifically to point to the location of CmdStan. I followed step 1.2 because I'm using Windows. Step 1.1 is for Linux/OS X, correct?

Hi Eric,
I’m talking about http://mc-stan.org/users/interfaces/cmdstan.html http://mc-stan.org/users/interfaces/cmdstan.html, particularly appendix B.3.
We need to get bernoulli.exe sample data file=bernoulli.data.R  (on page 10 of that manual) running.
Is that running on your system?
Best,
Rob

On Aug 10, 2017, at 14:15, Eric Chameroy<EMAIL_ADDRESS>wrote:
I believe I followed the instructions correctly. I created my .juliarc.jl file specifically to point to the location of CmdStan. I followed step 1.2 because I'm using Windows. Step 1.1 is for Linux/OS X, correct?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/goedman/Stan.jl/issues/47#issuecomment-321675547, or mute the thread https://github.com/notifications/unsubscribe-auth/AADL6CB3XN8DlC1NMwo5bXcBaFlY26pCks5sW3MEgaJpZM4OzeWq.

Hi Rob,
Sorry, I did not have CmdStan properly set up. Thank you for pointing me to the proper documentation. I do believe CmdStan is running correctly now. Thanks for your patience. I feel somewhat silly for not having figured that out on my own.
However, I still get errors when running Pkg.test("Stan") and executing the code in a notebook, although, now the errors are different then before.
Here are the output files.
CmdStan:
bernoulli.data.R.txt
bernoulli.stan.txt
Notebook output:
bernoulli.exe_build.log.txt
bernoulli_run.log.txt
bernoulli.stan.txt
bernoulli_1.data.R.txt
I would send you output files from Pkg.test("Stan"), but I don't know where to find them. I apologize if I'm uploading too many files - I'm not sure if you need anything other than the logs.
Best,
Eric

Hi Eric,
That looks a lot better!
Can you run your example in the REPL? Or one of the examples in Stan.jl?
Or Pkg.test(“Stan”) in REPL should display error messages that you can copy to a file.
Rob

On Aug 11, 2017, at 13:12, Eric Chameroy<EMAIL_ADDRESS>wrote:
Hi Rob,
Sorry, I did not have CmdStan properly set up. Thank you for pointing me to the proper documentation. I do believe CmdStan is running correctly now. Thanks for your patience. I feel somewhat silly for not having figured that out on my own.
However, I still get errors when running Pkg.test("Stan") and executing the code in a notebook, although, now the errors are different then before.
Here are the output files.
CmdStan:
bernoulli.data.R.txt https://github.com/goedman/Stan.jl/files/1218935/bernoulli.data.R.txt
bernoulli.stan.txt https://github.com/goedman/Stan.jl/files/1218936/bernoulli.stan.txt
Notebook output:
bernoulli.exe_build.log.txt https://github.com/goedman/Stan.jl/files/1218941/bernoulli.exe_build.log.txt
bernoulli_run.log.txt https://github.com/goedman/Stan.jl/files/1218944/bernoulli_run.log.txt
bernoulli.stan.txt https://github.com/goedman/Stan.jl/files/1218950/bernoulli.stan.txt
bernoulli_1.data.R.txt https://github.com/goedman/Stan.jl/files/1218952/bernoulli_1.data.R.txt
I would send you output files from Pkg.test("Stan"), but I don't know where to find them. I apologize if I'm uploading too many files - I'm not sure if you need anything other than the logs.
Best,
Eric
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/goedman/Stan.jl/issues/47#issuecomment-321907902, or mute the thread https://github.com/notifications/unsubscribe-auth/AADL6DJnPbsYtFTlnsB24EgYP2B7-ZQVks5sXLWbgaJpZM4OzeWq.

Sure. I didn't know I could just copy and paste from the REPL. Here is the Pkg.test("Stan") output:
PkgtestStanREPL.txt
Eric

Aah, this looks like a very old issue (closed issue #9) that was brought up in 2014 (but for some reason I closed it, probably because I had no way to test).
But the error message definitely points to where I try to delete the tmp directory. Apparently I can't do that in Windows.
One quick test would be to copy and paste all text in the Examples/NoMamba/Bernoulli/bernoulli.jl file into the REPL and press return.

I tried running the code in the REPL. Here's the end result:
bernoulli_example_REPL.txt
I also wasn't having any luck using the "ProjDir=" argument. At first I tried "ProjDir=pwd()", but got an error. When I dropped the "ProjDir=" part and just used "pwd()", I had a little better success. Here's the notebook file I was using to better illustrate what I'm talking about:
Stan Example.zip
Eric

Looking through the notebook, we’re pretty close in fact. CmdStan ran fine (rc=0, Mamba chain object looks fine)
Can you remove the first line after if rc=0 (i.e. sim1 = …) and try the notebook again?
Best,
Rob

On Aug 11, 2017, at 16:02, Eric Chameroy<EMAIL_ADDRESS>wrote:
I tried running the code in the REPL. Here's the end result:
bernoulli_example_REPL.txt https://github.com/goedman/Stan.jl/files/1219261/bernoulli_example_REPL.txt
I also wasn't having any luck using the "ProjDir=" argument. At first I tried "ProjDir=pwd()", but got an error. When I dropped the "ProjDir=" part and just used "pwd()", I had a little better success. Here's the notebook file I was using to better illustrate what I'm talking about:
Stan Example.zip https://github.com/goedman/Stan.jl/files/1219265/Stan.Example.zip
Eric
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/goedman/Stan.jl/issues/47#issuecomment-321936949, or mute the thread https://github.com/notifications/unsubscribe-auth/AADL6Hdym5-As4vUEBhK4s2i85gwARmLks5sXN2GgaJpZM4OzeWq.

No errors.

Yip, the call to stan(…) returns a Tuple: (rc, Mamba.Chain object).
For the filtering you just need the Mamba.Chain object.
It does bother me that I can’t remove the tmp dir using a run command and also that @FILE does not work on Windows.
Rob

On Aug 11, 2017, at 16:49, Eric Chameroy<EMAIL_ADDRESS>wrote:
No errors.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/goedman/Stan.jl/issues/47#issuecomment-321941763, or mute the thread https://github.com/notifications/unsubscribe-auth/AADL6NPfdRwGav08JAs3Y8Fb_JsNu2R_ks5sXOiIgaJpZM4OzeWq.

A few more remarks:

The @__FILE__ issue I now understand, this only works if the file is loaded from a file, which is not the case in a notebook. In a notebook it returns "", which means the tmp directory is created in the current working directory.

I've added a slightly edited version of Eric's notebook to the master version of Stan.jl. The notebook should become visible after Pkg.checkout("Stan").

Note that if you run a notebook from within the Stan.jl directory, Pkg.update() will in the future skip updating Stan.jl. A better approach is to copy the notebook to a different location and then start/run iJulia.

If someone figures out why I can't remove a tmp dir in Windows (or what run command I should use), that would be appreciated.

I've done two things:
I copied bernoulli.jl from the location of the Stan package to my desktop. Ran it using the command line - no errors.
I also created a file, stan_example.jl that used the example code from the Stan.jl documentation. I could also get that to execute using the command line, but only after commenting out the "sim1 =" line.
bernoulli.jl.txt
stan_example.jl.txt
So, will this issue cause problems for me if I continue working on Windows? Is there a work around?
Eric

Hi Eric,
I don’t think you will run into problems.
The “sim1 = ...” is not working because it should read “rc, sim1 = ...”, exactly as in the line above the if statement in your script. But that would just run the simulation again, you already have the results in sim1.
Does that clarify things?
Sent from Rob Goedman's iPad Pro

On Aug 13, 2017, at 16:40, Eric Chameroy<EMAIL_ADDRESS>wrote:
I've done two things:
I copied bernoulli.jl from the location of the Stan package to my desktop. Ran it using the command line - no errors.
I also created a file, stan_example.jl that used the example code from the Stan.jl documentation. I could also get that to execute using the command line, but only after commenting out the "sim1 =" line.
bernoulli.jl.txt
stan_example.jl.txt
So, will this issue cause problems for me if I continue working on Windows? Is there a work around?
Eric
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

So what's happening is that the first call to stan() is returning a tuple. The second call is trying to overwrite the tuple created by the first call, creating the error. Correct?
Thank you for all of your help Rob. I really appreciate it.
Eric

After the first call, rc is getting the 1st entry of the Tuple that stan() returns. This is a return code which I set in stan() to differentiate what can go wrong. The 2nd component of the Tuple returned by stan() contains the simulated values (converted to a Mamba.Chains object). This is assigned to sim1. Thus sim1, a Mamba.Chains object, can be used to filter a subset of all sampled parameters and the script assigns those to sim.
If the left hand side only is sim1, sim1 ends up as a Tuple and can’t be used directly to filter a subset and this was the error Julia reported.
Sent from Rob Goedman's iPad Pro

On Aug 13, 2017, at 20:50, Eric Chameroy<EMAIL_ADDRESS>wrote:
So what's happening is that the first call to stan() is returning a tuple. The second call is trying to overwrite the tuple created by the first call, creating the error. Correct?
Thank you for all of your help Rob. I really appreciate it.
Eric
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for the explanation.
Again, thanks for your help.
Eric


Folded multiply optimizes poorly on WebAssembly

https://rust.godbolt.org/z/dPYf4M
The fallback hash is designed around the assumption that the 128-bit multiply will be optimized into a 64-bit multiply with 128-bit output, but when compiling for WebAssembly it turns into a call to __multi3 which emulates a full 128-bit multiply.
I think this is an inherent limitation of WebAssembly rather than a codegen issue, as far as I can tell there's no way to get the upper part of a 64-bit multiply in wasm. A secondary fallback that avoids 128-bit math might be beneficial for targets like this.

A quick test to see how much of a performance cliff this causes
use ahash::AHasher;
use fnv::FnvHasher;
use rand::prelude::*;
use std::hash::{Hash, Hasher};
use std::time::Instant;

fn bench(name: &str, data: &[u64], mut hasher: impl Hasher) {
    let start = Instant::now();
    data.hash(&mut hasher);
    let hash = hasher.finish();
    let time = start.elapsed();

    println!("{}: {:?} ({})", name, time, hash);
}

fn main() {
    let mut data = vec![0u64; 8_000_000];
    thread_rng().fill(data.as_mut_slice());

    bench("ahash", &data, AHasher::default());
    bench("  fnv", &data, FnvHasher::default());
}

Native (no AES):
$ cargo run --release
ahash: 7.0502ms<PHONE_NUMBER>3929795835)
  fnv: 60.6068ms<PHONE_NUMBER>109644555)

WebAssembly:
$ cargo build --target=wasm32-wasi --release
$ wasmer target\wasm32-wasi\release\wasmtest.wasm
ahash: 94ms<PHONE_NUMBER>958052022)
  fnv: 109ms<PHONE_NUMBER>2428832362)

That's still acceptable performance on wasm, but AHashes usual performance edge is almost completely negated

Have you tried running with the LLVM compiler, runtime speed performance might be increased!
$ wasmer run target\wasm32-wasi\release\wasmtest.wasm --llvm

$ wasmer run target\wasm32-wasi\release\wasmtest.wasm --llvm
error: failed to run `target\wasm32-wasi\release\wasmtest.wasm`
╰─> 1: The `llvm` compiler is not included in this binary.

I'm using the Windows binary from https://github.com/wasmerio/wasmer/releases/tag/1.0.2
Does the LLVM backend only come with the Linux/OSX versions?
$ wasmtime target\wasm32-wasi\release\wasmtest.wasm
ahash: 37.4199ms<PHONE_NUMBER>860778540)
  fnv: 82.4112ms<PHONE_NUMBER>4453256082)

wasmtime shows better performance overall, but ahash is still only 2x faster than FNV compared to 10x faster in native code

Which version of Wasmer are you using, and in which system?
The prebuilt binaries should all include llvm except for Windows

It occurred to me that hashing the entire dataset at once isn't really representative of how ahash/fnv would behave in a hashmap keyed by u64, where each hasher would only be used to hash a single value. Reworked the benchmark:
use ahash::RandomState;
use fnv::FnvBuildHasher;
use rand::prelude::*;
use std::hash::{BuildHasher, Hash, Hasher};
use std::time::Instant;

fn bench(name: &str, data: &[u64], builder: impl BuildHasher) {
    let mut total = 0u64;

    let start = Instant::now();
    for n in data.iter() {
        let mut hasher = builder.build_hasher();
        n.hash(&mut hasher);
        let hash = hasher.finish();

        // side effect to keep the hash from being optimized away
        total = total.wrapping_add(hash);
    }
    let time = start.elapsed();

    println!("{}: {:?} ({})", name, time, total);
}

fn main() {
    let mut data = vec![0u64; 8_000_000];
    thread_rng().fill(data.as_mut_slice());

    bench("ahash", &data, RandomState::default());
    bench("fnv1a", &data, FnvBuildHasher::default());
}

$ cargo run --release
ahash: 16.2001ms<PHONE_NUMBER>360615028)
fnv1a: 21.3877ms<PHONE_NUMBER>369249413)

$ cargo build --target=wasm32-wasi --release
$ wasmtime target\wasm32-wasi\release\wasmtest.wasm
ahash: 131.5265ms<PHONE_NUMBER>7417786512)
fnv1a: 34.5373ms<PHONE_NUMBER>142809863)

It's actually worse than I thought, ahash goes from a slight lead over fnv in native code to being nearly 4x slower in wasm

I was curious and I just tested times with the latest Wasmer in the new macOS M1.
Here's my timings:
Wasmer (LLVM + Native)
➜  aHash git:(master) ✗ wasmer target/wasm32-wasi/release/example.wasm --llvm --native
ahash: 34.327ms<PHONE_NUMBER>2678093573)
fnv1a: 15.481ms<PHONE_NUMBER>3041238450)

Wasmer (Cranelift + JIT - default)
➜  aHash git:(master) ✗ wasmer target/wasm32-wasi/release/example.wasm --cranelift
ahash: 42.439ms<PHONE_NUMBER>7232338973)
fnv1a: 25.012ms<PHONE_NUMBER>7822909892)

Native binary
➜  aHash git:(master) ✗ cargo run --release
[...]
    Finished release [optimized] target(s) in 0.03s
     Running `target/release/example`
ahash: 5.835041ms<PHONE_NUMBER>8906542966)
fnv1a: 30.808083ms<PHONE_NUMBER>817530018)

FNV is faster in Wasmer than in a native binary on the M1? That's bizarre, I wonder what's going on there.
Nevertheless, ahash still struggles when compiled to WebAssembly.

FNV is faster in Wasmer than in a native binary on the M1? That's bizarre, I wonder what's going on there.

Yeah, I'm not 100% sure on why is that.
We have seen similar speedups in other benchmarks due to 32-bit pointers in Wasm instead of 64-bit ones in Native... that might be it?

Nevertheless, ahash still struggles when compiled to WebAssembly.

Why you think is that? Is ahash using custom-encoded asm instructions when executed natively?

Why you think is that? Is ahash using custom-encoded asm instructions when executed natively?

AHash can optionally use AES crypto instructions to accelerate hashing on x86 targets, but all of my timings are taken with that feature disabled so the same portable fallback algorithm is used in both native and wasm tests. As I said in the OP, I think the problem is that the fallback algorithm assumes the processor can produce a 128-bit result from multiplying two 64-bit integers together, but WebAssembly doesn't support that operation and Rust has to generate very inefficient code to work around it.

I had hoped that LLVM would be able to work out that it only needed to perform a 64bit multiply.
I have created the https://github.com/tkaitchuck/aHash/tree/miri branch to try out experimental options.

LLVM can work it out, but there isn't necessarily a clean way to lower that to machine code
I checked all of the targets available on Godbolt to see which ones have fast folded_multiply
Fast (1-2 mul instructions)

x86_64
aarch64
mips64
powerpc64
riscv64gc
s390x

Not ideal (~10 math instructions)

i686

Slow (calls __multi3)

arm (all 32-bit variants)
thumb (all variants)
mips
powerpc
sparc64
wasm32

Many of those aren't particularly relevant anymore, but 32-bit ARM and Thumb are still around in embedded devices

The merged change creates a code path that should be good but is definitely not great, as it requires 4 multiplies per 64 bits of input. I am creating a new issue to track optimizing this: https://github.com/tkaitchuck/aHash/issues/79

The fix is included in version 0.7.2

Thanks!


Errors: Http 500 - CF 1101 // Http 503 - CF 1105 // Http 504

Hi again. I'm back at it once again trying to export a 450 MB-large database. My own worker uses Hono and I would constantly see the Inspector command exceeded memory limits errors.
I've decided to git clone yours and replace the bindings in the /example worker with the very of my own. I'm seeing 500s again. Sometimes I will receive Error 1105 from Cloudlfare, or other Http 504 from both... most of the time it is Error 1101. It almost always won't tell what went wrong. A chase in the blind.
I've played a lot with backup options and managed to get some uploading going on but I would bump into CPU time limit reached or worker has been executing for two long. The later I was able to get around using http://localhost:8787/__scheduled?cron=*+*+*+*+* route however the others are still there.
Example photo of the struggle:

It might as well be a limitation I found myself that won't be able to get out until Cloudflare allows official export and it's nothing that we can do here but I thought it's worth writing here to see if you've got any ideas. Cheers.

Try 100mb limit, instead of 10mb/30mb

const options: CreateBackupOptions = {
    fileName: `backups/${new Date().toUTCString()}.sql`,
    queryLimit: 50,
    maxBodySize: 100,
};

results in HTTP 504

Can you try deploying the worker and see if it happens on a real worker?
I suspect that the dev worker is more rate limited than a real worker.

Unfortunately same outcomes...
I have 11 tables and around 95k rows:
.
I'm thinking if it'd be realistic writing support and having raised limits to get through. I'm experiencing very high latency consuming from classic app and thus need to host elsewhere. Maybe you know if Cloudflare would be able to offer one-time sqlite dump?

I had success using https://github.com/Cretezy/cloudflare-d1-backup tool so I'm going to close this issue. I suppose it's not the nicest way to spend the time given that official export is coming in the next season (hopefully) and ideally the target audience is not going to use it for storing lots of (archival) data.


[TensorIR] Fix parser autocompletion mode

This PR will fix the Script parser autocompletion mode and add checks for opaque blocks (which is not allowed now)
Co-authored-by: Ruihang Lai<EMAIL_ADDRESS>cc @tqchen @junrushao1994

Thanks @Hzfengsy @MasterJH5574 . This PR is merged


Concept properties are not being saved while using the operation apply-codesystem-delta-add

Concept properties are not being saved while using the operation $apply-codesystem-delta-add
To Reproduce
Steps to reproduce the behavior:

POST fhir/CodeSystem/$apply-codesystem-delta-add
send the following request
{
"resourceType": "Parameters",
"parameter": [
{
"name": "system",
"valueUri": "test.com/testProp"
},
{
"name": "codeSystem",
"resource": {
"resourceType": "CodeSystem",
"concept": [
{
"code": "testPropCode1",
"display": "testPropDispaly1",
"property": [
{
"code": "code1",
"valueString": "testPropPropertyVal1"
}
]
}
]
}
}
]
}

the concepts are being added well but properties are not saved at all.
Expected behavior
Concepts and their properties should saved
Environment:

HAPI FHIR 4.1.0
OS: [windows]
Browser [chrome]

@jamesagnew :I am interested in solving this problem; I have implemented the code fix for it

@jamesagnew
In case no one is working on this; I would like to work on it.


SVG used as background-image not changing color in dark mode

Hello, we are using a couple of Bootstrap SVG icons in our CSS, like:
background-image: url("data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='16' height='16' fill='currentColor' class='bi bi-trash' viewBox='0 0 16 16'%3E%3Cpath d='M5.5 5.5A.5.5 0 0 1 6 6v6a.5.5 0 0 1-1 0V6a.5.5 0 0 1 .5-.5zm2.5 0a.5.5 0 0 1 .5.5v6a.5.5 0 0 1-1 0V6a.5.5 0 0 1 .5-.5zm3 .5a.5.5 0 0 0-1 0v6a.5.5 0 0 0 1 0V6z'/%3E%3Cpath fill-rule='evenodd' d='M14.5 3a1 1 0 0 1-1 1H13v9a2 2 0 0 1-2 2H5a2 2 0 0 1-2-2V4h-.5a1 1 0 0 1-1-1V2a1 1 0 0 1 1-1H6a1 1 0 0 1 1-1h2a1 1 0 0 1 1 1h3.5a1 1 0 0 1 1 1v1zM4.118 4 4 4.059V13a1 1 0 0 0 1 1h6a1 1 0 0 0 1-1V4.059L11.882 4H4.118zM2.5 3V2h11v1h-11z'/%3E%3C/svg%3E");
The icons are not changing the color when dark mode is set, is there anything we should add to our custom CSS after importing pico.css?
Thank you very much,
Martin

I tried to add a style (with prefers-color-scheme) declaration within the SVG icons without any success

I believe you cannot use css variable with url(). But css mask is the way to go.
You can read more in this great article  Coloring SVGs in CSS Background Images

Yes, we can't dynamically change a background image SVG color.
That's why Pico has light icons and dark icons.


interrupted aortic arch, type c

Add new child term of interrupted aortic arch MP:0004157
interrupted aortic arch, type c
DEF: complete discontinuation/blockage between the ascending and descending aorta with the interruption located between the brachiocephalic (innominate) artery and the left common carotid artery
SYN: interrupted aortic arch type c
SYN: interrupted aortic arch type C
SYN: IAA-C
SOURCE:
https://radiopaedia.org/articles/interrupted-aortic-arch
https://pedecho.org/library/chd/iaa
https://en.wikipedia.org/wiki/Interrupted_aortic_arch
https://www.childrensmercy.org/departments-and-clinics/heart-center/interrupted-aortic-arch/
Annotations in J:130481, J:128012, J:71240
Compare with:
Interrupted aortic arch type C HP:0011614
DEF: Non-continuity of the aortic arch with an atretic point or absent segment between the innominate and left carotid arteries.

Added new term:
interrupted aortic arch, type c MP:0031517


Upgrade github.com/opencontainers/image-spec to v1.0.2

To pick up the fix for https://www.cve.org/CVERecord?id=CVE-2021-41190.
Signed-off-by: Michi Mutsuzaki<EMAIL_ADDRESS>

will backport https://github.com/cilium/cilium/pull/23140 instead


Definition of Classes in Signature

kb=TripleStoreKnowledgeBase()
for i in kb.concepts:
     ...

Returns the results of  self.ontology.classes_in_signature()
def classes_in_signature(self) -> Iterable[OWLClass]:
       query = owl_prefix + "SELECT DISTINCT ?x WHERE {?x a owl:Class.}"
       yield from get_results_from_ts(self.url, query, OWLClass)

Should we relaxed this assumption by returning things that are rdfs:Class ?
A user may set a flag to true (e.g. kb=TripleStoreKnowledgeBase(inferrence=True), we can very well find a better name :) )

Ontolearn works with OWL classes and having to work with classes of type rdfs:Class would be very resctrictive. A class of type rdfs:Class is less expressive which would contradict the rest of the methods that work with classes of type owl:Class. Relations like owl:equivalentClass, owl:disjointWith are not supported by rdfs:Class as well as complex class constructors like owl:unionOf, owl:intersectionOf, owl:complementOf.
If we were to implement this change then there is a chain of other changes that need to be considered, like checking on each reasoner methods that works with classes whether we have to do with an owl:Class or a rdfs:Class so we avoid possible errors. Another solution is to implement another reasoner for simple ontologies which uses only RDFS classes. To me its a lot of trouble but its you who can decide if this is worth it (I'm not sure what you want to use this for).

Ontolearn works with OWL classes and having to work with classes of type rdfs:Class would be very resctrictive
For each RDF class definition ( e.g. x type rdfs:Class), we will infer x type owl:Class.
Why do you think that such inference be restrictive ?
Relations like owl:equivalentClass, owl:disjointWith are not supported by rdfs:Class as well as complex class constructors like owl:unionOf, owl:intersectionOf, owl:complementOf.
Since we make the owl:Class inference for each RDF classes, we can use all complex constructors.


Lint for Mozilla PKI Policy for ECDSA encoding requirements

See https://github.com/mozilla/pkipolicy/blob/master/rootstore/policy.md#512-ecdsa

When ECDSA keys are encoded in a SubjectPublicKeyInfo structure, the algorithm field MUST be one of the following, as specified by RFC 5480, Section 2.1.1:
The encoded AlgorithmIdentifier for a P-256 key MUST match the following hex-encoded bytes: > 301306072a8648ce3d020106082a8648ce3d030107.
The encoded AlgorithmIdentifier for a P-384 key MUST match the following hex-encoded bytes: > 301006072a8648ce3d020106052b81040022.
The above encodings consist of an ecPublicKey OID (1.2.840.10045.2.1) with a named curve parameter of the correponding curve OID. Certificates MUST NOT use the implicit or specified curve forms.
When a root or intermediate certificate's ECDSA key is used to produce a signature, only the following algorithms may be used, and with the following encoding requirements:
If the signing key is P-256, the signature MUST use ECDSA with SHA-256. The encoded AlgorithmIdentifier MUST match the following hex-encoded bytes: 300a06082a8648ce3d040302.
If the signing key is P-384, the signature MUST use ECDSA with SHA-384. The encoded AlgorithmIdentifier MUST match the following hex-encoded bytes: 300a06082a8648ce3d040303.
The above encodings consist of the corresponding OID with the parameters field omitted, as specified by RFC 5758, Section 3.2. Certificates MUST NOT include a NULL parameter. Note this differs from RSASSA-PKCS1-v1_5, which includes an explicit NULL.

Resolved by https://github.com/zmap/zlint/issues/358


systemd: use getty-generator for vconsole instantiatiation

systemd has a native feature that allows generation and instantiatiation
of<EMAIL_ADDRESS>instances for virtualizer consoles.
Signed-off-by: Oleksandr Kravchuk<EMAIL_ADDRESS>

@0390b8ae1061dceaa8cb97f13870fd9979c009a6 also added getty.target.wants to FILES_${PN}.

I wonder why serial-getty-generator did not dynamically create getty services before in PELUX. It has been a feature in systemd for a very long time (the start of the project almost... to create them on demand and saving boot time in that way.
But if it works now, let's just remove this and be happy that we remove some services from the startup. :)

@SKumarDas do you remember why this change was brought in?

@sashko This change was made to facilitate one more console because before this change we had only one and it was taken by the GUI.

No time to test this myself, but approving assuming we still get virtual consoles.

Feel free to come by my desk and check it out on NUC or Raspberry Pi I have here.


Add quick pick menus to Rasa docs

Proposed changes:

Add table of contents to relevant pages for easier navigation

Status (please check what you already did):

[ ] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

@erohmensing @MetcalfeTom did we need to do this for Rasa X as well?

@akelad in my opinion, none of the pages needed it which didn't already have it.  Do you have an example of one you'd like it for?


[luasocket] bump to 3.1.0

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Updated with the suggestions from @JonLiu1993
Thanks.

@JonLiu1993 @Cheney-W is anything else expected of me?

It's intentional to avoid failing when the files don't exist. There use to
be conditionals on them and even they didn't cover community triplets.
On Tue., May 2, 2023, 2:45 p.m. Javier Matos Denizac, <
@.***> wrote:

@.**** commented on this pull request.
In ports/luasocket/portfile.cmake
https://github.com/microsoft/vcpkg/pull/31078#discussion_r1182921430:

if(VCPKG_LIBRARY_LINKAGE STREQUAL "dynamic")

Handle socket dll name

file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/bin/socket/core.dll")
if(EXISTS "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb")

   file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/bin/socket/core.pdb")

endif()
file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.dll")
if(EXISTS "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.pdb")

   file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.pdb")

endif()

file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/bin/socket/core.dll" RESULT temp)
file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/bin/socket/core.pdb" RESULT temp)
file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.dll" RESULT temp)

I see you are storing the results of the rename in temp, but I don't see
temp being used for anything. Is this intentional or am I missing
something?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/pull/31078#pullrequestreview-1409691311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJBCOIXLX55BFO6XTI6FTXEFI3RANCNFSM6AAAAAAXI4TYBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://cmake.org/cmake/help/latest/command/file.html#rename
On Tue., May 2, 2023, 6:15 p.m. Stephen E. Baker, @.***>
wrote:

It's intentional to avoid failing when the files don't exist. There use to
be conditionals on them and even they didn't cover community triplets.
On Tue., May 2, 2023, 2:45 p.m. Javier Matos Denizac, <
@.***> wrote:

@.**** commented on this pull request.
In ports/luasocket/portfile.cmake
https://github.com/microsoft/vcpkg/pull/31078#discussion_r1182921430:

if(VCPKG_LIBRARY_LINKAGE STREQUAL "dynamic")

Handle socket dll name

file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/bin/socket/core.dll")
if(EXISTS "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb")

   file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/bin/socket/core.pdb")

endif()
file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.dll")
if(EXISTS "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.pdb")

   file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.pdb")

endif()

file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/bin/socket/core.dll" RESULT temp)
file(RENAME "${CURRENT_PACKAGES_DIR}/bin/socket/socket.core.pdb" "${CURRENT_PACKAGES_DIR}/bin/socket/core.pdb" RESULT temp)
file(RENAME "${CURRENT_PACKAGES_DIR}/debug/bin/socket/socket.core.dll" "${CURRENT_PACKAGES_DIR}/debug/bin/socket/core.dll" RESULT temp)

I see you are storing the results of the rename in temp, but I don't see
temp being used for anything. Is this intentional or am I missing
something?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/pull/31078#pullrequestreview-1409691311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGJBCOIXLX55BFO6XTI6FTXEFI3RANCNFSM6AAAAAAXI4TYBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's intentional to avoid failing when the files don't exist. There use to be conditionals on them and even they didn't cover community triplets.

A variable name like unused would have made this clear.
And I think at least for the DLLs it would have been good to have proper conditionals.


HWKAPM-653 Ensure all possible property names are included in the UI …

…filter dropdown when a business transaction has not been selected

@pavolloffay Could you review please?
Reviewer notes:

Original aim was to ensure all property names were displayed even when no business txn was selected
However took opportunity to optimise the implementation, using ES aggregation rather than pulling all fragments and processing locally

@pavolloffay Thanks!


[lists] Add custom products not in the database (shopping list)

Is your feature request related to a problem? Please describe.
When compiling a shopping list, you are not necessarily buying stuff that is already at OFF.
Describe the solution you'd like
The ability to add custom products, including the data listed in #3373 and a photo (from a price list or elsewhere). The data must remain private unless specified in a complex way, and the UI should be clearly marked as such, so you don't confuse what you are entering.
Describe alternatives you've considered
Use Privacy Friendly Shopping List or other customizable apps. But they are not perfect.

This would need sizable coding resources on the mobile side, and probably some server side work to turn the categories taxonomy into a standardized multilingual lists of products.

Previously reported in #2929 and #2928


fix: replace string formating by dict

What model is being used for Bedrock for JSON generation? Also what prompt?


Maya: Preserve time units on load and update with MayaUSDReferenceLoader

Changelog Description
Preserve time units on load and update with MayaUSDReferenceLoader.
Without the current frame, playback range and FPS may be altered by the loading/updating. This ensures to preserve the state of the local scene.
Additional info
I also tried using the optionVar of the import options to disable 'importing' the FPS or frame ranges from the loaded file using option vars importOptionsImportFramerate and animRangeOptDefault (also see e.g. C:\Program Files\Autodesk\Maya2024\scripts\others\fileOptions.mel) but those didn't seem to affect referencing so I ended up more explicit context managers instead.
Testing notes:

Loading or updating a USD file with Reference Maya USD loader should not change timeline / current frame.

Without this PR it usually sets e.g. the start frame to 0 if there's no frame range in the loaded USD file.

With this now part of ayon_maya - should a PR like this bump maya addon version?

With this now part of ayon_maya - should a PR like this bump maya addon version @iLLiCiTiT ?

It should.


Config UI: General layout

extracted part from #12958
prerequisite for "password change" #11732 and other non-device configurations

↔ use full layout width on larger screens
🍱 compact view of all non-device setting state

moved update/edit logic to separate screens/modals
cleaner ui. these settings will not be updated frequently
purpose of each setting or setting-group (e.g. port+host+protocol=network) can be explained on the separate screen

💬 refactored modal handling (introduced generic component)

⚠ Only "title" and "telemetry" are editable right now. Other settings are present (and disabled) to get a feeling for future layout requirements. Unimplemented entries will be hidden once Config UI available in a first non-experimental state.
🗒 Some texts are not translated yet, because this area is still subject to change, and we don't want to trigger unnecessary translation cycles. We'll translate them once we move out of experimental.
large screen: after

large screen: before

medium screen: after

medium screen: before

Wow, next release will have many nice improvements! Nice work! 😎