Promote the experimental conformance test suite to be the standard one

What would you like to be added:
The experimental conformance test suite is going to be promoted to standard. In order to do so, we need to:

remove all the experimental keywords used in the suite code
replace the old standard suite with the new one

/assign


Support React Native 0.62

Support React Native 0.62 & deprecate react-native WebView (fixed #1280)
Summary
I created this PR to support React Native 0.62, as it doesn't anymore have WebView - so this community-driven WebView is a real must.
Test Plan
We have tested this WebView on our React Native application, and also the example application has been updated.
Tested with iOS

Tested with Android

Tested with Expo SDK
https://github.com/jussikinnula/react-native-webview-expo-test (see also branches)
What's required for testing (prerequisites)?
yarn && yarn start:ios

In external projects, one can use npm i github:jussikinnula/react-native-webview#build or npm i github:jussikinnula/react-native-webview#build2 (the first one doesn't support Expo, second one has that fixed as well).
What are the steps to reproduce (after prerequisites)?
N/A
Compatibility

OS
Implemented

iOS
✅

Android
✅

Expo iOS
✅

Expo Android
✅

Expo Web
✅

Note! Expo Web doesn't actually support WebView, this new implementation works similarly as it has worked earlier by displaying a box with red border.
Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[X] I updated the typed files (TS and Flow)
[X] I added a sample use of the API in the example project (example/App.js)

I will remove "WIP" once necessary checklist steps have been taken in account. This pull request is merely added now to see work in progress by others in community, as well as possibility to test it (e.g. npm i github:jussikinnula/react-native-webview#build).

For testing Android, we need to wait until https://github.com/react-native-community/cli/issues/1159 gets fixed.

We have also tested the changed WebView on our app with real devices (both iOS and Android) with no issues.

I would say that this patch is nearing finishing. The only questions I have:

How about the versioning, do we want to add it on top of this patch or done separate (there's no breaking change, even when we provide "default" WebView for Expo SDK (or other "unsupported" platforms) instead of exposing "WebView" from react-native (which was removed).
What do you think of documentation, should React Native 0.62 upgrade be mentioned still in "breaking changes" list?
Do we want to wait until https://github.com/react-native-community/cli/issues/1159 is released before merging this? The only issue is with this project's Android example, which needs it in order to work. The fix is already merged, but not yet released.

@kaiguo, could you check why the Windows CI is failing? I guess the used Windows React Native doesn't work with React Native 0.62?

@kaiguo, could you check why the Windows CI is failing? I guess the used Windows React Native doesn't work with React Native 0.62?

##[error]     9>d:\a\react-native-webview\react-native-webview\node_modules\react-native-windows\reactcommon\build\x64\release\reactcommon\react-native-patched\reactcommon\yoga\yoga\bitutils.h(48): error C2220: warning treated as error - no 'object' file generated (compiling source file D:\a\react-native-webview\react-native-webview\node_modules\react-native-windows\ReactCommon\build\x64\Release\ReactCommon\react-native-patched\ReactCommon\yoga\yoga\Utils.cpp) [D:\a\react-native-webview\react-native-webview\node_modules\react-native-windows\ReactCommon\ReactCommon.vcxproj]

@kaiguo Kai Guo FTE, could you check why the Windows CI is failing? I guess the used Windows React Native doesn't work with React Native 0.62?

@jussikinnula yeah there were some breaking changes on react-native-windows side, I'll open a PR to upgrade Windows to 62.

@jussikinnula I opened a PR against your branch to upgrade windows to 62, https://github.com/jussikinnula/react-native-webview/pull/1, feel free to merge that in if everything looks good to you : )

@jussikinnula I opened a PR against your branch to upgrade windows to 62, jussikinnula#1, feel free to merge that in if everything looks good to you : )

Thanks, it looked good :)

@jussikinnula  how to install your MR to my apps?

@jussikinnula how to install your MR to my apps?

We are using currently npm i github:jussikinnula/react-native-webview#build version, it doesn't have the newest changes I rebased a moment ago. I would consider it proof-of-concept, so you can test your software with it but you should wait until this PR gets merged and use official build instead.

@jussikinnula (nice Finnish name there btw) would you mind resolving conflicts on this? I’ll get it merged next release.


Kong cluster

Summary
Will machines set up as a kong cluster distribute load equally? Is there any way to configure the load distribution?
SUMMARY_GOES_HERE
Steps To Reproduce

Additional Details & Logs

Kong version ($ kong version)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (registered APIs/Plugins & configuration file)
Operating System

Hi,
You will need to configure a load balancer in front of your Kong cluster to distribute the load for you. Each cluster node runs on a unique IP address (for obvious reasons), so the network architecture downstream will be responsible for forwarding traffic to the cluster.
BTW, this sort of question really belongs on Gitter or the mailing list. GitHub issues should be used for issues, following the template made available :)

Thanks for the reply. Will use mailing lists or gitter in the future. Thanks though.


Fix android prebuilt ghc

Motivation for this change
In order to build GHC to cross compile to android, we need an extra c flag, and we need a patch from master to fix hsc2hs. There is also an issue with forbidden references in dynamic exes, so for now we just don't enableShared:
RPATH of binary /nix/store/bbg892bbs7dqjqm9y1jpgwc2s5sw1xxb-aarch64-unknown-linux-android-ghc-8.4.2/lib/aarch64-unknown-linux-android-ghc-8.4.2/bin/ghc-iserv-dyn contains a forbidden reference to /tmp/nix-build-aarch64-unknown-linux-android-ghc-8.4.2.drv

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Hm. There's a lingering issue with libiconv when trying to build the hello haskell package.
@nix { "action": "setPhase", "phase": "setupCompilerEnvironmentPhase" }
setupCompilerEnvironmentPhase
Build with /nix/store/xnyaam87fjk1nqfpj7fgyyzji5q2flh1-aarch64-unknown-linux-android-ghc-8.4.2.
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking sources
unpacking source archive /nix/store/x3p4dk6kpx485f94d09hiqiy6cvz7i93-hello-<IP_ADDRESS>.tar.gz
source root is hello-<IP_ADDRESS>
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file hello-<IP_ADDRESS>/src/hello.hs
@nix { "action": "setPhase", "phase": "patchPhase" }
patching sources
@nix { "action": "setPhase", "phase": "compileBuildDriverPhase" }
compileBuildDriverPhase
setupCompileFlags: -package-db=/tmp/nix-build-hello-<IP_ADDRESS>-aarch64-unknown-linux-android.drv-0/package.conf.d -j4 -threaded
[1 of 1] Compiling Main             ( Setup.hs, /tmp/nix-build-hello-<IP_ADDRESS>-aarch64-unknown-linux-android.drv-0/Main.o )
Linking Setup ...
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: cannot find -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: cannot find -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: cannot find -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.so when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: skipping incompatible /nix/store/fh010pwx46whw9dwzxsawkwygwrgri5k-libiconv-1.15-aarch64-unknown-linux-android/lib/libiconv.a when searching for -liconv
/nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/ld: cannot find -liconv
collect2: error: ld returned 1 exit status
`cc' failed in phase `Linker'. (Exit code: 1)

But I think we can call this a separate issue.

/cc @Ericson2314 @angerman

Hey, just for reference, did you ever get "armv7a-android-prebuilt" to build after these changes? I am trying to figure out the cause of #42333 & not sure if it 8.4.* was ever working.

@matthewbauer No, I think I only ever tested this with arm64


Fix Set ReceiverID example

https://github.com/crsf-wg/crsf/wiki/CRSF_FRAMETYPE_COMMAND#command_model_select_id
The example shows this package:
c8 09 32 ee ea 10 05 14 crc-ba crc
The length (pkt[1]) is 9 bytes. But in reality there should be 8 bytes: [32 ee ea 10 05 14 crc-ba crc]
I checked this command by connecting to my rс. The result confirmed that there is an mistake in the wiki:
c8 08 32 ee ea 10 05 36 26 a0

Ah you're right, thanks! That's what happens when I try to make up an example on the fly. 😳 I've updated the page with your example instead, and added more info about the extra CRC as well.


custom port for dotnet platform

Thanks, Mohammad Reza.
Can you please resolve merge conflicts?

Please run npm run format and then commit those changes as well.


Some images are not displayed after file selection.

Hello Cornel
Thx for that great project. We use it and feel fine with it, but have some problems with some images.
I select the file (jpg) and the display area of the image keeps blank / gray.
Do i save the same image with paint it works fine. But the image seems to be ok, because i can open it in any image program and browser. I added a sample image which does not work.
Loui

I have this issue too in Safari!

I was running into a similar problem, basically what I ended up doing was this:
image.onload = function() {       that.cropper.setImage(image);     }
I had to do this because when the cropper loaded the image, the image wasn't fully loaded yet. After doing this, I havn't had any problems with images no showing.

Hello again
OK that does not solve our problem but maybe someone can help me. We found out the following error..
vendor.bundle.js:49694 ERROR RangeError: Offset is outside the bounds of the DataView
at DataView.getUint16 ()
at Function.webpackJsonp.../../../../ng2-img-cropper/src/exif.js.Exif.readTags (vendor.bundle.js:2960)
at Function.webpackJsonp.../../../../ng2-img-cropper/src/exif.js.Exif.readEXIFData (vendor.bundle.js:3105)
at Function.webpackJsonp.../../../../ng2-img-cropper/src/exif.js.Exif.findEXIFinJPEG (vendor.bundle.js:2887)
at handleBinaryFile (vendor.bundle.js:2816)
at Function.webpackJsonp.../../../../ng2-img-cropper/src/exif.js.Exif.getImageData (vendor.bundle.js:2827)
at Function.webpackJsonp.../../../../ng2-img-cropper/src/exif.js.Exif.getData (vendor.bundle.js:3163)
at ImageCropperComponent.webpackJsonp.../../../../ng2-img-cropper/src/imageCropperComponent.js.ImageCropperComponent.getOrientedImage (vendor.bundle.js:4713)
at vendor.bundle.js:4673
at ZoneDelegate.webpackJsonp.../../../../zone.js/dist/zone.js.ZoneDelegate.invokeTask (polyfills.bundle.js:6551)
--> Somone understand what's going wrong?

I had this problem once, and it went away after reinitializing the cropper (re-loading the page). And I haven't had it since.


Update migrate-dtu-to-vcore.md

Adding Hyperscale specific scenario for geo-replicated databases

Hyperscale migration required geo-replication to be turned off
https://learn.microsoft.com/en-us/azure/azure-sql/database/manage-hyperscale-database?view=azuresql&tabs=azure-portal#prerequisites

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

adding additional info as per the comments from Dimitri

@ms-amsolo Please close this and reopen it in sql-docs-pr.

Replaced by #9850.


Broken doc url

At url https://material-ui-next.com/demos/tables/ link for codesandbox.io at Sorting & Selecting is broken, page is blank and console reports 414 error

@ekv88 Thanks for raising the issue: 414 URI Too Long.
@CompuIves Do you have a working example with a POST on https://codesandbox.io/api/v1/sandboxes/define? I couldn't make it work so far, we are using a GET.

Here's a working example 😄
https://codesandbox.io/s/5x6rp53jxp

@CompuIves Thank you! I will try that!

Ok, fixing the issue is a matter of chaning this line from get to post:
https://github.com/mui-org/material-ui/blob/39cda071bb0e18baa327cf730ff8326dc5756efe/docs/src/modules/components/Demo.js#L152
@ekv88 Do you want to give it a try?

@oliviertassinari  Can I try this one?

:+1:

pull request https://github.com/mui-org/material-ui/pull/9491


update installation docs to use https

Fixes the old installation instructions using ssh, which would require extra permissions and typically errors out, to use the public https instead.

This should help fix/avoid #177 in the future


Switch to HTTPS for 362 Casks

Every changed URL was:

reachable using a valid HTTPS connection and
'curl head' returned a status code of 200.

I did not verify the whole download, as I'm currently on limited bandwidth. Some casks redirect from HTTPS to HTTP (e.q. sourceforge links) those are left with HTTP.
Should I add more verification?
The script used can be found at cask-switch-https. Does this script also fit somewhere into homebrew-cask? I would happily create a pull request.
Log testing all Casks with a http url/appcast/homepage: https://gist.github.com/tsparber/da281e1912fc0a1ced20
As usually there should be one PR per changed cask: Should I create 362 separate PRs?

Hmm travis fails:
The log length has exceeded the limit of 4 MB (this usually means that the test suite is raising the same exception over and over).

The job has been terminated

So creating 362 PR is the way to go?

The tests failed before it ever got that far:
  1) Failure:
Hbc::CLI::Home#test_0001_opens the homepage for the specified Cask [/Users/travis/build/caskroom/homebrew-cask/test/cask/cli/home_test.rb:25]:
--- expected
+++ actual
@@ -1 +1 @@
-[["/usr/bin/open", "--", "http://www.alfredapp.com/"]]
+[["/usr/bin/open", "--", "https://www.alfredapp.com/"]]

  2) Failure:
Hbc::CLI::Home#test_0002_works for multiple Casks [/Users/travis/build/caskroom/homebrew-cask/test/cask/cli/home_test.rb:32]:
--- expected
+++ actual
@@ -1 +1 @@
-[["/usr/bin/open", "--", "http://www.alfredapp.com/"], ["/usr/bin/open", "--", "https://www.adium.im/"]]
+[["/usr/bin/open", "--", "https://www.alfredapp.com/"], ["/usr/bin/open", "--", "https://www.adium.im/"]]

Fixed the test, at least the tests seem to work know. Audit is still running.

There are merge conflicts that need to be solved.

Does this script also fit somewhere into homebrew-cask?

developer/bin. You don’t need to submit it there, though, we can just link to it from maintaining.md, if you prefer, but it seems like a nice fit.
I have a few scripts that are regularly used by some maintainers (and even users) here, but I keep them in my own repo for a few reasons. Some of them aren’t homebrew-cask specific and can easily benefit other projects and the one that is specific to us (cask-repair) isn’t really targeted for development, and can be used by users. I also very much like to keep them organised and even have formulas for them all all my others “tiny-scripts”. I never gave it any thought, really, but perhaps (at least cask-repair) could make sense here.
Up to you if you want to submit it here, but it seems like you do, and it could be a nice addition.
Looking through the script, I was amazed at how much our style for bash scripting is alike (even down to when to use $var vs ${var}) though I see differences in other areas, but looking further into it it seems like you may have gotten a lot from my scripts directly.
No problem there, naturally, they are public domain, after all. But I wonder if it should duplicate so much functionality (like fetching casks), since it is non-essential functionality and it means the script will essentially have to “catch up” that part of the code to cask-repair.
Perhaps it should be more focused in checking the availability of https, and doing it for all casks. We can worry about that later, though. It also seems like something that would be nice to add to cask-repair, so perhaps it would make sense to have two scripts:

cask-repair for individual casks (with the added functionality of checking if https is available.
cask-switch-https that just checks for https urls (but always does it for all casks).

Looking through the script, I was amazed at how much our style for bash scripting is alike

That's no coincidence :wink:

There are merge conflicts that need to be solved.

Rebased.

Up to you if you want to submit it here, but it seems like you do, and it could be a nice addition.

Great, I will then create a PR.

Looking through the script, I was amazed at how much our style for bash scripting is alike (even down to when to use $var vs ${var}) though I see differences in other areas, but looking further into it it seems like you may have gotten a lot from my scripts directly.
No problem there, naturally, they are public domain, after all.

Yes, this script is intentionally based on your cask-repair to keep things similar, thanks for the 'template' :smiley:

Perhaps it should be more focused in checking the availability of https, and doing it for all casks.

I cleaned the script to focus on https, by default it performs the verify now and runs on all casks (with the option to run an individual cask if provided).

Checked manually and everything seems to be fine. Nice going on also fixing references to alfred’s url on the documentation and tests.
For future reference, github diffing limits. Seems like two PRs would’ve been enough, here, for us to at least be able to see all the files in the preview.


STORM-2407: KafkaTridentSpoutOpaque Doesn't Poll Data From All Topic-…

…Partitions When Parallelism Hint Not a Multiple Total Topic-Partitions

Introduce logic to poll data from the topic partitions assigned to each  task

+1.


V4 docs

This the new updated docs for v4.

[x] New API section.
[x] Add docblock in some classes
[x] Update current docs.

@freekmurze as you haven't been part of the v4 changes at all. Could you take a look now?
I think none of the features/changes is bad - but possibly you have a better naming idea for some, want something else to be added to v4 or anything. 🙂

@Gummibeer, I'm not sure about this section yet.

@Gummibeer What do you think about the new API section

I think that it's a good way. 👍
Could you walk through the docs and fix indentation and empty lines? You could also run prettier over all doc files, this should fix a lot of things already.

Okay, I will wait this week till @freekmurze reviewed v4. From my side it looks good now. 🥳


Unable to run gulp tasks after este package.json update (on Windows)

Steps to reproduce
Create clean install of este (or just delete node_modules) and run npm install (or gulp, gulp -p, etc.).
Expected Behavior
Install and compile dependecies.
Actual Behavior
It causes an error in my case (on Windows):
E:\Projects\_github\este\node_modules\babel-preset-react-native\transforms\transform-symbol-member.js:22
  const t = babel.types;
  ^^^^^
SyntaxError: Use of const in strict mode.
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Module._extensions..js (module.js:478:10)
    at Object.require.extensions.(anonymous function) [as .js] (E:\Projects\_github\este\node_modules\babel-register\lib\node.js:166:7)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (E:\Projects\_github\este\node_modules\babel-preset-react-native\configs\main.js:41:5)
    at Module._compile (module.js:460:26)

Did you update your Node to 6?

Yes, I updated, but it turned out that npm was not. After updating npm manually, it's working.
Thank you very much.


Implement starknet_traceBlockTransactions RPC call

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

Bugfix
Feature
Code style update (formatting, renaming)
Refactoring (no functional changes, no API changes)
Build-related changes
Documentation content changes
Testing
Other (please describe):

What is the current behavior?

Resolves: #NA
What is the new behavior?

Does this introduce a breaking change?

Other information

superseeded by #1416


Add User-Agent to builder requests

Motivation

see https://github.com/ChainSafe/lodestar/issues/5360

Description

Set on all requests by default
Set to current agent version by default
Allow to customize and disable via visible flag --builder.userAgent

builder
      --builder                        Enable builder interface
                                                      [boolean] [default: false]
      --builder.urls                   Urls hosting the builder API      [array]
      --builder.timeout                Timeout in milliseconds for builder API H
                                       TTP client                       [number]
      --builder.faultInspectionWindow  Window to inspect missed slots for enabli
                                       ng/disabling builder circuit breaker
                                                                        [number]
      --builder.allowedFaults          Number of missed slots allowed in the fau
                                       ltInspectionWindow for builder circuit
                                                                        [number]
      --builder.userAgent              User-Agent header attached to all builder
                                        requests
        [string] [default: "Lodestar/v1.6.0/dapplion/builder-useragent/8846e16"]

Closes https://github.com/ChainSafe/lodestar/issues/5360

:tada: This PR is included in v1.8.0 :tada:


Remove version: 8x8.VirtualOfficeDesktop version <IP_ADDRESS>

Consolidating into 8x8.Work. Related to: #135119
Microsoft Reviewers: Open in CodeFlow

URL: https://vod-updates.8x8.com/ga/work-64-msi-v8.5.3-1.msi
Status Code: 200
(Automated message - build 656)


COCO resize tools does not work for datasets without keypoints

Describe the bug
COCO resize tools does not work for instance datasets yet.
To Reproduce
Steps to reproduce the behavior:

Go to airo-mono/airo-dataset-tools/test/test-data/
Run airo-dataset-tools resize-coco-keypoints-dataset instances_val2017_small.json --width 50 --height 100

fixed in #99


Fix json object array alignment

Currently, json array macros are passing the outer "container" struct
to the Z_JSON_ELEMENT_DESCR macro, causing the alignment to be calculated
from the outer struct which may be incorrect. Fix this by using the
biggest shift from the nested objects to calculate the alignment as the
struct size is calculated based on the biggest alignment of it's members.
Also add a test for such case.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/73899

@mrfuchs @sir-branch please review


Add Flake8 support for tox test runner

Add flake8 as a code-style check to tox test runner. Since CircleCI runs the default tox envs, this means that code-style should be enforced for each pull request.
Closes #12

I hadn't noticed that I have some squashed flake8 errors set globally to reduce noise with our other projects. The project should now be flake8 clean.


Progress counter goes over 100 %

Epoch and update numbers are not set correctly anymore in the end of an epoch? Progress counter goes over 100 % of epoch 1.

The problem is probably caused by batch iterator not returning to the same position in the file, when recalling a previous training state.
_set_candidate_state
[301036] Validation set cost history: 1164.4 1088.1 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 [963.7]
[302000] (55.18 %) of epoch 1 -- lr = 0.001, cost = 2.07, duration = 4.77 ms

[547000] (99.94 %) of epoch 1 -- lr = 0.001, cost = 2.23, duration = 4.75 ms
[547337] Validation set cost history: 1164.4 1088.1 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 [963.7] 997.1 986.8 995.3 984.1 986.2 986.7 994.1 984.8 987.9
[663] (0.12 %) of epoch 2 -- lr = 0.001, cost = 1.78, duration = 6.80 ms

[54663] (9.99 %) of epoch 2 -- lr = 0.001, cost = 2.05, duration = 4.35 ms
[54734] Validation set cost history: 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 [963.7] 997.1 986.8 995.3 984.1 986.2 986.7 994.1 984.8 987.9 987.6 975.6
reset_state
[301031] Validation set cost history: 1164.4 1088.1 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 [963.7]
[301960] (55.17 %) of epoch 1 -- lr = 0.0005, cost = 1.59, duration = 7.60 ms

[409960] (74.90 %) of epoch 1 -- lr = 0.0005, cost = 2.69, duration = 3.23 ms
[410503] Validation set cost history: 1164.4 1088.1 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 963.7 962.8 975.3 966.4 958.3 [948.9]
[410960] (75.08 %) of epoch 1 -- lr = 0.0005, cost = 1.93, duration = 3.11 ms

[546960] (99.93 %) of epoch 1 -- lr = 0.0005, cost = 1.86, duration = 4.41 ms
[547337] Validation set cost history: 1088.1 1040.3 1034.5 1010.8 1013.9 1012.4 1015.2 999.3 1002.3 963.7 962.8 975.3 966.4 958.3 [948.9] 964.9 957.2 953.2 966.3 954.8
[547960] (100.11 %) of epoch 1 -- lr = 0.0005, cost = 1.54, duration = 6.42 ms


Please provide an example of using this from PowerShell

There doesn't seem to be anything on the net for consuming Kafka data from PowerShell. We would love to be able to consume from Kafka stream for provisioning to things like AD and Exchange. Can this client be used from PowerShell?

I think you're talking about a custom cmdlet? sounds like a good little project for someone to investigate this! we haven't looked at it yet.
https://msdn.microsoft.com/en-us/library/ms714395(v=vs.85).aspx

I'm pretty sure that what @mhowlett suggested is the right answer -- write a cmdlet (in C#, where you have full support for async/await, generics, and extension methods) and call that from PowerShell.
As a first draft, take the simple example here:
https://docs.microsoft.com/en-us/powershell/developer/cmdlet/how-to-write-a-simple-cmdlet
And marry it to the producer example in the readme...

thx @Jaykul

@copdips
How did it go? Are you able to share your solution?

@blomster
Sorry, I moved to Python to use Kafka in the end.
But if you're using powershell, and you dont need to send very very quickly the messages to Kafka, you can use whatever supported language to build a rest API that can receive messages from powershell (invoke-restmethod), and then forward them to Kafka.


MockTaskRunner blocks

Environment
azure-pipelines-task-lib version: 2.9.3
Issue Description
MockTaskRunner#run blocks.
https://github.com/microsoft/azure-pipelines-task-lib/blob/0a6b208c2e94339d2ee9d33d971d965ca25d61de/node/mock-test.ts#L68
How about not doing that?
You've already made it hard enough to mock http responses by launching the SUT in a different process.
Now we can't even run a mock http server from our test runner, because you're blocking that, too?
Wow.
Expected behaviour
This test suite
tests/_suite.ts
import * as path from 'path'
import * as assert from 'assert'
import * as ttm from 'azure-pipelines-task-lib/mock-test'
import axios from 'axios'
import ServerMock = require('mock-http-server')

describe('Sample task tests', () => {
    const server = new ServerMock({host: 'localhost', port: 0})
    before(done => server.start(() => {
        process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI = `http://localhost:${server.getHttpPort()}/`
        done()
    }))

    after(done => server.stop(done))

    it('non-blocking code completes', () => {
        server.on({path: '*'})
        return axios({
            method: 'GET',
            url: process.env.SYSTEM_TEAMFOUNDATIONCOLLECTIONURI
        })
            .then(res => assert.strictEqual(res.status, 200))
    })

    it('blocking code completes', done => {
        const tp = path.join(__dirname, 'success.js')
        const tr: ttm.MockTestRunner = new ttm.MockTestRunner(tp)
        server.on({path: '*'})
        tr.run()
        done()
    })
})

with these scripts
tests/success.ts
import ma = require('azure-pipelines-task-lib/mock-answer')
import tmrm = require('azure-pipelines-task-lib/mock-run')
import path = require('path')

const taskPath = path.join(__dirname, '..', 'index.js')
const tmr: tmrm.TaskMockRunner = new tmrm.TaskMockRunner(taskPath)

tmr.run();

index.ts
import tl = require('azure-pipelines-task-lib/task')
import axios from 'axios'
const url = tl.getVariable('system.teamFoundationCollectionUri')
axios.get(url!)

shouldn't deadlock.
That would mean offering a promise returning method like #runAsync by maybe calling fork instead.
Actual behaviour
Deadlock.
Since no such function exists, we only get a deadlocking synchronous #run.
$ npm run build && npm test

><EMAIL_ADDRESS>build /home/LOM0227/project/ALM.Wreckorator/asset_yml_check_whitelist/src
> tsc

><EMAIL_ADDRESS>test /home/LOM0227/project/ALM.Wreckorator/asset_yml_check_whitelist/src
> mocha tests/_suite.js

  Sample task tests
      non-blocking code completes

the second test never completes.
It shouldn't be this hard to mock http responses: some tasks will need to do network IO, and we need a way to test them.
Steps to reproduce
Transpile & run the above code.

I'd recommend doing this with a standard http mocking library (e.g. nock) - we do this all over the place in our tasks repo if you'd like examples (e.g. https://github.com/microsoft/azure-pipelines-tasks/blob/7e2d65456791dd520f9840c656a8df292b2cd65a/Tasks/AppCenterDistributeV3/Tests/L0PublishStore.ts#L4). You can also mock the source of the calls themselves (e.g. https://github.com/microsoft/azure-pipelines-tasks/blob/7e2d65456791dd520f9840c656a8df292b2cd65a/Tasks/NodeToolV0/Tests/L0FirstDownloadSuccess.ts#L59)
I can't speak to why this was originally written this way, but I do see a lot of rationale for doing it - the way our library is currently set up, even if we offered something like runAsync there's not a clear/easy way to wait on the result here. Additionally, it more closely mirrors the actual execution that happens on the agent this way.
I'm not necessarily opposed to doing this, but I don't see it rising in priority at the moment since there are other ways to do this.

Node.js is fundamentally a cooperative multitasking system that runs JavaScript in a single thread.
Pretty much every introduction to Node.js covers this and stresses the importance of not blocking the main thread by keeping synchronous calls to a minimum/avoiding them entirely.
Just from Node.js' own guides

Overview of Blocking vs Non-Blocking

Dangers of Mixing Blocking and Non-Blocking Code

Don't Block the Event Loop (or the Worker Pool)

Performing synchronous calls despite these clear warnings is a failure to cooperate, a fundamentally bad practice for Node.js.
In this project, it leads to some awkward workarounds, like the scattering of 'mock test runner' files required just to setup mocks and run the task, instead of allowing more straightforward testing setup & teardown directly from the test suite file.
If this Node.js library is going to depart from a basic standard and defy common expectations of Node.js libraries, then documentation should at least clearly point this out and explain the compensations necessary to support that departure where more natural approaches for more cooperative libraries would fail.
Would a Promise that resolves on completion of the forked process not wait on the result?
Promises & callbacks are commonplace for Node.js.

Needs better adherence to node conventions & more explicit documentation of departures.


Toggle Console Action is inserted into command input box

Clearing text directly when opening the console doesn't work since the event seems to be processed later but _commandInput.CallDeferred(LineEdit.MethodName.Clear); fixed the issue for me.


Added logic to explicitly close TCP socket for sync HTTPS client

Hello!
I noticed that example client does not include logic which explicitly closes TCP socket, I think it's good ideas to add it here.
In addition to leaked socket I hit quite nasty issue when my CDN (Fastly) does not implement proper TLS termination and shutdown() hands indefinitely. So I've added comment about this case too.

I had conversation with my friend from OpenSSL and I think this issue will need more investigation on my side. I use old OpenSSL (1.0) and it may be root cause of such behaviour. I'll repeat my tests with fresh OpenSSL and will report back.

After upgrade to OpenSSL 1.1.1q shutdown works fine and does not block execution. I can confirm that old OpenSSL (1.0.2d) was root cause.
Socket close is not needed too, shutdown moves socket to TIME_WAIT status which is perfectly fine.


Use proper way to include latest tag into binary

Description
Use proper way to include latest tag into binary
Type of change

Configuration update

Testing steps
To be checked on CI.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch coverage has no change and project coverage change: -0.06 :warning:

Comparison is base (5c87f48) 69.48% compared to head (f6f91da) 69.42%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1085      +/-   ##
==========================================
- Coverage   69.48%   69.42%   -0.06%
==========================================
  Files          25       25
  Lines        3657     3657
==========================================
- Hits         2541     2539       -2
- Misses        861      863       +2
  Partials      255      255

see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Revert removal of shared TaskQueue for page.screenshot()

Issue https://github.com/puppeteer/puppeteer/issues/6713.
Please, notice that I changed the return type of page.screenshot() back to Promise<Buffer | string>.
I did it because I couldn't find any place where it returns void.
Let me know if it was a mistake on my part, I'll change it back.

@jackfranklin sorry for the ping, but Travis appears not to be working for the last 5 days.

@jackfranklin sorry for the ping, but Travis appears not to be working for the last 5 days.

@jackfranklin sorry for the ping, but Travis appears not to be working for the last 5 days.

Yeah, we're aware of this. Travis essentially removed support for their free plan, so we'll have to migrate to GitHub Actions. #6726

@jackfranklin sorry for the ping, but Travis appears not to be working for the last 5 days.

Yeah, we're aware of this. Travis essentially removed support for their free plan, so we'll have to migrate to GitHub Actions. #6726

@dmitrysteblyuk thank you for this PR and for the bug report. This looks good to me. Now we've landed GitHub actions into the codebase, would you mind rebasing your PR? Then we can see if CI is happy 👍

@dmitrysteblyuk thank you for this PR and for the bug report. This looks good to me. Now we've landed GitHub actions into the codebase, would you mind rebasing your PR? Then we can see if CI is happy 👍

@mathiasbynens PTAL.

@googlebot I consent.


Fix pop-up labels on hexmaps when viewed on mobile

Picked up by L23, and to some degree replicated by me.
Using both Firefox & Chrome on phone, Calverley & Farsley are cut off (see below).
L23 report  Bramley and Stanningley also cutting off, although I see their label getting close to the edge but not cutting off. Asked what browser/phone combo they are using to help us solve this.
When looking at volunteers on the phone, I also struggle with Pudsey - I can't see the number of applications recieved.

@gilesdring are you happy for this to be considered ready?

Yes

@gilesdring In OI Lume Charts the standard hex map tooltips should fit to the screen properly. See e.g. custom tooltips and hex labels.

We will need to either switch hexmaps to standard version or implement the same logic in the L23 hexmap viz type.

Closed as a result of @slowe updating to the OI Lume Viz version of the hexmaps


[FLINK-32796][table] Try to create catalog store path if not exists

What is the purpose of the change
Try to create dirs that does not exists when opening file catalog store. If an error is thrown in case the given path not exists, SQL client will fail to start, also SQL gateway won't operate, cause even basic API calls will fail.
It makes sense to at least try creating any missing folders, which can probably help allow some annoying errors and restart rounds.
Brief change log
Add FileSystem.mkdirs(...) to FileCatalogstore.open() to try creating dirs the given path does not exist.
Verifying this change
Added unit test to verify a failing scenario. Every other existing unit test will now create the store root folder.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@ferenc-csaky Thanks for your contribution, Over looks good to me.
I thinks we should update the document about the default behavior when root directory does not exist.

Thanks for the review @hackergin! @PatrickRen if you have some time can you check on this one?

@ferenc-csaky thanks for your PR, as @hackergin mentioned, could you please update doc as well?

@ferenc-csaky thanks for your PR, as @hackergin mentioned, could you please update doc as well?

Yep, just pushed the commit to my other PR.

Thank you! Let me prepare the same changes against 1.19.

Thanks for the contribution @ferenc-csaky
Thanks for review @hackergin


Add simple string conversion and match expressions

What kind of change does this PR introduce? (check one with "x")
[] Bugfix
[X] Feature
[] Code style update (formatting, local variables)
[] Refactoring (no functional changes, no api changes)
[] Other... Please describe:
``` Adds a new functions for use in mapping expressions:
string.tostring(nnn) returns a string
string.match(expression,string) returns a bool

**Fixes**: #

**What is the current behavior?**

**What is the new behavior?**

@mellistibco What needs to be done to get this request merged ?


Add global --env option. Fix issue #208.

Fix for issue #208.

The env option is evaluated globally in https://github.com/opencfp/opencfp/blob/master/bin/opencfp#L15 but is not propagated to all commands.
Fix inspired by:
https://github.com/symfony/symfony/blob/2.7/src/Symfony/Bundle/FrameworkBundle/Console/Application.php#L46

Thanks @bcremer


FEAT: Transition to scikit-build-core CMake-based build process

This was initally started to transition to Hatchling. However, as I learned more about scikit-build-core, it seems that it handles all the steps required to build coincurve. However, I do not know Hatchling, so let me know your thoughts on how we should integate it.
This PR itself changes some of the original features of coincurve. CMake provides a convenient way to download the SECP256K1 library, so currently, I did not add this feature (though the current UPSTREAM_REF is used)
The other major addition is that the flow creates the CFFI source headers directly from the downloaded SECP256K1 original headers, so it should be quite straightforward to follow SECP256K1 updates.
The rest is fairly similar, create the SECP256K1 library (using the CMake they provide), create the CFFI C-code (rather than compile it with CFFI), then compile the C-Code and link it to SECP256K1 library.
The feature of using SYSTEM_LIB is similar, although I have not tested it as well as for the CMake-based PR.

@ofek So this is a scikit-build-core backend build. I try to see how to use hatch instead, but have not understood yet how to link scikit-built-core to hatch. Since this is a cmake build, would it not be simpler to try to develop a hatch-cmake` hook?

Henry (the maintainer) is working on making a plugin for Hatchling. Until then this current approach is fine. Which PR should I review? I don't quite know the plan.

https://github.com/ofek/coincurve/pull/159 is ready, it only add verification for shared lib build. Or do you think it should not be added?

This PR still has issues, but I think it is more maintainable than the setuptools+CMake.
So the plan was to move to CMake, then to use hatch back-end and create the hooks to setuptools - but then you pointed me to scikit-build-core ... so we can see this PR as a parallel dev and choose how to move forward - I am not so good at plan (r communication for that matter)
Do we need to better sync-up?

Definitely please move forward just with scikit-build-core!

So, if I undersand correctly, Henry will build a plugin for hatch which would use scikit-build-core and be able to interact with hatch, but I think what I thought we would have is a hatch plugin to interact/control scikit-build-core.
With the current PR, do you see how coincurve woulb benefit from hatch
I am trying to update the config with something like:
[[tool.hatch.build.hooks.build-scripts.scripts]]
out_dir = "out"
commands = [
    "mkdir -p build_with_cmake",
    "cd build_with_cmake && cmake -S .. -B .",
    "cd build_with_cmake && cmake --build .",
]
artifacts = [

to try to understand how hatch fits/improves the current status

The plugin would live at the build system layer i.e. Hatchling, which would invoke some logic within that other package. Hatch the CLI has nothing to do with this and we don't even have to use it if you don't want to.

Right, I meant hatchling, but the backend would be scikit-build-core, with a tools.scikit-built.hatchling.xxx then what would we use it for? what is coincurve missing without hatchling in the current state
I am thinking to try to PR a update of the current master with hatchling to better understand the differences and benefits

The plugin would live at the build system layer i.e. Hatchling

I took a brief look and it seems to me that it is similar to the setuptools integration

For more context, see this issue: https://github.com/PyO3/maturin/issues/1419
Basically, Hatchling will be the backend when there is a plugin and the only thing that other package would do is build the artifacts for Hatchling to consume.

Right, I think I understand, though the pyproject.toml should have hatchling as build-backend, then specify the configuration of scikit-build-core with a plug-in, correct?

In the future scenario Hatchling would be the backend with a plugin configured that would call scikit-build-core. In the final future scenario after that, scikit-build-core would remove its ability to be a backend (i.e. strip out all of the code to build wheels and source distributions) and it would only be used as a plugin for any backend that supported the future plugin architecture.
For now, we have to use only scikit-build-core.

What do you think about scikit-build-cli? It may not be too difficult to create a hatch plugin, this way we could move on to using hatchling and have a more finalized packaging, then once scikit-build-core has moved forward we'd use the more integrated interface

Do you mean a Hatchling build hook that invokes something as a subprocess to build the artifacts?

Yes, I thought that was the model you were looking for. I dug a bit more on scikit and I think the plugin is intended to be something likebuild-backend = scitkit-build-core.hatchling.build_meta, right, but it seems convoluted to me and more complex to engineer

No the future plugin would be a build hook like this https://github.com/ofek/hatch-mypyc so Hatchling would still be the backend, nothing else would be referenced in that section.
Sure if you can make that work that sounds good! You would want a custom build hook:

https://hatch.pypa.io/latest/plugins/build-hook/custom/
https://hatch.pypa.io/latest/plugins/build-hook/reference/#hatchling.builders.hooks.plugin.interface.BuildHookInterface.initialize
https://hatch.pypa.io/latest/plugins/builder/wheel/#build-data
https://github.com/ofek/hatch-mypyc/blob/5a198c0ba8660494d02716cfc9d79ce4adfb1442/hatch_mypyc/plugin.py#L340-L343

I am too superficial! This is exactly what Henry is working on (last commit 4 days ago). He is developing it inside a branch, so I thought he was implementing it the way he wraps setuptools, but looking at his test suite and example, it is what I am looking for:
[tool.hatch.build.targets.wheel.hooks.scikit-build]
dependencies = ["scikit-build-core"]
wheel.install-dir = "hatchling_example"
cmake.source-dir = "cpp"

lOl ... 10s of back-and-forth for me to finally get it :'(

I didn't know it was that far along, nice!

Oh, you're not collaborating with him? So, I am copying his progress into a standalone plugin that I will install locally to test out

I am in general but not implementing the plugin directly. For that, I think one of his undergraduates researchers is assisting.

Hmmm ... I don't want to be ridiculed by a graduate student though ... maybe I should drop him a note before I get entangled

Some progress, though still some hiccups:
-- Install configuration: "Release"
-- Installing: //_libsecp256k1.cpython-311-x86_64-linux-gnu.so
CMake Error at /tmp/tmpyk6rr366/cmake_install.cmake:80 (file):
  file INSTALL cannot copy file
  "/tmp/tmpyk6rr366/cm_python_module/_libsecp256k1.cpython-311-x86_64-linux-gnu.so"
  to "//_libsecp256k1.cpython-311-x86_64-linux-gnu.so": Permission denied.

from:
[tool.hatch.build.targets.wheel.hooks.hatch-scikit-build-core]
dependencies = ['scikit-build-core']
wheel.install-dir = 'build/scikit-build-core'
cmake.source-dir = '.'

# --- scikit-build-core ---
[tool.scikit-build]
# metadata.version.provider = "scikit_build_core.metadata.setuptools_scm"
cmake.verbose = true
# logging.level = 'INFO'
# wheel.packages = ['src/coincurve']
cmake.build-type = 'Release'

@ofek I invited you to the plugin repo. It is private for now. It is mostly from Henry, but I did not understand why he needed to update scikit-build-core. I implemented a version that seem to work with 0.8.2, but I'd like you to take a look. The build_dir test is failing because I don't quite understand the mypyc that it comes from. But it builds the wheel.
I am trying to understand how to control exactly what files are built by scikit and passed to hatchling, but I could use a helping eye and brain ...

@ofek It looks like with the new plugin,  scikit-build-core is creating the wheel, the installed coincurve in the tox directory only contains _libsecp256k1.xxx.so.
How does hatch retrieve the artifact from scikit-build hook. The hook builds the library correctly:
  -- Installing: /tmp/tmp17u80te9/wheel/platlib/coincurve/_libsecp256k1.cpython-311-x86_64-linux-gnu.so
  Preparing editable metadata (pyproject.toml) ... done

but after that it seems that I am missing something. With the previous implementation, the artifacts were passed in the build_data, I don't see it in this implementation - stuck for now

cc @henryiii

With python -m build:

*** Installing project into wheel...
-- Install configuration: "Release"
-- Installing: /tmp/tmpzvu_i59_/wheel/platlib/coincurve/_libsecp256k1.cpython-311-x86_64-linux-gnu.so
Successfully built coincurve-19.0.2.tar.gz and coincurve-19.0.2-cp311-cp311-linux_x86_64.whl

content:
unzip -l dist/coincurve-19.0.2-cp311-cp311-linux_x86_64.whl
Archive:  dist/coincurve-19.0.2-cp311-cp311-linux_x86_64.whl
  Length      Date    Time    Name
---------  ---------- -----   ----
      179  2020-02-02 00:00   coincurve/__about__.py
      297  2020-02-02 00:00   coincurve/__init__.py
     1117  2020-02-02 00:00   coincurve/context.py
     3985  2020-02-02 00:00   coincurve/ecdsa.py
      433  2020-02-02 00:00   coincurve/flags.py
    24235  2020-02-02 00:00   coincurve/keys.py
        0  2020-02-02 00:00   coincurve/py.typed
      310  2020-02-02 00:00   coincurve/types.py
     4183  2020-02-02 00:00   coincurve/utils.py
  1279192  2020-02-02 00:00   coincurve/_libsecp256k1.cpython-311-x86_64-linux-gnu.so
     3977  2020-02-02 00:00   coincurve-19.0.2.dist-info/METADATA
       99  2020-02-02 00:00   coincurve-19.0.2.dist-info/WHEEL
     9748  2020-02-02 00:00   coincurve-19.0.2.dist-info/licenses/LICENSE-APACHE
     1065  2020-02-02 00:00   coincurve-19.0.2.dist-info/licenses/LICENSE-MIT
     1218  2020-02-02 00:00   coincurve-19.0.2.dist-info/RECORD
---------                     -------
  1330038                     15 files

When I install that wheel and test it, it seems to work
Note that I could be missing something quite obvious

Ready for review?

Yes, it is ready - I got distracted with other projects - I am not very knowledgeable in CMake, so there maybe lots of improvements possible

@ofek Did you have some time to review? Perhaps finding someone knowledgeable in CMake to provide feedback would be good

Sorry for the wait! I just tried and am getting an error:
* Building wheel...
*** scikit-build-core 0.9.5 using CMake 3.29.3 (wheel)
*** Configuring CMake...
2024-05-31 20:02:10,042 - scikit_build_core - WARNING - Can't find a Python library, got libdir=None, ldlibrary=None, multiarch=None, masd=None
loading initial cache file C:\Users\ofek\AppData\Local\Temp\tmpz7b4bd21\build\CMakeInit.txt
-- Building for: Visual Studio 17 2022
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.22631.
-- The C compiler identification is MSVC 19.33.31630.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/VC/Tools/MSVC/14.33.31629/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Found Python: C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Scripts\python.exe (found suitable version "3.11.7", minimum required is "3") found components: Interpreter Development.Module Development.SABIModule
CMake Error at C:/Users/ofek/AppData/Local/Temp/build-env-ynqhppp2/Lib/site-packages/cmake/data/share/cmake-3.29/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
Call Stack (most recent call first):
  C:/Users/ofek/AppData/Local/Temp/build-env-ynqhppp2/Lib/site-packages/cmake/data/share/cmake-3.29/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  C:/Users/ofek/AppData/Local/Temp/build-env-ynqhppp2/Lib/site-packages/cmake/data/share/cmake-3.29/Modules/FindPkgConfig.cmake:114 (find_package_handle_standard_args)
  CMakeLists.txt:35 (find_package)

-- Configuring incomplete, errors occurred!
Traceback (most recent call last):
  File "C:\Users\ofek\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
    main()
  File "C:\Users\ofek\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyproject_hooks\_in_process\_in_process.py", line 335, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\ofek\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyproject_hooks\_in_process\_in_process.py", line 251, in build_wheel
    return _build_backend().build_wheel(wheel_directory, config_settings,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\hatchling\build.py", line 58, in build_wheel
    return os.path.basename(next(builder.build(directory=wheel_directory, versions=['standard'])))
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\hatchling\builders\plugin\interface.py", line 147, in build
    build_hook.initialize(version, build_data)
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\scikit_build_core\hatch\plugin.py", line 125, in initialize
    self._initialize(build_data=build_data)
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\scikit_build_core\hatch\plugin.py", line 224, in _initialize
    builder.configure(
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\scikit_build_core\builder\builder.py", line 235, in configure
    self.config.configure(
  File "C:\Users\ofek\AppData\Local\Temp\build-env-ynqhppp2\Lib\site-packages\scikit_build_core\cmake.py", line 221, in configure
    raise FailedLiveProcessError(msg) from None
scikit_build_core.errors.FailedLiveProcessError: CMake configuration failed

ERROR Backend subprocess exited when trying to invoke build_wheel

No pkg-config it looks like.

I wonder if there’s a way we can suppress those extra tracebacks that aren’t useful.

I was expecting it to Just Work ™ ... is that a requirement?

Can we drop Windows 32 bit? After that CI will pass (no pypkgconf wheel for that architecture).

Well, that organization is just a matter of preference (or non-formal CS background) it could be organized differently. I am also not very knowledgeable with CMake, so I needed a lot of debugging, separating the different phase helped me navigate


Rename icons folder to favicons

Done https://github.com/orlovmax/front-end-scaffold/commit/b46ac50d02f8c6abc346dc212bd019e36c328a95


Add badges in readme

I want to add badges in readme.
reference img

Great, I'm assigning you the issue . Go ahead


[Question]: Using llama-index , "PandasQueryEngine" to generate text 2 pandas string to execute and filter the data using dataframes, but the string generated raises error as pd is missing.

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Have imported using, from llama_index.core.query_engine import PandasQueryEngine.
But the pandas_str generated has "pd". Below is an example of the error.

Pandas Instructions:

"df[(df['Operating Region'] == 'Southern') & (df['Due Date'] < pd.to_datetime('today'))]['Assets ID'].nunique()"

Pandas Output: There was an error running the output as Python code. Error message: name 'pd' is not defined

What version are we supposed to use for this edit, as its not working in my case.
Version being used:  {llama-index-core==0.10.26}. please help


autosifter shaking the wrong sieve

i'm playing project ozone 3 with ex nihilo creatio version 1.12.2-0.3.7 installed
to automate resources i've made a settup with 3 autosieves each powered by their own waterweel, one for gravel, one for sand and one for dust
each is placed next to eachother with 3 empty spaced between
and while the waterwheel powers the correct sifter, for some reason, some sifters don't shake the sieve on top of them but those on top of other sifters
and that while the one on top of the powered sifter isn't moving at all

I believe PO3 has the sieve radius set higher than default (I think a radius of 4 instead of 2); the sieves don't actually check connectivity (just range) so some of the disconnected sieves might actually be valid to move. As a temp fix try moving them further apart.
Could you post an screenshot of your setup?

while waiting for the reply i had done some testing and indeed moving them 1 more block away from eachother fixes that problem, so i guess that means the problem has been solved
thanks for taking the time to read and help

It intentionally doesn't check connectivity as that would mean having do to
a map search instead of simply iterating over a grid which might hurt
performance badly if it is done often.
bandon41<EMAIL_ADDRESS>schrieb am Mi., 27. Feb. 2019, 08:56:

while waiting for the reply i had done some testing and indeed moving them
1 more block away from eachother fixes that problem, so i guess that means
the problem has been solved
thanks for taking the time to read and help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BloodyMods/ExNihiloCreatio/issues/177#issuecomment-467760912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHwpZm5_Q88Ui_riT-TFnLi6L6c-tvMeks5vRjosgaJpZM4bTcHB
.

I think I have it working (along with some minor changes to how it looks)
https://gfycat.com/CanineGraveGrayreefshark

Release 0.4 has changed the animation and fixed the config option to disable it.


GC issues

If we:

heap allocate a struct-based helix object (e.g. Duration::new)
GC runs

It will GC the Ruby object associated with the helix object, because the GC couldn't see the VALUE pointer on the stack
See https://github.com/sparklemotion/nokogiri/commit/0a1556a7c0ac1dc58c074a06f639bba3ddabab3f
and http://blog.reverberate.org/2016/06/12/native-extensions-memory-management-part1-ruby-mri.html

Seems like we should be able to write a failing test for this, right?

@chancancode @wycats is this still an issue?

Until we allow calling back into the Ruby VM (thus opening up the possibility of triggering a GC pass mid-rust-frame), I don't think this could happen


refactor: cleanup

This uses the token dialer branch as a base.

Removes the old switch code as it is no longer needed
The Stats code has been moved to a root directory, as it will be reincorporated soon.
All unused packages have been removed from package.json
Adds the bundlesize check back to ci

Note: The deps check in ci is excluding pull-stream and pull-handshake. Ping still needs to be refactored and as such is requiring these modules. Once that refactor is done, we can remove the exclusion from the ci deps check.

 🙌


version of GHC for the book

I noticed two conflicting directions:

https://github.com/input-output-hk/hs-opt-handbook.github.io/blob/dcf7054c8d9d89923e94faf0585c3a26def7ec19/index.rst?plain=1#L8

https://github.com/input-output-hk/hs-opt-handbook.github.io/blob/dcf7054c8d9d89923e94faf0585c3a26def7ec19/src/Preliminaries/how_to_use.rst?plain=1#L33

Maybe some sort of a constant would be useful to reference the required version of GHC...

Aha! Thanks for the review. Yes I must have missed this in when I updated the version. Constants would be nice, and would be my normal operating procedure with latex but sphinx makes it harder than it should be. Let me test some adding these in the rst_prolog and see if it works.

Nice!


server error: Server not initialized

Honestly it can't init language server.
Installation instruction 100% followed!
[Error - 00:17:53] Request textDocument/hover failed.   Message: server error: Server not initialized   Code: 1  [Error - 00:19:27] Request textDocument/hover failed.   Message: server error: Server not initialized   Code: 1

Getting the exact same issue.

@Thortok try removing the double quotes (") and remove the second setting (Script Cache path)

I think I have the same problem
[Info  - 6.10.17] Server initialized!
[Error - 6.10.17] Request textDocument/hover failed.
  Message: server error: Server not initialized
  Code: 1

i had the plugin working at one point, but not immediately.
then i went customizing vscode and when i came back, it didn't work anymore
since then i've reinstalled vs code portable 3 times (unzip the thing, add data folder, move cyberpunk source to the folder, add it to workspace,install vsix, set cyberpunk folder path)

Having the same issue:

Tried putting both CP2077 folder path and cache. Only cache, only game. No matter what I do I get the same message:

Request textDocument/hover failed.
Message: attempted to access the server while it was not initialized
Code: 1

Same issue on my end


how to add asp.net core project

i Installed MVC.Grid package from nuget. but not see inside js and css file.

Please follow installation steps, because nuget in asp net core doesn't add static content anymore.


Commandline options for partial reformatting

It'd be very nice if dfmt could only selectively reformat a file, without //dfmt off comments. This is needed for more user friendly editor integration: only reformat stuff related to the current cursor/selection.
clang-format has the options -cursor, -lines, -offset/-length for this purpose. See http://clang.llvm.org/docs/ClangFormat.html
I have been hacking on dfmt to try and implement -offset/-length functionality, and I got something to work, but... , it needs a more thought-out approach. The problem is that //dfmt on/off is too crude at the moment, and so I cannot just mimic that functionality. When formatting is disabled, the whole machinery is bypassed that keeps track of indentation etc.
For example (think of the //dfmt on .. //dfmt off range as the range specified on the commandline):
//dfmt off
void foo() {
    int i;
    //dfmt on
    if (i) {

    }
    //dfmt off
}

will be reformatted without indenting if(). However, I think it will make //dfmt on much more useful, as it will give more fine-grained control. Right now, I guess you can only enable/disable formatting on blocks that do not need indentation at their start.
I have not spent enough time thinking about how to go about this. I.e. what indentation to start with after //dfmt on ? Assume the user was doing "the right thing" before //dfmt on, i.e. just parse the whole file and start with the indentation that the if() thing would get when the part before //dfmt on would be correctly formatted?

A related problem is that running dfmt on a snippet (for example, using vim's equalprg) gives some annoying messages: (this is from running it on a unittest block)
stdin(9:42)[error]: Declaration expected
stdin(10:40)[error]: Declaration expected
stdin(10:41)[warn]: Empty declaration


lv_templ: new destination/directory: src/

In additon to the "Coding style guide" link is broken at https://docs.littlevgl.com/en/html/get-started/quick-overview.html#learn-the-basics
For the "Coding style guide"

wrong link is https://github.com/littlevgl/lvgl/blob/master/docs/CODING_STYLE
correct link is https://github.com/littlevgl/lvgl/blob/master/docs/CODING_STYLE.md

In additon to the "Coding style guide" link is broken at https://docs.littlevgl.com/en/html/get-started/quick-overview.html#learn-the-basics

It seems to be correct in the Git repo so my guess is that @kisvegabor just hasn't rebuilt the docs  yet. The fix should be public shortly.

lv_templ: new destination/directory: src/

Thank you!

It seems the doc compiler truncated the .md from the link.
I've fixed and rebuilt it.


Respect ty->usig in get_load_inst.

get_load_inst needs to consider the signed-ness of the type
that it's emitting a load instruction for. If the load is
unsigned, emit zero-extended load instructions.

This also has the unfortunate side-effect of breaking the varargs test.

Making gp_offset and fp_offset of struct __va_elem type int (instead of unsigned int) makes the tests pass; hmm...

Ahh, forgot about x86_64 zero-extension weirdness; fixed and passing tests again! Please review and pull.

You need to include a failing test case along with your fix.

Hm, thanks for the testcase request. I can only seem to make bc334b2n or 257c809/188b139 fail (but not both separately). I'll keep trying; I think it might be possible.

Seems like  257c809 / 188b139 are unnecessary. I'll close this pull request and open a new one with the minimal set of commits required.


Url Rewrite - No distinction between global and regular rules.

Currently global rules are treated like normal rules, while rather they should be treated to operate on both the path and path base (AFAIK).
Global rules:

 – Rules in this collection can be defined only on the server level. Global rules are used to define server-wide URL rewriting logic. These rules are defined within the ApplicationHost.config file, and they cannot be overridden or disabled on any lower configuration levels. Global rules always operate on the absolute URL's path (that is, the requested URI without the server name). These rules are evaluated early in the IIS request-processing pipeline (PreBeginRequest event).

Wait, what requires forwarding? global rules? I don't see that in the description, only that it mutates the full url.

@Tratcher you may be technically correct (i'm not sure, to be honest) and that's what i have implemented. for my use-case, we are using global rules specifically to create a reverse proxy. i went ahead and created a reverse proxy middleware which works for single requests but i'm concerned about concurrency and scale.
public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory msLoggerFactory, ILogger logger)
{
	msLoggerFactory.AddSerilog(logger);
	RewriteOptions options = new RewriteOptions().AddIISUrlRewrite(env.ContentRootFileProvider, "RewriteRules.xml");
	app.UseRewriter(options);
	app.RunReverseProxy();
}

without adding app.RunReverseProxy();, the server returns a 404 which makes perfect sense. we require the request forwarding.
i'm not really sure what the purpose of a global rule that rewrites the full url would be if it were not being used to implement a reverse proxy but i guess that's technically out of scope for this issue. it seems like a reverse proxy middleware (as i have implemented separately as seen above) would be a great additional sub-project for this repo. definitely interested to hear your thoughts.
i'll try to get the PRs for both issues (this one and https://github.com/aspnet/BasicMiddleware/issues/55) submitted by this weekend, if not sooner. still need to write some unit tests and do a git pull. you guys appear to have changed some of the code since i originally cloned the repo and fixed one of the bugs that i discovered and also fixed while doing this work.

Resolved in https://github.com/aspnet/BasicMiddleware/pull/169


Variational SVM Minibatching

What is the expected behavior?
Stochastically select a fixed mini-batch of the training set during each training evaluation of the cost function, rather than running over every sample during each cost evaluation. This is a standard method in machine learning.

The minibatching success rate problem, for which I believe this was re-opened, was fixed by PR #463


ENH make a version without the sysroot constraint

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin rerender

@ktlim @brianv0 This one is ready for review!
I had to change the pin-it script slightly to alias the old name in order to preserve the API.
Once this goes in, it should allow folks to build subsets of the stack on the conda-forge default sysroot.
cc @esheldon

What's the merge split into split?  Should that maybe be a rebase instead?

That was me pulling down the changes from the rerendering. We can squash and merge all commits.

Any other comments here @ktlim or should we merge?


Connection to ledger does not work

:link: zboto Link

Connection to ledger live works correctly, but it doesn't work on our platform
Stacks version on Ledger: v0.23.3

Hi @Eranity , please follow this thread (https://github.com/Zondax/ledger-stacks/issues/171), we've released a new version which is currently in testing. I'd request you to uninstall the app, and install the one available on Ledger Live 0.24.2. That should work. Once we roll-up the latest update you should be able to upgrade your app. Thanks!

Stacks 0.24.5 have been release last week on Ledger Live, can we please close this issue? @Eranity

Hello, thanks for the answer. Yes, we can close this usse.


Plot genes without highly_variable_genes

I'm new to scanpy, and I want to plot umap with some genes.
In this tutorial, it's written below

sc.pl.umap(adata, color=['CST3', 'NKG7', 'PPBP'])

But, I could show only highly variable genes, because other genes were discarded by the code below

adata = adata[:, adata.var.highly_variable]

So, how can I plot umap with genes without highly variable? How can I leave all genes in anndata?

You can keep the genes in adata.raw, make a copy of the anndata object (adata_old = adata.copy()) before subsetting, or give the hvg one a new name like adata_hvg = adata[:, adata.var.highly_variable].copy()

OK, I try it. Thank you so much.


Ability to change the object names / object name prefixes

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
I am currently syncing a folder as per the static website guide, in a folder with a path like ../landingpage/dist/ - this makes both each bucket item a bit annoying, for a resource-name perspective (i.e landingpage-synced-folder-../landingpage/dist/index.html) - would have preferred an option to change these, i.e just a function mapping path -> ressource_name.
Furthermore, the actual bucket object names get a peculiar path: bucket/../landingpage/dist/index.html.
Wanted behavior
Could one either get the possibility (via a lambda) to manually set these names and/or simply a flag to set the name relative to the path? (which would result in landingpage-synced-folder-index.html).

The peculiar path name is likely caused by this line here, where the provider assumes the path does not have a trailing slash


Difficulties adding new field to end of form

I am using Chrome. Video shows best.
http://screencast.com/t/pGu3cN7VKqZ

Please contact bootsnip guys. I didn't make the bootstrap 3 version.


Fixes #38

Edit readme to add proper documentation on NPM dependencies to install.

Apologies. I am not also entirely in the Node loop, but I looked it up and those still should be installed.
I was required to install websockets at the very least, and didn't notice the node_modules directory, but it seems that I'd have to do an npm install from there if I didn't globally install websockets anyway.
I've updated the PR for this, but let me know if it should be working a different way.

Having installed websocket, I still can't get browserlink to work. I tried running node browserlink.js before and after installing websocket, and these two warnings dissapeared.
Warning: Native modules not compiled.  XOR performance will be degraded.
Warning: Native modules not compiled.  UTF-8 validation disabled.

But the remainder of the error message didn't change:
events.js:182
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::9001
    at Object.exports._errnoException (util.js:1024:11)
    at exports._exceptionWithHostPort (util.js:1047:20)
    at Server.setupListenHandle [as _listen2] (net.js:1319:14)
    at listenInCluster (net.js:1367:12)
    at Server.listen (net.js:1467:7)
    at Object.<anonymous> (/Users/cog/.vim/bundle/browserlink.vim/browserlink/browserlink.js:75:8)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)

Oh, yeah, this thing doesn't work, never did.


Guzzle 6

Hey,
I notice this library is constrained to guzzle 5 because that is what the oauth subscriber supports.
Personally i'm not using oauth to authenticate my sylius based api, so this is a shame for me that i'm constrained to guzzle 5.
There is no guzzle 6 oauth2 subscriber unfortunately, so upgrading is not trivial.
Is there any appetite to get this library working with guzzle 6?
This would require a new subscriber or upgrading commerce guys.
I don't have much motivation to do this however, as I'm not using oauth2...

@pjedrzejewski just wondered, was there a discussion about not using oauth2 for the sylius api? i seem to remember reading something but can't find it now

@michalmarcinkowski maybe we can maintain 2 branches here, one for guzzle 5 and one for 6?

@bendavies I added guzzle-6 branch, feel free to open a PR with changes necessary to make it work with Guzzle 6.

Then issue is that we can't use the current oauth2 subscriber with guzzle6. So i wouldy remove it as a dependency.


retry a few times if the file number of html report is suspicious

should close #14.
it appears that the OLMS web server sometime returns the wrong response when we ask for the detailed html form. so we check that the file number is the same as the srNum (which comes from the base info), and if it is not we retry three times.
for the cases that it gave us the wrong response, this seems to work the very second retry.
unfortunately, there are some cases where the srNum and the file number are validly different, and so after three retries, we accept the info from the html response.

@hancush, can you please review.


Why not recommend Mutagen for Windows WSL2 installations?

I am just getting up to speed on Mutagen, but it seems interesting to me. I haven't really noticed the benefits yet, but only enabled today on my MacOS development machine. :)
I have never been completely happy with performance on Windows and I'm curious why Mutagen isn't recommended for a Windows WSL2 installation?
My Windows performance issues are all connected to using Windows native applications (PHPStorm and Git Fork). The PHP and MySQL execution inside of the WSL2 container (Ubuntu) is perfect. PHPStorm is somewhat usable, but Git Fork is completely unusable unless I install the wslgit utility (which pipes git commands into the WSL2 container's native git).
I'm also not a huge fan of having all the project files stored only on the WSL2 container file system instead of the Windows native file system.
So if I use choco to install ddev in Windows, instead of brew to install inside of WSL2, would Mutagen work to let me leave the files on the native Windows file system, but still execute PHP and run the database inside of a Linux container?
Or am I missing something?

If you're running ddev inside WSL2, you'll find that the performance is as fast as mutagen can provide, at least with my testing.
If you're having trouble with performance of Windows-side applications like PhpStorm and GitFork, mutagen (as provided with ddev) has absolutely no impact on those. The problem is with those apps reaching across from windows essentially over a network mount, which is slow. You may be interested in reading https://ddev.com/ddev-local/ddev-local-and-phpstorm-debugging-with-wsl2/

If you prefer to use ddev on the Windows side, it is vastly improved by mutagen, but the initial sync is dreadfully slow. But after that it's wonderful.  But that has nothing to do with WSL2 - it's running on the Windows site.
You're welcome to use mutagen on WSL2. It does no harm. Our CI tests mutagen on Linux, it works perfectly. But it doesn't deal with the problem you're experiencing, Windows side applications reaching across into the WSL2 filesystem.
(And if you enabled mutagen on macOS and didn't immediately see the benefits, something is terribly wrong. )

So if I use choco to install ddev in Windows, instead of brew to install inside of WSL2, would Mutagen work to let me leave the files on the native Windows file system, but still execute PHP and run the database inside of a Linux container?

Yes, it works great. Not as fast as WSl2, but yes, it works, slow syncing. But your issue was "Why not recommend mutagen for Windows WSL2 installations", which is a completely different topic.

Sorry, I'm still confused :)
If I put my PHP project files in a Windows directory, install ddev with choco, configure ddev to use Mutagen, then run ddev start --- wouldn't ddev create Docker container(s) for MySQL and and web, use Mutagen to sync the files from Windows into the web container, and execute PHP inside web container at the same WSL2 speeds as if I installed the PHP project files inside of the Ubuntu container myself?
I can tolerate an initial sync that is slow, but then I would have full speed Fork and PHPStorm access to the source project files since they are on the native Windows FS. Or will a I see a slowdown of PHP execution because the web container was created and initialized from the Windows version of ddev instead of the Linux version?

You'd have to think long and hard about the consistency issues that might result.

Understood, but I don't modify any project files except through PHPStorm and git. Only exception is CSS building (gulp and SASS). I think PHPStorm might execute the CLI commands inside the container, so I may need to manually configure some of that if PHPStorm is no longer using the //wsl$/ path.

I will give it a try and let you know what I find :)

So far so good. Installation was simple. After a git clone (executed through Git Fork), the initial sync of a Drupal 9 site took about 50 seconds. Performance seems good, but I didn't do any real benchmarks or comparisons yet. Git Fork performance is very fast. PHPStorm seems a little quicker as well. AND all my project files are in the native Windows file system.
One issue that might be more of a Mutagen issue in this environment -- the eXecution bit of the drush files inside of /vendor/ is not being set properly. So I'm currently getting permission denied errors when I try and run ddev drush commands.

If you're using ddev drush then you should see the permissions on /var/www/html/vendor/bin/* be fine (inside web container). Not sure what you're doing. If you're then trying to relay that from WSL2 into Windows you're likely to have some trouble, because windows doesn't have the concept of permissions bits.

I haven't been able to follow exactly what you're doing now. Are you using mutagen-enabled on Windows, ignoring WSL2? In that case, you may be looking at the symlinks in vendor/bin on the windows side using git-bash or whatever. Not only does Windows not really have the same permissions case as Linux (what is in the container) but symlink ownership and permissions don't translate directly. https://github.com/mutagen-io/mutagen/issues/282 is probably related.

I'm trying to keep the project files primarily inside of the Windows filesystem instead of putting them inside of the Ubuntu filesystem (my WSL2 Linux VM). I do have WSL2 installed and I believe that is recommended for Docker performance anyway. I'm just not using the Ubuntu/Linux CLI when I run ddev commands. Instead I'm using Windows PowerShell (or git-bash) to execute ddev commands.
Reasons for me to explore this are to avoid hacks to Windows GUI apps (Git Fork) and hopefully improve PHPStorm performance a little (although I don't notice any major improvements there). This is to avoid having Windows apps accessing the project files through the //wsl$/ path (which is known to be very slow and MS doesn't seem to be prioritizing any fixes). I also have some issues with WSL2 and my VPN (Cisco AnyConnect). There is a known issue that ends up breaking all network connectivity inside the WSL2 VM when the VPN connects.
I did solve my drush permissions issues. We (unfortunately) are committing vendor and other composer installed filed into our repo. So the simple solution was to delete the vendor directory and run ddev composer install after the git clone. This way, composer takes care of file permissions inside of the ddev container. I could probably be more selective with which composer packages I force to re-install, and there might be some .sh files in Drupal core that aren't getting their +x enabled.
Overall, it's working (using PowerShell instead of the WSL2 virtual machine CLI). Performance appears good all around. I do notice odd pauses when I try and execute ddev commands, but I suspect that is my Windows laptop or possibly security related software being nosey in the background. ddev xdebug on too upwards of 7 to 10 seconds to complete.
I do have to do more testing, but so far everything seems to work well with ddev and Mutagen and only using the Windows native file system. Having to run ddev composer install for every new developer installation is going to be problematic, but that could also be solved by removing composer packages from our git repo.

Hi @JonMcL -
First, let's keep the terminology straight. WSL2 for Docker is a completely different subject. Although Docker Desktop supports the legacy Hyper-V technique, it's basically obsolete and has nothing to do with these topics. Anybody using current Docker should use the WSL2 docker back-end. But that has nothing to do with DDEV or mutagen.
If you want to work on the WIndows filesystem, you absolutely need to work on the Windows side, use the Windows DDEV, and essentially ignore WSl2, (even though we know that Docker is using it).
And yes, DDEV works fine with mutagen on Windows, after you put up with the initial sync.
Thanks for changing the title. I'll go ahead and close this now, happy to continue the conversation here though, and would love to hear your reactions after using it for awhile.


bump pretty-bytes to ^4.0.2

pretty-bytes has a nth-level dependency with a security vulnerability (parse-json which vendors in unicode.js) bumping pretty-bytes significantly reduces the number of dependencies.
npm list results:
before:
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ ├──<EMAIL_ADDRESS>│   │ └──<EMAIL_ADDRESS>deduped
│   ├──<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ ├─┬<EMAIL_ADDRESS>│   │ │ └──<EMAIL_ADDRESS>│   │ └──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>deduped
│   ├─┬<EMAIL_ADDRESS>│   │ ├──<EMAIL_ADDRESS>│   │ ├─┬<EMAIL_ADDRESS>│   │ │ └──<EMAIL_ADDRESS>│   │ ├──<EMAIL_ADDRESS>│   │ └─┬<EMAIL_ADDRESS>│   │   ├─┬<EMAIL_ADDRESS>│   │   │ └──<EMAIL_ADDRESS>│   │   └──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ ├─┬<EMAIL_ADDRESS>│   │ │ ├─┬<EMAIL_ADDRESS>│   │ │ │ └──<EMAIL_ADDRESS>deduped
│   │ │ └─┬<EMAIL_ADDRESS>│   │ │   └──<EMAIL_ADDRESS>│   │ └─┬<EMAIL_ADDRESS>│   │   ├─┬<EMAIL_ADDRESS>│   │   │ ├──<EMAIL_ADDRESS>deduped
│   │   │ ├─┬<EMAIL_ADDRESS>│   │   │ │ └─┬<EMAIL_ADDRESS>│   │   │ │   └──<EMAIL_ADDRESS>│   │   │ ├──<EMAIL_ADDRESS>│   │   │ ├──<EMAIL_ADDRESS>deduped
│   │   │ └─┬<EMAIL_ADDRESS>│   │   │   └──<EMAIL_ADDRESS>│   │   ├──<EMAIL_ADDRESS>deduped
│   │   └─┬<EMAIL_ADDRESS>│   │     ├──<EMAIL_ADDRESS>deduped
│   │     ├──<EMAIL_ADDRESS>deduped
│   │     └──<EMAIL_ADDRESS>deduped
│   ├─┬<EMAIL_ADDRESS>│   │ ├─┬<EMAIL_ADDRESS>│   │ │ └─┬<EMAIL_ADDRESS>│   │ │   └─┬<EMAIL_ADDRESS>│   │ │     └──<EMAIL_ADDRESS>deduped
│   │ └─┬<EMAIL_ADDRESS>│   │   └──<EMAIL_ADDRESS>deduped
│   └──<EMAIL_ADDRESS>
after:
├──<EMAIL_ADDRESS>

@maxogden tests are passing, any chance to merge this and bump the patch version so folks can clear up security vulnerabilities?

@maxogden friendly bump.

I haven't had time to maintain this lately, anyone else wanna help do releases + testing? @mafintosh is also a contributor

It's been over a year now and the dependency on<EMAIL_ADDRESS>still produces a massive and completely unnecessary dependency tree for this module (and in turn any module which depends on nugget, eg electron through electron-download).
Please merge this!

@maxogden & @mafintosh
Friendly bump #2 ...
Please merge this.

@maxogden Friendly bump of this PR.
There are still some packages that use the package in this repo, and any time you use one of those packages you get an audit warning (and a dependabot warning on GitHub too).
I tried to fix this via npm audit fix and by adding overrides (see npmjs.com) to the package.json of the project I'm working on and neither method was a solution.
This PR is probably the best method to resolve the vulnerability, as I believe most packages that depend on this package would upgrade almost immediately.

@maxogden @mafintosh Friendly bump too.
Hope you'll see this!
I just tried to reach out to Max on Twitter, just in case.

published 2.0.2 with this PR


allow for multiple re-directs in Maricopa Vaccine County

If a data card requires multiple re-directs to reach the final URL, the scraper currently fails (I was assuming it would only ever re-direct once, but on the new data it requires two.)
This adds the ability to re-direct as many times as needed, by calling the _get_json() function until a valid URL is found.
This should be safe (never infinitely repeat) as if there's a bad URL/landing page, the _get_url() method should break, stopping any sort of loop. But, let me know if there are concerns about this.

This is a great change! Thank you


Adding string literals example to basic prop types

Thought it would be useful to have it there since it's very common. I know it's explained further in unions and enums with string values but for me, I didn't notice them first and needed to google it :smile

sure thing, thank you


KYC Integration

Code changes required to verify a user's phone number with the KYC service before continuing with project creation. If a user is not verified, they are to be redirected to console's phone verification page.

@pjcdawkins Looking for final review for KYC launch.


GPIO api updated

Notes:

Pull requests will not be accepted until the submitter has agreed to the contributer agreement.
This is just a template, so feel free to use/remove the unnecessary things

Description
A few sentences describing the overall goals of the pull request's commits.
Status
READY/IN DEVELOPMENT/HOLD
Migrations
If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.
YES | NO
Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[ ] Documentation

Deploy notes
Notes regarding the deployment of this PR. These should note any
required changes in the build environment, tools, compilers, etc.
Steps to test or reproduce
Outline the steps to test or reproduce the PR here.

GCC_ARM Build,
http://jenkins.pk-lhr.u-blox.net:8080/job/mbed/job/KM/job/Pull request build/29/console

Build Result: http://jenkins.pk-lhr.u-blox.net:8080/job/mbed/job/KM/job/Pull request build/33/console
Build is successful.


Security Vulnerability in jsonpath-plus Dependency

Description:
I am using the api package and recently encountered a critical security vulnerability caused by a transitive dependency on jsonpath-plus. This package is vulnerable to remote code execution (RCE) in versions prior to 10.0.7, as outlined in GitHub Advisory GHSA-pppg-cpfq-h7wr.
Issue:
When manually upgrading oas to 25.2.1 to mitigate the vulnerability, compatibility issues arise with @readme/api due to breaking changes introduced in oas versions between 25.0.2 and 25.2.1.
Request:
Please update the oas dependency to ^25.2.1 or later within @readme/api.
Steps to Reproduce

Install @readme/api.
Run npm audit to identify the vulnerability in jsonpath-plus.
Upgrade oas to 25.2.1.
Attempt to use @readme/api and observe compatibility issues due to breaking changes in oas.

References
GitHub Advisory GHSA-pppg-cpfq-h7wr


Confirm what structure to use for partial charge calculation

Do we want...

Initial geometry from assembly
Supercell after relaxation with UFF
Initial geometry after a few relaxation steps
....

I lean geometry after a few dft relaxation steps. That would be only a few minutes per MOF and avoids having to trust the forcefield


MCTS display

Using Graphviz (https://graphviz.org/) or NetworkX (https://networkx.org/), provide a MCTS viewer. This viewer will take a json file that describes the MCTS

MCST Display is available
It can be improved in many ways, but it is a long term work, without current specification.


Add 2 more NLP suppression levels, pause filter adaptation when output muted

Exporting my work on AEC so far, so I'm not deep in a silo.

Add more granular NLP levels
Pause filter adaptation while the output is muted

Thank you @strohel. This will hopefully improve the howling that happens after existing low-power mode, but I still need to look into solving more general howling that happens when the filter adaptation finds a bad local optima. I plan to do a bit more testing tomorrow.

I'm experimenting throttling of FIR filter adaptation based on the relative activity of the far end w.r.t. the near end, which is the purple dots in the plot below. The first 1/4 is where both sides are inactive, the 2nd with far-end active, the 3rd with near-end active, and the last 1/4 with both sides active.

Reverted the FIR filter adaptation pause based on far end detection. I will start simple with just the explicit hint for pausing filter adaptation and NLP level additions.

@strohel This is ready for review. It is basically back to where it was when the PR was opened.

Closing the PR; the aec-fixes branch will stay, that the webrtc-audio-processing crate will depend on. In the future, we will want to migrate back to the trunk of pulse-webrtc-audio-processing and merge this PR in, but not for now as it's going to be a lot of work having to validate all the processors that changed.


Missing references in some cases

TypeScript Version:  3.5.2

Search Terms:
Code
We have some code that looks like this:
./main.ts:
import { types } from './types';

type Test = types.Foo;

./types/index.ts:
import * as types from './types';
export { types };

./types/types.ts:
export type Foo = 1;

When I try to find references for Foo, I'm getting no results. "Go to definition" does  still work.
However, I can only reproduce this behaviour in our large TS project. I've tried and failed to create a reduced test case. If I start removing files from our large TS project, it will eventually begin to work, suggesting there's some sort of (memory?) threshold which we could be crossing.
Here is a video recording to demonstrate this: https://www.dropbox.com/s/xw7c3lk0mxhr5vj/Screen Recording 2019-08-21 at 21.40.59.mov?dl=0
I have also attached a TS server log, which was recorded at the same time as the above video.
Any ideas what could be going on here? What's the best way for me to debug this?
tsserver.log

Can you please share the screen recording and tsserverlog again. (The screen share link is not working so I cant figure out what you expect and what is happening.)

Just found another bug which might be related: https://github.com/microsoft/TypeScript/issues/33489

@OliverJAsh From the log it looks like when finding references its hitting the assertion. We would need code sample to investigate so please provide small repro if you can or you can share your source code. We can get NDA for getting the repro project if you need to. Thanks.

I would be happy to share my code, it's in a private GitHub repository. I
guess I'll need an email address from you, for me to add to it?
I tried creating a reduced test case but it was too difficult unfortunately.
On Mon, 30 Sep 2019, 18:17 Sheetal Nandi<EMAIL_ADDRESS>wrote:

@OliverJAsh https://github.com/OliverJAsh From the log it looks like
when finding references its hitting the assertion. We would need code
sample to investigate so please provide small repro if you can or you can
share your source code. We can get NDA for getting the repro project if you
need to. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/TypeScript/issues/33017?email_source=notifications&email_token=AAHBACIEYJ67YPJGW6ND5ZTQMIYCDA5CNFSM4IOOFC7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76MP5Y#issuecomment-536659959,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHBACM3GF66HDG2646LQNLQMIYCDANCNFSM4IOOFC7A
.

email is shkamat at microsoft dot com

Hi @sheetalkamat, I have sent an NDA to you and @DanielRosenwasser.

@sheetalkamat I have sent you an invite to access the repo.
To test this issue, the file I used in my video is located at app/routes/Explore/LandingPage/components/PopularCategories/PopularCategories-helpers.ts.

@sheetalkamat Sorry I forgot to mention, this is on the maintenance/named-namespace-imports-landing-pages branch.

@OliverJAsh Thank you for providing details. I was able to repro this and will look into this soon.


[service-bus] fixes receiveMessages settle issue when disconnected

Fixes #6065 and replaces #6338
This PR updates the onDetached method to call the finalAction method of the ongoing receive operation.
This also changes the behaviour of the finalAction method so that if the receiveMode is receiveAndDelete and there is a detachedError, the promise will resolve with the messages received so far. This change was made because the messages have already been removed from service bus so the user can safely act on them.
Testing
I tested by running the following code with the changes in this PR:
import {ServiceBusClient, ReceiveMode} from '@azure/service-bus';

async function run() {
  const client = ServiceBusClient.createFromConnectionString(
    "ConnectionString"
  );

  const queueClient = client.createSubscriptionClient('TOPIC', 'SUBSCRIPTION');

  const receiver = queueClient.createReceiver(ReceiveMode.receiveAndDelete);
    try {
      const messages = await receiver.receiveMessages(1000, 1000 * 5);
      console.log('Everything worked fine!', messages.length);
    } catch (err) {
      console.log(`I encountered an error:`, err);
    } finally {
      console.log('I eventually was called.');
    }
}
run();

I set the DEBUG environment variable to *error and waited for the receiver to be established, then disconnected from the network. I immediately saw the network error in my catch block.
Putting this PR in draft so I can trigger CI as I was having some issues running the CI locally.

/azp run js - servicebus - tests


dynamically generate markup for added sites

Idea: use jekyll/eleventy/something-else to take in a json file and output the html. Instead of doing it by hand every single time.

Yeah, I've been thinking about doing this. The site is written by hand in HTML/CSS at the moment. I just need to set a couple of hours aside to migrate to Jekyll. At the moment it's a catch 22 where I have 10 mins here and there to add a few sites, but I don't have a couple hours to re-design it.
I really need to prioritise that though.

I'd do it. I'm more comfortable with Eleventy, but I could do Jekyll if you want.

I really appreciate the offer, but it will be a fun weekend project. 😄

Had some time last night/today and managed to get this done. Also tweaked a couple other bits while I was at it.


feat: add Azure OpenAI service support

Reference
Quickstart: Get started using ChatGPT (preview) and GPT-4 (preview) with Azure OpenAI Service
Test Screenshot

Thanks


Add Polish translation

Signed-off-by: Atrate<EMAIL_ADDRESS>

Thanks for your input to the app. Sorry for not seeing the pull request in time