[tool_crash] FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2

This error happened in a project I hadn't touched for a long time (it had a constraint for Flutter 1.9.1)
The tool tried to access the directory .dart_tool, which doesn't exist.
Command
flutter pub upgrade --major-versions -n

Logs
FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.openSync (dart:io/file_impl.dart:479:5)
#2      _File.writeAsBytesSync (dart:io/file_impl.dart:604:31)
#3      _File.writeAsStringSync (dart:io/file_impl.dart:628:5)
#4      ForwardingFile.writeAsStringSync (package:file/src/forwarding/forwarding_file.dart:147:16)
#5      ErrorHandlingFile.writeAsStringSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:271:22)
#6      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
#7      ErrorHandlingFile.writeAsStringSync (package:flutter_tools/src/base/error_handling_io.dart:270:5)
#8      _DefaultPub.interactively (package:flutter_tools/src/dart/pub.dart:404:21)
<asynchronous suspension>
#9      PackagesInteractiveGetCommand.runCommand (package:flutter_tools/src/commands/packages.dart:344:5)
<asynchronous suspension>
#10     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)
<asynchronous suspension>
#16     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)

[✓] Flutter (Channel stable, 2.5.2, on macOS 11.6 20G165 darwin-arm, locale en-GB)
    • Flutter version 2.5.2 at /Users/minnovi/flutter_sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3595343e20 (13 days ago), 2021-09-30 12:58:18 -0700
    • Engine revision 6ac856380f
    • Dart version 2.14.3

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/minnovi/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/minnovi/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/minnovi/Library/Android/sdk
    • Java binary at: /Users/minnovi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.0, Build version 13A233
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/minnovi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/minnovi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/minnovi/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.60.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.27.0

[✓] Connected device (3 available)
    • RMX2155 (mobile)       • 6DCASCXKSK6DXWEY                     • android-arm64  • Android 10 (API 29)
    • iPhone 13 Pro (mobile) • BD92AF67-2F86-46C5-BCAE-9DC8B4B021B1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0 (simulator)
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 94.0.4606.81
    ! Error: iPhone is not connected. Xcode will continue when iPhone is connected. (code -13)

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.2.0+20
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 20e59316b8b8474554b38493b8ca888794b0234a

dupe? https://github.com/flutter/flutter/issues/68958


Updated node version from 18 to 20

Upgrading node version to 20.11.0

Which Traffic Control components are affected by this PR?

CDN in a Box
Automation (GitHub Actions)

What is the best way to verify this PR?

Pass all GH actions
If this is a bugfix, which Traffic Control versions contained the bug?

N/A
PR submission checklist

