🐛 [firebase_ml_vision] Unable to run Android app

Bug report
Describe the bug
When I am trying to run my app it fails with the error "Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.".
Error:
      aunching lib/main.dart on Android SDK built for x86 in debug mode...
      Running Gradle task 'assembleDebug'...

      FAILURE: Build failed with an exception.

      * What went wrong:
      Could not determine the dependencies of task ':app:compileDebugJavaWithJavac'.
      > In project 'app' a resolved Google Play services library dependency depends on another at an exact version (e.g. "[19.0.
      2]", but isn't being resolved to that version. Behavior exhibited by the library will be unknown.

      Dependency failing: com.google.android.gms:play-services-vision:20.0.0 -> com.google.android.gms:play-services-vision-co
<EMAIL_ADDRESS>but play-services-vision-common version was 19.1.0.

      The following dependencies are project dependencies that are direct or have transitive dependencies that lead to the art
      ifact with the issue.
      -- Project 'app' depends onto com.google.firebase:firebase-ml-vision@{strictly 24.1.0}
      -- Project 'app' depends on project 'firebase_ml_vision' which depends onto<EMAIL_ADDRESS>      0
      -- Project 'app' depends onto com.google.android.gms:play-services-vision-common@{strictly 19.1.0}
      -- Project 'app' depends onto com.google.android.gms:play-services-vision-image-label@{strictly 18.0.4}
      -- Project 'app' depends onto com.google.android.gms:play-services-vision@{strictly 20.0.0}

      For extended debugging info execute Gradle from the command line with ./gradlew --info :app:assembleDebug to see the dep
      endency paths to the artifact. This error message came from the google-services Gradle plugin, report issues at https://
      github.com/google/play-services-plugins and disable by adding "googleServices { disableVersionCheck = false }" to your b
      uild.gradle file.

      * Try:
      Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

      * Get more help at https://help.gradle.org

      BUILD FAILED in 1s

Steps to reproduce
I am not sure how to reproduce it as I did not make any changes to my project since two days. Today when I tried to ru the app it started failing. After that I updated all my dependencies to the latest one. But still it is not working.
Expected behavior
The app should be installed and open in the emulator or a physical device.

Flutter doctor
Click To Expand
[✓] Flutter (Channel stable, 1.22.3, on Mac OS X 10.15.7 19H15, locale en-US)
    • Flutter version 1.22.3 at /Volumes/myuser/Flutter/FlutterSDK/flutter
    • Framework revision 8874f21e79 (12 days ago), 2020-10-29 14:14:35 -0700
    • Engine revision a1440ca392
    • Dart version 2.10.3

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/myuser/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/myuser/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/myuser/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.10.0.rc.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.51.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (1 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

! Doctor found issues in 1 category.

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.10.3
Flutter SDK 1.22.3
lottery_pool_pro 1.0.0+1

dependencies:
- cached_network_image 2.3.3 [flutter flutter_cache_manager octo_image]
- cloud_firestore 0.14.3 [flutter meta quiver firebase_core firebase_core_platform_interface cloud_firestore_platform_interface cloud_firestore_web]
- cloud_functions 0.7.0+1 [meta flutter firebase_core firebase_core_platform_interface cloud_functions_platform_interface cloud_functions_web]
- cupertino_icons 1.0.0
- device_info 1.0.0 [flutter device_info_platform_interface]
- email_validator 1.0.6
- expandable 4.1.4 [flutter]
- firebase_admob 0.10.2 [meta flutter firebase_core]
- firebase_auth 0.18.3 [meta firebase_core firebase_core_platform_interface firebase_auth_platform_interface firebase_auth_web flutter]
- firebase_core 0.5.2 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_crashlytics 0.2.3 [flutter stack_trace firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface]
- firebase_messaging 7.0.3 [meta flutter firebase_core]
- firebase_ml_vision 0.9.9 [flutter]
- firebase_storage 5.0.1 [flutter firebase_core firebase_core_platform_interface firebase_storage_platform_interface quiver]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_signin_button 1.1.0 [flutter font_awesome_flutter]
- flutter_staggered_grid_view 0.3.2 [flutter]
- http 0.12.2 [http_parser path pedantic]
- image_cropper 1.3.1 [flutter]
- image_picker 0.6.7+14 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface]
- intl_translation 0.17.10+1 [analyzer args dart_style intl path petitparser]
- launch_review 2.0.0 [flutter]
- package_info 0.4.3+2 [flutter]
- permission_handler 5.0.1+1 [flutter meta permission_handler_platform_interface]
- purchases_flutter 1.4.3 [flutter]
- share 0.6.5+4 [meta mime flutter]
- shared_preferences 0.5.12+4 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- simple_connectivity 0.1.1 [flutter meta]
- url_launcher 5.7.10 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos url_launcher_windows]
- us_states 1.0.3
- xml2json 4.4.0 [xml]

dev dependencies:
- flutter_launcher_icons 0.7.5 [image args yaml]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- _fe_analyzer_shared 7.0.0 [meta]
- analyzer 0.39.17 [_fe_analyzer_shared args charcode cli_util collection convert crypto glob html meta package_config path pub_semver source_span watcher yaml]
- archive 2.0.13 [crypto args path]
- args 1.6.0
- async 2.5.0-nullsafety.1 [collection]
- boolean_selector 2.1.0-nullsafety.1 [source_span string_scanner]
- characters 1.1.0-nullsafety.3
- charcode 1.2.0-nullsafety.1
- cli_util 0.2.0 [path]
- clock 1.1.0-nullsafety.1
- cloud_firestore_platform_interface 2.2.0 [flutter meta collection firebase_core plugin_platform_interface]
- cloud_firestore_web 0.2.1 [flutter flutter_web_plugins http_parser meta firebase_core firebase_core_web cloud_firestore_platform_interface js]
- cloud_functions_platform_interface 3.0.1 [flutter meta firebase_core plugin_platform_interface]
- cloud_functions_web 3.0.1 [firebase_core cloud_functions_platform_interface flutter flutter_web_plugins firebase http_parser meta]
- collection 1.15.0-nullsafety.3
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- csslib 0.16.2 [source_span]
- dart_style 1.3.6 [analyzer args path source_span]
- device_info_platform_interface 1.0.1 [flutter meta plugin_platform_interface]
- fake_async 1.2.0-nullsafety.1 [clock collection]
- ffi 0.1.3
- file 5.2.1 [intl meta path]
- firebase 7.3.2 [http http_parser js]
- firebase_auth_platform_interface 2.1.3 [flutter meta firebase_core plugin_platform_interface]
- firebase_auth_web 0.3.2 [flutter flutter_web_plugins meta http_parser intl firebase_core firebase_core_web firebase_auth_platform_interface js]
- firebase_core_platform_interface 2.0.0 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.1 [firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_crashlytics_platform_interface 1.1.3 [flutter meta collection firebase_core plugin_platform_interface]
- firebase_storage_platform_interface 1.0.1 [flutter meta collection firebase_core plugin_platform_interface]
- flutter_blurhash 0.5.0 [flutter meta]
- flutter_cache_manager 2.0.0 [flutter path_provider uuid http path sqflite pedantic clock file rxdart]
- flutter_plugin_android_lifecycle 1.0.11 [flutter]
- flutter_web_plugins 0.0.0 [flutter characters collection meta typed_data vector_math]
- font_awesome_flutter 8.10.0 [flutter]
- glob 1.2.0 [async collection node_io path pedantic string_scanner]
- html 0.14.0+4 [csslib source_span]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- image 2.1.18 [archive xml meta]
- image_picker_platform_interface 1.1.1 [flutter meta http plugin_platform_interface]
- intl 0.16.1 [path]
- js 0.6.2
- matcher 0.12.10-nullsafety.1 [stack_trace]
- meta 1.3.0-nullsafety.3
- mime 0.9.7
- node_interop 1.2.0 [js]
- node_io 1.1.1 [node_interop path]
- octo_image 0.3.0 [flutter flutter_blurhash]
- package_config 1.9.3 [path charcode]
- path 1.8.0-nullsafety.1
- path_provider 1.6.24 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+6 [flutter]
- path_provider_platform_interface 1.0.4 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+3 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.2 [meta]
- permission_handler_platform_interface 2.0.1 [flutter meta plugin_platform_interface]
- petitparser 3.1.0 [meta]
- platform 2.2.1
- plugin_platform_interface 1.0.3 [meta]
- process 3.0.13 [file intl meta path platform]
- pub_semver 1.4.4 [collection]
- quiver 2.1.5 [matcher meta]
- rxdart 0.24.1
- shared_preferences_linux 0.0.2+4 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+11 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+3 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- sky_engine 0.0.99
- source_span 1.8.0-nullsafety.2 [charcode collection path term_glyph]
- sqflite 1.3.2+1 [flutter sqflite_common path]
- sqflite_common 1.0.2+1 [synchronized path meta]
- stack_trace 1.10.0-nullsafety.1 [path]
- stream_channel 2.1.0-nullsafety.1 [async]
- string_scanner 1.1.0-nullsafety.1 [charcode source_span]
- synchronized 2.2.0+2
- term_glyph 1.2.0-nullsafety.1
- test_api 0.2.19-nullsafety.2 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0-nullsafety.3 [collection]
- url_launcher_linux 0.0.1+4 [flutter]
- url_launcher_macos 0.0.1+9 [flutter]
- url_launcher_platform_interface 1.0.9 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.5+1 [url_launcher_platform_interface flutter flutter_web_plugins meta]
- url_launcher_windows 0.0.1+3 [flutter]
- uuid 2.2.2 [crypto convert]
- vector_math 2.1.0-nullsafety.3
- watcher 0.9.7+15 [async path pedantic]
- win32 1.7.3 [ffi]
- xdg_directories 0.1.2 [meta path process]
- xml 4.5.1 [collection convert meta petitparser]
- yaml 2.2.1 [charcode collection string_scanner source_span]

Duplicate of #3995

Closing this as a duplicate of #3995, please follow the mentioned issue


ImportError: cannot import name 'GWGenerator' from 'dragonnfruit.io'

I think the name was changed to GenomewideGenerator

Where is this issue arising?

I was following the code snippet in the README.
from dragonnfruit.io import GWGenerator

.
.
.

X = torch.utils.data.DataLoader(
	GWGenerator(
		sequence=sequence,
		signal=signal,
		neighbors=neighbors,
		cell_states=cell_states,
		read_depths=read_depths,
		trimming=(2114 - 1000) // 2,
		window=1000,
		chroms=['chr1'],
		random_state=None),
	pin_memory=True,
	num_workers=8,
	worker_init_fn=lambda x: numpy.random.seed(x),
	batch_size=128)


statically built ffmpeg shows ver v1.28 even though it was built from 4.4

Because of this ffprobe is also old.
I know the question is vague but can you choose the version you build statically?
the build command used was:
./build-ffmpeg --build --enable-gpl-and-non-free --full-static
thanks

The build script has been fetching the Latest snapshot of ffmpeg. I've never seen this type of result before, neither in this repository's Actions Tabs nor in my personal scratch script.
There may be some environment variable interfering with your build system. It's the only thing I can explain.

Try to change this?

Same problem with 1.31. I've even replaced 1.31 with 4.4 in "SCRIPT_VERSION=1.31" inside build-ffmpeg script, but final ffmpeg binary still 1.31 instead of 4.4. Weird this that there's no 1.31 text anywhere else in distributive.

Same for me. Mine got version number 1.33, the current script version.

Same here I have ffmpeg-build-script version 1.4 and the built ffmpeg is reporting,
ffmpeg version v1.40 Copyright (c) 2000-2022 the FFmpeg developers

when it should be 5.1 which the version of ffmpeg being downloaded and built.

If I modify ffmpeg-build-script to rename the SCRIPT_VERSION variable to BUILD_SCRIPT_VERSION, then instead of the built ffmpeg reporting the version of ffmpeg-build-script (e.g. 1.40) it now reports a version containing a hash:
ffmpeg version fb979f1 Copyright (c) 2000-2022 the FFmpeg developers
which is the hash of the most recent commit in my fork of the ffmpeg-build-script repo, so something in the ffmpeg build is being (too) clever about determining the version number.

After further investigation I can report that the problem with the reported version number occurs because of the way the ffmpeg build determines the version number to incorporate into the built artifact. In the ffmpeg source, version determination is performed by a shell script ffbuild/version.sh which looks for the presence of various files, and also determines whether the source is being built inside a cloned git repository.
If this repository (ffmpeg-build-script) is cloned, and the build performed within that clone, the ffmpeg build mistakenly picks up version information from the ffmpeg-build-script Git repository, rather than from the ffmpeg Git repository as it was designed to do.
The solution then, is to not execute build-ffmpeg from within a clone of the ffmpeg-build-script repository, but instead from a non-git repo directory. One way to do this is to stop the clone being a clone by deleting the .git directory after cloning:
$ git clone https://github.com/markus-perl/ffmpeg-build-script.git
$ cd ffmpeg-build-script
$ rm -rf .git/
$ ./build-ffmpeg --build --build --enable-gpl-and-non-free

After the build is complete, invocations of ffmpeg -version should now report the correct release version number:
$ ./workspace/bin/ffmpeg -version
ffmpeg version 5.1.2 Copyright (c) 2000-2022 the FFmpeg developers
...

Thanks for investigating this issue.
I'd like to recommend renaming the .git folder instead of deleting it. This way, it would be easier to update the cloned repo in the future by renaming the folder back again instead of repeatedly cloning:
$ git clone https://github.com/markus-perl/ffmpeg-build-script.git
$ cd ffmpeg-build-script
$ mv .git .git.bak
$ ./build-ffmpeg --build --build --enable-gpl-and-non-free

Rename back to update the repo
$ mv .git.bak .git
$ git pull

@rob-smallshire Thank you for investigating this issue. Depending on your use the easiest workaround is clearly to rename/delete the .git folder. In a future version of the build script we may can patch the version.sh file to just ignore the .git folder.


Feature/additive expert ensemble

Changes proposed in this pull request:

Implements Additive Expert Ensemble (AddExp) for classification
Kolter and Maloof. Using additive expert ensembles to cope with Concept drift. Proc. 22 International Conference on Machine Learning, 2005.

Checklist

[x] Code complies with PEP-8 and is consistent with the framework.
[x] Code is properly documented.
[x] Tests are included for new functionality or updated accordingly.
[x] Travis CI build passes with no errors.
[x] Test Coverage is maintained (threshold is -0.2%).
[x] Files changed (update, add, delete) are in the PR's scope (no extra files are included).

Thanks @AndreFCruz !
Was waiting for this PR to start preparing a new release :-)


