Flutter WebView - Hybrid Composition Crashes the App

Steps to Reproduce

Open WebView and enable hybrid composition using: WebView.platform = SurfaceAndroidWebView() (When disabled - the bug doesn't occur)
Go back to the previous page where you came from
Try to rotate the page to landscape mode

Expected results:
The screen rotating normally.
Actual results:
The app crashed.

Logs

[+3933 ms] I/WebViewFactory(11251): Loading com.google.android.webview version 85.0.4183.101 (code 418310183)
[  +70 ms] I/cr_LibraryLoader(11251): Loaded native library version number "85.0.4183.101"
[        ] I/cr_CachingUmaRecorder(11251): Flushed 5 samples from 5 histograms.
[  +57 ms] W/chromium(11251): [WARNING:dns_config_service_posix.cc(342)] Failed to read DnsConfig.
[  +61 ms] W/om.webview_tes(11251): Accessing hidden method Landroid/media/AudioManager;->getOutputLatency(I)I (greylist, reflection, allowed)
[   +2 ms] W/cr_media(11251): Requires BLUETOOTH permission
[  +61 ms] E/libEGL  (11251): call to OpenGL ES API with no current context (logged once per thread)
[   +2 ms] W/Gralloc3(11251): allocator 3.x is not supported
[ +889 ms] D/OnePlusJankManager(11251):  Chor uploadMDM JANK_TYPE_ONCE mViewTitle = com.webview_test/com.webview_test.MainActivity--- jank level = 1
[+2909 ms] E/flutter (11251): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(43)] java.lang.RuntimeException: Methods marked with @UiThread must be
executed on the main thread. Current thread: 1.raster
[        ] E/flutter (11251):   at io.flutter.embedding.engine.FlutterJNI.ensureRunningOnMainThread(FlutterJNI.java:992)
[        ] E/flutter (11251):   at io.flutter.embedding.engine.FlutterJNI.destroyOverlaySurfaces(FlutterJNI.java:873)
[        ] E/flutter (11251):
[        ] F/flutter (11251): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1286)] Check failed: CheckException(env).
[ +255 ms] F/libc    (11251): Fatal signal 6 (SIGABRT), code -6 (SI_TKILL) in tid 11361 (1.raster), pid 11251 (om.webview_test)
[ +221 ms] *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
[        ] Build fingerprint: 'OnePlus/OnePlus7Pro_EEA/OnePlus7Pro:10/QKQ1.190716.003/2007240000:user/release-keys'
[        ] Revision: '0'
[        ] ABI: 'arm64'
[        ] Timestamp: 2020-10-03 13:35:54+0300
[        ] pid: 11251, tid: 11361, name: 1.raster  >>> com.webview_test <<<
[        ] uid: 10762
[        ] signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
[        ] Abort message: '[FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1286)] Check failed: CheckException(env).
[        ] '
[        ]     x0<PHONE_NUMBER>000000  x1<PHONE_NUMBER>002c61  x2<PHONE_NUMBER>000006  x3  000000751aa8d490
[        ]     x4<PHONE_NUMBER>008000  x5<PHONE_NUMBER>008000  x6<PHONE_NUMBER>008000  x7<PHONE_NUMBER>000000
[        ]     x8<PHONE_NUMBER>0000f0  x9  7eaf89dcede28044  x10<PHONE_NUMBER>000001  x11<PHONE_NUMBER>000000
[        ]     x12 fffffff0fffffbdf  x13<PHONE_NUMBER>000003  x14<PHONE_NUMBER>000001  x15 0000b3cba0e98417
[        ]     x16 000000760e228738  x17 000000760e206be0  x18<PHONE_NUMBER>572000  x19<PHONE_NUMBER>002bf3
[        ]     x20<PHONE_NUMBER>002c61  x21 00000000ffffffff  x22<PHONE_NUMBER>000568  x23 000000751fc63a80
[        ]     x24<PHONE_NUMBER>000041  x25 000000751aa8e020  x26<PHONE_NUMBER>000000  x27<PHONE_NUMBER>000041
[        ]     x28<PHONE_NUMBER>000001  x29 000000751aa8d530
[        ]     sp  000000751aa8d470  lr  000000760e1b8404  pc  000000760e1b8430
[  +64 ms] backtrace:
[        ]       #00 pc<PHONE_NUMBER>073430  /apex/com.android.runtime/lib64/bionic/libc.so (abort+160) (BuildId: a2584ee8458a61d422edf24b4cd23b78)
[        ]       #01 pc<PHONE_NUMBER>286f44  /data/app/com.webview_test-FPKpAVtYdfmSs-YNkW1NDQ==/lib/arm64/libflutter.so (BuildId: 0c638458ff559f7abd5975412f6d530104a696e1)
[        ]       #02 pc<PHONE_NUMBER>0024fc  [anon:.bss]
[ +305 ms] Service protocol connection closed.
[   +1 ms] Lost connection to device.

Analyzing webview_test...
No issues found! (ran in 2.6s)

[✓] Flutter (Channel stable, 1.22.0, on Mac OS X 10.15.6 19G2021, locale en-IL)
    • Flutter version 1.22.0 at /Users/talbarda/Desktop/flutter
    • Framework revision d408d302e2 (4 days ago), 2020-09-29 11:49:17 -0700
    • Engine revision 5babba6c4d
    • Dart version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at /Users/talbarda/Library/Android/sdk
    • Platform android-30, build-tools 29.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • 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.9.1

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.1
    • Dart plugin version 192.8052
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
    • GM1913 (mobile) • d355cf87 • android-arm64 • Android 10 (API 29)

• No issues found!

I also met this problem

@tal412
The support for hybrid composition currently only seem to be in Master. Please see this merged PR.
Can you switch to latest master and try again to see if it works ?
Also see related issue.
Thanks.

The crash is caused by the exception visible in the logs:

java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: 1.raster

As the webview for Flutter is out of the developer preview since version 1.22.0 (on the stable branch) and the documentation suggests using SurfaceAndroidWebView(), it can be a bit confusing whether developers should use it or not since having a potential crash on rotation isn't seems to be a stable solution...

Using the master branch can also cause a crash.

@tal412
The support for hybrid composition currently only seem to be in Master. Please see this merged PR.
Can you switch to latest master and try again to see if it works ?
Also see related issue.
Thanks.

As @lbx-ai said, the crash also occurs using the master channel.

Issuer replicable on latest master. Crash occurs if we open webview interface by tapping on a button and go back to it and rotate screen.  If we use the official webview_flutter example and run as-is, then the crash doesn't occur.

crash log
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x960]-format:1
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x888]-format:1
E/BpSurfaceComposerClient(18584): Failed to transact (-1)
E/BpSurfaceComposerClient(18584): Failed to transact (-1)
I/WebViewFactory(18584): Loading com.android.chrome version 85.0.4183.127 (code 418312770)
I/cr_LibraryLoader(18584): Loaded native library version number "85.0.4183.127"
I/cr_CachingUmaRecorder(18584): Flushed 5 samples from 5 histograms.
W/ResourceType(18584): Failure getting entry for 0x7f1305b7 (t=18 e=1463) (error -2147483647)
W/chromium(18584): [WARNING:dns_config_service_posix.cc(342)] Failed to read DnsConfig.
W/cr_media(18584): Requires BLUETOOTH permission
I/zygote  (18584): Do partial code cache collection, code=25KB, data=30KB
I/zygote  (18584): After code cache collection, code=25KB, data=30KB
I/zygote  (18584): Increasing code cache capacity to 128KB
W/libEGL  (18584): EGLNativeWindowType 0xd34ff808 disconnect failed
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x888]-format:1
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x888]-format:1
I/zygote  (18584): Do partial code cache collection, code=62KB, data=59KB
I/zygote  (18584): After code cache collection, code=62KB, data=59KB
I/zygote  (18584): Increasing code cache capacity to 256KB
I/chromium(18584): [INFO:CONSOLE(4)] "JQMIGRATE: Migrate is installed, version 1.4.1", source: https://camellabs.com/wp-content/cache/minify/df983.js (4)
I/zygote  (18584): Do full code cache collection, code=116KB, data=106KB
I/zygote  (18584): After code cache collection, code=91KB, data=65KB
I/zygote  (18584): Do partial code cache collection, code=125KB, data=98KB
I/zygote  (18584): After code cache collection, code=125KB, data=98KB
I/zygote  (18584): Increasing code cache capacity to 512KB
E/BufferQueueProducer(18584): [ImageReader-540x888f1m3-18584-0] disconnect: not connected (req=1)
W/libEGL  (18584): EGLNativeWindowType 0xcda73808 disconnect failed
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x888]-format:1
E/BufferItemConsumer(18584): [ImageReader-540x888f1m3-18584-0] Failed to release buffer: Unknown error -1 (1)
E/chromium(18584): [ERROR:tile_manager.cc(793)] WARNING: tile memory limits exceeded, some content may not draw
E/chromium(18584): [ERROR:tile_manager.cc(793)] WARNING: tile memory limits exceeded, some content may not draw
W/libEGL  (18584): EGLNativeWindowType 0xd34ff008 disconnect failed
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [960x540]-format:1
W/libEGL  (18584): EGLNativeWindowType 0xd34ff808 disconnect failed
D/mali_winsys(18584): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [888x540]-format:1
E/flutter (18584): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(44)] java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: 1.raster
E/flutter (18584): 	at io.flutter.embedding.engine.FlutterJNI.ensureRunningOnMainThread(FlutterJNI.java:992)
E/flutter (18584): 	at io.flutter.embedding.engine.FlutterJNI.destroyOverlaySurfaces(FlutterJNI.java:873)
E/flutter (18584):
F/flutter (18584): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1287)] Check failed: CheckException(env).
F/libc    (18584): Fatal signal 6 (SIGABRT), code -6 in tid 18631 (1.raster), pid 18584 (example.newdemo)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'samsung/a2coreltedd/a2corelte:8.1.0/OPR6/A260GDDU3ASJ1:user/release-keys'
Revision: '1'
ABI: 'arm'
pid: 18584, tid: 18631, name: 1.raster  >>> com.example.newdemo <<<
signal 6 (SIGABRT), code -6 (SI_TKILL), fault addr --------
Abort message: '[FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1287)] Check failed: CheckException(env).
'
    r0 00000000  r1 000048c7  r2 00000006  r3 00000008
    r4 00004898  r5 000048c7  r6 d6d8a3fc  r7 0000010c
    r8 00000000  r9 0000021c  sl 00000378  fp e9f2d560
    ip d6d8a4f8  sp d6d8a3e8  lr f5908a2f  pc f590243c  cpsr 200b0030
