Windows Installer triggers QEMU USB reset, aborting installation

When trying to install Windows 11 ARM version, during the second step of the installation (Getting files ready for installation), this randomly ends with error 0x800701B1 (Windows cannot install required files. Make sure all files required for installation are available.)

This seems to happen at random moments during this phase, somewhere between 0 and 100%. A single time, without any specific modifications to the settings, I managed to get things going to 100%. After that, the rest of the install worked as expected and so did the installed Windows version.
When trying to restart setup, without a VM restart, I noticed that the CD/DVD-drive is no longer readable. Changing the ISO does not seem to help.

Configuration

UTM Version: 2.2.4 (36)
OS Version: 11.6
Intel or Apple Silicon? M1 Silicon (Macbook Air 2020)

Debug log
debug.log

@conath thanks a lot for looking into this.
Just tested your suggestion but had to use the following:
-device usb-storage,drive=drive1,removable=true,bootindex=1,bus=usb-bus.0
Looks like there bus=usb-bus and bus=usb-bus.0, usb-bus.1 did not exist.
Just tried this once so far and immediately managed to get to the next phase of the install.
I also had the impression that performance (for that copy phase) was a lot better in comparison with the defaults.
Will try this a few more times to be sure it's not just a coincidence and will update here.

Thank you for testing! It would seem we've discovered a QEMU bug.
However as you noted there is a speed benefit when we assign the USB controller manually. So I would suggest that the way the Drives configuration is interpreted needs to be changed. Any USB drives should be assigned to the XHCI bus, if available. @osy
This will have the side effect that it works around the bug.

Still hitting this occasionally @osy

I'm encountering this issue on 2.4.1.  Not seeing any error in the debug log but attached if you want to review..
Windows 11 22499.1000 / UTM 2.4.1 / MacOS 12.0.1 / M1 Pro 10-core MBP
debug3.log

I have also been able to reproduce this today using UTM 2.4.1 and Windows 11 ARM Insider Preview 22509.1011 ISO. I tried installing normally twice (with the GUI) and once using dism. Gui gets stuck at "Getting Files ready for installation" and dism stops working at 69% (not nice) and in both cases the USB bus is gone after the error until the VM is powered off and back on again.

Having this issue too. Wondering what QUMU parameters I could add to make it work?

@kad-kroont Here is an idea I have not tested: you could convert the installer ISO to QCOW2 and import into UTM as a disk drive. Set disk interface to NVMe and install from that.

@CONATH if you can provide steps/detail, I'll give it a try.
On Fri, Dec 17, 2021 at 11:47 AM CONATH @.***> wrote:

@kad-kroont https://github.com/kad-kroont Here is an idea I have not
tested: you could convert the installer ISO to QCOW2 and import into UTM as
a disk drive. Set disk interface to NVMe and install from that.
—
Reply to this email directly, view it on GitHub
https://github.com/utmapp/UTM/issues/3194#issuecomment-996867380, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3ZW52T5XSOQH24CHKQ57LURNSTFANCNFSM5GJMIP7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

--

Jason WhiteheadOwner/Operator *Hop
*vision LLC - Craft Beer Marketing Solutions *
*http://www.hopvision.com http://www.hopvision.com *

@.*** @.**>office:<PHONE_NUMBER>mobile:
(717) 343-4922 fax:<PHONE_NUMBER>

I have tried the suggestion of converting the ISO to a QCOW2. This doesn't work, I think because the ISO is multiple partitions, one is EFI, and the other contains install data. Not sure, just a guess. I don't think it maps to a normal NVMe drive with GPT partitions. The Windows installer tells me that files are missing, asn asks me to browse to the location of the necessary drivers. This isn't the exact error message but it's something along those lines. I wonder, if there is a way to put the ISO inside of a QCOW2, convert the ISO to QCOW2, mount both of them to the VM, and read the install data from the ISO inside of the QCOW2 after booting the converted ISO. I'm gonna try that tomorrow.

I had to use
-device
usb-storage,drive=drive0,removable=true,bootindex=1,bus=usb-bus.1
And it took me 8 attempts lol, just have to keep trying until it gets past 100% copying the files.

I downloaded 22000.1_arm64_en-us_multi_a1b78728_convert from https://uupdump.net/
then I had to do the regedit.exe workaround to be able to get past "This PC can't run Windows 11" secure boot check.
then I get this error after attaching the CD/DVD (ISO) Image the default "Import drive" way. It says interface is "USB"
qemu-system-aarch64
 -L /Applications/UTM.app/Contents/Resources/qemu
 -S
 -qmp tcp:<IP_ADDRESS>:4444,server,nowait
 -nodefaults
 -vga none
 -spice "unix=on,addr=/Users/brandonros/Library/Group Containers/WDNLXAD4W8.com.utmapp.UTM/EB1C7DE1-027D-4A35-A1F6-AD4B41536A01.spice,disable-ticketing=on,image-compression=off,playback-compression=off,streaming-video=off,gl=on"
 -device virtio-ramfb-gl
 -cpu cortex-a72
 -smp cpus=4,sockets=1,cores=4,threads=1
 -machine virt,highmem=off
 -accel hvf
 -accel tcg,tb-size=1024
 -drive if=pflash,format=raw,unit=0,file=/Applications/UTM.app/Contents/Resources/qemu/edk2-aarch64-code.fd,readonly=on
 -drive "if=pflash,format=raw,unit=1,file=/Users/brandonros/Library/Containers/com.utmapp.UTM/Data/Documents/Windows 11.utm/Images/efi_vars.fd"
 -boot menu=on
 -m 4096
 -device intel-hda
 -device hda-duplex
 -name "Windows 11"
 -device qemu-xhci,id=usb-bus
 -device usb-tablet,bus=usb-bus.0
 -device usb-mouse,bus=usb-bus.0
 -device usb-kbd,bus=usb-bus.0
 -device ich9-usb-ehci1,id=usb-controller-0
 -device ich9-usb-uhci1,masterbus=usb-controller-0.0,firstport=0,multifunction=on
 -device ich9-usb-uhci2,masterbus=usb-controller-0.0,firstport=2,multifunction=on
 -device ich9-usb-uhci3,masterbus=usb-controller-0.0,firstport=4,multifunction=on
 -chardev spicevmc,name=usbredir,id=usbredirchardev0
 -device usb-redir,chardev=usbredirchardev0,id=usbredirdev0,bus=usb-controller-0.0
 -chardev spicevmc,name=usbredir,id=usbredirchardev1
 -device usb-redir,chardev=usbredirchardev1,id=usbredirdev1,bus=usb-controller-0.0
 -chardev spicevmc,name=usbredir,id=usbredirchardev2
 -device usb-redir,chardev=usbredirchardev2,id=usbredirdev2,bus=usb-controller-0.0
 -device usb-storage,drive=drive0,removable=true,bootindex=0,bus=usb-bus.0
 -drive "if=none,media=cdrom,id=drive0,file=/Users/brandonros/Library/Containers/com.utmapp.UTM/Data/Documents/Windows 11.utm/Images/22000.1_MULTI_ARM64_EN-US.ISO,cache=writethrough"
 -device nvme,drive=drive1,serial=drive1,bootindex=1
 -drive "if=none,media=disk,id=drive1,file=/Users/brandonros/Library/Containers/com.utmapp.UTM/Data/Documents/Windows 11.utm/Images/disk-0.qcow2,cache=writethrough"
 -device virtio-net-pci,mac=5A:DA:4F:C4:DA:1D,netdev=net0
 -netdev vmnet-macos,mode=shared,id=net0
 -device virtio-serial
 -device virtserialport,chardev=vdagent,name=com.redhat.spice.0
 -chardev spicevmc,id=vdagent,debug=0,name=vdagent
 -uuid EB1C7DE1-027D-4A35-A1F6-AD4B41536A01
 -rtc base=localtime

@jensdepuydt On your website you include a section about adding the QEMU parameter to bypass this issue. Is this still up to date?

I have also been able to reproduce this today using UTM 2.4.1 and Windows 11 ARM Insider Preview 22509.1011 ISO. I tried installing normally twice (with the GUI) and once using dism. Gui gets stuck at "Getting Files ready for installation" and dism stops working at 69% (not nice) and in both cases the USB bus is gone after the error until the VM is powered off and back on again.
Edit: same error as in the original issue at the top of the page.

Do you see the same error message in the debug log after the USB bus argument change?

I have also been able to reproduce this today using UTM 2.4.1 and Windows 11 ARM Insider Preview 22509.1011 ISO. I tried installing normally twice (with the GUI) and once using dism. Gui gets stuck at "Getting Files ready for installation" and dism stops working at 69% (not nice) and in both cases the USB bus is gone after the error until the VM is powered off and back on again.
Edit: same error as in the original issue at the top of the page.

@conath Do you see the same error message in the debug log after the USB bus argument change?

@osy No, I saw the same Windows Installer error. The debug log error was not present. Interestingly, in a more recent test I noticed parts of or the entire USB bus seemed to stop working when the install fails this way.

I've been able to trigger this but not consistently. If someone can find a way to more consistently trigger it and/or with QEMU unmodified, we can better debug it.

I faced the same issue every time I tried to install something from a USB/CD Drive on a working Windows VM, the drive just becomes unavailable at some point. Tried adding the drive file after boot, ejecting it, changing the file. Same problem every time. Converting to qcow2 and switching USB3 / shared device count has no effect on it. Nothing related to USB in the logs. Setting -device usb-storage,drive=drive1,removable=true,bootindex=1,bus=usb-controller-0.0 in qemu tab worked.

I tested ~20 installs without issue using the latest commit.

I tested ~20 installs without issue using the latest commit.

@osy thank you so much. Could you cut a binary release please so users do not need to download that commit + build the app?

Patience


docs(api): added docs for useLatest hook

Close #121

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


[pbjson] Add new port

Fixes PR #13131
src file added

Hi @tejas15802
Thanks for adding this port.
It seems that you just add the source code to this PR.
Please refer to this doc https://github.com/microsoft/vcpkg/blob/master/docs/examples/packaging-github-repos.md to learn about how to add a new port.
You can also search other ports in vcpkg to learn more about this.
BTW, please sign CLA first.

@NancyLi1013 Than code doesn't owned by me. So I'm little confused on what option I should be choosing for signing

can you please guide me choose the appropriate option @NancyLi1013

@tejas15802
You can click here https://cla.opensource.microsoft.com/microsoft/vcpkg?pullRequest=13160 to sign CLA.

