VS Code Crashes When SMB Volume Gets Disconnected

Issue Type: Bug
If VS Code opens a file from a SMB Volume (say /Volumes/mounted_drive/*), and the respective Volume gets disconnected (e.g. network outage, timeout), then VS Code will freeze and crash. In the past, this also led to problems with the Finder, which meant that my Mac needed a hard reboot (Force Kill wouldn't work). I assume this is because of how VS Code deals with deleted files, but I don't know enough about the source to have an opinion.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:57:51.166Z)
OS version: Darwin x64 19.2.0
Fetching remote diagnostics for 'ssh-remote+tigergpu.princeton.edu' timed out.

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.44GB free)

Process Argv

Screen Reader
no

VM
0%

Fetching remote diagnostics for 'ssh-remote+tigergpu.princeton.edu' timed out.
Extensions (12)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.0.11

gitlens
eam
10.2.0

beautify
Hoo
1.5.0

vscode-map-preview
jum
0.4.8

python
ms-
2019.11.50794

remote-ssh
ms-
0.48.0

remote-ssh-edit
ms-
0.48.0

Go
ms-
0.12.0

material-icon-theme
PKi
3.9.2

java
red
0.55.1

markdown-all-in-one
yzh
2.6.1

material-theme
zhu
3.2.1

(2 theme extensions excluded)

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

Same problem, this is the info from the console application
Process:               Code Helper (Renderer) [2633]
Path:                  /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper (Renderer).app/Contents/MacOS/Code Helper (Renderer)
Identifier:            com.github.Electron.helper
Version:               0
Code Type:             X86-64 (Native)
Parent Process:        Electron [2626]
Responsible:           Electron [2626]
User ID:               501
Date/Time:             2020-01-09 13:32:01.768 +0100
OS Version:            Mac OS X 10.15.2 (19C57)
Report Version:        12
Bridge OS Version:     3.0 (14Y908)
Anonymous UUID:        39B90B9D-92DB-4634-9889-F5C50E50410D
Sleep/Wake UUID:       E5AF2BA5-7101-4B5D-89E5-0AB7B60F88D4
Time Awake Since Boot: 13000 seconds
Time Since Wake:       36 seconds
System Integrity Protection: enabled
Crashed Thread:        0  CrRendererMain  Dispatch queue: com.apple.main-thread
Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY
Application Specific Information:
abort() called
Thread 0 Crashed:: CrRendererMain  Dispatch queue: com.apple.main-thread
0   libsystem_kernel.dylib        	0x00007fff658647fa __pthread_kill + 10
1   libsystem_pthread.dylib       	0x00007fff65921bc1 pthread_kill + 432
2   libsystem_c.dylib             	0x00007fff657eba1c abort + 120
3   com.github.Electron.framework 	0x0000000105ed8b51 0x100543000 + 93936465
4   com.github.Electron.framework 	0x0000000105ed868c 0x100543000 + 93935244
5   com.github.Electron.framework 	0x0000000105ec8041 uv_run + 401
6   com.github.Electron.framework 	0x00000001028b6fd1 0x100543000 + 37175249
7   com.github.Electron.framework 	0x000000010293bfd0 0x100543000 + 37720016
8   com.github.Electron.framework 	0x000000010294b4bb 0x100543000 + 37782715
9   com.github.Electron.framework 	0x000000010294b997 0x100543000 + 37783959
10  com.github.Electron.framework 	0x00000001029aa103 0x100543000 + 38170883
11  com.github.Electron.framework 	0x00000001028e199a 0x100543000 + 37349786
12  com.github.Electron.framework 	0x00000001029a9a6f 0x100543000 + 38169199
13  com.apple.CoreFoundation      	0x00007fff2e25bb21 CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION + 17
14  com.apple.CoreFoundation      	0x00007fff2e25bac0 __CFRunLoopDoSource0 + 103
15  com.apple.CoreFoundation      	0x00007fff2e25b8d4 __CFRunLoopDoSources0 + 209
16  com.apple.CoreFoundation      	0x00007fff2e25a740 __CFRunLoopRun + 1272
17  com.apple.CoreFoundation      	0x00007fff2e259bd3 CFRunLoopRunSpecific + 499
18  com.apple.Foundation          	0x00007fff308fd188 -[NSRunLoop(NSRunLoop) runMode:beforeDate:] + 212
19  com.github.Electron.framework 	0x00000001029aa611 0x100543000 + 38172177
20  com.github.Electron.framework 	0x00000001029a9438 0x100543000 + 38167608
21  com.github.Electron.framework 	0x000000010294be68 0x100543000 + 37785192
22  com.github.Electron.framework 	0x000000010291dbc7 0x100543000 + 37596103
23  com.github.Electron.framework 	0x0000000105c3b270 0x100543000 + 91193968
24  com.github.Electron.framework 	0x0000000101c7cdcf 0x100543000 + 24354255
25  com.github.Electron.framework 	0x000000010401a960 0x100543000 + 61700448
26  com.github.Electron.framework 	0x0000000100f073d4 0x100543000 + 10240980
27  com.github.Electron.framework 	0x0000000100545854 AtomMain + 84
28  com.github.Electron.helper    	0x00000001004fa705 0x1004f9000 + 5893
29  libdyld.dylib                 	0x00007fff6571d7fd start + 1

Here are my log files! I actually have two crash files in the console. Here is the info:
Crash_Archive.zip

I'm also encountering this issue, almost every day since I work on network files.
Please fix it, and tell me if you need some more info.

@antoniomacri can you check the latest insiders https://code.visualstudio.com/insiders and if the crash happens, please attach the crash report following the steps at https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues

I am still bumping into this issue often, since I work mainly with files on a removable drive. Is a fix in the works?

Merging to https://github.com/microsoft/vscode/issues/106879


Port to 4.0

An issue relating to accessing a pointer outside an object is preventing us from getting the noise height for terrain generation, which is needless to say important. Along with that, OpenSimplexNoise has been replaced with FastNoiseLite, it'd be wise to begin transitioning focus and work on 4.0

After some digging, as of now 4.0 isn't stable enough to even begin writing a GDExtension for chunk generation, along with the decision to use Rust instead of C++ for generation.
So till Rust 4.0 bindings release, 4.0 isn't possible!


[Feature Request] Download lyrics from other sources if Spotify doesn't have them.

Spotify doesn't show the lyrics from Musixmatch sometimes, even if it's on it. Then if it's not on Musixmatch, it could look on rclyricsband.com, lrclib.net, Genius and maybe some others.

As there's every chance they won't add this feature, for a similar feature I would recommend using an open source android app called SongSync. Should be able to find the github pretty easily and it has more options than the Spotify lyrics should they be unavailable


Only allow one body parameter

Per the Swagger 2.0 specification, an operation can have only one body parameter and we need to enforce this.
Originally reported here: https://github.com/apigee-127/swagger-tools/issues/461

I think this can be closed:

check in validators here
test for it here

I verified on API studio that the proper error is thrown when two body params are defined.

Good catch.


NVIDIA-SMI has failed because it couldn't communicate with the NVIDIA driver. Make sure that the latest NVIDIA driver is installed and running.

Version
Microsoft Windows [Version 10.0.22000.318]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

When I used the installed Ubuntu (Microsoft store name, the version is 20.04) before, the result is COULD NOT communicate. Still, the Ubuntu that I installed the new Ubuntu 20.04 by Microsoft Store is regular and return the expected result.
Expected Behavior

Actual Behavior

Diagnostic Logs
No response

I am having the same problem with Windows 11, WSL2, Ubuntu 22.04, Driver NVIDIA-SMI 528.49

I am having the same problem with Windows 11, WSL2, Ubuntu 22.04, Driver NVIDIA-SMI 528.49

Maybe you can create an new WSL2 instance, it is working for me.

My environment:
Three Ubuntu images on WSL2 with Windows 11:

UBuntu
Ubuntu-22.04
Ubuntu-20.04

I Tried the following:

close all ubuntu terminal sessions
wsl --shutdown
wsl -d Ubuntu-22.04                                                          : OK
Try: NVIDIA-SMI  [ed@mlSys23:/mnt/c/Users/EdSey]                             : OK

NVIDIA-SMI 525.89.02 Driver Version: 528.49 CUDA Version: 12.0

cd
Try: NVIDIA-SMI                                                              : OK

NVIDIA-SMI 525.89.02 Driver Version: 528.49 CUDA Version: 12.0

wsl --shutdown
wsl -d Ubuntu
Try: NVIDIA-SMI  [ed@mlSys23:/mnt/c/Users/EdSey]                             : NF

Since Ubuntu was woking in the past:

I was running pyTorch with both CPU & GPU
some sequence of events has caused the link from the display driver
to the Ubuntu image to be broken

Since I have a couple of months of development on the Ubuntu image,
Does anyone have thoughts on how to re-establish the linkage to the display
driver for CUDA without removing the Ubuntu image and re-installing?


Break down design for deployment environment

Design and deploy website manual for woffle

Couscous is to be used for this, currently it does not push or load the associated CSS so it only renders as intended locally. This should be fixed.


[stable/prometheus-operator] CoreDNSDown alert is not created for coredns

Describe the bug
The alert CoreDNSDown is only created when kube-dns is used, not when coredns is used.
Version of Helm and Kubernetes:
helm v3.0.2
k8s v1.17.2
Which chart:
stable/prometheus-operator
What happened:
The promehteus alerts does not get created when coredns is used.
What you expected to happen:
The alert CoreDNSDown gets created when coredns is used.
How to reproduce it (as minimally and precisely as possible):
Install prometheus-operator with coredns.enabled = true
Anything else we need to know:

As mentioned in #20609, this should be fixed upstream.
But taking a look at the script which integrates these alerts into this repository, the alert is part of kube-prometheus release-0.1. This release only supports k8s up to version 1.14. I installed prometheus-operator on k8s 1.17.
So, shouldn't this alert be installed at all?


Add: n_repeated_element_in_size_2n_array

Codecov Report

Merging #89 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #89   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          36     37    +1
  Lines         405    412    +7
=====================================
+ Hits          405    412    +7

Impacted Files
Coverage Δ

...ze-2n-array/n_repeated_element_in_size_2n_array.go
100% <100%> (ø)

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 b2d9012...41f8bc1. Read the comment docs.


What is lazy evaluation in Bacon.js?

I am not able to understand what is lazy evaluation in Bacon.js.
I wrote the example provided by Bacon using map and flatMap and I get the same result.
Here is the HTML
<input id="itemname" type="text" />
<input id="additem" type="button" value="Add Item" />
<input id="purchase" type="button" value="Purchase" />

Here is the JS for code using map
var items = $("#additem").asEventStream("click").map(function(e){
    console.log("Executing");
    return document.getElementById("itemname").value;
}).toProperty();

var submittedItems = items.sampledBy($("#purchase").asEventStream("click"));

submittedItems.onValue(function(){})

Here is the JS for code using flatMap
var items = $("#additem").asEventStream("click").flatMap(function(e){
    console.log("Executing");
    return document.getElementById("itemname").value;
}).toProperty();

var submittedItems = items.sampledBy($("#purchase").asEventStream("click"));
submittedItems.onValue(function(){})

For both the version of JS there is nothing logged even if I click the buttons. According to documentation the second one should output "Executing" message on the console.
Both the code works if I attach a subscriber using onValue.
Please help me understand what's wrong?

There's two kinds of "laziness" involved in Bacon.js:

When an Observable has zero subscribers, it will not connect to its data source. As a result, nothing will happen unless you add at least one subscriber. Subscribers are added using onValue/ forEach. The log method also adds a subscriber.

Some methods, such as map and combine* also evaluate the event value lazily. This means that the function that you give to map as an argument won't be called unless the value is needed. This brings performance benefits in some cases such as when you combine a bunch of values from different sources using Bacon.combineWith but you need only some of the values, which you'll extract using sampledBy. In this case, the function given to combineWith will only be called for the values that are actually extracted using sampledBy.

For comparison, the first kind of laziness is implemented in other libraries too (rxjs, kefir at least), while the second kind is (afaik) only implemented in Bacon.js. I hope this helps!

I updated the FAQ with regard to laziness. https://github.com/baconjs/bacon.js/wiki/FAQ

@raimohanska The above example is to demonstrate the second point. But why does it work the same way even if I use flatMap?

It's right there in point 1: I cannot see any subscribers in your example. If a tree falls and no-one is listening, it doesn't actually fall :)

Added subscriber in the code. When I use flatMap and click on the purchase button after clicking on add item nothing happens. But it works in case of using map. What's the issue now?

Cannot see your code, so it's hard for me to tell.

I have provided all the code above. That's is all the code.

Ah cool, updated code above. Lemme see..

Cannot say with brief look. A fiddle or similar executable demonstration case would help.

I removed JQuery stuff, and tried the following.
var Bacon=require("../dist/Bacon")
var addE=Bacon.Bus()
var submitE=Bacon.Bus()
var items = addE.flatMap(function(e){
      console.log("Executing", e);
      return e;
}).toProperty();

var submittedItems = items.sampledBy(submitE);
submittedItems.onValue(function(x){
  console.log("submitted", x)
})

addE.push("new item")
submitE.push("submit")

Results are as expected:
➜  bacon.js git:(master) node testing/660.js
Executing new item
submitted new item
```

Same results when replacing `flatMap` with `map`.

Here is the jsFiddle https://jsfiddle.net/u01ajhsf/4/
When I use flatMap the value of the text field is never logged.

Oops it's working now. Anyways thanks for the clarifications. Awesome library :)

:) No problem, mate! I'm glad you got it working. The FAQ is now also a bit improved so I guess this was a win-win.


Publish Core Package

[ ] Writeup project content description
[x] Change name and repo in Cargo.toml
[x] Change scope in create_wasm.sh
[x] Run create_wasm.sh
[x] Check pkg/package.json
[x] Run npm pack --dry-run
[x] Run npm publish

Hi @hildjj - i transferred the project and refned the create_wasm.sh a bit to make a cargo clean beforehand and check for wasm-pack.
However - the package.json being generated is nearly ready to be published. But with the initial thoughts on naming you proposed calling the npm release @one-ini/wasm and i absolutely agree with this attempt.
For the cargo release the package might be named one-ini_core or one-ini but for other platforms other naming conventions might be better suiitable (like having a @one-ini/js in the future. Unfortunately wasm-pack does not allow (afaik) to adjust the package name.
So i'd like propose an "automated" adjustment of the package name in the package.json to @one-ini/wasm. I would prefer having this codified in a implicit build tool (i thought about the build.rs but it seems not suitable for tasks after the build.
A future evolvement could break out the core library for parsing and ast generation to the "core" lib and creating a separate repo for the wasm generation and packing. But i'd suggest to wait for that step until things settled at 1.0.
Any thoughts?

I think we also need to change to the default allocator before release.
The following line added to create_wasm.sh should fix the package name:
node -e 'n="./pkg/package.json";p=require(n);p.name="@one-ini/wasm";fs.writeFileSync(n,JSON.stringify(p,null,2))'

I like the idea of splitting this up a little in a future version, since for editorconfig I don't need the serde_json bits.  But let's publish what we have now.

I published version 0.1.0 to npm.

I think we also need to change to the default allocator before release.

I added WeeAlloc as global memory allocator, it reduced the export size by a few kilobytes. Unfortunately WeeAlloc was recently enlisted as unmaintained risk. In a short research i couldn't find an alternative.
In general i have the feeling the package is too big but i'll have to dig into it to find out more about the actual sizes.


Exhaustive Generic Join Test Cases

We need to test all the different combinations of tile groups, value types, join types, and join algorithms. This needs to be generic so that we can easily re-use the code whenever we add a new join executor.

Added a generic join test suite in 97a5dcc233f7c20dd55e41317d0d3ee1a459faf0. It evaluates different join types (Inner, Left outer, Right outer, Full outer) and join algorithms (Nested loop join, Merge join, Hash join). We deal with logical tiles here, so we don't worry about tile group layout in this test suite. We do value type testing in another test suite.

Added a generic join test suite in 97a5dcc233f7c20dd55e41317d0d3ee1a459faf0. It evaluates different join types (Inner, Left outer, Right outer, Full outer) and join algorithms (Nested loop join, Merge join, Hash join). We deal with logical tiles here, so we don't worry about tile group layout in this test suite. We do value type testing in another test suite.


Outdated macOS build

The macOS build currently only targets x86_64 architecture. Adding an arm64 binary or replacing the current x86_64 binary with Universal 2 is necessary for it to run on macs from 2021 onwards.

Thanks for the heads up, I've updated the build script.
Can you try the most recent build if it works for you?
I'm not a mac user, so I can't really test it myself.

I assume this is fixed with 21c1c81360421d5332f56d07c7580a7e6ed3c1f6.


Doc: fix example in Array sort_custom()

Currently, example code does not work, as return is outside of sort().

Cherry-picked for 3.0.7.


fix apt gpg/asc

switch from deprecated apt_key to get_url with gpg/asc

I'd prefer using the method of deb822_repository (requires Ansible 2.15, so would require bumping the role requirements too).


Remove special case for freebsd in favor of later shell validity check

Signed-off-by: Scott Hain<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@chef-cookbooks/ben-team @tas50

(this will be covered with this check: https://github.com/chef-cookbooks/users/pull/415)


Use legacy branch of Migen/MiSoC submodules

Migen/MiSoC is evolving. That's probably not the right time to switch.
So we now have to use use legacy branches of Migen/MiSoC instead of master.

Same for LiteEth and LiteScope: we now need to clone legacy branches.


How to override sequence functions.

Hi, I just want to know how can I hide the original sequence (next, back) controls and use my own controls.
I'm going to use a slider so I need to do this.
So the question is, exist any way to simply call a function to call the next image in sequence.
Thank you

I just want to know how can I hide the original sequence (next, back) controls and use my own controls.

You need to set showSequenceControl to false: http://openseadragon.github.io/docs/OpenSeadragon.html#Options

So the question is, exist any way to simply call a function to call the next image in sequence.

Yes, you need to use viewer.goToPage and viewer.currentPage methods
http://openseadragon.github.io/docs/OpenSeadragon.Viewer.html#goToPage
http://openseadragon.github.io/docs/OpenSeadragon.Viewer.html#currentPage

Thanks a lot, this works perfectly.


npm build fails because of slot inside table

Error
The build fails with the error
ERROR in ./src/DataTable.vue

  64 | 					</td>
  65 | 					<slot name="tbody-tr" :row="row"></slot>
    |      ^
  66 | 				</tr>

  Tag <slot> cannot appear inside <table> due to HTML content restrictions. It will be hoisted out of <table> by the browser.

Reproduce

run npm install
run npm run build

Discussion
Are the slots actually in use? Can't find something in the documentation.

I did not expect people to actually use this library so I have hardcoded some functions oriented to my needs. Now that it's becoming more popular and this should be removed.
The purpose of those slots were to add the ability of putting buttons in the datatable, so to manage rows.
See how I actually used those slots:

Putting a th / td around the slots solves the build error. Do the slots work for you with this change as expected?

That would work but that would take up a column space even when the user does rather prefer to not use the slots at all (right?).
If the <th>/<td> happens to take up space when empty then that would be a purposed solution.

could this be fixed by checking if the slot has data with a v-if?
like this

@rodrigoAbril Yeah sounds well. This bug should be closed, regardless of how I use it in my project. The problem is that one needs to be able to add multiple th/tds, not just one.
Other than that, the solution looks good, I think. The reason for this weird feature is to able to add things like buttons and such inside the datatable in each row. I also suggest documentation that, as it's entirely undocumented now, and that needs a concrete example on how to get buttons to work, because I remember that not being so easy.

Does this bug still exist


[Bug] The app is very slow on my Mi Tv Stick 4K, the audio and video are delayed and out of sync, please fix this

Device
TV
Operating System
Android
Browser
Other
Browser Version
125.0.6422.14
Better xCloud Version
0.11.0 (14)
Reproduction Steps
When opening the app it takes time to load, the game loads slowly, and when playing the audio is delayed and out of sync.

https://github.com/redphx/better-xcloud-android/issues/41


curlconverter adds content-type even if original request doesn't have any

Headers from firefox:

Headers from curlconverter:

what's the original curl command? firefox is probably generating a curl command that implicitly adds a content type.

Oh yeah, i forgot the command:
curl 'https://cwip-shaka-proxy.appspot.com/no_auth' --compressed -X POST -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/127.0' -H 'Accept: */*' -H 'Accept-Language: it-IT,it;q=0.8,en-US;q=0.5,en;q=0.3' -H 'Accept-Encoding: gzip, deflate, br, zstd' -H 'Origin: https://bitmovin.com' -H 'Connection: keep-alive' -H 'Referer: https://bitmovin.com/' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: cross-site' --data-raw "DATA"
As you can see there isn't that header

If we run a netcat server
nc -kl 8888

and change the command to send a request to it
curl 'http://localhost:8888/no_auth' --compressed -X POST -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/127.0' -H 'Accept: */*' -H 'Accept-Language: it-IT,it;q=0.8,en-US;q=0.5,en;q=0.3' -H 'Accept-Encoding: gzip, deflate, br, zstd' -H 'Origin: https://bitmovin.com' -H 'Connection: keep-alive' -H 'Referer: https://bitmovin.com/' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: cross-site' --data-raw "DATA"

we can see that curl does send that header
POST /no_auth HTTP/1.1
Host: localhost:8888
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/127.0
Accept: */*
Accept-Language: it-IT,it;q=0.8,en-US;q=0.5,en;q=0.3
Accept-Encoding: gzip, deflate, br, zstd
Origin: https://bitmovin.com
Connection: keep-alive
Referer: https://bitmovin.com/
Sec-Fetch-Dest: empty
Sec-Fetch-Mode: cors
Sec-Fetch-Site: cross-site
Content-Length: 4
Content-Type: application/x-www-form-urlencoded

DATA

so this is a bug in Firefox's generation of curl commands, it needs to account for implicit curl headers https://hg.mozilla.org/mozilla-central/file/tip/devtools/client/shared/curl.js

https://bugzilla.mozilla.org/show_bug.cgi?id=1921645


Add each fronius sensor as own template

Breaking Change:
Description:
This update adds the often used templates to the state engine per default and hides the overall components. The reason is that there is almost no use case without the templates and they are tricky yet repetitive to define.
Related issue (if applicable): as an example issue 11 on the libraries page
Pull request with documentation for home-assistant.io:  home-assistant/home-assistant.io#10025
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

Please rebase to solve the merge conflict.


Log to file for go-machine-service

Moving issues into rancherio/rancher.
https://github.com/rancherio/rancher/issues/320


chore(RHTAPREL-730): update taskGitRevision default

This commit updates the default parameter of taskGitRevision in the development branch pipelines to be development instead of main.

This commit updates the default parameter of taskGitRevision in the development branch pipelines to be development instead of main.

OK, but I wonder what the effect of this is. The only place that creates these pipelines is the release controller, right? And hopefully it should provide the right value for this. (If not today, then soon.) So ideally we wouldn't have to worry about this at all. Which would be great, because then we wouldn't need to make any changes when promoting stuff from development to stage to prod.

This commit updates the default parameter of taskGitRevision in the development branch pipelines to be development instead of main.

OK, but I wonder what the effect of this is. The only place that creates these pipelines is the release controller, right? And hopefully it should provide the right value for this. (If not today, then soon.) So ideally we wouldn't have to worry about this at all. Which would be great, because then we wouldn't need to make any changes when promoting stuff from development to stage to prod.

Huh? Right now, if you point your RPA at the development branch of the release-service-catalog repo, your pipeline is going to run with all the tasks from the main branch (which is now read only) instead of the development branch. That is why we need this PR. Our goal is to one day have the controller pass the revision based on what overlay it is deployed with, but we are not close to having that implemented and we cannot run any versions of the pipelines beyond what is in main today if we don't merge this

This commit updates the default parameter of taskGitRevision in the development branch pipelines to be development instead of main.

OK, but I wonder what the effect of this is. The only place that creates these pipelines is the release controller, right? And hopefully it should provide the right value for this. (If not today, then soon.) So ideally we wouldn't have to worry about this at all. Which would be great, because then we wouldn't need to make any changes when promoting stuff from development to stage to prod.

It is also in the acceptance criteria here https://issues.redhat.com/browse/RHTAPREL-730 (I just forgot about it)

Huh? Right now, if you point your RPA at the development branch of the release-service-catalog repo, your pipeline is going to run with all the tasks from the main branch (which is now read only) instead of the development branch. That is why we need this PR. Our goal is to one day have the controller pass the revision based on what overlay it is deployed with, but we are not close to having that implemented and we cannot run any versions of the pipelines beyond what is in main today if we don't merge this

Hmm, ok, then I guess we need this for now. At least it shouldn't be so painful now that it's set only in one place per pipeline.

It is also in the acceptance criteria here https://issues.redhat.com/browse/RHTAPREL-730 (I just forgot about it)

OK, you're right. It also says that the other two branches should use the correct references too. So two questions:

Why is RHTAPREL-730 already closed if the AC are not met yet?
Where are the PRs to modify the staging and production branches similar to this PR? ;)