backtrace:
    #00 pc 0001a43c  /system/lib/libc.so (abort+63)
    #01 pc 0001252f  /data/app/com.example.newdemo-6ZzY8xph5Mu84iY3_pvSZg==/lib/arm/libflutter.so (offset 0xff8000)
Lost connection to device.

flutter doctor -v
[✓] Flutter (Channel master, 1.23.0-8.0.pre.132, on Mac OS X 10.15.4 19E2269
    x86_64, locale en-IN)
    • Flutter version 1.23.0-8.0.pre.132 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 5e97eed895 (23 hours ago), 2020-10-05 00:42:50 -0700
    • Engine revision a8d7d97bef
    • Dart version 2.11.0 (build 2.11.0-185.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • 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-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (4 available)
    • SM A260G (mobile) • 5200763ebcfa861f • android-arm    • Android 8.1.0 (API
      27)
    • macOS (desktop)   • macos            • darwin-x64     • Mac OS X 10.15.4
      19E2269 x86_64
    • Web Server (web)  • web-server       • web-javascript • Flutter Tools
    • Chrome (web)      • chrome           • web-javascript • Google Chrome
      85.0.4183.121

• No issues found!

Steps to Reproduce

Run flutter create bug.
Update the files as follows:

main.dart
import 'dart:io';

import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Web Views',
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  final _links = 'https://flutter.dev';

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        body: Center(
      child: _urlButton(context, _links),
    ));
  }

  Widget _urlButton(BuildContext context, String url) {
    return FlatButton(
      color: Theme.of(context).primaryColor,
      padding: const EdgeInsets.symmetric(horizontal: 50.0, vertical: 15.0),
      child: Text(
        url,
        style: TextStyle(color: Colors.white),
      ),
      onPressed: () => Navigator.push(context,
          MaterialPageRoute(builder: (context) => WebViewContainer(url))),
    );
  }

  void _handleURLButtonPress(BuildContext context, String url) {
    Navigator.push(context,
        MaterialPageRoute(builder: (context) => WebViewContainer(url)));
  }
}

class WebViewContainer extends StatefulWidget {
  final url;

  WebViewContainer(this.url);

  @override
  createState() => _WebViewContainerState(this.url);
}

class _WebViewContainerState extends State<WebViewContainer> {
  var _url;
  final _key = UniqueKey();

  _WebViewContainerState(this._url);

  @override
  void initState() {
    super.initState();
    if (Platform.isAndroid) WebView.platform = SurfaceAndroidWebView();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(),
        body: Column(
          children: [
            Expanded(
                child: WebView(
                    key: _key,
                    javascriptMode: JavascriptMode.unrestricted,
                    initialUrl: _url))
          ],
        ));
  }
}

pubspec.yaml
name: bug
description: A new Flutter project.

# The following line prevents the package from being accidentally published to
# pub.dev using `pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: ">=2.7.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  # The following adds the Cupertino Icons font to your application.
  # Use with the CupertinoIcons class for iOS style icons.
  cupertino_icons: ^1.0.0
  webview_flutter: ^1.0.0

dev_dependencies:
  flutter_test:
    sdk: flutter

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  # To add assets to your application, add an assets section, like this:
  # assets:
  #   - images/a_dot_burr.jpeg
  #   - images/a_dot_ham.jpeg

  # An image asset can refer to one or more resolution-specific "variants", see
  # https://flutter.dev/assets-and-images/#resolution-aware.

  # For details regarding adding assets from package dependencies, see
  # https://flutter.dev/assets-and-images/#from-packages

  # To add custom fonts to your application, add a fonts section here,
  # in this "flutter" section. Each entry in this list should have a
  # "family" key with the font family name, and a "fonts" key with a
  # list giving the asset and other descriptors for the font. For
  # example:
  # fonts:
  #   - family: Schyler
  #     fonts:
  #       - asset: fonts/Schyler-Regular.ttf
  #       - asset: fonts/Schyler-Italic.ttf
  #         style: italic
  #   - family: Trajan Pro
  #     fonts:
  #       - asset: fonts/TrajanPro.ttf
  #       - asset: fonts/TrajanPro_Bold.ttf
  #         weight: 700
  #
  # For details regarding fonts from package dependencies,
  # see https://flutter.dev/custom-fonts/#from-packages

Crash occurs if we open webview interface by tapping on a button and go back to it and rotate screen.

Logs

E/flutter ( 5036): [ERROR:flutter/shell/platform/android/platform_view_android_jni_impl.cc(44)] java.lang.RuntimeException: Methods marked with @UiThread must be executed on the main thread. Current thread: 1.raster
E/flutter ( 5036): 	at io.flutter.embedding.engine.FlutterJNI.ensureRunningOnMainThread(FlutterJNI.java:992)
E/flutter ( 5036): 	at io.flutter.embedding.engine.FlutterJNI.destroyOverlaySurfaces(FlutterJNI.java:873)
E/flutter ( 5036):
F/flutter ( 5036): [FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1287)] Check failed: CheckException(env).
F/libc    ( 5036): Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 5520 (1.raster), pid 5036 (mple.webviewbug)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'google/sdk_gphone_x86/generic_x86:10/QSR1.200715.002/6695061:userdebug/dev-keys'
Revision: '0'
ABI: 'x86'
Timestamp: 2020-10-06 14:22:48+0000
pid: 5036, tid: 5520, name: 1.raster  >>> com.example.webviewbug <<<
uid: 10135
signal 6 (SIGABRT), code -1 (SI_QUEUE), fault addr --------
Abort message: '[FATAL:flutter/shell/platform/android/platform_view_android_jni_impl.cc(1287)] Check failed: CheckException(env).
'
    eax 00000000  ebx 000013ac  ecx 00001590  edx 00000006
    edi ee14a33e  esi c6d90820
    ebp f1b73ef0  esp c6d907c8  eip f1b73ef9
backtrace:
      #00 pc 00000ef9  [vdso] (__kernel_vsyscall+9)
      #01 pc 00092328  /apex/com.android.runtime/lib/bionic/libc.so (syscall+40) (BuildId: 471745f0fbbcedb3db1553d5bd6fcd8b)
      #02 pc 000ad651  /apex/com.android.runtime/lib/bionic/libc.so (abort+193) (BuildId: 471745f0fbbcedb3db1553d5bd6fcd8b)
      #03 pc 011a329e  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #04 pc 011903a6  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #05 pc 012024a6  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #06 pc 01200303  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #07 pc 011ec5c6  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #08 pc 011ed23e  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #09 pc 011ee09c  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #10 pc 0119af8b  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #11 pc 011ecb18  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #12 pc 011ec880  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #13 pc 011fadae  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #14 pc 011a137f  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #15 pc 011a40af  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #16 pc 011a3fc0  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #17 pc 011aa74e  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #18 pc 011aa77c  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #19 pc 00018487  /system/lib/libutils.so (android::SimpleLooperCallback::handleEvent(int, int, void*)+39) (BuildId: 288ba3aff5b46dbd7e74be954af88b83)
      #20 pc 00019414  /system/lib/libutils.so (android::Looper::pollInner(int)+1044) (BuildId: 288ba3aff5b46dbd7e74be954af88b83)
      #21 pc 00018f4e  /system/lib/libutils.so (android::Looper::pollOnce(int, int*, int*, void**)+62) (BuildId: 288ba3aff5b46dbd7e74be954af88b83)
      #22 pc 00013780  /system/lib/libandroid.so (ALooper_pollOnce+96) (BuildId: 2b4362d3e0db6d6e8fbea7ad8414fbae)
      #23 pc 011aa68b  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #24 pc 011a3f6e  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #25 pc 011a3ac6  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #26 pc 011a7bf7  /data/app/com.example.webviewbug-fq05CcfxTr_0faANq5UXKQ==/lib/x86/libflutter.so (BuildId: dde17faddbb0a4d4af41fe660a89699680fb791a)
      #27 pc 0011a8e5  /apex/com.android.runtime/lib/bionic/libc.so (__pthread_start(void*)+53) (BuildId: 471745f0fbbcedb3db1553d5bd6fcd8b)
      #28 pc 000af6a7  /apex/com.android.runtime/lib/bionic/libc.so (__start_thread+71) (BuildId: 471745f0fbbcedb3db1553d5bd6fcd8b)
Lost connection to device.

flutter analyze
Analyzing webviewbug...
No issues found! (ran in 8.8s)

flutter doctor -v
[√] Flutter (Channel master, 1.23.0-8.0.pre.167, on Microsoft Windows [version 10.0.19041.508], locale fr-FR)
    • Flutter version 1.23.0-8.0.pre.167 at D:\fork\flutter
    • Framework revision f551178408 (5 hours ago), 2020-10-06 11:36:50 +0200
    • Engine revision 443cd5a1e1
    • Dart version 2.11.0 (build 2.11.0-186.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\hamdi\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Users\hamdi\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30225.117
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.49.3)
    • VS Code at C:\Users\hamdi\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.15.0

[√] Connected device (2 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)
    • Windows (desktop)                  • windows       • windows-x64 • Microsoft Windows [version 10.0.19041.508]

• No issues found!

Still crashes in Flutter 1.22 with webview_flutter 1.0.1 which was released today.

Issue not related to WebView plugin only.
By following the steps mentioned here https://flutter.dev/docs/development/platform-integration/platform-views?tab=android-platform-views-java-tab#hybrid-composition. Just navigate to new screen with the native view, go back and rotate the screen : the error is reproduced.

@blasten Some of the overlay surface JNI APIs are being invoked on the raster thread instead of the platform thread.
Is something going wrong with the raster thread merger?

I was able to repro this issue. I'm working on the fix. Thanks for the report!

I was able to repro this issue. I'm working on the fix. Thanks for the report!

@blasten any news? :)

This will be cherry picked. cc @christopherfujino

Reopening as Hybrid Compositon is causing App crash when opeing AlertDialog.
I am raising issue here, as it is flutter's issues instead of the plugin. Video - https://photos.app.goo.gl/UjMA5AYGsH2iXnJm7
Here is the link to issue- Link.


