blank screen

after new steam client beta update its just blank screen:

it was happened after Steam Client Beta - April 27th update
did there any way to uninstall this theme until it fixed?, i cant find docs to do that.

Happens to me too, usually I just need to restart and that fixes it. If you want to uninstall, just delete the ~/.steam/steam/skins/[cpt theme] folder, which should work.

just delete the ~/.steam/steam/skins/[cpt theme] folder, which should work.

its not working, and after some investigation i found that its not catppuccin issue, i created report on ValveSoftware/steam-for-linux
the only thing i found strange is: [0428/193415.367178:WARNING:gpu_process_host.cc(1256)] The GPU process has crashed 1 time(s)

im fully reinstalled steam(including deleting local/share/steam) and now everything works.

Still having this issue, this is the output of the log files:
Recreated steam folder from scratch.
➜  logs tail -f webhelper.txt  [2023-05-09 21:37:03] Error: AcquirePixmap: failed to create glx pixmap for window: 0x320006c [2023-05-09 21:37:11] SP Shared JS Context-'SharedJSCo': CreatingPopup name:notificationtoasts_4_desktop browser:65536 pid:71862 type:4 flags:1340266: (283.00, 70.00) -2147483648.00x-2147483648.00: url:about:blank?createflags=1340170&browserType=4 [2023-05-09 21:37:11] CreateOutputWindow: Creating browser window at: 805240832,805240832 size: 283x70 [2023-05-09 21:37:11] Loaded GL 4.6 [2023-05-09 21:37:11] notificationtoasts_4_desktop: Created window: size: 283,70 pos: 805240832,805240832 mode: OpenGL window: 0x2e000e8 [2023-05-09 21:37:11] notificationtoasts_4_desktop: AfterCreated handle:1376274 type:4: (0, 0) 283x70 [2023-05-09 21:37:11] Error: AcquirePixmap: failed to create glx pixmap for window: 0x3200071 [2023-05-09 21:37:32] Browser - launching child process with: /home/icyjiub/.local/share/Steam/ubuntu12_64/steamwebhelper --type=renderer --disable-seccomp-filter-sandbox --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --field-trial-handle=12398331292911151086,8834172278350125781,131072 --enable-features=CastMediaRouteProvider --disable-features=SameSiteByDefaultCookies --enable-blink-features=ResizeObserver,Worklet,AudioWorklet --disable-blink-features=Badging --lang=en-US --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --product-version=Valve Steam Client --buildid=1683666132 --steamid=0 [2023-05-09 21:38:52] Browser - launching child process with: /home/icyjiub/.local/share/Steam/ubuntu12_64/steamwebhelper --type=renderer --disable-seccomp-filter-sandbox --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --field-trial-handle=12398331292911151086,8834172278350125781,131072 --enable-features=CastMediaRouteProvider --disable-features=SameSiteByDefaultCookies --enable-blink-features=ResizeObserver,Worklet,AudioWorklet --disable-blink-features=Badging --lang=en-US --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --product-version=Valve Steam Client --buildid=1683666132 --steamid=0 [2023-05-09 21:38:59] Browser - launching child process with: /home/icyjiub/.local/share/Steam/ubuntu12_64/steamwebhelper --type=renderer --disable-seccomp-filter-sandbox --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --field-trial-handle=12398331292911151086,8834172278350125781,131072 --enable-features=CastMediaRouteProvider --disable-features=SameSiteByDefaultCookies --enable-blink-features=ResizeObserver,Worklet,AudioWorklet --disable-blink-features=Badging --lang=en-US --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --product-version=Valve Steam Client --buildid=1683666132 --steamid=0 [2023-05-09 21:39:29] Browser - launching child process with: /home/icyjiub/.local/share/Steam/ubuntu12_64/steamwebhelper --type=renderer --disable-seccomp-filter-sandbox --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --field-trial-handle=12398331292911151086,8834172278350125781,131072 --enable-features=CastMediaRouteProvider --disable-features=SameSiteByDefaultCookies --enable-blink-features=ResizeObserver,Worklet,AudioWorklet --disable-blink-features=Badging --lang=en-US --log-file=/home/icyjiub/.local/share/Steam/logs/cef_log.txt --product-version=Valve Steam Client --buildid=1683666132 --steamid=0 ^C ➜  logs  ➜  logs tail -f cef_log.txt   [0509/213659.114239:ERROR:gl_surface_presentation_helper.cc(259)] GetVSyncParametersIfAvailable() failed for 1 times! [0509/213659.128999:ERROR:gl_surface_presentation_helper.cc(259)] GetVSyncParametersIfAvailable() failed for 2 times! [0509/213659.141844:ERROR:gl_surface_presentation_helper.cc(259)] GetVSyncParametersIfAvailable() failed for 3 times! [0509/213709.194354:WARNING:xproto_util.cc(51)] X error received: serial 7074, error_code 3 (BadWindow (invalid Window parameter)), request_code 4, minor_code 0 (X_DestroyWindow) [0509/213710.184840:WARNING:xproto_util.cc(51)] X error received: serial 7226, error_code 3 (BadWindow (invalid Window parameter)), request_code 4, minor_code 0 (X_DestroyWindow) [0509/213719.450079:WARNING:xproto_util.cc(51)] X error received: serial 8086, error_code 3 (BadWindow (invalid Window parameter)), request_code 4, minor_code 0 (X_DestroyWindow) [0509/213732.746373:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213852.974370:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213859.770820:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213929.814344:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213955.174208:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213957.107920:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/214005.075650:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/214006.400491:WARNING:spdy_session.cc(3352)] Received HEADERS for invalid stream 29 [0509/214006.911635:WARNING:spdy_session.cc(3352)] Received HEADERS for invalid stream 31 [0509/214016.240538:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer ^C ➜  logs tail -f steamwebhelper.log  [0509/213732.746373:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213852.974370:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213859.770820:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213929.814344:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213955.174208:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/213957.107920:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/214005.075650:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer [0509/214006.400491:WARNING:spdy_session.cc(3352)] Received HEADERS for invalid stream 29 [0509/214006.911635:WARNING:spdy_session.cc(3352)] Received HEADERS for invalid stream 31 [0509/214016.240538:INFO:crash_reporting.cc(271)] Crash reporting enabled for process: renderer


Proto compile error for TextToSpeech API

When I try to use the the google-cloud-texttospeech-v1 feature, I get this compile error:
error[E0412]: cannot find type `Operation` in module `super::super::super::super::longrunning`
   --> /home/.../.cargo/registry/src/index.crates.io-6f17d22bba15001f/gcloud-sdk-0.20.6/src/../genproto/google.cloud.texttospeech.v1.rs:622:70
    |
622 |             tonic::Response<super::super::super::super::longrunning::Operation>,
    |                                                                      ^^^^^^^^^ not found in `super::super::super::super::longrunning`

error[E0698]: type inside `async fn` body must be known in this context
   --> /home/.../.cargo/registry/src/index.crates.io-6f17d22bba15001f/gcloud-sdk-0.20.6/src/../genproto/google.cloud.texttospeech.v1.rs:634:25
    |
634 |             let codec = tonic::codec::ProstCodec::default();
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^ cannot infer type for type parameter `U`
    |
note: the type is part of the `async fn` body because of this `await`
   --> /home/ayomidebamidele/.cargo/registry/src/index.crates.io-6f17d22bba15001f/gcloud-sdk-0.20.6/src/../genproto/google.cloud.texttospeech.v1.rs:646:48
    |
646 |             self.inner.unary(req, path, codec).await
    |                                                ^^^^^

error[E0698]: type inside `async fn` body must be known in this context
   --> /home/../.cargo/registry/src/index.crates.io-6f17d22bba15001f/gcloud-sdk-0.20.6/src/../genproto/google.cloud.texttospeech.v1.rs:634:25
    |
634 |             let codec = tonic::codec::ProstCodec::default();
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^ cannot infer type for type parameter `M2` declared on the method `unary`
    |
note: the type is part of the `async fn` body because of this `await`
   --> /home/../.cargo/registry/src/index.crates.io-6f17d22bba15001f/gcloud-sdk-0.20.6/src/../genproto/google.cloud.texttospeech.v1.rs:646:48
    |
646 |             self.inner.unary(req, path, codec).await
    |                                                ^^^^^

Steps to Reproduce
OS
64-bit Pop!_OS 22.04 LTS
cargo.toml
[package]
name = "ctest"
version = "0.1.0"
edition = "2021"
[dependencies]
gcloud-sdk = { version = "0.20", features = [
    "google-cloud-texttospeech-v1",
] }
tonic = { version = "0.9", features = ["tls"] }
prost = "0.11"
prost-types = "0.11"
tokio = { version = "1.20", features = ["full"] }
tracing = "0.1"
tracing-subscriber = { version = "0.3", features = ["env-filter"] }

main.rs
#[tokio::main]
async fn main() -> Result<(), Box<dyn std::error::Error>> {
	Ok(())
}

Hey, it seems sometimes GCP features have deps between each other on proto level.
To fix this try this:
features = ["google-cloud-texttospeech-v1", "google-longrunning"]

I left you an example here:
https://github.com/abdolence/gcloud-sdk-rs/tree/master/examples/text-to-speech-client

Yep, that fixed it! Thank you :)


CI: Start (periodically) testing Rails Edge

The best way to test a brand new stable release of Rails is to have already been testing Rails Edge beforehand. Let's add Rails Edge coverage with our CI system, having it run periodically (not as often as on every PR) with perhaps the same cadence as Ruby preview build runs.

Thank you for adding this ability to our CI.  It will become our foglights to tell if there will be issues with the upcoming rails version.


Incomplete instructions - FrameworkReference needed

[Enter feedback here]
The nuget instructions on this page don't work. I believe the correct path is to remove the ASPNET related NugetPackages and add
<FrameworkReference Include="Microsoft.AspNetCore.App" /> in an  in the project csproj files.
The net7.0 nuget packages dont exist or are not available following the instructions in this page, so developers need to remove certain ones first, then add the FrameworkReference on affected projects.

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

ID: e9669d35-3de1-997d-258c-745ebbdbface
Version Independent ID: e9669d35-3de1-997d-258c-745ebbdbface
Content: Migrate from ASP.NET Core 6.0 to 7.0
Content Source: aspnetcore/migration/60-70.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Microsoft.AspNetCore.App hasn't been used since 2.2

Thank you very much for kindly reviewing this so quickly.
The project I'm upgrading was a .NET Core 3.1 aspnetcore site, I moved from 3.1 to 6 following that documentation, and now doing 6->7 with this page and the combination of instructions between the two projects just don't work or are incomplete/missing something. (Im stuck with deprecated 2.x nuget packages, or missing Frameworks).
Following up on your comment; I guess, the Microsoft.AspNetCore.App package doesn't exist as a nuget package anymore, but that moniker is still used in VS
From a VS template project for .NET 7:

Either way, there do not appear to be .NET 7 ASPNET nuget packages to actually upgrade. [Perhaps they were supposed to be removed in the 3.1->6 conversion? But then, what are they replaced with?]
The ones Nuget Package Manager suggests are v2 and are all marked deprecated. There also doesn't appear in Visual Studio directly to add a FrameworkReference to a project.
I don't know what it is, but this particular documentation is clear as mud, at least for this 3.1->7 use case. I might just blow the project away and bring in my class files into one of the VS default template projects or something.


missing a dismissPeekViewHandler

I need to execute some code whenever PeekView is dismissed.
Could you please add a 'dismissPeekViewHandler' parameter in your configureView function which is executed whenever the PeekView is dismissed, or something similar?
thx

I agree with Dandiccf.
I strongly need a completion handler to perform a task after dismissing the PeekView.
Thank you.

Hi, sorry for the delay in response. I've update the source code and push the changes, hope it's helpful for you guys.

Hi there,  works perfectly - thx


Test Cases - Include initial unit tests

Initial test cases using PyTest

LGT with the testing. Ship it.


Use templates instead of hard coded strings

This has been partly accomplished with the move to Serde (for JSON, that is).


Fix the catapult server for chrome 120

After working for over two year without any changes, the latest Chrome update resulted in the catapult server no longer being seen by Chrome.
This fixes that, also added docs on how to sniff the traffic on the 9222 port.

@efernandez , @jasonimercer, @Omar-Elmofty fyi.

 Just asking to get an idea if I encounter this in the future

The catapult server just won't show up in chrome://inspect/?tracing#devices, even though it is running.

Is this change backwards compatible?

Seems to be, just tested with Chromium 112, the ascii arrow doesn't show up, but the trace button does and it opens a working tracing interface.


[Question]: Is it possible to get result from single-value request

Hello,
I've been struggling for a couple of days now for getting a sensible result from single-value Mikrotik API requests.
For example, this one:
/ip/dhcp-server/lease/print
=active=
=count-only=
?server=LAN-8-floor

<<< /ip/dhcp-server/lease/print
<<< =active=
<<< =count-only=
<<< ?server=LAN-8-floor
<<<
>>> !done
>>> =ret=7
>>>

As we can see in this example API returns an unnamed/anonymous object that doesn't fall into re={…} response:
{ret: 7}

In the debug session we see no response to such requests.
The immediate trace reply is:
TRC response channel closed

My configuration is as follows:
modules:
  - name: testx
    commands:
      - command: |-
          /ip/dhcp-server/lease/print
          ?server=LAN-8-floor
          =active=
          =count-only=
        metrics:
          - metric_name: dhcp_entries
            param_name: ret
            param_type: int
            metric_type: gauge
            labels:
              - param_name: source
                param_type: string
                value: dhcp

So, is it possible somehow to get this value?

Hi,
good idea, I have implemented this feature.
Works exactly like you tried, with param_name: ret.
I haven't created a release yet, so for testing with Docker use the mikrotik-exporter:dev image.
Let me know if everything works, then I'll create a release!

Hi,
Thanks for the quick update. I've tested your new :dev image with my configuration and it is working good.
One note, though. I still had to rebuild my own image from your source applying the patch from issue #54 .
Otherwise container logs are being spammed with lines and lines of time parse errors:
swoga-exporter      | 2024-02-29T12:33:18Z ERR failed to parse datetime error="parsing time \"2024-02-29 14:17:52\": month out of range" command_no=1 connection_no=0 metric_name=last_link_up module=interface request_no=413 sentence_no=13 target=router-traffic word="2024-02-29 14:17:52"
swoga-exporter      | 2024-02-29T12:33:18Z ERR failed to parse datetime error="parsing time \"2024-02-29 02:49:16\": month out of range" command_no=1 connection_no=0 metric_name=last_link_down module=interface request_no=413 sentence_no=19 target=router-traffic word="2024-02-29 02:49:16"

Hi,
thanks for the note!
Somehow I didn't notice this during testing and we don't have v7 in production yet.
Have pushed a fix please try again with the dev image!

New release with both fixes is available! v0.8.0


[2.5] irmin-pack: fix a bad performance regression for read-only instances

Pack files do not have generation numbers in V1, so force_generation was
always returning a corrupted value, which was forcing a cleanup of all
the existing caches on every sync.

What about his line? https://github.com/mirage/irmin/blob/73eaa2b5b9111c3841423e3e0fbd2c18ab3cadf8/src/irmin-pack/IO.ml?ts=4#L176

NVM, I answered myself :) (Can't be called with V1)

@Ngoguey42 unsafe_clear is called by clear which fails if the the store has version V1.

Updated to only do one syscall per sync to check that nothing has to change.


AppRTC - Attach "enter" key event to join button if room name is filled in

Ported from https://github.com/GoogleChrome/webrtc/issues/461
It would be nice if hitting enter after entering the room name on the apprtc landing page would join the room.
Something like this

fixed


Fixed bug causing data race condition in relay-server

Purpose of PR ?
FIXES   kubearmor/KubeArmor#1141
This PR fixes the data race condition present in relayServer.go .
Checklist:

[ ]  Breaking change (fix or feature that would cause existing functionality to not work as expected)

@Aryan-sharma11 in the log stack in issue here

goroutine 2689 [running]:
runtime.throw(0x18dcde8, 0x15)
	/usr/local/go/src/runtime/panic.go:1116 +0x72 fp=0xc00115fe18 sp=0xc00115fde8 pc=0x437c52
runtime.mapdelete_faststr(0x16c2400, 0xc000459560, 0xc0008f7b20, 0xc)
	/usr/local/go/src/runtime/map_faststr.go:306 +0x37f fp=0xc00115fe80 sp=0xc00115fe18 pc=0x41651f
github.com/kubearmor/kubearmor-relay-server/relay-server/server.connectToKubeArmor(0xc0008f7b20, 0xc, 0x18ca35b, 0x5, 0x0, 0x0)
	/usr/src/kubearmor-relay-server/relay-server/server/relayServer.go:630 +0x7ce fp=0xc00115ffb0 sp=0xc00115fe80 pc=0x1544fee
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1374 +0x1 fp=0xc00115ffb8 sp=0xc00115ffb0 pc=0x46cdc1
created by github.com/kubearmor/kubearmor-relay-server/relay-server/server.(*RelayServer).GetFeedsFromNodes
	/usr/src/kubearmor-relay-server/relay-server/server/relayServer.go:678 +0x276

we can see that the panic in the code is caused by runtime.mapdelete_faststr  which means we are trying delete a map which does not exists
also from the stack trace we get that panic occurs in the connectToKubeArmor function
try looking in the code where are we trying to delete a variable and just add a check to see if it exists or not for deleting

Looking good. This should fix the concurrent map writes. Also, for the data race condition sent on slack, can you please follow this while incrementing the wait group counter to ensure that we follow the best practice - https://pkg.go.dev/sync#WaitGroup.Add
Basically we need to move these calls out of the go routines -
https://github.com/kubearmor/kubearmor-relay-server/blob/d77ba972ebb02e9020a339c07c135acab55a76a6/relay-server/server/relayServer.go#L385

Done

@Aryan-sharma11 gosec tests are failing in all PRs (unrelated to the changes here). Can you look into this and update the workflow? Maybe it just needs a go version update.

@Aryan-sharma11 gosec tests are failing in all PRs (unrelated to the changes here). Can you look into this and update the workflow? Maybe it just needs a go version update.

Okay sure, I'll look into it


How to configure artisan command to be used

Since I installed the plugin I'm constantly receiving the error "Executable artisan not found" when saving my file or switching buffers.
The thing is that I cannot run artisan in the current project, as I have a static copy and the actual artisan command must run inside a Docker container.  But I could configure the correct command to be used if I knew how to do that.
Is there a configuration option for that?
Thanks in advance!

Hey, sorry to heard that you have problems.
how the plugin works by default it check if the file artisan is present in the root it assumes is a laravel project.
To define the environment first check the environment variable NVIM_LARAVEL_ENV which you can set up in your .env file.
here you can set local sail or docker-compose given your case you can use docker-compose and you need to define the container name by default is app but you can also set the environment variable APP_SERVICE with the name of the container.
this the command will run as
docker compose exec -it <container name> <command> by default will tried to run php (and artisan) commands, composer and npm commands.
If this is not enought for you, you can add your own overwriting the configuration in the setup function in
environments.definitions and set your own of type LaravelEnvironmentConfig
---@class EnvironmentCondition
---@field file_exists string[]|nil
---@field executable string[]|nil

---@class LaravelEnvironmentConfig
---@field name string
---@field condition EnvironmentCondition
---@field commands table|nil

in the lua/laravel/config/environments.lua file you can see examples how the ones I mentioned before are set.
Hope this explanation helps you to have it set up for your environment.

Hey, were you able to solve it ?  I will close it since two weeks since the last message, feel free to re-open it with more comments.


TESTING ONLY Broken test: DO NOT MERGE!

Test of process. Do not merge this change!

The test suite was run, and failed. Log: https://staging-linkeddata-rewriter-nectar.ardc.edu.au/test-logs/PR114.txt

Process test complete; closing.


Example code:
for(I = 1, 100){
   if(!propCanCreate()){ break }

    propSpawn("models/props_junk/CinderBlock01a.mdl", entity():pos(), ang(), 1)
    print(I)
}

this should stop printing at the burst limit, but it prints numbers well above that.

On further inspection it seems like propCanCreate() doesn't account for anything else wanting to block prop spawns.

Nevermind, it seems like it was a problem on my end


What is the difference between specifying app_type as Daas and DFS?

Comment
What is the difference between specifying app_type as Daas and DFS?

app_type as daas works when all service deploy localhost
DFS works only run engine localhost, and manager works on cloud service: https://cloud.tapdata.net/console/v3/#/workbench

Thank you for your reply! I see that the release v3.2 dfs version uses three databases (dfs_tm, dfs tm obs, and dfs tm log). Where can I obtain the initialization scripts for these databases?


Slow addition of TFs

I'm trying to create a MTCollection for a large data set (> 1000 edi files). Using a loop with mt_object.read(fn) /     md.add_station(mt_object), it takes > 5 hours. It slows down with iteration, as if there is some reinitialization/copy every time the add function is called.
Is this a feature or is there perhaps a more efficient bulk read function?

@agrayver Interesting, I just tried something similar and it was also slow.  There must be a bottleneck somewhere and it is likely to do with setting up the groups for each station.  In mine there were a lot of EDI files that had no survey name, so MTCollection kept creating unknown_survey_## and that seemed to slow it down quite a bit.  I'll work on some efficiencies hopefully in the next week or so.

@agrayver I updated the code so that building a MTCollection is much faster (starting with v2.0.10).  It should take between .5-5 seconds to add a transfer function depending on the metadata inside.  One way to speed it up is to set survey to be the same for all transfer functions if that makes sense to do so.
from mtpy import MTCollection

with MTCollection() as mc:
    mc.open_collection("path/to/collection.h5"
    mc.add_tf(list_of_transfer_functions, new_survey="common_survey_name")

If that works for you feel free to close this issue.

thanks for looking into this


What is the source of fasta file in refgenie?

Hello, I want to know the source of fasta file, from NCBI, UCSC, Ensembl. My PI want to use the ensemble fasta file, but I think your software is more convenient.
Thank you.

you can see the source of the fasta assets here: https://github.com/databio/databio_genomes/blob/master/asset_pep/recipe_inputs.csv#L2-L9
Let us know if you have any other questions or you'd like us to host fasta assets from other sources. Alternatively, we can assist you in creating your own fasta recipe which can be used to build assets with refgenie build command.

Thank you for your help


Cannot install or build Docker image

Looks like this line is causing error    https://github.com/twosigma/beakerx/blob/fabee2856ef04acb512e8d4e1b8c3eb31767f13c/beakerx/beakerx/handlers.py#L34
The error message from Conda is:
AttributeError: module 'tornado.web' has no attribute 'asynchronous'

Looks like tornado had a new release since beakerx was last released. I dig around in the current container from Dockerhub and it has a different version of tornado (5.0.2) from what conda is trying to install with beakerx right now (6.0.1)

I encounter the same issue building my own docker image for beakerx.

Thanks for reporting, we are updating our code to stand with tornado 6.0.  It should come out soon.

This bug seems to be fixed in version 1.4.0 but conda-forge still has version 1.3.0 so I had to use pip to install instead (documentation).
install beakerx
beakerx install


[cache] Update tx architecture copy -> diff

Currently, the entire cache is copied when making a new transaction. This will create performance and memory issues.
Instead of having the transaction snapshot the cache at a moment in time, the transaction should track only its diffs.
This is essentially implementing a (read committed)[https://www.postgresql.org/docs/current/transaction-iso.html] isolation level from relational databases.
Making this change will have a couple of downstream impacts

tx.get will check the transaction entries and fall back to the cache if the transaction hasn't written to the key yet.
tx.delete will actually need to write a tombstone in the transaction entries and the actual delete will occur when the transaction is committed to the cache.

updated delete in https://github.com/data-eden/data-eden/pull/104


Fix: Check ratings a bit more strict

This PR

[x] checks if the ratings given are numeric

Casting a string like 'blabla' to an int will make it 0 in php. Which isn't really what we want here.

Thanks @BackEndTea and @localheinz


Make desk run work with an argument vector.

This change adds a new argument vector form of the desk run command, allowing an existing command to be wrapped without requiring quoting/escaping of the command.
In this new form, the argument vector is executed without variable expansion by the shell. This could be supported using eval, but it would require some tradeoffs that aren't really worth it IMO (ssh does something like this, and it generally makes a mess of anything but the simplest of commands).

This addresses #88.

Looks good to me, thanks!


Add "sort-keys" rule

Requested here: https://github.com/brianchandotcom/liferay-portal/pull/75764#issuecomment-511905751
This is straightforwardly achievable with the sort-keys rule. Object property order could in theory matter, but I think it's quite unlikely that our projects rely on it anywhere, especially because the rule doesn't sort across spreads:
// This rule does not report unsorted properties that are separated by a spread property.
let obj = {b: 1, ...c, a: 2};

We're already using sort-keys in Clay to good effect.

Done via https://github.com/liferay/eslint-config-liferay/pull/63


SWITCHYARD-1997 Dynamic "per message" endpoint

Support overriding endpoint via context property - service name for SCA, URI for REST

Can one of the admins verify this patch?

ok to test


Transaction support

Transaction which is supported on Mongo v4 and above is almost ready. Just need 1-2 days to check completely. Thanks for your help @Smolevich .

Codecov Report

Merging #2019 into master will increase coverage by 0.25%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2019      +/-   ##
============================================
+ Coverage     87.04%   87.30%   +0.25%
- Complexity      667      677      +10
============================================
  Files            31       31
  Lines          1521     1544      +23
============================================
+ Hits           1324     1348      +24
+ Misses          197      196       -1

Impacted Files
Coverage Δ
Complexity Δ

src/Jenssegers/Mongodb/Connection.php
92.94% <100.00%> (+1.63%)
46.00 <8.00> (+8.00)

src/Jenssegers/Mongodb/Query/Builder.php
89.09% <100.00%> (+0.20%)
167.00 <2.00> (+2.00)

...Jenssegers/Mongodb/MongodbQueueServiceProvider.php
100.00% <0.00%> (+20.00%)
2.00% <0.00%> (ø%)

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 a7acac7...e590578. Read the comment docs.

There is still work to do with handling deadlock in transaction, especially configuring exception messages of mongodb.

@ilyasokay do you have plans to continue work with PR?

@ilyasokay do you have plans to continue work with PR?

@Smolevich  Sorry for the delay, I had some urgent works to do; but I will continue this pr as soon as possible.

Closing in favor of https://github.com/jenssegers/laravel-mongodb/pull/1904


[DOC-2892] Update GCP connector details - metadata server IP requirement

Update GCP connector details - metadata server IP requirement
For reviewers, a preview is available.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


add test dependency to publish

Codecov Report

Merging #9 (f9fa5cd) into main (3b349a6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   82.47%   82.47%
=======================================
  Files          14       14
  Lines         291      291
=======================================
  Hits          240      240
  Misses         51       51

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 3b349a6...f9fa5cd. Read the comment docs.


CONTRIBUTING.md doesn't mention the make webpack step

...which means if you only follow what https://github.com/resin-io/etcher/blob/master/docs/CONTRIBUTING.md says, then you get a version of Etcher that doesn't work properly and just prints:
gui.js Failed to load resource: net::ERR_FILE_NOT_FOUND

in the DevTools console.

...and I've just noticed that it doesn't mention the make cli-develop step either?

I didn't even know cli-develop existed o.O

make help is your friend ;)

Should I create a separate issue for make cli-develop being missing from CONTRIBUTING.md ?

Yeah, or just make a PR? ;P


partial for partners

In the interests of keeping pr's small here is one just for the partial to allow the edit/update and new/create stories to run in parallel.

error messages in form to be implemented in a later pr abstracted out into
custom form helper to be used across application
have had to require CGI in tests to escape provinces for tests - is there a
better way
have used minitest assert_select in tests because that is my preference but
I am open to suggestions if this offends people

See screenshot below:-

:+1:

:+1:

Very happy with the small focused PR, and the partial looks good.
What's our final decision on view specs? At one point, we were leaning towards limiting them to branches & conditionals - i.e. testing any logic that might be in a view but nothing else.

I think possibly the only test that merits considering dropping is the first test (the form check), but I think it still has value. Given that its a partial used in multiple places I think checking input fields are present and render the appropriate values is prudent.
There shouldn't be much overhead given the exclusive use of fixture data and build_stubbed calls.
In #252 the only test I'm running is that the partial is called, because all the testing is effectively in the partial.
Given we should be ruthlessly minimising any logic in views I think we do need some (minimal) content checking, especially as cucumber should not be doing this. However, it needs to be abstract enough not to require rewriting for every change but give enough confidence that the right information in the view is being rendered.
If you think I have moved away from the vision of how our view test should be, do you want to put something on the table - a list say of what you envisioned seeing, and then we can move closer to it.

The asserts have to be multiline, @NikitaAvvakumov typo catch made me spot a floor in the tests. However I have refactored out the render into the before block and reduced them to a single test to make things more readable and compact.


Review NOTICE file for incubator-openwhisk-runtime-java

Init. PR: https://github.com/apache/incubator-openwhisk-runtime-java/pull/29#pullrequestreview-109322187
file: https://github.com/apache/incubator-openwhisk-runtime-java/blob/master/NOTICE.txt

Closing in favor of tracking issues with the LICENSE file here:https://github.com/apache/incubator-openwhisk-release/issues/102


Adds breakpoints for responsive animations

Allows you to use breakpoints to define different behaviors depending on the user screen size.
Example of how to use :
<img data-aos="xs:fade-right md:fade-bottom xl:flip-right" data-aos-delay... />
Related Issue
Personal requirements led me to this commit.
How Has This Been Tested?
Could probably get minor corrections such as moving the generic  ('fade', 'flip', 'slide' and 'zoom') into each breakpoint to avoid any potential problem.

Cool Idea! Would love to see it merged!


Update readme to use API token

Description of your changes
Fixes #6 by changing the readme to use API instead of session tokens which do not expire.
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review.

lgtm
Thanks Max!


[python] Restore soma indexer in hvg

Previously removed due a memory bug, now fixed by https://github.com/single-cell-data/TileDB-SOMA/issues/2662

The issue before was memory, right? Would you want to introduce a memory test here?
I've been using https://github.com/bloomberg/pytest-memray for this

The issue before was memory, right? Would you want to introduce a memory test here?
I've been using https://github.com/bloomberg/pytest-memray for this

That would be great. This should be part of the upcoming "expanded regression testing" work for sure.


Masking and binning awkward.highlevel.Array leads to wrong results

Version of Awkward Array
1.8.0
Description and code to reproduce
Hej Jim, thanks for the nice package.
I encountered a strange behavior when making a histogram of a masked Awkward Array. I am not sure if the issue arises from awkward or numpys masked arrays (or maybe even both). When converting the awkward.array into a numpy.masked array the entries for the masked fields change. In addition when making the histogram of the masked array it looks like that the masked values are not removed before making the histogram (while they are removed when using other numpy functions like np.sum).
I made a short video to illustrate the issue.
https://user-images.githubusercontent.com/43881800/166879012-6057a1af-c379-4ac2-9889-7b2e142f401b.mp4
Below you can find the code used in the video:
import numpy as np
import awkward as ak
import pickle
import sys
print(sys.version, ak.__version__, np.__version__)

# Load and prepare data:
with open("test_data_awkward_numpy_bug", "rb") as fp:   # Unpickling
    list_root = pickle.load(fp)

test_data = ak.Array(list_root)
first_photon = ak.min(test_data['pmthitTime'], axis=1)
first_photon = first_photon * 10**9
test_mask = (test_data['nNVpmthits'] >= 300 ) & (test_data['nNVpmthits'] < 700)
test_mask = test_mask & (first_photon < 20)

# Bug observed in awkward arrays using numpy.ma.core.MaskedArray
# No sure if the bug is in awkward or numpy.
t = ak.to_numpy(test_data['nNVpmthits'][test_mask])
print(t[:20])
print(t.data[:20])
a, _ = np.histogram(t, bins=100, range=(0, 1500))
b, _ = np.histogram(t.data[~t.mask], bins=100, range=(0, 1500))
print(a)
print(b)

I also made a pickle file of the used data:
test_data_awkward_numpy_bug.zip

(btw sorry for the poor video quality, but I had to compress it in order to upload it)

Unlike NumPy, Awkward array isn't guaranteed to preserve the original values of masked elements.
As to the histogram results, I'm not sure that np.histogram supports masked arrays at the moment #832. I suspect you will want to first ravel the contents e.g. with np.ravel.
Technical Explanation :microscope:
The reason for this is that there are several representations of missing values in Awkward. Some of them (BitMaskedArray / ByteMaskedArray) mask elements, meaning that len(array) == len(mask), whereas others index elements (IndexedOptionArray, where len(array) may not equal len(index).
The consequence of having an indexed option type is that by masking an array you are not guaranteed to later be able to recover the masked values.

Hej Angus, thank you very much for the quick reply.

Awkward array isn't guaranteed to preserve the original values of masked elements.

Okay, this is something I expected already.

I'm not sure that np.histogram supports masked arrays at the moment

Okay, thanks. Yes the current behavior is a bit funky. At least none of us expected this behavior. Okay then I will warn our collaboration about it. Thank you very much for the quick help.

FYI: I think you mean ak.ravel before converting to numpy, is not it? At least np.ravel is not doing anything on the masked array as my data is already flat.

np.ravel invokes ak.ravel, which does two things - it flattens and it drops None values. Both are generally useful when filling histograms :)

If you're happy with the solutions posed here, then I shall close the issue :)

Sure, thanks for your help. I think the issue is more related to numpy showing this unexpected behavior, Though I think it is quite a pitfall and a bit worrisome if you are doing simulations, working with uproot etc.


AUTO DARK MODE

Description of the new feature / enhancement
I want PowerToys to automatically recognize the sunrise and sunset time of each day or user-defined time, so that Windows OS can switch to day/night mode automatically.
Scenario when this would be used?
It can be used to protect your eyesight,especially for those who have a long screen time almost every day such as Programmers.
Supporting information
No response

Duplicate of #1331

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


Need proper introduction for CSS id selector

Describe the Issue
I believe that step 8 of the accessibility quiz is the first time the CSS id selector is introduced. (I searched the previous courses and did not find it. Even though the registration form uses the id attribute on some HTML elements it does not style them with the id selector.) The instructions for step 8 ask the user to style the image using its id as the selector but they don't explain how to do that. The user only gets this information from a hint if they don't do it correctly.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-accessibility-by-building-a-quiz/step-8
Your code
N/A
Expected behavior
I think it is preferable if we explain how to use the id selector in the instructions instead of having the user fail and then telling them how to do it through a hint. Since step 8 already has a considerable amount of content explaining the max function, one option would be to create a new step just for explaining the id selector. Step 9 has the user add padding to the image, so we could create a new step before the current step 8 with an explanation on how to target by id and have them add the padding in that step instead (since they are already familiar with that property). This new step would be similar to step 26 in the Cafe Menu..
But if adding a step is a pain then we could swap step 8 and step 9 since step 9 has a relatively small amount of content, and add the explanation of the id selector there.
Or we could look for a place in a previous course to use the id selector. It does seem a little odd to wait so long to use it since it is a pretty popular selector.
Or I suppose we could do nothing and keep relying on the hint to give them the information after the fact. To be honest, I don't recall seeing too many issues about this in the forums. I did a search and maybe found one or two that were about this issue out of the thirty or so I looked at. So perhaps the hint method is doing its job? It just seems to me that an important and common selector like this should be clearly explained up front. This isn't something they would be able to deduce from information they have already learned. They would need to go to an outside source to figure it out since it is a completely new selector.
Screenshots
No response
System
N/A
Additional context
No response

I think the id selector should be used and explained in "Learn Basic CSS by Building a Cafe Menu" which introduces CSS selectors. I see no reason why it can't be.
Maybe for the wrapping div element (so #menu instead) or for the footer element.

If no one is working on this issue i'd like to give it a shot

I like the idea of introducing it earlier, yeah.
I would propose editing Step 20 of the Cafe Menu to set the id="menu" attribute on the div, then editing Step 21 to teach the #menu selector.

For anyone doing this PR remember the change has to be reflected through all the seed code. So it is not just changing two challenges. But the two, and all following seed code (CSS selector and HTML attribute). And BTW, this challenge has 91 steps.


Create fulton58

@CollectivelyCode lib/domains/org/fulton58: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@CollectivelyCode lib/domains/org/fulton58: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@CollectivelyCode lib/domains/org/fulton58: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@CollectivelyCode lib/domains/org/fulton58: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@CollectivelyCode lib/domains/org/fulton58: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@CollectivelyCode I have to close this request as a duplicate. The record for this school exists already. If you experience any trouble when requesting a free license at https://www.jetbrains.com/shop/eform/students please send a screenshot with the error message to<EMAIL_ADDRESS>


Visual C++ Redistributable Package required for SQL Server 2019

After quite a bit of trial and error I managed to get SQL Server 2019 TDE/EKM using Azure Key Vault working. Although the documentation version for this page was set to SQL Server 2019, there is no documentation pertaining to which version of the Visual C++ Redistributable is required to enable the SQL Server Connector for SQL Server 2019. The page currently notes that SQL Server versions 2014 and older require Visual C++ Redistributable Packages for Visual Studio 2013, while SQL Server 2016 requires Visual C++ Redistributable for Visual Studio 2015.
The Visual C++ Redistributable installed with SQL Server 2019 is the Visual C++ Redistributable Package for Visual Studio 2017, which does not currently work with the SQL Server Connector. One would assume that since the documentation shows SQL Server 2016 "supposedly" requires the Visual Studio 2015 version, that SQL Server 2019 would use the same; however SQL Server 2019 currently requires the Visual C++ Redistributable for Visual Studio 2013 in order to enable the SQL Connector.
To Do:
Please verify that your documentation for SQL Server 2016 is correct.
Please add documentation for SQL Server 2017.
Please add documentation for SQL Server 2019.

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

ID: 4997f245-c966-51a3-34c7-705c3cfbd4b0
Version Independent ID: 129fbe40-3b02-f2c1-ef13-b42d86ffdb77
Content: Setup TDE Extensible Key Management with Azure Key Vault - SQL Server
Content Source: docs/relational-databases/security/encryption/setup-steps-for-extensible-key-management-using-the-azure-key-vault.md
Product: sql
Technology: security
GitHub Login: @jaszymas
Microsoft Alias: jaszymas

@astretch -- thank you for your thorough feedback.
@jaszymas -- please look into this issue.

Hello @astretch - I apologize for the delay. We have updated the section regarding the SQL Server Connector, but it looks like the Visual C++ Redistributable portion has not been updated.
@Rupp29 @shohamMSFT - Hey Adrian/Shoham, do you mind checking on this issue and see if we can get an update? It looks like even the download site has outdate prereqs.
https://docs.microsoft.com/en-us/sql/relational-databases/security/encryption/setup-steps-for-extensible-key-management-using-the-azure-key-vault?view=sql-server-ver15&tabs=portal#prerequisites

I had to reach out to the engineer to track down the correct answer for the C++ versions.
I just heard from the engineer this morning, he said he was looking into it.
Once that is confirmed, I can update the docs with the appropriate information.
::Adrian
From: Van To @.>
Sent: Thursday, April 1, 2021 12:02 PM
To: MicrosoftDocs/sql-docs @.>
Cc: Adrian Rupp @.>; Mention @.>
Subject: Re: [MicrosoftDocs/sql-docs] Visual C++ Redistributable Package required for SQL Server 2019 (#4670)
Hello @astretchhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fastretch&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514408524|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=AjLfVqbJA%2BzEuoDrB458IheQrpPpmMhGl3jZxlbbszA%3D&reserved=0 - I apologize for the delay. We have updated the section regarding the SQL Server Connector, but it looks like the Visual C++ Redistributable portion has not been updated.
@Rupp29https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRupp29&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514408524|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=9W01%2BBbMHa8Bd9A5lkVknJBY7LPIsniNMY3tXv%2FiS2c%3D&reserved=0 @shohamMSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FshohamMSFT&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514418517|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PbwzZmvNMD9ohgQCIKZiXpLSPEttfErc6sH9kEjYnLY%3D&reserved=0 - Hey Adrian/Shoham, do you mind checking on this issue and see if we can get an update? It looks like even the download site has outdate prereqs.
https://docs.microsoft.com/en-us/sql/relational-databases/security/encryption/setup-steps-for-extensible-key-management-using-the-azure-key-vault?view=sql-server-ver15&tabs=portal#prerequisiteshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fsql%2Frelational-databases%2Fsecurity%2Fencryption%2Fsetup-steps-for-extensible-key-management-using-the-azure-key-vault%3Fview%3Dsql-server-ver15%26tabs%3Dportal%23prerequisites&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514418517|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=eSG0qOOWBgoAFP00mPn%2FD8fnS5LdJw6UFAXdm8UP40M%3D&reserved=0

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fissues%2F4670%23issuecomment-812109440&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514428511|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=msZWfNt%2F0j%2F0%2FtIU0CkLM0Vf5uYXyVWciAT4Gr%2FcZ5U%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZWNZRYEUPX4LNR6SWQL7TTGS7MJANCNFSM4MT76DOQ&data=04|01|arupp%40microsoft.com|dafb099a810242d3037508d8f540b20f|72f988bf86f141af91ab2d7cd011db47|1|0|637529005514428511|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=r%2BOGNEumAj0RJP5dMklWqN4B7nukq6l3d73JTUJy%2FHA%3D&reserved=0.

@Rupp29 - Thanks Adrian!
#reassign:Rupp29

@Rupp29 - Hey Adrian, checking back on this. Did we get an answer?

Nothing yet.
I’m not sure how to move ahead on this.
I don’t have to the means to test this myself.
Adrian Rupp
Sr. Program Manager
Azure SQL DB - Security
VoIP:<PHONE_NUMBER>
Mobile:<PHONE_NUMBER>
Email: @.***

From: Van To @.>
Sent: Friday, June 11, 2021 2:35:37 PM
To: MicrosoftDocs/sql-docs @.>
Cc: Adrian Rupp @.>; Mention @.>
Subject: Re: [MicrosoftDocs/sql-docs] Visual C++ Redistributable Package required for SQL Server 2019 (#4670)
@Rupp29https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRupp29&data=04|01|arupp%40microsoft.com|192672f3899343694f0a08d92d20da6b|72f988bf86f141af91ab2d7cd011db47|1|0|637590441399316771|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=wFddAToDbm2XBwcCgaAFFz%2FIZU70lcuPEoXRDb%2FZ8aY%3D&reserved=0 - Hey Adrian, checking back on this. Did we get an answer?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsql-docs%2Fissues%2F4670%23issuecomment-859922209&data=04|01|arupp%40microsoft.com|192672f3899343694f0a08d92d20da6b|72f988bf86f141af91ab2d7cd011db47|1|0|637590441399326766|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=zYsePt4wdvg0tkeHTPDZqtgZNE4i%2B4waLYQxYCUJ1Is%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZWNZU6X3ZNUKUWSTLWPNLTSJ6STANCNFSM4MT76DOQ&data=04|01|arupp%40microsoft.com|192672f3899343694f0a08d92d20da6b|72f988bf86f141af91ab2d7cd011db47|1|0|637590441399326766|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=HRAGn1VBEw8l6uCCn%2F0Jn4rZOOG1RC6K8JvtWKl%2FBTY%3D&reserved=0.

@Rupp29 fixed this in https://github.com/MicrosoftDocs/sql-docs/commit/7d2922dfe23c77d6ef52e8a7bef7bc302cf94f03


terminal.onCharInput can crash if dart.io.stdin is already listened to

dart.io.stdin itself isn't a broadcast stream.
IDEs subscribed to stdin can cause the tool to crash in terminal.onCharInput

Does the IDE need to run flutter with --non-interactive?  Or do we need to wrap stdin in a broadcast stream?  Or something else?  @tvolkert @devoncarew

Letting dart.io.stdin be exposed as a broadcast stream seems like a generally safe thing to do unless I'm missing some context. Though it seems like that file was last touched 5 years ago anyway.

I presume there is a memory cost?  @zanderso might know.

If there is an additional memory cost, I would guess that it is pretty small. I would guess that dart:io stdin is not a broadcast stream by default because broadcast streams silently drop events when there are no listeners, which would be bad if you really want all the bits that are sent your way. Also, if what you really want is a broadcast stream you can always say stdin.asBroadcastStream().

/cc @floitschG in case he has any additional information.

The problem with a broadcast stream is, that it justs dumps events into the void when nobody is listening. For stdin that would probably be worse that not being able to have multiple listeners on stdin.
In general, we think that stdin is a resource that can't just be used independently by different parts of a program. That is, there should be some part of the program that is responsible for managing access to stdin. That part can then also deal with multiple listeners by using the StreamQueue class from package async (for example).
Related issues:
https://github.com/dart-lang/sdk/issues/24881
https://github.com/dart-lang/sdk/issues/24464

IDEs subscribed to stdin can cause the tool to crash in terminal.onCharInput

This is an external process subscribed to stdin, or flutter_tools listening to stdin in more than one place?
We should likely be able to infer a non-interactive global flag from running in flutter daemon or flutter run --machine. Though, I'd want to look at places where we're asking for user input and see if those can't be eliminated (https://github.com/flutter/flutter/issues/10626).

I don't think there is anything actionable here for us. Closing for now


Debian: type initializer for 'LibGit2Sharp.Core.NativeMethods' threw an exception

See
https://github.com/libgit2/libgit2sharp/issues/1533#issuecomment-661287726
and
https://github.com/dotdevelop/dotdevelop/issues/52#issuecomment-771183443
the branch to fix it is this: https://github.com/dotdevelop/libgit2sharp/tree/dd/issues/2
the git hash for libgit is 6777db8e8343014d57c41b335e0c60af44d4e9f2 ->

so, according to @Molytho , the problem is in the mismatch from
https://github.com/dotdevelop/libgit2/blob/6777db8e8343014d57c41b335e0c60af44d4e9f2/include/git2/status.h#L170-L197
to
https://github.com/dotdevelop/libgit2sharp/blob/3a544afbe551ae4ae6ff4b4f732f1341d3b5b966/LibGit2Sharp/Core/GitStatusOptions.cs#L6-L20

I started testing and I think I found a solution/workaround. I don't know how to set the project up properly to use a self build nuget for libgit2-sharp so I changed it to a project reference so this might not be 100% accurate.
First I compiled it with a clean clone of this repo. It didn't work.
Then I added the line
public void* Baseline = (void*)0;
to the end of the GitStatusOptions class. After that it worked fine.

It's surprising that this mis-match hasn't caused problems long before now!  It seems to be in the current master and latest release (maint/v0.26) branches of github/libgit2/libgit2sharp, and not just the mono/dotdevelop forks.
Well tracked down @Molytho!

maybe it has to do with a more sever (and saver) memory access control in recent versions of mono?
and, yes, @Molytho did a great job, and he should upstream that error in the github/libgit2/libgit2sharp - repo.

Could the LibGit2Sharp.NativeBinaries be updated with this change, together with the dotdevelop build, please (even though the 'no TLS stream' issue is still outstanding)?  Thanks.

Yes. Done..

The fix got merged to upstream:
https://github.com/libgit2/libgit2sharp/commit/b78b2900dfe3dc835e0e79f6e542672826bdbc9a

That's great. Many thanks @Molytho !


Use setuid to launch slurm services during container startup

In the new AiiDA2 image, instead of needing to run container as root, we could perhaps just setuid on the slurm launch script.
https://en.wikipedia.org/wiki/Setuid

Turns out setuid is ignored for bash (and other interepreted) scripts.
https://unix.stackexchange.com/questions/364/allow-setuid-on-shell-scripts
https://www.baeldung.com/linux/setuid-and-user-ids
https://www.liquidweb.com/kb/how-do-i-set-up-setuid-setgid-and-sticky-bits-on-linux/
We might still try to set setuid on the slurmd, slurmctld and / or munged executables, but that's probably even less secure of than what we're doing now.


Dev 2 2 0 kobeya 1002

Ticket Link
Groupings-1002
Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

Member

Should this be fixed in master as well?

Yes.  Please also fix in master. Thanks.


Enforce pure YAML syntax

Feature request
Desired Behavior
Add a rule to enforce pure YAML. There are a number of benefits to this (linked below). The rule would disallow the use of key=value in a task. For example, instead of writing a mount directive like this:
---
- name: Ensure the floppy disk is mounted.
  mount: "name=/mnt/floppy src=/dev/fd0 fstype=fuse opts=allow_other,user=nobody,nonempty dump=0 passno=0 state=mounted"

It should be written like this:
---
- name: Ensure the floppy disk is mounted.
  mount:
    name: /mnt/floppy
    src: /dev/fd0
    fstype: fuse
    opts: "allow_other,user=nobody,nonempty"
    dump: 0
    passno: 0
    state: mounted

This could certainly be off by default, but in my opinion all Ansible roles should be written this way. Here are some useful links justifying this choice:

Why you should use YAML as YAML with Ansible
OpenShift Ansible Best Practices
YAML Best Practices for Ansible Playbooks

I've written an ansible-review rule that does this - I'll see if I can make it an ansible-lint rule.
I may well add it to an extras rule set rather than the defaults, not sure how popular new rules always are (I was happy to add a few new ones with v3.0)

Looking at how ansible-lint currently runs checks, this is much easier to do with ansible-review (I could add a fourth matcher that just runs against the content of a file, but this is already implemented in ansible-review)
https://github.com/willthames/ansible-review/blob/master/examples/standards.py#L21
https://github.com/willthames/ansible-review/blob/master/examples/standards.py#L233
ansible-review can also run whatever lint checks you require against certain classes of file.

ansible-lint 3.1.0 released with this change


[Scheduled] flutter pub upgrade --null-safety

Auto-generated by create-pull-request
See: https://github.com/peter-evans/create-pull-request/blob/master/docs/concepts-guidelines.md#triggering-further-workflow-runs

Codecov Report

Merging #24 (e96a9ca) into main (53aeae4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   16.66%   16.66%
=======================================
  Files           1        1
  Lines           6        6
=======================================
  Hits            1        1
  Misses          5        5

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


Missing assert header files

Compilation fails because of missing #include  in MatlabIO.cpp.
Compiler gcc 4.7.3

I have the same question when I compile the main.cpp. In fact, I don't know how to use this code. do you have solve this question?


[#240] Edited Backend Readme file to have the correct port.

Description:
This is related to issue #240 the backend port lined in the Backend readme file was wrong. This PR would update it to the correct one.
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] My code follows the code style of this project
[ ] My code has been commented
[ ] My change requires a change to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

LGTM :)


Clamp Idle capacity Growth By tx Pacing

Clamp the txPortal.capacity ceiling as a function of tx pacing. When tx calls are more frequent, the upper ceiling of the txPortal.capacity will be reduced by a smaller factor, and when tx calls are infrequent, the txPortal.capacity will be reduced by the larger delay between tx calls.
This should allow infrequent transmitters to "soft start" in the middle of a session.

Looking like a non-issue in westworld3.1.


Bug: Opening Git directory automatically switches branches

Description
Hi,
few weeks ago, I opened my project in Files and it prompted me tu sync it with Git - I clicked the second button to cancel it. Visual Studio said, that it needs a restart to apply new changes in my code (I though that it was because of one specific build error), so I restarted it. After opening VS, I was shocked. It removed my whole progress to the last commit, which was 4 hours of work. I had to write my code again, without any option to revert it.
Two days ago, I opened my project in Files and it prompted me to sync my progress on my branch with Git. I remembered, that last time I picked "Cancel", so now I picked to keep the local changes (I don't remember how it was named) on my branch. Luckily, it didn't remove my whole progress, but it brough back almost 2 month old changes that created merge conflicts. I had to resolve every conflict to get my progress back. It looked like it was merging with stashed changes (I really don't know why, because I didn't create them).
Why does it want to merge old random files and delete my progress when I'm on the same branch for like 3 months already? I did not change my branch at all. Files always showed my current branch whose name was correct.
For now I'll stop using Files until it's fixed. I would love to see an option to log off from the GitHub integration since I don't find it usefull for myself.
Steps To Reproduce

Open your project folder
There's a small chance that a content dialog appears with options what to do with your branch settings

Requirements

Identify why the content dialog is displayed
Make sure it only shows if the user specifically chooses an action

Files Version
<IP_ADDRESS>
Windows Version
10.0.22635.2552
Log File
debug.log

Thanks for the report, I've added this to the project board.

Update: I found out that this is the dialog that randomly pops up. It's for switching the branch, but even when you click "Zrušit" (Cancel) it still removes your progress.

The second time I was it I chose the "Keep my changes on 'Beta'" (the 2nd one), but it created merge conflicts.

For now I have just revoked File's access to my GitHub account, which should make it safe to use without the bug appearing. This should disable the GitHub integration, right? :D

@Pietro228 the GitHub integration is only for pushing changes to the remote repo. It sounds like this issue can still affect your local changes.

It wont be edited if you don't open the git folder with Files.

Please let us know if anyone else is able to reproduce this issue.

@Pietro228 the GitHub integration is only for pushing changes to the remote repo. It sounds like this issue can still affect your local changes.

That's a pity :(

It wont be edited if you don't open the git folder with Files.

But I need to access it frequently, because I have to work with binaries.

Please let us know if anyone else is able to reproduce this issue.

Alright.

You can change the branch in Visual studio or even the command line.

I know, but I don't want to change it. I'm a solo dev, so I don't need to switch it that frequently

Thank you for taking the time to report this, we'll continue investigating and will let you know if/when we identify the cause.

We still haven't been able to repro this issue but we'll keep looking.

@Pietro228 have you been able to repro this since opening the issue?

@yaira2 I haven't used Files since then, because it's still not that stable for my needs. However when I try to reproduce it now, it doesn't appear. Maybe it could appear randomly one day, but I don't use Files on daily.

However when I try to reproduce it now, it doesn't appear

We've also been monitoring for similar reports and haven't heard any so it's likely our refactoring fixed the issue. That said, if anyone has is able to repro this issue, please let us know so that we can investigate further.


Failure to save notebooks due to issues with jupyter_server_fileid on jupyterlab > 3.6

Based on report in https://2i2c.freshdesk.com/a/tickets/476, me and @pnasrat had a debugging session trying to figure out why File saving is broken for some users (turned out to be very few!) on the UToronto hub as a result of #2240.
Our conclusions are:

JupyterLab 3.6 brings in something called ydoc, related to real time collab
This brings in a fileid server extension (that wasn't present in 3.5) - https://github.com/jupyter-server/jupyter_server_fileid
This puts file ids on a sqlite db
This is on the critical save path - if there are any failures here, no notebooks can be saved
And there was failures

[W 2023-02-22 14:05:53.096 SingleUserLabApp manager:359] jupyter_server_fileid | extension failed loading with message: unable to open database file
[E 2023-02-22 14:05:53.096 SingleUserLabApp manager:360] jupyter_server_fileid | stack trace
    Traceback (most recent call last):
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/manager.py", line 355, in load_extension
        extension.load_all_points(self.serverapp)
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/manager.py", line 229, in load_all_points
        return [self.load_point(point_name, serverapp) for point_name in self.extension_points]
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/manager.py", line 229, in <listcomp>
        return [self.load_point(point_name, serverapp) for point_name in self.extension_points]
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/manager.py", line 222, in load_point
        return point.load(serverapp)
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/manager.py", line 148, in load
        return loader(serverapp)
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/application.py", line 453, in _load_jupyter_server_extension
        extension.initialize()
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/application.py", line 420, in initialize
        self._prepare_settings()
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server/extension/application.py", line 302, in _prepare_settings
        self.initialize_settings()
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server_fileid/extension.py", line 23, in initialize_settings
        file_id_manager = self.file_id_manager_class(
      File "/opt/conda/lib/python3.10/site-packages/jupyter_server_fileid/manager.py", line 244, in __init__
        self.con.execute("PRAGMA journal_mode = WAL")
    sqlite3.OperationalError: unable to open database file

So we should find a way to report this upstream - this extension is in the critical path of file saving even if we do not touch real time collaboration at all
sqlite and NFS do not like each other
In the meantime, we can unblock utoronto by pinning to jupyterlab < 3.6

@pnasrat also found https://www.sqlite.org/wal.html, which states WAL will not work over network filesystems. It fails arbitrarily.
In general, I also think the fileid stuff should not be enabled unless you are doing RTC?

Created https://github.com/2i2c-org/infrastructure/issues/2245 to track getting rid of sqlite from NFS

Reported https://github.com/jupyter-server/jupyter_server_fileid/issues/60 upstream

BTW, I think the issue title should be >= 3.6?

As a note, based on https://github.com/jupyter-server/jupyter_server_fileid/issues/60#issuecomment-1444605208, this could also have been the nbsignatures.db file causing issues, rather than the file_id file.
This brings up the question of why these failed at all, which could just be NFS being NFS.
This also means https://github.com/2i2c-org/infrastructure/issues/2245 is decently important if we want to avoid a repeat.

Given #2245 is filed and the issue has been reported upstream I believe this can be closed, but putting in Waiting for the moment

I agree, closing it.


[ServiceBus] Support AAD token provider

Closing this as the work is complete, and coming out with our Track-2 (1.0) release; preview will be coming in the next few weeks.
If there is a high-pri need to have this in the 0.50 library, don't hesitate to reopen this or make an issue that we'll triage accordingly.


Fix OGG audio loops

Fixes #36289 regression, breaking audio loops under certain conditions.

Thanks, seems to work fine!

Cherry-picked for 3.2.1.


XAML designer will not render any forms

Describe the bug
I have installed the WinUI/ProjectReunion/WindowsAppSDK extensions in VS2019 and followed all the steps in the tutorials but I cannot get the form to paint in the IDE. I have checked all of the settings in the options menus and my other XAML projects render the form just fine. NOTE: The app will run and compile without issue, but will not render in the IDE.
I'm using VS2019 Version 16.10.3 on Windows 10 Pro, build 19042.1110
Steps to reproduce the bug

Steps to reproduce the behavior:

Create blank app WinUI project.

Using NuGet just makes the problem worse...

@fabiant3 FYI.
@GuildOfCalamity Xaml designer is not yet supported by WinUI3.

@fabiant3 FYI.
@GuildOfCalamity Xaml designer is not yet supported by WinUI3.

So how are we supposed to do the visual design??

Hi @GuildOfCalamity,
The XAML Designer for WinUI 3 is not in the plan for 2021.  I agree VS should aware of users that WinUI doesn't support XAML Designer, although I thought the designer is disabled by default in the WinUI project templates to avoid these misunderstandings.
Also, I believe that Alvin was using a preview version of WinUI 3 in UWP in his book. A lot of things happening since Alvin released the book. For example:

WinUI 3 is now part of the Windows App SDK.
Windows App SDK has different releases: Experimental, Preview, and Stable.
The Experimental release contains WinUI 3 running in UWP, in addition to WinUI 3 in Desktop (Win32).
WinUI 3  the Preview and Stable releases contain only WinUI 3 in Desktop
XAML Designer only worked for WinUI 3 in UWP in the past, given we were reusing the UWP XAML designer (actually, we adjusted several things to make it work). The UWP XAML designer is not compatible with WinUI 3 in Desktop given WinUI 3 in Desktop has substantial changes that require a rearchitected of the XAML Designer.

Without XAML designer, the recommended development inner loop is to use XAML Hot-Reload, Live Visual Tree, and in-app command bar to design the app while the app is running so you can apply changes and see the result immediately.
Closing this bug given it's a dupe of https://github.com/microsoft/microsoft-ui-xaml/issues/4195

Also, I've seen screenshots of people working with the designer using WinUI 3.0. Alvin Ashcraft's new book proves that it worked at some point.

If you're referring to figures 11.1 and 11.2 (from chapter 11) these show a WinUI2/UWP app running. I believe the intent was to show how it will eventually be (& was hoped for at the time of writing.)
This also hints at a possible workaround.
As WinUI3 XAML is very similar to that used in UWP apps you can write your XAML in a UWP app (including seeing it visualized in Visual Studio) and then copy it into a WinUI3 app. Or set up a solution with both projects that share the XAML.
If you're new to building with XAML, you might also want to try out XAML Studio (Microsoft Store link) as that will also allow you to visualize the entered XAML with a version of XAML that is very similar to that available in WinUI3.
As a further alternative, you can often get a better experience than with the Visual Designer by using XAML Hot Reload (also covered in Chapter 11 of Alvin's book.)
Another option we can look forward to in the future is XAML Live Preview which will hopefully be available for more than just WPF in a future Visual Studio preview.
Not an excuse for the current situation but some possible workarounds and a suggestion that things will get better in the future.

Thanks for the info!

I did just try VS2022 starting with a blank UWP project, but my god is it buggy... seems to have no idea what a grid is???

@GuildOfCalamity This would happen if there were some kind of problem with the References in the project. For a normal working blank project it should look like this:

Duplicate of #4195


i need to know what this value mean in PEX's database

Hello..
i need to know how to analyze this value in database!

like how match time it need to expire?
Thank you.

it's seconds since the epoch. timed permissions were a bit funky on 1.23 so good luck?


Discussion of experiment and analysis data

Experiment Result Data
Currently we have issues with some miss-matches between the qiskit-experiments result API and the Result DB API which we have brute forced to work, but in an somewhat unsatisfactory way.
Result DB API has

Experiment: Identifies an experiment as a device and list of jobs, figures, analysis results
Analysis Result: Identifies a single result consisting of type, fit, quality, chisq, components etc

Qiskit-Experiments has

Experiment: class that runs experiments
ExperimentData: container of jobs, figures, analysis results
AnalysisResult: free form result dict forced to match ResultDB result but doesn't really work.

Essentially what qiskit-experiments considered an "analysis result" (ie the output of Analysis.run) is what the ResultDB calls an "Experiment".
Two main issues are:

The individual Analysis results are not design in a way that works well with the result DB API.

Our ExperimentData container is designed to be able to add extra data and re-run analysis, but that should probably be uploaded as a separate "Experiment" into the DB to separate the list of all results and figures under a unique experiment id and corresponding job IDs, since the now full set of job ids only refers to the second analysis, not thee first one which was a subset of the jobs stored.

AnalysisResult vs DbAnalysisResultV1
Currently we have the service DbAnalysisResultV1 class of the form
DbAnalysisResult(
  result_data: Dict,
  result_type: str,
  device_components: List[Union[DeviceComponent, str]],
  experiment_id: str,
  result_id: Optional[str] = None,
  chisq: Optional[float] = None,
  quality: Optional[str] = None,
  verified: bool = False,
  tags: Optional[List[str]] = None,
  service: Optional[DatabaseServiceV1] = None,

We want fit result classes that can be converted to the above in a sensible way, but we dont always want to deal with all this formatting in the Analysis classes directly. This conversion from more user friendly classes should be handled by the service code as part of its saving/loading.
Currently this is handled through an adhoc mixture of AnalysisResultdata, AnalysisResult classes by the BaseAnalysis class. Even though our Analysis._run_analysis method in principle returns a list of result data objects (currently AnalysisResultData) no experiments use this functionality and return a single blob containing everything. We then have a separate AnalysisResult class which this is a direct subclass of DbAnalysisResult which automatically adds all extra fields.
One way we could look at improving this is by making our _run_analysis return a set of analysis results designed to match a single entry. Something like
@dataclass
class AnalysisResult:
    name:str = None
    value: any
    stdev: Optional[any] = None
    unit: Optional[str] = None
    chisq: Optional[float] = None
    quality: Optional[str] = None
    verified: Optional[bool] = None
    extra: Optional[dict] = None

Everything except name and value would be optional to allow entry of arbitrary data, but the remaining fields can be used where appropriate.
Value can be any python object (that can be serialize by the JSON encoder), but we have special handling in the service code for when value/stddev are floats for display in the result DB.
Some examples of how we could use this for existing experiments
T1 Example
Currently the  T1 experiment returns a result object that looks like
{
    "value": popt[1],
    "stderr": popt_err[1],
    "unit": "s",
    "label": "T1",
    "quality": quality
    "fit": {
        "popt": popt,
        "popt_keys": param_keys,
        "popt_err": popt_err,
        "pcov": pcov,
        "reduced_chisq": reduced_chisq,
        "dof": dof,
        "xrange": xdata_range,
    }
}

To convert this to above DB friendly AnalysisResults could depend on how we want to highlight the results in the DB for the given experiment. Eg for T1 we could list everything as separate results such as:
[
    AnalysisResult(name="T1", value=popt[1], stddev=popt_err[1], unit="s", quality=quality),
    AnalysisResult(name="a", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="tau", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="a", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="pcov", value=pcov)
]

One problem with this is there is a lot of data that isn't that relevant and clutters the result view with lots of table entries. So if those aren't pertinent to the main experiment result we could use the extra tag to include it in fewer results such as
AnalysisResult(
        name="T1",
        value=popt[1],
        stddev=popt_err[1],
        unit="s",
        quality=quality,
        extra={
            "popt": popt,
            "popt_keys": param_keys,
            "popt_err": popt_err,
            "pcov": pcov,
            "reduced_chisq": reduced_chisq,
            "dof": dof,
            "xrange": xdata_range,
        }
)

It would be up to the writers of experiments to decide how to break up its analysis results.
State Tomography
Another example which currenlty does not fit well with Result DB is tomography experiments. Here we have raw data like
{
 'state': rho,
 'state_fidelity': fid,
'fitter_metadata': {'fitter': 'linear_inversion',
                     'fitter_time': t},
 'state_metadata': {'eigvals': evals,
                    'positive': True,
                    'positive_delta': 0.0,
                    'trace': trace},
}

This could be broken up into something more useful for DB like
[
    AnalysisResult(
        name = "state_fidelity",
        value = fid,
        quality = quality),
    AnalysisResult(
        name="state",
        value=rho,
        quality=quality,
        extra={
            'fitter': 'linear_inversion',
            'fitter_time': t
        }),
    AnalysisResult(
        name = "eigvals",
        value = evals,
        quality = quality
    )
]

where in this case quality could for example be "good" if eigvals > 0 and bad otherwise.
ExperimentData vs DbExperimentDataV1
Each collection of analysis results and figures produced by Analysis.run needs to be identified together (ie DB Experiment). In our current code it is not since ExperimentData can have analysis re-run arbitrarily many times, possibly with additional data being added in between.
Here we could have a class like AnalysisData which is this collection and is closer to what Result DB calls an DbExperimentData than our current ExperimentData class is. Then our ExperimentData coudl contain multiple experiment_ids and AnalysisData objects from re-running or doing repeated analysis etc. In this case the DB experiment_id tag could refer to AnalysisData, and re-running analysis uploads the new results under a new "experiment id" while still referencing the same jobs as the previous experiment.
One issue in this case would be how to store an experiment with no-analysis, and then add analysis later.
Another option would be to change how ExperimentData works to force it to only contain a single collection of analysis results, and anytime analysis is re-run it would make a new copy of ExperimentData without previous analysis results/figures and with a new id, and then add the new results.

Our ExperimentData container is designed to be able to add extra data and re-run analysis, but that should probably be uploaded as a separate "Experiment" into the DB to separate the list of all results and figures under a unique experiment id and corresponding job IDs, since the now full set of job ids only refers to the second analysis, not thee first one which was a subset of the jobs stored.

👍 agreed

Thanks Chris for summarizing this. Here is my thoughts.

miss-matches between the qiskit-experiments result API and the Result DB API

Yes, I agree. I commented this in PR #113. Here is relevant discussion with @jyu00 https://github.com/Qiskit/qiskit-experiments/pull/113#issuecomment-870087005
In your example,
[
    AnalysisResult(name="T1", value=popt[1], stddev=popt_err[1], unit="s", quality=quality),
    AnalysisResult(name="a", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="tau", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="a", value=popt[1], stddev=popt_err[1], chisq=reduced_chisq),
    AnalysisResult(name="pcov", value=pcov)
]

In this form the last entry is somewhat meaningless since it doesn't contain the label for each matrix entry.
So I more prefer
AnalysisResult(
        name="T1",
        value=popt[1],
        stddev=popt_err[1],
        unit="s",
        quality=quality,
        extra={
            "popt": popt,
            "popt_keys": param_keys,
            "popt_err": popt_err,
            "pcov": pcov,
            "reduced_chisq": reduced_chisq,
            "dof": dof,
            "xrange": xdata_range,
        }
)

however we may not want to let developer decide what is the primary parameter. For example, the baseline value of RB experiment tends to be considered as an extra parameter, but we could guess a leakage error from the asymptote, i.e. if the value of 1Q RB is off from 0.5 this may indicate leakage, and this could motivate us to run further leakage analysis. Once we updated the policy of the primary parameter, we cannot guarantee consistent data structure with previous experiments.
Note that if the experiment has more than two parameters to report, this data structure will rapidly become inefficient in data volume. For example, Hamiltonian tomography #105 will need to report 6 parameters. It's more efficient to take the data structure in the first example in this case.

Our ExperimentData container is designed to be able to add extra data and re-run analysis, but that should probably be uploaded as a separate "Experiment" into the DB

Yes, I agree. To make the link between entries clearer, I think experiment ID structure should be something like
93308403-ae72-43bd-9430-6cb3491f405a.000

where the last three digit followed by a dot represents number of trial for the experiment data entry. This allows us to manage database view with versions, i.e. somewhat like arXiv webpage where we can track entry with the version and comment (in DB we have note field).
This discussion make me think result_type of the DbAnalysisResult should be

class AnalysisType(IntEnum):
    CurveAnalysis = 0  # any curve fit type analysis
    MatrixAnalysis = 1  # QST, QPT, readout mit
    QuantumVolume = 2  # perhaps curve analysis

so that we can do kind of schema validation for result format. Then we can add analysis method as
result_data = {
   "analysis_class": T1Analysis,
   "fit_model": "a exp(-t/tau) + b"
}

as a part of metadata, i.e. extra in your example.

In this form the last entry is somewhat meaningless since it doesn't contain the label for each matrix element.

There is no reason we couldn't add the matrix element labels to the extra field.
One shouldn't think of result_type as denoting a type system. It is a key that users may search on in the database. So, it allows me to pull all the T1, T2star, results out of the database for comparative analysis (e.g. variation over time, qubit index, device, design, etc.). I support the idea of using common constructors to ensure uniformity across all curve fit analyses, but from an experiment service perspective, they should have unique result_types.

Also, 💯 to the idea of incorporating the fit model in an easy to read format like @nkanazawa1989 suggests with:
"fit_model": "a exp(-t/tau) + b".

Here we could have a class like AnalysisData which is this collection and is closer to what Result DB calls an DbExperimentData than our current ExperimentData class is.

I have to say that I already think we have too many levels of object hierarchy. I'd rather go the other way and figure out how to make ExperimentData disappear as the chain already feels too long.

After discussing these issues with @jyu00 we came up with a few things to try and implement.
As first step we are going to make the following changes to DbExperimentDataV1 since they should be straightforward to implement and fix some of the current issues:

If you add more data to an ExperimentData if it will make a new copy of the ExperimentData (and DB experiment) containing the previous jobs and new jobs, and only store the new analysis result (if analysis is run) in the new experiment.
If you re-run only analysis without adding extra data it will also make a new copy of the ExperimentData (and DB experiment) containing only the new analysis result (the previous result will be unchanged and accessible under original experiment id). We can add support for updating the existing results when re-running analysis later.

For the AnalysisResult classes themselves, I am going to try refactoring #236 to make changes directly to the DbAnalysisResultV1 class rather than add a new class, and then use this class directly in the experiment analysis rather than the current AnalysisResultData class. I think then adding specialized classes for certain types of values like @nkanazawa1989 has started in #237 is then a good solution so we can remove some fields like unit, stderr/variance, chisq from the base analysis result class.

What will happen if you run an experiment multiple times? Something like:
expdata1 = exp.run(...)
expdata2 = exp.run(...)  # This is not Case 1, because we don't add more data to expdata1, it's a new fresh run of `exp`

Is it working now? Will it work after the change?

@yaelbh that won't change, expdata1 and expdata2 will continue to be independent sets of data + analysis for the defined experiment

@chriseclectic Can this be closed now?


Drawing up mock ups of the website

[ ] Home Page Outline
[ ] Task View Outline
[ ] Login/Signup Outline
[ ] Calendar Outline

[ ] Add Task Outline


Version 2 Testing

I have started this pull request for implementing tests for version 2 of the library. Currently I have added two to verify user functionality.
I also added a function called recurse_dict in common.py which replaces the resource self with self_ as not to conflict with the python self.

@rossdylan Here is the newest pull request, not sure how to add you as a subscriber.

@gmjosack @rossdylan The user_test doc is getting quite long with all of the same users. Do you think it would be best to move these into a separate file and import them?

Sorry for the delay here. @khardsonhurley and I worked offline to figure this out, was a mocking bug, over a pygerduty bug.


:bug: fix Module name typo and add child spec

Hello,
Is there a rough time frame of when this may be merged?

@punchcafe sorry for the delay.


Publish 1.4 to mod platforms

V1.4 of the mod seems to be out, but it is not on CurseForge yet, please publish it there. Similarly, publishing to Modrinth would be nice, it does support author payments now.

Made #27 about modrinth support, and I don't think they are going back into CurseForge because author officially stated they are not updating on that platform anymore

I won't be publishing my mods on either site anymore. I'm disagreeing with Curseforge's shitty practices, and with Modrinth moderators' stance on bans.
Since pretty much everything I do is licensed under MIT anyone could upload my mods to their favorite mod site and even receive payouts. But Github is the only official place as of now.


Open Source?

Wasn't ReportAnalyzer Open Source?

I have not released the code for Report Analyzer thus far.


[ SPARK-20739][CORE][TEST]Supplement the new unit tests to SparkContextSchedulerCreationSuite

What changes were proposed in this pull request?
This PR adds the new unit tests to support test 'SPARK_REGEX(sparkUrl)' branch in createTaskScheduler methods.
How was this patch tested?
The new unit test.

Can one of the admins verify this patch?

OK. It's minor, and not 100% sure this is the best place to test, but, it follows the pattern in the rest of the file and. This didn't need a JIRA, and please see my comments on the JIRA.

@heary-cao it seems like the additional tests are primarily targeted towards testing spark regex parsing while creating a task scheduler . If that's actually the intent, maybe we should have a more surgical test?

Sorry,
surgical test?
I didn't understand .
Can you explain in more detail?

He's saying that you're really just testing arg parsing. And it's mostly tested elsewhere, although maybe not this exact case. I don't know if this adds much value.

ok,
It does not produce value, that is junk code.
close this PR.


Revamp guns: Revival

Again, replacing the cev eris sprites and purging some unused sprites. Should be ready in 2 days max.

Ok, this is ready for review.

There's a problem: the reason why I didn't linked specific commits is because I couldn't find them (also, yay, more conflicts). As in I couldn't find any of the original old commits that first introduced the bulldog shotgun or the laser rifle from ds13.

You don't need the original commit that added a sprite, in fact you shouldn't use that one since sprites can change.


Publishing 0.2.0 to NPM

@arcticicestudio any chance this can be done?

Hi @SNDST00M :wave:, thanks for your contribution :+1:
Looks like I missed this one after pushing the version. Thanks for the notice, I'll push it later!

Alright, done 😄


Add networking operations.

recvfrom & sendto not implemented for io-uring.

Switch recvfrom to blocking impl; sendto requires a very high kernel version.

All fixed using *msg.


Clean up of codebase - Part I (server)

Before I add new feature, I wanna do some clean up as the files are getting bigger and bigger. I'll clean up the server first then the client next.

@noklam This looks good, But I have one question, you created a file bundled/tool/lsp_lsp.py which is empty Is this file needed?


Fix typo in Replicas and dynamic repositories.md

There's a typo in how-to of Replicas and dynamic repositories 🙇

Thank you!