@tejas15802
You can click here https://cla.opensource.microsoft.com/microsoft/vcpkg?pullRequest=13160 to sign CLA.

I'm not the author of that code, so I'm not sure If I can sign it

This is not related to the author. So it doesn't matter to sign this. Everyone needs to sign this if he wants to contribute to Microsoft open source projects. Since the source codes have included LICENSE file, you don't need to worry about this.

I don't what is causing the issue, can someone help me out?

@tejas15802
You just need to keep CONTROL, portfile.cmake files in this PR. Other files are not needed here. You should remove these files.
BTW, I noticed that this is a header-only library, in this case, you just need to install its headers to vcpkg.
Please refer to this port https://github.com/microsoft/vcpkg/tree/master/ports/libmorton.

Have I done it right this time, coz it's taking long to verify

/azp run

should I look @ the code once again?

should I look @ the code once again?

Sorry, I don't make sense. What do you mean?
If you mean whether you need to update changes or not in this PR, I think it's unnecessary now.
Since the current status seems to be not related with this PR. I will try to rerun it.

should I look @ the code once again?

Sorry, I don't make sense. What do you mean?
If you mean whether you need to update changes or not in this PR, I think it's unnecessary now.
Since the current status seems to be not related with this PR. I will try to rerun it.

Yes that's what I meant. Mm Okay!

Hi @tejas15802
Sorry for the long delay.
It seems that the repository doesn't exist now.
Could you please help take a look and merge master changes to this PR?

I actually deleted the repo, cuz I thought I made some kind of mistake and what I did isn't going to work

I actually deleted the repo, cuz I thought I made some kind of mistake and what I did isn't going to work
@NancyLi1013

Hi @tejas15802
Sorry for the long delay.
It seems that the repository doesn't exist now.
Could you please help take a look and merge master changes to this PR?

Are you willing to create a new repo to make a new PR for the new port?
Without the original repo, this PR cannot continue working.

Are you willing to create a new repo to make a new PR for the new port?
Without the original repo, this PR cannot continue working.

sorry I'm not.

/azp run

Thanks for the PR; we're closing this for now since the repo has been deleted. If you'd like to continue working for the port, please open a new PR.


Device doctor does not support brew in new locations

New versions of brew are installing binaries in a different location which is not supported by device doctor. This is causing multiple devices to get quarantined.

Changes have landed in the salt states repository.


[Perf] Optimize the size

React Tiptap Editor is 11MB in size after bundling.

@condorheroblog yea, I see, vendor file is heavy

I will move shiki and katex to external library

@condorheroblog can you install the latest version. hope bundle size is smaller

Good Job👍

But I don't know why the vendor file is still a little bit bigger.😂

yea. we have many dependencies. I will check later.


Jribbble 3.0

This is a rewrite for Dribbble API v2 http://developer.dribbble.com/v2
A lot of functionality has been removed from the Dribbble API, so v1 functionality is no longer available. Jribbble 3.0 can;

Get the current user
Get the current user's shots
Get the current user's projects

User access tokens generated with Dribbble-approved applications can:

Get the current user's likes
Get a list of Popular shots

I'm not sure how Dribbble's approval process works. For now I included Jribbble methods to fetch likes and shots. They will 403 if your application has not been approved.
With the reduced functionality, this PR also removes the jQuery dependency. Jribbble now works without any other dependencies and is only ~1.7KB when minified. And will be smaller when gzipped.
fixes #16
fixes #18

Ah, yeah @marcobiedermann I've not tried to contact them for popular shots for v2. I've tried in the past and had no luck with their contact form, but have been able to get them on the api twitter account https://twitter.com/dribbbleapi


Fix Change Log Macros

REF is not defined and you don't specify std in XREF.

These XREFs are broken. XREF doesn't work with sub-packages, which is why REF was introduced. Also, XREF's order of arguments is different form REF's.
Better define REF instead.

I hope the point is clear now, that we should stop to do this manually and starting from the next release we will hopefully have simple, plain-text release files :)

Closing in favor of https://github.com/dlang/phobos/pull/4364


Request class for sms statusCallback

Is there a specific class we can use for the sms status callbacks? I see StatusCallbackRequest, but it inherits from VoiceRequest. Can we get something like an SmsStatusCallbackRequest that we can use instead of checking Request.Form?
So specifically, I'm talking about the webhook outlined here for when you want the get the updated status of a message:
https://www.twilio.com/docs/sms/tutorials/how-to-confirm-delivery-csharp

Good idea @rossiter10, I will look at adding this.

Done! Just uploaded 5.20.1 to Nuget.


[BUG] host shutdown with a Ctrl-C exits with a warning

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[ ] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
Stopping the host with Ctrl-C successfully kills the host, but it ends with a warning that the wasmtime epoch interrupt thread is timing out:
Error: failed to shutdown host

Caused by:
    epoch interrupt thread timed out

Steps to reproduce

