scrcpy server does not restore Android settings in case of unexpected termination/ADB disconnection

Environment

OS: N/A
Scrcpy version: 3.0
Installation method: N/A
Device model: N/A
Android version: 15

Describe the bug
Whenever ADB disconnects (over Wi-Fi, probably because device changes frequency or roamed to different AP), server-side doesn't appear to restore settings as part of the exit cleanup. For example, when scrcpy is running with the--stay-awake and ADB unexpectedly terminates, the Stay awake option in Developer options remains toggled on. Any subsequent scrcpy connection takes this as a base setting and doesn't turn it off when properly exiting either. It has to be manually disabled in Developer options.
What is the server-side behavior for unexpected disconnections? Is the exit cleanup only possible with ADB commands from the client-side? I'm hoping it's possible for scrcpy server to detect abrupt disconnections and execute the cleanup routine independent of the client.

That also makes wonder, how is virtual displays handled when this happens. I hope it doesn't continue to exist and consume resources without recourse.

Which device is it?
On unexpected disconnection, cleanup should still be executed.
For example, if I run scrcpy --show-touches, and I unplug the USB cable, "show touches" is immediately disabled on the phone.
I have a device (Nexus 5 with Android 6) where the cleanup process is killed before it completes, probably an Android bug. But in general, it works.

That also makes wonder, how is virtual displays handled when this happens

The virtual display is destroy anyway. Currently, the content (the app launched in virtual display) is also destroyed. In future versions, it would be great to have a flag to keep the running app (moved to the main display for example). See https://github.com/Genymobile/scrcpy/issues/5563#issuecomment-2508906274

Which device is it?
On unexpected disconnection, cleanup should still be executed.

Pixel 8 Pro/GrapeheneOS, but I've observed it many times on my last phone Motorola Edge 30/LineageOS, that's why I've left device model as N/A.
Is there anything else I can provide to debug this, or is it being Android bug pretty much confirmed?

The virtual display is destroyed anyway. Currently, the content (the app launched in virtual display) is also destroyed. In future versions, it would be great to have a flag to keep the running app (moved to the main display for example). See https://github.com/Genymobile/scrcpy/issues/5563#issuecomment-2508906274

Oh, good. Yes, keeping the app running would be great.

Can you reproduce the problem with scrcpy --show-touches?

I just realized, it might possibly a WONTFIX. When Android decides to roam to a different BSSID, ADB is automatically switched off... I'm guessing server cleanup doesn't have the chance to run before being killed alongside ADB. Perhaps if something like Shizuku was involved, it might be different? But Shizuku also depends on wireless ADB...

Can you reproduce the problem with scrcpy --show-touches?

I'll try tomorrow (past midnight here). Should be easy for me to reproduce by simply kicking off device from my AP.

I'm guessing server cleanup doesn't have the chance to run before being killed alongside ADB.

Yes, it can run when adb is killed. See https://github.com/Genymobile/scrcpy/blob/master/server/src/main/java/com/genymobile/scrcpy/CleanUp.java

Can you reproduce with --show-touches (which is more obvious to test) 100%, or is it random?
https://github.com/Genymobile/scrcpy/blob/master/doc/device.md#show-touches

AFAIK, there are two possible problems:

on some devices, Android unexpectedly kills the clean up process at the same time as the scrcpy process;
currently, if scrcpy terminates before the cleanup process is started (race condition), typically due to an error, the initial state is not restored (can be easily reproduced with scrcpy --show-touches --video-encoder=fail)

I will fix the second problem soon.

Can you reproduce with --show-touches (which is more obvious to test) 100%, or is it random?

Yes, I can reproduce the behaviour consistently with both.

