Unable to run wsl.exe when connecting from Windows native OpenSSH server

Version
WSL 版本： <IP_ADDRESS> 核心版本： <IP_ADDRESS> WSLg 版本： 1.0.47 MSRDC 版本： 1.2.3575 Direct3D 版本： 1.606.4 DXCore 版本： 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp Windows版本： 10.0.19045.2311
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
預設發佈: Arch 預設版本: 2
Distro Version
ArchWSL
Other Software
OpenSSH_for_Windows_8.1p1, LibreSSL 3.0.2 (Installed with https://learn.microsoft.com/en-us/windows-server/administration/openssh/openssh_install_firstuse)
Repro Steps

Enable the OpenSSH component in Optional Features and make sure it is running in services.msc
Make sure WSL2 (Store version) is running correctly
Use ssh to connect to it from another machine, and you should get a cmd.exe shell from that
Execute wsl or wsl.exe and get an error message: 系統無法存取該檔案。 (Sorry, but I can't get it to report English version of error message)
Execute powershell to enter powershell
Test Get-Command wsl and it should be showing C:\Windows\system32\wsl.exe
Execute wsl or wsl.exe and get the same error message: 系統無法存取該檔案。

Expected Behavior
Executing wsl in SSH session works in previous version (non Microsoft Store version), but it breaks for some unknown reasons after I updated to Microsoft Store version
Actual Behavior
Getting error 系統無法存取該檔案。 (Sorry, but I can't get it to report English version of error message) when I try to execute wsl in a SSH session from cmd.exe or powershell.exe
Diagnostic Logs
No response

/dupe #9231


Add unit tests for bobbit.history

This PR adds unit tests for bobbit.history. Addresses task in #17.

@shawalli Besides the in-place reverse, everything else looks OK.  Go ahead and revert that and I'll test it out.  Thanks.

The original implementation was throwing an error when I ran it, that's why
I changed it. Would you want me to copy the deque and reverse that? Or
perform the for loop in reverse order?
On Sun, Oct 4, 2020, 4:43 PM Peter Bui<EMAIL_ADDRESS>wrote:

@shawalli https://github.com/shawalli Besides the in-place reverse,
everything else looks OK. Go ahead and revert that and I'll test it out.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pbui/bobbit/pull/19#issuecomment-703313396, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4KLREEORSW24DSO2RYQLSJDNAXANCNFSM4SD5EG4A
.

I think we should do:
 messages = list(reversed(messages))

@shawalli Thank you for your contribution.


String truncation warnings

Hello!
I compile  my project with lcms, and I turn up the warning levels when i build.
Here are a number of string truncation warnings I get:
https://my.cdash.org/viewBuildError.php?type=1&buildid=1943614
Is there a way of suppressing these warnings, if they are not hiding any bugs?
Thanks very much,
Aaron

I will simply suppress these warnings on my side.


Feature Request: Support environment variable for authentication file

When running tools  like podman in a CI system like Jenkins, we would love to pass folder specific credentials to push to dev, prod repositories and not expose the credentials for the user running Jenkins. Docker already supports this with the DOCKER_CONFIG environment variable
Proposal being and registry push/pull operations tries to use an environment variable named something like REGISTRY_AUTH_FILE (open to better names) and use that for primary operations otherwise fallback to the default auth.json

This does not sound that difficult.
@umohnani8 @rhatdan WDYT?

For podman pull/push/build and maybe more commands that I'm forgetting, we have an --authfile option that points to the directory that podman login has dropped the config.json authentication file into.  Can you use this or are you looking for other credentials?  If so, might be a short term solution, but we should be able to add an environment variable too if that helps.

I could live with the authfile option for now, let me explain a use case:
For dev builds it uses a credentials accessible for the dev builds only and the path to file gets exported only for that particular build, same with other environments too.  So having an environmental variable helps prevent config leak and ability to use different credentials per environment. Moreover an environment variable is much easier to use and prevents passing extra options


Adding fixes to lib.properties to allow compiling with JDK 11

Signed-off-by: Oren Kaminer<EMAIL_ADDRESS>

@denis-itskovich can be closed in favor of #27

Implemented in #27


Add auto configuration for shelly devices

Currently shelly devices are not auto configured, please find below the tree of an regular shelly device (shelly 1)

Could you please test with 0.0.7 if it works?

Could you please test with 0.0.7 if it works?

looks great with 0.0.8, please note I have only the shelly 1 so not able to test other devices of the shelly manufacture for you.
closing this one


Bug/fix carrousel

fixed the error about the title in title carrousel

closing, fix included in another PR


docs: fix custom scorecard test code sample

Description of the change:
Updates the code sample in the custom scorecard test documentation replacing the github.com/operator-framework/operator-registry/pkg/registry  import for apimanifests "github.com/operator-framework/api/pkg/manifests" and  making the required fixes. Also changes the function signature to take a pointer to a bundle.
Motivation for the change:
Was going through the documentation and noticed that it wasn't working. Then checked against sample code from the repo, e.g. https://github.com/operator-framework/operator-sdk/blob/master/internal/scorecard/tests/basic.go#L19

@camilamacedo86 if it helps, the changes in the docs match code that is currently found in the repo, e.g. https://github.com/operator-framework/operator-sdk/blob/master/internal/scorecard/tests/basic.go#L19


Put request with multipartform/data content type sends an empty request

New Issue Checklist

I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
flutter / web / android

Dio Version
3.0.0

Repro rate
all the time (100%)

Issue Description and Steps
when sending a put/patch request with data being a FormData object it just sends empty body, I have tried to set the "Content-Type" to "multipart/form-data" but nothing changed.

same here

I've seen similar issue on 2018
https://github.com/flutterchina/dio/issues/88
Suggestion in this issue was to send map instead of form data
But I'd like my library to be able to send form data and attach multipart files as part of request
It's crucial
Please take a look
I can also see that there's actually logic both for form data and multipart files

I've seen similar issue on 2018
#88
Suggestion in this issue was to send map instead of form data
But I'd like my library to be able to send form data and attach multipart files as part of request
It's crucial
Please take a look
I can also see that there's actually logic both for form data and multipart files

I've seen that one..
As you said I'd like to send multipart files, BTW I ended up switching all the PUT mapping endpoints in my server to POST mapping😞

Having the same issue.

Seems to happen to me, unfortunally it is very hard to understand at first

I am having this issue with dio ^5.3.2
Why is this closed?

I am having this issue with dio ^5.3.2 Why is this closed?

im having this issue too
Future update1() async {
try {
var formData = dio.FormData.fromMap({
'fullName': fullNameController.text,
'file': await dio.MultipartFile.fromFile(pickedImagePath.value,
filename: pickedImagePath.value)
});
  final response = await userService.updateProfile(payload: formData);
  if (response.status == 'success') {
    logger.log("Success:: ${response.message!}");
  } else {
    logger.log("Success:: ${response.message!}");
  }
} catch (exception) {
  logger.log("failed:: ${exception.toString()}");
}

}
this is a PUT reuqest. but now it send the finstance of the file and not the file path as supposed string.

Please submit new issues with minimal reproducible examples and updated info. We're generally not handling issues before Feb 13, 2023.


FAQ, Documentation

The idea here is a log of the questions asked about the software and answers given to keep a log of where the UI/UX does not imply the answers. These need to be integrated into some documentation, manual, or main video series about the software. Any good documentation should address these points.

to unlock rail and manually adjust the laser select “automatically lock rail”?

You can also click the unlock button in navigator that looks like an unlocked lock. Or hit F12 and type "unlock". The automatically unlock just makes sure it tends to remain unlocked after jobs or jogging.
Since unlock is console command, you can also also bind it to a key. Though currently the binds aren't saving between sessions. But, "bind space unlock" in console or something like that in keymap would be nice if you need to do that.
But, really you don't need to unlock the rail, and move it around. That's more of a thing required in Whisperer. You can put the project at the position in your bed that you want, and run it from there. The main scene is your bed scene. So you can just locate it how and where you want it, then run it, and whereever your laser is, it will go to the start location and begin work.

the power of the laser is still controlled by the panel?

The power is controlled by the panel. That does the amount of current to the laser and must remain within bounds. The power here is PPI pixels per inch. If you put it at 500 it will be half as strong of a line as it would be at 1000. But it does this modulating the pulses in software. The modulation of the current to the laser is typically PWM and the modulation here is PPI.

do you always leave PPI power at 1000? Why should it be changed and when?

You would change the PPI if you wanted to engrave something lighter or use less power. Like if you wanted to engrave a piece of tissue paper, you literally could not engrave that at any speed fast enough to not burn it, you'd use the PPI.  Or if you want to do a very light outlining of something. Sometimes if you're rastering a solid black thing adjusting the PPI is an easy fix to it being too dark.

the speed settings should be the same as K40 Whisperer?

The speed settings are the same as Whisperer. In fact, it runs a slightly updated version of the code in Whisperer which I also wrote as I spent a month or so properly reverse engineering the speedcodes for the M2 Nano.

Does Meerk40t support the following svg... it is a dashed line... k40w treats it as a solid line... what I am looking to do is have my laser cut the dashed lines I would use to stitch leather.. otherwise its me having to cut the dashed lines.
dash lines .zip

No, MeerK40t doesn't process dashed lines. But, I specifically looked at dash arrays when I was coding the svg spec. But, I didn't think it was useful enough to include (given the effort it would take to implement), but more generalized dashed lines seemed like they would be useful. The loaded SVG files have the information but MeerK40t never does anything with it, though it has access to that information.
I did however make a PPI advanced setting that hasn't made it into production called Dot Length (it's in advanced, in operation properties and disabled). The core idea was to set a dot length for the PPI carry forward so you could basically make dashed lines yourself with that. If set very short, they would be let you avoid some potential stuttering. If made longer they could give a dashed effect.
I've been meaning to check whether this bunch of chained plotting generators could be turned into something more elegant. If it can be I'll look into, implementing a dashed line. Though I'm only a bit away from a version update. So if something elegant doesn't present itself, I likely wouldn't include it. Since I can't really risk getting too far into the weeds. But, there's some chance that there's a really elegant solution that'll let me implement dashed lines and restore grouped pulses being triggered on the fly. As well as these other settings, or adding other features to the pulse train without causing exponentially more complexity.

@celticht32 In inkscape, go Path->Stroke to Path Then Path->Break Apart -- Make sure they have a correct stroke as they'll be little closed shapes. But, that'll get you passed that road block and move you on to more work. It'll be a bit before I get any line dashing in MeerK40t. I'm too afraid of screwing up the next version to switch yet.

The reality of documentation is that you need to have a few volunteers to focus on this. For this you need a user-friendly and supportive community and actively to encourage users to contribute.


Bad update?

Has there been a refactoring and an update but the a new README.md was not added? For instance, all of the png's are now missing and show as broken links in the readme.  Also the examples in the readme currently don't seem to work.

Yes.
I was working on updated docs for v1.0, due to the huge amount of changes.
I took the step to remove old documentation on Jun 6th, intending to update it the next day. It is now 4 days later.
The current ETA is to have re-hauled docs out by the end of this weekend, on Jun 13th.
Apologies for the interim status. The old docs had to be purged at some point, and I did it too early.

Closed by #82


Link that has a port with no label range

Of the three main static topology files, there's a link that has one port that has no label_range:
https://github.com/atlanticwave-sdx/datamodel/blob/1a6404a653510341e41218bb2f663aa2f16a9749/src/sdx_datamodel/data/topologies/zaoxi.json#L16-L21
I think this is the only such one. Should this be treated like an error?  If so, I believe schema should be updated accordingly, and error handling should be updated accordingly.

This may be a (intentional or unintentional) mistake.(1) It may be interpreted as 'all vlans' are available'; or (2) an error. Otherwise, it would prevent your code find a vlan assignment along a path, right?

Yes, this makes VLAN assignment a little more tricky.  I am leaning towards treating this as an error, unless we have have a reason to interpret it otherwise.

Sure. Then please add a vlan range to this port so your code can work.
Later, in practice, let's see how the real topology from OXPs would show up.

After thinking about this a little more, I think not adding a label_range here makes sense, at least for now.  Because of this particular missing label_range, an error case was surfaced in pce, and by handling that we're able to make pce a little more robust (I hope!).  We can't assume perfectly well-formed topology data, at least until we have stricter validation in place.


Fix incorrect warning for panel defaultSize < minSize

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
I was getting a constraint warning when the panel was rendered as collapsed on page load. Then it makes sense that the defaultSize is smaller than the minSize. So no need to emit the warning when the defaultSize is the collapsedSize.
Here is the diff that solved my problem:
diff --git a/node_modules/react-resizable-panels/src/utils/validatePanelConstraints.ts b/node_modules/react-resizable-panels/src/utils/validatePanelConstraints.ts
index 58f258c..d78b08a 100644
--- a/node_modules/react-resizable-panels/src/utils/validatePanelConstraints.ts
+++ b/node_modules/react-resizable-panels/src/utils/validatePanelConstraints.ts
@@ -33,7 +33,7 @@ export function validatePanelConstraints({
     if (defaultSize != null) {
       if (defaultSize < 0) {
         warnings.push("default size should not be less than 0");
-      } else if (defaultSize < minSize) {
+      } else if (defaultSize < minSize && defaultSize != collapsedSize) {
         warnings.push("default size should not be less than min size");
       }

This issue body was partially generated by patch-package.

That sounds like a reasonable change.

Fixed in a887486. Will go out with 1.0.8

Fixed in b27bcfd. Will go out with 1.0.8.

This fix just went out with 1.0.8

This fix just went out with 1.0.8

❤ → ☕ givebrian.coffee


Charles not showing any useful information

I am using Charles 4.2.6 on Mac OS 10.13.5.  I do not see any "POST"s as shown in your screen shot, i only see "CONNECT" and there are no paths associated to any a1.tyuaus.com listings and therefore no query string to look at to get the uuid, localKey, etc.
Is there some place else to look for these values, or is there a setting in Charles I am missing?

Thank you.

You have to go to
Proxy -> SSL Proxy Settings
Then activate the checkbox and typ in . to the Location.


fix: remove admin container from docker-compose

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tickets
N/A

License
MIT

Doc PR
N/A

See https://github.com/api-platform/api-platform/pull/1567.

A branch feat/next has been created.


Copying css during build.

Fixes: https://github.com/MattMcFarland/react-darkroom/issues/9
The /src/css directory is now copied into /lib/css the way the fonts directory is.
Also adding yarn lock.

Thank you so much for this, I have had trouble keeping up on this repo, and your contribution is very helpful <3 thanks a ton


moment('W01 2020', '[W]WW YYYY').format('[W]WW YYYY');  resolves to W01 2019

Is this a bug that others can observe or am I misunderstanding the format here?

This gets parsed to the ISO week of year, which does not start in 2020.
moment('W01 2020', '[W]WW YYYY').format()
"2019-12-30T00:00:00-08:00"

Thank you so much @marwahaha your amazing! This one had me upside down for a while.


just wanted to say-- awesome work!  This is so cool.

@fiatjaf My team has a huge Trello board that takes a long time to load, so I've been looking for something like this for a long time.  I just stumbled across this repo after checking out http://webapps.stackexchange.com/questions/85141/quick-add-card-in-trello-from-anywhere-similar-to-todoist/88514#88514, and all I can say is wow!  trelew is everything I could have hoped for, plus more things.  Great work 👍

Thank you, those were part of my motivations for writing it too. I'm glad it's working for you, but at the sime sorry to deliver incomplete work, as you saw in https://github.com/websitesfortrello/trelew/issues/16.


build_sysext: Detect dev rebuild of release tag, fix build ID issue

build_sysext uses a base squashfs (basically a full snapshot of the Flatcar OS image) to build custom sysexts on top. Before building it ensures the base image actualy matches the OS version in the repository root.
The version string includes a BUILD_ID which might be auto-generated (by including common.sh) if it is not present in the version file - e.g. when the version is an official release (tag). This build ID auto-generation causes issues with the version check when image build and sysext build scripts run independently - each will generate its own build ID, and this will cause build_sysext's version check to fail.
build_sysext will now use the build id from the base squashfs when it is not set in the source tree's version.txt to work around that issue. This is a more general solution than 361eda220b368c3c3a959357c54db10d4c2f1d1a (which this patch reverts) as it directly addresses the issue in build_sysext instead of working around it in sysext_prod_builder.
How to use

Check out this branchgit clone https://github.com/flatcar/scripts.git
cd scripts
git checkout t-lo/fix-sysext-build-id-issue-in-image-to-vm

Build it./run_sdk_container -t
       ./build_packages
       ./build_image
       ./image_to_vm.sh --from ../build/images/amd64-usr/latest/ --format' 'qemu_uefi'

Testing done
Ran the above builds.
Cherry-pick / backport
To all active mainenance branches except LTS (which doesn't use sysext), i.e. flatcar-3975 flatcar-4012 flatcar-4054

Tests are green and PR is approved, but main has moved on. Will force rebase and merge since it's a small change and well understood.

Merged and cherry-picked for flatcar-3975, flatcar-4012, flatcar-4054.


systemd: restart service on failure

Restart=on-failure option will tell systemd to restart the service if it exited with non-zero exit code.
See Restart option in systemd manual. In particular, it recommends the following:

Setting this to on-failure is the recommended choice for long-running services, in order to increase reliability by attempting automatic recovery from errors.

The default value for StartLimitIntervalSec will limit service to be restarted not more than 5 times within 10 seconds.
There is also a possibility to set this parameter optionally, but I think it would be a good default behavior.

Pull request acceptance prerequisites:

[x] Signed the CLA (if not already signed)
[x] Rebased/up-to-date with base branch
[x] Tests pass
[ ] Updated CHANGELOG.md with patch notes (if necessary)
[ ] Updated CONTRIBUTORS (if attribution is requested)

The systemd service template in this module is intentionally the same as the upstream one that the Elasticsearch service ships with, because changing the Puppet one can introduce inconsistencies with behavior in the default setup. See some related discussion here on the elasticsearch repo and the related discussion in the Ansible module.
With that in mind, I think the best option would probably be to drop a file as a service override, like the following (disclaimer: not tested, may or may not be right):
file { "/etc/systemd/system/elasticsearch-${instance}.service/override.conf":
  ensure => file,
  content => "[Service]\nRestart=on-failure\n",
}

Thoughts?

Thank you for pointing to the relevant discussions. Yes, it seems like the same workaround is currently advised for Chef and Ansible, so we should probably do the same in Puppet. Considering this, closing the PR.


[docs] Update Getting Started Standalone

Motivation
Update the getting-started-standalone.md file.
Modifications

Update the module What your package contains: Per the latest Pulsar version, the instances folder is also the default directory in Pulsar.
Update the command pulsar-daemon start standalone to bin/pulsar-daemon start standalone according to the whole context(eg. bin/pulsar is used in this chapter), which can be more easily executed by users. If not, users will be promoted that pulsar-daemon: command not found.

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] doc-required

Hi @tuhaihe I've changed doc-required to doc since this PR contains doc changes. Instructions of how to use doc labels: https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-r-A7QdFDBwmZh6ytB4guwMoXHqc0/edit

@Anonymitaet Thanks for your changes.


App owner

In production, I have issues when running app when files are owned by root, and it is not secure
When app starts i make www-data user own the whole app directory recursively
Locally, I also need to edit files as "weeger"
I made "weeger" part of the www-data group
But ! I'm still unable to edit my file

Fixed


Update desktop download links from 3.4.1 to 3.5.0

+1 Made a grammar change unrelated to your changes.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 12:36:29 on 2023.06.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 3, errors out: 0
Original problem ID: 28479
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=28479

Problem has been resolved in 12m 0s at 12:48:29 on 2023.06.09
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 28479


cmake: don't build PTX and 3.5 arch if cuda arch detection fails

Description
Currently, if USE_CUDA=ON but architecture detection fails due to building on a machine without GPU, we will build for 3.0;3.5;5.0;5.2;6.0;6.1;7.0;7.0+PTX;7.5;7.5+PTX. Thereby we create too large object files, triggering https://github.com/apache/incubator-mxnet/issues/16852.
Thus, this PR introduces patched select_compute_arch.cmake, such that the fallback arches are 3.0;5.0;5.2;6.0;6.1;7.0;7.5, ie. dropping PTX and 3.5 so that #16852 is not triggered.
We don't need 3.5, because code compiled for 3.0 also runs on 3.5 hardware and 3.5 hardware is 7 years old. If someone really cares about performance on 3.5, they can still manually set MXNET_CUDA_ARCH=3.5.
Upstream bug https://gitlab.kitware.com/cmake/cmake/issues/20275
Changes

[X] cmake: don't build PTX and 3.5 arch if cuda arch detection fails

We don't need 3.5, because code compiled for 3.0 also runs on 3.5 hardware - I do not believe that is the case, those archs are not binary compatible. I would say that the better choice would be to remove SM3.0 (as SM3.5 is still accessible via p2 AWS instance). We do not need PTX for 7.0 if we include PTX for 7.5.

Not including PTX at all would make it impossible to run MXNet on future GPUs without recompilation.

We don't need 3.5, because code compiled for 3.0 also runs on 3.5 hardware - I do not believe that is the case, those archs are not binary compatible. I would say that the better choice would be to remove SM3.0 (as SM3.5 is still accessible via p2 AWS instance). We do not need PTX for 7.0 if we include PTX for 7.5.

Why is this wrong? Binary forward compatibility is explicitly described in Nvidia docs https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#binary-compatibility

Not including PTX at all would make it impossible to run MXNet on future GPUs without recompilation.

That's true, but given the amount of cuda code in MXNet, PTX JIT compilation does not provide a good user experience (program will hang very long?) and is thus not enabled in the Makefile builds as well AFAIK.

ping @ptrendx

Ping @ptrendx

Ok, I think you are right that 3.0 will in fact work for 3.5. That said, again, do we actually care about SM 3.0 (and 5.0 for that matter, as it was used only in a few GPUs)?

Ok, I think you are right that 3.0 will in fact work for 3.5. That said, again, do we actually care about SM 3.0 (and 5.0 for that matter, as it was used only in a few GPUs)?

I'm not sure if we have any data about the distribution of SMs of our users. Thus my main motivation here is to preserve backwards compatibility.
Also, this change does not affect our binary distribution (python wheels), but just the default for manual build-from-source builds. So I expect if performance is important to whomever is doing the build, they will figure out to set the right architecture. Thus choosing a conservative default would be fine?
Is there any data about how large the performance difference is between running 3.0 binary compared to 3.5 binary on a 3.5 architecture?

I see. There should not be much difference - the only difference in the generated instructions could be usage of ldg in SM_35 case, but I checked and it is used only in 2 places in MXNet.

@ptrendx let's merge this PR as is? Without it users can run into #16852 easily.


problem logging in

tried logging into client, getting this error
[Error] There was an error deserializing the object of type System.Collections.Generic.List`1[[POEApi.Model.JSONProxy.Character, POEApi.Model, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]. Encountered unexpected character '<'.

Are your sure your account name is correct?

Im using the same info i log into the path of exile site with, am i supposed to do something different since im a steam user?


feat: SDP-9629: log more info on deployment error

... by letting skaffold figure out and log the reason in case of a failed Helm chart deployment.

I also see all the install steps logging this message as a Warning:
time="2023-11-15T20:45:33Z" level=warning msg="Could not get docker context: starting command docker context inspect --format {{.Endpoints.docker.Host}}: exec: \"docker\": executable file not found in $PATH, falling back to the default docker host" subtask=-1 task=DevLoop

Is there anyway to not log this - I fear this may throw users off when they are trying to debug why a deployment failed.

I also see this message:
There is a new version (2.9.0) of Skaffold available.

Since this is our first version being released with skaffold, should we update to use the latest version?

@pinaki124 I addressed your concerns now:

To prevent skaffold from logging the docker-related warning, I added a fake docker script.
I made the Action use the new skaffold 2.9.0 version.
To support the full timeout input value format Helm supports, I added a duration2seconds bash script and corresponding bats tests.

Please re-review!

I am closing this one in favour of #3 which merges into the new dev branch instead of main to avoid breaking things for users referring to the main branch.

Reopening this again and closing #3 since I realized we cannot replace the helm-install@v1 usage with a pre-configured Action anymore within our organization and I assume nobody is using the Action from the main branch directly anyway but a pinned tag.
Thus, I am updating the image reference to point to the new public image version before merging this one...

@jpbriend pointed out that there is bats container image that could be used here. I am also applying that to the PR...


elements diplayed out of the screen are not clickable --> i.e., scroll bar is not moving in my application to click on the element which is present out of the window seen

Hi there!
Thanks for submitting an issue to Protractor.
To help us help you better, please do the following before submitting an issue:

Review the questions section of CONTRIBUTING.md.
Make sure you are not asking a usage or debugging question. If you are, use StackOverflow, Google Group discussion list, or Gitter to get help.
Provide a minimally reduced test case. This makes it much more likely that your bug will be fixed. Protractor has a test Angular application available at http://www.protractortest.org/testapp which you can use for the reproducible test case.
Fill in the information that corresponds to your type of issue below.
Delete this intro and any unrelated text :smile: (if you do not we'll assume you haven't read these instructions and automatically close the issue.)

Bug report

Node Version: ``
Protractor Version: ``3.3
Angular Version: ``
Browser(s): ``chromev55
Operating System and Version ``win7
Your protractor configuration file
A relevant example test
Output from running the test
Steps to reproduce the bug
The URL you are running your tests against (if relevant)

Feature Request

Reasons for adopting new feature
Is this a breaking change? (How will this affect existing functionality)

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


Improved config transform

Fix to the previous PR (https://github.com/Haacked/RouteMagic/pull/20), as discussed in the comments there.

:sparkles:


Add support for pypy3 hashbang

This was merged and released as py2deb 2.2. I'll quickly move on to #30 now :innocent:.


Update virtual-machine-network-throughput.md

add description for how each flow is distinguished.

@asudbring
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Address clippy warnings

Since we have yet to get this check passing, we have been slacking on keeping contributors accountable for their linting habits.
Get the clippy check passing.
Some warnings are generated by deprecated methods or types. @nopestack may be of more help with this, as they may be actively being worked on or require minimal effort to remove entirely.

@andrewvious

@eureka-cpu Going to split up this issue into two or three PR's so it can be easier reviewed.

[ ] Removal of unused variables and imports, as well as code quality fixes
[ ] Reassignment or Removal of deprecated methods
[ ] Remove or ignore unused fields, in any struct


CalculateGenotypePosteriors's globalPrior usage seems wrong

@ldgauthier commented on Wed Oct 21 2015
@vruano and I both looked at this and we can't figure out why this globalPrior (by default 0.001 = SNP heterzygosity) is being added to ALL alleles (alt and ref). If no other AC priors are used, this seems to have the potential to apply a high prior to skew the genotypes in favor of homozygotes.
I'll look at this while I'm doing the analysis for CGP in production recommendations.

@vdauwera commented on Tue May 17 2016
Is this still a thing?

@ldgauthier commented on Tue May 17 2016
It is still a thing in that is has been untouched and uninvestigated.  I might get some time this week.

@ldgauthier commented on Tue May 17 2016
Actually @davidbenjamin since you've been thinking a lot about genotypes and priors recently, do you think you'll have time to take a look?  Chris Hartl did the model, so I don't know all the details, but it shouldn't be too hard to untangle.

@davidbenjamin commented on Tue May 17 2016
@ldgauthier Sure, I can take a look.

@davidbenjamin commented on Tue May 17 2016
I agree with @ldgauthier and @vruano.  The same prior pseudocount is being added to all alleles, alt and ref, and this is incorrect.  Since the default pseudocount is 0.001 this is a symmetric sparse prior which, as @ldgauthier said, favors a small number of alleles and hence biases in favor of homozygous genotypes.
A more correct thing to do -- not necessarily the most correct, but not manifestly wrong -- would be to give 1000 prior pseudocounts to the ref allele and 1 prior pseudocount to each alt allele (for SNPS, at least).
A couple more things I noticed in this code:

PosteriorLikelihoodsUtils has a nomenclature problem.  Likelihoods, priors, and posteriors are getting conflated and combined in various amusing permutations, chief among them being that a posterior likelihood is an oxymoron.
It seems like there may be an overcounting problem in PosteriorLikelihoodsUtils::calculatePosteriorGLs.  If I follow correctly, lines 82-100 get the total counts for each allele in the resources and in the input, via the MLEAC or AC fields, and add the prior pseudocounts.  Lines 111-144 grab the genotype likelihoods from the input.  In line 149 we pass the allele counts and likelihoods to calculatePosteriorGLs.  This method uses the allele counts (prior + resources + input AC) to define a Dirichlet distribution which then serves as the prior on genotypes.  Finally, on line 203 we multiply (add in log space) this prior by the genotype likelihoods to get the posterior probabilities on genotypes.  It appears to me that we have double-counted the input data, once to get its AC field and once to get its GLs.  I believe the correct thing to do is use only the resources to define a prior which is then combined with the GLs to get the posterior.

@ldgauthier commented on Tue May 17 2016
I will take the blame (both figuratively and the literal git blame) for PosteriorLikelihoodsUtils nomenclature problems.  I either initiated them or didn't fix them when I refactored.
I also intuitively prefer resources only without using the input AC, but that being said we've seen better results using both, specifically for a Finnish cohort with 100 founders. In the DSDE/ATGU meetings the use of the input AC was discussed as being analogous to a single step of EM.  Would the true EM apply a different update for each sample in the callset?

@davidbenjamin commented on Tue May 17 2016
The better results using the double-counting might have something to do with the incorrect prior -- if the prior is skewing toward homozygosity, then double-counting your variant data might counteract this and rescue some variant genotypes, which will be mainly hets.
The EM model that people implicitly seem to have in mind is alternating E steps on each sample to get genotype posteriors with M steps to learn the allele frequencies.  So let's work out what happens if you do just one iterations:

Initialize allele frequencies to the mean of the Dirichlet heterozygosity prior; i.e. ~1 for ref, ~1/1000 for each alt, plus any allele counts from the resources.  Genotype priors come from the multinomial distribution (one genotype is a draw of 2 alleles) of these allele frequencies.
(E step) genotype posteriors are the product of genotype likelihoods with the priors from step 0).  Pseudocounts are the sum of expected posterior allele counts.
(M step) MLE allele frequencies are the mode of the Dirichlet parameterized by the sum of the original step 0) prior+resources pseudocounts with the E step pseudocounts from step 1).

Hmmm that does sound a lot like what the code is doing now.  I suppose it's defensible after all.

@ldgauthier commented on Thu May 19 2016
So it sounds to me like the action item here is to fix the Dirichlet heterozygosity prior.  I like the idea of adding one count for the ref and 1/1000 for each alt (rather than, for example, 1000 for ref and one for alt) so the heterozygosity prior does something in the absence of external resource counts, but doesn't overwhelms them if they are present.  @davidbenjamin Can you think of a more rigorous justification for the scaling of counts between sample genotype allele counts and the heterozygosity?

@vdauwera commented on Mon Nov 14 2016
Is this still alive? To be continued in GATK3 or 4?

@ldgauthier commented on Tue Nov 15 2016
You can move to GATK4.

@vdauwera @ldgauthier the new qual score model does exactly this.  IMO we should simply have this CLI use AlleleFrequencyCalculator.

Issue moved to broadinstitute/gatk #2918 via ZenHub


Cloud sync errors

Not sure if this ie new, old, me or something temporary, but these have been starting to show up in my dev tools, both live and via localhost:
POST https://zyh2ho4s6.dexie.cloud/sync 500

Failed to sync client changes HttpError: 500
    at index.0c75a52a.js:28:16686
    at Generator.next (<anonymous>)
    at s (index.0c75a52a.js:24:22083)
index.0c75a52a.js:32 error in syncIfPossible() HttpError: 500
    at index.0c75a52a.js:28:16686
    at Generator.next (<anonymous>)
    at s (index.0c75a52a.js:24:22083)

Could this be related to me having a list ID containing a dot (".")? (I invited the user<EMAIL_ADDRESS>which produced a list ending in .nu whereas email adresses ending in .org does not produce lists with dots in their ID:s)
Then (related?) I get substr errors like this:

Any findings here..? I've tried deleting my local db but still get these sync issues. Current (failing) sync payload is 46 objects in changes array, I guess this empties out once the sync is successful..?

I found where in the code the issue happens yesterday but have not found a way to reproduce it yet. If it would be possible to repeat it, please have devtools open when repeating it and right click the failing request on devtools / Network tab, and choose copy as cURL and send to me in a private email or slack message. I just sent you an invitation on Slack.

This issue should have been solved for a couple of weeks ago. If not please reopen.


Add manual restore docs for MariaDB

Checklist

[x] The PR has a meaningful title. It will be used to auto generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[x] Link this PR to related issues or PRs.

I'll adjust the versions once AppCat is tagged.


Copy code from google-cloud-cpp-spanner

... to google/cloud/spanner.
We should investigate how to preserve history as well when doing this copy.

Re: preserving history, see the notes in https://github.com/googleapis/google-cloud-cpp/pull/3728, which moves release/ and preserves history.

Remember to update the ci/ scripts as needed, e.g.:

ci/kokoro/docker/build-in-docker-cmake.sh has a list of expected directories that get installed, which will likely need to be updated.
ci/kokoro/docker/publish-refdocs.sh has the list of directories that produce documentation.
ci/etc/integration-tests-config.* list the environment variables necessary to run the integration tests and examples.
ci/etc/quickstart-config.* lists the directories with a quickstart example and now to invoke that example.

We also need to copy the API baseline from ci/test-api and update the check-abi.sh script (which needs to be renamed, but that is a separate issue).


No license

I'm thinking about using this for a little project, but I don't see a license for the code. Is it MIT licensed (or something else), and can you add a file with the license? Thanks.

I added the MIT license, in the file LICENSE.md, and also stated so in the readme file.
If this doesn't suit your needs please let me know, because the idea is to basically let anyone do anything with this... :)
I'll be happy if you let me know which project you're using this for ?
(But even if you don't or can't i'll understand)


Unignore gradle-wrapper.jar

Since gradle-wrapper.jar is in Git, it should be actively tracked as well.

shouldnt we commit the latest jar version as well?

https://github.com/projectnessie/gradle-build-plugins/tree/main/gradle/wrapper
wrapper jar is from 8.0.1 still (or it's unchanged?)
no big deal i guess


Spec File: allow direct altering of status

see #1994

@rovats

Buttons on the below component:

State machine as follows:


Can't access Dashboard or History with cancan permissions

I'm using cancan to restrict access (read and update) to certain records.
User has a role which corresponds to a provider field in the other models. List, show and edit pages are filtered perfectly based on this role. However only Admin user is able to access the dashboard and histories.  I have tried adding "can :history" to all users to no avail. (I also tried :see_history).
class Ability
include CanCan::Ability
def initialize(user)
  if user
    can :access, :rails_admin
    #can :read, :all
    #can :index, [Person, Parent, Eoi]
    can :history
  end

  if user.role == 'admin'
    can :manage, :all
  end

  if user.role == 'Durham'
    can :manage, Person, :provider => 'Durham'
    can :manage, Parent, :provider => 'Durham'
  end

I don't think this is implemented in rails_admin (I just took a quick glance at the code). That is a good idea though. I have a similar use case coming in one of my app.

can :history, :all ?

Thanks for suggestion.
I tried
can :history, :all
doesn't work either.  I have also tried (as there is also user info on the dashboard)
can :history, :all
can :manage, User
but still no access to dashboard or history.  If I add "can :manage, :all" it does work. But that removes the restrictive permissions of course.

can you add some failing specs to
spec/requests/authorization/cancan_spec.rb

I'd really like to tackle that.

@damiena Any update here? I'm seeing the same thing in my app after upgrading to Rails 3.1.3

@jmccartie   can :dashboard doesn't seem to work for me
@bbenezech sorry, I've not had a chance to come up with any failing specs, I've had too much other work on over the last few months.
I've simply routed past the dashboard for now but I will try to get back onto fixing it when I can.

@damiena
Here's my working ability file:
    if user.role?
      can :dashboard
      can :access, :rails_admin
    end

    case user.role.to_sym
      when :admin then can :manage, :all
      when :human_resource_team
        can :manage, [Job, Employee]
        can :read, [Location]
      when :message_team then can :manage, [Message, MessageGroup, TalkItOverItem, Format, Speaker]
      when :campus_team then can :manage, [Location, LocationTime, FacebookLink, Employee]
    end

here's mine...
'''ruby
class Ability
include CanCan::Ability
def initialize(user)
if user.role?
can :dashboard
can :access, :rails_admin
can :manage, User
end
if user.role == 'admin'
  can :manage, :all
  can :dashboard
end

'''

What about Gemfile?
Me:
gem "devise", "~> 1.4.7"
gem "cancan", "1.6.7"


close navigation drawer on sync menu selection

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1745
Description
Closed navigation drawer before returning from onNavigationItemSelected() mehod
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

Hi @PallaviGanorkar I have also added a commit to make sure "sync" isn't selected by returning false instead of true, because it shouldn't be aselected item, it's just a one-time option.


Adding options.html to build is missing

options.html page is required for adding some extension settings. How to include that while building project? @jessepollak

Have you tried adding an options.html file to the manifest.json file here?

Yes, but the part of adding that to build is missing. I had to manually copy that options.html to build folder to make it work. Probably need to include that html and options.js file in build.js . Any hint will also help.

I just added support for an options page and JS in this commit. If you merge the add-options-page branch into yours, you should be good to go!


jclulow/node-progbar#13 support an optional alternate output stream for <ProgressBar>.log()

This would fix #13.
Worked in my limited testing.

@jclulow ping?


Invalid MIC on Relay Feature

Hey!
I am testing the new branch (feature/relay) with the ChirpStack network server and I am getting some issues after the device is successfully connected to the NS through the Relay RX device.
After a TX DONE on RXR (Join Accept sent to the device), the Relay RX starts a CAD 1s.
The device, after a Join accept, starts an Alarm to send uplink in 60 s. When the device sends the WOR, the Relay tells me a
Invalid MIC and Unknown ED (0x017501A3).
What am I doing wrong? Is this behavior expected or could it be a possible bug?
Thanks!

Hi!
I found the problem, it was my setting on Chirpstack about ForwardListReq.

Hi @EduardoPfeifer, could you be kind enough to clarify more on the settings of Chirpstack with regards to the ForwardListReq? Im encountering the same issue of Invalid MIC and Unknown ED. As i do not own the Chirpstack server, i would need to raise it to the owners for the appropriate changes. Hope you can assist me on this. Thanks

Hi @Mathannbala ,
In Chirpstack, you need to link your end-device and your relay together
To do that,select you ED  in your application , click on "selected device" then "Add to relay"

I hope it help

@lbm-team Thank you so much for the prompt response, I've done so and my end device is able to connect via the relay. However, I noticed that after a certain amount of transmissions, I'm seeing " missed data" on the relay, not sure if this is expected behaviour


Random failed with distro

I have to launch one, two or three times the disto for it to work on my laptop.
I think it is the bug in WSL you have already create:
https://github.com/microsoft/WSL/issues/6917
As a workaround, instead of calling this:
wsl.exe -d wsl-vpnkit service wsl-vpnkit start
I remove background option of wsl-vpnkit.service and keep wsl.exe process in background like this:
wsl.exe -d wsl-vpnkit sed -i '/--background/d' /app/distro/wsl-vpnkit.service
wsl.exe -d wsl-vpnkit sed -i '/--stdout/d' /app/distro/wsl-vpnkit.service
wsl.exe -d wsl-vpnkit sed -i '/--stderr/d' /app/distro/wsl-vpnkit.service
nohup sh -c "wsl.exe -d wsl-vpnkit service wsl-vpnkit start" 2>&1 </dev/null &

Have you a better solution?

Do the logs have anything when you start the service and it doesn't work?
wsl.exe -d wsl-vpnkit cat /var/log/wsl-vpnkit.log

starting wsl-vpnkit
vpnkit.exe exists at /mnt/c/Users/username/wsl-vpnkit/wsl-vpnkit.exe
npiperelay.exe exists at /mnt/c/Users/username/wsl-vpnkit/npiperelay.exe
waiting for /var/run/wsl-vpnkit.sock ...
found /var/run/wsl-vpnkit.sock
waiting for eth1 ...
found eth1
ip: RTNETLINK answers: File exists
successfully created TAP device eth1
starting in connect mode with path=/var/run/wsl-vpnkit.sock and tap=eth1
2021/10/11 23:11:04 open //./pipe/wsl-vpnkit: The system cannot find the file specified.
EOF reading from socket: closing

Failed to read hello from client
Failed to negotiate vmnet connection
wsl-vpnkit.exe: [INFO] Setting handler to ignore all SIGPIPE signals
wsl-vpnkit.exe: [INFO] ...multiple logs here...
wsl-vpnkit.exe: [INFO] TCP/IP ready
wsl-vpnkit.exe: [INFO] TCP/IP stack connected

I would also like to see the pstree when it fails.
wsl.exe -d wsl-vpnkit pstree

init-+-init---wsl-vpnkit-+-wsl-vpnkit---socat---npiperelay.exe
     |                   |-wsl-vpnkit---wsl-vpnkit.exe
     |                   `-wsl-vpnkit---vpnkit-tap-vsoc---vpnkit-tap-vsoc---4*[{vpnkit-tap-vsoc}]

Also, if you want to run the script in the foreground for nohup you can just use the script directly without the service script.
wsl.exe -d wsl-vpnkit /app/wsl-vpnkit

In order to keep process running, I use daemonize tool and launch this in my distro:
daemonize $(which wsl.exe) -d wsl-vpnkit wsl-vpnkit

@philippe-granet do you still have this issue? Although if daemonize is working for you, that might be the best solution for now.


Fix network interpretation of Commuter Rail dataset

The input dataset has some labelling inconsistencies that result in the networking algorithm to fail and create the mess pictured above.

The Commuter Rail dataset is currently not used for any network interpretation, so this can be backlogged indefinitely until the network version of this dataset is needed.


Dataset

It would be great if you can share a link to Sketchy Dataset.

you can download it here, http://sketchy.eye.gatech.edu/

I requested access to the Google Drive where the dataset is stored. Please give me access to the shared folder.

I requested access to the Google Drive where the dataset is stored. Could you please give me the download permission? Thanks.

https://drive.google.com/file/d/0B7ISyeE8QtDdTjE1MG9Gcy1kSkE/view?usp=sharing&resourcekey=0-r6nB4crmdU-LK7H38xnOUw
try this link

what is the difference b/w tx_000000000000 and tx_000100000000

see README in dataset
tx_000000000000 : image is non-uniformly scaled to 256x256
tx_000100000000 : image bounding box scaled to 256x256 with
an additional +10% on each edge; note
that due to position within the image,
sometimes the object is not centered


Dashboard loaded from json passed in URL is misplaced

See upstreams Freeboard/freeboard/issues/139.

Fixed by stufisher@15b88ba. Closing.


lyxal: Watch realiqtestonline(?!.com)

lyxal requests the watch of the watch_keyword realiqtestonline(?!\.com). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

realiqtestonline(?!\.com) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Add CVE-2016-5385 for Artax

Code has been introduced in https://github.com/amphp/artax/commit/6f8b6c2b#diff-da1906ee835fb5c0e62dc9db19aff36fR27.
git tag --contains 6f8b6c2b lists the following tags to be affected contain the above commit:
v1.0.0
v1.0.0-alpha
v1.0.0-beta
v1.0.0-beta2
v1.0.0-rc1
v1.0.0-rc2
v1.0.0-rc3
v1.0.0-rc4
v1.0.0-rc5
v1.0.0-rc6
v1.0.1
v1.0.2
v1.0.3
v1.0.4
v2.0.0
v2.0.1
v2.0.2
v2.0.3
v2.0.4

I guess >=1 doesn't cover v1.0.0-alpha? What would be the right constraint here?

If you want to specifically include alpja version, I think you should use >=1.0.0-alpha

I changed it to >0.7.1 as 0.x is anyway unsupported and will not see any releases anymore.


Expanding row in grid does not work correct in Master branch

Expanding a row in a grid makes the grid look very strange. The code is:
onExpand: function (event) {
   $('#'+ event.box_id).css({ margin: '0px', padding: '0px', width: '100%' }).animate({ height: '105px' }, 100);
   setTimeout(function () {
      $('#'+ event.box_id).load('process_grid.php', {"recid":event.recid, "cmd":"get-record" });
      w2ui.grid_comp_users.resize();
   }, 300);
},

In the expanding area the left column is filled with "+", that belongs to the row below the expanded area.
This error does not exists in the 1.4.x. I downloaded a Master branch from 10:th of October, and the error did not exists in that branch version.
I also get an error message in the console when rendering the grid: ReferenceError: event is not defined. Looking in the code w2ui.js 1089:
function active (new_w2ui_name) {
        if (new_w2ui_name == null) return w2ui_name;
        var obj = w2ui[new_w2ui_name];
        if (obj && obj.focus) {
            if (obj.focus.call(obj, event) === false) return false; <<-- The error
        }
        w2ui_name = new_w2ui_name;
        return true;
    }

Sub-grids were rewritten. There were too many problems with current approach and now it is different in 1.5.x, See https://github.com/vitmalina/w2ui/blob/master/demos/examples/grid-7.html


[WIP] Contextual narrowing to object

Related: https://github.com/microsoft/TypeScript/issues/48468

@typescript-bot pack this
@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this

Seems like there's a slowdown from this - maybe a fluke?
@typescript-bot perf test faster

@typescript-bot pack this
@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test faster

RWC reveals a few interesting things:

Substituting unknown for the error messages is definitely a bit confusing.

This gets worse with unions - e.g.
let f = <T, U extends UsefulType>(x: T | U): object => x

So whereas this used to report that T | U wasn't assignable to object, it now reports unknown isn't assignable to object.

Going to rerun with latest changes.

Hmm, yeah, I don't much like the error message changes in the RWC baselines. The unknown makes it seem like the expression doesn't have a type at all.

Discussed with @ahejlsberg offline - this PR is a bit too corner-casey and produces worse messages when a type isn't narrowed. We're interested in seeing if we can leverage smarter intersection types with {} while narrowing generics, which is a change that will have to wait for 4.8 at the least. For now we'll have to live with the break, which people can get around by adding a constraint of {} | null | undefined.


[Bug] Some Prettier options don't apply

Describe the bug
I customized the default @antfu/eslint-config for my needs, then I also added this plugin for trying to format css in vue files.
My configuration:
import antfu from "@antfu/eslint-config";
import format from "eslint-plugin-format";

export default antfu(
	// General
	{
		"typescript": true,
		"vue": true,
		"unocss": true,
		"stylistic": {
			"indent": "tab",
			"quotes": "double"
		},
		"formatters": {
			"css": true // <-- this here
		},
		"rules": {
			// my custom rules
		}
	},

	// Style
	{
		"files": ["**/*.css"],
		"languageOptions": {
			"parser": format.parserPlain
		},
		"plugins": {
			format
		},
		"rules": {
			"style/indent": "off",
			"format/prettier": ["error", {
				"parser": "css", // also tried "scss"
				"useTabs": true,
				"vueIndentScriptAndStyle": true // <-- this doesn't apply
			}]
		}
	}
);

I'm trying to indent the Vue style block of 1 tab, so I have a consistent indentation for the template and the script tags.
From my understanding, the format/prettier rules should reflect what's available in Prettier, but only the first 2 rules actually shows warnings/errors in my application.
Tabbing my code to indent the style tag results in error: Delete ↹eslint(format/prettier)
Are only some rules supported?
Reproduction
nuxt-template
System Info
System:
  OS: macOS 14.2.1
  CPU: (8) arm64 Apple M2
  Memory: 64.59 MB / 24.00 GB
  Shell: 5.9 - /bin/zsh
Binaries:
  Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
  Yarn: 1.22.19 - /usr/local/bin/yarn
  npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
  pnpm: 8.14.1 - ~/.nvm/versions/node/v20.10.0/bin/pnpm
Browsers:
  Safari: 17.2.1
  Arc: 1.25.1

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Could you provide an reproduction?

@antfu updated the reproduction section, I thought the config snippet was enough.

I see, vueIndentScriptAndStyle is not effective that is because Prettier isn't formatting the Vue file but only CSS part (so it doesn't know it's dealing with a Vue file). This should be handed by https://github.com/antfu/eslint-processor-vue-blocks/issues/4, where I created an issue there. It would be best if someone could help implement that and send a PR, as this would be quite a low priority for me.
Close here and redirects to https://github.com/antfu/eslint-processor-vue-blocks/issues/4


[Bug] A mini RayService requires more than 90 seconds to converge

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
ray-operator
What happened + What you expected to happen
See #837 for more details.
test_sample_rayservice_yamls.py is very flaky (~20% to fail). Previously, it was very stable. https://github.com/ray-project/kuberay/pull/731 said that "I have run more than ten times on my cluster, and the result is always pass.".

[root cause]: RayServiceAddCREvent fails to converge due to the absence of rayservice-sample-serve-svc in 90 seconds. It is too slow.

Reproduction script
Run the following command several times.
RAY_IMAGE=rayproject/ray:2.1.0 OPERATOR_IMAGE=kuberay/operator:nightly python3 tests/test_sample_rayservice_yamls.py

Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Just recording the time it took when I ran this test on my laptop:
--- RayServiceAddCREvent<PHONE_NUMBER>8118286 seconds ---

It becomes stable after #1000. The RayService test passes 25 times consecutively.

RAY_IMAGE=rayproject/ray:2.4.0 OPERATOR_IMAGE=kuberay/operator:nightly python3 tests/test_sample_rayservice_yamls.py

The RayServiceAddCREvent converges in 32 seconds. I guessed the root cause of this issue is RAY_IMAGE. The current test framework will not replace the RAY_IMAGE automatically, so the Pod will pull the image from DockerHub instead if RAY_IMAGE is not the same as the value of image in the YAML file.


Can containerd be restarted and recover all running containers?

Wondering if this supported or not. If not, will that be on the roadmap.

yes. Every container has a containerd-shim to supervise the container, even if the containerd stop, the container can keep on running, and once containerd restart, it can recover all running contaners by restore the /run/docker/libcontainerd

ping @crosbymichael ; I'm fairly sure that capability would be preserved, but @coolljt0725 is correct that it is definitely in the code used by the Docker engine today in 1.11 and 1.12 (the v0.2.x branch), but not in master at the moment.

Yes, it will ;)
The only difference is that the shim will not be a hard requirement.   If you don't want it then you don't have to use it but it will be the default just as it is today.


BTFS-1025 add plugin for btfs nodes

new plugin for running btfs nodes called localbtfs.  note we need to configure the Addressees Announce setting to [] instead of the external ip address so the swarm command will work.  see updated README

@taiyangc i refactored the repo based on your comments by removing the non btfs plugins.


Segfault at TEcs.bmx:277

ECS is causing a segfault on new release of bmx-ng (v<IP_ADDRESS>).
https://github.com/fightlessbirds/rcge/blob/83c2a47346876c7759f92233b8090a5a0a18a549/src/rcge.mod/ecs.mod/TEcs.bmx#L277

Fixed in bmx-ng-v0.129


Same layout on other users.

https://uploads.disquscdn.com/images/fc4a5dd3a520efc32c5ebc1c32cc42467f3fb7b3cdd894b3bf6e038fe1e3327b.png
Hi Muaz Khan thank you for very helpful library.
Please check the link for the picture .
Now this is the screenshot of my screen. If we can see there a two videos on 1 page. Sir Is it possible that the layout of the video of one user is the same to others? Like on the picture this our 2 different browser joining on the same room and if we can see the layout of the video is different . If who is the LOCAL video of the room that is on the first (as always) . Now I repeat sir is it possible that the layout of the video is the same layout seeing by other users?
https://uploads.disquscdn.com/images/4d37f938fac54950ebbc3cf5c4580782747280b775f1acb5f1a0beebb9b08395.png
Example on the second image. This is the layout seeing by other users. The big layout on the top is the teacher and the small video on the bottom is teachers. And it displays the same layout in all users.
Please help me . BIG THANKS SIR!!!

HTML:

CSS: (Set Styles Yourself)
.videos-layout {}
.teacher-big-screen {}
.students-videos {}

.teacher-big-screen video {
    width: 100%;
}

.students-videos video {
    width: 360px;
    margin: 0 5px;
}

JavaScript:
connection.onstream = function(event) {
    if(event.stream.isScreen) {
        $('.teacher-big-screen').html('').append(event.mediaElement);
        return;
    }

    $('.students-videos').append(event.mediaElement);
};

Code for the Presenter ONLY
connection.session = {
    audio: true,
    screen: true,
    // broadcast: true : Enable this line ONLY if you want ONE_to_Many broadcast
};
connection.dontCaptureUserMedia = true;

// manually capture screen and microphone
getScreenConstraints(function(error, screen_constraints) {
    navigator.mediaDevices.getUserMedia({
        video: screen_constraints
    }).then(function(screen) {
        var video = document.querySelector('video');
        video.muted = true;
        video.srcObject = screen;

        navigator.mediaDevices.getUserMedia({
            audio: true
        }).then(function(mic) {
            var finalStream = new MediaStream();
            screen.getVideoTracks().concat(mic.getAudioTracks()).forEach(function(track) {
                finalStream.addTrack(track);
            });
            finalStream.isScreen = true;
            connection.attachStreams.push(finalStream);

            // video.srcObject = finalStream;

            var event = {
                type: 'local',
                stream: finalStream,
                mediaElement: video
            };
            connection.onstream(event);
        });
    });
});

Code for all participants
connection.session = {
    audio: true,
    video: true,
    // broadcast: true : Enable this line ONLY if you want ONE_to_Many broadcast
};

@muaz-khan The code is about screen sharing i don't need screen sharing sir. What I need sir is same videos layout in all users. I think you miss interpret my concern.

PLEASE HELP ME .
This is my code for index.html

     0

        <input type="text" id="room-id" placeholder="room-id">
        <button id="open-room">Open</button>

Undo

x
<select id="undo-options" size="4">
    <option>Last Shape</option>
    <option>All Shapes</option>
    <option>Last Multiple</option>
    <option disabled>Specific Range</option>
</select>

    Digit/Number:

    <input type="text" id="number-of-shapes-to-undo" style="padding: 0;width: 90px;text-align: center;border-radius: 0;margin-top: 6px;">

<button id="btn-undo" style="float:right;">Undo</button>

x
<select id="data-url-format" size="4">
    <option>image/png</option>
    <option>image/jpeg</option>
    <option>image/gif</option>
    <option>image/webp</option>
</select>

<button id="btn-getDataURL" style="float:right;">Get DataURL</button>

x

This is my code for the CSS

THIS IS THE OUTPUT WHEN I RUN .

 AND THE USER 1 OR THE ROOM CREATOR IS THE BIGGER SCREEN COMPARE ON OTHER PARTICIPANTS .
AND THIS IS THE LAYOUT I WANT TO ACHIEVE .

Just added a demo: demos/dashboard.html
Online URL:

https://rtcmulticonnection.herokuapp.com/demos/dashboard.html

Wow thank you so much muaz-khan.
1 thing the name in the chat is not friendly name it is a random scrambled
letters. How can I add a enter a name first and that name will appear to
chat .
Thank you so much!!!
On Sat, Sep 15, 2018 at 5:54 PM, Muaz Khan<EMAIL_ADDRESS>wrote:

Just added a demo: demos/dashboard.html
https://github.com/muaz-khan/RTCMultiConnection/blob/master/demos/dashboard.html
Online URL:

https://rtcmulticonnection.herokuapp.com/demos/dashboard.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421547606,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnHiaDgjLZ_bAdqCwqqizw0ez9A8yks5ubM5MgaJpZM4WmnS3
.

I know that is a connection.extra API event i follow the codes on your
youtube video entatiled add extra. But I think your code there is
deprecated because it do nothing when I add the codes on my codes.
Please help thank you for being a kind person.
On Sat, Sep 15, 2018 at 11:46 PM, Benedict Aquino<EMAIL_ADDRESS>wrote:

Wow thank you so much muaz-khan.
1 thing the name in the chat is not friendly name it is a random scrambled
letters. How can I add a enter a name first and that name will appear to
chat .
Thank you so much!!!
On Sat, Sep 15, 2018 at 5:54 PM, Muaz Khan<EMAIL_ADDRESS>wrote:

Just added a demo: demos/dashboard.html
https://github.com/muaz-khan/RTCMultiConnection/blob/master/demos/dashboard.html
Online URL:

https://rtcmulticonnection.herokuapp.com/demos/dashboard.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421547606,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnHiaDgjLZ_bAdqCwqqizw0ez9A8yks5ubM5MgaJpZM4WmnS3
.

Last thing sir please help me this is the major part I never did.
How to add a pdf to canvas and sync to all users? And if one user switch
the pdf it will sync to all users .
How it that possible Mr . muaz-khan
On Sun, Sep 16, 2018 at 12:20 AM, Muaz Khan<EMAIL_ADDRESS>wrote:

Now users can enter their full names.
Please check the demo again:

https://rtcmulticonnection.herokuapp.com/demos/dashboard.html

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421594024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnDTDeqDB66s1NK0PNyILp28J0YqBks5ubSi7gaJpZM4WmnS3
.

PDF syncing is possible. I'll add this feature as well (in the same demo).

WOW !!! Thank you so very much mr muaz-khan!!!
I can't explain how happy I A'm because I'm doing that feature for almost 2
weeks and nothing happens even tho read many articles many blogs.
I'm going to wait for this now sir.
Again. THANK YOU SO MUCH!!
On Sun, Sep 16, 2018 at 12:38 AM, Muaz Khan<EMAIL_ADDRESS>wrote:

PDF syncing is possible. I'll add this feature as well (in the same demo).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/muaz-khan/RTCMultiConnection/issues/628#issuecomment-421596699,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnlPnN5ivse3-8rSIs5Ghxkig3nRycP1ks5ubS0VgaJpZM4WmnS3
.


drive navigation feature in FileChooser widget under windows

This is in of concern with the windows operation system.
Currently the FileChooser widget allows navigation through directories which are present only in the drive to which the path property belongs to.
Though the drive can be explicitly changed through modification of the path property, it cannot be changed through the user action once the widget gets fired up.

My suggestion: Display the drive list once the user clicks ..\ (one up directory) from the root directory of the drive, and later change the path to the particular drive which user selected.

Is this something being worked on?

I'm pretty sure it's not being worked and there are no current plan to change this.
If you think you can provide a solution and make a case for it, feel free to discuss it and create a PR, but did you look at the linked garden project to see if it fit your needs?


[prim_flop_2sync] Make the prim a standard non-generated prim

This is done to ease CDC instrumentation for DV simulations.
See #10432 for context.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

wait sorry, i thought the original idea was to keep it a wrapped prim but have an ifdef option underneath, since it's likely in the future we will have technology specific sync cells. did i remember wrong? Or is the idea there will be another level of technology specific prim cell beneath this one?

wait sorry, i thought the original idea was to keep it a wrapped prim but have an ifdef option underneath, since it's likely in the future we will have technology specific sync cells. did i remember wrong? Or is the idea there will be another level of technology specific prim cell beneath this one?

ah yes now i remember.  looks good to me.


Time filtering and bool parameters

New TimeFilter class and time parameter in SurfaceCollection.filter method.
Get time info:
case.surfaces.timestamps

case.surfaces.intervals

Construct time filter:
from fmu.sumo.explorer import TimeFilter, TimeType

# get surfaces with timestamps
time = TimeFilter(type=TimeType.TIMESTAMP)

# get surfaces with timestamps in range
time = TimeFilter(type=TimeType.TIMESTAMP, start="2018-01-01", end="2022-01-01")

# get surfaces with intervals
time = TimeFilter(type=TimeType.INTERVAL)

# get surfaces with intervals in range
time = TimeFilter(type=TimeType.INTERVAL, start="2018-01-01", end="2022-01-01")

# get surfaces with exact matching on t0 and t1
time = TimeFilter(type=TimeType.INTERVAL, start="2018-01-01", end="2022-01-01", exact=True)

# get surfaces where interval overlaps {start} and {end}
time = TimeFilter(type=TimeType.INTERVAL, start="2018-01-01", end="2022-01-01", overlap=True)

# get surfaces with any type of time data
time = TimeFilter(type=TimeType.ALL)

# get surfaces without time data
time = TimeFilter(type=TimeType.NONE)

Apply time filter:
surfs = case.realization.surfaces.filter(time=time)

Is the example notebook up to date and working?


Can we declare version 1.0?

Is there anything more we need to do until we can call this version 1.0? We can definitely hold off on any additions until future versions. Are we happy with the interface and naming of the codecs and functions? Those are the things we cannot change later without breaking source compatibility.

I say go!

I agree, the last encoded value fixes did it for me.

Let's do it.

I ran into a bug when porting some old code to this API, using it as more of a graph based API. Turns out that it is not possible right now to put an encoded_value in a std::vector and encode it using the array_t codec, because there is a static assertion in the codec that requires that the vector element type (encoded_value) is the same as the inner codec's object type (encoded_value_ref). I'm assuming that this is also the case for maps. We should be able to solve this with a different static assertion, since encoded_value_ref can implicitly convert to encoded_value. It should not lead to any changes in the public APIs, but we should fix it before declaring 1.0.0, because it makes it impossible to use for some simple use cases.

@punchfox Instead of using std::is_same, do you think we could use two std::is_convertible assertions (one in each direction) instead?

Another question: What should we do to make it 1.0.0? Add a git tag? Is that enough?
Also, after tagging it, should we announce it? Where? Add it to json.org? hacker news?

@punchfox I'm working on a fix for the is_same issue

@peck Great! Was gonna do it, but I think I will have lunch instead then. :smile:
A tag should be enough. Any fixes can be new patch versions. Will we support 1.x and 2.x at the same time in the future?

@punchfox I think supporting two major versions in parallel will depend on the reason for the break and how widely adopted the library is. I would take that decision later

I just published version 1.0.0.


[CI/Build] fix Dockerfile.cpu on podman

podman does not like RUN --mount with relative paths when WORKDIR is not set.
fixes #8502

I confirm that this fix the problem of building the Dockerfile.cpu on podman.


hashids.php versioning doesn't match versioning on hashids

If I look at the compatibility table at http://hashids.org/php/, it suggests that all implementations stick to the same versioning and should be at ~1.0.0.
hashids.php has already bumped the major release twice and thus is now at ~3.0.0, is this an issue?

...is this an issue?
We've discussed this internally and decided to bump the PHP package because of breaking changes. There was no way around it if we wanted to benefit from the new features.


Bug:

Current behavior
set_font_name("B Titr") function changes the font name but not it's actual font in excel.

Expected behavior

Sample code to reproduce
let second_merge_format = Format::new()
        .set_font_name("B Titr")
        .set_font_size(28)
        .set_bold()
        .set_align(FormatAlign::VerticalCenter)
        .set_align(FormatAlign::CenterAcross);

### Environment

```text
- "0.80.0"
- rust_xlsxwriter = "0.80.0"
- 1.83.0
- Excel version: 2021
- OS: Windows 10

Any other information
No response

Some fonts require additional information such as the character set and font family.
Could you upload a simple file created in Excel with some text in one cell only with the "B Titr" font.

use rust_xlsxwriter::{Format, FormatAlign, FormatBorder, Workbook};

fn main() {

    let mut workbook = Workbook::new();

    let worksheet = workbook.add_worksheet();

    worksheet.set_column_width(0, 50).expect("TODO: panic message");
    worksheet.set_row_height(0, 100).expect("TODO: panic message");

    let format = Format::new()
        .set_border(FormatBorder::Thick)
        .set_font_size(72)
        .set_bold()
        .set_font_name("B Titr")
        .set_align(FormatAlign::VerticalCenter)
        .set_align(FormatAlign::CenterAcross);

    worksheet.write_with_format(0, 0, "سلام", &format).expect("TODO: panic message");

    workbook.save("file.xlsx").expect("TODO: panic message");

}

@mehranj73 I'm looking for an actual xlsx file, created in Excel, with some text in the "B Titr" font in cell A1.

test_file.xlsx
Is this acceptable?

Thanks. That is just what I needed.
The sample file also includes a character set for the font and and it doesn't have the default font family. I set the following font properties to match:
use rust_xlsxwriter::{Format, FormatAlign, FormatBorder, Workbook, XlsxError};

fn main() -> Result<(), XlsxError> {

    let mut workbook = Workbook::new();

    let worksheet = workbook.add_worksheet();

    worksheet.set_column_width(0, 60)?;
    worksheet.set_row_height(0, 100)?;

    let format = Format::new()
        .set_border(FormatBorder::Thick)
        .set_font_size(72)
        .set_bold()
        .set_font_name("B Titr")
        .set_font_charset(178)
        .set_font_family(0)
        .set_align(FormatAlign::VerticalCenter)
        .set_align(FormatAlign::CenterAcross);

    worksheet.write_with_format(0, 0, "سلام 100000", &format)?;

    workbook.save("gh125.xlsx")?;

    Ok(())
}

Which gives this output:

Does that look correct?

Wow thanks. How did you figure out which family and character set should you use?

How did you figure out which family and character set should you use?

It was in the xl/styles.xml subfile in the xlsx file:
    <font>
      <sz val="72"/>
      <color theme="1"/>
      <name val="B Titr"/>
      <charset val="178"/>
    </font>

I've only ever had to do that once or twice before and as far as I remember it was for Arabic fonts as well.
Before we close this could you do a bit of testing and see if it is the character set or the font family or both that has the required effect on the output.

It works great. Thanks again.


Use the player inside a real game

Hello,
I am starting porting one of my old Amiga games (Quadranoid) on Godot (after  25 years!) and I'd like to use your mod player to play original PRO tracker files.
Is it possible to use your software as a standalone library to play modules?
Thanks! :-)

Hello!  Thank you for your interest in my project.  When writing it, I tried to separate the interface from the implementation the best i could, in the hopes it wouldn't be too difficult to adapt it to a game project if necessary.  However, the code hasn't been tested outside of the scene it belongs to, so I couldn't say for sure!
If you check the code you can see most of the classes are in the main file.  All of the interface functions are at the top, and the replayer code is below that. Feel free to make your own audio stream generator in another scene and try using the replayer to fill the buffer with data from a mod you've loaded into the replayer.  I'll be happy to know the results!
P.S.  One more thing to note;  the file access routines are not guaranteed to work on every platform in the same way, so to be safe you should load a path from the resource folder, and remember that mods aren't included in the pack file by default, so you'll have to include them when exporting to a real project.
On my puny old i3, gdsmod takes up around 6% cpu to render a mod and fill the sound buffer at runtime.  Please set your expectations accordingly. 🙏  Thank you!

Ah, and moreover, whether you're able to successfully play a mod from outside the scene or not, please let me know.  I may be able to try moving the replayer to its own separate singleton myself if it needs more work to get working in your game than a simple copy-and-paste job to an autoload.

Hello Nobuyuki,
thank you for your kind reply :-)
I am just starting now writing my game and as soon as I'll arrive to the
tracker I'll write to you.
Thanks!
Fabio
On Tue, Nov 5, 2019 at 3:35 PM Nobuyuki<EMAIL_ADDRESS>wrote:

Hello! Thank you for your interest in my project. When writing it, I tried
to separate the interface from the implementation the best i could, in the
hopes it wouldn't be too difficult to adapt it to a game project if
necessary. However, the code hasn't been tested outside of the scene it
belongs to, so I couldn't say for sure!
If you check the code you can see most of the classes are in the main
file. All of the interface functions are at the top, and the replayer code
is below that. Feel free to make your own audio stream generator in another
scene and try using the replayer to fill the buffer with data from a mod
you've loaded into the replayer. I'll be happy to know the results!
P.S. One more thing to note; the file access routines are not guaranteed
to work on every platform in the same way, so to be safe you should load a
path from the resource folder, and remember that mods aren't included in
the pack file by default, so you'll have to include them when exporting to
a real project.
On my puny old i3, gdsmod takes up around 6% cpu to render a mod and fill
the sound buffer at runtime. Please set your expectations accordingly. 🙏
Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nobuyukinyuu/gdsmod/issues/8?email_source=notifications&email_token=AAVWNV3Z62V3MKBTI4RXKY3QSGAE3A5CNFSM4JI7CQE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDC5JPA#issuecomment-549835964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVWNV4LVBYRUOESNR2YJDLQSGAE3ANCNFSM4JI7CQEQ
.

--
Ciao,
Fabio

I would also like to use it in a game.  I really like the idea of using mods instead of big ogg files, and it seems to perform pretty well. Maybe a plugin that provides a new Node-type with an import-plugin (like this would make it easier to use?
I tried digging in, but couldn't quite figure out how to pull the player out of the demo scene. It looks like load_module in ModReplayer.gd will load it, and inject it into an Audiobuffer. Is this right? Maybe I could extend an audio-buffer to a new type and add an import-type in a plugin.

Also, as a side-note, I think it will allow mod-files in HTML5 exports, which is great (vs native addons, which only work on specific native targets that they've been compiled for, and cause lots of other issues.)

I made some progress using it as a plugin that should define a new AudioStreamSample so it can be attached to a AudioStreamPlayer.
Still can't wire it up to a AudioStreamPlayer, though:
 editor/import/editor_import_plugin.cpp:91 - Condition "!(get_script_instance() && get_script_instance()->has_method("get_import_options"))" is true.
 editor/import/editor_import_plugin.cpp:61 - Condition "!(get_script_instance() && get_script_instance()->has_method("get_preset_count"))" is true. Returned: 0
 Cannot erase nonexistent section "params".
 editor/import/editor_import_plugin.cpp:91 - Condition "!(get_script_instance() && get_script_instance()->has_method("get_import_options"))" is true.
 core/class_db.cpp:540 - Condition "!ti->creation_func" is true. Returned: __null
 Class type: 'AudioStreamGeneratorPlayback' is not instantiable.
 res://addons/gsmod/import_plugin.gd:22 - Invalid call. Nonexistent function 'get_frames_available' in base 'Nil'.
 core/os/file_access.cpp:667 - Condition "!f" is true. Continuing.
 Cannot open file 'res://.import/mdemo4_3.mod-718abadf4df1f2ebca0f0ecf2fe47da5.res'

Maybe someone with more godot-knowledge can see what I'm missing?
gsmod_demo.zip

I decided to go another direction: make a minimal self-contained scene, so I could just drop it into my game.
gsmod_minimal.zip
I'll keep working on a plugin (maybe just a new type that is an instance of the scene, that lets you set the file) but I figured this could be useful to others as-is.


Add a Guide folder for Beginners in CP

The Guide to Beginners in CP and DSA will contain-
1- The Link to the Coding platform for CP like codeforces,  hackerrank, codechef
2- Books for reference for Competitive Programming (CP)
3- The Roadmap of Sequence of topics to learn to master DSA

@aaryahjolia  @AnushkaGupta2003  Can you assign this issue to me


#334 - Fix bug related with incomprehensible error messages related with failures on setup

Thanks @vladyslav-fenchak!


Use string_tree module in favor of deprecated string_builder module of Gleam

Since release 1.6.0 of Gleam, string_builder has been deprecated in favor of string_tree.
This PR:

Updates version of dependency (gleam_stdlib) from 0.41.0 to 0.43.0

Or is this something that you'd prefer doing in a separate PR?

Adds conversion mechanism from Element to StringTree

Not sure how long string_builder will be kept around for. Assuming that string_builder won't be kept for too long I opted for duplicating some parts instead of generalizing them. Should there be any changes required, it isn't that much of an effort to keep both in sync.

Refactors VDOM to String conversions to make use of StringTree instead of StringBuilder internally
Deprecates functionality using StringBuilder

Making it easy to find all affected functionality that might be removed in the future.
Those @deprecated(...) might need some line breaks. Running gleam format didn't help. Might make sense to add an .editorconfig file to this repository?

Going through the individual commits should make it easier for you to review all changes in digestible chunks.
Are there tests or anything else that I missed?

Oops I'm sorry, I missed this PR when I was pushing out an update this morning! Thank you for the effort anyway, it's appreciated 💕

Another time!


Fixes large text layout

What does the pull request do?
It changes the TextFormatterImpl to retain already shaped runs to improve the general performance of large text layouts.

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Avaloniaui.net with user documentation

Breaking changes

Fixed issues
#4434

There is still room for improvement. Without this change, it took minutes to create the large text layout now its a matter of milliseconds

@Gillibald is this ready? so excited :D

@Deadpikle Could you verify that build 9736 works for you?

@Gillibald @jmacato Changes work great!
Before (9703):

After (9736):


Send Function: typeMismatch(Swift.Array, Swift.DecodingError.Context(codingPath: [], debugDescription: "Expected to decode Array but found a dictionary instead.", underlyingError: nil))

I am getting the above error when trying to make a transaction. I set up infura Ropsten network by:
let endPoint = "https://ropsten.infura.io/v3/my_id"
let web3 = web3swift.web3(provider: Web3HttpProvider(URL(string: endPoint)!)!)
And here is my code where I am using the send function:

Please help if possible, been stuck with this error for days

same issue

Hello, been having same issue. Finally figured it out. So when you initialze a new web3Wallet you need to add a KeystoreManager to your Web3 instance. Here is an example:
      let password = "web3swift"         let formattedKey = privateKey.trimmingCharacters(in: .whitespacesAndNewlines)         let dataKey = Data.fromHex(formattedKey)!         let keystore = try EthereumKeystoreV3(privateKey: dataKey, password: password)!         let keyData = try JSONEncoder().encode(keystore.keystoreParams)         let address = keystore.addresses!.first!.address         let keystoreManager = KeystoreManager([keystore])         web3.addKeystoreManager(keystoreManager)         return Web3Wallet(address: address, data: keyData, name: name, type: .normal)


docs: add DD-478 Bitcoin > ICP for Bitcoin page

Thank you for your contribution to the IC Developer Portal. This repo contains the content for https://internetcomputer.org and the ICP Developer Documentation, https://internetcomputer.org/docs/.
If you are submitting a Pull Request for adding or changing content on the ICP Developer Documentation, please make sure that your contribution meets the following requirements:

[X] Follows the developer docs style guide.
[X] Follows the best practices and guidelines.
[X] New documentation pages include document tags.
[X] New documentation pages include SEO keywords.
[X] New documents are in the .mdx file format to support the previous two components.
[X] New documents are registered in /sidebars.js, otherwise, it will not appear in the
side navigation bar.
[ ] Make sure that the .github/CODEOWNERS file is
filled with new documents that you added. This way we can ensure that future Pull Requests are reviewed by the right people.

Thank you for the review, @jessiemongeon1! I will not be able to take a look until Friday


COMPRESS=DEFLATE

How to handle DEFLATE compressed data streams, and are there any plans for related support?

Do you mean specifically for things like attachments? I don't think DEFLATE can be used on the entire connection? For attachments and the like, the imap crate just gives you the raw multiparts, so decoding their contents is up to you.


One-shot rpc invocation option instead of a persisted ws session

Is your feature request related to a problem? Please describe.
Websocket sessions could reach thousands when thousands of users have the client app open.
Yet, the only option to reduce that is to establish a ws connection in each invocation or establish
a ws connection that lives with the lifecycle of the app.

Don't get me wrong, I want long lived ws connection for communication between services but
not for communication between user client applications and a service.

Describe the solution you'd like
I don't exactly know if websocket sessions are very light or not for thousands of simultaneous long-lived
connections  to be a problem but if so, there should be a way around that.
Maybe supporting different kinds of RemoteServices were one has instance-per-connection policy and
another is invocation-per-connection policy?
Describe alternatives you've considered
If kotlin-rpc was aimed to service-to-service communication, I think a gRPC kotlin wrapper would be a better choice.
On the other hand (if kotlin-rpc  was aimed to client-to-service communication), I think the only alternative would
be Ktor resources (yuk!)
Additional context
I am creating a KMP project were the client app and the server app are both in Kotlin and I am searching for an
elegant more "natural" way of communication between the two.

Hi, @lsafer-meemer! Thank you for the feature request. We are thinking about protocol enhancements, and we'll take this into consideration during the design

Or maybe keep the websocket connection open as long as one of the registered services are invoked. After an idle period of  ( for example ) 5 seconds close the websocket and create a new websocket when there is another invocation of a registered service?
But to the be brutally honest, I have no idea how resource intensive it is to keep thousands of ( maybe mostly idle ) simultaneous websocket connections alive. Ktor is kinda great, so maybe they figured something smart out :)

I haven't found a way to detect when the underlying websocket session has closed. For example trying to add webSocketSession.outgoing.invokeOnClose {} throws an IllegalStateException saying that there already is another handler registered.


kill all the jobs when force-complete, aborted

fixes #8252

@amaltaro, This is not most efficient solution but seems to be simplest. Still there is chance for race condition for closed-out, announced -archived. But it should be very unlikely.
The extra overhead would be ~1 sec per request when request is completed with all killed job. I am not sure whether that is two much or not current production reqmgr contains ~200 completed requests. If this is too much overhead, we can create the new query just checking all the requests are completed by just checking the subscription for available and acquired subscription.

test this please

test this please

@alexanderrichards, Alan could review again. I ran the test in vm and added unittest.

test this please

test this please

test this please

test please

test this please

test this please

test this please


[ENH] Add metrics module for comparing networks-like graphs

Please review this PR first before #30. cc @robertness
Changes proposed in this pull request:

Implements confusion_matrix_networks function for comparing networkx-like graphs for evaluating causal discovery algorithms
Update poetry files

Is part of #30 for the sake of lowering LOC diff.
Before submitting

[ ] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[ ] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[ ] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

After submitting

[ ] All GitHub Actions jobs for my pull request have passed.

Codecov Report

Merging #48 (bf43cff) into main (c31f0a8) will decrease coverage by 1.49%.
The diff coverage is 25.00%.

:exclamation: Current head bf43cff differs from pull request most recent head d37735d. Consider uploading reports for the commit d37735d to get more accurate results

@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
- Coverage   67.17%   65.68%   -1.50%
==========================================
  Files          12       13       +1
  Lines         655      679      +24
  Branches      125      126       +1
==========================================
+ Hits          440      446       +6
- Misses        174      192      +18
  Partials       41       41

Impacted Files
Coverage Δ

dodiscover/metrics.py
25.00% <25.00%> (ø)

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

I have updated the docstring to elaborate that this function is solely used to compare "adjacency structure" of graphs

See comments. Also, should there be a unit test?

Good catch. I added a unit-test, which caught some subtleties I think should be added.


GPII-3391: New dataloader for aws

This brings new dataloader to AWS infra as well. I've verified state of AWS database and it's consistent with GCP (at least the entries relevant for this change). Steps to deploy are same as the ones for gcp part (see https://github.com/gpii-ops/gpii-infra/pull/163 for details).
This PR also removes docs about dataloader, as there's nothing special about the dataloader anymore, it now lives as part of universal and not an individual project. Dataloader itself is documented at https://github.com/GPII/universal/blob/master/documentation/DataLoader.md and does not require any special treatment from the infra point of view.

[ ] Do DB cleanup (described in #163 ) on stg environment
[ ] Merge this PR
[ ] Do DB cleanup (described in #163 ) on prd environment
[ ] Promote CICD pipeline to prd

LGTM

LGTM


Add List type

Decide between Scott encoding and Plutus builtin primitives (ChooseList, HeadList, ..). Compare CPU costs, code size, etc.
Goal: we need list type for the signatories in #5

@L-as added a List type using UPLC builtins here: https://github.com/Plutonomicon/plutarch/commit/24acc6bbf83d3b94abc9dcd4913ccc44edb2dfc4

As I've noted elsewhere, we need a Scott-encoded lists so that we can have a list of arbitrary data structures using Scott encodings.

A lot has changed since this issue was opened, so closing.


ReferenceError: e is not defined error occurs when running next dev --turbo

Describe the bug
When using next dev --turbo with a Next.js application, the following error message appears:
ReferenceError: e is not defined
    at Module.Kind (/path/to/.next/server/chunks/ssr/node_modules_xyz.js:1376:18)
    at Kind (turbopack://[project]/node_modules/@urql/core/src/gql.ts:84:30)
    ...

The application works without issues when running the next dev command.
The code is the same as in the following link when the error occurs:
https://github.com/urql-graphql/urql/blob/main/examples/with-next/app/page.tsx
Reproduction
https://github.com/urql-graphql/urql/tree/main/examples/with-next
Urql version

@urql/next: 1.1.2
urql: 4.2.0
next: 14.2.16

Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

You might have to reload the turbo cache apparently when a transitive dep gets bumped, turbo starts failing.

I've reloaded the turbo cache, but the issue persists.😢

closing this since it's a bug in Next and there's nothing we can do on our end. You can inspect the installed package to verify that the code that is missing in your output bundle is indeed present.
Wiping the turbo cache manually resolves this


Using service pricipal not working

Hello together,
I try to run the assessment with an service principal and setup delegated graph permission for Directory.Read.All and Policy.Read.All but I get the errors below.
I think we mss some permissions in ms graph.
Catch-MsGraphError : Applications without a signed-in user are not allowed access to this report or data.
At C:\Program Files\WindowsPowerShell\Modules\AzureADAssessment\2.3.7\internal\Get-MsGraphResults.ps1:372 char:33
+                                 Catch-MsGraphError $_
+                                 ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Write-Error], WebException
    + FullyQualifiedErrorId : AccessDenied,Catch-MsGraphError

When it is not allowed to use service principals, for what is the option to Connect-AADAssessment -Certificate ...
Can someone help me?
Thanks
Arne

Can you share the full Connect-AADAssessment command you are running?

@atiedemann I suspect you need to add "Directory.Read.All" and "Policy.Read.All" application permissions to your app registration and then do admin consent. Could you validate your app registration looks like this?

I'll try it again tomorrow and give you feedback.
Thanks for your response.

@HWf7iz3HuBNbQ8 This was because I do not find the permission at application level. :-(

Good morning, so I tested it again with Application permission and I update the module to the latest version, now it is working.
Thanks a lot
Arne

What I now get ist that the AuditLog.Read.All permission are missing and I get the following error:
Catch-MsGraphError : Calling principal does not have required MSGraph permissions AuditLog.Read.All
At C:\Program Files\WindowsPowerShell\Modules\AzureADAssessment\2.3.11\internal\Get-MsGraphResults.ps1:372 char:33

So I think that these permission must be added to the application.
After I add the permission, the error is gone.

@atiedemann Thanks for the update. And also, thanks for calling out AuditLog.Read.All.