Because I completely forgot about this part
I was planning to use my promotion script for that as a test since no one is using staging or production yet anyways

Because I completely forgot about this part
I was planning to use my promotion script for that as a test since no one is using staging or production yet anyways

ok, no problem
oh, you already have a promotion script, cool! in that case disregard my other comment about doing this manually for now in another (merged) PR ;)

Because I completely forgot about this part
I was planning to use my promotion script for that as a test since no one is using staging or production yet anyways

ok, no problem
oh, you already have a promotion script, cool! in that case disregard my other comment about doing this manually for now in another (merged) PR ;)

Very WIP promotion script ;)
But there will be PRs soon for the staging and production branches
I reopened the JIRA


[stable/postgresql] Add desaintmartin as maintainer.

What this PR does / why we need it:
Since there is no longer any official maintainer of stable/postgresql, I propose myself. Hopefully I am not the only one. :)
List of contributions to this repo: https://github.com/helm/charts/pulls?utf8=✓&q=is%3Apr+author%3Adesaintmartin+
cc @unguiculus for approval
cc @cpanato for reference

@desaintmartin

need to bump the chart version
need to sign the DCO  (https://www.helm.sh/blog/helm-dco/index.html)

and i will defer to the helm charts mantainers to check this

Done!

still need to bump the chart version

cc @mattfarina @scottrigby
cc @carrodher as future owner

/ok-to-test
/lgtm


chore: promote h52 to version 0.1.2-SNAPSHOT

chore: promote h52 to version 0.1.2-SNAPSHOT
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Showing sTokens positions on the Me page

What would you like to share?
List the user's sTokens on the /me page.
Mock:

Additional information
No response
Would you like to work on this issue?

[ ] Yes, I want to work on this issue!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

#14 fixes this


Force the matrix to have only integers as values

These floats crash the program when running the normalization in custom mode.

It also sounds like the 3D matrix choice is never selected. Bug to fix.


Recycle Bin shown as Shell:RecycleBinFolder

Description
The Recycle Bin is having errors. Please look into the Steps to Reproduce on how that error occured. I have described it there. I have no idea what is causing this kind of behaviour, so I don't know  what the issue is. Thanks!
Steps To Reproduce

Ensure that The Files App is configured to open at Start

Open the  app

You will notice at the sidebar that the Recycle Bin is shown as Shell:RecycleBinFolder and after clicking into it, an error pops up.

You can also fix it by closing the files app and quiting the background process.

Restart the Files App and the Recycle Bin is restored as it should be and also working as normal

Requirements
This Error should be fixed as soon as possible as it gets annoying to restart the Files App everytime that happens.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4169
User ID
files-community
Log File
debug.log

Thanks for the report.
This is the same as #15083. Click Subscribe to get notified of the further info on this when we have an update.

One thing I can tell since I got started working on shell namespaces, navigations via shell namespaces in some place seem to be working.

Can it be fixed by myself as said in the workaround  in Issue #15083?

Try it and see


adding slack#/types/date

Summary
Adds slack#/types/date to the Schema. At runtime it's a string (it's format is YYYY-MM-DD).
Testing
To test this you can use an Open Form step and add a field of Schema.slack.types.date - it should render a date-picker like in the screenshot.

You can embed that output into a Send a message step's message input and you should see the date string you selected, like so:

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Do we have an example of what it looks like to set this value, either as a default or when as a parameter passed to a function? Is it just the below?
MyWorkflow.addStep(FnThatAcceptsDate, {
  date: "2022-08-19"
});

Do we have an example of what it looks like to set this value, either as a default or when as a parameter passed to a function? Is it just the below?
MyWorkflow.addStep(FnThatAcceptsDate, {
  date: "2022-08-19"
});

Yup, I was able to default my form's date like that:
{
  name: "when",
  title: "When",
  type: Schema.slack.types.date,
  default: "2022-08-09",
}


fix: make prompter multi-select indicator more distinct

Description of changes

Updates the selected/unselected indicator in prompt multi-select lists to be more distinct
Before the indicator was a green vs gray ✓, now it is a blue ● or gray ○
PR also includes lint fixes. The only functional change is lines 269-271
Issue #, if available

Description of how you validated changes
Manually validated. There's not really a way to write a unit test for this because it is just tweaking a parameter on the enquirer input
Checklist

[x] PR description included
[x] yarn test passes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #11742 (93cedc2) into dev (984c0f8) will increase coverage by 0.77%.
The diff coverage is 63.71%.

@@            Coverage Diff             @@
##              dev   #11742      +/-   ##
==========================================
+ Coverage   49.45%   50.23%   +0.77%
==========================================
  Files         692      705      +13
  Lines       33424    34087     +663
  Branches     6840     7005     +165
==========================================
+ Hits        16529    17122     +593
- Misses      15370    15484     +114
+ Partials     1525     1481      -44

Impacted Files
Coverage Δ

...y-appsync-simulator/src/resolvers/unit-resolver.ts
90.32% <0.00%> (ø)

...ages/amplify-appsync-simulator/src/schema/index.ts
70.68% <0.00%> (-1.25%)
:arrow_down:

...tor/src/velocity/util/transform/dynamodb-filter.ts
4.93% <0.00%> (ø)

...ync-simulator/src/velocity/util/transform/index.ts
21.05% <0.00%> (ø)

...c/provider-utils/awscloudformation/import/index.ts
31.02% <ø> (ø)

...vice-walkthrough-types/cognito-user-input-types.ts
100.00% <ø> (ø)

...wscloudformation/utils/auth-sms-workflow-helper.ts
43.75% <0.00%> (ø)

...ges/amplify-category-custom/src/commands/custom.ts
0.00% <0.00%> (ø)

...ion/service-walkthroughs/lambdaLayerWalkthrough.ts
11.21% <0.00%> (ø)

...mation/service-walkthroughs/scheduleWalkthrough.ts
16.49% <0.00%> (ø)

... and 214 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding FlyByWire Statistics app

Purpose
An app that tracks the number of flights using the FlyByWire Simulations systems, with a nice animation that works well for the Normal rate, but even for the Turbo speed. For Normal speed it will float the FlyByWire Logo and the count from the left, and exit the frame to the left. For other speeds, it will float in, then stay and be replaced with the next app.

About FlyByWire Simulations
FlyByWire Simulations is a non-profit group of volunteers who build FOSS (GPL licensed) add-ons and infrastructure for Microsoft Flight Simulator 2020 (and 2024 in the future). Our current mainline add-on, called A32NX, is a representation of the A320Neo for MSFS and we are actively working on a representation of the A380 as well. All of this is done out of love and passion for the Flight Simulator Community and our interest in development in this field.
Our goal is to provide not only high quality add-ons for free to the community, but also allow others to use our base systems and installer platform for their own add-ons (there are other organizations using our systems for their A330, A319, A321, ... aircraft for MSFS). By doing so we hope to enhance the quality and collaboration amongst the community, while enjoying ourselves, learning from each other and building strong friendships.

I have read the CLA Document and I hereby sign the CLA


Structure Change for Deaths

Has the structure changed for api calls for information relating to deaths?  I keep getting 404 error codes using this structure:
Request failed .... 404 - Not Found
Response .......... {"response":"Invalid parameter 'newDeathsByDeathDate' in the requested JSON structure. Did you mean 'newDeaths28DaysByPublishDate'?","status_code":404,"status":"Not Found"}
Tried checking the documentation and there is no mention of this field.
Changing the field to newDeaths28DaysByPublishDate, stopped the python error, but no data was returned.
Have I missed a change announcement somewhere?
Thank you.

Release of death figures as ...DeathsByDeathDate has been discontinued.
Re the newDeaths28DaysByPublishDate, could you please share your code here?

I can answer nimzoking’s query having spent some time looking at this particular issue.
The documentation defining the valid list of metrics for the API’s structure was apparently updated on 13th of August 2020 at 17:30pm. In order to find the list, one has to click on the link ‘See a list of valid metrics for structure’ in the Developer’s guide on the Coronavirus dashboard (https://coronavirus.data.gov.uk/developers-guide).
The API documentation on https://publichealthengland.github.io/coronavirus-dashboard-api-python-sdk/pages/examples/general_use.html#structure points to the same Developer’s guide for the metrics.
It is not very helpful that the examples of how to format the Structure on both websites include two metrics are no longer valid; ie "newDeathsByDeathDate" and "cumDeathsByDeathDate".

Thank you, I did missed the URL to expand, and was simply concentrating on the examples.
That said, I still get no data returned.
Example Return:
{'data': [{'Date': '2020-08-17', 'areaName': 'Manchester', 'newCasesByPublishDate': 21, 'cumCasesByPublishDate': 3809, 'newDeaths28DaysByPublishDate': None, 'cumDeaths28DaysByPublishDate': None}], 'lastUpdate': '2020-08-17T14:15:27.000000Z', 'length': 1, 'totalPages': 1}
Is None the expected result for the cumulative cases?  I am getting a result from cumulative cases, but nothing for cumulative Deaths.  Can I safely assume, no deaths in previous 28 days?
Structure I have set up:
cases_and_deaths = {
"Date": "date",
"areaName": "areaName",
"newCasesByPublishDate": "newCasesByPublishDate",
"cumCasesByPublishDate": "cumCasesByPublishDate",
"newDeaths28DaysByPublishDate": "newDeaths28DaysByPublishDate",
"cumDeaths28DaysByPublishDate": "cumDeaths28DaysByPublishDate"
}

Hi Carl,
I have just checked with the website to see whether death data is available yet -  https://coronavirus.data.gov.uk/deaths?areaType=region https://coronavirus.data.gov.uk/deaths?areaType=region&areaName=North West &areaName=North%20West. Unfortunately, it is only available at the Nation level.
I ran the attached script and got data for England, but nothing when I tried the North West region. No point in trying Manchester yet – patience is required!
Regards
Martin
From: Carl Tillotson<EMAIL_ADDRESS>Sent: 17 August 2020 21:57
To: publichealthengland/coronavirus-dashboard-api-python-sdk<EMAIL_ADDRESS>Cc: grandpamartin<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [publichealthengland/coronavirus-dashboard-api-python-sdk] Structure Change for Deaths (#12)
Thank you, I did missed the URL to expand, and was simply concentrating on the examples.
That said, I still get no data returned.
Example Return:
{'data': [{'Date': '2020-08-17', 'areaName': 'Manchester', 'newCasesByPublishDate': 21, 'cumCasesByPublishDate': 3809, 'newDeaths28DaysByPublishDate': None, 'cumDeaths28DaysByPublishDate': None}], 'lastUpdate': '2020-08-17T14:15:27.000000Z', 'length': 1, 'totalPages': 1}
Is None the expected result for the cumulative cases? I am getting a result from cumulative cases, but nothing for cumulative Deaths. Can I safely assume, no deaths in previous 28 days?
Structure I have set up:
cases_and_deaths = {
"Date": "date",
"areaName": "areaName",
"newCasesByPublishDate": "newCasesByPublishDate",
"cumCasesByPublishDate": "cumCasesByPublishDate",
"newDeaths28DaysByPublishDate": "newDeaths28DaysByPublishDate",
"cumDeaths28DaysByPublishDate": "cumDeaths28DaysByPublishDate"
}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/publichealthengland/coronavirus-dashboard-api-python-sdk/issues/12#issuecomment-675110556 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQUPFW4QFGHPVNQC2FV5WATSBGKQFANCNFSM4P7X53AA .  https://github.com/notifications/beacon/AQUPFW3IN5FDL3WLLA7H3XTSBGKQFA5CNFSM4P7X53AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFA6V5HA.gif
from uk_covid19 import Cov19API
filt = [
'areaType=nation',
'areaName=England'
]
struct = {
"date": "date",
"newDeaths28Days":"newDeaths28DaysByPublishDate",
"cumDeaths28Days":"cumDeaths28DaysByPublishDate"
}
api = Cov19API(filters=filt, structure=struct)
data = api.get_csv(save_as="England deaths_1.csv")
print(data)

Hi @grandpamartin, have you tried using the newDeaths28DaysByPublishDate and cumDeaths28DaysByPublishDate metrics? The previous metrics have been retired and these new ones have been added.
I will update SDK docs as soon as I find a bit of time in my diary. Apologies for the inconvenience.

@nimzoking Yes, it is the expected answer. Cases by PublishDate are only available for nations, not UTLAs or LTLAs.
Try newCasesBySpecimenDate or cumCasesBySpecimenDate instead. You should get the full data then.
Hope this helps.

Yes, but I only get the data for nations. Nothing for regions which the dashboard says
Is also available on the new Deaths page. Try again tomorrow.
Sent from my iPad
Martin
On 17 Aug 2020, at 11:03 pm, Pouria Hadjibagheri<EMAIL_ADDRESS>wrote:

Hi @grandpamartin, have you tried using the newDeaths28DaysByPublishDate and cumDeaths28DaysByPublishDate metrics? The previous metrics have been retired and these new ones have been added.
I will update SDK docs as soon as I find a bit of time in my diary. Apologies for the inconvenience.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@grandpamartin Oh you mean the 28-day deaths. Yes, it was only the latest at LA-level today. We are waiting for the data to come through... It'll be pushed to the database and released as soon as we receive it. Our ETL, database, and API infrastructures are set up and ready to go...

I have just checked the Dashboard and found that Daily deaths within 28 days of positive test by date reported are now shown for Nations and Regions. The North West region has the most at 6 new deaths. By a little trial and error using the API I found that Kirklees has 1 newDeaths28DaysByPublishDate. The cumDeaths28DaysByPublishDate column is empty. When I looked at Manchester, 0 is shown for the new deaths and the cumulative column is also empty. Seems to mean that the API is working down to utla level at least.
Incidentally, cases are being shown for Nations and below by Specimen Date (as a time series) and for the latest date only by Publish Date as well. It seems that the UK case data is only by Publish Date since Northern Ireland does not submit by Specimen Dates (I think).

@grandpamartin cumDeaths28DaysByPublishDate is indeed empty. We're working on it. We added a bunch of new metrics today again... do have a look.
The API is working regardless of level, it's just that we don't have the data at all levels yet.
We don't have all the metrics for every region. This is primarily because different DAs/LAs have been producing different metrics. We are working to standardise them and make at least one metric available for everything... but it takes some time.


tikz support [was: mathjax graphics]

mathjax should support tikz graphics and animations!
tikz animations are javascript!!

I been doing some digging because of my own hypothetical use case and found some info possibly relevant to this thread, along with a few brainstorm ideas to add.  If any parts are not seen as so relevant, my apologies in advance, this just seemed a likely appropriate place to brain dump some of my research results on this subject:
If I understand correctly from looking here: (https://www.ctan.org/pkg/pgf?lang=en) Tikz is built on PGF, and both are largely built with LaTeX macros.  It looks like there is some C/C++ code and some LUA in the project source (https://www.ctan.org/tex-archive/graphics/pgf/base) but most of it looks to be tons of TeX files containing macros (https://www.ctan.org/tex-archive/graphics/pgf/base/tex).
If my admittedly limited understanding is correct (LaTeX, PSTricks, PGF and TIKS are all new to me as of about 48 hours ago) maybe all that would be needed is porting some of the LUA and C/C++ code to javascript (or maybe compiling them 'as is' for a Nodejs plugin?) and then the functionality could just be lifted from the TeX files in the original project.
Issues with this I can currently guess at are:

That the huge amount of TeX macros in the PGF project might take a long time to (pre?)process as there are many of them, and some of them look pretty large.

I don't think any of this stuff was originally built with real time rendering in mind, which might be a problem if Tikz/PGF in browser is then supposed to support real time changes.  Seems there are also 3D rendering packages for these (like for visualizing volume integrals and much more, etc), but these are almost probably built with software rasterizing through TeX 2D drawing commands (admittedly this is a guess on my part).  A web implementation would want to use WebGL if it was going to show real time changes, although even draws to a 2D canvas would presumably be faster than making SVG and the other current formats.

Just brainstorming here, but maybe some javascript regex could easily convert <LaTeX/PGF/TIKS/other...> scripts to a JSON parsable object array of interpret-able commands describing render calls to Canvas and/or WebGL.  That might be relatively fast and get an initial implementation working.  However it will not solve the obvious optimization problems when using existing macros for rendering 3D via 2D TeX/PGF that exist from the old-school packages.  On the other hand, once the common existing Macros/Packages were identified, a WebGL implementation of them could come with the system or be plug-able side project spin-offs for the main system.
Anyway, just some thoughts after a couple days of pondering and researching the subject.  Hopefully they help somebody or strike a new idea/approach for somebody reading.

Thanks for sharing your thoughts on this. Just a few points form our end, kind of a summary of what we (the MathJax team's) have encountered in discussions around this.

Reach out to tikz's author.

As we've written in the past, the team does not have sufficient expertise to even consider re-implementing pgf/tikz. It solves a radically different problem than TeX math mode does so it doesn't fit into MathJax architecture at this time (which focuses entirely on math mode).
We had a meeting with Till Tantau a few months ago. He's great, he's interested in the problem of making tikz happen on the web; he lacks time / resources.

Like TeX, pgf/tikz is tool for print layout. There will be many features that do not make sense on the web. (For example, referencing nodes outside the current picture, cf. section 16.13ff in the pgf manual.)
if you only need server-side processing in nodejs, then perhaps building a latex bridge is the better choice (a generic one like https://www.npmjs.com/package/latex-file, or a specialized one like https://www.npmjs.com/package/flowtex). tikz already has a tex4ht driver that generates SVG.

of course you can also try to transpile all the things, cf. https://github.com/manuels/texlive.js.

There are sufficiently good web-based tools for most tikz use cases. If you walk through the tutorial section of the tikz manual, it's hard to find something you don't have multiple options for already: for geometry, jsxgraph, geogebra, mathbox etc; for petri-nets, UML etc there is jointjs, js-sequence diagrams, jsplumb and many others; for graphs, bar, and pie charts there are a gazillion things out of the data visualization world. So, just like MathJax, one idea might be to look for a suitable subset of tikz and focus on high-quality conversion instead of faithful implementation. That is, use it as an API to generate code to feed into one of the major graphics libraries supports. Doing this for narrow tizk-based packages might also be an option (e.g. chemfig).

In any case, we're happy to help and share our knowledge. But MathJax does not have any plans (or resources) to pursue graphics at this time.

What about setting up a mathjax-based tikzjax?

tikzwolke.com converts script tags (containing TikZ code) into SVGs and there's a live demo at https://demo.tikzwolke.com/ and the code is at https://github.com/kisonecat/tikzwolke
In the <head> of your HTML, include
<script src="https://tikzwolke.com/v1/tikzwolke.js"></script>

Then in the <body>, include TikZ code such as
<script type="text/tikz">
  \begin{tikzpicture}
    \draw (0,0) circle (1in);
  \end{tikzpicture}
</script>

Ultimately the TikZ is compiled in the cloud, but the TikZ source is hashed and Amazon S3 is used as a cache.

The code at https://github.com/kisonecat/tikzjax now provides TikZ running entirely in the browser.
(With web2js, the Pascal source for e-TeX is compiled to WebAssembly; it's run under node and interrupted after loading the LaTeX format and TikZ, and the resulting core dump is shipped to the browser to very quickly get the browser to a place where TikZ is running.  With dvi2html the DVI is converted to SVG.)


fix: updates the default url for self-hosted Keploy

the default port of Keploy server is updated to 6789, therefore the default url in server-config needs to be updated in Keploy class.

The default url is updated in this PR. Closing this PR


Change email regex

I'm submitting a ...
[X] bug report
[X] feature request
[ ] question about the decisions made in the repository

Change email regex: http://emailregex.com/
 let EMAIL_REGEXP = /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/;

Hey @Newan, thanks for the information, would you be able to provide a pull request?

O.k was created #817


[HUDI-7384] [HUDI-7405] [secondary-index] Secondary index support

Initial commit. Supports the following features:
1. Modify schema to add secondary index to metadata
2. New partition type  in the metadata table to store secondary_keys-to-record_keys mapping
3. Various options to support secondary index enablement, column mappings (for secondary keys) etc
4. Initialization of secondary keys
5. Update secondary keys on inserts/upsert/deletes
6. Add hooks in HoodieFileIndex to prune candidate files (to scan) based on secondary key column filters.
7. Add ability in HoodieMetadataLogRecordReader to use unmerged log record scanner (HoodieUnMergedLogRecordScanner)
8. Use the unmerged log record sacnner to buffer records from secondary index partition (of metadata table)
9. Add support for merging secondary index records (from delta log files and base files)
Limitations:
1. Supports only one secondary index at the moment.
2. HoodieUnMergedLogRecordScanner uses a in-memory buffer to store records and subsequently merge the records
3. Scanning of the secondary index partition is done sequentially (both on the query side and the index-maintainance side)
Pending items:
1. Integrate with compaction
Change Logs
Initial commit. Supports the following features:
1. Modify schema to add secondary index to metadata
2. New partition type  in the metadata table to store secondary_keys-to-record_keys mapping
3. Various options to support secondary index enablement, column mappings (for secondary keys) etc
4. Initialization of secondary keys
5. Update secondary keys on inserts/upsert/deletes
6. Add hooks in HoodieFileIndex to prune candidate files (to scan) based on secondary key column filters.
7. Add ability in HoodieMetadataLogRecordReader to use unmerged log record scanner (HoodieUnMergedLogRecordScanner)
8. Use the unmerged log record sacnner to buffer records from secondary index partition (of metadata table)
9. Add support for merging secondary index records (from delta log files and base files)
Limitations:
1. Supports only one secondary index at the moment.
2. HoodieUnMergedLogRecordScanner uses a in-memory buffer to store records and subsequently merge the records
3. Scanning of the secondary index partition is done sequentially (both on the query side and the index-maintainance side)
Pending items:
1. Integrate with compaction
Impact
Support secondary index on columns (similar to record index, but for non-unique columns)
Risk level (write none, low medium or high below)
Medium. New and existing tests
Documentation Update
NA. Will be done later
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Rebase and resolve conflicts. Fix a bug related to MOR tables with secondary index.

Moved away from using HoodieUnMergedLogRecordScanner.  Added new buffer in HoodieMergedLogRecordScanner (based on SpillableDiskMap) to handle non-unique keys (secondary keys)

Hi @bhat-vinay! Is this design of secondary index through MDT is the only one to be implemented or there plans to make some other Index Types? As I remember there was RFC for Lucene Index and maybe some other types in future?

Hi @bhat-vinay! Is this design of secondary index through MDT is the only one to be implemented or there plans to make some other Index Types? As I remember there was RFC for Lucene Index and maybe some other types in future?

Please get in touch with @codope for the latest update on this. AFAIK, lucene based secondary index is not planned at this time and MDT based secondary index is the one being developed.

Hi @skyshineb , we do plan to add more index types. If you are interested in contributing to lucene based secondary index, I can help you to get started with multi-modal indexing framework.

hi @codope! I planned to test this MDT implementation and the Lucene(one which I took from previous SI attempt and finished myself). And figure out is it profitable to use Lucene or not. But why this PR got closed?


Automate the updating of the application version matrix (Kubeflow v1.0 onwards)

From Kubeflow v1.0 onwards, the docs contain an application version matrix: https://master.kubeflow.org/docs/reference/version-policy/#kubeflow-application-matrix
The table shows, for each application, the application version that is in the manifest at the time when  the current version of Kubeflow (such as Kubeflow v1.0.0) was released.
It would be nice to automate the updating of the version numbers in the table.

Hello @sarahmaddox .
Just reviewing this issue and wondering if you have further suggestions or a list of requirements?
Thanks!

This is not my template, I this this is a good start to see what might come handy in the future.
https://github.com/kaushalmodi/ox-hugo/blob/master/test/site/content/real-examples/multifractals-in-ecology-using-r.md

Hallo @kbhawkey this issue came up in a discussion with @jlewi and @abhi-g and @jbottum . The version numbers of the applications are held in the manifest files in this repo:
https://github.com/kubeflow/manifests
We'd need some script to run (perhaps as part of the Kubeflow release process) that pulls the relevant number out of the relevant file and plugs the number into the page. We'd probably need to use parameters, something like what this PR did:
https://github.com/kubeflow/website/pull/1620
That's about as much info as I have. @jlewi do you have any further suggestions?


Cannot read property 'name' of undefined

element = this.validationTargetFor( this.clean( element ) );
this.lastElement = element;
this.prepareElement( element );
3 lines from jquery.validation
first line makes the element become undefined if it is in the ignore list for jquery validation
so the $("element").valid() throws an error "Cannot read property 'name' of undefined"
What you should do is to make it console.log("Element  is in the ignore list, see validator.settings.ignore")
I wrestled with this problem for 2 days because I didn't know about the ignore list and all I was getting was that stupid error and I keep my required checkboxes hidden because I use their labels for visual styles

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually, especially after several years with no activity. It doesn't mean I'm abandoning the project, just that I'm unable to work through 200+ open issues with the little time I can afford to spend on this project.
To the reporter (or anyone else interested in this issue): If you're still affected by the same issue, please consider opening a new issue, with a testpage that demonstrates the issue with a current version of the plugin. Even better, make an attempt to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


ci: Add macos-14 to CI

Relies on https://github.com/rhysd/actionlint/issues/393, or we add a config file in the meantime

It has been announced that macos-latest will switch from 12 to 14 in a few months, so it may be better to specify the version without using latest.

It has been announced that macos-latest will switch from 12 to 14 in a few months, so it may be better to specify the version without using latest.

Should we keep 12 after that change, though?

Should we keep 12 after that change, though?

I suspect that even at this stage there is no need to keep 12, since macOS cross-compilation seems to work fine, wouldn't it be enough to just switch from latest to 14? Due to differences in machine performance, 14 tends to finish processing faster.
The smaller size of the matrix will also ease CI congestion.

I left the old mac-os ones there for the moment; we can remove when this becomes latest, or I'm fine with someone doing it sooner if they think that's better


Split out the caching and source-map-fixing into separate modules.

NYC now does 3 things:

Installs an (optionally cacheable) transform via append-transform that captures coverage and source-map data.
Remaps istanbul reports using source-map data.
Uses spawn-wrap to make all this happen across forked processes.

I think 1 and 2 need to be split out.

:+1:

Remaps istanbul reports using source-map data.

There's https://github.com/SitePen/remap-istanbul but it already does way too many things besides remapping the reports.
Even if 1 and 2 are split out they may still end up coupled to each other. Not sure I see the point, unless they're moved into Istanbul itself maybe.

I think 1 would almost certainly depend on 2 (but not necessarily the other way around).

Not sure I see the point

It would allow all our hard work getting source-maps and caching working to be used in other contexts.

unless they are moved into Istanbul itself maybe

That may be the ideal solution, but I have got to imagine that there is a good reason that source-map support hasn't found it's way to master yet over there.
// @gotwarlost

It would allow all our hard work getting source-maps and caching working to be used in other contexts.

Sure, assuming those other contexts discover the modules.

Sure, assuming those other contexts discover the modules.

You take the risk of being ignored every time you type npm publish. That's just life.
For my part, I am really curious how much overhead spawn-wrap and foreground-child add? How much faster would AVA be if we used those split out modules and provided a --coverage option that called them in every forked process? Our babel.js script runs before any user scripts, so we can already provide many of the guarantees that spawn-wrap provides for NYC. Of course, that wouldn't cover users who are going to be forking again from within their test files. In that case - they need to use NYC.

@jamestalmage it looks like @gotwarlost has been working on his own source-map library as part of the istanbuljs org, could we look at merging efforts?

Please correct me if I'm wrong but this is resolved by the existence of caching-transform and istanbul-lib-source-maps?

I think caching is resolved and any further source-map work should be incorporated to istanbul-lib-source-maps.


Polymorphic Through

Using: "sequelize-typescript": "0.4.0-beta"
I'm having some trouble figuring out how to do polymorphic model definitions so that the queries work as expected. I'm trying to do a Subscription table eg:
Table: Subscription
integer: userId
integer: targertId
string: targetType  -> One of USER / POST / ETC

I'm focusing on getting USER to work first. I want to be able to get all the users a particular user has subscribed to. So I have something like this:
class User extends Model<User> {
...
    @HasMany(
    () => Subscription,
    {
      foreignKey: "targetId",
      scope: {
        targetType: "USER",
      },
    }
  )
  usersSubscribedTo: IUser[];
...
}

class Subscription extends Model<Subscription> {
...
  @ForeignKey(() => User)
  @AllowNull(false)
  @Column
  userId: number;

  @BelongsTo (
    () => User,
    "userId",
  )
  user: IUser;

  @AllowNull(false)
  @Column
  targetId: number;

  @AllowNull(false)
  @Column
  targetType: string;

  @BelongsTo (
    () => User,
    {
      foreignKey: "targetId",
      constraints: false,
      scope: { targetType: "USER" },
    },
  )
  userSubscribedTo: User;
...
}

This looks kind of ok to me, but with this I will get:
error TS2345: Argument of type '{ foreignKey: string; constraints: false; scope: { targetType: string; }; }' is not assignable to parameter of type 'AssociationOptionsBelongsTo | undefined'.
  Object literal may only specify known properties, and 'scope' does not exist in type 'AssociationOptionsBelongsTo | undefined'.

Update
How could I achieve it if I treat Subscription as a through table:
  @HasMany({
    () => User,
    through: () => Subscription,
    constraints: false,
    scope: { targetType: "USER" },
  }, "targetId")
  usersSubscribedTo: IUser[];

So you want to implement a many-to-many relation, where the subscription table is the through table?

Correct. So that ultimately I can query things like:
user.subscribedUsers
user.subscribedPosts
post.subscribedUsers

Did you tried something like:
@Table
export class User extends Model<User> {

    @BelongsToMany(() => User, () => Subscription, 'subscriberId', 'userId')
    usersSubscribedTo: User[];
}

@Table
export class Subscription extends Model<Subscription> {

    @PrimaryKey
    @ForeignKey(() => User)
    @Column
    subscriberId: number;

    @PrimaryKey
    @ForeignKey(() => User)
    @Column
    userId: number;
}

If you remove the subscriberType column then that's not polymorphic.

Ah, sry, my fault. This would be the proper way then:
@Table
export class User extends Model<User> {

    @Column
    name: string;

    @BelongsToMany(() => User, {
        through: () => Subscription,
        foreignKey:  'subscriberId',
        otherKey: 'targetId',
        constraints: false,
        foreignKeyConstraint: true,
        scope: {
            targetType: 'user'
        }
    })
    usersSubscribedTo: User[];

    @BelongsToMany(() => Post, {
        through: () => Subscription,
        foreignKey:  'subscriberId',
        otherKey: 'targetId',
        constraints: false,
        foreignKeyConstraint: true,
        scope: {
            targetType: 'post'
        }
    })
    postsSubscribedTo: Post[];
}

@Table
export class Subscription extends Model<Subscription> {

    @PrimaryKey
    @ForeignKey(() => User)
    @Column
    subscriberId: number;

    @PrimaryKey
    @Column
    targetId: number;

    @Column
    targetType: string;
}

But when querying the data, the created sql string doesn't seem to work unfortunately. I will look into this soon.

What I can already say is, that querying data with this model configuration doesn't work with pure sequelize, too.

OK, it is possible. But therefor it is necessary to set through options to the belongsToMany options. See #122

I see. I appreciate you looking into this in such a short time @RobinBuschmann :)

ThroughOptions are now available since 0.6.0-beta.3 (latest release: 0.6.0-beta.5).


install.ps script does not install in my machine

Description
The installation script does not run properly in Windows 10
Steps to Reproduce

Expected behavior:

Actual behavior:

Environment

Additional Information

Closing due to duplicate of #429


Create FileArchive schema and ContentTypes for the EBRAINS image service

We introduce the FileArchive schema to point to specifically structured file location containing typically a large number of files, that should not be indexed by a graph database.
Use case within EBRAINS: image service buckets / containers for Neuroglancer or DCI Capture (@xgui3783 correct?)
The respective content types are attached to the PR (@apdavison & @jagru20 please help with the review)
@marcnm @alexisdurieux @dickscheid FYI
[the background of this PR is also discussed in #318]

@olinux we need to have some clarification for the image-service bucket with the DZI files. the rest is ready to be reviewed.

@jagru20 I'm not yet happy about the content types. I will most likely change them if I get the feedback I need.

@olinux @jagru20 @apdavison I've added the content types of Microsoft Deep Zoom software also to this PR since they were missing in our list.

@jagru20 @olinux @apdavison:
The PR is ready to be reviewed again. With the help of @xgui3783 I've updated the content types for the image service and, for completion and references, added the individual content types of the related software (deep zoom and neuroglancer).
@marcnm @alexisdurieux In the long run would be good if you could online document the specifications for each of the image-service bucket/container types so that we can add a specification link to the respective content types.


Add bash completion instructions

Thank you for this scripts. This PR adds instructions on how to setup git-completion.bash to work with commands branch and merge.
Closes #35

Updated 👍

Oh my god, it's not even my birthday
Bless you Pavel 🙏
Is it OK if I add you to the Contributors list in the README and grant you commit access? Per repo policy

Haha, that's awesome, I am glad it helped out 😄Thank you @jamiew!


add implementation of atkin sieve algorithm

overview

add implementation of atkin sieve algorithm
extract the eratosthene processing of the existing genPrimeNums function as the sgenEratosthenesSieve function
(other files that need to be updated will be updated when the primes initialization function is added.)

reference information

none

spec and other changes will be handled in a separate issue (#52)


Pitch instruments in drum channels and vice-versa

As seen in Dogebox and Shitbox.

oops, forgot to close this


fix for release helm chart and documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 58.895%

Files with Coverage Reduction
New Missed Lines
%

internal/controller/nicdevice_controller.go
4
76.67%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.1%

Covered Lines:
1791

Relevant Lines:
3041

💛  - Coveralls


Support Manual Control Mode

See https://github.com/ParadigmHyperloop/hyperloop/pull/47 and https://github.com/ParadigmHyperloop/manual_client/pull/1.  We should be deprecating manual_client soon and implementing the functionality over the Web -> ODS -> Pod interface

I envision a panel expands whenever the pod's mode changes to Manual with all the controls. Changing any of the controls will issue a new manual command as documented in  https://github.com/ParadigmHyperloop/hyperloop/pull/47 to ODS.
There should also be a new modal created called Change State with a dropdown/list of all the states to change to and a button to execute the state change.  This will allow the pod to be taken in and out of manual mode


A script to generate a filelist for any bazel target

Script to generate a list of source files (.sv and .svh)
Usage:
   ./generate_filelist.sh <target> [output_path]

Arguments:

<target>       : Bazel target for which to generate the file list (required).
[output_path]  : (Optional) Directory where the output file will be saved.
If not provided, the file will be saved in the current directory.

Output:
Generates a filelist with the name filelist_paths_<sanitized_target>.f
containing a list of paths to the source files (.sv and .svh) related to the given target.
The file starts with +incdir+./macros and includes ./ for each file.
Examples:

Default behavior (current directory):
./generate_filelist.sh //arb/rtl:br_arb_fixed_elab_test

Output file: ./filelist_paths_arb_rtl_br_arb_fixed_elab_test.f

Custom output path:
./generate_filelist.sh //arb/rtl:br_arb_fixed_elab_test /path/to/output

Output file: /path/to/output/filelist_paths_arb_rtl_br_arb_fixed_elab_test.f

agh, can't merge since you @hamid-chipstack didn't sign commits. i guess i can turn it off..


Added "Resolution" section

Added "Resolution" section with the first, basic analysis step.

@EliotSeattle - This PR is ready to go to live/master.


add Julia language

add Julia programmes

@harshraj8843 Does this need to be added to the tracked issues?


Left rotate array by D places

Description 📜
It rotates the position of elements in the array by D places.

Domain of Contribution 📊

[x] Competitive Programming

Documentation

/assign

/assign

/assign

I will provide documentation for this issue.
On Thu, 30 Sep, 2021, 10:18 PM github-actions[bot], <
@.***> wrote:

This issue has been assigned
https://github.com/girlscript/winter-of-contributing/issues/1144#issuecomment-931491817
to @tamalbag117 https://github.com/tamalbag117!
It will become unassigned if it isn't closed within 12 days. A maintainer
can also add the pinned label to prevent it from being unassigned.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/1144#issuecomment-931492293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU25MUQWTQJLOQCQJZ6IX5LUESIFZANCNFSM5ELHCEAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[Bug] Duplicate keys in VanillaRegistries bootstrap

What happened?
Attempting to use VanillaRegistries.createLookup() results in an exception when WorldWeaver is installed. While this method doesn't seem to be called by vanilla unless SharedConstants.IS_RUNNING_IN_IDE is enabled, other mods are dependent on it, such as the Fabric API.
I tested this in WorldWeavers's own dev environment using revision 04e0d37476a7dbad15e6af0533cf09153969fb2d. The Minecraft version is 1.21 (pointing this out as the issue template only allows me to select 1.20.1).
Steps to reproduce:

Run runTestmodClient in the wover-biome-api subproject
Hook into the loading process by placing a breakpoint in net.minecraft.client.main.Main#main, anything early enough works
Method 1: Call VanillaRegistries.createLookup() directly using the IDE evaluator.
Method 2: Set SharedConstants.IS_RUNNING_IN_IDE to true. Wait for vanilla to try calling VanillaRegistries.createLookup() from Commands.validate()
Experience a crash

Presumably, any mod that uses the fabric biome api would trigger a crash this way, although I couldn't think of any to test this out with.
BCLib
N/A
Fabric API
0.100.3+1.21
Fabric Loader
21.0.8
Minecraft
1.20.1
Relevant log output
https://mclo.gs/g58EdI9

Other Mods
No response

Thanks for reporting.

Should be fixed with the latest version.


quarto is not started at root directory, thus no renv

Related to quarto-dev/quarto-cli#855
It appears, the extension for VSCode executes quarto from the directory of the .qmd file thus renv is not accessed since at project's root.
The same behaviour might also be in RStudio, but I dit not check.
Setup project
R -q -e 'renv::init()'

See output
#> * Initializing project ...
#> * Discovering package dependencies ... Done!
#> * Copying packages into the cache ... Done!
#> The following package(s) will be updated in the lockfile:
#>
#> # CRAN ===============================
#> - renv   [* -> 0.15.4]
#>
#> * Lockfile written to '~/Projects/misc/renv.lock'.
#> * Project '~/Projects/misc' loaded. [renv 0.15.4]

R -q -e 'renv::install("ggdist")'

See output
#> Installing HDInterval [0.2.2] ...
#>         OK [linked cache]
#> Installing numDeriv [2016.8-1.1] ...
#>         OK [linked cache]
#> Installing distributional [0.3.0] ...
#>         OK [linked cache]
#> Installing ggdist [3.1.1] ...
#>         OK [linked cache]

Rscript -q -e 'writeLines("---\nformat:\n  html:\n    self-contained: true\n---\n\n```{r}\nlibrary(ggdist)\n```\n", "test.qmd")'ined: true)
mkdir subdir
Rscript -e 'writeLines("---\nformat:\n  html:\n    self-contained: true\n---\n\n```{r}\nlibrary(ggdist)\n```\n", "subdir/test.qmd")'

Render with quarto cli from project's root
[Success] Render qmd in project's root
quarto render test.qmd

See output
#> processing file: test.rmarkdown
#>   |.......................                                               |  33%
#>   ordinary text without R code
#>
#>   |...............................................                       |  67%
#> label: unnamed-chunk-1
#>   |......................................................................| 100%
#>   ordinary text without R code
#>
#>
#> output file: test.knit.md
#>
#> pandoc
#>   to: html
#>   output-file: test.html
#>   standalone: true
#>   self-contained: true
#>   section-divs: true
#>   html-math-method: mathjax
#>   wrap: none
#>   default-image-extension: png
#>   filters:
#>     - crossref
#>
#> metadata
#>   document-css: false
#>   link-citations: true
#>   date-format: long
#>   lang: en
#>
#> Output created: test.html

[Success] Render qmd in a subdirectory of project's root
quarto render subdir/test.qmd

See output
#> processing file: test.rmarkdown
#>   |.......................                                               |  33%
#>   ordinary text without R code
#>
#>   |...............................................                       |  67%
#> label: unnamed-chunk-1
#>   |......................................................................| 100%
#>   ordinary text without R code
#>
#>
#> output file: test.knit.md
#>
#> pandoc
#>   to: html
#>   output-file: test.html
#>   standalone: true
#>   self-contained: true
#>   section-divs: true
#>   html-math-method: mathjax
#>   wrap: none
#>   default-image-extension: png
#>   filters:
#>     - crossref
#>
#> metadata
#>   document-css: false
#>   link-citations: true
#>   date-format: long
#>   lang: en
#>
#> Output created: test.html
#>

Render using quarto-vscode
code .

[Success] Render "test.qmd" via quarto-vscode extension "button".
See output
#> /Applications/quarto/bin/quarto preview test.qmd --no-browser --no-watch-inputs
#>
#>
#> processing file: test.rmarkdown
#>   |.......................                                               |  33%
#>   ordinary text without R code
#>
#>   |...............................................                       |  67%
#> label: unnamed-chunk-1
#>   |......................................................................| 100%
#>   ordinary text without R code
#>
#>
#> output file: test.knit.md
#>
#> pandoc
#>   to: html
#>   output-file: test.html
#>   standalone: true
#>   self-contained: true
#>   section-divs: true
#>   html-math-method: mathjax
#>   wrap: none
#>   default-image-extension: png
#>   filters:
#>     - crossref
#>
#> metadata
#>   document-css: false
#>   link-citations: true
#>   date-format: long
#>   lang: en
#>
#> Output created: test.html
#>
#> Watching files for changes
#> Browse at http://localhost:5728/

[Failure] Render "subdir/test.qmd" via quarto-vscode extension "button".
See output
#> /Applications/quarto/bin/quarto preview test.qmd --no-browser --no-watch-inputs
#>
#>
#> processing file: test.rmarkdown
#>   |.......................                                               |  33%
#>   ordinary text without R code
#>
#>   |...............................................                       |  67%
#> label: unnamed-chunk-1
#> Quitting from lines 8-9 (test.rmarkdown)
#> Error in library(ggdist) : there is no package called 'ggdist'
#> Calls: .main ... withVisible -> eval_with_user_handlers -> eval -> eval -> library
#>
#> Execution halted

quarto
quarto --version
#> 0.9.399

We just made a change in our vscode extension (released yesterday) to start from the project dir: https://github.com/quarto-dev/quarto-vscode/commit/00c5382fd4591ed9966bb478ecdcf0679ab7e351
If you update the extension does the behavior change?

I am using v1.20.1, is there another version?

That is the right version! Could you send me a link to a git repo where this repros and I'll run it down.

Here it is: https://github.com/mcanouil/quarto-renv

It looks like the problem is that there is no _quarto.yml file at the root so neither quarto nor vscode knows to use that root as the source of a possible venv (we don't just chase up the directory structure looking for an renv, we need some other "tell" to indicate we should use it).
So adding a project file at the root definitely solves the problem for vscode. With this commit (https://github.com/quarto-dev/quarto-cli/commit/f82b3d93d3340cae1d952574d481e31fb0708736, avail in release v0.9.403) this is as simple as just touch _quarto.yml in the root directory (previously you needed to actually have some yaml in there, but there is no reason that the mere existence of the file shouldn't imply a project).
All of that said, I think our vscode extension should also be running the .qmd files from the workspace root in the case that there isn't a project (which should fix the problem even w/o the _quarto.yml). I'll work on that next.

Here is the related fix in quarto-vscode: https://github.com/quarto-dev/quarto-vscode/commit/76ba6f454026f5a2debd98013284f47fdda10f5b
This will ship w/ the next extension release (probably a few days)

Thank, I'll check both cli and vscode extension when released.


Assigning colonists to specific player

Discord user @Jayden#4755 requested the following:
Assign specific colonists to 1 player, thus preventing other players from managing the priorities of said colonists. This would increase the amount of cooperation that needs to happen between players.

As it seems the user who requested this has left the Discord server, do we still want to go ahead with this request? I still believe it is not needed, since RimWorld is meant to be a cooperative game as a whole, and if players want to play like this, they can just agree to not touch others pawns.

As it seems the user who requested this has left the Discord server, do we still want to go ahead with this request? I still believe it is not needed, since RimWorld is meant to be a cooperative game as a whole, and if players want to play like this, they can just agree to not touch others pawns.

Yes as this would be useful in pvp and when playing with randoms, this should be linked into an update with multiple faction control please.

I see a couple of scenario's you might have to into consideration

what happens when a pop is assigned to a player that left the game?
-- is the pop paused? but what if it is in a city where a other person is playing of same faction
--- is same faction, the other faction members could take over control
---- control will be given back when the player returns
--- if in other city pause the city
perhaps a admin should always have to possibility to assign every pop to a certain player
what if a player never returns? are the pops always paused, even if other players in the same faction are still playing?
-- maybe some kind of faction owner role, has the right to assign every pop within faction to him or other player in faction.
--- can we give pop to players outside of faction?
---- can we give pops to rimworld factions?

I would like to see your thoughts about this. If sharing my thoughts this way is not appreciated, sorry in advance.

Closing this because of inactivity and it does not seem to be viable to implement.


[Feature] Changing Breakpoints After Initialization

What are you trying to add to Cupertino-Pane?
The ability to change breakpoints after the cupertino-pane has been created.
Describe your feature request detailed
Sometimes there are different contexts, where after the cupertino-pane has been created, that a person would want to change the breakpoints without having to close and reinitialize a new cupertino-pane. A possible example is changing the top breakpoint based on an option that the user sets by pushing a button. Creating a method that could update the breakpoints, later on, would be helpful to streamline many apps.
Is there an alternative at the latest version?
[ ] Yes (describe the alternative)
[x] No
Is this related to an issue?
[ ] Yes (Give a link to the issue)
[x] No

@akharag Will be available on next release 1.1.9


[HUDI-7296] Reduce CI Time by Minimizing Duplicate Code Coverage in Tests

Change Logs
testBootstrapRead and TestHoodieDeltaStreamerSchemaEvolutionQuick have many combinations of params. While it is good to test everything, there are lots of code paths that have extensive duplicate testing. Reduce the number of tests while still maintaining code coverage
Impact
faster CI
Risk level (write none, low medium or high below)
low
Documentation Update
N/A
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@jonvex, when is "fullTest" set to "true"?

CI passes. Can merge once @jonvex confirms the cases above.


Add e2e integration test for backup block deletion

We need an e2e integration test, that verifies the correct deletion of the backup blocks.
Have a look at the manual test scenario for an idea of what that should look like.
longhorn/longhorn#1431
Test skeleton is here

Assign to @joshimoo for now, until we have the test skeleton approved.

@yasker test skeleton merged https://github.com/longhorn/longhorn-tests/pull/348

Already implemented by Mohamed!
The original issue is longhorn/longhorn#1435


fix state getter using this syntax

It will throw TypeError: Cannot read property 'firstName' of undefined if not removing user property.

Oh.. right.. after I changed it from using external state reference(no longer works). Thank you.


refs #192 job execution context impl returns correct recovering job key

TriggerKey is always (name,group) pair, name first.

Hi @liyuankui Thank you for the PR! I have created a new PR#349 with your contribution against 2.3.x branch. Will close this one as dup.


Negative terminals for the confidence interval with a positif lift

Bug description
Analysing the test resultats, we obtain a positive lift but negative terminals for the confidence interval :
GeoLift Results Summary
##################################
##### Test Statistics #####
##################################
* Average ATT: 0.243
* Percent Lift: 3.7%
* Incremental Y: 506
* P-value: 0.1
* 90% Confidence Interval: (-1280.739, -46.217)
##################################
##### Balance Statistics #####
##################################
* L2 Imbalance: 0.668
* Scaled L2 Imbalance: 0.0995
* Percent improvement from naive model: 90.05%
* Average Estimated Bias: NA
##################################
##### Model Weights #####
##################################
* Prognostic Function: NONE
* Model Weights:
* 28320: 0.0574
* 95250: 0.0457
* 95130: 0.0377
* 63118: 0.0329
* 95400: 0.0324
* 20600: 0.031
* 30127: 0.0309
* 72230: 0.0296
* 95200: 0.0273
* 6270: 0.0262
* 44100: 0.0259
* 69290: 0.0235
* 63200: 0.0231
* 6100: 0.0225
* 89100: 0.0224
* 30110: 0.0221
* 44300: 0.0212
* 54440: 0.0199
* 1000: 0.0196
* 84700: 0.0196
* 44600: 0.018
* 6220: 0.0171
* 24000: 0.0167
* 90000: 0.0166
* 14370: 0.0162
* 84300: 0.0142
* 28200: 0.0141
* 16710: 0.0139
* 19100: 0.0134
* 69008: 0.0125
* 14000: 0.0122
* 28120: 0.012
* 6800: 0.0115
* 95310: 0.0115
* 69004: 0.0114
* 11000: 0.0112
* 6400: 0.0108
* 21800: 0.0101
* 69360: 0.0099
* 6600: 0.0098
* 95800: 0.0097
* 64230: 0.0095
* 42160: 0.0093
* 69800: 0.008
* 42230: 0.008
* 45700: 0.0079
* 26100: 0.0075
* 6150: 0.0071
* 69200: 0.0065
* 41200: 0.0065
* 28700: 0.0061
* 49000: 0.0059
* 44000: 0.0058
* 73000: 0.0057
* 25000: 0.0055
* 69003: 0.0048
* 87000: 0.0048
* 30000: 0.0047
* 21000: 0.0044
* 63100: 0.0039
* 84100: 0.0037
* 95170: 0.0037
* 42000: 0.0036
* 8000: 0.0033
* 64000: 0.0025
* 95100: 0.0021
* 40000: 0.0021
* 64310: 0.002
* 7200: 0.002
* 22000: 0.0019
* 69100: 0.0017
* 69007: 0.0017
* 86000: 8e-04
* 35000: 7e-04
* 66350: 7e-04
* 6000: 6e-04
* 6300: 5e-04
* 56100: 4e-04
* 66000: 1e-04
* 88200: 1e-04
* 29200: 1e-04

Session information
R version 4.2.1 (2022-06-23)
Platform: aarch64-apple-darwin20 (64-bit)
Running under: macOS Monterey 12.2.1

Matrix products: default
LAPACK: /Library/Frameworks/R.framework/Versions/4.2-arm64/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods
[7] base

other attached packages:
[1] GeoLift_2.5.0 msaR_0.6.0

loaded via a namespace (and not attached):
 [1] augsynth_0.2.0         ggrepel_0.9.1
 [3] Rcpp_1.0.9             mvtnorm_1.1-3
 [5] ape_5.6-2              lattice_0.20-45
 [7] listenv_0.8.0          tidyr_1.2.1
 [9] zoo_1.8-10             assertthat_0.2.1
[11] digest_0.6.29          foreach_1.5.2
[13] utf8_1.2.2             lfe_2.8-8
[15] parallelly_1.32.1      R6_2.5.1
[17] plyr_1.8.7             evaluate_0.18
[19] Boom_0.9.10            ggplot2_3.3.6
[21] highr_0.9              pillar_1.8.1
[23] panelView_1.1.11       rlang_1.0.6
[25] rstudioapi_0.14        Matrix_1.4-1
[27] osqp_<IP_ADDRESS>           stringr_1.5.0
[29] BoomSpikeSlab_1.2.5    directlabels_2021.1.13
[31] htmlwidgets_1.5.4      munsell_0.5.0
[33] proxy_0.4-27           compiler_4.2.1
[35] xfun_0.35              pkgconfig_2.0.3
[37] globals_0.16.1         htmltools_0.5.3
[39] CausalImpact_1.2.7     tidyselect_1.1.2
[41] gridExtra_2.3          tibble_3.1.8
[43] quadprog_1.5-8         dtw_1.22-3
[45] codetools_0.2-18       fansi_1.0.3
[47] future_1.28.0          dplyr_1.0.10
[49] MASS_7.3-57            grid_4.2.1
[51] gsynth_1.2.1           xtable_1.8-4
[53] nlme_3.1-157           jsonlite_1.8.4
[55] gtable_0.3.1           lifecycle_1.0.3
[57] DBI_1.1.3              magrittr_2.0.3
[59] scales_1.2.1           cli_3.4.1
[61] stringi_1.7.8          reshape2_1.4.4
[63] doRNG_1.8.2            doParallel_1.0.17
[65] ellipsis_0.3.2         xts_0.12.1
[67] generics_0.1.3         vctrs_0.5.1
[69] sandwich_3.0-2         Formula_1.2-4
[71] iterators_1.0.14       tools_4.2.1
[73] glue_1.6.2             purrr_0.3.4
[75] rngtools_1.5.2         MarketMatching_1.2.0
[77] abind_1.4-5            parallel_4.2.1
[79] fastmap_1.1.0          yaml_2.3.6
[81] colorspace_2.0-3       LowRankQP_1.0.5
[83] knitr_1.41             bsts_0.9.8

Additional context
Test Parameters  :
GeoTest <- GeoLift(Y_id = "Y",
data = GeoTestData_Test,
locations = zipcode$Zipcode,
treatment_start_time = 93,
treatment_end_time = 97,
ConfidenceIntervals = TRUE)

It is a TWO-SIDED LIFT TEST
method : default setting
fixed_effects : default setting

Pre-treatment period : from the 01/01/2021 to the 03/10/2022
Units : 4

Hi @NoemieCh  The reason why there is often a discrepancy between the p-value and the Confidence Interval is that these two come from different methodologies. Specifically, the p-value for the aggregated Lift are calculated using conformal inference and the CIs are calculated through jackknifing methods. That is the reason for the discrepancy.

Hi, I have added here a link that can help understand more.  If you have no more questions i'm closing this issue :)


importing module with backticks breaks IntelliSense

VSCode Version: latest (1.10.0-insider)
OS Version: Windows 10 + Docker

Steps to Reproduce:

import a module in node.js with backtick qoutation. Example: const fs = require(fs);

check IntelliSense autocomplete. Example: fs dot

With double quote, IntelliSense works:

However, with back-tick IntelliSense doesn't work.

This issue was moved to Microsoft/TypeScript#14345


ec: Add PPC64 vector assembly version of p521 field operations

Only field multiplication and squaring (but not reduction) show a
significant improvement.  This is enabled on Power ISA >= 3.0.
On a Power 9 CPU an average 10% performance improvement is seen (ECHDE:
14%, ECDSA sign: 6%, ECDSA verify 10%), compared to existing code.
On an upcoming Power 10 CPU we see an average performance improvement
of 26% (ECHDE: 38%, ECDSA sign: 16%, ECDSA verify 25%), compared to
existing code.
There are some setup commits.  This includes one that updates crypto/perlasm/ppc-xlate.pl to do some register number hacking, allowing more readable Power perlasm to be written.
Checklist

Note that I don't understand the Actions failure.  Our commits don't touch the files mentioned in the check_update git diff output.

Request for @amitay to ACK this because some of the commits carry his authorship and Signed-off-by: line.

This has been tested on IBM Power 9 and (pre-release) Power 10 systems.
The assembly was original hand-coded by Amitay. It is a fairly direct translation of crypto/ec/ecp_nistp521.c:felem_mul() and crypto/ec/ecp_nistp521.c:felem_square().  Initial testing was done in a standalone test-harness/micro-benchmark, which is not included here.
The Perl wrapping is quite straightforward, it just abstracts out some common functions and (re)produce Amitay's code.
Testing has been done using openssl speed ecdhp521 ecdsap521 to confirm the performance improvements and to (hopefully ;-) confirm correctness. I've confirmed that the current code falls back as expected to run the existing code using OPENSSL_ppccap=0 (on a Power 9 box).
I've also run openssl speed across all algorithms (on a Power 9 box) to sanity check that there are no unexpected failures (with and without OPENSSL_ppccap=0).  I hope that this also shows that the crypto/perlasm/ppc-xlate.pl changes are backward compatible.

Request for @amitay to ACK this because some of the commits carry his authorship and Signed-off-by: line.

This PR is fine by me.

Is @amitay on the CLA list? If not, and if the comment thread above is acceptable to the project, does this mean that anyone on the CLA list can submit non-CLA work by other authors?

Is @amitay on the CLA list?

Yes.

does this mean that anyone on the CLA list can submit non-CLA work by other authors?

No.

Thanks @mattcaswell .

This pull request is ready to merge

Merged, thanks for the contribution.


[BUG] Eventually which uses no Should does not get auto-fixed

Describe the bug
There is an Eventually which uses no Should:
                        Eventually(func(g Gomega) {
                                vm, err = virtClient.VirtualMachine(vm.Namespace).Get(vm.Name, &k8smetav1.GetOptions{})
                                g.Expect(err).ToNot(HaveOccurred())
                                g.Expect(controller.HasFinalizer(vm, v1.VirtualMachineControllerFinalizer)).To(BeFalse())
                        }, 2*time.Minute, 1*time.Second)

It gets reported like this:
$ ginkgolinter ./...
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2290:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2302:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2311:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"

Example run:
rmohr@rmohr-kubevirt ~/gerrit/kubevirt/tests (dev)$ ginkgolinter -fix ./...
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2290:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2302:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2311:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
rmohr@rmohr-kubevirt ~/gerrit/kubevirt/tests (dev)$ ginkgolinter ./...
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2290:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2302:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"
/home/rmohr/gerrit/kubevirt/tests/vm_test.go:2311:4: ginkgo-linter: "Eventually": missing assertion method. Expected "Should()" or "ShouldNot()"

However when running  the linter with -fix, it does not get fixed.
To Reproduce
Steps to reproduce the behavior:

Create an Eventually which uses inner assertions only, without any Should
apply the linter with -fix
It gets not fixed

Expected behavior
It would be great if that could be automatically fixed as well. Or is this even something to report? Maybe it should check the function signature as well?

Thanks @rmohr
This is on purpose and it also documented here: https://github.com/nunnatsa/ginkgolinter#missing-assertion-method
The reason for that is that the linter can't guess the right test logic. In this case I want the user to decide how assert the test case. I think that default handling like Should(Succeed()) may not always be the right choice.

Makes sense. :+1:


Declaring an existing module / namespace

I want to write a type declaration that allows me to write Typescript that compiles to the following output

class MyTest{

    public  Test() {
            var instance = new mymodule.MyClass();

    }
}

There should be no import generated, because the Javascript library gets already imported somewhere else.
I tried already many possible solutions to solve it, one of them is the following typescript declaration

export module mymodule{
 export class MyClass{
 }
}

I try to use import type, because I don't want to get the import compiled to the Javascript file.
import type { mymodule } from "./Declaration.ts"

class MyTest{

    public  Test() {
            var instance = new mymodule.MyClass();

    }
}

The compiler fails with the following error message:
 mymodule' cannot be used as a value because it was imported using 'import type'
When I use import (without type), then I get the import compiled into JavaScript, which fails at runtime.
Is there a possibility to write the declaration that it simply works without emitting any import?

It looks like this is a question rather than a bug report. This issue tracker is for tracking bugs and active work on TypeScript itself, rather than a general forum for programmers using TypeScript to get help or ask questions.
You can ask questions on sites like Stack Overflow. We are not able to provide one-on-one support on the issue tracker. Please read the issue template carefully - it has important information on what kinds of reports can be acted on here, as well as links to useful TypeScript resources. Thanks!