I should work with:
diff --git a/server/src/main/java/com/genymobile/scrcpy/CleanUp.java b/server/src/main/java/com/genymobile/scrcpy/CleanUp.java
index f372855bf..3df2199de 100644
--- a/server/src/main/java/com/genymobile/scrcpy/CleanUp.java
+++ b/server/src/main/java/com/genymobile/scrcpy/CleanUp.java
@@ -108,6 +108,7 @@ public final class CleanUp {
     private void run(int displayId, int restoreStayOn, boolean disableShowTouches, boolean powerOffScreen, int restoreScreenOffTimeout)
             throws IOException {
         String[] cmd = {
+                "setsid",
                 "app_process",
                 "/",
                 CleanUp.class.getName(),

But I think setsid is not available everywhere.
However, this might work:
diff --git server/src/main/java/com/genymobile/scrcpy/CleanUp.java server/src/main/java/com/genymobile/scrcpy/CleanUp.java
index f372855bf..f81b5c183 100644
--- server/src/main/java/com/genymobile/scrcpy/CleanUp.java
+++ server/src/main/java/com/genymobile/scrcpy/CleanUp.java
@@ -108,6 +108,8 @@ public final class CleanUp {
     private void run(int displayId, int restoreStayOn, boolean disableShowTouches, boolean powerOffScreen, int restoreScreenOffTimeout)
             throws IOException {
         String[] cmd = {
+                "/system/bin/sh",
+                "-c",
                 "app_process",
                 "/",
                 CleanUp.class.getName(),
@@ -116,6 +118,7 @@ public final class CleanUp {
                 String.valueOf(disableShowTouches),
                 String.valueOf(powerOffScreen),
                 String.valueOf(restoreScreenOffTimeout),
+                "&",
         };

         ProcessBuilder builder = new ProcessBuilder(cmd);

Please test this binary (to replace in 3.0.2):

scrcpy-server SHA-256: df21e420edd7b723fb6ca70a98846f87c04d5719698d21af9eefea27f0b037b

No change, at least on my device. Yanked the cable and I still get visible touches, and stay awake remains active.

It fixes the problem on my Nexus 5 ("show touches" must be disabled when scrcpy starts, because it restores the initial value).
adb shell settings put system show_touches 0
scrcpy --show-touches
# unplug USB cable

Oh no, sorry, it's only the setsid version which works for me. Try:

scrcpy-server SHA-256: d325a23eba4d14c01a7adfc7a705f4f1a8cc2b0a009f817bf970ee2f30a19fb

No dice with this one either, on my device I still get the same behaviour (of course I reset both show touches and stay awake to off before trying, like I did when I tried the other test binary).
Maybe @oxwivi will have better luck with theirs.

Maybeyour device does not have setsid. What is the result of adb shell which setsid? And adb shell which nohup?

Looks like it has both:
$ adb shell which setsid
/system/bin/setsid
$ adb shell which nohup
/system/bin/nohup

And with:

scrcpy-server SHA-256: 09a7caf75fb694ee08214a415f1a0712ea8fb821a06f9b0d1301351b93ca495

?

Still no change. Perhaps with some devices it's just inevitable...

I've done some experimenting on another computer, using the standard 3.0.2 scrcpy-server, running a logcat viewer on the phone.
In my case, it looks like the server doesn't even recognise the USB cable being yanked, and instead it runs the cleanup process when USB is plugged back in.
In the following screen recording, I start out with USB connected (as confirmed by the message in notification panel), then start scrcpy --show-touches, which is reflected in the logcat.
I tap around and open the notification panel again, then I disconnect the USB cable, but nothing appears in the log.
At around 0:28 in the video, I open the notification panel yet again to show USB is no longer connected, then tap around the screen some more and touches are still shown.
At 0:53, I reconnect the cable, and only then does the cleanup process run, and touches are no longer shown.
https://github.com/user-attachments/assets/546c2fbb-4da4-488e-8966-0a9038530402
Hope this helps.

Weird. If instead of disconnecting the USB cable you execute adb kill-server, what's the behavior?

Same behaviour: nothing in logcat after doing adb kill-server on the computer. Then, when I start scrcpy again on the computer, it runs the cleanup process.

Looks like a device bug, the read() does not wake up if the parent process terminates:
https://github.com/Genymobile/scrcpy/blob/baa10ed0a36ec775712be85f22d3db3f0a6e19f2/server/src/main/java/com/genymobile/scrcpy/CleanUp.java#L172
(or maybe the parent process does not terminate properly?)

I can confirm this bug on a pixel 9 running latest android 15 QPR1. Not sure when it started for me, maybe after moving to A15?
But for a month or so now I've noticed my phone screen staying awake all night on my bed while its been plugged in.
I thought maybe it was a fluke after reading the /doc/device.md and I manually used adb to restore the value. But I just checked again and it's the same problem.
My usecase is that i normally setup a tcpip connection via usb, and most of the time i just yank the cable to go wireless. From there on i launch a direct tcpip connection via hostname. Connecting via hostname and exiting does not seem to create this problem.
I would say that this issue is large enough for me that i would avoid --stay-awake since it's caused my phone screen to stay on while charging overnight several times.
Please let me know if i can provide more information to assist with diagnostics.

Please test #5613, which solves some of the problems (but not https://github.com/Genymobile/scrcpy/issues/5601#issuecomment-2521255574).

Please test #5613, which solves some of the problems (but not #5601 (comment)).

Thanks, I am now testing this. I tried to re-produce with my usecase and it seems ok but i was not necessarily able to consistently reproduce the issue before this test. I will continue testing over the weekend and report back.

Can you reproduce the problem with scrcpy --show-touches?

Can reproduce. Very sorry about my radio silence yesterday, I was far more busy that I expected.

Please test #5613, which solves some of the problems (but not #5601 (comment)).

Sorry, how do you specify different server bins?

Sorry, how do you specify different server bins?

Either set the SCRCPY_SERVER_PATH environment variable to the scrcpy-server you want to use. Or just make a backup and replace scrcpy-server in your release folder.

Please test #5613, which solves some of the problems (but not #5601 (comment)).

Still the same. stdout shows that it has picked up the downloaded scrcpy-server, but the settings remains toggled after I kick my phone off of Wi-Fi while scrcpy is running:
scrcpy 3.0.2 <https://github.com/Genymobile/scrcpy>
INFO: FLAC audio: audio buffer increased to 120 ms (use --audio-buffer to set a custom value)
WARN: Could not disable minimize on focus loss
INFO: ADB device found:
INFO:     -->   (usb)  adb-3A090DLJG000YG-lDVe9W._adb-tls-connect._tcp            device  Pixel_8_Pro
/home/$USER/Downloads/scrcpy-server: 1 file pushed, 0 skipped. 141.0 MB/s (90504 bytes in 0.001s)
[server] INFO: Device: [Google] google Pixel 8 Pro (Android 15)
INFO: Renderer: opengl
INFO: OpenGL version: 4.6 (Compatibility Profile) Mesa 24.2.8
INFO: Trilinear filtering enabled
INFO: Texture: 1920x1080
[server] INFO: New display: 1920x1080/308 (id=7)
WARN: Device disconnected

My situation has definitely improved, I am not able to re-produce as consistently. I did have one time where the flag did not reset, and I am not sure what I did to cause this. If I can narrow down the steps to re-produce I will present them here.


[$500] Workspace - In Workspace, trying to add a number contact & selecting it, contact disappears

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.22
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Launch app
Tap Profile icon
Tap Workspaces ---- Workspace ---- Members
Tap invite
Enter a number eg: +919787766754
From the list, tap and select the contact

Expected Result:
In Workspace, trying to add a number contact & selecting it, contact must not disappear
Actual Result:
In Workspace, trying to add a number contact & selecting it, contact disappears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/dae43a43-e2fe-4265-8d38-4623d49019b8
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012bf34b264efa7a44
Upwork Job ID:<PHONE_NUMBER>212407808
Last Price Increase: 2024-01-05

This is a regression from https://github.com/Expensify/App/pull/31648, which is still within the regression period.
Confirmed by reverting the PR locally:

cc: @abdulrahuman5196 @thienlnam

Thanks Pavlo - assigning from that PR

Hi, Thanks for the PR. It seems this is not a deploy blocker. So Will work on review in my morning.

Hi @abdulrahuman5196 any update?

@kevinksullivan  the PR was merged. Since this is a regression issue, we can close the issue once we verify in production after it reaches there.

the PR was merged. Since this is a regression issue, we can close the issue once we verify in production after it reaches there.

Yeah. Already pointed out here. We can close this issue.


Fix a error: no proper invocation of AM_INIT_AUTOMAKE was found

I'm not sure what the best way to fix that warning is (well, besides dumping autotools).  When I add the AM_INIT_AUTOMAKE, I get an error that there is no Makefile.am, which actually breaks the build rather than simply adding noise.

I followed the recommendations:
https://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_005fINIT_005fAUTOMAKE-invocation

They want you to use automake, but JWM doesn't really (and I certainly don't want to depend on more autotools!).  The only reason JWM invokes automake at all is to copy some of the scripts that configure depends on (maybe there's a way to get those without invoking automake?).  In any case when I add that, the autogen.sh script returns a non-zero exit code in addition to complaining that I'm not fully using all the autotools available to me.  The non-zero exit code causes the other scripts I use to package JWM to fail (I could make them ignore the status, but that seems unsavory).
I really need to get rid of this autotool mess at some point.

@joewing are you thinking of using cmake instead?  Or did you have another idea?

It is necessary to solve the problem:
$ ./autogen.sh
configure.ac: error: no proper invocation of AM_INIT_AUTOMAKE was found.
configure.ac: You should verify that configure.ac invokes AM_INIT_AUTOMAKE,
configure.ac: that aclocal.m4 is present in the top-level directory,
configure.ac: and that aclocal.m4 was recently regenerated (using aclocal)
automake: error: no 'Makefile.am' found for any configure output

@Israel- cmake might be the solution (I've used it before and it seemed decent).  I do like the fairly standard setup that autotools provide, but in general it seems like they cause more problems than they solve.
@rdnvndr The problem is, that with this change autogen.sh returns an error status.  Without the change it complains, but it returns a successful status (and works despite the complaining).  Even with this change, it still complains about the missing Makefile.am.  So I won't merge this because it breaks my release process.

@joewing Thank you for comments!


support shields.io styles

it would be nice to render different shields.io visual styling passing the style argument
https://badges.netlify.com/api/badges.svg?branch=master&style=flat-square

will migrate to badgen when possible.


The error thrown by run() is wrong when the port is blocked

@settlersxp commented on Tue Apr 02 2019
https://github.com/Azure/azure-event-hubs-python/blob/737c5f966557ada2cf10fa0d8f3c19671ae96348/examples/send.py#L36
The error shown is "Please confirm target hostname exists" which is incorrect after a very long wait time. Probably the wait time is equal to the timeout I have set, did not bother to check.
Steps to reproduce:

Set valid azure credentials and project information
Do not set a proxy
Block with a firewall the ports 5671 and 5672
Try to open a new consumer and notice that the error message is "Please confirm target hostname exists"  instead of anything related to blocked ports.

@annatisch commented on Tue Apr 02 2019
Thanks for this report @settlersxp!
I would say it's likely that the underlying error raised if the same in this case (something like "endpoint could not be reached") - which is why it isn't distinguished.
I'll see if there's anything that can be done at the uAMQP level to address this :)

@settlersxp commented on Tue Apr 02 2019
Makes sense, if it's difficult to distinguish the two errors adding the bit with "or make sure the ports 5671 and 5672 are open in the firewall" should do the trick. Afterall some techy people will use the SDK, there is some tolerance to debug and workarounds.

@yunhaoling , Hi,  I still met similar error with read_device_to_cloud_messages_sync.py under iot-hub/Quickstarts/read-d2c-messages
when run above scripts, below error printed:
EventProcessor instance '38b7c210-6a89-4252-a91c-9a74163b785b' of eventhub 'iothub-ehub-iotzy-5817228-77db27f20f' consumer group '$Default'. An error occurred while load-balancing and claiming ownership. The exception is AuthenticationError("Unable to open authentication session on connection b'82aba880-ede3-4af4-a01b-c95879831eab'.\nPlease confirm target hostname exists: b'ihsuprodkwres038dednamespace.servicebus.windows.net'\nUnable to open authentication session on connection b'82aba880-ede3-4af4-a01b-c95879831eab'.\nPlease confirm target hostname exists: b'ihsuprodkwres038dednamespace.servicebus.windows.net'"). Retrying after 11.374749498916175 seconds
An exception: Unable to open authentication session on connection b'82aba880-ede3-4af4-a01b-c95879831eab'.
Please confirm target hostname exists: b'ihsuprodkwres038dednamespace.servicebus.windows.net'
Unable to open authentication session on connection b'82aba880-ede3-4af4-a01b-c95879831eab'.
Please confirm target hostname exists: b'ihsuprodkwres038dednamespace.servicebus.windows.net' occurred during the load balance process.
I  am sure that my connect string is correct,  I copy it from portal web.
I tried both 2 shared access key:   sercvice and iothubowner.   All have this error.
Could you give some hint about  how I debug what is the real problem?  I run read_device_to_cloud_messages_sync.py on ubuntu 18.04,  and I don't enable firewall on my host,  so I guess it should not because port 5671 and 5672 is blocked by firewall.
Thanks

@yunhaoling
I also met below error during send_c2d_message:
if not self._client_ready():

File "/azure/venv/lib/python3.8/site-packages/uamqp/client.py", line 525, in _client_ready
self.message_handler.open()
File "/azure/venv/lib/python3.8/site-packages/uamqp/sender.py", line 211, in open
raise errors.AMQPConnectionError(
uamqp.errors.AMQPConnectionError: Failed to open Message Sender. Please confirm credentials and target URI.
my connect string is correct, since I can use that to handle direct method test.


Actionable cards with hover

It would be a nice feature if card elements could be made actionable (similar to actionable list group items).
So for example add a card-action to any of the applicable card children:

        You could've used me for accordions

My current workaround for a similar effect is to use one list-group-item within a list-group like so:

            Close enough, but it's not exactly ideal

Also I'm forced to use  and href="#" to get a pointer cursor and I cannot enjoy additional styling for example in the card-header or card-footer.

Seems related to : https://github.com/twbs/bootstrap/issues/18294


Linux Privileged ports and Debugging

Is there any other way to debug an app that run in privileged ports on Linux, than by attaching?
For now, I use commands like:
authbind --deep dotnet run --launch-profile "Development"
Then I use in Vscode, I select ".NET Core Attach".
But how can I launch vsdbg under authbind --deep for all my debug profiles?

Also, Wikipedia lists another alternative to authbind: https://en.wikipedia.org/wiki/Authbind. Do you know if that will work for your scenario? I am assuming you would use the path to dotnet (ex: /usr/share/dotnet/dotnet) with it.

@gregg-miskelly

Does it show you the stdin/out of child process that it it launches? Will it stick around until the child process exits?

Well, if not done properly, everything will lead to "Permission denied", as ports <1024 need rights…
Anyway, it was late, and I did not try the only simple and right way (deep is very important here):
authbind --deep code
Now everything is OK.
And If you use setcap, know that each time the binary is updated, the rights are removed (set per binary).
Moreover, if you use "setcap CAP_NET_BIND_SERVICE+p /path/to/binary"
you have to deal manually with all dependency like:

"setcap CAP_NET_BIND_SERVICE+p /usr/share/dotnet/dotnet"
"setcap CAP_NET_BIND_SERVICE+p ./.vscode/extensions/ms-vscode.csharp-1.15.2/.debugger/vsdbg"

and so one… (the 2 above are not sufficient).  Without forgetting that at each update, the process need to be done again. And if an update add some dependency, you will have to update scripts, whereas with authbind, rights are set per user.


Dashboard broke over time

https://godhatesfags.church/ziAn5.png
as you can see ive just uploaded an image and images before this but the dashboard stop updating, also no longer list what user uploaded the images. maybe my sqllitedb broke? and like 10+ commits behind so maybe this was fixed?

Are you sure you uploaded the pictures while being logged in? That looks like an anonymous upload

Randomly updating again.


Adds numeric, string, position, and url CSS types

Adding a number of fundamental and domain specific CSS types. Also creates a types group under css.
A few include -type as part of the name to disambiguate from similarly named objects.
The top level dimension type is included with numeric types, but it could also reasonably be its own feature (with or with out including sub-types such as angel). As included here, it's included with other numeric types and each sub-type has its own feature.

Converting this to draft to retain the conversation. Will take up these changes and split up into appropriate new PRs

The split out PRs have been merged 🎉


DPL-433-1 - As a user I need to be able to select the new Run Design options for SMRT Link 11

User story
As a user I need to be able to select the new Run Design options for SMRT Link 11
Who are the primary contacts for this story
James / Steve
Who is the nominated tester for UAT
James and the rest of the long read team
Acceptance criteria
To be considered successful the solution must allow:

[ ] The availability of the current active SMRT Link versions with a flag for the default version
[ ] The availability of the current SMRT Link options and which versions they are attached to
[ ] Deprecation of existing stand alone SMRT link columns
[ ] Migrate existing SMRT link columns to the smrt link options store
[ ] Validation of SMRT link options for each Pacbio well depending on which version the run is attached to

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/traction-ui#847

Additional context
This is the back end work for DPL-433 and will provide the configuration options for the ui.

Tasks for deprecating existing columns:

[ ] rename on_plate_loading_concentration to on_plate_loading_concentration_deprecated
[ ] rename binding_kit_box_barcode to binding_kit_box_barcode_deprecated
[ ] rename pre_extension_time to pre_extension_time_deprecated
[ ] rename loading_target_p1_plus_p2 to loading_target_p1_plus_p2_deprecated
[ ] rename movie_time to loading_target_p1_plus_p2
[ ] add on_plate_loading_concentration, binding_kit_box_barcode,  pre_extension_time, loading_target_p1_plus_p2 and movie_time to the store
[ ] ensure that all existing tests pass
[ ] Modify the rake task to migrate new pacbio well smrt link options.
[ ] Add a test to ensure new pacbio well smrt link options are migrated successfully

Tasks for smrt link version and option information:

[ ] create a smrt link versions model
[ ] create a smrt link options model
[ ] create a join table so that versions can have multiple options
[ ] create a smrt link versions end point so that each version with it's associated options can be retrieved

Tasks for validating smrt link options:
create a test file for the smrt link options validator which will need to test that validations are run for the well option
use the smrt link versions and options to drive the validator
use the validations for each version to drive the well smrt link option validation. This is tricky and required loading validations dynamically
create a yaml file with all of the current smrt link option names which is included as part of the well store
remove the standard smrt link validations from well and ensure the tests still pass

Tasks for moving run to smrt link versions:

[ ] create a new migration to add a belongs_to relationship for smrt link version. This will involve changing the existing field from a string
[ ] modify the run so that the attribute now defaults to the smrt link version default rather than the constant
[ ] modify the run tests to test this behaviour
[ ] Remove the smrt link versions default constant


issue86 macfix

New Localdev image fails to build on MacOS with error on groupmod
issue86 TEST ONLY
change owner of docker.sock to localdev

I believe the fix should look more like the windows fix but I'll confirm

Hey @gamerson with @ipeychev's help we confirmed that it appears your solution is the only one that will work on Mac. However before we merge I want to consolidate the solution on Lin & Win. I'll send some follow up changes once I've tested on those platforms.

I've just tested this, and I'm seeing a new msg that could be a concern
Writing Kubernetes config: could not create any of the following paths: /run/user/502/tilt-dev/tilt-default/cluster
█

@gamerson is that a REST uri or a file system path? If it's a FS is that inside the localdev-server container or on local system?

@gamerson is that a REST uri or a file system path? If it's a FS is that inside the localdev-server container or on local system?

This is a file system path, inside the localdev-server. This is from the Tiltfile resource inside of Tilt UI, so its a message from Tilt server itself.

Looks like tilt is trying to serialize out the config for connecting to the kube context and is running into permission error writing to /run/user/502. The /run path is same as /var/run in our localdev server and its owned by root.  I'm not sure why tilt things it should be writable.
One option might be to make /var/run/user recusrively owned by our localdev user instead of just  /var/run/docker.sock.

@rotty3000 I'm good with merging this.


Update languages.yml

Description
Add Extension for Solidity.
Checklist:

[x] I am associating a language with a new file extension.

[x] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?q=extension%3Asol+contract&type=Code

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/anuditnagar/assetreg/blob/master/Contracts/VehicleRegistry.sol

Sample license(s):

[x] I have included a change to the heuristics to distinguish my language from others using the same extension.

Please look for previous issues attempting the same change.


TEST: Azure DELETE

This PR adds tests for Azure Delete

PR has been updated. Reviewers, please be cautious.

CONFLICT (add/add): Merge conflict in tests/unit/api/objectReplicationMD.js
CONFLICT (add/add): Merge conflict in tests/unit/api/objectGet.js
CONFLICT (add/add): Merge conflict in tests/functional/aws-node-sdk/test/object/get.js
CONFLICT (add/add): Merge conflict in lib/data/wrapper.js
CONFLICT (add/add): Merge conflict in lib/api/objectPutCopyPart.js
CONFLICT (add/add): Merge conflict in lib/api/objectGet.js
CONFLICT (add/add): Merge conflict in lib/api/objectCopy.js
CONFLICT (add/add): Merge conflict in lib/api/completeMultipartUpload.js
CONFLICT (add/add): Merge conflict in lib/api/apiUtils/object/storeObject.js
CONFLICT (add/add): Merge conflict in lib/api/apiUtils/object/createAndStoreObject.js

For now we return error: 400 - InvalidArgument if deleting specifying a version.

As discussed, please add a test trying to delete a specific version and test that the result is no such key and then we can merge.

@ironman-machine try

Hello @nicolas2bert
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/14652' with the following env. args:
{
    "SCALITY_INTEGRATION_BRANCH": "ultron/master",
    "REPO_NAME": "S3",
    "DEFAULT_BRANCH": "master",
    "SCALITY_S3_BRANCH": "test/azure/delete"
}

:sunny: :+1: circleCI test succeeded!


Reading variables from ~/.aws/config and ~/.aws/credentials is broken

If data is present in either ~/.aws/config or ~/.aws/credentials, whatever bash magic is happening in the scripts isn't working. I enabled debug mode in the scripts by adding  #!/bin/bash -ex at the top to see what's going on, and this is what I get:
+ echo '
      ###    ########   #######  ########
     ## ##   ##     ## ##     ## ##     ##
    ##   ##  ##     ## ##     ## ##     ##
   ##     ## ##     ## ##     ## ########
   ######### ##     ## ##     ## ##
   ##     ## ##     ## ##     ## ##
   ##     ## ########   #######  ##
'

      ###    ########   #######  ########
     ## ##   ##     ## ##     ## ##     ##
    ##   ##  ##     ## ##     ## ##     ##
   ##     ## ##     ## ##     ## ########
   ######### ##     ## ##     ## ##
   ##     ## ##     ## ##     ## ##
   ##     ## ########   #######  ##

+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export MACHINE_TYPE=aws
+ MACHINE_TYPE=aws
+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export AWS_ACCESS_KEY_ID=AKIAIR4X6GNTAQLFE36Q
+ AWS_ACCESS_KEY_ID=AKIAIR4X6GNTAQLFE36Q
+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export AWS_SECRET_ACCESS_KEY=rWqNA6sq87qjusvG/vF1x/UnRmYpUDlTsQYSxiwV
+ AWS_SECRET_ACCESS_KEY=rWqNA6sq87qjusvG/vF1x/UnRmYpUDlTsQYSxiwV
+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export MACHINE_NAME=ilmarinen-adop
+ MACHINE_NAME=ilmarinen-adop
+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export AWS_VPC_ID=vpc-c303f0a7
+ AWS_VPC_ID=vpc-c303f0a7
+ getopts t:m:a:s:c:z:r:u:p: opt
+ case ${opt} in
+ export AWS_DEFAULT_REGION=eu-west-1
+ AWS_DEFAULT_REGION=eu-west-1
+ getopts t:m:a:s:c:z:r:u:p: opt
+ '[' -z aws ']'
+ CLI_COMPOSE_OPTS=
+ case ${MACHINE_TYPE} in
+ provision_aws
+ '[' -z ilmarinen-adop ']'
+ '[' -z vpc-c303f0a7 ']'
+ '[' -z ']'
+ echo 'No availability zone specified - using default [a].'
No availability zone specified - using default [a].
+ export VPC_AVAIL_ZONE=a
+ VPC_AVAIL_ZONE=a
+ '[' -f /Users/oscar.renalias/.aws/credentials ']'
+ echo 'Using default AWS credentials from ~/.aws/credentials'
Using default AWS credentials from ~/.aws/credentials
+ '[' -z AKIAIR4X6GNTAQLFE36Q ']'
++ grep -v '^\[' /Users/oscar.renalias/.aws/credentials
++ sed 's/^\(.*\)\s=\s/export \U\1=/'
+ eval aws_access_key_id = AKIAIR4X6GNTAQLFE36Q aws_secret_access_key = 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
++ aws_access_key_id = AKIAIR4X6GNTAQLFE36Q aws_secret_access_key = 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
./quickstart.sh: line 119: aws_access_key_id: command not found

The version of bash in OS X 10.11 definitely doesn't like whatever is going on.

I'm really not a fan of us trying to re-use the .aws directory as that's normally handled by some Python magic in the AWS CLI so that part might be stripped out in favour of something less inventive but not as fragile.
Also, I hope those credentials aren't active any more!

Aka redact your credentials before posting on public forums, i.e. don't
wave your secret key around in public! :-)
On 18 May 2016 12:09 a.m., "Nick Griffin"<EMAIL_ADDRESS>wrote:

I'm really not a fan of us trying to re-use the .aws directory as that's
normally handled by some Python magic in the AWS CLI so that part might be
stripped out in favour of something less inventive but not as fragile.
Also, I hope those credentials aren't active any more!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Accenture/adop-docker-compose/issues/71#issuecomment-219880930

No, that API key is no longer valid 😇

Regarding the issue, why not drop this approach and simply force everyone to expose these values as environment variables?

That's the direction I'm heading in. We had that originally and people wanted to use the credentials file but then it turns out there are some cases where people have multiple sets of credentials that it couldn't handle, and it'd also use the contents of the file even if the parameters were provided etc.
I'd happily strip it all out in favour of environment variables and drop the parameters from the script too.

Yeah, I also had an issue with the script using credentials from the file despite the fact that I was providing them as parameters.

The issue of trying to use the stored credentials even when you provide them on the command line is that you're using the bitwise operators (&, |) instead of the logical operators (&&, II) in your compound bash if tests. I'll try to fix and submit a pull request for this if I get time.


How about removing the 'full control' restrict in archwsl.json at scoop extras?

I install archwsl via scoop.
it says as followings :
Installing 'archwsl' (<IP_ADDRESS>) [64bit]
Loading Arch.zip from cache
Checking hash of Arch.zip ... ok.
Extracting Arch.zip ... done.
Linking D:\Scoop\apps\archwsl\current => D:\Scoop\apps\archwsl\<IP_ADDRESS>
Creating shim for 'Arch'.
Creating shortcut for Arch Linux (Arch.exe)
Persisting data
Running post-install script...
WARN  Full Control access to the scoop directory is necessary to install WSL distribution.
WARN  Change directory security and reinstall ArchWSL.
'archwsl' (<IP_ADDRESS>) was installed successfully!

Trying to read the archwsl.json and regardless of the 'Full Control' permission restrict, I find it works well without the restrict. just like the following steps:

in the scoop bucket dirs add a local dir like D:\Scoop\buckets\local
add a archwsl.json copy here remove the restrict of $permission

{
    "##": "'rootfs' must be a 'real' directory (at least on Windows 10 RS4)",
    "version": "<IP_ADDRESS>",
    "description": "Install ArchLinux as a WSL Instance",
    "homepage": "https://github.com/yuk7/ArchWSL",
    "license": "MIT",
    "notes": "Even when you are logging in as 'root', some operations (like service command) require Windows administrator privileges",
    "url": "https://github.com/yuk7/ArchWSL/releases/download/<IP_ADDRESS>/Arch.zip",
    "hash": "e62abb2568d59d165946ce462029ca032102ba20c7c12b34440cf9074b7de6cd",
    "post_install": [
        "warn 'DO NOT KILL THE PROCESS. Installation is still running!'",
        "Copy-Item \"$dir\\Arch.exe\" \"$persist_dir\\data\\Arch.exe\"",
        "$res = Invoke-ExternalCommand \"$persist_dir\\data\\Arch.exe\" 'install', \"$dir\\rootfs.tar.gz\"",
        "if(!$res) { error 'ArchWsl installation failed!'; return }",
        "Remove-Item \"$persist_dir\\data\\Arch.exe\" -Force",
        "Remove-Item \"$dir\\rootfs.tar.gz\" -Force"
    ],
    "uninstaller": {
        "script": [
            "if ($cmd -ne 'uninstall') { return }",
            "$res = Invoke-ExternalCommand \"$dir\\Arch.exe\" 'isregd'",
            "if(!$res) { error 'ArchWsl is not registered!'; return }",
            "Invoke-ExternalCommand \"$dir\\Arch.exe\" 'clean', '-y' | Out-Null"
        ]
    },
    "bin": "Arch.exe",
    "shortcuts": [
        [
            "Arch.exe",
            "Arch Linux"
        ]
    ],
    "persist": "data",
    "checkver": "github",
    "autoupdate": {
        "url": "https://github.com/yuk7/ArchWSL/releases/download/$version/Arch.zip"
    }
}

use scoop install local/archwsl

it works well. So maybe this permissions restrict can be removed ?

no problem, thanks.
Yes. I don't think it need administrator privileges.

Thanks for your reply. But I only test it on win10 ltsc, maybe open a issue instead.


[5.1] introduce SoftDeletes contract (and fix soft deleted parents on HasManyThrough)

The same as #8246 but additionaly introduces SoftDeletes contract in order to be able to reliably determine whether a model uses soft deletes.

There's no forcing the end-user to implement the interface when using the trait.
How can we reliably tell that it's not using soft deletes when it's not implementing the interface?

@JosephSilber You can never be sure about that, when class is not implementing the interface. It's the opposite - interface's purpose is to make sure the class does something, if it implements it.
Now, it's just the same as User implementing Authenticatable by simply adding the Authenticatable trait. I wouldn't worry about forcing anyone.
The problem, and the reason for the interface here, is exactly the same to what you are saying - you can't rely on whether a class uses the trait or not, when you  want to check if it soft deletes. Moreover, testing is much easier with interface vs. just using the trait.

I'm saying that you're relying on the user to implement the SoftDeletes contract for HasManyThrough to work properly.
Checking for the use of the SoftDeletes trait is more expensive, but more accurate.

Checking for the use of the SoftDeletes trait is more expensive, but more accurate.

We need to do that, yes. We can't rely on the interface. Please remove the interface.


Savings Plans vs. RI's

It would be super helpful if this documentation explained the differences between Reserved Instances and Savings Plans, and why a customer would choose one over another.

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

ID: 633b666b-e9a4-3a8d-f758-a646800967a7
Version Independent ID: 0947bfc8-441f-580e-0596-359d3b83380f
Content: What are Azure savings plans for compute? - Microsoft Cost Management
Content Source: articles/cost-management-billing/savings-plan/savings-plan-compute-overview.md
Service: cost-management-billing
Sub-service: reservations
GitHub Login: @bandersmsft
Microsoft Alias: banders

@christopherhouse Yes, you're right. We updated https://learn.microsoft.com/azure/cost-management-billing/savings-plan/discount-application yesterday to explain the difference between reservations and savings plans and when one is better than the other.
I hope this helps.
#please-close


avoid sending model through task

Addressing Bobby's comments in https://github.com/NVIDIA/spark-rapids-ml/pull/368#discussion_r1296465594

build

Think you have another one in the _transform_internal() method.

build


Fix: e2e tests and added documentation

Signed-off-by: Asif Sohail Mohammed<EMAIL_ADDRESS>Description

Fixed assertions in EndToEndLogMetricsTest test
Added EndToEndLogMetricsTest test to workflows
Cleaned up ParallelGrokStringSubstituteLogTest
Added documentation for EndToEndLogMetricsTest, ParallelGrokStringSubstituteLogTest

Issues Resolved
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2267 (7bb37ce) into main (c797d5c) will increase coverage by 0.07%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2267      +/-   ##
============================================
+ Coverage     93.66%   93.73%   +0.07%
- Complexity     1825     1826       +1
============================================
  Files           224      224
  Lines          5284     5284
  Branches        426      426
============================================
+ Hits           4949     4953       +4
+ Misses          233      226       -7
- Partials        102      105       +3

Impacted Files
Coverage Δ

...rwarder/discovery/AwsCloudMapPeerListProvider.java
92.85% <0.00%> (-2.39%)
:arrow_down:

...opensearch/dataprepper/pipeline/ProcessWorker.java
84.61% <0.00%> (+11.53%)
:arrow_up:

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


chore(master): release 1.1.76

:robot: I have created a release beep boop
1.1.76 (2023-09-05)
Bug Fixes

deps: update dependency laravel/framework to v10.22.0 (a041cf9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.76 :sunflower:


Error with --prod

Hey there,
Everything seems to work perfectly when I run it in debug, but once I build for prod:
ionic cordova build android --prod --release
The build process works fine but when I run the app I can't open the page in which i'm using the ionic-select-searchable, any thoughts?
I'm using lazy loading and importing the SelectSearchableModule in AppModule and the page module as well
@ionic/cli-utils  : 1.19.2
ionic (Ionic CLI) : 3.20.0

global packages:
cordova (Cordova CLI) : 7.1.0

local packages:
@ionic/app-scripts : 3.1.11
Cordova Platforms  : android 6.3.0
Ionic Framework    : ionic-angular 3.9.2

System:
Android SDK Tools : 26.1.1
Node              : v8.9.1
npm               : 5.6.0
OS                : Windows 10

Environment Variables:
ANDROID_HOME : C:\Android\sdk

Misc:
backend : pro

Using version:     "ionic-select-searchable": "^2.8.2",

Hey @eakoriakin thanks for the quick response.
All fixed now, the cause of the issue was that I didn't import the SelectSearchableComponent in the page itself and using it with the onChange function.
Used to be like this: (works in debug)
getCarNames(event){....}
Now I changed based on the docs to:
getCarNames(event: {
component: SelectSearchableComponent,
value: any
}) {....}
And it works like charm when using --prod --release
Again thanks @eakoriakin for such a great component


More inline docs needed for inline completion API

Testing #124979
There are several functions, classes, and interfaces pertaining to the inline completion API that are missing JS doc comments. I didn't have any trouble converting a CompletionProvider to an InlineCompletionProvider, but we could still have more JS doc comments.

Thanks! Will add more documentation over time.


Updates for 12.7.0

Remove changes to README and changes to samples, we don't update those during a preview.

Also in the changelog, I added an extra hyphen last time on lines 40 and 41. Please remove.


Error for latest React (15.6.1)

Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object.
"react": "^15.6.1",
"react-dom": "^15.6.1",

Problem seems to be related to this code chunk:
    return _react2.default.createElement('input', _extends({ ref: function ref(_ref) {
        return _this2.input = _ref;
      } }, props, { onFocus: _this2.onFocus, onBlur: _this2.onBlur }));

BTW, why not keeping source files in npm as well? Probably this is issue releated to compiled code only.

@ixrock Can't reproduce. Can you  make a demo?
Sources aren't included to reduce package size. Also node_modules is usually excluded from processing with babel-loader so import InputMask from 'react-input-mask/src' won't compile anyway.

Problem solved by importing package as (i'm using it with typescript):
import InputMask = require("react-input-mask/dist/react-input-mask.min.js");
previously was:
import InputMask = require("react-input-mask");
Maybe it's better to specify "main" in package.json linked to compiled minified version?

And btw, is it you maintainer for https://www.npmjs.com/package/@types/react-input-mask ?

@ixrock I'll appreciate if you'll share your project. Still want to make it work with a regular import.
About types – no i'm not a maintainer because i don't use TypeScript.

It's private company repo, so cannot share it, sorry.
If you don't use typescript maybe you won't face this problem or maybe it's webpack's specific loader problem or who knows..
If you want to make it work with regular import just define "main": "react-input-mask/dist/react-input-mask.min.js" in package.json

@ixrock The weird thing is that i've set up TypeScript environment and it just works without any changes in import path or main field. Can you try to "broke" this project? It starts a webpack server with npm run dev.
react-input-mask-test.zip

Didn't run your project yet, but can suppose it's cause I don't use babel at all.
This is how loaded ts-files in my webpack's config:
module: {
  rules: [
    {
      test: /\.tsx?$/,
      use: 'awesome-typescript-loader'
    },
  ]
}

@ixrock Still works.
https://github.com/sanniassin/react-input-mask-ts-bug

Hmm.. I found the problem and it works now for me with normal import:
import InputMask from "react-input-mask"
Recently I've updated typescript to latest version and probably something there was changed related how such form of import works:
import InputMask = require("react-input-mask");
I'm using such form of import for packages without types or for specific cases like for lodash:
import isFunction = require("lodash/isFunction"); because if I'll do import isFunction from "lodash/isFunction"; it says it has no default import and shows error..
Thanks for your help! 👍

For what its worth, I am seeing this randomly with React 15.5.4 and webpack 3.3.0
Sometimes it works, sometimes it fails. Not using typescript.
Same exact error: Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object.

@pbarbiero Whether it disappears after page reload or specific page with specific js bundle always fails?

We have code splitting, so several bundles. It appears to always fail when a second bundle tries to use it, and then it breaks it for all bundles after that.

@pbarbiero Do you use require('react-input-mask') instead of import? There is some issues with CommonJS build.

We will just stick with the workaround for now.
Do you want to reopen this issue or would you like me to open a new one?

@pbarbiero New one will be better, thanks!

This is still an issue w/ using CommonJS require() and Webpack. Tested with v1.2.2.
const InputMask = require('react-input-mask'); // React.createElement: type is invalid
const InputMask = require('react-input-mask/dist/react-input-mask.min'); // works

@sedenardi Why not use ES import syntax? Since modern versions of Webpack always use ES modules over CommonJS ones (https://github.com/webpack/webpack/issues/4742), you have to either import with
import InputMask from 'react-input-mask';

or
const InputMask = require('react-input-mask').default;

@sanniassin It's party of a fairly large project whose views haven't been cut over to ES import syntax yet.
Appending .default to the require statement doesn't work either, resulting in this error:
Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
Not sure if this makes a difference, but I'm doing server-side rendering. This error comes from the server when I render the markup.

@sedenardi I decided to get rid of ES module to avoid this kind of confusion. Going to release new version in a few days.


Added the program Arithmetic Slices (Using DP)

Hi
Kindly add the hacktoberfest label for my pull request
Thanks

Sure
I'll do it

Hi
The requested changes have been made
Kindly check


RepeatableCommands=true, need for repeatableCommandArity="0..1"

I've created simple spring boot app which takes two commands A and B, each with required option -r and enabled repeatable commands so that I can run it with arguments 'A -r 1 B -r 2'
Everything works great however when I run with arguments 'A -r 1 B -r 2 B -r 3'
Issue: my command B is executed twice with the value 3 instead of 2 and 3
Feature request: I'd like to prevent the command listed multiple times, is it possible?
Repo is here
https://github.com/sergproua/learn-picocli

You may have found a bug.
It is still unclear to me whether the bug is Spring-related (Spring may be caching the instance of CommandB so the same instance gets reused for the second B commandline argument), or whether there is an issue in the picocli parsing logic.

Feature request: I'd like to prevent the command listed multiple times, is it possible?

This may be a duplicate of https://github.com/remkop/picocli/issues/1194;
can you check if the suggested solution works for you?

Issue: my command B is executed twice with the value 3 instead of 2 and 3

You may have found a bug.
It is still unclear to me whether the bug is Spring-related (Spring may be caching the instance of CommandB so the same instance gets reused for the second B commandline argument), or whether there is an issue in the picocli parsing logic.

It is Spring @Component, Singleton scope used by default. I added below and it is working as expect.
@Component
@Scope(value = ConfigurableBeanFactory.SCOPE_PROTOTYPE)
Thanks!
Serge

Feature request: I'd like to prevent the command listed multiple times, is it possible?

This may be a duplicate of #1194; can you check if the suggested solution works for you?

Feature request: I'd like to prevent the command listed multiple times, is it possible?

This may be a duplicate of #1194; can you check if the suggested solution works for you?

Yes, #1194 solution works however it executes first command then second then throws an error. I think it is ok to close this ticket and I'll propose a better solution in that ticket.
Thanks!

duplicate of #1194


Update the base maps to include more of the Aleutian chain and NWT

In the new year, we will want to improve our base maps to include more of the Aleutian chain and NWT. The current tiles are cut off too early in lower latitude and we can update with the newest data from OSM.

Just to clarify this a little bit more, it's not the issue raised in #81 -- this is referring to the base layers themselves!

This is done!  There's some possible leftover work to rebuild the underlying styles from a currently-updated, upstream style repo -- that might clear up a few quirks with rendering (label sizes and stuff, and generally it's just a good idea to stay up to date) but we can revisit that later.


[vite] Internal server error: Cannot read properties of undefined (reading 'message')

Reporting a bug?
When starting the vite dev server, the process stops with the following message:
[vite] Internal server error: Cannot read properties of undefined (reading 'message')
  Plugin: vite-plugin-vue-i18n
  File: /home/projects/vite-plugin-vue-i18n-issue/src/i18n/locales/en.json
      at newOptions.onError (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/bundle-utils/lib/codegen.js:102:48)
      at newOptions.onError (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/bundle-utils/lib/codegen.js:102:36)
      at newOptions.onError (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/bundle-utils/lib/codegen.js:102:36)
      at emitError (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:197:13)
      at readTokenInPlaceholder (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:646:21)
      at readToken (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:776:20)
      at Object.nextToken (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:823:16)
      at parseMessage (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:1029:50)
      at parseResource (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:1096:25)
      at Object.parse (/home/projects/vite-plugin-vue-i18n-issue/node_modules/@intlify/message-compiler/dist/message-compiler.cjs.js:1111:21)

The project is using both vue-i18n and @intlify/vite-plugin-vue-i18n. The messages for the default language are provided by an external JSON file, which includes a long value, with some escaped characters inside.
For more details, please see the provided Stackblitz project.
Expected behavior
Since both the json configuration file and the plugin configuration are valid, the project should be served correctly and the string should be successfully interpolated by the vue-i18n plugin.
Reproduction

Run this Stackblitz project https://stackblitz.com/edit/vite-plugin-vue-i18n-issue?file=vite.config.json

Issue Package
vite-plugin-vue-i18n
System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (4) x64 Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz
    Memory: 5.15 GB / 15.90 GB
  Binaries:
    Node: 16.13.2 - C:\Program Files\nodejs\node.EXE
    npm: 8.1.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 103.0.5060.114
    Edge: Spartan (44.19041.1266.0), Chromium (103.0.1264.62)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @intlify/vite-plugin-vue-i18n: ^5.0.0 => 5.0.0
    vite: ^3.0.1 => 3.0.1
    vue: ^3.2.37 => 3.2.37
    vue-i18n: ^9.1.10 => 9.1.10

Screenshot

Additional context
I tried editing the JSON configuration file, shortening the longest string and removing some escaped characters. It seems to be working after deleting enough characters, but I could not pinpoint the exact source of the issue.
Validations

[X] Read the Contributing Guidelines.
[X] Read the README
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.

I also encountered this problem, and the reason seems to be because of the @ symbol

Thank you for your reporting!
I've checked your repro codes!
I've found out there are special characters in your locale resources.
https://vue-i18n.intlify.dev/guide/essentials/syntax.html#special-characters
you need to avoid special characters by using literal interpolation.


Switch children should only be Routes

Update docs to state that the elements passed to a <Switch> should be <Route>s. I used "should" instead of "must" because technically any component with the correct props will work.

Yeah, that reads a little bit clearer. Thanks!


Add nosystemd build tag

This allows building the exporter without systemd headers, eg:
env CGO_ENABLED=0 go build -tags nosystemd

Closes #11

Nice work!
I tested the builds only, since we are not using the systemd functionality.
We still need to update the readme after this update.


Add option to dromedary.py to generate Opam switch and install packages

Add the possibility to create a new Opam switch at a given path or with a given name, setting its compiler and install packages in this newly created switch. This Opam switch's packages are used to generate the BUCK file.
Example configuration:
{
    "name": "./third-party",
    "compiler": "ocaml-variants",
    "packages": [
        "menhirLib",
        "sedlex=3.2",
        "alcotest>=1.7.0"
    ]
}

Thank you for the PR.
I need to think about it a bit, but overall I think it is a good idea. I'll get back to you soon

Sorry I got sidetracked this week.
I think it's a good idea. Will try to review asap

I've addressed everything except for the List[Any] and the printing of messages to stdout.
Also added a flag to always answer question with "yes" to opam install.

Many thanks for the update @Release-Candidate ! Will stamp this after @vsiles import the latest change again.

Thank you @grievejia and especially @vsiles for helping me adding dromedary.py to ocaml-scripts!


[Feature] Possibility to define multiple success status. Add HTTPMultiResponder

[There was a discussion about having the ability to define multiple success status. As it was proposed in discussion and it seemed to possibly be forgotten, I'll add this here as an issue.]
It's easy to specify multiple error statuses with u.SetExpectedErrors(...), and I can override the "success" status with nethttp.SuccessStatus(http.StatusCreated) when creating the handler. But: my API needs to return either 200 or 201 status based on some logic, with essentially the same JSON response.
I cannot find a way to do that nicely: I don't want to override the whole response serialization, and I don't think treating 200 or 201 as an "error" is the right approach. Any ideas?
Originally posted by @haimgel in https://github.com/swaggest/rest/discussions/153

Oh, sorry, I only searched for HTTPMultiResponder and couldn't find a PR. Thx for your outstanding work! <3


[Table] In-line table editing, full table

PENDING - Will fill in details.  Creating as placeholder for now.

Switching table to edit mode: https://github.ibm.com/Watson-IoT/pal-tracking/issues/20
Data picker & Numeric spinner in cells: https://github.ibm.com/Watson-IoT/pal-tracking/issues/6
Validation per cell change with error state indicator: https://github.ibm.com/Watson-IoT/pal-tracking/issues/32
In Edit mode, show "Notes" button for adding notes to change log: https://github.ibm.com/Watson-IoT/pal-tracking/issues/43

This item has been delivered. Closing.


Syntax highlighting issue when using positional args in class instantiation

There is a minor syntax highlighting issue where the left parenthesis has the wrong color when instantiating a class with a number.

@saolsen blame says you've been here. Assigning it to you, feel free to reassign.

Hm, I'm not able to reproduce this... This is using vscode?


Preserve Dovecot /etc/dovecot/dovecot.conf configuration file

Is your feature request related to a problem? Please describe.
On the Dovecot reinstallation during the conversion, the /etc/dovecot/dovecot.conf is overwritten with a default file. Normally it is advised not to have any custom tweaks in that file, but this does not work reliably for the "auth_mechanisms" setting. Removal of digest-md5 from that line is crucial, as otherwise some mail apps only login after a second or third attempt. This triggers Fail2Ban (one false login per login attempt), and this again blocks the user. For that reason, users have asked to have digest-md5 removed by default, e.g. in https://plesk.uservoice.com/forums/184549-feature-suggestions/suggestions/41190802-remove-digest-md5-from-dovecot-and-postfix-configs. Unfortunately this hasn't made it into the Plesk default setup yet, so the only way to reliably remove digest-md5 is to remove it from the dovecot.conf file. It is not sufficient to create an additional configuration file for that (tests had shown it just won't work reliably). This means that users depend on preservation of their /etc/dovecot/dovecot.conf file.
Describe the solution you'd like
Save /etc/dovecot/dovecot.conf in phase 1 of conversion, restore it in the last phase.
Describe alternatives you've considered
Save it manually, restore it manually.

Should be fixed by #207, #208. Please find attached centos2alma - centos2alma.zip


[BUG] Crash on new world creation when paired with Fisk's Superheroes (1.7.10)

Version of Advanced Rocketry

1.7.10 - 1.4.0-32

Have you verified this is an issue in the latest unstable build

This is the latest Alpha build available on Curseforge

Version of LibVulpes

1.7.10 - 0.2.8-37-universal

Version of Minecraft

1.7.10

Does this occur without other mods installed

The problem arises in a profile with only these 3 mods: LV, Ar, and FiskHeroes

Crash report

PasteBin

Description of the problem

Make a Minecraft 1.7.10 Forge <IP_ADDRESS>4 profile and add LibVulpes, Advanced Rocketry, and Fisk's Superheroes;
Start the game and make a New World;
See the game crash.

Why not use just Advanced Rocketry?

Why not use just Advanced Rocketry?

Because I'd like to make a modpack where one is able to travel space and be a superhero?
besides, the guys over at the other mod commented the exact opposite, why not use just their mod? kinda funny


Add placement left to paper

https://vczb.github.io/sagu-ui/?path=/story/paper--default

Hey, how's it going? I'm trying to submit a PR for this issue for Hacktoberfest but I'm having some issues pushing my branch. I've tried both HTTPS and SSH and still no luck. Anything I'm missing here?

Hey, how's it going? I'm trying to submit a PR for this issue for Hacktoberfest but I'm having some issues pushing my branch. I've tried both HTTPS and SSH and still no luck. Anything I'm missing here?

Hi @anechol , so strange, this repo doesn't have anything different. Can you please share the error message

So with https, I'm getting the following error:
fatal: unable to access 'https://github.com/vczb/sagu-ui.git/': The requested URL returned error: 403
I have a personal access token created and have been using that. I have also tried SSH. I have a key added to my account and the ssh agent and all of that and still getting this error:
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I might be missing something here, but I am able to push to own repos just fine. 🤔 I am using WSL2 if that matters, but I haven't had this issue before.

did you create a new branch ?
git checkout -b my-branch-name

So with https, I'm getting the following error:
fatal: unable to access 'https://github.com/vczb/sagu-ui.git/': The requested URL returned error: 403
I have a personal access token created and have been using that. I have also tried SSH. I have a key added to my account and the ssh agent and all of that and still getting this error:
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I might be missing something here, but I am able to push to own repos just fine. thinking I am using WSL2 if that matters, but I haven't had this issue before.

you need fork this repository and create a new branch to submit a PR
guides:

https://opensource.com/article/19/7/create-pull-request-github
https://www.freecodecamp.org/news/how-to-make-your-first-pull-request-on-github-3/

AH I missed the forking step haha. Thanks, I'll get a PR up later and see your thoughts on it.

AH I missed the forking step haha. Thanks, I'll get a PR up later and see your thoughts on it.

You are welcome, any questions I'm available


RAILS_ROOT is gone in rails 3.1.0

attachment_fu still makes use of RAILS_ROOT in multiple places.  This causes the rails app to fail on startup.

I think the problem is about RAILS_ROOT. You have to change the line 4 on attachment_fu.rb file in the plugin folder from RAILS_ROOT to Rails.root.

Yes, I know that.  It would be nice if the maintainer did the same so that every single user didn't have to :-/


[summer2021]Top Performance MySQL Protocol Server in Rust

[ ] add mysql server
[ ] integrate into key path

[ ] primary data types adaption
[ ] concurrent safety for existed CH compatible server

[ ] functionalities enablement

[ ] auth, interaction with #126
[ ] metrics, interaction with #165
[ ] prepared statements
[ ] compression
[ ] encryption

[ ] unit tests and integ tests
[ ] benchmarks
[ ] docs

What is the minimum version of MySQL/MariaDB we plan to support?  I can try to initial a demo to run it.

This is a topic of summer 2021. If you are interesting, I invite you become a co-mentor for the student of this topic.
I have committed a crate to be as the protocol server. This is a fixed version for a popular mysql protocol server.


property access callback

A couple of queries:

Is it possible to get callback whenever a property is accessed on an object even if the property is not defined on the object? This will help with lazy initialization and also with memory issues. The property accessed if valid, will be added as a property to the object in the c# code.

Can the API "JsCreateExternalObject(
In_opt void *data,
In_opt JsFinalizeCallback finalizeCallback,
Out JsValueRef *object
);"
hold a c# object as the "data"? Currently we are holding a int as data which maps to our c# object.

Proxy (a new feature in ECMAScript 2015) should be able to solve this problem nicely.
Yes you can hold c# object as data, but remember to put a hold on the object otherwise it may get garbage collected.

Thanks for the reply,
I understand we can create proxy in javascript code but is there a way to create it programatically? I did not find the key word "proxy" in the javascript reference(https://msdn.microsoft.com/en-us/library/dn249536(v=vs.94).aspx )

Thanks for clarifying :). Unfortunately as for now there is no API that directly creates proxies. A kind of hacky way I guess is to have a script that creates a proxy you want and then run it with JsRunScript, which returns the resulting proxy object to C# environment.


Song rows aren't droppable in some positions

I'm not sure if it's always been there, but in Chrome 105.0.5195.112, I'm seeing annoying behavior when dragging rows in song-table to reorder them where a red crossed-out circle is briefly visible for a single Y coordinate between each pair of rows. If I happen to end the drag there, the drop doesn't happen.
At first I thought that the #drag-target indicator div was responsible, but I see the issue when I set pointer-events: none on it or even leave it at display: none. Getting rid of the table's line-height: 1.2 doesn't help either.
I should check if this happens in Firefox, and also when using a low-res display -- this device has a pixel ratio of 1.600000023841858, which has triggered Chrome bugs in the past.

I think that this happens whenever the pointer enters an element that doesn't cancel the dragenter event. The spec says that all DnD events bubble, and dragenter seems to bubble when I make a minimal test case, but for some reason my dragenter handler on document.body doesn't seem to be getting called here.
Adding another dragenter handler on the table element seems to at least fix the issue between song rows. I tried creating a fullscreen overlay element to handle all of the DnD events, but it wasn't receiving any events for some reason.


Revert "Revert "Makefile: Compile zlib .c files with DMD via importC""

Reverts dlang/phobos#8870
The regression for this was fixed by https://github.com/dlang/dmd/pull/15975

Nice, thank you and Rainer!

@thewilsonator you may need to open a new PR with a branch on your fork of the repo.

https://github.com/dlang/phobos/pull/8873


Add Quick Settings tile to change wallpaper

I have set up the wallpaper changer with a timer. Sometimes, I'm not fond of the chosen wallpaper and I would like to have a quick settings tile to change it.
Documentation on Quick-Settings tiles: https://developer.android.com/develop/ui/views/quicksettings-tiles

It's added

I've tested it and it works like a charm. Thanks


Fixing build break with upstream lldb (SwiftASTContext::GetFloatTypeSemantics)

SwiftASTContext::GetFloatTypeSemantics not implemented was causing build break.

@swift-ci please test


chore: postman

description
This PR adds Postman to the list of GUI clients, since gRPC is now fully supported by Postman, alongside other protocols like REST (of course), GraphQL, Websockets, and so forth.
context
Hello! I'm new at Postman, and we actually fully support gRPC and we'd love to guide any new developers using gRPC for the first time to check it out.
Totally un-opinionated on where we add the link, but I added it at the top since I do ultimately think it's a great resource to use for someone testing out a gRPC service (or services).

Thanks @bojand!


Update About page i18n

Also fixed some images
Closes #3419

Done

Thanks!
By the way, I'm working on creating a CodeCombat Cookbook, and this morning I added a recipe for Translating page strings. I added the guideline to mark strings with DNT to avoid this problem in the future. Also, can you look over the recipe and let me know if I missed anything for good translation rules?

I think everything is covered. It should be great to help new people that don't know how to start :+1:


refactor(fromEvent): clearfy the type definition of Observable.fromEvent()

This makes the type definition more compatible with RxJS v4

I updated:

Remove the change splitting `setupSubscription()
Rename type names of EventEmitter kinds

@kwonoj
I derived the naming convention EventTargetLike from TypeScript's PromiseLike<T> but I feel EventTargetKind may be a bit more better.
How about do you feel?

Both of them seems fine to me. Wish there's kind of dictionary for these kind of naming ;) Though I'm bit leaning towards to latter (EventTargetKind), if there's existing reference especially from TypeScript, that may be better choice.
BUT since I'm not good at English enough, consider it as personal opinion and gladly welcome any suggestions. :) Let's wait for some time if any opinion / suggestions around.

Kind in a type system context means something else. I think Like makes more sense here.

updated

@saneyuki, I'm seeing one of commit is marked as fixup (7f3b468), is it expected to be squashed?

@kwonoj squashed!

Merged with 6f53dbf, thanks @saneyuki .


Improve language of email notification for removing alternate email address

This sentence should be changed since it uses first person and has a comma splice.

I just wanted to let you know, the email address<EMAIL_ADDRESS>has been removed from your account.

Better version (removes the beginning clause)

The email address<EMAIL_ADDRESS>has been removed from your account.

Screenshot

But the change is not as polite, yes? :smiling_imp:

@lbanner It seems uncharacteristic of the OSF emails to have first person narration. Robots don't need to be polite!

I like the suggested change. Ping @jlcohoon for a final "go" or "nogo" on this.

I like it. There should be a second email that pairs with this (the alternate and primary get this email). Both should be updated.


changes to be seen on checkboxes in filter menu

Fix #82
I'm not sure if these changes will be seen on the public and internal sites so I have to still check that.

Superseded by #105


[WIP] PoC: Serialize todos as git notes

WIP - Do Not Merge

Draft PoC to address #285

Notes

git notes are not cloned/pulled/pushed automatically. this means that either the user has to manually manage the notes refs, which is probably a non-starter, or else the utils would manage it through installing git hooks or explicit exec calls. The latter 2 options may be unexpected side-effects for a broader user-base, but could be messaged when installing the utils.
notes show up in log messages, and don't appear to be truncated reasonably. To make the log easier to read and obscure todos, we can use a ref to refs/notes/todos instead of the default refs/notes/commits.
i don't think git itself imposes a size limit, though GitHub specifies 100MB max file size, which presumably would apply to the notes object.

discussing more, the requirement for git notes to be managed independently of the corresponding branch is a non-starter. we shouldn't install git hook or auto-push since that'd be super surprising to developers, and we shouldn't require more work to manage a separate todo ref.


Taxon constraint:  transcytosis

Please provide as much information as you can:

GO term ID label:
GO:0045056 transcytosis

Seems to be multicellular phenomena?
Macromolecules are captured in vesicles on one side of the cell, drawn across the cell, and ejected on the other side. Wikipedia

Request to add a taxon constraint:

**only in taxon: **

**never in taxon: **
never in yeast

(picking up a single annotation from PAINT)

Supporting evidence if available (e.g PMID):

Added parent GO:0032501 ! multicellular organismal process

Fixed taxon constraint in #22336


Leverage new Godot 4.0 features

Godot 4.0 introduces many new features which should let us delete a lot of custom code, or improve performance.

[ ] #1380
[ ] #1791
[ ] #1941
[ ] #1942
[ ] #1943
[ ] #1944
[x] #1945
[ ] #1946
[ ] #1947
[ ] #1948
[ ] #1949
[ ] #1950

Additionally, there are many new GDScript features and changes changes. I have not created stories for these changes yet, because some of these changes might break stuff which needs to be fixed during #1951 (such as the Super keyword), and some of these changes might be handled automatically by the upgrade tool (such as the property syntax).
After #1951 is complete, we should create a handful of new stories related to updating out GDScript symtax.

Cancelling the Godot 4 migration due to critical missing features. More information available on the godot-4-transition page in the wiki.


Add polestar filter to crew retrieval

This PR allows users to filter/disable their owned polestars, to stop them from being used in the search for viable combos.
The added button shows the total number of owned polestars and how many of them are currently used to find retrievable crew. Upon clicking, a modal opens that allows users to disable polestars:

There's also a new .quantity property of owned polestars that so far has no use. If this PR is accepted, I'll eventually add a filter option to only use polestars that have duplicates or with at least x quantity.

Ooooooh. Does it show okay on mobile?

It should. I haven't tested it on a mobile device, but here's what the responsive mode in the browser's dev tools make it look like (depending on the resolution/display size). The modal content can be scrolled:


slow in browser!

Hi,
I've just found a playing-slow issue, which might be very important.
The thing is, when I play a rtsp media with the flash debuger, everything is all right; but when I play the same media within a browser, for example, IE, firefox, chrome, etc., the video showing is not identically as realtime as playing in the debuger. Sometimes it showed a slow-fast-fast rhythm, while sometimes it could be freezing for half a second. And no error info showed at all.
Both the player client and rtsp server are running on Windows.
Is there anyone encountered the same problem?
I wish noseglid could spend some time taking care of this issue.
Thanks very much!

hello, xinlingyun, do you mind share me your play rtsp playing demo ?  we don't know how to build play client and rtsp server, thanks


Manifold calculating code not always providing circle manifolds for corner collisions

As MelvMay pointed out in issue #428 , some shape-shape type collisions with corners don't generate circles typed manifolds. This becomes more of a problem for those shapes that don't have rounded corner collision handling as they have larger m_radius settings. As Erin's code stands as of commit 374664b (from June 19, 2016), only circle-circle and circle-edge type collisions support rounded corner collisions.

This capability is now in my dev branch (prior to commit dfd51ec). The following screenshot is an example of it in my dev branch GUI Testbed with visual highlighting of what's involved:

This picture shows two polygons - a triangle (Shape A) and a rectangle (Shape B) - colliding at each others vertical corners. The curved outlines surrounding both polygons show their "skins" with radiuses much larger than the default in order to highlight the effect. The white circles encircle the contact points generated for the e_circles type contact manifold (where their radii are half the vertex radiuses of 0.8 and 1.6 respectively). The blueish green line shows the simplex generated for calculating their distances. The red dot shows the midpoint of the aforementioned simplex line.
Algorithmically speaking, I achieved this solution essentially by removing the addition of the skin radius total from the incident edge clipping and then checked for the vertices being within that total radius of each other when edge collision wasn't detected. If the vertices were within the total radius of each other, then a 'e_circles' type manifold is returned from the manifold calculating code (instead of the face type manifold that would have formerly been returned).

Feature request: rounded polygonal shapes

Rounded polygons are available in version 3.0


chore: update ydh email domain

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
479

Relevant Lines:
479

💛  - Coveralls


feat: artifacts whitelist

in order to be compatible with hardhat newer than v2.8.2, we need to add artifacts whitelist after generating abi files from fe programs

I don't know what an artifacts whitelist is or does. Can you point me in the right direction?

@with-heart sorry for the confusion, the direction is just the link I posted on the first comment, the big map is that

after hardhat v2.8.x the compile task will eventually execute an subtask to remove all the obsolete artifacts files https://github.com/NomicFoundation/hardhat/blob/master/packages/hardhat-core/src/builtin-tasks/compile.ts#L1401, obsolete means do not correspond to the current solidity files, for our plugin, the source file is Fe contract files, so all the artifacts files generated by this plugin will automatically removed after the compile task ran out.
to solve this, there is a whitelist for obsolete artifacts for skipping the remove process https://github.com/NomicFoundation/hardhat/blob/master/packages/hardhat-core/src/internal/artifacts.ts#L198
in order to add our artifacts files to the whitelist, we simply call addValidArtifacts for the rescue https://github.com/NomicFoundation/hardhat/blob/master/packages/hardhat-core/src/internal/artifacts.ts#L51


Pretty ounces

Use 30ml as one ounce: (fractions of) 30 are the increment used by IBA,
so this does away with weird numbers
Format floats as fractions: 0.5 -> ½
Add utility to handle plurals

This changes the current behavior:

To:

Note that fractions are also applied to other floats:

Unfortunately fractions are a bit hard to read. We could also opt to do 1/2. Or make it optional.
Finally: I’m bad at Elm. The code isn’t great.

Awesome! Your ramp-up on Elm is impressive.
Okay, so font-wise! It looks like… Google Fonts aren't loading for you, and you're seeing Menlo? IBM Plex has pretty decent fractions:

Interesting, 🤔 I see a difference in Chrome and Safari in the fonts. The CSS loads Sans:
https://github.com/tmcw/flair/blob/f8057b522e00438069af7c666719c8c73276761b/front/public/index.html#L9
but references Mono:
https://github.com/tmcw/flair/blob/622119d8a95655d151cac73dfb47ef0181c5f5f1/front/src/Main.elm#L1170
Maybe you have it installed locally?


Add Foreign Keys to the tables

Foreign Keys are commented, what's the reason for this? (it can be very handy when using graphql tools like Hasura etc.)

I don't remember why, I think it was because if we have FKs then we need to insert things in order, I think you can just add them after importing, did you try that?

Wait indexes already are meant to be run afterwards. They are commented because some entries don't respect them. Feel free to tinker around with any index combination you like and submit a PR!


[MERGE ONLY AFTER #375] Add PROG-INITIAL-REWIRING-HEURISTIC

Per ecpeterson's comments in #361, update COMPILER-HOOK to call a new function, PROG-INITIAL-REWIRING-HEURISTIC, to try and determine whether a :NAIVE initial rewiring strategy is likely to succeed, and otherwise fallback to *INITIAL-REWIRING-DEFAULT-TYPE*, which now defaults to :PARTIAL, rather than :NAIVE.
 The idea is that these QPU files can be used in testing as representative of real Aspen chipspecs.
Additionally, add a (very basic) chip-reader test and the stub of a regression test for #378.
Fixes #361
Note that the failing test in QUILC-TESTS is due to #375; therefore, this PR should not be merged until after #375.

I also tested these changes against the Python/pyquil example programs provided by kylegulshen and msohaibalam in #361, all three of which now report success rate of 100% if no INITIAL_REWIRING pragma is provided.

"32 files changed" had me briefly spooked!

Heh. I could always pair down the set of Aspen chip files if we feel like checking them all in is overkill. I opted to keep them "just in case"

I might flip-flop on this advice: I think chip-reader is doing the right
thing with just omitting the dead higher-order object. If we really
supported this, we’d have to pepper the addresser (at least—perhaps the
whole codebase) with hardware-object-dead?s.
At any rate, it is verboten to be a live object with dead subobjects—so if
we’re hewing to what the reader is doing, you could still rewrite the
predicate as “missing or (order 0 and dead)”.
On Thu, Aug 15, 2019 at 5:25 PM appleby<EMAIL_ADDRESS>wrote:

@appleby commented on this pull request.
In src/addresser/initial-rewiring.lisp
https://github.com/rigetti/quilc/pull/380#discussion_r314505641:

(loop :for inst :across (parsed-program-executable-code parsed-prog) :do
(cond
((typep inst 'pragma)
(when (typep inst 'pragma-initial-rewiring)
(return-from prog-rewiring-pragma (pragma-rewiring-type inst))))
(t

  (return))))

;; If we didn't find anything, no instructions, nothing!, then just
;; return the default.
initial-rewiring-default-type)

  (return-from prog-rewiring-pragma nil)))))

+(defun %naively-applicable (gate-app chip-spec

                       &aux (qubit-indices (application-qubit-indices gate-app)))

"Return true if the given GATE-APP's arguments correspond to a valid HARDWARE-OBJECT in CHIP-SPEC."
(and (notany (a:curry #'chip-spec-qubit-dead? chip-spec) qubit-indices)

  (lookup-hardware-object-by-qubits chip-spec qubit-indices)))

Done. Note that I dropped the (notany (a:curry #'chip-spec-qubit-dead?
chip-spec) qubit-indices) altogether. Do we want to retain that to guard
against cases where the chipspec has an "alive" order N>0 object but one or
more of it's qubits are dead? Or is that sufficiently verboten that it's
not worth checking?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rigetti/quilc/pull/380?email_source=notifications&email_token=AHFKLFNS6JCJVCMXXLUAHETQEXCVZA5CNFSM4IKM55SKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCBXINHI#discussion_r314505641,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFKLFIYQMZWHQBNEMOUTDTQEXCVZANCNFSM4IKM55SA
.

I might flip-flop on this advice: I think chip-reader is doing the right thing with just omitting the dead higher-order object. If we really supported this, we’d have to pepper the addresser (at least—perhaps the whole codebase) with hardware-object-dead?s. At any rate, it is verboten to be a live object with dead subobjects—so if we’re hewing to what the reader is doing, you could still rewrite the predicate as “missing or (order 0 and dead)”.

Sounds reasonable. I switch back to that then, but only check for dead order-0 objects.

I feel good about merging this if you're all set.

I feel good about merging this if you're all set.

Yep, good to go.


No CSS or JS files running on windows

Hi, I'm trying to run this on windows via this tutorial
https://drizin.io/installing-gitea-git-server-developed-in-go-in-windows/
I get the site up and running just fine, but it's getting a 404 when it tries to find the index.css.  What am I missing here?

That blog post is out of date. You have a few options,

download a pre-compiled version: https://dl.gitea.io/gitea/1.12.2
use a package managed version: https://chocolatey.org/packages/gitea
use nodejs and build the js/css your self (here are directions on how to build from source)


feat: deploy fetch missing dependencies

What does this PR do?
Explain what the PR does
Testing
If relevant, explain how to test the change locally

Went a different approach here.


Observation import to postgresql is much slower than other resource types

Even though each observation resource is small in size, but it's the slowest in the bulkdata import tests for Postgresql.
Compared the imported data, and seems only the composite table of observation is not empty and filled with mostly(and a lot) null values, we should investigate if this does have a big impact to the performance and find solution to improve this.

There have been significant improvements to the Schema.  When #1366  is close, this should be re-evaulated and closed if still an issue.

I moved to the Icebox


Remove duplidated day in bootstrap-datepicker.kh.js

File name should be .km rather than .kh as in https://github.com/svenfuchs/rails-i18n/blob/master/rails/locale/km.yml

Thanks for the PR, can you add a new translation file with the correct language code and add a deprecation message in the incorrect language code file? See bootstrap-datepicker.kr.js for an example

Thank you very much for your quick response. Yes, I have added deprecated message in the .kh, and create a new bootstrap-datepicker.km.js. Please kindly review it :)

@acrobat My apology for annoying, but just a following up if there is anything else you need in order to for this PR to be merged?

Nope, just lost track of the PR. Thanks @somethvictory!

Thank you very much :)


Writes 0xffffff00 to APPROTECT instead of flashing a bin file

OpenOCD reset required for the nRF52 to read in UICR registers for any changes to take effect.  Or you can power cycle the nRF52.
Shutting down and powering off your Raspberry Pi assumes you are powering the nRF52 from the Raspberry Pi and there is not other supply to the nRF52.

Tested OK, flash protect works great, sorry it took so long. Thanks so much! :-)


Elastic Search plugins support

Hello, I am wondering if Elassandra will support the ecosystem of ES plugins, in specific the watcher plugin for sending alerts?

Hi,
It supports plugins, but, as far as i known, watcher is a commercial plugin that require a licence from Elasticsearch.
You need to trigger some events ?
Vincent.

Le 11 oct. 2016 à 22:16, somecallmemike<EMAIL_ADDRESS>a écrit :
Hello, I am wondering if Elassandra will support the ecosystem of ES plugins, in specific the watcher plugin for sending alerts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/vroyer/elassandra/issues/51, or mute the thread https://github.com/notifications/unsubscribe-auth/AJzHmdu4s1G9MSdqEYMi7msuCmZNl-tiks5qy-6igaJpZM4KUDpZ.

Vincent,
Yes we are interested in using ES for SEIM event alerting.  ES is a perfect system for storing indexed data and especially retrieving data based on grok processing that can identify trends.

@somecallmemike you might want to give sentinl a try, it should work great with Elassandra as it run within the Kibana engine


Implement Mixer-E2E test framework Mixer Client library -> adapters and back

… inspects data on adapters.
-Implemented a spy adapter. Ideally it can completely auto generated.
-Added single sample test to demonstrate how to use the spy adapter and helper functions.
Next step: Add more test : More scenarios * All template variety.
Release note:

This change is

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
a604db63926a2654761470cdfb2a19e96ddd8d09
link
/test mixer-presubmit

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

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
020c268ea2ec4c6a18ec00d3ccf5cd1828d4a985
link
/test mixer-presubmit

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

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
59f396af375f7e0fdb7c47392d0c43cdfa89814e
link
/test mixer-presubmit

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

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
164cc554b1f6821aa3e4c0d1297fc6a0038d0cfb
link
/test mixer-presubmit

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

@guptasu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
5a3d5a74a553e3639bbf834e46a8c7de8959d3f9
link
/test mixer-presubmit

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

abandoning this PR for : https://github.com/istio/mixer/pull/1176


Misc. Quality of Life Improvements Pt 2

upon startup, report if everything is good
miner immediately start, don't wait for a new block
The miner should notify the user when factomd is not synced, and therefore mining cannot be started. Currently, the miner just sits, and does nothing.

could you specify what "everything is good" means?

@WhoSoup I'll edit the original description with it.


fix build ios project name

There is an error building the project in ios.
the cause is that if in the config.xml the project name tag is placed like this.

the following error is generated

This is due to the characters new line \ n, return \ r and spaces before and after the project name.
Partial solution: not use these characters in the name del projecto.
Definitive solution with regular expressions: replace (/ [\ r \ n] + / gm, '') .replace (/ (^ [\ s] + | [\ s] + $) / g, '')

Thanks @Leo1305Pineda this looks like a good change, we'll make sure it gets in.

@adrianyg7
Was this included in another PR?

@echo-branch no
@Leo1305Pineda how did you get a config.xml file containing new-lines/return/spaces on name tag? Neither Cordova, PhoneGap or Ionic cli will produce such thing.

The framework does not generate the spaces, the return or the new line.
This happens if the developer is the one who unintentionally causes them,
will not allow compilation and the console does not provide more details
about the reason for the problem.
El jue., 9 jul. 2020 a las 20:02, Adrián Márquez<EMAIL_ADDRESS>escribió:

@echo-branch https://github.com/echo-branch no
@Leo1305Pineda https://github.com/Leo1305Pineda how did you get a
config.xml file containing new-lines/return/spaces on name tag? Neither
Cordova, PhoneGap or Ionic cli will produce such thing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking-attribution/pull/640#issuecomment-656408151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBJQPYWDG4VHIBGONZWAJTR2ZLAZANCNFSM4LFAFOHA
.


[Feature]: Dynamic docs with code animation by codehike or snappify or animate-code

Package Scope
etc
Description
https://codehike.org/
https://snappify.com/editor
https://github.com/onurkerimov/animate-code
Possible Solution
Like below gif or React code

etc.
No response

effect-ts:

It's first POC...! By this way, when first enter the main page, I want to convince anyone want to use suspensive intuitively and immediately why they should use Suspensive with just one page that includes a flowing animation.


Converter USD to EUR

Student, group: Nailya Zagrutdinova, 3530901/90201
URL: https://github.com/nenailya/Converter
Git approach: git-flow

`+