[ ] This PR has tests
[ ] This PR has documentation
[ ] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Looking at the end-to-end tests run (under Run everything and test:
  │ Cypress:        13.6.2                                                                         │
  │ Browser:        Electron 114 (headless)                                                        │
  │ Node Version:   v18.20.2 (/usr/local/bin/node)                                                 │

so it's still using Node 18 for the end-to-end tests, but it should use Node 20 instead.

These titles:
https://github.com/apache/trafficcontrol/blob/03f7f149323cc22e4fe6fc63a064783f0d4c7c77/.github/workflows/tpv2.yml#L63
https://github.com/apache/trafficcontrol/blob/03f7f149323cc22e4fe6fc63a064783f0d4c7c77/.github/workflows/tpv2.yml#L96
https://github.com/apache/trafficcontrol/blob/03f7f149323cc22e4fe6fc63a064783f0d4c7c77/.github/workflows/tpv2.yml#L128
should be changed to
      - name: Node ${{ env.NODE_VERSION }}

Fixed in c44f5aa

TPv2 still runs on Node 16 in CDN in a Box for Developers.
https://github.com/apache/trafficcontrol/blob/a89223526370e10b49ba26d0e302e1f3a4617374/dev/tpv2/Dockerfile#L26


Improve instructions for tags and releases

Do we want to emphasize on also providing an input for the tag?
Should we also explain the differences between a tag and a release somewhere to indicate one is of git and the later is specific to GitHub?
Error when no tag is provided

From https://github.com/githubtraining/release-workflow/issues/81

Closing since this has not generated any more bug reports


Importing null value for asset field doesn't remove asset/s

I imagine this is a bug...
To clean up my entries from #211, I created a feed of titles, slugs, and empty asset fields (news gallery) for all entries which should have an empty asset field.
I expected that running that import (i.e. null values for asset fields) would update matched entries and remove assets from the specified asset field field. However, the assets remain intact. I tried an empty string, and null, but neither worked.

Can you give the latest master a go?

Thanks for the fix! Solved the issue. Had my fingers crossed it would solve #211 as well if null fields were present, but no such luck.


Redirect to localhost address when trying to add account using subscription option

Storage Explorer Version: 1.19.0
Build Number: 20210414.2
Platform/OS: Windows 10 LTSC
Architecture: x64
Regression From:  1.18.0
Bug Description
Got redirect to localhost:49158 when trying to add an account using the "Add an account..." button and then select Subscription.
I Selected med Azure account and then i got redirected to https://localhost:49310/?code=
Steps to Reproduce

Press "Add an account...", the last button before "Open Explorer".
Select Subscription
You get redirect to a page where you can select your account
Now you are redirected to a localhost page with an error saying:

This site can’t provide a secure connectionlocalhost sent an invalid response.
ERR_SSL_PROTOCOL_ERROR
Expected Experience
To have my account added to Azure Storage Explorer.
Actual Experience
An error page saying
"This site can’t provide a secure connectionlocalhost sent an invalid response.
ERR_SSL_PROTOCOL_ERROR"
Additional Context
I had multiple account setup before the update to 1.19.0, after the update I was logged out from one of the accounts and now i'm trying to add the account again.

@MikaelPorttila what browser are you using? Did you try this at least more or once?

@MRayermannMSFT
I was using Brave Version 1.23.71 Chromium: 90.0.4430.72 (Official Build) (64-bit)
I tried this three times in total, twice when i tried the first round and one more time to capture the screenshot

@MikaelPorttila could something about your browser or something else be forcing HTTPS to be used always/no matter what?
When we do sign-in with your browser, the sign-in page redirects to a localhost address/port that our auth library is listening on. HTTP has to be used for that local host address though.

If you can't think of anything that would be causing this, then for now you can change to using integrated sign in. Please read this doc/section "Changing where sign in happens" on how to do that: https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-sign-in#changing-where-sign-in-happens

I downloaded Brave, and unfortunately wasn't able to reproduce. But it does look like it has some settings which could be making this happen?
For example, connections can be force upgraded on a site by site or global basis.

Perhaps try turning off "upgrade connections to https" either for localhost or just the setting in general?

Hi,
Thank you for troubleshooting this issue!
Tried to turn off the HTTPS and shield for the localhost page but still got redirected to HTTPS by something, I just changed the default browser to Firefox, logged in and changed back to Brave again and this solved all my issues for now and I can continue my work. 👍
Thoughts: Lower/turn off security to allow sign in feels like poor practice, maybe the solution for this issue should be to solve the root issue of using HTTP, one of these days other vendors will take after Brave and redirect to HTTPS and HTTPSEverywhere (Popular Chromium extension) users are in theory also affected.
You can close this issue unless you don't want to handle the case "What if HTTP wasn't an option".
Thank you again for all the help!

I am also having this issue, with authenticating and re-authenticating. I accidentally created a duplicate ticket (4417)

Experienced the same issue in Version 1.19.1 / Build: 20210425.1
switch to [Edit] > [Options[ > [Sign In] > [integrated sign] was a successful workaround

I had the same issue and used the same workaround. Could this be related to HSTS?
Options were renamed to Settings on my build

Same issue with Chrome v92.0.4515.131. Just re-installed storage explorer (v1.20.1), same thing.
Above work around works.

Same problem here. Just switched to integrated sign-in and worked fine.

I was able to work around this by going to
chrome://net-internals/#hsts
Delete domain security policies
Entering 'localhost' and clicking delete

Adding comments to both issue numbers in case someone searches it.
Disabling following flag is the solution in MS Edge (tested in version 104) - edge://flags/#edge-automatic-https
Cache needs to be cleared otherwise HSTS will report 'localhost' domain with flag dynamic_upgrade_mode: FORCE_HTTPS
You can query HSTS by going to edge://net-internals/#hsts
Alternative workaround is as mentioned previously, change Azure Storage sign-in with setting to Integrated Sign-in

I had the same issue but changing to Integrated Sign-In solved my problem. (Version 1.27.1)
Edit => Settings => Sign In WIth

It is now 2023, and whoever team owns this app does not seem to care.

I had the same issue with current Edge browser 121.0.2277.98 (Official build). I couldn't find option to upgrade HTTP to HTTPS but in flags something mentioned.., However I just paste link into Firefox and it worked. But as SignorLuigi wrote I changed for integrated Sign-in and will see

In 1.36.2 (December 2024), deleting HSTS for localhost in Chrome fixes the issue. Make sure to not close the login tab, to remain logged in. Before, I tried Integrated Sign-in but it didn't log me in.


fix result column size error when handling const column in truncate_decimal128

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #4282
Problem Summary(Required) ：

This issue is cause by a mishandling of the size of the returned column in truncate_decimal128 function.
The size of c0 maybe changed after invoke get_data_column_of_const, we should keep the original size as the result column size.
Another small change in this PR is to adjust the build script to only compile jdbc_bridge when building BE.

better split to two pr

run starrocks_fe_unittest


Escaped quotes test regex broken

On the current staging, two challenges ("Escaping Literal Quotes in Strings" and "Quoting Strings with Single Quotes") do not pass all tests with standard solution code.
Namely, tests asserting regex match lengths for non-escaped double quotes are failing:
code.match(/[^\\\\]\"/g).length === 2 in the former challenge and
code.match(/\"/g).length === 4 in the latter.
(Note: each of these is only one half of their respective test, and it might be beneficial in this case to split each test into two parts so users know where to look for their mistake instead of counting every quotation mark, as well as speeding up debugging if something breaks the regex again later on.)

@BKinahan Any idea of the cause of this? I'm working on the "Quoting Strings with Single Quotes" challenge. Would splitting the test into two fix it?

@dhcodes Splitting it into two tests is how I found that the two quoted-above halves of the tests are what are causing the trouble, but I'm not sure of the exact cause. I'm reasonably sure it has to do with escaping though, and you might gain some insight by adding to the tail or tests some code which logs the results of the calls to match so you can see what the results actually are. If I recall each was matching more times than it was supposed to.
Good luck!

Looks like several changes done internally now effect the number of quotes in the original code.
First is head/tail is now included in the code. If this has not always been the case then that will require a new issue to be opened.
The other is now all JS code is pushed through babel, which adds "use strict"; key. This won't change as it is a requirement for the new challenges coming down the pipeline.
Can someone confirm that code did not contain the head/tail previously?

After reviewing the code, looks like to quickest solution is to change the number of expected to 12 as changing the internals is far more involved.

I've made those changes here https://github.com/FreeCodeCamp/FreeCodeCamp/pull/10518

The PR above only addresses the problem in Escaping Literal Quotes in Strings. Quoting Strings with Single Quotes still isn't working for a similar reason.

Escaping Literal Quotes in Strings seems to be broken in production but not in dev.

Not sure if this is the place to comment. But the Quoting Strings with Single Quotes still seems to be buggy and won't pass.

I'm stuck on the Quoting Strings with Single Quotes challenge. I'm 100% certain I've got the right answer in but it's failing two tests (remove all the backslashes, you should have two single quotes and four double quotes). Very frustrating, can't get past this problem :(

Tested on Chrome and Firefox after new updates. Both are working. Closing this issue unless it crops up again.


Overhaul duration calculation functions

The current implementation of the relative date display appears to be brittle and actually being broken, for instance with this PR's date being 2 Dec 2024, the date 28 Jan 2024 is being very wrongly displayed as "9 days ago".

This pull request is an attempt to rebuild the computations from the ground up so as to be more consistent and robust:

The relative display format is now implemented using a separate function, taking into account the actual dates in question rather than relying on the information-stripped duration.
The duration display format has a new implementation and no longer relies on a reference date.
Functions applyDuration and elapsedTime now give consistent results.

Some semantics had to be changed and so I expect questions and probably some alterations to the details.

This PR seems too large to reasonably review and looking through some of the changes seem to not really change logic but just refactor for code style which makes it harder to review.

This is a complete rework of the logic, you can see the functions for elapsed time and rounding totally reimplemented with little resemblance to the original code, or you can try comparing outputs from the new implementation against the current one. The vast majority of changes are in the test suites, which I have also rebuilt to more comprehensively test the new behavior.

Is it possible to break this PR down into smaller pieces and optimise for a clean diff?

I could split this into multiple parts if that is what you desire, namely the new elapsed time calculation, the new rounding function, the relative time implementation, and the improved duration application function. However adopting them one by one may lead to inconsistent behavior, unless the splitting is just for the purpose of more convenient code review.

unless the splitting is just for the purpose of more convenient code review.

This is exactly the case. I am happy for us to release as one but it would be nice to get more detail about the changes in a few separate PRs especially when it comes to the tests.

For example we have found users tend to prefer the calendar year being used for such long durations, meaning Jan 2024-Sep 2022 is often rounded up to the much larger "2 years ago" - if we mark it as "1 year ago" we tend to get complaints 😕.

To specifically address this, my new implementation does output this duration as "in 2 years"/"2 years ago", the code only looks at the year difference in this case.
> new Intl.RelativeTimeFormat("en", {amounts: "auto"}).format(...relativeTime(new Date("2022-09-25"), "second", new Date("2024-01-15")))
'2 years ago'

I also think we need to assess each one to see if the new value makes more sense

I completely understand this and so to make it clear I am not in any rush to get this new behavior merged right away, it will of course take some time to discuss all the little details.

This is exactly the case. I am happy for us to release as one but it would be nice to get more detail about the changes in a few separate PRs especially when it comes to the tests.

Thank you for the confirmation. This will be changed to just a draft then and as I make the new set of PRs with more detailed descriptions it will be closed in favor of them.

Closing in favor of split pull requests.


[ADP-1963] publish script integrity hash data in ApiTransaction from stored cbor

I have added script validity field to get /transaction and get /transactions by parsing the CBOR from the transaction history
Comments

Issue Number
ADP-1963

bors try

bors r+


plotman fails if you have a plot going outside the plotman context

$ chia plots create -k 32 -n 4 -b 4000 -r 4 -t /chia-plot0 -d /chia-buffer
$ pip install --force-reinstall git+https://github.com/ericaltendorf/plotman@development
$ plotman status
 plot id    k            tmp                        dst    wall   phase   tmp     pid   stat      mem    user    sys     io
--------   32    /chia-plot1   /mnt/synology/usbshare1/   22:22     ?:?     0   11921    SLP     4.9G   21:47   1:02   4:50
^ one is already running outside plotman context

$ plotman plot

Traceback (most recent call last):
  File "/home/farmer3/chia-blockchain/venv/bin/plotman", line 8, in <module>
    sys.exit(main())
  File "/home/farmer3/chia-blockchain/venv/lib/python3.7/site-packages/plotman/plotman.py", line 143, in main
    wait_reason = manager.maybe_start_new_plot(cfg.directories, cfg.scheduling, cfg.plotting)
  File "/home/farmer3/chia-blockchain/venv/lib/python3.7/site-packages/plotman/manager.py", line 92, in maybe_start_new_plot
    tmpdir = max(rankable, key=operator.itemgetter(1))[0]
TypeError: '>' not supported between instances of 'NoneType' and 'int'

in reference to: https://github.com/ericaltendorf/plotman/issues/199#issuecomment-830660078
also - if you use plotman 1st then kick off a job manually, plotman errors and fails

The following workaround fixed my issue in main:
94,96c94,100
<                         job = Job(proc, logroot)
<                         if not job.help:
<                             jobs.append(job)
---
>                         try:
>                             job = Job(proc, logroot)
>                             if not job.help:
>                                 jobs.append(job)
>                         except Exception as e:
>                             # print("Error:", e)
>                             pass
100c104
<
---
>

Ideally, plotman would also read logfiles from ~/.chia/plotter, but I don't know if that's feasible.

I am interested in adjusting how log files are identified.  This may involve configuring multiple log directories to check or possibly other means of detection.  Though, this issue is for when we still don't find a log file it seems.  Remember that creating a log file is a purely optional thing.  There's no guarantee that chia plots create output is directed to a file.
Anyways, I made an attempt at a fix in #258.  I haven't run it yet but if you want to give it a try, let me know how it goes.


Discontinue TOML support

Do you want to request a feature or report a bug?

Feature
What did you expect to see?
Requesting discontinuation of a useless feature.
Reasons:

YAML is industry standard; no need for multiple configuration formats
TOML doesn't offer any benefit over YAML
TOML is a poor fit for highly nested configuration such as what Traefik tends to have
Maintaining code and documentation for both formats requires unnecessary effort
Having documentation showing examples in multiple formats is confusing for newcomers

Hello,
Traefik v2 support TOML and YAML

@ldez I'm curious, what advantage does TOML have over YAML? Can anyone honestly say this kind of configuration is easy to read: https://docs.traefik.io/reference/dynamic-configuration/file/ ?
The language spec of TOML might be simple, but deciphering such nested configuration files is far from simple. In small projects without highly nested configuration structure, TOML might make sense, but I doubt it makes sense in this case. My main point is that having multiple configuration file formats creates confusion and discourages adoption.
Please at least consider making YAML the default and recommended option.

I talked with a friend about this exact topic and toml makes sense when you don't have 3 level deep nested options but with traefik you easily have 5 or 6 levels deep at which toml repeats itself way to much.


[v0.43.x] Updated Delegators Rewards Query Frequency

Description

Checklist

[x] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[x] Re-reviewed Files changed in the Github PR explorer.

Closing this as we will simply pull the changes into cosmos/v0.43.x once that #199 is merged


Remove the pre-generated pagination.

Issue
Addresses #54

[x] This change addresses only certain aspects of the issue

Description
I had the incorrect type on the axios response, which led to the
metadata being undefined, so no pages were being generated.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have reviewed the changes in a desktop view
[x] I have reviewed the changes in a mobile view
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have assigned reviewers
[ ] I have updated the changelog to include a description of my changes

Make sure this doesn't close the related issue when you merge this.
Also should we just delete these pages instead, since this isn't how pagination is going to work?

Done! And the related issue is already closed now.


Search bar result cannot be opened in editor

Issue Type: Bug
I like the "Open in editor" feature of search bar. But it doesn't work from some point on(I don't know the exactly time), and if I click the link "Open in editor", an error notification occurs like following:
"Unable to open '': Unable to resolve resource search-editor:#0.36733523645139976."
I really like this feature, hoping you fix it.
VS Code version: Code 1.57.0 (b4c1bd0a9b03c749ea011b06c6d2676c8091a70c, 2021-06-09T17:22:31.215Z)
OS version: Darwin x64 19.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 5

Memory (System)
16.00GB (0.69GB free)

Process Argv
-psn_0_21628063 --crash-reporter-id 8cd49c1f-5018-4fc8-ac0b-3c95b4de8155

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

bracket-pair-colorizer
Coe
1.0.61

doxdocgen
csc
1.3.2

vscode-simpler-icons
dav
1.6.5

gitlens
eam
11.5.1

vscode-great-icons
emm
2.1.77

vscode-glsl
GeF
0.2.2

theme-karyfoundation-themes
kar
20.0.3

git-graph
mhu
1.30.0

python
ms-
2021.5.926500501

vscode-pylance
ms-
2021.6.1

jupyter
ms-
2021.6.999406279

cmake-tools
ms-
1.7.3

cpptools
ms-
1.5.0-insiders

cmake
twx
0.0.17

vim
vsc
1.21.2

markdown-all-in-one
yzh
3.4.0

material-theme
zhu
3.10.14

vscode-proto3
zxh
0.5.4

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255cf:30319632
vscgsv2:30307504
vscorehov:30309549
vscod805cf:30301675
binariesv517:30312825

Thanks for the report, do you maybe have
"workbench.editorAssociations": {
    "*": "default",

set in your settings.json?
cc @lramos15 this seems to be a common problem, is there a better way to be contributing these overrides to not need to maintain a separate allowlist for things broken by "*": "default", associations? As is I've been using:
Registry.as<IEditorRegistry>(EditorExtensions.Editors).registerEditor(
	EditorDescriptor.create(
		SearchEditor,
		SearchEditor.ID,
		localize('searchEditor', "Search Editor")
	),
	[
		new SyncDescriptor(SearchEditorInput)
	]
);

which should mean all SearchEditorInput's get opened with the SearchEditor, regardless of workbench.editorAssociations. In this and the welcome page, there's simply no way any other editor could know how to open the input.

cc @lramos15 this seems to be a common problem, is there a better way to be contributing these overrides to not need to maintain a separate allowlist for things broken by "*": "default", associations? As is I've been using:

I am interested and how users are setting this setting as it means they ignore all custom editors. What I suggest is if it truly cannot be opening by anything else then register it as exclusive instead of default as then it will pick yours regardless of setting.

There's no way an extension can open an untitled search editor, but a saved search editor is fine as the backing store is just the text file. I'd expect the override service not to be used in the untitled case, as far as I'm aware it's only for opening file resources via the UI.
I guess the root of the question is why is the override service getting involved with actual Inputs? I'd expect it to manage going from a URI to the Input, but once there's an input shouldn't it correspond to a single view type, as determined by IEditorRegistry#registerEditor?

There's no way an extension can open an untitled search editor, but a saved search editor is fine as the backing store is just the text file. I'd expect the override service not to be used in the untitled case, as far as I'm aware it's only for opening file resources via the UI.

You could register a separate editor entry for the untitled scheme similar to what notebooks do for cells as a temporary solution.

I guess the root of the question is why is the override service getting involved with actual Inputs? I'd expect it to manage going from a URI to the Input, but once there's an input shouldn't it correspond to a single view type, as determined by IEditorRegistry#registerEditor?

This is part of the end goal of the debt, I still have a lot of code to get there. So no we don't know (yet).

cc @lramos15 what's the status on this? Are search-editor schemes now disallowed from participating in override process?

cc @lramos15 what's the status on this? Are search-editor schemes now disallowed from participating in override process?
I got rid of the avenues which caused users to end up with the broken setting. The search-editor schemes are allowed to participate in the override service, if you would like me to disallow them let me know as I'm not familiar with the repercussions there. Here are the following exempt schemes

		Schemas.extension,
		Schemas.webviewPanel,
		Schemas.vscodeWorkspaceTrust,
		Schemas.walkThrough,
		Schemas.vscodeSettings

Are you seeing this as a common issue? It seems like most of the issues are from the "*": "default" setting which should be fixed.

I haven't seen this one as often, we can wait to see if it's fixed by "*": "default" improvements.


rapidfuzz-cpp: 1.1.0 -> 1.1.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/maxbachmann/rapidfuzz-cpp/releases.
meta.description for rapidfuzz-cpp is: "Rapid fuzzy string matching in C++ using the Levenshtein Distance"
meta.homepage for rapidfuzz-cpp is: "https://github.com/maxbachmann/rapidfuzz-cpp"
meta.changelog for rapidfuzz-cpp is: "https://github.com/maxbachmann/rapidfuzz-cpp/blob/v1.1.1/CHANGELOG.md"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.1 with grep in /nix/store/zgwhmlr2zknwiz9dqa6lbvx0gws3hp75-rapidfuzz-cpp-1.1.1
found 1.1.1 in filename of file in /nix/store/zgwhmlr2zknwiz9dqa6lbvx0gws3hp75-rapidfuzz-cpp-1.1.1

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath
puddletag
python310Packages.levenshtein
python310Packages.rapidfuzz
python39Packages.levenshtein
python39Packages.rapidfuzz
rapidfuzz-cpp
spotdl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zgwhmlr2zknwiz9dqa6lbvx0gws3hp75-rapidfuzz-cpp-1.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A rapidfuzz-cpp https://github.com/r-ryantm/nixpkgs/archive/9c488703000f51788450d1e37840759a49b9c745.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zgwhmlr2zknwiz9dqa6lbvx0gws3hp75-rapidfuzz-cpp-1.1.1
ls -la /nix/store/zgwhmlr2zknwiz9dqa6lbvx0gws3hp75-rapidfuzz-cpp-1.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

7 packages built:

puddletag
python310Packages.levenshtein
python310Packages.rapidfuzz
python39Packages.levenshtein
python39Packages.rapidfuzz
rapidfuzz-cpp
spotdl

Maintainer pings
cc @dotlambda for testing.

Result of nixpkgs-review pr 184630 run on x86_64-darwin 1

6 packages built:

python310Packages.levenshtein
python310Packages.rapidfuzz
python39Packages.levenshtein
python39Packages.rapidfuzz
rapidfuzz-cpp
spotdl


refactor: fix warning on new static usage

Description
As it is just a test fixture, I think it's safe to turn it final.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @paulbalandan


doc missing from pypi tarball

Could you please add CHANGELOG.md and LICENSE.md to the pypi release?

I've just released 2.10.0, can you please confirm you now see the .md files? Cheers


how do I stop it from doing that

Describe your bug here. If you are modding, try testing your bug in a clean version of the engine instead. Also, if you're using an older version of JSE, please try the latest version/action build. Also, be sure to check the pinned 'JS Engine: Known Issues' issue, to check if your issue hasn't already been found!

Command Prompt/Terminal/Crash logs (if existing)
No response
Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
put a script in the scripts folder that has to do with the UI
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
a
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.
Yes

sir..
that's a script.
that's an issue with the script, and not the engine itself.

sir..
that's a script.
that's an issue with the script, and not the engine itself.

ok

change it to conversation(General)
how do I fix the script tho

check that the text's camera isn't set to "camGame"

I want the script pls

change it to conversation(General) how do I fix the script tho

ok?

change it to conversation(General)

But why exactly you want that?


Update precommit

IMPORTANT: Please search among the Pull request before creating one.
Title

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] New feature (non-breaking change which adds functionality)

Description

See title
How has this been tested?

NA
Closes

NA

3.9 CI can be updated after new scVelo release, since currently, umap-learn is pinned there (and for 3.9, we need higher numba => higher umap version).


Problems when run example QM9

Command:
CUDA_VISIBLE_DEVICES=0 python scripts/train.py --conf examples/ET-QM9.yaml --dataset QM9 --log-dir output/
Output:
  File "/benchmark/science-computing/torchmd-net/torchmdnet/utils.py", line 153, in __call__     raise ValueError(f"Unknown argument in config file: {key}") ValueError: Unknown argument in config file: distributed_backend
Any problems for the example?

This problem can be solved by removing "distributed_backend" in file "examples/ET-QM9.yaml".

@limin2021 could you make a PR removing that line from the example?

Sorry, I can't follow you. Can you make it clear?

Done.


Changing metrics pairs to string,object to show metrics under Measure Property

@khilan2goel,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

👍  From functional perspective, this PR looks good to me.


Chromeos / Crouton / Crosh support

Feature Request
What feature are you missing?
I'd like to be able to use Prisma services on my Chromebook using crouton via crosh.
How could this feature look like in detail? Tradeoffs?
The problem is that when you launch a new browser window and that window can't be opened prisma will exit with code 3.
Crosh won't open links but you could do what Create React App does and ignore the fact that the browser hasn't opened and print the url which I can type on the browser.
I have run into issues with this using prisma login and prisma playground.
Right now I have to log onto XFCE so that the browser opens when I run playground for instance, then I close XFCE and open the url on chromeos and all is fine.

They use this implementation to open the browser window: https://github.com/facebook/create-react-app/blob/next/packages/react-dev-utils/openBrowser.js
I haven't found Prisma's yet but I'm trying to stitch a PR myself.

I have submitted PRs for those issues.
I guess this can be closed.
Thanks for your awesome work!

Thanks a lot for your contributions @degroote22! We'll review them soon. In the meanwhile, I'll reopen this issue and will close it once those PRs land 💯


Failures on [dart2js] Support pseudo-conflicting supertypes

There are new test failures on [dart2js] Support pseudo-conflicting supertypes.
The tests
co19/LanguageFeatures/nnbd/overriding/override_checking_A06_t042 RuntimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A06_t068 RuntimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A06_t092 RuntimeError (expected Pass)

are failing on configurations
dart2js-hostasserts-strong-linux-x64-chrome

I think this has been fixed, probably by @fishythefish .


nixos-install manual: remove --chroot option

Motivation for this change
This option has been removed from the tool in favour of nixos-enter.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

backport:
[detached HEAD 4647755b9fd] nixos-install manual: remove --chroot option
Author: Linus Heckemann<EMAIL_ADDRESS>Date: Tue May 1 07:36:35 2018 +0100
1 file changed, 11 deletions(-)


An extra 'Tags' shows in the 'upload Files' dialog for one new created blob container with the tags feature is disabled

Storage Explorer Version: 1.16.0-dev
Build: 20201023.5
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one storage account (make sure the tags feature is disabled)-> Blob Containers.
Create a new blob container -> Click 'Upload -> Upload Files...'.
Check whether the 'Tags' shows in the 'Upload Files' dialog or not.

Expect Experience:
No 'Tags' shows in the 'Upload Files' dialog.
Actual Experience:
An extra 'Tags' shows in the 'Upload Files' dialog.
.
More Info:

This issue also reproduces for one new cloned blob container.
This issue also reproduces in the 'Upload Folder' dialog.
This issue doesn't reproduce after manually refreshing the Blob Containers node.

This issue is fixed on build rel/1.16.0_20201030.5. so close it.


Kafka 3.0.0 compatibility

Hi,
Not an issue, just a quick update. We upgraded our Kafka clusters to Kafka 3.0.0 / Strimzi 0.26 . The connector in release 0.9.1 (Scala 2.13 version) is working fine. Compatibility Matrix could then be updated accordingly.
Maybe also add that for those who are using Strimzi to manage their Kafka resources, one must use the Scala 2.12 release of the connector when using Strimzi < 0.25 and the Scala 2.13 release of the connector when using Strimzi >= 0.25.
Regards,
Thibaut

@thibthibus Thanks for the information. That is very helpful for the users. If you could create a pull request to update the documentation, that would be great. Thanks.

@thibthibus  Thanks for the PR.
#128 merged


Feature pg search

Why:

I didn't have access to :solr or :sphinx and wanted to still do a search.
Using PostgreSQL the best searching gem I've used was pg_search

This change addresses the need by:

Add a check for :pg_search option since it works differently than the other searches.
Add searchable to a message.
Remove searchable for receipts if pg_search is enabled.

Is this PR valid? I need to remove my solr dependency and before I disable the search option I want to know if you plan to support pg_search

well I did a little rework and it seems to be working #461


Migrate from github.com/golang/protobuf to google.golang.org/protobuf

Please provide a description of this PR:
xds and go-control-plane don't have any direct dependency with github.com/golang/protobuf but with google.golang.org/protobuf instead
Migrate from github.com/golang/protobuf to google.golang.org/protobuf
This doesn't address every use of github.com/golang/protobuf but reduce it's use

Hi @mmorel-35. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

can you also update https://github.com/istio/istio/blob/master/tools/golangci-override.yaml

/ok-to-test

@mmorel-35: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mmorel-35: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test release-notes

lint_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mmorel-35: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test release-notes

lint_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test lint

integ-ambient_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mmorel-35: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test lint

integ-ambient_istio
130095312e2ab30a252ddde75103e9394ce5c9d6
link
true
/test integ-ambient

release-notes_istio
9b9ae31277fbb880c1a96f47bcb0b1d00b5446f5
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes
No user facing changes, so the label covers it.


Obsolete foot and foot-terminfo

New releases are available on AUR but not in manjaro-sway -- do Github workflows need updating again?
Note that 1) this issue is also blocking an ncurses update, and 2) you may want to add a TERM=foot-extra to foot.ini in order to continue using foot's terminfo after the update.
See the latest comments here: https://aur.archlinux.org/packages/foot/

should we be targetting tmux? I guess for most users, defaulting to xterm-256color would be just fine, wouldn't it?

I think it's fine to simply use the ncurses terminfo, see also here:https://codeberg.org/dnkl/foot/src/branch/master/INSTALL.md#other. In that case, foot-terminfo is not needed anymore and should be uninstalled.
For sshing, the alias in the default .zshrc is to use xterm-256color anyways.

on the other hand: there is no real harm, according to that readme. and a couple of addons for the tmux'ers among us. I'll just merge your patch.

Thanks! foot-extra seems to be incompatible with manjaro-zsh-config, though:
/usr/share/zsh/manjaro-zsh-config:bindkey:86: cannot bind to an empty key sequence
/usr/share/zsh/manjaro-zsh-config:bindkey:87: cannot bind to an empty key sequence

I recommend reverting that change (and hence keep using ncurses terminfo) for now.

I updated the system and now foot is throwing this error when I open it:
/usr/share/zsh/manjaro-zsh-config:bindkey:86: cannot bind to an empty key sequence
/usr/share/zsh/manjaro-zsh-config:bindkey:87: cannot bind to an empty key sequence

zeioth@zeioth-pc Linux 5.14.18-1-MANJARO x86_64 21.2.0 Qonos
/home/zeioth/.zshrc:bindkey:295: cannot bind to an empty key sequence
/home/zeioth/.zshrc:bindkey:296: cannot bind to an empty key sequence
~ >

I am confused. On my machine
/usr/share/zsh/manjaro-zsh-config:bindkey:86: cannot bind to an empty key sequence
/usr/share/zsh/manjaro-zsh-config:bindkey:87: cannot bind to an empty key sequence

occurs only as soon as I uninstall foot-terminfo

This is how I reproduced the error after observing it for the first time:

Override term in ~/.config/foot/foot.ini to term=foot, then run foot (not as a client, but a separate instance) -- observe that everything's working fine
Revert back to term=foot-extra, run foot and observe the error

$ pacman -Q foot
foot 1.10.0-3
$ pacman -Q foot-terminfo
foot-terminfo 1.10.0-1
$ cat .config/foot/foot.ini
include=/usr/share/sway/templates/foot.ini

# rename to foot.ini and add your custom foot overrides hereafter
# font=RobotoMono Nerd Font Mono:size=11
dpi-aware=true
term=foot-extra

I could've sworn I tested before uninstalling foot-terminfo :D
Anyways, re-installing foot-terminfo and reverting to term=foot-extra seems to be resolving the issue for me similar to @boredland.
That still leaves this, though:

I tried with:
yay -S foot-git foot-terminfo-git

With the same result.

In case it helps debugging, to add to the above:
$ yay -Q manjaro-zsh-config
manjaro-zsh-config 0.22-2

soo... this issue is nearly resolved, isn't it? term=foot-extra is part of the defaut ini now and no changes are required from the users perspective.
Only remainder: adding foot-terminfo to the settings package dependencies.

Agreed. Thanks a lot and sorry for the confusion!

My prev issue still remain, I'll research what's going on after work.

Ah, sorry I thought it was the same issue.

Hey sorry for the delay:
I fixed the issue by adding this line to ~/.config/foot/foot.ini
term=foot-extra

The new foot is supposed to get the value of term from ncurses by default (or that's what AUR says), but doesn't seem the case for me.  Ready to be closed.

Doesn't /usr/share/sway/templates/foot.ini already have that line?

I checked and yes it does. Most likely this was caused by my local config:
# I was using include in foot.ini like
~/.config/foot/base.ini → foot.ini
# instead of
/usr/share/sway/templates/foot.ini → foot.ini

This is something I usually do to avoid breaking changes from upstream but this time backfired.
Thank you.


AppBar demo is out of date or bug

I use the following code from the demo:
<AppBar
title="Title"
iconElementLeft={}
iconElementRight={
<IconMenu iconButtonElement={

}>

} />
The right side menu items dont show up when I click the icon. I see a very tiny empty rectangle. When I put text between the <MenuItem..>Text  the Text shows up. But when I click those items..they dont show any sort of animation or a click response..the menu just closes.
This is using the latest (0.13) as of today.

The part of your code and your description is not clear.

I have this issue too, I have read that it comes from the InjectTapEventPlugin, or that the menu-item imported package is the wrong one :(
When you put text into the tags it doesnt work because it isnt working as a button but as a text child.
Have a good day and I hope I have helped you :)

That should be fixed with https://github.com/callemall/material-ui/pull/2382.


[WIP] ARROW-1257: Plasma documentation

Thanks a lot to @crystalzyan who did all the heavy lifting for this PR!

Hey Philipp,
Five things came up when I was reviewing plasma.rst (the plasma python tutorial):
Mac OS X Installation Instructions Don't Work For Me
... I still don't know what's wrong with my mac, but trying to follow the installation instructions for plasma still don't work. I get this error when trying to import plasma:

I've tried the installation instructions with both your pcmoritz/arrow plasma-cython branch, and the actual apache/arrow repo, I did update my dependency packages, but this still happens. Did something go wrong in the install pyarrow + plasma step?
Also, there's a paragraph in the Mac OS X Installation Instructions I had left which goes something like:

Plasma also requires the build-essential, curl, unzip, libboost-all-dev, and libjemalloc-dev packages. MacOS should already come with curl, unzip, and the compilation tools found in build-essential.

This was honestly more of a note to myself than anything, so I'm not sure if this paragraph is still necessary (it might confuse users).
Does PlasmaClient.get Take in only Lists?
I noticed that in the tutorial, all calls made to PlasmaClient.get seem to require list brackets for the argument and return result:

[buffer2] = client2.get([object_id])

It's a little unexpected, but this is how the method's syntax behaves, correct? If so, we could maybe add a sentence in the Getting an Object section that the PlasmaClient.get method only takes in/outputs lists. (in contrast to Ray.get).
Also, if it's like ray.get in that it can get multiple object ids at once, we might want to include a code example of that capability:

Note that client.get takes in the single argument object_id in a list, and outputs the single plasma object in a list. This is because the syntax for client.get supports getting multiple Object IDs as well. To get multiple objects at once, you would similarly pass in-and-out the objects as a list, like follows:

[buffer_A, buffer_B] = client2.get([object_id_A, object_id_B])
Reword the Timeout Explanation
Under the Getting an Object section, you included a mention of the timeout_ms argument for the PlasmaClient.get function:

If the object has not been sealed yet, then the call to client.get will block until the object has been sealed by the client constructing the object. Using the timeout_ms argument to get, you can specify a timeout for this (in milliseconds). After the timeout, the interpreter will yield control back.

These last two sentences are very brief and do not show a code example of the syntax of passing timeout_ms argument, which I would suggest to add as a comparison to a normal call to PlasmaClient.get.
Also, I'll mention that I actually found these two sentences confusing at first, since the word get wasn't even highlighted as code or written out in full, so I thought that it was part of the grammar of the sentence and that it wasn't a mention to the Python function PlasmaClient.get.
We could instead do something like:

If the object has not been sealed yet, then the call to client.get will block until the object has been sealed by the client constructing the object. However, we can limit how long client.get can block by passing in an optional timeout_ms argument.

By setting timeout_ms, we specify a timeout for this function call (in milliseconds). This timeout will force the interpreter to exit client.get early (regardless of success) if the function takes longer than timeout_ms milliseconds. Here is an example of using timeout_ms with client.get:

[buffer2] = client2.get([object_id], timeout_ms=100) // This function will timeout in 100 ms
Pandas Reference Link Broken
In the Storing Pandas DataFrames in Plasma section, I had originally included an rst link to the Using PyArrow with Pandas page of the arrow documentation. This was to let users know that they could check out the conversion charts between pandas and Arrow:

One can instead use pyarrow and its supportive API as an intermediary step to import the Pandas DataFrame into Plasma. Arrow has multiple equivalent types to the various Pandas structures, see the :ref:pandas page for more.

However, this :ref: link is currently broken, since the corresponding link anchor I put in pandas.rst has been removed. We should remove this broken link reference entirely, then.
Include One-Liners for Converting Plasma Objects Back to Arrow/Pandas
This is just an idea for the sake of convenience, but after we explain the users the conversion steps for PlasmaBuffer -> Arrow reader -> Arrow tensor -> numpy array in Getting Arrow Objects from Plasma (similarly, PlasmaBuffer -> Arrow BufferReader -> Arrow RecordBatchStreamReader -> Arrow RecordBatch -> Pandas DataFrame in Getting Pandas DataFrames from Plasma), we could also provide an equivalent condensed one-liner for the code example. This is so to show that all the conversion steps aren't really that intimidating or difficult:
For Arrow:

We can condense the entire procedure described above into one-liners as follows:

# Get the arrow object by ObjectID.
[buf2] = client.get([object_id])

# Equivalent one-liner to convert Plasma buffer back to Arrow tensor
tensor2 = pa.read_tensor(pa_BufferReader(buf2))

# Equivalent one-liner to convert Plasma buffer back to numpy array
array = pa.read_tensor(pa_BufferReader(buf2)).to_numpy()

For Pandas:

The above conversion procedures may seem lengthy, but we can put them all together into a one-liner as follows:

# Fetch the Plasma object
[data] = client.get([object_id])

# Equivalent one-liner to convert Plasma buffer back to Pandas dataFrame
result = pa.RecordBatchStreamReader(pa.BufferReader(data)).read_next_batch().to_pandas()

I pushed a few small changes.
This looks good to me, nice job @crystalzyan :)


Bug report: URL Decode function not properly decoding URLs

Describe the bug:
The "URL Decode" operation in CyberChef is not properly decoding certain URL-encoded strings.
To Reproduce:

Go to CyberChef (https://gchq.github.io/CyberChef/)
Enter the following URL-encoded string in the input box: https://fa.wikipedia.org/wiki/طرح‌واره_درمانی
Add the "URL Decode" operation to the recipe
Run the recipe
The output will be incorrect

Expected behavior:
The "URL Decode" operation should properly decode the input URL-encoded string to the correct URL.
The output should be:
https://fa.wikipedia.org/wiki/طرح‌واره_درمانی
However, the output I am getting is:
https://fa.wikipedia.org/wiki/Ø·Ø±ØâÙØ§Ø±Ù_Ø¯Ø±ÙØ§ÙÛ
Screenshots

Additional context
The issue is not specific to a particular input string and occurs consistently in the current version of CyberChef. It's important to note that copying the decoded output using the "Copy" button works correctly.

You must change encoding.


Antivirus

Hi,
it's not an issue.
but is it possible instead of killing the antivirus to whitelist the Malware?
i think it would be more stealthy, but don't know if it is possible.

I think that's not possible and if it's possible every antivirus will have his own way but I will try my best thank you


Make wiki pages visit fast

Fix #20156
We reuse the code from the repository code view instead of the current code.
Previously it took 5653ms for https://gitea.com/henri/wiki/wiki/?action=_pages in my local machine, now it's about 300ms .

Does this need backport?


Website content is being duplicated again

Look under "Games". 2048 is duplicated twice for Objective-C and Swift.

Issue detail in #903

Started a deploy + clearing cache 🙏


[SPARK-10072][STREAMING] BlockGenerator can deadlock when the queue block queue of generate blocks fills up to capacity

Generated blocks are inserted into an ArrayBlockingQueue, and another thread pulls stuff from the ArrayBlockingQueue and pushes it into BlockManager. Now if that queue fills up to capacity (default is 10 blocks), then the inserting into queue (done in the function updateCurrentBuffer) get blocked inside a synchronized block. However, the thread that is pulling blocks from the queue uses the same lock to check the current (active or stopped) while pulling from the queue. Since the block generating threads is blocked (as the queue is full) on the lock, this thread that is supposed to drain the queue gets blocked. Ergo, deadlock.
Solution: Moved blocking call to ArrayBlockingQueue outside the synchronized to prevent deadlock.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41069/
Test FAILed.

+1. LGTM

Jenkins, test this please.

Test build #1646 has started for   PR 8257 at commit fcb44ce.

Merged build triggered.

Merged build started.

Test build #41081 has started for   PR 8257 at commit fcb44ce.

Test build #41081 has finished for   PR 8257 at commit fcb44ce.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41081/
Test FAILed.

Merged build finished. Test FAILed.

Test build #1646 has finished for   PR 8257 at commit fcb44ce.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

LGTM2

LGTM

LGTM

Test build #1647 has started for   PR 8257 at commit fcb44ce.

Merged build triggered.

Merged build started.

Test build #41103 has started for   PR 8257 at commit cb7bba2.

Test build #41103 has finished for   PR 8257 at commit cb7bba2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41103/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #41119 has started for   PR 8257 at commit cb7bba2.

Test build #1647 timed out     for PR 8257 at commit fcb44ce     after a configured wait of 175m.

retest this please

Merged build triggered.

Merged build started.

Test build #41122 has started for   PR 8257 at commit cb7bba2.

Test build #41122 has finished for   PR 8257 at commit cb7bba2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41122/
Test PASSed.

Merged build finished. Test PASSed.

Test build #41119 timed out     for PR 8257 at commit cb7bba2     after a configured wait of 175m.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41119/
Test FAILed.

Test build #1651 has started for   PR 8257 at commit cb7bba2.

Test build #1651 has finished for   PR 8257 at commit cb7bba2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1652 has started for   PR 8257 at commit cb7bba2.

Test build #1653 has started for   PR 8257 at commit cb7bba2.

Test build #1652 has finished for   PR 8257 at commit cb7bba2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1654 has started for   PR 8257 at commit cb7bba2.

Test build #1653 has finished for   PR 8257 at commit cb7bba2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1658 has started for   PR 8257 at commit cb7bba2.

Test build #1662 has started for   PR 8257 at commit cb7bba2.

Test build #1658 has finished for   PR 8257 at commit cb7bba2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merging this to master and 1.5. Thanks @zsxwing and @harishreedharan for reviewing this.

Test build #1662 has finished for   PR 8257 at commit cb7bba2.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class FilterNode(condition: Expression, child: LocalNode) extends UnaryLocalNode
abstract class LocalNode extends TreeNode[LocalNode]
abstract class LeafLocalNode extends LocalNode
abstract class UnaryLocalNode extends LocalNode
case class ProjectNode(projectList: Seq[NamedExpression], child: LocalNode) extends UnaryLocalNode
case class SeqScanNode(output: Seq[Attribute], data: Seq[InternalRow]) extends LeafLocalNode
public final class UTF8String implements Comparable<UTF8String>, Externalizable

Test build #1654 timed out     for PR 8257 at commit cb7bba2     after a configured wait of 175m.


[FEATURE] Deploy MONA from Azure Marketplace

What if we use Azure Marketplace and a Solution Template offer to deploy MONA in the clients subscription?
I do not have a complete plan on how to do this, this this post is a mix of ideas and questions.
Collect information from the user
A createUiDefinition.json file can be used to collect more information from the user, such as options we have both as required and optional in the basic-deploy.sh script.
Deploy ARM template
How much of the CLI commands in basic-deploy.sh can be created with ARM templates and the variables from createUiDefinition?
Is it possible to run a script during deployment of a Marketplace solution, and if so, could we use basic-deploy.sh in a headless mode with arguments collection from createUiDefinition?
Why would it be worth the time?
When I speak to customers, partners and actually even Microsoft employees, they don't see a clear and easy path to get their solutions to Azure Marketplace. If we could make it as simple as a couple of clicks in the Marketplace I believe we can improve the number of solutions in the marketplace, and also show that we eat our own dog food.
– How do you get started with Azure Marketplace?
– Just install it from Azure Marketplace. :)
Let's discuss if this is a good idea and possible for us to achieve.

I. Love. This. Idea. Unfortunately, we're limited in what we can add to the Azure Marketplace. What if instead, we could simplify the deployment by having the user just click a Deploy to Azure button? Would that be a step in the right direction?

Yes. I think it would be.
Is it still possible to have some sort of UI to ask the user questions that is used as variables in deployment templates? Basically the same thing as we do with options to the cli script today. Resource group, name, etc.
I guess a server endpoint could take a querystring with parameters and generate the arm templates server side if there's no build in way to pass arguments/variables to the "Deploy to Azure" link.

Assuming the ARM template uses parameters and has defaults , then these should be asked for in teh azure UX ,

The portal displays a pane that allows you to easily provide parameter values.
The parameters are pre-filled with the default values from the template. The camel-cased parameter name, storageAccountType, defined in the template is turned into a space-separated string when displayed on the portal.

I thought about this a lot over the holidays. To achieve this in a clean way, I think we'd need to move Mona to containers which I'm definitely open to. We could handle the remaining configuration -- AAD, etc. -- using a deploymentScript action in the bicep template. Let's discuss this further soon. Seems like we might be trending toward containers some anyway per #88.


Elm syntax highlighting

It looks like pygments.rb doesn't support Elm syntax highlighting, even though Pygments does. I'd really like to use Softcover for writing about Elm, but this has become a soft blocker. To further complicate things, it looks like pygments.rb is dead or dying.
What would y'all recommend to get unblocked? I'll write my book without syntax highlighting if I need to, but I'd really prefer having it.
(Also @mhartl thanks for the Rails Tutorial. I learned web dev using it when you first published, and I wouldn't be where I am now without your work. Thank you.)

The pygments.rb project seems active enough, with commits within the last month. If it's really true that pygments.rb doesn't support all current Pygments lexers, you might consider reaching out to see if they might be willing to add them.
At some point, if necessary we could switch Softcover to shell out directly to pygmentize, though that would take some dev work on my part, which unfortunately isn't a top priority right now.
Reaching out to the pygments.rb team is probably your best bet, especially if you can figure out how to add the necessary lexer as a pull request!
Thanks for the kind words about the Rails Tutorial. I'm glad you found it useful!

Will do, thanks @mhartl. For now I've got it working with the Haskell syntax highlighter… there are a few key differences but it works for 90% of what I need.

:+1:


Statistics anomaly : f5_counterBitsIn showing 1.1 Tbps throughput

Environment

Telemetry Streaming Version: 1.12.0
BIG-IP Version: <IP_ADDRESS>

Summary
A clear and concise description of what the bug is.
Please also include information about the reproducibility and the severity/impact of the issue.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "class": "Telemetry",
    "My_Poller": {
        "class": "Telemetry_System_Poller",
        "interval": 60
    },
    "Custom_Endpoints": {
    "class": "Telemetry_Endpoints",
        "items": {
             "detailedCPU": {
                 "name": "detailedCPU",
                 "path": "/mgmt/tm/sys/host-info"
             },
             "systemHostname": {
                 "name": "system",
                 "path": "/mgmt/tm/sys/global-settings"
            }
        }
    },
    "Custom_System_Poller1": {
        "class": "Telemetry_System_Poller",
        "interval": 60,
        "endpointList": "Custom_Endpoints"
    },
    "My_Pull_Consumer": {
        "class": "Telemetry_Pull_Consumer",
       "type": "Prometheus",
        "systemPoller": ["My_Poller", "Custom_System_Poller1"]
    },
    "My_System": {
        "class": "Telemetry_System",
        "enable": "true",
        "systemPoller": ["My_Poller", "Custom_System_Poller1"]
    }
   }

Expected Behavior
Throughput should not exceed values in MBPS. I confirmed the same from F5 statistics GUI .
Actual Behavior
Seeing throughput in Tbps in our grafana dashboards

Are you doing anything in Grafana to convert the bitsIn value from a total count to a rate value?

@dstokesf5
I meant, GUI displayed the correct value. It's not frequent. I couldn't confirm it with iControl REST APIs. We see the deviation in our grafana rate queries.
We are plotting aggregated views of mgmt and network interface. Please, refer to the query in the end
This anomaly is not limited to one metric. Following metrics were also been reported with undesriable values:
f5_counters_bitsOut, f5_numberReqs, f5_2xxResp, f5_serverside_bitsIn
Yes, We are using a rate function as below:
sum(rate(f5_counters_bitsIn{host=~"(.+)"} [2m])) by (host)

It was a spike seen in grafana for networkInterface=1.1 . It lasted only for 1-2 minutes.

How often do you see this data anomaly? It would be very helpful to know what the /mgmt/tm/net/interface/stats iControl REST endpoint and the Prometheus consumer (/mgmt/shared/telemetry/pullconsumers/My_Pull_Consumer) are showing when you see this anomaly. That will help us determine if there is an issue with the iControl REST API, Telemetry Streaming, or your Grafana configuration.

This anomaly is randomly occurring on different nodes in production. Frequency could be weekly once or twice.
How can I get the historical stats using the iControl REST endpoint?

Unfortunately, I am unaware of a way to get historical data from iControl REST.
Do you see this same data anomaly with the new OpenTelemetry consumer? https://clouddocs.f5.com/products/extensions/f5-telemetry-streaming/latest/setting-up-consumer.html#opent


Incorrect @return broke PHPStorm autocomplete

In \BehEh\Flaps\Flaps::getFlap given incorrect PHPDoc: leading backslash missed and this broke PHPStorm' autocomplete (see attached screenshot). IDE resolves BehEh\Flaps\Flap as nested namespace \BehEh\Flaps\Flaps\BehEh\Flaps\Flap.
If I add backslash in @return autocomplete works fine.
    /**
     * Creates a new Flap and returns it, setting default violation handler.,
     * @param string $name the name of the flap
     * @return BehEh\Flaps\Flap the created flap here <- here
     */

Thanks for the report!


(Supersedes #950) Bump elixir

Pull Request Test Coverage Report for Build 2116f47584efefffb76337471a27460519e983e2-PR-953

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.647%

Totals

Change from base Build befbc748d864a212257ce3eeee33c70b1c52a158:
0.0%

Covered Lines:
1058

Relevant Lines:
1564

💛  - Coveralls


Arrow is broken

Arrow is broken in tooltip

That seem like an element (a bullet point?) overlapping the circle :/

Is the arrow itself. I propose we remove it definitely

Oh, true, I didn't notice that!


Deleting a restriction is creating a new restriction in the TR window

While trying to delete a wrongly added restriction (no_u_turn) in iD, through the TR window, ended up creating a new restriction (only_u_turn) in addition to deleting the old one without any visible warnings.

osm Change file:

<?xml version="1.0" encoding="UTF-8"?>
<osmChange version="0.6" generator="iD">
   <create>
      <relation id="-8" version="0">
         <member type="way" role="from" ref="417310791" />
         <member type="way" role="via" ref="573973104" />
         <member type="way" role="to" ref="573973107" />
         <tag k="type" v="restriction" />
         <tag k="restriction" v="only_u_turn" />
      </relation>
   </create>
   <modify />
   <delete if-unused="true">
      <relation id="8155829" version="1">
         <member type="way" role="from" ref="417310791" />
         <member type="way" role="via" ref="573973104" />
         <member type="way" role="to" ref="573973107" />
         <tag k="restriction" v="no_u_turn" />
         <tag k="type" v="restriction" />
      </relation>
   </delete>
</osmChange>

@bhousel
Area in OpenStreetMap

click the way in iD
Delete no_left and no_u turn-restriction
While deleting no_left - first click it changes to only_left (blue) and second click deletes the TR.
While deleting no_u - the first click (in single action) both only_u and deletion is happening (no visible change in colors).

(in gif you can notice, deletion of no_u doesn't change to blue but directly deletes but doesn't give indication of TR creation)
Follow the steps as per to reproduce the problem

Awesome thanks @srividyacb and @abhisheksaikia for the detailed report 👍


fix: first 2 issues in #880

Fixes sizing of searchwithin
Description
This is the fix for the first two issues described in #880
How and where has this been tested?

How this was tested: Using local build of searchwithin
Browser(s) and OS(s) this was tested with: Firefox and Chrome on Mac

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[x] This pull request is ready to merge.

I moved the change in spectrum-tokens-17 which is the branch we are prepping for the next release.
https://github.com/adobe/spectrum-css/commit/533a51b6e87432578ed0902d97bd889fed85817c

min-width: inherit is missing from the merge. Is that intentional?

I tried something different. I can put it back.

@Harbs I put back the inherit


ARROW-2909: [JS] Add convenience function for creating a table from a list of vectors

Simplifies the creating of a Table from JS Arrays:
const LENGTH = 20000;

const idVec = Arrow.vector.IntVector.from(
  Uint32Array.from({length: LENGTH}, () => Math.round(Math.random() * 2E9))
);
const latVec = Arrow.vector.FloatVector.from(
  Float32Array.from({length: LENGTH}, () => Number((Math.random() * 180 - 90).toFixed(1)))
);
const lngVec = Arrow.vector.FloatVector.from(
  Float32Array.from({length: LENGTH}, () => Number((Math.random() * 360 - 180).toFixed(1)))
);

const table = Arrow.Table.fromVectors([idVec, latVec, lngVec], ['id', 'lat', 'lng'])

onsole.log(table.schema.fields.map((f) => f.name));
// [ 'id', 'lat', 'lng' ]

console.log(table.schema.fields.map((f) => f.type));
// [ Uint32 [Int] { TType: 2, children: undefined, isSigned: false, bitWidth: 32 },
//   Float32 [Float] { TType: 3, children: undefined, precision: 1 },
//   Float32 [Float] { TType: 3, children: undefined, precision: 1 } ]

Codecov Report

Merging #2322 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2322      +/-   ##
==========================================
- Coverage   84.46%   84.46%   -0.01%
==========================================
  Files         293      293
  Lines       45112    45112
==========================================
- Hits        38103    38102       -1
- Misses       6978     6979       +1
  Partials       31       31

Impacted Files
Coverage Δ

cpp/src/arrow/util/thread-pool-test.cc
98.91% <0%> (-0.55%)
: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 f019601...9fb6050. Read the comment docs.

@trxcllnt what do you think?

Note the README change in this PR will only work once #2445 is merged

@TheNeuralBit could you rebase?


Implemented ddnss

libdns implementation for  ddnss(.de) a german dns service.

I've done your requested changes. Updated the copyright, too

Question: I've done the changes for caddy-dns too. Where can i make the pull request for it?

@mholt Question: I've done the changes for caddy-dns too. Where can i make the pull request for it?

(Are you using this with Caddy? If so I'll create a repo in caddy-dns for you to make it a Caddy module as well!)


Create an Asset Link URL Resolver

Motivation
Our current website serves up many PDF files. They can be easily found using google.
Such as this: https://www.google.com/search?sxsrf=ALeKk032G0vKTmeNlvlepndVXztTtzAJpg%3A1582830339114&source=hp&ei=AhNYXrCKN5nN0PEP7eyykA8&q=ARRT+rules+and+regulations&oq=ARRT+rules+and+regulations&gs_l=psy-ab.3..0.8034.11005..11150...2.0..0.209.4317.0j24j2......0....1..gws-wiz.....10..35i39j0i131j35i362i39j0i10j0i22i30j0i22i10i30.0Ws3sH2JO30&ved=0ahUKEwiwk9zAtvLnAhWZJjQIHW22DPIQ4dUDCAg&uact=5
You will see that one of the links is this: https://www.arrt.org/docs/default-source/governing-documents/arrt-rules-and-regulations.pdf?sfvrsn=3f9e02fc_42
As you can see it is being served up from our domain.
However, it seems that with Kentico Kontent we do not have flexibility of defining an Asset Link URL Resolver.
We can create Content Item Resolvers, but not Asset Link Resolvers.
Instead what is being surfaced right now in the above example might be something like this:
https://preview-assets-us-01.kc-usercontent.com:443/406ac8c6-58e8-00b3-e3c1-0c312965deb2/ba8df0e1-69da-44f4-b197-9aceafb39979/arrt-rules-and-regulations.pdf
This has some negative ramifications for our business as follows:

Google will drop the PDF from their index and people will no longer be able to find it from our domain.
Google might index it, but users will think it is from an unofficial source (kc-usercontent.com) domain.
When we get around to building our search of our website, will we find the content since it's not from our domain (probably a search engine setup specific issue)

We need the ability to define an asset link resolver in the Kentico client builder registration.
Right now the approach we are attempting is the following:

Create .NET Core middleware that intercepts the response body content and does string substitution.

            app.Use(async (context, next) =>
            {
                var body = context.Response.Body;

                using (var updatedBody = new MemoryStream())
                {
                    context.Response.Body = updatedBody;

                    await next();

                    context.Response.Body = body;

                    updatedBody.Seek(0, SeekOrigin.Begin);
                    var newContent = new StreamReader(updatedBody).ReadToEnd();

                    await context.Response.WriteAsync(
                        newContent.Replace("preview-assets-us-01.kc-usercontent.com:443", "localdevwwwcore.arrt.org/Assets"));
                }
            });

Is this the best way to do it for now?
Not sure on performance or if those kc-usercontent.com links can dynamically change and we need to define a whole list of potential CDN servers?
Proposed solution
Introduce Asset Link Url Resolver, much like the Content Item resolver you already have.
Alternative solutions:

Specify the URL link in Kentico.Ai portal to define this information?
Another alternative is to allow customers to map their own domain to your CDN servers. For instance we could map https://assets.arrt.org to resolve to https://preview-assets-us-01.kc-usercontent.com:443/406ac8c6-58e8-00b3-e3c1-0c312965deb2/ba8df0e1-69da-44f4-b197-9aceafb39979/ so that what is returned in your JSON is already pointing to the correct location

Update. I created a middleware as follows:
    public class AssetLinkMiddleware
    {
        private readonly RequestDelegate _next;

        public AssetLinkMiddleware(RequestDelegate next)
        {
            _next = next;
        }

        public async Task InvokeAsync(HttpContext httpContext, IOptions<ProjectOptionsBase> projectOptions)
        {
            var body = httpContext.Response.Body;

            using (var updatedBody = new MemoryStream())
            {
                httpContext.Response.Body = updatedBody;

                await _next(httpContext);

                httpContext.Response.Body = body;

                updatedBody.Seek(0, SeekOrigin.Begin);
                var newContent = new StreamReader(updatedBody).ReadToEnd();
                string assetUrl = projectOptions.Value.AssetUrl;
                string[] kenticoCdnUrls = projectOptions.Value.KenticoAssetCDNUrls;

                for (int i = 0; i < kenticoCdnUrls.Length; i++)
                    newContent = newContent.Replace(kenticoCdnUrls[i], assetUrl);

                await httpContext.Response.WriteAsync(newContent);
            }
        }
    }

Registered in Startup.cs as follows:
 if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
            }
            else
            {
                app.UseStatusCodePagesWithReExecute("/Error/{0}");
                app.UseExceptionHandler("/Error/500"); //Internal server error occurred
                // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
                app.UseHsts();
            }

            // Add IIS URL Rewrite list
            // See https://docs.microsoft.com/en-us/aspnet/core/fundamentals/url-rewriting
            var options = new RewriteOptions().AddIISUrlRewrite(env.ContentRootFileProvider, "IISUrlRewrite.xml");
            app.UseRewriter(options);

            app.UseHttpsRedirection();

            // Write streamlined request completion events, instead of the more verbose ones from the framework.
            // To use the default framework request logging instead, remove this line and set the "Microsoft"
            // level in appsettings.json to "Information".
            app.UseSerilogRequestLogging();

            app.UseStaticFiles();
            app.UseRouting();

            app.UseWhen(context => context.Request.Path.StartsWithSegments("/webhooks/webhooks", StringComparison.OrdinalIgnoreCase), appBuilder =>
            {
                appBuilder.UseMiddleware<SignatureMiddleware>();
            });

            app.UseMiddleware<AssetLinkMiddleware>();

            app.UseEndpoints(endpoints =>
            {
                endpoints.MapControllerRoute(
                    name: "areas",
                    pattern: "{area:exists}/{controller=Home}/{action=Index}/{id?}");

                endpoints.MapControllerRoute(
                    name: "sitemap",
                    defaults: new { controller = "Sitemap", action = "Index" },
                    pattern: "sitemap.xml");

                endpoints.MapControllerRoute(
                    name: "preview",
                    pattern: "preview/{*urlSlug}", defaults: new { controller = "Preview", action = "Index" });

                endpoints.MapControllerRoute(
                     name: "news",
                     pattern: "news/{action=Index}/{page?}", defaults: new { controller = "News", action = "Index" });

                endpoints.MapControllerRoute(
                    name: "default",
                    pattern: "{controller=LandingPage}/{action=Index}/{codename?}");

                endpoints.MapDynamicControllerRoute<SitemapRouteTransformer>("pages/{**slug}");
            });

My ProjectOptionsBase:
    public class ProjectOptionsBase
    {
        .....
        .....
        public string[] KenticoAssetCDNUrls { get; set; }
        public string AssetUrl { get; set; }
    }

My appsettings.config that maps to my project options base:
  "KenticoAssetCDNUrls": [
    "preview-assets-us-01.kc-usercontent.com:443"
  ],
  "AssetUrl": "localdevwwwcore.arrt.org/Assets",

So let me summarize the requirements and let's see if I understand the problem correctly:

You want to serve assets from your own domain (arrt.org) for SEO reasons (basically)
You want a single source of truth for PDFs.
You want to be able to work with the assets in Kontent - use them in rich text, etc.
You want to store the PDFs in Kontent.

Solutions:
You suggested creating an Asset Link Url Resolver. This is only part of the solution as it'd only fix the links in the rich text elements but the PDFs still wouldn't be served from the correct domain.
Since Kontent doesn't allow mapping custom domains to the CDN servers (which is something that you can request in the roadmap) you'd need to create a proxy/middleware that would translate arrt.org links to CDN links or that would serve the PDFs directly from the arrt.org domain (by downloading the content from the CDN first - you'd probably lose some performance advantage here).
In order to create such a translation table, you'd need to identify the asset in question to get its URL.
To do that, you can utilize the Management API's Asset endpoint or traverse through all content items using the items-feed endpoint. I'd go for the former as the latter is more performance demanding. The same algorithm should then be used in the Asset Link Url Resolver to create the hrefs.
It is possible to bypass all this by creating a wrapping content type called "PDF" with fields like "Name", "URL slug", and "Asset". This way, you can easily render the rich text with correct links + query PDF assets by "URL slug" (which could be extracted from a route and passed to the controller).
Similarly, if (4) is not true and you want to serve the PDF from where they already are you can create a custom content type called "PDF" with "Name" and "URL" elements and link them from your rich text fields.
One more question for you: do you want to preserve the current URLs of the assets? Because if your current route pattern is not deterministic/predictable this can be potentially quite complicated.

The middleware I posted has actually solved this issue for us. (FYI, I updated it above since it had a bug with binary data, made it target only text/html now and that seems to fix it).
What it does is the following:

It rewrites the HttpContext.Response.Body by looking for all instances of the following:

preview-assets-us-01.kc-usercontent.com:443
assets-us-01.kc-usercontent.com:443
It then remaps the that to this URL:
localdevwwwcore.arrt.org/Assets
This makes the following URL:
https://preview-assets-us-01.kc-usercontent.com:443/406ac8c6-58e8-00b3-e3c1-0c312965deb2/ba8df0e1-69da-44f4-b197-9aceafb39979/arrt-rules-and-regulations.pdf
Turn into this:
https://localdevwwwcore.arrt.org/406ac8c6-58e8-00b3-e3c1-0c312965deb2/ba8df0e1-69da-44f4-b197-9aceafb39979/arrt-rules-and-regulations.pdf
Then our AssetsController.cs:
    public class AssetsController : BaseController<AssetsController>
    {
        private readonly IOptions<ProjectOptions> _options;
        private readonly IAssetService _assetService;
        public AssetsController(IDeliveryClient deliveryClient, ILogger<AssetsController> logger, IOptions<ProjectOptions> options,
            IAssetService assetService) : base(deliveryClient, logger)
        {
            _options = options;
            _assetService = assetService;
        }

        public async Task<IActionResult> Index(string urlFragment)
        {
            //Check if the binary data is in our distributed cache based off of the unique url Id
            Logger.LogInformation("Retrieving asset URL: {urlFragment}", urlFragment);

            //TODO: Check if this data is in the distributed cache to avoid a call to Kentico Kontent
            var asset = await _assetService.GetKenticoAsset(_options.Value.KenticoAssetCDNUrls, urlFragment);

            //If not in there, go fetch the binary data using an HttpClient and put into cache
            System.IO.File.WriteAllBytes("C:\\Temp\\" + asset.FileName, asset.FileData);

            //Return the file result data
            if (asset != null)
                return File(asset.FileData, asset.ContentType, asset.FileName);

            Logger.LogInformation("Could not find Kentico Asset at URL: {urlFragment}", urlFragment);
            return new NotFoundResult();
        }
    }

Code of GetKenticoAsset:
 /// <summary>
    /// Kentico Asset Service
    /// Provides a way to get information about kentico assets to serve up from our own domain
    /// </summary>
    /// <remarks>
    /// 2/27/2020 - MRO: Initial creation
    /// </remarks>
    public class AssetService : IAssetService
    {
        private readonly HttpClient _httpClient;
        private readonly IMimeMappingService _mimeMappingService;

        public AssetService(HttpClient httpClient, IMimeMappingService mimeMappingService)
        {
            _httpClient = httpClient;
            _mimeMappingService = mimeMappingService;
        }

        public async Task<KenticoAsset> GetKenticoAsset(string[] cdnUrls, string urlFragement)
        {
            var asset = new KenticoAsset();

            //Build all the original possible kentico cdn Urls
            List<string> cdnFullUrls = new List<string>();
            foreach (var itm in cdnUrls)
            {
                cdnFullUrls.Add($"https://{itm}/{urlFragement}");
            }

            byte[] data = await _httpClient.GetByteArrayAsync(cdnFullUrls[0]);

            string fileName = Path.GetFileName(urlFragement);

            asset.ContentType = _mimeMappingService.Map(fileName);
            asset.FileData = data;
            asset.FileName = fileName;
            asset.KenticoUrls = cdnFullUrls.ToArray();

            return asset;
        }
    }

    public interface IAssetService
    {
        Task<KenticoAsset> GetKenticoAsset(string[] cdnUrls, string urlFragement);
    }

I hope this make sense what I'm asking for. Either a Asset Link resolver that doesn't make me write all this code to re-write the HttpContext.Response.Body.
Here is a mockup of a proposal for Alternative Solution #1 above (Specify the URL link in Kentico.Ai portal to define this information?)
So when the JSON comes back it's already altered to go to the correct place. We would then just go construct the CDN url and fetch it with our own HttpClient:

BTW, I updated the code in my replies above, so please re-review.

Here is information that might be helpful for Alternative solution # 2 (Using DNS CNames to point to your Fastly CDN): https://docs.fastly.com/en/guides/working-with-domains
If you surfaced a way to map our domain to your CDN in app.kontent.ai settings page perhaps that is another way to solve this. Then we could use your CDN servers (which would be ideal) and we still would get the benefit of the content appearing to come from our domain (assets.arrt.org for example)

Hi, I did some investigation on how other customers build their applications with custom domains for assets. They mentioned using a CDN and setting the rules for caching to save some cross-site requests and traffic. At the moment, building a middleware service, as you did, would be the best way to do asset white labeling. By the way, since there are already some requests for this feature, we've added it to considering section on our roadmap. You can vote there for features that are important for you as well as submit your own ideas.
Best regards,
Lukas Turek
Product Manager

@lukasturek I tried finding that feature request on your roadmap but couldn't find it. The link appears to be broken.

@lukasturek I tried finding that feature request on your roadmap but couldn't find it. The link appears to be broken.

I have adjusted the link

FYI, we had to back out the middleware we were using. I couldn't get it to work 100% of the time (weird blank page issue randomly) when used in conjunction with Microsoft's response caching middleware :(

So you're back with the CDN links, right? We have no immediate plan to address this, we first need to stabilize the v14 release. I guess we can take a look at this one then. Could you perhaps draft a PR addressing the issue here? It might speed up things.
(Sorry for my late responses...We just had a baby so I was busy changing dipers :))

Congrats!
Yeah we are using the cdn links now. You want a pull request containing what?

thanks! :)
PR of the suggested "Asset Link Url Resolver"...if that's what would help you solve the issue (e.g. replace the current middleware). We can at least start iterating from there to see if it's a way to go.

So I played with this a bit more and I can't repro it anymore (atleast locally). It may have something to do with our hardware load balancer as well. So, more investigation is needed.

ok. I'll leave this open and wait for more input.