consider supporting fetching instance metadata

One thing we could consider is streamlining the process of configuring an instance to handle instance metadata.
Here we'd have an opinionated flow to fetch from the same sources as Ignition/afterburn to describe the desired system configuration, which would here be:

container image reference
configmap and secrets
actually in general all bootc install options

In this flow, the OS vendor would ship e.g. AMIs that include a systemd unit that does this by default, and checks whether it has bootc-compatible metadata.  If it doesn't (e.g. it looks like stock cloud-init config) then we ignore it?

I think we won't do this in this project.


Document Method of Comparing Policy/Rule Sizes against Limits

After discussion with engineering, it was found that pulling "Get-DLPCompliancePolicy" and "Get-DLPComplianceRule" > Outputting this info to a TXT file > Validating size of TXT file.
There is no current documentation that outlines this is an adequate way to confirm Rule Size - as such, would like this published in the "Limits" section (https://learn.microsoft.com/en-us/microsoft-365/compliance/dlp-policy-reference?view=o365-worldwide#:~:text=Also%2C you need to be aware of the following constraints of the platform%3A).

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

ID: 77dd99ae-2687-1e3f-341e-04a8e3d4acc2
Version Independent ID: 1948ad4b-3bc1-4f49-fded-9582a854cbbd
Content: Data Loss Prevention policy reference - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/dlp-policy-reference.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@MSFT-SeanG Thanks for your feedback! New issues and pull requests for this content are closed while we update this content to support upcoming feature improvements.


Post Deployment Script issue

I Have followed the instructions to add a post deployment script and it fails to work what is wrong with the below code:
these are the pieces I added to the project
	<PostDeploy Include="DeploymentScripts\Script.PostDeployment.sql" />
	<None Include="DeploymentScripts\Scripts\Event.SOFTSTOPEXCEPTION.sql" />
	<None Include="DeploymentScripts\Scripts\Task.DEFAULT_MANUAL_ACTIONREQUIRED_SHERLOCKTASK.sql" />

and in the post deployment script it self :

:r .\Scripts\Event.SOFTSTOPEXCEPTION.sql
:r .\Scripts\Task.DEFAULT_MANUAL_ACTIONREQUIRED_SHERLOCKTASK.sql

1>Adding DeploymentScripts\Script.PostDeployment.sql to the model
1>Unhandled exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
1> ---> Microsoft.SqlServer.Dac.Model.DacModelException: Add or update objects failed due to the following errors:
1>EXEC : error SQL46010: Incorrect syntax near .

Thanks for reaching out! Could you try replacing what you currently have in your project file with the following:
<Content Remove="DeploymentScripts\**\*.sql" />
<PostDeploy Include="DeploymentScripts\Script.PostDeployment.sql" />

That way we make sure that none of the scripts underneath DeploymentScripts are added to the model directly, but instead included in the overall post-deployment script.

Thanks for reaching out! Could you try replacing what you currently have in your project file with the following:
<Content Remove="DeploymentScripts\**\*.sql" />
<PostDeploy Include="DeploymentScripts\Script.PostDeployment.sql" />

That way we make sure that none of the scripts underneath DeploymentScripts are added to the model directly, but instead included in the overall post-deployment script.

Ahh! thank you, that fixed it. I thought I was going crazy


Fix Data Preview- 508 Issue

Required Fix

[ ] RF-6
[x] RF-9
[x] RF-14

Context
See EPA CAMD Accessibility Report  for how to fix and additional details

Issue #
Issue Description

RF-6
There is a button present that is   lacking information which a user needs  to understand its purpose. This issue is a violation of Section 508 and WCAG 2.0 Success Criterion 4.1.2

RF-9
Form fields that are grouped together, such as radio buttons and check boxes, do not have fieldset and legend tags. These tags provide understandability for assistive technology users. This issue is a violation of WCAG 2.0 Success Criterion 1.3.1

RF-14
An actionable element on a web page triggers an update or adds new content with no change in URL, but there is no method to alert Assistive Technology users of the update. This issue is a violation of Section 508 and WCAG 2.0 Success Criterion 1.3.1.

Testing
Tester: @aprematta

One issue found with RF-6, the rest pass:
RF-6 -
All Close buttons are labeled close instead of Close . I would actually say remove instead of close.

Testing complete, closing ticket


pdf -> svg

Steps to reproduce:
wget https://github.com/coolwanglu/pdf2htmlEX/raw/master/test/browser_tests/fontfile3_opentype.pdf
node ./examples/node/pdf2svg.js /tmp/fontfile3_opentype.pdf

Console:
# Document Loaded
Number of Pages: 3

# Page 1
Size: 596x842

Warning: Unimplemented operator dependency
Error: Error: The "Blob" constructor is not supported.

Which browser and browser version do you use? It most likely is not supported or a blob polyfill is necessary.

node.js (from command line)

Node.js does not support the Blob constructor. I'm not sure if we can fix this by using a polyfill though.


put example topology back in original state

adding 2 new example topolgies: demo_topo_1pop.py and demo_topo_3pop.py.
bring sonata_simple_topology.py back in original state.
I think the flags in dummygatekeeper.py are already the original state.

Can we also change back USE_DOCKER_MGMT = False to USE_DOCKER_MGMT = True in the dummygatekeeper.py ? Because if the docker management interfaces are deactivated, e.g., docker stats will not work. Or does this flag has no effect anymore?

The docker0 mgmt interfaces will be always activated.
USE_DOCKER_MGMT=False means: deploy the management interfaces defined in the NSD/VNFDs next to the docker0 interfaces. (interfaces defined with type=management). This is the normal/original way the dummygatekeeper works.
USE_DOCKER_MGMT=True means: do not deploy the management interfaces from the NSD/VNFDs, but use the already existing docker0 interfaces instead (basically using the docker0 network as the management network of the service)
I plan to still document this on the wiki...

Ah now I got it. Then its fine.


Bbox assertion error when using ENB models (eval & pretrain as well): assert (boxes1[:, 2:] >= boxes1[:, :2]).all()

Hi, I am using your requirements file, the same libraries, but I am receiving this bbox assertion error, only when using ENB models (ENB3 & 5), and everything is fine when using ResNet backbone.
It seems that the bbox predictions are all Nan. I've found this error in DETR but no clear solution to it (I tried different lr, different batch sizes).
In eval it appears right away, but in the pretraining mode, it is very random, at different iterations.
Epoch: [0]  [  940/78534]  eta: 8:58:29  lr: 0.000100  lr_backbone: 0.000010  lr_text_encoder: 0.000001  loss: 84.7997 (98.7713)  loss_bbox: 1.9616 (3.2480)  loss_bbox_0: 2.0860 (3.2589)  loss_bbox_1: 2.0562 (3.2474)  loss_bbox_2: 1.9565 (3.2655)  loss_bbox_3: 2.0650 (3.2771)  loss_bbox_4: 1.9537 (3.2593)  loss_ce: 10.5923 (11.1107)  loss_ce_0: 10.5793 (11.1826)  loss_ce_1: 10.5315 (11.0932)  loss_ce_2: 10.4743 (11.1262)  loss_ce_3: 10.5987 (11.1445)  loss_ce_4: 10.5877 (11.0967)  loss_giou: 1.7100 (2.0738)  loss_giou_0: 1.8241 (2.0809)  loss_giou_1: 1.7486 (2.0892)  loss_giou_2: 1.7185 (2.0759)  loss_giou_3: 1.8257 (2.0803)  loss_giou_4: 1.6419 (2.0610)  cardinality_error_unscaled: 4.8750 (5.8658)  cardinality_error_0_unscaled: 4.8750 (7.5942)  cardinality_error_1_unscaled: 4.8750 (6.0007)  cardinality_error_2_unscaled: 4.8750 (6.0588)  cardinality_error_3_unscaled: 4.8750 (5.8966)  cardinality_error_4_unscaled: 4.8750 (5.8688)  loss_bbox_unscaled: 0.3923 (0.6496)  loss_bbox_0_unscaled: 0.4172 (0.6518)  loss_bbox_1_unscaled: 0.4112 (0.6495)  loss_bbox_2_unscaled: 0.3913 (0.6531)  loss_bbox_3_unscaled: 0.4130 (0.6554)  loss_bbox_4_unscaled: 0.3907 (0.6519)  loss_ce_unscaled: 10.5923 (11.1107)  loss_ce_0_unscaled: 10.5793 (11.1826)  loss_ce_1_unscaled: 10.5315 (11.0932)  loss_ce_2_unscaled: 10.4743 (11.1262)  loss_ce_3_unscaled: 10.5987 (11.1445)  loss_ce_4_unscaled: 10.5877 (11.0967)  loss_giou_unscaled: 0.8550 (1.0369)  loss_giou_0_unscaled: 0.9121 (1.0405)  loss_giou_1_unscaled: 0.8743 (1.0446)  loss_giou_2_unscaled: 0.8593 (1.0379)  loss_giou_3_unscaled: 0.9129 (1.0401)  loss_giou_4_unscaled: 0.8209 (1.0305)  time: 0.4399  data: 0.0077  max mem: 10505
Traceback (most recent call last):
File “main.py”, line 646, in
main(args)
File “main.py”, line 549, in main
train_stats = train_one_epoch(
File “/home/ubuntu/efs/users/oignat/internship/mdetr/engine.py”, line 72, in train_one_epoch
loss_dict.update(criterion(outputs, targets, positive_map))
File “/home/ubuntu/better_glip/lib/python3.8/site-packages/torch/nn/modules/module.py”, line 1130, in _call_impl
return forward_call(*input, **kwargs)
File “/home/ubuntu/efs/users/oignat/internship/mdetr/models/mdetr.py”, line 666, in forward
indices = self.matcher(outputs_without_aux, targets, positive_map)
File “/home/ubuntu/better_glip/lib/python3.8/site-packages/torch/nn/modules/module.py”, line 1130, in _call_impl
return forward_call(*input, **kwargs)
File “/home/ubuntu/better_glip/lib/python3.8/site-packages/torch/autograd/grad_mode.py”, line 27, in decorate_context
return func(*args, **kwargs)
File “/home/ubuntu/efs/users/oignat/internship/mdetr/models/matcher.py”, line 75, in forward
cost_giou = -generalized_box_iou(box_cxcywh_to_xyxy(out_bbox), box_cxcywh_to_xyxy(tgt_bbox))
File “/home/ubuntu/efs/users/oignat/internship/mdetr/util/box_ops.py”, line 51, in generalized_box_iou
assert (boxes1[:, 2:] >= boxes1[:, :2]).all()

I have the same issue. Any progress ?

This error most likely indicates divergence in your model. You can check by adding asserts like:
assert not boxes1.isnan().any().item(), "nan in boxes1"

in the generalized_box_iou function which is currently failing.
If this is established, there could be several reasons that is happening, in order of likelihood:

Total batch-size is not big enough. We trained with total batch size = 64 (achieved with batch 2 per gpus and 4 nodes of 8 gpus). Training is possibly going to be stable at bs 32, and maybe at 16 but lower than that is unlikely to work
Mixed precision is tricky to get working. If you tried some sort of AMP autocast or FP16 training that could be the reason for the divergence
Corruption in your training data that somehow triggers gradient explosion.
In some rare cases, if the weight decay is too high it can create nans as well, although it is unlikely in that case because it happens early in training.

Hope this helps

I got the error when running inference with the released checkpoint of MDETR-EB3 on RefCOCO.

same problem

I got the same problem too when using the pre-trained MDETR-EB5 from torch.hub during inference. Does anyone know how to fix this problem?

I got the error when running inference with the released checkpoint of MDETR-EB3 on RefCOCO.

Hi,Do you have checked the img feature from EB3,in my case the feature is very large ,sometimes is NAN

This error most likely indicates divergence in your model. You can check by adding asserts like:
assert not boxes1.isnan().any().item(), "nan in boxes1"

in the generalized_box_iou function which is currently failing.
If this is established, there could be several reasons that is happening, in order of likelihood:

Total batch-size is not big enough. We trained with total batch size = 64 (achieved with batch 2 per gpus and 4 nodes of 8 gpus). Training is possibly going to be stable at bs 32, and maybe at 16 but lower than that is unlikely to work
Mixed precision is tricky to get working. If you tried some sort of AMP autocast or FP16 training that could be the reason for the divergence
Corruption in your training data that somehow triggers gradient explosion.
In some rare cases, if the weight decay is too high it can create nans as well, although it is unlikely in that case because it happens early in training.

Hope this helps

Hi I'm trying FP16 and running evaluation on Flickr30k val set. Its output is also giving "nan", only for few samples though. What is this divergence issue you're speaking of? @alcinos


Support CRUD with strongly typed sorting, filtering and pagination (paging) out of the box

Feature request:
Support sorting, filtering and pagination out of the box.
Pretty much all API's need this, unless you're doing a simple demo. So instead of every developer having to invent and implement these standard feature I think it would be greatly beneficial if this came with the framework.
You already have some work you can draw on. https://github.com/OData/WebApi already give you access to sorting, filtering and pagination via the [EnableQuery] attribute. But OData might not be suitable for everyone.
The current documented way of doing the above is relying on switching on strings.

public async Task<IActionResult> Index(
    string sortOrder,
    string currentFilter,
    string searchString,
    int? page)
{
    ViewData["CurrentSort"] = sortOrder;
    ViewData["NameSortParm"] = String.IsNullOrEmpty(sortOrder) ? "name_desc" : "";
    ViewData["DateSortParm"] = sortOrder == "Date" ? "date_desc" : "Date";

    if (searchString != null)
    {
        page = 1;
    }
    else
    {
        searchString = currentFilter;
    }

    ViewData["CurrentFilter"] = searchString;

    var students = from s in _context.Students
                   select s;
    if (!String.IsNullOrEmpty(searchString))
    {
        students = students.Where(s => s.LastName.Contains(searchString)
                               || s.FirstMidName.Contains(searchString));
    }
    switch (sortOrder)
    {
        case "name_desc":
            students = students.OrderByDescending(s => s.LastName);
            break;
        case "Date":
            students = students.OrderBy(s => s.EnrollmentDate);
            break;
        case "date_desc":
            students = students.OrderByDescending(s => s.EnrollmentDate);
            break;
        default:
            students = students.OrderBy(s => s.LastName);
            break;
    }

    int pageSize = 3;
    return View(await PaginatedList<Student>.CreateAsync(students.AsNoTracking(), page ?? 1, pageSize));
}

source: https://docs.microsoft.com/en-us/aspnet/core/data/ef-mvc/sort-filter-page?view=aspnetcore-2.0

While versatile it's rather cumbersome/annoying to write and maintain. Just imagine an entity where you'd want to sort and filter on 10 properties and not just do it on single properties but do stuff like (pseudo example) /resource?filter=PropA=foo & sort=PropB,desc & sort=PropC. That's a nightmare to write.
Why can't we have something like this:
public PagedResult<Foo> Get(QueryParams<Foo> params)
{
    var filteredSortedPagedFoos = _db.Foos.ApplyQuery(params);
    return Ok(filteredSortedPagedFoos);
}

While this might be a bit over simplified it should demonstrate the "out of box" feeling I'm looking for :)
I basically want something similar to what Spring Boot has "out of the box".
@RepositoryRestResource
public interface FooRepository extends JpaRepository<Foo, Long> {}

This code snippet gives me a fully functional CRUD controller with paging and sorting. Not filtering though. While I don't advocate Spring Boot because it's absolutely terrible compared to ASP.NET Core that particulier feature is pretty cool. Though it's rather limited and has all sorts of problems - as does everything with Spring Boot... Anyway...
Last but not least. The implementation should take DTO usage into account.
I know there are some packages that try to solve this "problem". I haven't found one that fully solves it. Since it's something every API or MVC site needs I think it would be handy with a default way to implement it.

There are a lot of different ways to achieve this functionality (OData, GraphQL, etc.). As such I'm not sure we want to bake one particular way of achieving this functionality into ASP.NET Core. Instead I would prefer to see a variety of community offerings surface with different ways of achieving this functionality that meet the needs of different apps.
@glennc @rynowak  What are your thoughts on this?

If we did something new like this, it would be JSON API - it's a pretty substantial spec, similar in scope to OData, but it's native to the domain of HATEOAS+JSON. On a personal note, in my opinion it uses the HATEOAS concepts successfully for things you'd actually want to do 😆
There's an existing OData framework build on top of ASP.NET Core https://blogs.msdn.microsoft.com/odatateam/2018/07/03/asp-net-core-odata-now-available/ - I'd recommend that you check this out @snebjorn
GraphQL is a totally different programming model. It would require an investment roughly the size of OData and how you program it, probably wouldn't have much in common with controllers.

Moving this out to the future release, so we have time to think about this.


Feature suggestion: Convert binary data to hex and back for external editors

As discussed with @mhils recently.
No tests, neither are functions placed in proper modules.
Just a demo on how binary editing could be done with conversion to a readable format, before handing it over to an external editor.
For a quick test implementation, this already works pretty well. Formatting functionality could use some optimization, though.
Workflow

Binary message body in mitmproxy (hex view)

Entering edit mode with e key (data gets converted to readable representation if not valid UTF-8, editor is Nano in this case)

Edit the "hexlified" data as you like (adding 0xdeadbeef here)

After saving in the editor, data gets converted back to binary (comments prefixed with # get stripped)

Agree on this.
(Although I'd love to see radare2 as inline hexeditor !1elf!!)
Leaving this open for future discussions and prototype for fallback in absence of console hexeditor

This was a useful exchange, thanks. I'm closing this here in favor of #5231. :)


Skywalking multi-database support

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[x] Requirement
[ ] Feature or performance improvement

Question
Does Skywalking have any plans to support multiple databases, such as MySQL, SQL Server, Oracle？

MySQL is already available, with H2 and TiDB. PostgreSQL is tracked at #6321. But Oracle? No plan on the Apache repo. But the extension mechanism supports you or someone to provide implementations on other 3rd party repo.


License?

Hello,
I've been using the matplotlib code from your WF repo for some special cases of the WF model I've been coding. I'd like to host this on GitHub with an MIT License, but I can't find a license on your repo. Do you plan on having one? If not I'll just write something different.

Glad this is helpful. The Wright-Fisher notebook can definitely be MIT License. I'll get a license included for the repo as a whole shortly.

@sarahCobey ---
Are you comfortable with licensing all slides as CC 4.0: http://choosealicense.com/licenses/cc-by-4.0/?
And software as MIT: http://choosealicense.com/licenses/mit/?
If so, I'll add the licenses.

Yes, that sounds good.

Sarah Cobey, PhD
Assistant Professor
Ecology & Evolution
University of Chicago
(617) 756-7204 (cell)
sarahcobey (Skype)
cobeylab.uchicago.edu
On Sun, Jul 17, 2016 at 6:31 PM, Trevor Bedford<EMAIL_ADDRESS>wrote:

@sarahCobey https://github.com/sarahCobey ---
Are you comfortable with licensing all slides as CC 4.0:
http://choosealicense.com/licenses/cc-by-4.0/?
And software as MIT: http://choosealicense.com/licenses/mit/?
If so, I'll add the licenses.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/trvrb/sismid/issues/10#issuecomment-233210620, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABI3QjCGN36cOjlnTjf5d_SFATM7ujZlks5qWrs2gaJpZM4GXz-J
.


[QUERY] Azure Communication Service .Net SDK: what does AddTextAnalyticsClient() in Call.cs do?

Library name and version
Azure.Communication.Calling 1.0.0-beta.29
Query/Question
Hi! I found this method in Call.cs class and I try to realize what it does. I expext there is feature to analyze text transcript during video call, it is correct? Can anyone share some code examples or just explain ho to use it? I add my analysis client and what next? How I will get response with analysis from client during video call? Thank you!
Environment
Windows 10 OS Build 17763.2565
Microsoft.NETCore.UniversalWindowsPlatform version 6.2.13
Microsoft Visual Studio Community 2022 (64-bit) Version 17.0.4

@h0las-dev Which call.cs you are referring to? Can you please share the file path ?

@SaurabhSharma-MSFT yes, it's Call.cs from Azure.Communication.Calling namespace in Azure.Communication.Calling SDK

@h0las-dev I responded over on https://docs.microsoft.com/en-us/answers/questions/737970/index.html.
TL;DR AddTextAnalyticsClient() I found as an extension method in Azure.AI.Analytics and it didn't match any signatures on the Call object. If your code is GitHub, we can take a closer look.

Oh, I'm really sorry, it was my fault, I didn't check it's just extension. Thank you!


Feature request: Global way to catch exceptions

Is your feature request related to a problem? Please describe.
Today, if I read the doco right, in order to process potential exceptions, notifications are to be used.
In this context, given 3 jobs, I have to register 3 different notification handlers for the sole purpose of logging exceptions.
However, more often than not, when an exception reaches out of a job, all we can really do is log it in an telemetry container to examine it further.
Describe the solution you'd like
Ideally, being able to register one type through an interface that would be globally leveraged by all jobs for notification purposes, without having to link each job individually to it.

In this context, given 3 jobs, I have to register 3 different notification handlers for the sole purpose of logging exceptions.

Yes - fair point and I like the feature request!
API-wise, how would you like something like:
builder.Services.AddNCronJob(o => {
  o.AddJob<MyJob>(...);
  o.AddGlobalExceptionHandler<MyHandler>();
});

It would follow ASP.NET in the sense that you can define multiple and can tell the system: "Hey, I handled everything, no need to rethrow or bubble up".

I'd :heart: that

@nulltoken
I will make a prerelease, if you want to playaroud.
The basic semantics didn't change at all:
builder.Services.AddNCronJob(o => {
  o.AddJob<MyJob>(...);
  o.AddExceptionHandler<MyHandler>();
});

If the handler returns false the next in line will handle the exception. Returning true breaks the circuit.
The order of execution:
Job -> ExceptionHandler -> JobNotificationHandler -> Dependent Job

Independent if a job is handled or not, the JobNotificationHandler will see the exception passed in. Currently the flag is only to tell the system whether or not the next handler should be invoked.

Latest preview: https://www.nuget.org/packages/NCronJob/3.1.0-preview

@linkdotnet Great! Thanks a lot for this.
Just out of curiosity, do you have a branch pointer I could peek at? That would allow me to learn a bit more about the lib.

@nulltoken
This time, because it was really straightforward I committed a sin and directly pushed to main. :D
Here the commit in question: https://github.com/NCronJob-Dev/NCronJob/commit/adaabb2e27ddafb7cf26f206b7783ea63e4a34de

With DependentJobs and IExceptionHandler I may consider removing IJobNotifactionHandler in the future.
at least that was the original motivation for those - and with that feature, I don't see much benefit anymore.

Here the commit in question: adaabb2

Looks very neat!
Just a nitpick: In https://github.com/NCronJob-Dev/NCronJob/blob/adaabb2e27ddafb7cf26f206b7783ea63e4a34de/tests/NCronJob.Tests/GlobalExceptionHandlerTests.cs, shouldn't the test exception handlers write different values to the channel writer?
This may protect against potential future regressions would the order of execution change.

Oh boy - yes. That happens if you just copy&paste stuff and test it once manually!
Also the "true" case should be checked.

Also - I will check for exceptions from the exceptionhandler. They shouldn't interfere with the system.

Also - I will check for exceptions from the exceptionhandler. They shouldn't interfere with the system.

Good catch! I didn't this of this. And it will eventually happen thanks to Murphy's law

Indeed! I will push a new preview version 3.1.1-preview for this

I will push a new preview version 3.1.1-preview for this

I gave it a try. And that makes my life a lot easier. Thanks for that.
One thing that is missing (at least in my context), would be to get a bit more data about the Job that triggered the exception.
I can extract the instance Id from the context, but in order to make the log easier to parse, I'd like to decorate the telemetry with the Job type.
Is there a built-in way in NCronJob to retrieve this? Or would it be possible to potentially extend the context with an additional property?

I am somewhat amazed that I didn't expose the jobName.
See: https://docs.ncronjob.dev/advanced/dynamic-job-control/#job-names
Maybe that would help if it is part of the IJobExecutionContext?

Maybe it wasn’t needed before because surfaced in a job context?
However, I’d really appreciate that surfacing 🙏

Sure, I will add both information

Newest preview will have these information: https://www.nuget.org/packages/NCronJob/3.1.2-preview

Awesome! I’ll provide feedback tomorrow.


Allow ".ts" in include/require statements.

Suggestion
In order to align with Deno usage, as well as potentially future support for a browser loader, it would be beneficial to be able to include files with the .ts extension included.
The option should be able to be configured. When enabled output of ".ts" extension should be omitted on the output.
If a module reference begins with http:, https: or // then the default behavior should be left in place. Possibly use separate option to download/process such modules placed in (outdir)/_cache/https__www.foo.com/path/to/mod.js.  where the protocol followed by two underscors is presented... preference for https, fallback to http in case of // at the front.
There's a pretty deep rabbit hole this could go down... that said, it would be really nice if it were easier to support Deno along with node and browser, and this would go a long way toward doing that.  Since Deno can reference project code directly in typescript, it would allow projects to be Deno first, but still allow for build targets for node and browser usage.
🔍 Search Terms
List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.

deno
".ts"
extension

✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

📃 Motivating Example

💻 Use Cases

#38149?

@MartinJohns thank you...

While there are other references.. the #38149 was closed along with other mentioned issues.
https://github.com/microsoft/TypeScript/issues/38149#issuecomment-1185887354
To me, given that part of the purpose of TS should be to support browser usage, that explicitly requires an extension... it makes far MORE sense to include the output extension for files that TSC processes, not less.  Just because it's easier to ignore the issue, does emphatically NOT mean it's not an issue.  In fact, TSC should ADD the extension where none was specified but it will be transpiling the referenced location via magic extension detection.

the #38149 was closed along with other mentioned issues.

No idea what you're talking about. It's still open.

TSC should ADD the .js extension where none was specified when it will be transpiling the referenced location via magic extension detection.

Rewriting paths has come up again and again, and the TypeScript team called this a huge no-go. Just check this issue: https://github.com/microsoft/TypeScript/issues/49083 This is also something different from your initial issue.

@MartinJohns it's pretty much the same issue... Being able to use the .ts file extension and outputting .js for reference where the reference was outputted with a different extension.
Those is for better compatibility with the browser and deno, while it also would not break node.

We're not likely to transform paths, but the closest thing we may be considering is https://github.com/microsoft/TypeScript/issues/37582.


Error when running example

I met an error when running codes with the following example Simulate spatial transcriptomic data
Code shows
set.seed(123)
example_simu <- scdesign3(
sce = example_sce,
assay_use = "counts",
celltype = "cell_type",
pseudotime = NULL,
spatial = c("spatial1", "spatial2"),
other_covariates = NULL,
mu_formula = "s(spatial1, spatial2, bs = 'gp', k= 400)",
corr_formula = "1"
)

Error information:
Input Data Construction Start
Input Data Construction End
Start Marginal Fitting
Warning message in mclapply(seq_len(n), do_one, mc.preschedule = mc.preschedule, :
“scheduled cores 1, 2 did not deliver results, all values of the jobs will be affected”
Error in names(answer) <- dots[[1L]]: NULL是不能有属性的

Hey, Dongyuan, thank you, I fix it with your suggestion.


[DO NOT MERGE][NL DateTime] Support Dutch DateTimeModel in .NET

First version of DateTimeModel, containing Date + DateTime + Holidays

current status:
DateExtractor:  ❌(61 of 113 still unsupported)
DateParser: ❌(61 of 113 still unsupported)
DateTimeExtractor: ❌(111 of 73 still unsupported)
HolidayParser: ✅ (1 of 12 unsupported)
HolidayExtractor: ✅ (9 supported)
depends on: #1100, #1098, #1097, #1094 (but can cherry pick them out if necessary)
work in progress, pr for review purposes

@tellarin done! However it looks like many .cs files just had a small change, due to the new method of generating resources. I will have a look if I can reproduce this in master and create a new PR with only those files, otherwise I will revert it.


Widget for zero-shot image classification

Would have as inputs:

an image dropzone
a field with possibles labels

as suggested by @patil-suraj (for CLIP?)
Should be decently easy once huggingface/huggingface_hub#87 is merged

Duplicate


Host-to-Plugin and Plugin-to-Host question

Hello,
I am developing an application in which the host will load some Plugins and then call their start method.
I need to be able to do 2 things after loading a plugin:

Start the plugin in a thread so that it runs independent from the host. This is the case when the host is starting up a type of client CLI plugin or as Web server plugin.

I have learned how to call methods in plugins, but I am not sure how to call a set of host methods from a plugin. I think that this is in the interface but am not clear on how it is done.

Can you please assist with these 2 cases?
thanks

Is there any update on this question?

You would use Dependecy Injection to do this easily... i think there is an example in the samples

If you want to call a method in the host, wrap it in a Task.Run() call.
await Task.Run(plugin.Start);

If you want to call a method from the host assembly, you could have a function in your plugin that accepts a function, lambda, etc. as a sort of 'callback'.
public interface IPlugin{
    void RegisterCallback(Action funcFromHost);
}

public class Plugin : IPlugin {
    public Action FuncFromHost;
    public void RegisterCallback(Action funcFromHost) {
        this.FuncFromHost = funcFromHost;
    }
}

// In host
static async Main() {
    var plugin = ...;
    plugin.RegisterCallback(hostFunc);
    await Task.Run(plugin.Start);
}

I haven't tested this.


build: refine validating Windows VHD free size

What type of PR is this?
/kind feature

What this PR does / why we need it:
refine validating Windows VHD free size
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2627

Relevant Lines:
3727

💛  - Coveralls


Integrate enhanced meltdown handling for dependency-watchdog probe

How to categorize this PR?

/area disaster-recovery
/area control-plane
/kind enhancement
What this PR does / why we need it:
This PR integrates the changes brought in with  release v0.7.0 of dependency-watchdog.
The release brings in 2 prominent changes.

The changes introduced with the DWD PR#39 are -

Enhanced the dependency-watchdog probe config yaml with new fields introduced to enable scale[Up|Down] delays.
Also dependency for each scale resource is also enabled to ensure that we only scale[Up|Down} resource provided the dependents are already in the desired state for the given scale operation.

As a result of the above 2 changes the new config file is adapted to be bring down kube-controller-manager along with machine-controller-manager and cluster-autoscaler.
Also while scaling up we give a 120 seconds delay to let the kubelet update the node status before scaling up kube-controller-manager. We give another 60 seconds delay to machine-controller-manager after kube-controller-manager is up to ensure that the node status is updated correctly by KCM.
cluster-autoscalar shall come up immediately once machine-controller-manager is up.
This shall ensure that we don't accidentally let MCM and Cluster Autoscaler to replace machines and wait for the api-server availability before taking any action.
These new changes are reflected with new config file as depicted below and are accordingly adapted in the component files for dependency-watchdog.
 probes:
 - dependantScales:
   - replicas: null
     scaleRef:
       apiVersion: apps/v1
       kind: Deployment
       name: kube-controller-manager
     scaleUpDelaySeconds: 120
   - replicas: null
     scaleUpDelaySeconds: 60
     scaleRef:
       apiVersion: apps/v1
       kind: Deployment
       name: machine-controller-manager
     scaleRefDependsOn:
     - apiVersion: apps/v1
       kind: Deployment
       name: kube-controller-manager
   - replicas: null
     scaleRef:
       apiVersion: apps/v1
       kind: Deployment
       name: cluster-autoscaler
     scaleRefDependsOn:
     - apiVersion: apps/v1
       kind: Deployment
       name: machine-controller-manager

The second list of changes are the ones introduced with the switch for leader election to endpoint leases PR#37 introduced by @ary1992.
These change requires the existing cluster role  for the dependency-watchdog-probe to be enhanced with following -
  - apiGroups:
    - coordination.k8s.io
    resources:
    - leases
    verbs:
    - create
  - apiGroups:
    - coordination.k8s.io
    resourceNames:
    - dependency-watchdog-probe
    resources:
    - leases
    verbs:
    - get
    - watch
    - update

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

Kindly review if the delay introduced for KCM and MCM are sufficient to avoid a meltdown.
Changes are tested with a local-setup of Gardener applying these on a shoot.
However one round of scaffolding test to upgrade an existing gardener cluster to the image of this PR and a newer version of DWD needs to be verified still and shall be done while the review of the PR is in progress.
We also noticed that DWD release is not updated to latest Go release and is still working with 1.13. It was an oversight as there has been no release for DWD for sometime, but we wish to include it in the imminent patch release because we still have some pending tasks to follow up -

Update the README.md with the changes introduced.
Update the test-cases for DWD to work with the config changes enhancements.
Work on the issue in DWD in reading the kubeconfigs when loading the rotated secrets#PR36
In lieu of these imminent changes we can upgrade Go version to 1.17.7  with the next patch release of DWD to be integrated with g/g.

Release note:

"NONE"

Currently the e2e tests are failing as I've imported github.com/gardener/gardener/extensions/pkg/controller/worker/genericactuator to get the MCMDeploymentName here  as there was no constant available within g/g to capture this.
If someone has a better idea of ensuring this can be traced to then I can adapt that as well.

/assign

Currently the e2e tests are failing as I've imported github.com/gardener/gardener/extensions/pkg/controller/worker/genericactuator to get the MCMDeploymentName here as there was no constant available within g/g to capture this. If someone has a better idea of ensuring this can be traced to then I can adapt that as well.

Actually, MCM is an implementation detail of the Worker controller, but I think as part of this PR it is fair to let g/g populate it. If an extension does not use MCM and rather manages the machines on its own or via different means then DWD will simply have nothing that needs to be done/scaled-down.
Potentially we can move the constant to the v1beta1constants package (where most other constants are) to resolve the import restriction issue.

@rfranzke Should the k8s dependencies and c-r also be bumped to latest, I checked DWD is using v0.17 of k8s dependencies and v0.5.5 of c-r.

@acumino I think this is not related to this PR but rather to the maintenance of https://github.com/gardener/dependency-watchdog/, is it?

@acumino I think this is not related to this PR but rather to the maintenance of https://github.com/gardener/dependency-watchdog/, is it?

Yes, but @ashwani2k mentioned he will cut the patch release with latest go version, I guess k8s dependencies should be update alongside. WDYT?

I don't know whether this is strictly required, while it's certainly a reasonable thing to do. This must be decided by the maintainers of https://github.com/gardener/dependency-watchdog/.

Adapted the release notes. Kindly review as some of them I don't have the context to map them to the right category.
Also this is different as in the DWD v0.7.0 release notes something were wrongly reflecting under others so i've fixed them as well here. I'm not sure if it has implications in the way they will reflect as it will be different.

I'm not sure how it got closed.
I think my Mac was in hanging state as it was not able to move the cursor and it clicked instead of Comment the Close with comment button.

/unhold

@kris94 Can you please add the release milestone and merge this one?

reviewed/ok-to-test


fix: bitmap inconsistent encoding

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
fix https://github.com/singularity-data/risingwave/issues/4598
Bitmap::all_high_bits uses Bitmap::num_bytes, which will be multiples of 64. This is different from the behavior of BitmapBuilder.
Bitmap::to_protobuf assumes all bytes are useful, but this is not true because of previous reason. So is null will panic, due to the usage of all_high_bits
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

BTW, all_high_bits's bits are still different from builder. Should they be consistent? 🥸


Update Dataset.dataset_size after transformed with map

After loading a dataset, if we transform it by using .map its dataset_size attirbute is not updated.

@albertvillanova I would like to take this up. It would be great if you could point me as to how the dataset size is calculated in HF. Thanks!


Add to the participants please

Hello.
I am interested in new technologies and language.
I am currently learning Python.
This project is interesting for my self-education
Thank you for the opportunity
Regards, Alex

Welcome to openAOD! I have sent an invite to join our organization. We also recommend joining our Discord server : https://discord.gg/wZ83xSJSsN. If you have any questions regarding contributing or anything else feel free to write here or ping me on Discord. Cheers. Have a great day ahead of you!


Generating all matches of a regex

Is it possible to generate a list of all matches (instead of just one match) of a regular expression using randexp.js?
For example, this regex: /(red|gray) (wolf|fox)/ would have these possible matches:
["red wolf","red fox","gray wolf","gray fox"]

look at https://github.com/alixaxel/genex.js


Update images.yaml - set logging responsibles

How to categorize this PR?

/area logging
/kind enhancement
What this PR does / why we need it:
Sets 'cloud.gardener.cnudie/responsibles' labels for logging component
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm

@JordanJordanov: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-e2e-kind-operator	32ebe22 link	true	/test pull-gardener-e2e-kind-operator
Full PR test history. Your PR dashboard. Command help for this repository. Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see our testing guideline for how to avoid and hunt flakes.

is another occurrence of #8036
/test pull-gardener-e2e-kind-operator

/test pull-gardener-e2e-kind-operator


Template for creating new operators

I would like to see some sort of RFC process for adding new operators to Rx itself.  Currently, there is no defined standard for acceptance criteria for what is included and not included in the library itself.
Taking some hints from the Rust RFC Process, we can glean a bit on how others handle substantial changes to the language/runtime/etc.  Changes such as these are not to be taken lightly, as Rx can suffer from an API bloat as we cover a wide array of use cases already.  If a new solution can simply be made from existing operators, that should be preferred, over writing a new core operator.  Conversely, it could also remain in user land for the users who really need said operator to import it as needed from another repository.
Here is what I think the minimum bar should be in order to add a new operator to Rx:

A thorough explanation of the operator and its use cases.  This also includes error handling, completion, early unsubscription, and infinite case semantics just as we define in our unit tests.

An explanation of how current operators cannot accomplish this task, or if they can, but performance is not ideal.

Complete documentation including marble diagrams for further discussion

From there, the core team should enter discussions on whether to proceed or not.

Currently, there is no defined standard for acceptance criteria for what is included and not included in the library itself.

There is something: https://raw.githubusercontent.com/ReactiveX/rxjs/master/.github/PULL_REQUEST_TEMPLATE.md

If a new solution can simply be made from existing operators, that should be preferred, over writing a new core operator.

Complicated to follow this rule, since we can do .mergeMap(x => Observable.of(f(x))) instead of map(f). Should we drop map?

A thorough explanation of the operator and its use cases. This also includes error handling, completion, early unsubscription, and infinite case semantics just as we define in our unit tests.
An explanation of how current operators cannot accomplish this task, or if they can, but performance is not ideal.
Complete documentation including marble diagrams for further discussion

Agreed a lot.

@staltz

There is something: https://raw.githubusercontent.com/ReactiveX/rxjs/master/.github/PULL_REQUEST_TEMPLATE.md

But that's already for a pull request, and this discussion should happen before a pull request even happens, because we don't want to waste anyone's time in having them implement the full operator without a chance of it being accepted.

Complicated to follow this rule, since we can do .mergeMap(x => Observable.of(f(x))) instead of map(f). Should we drop map?

No, because as I stated, if performance is not ideal, then it should be considered.


CMO late joined with no backback equiped

Description
I got a report of a player spawning without a backpack. Checking through logs they latejoined through the terminal as a CMO on meta station. Round ID 55383
Additional context
I dont know if this is a preset but both the player and I were confused how it happened.

redo the loadout to fix

you can unequip alot of stuff in loadouts, might be because of that.

you can unequip alot of stuff in loadouts, might be because of that.

no its because they're spawning with an invalid prototype (the old CMO backpack that got deleted)

looks like this might be fixed?


Fixing the flaky test TupleSketchIntegrationTest

The sketch bytes size can be 1740 or 1756.

Codecov Report

Merging #11290 (351f6c6) into master (a6e23c1) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #11290     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2230     2156     -74
  Lines      120140   116983   -3157
  Branches    18218    17772    -446
=========================================
  Hits          137      137
+ Misses     119983   116826   -3157
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

integration2temurin11
?

integration2temurin17
?

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
?

unittests2temurin20
0.11% <ø> (ø)

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

Discussed offline. Currently we have one integration test per function to test the wiring of v2, which is not scalable and way too expensive. We should consolidate them into one test for all these queries. We can take this separately.


[ST] Execution reverted ezETH ETH->Blast

Describe the bug
TX fails on call, eg:

https://etherscan.io/tx/0x081166f1a652af6ec1cc23b1e58757c46650f79d7e589913a4d26398ea15cad0?locale=en-US&utm_source=imtoken
https://etherscan.io/tx/0x2f507998f2f6abc20c7c695c582578d5cf388e6d3a6ceee8fff7764804e10932?locale=en-US&utm_source=imtoken
https://etherscan.io/tx/0x0012fd6fefcedf6bfa7f674871c498c4492696b67eff232c82dd928af4309f84

Successfull transfer which was sent right after tx №3 (in the list above): https://etherscan.io/tx/0xd832aa2ca3bb5cde8cf37025e9608ef162d8af05d983453bf5f74d8c7262cbcb
Additional context
The user spent ~75$ paying fees for nothing. There's enough allowance and enough ETH for gas in the user's wallet.
@oncall

https://etherscan.io/tx/0x081166f1a652af6ec1cc23b1e58757c46650f79d7e589913a4d26398ea15cad0?locale=en-US&utm_source=imtoken
https://etherscan.io/tx/0x2f507998f2f6abc20c7c695c582578d5cf388e6d3a6ceee8fff7764804e10932?locale=en-US&utm_source=imtoken
https://etherscan.io/tx/0x0012fd6fefcedf6bfa7f674871c498c4492696b67eff232c82dd928af4309f84

all of these succeed in simulations provided the gas is increased. do you know how these transactions were initiated @oswy-cpu ?

bridge UI?

@LayneHaber thanks Layne, I'll ask the user and will suggest him to bump the gaslimit

was not able to repro locally in the bridge ui, fwiw the gas for these transactions is not calculated within the SDK but from the users injected provider or the static provider on the bridge page. I would also recommend they rotate providers in their metamask.
going to close this for now


Create JSX component of Accordion using Tailwind

Create JSX component of Accordion  using Tailwind. First create an issue using the given template and follow guidelines on Readme

Hi 👋
I would like to make my contribution as part of Hacktoberfest'21
Can you assign this issue to me?

TW Accordion #39


Rust syntax highlighting broke

Before:

After:

Everything looks closer to when I have rust-analyzer disabled:

I honestly really liked the old styling because not many themes had the amount of clarity catppuccin did. If there was some way to get a mix of the old behavior as well as the new styling, that would be awesome!

Please see https://github.com/catppuccin/vscode/issues/62, these changes were all intentional, and offering two entirely different syntax styles (2.x/3.) would make maintaining this theme extremely difficult.
I just added Chromatic UI testing (https://github.com/catppuccin/vscode/pull/154) to ensure that any future syntax changes in 3.x will not cause unforeseen changes; so I'm committed to make 3.x, and it's style, the best version of this extension.
If you're still happier with the old theme, you can always install an older version from the marketplace:


Add support for maintainer in PKG-INFO

Fixes #1288.
I have not really gone through this with a fine-toothed comb to see if it meets all of the specifications of PEP 345, but this is at least a step in the right direction.

Looks like you dropped checking for project_urls when moving the sieve from write_pkg_file() to get_metadata_version(), where it was previously setting a minimum metadata version of 1.2 per the Project-URL entry for PEP 345. Was that intentional, or an oversight? I'm just trying to figure out whether I should push a PR to readd the check for project_urls or whether it wasn't really necessary in the first place.

@fungi I believe the reason that I dropped checking for project_urls because PEP 345 specifies that both Requires-Python and Project-URL are required, and so anything using version >= 1.2 must have Requires-Python.
That said, this is a heuristic based on metadata generated mostly from setup(), so probably the original checking for "if you request any features of 1.2, we should create a valid version 1.2+ file".
So it was intentional but probably wrong. Feel free to make a PR to add it back.

Thanks, will do. For the record, I found this specifically because I noticed after a new setuptools release I was suddenly ending up with PKG-INFO files claiming to be metadata version 1.1 compliant even though they included Project-URL entries.


fix(deps): update dependencies

:tada: This PR is included in version 8.2.2 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.9 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.9 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.12 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Use WireMock in kamelet-chucknorris example project tests

The service is quite flaky so it'd be better to mock it.

Done in https://github.com/apache/camel-quarkus-examples/pull/273.


Update to use byebug directly so we can use newer pry

Hoping this will fix build issue on ruby head

Coverage remained the same at 96.03% when pulling 941e3ba687165e98659bd78127ca32a4e1b30be9 on dillonwelch:pry-pls into e7abd250388ec8e44c4a96a6d703d2b0655c3ce2 on devise-security:master.

https://github.com/deivid-rodriguez/pry-byebug/pull/346 looks like there are some issues with the gem and latest versions of Ruby.


So, Keybase has cancelled the Space Drop. They claim that it’s because of fake accounts. But I’m calling BS. They cancelled everyone after the 1st 200k accounts no matter what (I say this because if they’d done as they said they did, then I and many others wouldn’t have been booted). Then say they split the remaining funds up evenly amongst the “real” accounts.
But...Something is fishy.
I’m looking through keybase’s space drop payments. And it seems their source wallet has been creating new wallets and depositing the exact amount in each new one that should have been deposited in existing accounts. Why would they need to create so many new wallets if they’re supposedly SHRINKING they number of wallets?! Why would a source wallet need to create a new one?! There’s one new wallet sitting somewhere with the XLM that was promised to me and I’m sure there are numerous other new wallets with other’s XLM in it, and these new wallets belong to Keybase.
And conveniently, Keybase admins won’t address anyone’s questions on this page and continue to ignore everyone.
My account is real and verified. But keybase unverified me (so that I wouldnt get paid) and left my account open (must NOT be one of the accounts that they find to be unworthy of staying open, just not worthy of being paid).
Read their BS excuse here: https://keybase.io/a/i/r/d/r/o/p/spacedrop2019
Here is the link to the SpaceDrop source wallet showing the creation of all these new “ghost wallets”: https://stellar.expert/explorer/public/account/GDV4KECLSZLKRVH4ZTWVAS4I3W2LPAPV66ADFFUZKGIVOTK6GMKGJT53

wow!


Gallery item detail 3.1

Once https://github.com/kodadot/nft-gallery/issues/958 is closed and #1134 refactored into better code we can continue to main-nuxt best with this

Check whenever folded chart and price chart loads data
https://github.com/kodadot/nft-gallery/issues/957 show only when you can do something with NFT
https://github.com/kodadot/nft-gallery/issues/956
https://github.com/kodadot/nft-gallery/issues/990 is related to adding under history chart

Pls fix this
Pastels.Fractal._.Low.Carbon.NFTs.-.Mozilla.Firefox.2021-11-18.18-01-30.mp4

Fixed in https://github.com/kodadot/nft-gallery/pull/1246

Meanwhile done


don't add followers if there are none

I noticed a few errors in the logs when update_task was being invoked. The asana client returns a ValueError when there is either an empty array of followers or an array of empty strings. I'll cover the first case and see if I need to cover the second.
Pull Request synchronized with Asana task

I'm curious how we can get into this situation... There should always be at least one follower added (since we add all mentioned users and the author or commenter themself). Do you have an idea of what would trigger this? Perhaps a comment from someone who isn't in the Github -> asana user id mapping?

@gregeinfrank If I'm reading the logs correctly, it looks like this pr https://github.com/Asana/asanadotcom/pull/7565 is one of the prs that caused an error.
the reviewer has an entry in the sgtm-users table but there are no other participants in the PR (other than bots). I don't have asanadotcom admin permissions but I suspect that there are no followers being passed in otherwise we would be receiving a different error message


White, empty interface

Hi all.

As you can see, tusk is showing a white space.
When I use terminal to run Tusk these come up:
[am@awesomebook ~]$ "/opt/Tusk/tusk-app" %U
{ Error: Cannot find module '../tray'
    at Module._resolveFilename (module.js:485:15)
    at Function.Module._resolveFilename (/opt/Tusk/resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.get [as Tray] (/opt/Tusk/resources/electron.asar/browser/api/exports/electron.js:11:16)
    at Object.exports.create.win [as create] (/opt/Tusk/resources/app.asar/tray.js:129:22)
    at App.app.on (/opt/Tusk/resources/app.asar/index.js:112:10)
    at emitTwo (events.js:130:20)
    at App.emit (events.js:213:7) code: 'MODULE_NOT_FOUND' }
[am@awesomebook ~]$

I'm running on antergos, KDE desktop, and tried both 0.10.1 (still available on AUR) and version 0.11.0.

Looks a lot like a connectivity issue, try pressing Ctrl + R to reload, hopefully that will resolve it.

Closing, for now, will reopen if needed in the future.


-c yes continues even if .terraform directory is missing

if for some reason the .terraform directory is missing and the parameter -c yes was passed to aws2tf then it will continue running and fail to import with this error over and over again
ls: cannot access '../.terraform': No such file or directory
pi2 using initing TF provider
aws_iam_instance_profile ecsInstanceRole import

I'll add a check for -c - to ensure the previous run exists

added check


Release Google.Cloud.Support.V2 version 1.0.0

Changes in this release:
No API surface changes; just dependency updates and promotion to GA.

Created release for Google.Cloud.Support.V2-1.0.0


[feat] Added forceSpectator flag.

Codecov Report

Merging #22 into master will increase coverage by 0.55%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   82.46%   83.01%   +0.55%
==========================================
  Files           2        2
  Lines         154      159       +5
  Branches       54       54
==========================================
+ Hits          127      132       +5
  Misses         27       27

Impacted Files
Coverage Δ

src/server/datastore/ephemeral.ts
78.86% <100.00%> (ø)

src/utils.ts
97.22% <100.00%> (+0.44%)
:arrow_up:

Continue to review full report at Codecov.

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


MultipleStores

Hey! I'am using multiple stores in my project, and I want to knew.
Is each time when I call NewStore*(*sql.DB) it will open a new connection to db or pg driver will manage connections and use only limited pool of connections?

All New*Store does is use the provided *sql.DB, if you give all the stores the same instance of DB, they will use just that one. New*Store does not create any connections by itself.

And a new question)
From 1 table I have fk to another table, ex:
type ONE struct{
id int64
}

type TWO struct{
One  One `fk:"one_id"
}

so, how can I get type Two, if I have ONE.ID?

According to your model, TWO has an inverse key, so it would be like this:
type ONE struct {
    kallax.Model
    ID int64 `pk:"autoincr"`
}

type TWO struct {
    kallax.Model
    One One `fk:"one_id,inverse"`
}

Note that a normal fk means the foreign key field is in the other model, if the foreign key is in the same model you need to use inverse.
After generating the code, you should have all this available:
store := NewTWOStore(db)
two, err := store.FindOne(NewTWOQuery().FindByOne(one.ID))
if err != nil {
        // handle err
}

FindByXXX is generated for every relationship in a model.
Hope this helps you!

Sorry man, my bad, missed it in tutorial! Thanks!!!!!

No problem! :)

@erizocosmico is store instance thread-safe?


add network driver based on the Virtio Over MMIO

Base on the work of @duanyu-yu, we developed a network driver based on the Virtio Over MMIO interface, which is part of the Virtual I/O Device Specification. Details are published at https://docs.oasis-open.org/virtio/virtio/v1.1/csprd01/virtio-v1.1-csprd01.html
This network driver doesn't depend on the PCI bus and can be used within micro VMs. The current version is developed and tested on Qemu's microvm virtual platform. Details are published at https://qemu.readthedocs.io/en/latest/system/i386/microvm.html
The current version can be tested with the example web server of RustyHermit's Git repository. Please use following command to build the web server:

cargo build --no-default-features --features smoltcp -p httpd

Afterwards, the following command starts the web server withinthe microvm:

qemu-system-x86_64 -M microvm,x-option-roms=off,pit=off,pic=off,rtc=on,auto-kernel-cmdline=off -global virtio-mmio.force-legacy=off -nodefaults -no-user-config -display none -smp 1 -m 1G -serial stdio -kernel loader/target/x86_64-unknown-hermit-loader/debug/rusty-loader -initrd target/x86_64-unknown-hermit/debug/httpd -enable-kvm -cpu host -device isa-debug-exit,iobase=0xf4,iosize=0x04 -netdev tap,id=tap10,ifname=tap10,script=no,downscript=no -device virtio-net-device,netdev=tap10

r=me


How to get depth of given area in the picture that get by D435i? use python thanks!

Issue Description
<Describe your issue / question / feature request / etc..>
How to get depth of given area in the picture that get by D435i? use python thanks!

Hi @qingweihk  If you want to get a depth reading of the distance of an object that the camera is observing, Intel's distance_to_object Python tutorial may meet your needs.
https://github.com/IntelRealSense/librealsense/blob/jupyter/notebooks/distance_to_object.ipynb

Hi @qingweihk   Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


fix(hooks): token drop down console error

What kind of PR is this?:

/kind fix
What this PR does / why we need it:
Fix token dropdown button hooks error

Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Error will only show while selecting dTokens in token dropdown on the first time only, to reproduce it again will need reload the app
Developer Checklist:

[x] Read your code changes at least once
[x] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[x] No console errors on web
[x] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Still able to replicate after a few tries

@pierregee Done


Fix custom job and no active job logic and logs

Issue #, if available:
Description of changes: This change -

Fixes the logic and error codes to handle custom job
Identifies no active job and removes unnecessary file close
Removes redundant handling of no active job/parse error
Cleanup log in cases of custom job parsing failure and no active jobs

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Let's add a unit test for the new length check if we can.

Added unit tests in the commit https://github.com/aws/ota-for-aws-iot-embedded-sdk/pull/178/commits/1443c4d5e7978945ae5f08d8fc08a77997444c93


Terminal erratic behavior + duplicate path after updating VSCode

VSCode Version: 1.13.0 (1.13.0)
OS Version: macOS Sierra - 10.12.5
using ZSH

Steps to Reproduce:

updated to latest VSCode version
open terminal instance
look at path (see image)

when interacting with terminal (typing/clicking) there is a flicker and text flashes while the path looks to correct itself... but always returns to its duplicated state.

Same here : https://github.com/Microsoft/vscode/issues/28562

Duplicate of #26038
@Tyriar would the fix for #26038 also fix the flicker/flash issue?

@ramya-rao-a yep, the flicker happens whenever the terminal is "refreshed", for example when a keypress enters a char.


[release] 1.11.1

Do post-release for 1.11.1

Since we don't actually release a Jaeger image we can revert to Jaeger 1.58 without needing to release again.
I seen the same last night but was too tired to write the PR.

@puckpuck the PR is ready, it is just matter of getting it merged 😅


Adding ignoredUnusedImportPaths to sortConfiguration

In files that are using JSX, React has to be imported, even if it's not used explicitly. For cases like that one, I've added ignoredUnusedImportPaths, array of paths that should be ignored when removing unused imports.

🙏

Is there any way we make this real?


Make CORS origins configurable

In the latest version of boardgame.io, the permitted origins must be provided.
At the moment this is set to the following in gameServer.js.
origins: [
    '*'
]

This would probably be better set up from a config file
This will also affect the deployment process

When setting up such a config file, one must not forget to also adapt the docker and heroku deployments.


Adding more query types

This is a feature request. Presently the only queries are
pwd
ls

but I believe that there are other useful features that could be added. Looking at texosquery my list would be
osname
osversion
date-time
time-zones
tmpdir

There is already the ifplatform package, but it requires shell-escape (otherwise it just distinguish between Windows or not Windows). Possibly osversion is not that important as well as tmpdir. On the other hand, date-time and time-zones can be very useful; currently pdftex and luatex can provide very compact date information already, but xetex and perhaps ptex based engines don't.

currently pdftex and luatex can provide very compact date information already, but xetex and perhaps ptex based engines don't.

If you meant \pdfcreationdate and \pdffilemoddate pdfTeX primitives, they are both required by the l3kernel (see here) hence both have engine-independent expl3 correspondences: \c_sys_timestamp_str and \file_timestamp:n.

I was wondering what the queries such as:
date-times and time-zone
were intended to supply.
Please explain the details.

@muzimuzhi @car222222
You can look at the documentation of texosquery. The OS is able to provide faster access to a wealth of information about dates and time zones. This information is obviously accessible with LuaTeX, but not with other engines. With \(pdf)creationdate only the time zone offset is available, not the ID or daylight saving.
This might simplify the working of packages such as datetime2, for instance.


PR 41 broke the functionality, here's a fix to it

Screencast from 15-05-24 06:48:20 PM IST.webm
solution to #44

LGTM 👩🚀


Login Loop

After adding the application inside the Teams, the signing process keeps in loop.
We successfully log in the application inside Teams, and in the end, the application requests a new sign-in.
This happens for the readers and for the authors.

JoeJony- I am experiencing the same issue only it only occurs with iOS devices, Android, desktop client & web all work fine. I opened up a ticket with MSFT and have not made any progress.

Please check below options and try.
• Check if the app registration, Company Communicator deployment and instance of Teams are in the same tenant. If they are different, then at the time of providing parameters, in the custom template while deploying, the tenant id of teams instance needs to be provided.
image
To get the teams instance tenant id –
Go to any team -> click on three dots besides the team name -> click on ‘get link to team’ -> Copy the link -> Extract the ‘tenantId’ field from the copied link.
To change tenant id after deployment -
Go to ‘Configuration’ settings of App Service -> change the "AllowedTenants" & "Tenant id" field value to the one you copied and Save -> Restart the App service -> refresh the tab under Teams.
• In the app registrations check if the supported account types is multiple organizations.

Everything is correct in my deployment, yet I am stuck in the Sign in loop....any other suggestions?

@kahopkin The tenant Id should match the tenant where you plan to use the Company Communicator application.
Example: You may have an Azure subscription in tenant A, and may want to deploy the application for tenant B. In this scenario, you should provide tenant B's ID.
If you've already deployed, update the tenant Id in the configuration for all the resources.
Closing the issue, please re-open if you still face it.


How do I change the text used in calendar

I want to change the text Select month to my own language and also  Select year how can I do this?

What do you mean by "to your language"? Currently, it is in English, and are you looking to have it translated into your language? These strings are part of the library and require translation in the string files. May I know which language you need?

dutch select month should be selecteer maand and select year should be select selecteer jaartal

Are they correct?
Selecteer maand
Volgende maand
Vorige maand
Volgende week
Vorige week
Selecteer jaar

yes they are correct now

When are they gonna be updated?

With the release of the next version.

Awesome thanks for adding it in :D amazing library. Do you have a rough estimate when the next version is released?

I do not have any time this weekend, but I will attempt to fix some more issues and release a new version the weekend after...


Machine mac-10 low of disk space.

Machine mac-10 low of disk space.

Problem has been resolved and fixed by removing old files under directory /Users/flutter/remove free up disk space.


jpeg-xl not showing full color range

System Details

Windows 10 22H2
Thorium 126

Problem
When opening https://jpegxl.info/test-page/ on both Firefox Nightly 131a and Thorium 126, observing the red jpeg xl image shows different results - on the Thorium image the logo in the image is not visible, meaning that the full color range of the image is not shown. On Firefox, the logo is visible.
Screenshots
Firefox:

Thorium:

Additional Notes
Thank you for your work!

After further research I have discovered that "The red box is actually meant to be blank if you don't have a HDR screen" and the issue is in the FF implementation of the codec.


how to remove producer from producerlist？

I have build a private network which have 5 BP,when I stop a BP,for example:A,I can see other BP‘s log ,when  wait for A produce block,because A have been stoped,it wait 6 seconds,and in this 6 seconds,no blocks was produced,how to avoid this ?

in this 6 seconds,no block was produced,so any tx will accepted but not write in block,and account balance refer to above tx have no change,this is a risk.

Unregister an existing producer
Usage: cleos system unregprod  account
Positionals:
account TEXT                The account to unregister as a producer (required)

I have run undelegatebw ，voteproducer ，unregprod cmd ,and scaled of A has down to 0 in producedlist,but there still wait 6 seconds,and no block was produced in this 6 seconds.

There's no way to reduce the number of producers. In your case there's no way to remove producers since the number of producers is less than 21. However you can vote another 16 new producers so that would be 5+16=21 producers then A will run out of the top 21 producer list.


flows are glitchy in safari

Screen Recording 2020-07-27 at 4.19.47 PM.mov.zip
See video for more info.

I made a flow with no real query and then asked it to run.


Checking the validity of transaction before deleting.

Describe the bug
The deleteTransaction function does not check if the id parameter is a valid transaction. This could lead to an invalid transaction being deleted.
Expected behaviour
The function should first check, if the transaction which is being deleted is a valid one or not
Screenshots
No response
Additional context
No response

please assign me this issue under GSSoC '23, I will fix it..

Hey @KritikaJoshi22, are you planning on making changes to the contract? If yes, can you elaborate on how you will check if the transaction is valid?