x/vgo: no such file or directory

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
[adam@adams-macbook]$ vgo version
go version go1.10.2 darwin/amd64 vgo:2018-02-20.1

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
[adam@adams-macbook]$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/adam/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/adam/Scripts/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.10.2/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.10.2/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/9l/dqqy91t557j8lsrdv1w_krq00000gn/T/go-build692479027=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
cat <EOF>main.go
package main

import (
_ "github.com/mattes/migrate"
_ "github.com/mattes/migrate/database/cockroachdb"
_ "github.com/mattes/migrate/source/go-bindata"
)

func main() {
}
EOF
vgo vendor

What did you expect to see?
completed go.mod file
What did you see instead?
[adam@adams-macbook]$ vgo vendor
vgo: resolving import "github.com/mattes/migrate"
vgo: finding github.com/mattes/migrate (latest)
vgo: adding github.com/mattes/migrate v1.3.2
vgo: import "github.com/belugacdn/test" ->
	import "github.com/mattes/migrate/database/cockroachdb"<EMAIL_ADDRESS>open<EMAIL_ADDRESS>no such file or directory
vgo: import "github.com/belugacdn/test" ->
	import "github.com/mattes/migrate/source/go-bindata"<EMAIL_ADDRESS>open<EMAIL_ADDRESS>no such file or directory
vgo: import "github.com/belugacdn/test" ->
	import "github.com/mattes/migrate/database/cockroachdb"<EMAIL_ADDRESS>open<EMAIL_ADDRESS>no such file or directory
vgo: import "github.com/belugacdn/test" ->
	import "github.com/mattes/migrate/source/go-bindata"<EMAIL_ADDRESS>open<EMAIL_ADDRESS>no such file or directory
vgo: resolving import "github.com/fatih/color"
vgo: finding github.com/fatih/color (latest)
vgo: adding github.com/fatih/color v1.7.0
vgo: resolving import "gopkg.in/mattes/migrate.v1/driver/bash"
vgo: finding gopkg.in/mattes/migrate.v1 (latest)
vgo: adding gopkg.in/mattes/migrate.v1 v1.3.2

found this as part of a much larger project which is not using null imports, but, this reproducer was simpler and demonstrates the issue

CC @rsc @bcmills

@AdamJacobMuller the repro you provided doesn't seem like valid shell to me, so I've taken a guess at the following:
cd `mktemp -d`
export GOPATH=$PWD
mkdir hello
cd hello
cat <<EOD > main.go
package main // import "example.com/hello"

import (
  _ "github.com/mattes/migrate"
  _ "github.com/mattes/migrate/database/cockroachdb"
  _ "github.com/mattes/migrate/source/go-bindata"
)

func main() {
}
EOD

At this point we must create a go.mod to tell vgo we are a Go module:
echo > go.mod

Notice it will take the import path from main.go.
Now if we look at the available releases of github.com/mattes/migrate you will see the latest is v3.0.1. The thing to notice at this point is that this project has not been "converted" to be a Go module, that is there is no go.mod. This is important because it's major version is >= 2. As such for now we need to fallback to explicitly getting a version of github.com/mattes/migrate.
But then another thing to notice is that the CockroachDB support that you want to rely on has not actually made it to a release yet.
So taking these points together (project not converted to be a Go module, major version >= 2 and changes only available in master) we need to:
vgo get github.com/mattes/migrate@master

At which point we can see how go.mod has been changed:
cat go.mod

which gives:
module example.com/hello

require (
        github.com/cockroachdb/cockroach-go v0.0.0-20180212155653-59c0560478b7
        github.com/lib/pq v0.0.0-20180523175426-90697d60dd84
        github.com/mattes/migrate v0.0.0-20180508041624-4768a648fbd9
)

Notice the pseudo-versions being used, pseudo versions that correspond (at the time of writing) to the HEAD or master.
Finally do a build to check:
vgo build

I'll close this issue for now therefore because I think your expectation that vgo would create a go.mod file is not correct; you need to tell vgo where you want to define your module.
But if there are any other issues, please feel free to comment back here or open a new issue.

@myitcv thanks, I was not aware of the vgo get github.com/mattes/migrate@master syntax to fetch a specific release, works perfectly!
Thank you,
-Adam


Auth loop when connecting to tunnel with vscode.dev

Run code tunnel
Follow the auth steps
Open the vscode.dev link

🐛 There's an auth loop:
https://user-images.githubusercontent.com/22350/230055098-94df033e-77f0-4dd1-bb80-90d4d0613ce5.mov

Duplicates https://github.com/microsoft/vscode-remote-release/issues/8323


Enhance the benchmark runner to support multiple top level objects use case.

Description:
This PR adds support for multiple top level use case, so that we have an apple-to-apple performance comparison for Ion/JSON/CBOR.
Note:

This PR focuses on io_type=file, we might consider adding io_type=buffer later but we need to figure out how to fairly use JSON and CBOR's load/dump APIs to iterate each of multiple top level object - https://github.com/amazon-ion/ion-python/issues/325.

When we benchmark read speed for JSON document that includes multiple top level objects, we suppose that the given data is in JSONL. So we have to modify our JSON sample data - code here

I'm not sure if protobuf has the same issue for the multiple top-level objects use case. This need to be figure out when we come back to protobuf in the future - https://github.com/amazon-ion/ion-python/issues/326.

Three files I used for test locally, I upload them for visible but they will not be included when merge this PR into the main branch. They are:
1 amazon/ionbenchmark/json_newline_vs_list_repro.py
2 amazon/ionbenchmark/format_conversion_correct.py
3 amazon/ionbenchmark/format_conversion_inaccurate.py

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

File multiple_top_level_object.json and multiple_top_level_object.cbor includes equivalent data:
{"name":"John", "age":30, "car":null}
{"name":"Mike", "age":33, "car":null}
{"name":"Jack", "age":24, "car":null}