Start a wasmcloud host, wadm, nats
Start a wadm manifest (I've been using the rust http hello app)
Ctrl-C in a terminal to stop the host
Observe the error

Expected behavior
I expected the host to stop without that warning.
If it's a bug in the way we're stopping wasmtime the we should fix it, otherwise we should downgrade the warning if there is nothing for us to do.
Environment

OS: OSX
Shell zsh
wasmcloud Version: v1.3.0

Screenshots / Logs / Additional context
No response

It's actually a duplicate of https://github.com/wasmCloud/wasmCloud/issues/2391, which was closed a few months ago. Not sure if it's a regression though, I think this is simply a race condition it shutdown sequence.
I also encounter this one quite often

Yep I think this isn't a regression as much as it just wasn't truly resolved 😅

Digging in, this is almost certainly due to any running component taking a reference to the underlying Runtime Engine and the epoch thread not exiting until every reference is dropped.


Queue georeference caching for later in DwC import

Codecov Report

Merging #2741 (6258769) into development (d564e5b) will decrease coverage by 30.71%.
The diff coverage is 37.50%.

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

@@               Coverage Diff                @@
##           development    #2741       +/-   ##
================================================
- Coverage        87.20%   56.48%   -30.72%
================================================
  Files             1874     1345      -529
  Lines            65729    35494    -30235
================================================
- Hits             57318    20050    -37268
- Misses            8411    15444     +7033

Impacted Files
Coverage Δ

...pp/models/dataset_record/darwin_core/occurrence.rb
7.90% <0.00%> (-52.51%)
:arrow_down:

app/jobs/dwc_occurrence_cache_georeference_job.rb
75.00% <75.00%> (ø)

spec/support/concerns/random_token_methods.rb
2.63% <0.00%> (-97.37%)
:arrow_down:

spec/support/concerns/controller/delete.rb
9.09% <0.00%> (-90.91%)
:arrow_down:

spec/support/concerns/request/api.rb
11.11% <0.00%> (-88.89%)
:arrow_down:

app/models/identifier/global/issn.rb
14.28% <0.00%> (-85.72%)
:arrow_down:

app/models/identifier/global/isbn.rb
8.33% <0.00%> (-85.42%)
:arrow_down:

app/models/protonym/becomes.rb
16.43% <0.00%> (-79.46%)
:arrow_down:

app/models/identifier/global/uri.rb
21.42% <0.00%> (-78.58%)
:arrow_down:

lib/catalog/description_from_observation_matrix.rb
13.22% <0.00%> (-78.52%)
:arrow_down:

... and 1107 more

Continue to review full report at Codecov.

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


Switch default visibility from package to public

Motivation
The package access modifier seems to have issues for files produced by plugins, so let's switch back to public.
Modifications
Switch the default access modifier to public.
Result
The default modifier is public again.
Test Plan
Updated tests.

I think we should take a beat and consider whether public is the right default here. I think there's a strong case to be made for internal to make public an opt-in and/or making the accessModifier a required config and command line option to make it explicit. The former, at least, would be more consistent with other workflows for factoring handwritten code into modules and with other generators (e.g. grpc-swift).

Yup, discussed offline, I'll switch it to internal tomorrow.

It was user error, all is well if you use the 5.9 Swift tools version.


Add command line argument -o filename.txt  to store search results in a file.

Ability to write output to a file instead of a standard output. When a -o filename flag is passed, the output from the program should be added to the file instead of printing it on standard output.
Example:
$ ./mygrep lorem loreipsum.txt -o out.txt
$ cat out.txt
lorem ipsum
a dummy text usually contains lorem ipsum

Based on some reading found this is a know behaviour of flag in Go

This is simply the way the flag package deals with arguments. Argument handling is always somewhat convention driven, and the flag package definitely does not follow the gnu-style that many are accustomed to.

Ref: golang flag stops parsing after the first non-option

Closing this and creating a new issue to work on workaround for this.


EHOSTDOWN -- Node Problem?

I receive this error when trying to send commands using the Home app/shortcuts on my iPhone.
Note that Google commands to these bulbs work just fine, so it's definitely either HomeBridge or the plugin.  Wyze through HomeBridge works fine.
I've Googled "EHOSTDOWN" and got...a ghost town (heh) of results with regard to this plugin's usage of that term (as in, none at all).
It seems to be a Node.js specific error as other HomeBridge plugins have thrown it at times like this example.  This example also resolved to a bug in Node.

This means that the plugin is lost its connection to the bulb, I would check your network settings


ModuleNotFoundError: No module named 'qlinear'

I am trying the Transformers examples following the "Setup Transformers" and "Llama 2 - Pytorch" instructions step by step. However, an error occurred, which is as follow:
python run_awq.py --w_bit 4 --task quantize
Traceback (most recent call last):
  File "C:\Users\zxfeng\Documents\AI_Proj\RyzenAI-SW\example\transformers\models\llama2\run_awq.py", line 14, in <module>
    import qlinear
ModuleNotFoundError: No module named 'qlinear'

Is there any solution for this problem? Looking forward to your reply soon.

Hello, I am  also facing same problem and not able to find a way to install qlinear. can you guys help us in resolving the issue?

Hello, I am also facing same problem and not able to find a way to install qlinear. can you guys help us in resolving the issue?

hi, my problem is because I didn't install RyzenAI packets successfully which is in the "Step 3: Build dependencies
pip install ops\cpp --force-reinstall". you should go through step2 and step3 again.(If there is a RyzenAI packets in your python environment, you should uninstall it first)
Most importantly, you should run all the command in CMD environment. DO NOT USE WINDOWS TERMINAL!!!
Hope this can help you!

Hello, I am also facing same problem and not able to find a way to install qlinear. can you guys help us in resolving the issue?

hi, my problem is because I didn't install RyzenAI packets successfully which is in the "Step 3: Build dependencies pip install ops\cpp --force-reinstall". you should go through step2 and step3 again.(If there is a RyzenAI packets in your python environment, you should uninstall it first) Most importantly, you should run all the command in CMD environment. DO NOT USE WINDOWS TERMINAL!!! Hope this can help you!

@fengzhaoxin Anaconda Prompt works?


UI - Make icons font-awesome instead of bg image

Hey @SebastienBoissiere

Can you update this alert image icon to be a https://fontawesome.com/icons/exclamation-triangle?style=solid OR

...so it's consistent with other icons, like the good and bad for example
AB#59075

Hi Tiago
CC: @adamcogan
Done, the icon has been replaced by a Fontawesome icon

Figure: Icone is now from Fontawesome


[BUG] Fetcher is not executed after initial request despite unique Key.

Description
Hi. I've built my Store using a GraphQL fetcher and a Room source of truth, both with Coroutines. I use Int as the request Key to avoid Apollo's generated types in my domain package. I have an interactor named GetLesson exposing API that calls Store's get extension.
The UI is a RecyclerView showing List<Lesson>. When an item is clicked, getLesson.run(lesson.id) begins data loading work.
The first click always completes the entire use-case exactly as expected. But for any later clicks, fetcher and writer are not executed, and then reader always emits the record of the first event.
The only variable request parameter is lesson.id. I've confirmed StoreBuilder is receiving unique lesson.id values with each click. As an experiment, I also converted my request Key to use Apollo's generated LessonQuery but the issue wasn't solved.
I wonder if I’ve misconfigured something or found something. It’s most likely the first scenario. Does anything stand-out to you that might help my issue? Thanks for your insight; keep up the great work!
Expected behavior
A unique request completes Store’s path through fetcher, writer, and reader.
Smartphone:

Device: OnePlus 5T
OS: Android 9
Store Version: 4.0.0-alpha06
Apollo Version: 2.0.3
Room Version: 2.2.5
Coroutines Version: 1.3.5

Additional context
@Dao
abstract class LessonDao {

    @Insert(onConflict = OnConflictStrategy.REPLACE)
    abstract suspend fun insertLessons(lesson: List<LessonEntity>)

    @Insert
    abstract fun insert(lesson: LessonEntity)

    @Query(LessonSql.QUERY_SELECT_ALL)
    abstract fun loadLessons(): Flow<List<LessonEntity>>

    @Query(LessonSql.QUERY_DELETE_ALL)
    abstract suspend fun deleteAllLessons()
}

Any chance you could share your store configuration?
My guess is you’re running into the same issue as this🙂

Thanks @ychescale9! This turned out to be bug a of my own. Sorry for the trouble.


Only show damage if you are not on a collect quest

I got a traceback when on the Attack of the Mundane quest. This fixes that, we'll need to do another collect quest to see if it is correct.

I also cleaned up how animal (pets, mounts) names look to so we get them all formatted in a nice pretty fashion.


Import some localized strings from .NET Framework

A lot of the resource strings used in CoreWF are the same as in the .NET Framework. These could be copied over.

This was done as part of #21


Classifying hard and soft sweeps

Hi,
I would like to use HaploSweep to distinguish between simulated soft and hard sweeps, and I was hoping you could help me with this.
Currently, the HaploSweep manual states -w can be used to choose a window size in SNPs. I was wondering if window sizes can be defined using base pairs (as in Figure S11) instead of SNPs.
Additionally, it would be very helpful if you could share commands/scripts that can be used to reproduce Figure 5 in the paper.

Hi, I met the same issuse.
In paper, I rdad "we expect a higher RiHS value for the selected allele compared to neutral evolution and hard sweeps."
Does a high RiHS value seem to indicate soft selection sweep?

sorry, it is " In the context  of a soft sweep, we expect a higher RiHS value for the selected allele compared to neutral evolution and hard sweeps. "


Names and default names for keys, constraints and indices

While working on #12821, I noticed some that may merit some discussion.
When scaffolding, RelationalScaffoldingModelFactory checks the database name for keys, constraints and indices, comparing them against the default name EF Core would assign to this index. The idea is presumably to avoid useless scaffolding HasName() calls which would bloat OnModelCreating().
Now, at least in the PostgreSQL case (but I think also the others), a name is always returned for keys/constraints/indices. PostgreSQL has its own defaults: primary keys that haven't been explicitly named are called {table}_pkey, etc. This means that for a typical database that has been created without specifying names, HasName() will be scaffolded everywhere because the PostgreSQL defaults aren't the same as the EF Core defaults (PK_{table}).
There seem to be two options:

Current behavior: always take the name reported by the database. This bloats OnModelCreating() with HasName() calls on every single key, constraint and index. But it does have the advantage of round-trippability (exact names are preserved from the scaffolded database).
Alternative behavior: have the database model factories omit names when those names are the database defaults, leading to null names and less HasName() everywhere. Since EF Core must specify names when creating keys/indices, it would see nullls and default create indexes and keys with its own defaults.

Basically it's a choice between round-trippability (current behavior) and a leaner OnModelCreating(). I'm not sure to what extent the former is an actual goal.
What do you guys think?

@roji If I remember rightly, I think the idea was that the names generated by EF by default for PostgreSQL would match the "most common convention" for names created in that database. It sounds like for PostgreSQL this would be to match the names that the database will use if no name is provided.
However, it would be a breaking change to start generating the names in this way now, in that a new migration would be generated to rename all the constraints to their new values. We will discuss in triage what options there might be here.

@bricelam to link related issues.

I see... If I read the code right, that isn't possible currently, is it? I mean, the default name generation seems to be static methods which the providers cannot override etc.
Regardless yeah, it would be a breaking change but it definitely seems like the most correct way to deal with this... Maybe for 3.0?

@roji I looked into this code and we effectively broke the extensiblity model here when we simplified the service provider and model building process in 2.0--dumping "one model to rule them all". This is definitely something we should look at for 3.0.
We also discussed this in triage, and I believe there is a desire to be able to reverse engineer in two ways:

To a model that contains all information needed to then continue managing that database with Migrations. This mode would require thinks like constraint names to be included.
To a model that will only be used by the EF runtime and therefore can be smaller because it doesn't include things that EF doesn't care about.

@bricelam is going to link issues related to this. I don't think we have decided exactly how APIs, switches, etc will look, nor what the default will be.

I only care about: "To a model that will only be used by the EF runtime and therefore can be smaller because it doesn't include things that EF doesn't care about."

I guess this is the right discussion to also think about round-trippability, which is related but not exactly identical to the first reverse-engineering "mode".
Personally having two different reverse-engineering modes (command-line switch?) seems a bit complicated too me, it will probably be difficult for users to understand etc. My gut feeling is that a bit of bloat in the model (stuff like constraint and index names) isn't that bad, or at least not bad enough to warrant a whole new mode. After all, if users don't care about that stuff they can just manually delete it from the model (although if you're working on "iterative" reverse-engineering to continue to update models that might get tricky as false changes get detected).
Just my two cents...

Some notes:
With #10890 we could make the default reverse engineering experience not include constrint names. Part of #2167 is to add a gesture to Scaffold-Database to indicate you want to start using Migrations with it; this would signal that we should reverse engineer things like constraint names. The update model from database feature probably doesn't need the names since it doesn't need to rename anything (it just re-scaffolds everything) and a structural comparison is probably sufficient.

Triage: keeping this open for 3.0 to look at:

Re-enabling extensibility point for provider default constraint names
Considering then what to do with the PostgreSQL provider, and possibly others--i.e. should they take a breaking change in 3.0?

Other aspects discussed here are tracked by #10890, #2167.

Can I add something to this debate. In summary I would like the following two options added to reverse engineering:

Have an option that includes the constraint names in the reverse engineering process
Add a feature like @ErikEJ ExtraOnModelCreating method that can be overridden in another partial class.

My use case for asking for this is as follows:
I am reverse engineering a couple of my client's database which were created via SQL scripts. To ensure that any changes to the database (which happens often) the plan is to run a unit test that uses my CompareEfSql class to check that the EF Core's Model is still in step with the database it accesses. Because my CompareEfSql class checks everything it throws out errors because the primary key constraint names are not the same.
I could use @ErikEJ EF Core Power Tools and use his ExtraOnModelCreating method to alter the primary keys using the approach suggested by Andrew Lock (I tried that and it worked great - Andrew's suggestion opens up some really useful features). The problem is the client wants a single command that they can run, which rules out EF Core Power Tools. I also have a big list of tables that should be scaffolded, so a command they can copy/paste is better.
Mt CompareEfSql tool has a feature to suppress errors so I can get around this, but I would prefer to either get the Primary Key constraint name to be set from the database, or (and!) have @ErikEJ overridable ExtraOnModelCreating method added to the created OnModelCreating.

@JonPSmith as @ajcvickers wrote above, we probably want to have EF generate default constraint names that match the database's default. In other words, if you don't explicitly set constraint names in a code model, a PostgreSQL database created from that model would have PostgreSQL default constraint names, while an SQL Server database would have SQL Server's default constraint names.
If we go this way, then reverse engineering would probably behave accordingly - a default constraint name wouldn't result in a HasName(). This would provide both round-trippability and a lean model.
Unless I'm mistaken, there's nothing in the above that prevents something like CompareEfSql from comparing an EF Core model to an existing database - as long as the tool interprets the lack of an explicit constraint name in the model as a database default. Providers would possibly expose logic for letting tools such as CompareEfSql know what the default name generation strategy is, allowing your tool to understand if a constraint name encountered in the database is a default name or not.
Note that all this isn't finalized, just my idea of where we may want to go.

Hi @roji,
Thanks for the quick feedback. Adding the constraint name if its not the default sound like a good idea and it would also solve my problem.
You might also consider what @ErikEJ does in his EF Core Power Tools, i.e. providing a method at the end of the OnModelCreating which calls an overridable method (@ErikEJ - can you provide info. I only found it by using your tool and I couldn't find it in your docs). That just gives you lots of room for doing complex things.
One minor point. CompareEfSql uses the DbContext's Model property, with entityType.GetIndexes() to get the indexes and then ...Relational().Name to get the constraint name. This always returns a constraint name - either the default or one provided by HasName(). That means I don't need to do anything in CompareEfSql special - it just works.

Thanks for the quick feedback. Adding the constraint name if its not the default sound like a good idea and it would also solve my problem.

Great, I hope I'll get around to looking at this issue soon.

You might also consider what @ErikEJ does in his EF Core Power Tools, i.e. providing a method at the end of the OnModelCreating which calls an overridable method (@ErikEJ - can you provide info. I only found it by using your tool and I couldn't find it in your docs). That just gives you lots of room for doing complex things.

I assume you're asking for this because you want to re-scaffoled an existing context from a database which has changed, and want to make sure some configuration isn't overwritten. If so, take a look at #12245 which seems to be the same, but more in general, at #831 which is a more general approach to updating the model. This would definitely be out of scope for this issue.

One minor point. CompareEfSql uses the DbContext's Model property, with entityType.GetIndexes() to get the indexes and then ...Relational().Name to get the constraint name. This always returns a constraint name - either the default or one provided by HasName(). That means I don't need to do anything in CompareEfSql special - it just works.

I understand, but I'm not sure if that's the right behavior for default names. Imagine a model scaffolded from one database (e.g. SQL Server), used to create another (e.g. PostgreSQL); is seems ideal if the default constraint names from SQL Server are not scaffolded in any way, allowing PostgreSQL's default names to be used. In other words, in my current mental model (which may change) you would see null as the constraint name, and would need to reconstruct the database's default name, possibly with the provider's help.

https://github.com/aspnet/EntityFrameworkCore/blob/34a6ae7e583f9ed0d8ea57354824b1938945891f/src/EFCore.Relational/Metadata/RelationalKeyAnnotations.cs#L50-L51
Current code makes sure that you always get a constraint name.

@smitpatel yeah, this is one reason this would be a breaking change. I'll think about this more in-depth and then we can discuss.

@roji,
Thanks for the link to  #12245, with @ErikEJ OnModelCreatingPartial. I do think that is a useful idea.

I understand, but I'm not sure if that's the right behavior for default names. Imagine a model scaffolded from one database (e.g. SQL Server), used to create another (e.g. PostgreSQL); is seems ideal if the default constraint names from SQL Server are not scaffolded in any way, allowing PostgreSQL's default names to be used. In other words, in my current mental model (which may change) you would see null as the constraint name, and would need to reconstruct the database's default name, possibly with the provider's help.

From my point of view EF Core's Model has to be database specific to be any use. It contains things like the ColumnType, DefaultValueSql and many more that have to be for the current database. I can't speak for others but I use Model to find out things about the database I'm trying to access, so it needs to reflect that database's schema.

@JonPSmith You may be interested in this: https://github.com/ErikEJ/EFCorePowerTools/issues/196

This could be handled by default value conventions


update pixel boars

please resubmit under "/pixel-boars" as we can not merge as "/pixel boars". thank you


Exempt Swift Interface Files From Whole-Module ObjC Errors

We don't expect swiftinterface files to have anything meaningful to
report from these checks anyhow - they will have been run at module
generation time anyways.
In fact, the linked radar demonstrates how this kind of checking can be
detrimental to the usability of the compiler. The test committed here
declares the CoreFeatures framework, an Objective-C (root) class and a
set of Objective-C protocols. Now for the fun part: The Objective-C
class is made to conform to the Objective-C protocols in a Swift
extension. This extension is then printed into the generated
CoreFeatures-Swift header and made available to clients of the module.
Now, because we have an Objective-C protocol and a base class that is
imported from Objective-C, we mirror-in the members of RootProtocol.
The sum total of the monster we have built is that we now have a Swift
class that implements some requirements, and an Objective-C protocol
that has requirements mirrored into that same class. The result is
a raft of spurious selector conflict diagnostics that the user cannot
work around unless they own both the class and the protocol.
Exempt swiftinterface files from this checking.
rdar://69550935

@swift-ci test

@swift-ci test source compatibility

@swift-ci test source compatibility

⛵


Update application-proxy-add-on-premises-application.md

The text provided for the reg file is not correctly formatted to add/edit the registry key. The new text creates the registry key correctly.

@cchapin-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cchapin-ms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cchapin-ms - I am checking on this one. The format was changed in this commit https://github.com/MicrosoftDocs/azure-docs/commit/4f909e7348f788d9509de73ef392b854e6778e3d#diff-bb782a0b66cdebc7f6e98545f673d67f519ebf498443e83cc5472dce98b36174 so your PR would be changing it back. I am not sure if both are valid or one is valid in one environment and another in another environment. What do you think?

The change to line 49 in this commit does not work because it is not valid format for a registry file. The original text works correctly to create the dword value required.
Chris Chapin
Sr. Cloud Solutions Architect
Office:    +1<PHONE_NUMBER>
Mobile:  +1<PHONE_NUMBER>
chris.chapin<EMAIL_ADDRESS><EMAIL_ADDRESS>From: Ken Withee<EMAIL_ADDRESS>Sent: Monday, February 1, 2021 2:57 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Chris Chapin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Update application-proxy-add-on-premises-application.md (#69653)
@cchapin-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcchapin-ms&data=04|01|Chris.Chapin%40microsoft.com|29aaee6d4e0c494e780608d8c6eb9182|72f988bf86f141af91ab2d7cd011db47|1|0|637478062361765054|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PFlJCkxVWetelzmnjmEAJAE2twLY8V4CPNJA0swfpC4%3D&reserved=0 - I am checking on this one. The format was changed in this commit 4f909e7#diff-bb782a0b66cdebc7f6e98545f673d67f519ebf498443e83cc5472dce98b36174https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fcommit%2F4f909e7348f788d9509de73ef392b854e6778e3d%23diff-bb782a0b66cdebc7f6e98545f673d67f519ebf498443e83cc5472dce98b36174&data=04|01|Chris.Chapin%40microsoft.com|29aaee6d4e0c494e780608d8c6eb9182|72f988bf86f141af91ab2d7cd011db47|1|0|637478062361775005|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=dgmTZejjzaO8oi7xyrXSrmtQR9s5Nw50ldHv%2Fl1ugco%3D&reserved=0 so your PR would be changing it back. I am not sure if both are valid or one is valid in one environment and another in another environment. What do you think?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F69653%23issuecomment-771117410&data=04|01|Chris.Chapin%40microsoft.com|29aaee6d4e0c494e780608d8c6eb9182|72f988bf86f141af91ab2d7cd011db47|1|0|637478062361775005|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=rHs6sCFITGPZuwAeUreqvc2ZHafCS8GQ5HUGJbE1BHs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPR2V6YIIYWMUDK5LVBZWOLS44BRRANCNFSM4WXHU5PQ&data=04|01|Chris.Chapin%40microsoft.com|29aaee6d4e0c494e780608d8c6eb9182|72f988bf86f141af91ab2d7cd011db47|1|0|637478062361784949|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2FhEa95FARXF%2B5p%2FDtX8BiSz609L4pg0T%2Fa0cqONJmXs%3D&reserved=0.

Here is the issue from before: https://github.com/MicrosoftDocs/azure-docs/issues/66423

#sign-off


checkSignature is not working on nextjs app router

Summary
I use checkSignature from '@meshsdk/core', and I followed thsi doc:  https://meshjs.dev/guides/prove-wallet-ownership
First, I use pages/api/verify-sign, It works.
But when I use the app router, I moved the code to app/api/verify-sign/route.ts. I debug step by step, and when I add
const isValid = checkSignature(nonce, signature, address),
the entire API returns a 404 with no output. However, when I remove that checkSignature line, the API returns normally, and the logs are printed. In my console, I can see the warning:
 Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./src/app/api/verify-sign/route.ts"

my next config file :
`import type { NextConfig } from "next";
const nextConfig: NextConfig = {
reactStrictMode: true,
webpack: function (config, {isServer}) {
config.experiments = {
asyncWebAssembly: true,
layers: true,
};
if (!isServer) config.target = "web"
return config;
},
async rewrites() {
return [
{
source: '/cgent/:path*',
destination: 'http://localhost:8080/cgent/:path*',
},
];
}
};
export default nextConfig;
`
Steps to reproduce the bug
No response
Actual Result
no
Expected Result
no
SDK version
1.8.0
Environment type

[X] Node.js
[X] Browser
[ ] Browser Extension
[ ] Other

Environment details
No response

@mackieq you have reference example repo?

I created a project using the template npx meshjs my-app and found that the configuration file in this template lacks serverExternalPackages: ["@meshsdk/core", "@meshsdk/core-cst", "@meshsdk/react"], which caused the issue I mentioned above.


Autocompletion for paths

When entering a repository path, users should be able to hit TAB for autocomplete, like in bash.

Can this issue be taken?

I believe it was not. Yours if you are interested!

Hey, Can I work on this issue?

Ur a issue
On Fri, Aug 20, 2021, 11:00 AM namanrana @.***> wrote:

Hey, Can I work on this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sourcerer-io/sourcerer-app/issues/138#issuecomment-902829650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTSXVY5B3YMU6E7GMTN7UTT52C2PANCNFSM4EHOW23Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


System throws error AFTER load

NOTE: Probably the same root cause with https://github.com/epam/ketcher/issues/5047 and https://github.com/epam/ketcher/issues/5048
Steps to Reproduce

Go to Macromolecules - Sequence mode <---IMPORTANT
Load from file: System throws error AFTER load .zip

Actual behavior
System loads canvas BUT rise the error

Expected behavior
No error

Environment details:

Ketcher Version 2.23.0-rc.1 Build at 2024-07-02; 09:03:38
Indigo Toolkit Version 1.21.0-rc.3.0-g1570d3f9f-x86_64-linux-gnu-11.2.1
Windows 10 Pro
Chrome Version 126.0.6478.127 (Official Build) (64-bit)

Issue found while testing - https://github.com/epam/ketcher/issues/3532

Fixed.

Ketcher Version 2.23.0-rc.2 Build at 2024-07-10; 19:39:22
Indigo Version 1.22.0-rc.2.0-ga21fb5d8f-wasm32-wasm-clang-19.0.0
Win 10
Chrome Version 126.0.6478.127 (Official Build) (64-bit)


python language

contains python language, rewrite it on another language pls

also some code is stolen from that one project

why did you close kwuuh


Drawing thin_lines is affecting wireframes

Drawing thin_lines is affecting wireframes.
https://github.com/koide3/iridescence/assets/7959916/ae01a8ba-644b-4cf6-a979-5d02be82e7f0
Is it necessary to set the line width back to default(1.0)?
https://github.com/koide3/iridescence/blob/0172a680ce34de115049606a2b45aa6152732e11/src/glk/thin_lines.cpp#L64

Thanks for reporting the issue. I fixed it with https://github.com/koide3/iridescence/pull/99. I also added an option to change the line width of wireframe meshes.


made this compile on .net 5

crude but effective, not build on .net 4.6.1 or net standard 2
I've also disabled Analyzers since I don't need them for this project..

a few things to do:

restore feature-parity

re-include original .net versions

try and make a github workflow to create the whole thing for us

make different workflows as per .net version and platform (ie: x86/arm)


Fixes Footer Issue#10

Trying to make the footer look nicer as requeted.
Suggestions? Changes?

Very well done @jaas666 🙌
PR will be merged soon


SpellTable Integration

Integrate with SpellTable to build SpellBot like features.

Currently blocked by WoTC not allowing SpellTable API access.

.


Monitoring custom motor instrument / Tutorial example for EPICS hardware

I am new to EPICS instruments and I am not sure if this is the right place to ask this question.
I have a Newport XPS-D4 control stage and I wish to monitor the positions continuously. I control the stage according to here. The code is like this:
from newportxps import NewportXPS
from newportxps.XPS_C8_drivers import XPS, XPSException
import pysftp
from ophyd import EpicsSignal

hostname = 'XPS-1b81'
xps = NewportXPS(hostname)

for sname, info in xps.stages.items():
    print(sname, xps.get_stage_position(sname), info)

Output:
Z.Z-MFA 0.9999984006 {'stagetype': 'MFA@MFA-CC@XPS-DRV11', 'max_velo': 2.5, 'max_accel': 3.3333333333333335, 'low_limit': -0.0001, 'high_limit': 25}
XY.X-MFA 0 {'stagetype': 'MFA@MFA-CC@XPS-DRV11', 'max_velo': 2.5, 'max_accel': 3.3333333333333335, 'low_limit': -0.0001, 'high_limit': 25}
XY.Y-MFA 0 {'stagetype': 'MFA@MFA-CC@XPS-DRV11', 'max_velo': 2.5, 'max_accel': 3.3333333333333335, 'low_limit': -0.0001, 'high_limit': 25}

I can follow tutorial codes using simulated hardware, but how can I add e.g. this stage controller instrument to monitor the positions of all the axes in here:
x = EpicsSignal(XXX, name='x')

def callback(value, old_value, **kwargs):
    print(f"Value changed from {old_value} to {value}.")

token = x.subscribe(callback)

Taking a quick look, it seems the NewportXPS object provides a custom Python interface to the user and communicates to the device using its TCP protocol. Despite being located in the pyepics GitHub organization, I don't think there is any EPICS involved in it, so ophyd's EpicsSignal is no help here.
I believe the easiest way to integrate this with bluesky/ophyd would be to write a class that takes in an instance of NewportXPS  and provides the "bluesky interface". That is, it acts as an adapter between the custom Python interface of NewportXPS and the hardware abstraction provided by ophyd and expected by bluesky.
class NewportXPSDevice:
    def __init__(self, xps):
        self._xps = xps

    def read(self):
        ...

    def subscribe(self)
        ...

Depending on how much structure NewportXPS exposes (and how much of it you need) it may or may not be useful to reuse the ophyd.Device and ophyd.Component classes to make NewportXPS a Device subclass with sub-Devices.

Ok thank you @danielballan. Do you happen to have an example of how that can be written for non-EPICS instruments?

Yes, exactly.

Okay, any example on how the subscribe function should be defined?

The implementation we use is in https://github.com/bluesky/ophyd/blob/22089ca1795dd41b214c3589fb25af84f644356d/ophyd/ophydobj.py#L102-L125 which handles the "how do I do dispatch at the ophyd level" and we then make use of the monitoring functionality of epics to subscribe a function that calls _run_subs when the underlying control system.

it seems that the user's request has been comprehensively answered, can we close this @tacaswell ?


Open page with release notes on each extension update.

On first launch after an update we should show a webpage, with the release notes for the update. That way we can inform the user of the new features, UI, etc... With pointers to the new documentation.

Opening a webpage without the user asking does not seem like a good idea.
The goal of this feature should be part of our Marketplace page, for which there is already a link when an update is available.


Fix async onDispose during graceful shutdown

Problem
There is a bug during graceful shutdown. If onDispose returns a promise it won't wait for it to resolve before the process terminates. This bug doesn't happen when a room is being automatically disposed if clients have left and autoDispose property is set to true.
Steps to reproduce
In a room handler, setup the onDispose method to something like this:
onDispose() {
  return new Promise((resolve, reject) => {
    console.log("onDispose: Timeout started");
    setTimeout(() => {
      console.log("onDispose: Timeout complete");
      resolve();
    }, 2000);
  });
}

Then connect clients to a room and terminate the running process (CTRL+C on the terminal)
You should see the first message onDispose: Timeout started printed but not the second.
Cause
In Room.ts the constructor sets up a listener for "dispose" to handle proper disposing of the room (and eventually running this.onDispose if set). During graceful shutdown, the disconnect() method also registers another listener for "dispose" which simply resolves it without waiting for the other listener to complete any async calls.
Solution
My proposed solution is to emit a "disposeComplete" event when the main "dispose" listener (in  Room.ts constructor) completes, and have the disconnect method register a listener for "disposeComplete" instead.

Nice catch @LightningK0ala! Thanks a lot for this.
(Not sure why GitHub didn't recognize this as merged, but here's your commit in the history ❤  https://github.com/gamestdio/colyseus/commit/e8c67f28c7ff76614f54cd7eb0a42965cebe8785)


Respect

Awesome job!!! 💥 A master's project is always better. 🙏

Thank you for your feedback and the compliment on our project! We're glad you find it valuable. 🙏
If you have any specific suggestions or issues you'd like us to address, please let us know. We'll review it further and consider it for future updates. (. ❛ ᴗ ❛.)

fine 👺


fix issue action documentation in token contract

replace https://github.com/EOSIO/eosio.contracts/pull/414 by doing the change in the develop branch.

related https://github.com/EOSIO/eosio.contracts/pull/390

This is a duplicate of an earlier pull request on the same issue https://github.com/EOSIO/eosio.contracts/pull/390 Accepting #390 instead of this one.


cannot flash or erase_flash in v4.0.1 (IDFGH-3494)

Hi,
I just started to migrate from 3.3 to 4.0.1.
I followed the installation guide and I want to flash the ESP32 but it won't work.
Using the same hardware I can flash, erase_flash without any problem in version 3.3.
I only can use idf.py -p /dev/cu.usbserial-14Y0DC69 monitor
The erase_flash option gives the following output:
idf.py -p /dev/cu.usbserial-14Y0DC69 erase_flash
Checking Python dependencies...
Python requirements from /Users/skor/esp/esp-idf/requirements.txt are satisfied.
Executing action: erase_flash
Running esptool.py in directory /Users/skor/esp/hello_world/build
Executing "/Users/skor/.espressif/python_env/idf4.0_py2.7_env/bin/python /Users/skor/esp/esp-idf/components/esptool_py/esptool/esptool.py -p /dev/cu.usbserial-14Y0DC69 -b 460800 --before default_reset --after hard_reset erase_flash"...
esptool.py v2.8
Serial port /dev/cu.usbserial-14Y0DC69
Connecting........_
Detecting chip type... ESP32

A fatal error occurred: Failed to read register address 6001a00c (result was 01050000)
esptool.py failed with exit code 2

Any idea why it works for years now with make and NOT with idf.py?

Hi @korstiaanS,
Thanks for the detailed report.
The failure is actually coming from esptool.py not idf.py. Both the GNU Make build system and idf.py execute esptool to perform the flashing steps.
ESP-IDF v3.3 used esptool v2.8-dev, and ESP-IDF v3.3.1 and v4.0.1 both use esptool v2.8. Do you happen to know if the same esptool version was being used successfully before (esptool will print the version to the console each time it runs.)
The error message looks like a error occurred while the serial connection was reading the information from the chip. Usually I'd suggest this was a hardware problem (signal integrity or power stability), or a driver problem, rather than a software problem.
Is there any difference in the serial port device path or the baud rate configured with ESP-IDF v3.3 vs v4.0? (You can set the baud rate by passing the -b RATE option to idf.py.)

Hi,
The v3.3 is on my MacBook Pro and the v4.0.1 is running on a virtual machine on Parallels.
Maybe that's the reason why it won't work properly?

Hi @korstiaanS ,
Is MacOS the OS in both cases?
Are you passing through the serial port to the VM guest as a USB device or a serial device? If serial, then suggest trying USB instead - but it looks like it's already USB.
Otherwise, would suggest trying the things mentioned in the previous comment.

One other though, if it's a different OS version in the Parallels VM then make sure the USB to serial driver is updated to the latest version. Some issues have been reported with MacOS Catalina drivers in particular.

Hi,
Yes same macOS in both cases.
I'm passing through as a USB device.
My physical mac use pytool 2.8-dev and the virtual one v2.8
I installed FTDI driver 2.4.2 but I'm not sure if the OS is using this driver since it is coming in via parallels?
Changing baud rates won't help either.

Can you try updating the FTDI driver in the VM guest as well?

I did it inside the VM.
I installed the VCP driver. Is that the good one.
https://www.ftdichip.com/Drivers/VCP/MacOSX/FTDIUSBSerialDriver_v2_4_2.dmg
I'm just wondering if macOS really uses this driver?

I also see that my fysical MBP (the working one) has no FTDI driver installed/running and that my VM has now the FTDI software installed BUT it shows 'No' in the loaded column.

It works:
I found a driver v2.4.4 on the net and if I install this driver inside my VM it works.
This is the post:
http://www.ftdicommunity.com/index.php?topic=505.0
So now it works on a VM in parallels!

Hi @korstiaanS,
Thanks for providing the link and glad you found a configuration that works in the VM.
If the issue is now resolved, can this issue be closed?

Hi,
Thank you!
I will close the case.


Formatter should be determining line break type from file

Always using CRLF is wrong since people may be exchanging files between systems without SCC that may be able to correct line endings.

#1285


check ch02 on latest softw.

Last check before finalizing the PDFs.

Hello @rasbt! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file ch03/ch03.py:

Line 38:1: E303 too many blank lines (5)
Line 38:80: E501 line too long (82 > 79 characters)
Line 38:83: W291 trailing whitespace
Line 39:2: W291 trailing whitespace
Line 40:45: W291 trailing whitespace
Line 41:2: W291 trailing whitespace
Line 42:5: W291 trailing whitespace
Line 43:2: W291 trailing whitespace
Line 44:28: W291 trailing whitespace
Line 45:2: W291 trailing whitespace
Line 638:1: E265 block comment should start with '# '


Accessing Devices for rootless container

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
Running deconz in a rootless podman container will lead to missing gateway firmware info / Zigbee channel not being shown in web-gui.
Running the same container as root will work flawless.
Using: ConBeeII
FirewallD and SELinux have been disabled for testing.

Steps to reproduce the issue:

User has been added to dialout group do access device
sudo usermod -a -G dialout $USER

Podman container settings
podman run -d \
    --name=deconz \
    --restart=always \
    -v /etc/localtime:/etc/localtime:ro \
    -v /home/username/deconz:/root/.local/share/dresden-elektronik/deCONZ:Z \
    --device=/dev/ttyACM0 \
    -e DECONZ_WEB_PORT=40850 \
    -e DECONZ_WS_PORT=40860 \
    -e DEBUG_INFO=1 \
    -e DEBUG_APS=0 \
    -e DEBUG_ZCL=0 \
    -e DEBUG_ZDP=0 \
    -e DEBUG_OTAU=0 \
    -p 40850:40850 \
    -p 40860:40860 \
    marthoc/deconz

start podman container rootless
Describe the results you received:
Web ui wont show firmware/device information. Zigbee channel not shown and cant be changed.
Describe the results you expected:
Same behavior root/rootless container startup
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Version:            1.6.2
RemoteAPI Version:  1
Go Version:         go1.13.1
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.13.1
  podman version: 1.6.2
host:
  BuildahVersion: 1.11.3
  CgroupVersion: v2
  Conmon:
    package: conmon-2.0.2-1.fc31.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.2, commit: 186a550ba0866ce799d74006dab97969a2107979'
  Distribution:
    distribution: fedora
    version: "31"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    name: crun
    package: crun-0.10.6-1.fc31.x86_64
    path: /usr/bin/crun
    version: |-
      crun version 0.10.6
      spec: 1.0.0
      +SYSTEMD +SELINUX +APPARMOR +CAP +SECCOMP +EBPF +YAJL
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 4
  eventlogger: journald
  hostname: ruben
  kernel: 5.3.12-300.fc31.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.0-20.1.dev.gitbbd6f25.fc31.x86_64
    Version: |-
      slirp4netns version 0.4.0-beta.3+dev
      commit: bbd6f25c70d5db2a1cd3bfb0416a8db99a75ed7e
  uptime: 47m 59.45s
registries:
  blocked: null
  insecure: null
  search:
  - docker.io
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - quay.io
store:
  ConfigFile: /home/username/.config/containers/storage.conf
  ContainerStore:
    number: 4
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.7-1.fc31.x86_64
      Version: |-
        fusermount3 version: 3.6.2
        fuse-overlayfs: version 0.7
        FUSE library version 3.6.2
        using FUSE kernel interface version 7.29
  GraphRoot: /home/username/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 4
  RunRoot: /run/user/1000
  VolumePath: /home/username/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman.x86_64                                     2:1.6.2-2.fc31

Additional environment details (AWS, VirtualBox, physical, etc.):
dmesg output
[  452.998689] usb 1-5: USB disconnect, device number 2
[  453.248939] usb 1-5: new full-speed USB device number 3 using xhci_hcd
[  453.376788] usb 1-5: New USB device found, idVendor=1cf1, idProduct=0030, bcdDevice= 1.00
[  453.376795] usb 1-5: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[  453.376799] usb 1-5: Product: ConBee II
[  453.376804] usb 1-5: Manufacturer: dresden elektronik ingenieurtechnik GmbH
[  453.376807] usb 1-5: SerialNumber: DE1997763
[  453.379022] cdc_acm 1-5:1.0: ttyACM0: USB ACM device
[  457.243954] usb 1-5: USB disconnect, device number 3
[  457.244248] cdc_acm 1-5:1.0: failed to set dtr/rts
[  457.556997] usb 1-5: new full-speed USB device number 4 using xhci_hcd
[  457.684774] usb 1-5: New USB device found, idVendor=1cf1, idProduct=0030, bcdDevice= 1.00
[  457.684781] usb 1-5: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[  457.684786] usb 1-5: Product: ConBee II
[  457.684790] usb 1-5: Manufacturer: dresden elektronik ingenieurtechnik GmbH
[  457.684794] usb 1-5: SerialNumber: DE1997763
[  457.686380] cdc_acm 1-5:1.0: ttyACM0: USB ACM device
[  676.791514] TCP: request_sock_TCP: Possible SYN flooding on port 40850. Sending cookies.  Check SNMP counters.
[ 1591.589068] TCP: request_sock_TCP: Possible SYN flooding on port 40850. Sending cookies.  Check SNMP counters.
[ 1948.399153] TCP: request_sock_TCP: Possible SYN flooding on port 40850. Sending cookies.  Check SNMP counters.

podman logs deconz

[marthoc/deconz] Starting deCONZ...
[marthoc/deconz] Current deCONZ version: 2.05.71
[marthoc/deconz] Web UI port: 40850
[marthoc/deconz] Websockets port: 40860
[marthoc/deconz] VNC Disabled
libpng warning: iCCP: known incorrect sRGB profile
23:54:17:651 HTTP Server listen on address <IP_ADDRESS>, port: 40850, root: /usr/share/deCONZ/webapp/
23:54:17:661 CTRL. 3.16.2
23:54:17:786 parent process /bin/sh
23:54:17:786 gw run mode: normal
23:54:17:786 GW sd-card image version file does not exist: /root/.local/share/dresden-elektronik/deCONZ/gw-version
23:54:17:789 DB sqlite version 3.16.2
23:54:17:789 DB PRAGMA page_count: 35
23:54:17:790 DB PRAGMA page_size: 4096
23:54:17:790 DB PRAGMA freelist_count: 0
23:54:17:790 DB file size 143360 bytes, free pages 0
23:54:17:790 DB PRAGMA user_version: 6
23:54:17:790 DB cleanup
23:54:17:790 DB create temporary views
23:54:17:912 don't close database yet, keep open for 900 seconds
23:54:17:915 started websocket server at port 40860
23:54:17:918 GW firmware update not supported on x86 linux headless
23:54:17:919 found node plugin: libde_rest_plugin.so - REST API Plugin
23:54:17:920 found node plugin: libde_signal_plugin.so - Signal Monitor Plugin
23:54:17:927 found node plugin: libstd_otau_plugin.so - STD OTAU Plugin
This plugin does not support propagateSizeHints()
This plugin does not support propagateSizeHints()
23:54:18:733 wait reconnect 15 seconds
23:54:19:783 wait reconnect 14 seconds
23:54:20:728 New websocket xx:35744 (state: 3)
23:54:20:833 wait reconnect 13 seconds
23:54:21:864 wait reconnect 12 seconds
23:54:22:864 wait reconnect 11 seconds
23:54:23:864 wait reconnect 10 seconds
23:54:23:997 Announced to internet http://dresden-light.appspot.com/discover
23:54:24:864 wait reconnect 9 seconds
23:54:25:864 wait reconnect 8 seconds
23:54:26:864 wait reconnect 7 seconds
23:54:27:808 New websocket xx:38428 (state: 3)
23:54:27:864 wait reconnect 6 seconds
23:54:28:864 wait reconnect 5 seconds
23:54:29:864 wait reconnect 4 seconds
23:54:30:079 Websocket disconnected xx:35744 (state: 0)

Does your user have permission to access /dev/ttyACM0? That's likely the issue here. You'll need to have permission not tied to a group, as well, as Podman drops supplemental groups in containers by default (though I believe we added a way around that recently...)
@giuseppe How do we invoke Podman to not drop supplemental groups when crun is in use, and did that make it into F31 yet?

You're right i only added the user to the group dialout.
Now i added the user directly to the device itself and that solved the problem.
SELinux did block the access however. Seems like i haven't noticed that before while testing it.
Perhaps that would be an helpfull entry to the readme in the section where the dialout group is mentioned.
Thank you very much for your quick support! This helped alot!

Please open a PR with the fixed wording.


Breaks with AWS SDK v3.0

AWS version 3.0 was announced on 5/27/15: http://blogs.aws.amazon.com/php/post/Tx29HZZVCASVU21/Version-3-of-the-AWS-SDK-for-PHP
Since you don't specify which version to use, 'Aws\Common\Aws'  class reference doesn't work. Please refer to http://docs.aws.amazon.com/aws-sdk-php/v3/guide/guide/migration.html for migration information.
As a temporary fix, one can add the following to their main composer.json file.
"aws/aws-sdk-php": "2.*"

Thanks @canreo. I was not aware of this release.
For now this extension will only work with the SDK version 2.0. I will update the extension to work with version 3.0 ASAP.
Closed with f24782963d60741ba6e511337ed72bab86d80128

The extension was updated to work with AWS version 3 in the release 2.0


Unable to import JEEFACEFILTERAPIGEN from npm module

I was trying to import JEEFACEFILTERAPIGEN from npm module to create multiple videos on a page. I can't do it since  JEEFACEFILTERAPIGEN is not exported from jeelizFaceFilter.module.js.
import {
JEEFACEFILTERAPI,
NN_DEFAULT,
NN_4EXPR,
NN_LIGHT
} from 'facefilter';
I need to import JEEFACEFILTERAPIGEN, can I do that?
Thanks for your help.

You need to detect faces on multiple videos simultaneously?

You need to detect faces on multiple videos simultaneously?

Yes. I need to detect faces from videos that come from the same camera and add different objects to each face.

If you want to detect multiple faces on the same video you don't need JEEFACEFILTERAPIGEN, JEEFACEFILTERAPI handles multi faces detection.

If you want to detect multiple faces on the same video you don't need JEEFACEFILTERAPIGEN, JEEFACEFILTERAPI handles multi faces detection.

 Those videos come from the same camera.
Like in here , but both videos come from the same camera.

Hi,
I have replaced JEEFACEFILTERAPIGEN global variable by the create_new() method (see https://github.com/jeeliz/jeelizFaceFilter#multiple-videos) so that it is possible to use it with the ES6 version.

Hi,
I have replaced JEEFACEFILTERAPIGEN global variable by the create_new() method (see https://github.com/jeeliz/jeelizFaceFilter#multiple-videos) so that it is possible to use it with the ES6 version.

Awesome! Thanks a lot for your great work. I will try it.


Bugs on the Polygon textureCoordinates attribute

What happened?
I calculated the textureCoordinates property of Polygon, but the texture is incorrect. I reversed the positions array of textureCoordinates and the texture is normal. I want to know why this is because, in theory, The result obtained from an array without inversion is correct. But now it's the opposite.Is there a way to identify whether it is necessary to reverse t the positions array of textureCoordinates .thanks
Reproduction steps
1.http://localhost:8088/Apps/Sandcastle/index.html#c=lZVbj+I2FMe/isUTVIzjaxyzMGpFK7VSVx2pVV/KPngTL7hNYuQYZtlqv/ueXIAMO7OXgIRjn9/f9rmR+7qJ6Ojsow1ohWr7iNa2cYcK/93NTTeTvHtf+zoaV9uwmcxeberSRmRCMKcV+mdTo+G5jsZKaxMijEzNpohgLSmhOtNUMqU0ncNUpjTJiOKE0JSpjM3mV6HndUiLaSIY01xIpVPBiRphL1BoTi5GL5jcng8QTATXLCNEKSkZIaPzPS/SXYoImeoMQMFSuBmjcnam3oADk6RzHw72aENjp7NXSRJ3rkH5ztRbi3xdnsD+aAKqTLTBmfJpfH6rzNa+HpYegt/bEE/T/6+eSxL0BXx9aKKvvirSPq41WqDNBOMEvq7aJvSO4n/3281k/sQUtgx2b01cvOxfwtAPg9Wh89PoffZ047PWC14mc0qeEB/Pmbn3ro5kdctw/C746me7DdY2U4TuKIf9QQXSYkzSbyBpS96C7Fu3HIGFa44r9DkH89HUuZ32l5n3G4yow9cp1lO0pfKu0qN9Hw/BFg++PG19DTnR1z62dXTR2QabougToDaVXfTu3Ux+eR/DobDFRQAu3Cs8urhDkDd3e9+AAszsrNvuYoNMXaC8S7MzhXLvQ+FqSLhmyJ1BZoGGpNs5G0zId6cF+qIvf2qr59d+pym0oE1NqcCMgH815xnUqUhhA8awIlpJDqXIOKFUwiSXGiuqoD7TLJOZVGJ+wSnUPOVSScKuOMkUgbVMtoY8hQhqKZiQkmSCp98FixRLokWmldCKsmz+XQcXCtNUg3yaKq0pH9E0hY4jUhiM6JRTyTlYn2GdUsJERlKeKtF7/M25nQ1BWl9jdIlKG6c+us2i61sD8nH4hfg/DAZ9TBYI8sX2q73JuQ0tLqNutVXo6rb9DLn4wfvqLz+9yczWaDKfLJt4Ku39+Vw/umrvQ0SHUE6hOUVb7cv26MnbQ/6fjThvOrA1XSZjdFm4I3LF6pk/OJSXpmlg5d2hLP90H+xmcr9MwP4ztPQGXLX9A3p4aU6t2Y7e/95PYoyXCbw+T0bvy7cm3Ch/Ag
2.Open comment code : array. reverse()
Sandcastle example
No response
Environment
Browser:
CesiumJS Version<EMAIL_ADDRESS>Operating System:

//

Hi there,
I think the root of this issue may be that polygons are design to be tangent to the ellipsoid surface rather than perpendicular. I would suggest a billboard or possibly a Wall Geometry for your use case.
CesiumJS is a big project, so we use GitHub for feature requests and bug tracking exclusively. In the future, please take any questions to the Cesium Forum where there are members of the community and developers from the team who can help. Thanks!


fix a typo

thanks for your share the wonderful notebook， i think there have a small typo in notebook.
Guiding with some  osrt of perceptual loss
Guiding with some  sort  of perceptual loss

Hi @suchot! Thanks a lot for your fix! However it replaces all cell outputs, so I created a new PR #30, in which you were added as an author. Please, let us know if that's ok for you :)

It's ok， no problem.


The controller BUG

Hi,The filter field in the controller displays no data if it is in Chinese, only when it is in English.What should I do?

Check if the related field of the control is correct.

If it is not correct only in Chinese, you can check if Chinese conditional query is supported in this View.

Yes, you are right, my view does not support Chinese queries

However, it is normal for me to query in Chinese on the virtual machine

I solved this problem because it should be the coding problem of the database.
Solution: add "after" to the url connecting to the database?UseUnicode = true&characterEncoding = utf-8 & useSSL = false"
THANKS


Docker-compose volumes don't mount

Hello!
I have a docker-compose that I'm using to build Django + React project.
Yesterday I've wanted to change some staff and figured out that project didn't build. After some investigation I found that volumes don't mount at all (for django and and react frontend)
I'm using this scheme
  backend:
    restart: always
    build:
      context: .
      dockerfile: ./docker/local/Dockerfile.backend
    ports:
      - "8000:8000"
    entrypoint: /backend.sh
    volumes:
      - ./api:/backend

I tried to change the path to "full path" or @PWD/api - didn't help.
I have Version 15.1.3 (47255)

When I removed the volumes mountings and copied the code directly inside the Dockerfile.backend file it started to work, but I would like to keep the advantage of mounting folders during coding, because in this case I don't need to build project every time I change something in UI (React) project

Any suggestions how to fix this?

Can you please provide a bit more details how to reproduce the problem? Dockerfile would be helpful.

Hello!
As I wrote before the problem is that the 'backend' folder contains only backend.sh and 'requirements' folder, because they are copied 'inside' Dockerfile, but it does't contain the source code ('./api') that is mounted inside docker-compose.
There is my Dockerfile
FROM python:3.7

ENV PYTHONUNBUFFERED 1

ENV DJANGO_SETTINGS_MODULE config.settings.local

# creates working directory
WORKDIR /backend

COPY ./docker/local/backend.sh /

# uncomments next line to make it works
# COPY ./api /

# copy and install requirements
COPY ./api/requirements /requirements

RUN pip install -r /requirements/dev.txt

EXPOSE 8000

and entrypoint script
#!/bin/bash

set -e

python /manage.py runserver <IP_ADDRESS>:8000

exec "$@"

Thank you for your help!

@a-samarin Let's make sure that your Mac host folders are actually mapped into the VM. Please run these commands in the working directory on Mac host:
ls ./api
docker-machine ssh <vm_name> -- ls $(PWD)/api

Do they return the same output? I guess, there should be the requirements dir and other stuff.

@legal90 , files and folders aren't mapped
first of all, '$(PWD)/api'  is a wrong path
docker-machine ssh default -- ls $(PWD)/api
ls: /Users/anton/api: No such file or directory
exit status 1

but
docker-machine ssh default -- ls $(PWD)/Projects/[ProjectName]/api
returns nothing
on Mac host
ls  projects/[ProjectName]
api				docker-compose-dev-yml		docs
app				docker-compose-test-dev-yml
docker				docker-compose.yml

As you can see  inside 'project' directory several folders and files on my Mac, but inside docker-machine this command returns only 'api' folder
docker-machine ssh default -- ls $(PWD)/Projects/BottingMe
api

Hope this helps. If you need more information I'll be happy to provide it.

OK, so the working dir in your case is /Users/anton/Projects/BottingMe and it seems there is some issue with sharing this folder (or the entire content of /Users) between the Mac and Docker Machine VM.
Please restart the vm, docker-machine restart default and check if the files are on place?:
docker-machine restart default
# ...
eval $(docker-machine env default)
docker-machine ssh default -- ls -la /Users/anton/Projects/BottingMe

If it doesn't help - try creating a new VM and check there:
docker-machine create -d parallels test-vm
# ...
eval $(docker-machine env test-vm)
docker-machine ssh test-vm -- ls -la /Users/anton/Projects/BottingMe

this command fixed mapping
docker-machine ssh default -- ls $(PWD)/Projects/bottingme/Api
README.md
__init__.py
config
expbot
home
manage.py
requirements

I've successfully built 'django' part of project. Could you please not close the issue  I'll try to build whole project today
Thank you for your help

@legal90, I've built the full project successfully. Thank you!


Fix docker-compose demo on M1 chip

This enforces the default platform that Docker use to build the containers. This PR applies the suggested fix
https://github.com/cashapp/pranadb/issues/351

[ ] Tested on @xuebingli M1 laptop.

@kalfonso Is this still being worked on, or can I close it?

I'm going to close this. Please re-open it if there is something to merge.


License and deps issues

While working on a conda-forge recipe for mccode-antlr with @willend, we discovered a few issues:

No LICENSE file in the repo. For now we wrote BSD-3-Clause in the packaging meta data, but needs to be clarified. And conda-forge packages always needs a LICENSE file.
The pyproject.toml specifies >=py3.9, but specifies deps for <py3.8 which could be cleaned up.
The pinned dependency antlr4-python3-runtime==4.13.2 seems like it will give a lot of potential issues in a mcstas conda env. Could it be lifted please?

As part of #91 these issues have been addressed. Feel free to re-open this issue if you find that anything is lingering.


How do you even use this?

This project is missing pretty much all usable documentation, same for Spigot and Bukkit and Paper as well. I don't know how anybody learns how to use any of this.
I'm just trying to make a long running task in a separate thread and all I have to go on is old questions people have asked which talked about this project because there's never any proper well-written up-to-date and short and concise tutorials anywhere.
The Usage page on the wiki is all I have on how to use this and it's not even a tutorial, it's just a copy and paste blob of code that doesn't make any kind of sense at all.
Can we start with a HelloWorld tutorial that's as short and simple as possible then progressively explain each and every feature stepping me through bit by bit one at a time with each tutorial being as small, short, and simple as possible until we progress to the intermediate and then advanced stages.
I'm never going to understand any of this without that and I don't see how anybody else can either.
Sorry for rant, I feel like I'm climbing mount everest and back just to do the simplest things which adds days and weeks to the simplest plugins because there's almost always never any tutorials anywhere by anyone unless it's 10 years out of date and poorly written. It's often just here's a blob of code or JavaDoc, figure it out yourself.

Creating documentation takes time which is generally in short supply.
This issue tracker does not exist for the purpose of support conversations about using TaskChain. With that being said. There is a fair amount of documentation for this project.
See...
https://github.com/aikar/TaskChain/blob/master/README.md

There are many examples on this page: https://github.com/aikar/TaskChain/wiki/Usage


Cannot invoke an expression whose type lacks a call signature (ES6)

The following code works when transpiled with babel but vscode has a problem with it:
// module1.js
const str = 'abc';
export const fn = () => ({
  key: 'value'
});

export default [str, fn];

// module2.js
import {fn, default as module1} from './module1';

let str, obj;

// this works ...
str = module1[0];
obj = fn();

// ... but this shows the following error:
// Cannot invoke an expression whose type lacks a call signature.
obj = module1[1]();

Am I doing something wrong or is this a bug? I'm using version 0.10.3 on Windows 10.

The 'Format Code' command also turns export default [str, fn]; into export default[str, fn]; which seems to work.

Likely an issue with the TS version we use for JS.

Closed with the adoption of Salsa (https://code.visualstudio.com/Updates#_languages-javascript).


BlockHound integration forces all FastThreadLocalThread instances to be non-blocking

Expected behavior
Allow additional BlockHound integrations to configure the non-blocking state of threads if they wish.
Actual behavior
The current BlockHound integration forces any instance of FastThreadLocalThread to be non-blocking regardless of the return value of other integrations - https://github.com/netty/netty/blob/4.1/common/src/main/java/io/netty/util/internal/Hidden.java#L144
Due to a bug in how BlockHound has implemented integration comparison, this prevents other integrations from overriding this behavior and redefining the thread as a blocking thread if it so chooses - https://github.com/reactor/BlockHound/issues/273
This is an issue with Ratpack as it uses DefaultThreadFactory to produce pools of both compute and blocking threads which utilize the FastThreadLocal to store binding information for its Promise structure.
When integrating Ratpack with BlockHound, due to the issue above, it's impossible to redefine the blocking threads produced in this manner to be blocking instead of non-blocking.
Steps to reproduce
Create an additional BlockHound integration that attempts to set the thread blocking state for FastThreadLocalThread to be false and load it in additional to the current Netty integration (note, the integration must be loaded before the Netty one such that the Netty integration becomes the entry point of the function chain).
Minimal yet complete reproducer code (or URL to code)
    builder.nonBlockingThreadPredicate(current ->
      t -> !(t instanceof FastThreadLocalThread) && current.test(t)
    );

Netty version
4.1.73.Final
JVM version (e.g. java -version)
openjdk version "11.0.14" 2022-01-18
OpenJDK Runtime Environment Temurin-11.0.14+9 (build 11.0.14+9)
OpenJDK 64-Bit Server VM Temurin-11.0.14+9 (build 11.0.14+9, mixed mode)
OS version (e.g. uname -a)
Darwin xxxxxxxx 21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:28:41 PST 2021; root:xnu-8019.61.5~1/RELEASE_ARM64_T6000 arm64

@normanmaurer after digging into this more, I'm not sure Netty can really do anything here and it's actually an issue in Blockhound - https://github.com/reactor/BlockHound/issues/273
In the meantime, I did find a workaround in Ratpack, so I'll close this.


Web hook does not wait for completion of "mergeable" check

Description
Sometimes a webhook is triggered while the UI still displays

Merge conflict checking is in progress. Try again in few moments.

after some action. This means that the content of the mergeable field is stale, so the receiver (in my case buildbot) ignores the PR event with

Received event 'pull_request' from gitea
Gitea Pull Request ignored because it is not mergeable. (payload: action reopened, mergeable false)

A couple of seconds after the action, merge checking is completed and shows the PR as mergeable.
I have observed this on

a reopened action, and on
a synchronized action when the previous PR state had merge conflicts

which led me to the conclusion that the previous mergeable status is put in the payload.
I have observed this on a self-hosted Gitea 1.15, 1.16.something and now under 1.16.8.
Expected behaviour
The webhook firing should be delayed until the merge conflict checking has completed. Otherwise, wrong mergeable status is reported in the payload.
The instance is not under my control, so I unfortunately cannot provide most of the debugging info below. :-(
Details
Gitea Version
1.16.8
Can you reproduce the bug on the Gitea demo site?
did not attempt yet
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Self-hosted instance not under my control, details unknown.
Database
No response

Is there any more info you need on this bug report?

I can confirm this bug as it happened to me as well. The mergeable flag will even stay false regardless of the current state of the PR. Even when manually triggering the same webhook again.

Mergable checking may take very long.

OK. Does this have an impact on the propsed solution? I'd think it doesn't matter when exactly the webhook fires?

The webhook firing should be delayed until the merge conflict checking has completed. Otherwise, wrong mergeable status is reported in the payload.

I don't know whether people would like the webhook is triggered fast or just wait the mergable detection. Maybe we can have some votes here.

Users: If you prefer that webhooks trigger more quickly, but can contain an incorrect mergeable status, upvote this comment 👍

Users: If you prefer that the webhook payload always contains a correct mergeable entry, even if it means that webhooks are delayed (by on the order of a couple of seconds?), upvote this comment 👍

Seems like it's just the 2 of us and i've pushed a PR to the library i am using in order to skip this check entirely. It doesn't work for me and only causes issues. I would like to make use of it but for that it would need to work properly.
What timeframe we are talking about here. I guess it's depending on the size of the project. Could it be seconds, minutes or even hours?

btw, related issue #25849

Or could we have a mergable == checking before the check finished.

Or could we have a mergable == checking before the check finished.

...and then fire a second webhook as soon as the check is finished, potentially triggering a second CI build downstream? Is that not needlessly complicating any API handling logic of your consumers that want to avoid that?
It would be interesting to get a feeling for the answer to the above question:

What timeframe we are talking about here. I guess it's depending on the size of the project. Could it be seconds, minutes or even hours?

I have personally only ever seen O(seconds) (less than 5 surely, but haven't measured objectively) for completing the merge check, but I'm not maintaining the Linux kernel or other huge projects.
@lunny you seem to be concerned that

Mergable checking may take very long.

Do you have any data or experience to share here?

I would like to know how GH handle the problem especially for the big push.

I would like to know how GH handle the problem especially for the big push.

Apparently a background job is involved. Here are some API docs that touch mergeability checks:
https://docs.github.com/en/rest/guides/using-the-rest-api-to-interact-with-your-git-database?apiVersion=2022-11-28#checking-mergeability-of-pull-requests
https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#get-a-pull-request
However, those are about actively querying the PR status via API, not about the webhook timing, and if the webhook is delayed until the check is complete.
As consumers of webhooks passively receive them, I'd think mandating them to actively trigger other API calls to get up-to-date information is weird/undesirable, as opposed to waiting some time for the check to complete.
At least in my perspective, waiting O(seconds-minutes) longer for a webhook to fire would be a total non-issue, but modifying the CI runner (or any downstream consumer, really) that consumes those webhooks to implement additional logic to poll&re-check would be difficult or impossible :shrug: (and useless/ineffective anyway as long as #25849 is unfixed)

Maybe the attribute should have 3 states.

false: cannot be merged
checking: it's being checked, the consume can do an extra API to check later.
true: can be merged.

This will not wait for the checking because most CI doesn't need this field.

See https://docs.github.com/en/graphql/reference/enums#mergeablestate

because most CI doesn't need this field.

This assertion is surprising to me. Do most CI systems just start a build, only to fall down because the merge does not succeed? Or are you referring those that don't build the merge commit (as they should, arguably), but just the PR head?

Yes, I think Github encountered the same problem as us, so why there are 3 values for MergeableState.

Values for MergeableState

CONFLICTING
The pull request cannot be merged due to merge conflicts.
MERGEABLE
The pull request can be merged.
UNKNOWN
The mergeability of the pull request is still being calculated.

The PR head can have no conflict with the base branch(base commit) but have conflicts with the merge base. So I think even if it cannot be merged, the build may still run and succeed.


Access violation exception

Which version of the app?
WinUI 3 Gallery
Description
Upon closing the application (after selecting any of the basic input examples) the following unhandled exception is thrown...
Exception thrown at 0x00007FFFAF8047F8 (threadpoolwinrt.dll) in WinUIGallery.DesktopWap.exe: 0xC0000005: Access violation reading location 0x0000000000000000.
Screenshots

Windows version
Windows 11 22H2 (22621)
Additional context
No response

I am not able to repro this issue. Please re-open if the issue is still persisting.

@karkarl Were you logged into a domain/network account when you tried this? I've found that most of the issues arise from being logged into a domain account while using WinUI3. e.g. the print dialog works fine from my home computer (no domain login) but crashes when I try it from my work machine (logged into a domain).


x-axsis Ticks problem

Hi,
Is there any way to Hide auto-generate  X-axsis's tick,
just show the x-axsis whick in my Xvalues[]?
thanks.

Yeah, but what if you want to remove them completely as he asked? If I set Ticks: nil, it just uses the default ticks.


Create a dataset repo template

Depends on: #3
The template will help to quickly make a new valid dataset repository. It should include the needed CI files, metadata, and any other relevant things.

Here it is: https://github.com/RiverBench/dataset-template


[improve][misc] Allow custom Netty DnsServerAddressStreamProviders

Fixes #xyz

Master Issue: #xyz
Motivation

Modifications

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

/pulsarbot run-failure-checks

Closed via https://github.com/apache/pulsar/pull/18646


Refactor routing

Fixes #261
TODO:

[ ] Update docs

Codecov Report

Merging #271 into master will decrease coverage by 1.32%.
The diff coverage is 92.8%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
- Coverage   95.34%   94.02%   -1.33%
==========================================
  Files          32       32
  Lines        1204     1189      -15
  Branches      115      130      +15
==========================================
- Hits         1148     1118      -30
- Misses         41       51      +10
- Partials       15       20       +5

Impacted Files
Coverage Δ

bocadillo/constants.py
100% <ø> (ø)
:arrow_up:

bocadillo/compat.py
100% <ø> (ø)
:arrow_up:

bocadillo/errors.py
100% <ø> (+2.53%)
:arrow_up:

bocadillo/applications.py
98.36% <100%> (-0.47%)
:arrow_down:

bocadillo/hooks.py
100% <100%> (ø)
:arrow_up:

bocadillo/urlparse.py
100% <100%> (ø)
:arrow_up:

bocadillo/app_types.py
87.5% <100%> (-0.74%)
:arrow_down:

bocadillo/routing.py
91.2% <89.81%> (-8.01%)
:arrow_down:

bocadillo/views.py
95.89% <92.1%> (-4.11%)
:arrow_down:

bocadillo/middleware.py
95.55% <94.59%> (-4.45%)
:arrow_down:

Continue to review full report at Codecov.

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

Must release 0.15.1 first — see #278.