The commit 74d89ad refactor loader (read APIs) to separate files. And below are the new results for a log

name
file_size(B)
time_min(ns)
time_mean(ns)
memory_usage_peak(B)

Ion - after
22263038
5092575416.60
5109052881
276500

Ion - before
22263038

4730782811
171700

json - after
145330385
871989650.00
872934369
112139

json - before
145330385

875,974,056
77,446

cbor - after
116960762
985745625.00
991288347
87769

cbor - before
116960762

995,595,161
51,222

Below is the table showing the differences in write performance after the refactor.

name
file_size(B)
time_min(ns)
time_mean(ns)
memory_usage_peak(B)

Ion - before
22263038

2,509,895,173
42,573,352

Ion - after
22263038

4,139,845,225
42,573,352

json - before
145330385

3,981,317,292
143,755,789

json - after
145330385

4,018,459,501
143,756,301

cbor - before
116960762

1,303,162,089
19,301

cbor - after
116960762

1,291,955,078
19,813

Why is Ion read and write performance worse after the refactor?

Re,

I'm also not sure why IonLoadDump needs to change. My understanding is that we're not flushing the writer so it will just buffer until it closes, then flush the symbol table and values. Right?

I saw a 20% difference between dumps each top-level object and dump the whole document. I can take a look at that later.

The commit 2b177af addressed the comment. A few comments and highlights below.

Re: https://github.com/amazon-ion/ion-python/pull/315#discussion_r1442404936
It seems that the generator can't be fully consumed by test_fun; it will only parse the first value and return when I pass a generator to it. I refactored the benchmark_spec.get_data_object() method to return a generator and reused it in other places to avoid repeating code. However, for benchmarking, we still call list(the_generator) to convert it into a list before passing it to test_fun. Do you have any concerns or comments about this?

Re: https://github.com/amazon-ion/ion-python/pull/315#discussion_r1442408471
I'm going to investigate why this happens and determine which benchmarking methods are more accurate. Based on the previous results, there is a 20% difference between these two approaches.

After the refactor, the Ion write performance dropped from 2e^9 to 4e^9 I'm not sure why; I'm investigating it now.

There are some conflicts with the main branch. (I guess it's because of the recent bare_value PR) I'll work on them at the end.

It seems that the generator can't be fully consumed by test_fun; it will only parse the first value and return when I pass a generator to a defined test_fun. I refactored the benchmark_spec.get_data_object() method to return a generator and reused it in other places to avoid repeating code. However, for benchmarking, we still call list(the_generator) to convert it into a list before passing it to test_fun. Do you have any concerns or comments about this?

Can't we just pull off the generator and do nothing with the values in the test_fun?

The highlight No.3 in my comment above is solved and I updated the metrics table for dump APIs.
This is because for the before metrics, I checked out the latest ion-python commit to see how much improvement we have made for the write side. However, since the PR is not merged yet so that the after result is based on the latest released version 0.11.3. I updated it. In the future I will use this commit d425587 for benchmarking comparison in this PR (the current latest commit).

Investigating the build failure issue, it worked well locally.


Modifying the official documentation

When I ran the code from the documentation below, the 'insufficient_scope' error bothered me for a long time. After a period of debugging, it is found that the SQL file is missing a field, resulting in the authorized scope can not be saved into the database.
https://docs.spring.io/spring-authorization-server/docs/current/reference/html/guides/how-to-jpa.html#authorization-schema
https://docs.spring.io/spring-authorization-server/docs/current/reference/html/guides/how-to-jpa.html#authorization-entity
As I said above, the authorizedScopes field is missing in the SQL file.

#1010


Create the navigation graph, the fragments and the layout for the fragments.

the bug when clicking on the add button in the main fragment must be solved.
The user should be navigate to the add fragment but nothing happens

Issue was solved


Cannot show child tasks in multi level tree structure

I am trying to display a tree structure, but the Gantt chart only shows first-level children. My tree data has three to four levels. Is it possible to display a multi-level tree structure?

Data Structure:
->A
    ---->B
          ---->BA
          ---->BB
    ---->C
->D
->E
    --->F

I cannot see BA and BB node in the gantt display.

try this
<ngx-gantt [maxLevel]="urmaxlevel"></ngx-gantt>

Does version 12.2.3 already have this change?

Does version 12.2.3 already have this change?


Optionally log file and line number with each statement.

When adding new log messages, it would be great to have the file and line number printed with each statement. I think this could be as little as adding them to the log struct and then utilizing them in the formatter.
Maybe this is available already somehow?
https://github.com/reidmorrison/semantic_logger/blob/fd532c129c2bb664dfa2ce83731dc309f7ec1d88/lib/semantic_logger/base.rb#L77

Actually, maybe this is already available somehow?
https://github.com/reidmorrison/semantic_logger/blob/fd532c129c2bb664dfa2ce83731dc309f7ec1d88/lib/semantic_logger/base.rb#L77

I looked into this capability and agree that it would be very useful. The challenge is that __FILE__ and __LINE__ are defined at the point in the code they are used. I.e. By using these values inside of SemanticLogger would cause it to return the SemanticLogger file and line number rather than the file and line number of the caller.
The only place I have seen someone grab the file and line number without forcing the caller to pass it on every log entry would be to grab the stack trace by creating a new Exception. Unfortunately grabbing the stack trace is a relatively expensive operation.
Otherwise the only way to do it right now is to add it to every log entry:
logger.info("Hello there", file: __FILE__, line: __LINE__)

Running the above in a Rails console:
 Rails.logger.info("Hello there", file: __FILE__, line: __LINE__)

Results in the correct filename and line number be written to the log file:
2015-09-15 19:51:27.706031 I [13857:70365821133320] Rails -- Hello there -- {:file=>"(irb)", :line=>2}

Makes sense, and matches my understanding of the issue. I made a clunky workaround by using a method proxy around the logger which injects the line number using the caller kernel method. Works well enough, though performance isn't a huge issue for me. Here's what it looks like:
class Proxy
  def initialize(logger)
    @logger = logger
  end

  def method_missing(m, *args, &block)
    location = caller[0].split('/').last
    file, line = location.split(':')

    args[0] = "(#{file}:#{line}) #{args[0]}"
    @logger.send(m, *args, &block)
  end
end

Great idea to help narrow down log messages.
I just researched Kernel#caller and it is unfortunately expensive. Maybe we could limit it to error and above log messages.
What we have done is to give every class it's own logger instance using 'include SemanticLogger::Loggable' so that the class is added to every log message and then we can search for the message in that file. Not a perfect solution but it avoids the expense of calculating the stack trace. In our batch environment we were processing over 11,000 log messages per second ( before silencing info level messages ).
Wish there was a way in Ruby to quickly get to the caller without the need for obtaining a full stack trace.

Another use case where the stack trace would be very useful is when submitting an error to Bugsnag it would be nice to know where the error came from.
We could create a global configuration option to set the level at which to collect stack traces. If the stack trace is present, then it could log the filename and line number. And for Bugsnag or other event systems we can forward the stack trace directly to them.
So in low volume environments it could be dropped to :debug or :info and then in production it would be set to :error level.

Busy with the change to support capturing stack traces within the context of the caller. In Rails add the following to config/environments/development.rb:
  # Log level, suggest :trace or :debug in develooment
  config.semantic_logger.default_level = :trace
  # To add the filename and line number to all log messages
  # Recommend :error, :warn, or :fatal in production
  config.semantic_logger.backtrace_level = :trace

Or if running standalone:
  SemanticLogger.default_level = :trace
  # To add the filename and line number to all log messages
  # Recommend :error, :warn, or :fatal in production
  SemanticLogger.backtrace_level = :trace

@UnquietCode Can you give the master branch on github a try and let me know what you think about the implementation and if you have any feedback?

I am currently running on master and this feature is working great! I'm trying to debug another issue where using payloads is causing the logging statement to fail silently. If I can get a test setup going then I should be able to get you a failing test. Is there an internal logger which might be capturing internal errors?

Ok, that issue was unrelated (#19). This is working well for me. Thanks for adding the top level configuration option which, as you say, lets me squeeze a little more performance out of production environments.


Enable writing universal resolution out to requirements.txt

I think this is possible (and perhaps not even that hard) given the format we settled on. It would be a pretty neat feature.

I will try this some time for fun.

This is now supported.


Uncaught TypeError in Code

There is an uncaught TypeError in your code.
Image of Error

Possible Cause
The URL doesn't contain a fragment. This is a URL fragment:

This is the URL of the website when the problem occurred:

Possible Fixes

Make your code more defensive (i.e. check for all cases)
You can't actually use GitHub Pages because...

Bro....


Roll back to old PowerShell syntax highlighting

VSCode Version: 1.3.1
OS Version: All

For the 1.3.0 release of VS Code we introduced a new syntax file for PowerShell which ended up having a few critical bugs.  Since we keep finding more issues with the new syntax file I think it's probably good to revert back to the old one before finishing the 1.4 release.  I know you're in lock down mode right now, but is there any possibility that we could revert for this release so that users aren't troubled by the issues anymore?
I'll go ahead and submit a PR, feel free to accept or reject based on what you decide.
Thanks!
/cc @aeschli @egamma

Pull request here: https://github.com/Microsoft/vscode/pull/9923

Thanks guys!


Add field to telecommands table: readiness_level

Task Steps

Create an enum called TCMD_readiness_level_enum_t, with options:

// enum options:
TCMD_READINESS_LEVEL_IDEA_PHASE
TCMD_READINESS_LEVEL_NOT_IMPLEMENTED
TCMD_READINESS_LEVEL_IN_PROGRESS

// Especially for commands which only work over the umbilical UART:
TCMD_READINESS_LEVEL_GROUND_USAGE_ONLY

// For commands like "test the flash memory", which corrupts the filesystem but is flight-safe:
TCMD_READINESS_LEVEL_FLIGHT_TESTING

// For commands which are intended for normal operation in flight:
TCMD_READINESS_LEVEL_FOR_OPERATION

Add this enum to the command registration table (name: readiness_level)
Add an enum value to all existing telecommands

Reason
This field is mostly for documentation generation. We may also disable TCMD_READINESS_LEVEL_GROUND_USAGE_ONLY (and below) in-flight, or may require a special key to trigger those commands.

Fixed in #102


Failed to run Flutter Gallery on iOS Simulator

This issue seemed to be only affecting Flutter Gallery on my Google Corp Mac.
Steps to Reproduce
$ flutter run
Running "flutter packages get" in flutter_gallery... 1.3s\
Launching lib/main.dart on iPhone SE in debug mode...
Running Xcode clean...                               0.8s-
Running Xcode build...                               1.3s-
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
    diff: /Podfile.lock: No such file or directory
    error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.
Could not build the application for the simulator.
Error launching application on iPhone SE.

Flutter Doctor
Paste the output of running flutter doctor -v here.
$ flutter doctor -v
[✓] Flutter (on Mac OS X 10.13.3 17D47, locale en-US, channel master)
    • Flutter version 0.1.2-pre.17 at /Users/taodong/Code/flutter_repos/flutter
    • Framework revision 59fe5d0f46 (45 minutes ago), 2018-02-14 15:25:12 -0800
    • Engine revision f8778fce17
    • Tools Dart version 2.0.0-edge.3c4dccbd46f152be9e1b6ca95c57357e8e48057c
    • Engine Dart version 2.0.0-edge.3c4dccbd46f152be9e1b6ca95c57357e8e48057c

-

The process hung at this point.
I fixed the problem by opening Xcode and let it install "additional components". It would've been helpful if flutter doctor could tell me what I need to do.

Correction: Installing additional components from Xcode didn't really fix the problem.  I'll investigate more.

@InMatrix Were you able to run the gallery app on iOS simulator? I'm facing a similar problem in running the "platform_view" example app on an iOS simulator.


Disable unused SDL loggers

Fix: #175
This PR is ready for review.
Summary
Switch off TransportManagerLog and ProtocolFordHandlingLog SDL loggers
CLA

[x] I have signed the CLA

@dboltovskyi I like this idea a lot, but I think another approach can have more benefits.
If we use the log4j appenders property Threshold we can prevent log4j from trying to write anything at all, instead of just saving on the write speed by using /dev/null
Adding these two properties to the log4cxx.properties file would achieve the same effect and more:
log4j.appender.TransportManagerLogFile.Threshold=OFF
log4j.appender.ProtocolFordHandlingLogFile.Threshold=OFF

@iCollin Thank you for suggestion - it's really better than mine. I did an update in d848a7a


Changed azurecr references to ghcr

Thank you for helping make the Radius documentation better!
Please follow this checklist before submitting:

[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Changed azurecr references to ghcr
Auto-generated description

🤖 Generated by Copilot at 5cb6d26
Summary
🚚📝🆕

This pull request updates the documentation and code snippets to use the new public registry radius.ghcr.io instead of the old one radius.azurecr.io. This change affects the guides for installing Radius on Kubernetes, authoring and registering custom recipes, authoring apps with various resources and networking options, and deploying apps with environments.

Oh, we're the crew of the radius ship
And we sail the cloud with skill and wit
We've changed our registry from azurecr to ghcr
So pull the ropes and update the docs, me hearties!

Walkthrough

Update image URLs for demo app, AWS app, and volumes app from radius.azurecr.io to radius.ghcr.io in various code snippets and guides (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
Update image URL for Azure workload identity app from ghcr.io to radius.ghcr.io in container-wi.bicep code snippet (link)
Update template paths for MongoDB and Redis recipes from radius.azurecr.io to radius.ghcr.io in rad recipe list output (link)
Update Helm repository URL from radius.azurecr.io to radius.ghcr.io in Kubernetes installation instructions (link)
Update image URLs for Radius resource provider, deployment engine, and universal control plane from radius.azurecr.io to radius.ghcr.io in Helm configuration options (link)
Update target path and template path for custom recipe from myregistry.azurecr.io to myregistry.ghcr.io in recipe authoring instructions (link, link)

Issue reference
https://github.com/radius-project/docs/issues/889

Thanks for finding these! Our new registry is ghcr.io/radius-project instead of radius.ghcr.io. Could you update this PR to use ghcr.io/radius-project please?


Distube v4 is not creating a queue when using distube.play

When using the distube.play method, everything works, the bot is playing music in a voice channel. But when i try to check the queue or stop the song I get the following: DisTubeError [NO_QUEUE]: There is no playing queue in this guild
Further information:
This is the code i'm using:
const { SlashCommandBuilder } = require('discord.js');
const { DisTube } = require('distube');
const { SoundCloudPlugin } = require('@distube/soundcloud')
const { YtDlpPlugin } = require('@distube/yt-dlp');
const logger = require('../utils/logger');

module.exports = {
    data: new SlashCommandBuilder()
        .setName("play")
        .setDescription("play music")
        .addStringOption(option => option.setName('song').setDescription('For urls, supported sites: Soundcloud & Youtube'))
        .addStringOption(option => option.setName('modify').setDescription('Modify the playback, use stop to stop playing music')),
    async execute(interaction) {

        const distube = new DisTube(interaction.client, {
            searchSongs: 5,
            searchCooldown: 30,
            leaveOnEmpty: true,
            leaveOnFinish: true,
            leaveOnStop: true,
            plugins: [
                new SoundCloudPlugin(),
                new YtDlpPlugin()
            ]
        });
        const guild = interaction.client.guilds.cache.get(interaction.guildId)
        const member = guild.members.cache.get(interaction.member.user.id);
        const voiceChannel = member.voice.channel;
        distube
            .on('playSong', (queue, song) =>
                queue.textChannel?.send(
                    `Playing \`${song.name} - ${song.formattedDuration
                    }\`\nRequested by: ${song.user}`,
                ),
                // queue.addToQueue(song)
            ).on('error', (textChannel, e) => {
                logger.log('error', e);
                textChannel.send(
                    `An error encountered: ${e.message.slice(0, 2000)}`,
                );
            })
        if (voiceChannel && interaction.options.getString('song')) {
            distube.play(voiceChannel, interaction.options.getString('song'), { guild, textChannel: interaction.channel, member: member }).catch((error) => {
                logger.log('error', error);
                interaction.reply("something has gone wrong, did you provide a valid URL?");
            });
            interaction.reply("Searching for your song... 🎵");
        } else if (interaction.options.getString('modify') == "stop" && voiceChannel && !interaction.options.getString('song')) {
            distube.stop(guild).catch((error) => {
                    logger.log('error', error);
                    interaction.reply("something went wrong when stopping the music...")
            });
            interaction.reply('stopped the music!');
        } else if (!voiceChannel) {
            interaction.reply("You must join a voice channel to use this command!");
        } else {
            interaction.reply("Something has gone wrong, please check your command.");
        }
    }
}

Any help is appreciated!

 It should be created once for each client.
Check the example bot or join our support server to discuss about your problem.


Add VoD playback

You should be able to watch recorded streams, or VoDs.

I just tried this with the existing build flag (VOD_ENABLED) and found out that it seems to be working. When this issue was created, I temporarily gave up on the VoD feature, as for some reason the Twitch stream would not play in Sila, nor in visionOS Safari. It seems that has been fixed now; it must have been on Twitch's side.


Tax Rates, Tax Rules, Shipping Taxes Previlidiges

Bug Report
Have a look here:
https://github.com/bagisto/bagisto/blob/c1dceb0a011df016562c3d5461be6130375e3145/packages/Webkul/Checkout/src/Cart.php#L582-L591
$shipping->price is the total shipping amount of the cart in total.
If the shipping method has specified is_calculate_tax, then each $item->tax_amount will contain the tax of the shipping costs as a whole (calculated with the tax_rate of the item).
To calculate cart total tax_amount the Tax helper then adds up all $item->tax_amount's:
https://github.com/bagisto/bagisto/blob/c1dceb0a011df016562c3d5461be6130375e3145/packages/Webkul/Tax/src/Helpers/Tax.php#L73
So the cart/order tax_amount and grand_total will potentially include shipping taxes more than once.
E.g.: You have two different items in your cart, say with identical tax_rate, then you will pay twice the amount of shipping tax. But you should only pay it once.
So the calculation for the tax_amount for the order is incorrect, or did I miss someting? Btw. we can reproduce this when placing orders.
This is a conceptual problem however. Bagisto does not specify which tax_rate to use for shipping.
In some countries inside the EU shipping tax calculation for carts with different tax_rates goes like this: Total net shipping amount will be split to each item, depending on their net amount ratio with respect to total net amount of cart. Then you have a net shipping amount per item. Then, the tax for this item net shipping amount is multiplied by the tax_rate of the item to obtain gross shipping amount for this item.  Should be easily implementable with the current system, though I am not sure if all countries in the world do it like this!
Example:
cart item 1, net price: 10 €, tax_rate: 0.2
cart item 2, net price: 20 €, tax_rate: 0.1
cart shipping_net: 10€
-> shipping_item1 -> net: 10 / 30 * 10 = 3.33€, gross: 3.33€ * 1.2 = 4€
-> shipping_item1 -> net: 20 / 30 * 10 = 6.66€, gross: 6.66€ * 1.1 = 7.33€
-> cart shipping gross: 11.33€
Also I am not sure when to round in that case...
Bagisto Version

master branch

@derMart,
I just verified the whole scenario actual for the free shipping price is zero and for the flat shipping price is added to individual items. So tax is then calculated accordingly.
And it mostly varies from country to country, policies, and rules, etc.
Can you share the sample for the actual result and expected result. So that I can check what is best suited.

Can you share the sample for the actual result and expected result. So that I can check what is best suited.

Sorry, I do not understand what you mean. Actual result currently is that the tax amount of shipping is added more than once into grand_total if you have more than one item in cart. Expected should be that the tax amount of shipping is there only once.
Example:
Assume tax_rate for all items including shipping should be 10

cart item 1: total = 10, tax_rate: 10
cart item 2: total = 20, tax_rate: 10
shipping carrier returns $shipping->price = 10

independent of the carrier type, I understood this price to be the shipping amount of the whole cart, not just of one item (that's how flat shipping is calculating $shipping->price)

Then currently following happens (if I am not mistaken):

cart item 1: tax_amount = (10 + 10) * 10 / 100 = 2
cart item 2: tax_amount = (20 + 10) * 10 / 100 = 3
cart tax_amount = 5 -> that is wrong: 10% of shipping price is included twice in there, not once
cart grand_total = 10 + 20 + 10 + 5 = 35
cart net total is 10 + 20 + 10 = 30
if you apply 10% of 30 you get 33 not 35
-> cart tax amount is wrong

How you wanna fix this depends on what kind of tax calculation you like to support for shipping. That is country specific and I can not make this decision for Bagisto which countries they like to support in their tax calculation or if there is a way to design this so flexible that each user can incorporate its own shipping tax calculation.

All right I will check and fix this. This will take some time. Currently, you can override the behavior if you want.
I will try to make it a flexible for shipping calculations.

This is due to my pull request, I just added as per the flat tax rate basis,
https://github.com/bagisto/bagisto/pull/4360#pullrequestreview-555118566
And I was not aware of this I just thought tax should be added to the item price as shipping prices are individually added.
I will provide the fix for this also and will try to make it flexible also but this will take some time.
As currently bunch of issues are open which needs to be fixed as well.

sure, then why not add Bug label until it is fixed?

Yes! Because it totally varies on the basis of country policies, rules, etc.
That's why we have given enable/disable the feature so that if anything went wrong then the developer can simply disable the feature and override the core functionality.

I just checked and researched, this is itself a big enhancement like defining tax rules, tax classes, whether include taxes or not, etc.
I am reverting my previous pull request,
https://github.com/bagisto/bagisto/pull/4360

Fixed on #9711


Pixel to Struct of Arrays (and back again!)

Description
As discussed in https://github.com/Ogeon/palette/discussions/303, it would be useful to have an interface for converting an array/Vec of colors to and from Struct of Arrays format. The first step should be to allow:

[ ] Construction of Rgb<S, Vec<T>> from some I: IntoIterator<Item = Rgb<S, T>>, and the same for the other color types.
[ ] Make it possible to get a color at some index. For example fn get_mut(&mut self, index: usize) -> Option<Rgb<S, &mut T>>.
[ ] Make it possible to iterate over the colors. For example fn iter_mut(&mut self) -> impl Iterator<Item = Rgb<S, &mut T>.
[ ] Make it simple to read from and assign to colors with reference components, such as Rgb<S, &mut T>.

Bonus features:

[ ] Make it possible to push and pop individual colors.
[ ] Add chunks and windows iterators.

The reference components will likely be necessary, due to the split into separate arrays, unless there's a smarter solution. They don't have to enable much more than reading and writing, though. It's also what would make it hard to implement the Index and IndexMut traits, if possible at all.
Motivation
The discussions in https://github.com/Ogeon/palette/discussions/303 covers it pretty well. One of the main reasons for a built-in interface is that it gives a way to correctly preserve the color type information across conversions. It also lowers the bar for working with SoA data.

Good summary :slightly_smiling_face:
Another feature that I think would be useful is some way to reuse the existing allocations. It might be an extend-like interface so that users could call clear on the existing data and re-initialize with an iterator.

Ah, yes, that should also be there! I'll edit the description to include them...


Hovering any link within a hovered window always tries to open

Hi,
when I hover a [[wikilink]] and push Ctrl, it opens the hovering window. So far so good.
The problem comes when that new window contains links to external (attached) files... that it goes opening them, and it makes the experience pretty uncomfortable.
I'd suggest adding settings like these:

Only using Ctrl key while hovering tries to open the window (equivalent to "Hover trigger delay = infinite")
Discard hovering attached files (they need to be explicitly clicked) [or just relay on using Ctrl for confirming we want to open the attached file]

Excellent job, anyways!

UPDATE:
seems it works better using a setting of a core plugin: Page preview

If was unchecked for Reading View... and that's when it happened the mentioned problem.
Anyways, I think would be a good idea to have a setting for requiring Ctlr for opening the hovering window.

The Page Preview settings you've pointed out are the official way to customize this behavior. I can call this out more explicitly in the documentation but I don't see a need for having separate modifier key controls within the Hover Editor settings.

Yeah, most probably it would be enough just mentioning in the description.
Thanks folk!


WSL extension broken after changing profiles and can't delete/reinstall

Type: Bug
I started on a working project in WSL2. Have been using it for months no problems.

I tried to create a new profile from the template "Python"
It prompted me to install WSL in that profile, I said yes
I changed back to my original profile
Now the original profile prompted me to install WSL, even though it had already been installed. In every profile I try I get this error:

2023-05-05 20:49:38.063 [error] Unable to delete the existing folder 'c:\Users\forcf\.vscode\extensions\ms-vscode-remote.remote-wsl-0.78.4' while installing the extension 'ms-vscode-remote.remote-wsl'. Please delete the folder manually and try again: Delete: Unable to delete the existing folder 'c:\Users\forcf\.vscode\extensions\ms-vscode-remote.remote-wsl-0.78.4' while installing the extension 'ms-vscode-remote.remote-wsl'. Please delete the folder manually and try again
    at e.ExtensionsScanner.extractUserExtension (c:\Users\forcf\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:88:12555)
    at async x.v (c:\Users\forcf\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:88:19429)
    at async x.y (c:\Users\forcf\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:88:21039)
    at async x.h (c:\Users\forcf\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\code\node\sharedProcess\sharedProcessMain.js:88:19004)

I can't delete the extension folder to manually reinstall because there is a file wslServer.sh which I do not have permissions to touch, even though I am an administrator.

Now everything on my computer is messed up because I've tried uninstalling everything, but even a fresh installation cannot install the WSL extension because of this file. please help.
VS Code version: Code 1.78.0 (252e5463d60e63238250799aef7375787f68b4ee, 2023-05-03T20:09:00.748Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G4 CPU @ 1.10GHz (8 x 1498)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.60GB (1.17GB free)

Process Argv
--crash-reporter-id c1118667-74ba-40f9-bac7-6ab9b5e651cb

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
6233i204:30730053
vsccsb:30705552
azdwalk:30721579
pythonms35:30701012
pythonfmttext:30731395
pythoncmvfstr:30731969
fixshowwlkth:30730052
hidesbindicator:30730055
pythongtdpath:30731978

Currently experiencing the same issue. It won't even let me look at the owner.
But I found a workaround:

Download the WSL extension from web version of the extension store: https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-wsl (if you deleted the rest of the extension like me)
Rename the extension to .zip
Copy the contents of the extension folder in the zip to C:\Users\user\.vscode\extensions\ms-vscode-remote.remote-wsl-0.78.4 or your location of the extension
This will make vscode think that the extension is properly installed
On next launch there will be an error that it cannot connect to wsl
Ignore it
Go to extension store and install older version

My own workaround: restarting my computer and VSCode will delete the ownerless file and allow me to continue. However, creating new profiles from a project in WSL continues to raise the same problem with installation of the WSL extension.

The extension folder can't be deleted as WSL is still referencing wslServer.sh.
The workaround is to run wsl --shutdown
I'm not sure why switching the profile requires deleting the wsl extension.

Can confirm the workaround offered by @aeschli above works.

I have this issue as well and it's really annoying.
My workaround is to killall node (the vscode-server) in WSL and then cleanup the extension (delete C:\Users\{{ username }}\.vscode\extensions\ms-vscode-remote.remote-wsl-x.xx.x), reload, and re-install it.

Could be a duplicate of https://github.com/microsoft/vscode/issues/180482

It prompted me to install WSL in that profile, I said yes

@caforbes Can you please provide me the screenshot of VS Code at this step? If possible, please share a gif/video of the steps you mentioned in the description.

I have this issue as welln my computer.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


TypeError: Config.init() got an unexpected keyword argument 'blip_model'

I get the following error when I try to run the interrogator:
Loading CLIP Interrogator 0.6.0...
load checkpoint from /home/trahloc/s/ai/stable-diffusion-webui/models/BLIP/model_base_caption_capfilt_large.pth
Traceback (most recent call last):
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/gradio/routes.py", line 488, in run_predict
    output = await app.get_blocks().process_api(
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/gradio/blocks.py", line 1431, in process_api
    result = await self.call_function(
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/gradio/blocks.py", line 1103, in call_function
    prediction = await anyio.to_thread.run_sync(
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/anyio/to_thread.py", line 33, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 877, in run_sync_in_worker_thread
    return await future
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 807, in run
    result = context.run(func, *args)
  File "/home/trahloc/mambaforge-pypy3/envs/sd/lib/python3.10/site-packages/gradio/utils.py", line 707, in wrapper
    response = f(*args, **kwargs)
  File "/home/trahloc/s/ai/stable-diffusion-webui/extensions/clip-interrogator-ext/scripts/clip_interrogator_ext.py", line 131, in image_to_prompt
    load(clip_model_name)
  File "/home/trahloc/s/ai/stable-diffusion-webui/extensions/clip-interrogator-ext/scripts/clip_interrogator_ext.py", line 63, in load
    config = Config(
TypeError: Config.__init__() got an unexpected keyword argument 'blip_model'
HTTP Request: POST http://localhost:7869/api/predict "HTTP/1.1 500 Internal Server Error"

I'm running Python 3.10.6 on an ubuntu 22.04 variant, pop os specifically. Running on port 7869 to avoid confliction with oobabooga which also runs on 7860. Included a list of all pip/mamba/conda modules installed in that environment.
mamba_list.txt
pip-packages.txt

The extension uses clip-interrogator 0.5 to work with old version of transformers that webui used. Now it's updated to use 0.6, update extensions and restart webui and you should be in working order.


Update URLPattern polyfill

Hi there,
The URLPattern polyfill has been updated, so imports work slightly different (it patches the global object) and it should correctly export the typescript types.

Updated dependency, fixed in next release


Fix: the problem with duplicate filename of iOS

The new file name contains id of PHAsset.

In Android, the access permission itself depends on the Activity, so the code contains the code for dynamic permission checking, which requires your FlutterView to be Attached to an Activity, and it is performed according to the standard Android specification (that is, the channel called by each dart methods are wrapped in code for dynamic permission checking).
And your background service does not seem to depend on Activity, so you need to use ignorePermissionCheck to ignore PhotoManager's permission check on the premise of ensuring that you have permission.
// Call the method to ignore permission check
await PhotoManager.setIgnorePermissionCheck(true);


Fixes and improvements for 'utterances.es.txt'

Mainly fixing accents. Otherwise, some words might be hard to recognize.
Making sure "videos/vídeos" are both recognized for Latin & Castillian Spanish variants.
Few additions.

@m0ngr31 : ping. missing accents makes usage weird due to having to do funny (unnatural) intonations to phrases in Spanish.

Sorry, never saw the notification originally

No worries, @m0ngr31. Thanks for reviewing, and for sharing this software!


[AutoDiff] Remove 'differentiableFunction(from:)' and 'linearFunction(from:)'

Remove unused APIs differentiableFunction(from:) and linearFunction(from:). They were never official APIs, are not included in the initial proposal, and are unused by existing supported client libraries (SwiftFusion and S4TF). Most importantly, they block crucial optimizations on linear map closures (#34935) and would need nontrivial work in SILGen to support.

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - ee216181050e4d9c9b5ca7056a581edd95e0ee65

Build failed
Swift Test Linux Platform
Git Sha - ee216181050e4d9c9b5ca7056a581edd95e0ee65

Build failed
Swift Test OS X Platform
Git Sha - ee216181050e4d9c9b5ca7056a581edd95e0ee65

Build failed
Swift Test OS X Platform
Git Sha - ee216181050e4d9c9b5ca7056a581edd95e0ee65

@swift-ci please test

@swift-ci please test

@rxwei @dan-zheng  I was making use of differentiableFunction for checkpointing, is there a way to achieve gradient checkpointing without differentiableFunction? Like here

Would something like this do it?
@differentiable
func makeRecomputedInGradient<T: Differentiable, U: Differentiable>(
    _ original: @escaping @differentiable(T) -> U
) -> @differentiable(T) -> U {

    @differentiable
    func newFunc(_ t: T) -> U {
        var result: U? = nil
        var result = result.withDerivative { v in v * pullback(at: t, in: original) }
        result = withoutDerivative(at: original(t))
        return result
    }

    return newFunc
}

func _original<T, R>(_ x: T, _ f: @differentiable (T) -> R) -> R {
  f(x)
}

@derivative(of: _original, wrt: x)
func _vjp<T, R>(
  _ x: T, _ f: @differentiable (T) -> R
) -> (value: R, pullback: (R.TangentVector) -> T.TangentVector) {
  (value: f(x), pullback: pullback(at: x, in: f))
}

func makeRecomputed<T, R>(
  _ f: @escaping @differentiable (T) -> R
) -> @differentiable (T) -> R {
  { x in _original(x, f) }
}

Thanks Richard!


Fix to permit saml custom service definition by overriding SamlRegisteredService

By using class equals method won't be possible to define a custom saml registered service by overriding the existing SamlRegisteredService class.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.fabio seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Mock Contract not working with calledOnContract

Hello,
I'm having difficulties with using the Mock Contract with the calledOnContract and calledOnContractWith matchers.
For reference I am using Ethereum Waffle with Hardhat.
Here is a sample of the used code:

// setup
import * as registrar from "../artifacts/contracts/Registrar.sol/Registrar.json";

chai.use(solidity);
const { expect } = chai;

// making the mock
registrarMock = await deployMockContract(creator, registrar.abi);

// inside of a test
await registrarMock.mock.registerDomain.returns(returnedId);

// <--- call registerDomain from another contract --->

// check to see if function was called
expect("registerDomain").to.be.calledOnContract(registrarMock);

calledOnContract throws an exception:  TypeError: argument must be a contract
at Object.ensure (node_modules\@ethereum-waffle\chai\dist\cjs\matchers\calledOnContract\utils.js:6:15)
      at Object.validateContract (node_modules\@ethereum-waffle\chai\dist\cjs\matchers\calledOnContract\calledOnContractValidators.js:8:13)
      at Proxy.<anonymous> (node_modules\@ethereum-waffle\chai\dist\cjs\matchers\calledOnContract\calledOnContract.js:8:38)
      at Proxy.methodWrapper (node_modules\chai\lib\chai\utils\addMethod.js:57:25)
      at Context.<anonymous> (test\basiccontroller.spec.ts:66:44)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)

Happens with both calledOnContract and calledOnContractWith methods.
Relevant Versions:
{
  "@nomiclabs/hardhat-waffle": "^2.0.1",
  "ethereum-waffle": "^3.3.0",
  "chai": "^4.3.3",
  "ethers": "^5.0.31"
}

Duplicate of #469

Closing as a duplicate. See https://github.com/TrueFiEng/Waffle/issues/469#issuecomment-1199181049


aslcontext.tsv and inheritance

The BIDS spec (v.1.5.0) has sections describing an aslcontext.tsv  file for use with ASL scans, and also the inheritance principle, which says that instead of providing multiple identical metadata files you can provide a single one at a higher level whose contents will apply to lower levels.
Unfortunately, if you attempt to place a single aslcontext.tsv file at the top level of the dataset directory, the bids validator (v1.6.2) will tell you "Files with such naming scheme are not part of BIDS specification." (The same file, renamed sub-<label>_ses-<label>_aslcontext.tsv and placed in the sub/ses/perf directory with its corresponding nifti file, passes validation.)
Either this is a bug in the validator that should be fixed or the spec should be updated to explicitly say that the inheritance principle does not apply to the aslcontext.tsv file. (To me it looks like the former, but I'm no ASL expert.)

Sounds like a good case for inheritance to me as well, but more ASL expert opinions would be appreciated.
Technical comment: This is the spot in bids-validator where aslcontext.tsv would need to be added: https://github.com/bids-standard/bids-validator/blob/5ee85764957b8b07681ac33c92b7d3052c01ce45/bids-validator/bids_validator/rules/top_level_rules.json#L1-L21

I'm preparing some ASL data now and would like this inheritance feature for aslcontext.tsv. Is there any reason not to allow this, or at least add it as a feature in the schema-based validator?

They are inheritable in the schema-based validator.

I'll try it there and let this issue here know what I find out.


Unable to run SeaShell Framework. Can't load commands rpc and listener due to exceptions

Describe the bug
I'm using WSL (Ubuntu) and MacOS Sonoma with different versions of Python (from 3.10 to 3.12) and the SeaShell refuses to load the rpc and listener commands.
Errors are different from what was reported in #22 and #19.
I've been trying to install and also reinstall the framework (as shown in the update section) with no results. Also, I manually updated the pex package from pip, and other solutions provided in those two issues.
To Reproduce
Just run the pip3 command mentioned in the install section of the Readme.
Screenshots

Desktop (please complete the following information):

OS: Windows 11 + WSL Ubuntu (<IP_ADDRESS>-microsoft-standard-WSL2) / MacOS Sonoma (14.5)

Thank you very much for the details. I've already fixed this problem. Can you please enter these commands and confirm if problem is solved:
pip3 uninstall pex
pip3 install git+https://github.com/EntySec/Pex

No further loading issues have been found. Thanks for the fix and the reply as well! I will close it.


feat: store and push last valid config

What this PR does / why we need it:

Which issue this PR fixes:
Fixes #4048

Special notes for your reviewer:
E2E run with these changes: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/5336108716/jobs/9670625415

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@rainest @pmalek @mheap I've updated the CHANGELOG as Travis suggested and I think we can merge this one. My intention is to create a new issue for fetching the latest good configuration from the Proxy and tackle it in a different PR (we need to add a go-kong update as well), along with the hash thingy (fundamental when implementing the fetch from proxy feature). Are you ok with this approach?

I think this is (still) good to go but IDK if ya'll want to continue anything in the open comment threads, so approving again but leaving it up to @mlavacca to close them/merge.

The thread's discussion will be addressed in a separated PR. I added some unit testing to address @czeslavo comment. Waiting for his approval.

@mflendrich Flagged some potential issues with fetching config from a running DP if running in DB-backed and using both KIC and deck. Agree to merge this PR as-is, but we should consider limiting "fetch from DP" mode to DB-less only in a future PR


GH-40843: [Java] Cleanup protobuf-maven-plugin usage

Rationale for this change
protobuf-maven-plugin usage in Arrow codebase does not follow plugins best practices like sharing the same output directory for different execution or not using test goals for generating test classes
What changes are included in this PR?

Add protobuf-maven-plugin plugin to top level pom.xml under pluginManagement to define version and common configuration for all modules
Remove unnecessary executions of test-compile goal when no test protobufs are present
Remove use of outputDirectory and clearOutputDirectory and let the plugin choose it for each execution (the default output directory is based on the phase (main vs test) and the language/plugin-id)
Replace use of compile/compile-custom goals with test-compile/test-compile-custom when generating test protobufs

Are these changes tested?
As those changes are in the build system, they are covered by the build framework and tests run as part of the build
Are there any user-facing changes?
None

GitHub Issue: #40843

@github-actions crossbow submit java

@github-actions crossbow submit -g java

@lidavidm seems like the failing CIs not related, could we re-run and verify?


Refactoring Exceptions

SHAP currently calls Exception a lot throughout the code base. This makes it tough to handle exceptions internally as one has to match to messages to infer why an exception was raised.
For example; I want to try to build an explainer object and raise an error in cases where a tree explainer (or similar) won't work:
try:
    explainer = shap.Explainer(mod)
except Exception as e:
    if "The passed model is not callable and cannot be analyzed directly with the given masker!" in str(e):
        # do something
    else:
        raise e

Since I want to avoid catching all the other Exceptions that might be raised I have to match against the message. This is brittle and ugly in my opinion, I would much rather have something like:
try:
    explainer = shap.Explainer(mod)
except InvalidMaskerError:
    # do something

I am opening this draft PR which replaces some Exception calls with "more" appropriate ones including custom ones. As to not waste time in case this is not desired I wrote this very quickly and hence I'm sure some exceptions aren't the ideal ones to raise.
What do we think of doing it this way? If there's a positive reception to the idea I'll do it properly and open this as a PR.

Thanks @alexisdrakopoulos, I like the idea! I agree that handling exceptions via string matching is not ideal, and there is clear room to improvement here. What you have looks like the right direction, so go for it. Once you have a PR ready I expect it will be a simple merge.

There are three tests for which I need to update the exception catching.

@slundberg can we try running tests?

Just looked it over and it all looks good to me, except for one line I commented on. After you get a chance to look at that I think this is ready to merge. Thanks!


Rename the main branch to "main"

I think this would be a good idea but cannot do this myself because I don't have the necessary rights on the repository.

I'll see what and how :)


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 2bb8539:
:clock10: Full build: the average full build time is 5 min(s) 58 sec(s), this is based on the last 623 build(s) for this repository.

Docs Build status updates of commit 2bb8539:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ResolveComReference task fails when executing build from child AppDomain

I've run into following problem, when trying to execute build in child app domain of main process:
Invoke-MSBuild : C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(2545,5):
error MSB4061: The "ResolveComReference" task could not be instantiated from "Microsoft.Build.Tasks.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a".
Could not load file or assembly 'Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies.
The system cannot find the file specified.

The issue appears to be caused by fact that build is being run from child app domain of main process (powershell.exe in my case). ResolveComReference (extended from AppDomainIsolatedTaskExtension) does not use ApplicationBase of child appdomain, but instead uses ApplicationBase of parent PowerShell process.
Log from Fuslogvw.exe (Assembly Binding Log Viewer)
*** Assembly Binder Log Entry  (2017-03-22 @ 22:53:23) ***

The operation failed.
Bind result: hr = 0x80070002. The system cannot find the file specified.

Assembly manager loaded from:  C:\Windows\Microsoft.NET\Framework64\v4.0.30319\clr.dll
Running under executable  C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
--- A detailed error log follows.

=== Pre-bind state information ===
LOG: DisplayName = Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
 (Fully-specified)
LOG: Appbase = file:///C:/Windows/System32/WindowsPowerShell/v1.0/
LOG: Initial PrivatePath = NULL
LOG: Dynamic Base = NULL
LOG: Cache Base = NULL
LOG: AppName = powershell.exe
Calling assembly : (Unknown).
===
LOG: This bind starts in default load context.
LOG: No application configuration file found.
LOG: Using host configuration file:
LOG: Using machine configuration file from C:\Windows\Microsoft.NET\Framework64\v4.0.30319\config\machine.config.
LOG: GAC Lookup was unsuccessful.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/Microsoft.Build.DLL.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/Microsoft.Build/Microsoft.Build.DLL.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/Microsoft.Build.EXE.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/Microsoft.Build/Microsoft.Build.EXE.
LOG: All probing URLs attempted and failed.

I think MSBuild might need to set a resolution hook for its own assemblies when starting the new appdomain for an isolated task.
This was also the root cause of https://github.com/tunnelvisionlabs/antlr4cs/issues/173.

This is still causing issues with Live Unit testing in VS 2019.
See https://developercommunity.visualstudio.com/content/problem/402919/live-unit-testing-doesnt-work-with-xamarinforms-in.html

I'm also having trouble with this. The workaround suggested here doesn't seem to be accurate any more. The suggested workaround:

Copy System.Runtime.InteropServices.RuntimeInformation.dll present under
C:\Program Files (x86)\Microsoft Visual Studio\Preview\Enterprise\Common7\IDE\PrivateAssemblies\ to
C:\Program Files (x86)\Microsoft Visual Studio\Preview\Enterprise\Common7\ServiceHub\Hosts\ServiceHub.Host.CLR.x86. (Change the paths to match your local VS install location.)

Problem is, that dll doesn't exist at that path in my installation (VS 2019, 16.2.4). I can find it in 17 other locations, but I wouldn't know which one to copy, if this is still a viable way to fix it.
Result of a search for the file:


Can't set video property on VideoLayer using options object

After updating to the latest release of Framer Studio, if you set the video property on the options object, Framer will through the following error:
Uncaught TypeError: Cannot set property 'src' of undefined

After some brief exploration, it looks like what is happening is that the video option is calling the video setter before this.video has been defined.
In the meantime users can get around this error by setting videoLayer.video specifically after instantiating it rather than passing it in the options object.

Thanks @mattsjohnston!


FR: option to copy coordinates when copying objects between steps

When copying an object/icon from one step to another, it would be helpful if the position of the object within the arena would be unchanged, instead of pasting to the cursor location (i.e. the offset would be 0).
If you do not desire to change the default behavior, an option to do this would still be greatly appreciated, i.e. by making a shortcut like ctrl+shift+V be "copy without changing location" (this could also be implemented via the ctrl+D "duplicate selected objects" feature).
This option would make it far easier to correct mistakes in multiple steps whilst keeping positions consistent, e.g. correcting the position of an object in multiple steps, adding a missing marker to all steps, etc.
Preferably this feature would also work when copying multiple objects.

Duplicate with Ctrl+D already exists and would normally be the shortcut for this, but that doesn't work here because as soon as you switch steps, there aren't any selected objects anymore. I'll have to decide on what the best keyboard shortcut and name for this is, but I think having a way to duplicate the objects that are on the clipboard would be useful and not hard to implement.

This is now implemented with the shortcut Ctrl+Shift+V


[3205] Fix up active trainee records submission ready

Context

https://trello.com/c/ndHeUlmN/3205-fix-up-active-trainee-records-submission-ready

Changes proposed in this pull request
Guidance to review

 Fix up active trainee records submission ready


Add anchors to bullet points in authoring guidelines.

With #156 it's possible to put linkable anchors in bullet points, so reviewers would be able to post links to specific guidelines. Two branches which could use it are already there and I do not know how to make the feature available there (my git-fu is almost non-existant), so I will add anchors later, after branches are merged.
Or if someone would explain to me how to do rebase properly, or would do this for me, it would be great.
Awaiting for #134  ,  #19

Or if someone would explain to me how to do rebase properly

Make sure your local master is up to date:
$ git checkout master # if you're not there yet
$ git pull

Then rebase the branch:
$ git checkout some-branch
$ git rebase master

There shouldn't be any conflicts so rebase should work.
Then, you'll need to force push the changes because now your local branch is different from what you had pushed before.
$ git push --force-with-lease
# If you don't have the tracking set up
$ git push --force-with-lease origin some-branch

or would do this for me

I can do it for you, but I just realized that I've never done this with PRs with review comments. So I'm not sure how force push will affect those suggestions.
Maybe it's safer to just add them after merge. Or you can just add them now and it'll work after merging.

Would it be possible o have something what would let me to copy a link pointing to a specific bullet point or excerpt of a guideline and paste it as a link AND when a user would arrive via this link, then a guideline I wanted to point them to would be highlighted?

From Zola experiment:
https://user-images.githubusercontent.com/639336/104797867-8be91d80-5776-11eb-9def-0c8ed7cd362b.mp4
Syntax:
- {{ anchored(id='novelty', text='**Make sure your kata adds some new value to Codewars.**') }}

Hovering over will reveal a link that you can click or copy the URL. Anchored content is styled differently when targeted.

Would it be possible o have something what would let me to copy a link pointing to a specific bullet point or excerpt of a guideline and paste it as a link AND when a user would arrive via this link, then a guideline I wanted to point them to would be highlighted?

From Zola experiment:
https://user-images.githubusercontent.com/639336/104797867-8be91d80-5776-11eb-9def-0c8ed7cd362b.mp4
Syntax:
- {{ anchored(id='novelty', text='**Make sure your kata adds some new value to Codewars.**') }}

Hovering over will reveal a link that you can click or copy the URL. Anchored content is styled differently when targeted.


Is this plugin ready?

Hey there
Thanks for the amazing work here.
I was just wondering, what is the status of this plugin? Is it ready to use for production? are there any known issues?

I used it to cover the use cases of two apps I was developing at the time. Both are on the store now. Of course YMMV.


"To" field on Invoice tab is editable for non-multisig account

Steps to reproduce:

Go to the Invoice tab under non-multisig account
Click on the To field
Edit the address

Actual result: The To field allows a user to edit a recipient address
Expected result: An address in the To field is not editable, like for a multisig account or on mobile
Video for non-multisig account
Video for a multisig account (as it should be)

Why allowing put custom address is a bad idea?

Because it breaks the feature's logic IMHO. Please compare with Receive feature on mobile.

For me it's more like a feature not a bug. For sure not P2.

I agree with a priority and could agree also with that it's a feature but more tend to consider it as a bug.

I wouldn't even put a P3 on it, sounds an enhancement to me anyway

I like here flexibility but leaving the decision to the discussion when the time will come.

@Eugenity  thanks for reporting, after discussion we agreed that it's better for flexibility reasons to give users the option to choose to whom they're gonna generating the invoice.


add detailed explanation of GameServer hot-update

Codecov Report

Merging #70 (3a4830d) into master (1192d4d) will decrease coverage by 0.17%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   29.28%   29.11%   -0.17%
==========================================
  Files          18       18
  Lines        2281     2294      +13
==========================================
  Hits          668      668
- Misses       1566     1579      +13
  Partials       47       47

Flag
Coverage Δ

unittests
29.11% <ø> (-0.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


garm-cli builds for Windows

I understand if you don't want the "server" part built for Windows, but can you build and publish Windows builds of the cli?

Sure thing! Will be happy to! I think the server may run on Windows as well. Will have to test it out. In any case, I will publish the client for the latest release (and all future releases), tomorrow.

Surprisingly, the server seems to work well on Windows. However, it has no Windows service manager support, so something like nssm will need to be used if anyone wants to run it as a service on Windows. I will add the necessary handlers to enable native Windows service support in directly in the binary. But that will be done later.

Windows binaries have been added to the latest release: https://github.com/cloudbase/garm/releases/tag/v0.1.3
Give them a shot and let me know if you encounter any issues.


incorrect error with static reference

This code should be accepted:
class C() {
    shared Anything val {
        return C.val; // error: not definitely specified: 'val'
    }
}

I don't see ehy this error is incorrect.

@gavinking did you read the example as if val were static? Otherwise, I can't see the justification for the error.
The code that produces the error can also be written in ways that don't produce errors:
class C([C+] others) {
    String(C) sRef => C.s;
    String s {
        noop(others.first.s); // ok
        sRef(others.first); // ok
        sRef(this); // ok but hardly useful
        others.map((c) => c.s); // ok
        others.map(C.s); // ok in 1.3.0, disallowed in 1.3.1
        return "";
    }
}

and the error does not occur when taking references to methods:
class D() {
    String s() {
        String(D) sRef = uncurry(D.s); // ok
        return "";
    }
}

@jvasileff We have always tried to reject obviously-recursive value definitions. Of course we don't do the same with functions. I don't see a problem here, and I would like to close this issue.

@gavinking this is a valuable thing to have, and occurs in real world code.
It is not a case of obviously incorrect code that will (almost) always result in a stack overflow, as it would be if val were static.
I'm virtually certain that we'd see many cases of this in your Java code, where getters call the same getter, but on other instances of the same type (see the workarounds linked above).
Another example might be for implementing string, if you have a tree and would like to collect children.collect(Person.string). Or perhaps reduce hash for all children when implementing hash.
I do hope you reconsider, and allow expressions of the form Type.nonStaticMember which are actually functions, always.

It's worth noting that (nearly?) every collection type in Ceylon uses this sort of recursive access to the string and hash values (although not necessarily using a functional approach with map, etc.).
And this is allowed:
class Collection({Object*} items) {
    shared actual String string
        =>  items.collect(Object.string).string; // no warning here!
}

but not:
class Collection({Collection*} items) {
    shared actual String string
        =>  items.collect(Collection.string).string; // error
}

I just ran into this again, but with a more confusing error message:
    Type withNullEliminated
        =>  if (declaration.inherits(unit.nullDeclaration)) then
                unit.nothingDeclaration.type
            else if (isUnion) then
                unionDeduped(caseTypes.map(Type.withNullEliminated), unit)
            else this;

results in:
Type.ceylon:1338: error: not definitely specified: 'withNullEliminated' has not been assigned by every conditional branch
                unionDeduped(caseTypes.map(Type.withNullEliminated), unit)
                                                ^


Merge prTesting_201907291232 branch to master branch

Docs Build status updates of commit 9829eb3:
:clock10: Pending: waiting for processors (9 builds ahead of you)
:warning: Docs Build is busy, currently there are 9 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 9829eb3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:bulb:Suggestion

Details

split/updateTestCon.md

[Suggestion] Missing attribute: author. Add a valid GitHub ID.
[Suggestion] Missing attribute: ms.author. Add a valid Microsoft alias.
[Suggestion] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Suggestion] Missing attribute: title. Add a title string to show in search engine results.
[Suggestion] Missing attribute: ms.topic. Add a valid value to indicate the article type.
[Suggestion] Missing attribute: ms.date. A freshness date is required for this content set. Add a date in format MM/DD/YYYY.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


The API does not take very long texts.

When sending little bit longer text to API, it will respond:

Expected size[0] in [0, 512], but got 1144 [Op:Slice]

This means you need to split texts so that they are under 500 tokens long, which is not optimal.

No. model is trained with a maximum sentence length of 128.


⚠ St. Lawrence Global Observatory CKAN has degraded performance

In c9fa71a, St. Lawrence Global Observatory CKAN (https://catalogue.ogsl.ca/en/) experienced degraded performance:

HTTP code: 200
Response time: 6424 ms

Resolved: St. Lawrence Global Observatory CKAN performance has improved in d27d2f1 after 6 minutes.


Arduino does not see the connected Badusb in the ports.

I have a problem, namely when I insert Badusb into Arduino and go to Tools -> Port I have nothing. Since I am Russian with bad English, I got tired of reading all the documentation in English, so I decided to ask

Hello!
First you have to upload the code, then arduino ide will ask you to connect the cable.
Here you can see a demo:
https://twitter.com/JoelSernaMoreno/status/1181296289323130882?s=19
First you have to do the installation as indicated in the documentation

If you do not want to work a lot, use the virtual machine. Read the VM for dummies section

Hi! Same problem. My DigiSpark ATTiny85 works without a hitch, your cable just hangs at Plug in device now... (will timeout in 60 seconds).

Btw, ping me on @hookgab on Twitter as you don't have DMs open to non verified users...

Hi @huuck
Check twitter DM


gas limit error with default config

When I run the demo app (without changes), I get the following error:
eth_sendTransaction
[Error: tx has a higher gas limit than the block]

Changing  the development:gas_limit property in blockchain.yml does not fix the issue.
What to do?

@m888m just released ethersim 0.1.6 which fixes this.  try npm update embark-framework
npm list ethersim should report 0.1.6

@iurimatias Unfortunately, I need to reopen the issue. Pls have a look:
First I did the update like you instructed. As you can see the new version of the embark-framework is not reflected in grunt-embark (I also re-installed grunt-embark, but to no avail)
<EMAIL_ADDRESS>/home/m/dmc/embarkTest/embark_demo
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
As a consequence, I still get the following errors:
m@888@m:~/dmc/embarkTest/embark_demo$ embark simulator
starting server...
listening on localhost:8101
eth_coinbase
eth_getBlockByNumber
eth_sendTransaction
[Error: tx has a higher gas limit than the block]
eth_newBlockFilter
eth_getFilterChanges
eth_getFilterChanges
eth_getTransactionReceipt
eth_getCode

What I also noted is that embark does not seem to find the local grunt-cli. I don't think so, but it contribute to thte issue
m@888@m:~/dmc/embarkTest/embark_demo$ embark run
>> Local Npm module "grunt-cli" not found. Is it installed?

Running "coffee:compile" (coffee) task
>> 0 files created.

Running "deploy_contracts" task
primary account address is : 0x7c62ea8ee123f1ea39820e2a8ee0b4d64872f8bd
address is : 0x7c62ea8ee123f1ea39820e2a8ee0b4d64872f8bd
trying to obtain SimpleStorage address...
error deploying


Cannot download SSMS 18.3

The link https://go.microsoft.com/fwlink/?linkid=2104251 from page https://docs.microsoft.com/it-it/sql/ssms/download-sql-server-management-studio-ssms?view=sql-server-2017 doesn't allow to open the correct page, so I cannot download the update

This has been fixed in https://github.com/MicrosoftDocs/sql-docs/commit/816648720165343d7f830b0a8c428bbb743945b2. The localized page will be synced in a few days. You can use those links in the en-us page before that.

@JeffSkyrunner Thank you for the feedback.
Closing the issue now, please comment below if there issues with the fix.


Create some diagrams about the translation process

Visualize the translation process with some diagrams and create documentation for better understanding.

This will be done in parallel with issue #127.

Diagrams can be found in the file elasticai.creator.vhdl.README.md.


console.log in the context of an onPress handler does not log and causes UI delays

Description
    return (
        <TouchableOpacity onPress={ (a,b,c) => { console.log(a,b,c); } } key={ item } style={ viewStyle }>
            <Text style={ textStyle }>Touch me</Text>
        </TouchableOpacity>
    );

Touching that element dims it. onPress function appears to not get called, no logging happens.
Swapping to react-native-router and moving the logging out of the onPress context seems to fix things.
Expected Behavior
I'd expect the log message to display something even if it was undefined.
Observed Behavior
Touching the "button" causes a "hang" that times out and then the next touch goes thru a similar lag.  I'm expecting to find out that logging during the onPress handler isn't supported.
Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts<EMAIL_ADDRESS>npm ls react-native<EMAIL_ADDRESS>npm ls expo<EMAIL_ADDRESS>node -v: v6.9.4
npm -v: 4.1.2
yarn --version: command not found
watchman version: command not found

Also specify:

Operating system: OSX El Capitan  10.11.6
Iphone 6, Nexus 6

Reproducible Demo
https://github.com/mark0978/react-native-bug-report

Thanks for the detailed report and repro!
I've seen similarly strange behavior in the past due to the fact that console.log is overridden to call JSON.stringify on arguments in order to send them back to the terminal -- maybe that's related somehow? cc @brentvatne

we're working on improving logging in expo (which is the tool that powers this for crna), you can track it here: https://github.com/expo/expo/issues/168

@brentvatne can you explain why this issue has been closed? It's still a problem as far as I can tell, and the linked issue expo/expo#168 does not seem to be addressing it.


Design meeting discussion: July 23, 2015

This issue is for discussion of the EF design meeting on July 23, 2015.
Items discussed were:

NuGet/DNX Commands
Cross-platform coding guidelines

Please comment on this issue to provide feedback, ask questions, etc.

We finally had to let go of our revEng...

Is -force switch missing or it's no longer needed in EF7?

On Update-Database? It is no longer needed due to the removal of automatic migrations.

Yes on Update-Database, I was wondering because calling Update-Database -TargetMigration PreviousMigration would most of the time result in data loss and it was stopped by an error in EF6.

Oh weird; I thought we only blocked data loss on automatic migrations. In EF7, we'll warn you that they have been generated in the Up method during Add-Migration.

Here are some other things I noticed :sweat_smile:
dnx [project] ef migrations script
    [from] -f, --from (required when --to)
    [to] -t, --to
    -i, --idempotent
    -c, --context
    -s, --startupProject

Definitely liking the s'es, seems to be more natural to read. :+1:
But why is --from required when --to is specified? I could think of two scenarios when only one of these parameters would be specified.

I am developing a system, and I just deploy as is, meaning I only need not applied migrations. I go to command line and run dnx . ef migrations script --from FirstMigrationFor_v1.1. This works fine now.
I am developing a system, it already has a tagged v1.0 and v1.1-beta releases. I passed all the QA madness and I'm ready to deploy v1.0 on production. I need scripts so I go to command line and run dnx . ef migrations script --to LastMigrationFor_v1 (--upto switch would sound even better). This would mean "Generate all the script needed for v1.0 release".

I read your blog post and I couldn't help but notice separating migrations by namespace/folder. Thus having a -n, --namespace switch would be nice too. This could be used to generate version specific scripts like dnx . ef migrations script --namespace MyProject.Migration.V1_1. It could be a nice wrapper for ranging --from --to commands, but it's not necessary for initial release :smiley:.
dnx [project] ef migrations list
    -s, --startupProject

It would be nice to have a "not applied" option later on.
And as with migrations --namespace switch to list migrations in a given namespace.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Use viewport height/width for constraining dialog on small displays i…

…nstead of body dimensions.

Is this testable with visual diff by setting the viewport super small and having a wide element on the page to force scroll?

We can add a test for that scenario, but it wouldn't really catch a regression for the specific use-case since it seems to be Android specific.  That's probably a worthwhile scenario nonetheless - will add.