flutter run (when spawned from external script) crashes when its stdin is written to

Steps to Reproduce

description: I was attempting to induce a hot reload from a build/test script written in rust,
this example illustrates the attempted use of q hotkey after flutter is running

install rust toolchain (https://www.rust-lang.org/tools/install)
cargo new flutter_stdin
paste https://gist.github.com/smokytheangel0/c2f4b97ab0037a1e1357b9848dd17274 into flutter_stdin/src/main.rs
cargo build in flutter_stdin
flutter create stdin_test in ../
cp flutter_stdin/target/debug/flutter_stdin stdin_test
./flutter_stdin in stdin_test

Logs

Unhandled exception:
FileSystemException: writeFrom failed, path = '' (OS Error: Broken pipe, errno = 32)
#0      _RandomAccessFile.writeFromSync (dart:io/file_impl.dart:881:7)
#1      _StdConsumer.addStream.<anonymous closure> (dart:io/stdio.dart:312:15)
#2      _rootRunUnary (dart:async/zone.dart:1138:13)
#3      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#4      _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#5      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#6      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:266:7)
#7      _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:773:19)
#8      _StreamController._add (dart:async/stream_controller.dart:649:7)
#9      _StreamController.add (dart:async/stream_controller.dart:595:5)
#10     _StreamSinkImpl.add (dart:io/io_sink.dart:158:17)
#11     _IOSinkImpl.write (dart:io/io_sink.dart:291:5)
#12     _StdSink.write (dart:io/stdio.dart:341:11)
#13     Stdio._stdioWrite.<anonymous closure> (package:flutter_tools/src/base/io.dart:276:12)
#14     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
#15     _rootRun (dart:async/zone.dart:1126:13)
#16     _CustomZone.run (dart:async/zone.dart:1023:19)
#17     _runZoned (dart:async/zone.dart:1518:10)
#18     runZoned (dart:async/zone.dart:1502:12)
#19     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
#20     Stdio._stdioWrite (package:flutter_tools/src/base/io.dart:275:5)
#21     Stdio.stdoutWrite (package:flutter_tools/src/base/io.dart:269:9)
#22     StdoutLogger.writeToStdOut (package:flutter_tools/src/base/logger.dart:270:48)
#23     StdoutLogger.printStatus (package:flutter_tools/src/base/logger.dart:265:5)
#24     printStatus (package:flutter_tools/src/globals.dart:128:10)
#25     TerminalHandler._commonTerminalInputHandler (package:flutter_tools/src/resident_runner.dart:1161:5)
#26     TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1303:13)
#27     _rootRunUnary (dart:async/zone.dart:1134:38)
#28     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#29     _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#30     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#31     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:266:7)
#32     _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:377:20)
#33     _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:252:5)
#34     _AsBroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:476:11)
#35     _rootRunUnary (dart:async/zone.dart:1134:38)
#36     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#37     _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#38     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#39     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:266:7)
#40     _SinkTransformerStreamSubscription._add (dart:async/stream_transformers.dart:70:11)
#41     _EventSinkWrapper.add (dart:async/stream_transformers.dart:17:11)
#42     _StringAdapterSink.add (dart:convert/string_conversion.dart:238:11)
#43     _StringAdapterSink.addSlice (dart:convert/string_conversion.dart:243:7)
#44     _Utf8ConversionSink.addSlice (dart:convert/string_conversion.dart:314:20)
#45     _ErrorHandlingAsciiDecoderSink.addSlice (dart:convert/ascii.dart:254:17)
#46     _ErrorHandlingAsciiDecoderSink.add (dart:convert/ascii.dart:240:5)
#47     _ConverterStreamEventSink.add (dart:convert/chunked_conversion.dart:74:18)
#48     _SinkTransformerStreamSubscription._handleData (dart:async/stream_transformers.dart:122:24)
#49     _rootRunUnary (dart:async/zone.dart:1134:38)
#50     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#51     _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#52     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#53     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:266:7)
#54     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:773:19)
#55     _StreamController._add (dart:async/stream_controller.dart:649:7)
#56     _StreamController.add (dart:async/stream_controller.dart:595:5)
#57     _Socket._onData (dart:io-patch/socket_patch.dart:1878:41)
#58     _rootRunUnary (dart:async/zone.dart:1138:13)
#59     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#60     _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#61     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#62     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:266:7)
#63     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:773:19)
#64     _StreamController._add (dart:async/stream_controller.dart:649:7)
#65     _StreamController.add (dart:async/stream_controller.dart:595:5)
#66     new _RawSocket.<anonymous closure> (dart:io-patch/socket_patch.dart:1423:33)
#67     _NativeSocket.issueReadEvent.issue (dart:io-patch/socket_patch.dart:927:14)
#68     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#69     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#70     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#71     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:169:5)

[✓] Flutter (Channel master, v1.15.4-pre.201, on Linux, locale en_US.UTF-8)
    • Flutter version 1.15.4-pre.201 at /home/j0/flutter
    • Framework revision 52a5311924 (5 days ago), 2020-02-27 10:41:08 -0800
    • Engine revision 060a7733a6
    • Dart version 2.8.0 (build 2.8.0-dev.10.0<PHONE_NUMBER>)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /home/j0/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling
      support)
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /home/j0/Android/Sdk
    • Java binary at: /home/j0/Downloads/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit
      https://flutter.dev/setup/#android-setup for detailed instructions.

[!] Android Studio (version 3.6)
    • Android Studio at /home/j0/Downloads/android-studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.41.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.8.1

[✓] Connected device (1 available)
    • Nexus 6P • 84B7N15A07006474 • android-arm64 • Android 8.1.0 (API 27)

! Doctor found issues in 2 categories.

this code which never writes to stdin does not produce an error, but obviously it also does not perform as intended: https://gist.github.com/smokytheangel0/96edd82a5ced3ac25703609538ee1464

Hi @smokytheangel0
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

This was not a post for assistance, I am under the impression that there is a bug in how the flutter tool deals with stdin, the rust code is only there to provide the minimum code to demonstrate the bug. I suppose you could reproduce the bug in other languages, the task was to operate the hot loaded key commands like 'r' for hot reload, 'q' for quit, etc, from a program that is piped to the stdin of the flutter tool.

my build system would very much benefit from being able to operate the flutter run interface through stdin, or a c binding, but I am blocked by this bug.

Hi @smokytheangel0
If you believe there is an issue with Flutter itself
To better address the issue, would be helpful
if you could post a minimal code sample to reproduce the problem without the 3rd party code/sdk
Thank you

I'm also hitting this. I wrote a ruby wrapper around the Flutter tool, using IO.popen and also hit this.

I'm also hitting this. I wrote a ruby wrapper around the Flutter tool, using IO.popen and also hit this.

I'm also hitting this. I wrote a ruby wrapper around the Flutter tool, using IO.popen and also hit this, without even trying to write to the tool's stdin. For example:
      IO.popen([@path_to_flutter, 'pub', 'get']).close

will result in essentially the same stacktrace as OP.

I was able to solve this by reading the stdout stream from my script. The failure was happening because the tool was trying to write to a pipe that was closed by the parent process. This was my fix:
-      IO.popen([@path_to_flutter, 'pub', 'get']).close
+      IO.popen([@path_to_flutter, 'pub', 'get']) { |io| io.read }

However, it would be better if the tool threw a more helpful error.

I'm also hitting this. I wrote a ruby wrapper around the Flutter tool, using IO.popen and also hit this, without even trying to write to the tool's stdin. For example:
      IO.popen([@path_to_flutter, 'pub', 'get']).close

will result in essentially the same stacktrace as OP.

I was able to solve this by reading the stdout stream from my script. The failure was happening because the tool was trying to write to a pipe that was closed by the parent process. This was my fix:
-      IO.popen([@path_to_flutter, 'pub', 'get']).close
+      IO.popen([@path_to_flutter, 'pub', 'get']) { |io| io.read }

However, it would be better if the tool threw a more helpful error.

I can reproduce this issue on stable and master. I used the rust code above, slightly edited to work on Windows. See code sample below.

Code sample
use std::io::{BufRead, BufReader, Write};
use std::process::Command;
use std::process::Stdio;

fn main() {
    let mut run = Command::new("cmd")
        .args(["/C", "flutter", "run"])
        .stdout(Stdio::piped())
        .stdin(Stdio::piped())
        .spawn()
        .expect("failed to run flutter run");

    let out = BufReader::new(
        run.stdout
            .take()
            .expect("failed to create bufreader from run stdout"),
    );
    for line in out.lines() {
        let read_line = line.expect("failed to unwrap line in run stdout printer");
        if read_line.contains("http") {
            break;
        }
    }

    {
        let std_in = run.stdin.as_mut().expect("failed to get mut stdin");

        std_in
            .write(b"q")
            .expect("failed to write 'q' to flutter run stdin");
    }

    run.wait().expect("failed to wait for child process");
}

Logs
Unhandled exception:
FileSystemException: writeFrom failed, path = '' (OS Error: The pipe is being closed.
, errno = 232)
#0      _RandomAccessFile.writeFromSync (dart:io/file_impl.dart:873:7)
#1      _StdConsumer.addStream.<anonymous closure> (dart:io/stdio.dart:286:15)
#2      _rootRunUnary (dart:async/zone.dart:1442:13)
#3      _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#4      _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#5      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#6      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#7      _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:774:19)
#8      _StreamController._add (dart:async/stream_controller.dart:648:7)
#9      _StreamController.add (dart:async/stream_controller.dart:596:5)
#10     _StreamSinkImpl.add (dart:io/io_sink.dart:136:17)
#11     _IOSinkImpl.write (dart:io/io_sink.dart:269:5)
#12     _StdSink.write (dart:io/stdio.dart:315:11)
#13     Stdio._stdioWrite.<anonymous closure> (package:flutter_tools/src/base/io.dart:358:12)
#14     Stdio._stdioWrite.<anonymous closure> (package:flutter_tools/src/base/io.dart:357:22)
#15     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:32)
#16     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:109:18)
#17     _rootRun (dart:async/zone.dart:1426:13)
#18     _CustomZone.run (dart:async/zone.dart:1328:19)
#19     _runZoned (dart:async/zone.dart:1861:10)
#20     runZonedGuarded (dart:async/zone.dart:1849:12)
#21     runZoned (dart:async/zone.dart:1780:12)
#22     asyncGuard (package:flutter_tools/src/base/async_guard.dart:109:3)
#23     Stdio._stdioWrite (package:flutter_tools/src/base/io.dart:357:5)
#24     Stdio.stdoutWrite (package:flutter_tools/src/base/io.dart:343:7)
#25     WindowsStdoutLogger.writeToStdOut (package:flutter_tools/src/base/logger.dart:726:12)
#26     StdoutLogger.printStatus (package:flutter_tools/src/base/logger.dart:524:5)
#27     TerminalHandler._commonTerminalInputHandler (package:flutter_tools/src/resident_runner.dart:1582:13)
#28     TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1685:13)
#29     _rootRunUnary (dart:async/zone.dart:1434:47)
#30     _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#31     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#32     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#33     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#34     _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:377:25)
#35     _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:244:5)
#36     _AsBroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:474:11)
#37     _rootRunUnary (dart:async/zone.dart:1434:47)
#38     _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#39     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#40     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#41     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#42     _SinkTransformerStreamSubscription._add (dart:async/stream_transformers.dart:63:11)
#43     _EventSinkWrapper.add (dart:async/stream_transformers.dart:13:11)
#44     _StringAdapterSink.add (dart:convert/string_conversion.dart:238:11)
#45     _StringAdapterSink.addSlice (dart:convert/string_conversion.dart:243:7)
#46     _Utf8ConversionSink.addSlice (dart:convert/string_conversion.dart:317:20)
#47     _ErrorHandlingAsciiDecoderSink.addSlice (dart:convert/ascii.dart:279:17)
#48     _ErrorHandlingAsciiDecoderSink.add (dart:convert/ascii.dart:265:5)
#49     _ConverterStreamEventSink.add (dart:convert/chunked_conversion.dart:72:18)
#50     _SinkTransformerStreamSubscription._handleData (dart:async/stream_transformers.dart:111:24)
#51     _rootRunUnary (dart:async/zone.dart:1434:47)
#52     _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#53     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#54     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#55     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#56     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:774:19)
#57     _StreamController._add (dart:async/stream_controller.dart:648:7)
#58     _StreamController.add (dart:async/stream_controller.dart:596:5)
#59     _Socket._onData (dart:io-patch/socket_patch.dart:2314:41)
#60     _rootRunUnary (dart:async/zone.dart:1442:13)
#61     _CustomZone.runUnary (dart:async/zone.dart:1335:19)
#62     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1244:7)
#63     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:341:11)
#64     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#65     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:774:19)
#66     _StreamController._add (dart:async/stream_controller.dart:648:7)
#67     _StreamController.add (dart:async/stream_controller.dart:596:5)
#68     new _RawSocket.<anonymous closure> (dart:io-patch/socket_patch.dart:1839:33)
#69     _NativeSocket.issueReadEvent.issue (dart:io-patch/socket_patch.dart:1322:14)
#70     _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
#71     _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)
#72     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:122:13)
#73     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:193:5)

flutter doctor -v (Stable)
[✓] Flutter (Channel stable, 3.0.5, on Microsoft Windows [Version 10.0.22000.856], locale en-IN)
    • Flutter version 3.0.5 at C:\Users\Nabeel Parkar\fvm\versions\stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (4 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\android-sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = C:\Program Files\BraveSoftware\Brave-Browser\Application\brave.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.6)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.2.32630.192
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.70.1)
    • VS Code at C:\Users\Nabeel Parkar\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.46.0

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.856]
    • Chrome (web)                 • chrome        • web-javascript • unknown
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 104.0.1293.47

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

flutter doctor -v (Master)
[✓] Flutter (Channel master, 3.1.0-0.0.pre.2216, on Microsoft Windows [Version 10.0.22000.856], locale en-IN)
    • Flutter version 3.1.0-0.0.pre.2216 on channel master at C:\Users\Nabeel Parkar\fvm\versions\master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c91b481263 (3 hours ago), 2022-08-10 23:19:55 -0400
    • Engine revision 1b3b3edac1
    • Dart version 2.19.0 (build 2.19.0-81.0.dev)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\android-sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = C:\Program Files\BraveSoftware\Brave-Browser\Application\brave.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.6)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.2.32630.192
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.70.1)
    • VS Code at C:\Users\Nabeel Parkar\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.46.0

[✓] Connected device (4 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.856]
    • Chrome (web)                 • chrome        • web-javascript • unknown
    • Edge (web)                   • edge          • web-javascript • Microsoft Edge 104.0.1293.47

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


build version current does not have a default for an un-tagged history

The auto-version controller should default to 0.0.0 or 0.0.1 if no version tags are found.

This was resolved by https://github.com/rapid7/builderator/pull/39.


How can I make sure the page load is complete? onLoadStop called twice on Android

[x] I have read the Getting Started section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
3.3.2

Plugin version
5.7.2+2

Android version
12 and 8

iOS version

macOS version

Xcode version

Device information: Samsung S21 and Samsung S7
Description
Expected behavior:
Call onLoadStop once when page load finished.
Current behavior:
onLoadStop called twice on Android (on iOS, onLoadStop called one time as expected)
Steps to reproduce

Implement inappwebview as sample code
Check printed messages in console

final Set<Factory<OneSequenceGestureRecognizer>> _gestureRecognizers = {const Factory(EagerGestureRecognizer.new)};
  final UniqueKey _webViewKey = UniqueKey();

  InAppWebViewController? webViewController;
  final InAppWebViewGroupOptions _webviewOptions = InAppWebViewGroupOptions(
    crossPlatform: InAppWebViewOptions(useShouldOverrideUrlLoading: true, mediaPlaybackRequiresUserGesture: false),
    android: AndroidInAppWebViewOptions(useHybridComposition: true),
  );

  @override
  Widget build(BuildContext context) {
    return BlocProvider<XBloc>(
      create: (context) => getIt.get<XBloc>(),
      child: BlocConsumer<XBloc, XState>(
        listener: (context, state) {
          if (state is LoginFailedState) {
          } else if (state is LoginCompletedState) {}
        },
        builder: (context, state) {
          return InAppWebView(
            gestureRecognizers: _gestureRecognizers,
            initialOptions: _webviewOptions,
            initialUrlRequest: URLRequest(url: Uri.parse(context.read<XBloc>().requestUrl)),
            key: _webViewKey,
            onWebViewCreated: (controller) => webViewController = controller,
            onLoadError: (controller, url, code, message) => log('onLoadError ${url.toString()}, $code $message'),
            onLoadStart: (controller, url) => log('onLoadStart ${url.toString()}'),
            onLoadStop: (controller, url) async {
              log('onLoadStop ${url.toString()}');
              final redirectUrl = context.read<XBloc>().redirectUrl;
              if (url.toString().startsWith(redirectUrl)) {
                BlocProvider.of<XBloc>(context).add(GetAccessEvent(url.toString()));
              }
            },
            onReceivedServerTrustAuthRequest: (InAppWebViewController controller, URLAuthenticationChallenge challenge) async {
              return ServerTrustAuthResponse(action: ServerTrustAuthResponseAction.PROCEED);
            },
          );
        },
      ),
    );
  }

From this issue on Stackoverflow "Android - Webview onPageFinished Called Twice", it seems related to how Chromium WebView is implemented on native side by Android (check this answer).
For your use case, I think you have 2 options:

use onProgressChanged and check when the progress == 100 to execute your code.
use a boolean flag inside onLoadStop to check if that part of code has been already executed.

Hello my friends,
The solution best way;
Problem --> Android device onPagefinished  calling twice. (At the IOS device has not problem)
Solution --> Before OnLoadStop You can Call onLoadStart method that  define bool param in this section.


Student code-in in the projects page

Describe the bug
On the projects page the heading i.e., 'STUDENT CODE-IN' should redirect to the home page of the website i.e., scodein.tech which not doing as expected.
To Reproduce
can add a link to the heading present in the nav of the projects page.
Expected behavior
Must redirect to the parent page/site scodein.tech

Issue resolved


RangeError: Maximum call stack size exceeded

When a module contains exported enum, it cause RangeError
Example:
export enum Repositories {
    User = "User"
}

 Fixing tinspector issue


[BUG, Android] The app crashes after opening the Profile page

[BUG, Android] The app crashes after opening the Profile page
Steps:

download the app
login into your account
tap on the Profile or navigate from the burger menu
observe the result

Actual result:
The app crashes after opening
Expected result:
Profile page opens without issues
Env:
Android QA app Version 2.5.1-3 (4238)
Attachments:
Logs - https://app.zenhub.com/files/166818382/439dbd18-1d19-47a6-a153-dcdea82a5ea2/download
https://images.zenhubusercontent.com/166818382/d1dea806-afb5-4ccc-8154-0ed492386281/32e3ebcc_5a01_4f4d_a937_c7f9b13f00be.mp4

@L03TJ3 the problem is that we tried to render the flag as component, and they are not longer svg components but pngs

fixed b7e4cf0d1c69e53f6a3d3395862adafafb01397d

@sirpy Only partial fixed for now. fixed on android but doesn't show up on web now

fixed web: https://github.com/GoodDollar/GoodDAPP/commit/f31b1463ce6f2bfdf8dcf072b834ddcc599ca911

verified on prod


Use with Material UI

I did find this and this in my search before posting from a few years ago.
I am using Material-UI v4 in my project and would like to make the styling match for this input component. I can just try to make the CSS match (though it is no small task). I was wondering how possible it would be to:

Add Support for MUI since this is a very widely used Component Library (I know how unlikely this is but hey, can't blame me for trying).

For me to somehow replace Geosuggest's input with MUI's?

If not I guess the CSS route will work but this means potential issues anytime MUI updates there CSS, as well as having to replicate the animations.
I currently use this package but the outputs are not structured very well and I much prefer the way your's does (parsing the address_components array).
Thank you!

Discussion of this in #447.


docs: update README to include dependency

In writing an ansible role to deploy to AWS, I found an undocumented dependency. npm run build fails without make, not being able to build leveldown.

Not sure why it is showing I haven't signed the CLA. It's signed.


[PROG] Insertion Sort

Program Description
Sorting an array using Insertion Sort.
Additional
Update the documentation (DOC.md in sorting folder) to include insertion sort.
Checklist

[x] This issue has not been mentioned in existing issues.

do i need to write code and explanation of insertion of sort and add to the folder?


ghw: create a new scope for every iteration of a generate-for

Currently it looks like we just merge all iterations together which results in several duplicate variables.

#6

Currently it looks like we just merge all iterations together which results in several duplicate variables.

I've put an example in Xcodo/MRE with code and GHW and FST files. I generated them with GHDL 4.1.0 on Windows.
GHW file: https://github.com/Xcodo/MRE/blob/main/wellen-6/wellen_6.ghw
FST:

GHW:


how to input my own pictures?

how could i use my own pictures and get the result? I couldn't find the Interface to change the pictures in demo.py.

You can modify the related dataset provider to read your own picture. The datasets/linemod/linemo_dataset.py for LineMOD dataset and the datasets/ycb/ycb_dataset.py for the YCB-Video dataset.

You can modify the related dataset provider to read your own picture. The datasets/linemod/linemo_dataset.py for LineMOD dataset and the datasets/ycb/ycb_dataset.py for the YCB-Video dataset.

Hi Ethan,
In this case, how would I deal with the intrinsic matrix?

Hello, do you implement the output of importing your own RGBD image into the model?


msz_per_channel？

Thanks for your work! Could you explain the  meaning of  the argument 'msz_per_channel' for me?

First of all, thank you for your interest.
"msz_per_channel" means the memory size (or budget size) per channel.
FreD selects informative frequency domain dimensions and utilizes them. In this case, the informative frequency domain dimensions are assumed to be the same for each channel. Therefore, in FreD, the budget size required to represent one synthetic instance can be expressed as (channel $\times$ msz_per_channel).
For example, let's assume ipc=1, msz_per_channel=64, and CIFAR-10.
Then, the budget size of each class is (ipc $\times$ channel $\times$ height $\times$ width) = 1 $\times$ 3 $\times$ 32 $\times$ 32 = 3072.
Since FreD represents one instance with a budget size of (channel $\times$ msz_per_channel) = 3 $\times$ 64 = 192, it can create a total of 3072/192=16 instances at the same budget size.
In the actual code, the formula is represented without the channel size since it disappears in the division operation.
If you have any further questions, please let me know.


Progress Spinner is not working in angular material v2.0.0-beta.12

Bug:
Progress spinner is not working in v2.0.0-beta.12. I am getting an error as:
compiler.es5.js:1689 Uncaught Error: Can't resolve all parameters for MatProgressSpinner:
([object Object], [object Object], [object Object], ?).
    at syntaxError (compiler.es5.js:1689)

Also, Angular cli is throwing a warning as:
WARNING in<EMAIL_ADDRESS>230:83-91 "export 'DOCUMENT' was not found in '@angular/common'

I checked the above file which is<EMAIL_ADDRESS>and realised that it is trying to import { DOCUMENT } from '@angular/common'; which is possibly not present in the version of angular common (v4.2.4), which I am using right now.
After commenting out this line, angular cli stopped throwing an error, but progress spinner is not working as this is required dependency for it.
What is the current behavior?
angular cli build is throwing warning and progress spinner is not working
What are the steps to reproduce?
Updating angular material from 2.0.0-beta.11 to 2.0.0-beta.12.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"dependencies": {
  "@angular/animations": "4.2.4",
  "@angular/cdk": "2.0.0-beta.12",
  "@angular/common": "4.2.4",
  "@angular/compiler": "4.2.4",
  "@angular/core": "4.2.4",
  "@angular/forms": "4.2.4",
  "@angular/http": "4.2.4",
  "@angular/material": "2.0.0-beta.12",
  "@angular/platform-browser": "4.2.4",
  "@angular/platform-browser-dynamic": "4.2.4",
  "@angular/router": "4.2.4",
  "core-js": "2.4.1",
  "rxjs": "5.4.2",
  "zone.js": "0.8.14"
},
"devDependencies": {
  "@angular/cli": "1.3.2",
  "@angular/compiler-cli": "4.2.4",
  "@angular/language-service": "4.2.4",
  "@types/jasmine": "2.5.53",
  "@types/jasminewd2": "2.0.2",
  "@types/node": "6.0.60",
  "codelyzer": "3.1.1",
  "jasmine-core": "2.6.2",
  "jasmine-spec-reporter": "4.1.0",
  "karma": "1.7.0",
  "karma-chrome-launcher": "2.1.1",
  "karma-cli": "1.0.1",
  "karma-coverage-istanbul-reporter": "1.2.1",
  "karma-jasmine": "1.1.0",
  "karma-jasmine-html-reporter": "0.2.2",
  "protractor": "5.1.2",
  "ts-node": "3.2.0",
  "tslint": "5.3.2",
  "typescript": "2.3.3"
}

@angular/material has the following peer dependencies
"@angular/cdk": "2.0.0-beta.12",
"@angular/core": "~4.4.4",
"@angular/common": "~4.4.4"

Hello @willshowell : Thanks for informing about this. Let me try this once to check if it is working for me or not.  I am wondering if it is documented somewhere? I didn't find peer dependency of this anywhere.

No, but you should have received a warning during npm install.

Oh yes! I missed that warning message while doing npm install. I installed peer dependencies and and now it is working. Thanks @willshowell


String serialization can produce invalid JSON for some input strings

Hello, I've recently run into some issues serializing and deserializing data using the Graph SDK when the data contains string fields that have special characters besides the already escaped tab/newline characters. For instance, if I create an email with a body with content type text that contains the backspace character (hex character code 0x08), serialize it using the Graph SDK, and then attempt to deserialize it I get an invalid JSON error message because the backspace character is not properly escaped. The set of characters that requires escaping is available in section 7 of RFC 8259
To better highlight these edge cases I've created a new branch in a fork of the repo with a set of (currently failing) tests. A diff showing just the added tests is here
While I was working on a patch for this issue I noticed that the Graph SDK is using custom logic to perform all character escaping in strings. This made me curious, is there a particular reason not to use golang's standard library json.Marshal or an instance of a standard library encoder? It seems like if HTML-safe encoding wasn't needed then an encoder with that feature turned off and manually removing the additional newline character it adds should do the trick. It also seems like it would avoid some of the issues where some, but not all, character escape sequences are handled since the golang standard library should be well tested for edge cases

Hi @ashmrtn
Thanks for using the Go SDK and for reaching out.
The go SDK is generated from metadata (OpenAPI description) using kiota. One of the key tenets of Kiota is the generated models are not tied to any specific serialization format/library. This is to allow decoupling, reusability, interoperability, for performance reasons.... But also because some JSON serialization libraries don't allow customization of deserialization. e.g. if an endpoint describes returning a collection of Directory Object, which are Users, Service Principals, Groups.... you want the derived type to be used so pattern matching works. Another example is a dateTime field that's usually represented either as a number or as a string, and should be deserialize to the right type instead of scalar string/number....
This is why kiota clients rely on auto-serialization (each model type describes how to serialize/deserialize itself in the code), which in turns relies on this implementation here. Now if we compare Go with dotnet/java, both those languages have a JsonWriter offered by the serialization library (System.Text.Json, GSON), which handles escaping, making sure the JSON is valid etc... while providing higher level APIs (writeBoolean, string, etc...) But Go doesn't seem to offer this, which is why we're doing this directly in a byte slice here.
The escaping is being done here at the moment, but it could be beneficial to replace things by another library if we could, or move it to a dedicated wrapper (outside of kiota's primitive concepts) and encapsulate it here.
https://github.com/microsoft/kiota-serialization-json-go/blob/b9ef4053b6157070b5aeef7c5383565c3b583109/json_serialization_writer.go#L55
Is this something you'd like to work on? Let us know if you have further questions

ah sorry, I don't seem to have added enough context to some of my statements. I meant using the golang standard library's JSON marshalling only for strings. I understand that there's other data types that need to be serialized differently since Serialize for various objects is used for the wire format sent to the Graph servers which means it needs to comply with what the servers expect
I can create a quick patch for the string serialization issue that this Github issue is referencing
If there was a desire to use the golang standard library serialization functions in the future, it could be possible if the generated code was expanded to implement the MarshalJSON method (see the standard library documentation) as that would allow the standard library to use those calls instead of its more generic logic. I don't know if I would have the time to work on that though, assuming there was interest for that feature

Do you mean to replace the replace alls, the string to byte conversion from the writeStringValue method so the escaping is handled natively? If so, please go ahead with a PR, it'd be a great improvement!


[1.16.5 & 1.18.2] Flux Networks tries to wirelessly charge slots from Curios after they no longer exist

(I really hope 1.16.5 and 1.18.2 still updated with bug fixes and I'm not just wasting my time here.)
Describe the bug
If Flux Networks is set to wirelessly charge curios slots, the game will sometimes crash if a curios slot is removed using any means.
To Reproduce
Create a new world
Place down a Flux Controller, create a new network, assign the flux controller to this network (giving power to the network is optional, the game will crash regardless).
Under the Wireless Charging section, enable the charging of Curios slots.
Then, repeatedly run the following two commands until the game crashes (replace "charm" with a slot type that is currently registered in-game):
/curios add charm @p 1
/curios remove charm @p 1
Alternatively, repeatedly equip and unequip an item that gives you extra curios slots.
It may take more than one attempt. I was (un)lucky enough to have it work first try for me when reproducing the issue with a limited mod list.
Reproduced using the following mods:
Flux Networks (FluxNetworks-1.16.5-<IP_ADDRESS>.jar)
Curios API (Forge) (curios-forge-1.16.5-<IP_ADDRESS>.jar)
As well as the following mods for convenience sake:
Just Enough Items (JEI) (jei-1.16.5-<IP_ADDRESS>.jar) (Self explanatory why I would want this for testing)
Charm of Undying (Forge) (curioofundying-forge-1.16.5-<IP_ADDRESS>.jar) (Without a mod that adds a curios slot, I would have to resort to using a datapack to create one. I did not feel like doing that. This issue can be reproduced fine without this mod, as long as you have some other method of adding a curios slot.)
Expected behavior
For the game not to crash.
Screenshots
Not applicable for this bug, but I can provide a crash report.

replicated in 1.18.2 as well, basically the exact same crash so i don't feel the need to send a second crash report

I have also encountered this on 1.18.2, but also found a way to naturally encounter it during gameplay rather than requiring commands that is reasonably likely to be done organically:
Blood Magic's Living Armor has the upgrade Socketing that adds 1-5 curio slots depending on the level, and removing a Living Chestplate with any level of this upgrade when wireless charging is enabled on a flux network will consistently crash the game. You can also remove these slots using a ritual that clears the upgrades on Living Armor, which will also consistently crash the game.
Crash report from an environment with only Blood Magic and its dependencies, Curios API, Flux Networks, & JEI

This should be fixed in 3847318fbd46b8e82cc5e71d97c9b1a49b58fd5c. 1.16 is dropped and won't get fixed.


Fixes zend-psr7bridge composer decription

:-1:

why :-1: ?

I think that if somebody wants to use Zend\Expressive\Router\Zf2, they should know that they need to install package zendframework/zend-psr7bridge. I think it is good now.

ok ;)


Vulnerability roundup 92: qemu-4.2.0: 1 advisory [7.5]

search, files

[ ] CVE-2020-16092 CVSSv3=7.5 (nixos-20.03)

Scanned versions: nixos-20.03: 925ae0dee63.
Cc @edolstra

See also: #94724, #88387, #93985, #92872, #91309, #90958

20.03 is Frozen and EOL.
As such, issue is closed.


Offchain storage get expected enum Result

Storage get return Result not Option

@sacha-l   Please review it


using co with node 0.10.x?

Hi,
co's home page says:
When using node 0.10.x and lower or browsers without generator support, you must use gnode and/or regenerator.
but when gnode an app, got:
ReferenceError: Promise is not defined
    at co (.../node_modules/co/index.js:50:14)

is there a way to polyfill this with Bluebird? or what is the version of co that can be used with node 0.10.x?
Thanks,
A.C.

co@4+ requires a Promise implementation. For versions of node < 0.11 and for many older browsers, you should/must include your own Promise polyfill.

So yeah, you should do global.Promise = require('bluebird') or similar.

Yes you can override global.Promise. Also, check out this current PR for "bring your own promise" in this repo. Kind of interesting (and related) https://github.com/tj/co/pull/244

@madbence @joshbeam yeah, this will work for most of the cases, but we using co in test framework ava and including global polyfill will affect code under testing (if code relies on Promise object, but don't polyfill it in test).


Logos missing from Community page

Describe the bug
Discord and YouTube logos are missing.
To Reproduce
Visit https://atsign.dev/community/
Expected behavior
All boxes have Logos
Screenshots

hey i can fix the issue..

I think I can do it

hey  @cpswan   assign me this i am working on this


Update pattern-matching.md

Fix in-page links

I am rerunning the tests. Let's see if it passes now

@gkepka could you open up a separate PR? Maybe the branch is too old?


Simplify use of enums

It would be very nice for zig-sqlite to deal with converting enums to ints and back on its own. Having to write this at call site causes unnecessary boilerplate. As seen in this example:
try select.bind(.{
        .id = filter.id,
        .name = filter.name,
        .status = @enumToInt(filter.status),
    });

instead of just being able to:
try select.bind(filter);

I'm not opposed to adding this but I would want to avoid using @enumFromInt so that parsing an invalid value doesn't invoke undefined behavior.
I'll keep this open as a note to myself for the next time I make some changes.


Cmo api

@DavidAntliff here is my latest code. I've also pushed changes to a forked version of the example project. This has the latest driver code integrated but I'll probably go back and rename .uninitialize to .close or something easier to type. Thoughts?
I'm not able to get the RMT input working for some reason. Testing here shows perfect output timing, minimal or no bus contention and the device responding, but there is no valid input reported from the RMT. I'm stumped and would appreciate another set of eyes.

@DavidAntliff had a chance to take a look at this yet? I also asked on the ESP32 forums for some help, had one suggestion but nothing that looks related to the rmt rx issue that I'm seeing with this code.

Not yet, I should have time later today or tomorrow. I'll see if anything about RMT RX stands out to me. Thank you for your efforts.

@DavidAntliff I may have one lead. I don't believe I'm enabling input on that gpio pin and going to try that once I wrap up a couple of other things.

Any luck with the GPIO input enable?

I should be back to try that today. After fixing a few things in libesphttpd I was looking into why the gpio driver has inconsistent measurement times and unfortunately can’t see any reason for it beyond other operations being done in the processor.
I’ll let you know as soon as I’ve retested it.

@DavidAntliff and enabling open drain fixed the issue. Everything looks solid, timing looks good, temperatures look good and no crc errors so far!
Thoughts on when you could take a look at it? I'd prefer working from your main repo rather than my fork.

Sounds good - I will look at it and test with my circuit tonight (~12 hours from now).

Am I right in thinking your PR needs the "latest" ESP-IDF rather than v2.1.1? I'm having trouble with rmt_get_ringbuf_handle - doesn't seem to be in 2.1.1.

@davidantliff sounds good to me! I’m really ok either way with the api but thought I’d explain why I did it. Oh, and from my work in embedded I always favor static allocation but the esp32 has so much memory it looks like people mostly don’t care! I’m used to the stm32 :-)
Did you see the changes I made to the ds18b20 library and example? I think I pushed those for your review but didn’t add pull requests to them. If you’d like me to take a look just let me know!

@davidantliff I also did a write up of the timing issues with the gpio driver. They look caused by overall system load. http://chmorgan.blogspot.com/2017/11/oscilloscope-assisted-debugging.html?m=1

Yes, I'm starting to find myself just dynamically allocate data structures on the ESP32 myself - plenty of RAM for my applications so far. It does make things a bit simpler, and if performance is a concern then I just allocate them early. However I like to keep the option for static allocation since that's also what I'm used to.
I saw your changes in the other libraries and I've pretty much copied them into my own branches, with a few editorial changes (I wanted to keep some of the old code commented out until I get a chance to revisit). You can submit PRs for your forks if you like, otherwise I'll just merge these:
https://github.com/DavidAntliff/esp32-ds18b20/commits/esp32-owb-pr4
https://github.com/DavidAntliff/esp32-ds18b20-example/commits/esp32-owb-pr4
Thanks also for the writeup. I've always felt that the GPIO approach was a compromise and I also ran into issues when I enabled WiFi. It was always my intention to use a peripheral like RMT (or similar) but I didn't find the time. Thankfully you've been able to contribute, and for that I appreciate your efforts.

I really wanted those crc errors fixed! Thanks for all of the help along the way.

@chmorgan Just for the record, I have had my ESP32 sampling from three DS18B20 devices every second for almost the last 10 days (+850k samples), using this new approach, and I have had zero CRC errors. Good work!

Very cool! I’d say that was a good test run.
On Fri, Dec 1, 2017 at 11:31 PM David Antliff<EMAIL_ADDRESS>wrote:

@chmorgan https://github.com/chmorgan Just for the record, I have had
my ESP32 sampling from three DS18B20 devices every second for almost the
last 10 days (+850k samples), using this new approach, and I have had zero
CRC errors. Good work!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DavidAntliff/esp32-owb/pull/4#issuecomment-348667508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ-AHCvcm-DdC6dVej49sZA3JsL08W6ks5s8NKdgaJpZM4QiTKA
.


Add app-hooks.md to manifest [REVIEW - DO NOT MERGE UNTIL APPROVED]

@sarahxxu related to Issue #46 , added app-hooks.md to the bottom of the Guides section of the docs. Please review and lmk if you prefer this doc to appear in a different order within the Guides section.

Thanks Wendy! Merged


Feature Card Section Updated

fix:- #123
The Feature Card Section has been updated .
The changes can be visible as i am attaching before and after effects.
Before:-

After :-

Card is not responsive


Assign address to a pci bridge while appending it

This will allow to determine the PCI address of the device attached to the bridge within the guest.

@jodh-intel Comments addresses, ptal.

@devimc @sboeuf Rebased.

@amshinde TestQemuArchBaseAppendBridges failed

@sboeuf k8s CI seems to be failing. Is this something we have seen before?

@amshinde no the failure is about the kata-log-parser. We have something in the pipe for that, in the meantime, please re-trigger the jenkins job.


feat: support i18n for extension. currently only supports English and Chinese.

Now only support configuration and command.
There are some output message still using english.
If you agree with this, I will finish the rest.
refs:
https://github.com/shanalikhan/code-settings-sync/issues/581

This is good.
Thanks !

If you agree with this, I will finish the rest.

What you will be adding more.
just send me and let me know i will accept and release this in newer version

@shanalikhan
I am trying to translate updateSetting command, please check it out and see what can we improve

I am thinking about that should I translate the console message?
console message is only for development, not for user.
so we don't need to translate it.
for example:
https://github.com/shanalikhan/code-settings-sync/blob/3aa9e6c77a6c2ab83f147a73eed926633309e90e/src/commons.ts#L136

I am planning to translate the extension configuration、command、status bar message、information message

This is great.
We dont need to translate the console messages.
Yes i have checked the updateSetting title in english and chinese.
So we will be focusing on English and Chinese only for now.
It would be great if you can create new page in Wiki and write some documentation for users to allow their language support for this extension.
What keys they need to localize and what file name should they create.
https://github.com/shanalikhan/code-settings-sync/wiki

@shanalikhan
I have updated a lot for this. it should be almost done.
thanks for your time to review!
here is the document to show how to support their language:
https://github.com/shanalikhan/code-settings-sync/wiki/How-to-support-my-language%3F

@axetroy
It would be great if you can you check the v3.2 branch. the localization isnt working at the moment. Release is pending due to this.
Thanks

@shanalikhan
Hi.
I have tried v3.2 in<EMAIL_ADDRESS>Japanese has worked.

Thanks
Its working now
Get Outlook for iOShttps://aka.ms/o0ukef

From: Axetroy<EMAIL_ADDRESS>Sent: Thursday, October 18, 2018 7:40:08 PM
To: shanalikhan/code-settings-sync
Cc: Shan Khan; Mention
Subject: Re: [shanalikhan/code-settings-sync] feat: support i18n for extension. currently only supports English and Chinese. (#582)
@shanalikhanhttps://github.com/shanalikhan
Hi.
I have tried v3.2 in<EMAIL_ADDRESS>Japanese has worked.
[2018-10-18 22 36 58]https://user-images.githubusercontent.com/9758711/47162486-9feb0e00-d326-11e8-992a-cd075b4b6097.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/shanalikhan/code-settings-sync/pull/582#issuecomment-431035145, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIXjnKtmgFcBqNmPF0y912tgnXTEvonAks5umJLIgaJpZM4VFzG_.


one Stripe customer record didn't get imported with subscriber status

No need for a deep investigation here, just a note in case it changes anything ahead of Flaming Hydra import from Hype.
User<EMAIL_ADDRESS>emailed us and said he couldn't login to Sludge.
Indeed, his email was a free member in our Ghost, not subscribed.
Our Stripe shows he did indeed sign up on Hype at 2023-07-21T05:36:26.000Z with transaction ID ch_3NWC0rLRnByG5VQf0ietrX01. He does show up as paid in our CRM export from Hype, too.
(I've given him free access through 2024.)
I think our Hype import, done sitting together, was Aug. 3, so I'm not sure why his one record seems to have been missed. All his user activity is here, not that that offers a clue.

This isn't priority, since this is the only user who's emailed us. Otherwise all seems fine.
It's possible we used a July export from Hype to import into Ghost, dunno. That's just one possibility. Not crucial to figure out.

Reminder for me to investigate how somehow the user<EMAIL_ADDRESS>got his card charged again on Nov. 29, when he had canceled. Somehow Stripe had his card 6673 also associated with
<EMAIL_ADDRESS>, which had ended its paid subscription years ago.


STAR-1340 Add more metrics for counters and batches

The patch adds 2 more metrics for counters:

the lock wait time,
the number of locks in a counter mutation.
No tests for them as OSS doesn't test existing metrics either.

It also adds the number of batches executed in the coordinator and
tests them.
Port of CNDB-4100

I think that the CounterMutationMetricsTest is not formatted according to the code style, please check/auto reformat perhaps?

@jakubzytka I don't see what is not formatted according the code style. Can you comment at the places where it doesn't match the style guide? Note it is not CNDB style guide, and Apache style doesn't indent in many cases.
Since there is no reliable formatting, it might happen that some code wasn't formatted, but I try to make sure that IntelliJ auto-formatted all new code.


Liz Line Cambridge Heath

Cambridge Heath announcement on Elizabeth Line cannot be played
e6df257cf344421f8a5e07c49dd64198

This file only has a middle inflection version, which is why it's broken. I'll use that version in place of the end inflection.


[AutoRelease] t2-iotfirmwaredefense-2023-06-27-39129(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4267
Live test success

issue link:https://github.com/Azure/sdk-release-request/issues/4267

/azp run prepare-pipelines

/azp run prepare-pipelines

/azp run prepare-pipelines


Maintenance: metrics unit testing strategy

Description of the feature request
Problem statement
Metrics unit tests currently are not structured the same as the Logger module,
In the logger only public methods are tested, following the recommendations here: https://github.com/goldbergyoni/javascript-testing-best-practices/#--14-stick-to-black-box-testing-test-only-public-methods
Opening a ticket to track discussions and work on improving this,
Related issues, RFCs
https://github.com/awslabs/aws-lambda-powertools-typescript/pull/102

The issue is still current so contributions are welcome.
This is a good issue for newcomers as it allows to understand how the utility works in depth, as well as for those who are looking at sharpening their Jest skills.
If anyone is interested in picking this up, please leave a comment here, then discuss your findings / proposed changes before opening a PR.

@dreamorosi Interested, I will look into this & share my findings.

Hi @arnabrahman, thank you for your interest in this issue ✨!
Looking forward to review / collaborate on your proposal and findings in the issue.
If you have any question while you look into it, please let me know both here or on our Discord.

Hello @dreamorosi, sorry for replying late to this.
I think this particular issue was first mentioned here by @saragerion . After looking at both the unit tests for Logger/Tracer, my understanding of what she meant by the issue is, in the Logger/Tracer package all the tests are written for public functions rather than testing a particular feature of the package. But in Metrics, here we are following a pattern of testing a particular behavior/feature of Metrics rather than taking a single function & testing every case of that function.
So, if we want to follow a similar pattern/structure for Metrics similar to Logger/Tracer, we would take a function & test every case of that function. For example, for addMetric function, we want to test

the behavior when a new metric is added
the behavior when an old metric is added
the behavior when adding a single metric (for single metric, it will publish StoredMetrics)

We can take this approach & apply it to every public function of Metrics.
These are my findings. I might be wrong about this, so let me know if i am in the right direction or not. Thanks 🙂

Hi @arnabrahman thank you for taking the time to look into this!
Your findings are spot on, and this is indeed the direction that I think we should be going with this issue.
On top of the points you discussed above, as part of this issue I think we should:

Maintain 100% coverage
Adapt the style and convention used in the Logger/Tracer unit tests (i.e. comments, spacing, etc.)
Whenever needed/possible, extract common setup / utilities in helper functions

If you're still interested in working on this issue, feel free to start working on a PR. I'll be happy to continue the conversation there.

@dreamorosi Yes, i am very much interested to work on this & contribute. I will start a PR soon & will also follow your suggestions.

Great, I have assigned the issue to you and change its status to "Working on it".
If you have any question during the implementation don't hesitate to reach out here or on Discord!

Having the opportunity to introduce a new functionality for high resolution metrics and written tests before extending the Metrics implementation i had the following observations described in issue #1373 .
The observations relative to  tests are the Points 3,4,6
Note in observation 4 ,
as contributor i wish i had tighter feedback loop when extending the metrics implementation.
In my observations i described that a solution consideration is that the Metrics tests which exercise the public methods to be of an  MetricsInterface abstraction and not of the Metrics concrete implementation.

Yes, currently i am working on this. I have already made progress and so far covered 100% test coverage as you have mentioned. Now, i need to work on refactoring some things, then i will be finished. Sorry, it's taking a bit time. And, yes i will let you know if i face any problem. Thanks @dreamorosi

Hey thank you for getting back to me so quickly, there's no rush at all - I just wanted to check!
Thank you!

Hey thank you for getting back to me so quickly, there's no rush at all - I just wanted to check!
FYI: I'm about to open a PR that adds 4 test cases related to a new feature on Metrics, hope it doesn't cause too much mess on your branch!
Thank you!

Ok, I have a question. After I rewrite all the tests for Metrics, what will happen to the old tests? Should I keep both old tests & new tests or should I only keep the new ones?

If coverage of the new tests is 100% and the structure/style/etc. is in line with the other modules then we could replace the existing ones.
I'll be able to give you a concrete answer once you open a PR though.

Having the opportunity to introduce a new functionality for high resolution metrics and written tests before extending the Metrics implementation i had the following observations described in issue #1373 . The observations relative to tests are the Points 3,4,6
Note in observation 4 , as contributor i wish i had tighter feedback loop when extending the metrics implementation. In my observations i described that a solution consideration is that the Metrics tests which exercise the public methods to be of an MetricsInterface abstraction and not of the Metrics concrete implementation.

I'm about to merge the linked PR as the scope of this issue (as defined here) was to standardize the tests according to what is being done in the other utilities of the project.
I will provide a longer answer to all your points in the other original issue.


SCB-317 Update the cors license

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
 A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.05%) to 73.937% when pulling<PHONE_NUMBER>3be6a758c1ecf19539398c96847111 on asifdxtreme:cors into e9a498a5475549fe8a2b4c60d4e005b1bf0c3b00 on apache:master.

Codecov Report

Merging #273 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #273   +/-   ##
=======================================
  Coverage   70.33%   70.33%
=======================================
  Files          17       17
  Lines        3418     3418
=======================================
  Hits         2404     2404
  Misses        846      846
  Partials      168      168

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 e9a498a...2308805. Read the comment docs.


javascript selected words highlight bug ?

VSCode Version: 1.29
OS Version: windows 10

Hi to all,
in javascript and typescript files we have strange bug with incorrect highlighting selected and matched words. After selection we can see right color about 1 second, after this highlight colors will change to default. Issue doesn't reproduce with other file extensions, like c++, json, etc.
Steps to Reproduce:

set next options:
"editor.selectionHighlightBorder": "#ff0000",
"editorOverviewRuler.selectionHighlightForeground":"#ff0000",
reload vs code to apply these options
open javascript or typescript file and select any word

Does this issue occur when all extensions are disabled?: Yes

@rebornix Can you please take a look or assign to the right owner. I don't think this is a javascript specific issue. Two problems here:

I think selection highlight in the overview ruler is a language has symbol occurrences highlighting (like js does). Not 100% sure about this however

For the missing borders, the borders are there but they are under the symbol match highlighting background. This results in them seeming very faint.

It's an known issue with our word/symbol/selection highlight.


[BUG] Wrong Vault Parts in Detail Screen

Describe the bug
The m description of the part on windows is displayed wrongly
Expected behavior
No Part 2500001 of 3 display
Screenshots

Version 1.0.4

The m numbers are completely different and random on every vault


Wallet redirects to testnet when Microstacks react client with mocknet (devnet) is used

Report 1
A user testing the protocol on the local Stacks cluster provided by Clarinet: We're using the Microstacks React client with "mocknet" (devnet) as the network which should point all contract calls to the local network, however, the Hiro Wallet continues to use testnet. This is currently blocking us from performing local testing using the Hiro Wallet. Thomas from Fungible Systems pointed that this is indeed a known issue with the Hiro Wallet, but we're not sure if that has been reported or being worked on. Would appreciate any help the wallet team could provide to unblock local testing.
Additional report

Cause

Possible solutions

kyranjamie — Today at 5:50 PM
I've reverted a change which we think might've been responsible. Would you be able to download the wallet build from this PR and let us know if it's fixed? https://github.com/hirosystems/stacks-wallet-web/pull/3045

@hstove can you confirm this is resolved for you in the latest wallet build?


Added installation instruction

Improved documentation by adding an installation instruction. Not every trader knows how to install the dependencies.

Thanks a lot for this patch, will merge this.
best regards,
Florian La Roche


Add battery voltage sensor and connectivity binary_sensor, move battery_state to binary_sensor

Add battery voltage sensor and connectivity binary sensor, move battery state to binary sensor

Added sensor for battery voltage.
Moved battery_state to a binary_sensor.
Added connectivity binary_sensor.

@carterbox I resolved your comments, and added SSID and RSSI as optional sensors that can be enabled.

Hey, sorry I did not do a PR for my commit, I was not thinking. 😬
@carterbox how do you do an autorelease with your action? Do you just tag it and it creates a release?

@carterbox how do you do an autorelease with your action? Do you just tag it and it creates a release?

I'm not sure what you mean. Are you talking about https://github.com/tache/homeassistant-kidde/blob/main/.github/workflows/release.yml ?


Is the CSV results file from the corrected IEEE spreadsheet or the original?

Hello @LiaungYip
Very nice work here.
Are the CSV results you posted from the "corrected" spreadsheet or the original spreadsheet with the wrong formulas?
Could you upload both if you have them still?
arcflash/ieee_1584/additional_test_cases/ieee_1584_spreadsheet_results.csv
Thank you,
JGrimard

This will be the results from the corrected spreadsheet.
The reason I calculated the spreadsheet results was to check the Python
code using a different calculation implementation. Only correct results
were useful for that purpose.
I didn't retain the results from the erroneous spreadsheet.
On Fri, Jul 21, 2023, 22:18 jgrimard @.***> wrote:

Hello @LiaungYip https://github.com/LiaungYip
Very nice work here.
Are the results you posted from the "corrected" spreadsheet or the
original spreadsheet with the wrong formulas?
Could you upload both if you have them still?
arcflash/ieee_1584/additional_test_cases/ieee_1584_spreadsheet_results.csv
https://github.com/LiaungYip/arcflash/blob/main/arcflash/ieee_1584/additional_test_cases/ieee_1584_spreadsheet_results.csv
Thank you,
JGrimard
—
Reply to this email directly, view it on GitHub
https://github.com/LiaungYip/arcflash/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQSV5HZVI6NCT4MMLAD6NLXRKFUPANCNFSM6AAAAAA2S47X5E
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you for the fast response.  I don't actually need the erroneous results.  I just wanted to use the correct results to run some tests on my Javascript implementation of the IEEE calculations that I'm working on but I wasn't sure if the results were from the corrected version or not.  Now I know they are correct.
Have a great day.

If you are testing a JavaScript project you can generate many more test
cases - I had to scale back the number of tests I did because Excel is very
slow. There should be some test case generation code in this repository.
On Fri, Jul 21, 2023, 22:36 jgrimard @.***> wrote:

Closed #3 https://github.com/LiaungYip/arcflash/issues/3 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/LiaungYip/arcflash/issues/3#event-9886067301, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQSV5CSZSRPRM6A66DY5XLXRKHWPANCNFSM6AAAAAA2S47X5E
.
You are receiving this because you were mentioned.Message ID:
@.***>

I am going to use the CSV file to validate that my JavaScript implementation is producing the correct results.  It is perfect for what I need.  144,000 test cases is much better than the two test cases provided in the IEEE calculations.


Add changelog for the v1.4.0 release

What this PR does / why we need it:
Add the changelog for the v1.4.0 release.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Does this PR introduce a user-facing change?:
NONE

/assign @kron4eg
cc @Mita-Kubermatic @shaase-ctrl

/hold

/hold cancel


Consider Moving PATs to Azure Key Vault

See example: https://github.com/microsoft/ebpf-for-windows/blob/c4b7fe423402fa66ff243ec02a451c354ca4be83/.github/workflows/upload-perf-results.yml#L80C1-L87C1
This would eliminate the need to store them with GitHub.

See https://github.com/microsoft/netperf/issues/209


State machine

The H2 state machine is wrong.
In particular, it tries to handle CONTINUATION frames as part of the state machine. That just won't work, in part because it makes handling trailers essentially impossible. For that reason, I think we can remove the CONTINUATION handling and refactor to consider HEADERS + CONTINUATION a single jumbo frame.

Resolved in d85b47d9a327956254cba526c564e86abb76d382


[feature request] option to show only sites with unread feeds

just a quick suggestion,
would it be possible to have an option to get newsboat to show only sites with unread feeds ?
i mean, at this moment i have so many subscriptions to various sites that i have to scroll a few pages up and down just to check if there is something new. if newsboat could have such an option, only sites that have unread feeds would be displayed and users wouldn't have to continuously scroll up and down all the time just to check for updates. so yes, either display only sites with unread feeds, or at least place them at the top of the list. if this is made as an option folks can choose wether to use it or not depending if they prefer current behaviour, or the one i'm describing here. would be great to also have a hotkey to toggle it on and off at will.

the feature request i suggested already exists. in config do:
show-read-feeds no
...
bind-key U toggle-show-read-feeds
bind-key u show-urls

Sorry I didn't reply in time. Glad you resolve it yourself! :+1:


Math magicians: App routing

It was great implementing the following features:

The webpage consists of 3 pages: Home, Calculator, and Quote.
Layout matches the wireframe.
Added some styles.
Used a route for every page.
A nav with the links to all three pages.

@Strangeal Thank you for a great review


fix: Include wallet-sdk-gomobile in code coverage, fix DID creator bugs

Signed-off-by: Derek Trider<EMAIL_ADDRESS>closes #47

Codecov Report
Base: 90.27% // Head: 85.27% // Decreases project coverage by -5.00% :warning:

Coverage data is based on head (e677f63) compared to base (e6bad1f).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
- Coverage   90.27%   85.27%   -5.01%
==========================================
  Files           4        8       +4
  Lines          72      129      +57
==========================================
+ Hits           65      110      +45
- Misses          5       12       +7
- Partials        2        7       +5

Impacted Files
Coverage Δ

cmd/wallet-sdk-gomobile/didcreator/didcreator.go
85.71% <100.00%> (ø)

cmd/wallet-sdk-gomobile/localkms/localkms.go
60.00% <0.00%> (ø)

cmd/wallet-sdk-gomobile/didcreator/keywrapper.go
71.42% <0.00%> (ø)

cmd/wallet-sdk-gomobile/didresolver/resolver.go
100.00% <0.00%> (ø)

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

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


Adds Advertisers.all [#104683988]

http://pivotaltracker.com/story/show/104683988

:ship:

Hey guys, is this working on version 0.1.1?

@Mau3189 - nope, it is not: https://github.com/G5/yahoo_gemini_client/blob/master/CHANGELOG.md


Build performance in VSCode is slow on RDS

I have 2 environment with the same repo.  On both, I run the build-process (CTRL+SHIFT+B).  Version of VSCode and the AL Extension is the same.  Performance of the build is "slightly" different:

my laptop - 20 seconds
our RDS environment - 20 minutes
This while the resources on the RDS are better.. .

This is what we see:

the "Microsoft.Dynamics.Nav.EditorrServices.Host" eats up all the CPU - of all cores:

When I kill the process, it goes back to about 5% utilization (on my laptop, I have a constant 30% utilization because of Hyper-V running)

Funny thing, if we run this on the same RDS-server (compile without VSCode), then there is no problem at all.. :
alc.exe /packagecachepath:"$($ProjectCachePath)" /project:"$($ProjectPath)" /out:"$($AppFilePath)" /analyzer:"$($ALCompilerCopPath)" /fullpaths
Conclusion is that there is something seriously wrong in the VSCode-part of the compiler...

We just noticed that the majority of the performance loss is not due to the NAV-Codeanalysis, but it seems it comes from the .Net-CodeAnalysis:

So what we will do this evening, is we will update our .Net Framework on our server and tomorrow, I'll come back to you...

Well, same version of VSCode, x64, same AL extension ...

Solved!
A windows update fixed it.  For completeness - this is the one that fixed the issue:

Ok, I wander how did you get that picture though? Since the current codebase does not call NormalizeWhiteSpace anymore?

That pic was indeed from the older version.  As my colleague took that screenshot, while I was testing the new version on my VSCode on that server... .


Update generated OSM descriptors

Adjust/update generated OSM descriptors
Comparing newly created, compatible descriptors with previously generated descriptors:
NSD:

[x] Add vim-network-name: default and type: ELAN for all vlds
[x] Change default-vnf to vnf

VNFD:

[x] No input interface and connection point for first VNF
[x] No output interface and connection point for last VNF
[x] Change default-vnf to vnf

Changing member-vnf-index-ref to string or choosing unique connection point or interface names is not necessary.


[WIP] Restructuring tests: an example of splitting basic tests into integration and e2e

Idea
This PR introduces initial changes so we have a concrete example when we talk about restructuring the CRC tests.
By adhering to Cucumber's design principles (thus not needing BeforeFeature hooks), we can upgrade to godog v0.11.0 and get support for jUnit file export. This means that we need to shift our more detailed (integration) tests away from Cucumber. Ginkgo was suggested as a possible candidate.
So this is a preview of splitting the tests into integration and e2e on the example of basic tests. The integration part is in Ginkgo and the e2e part is in Cucumber.
I reused a lot of Guillaume's Ginkgo set-up code from Tea Time discussion.
Still missing

Test on all 3 platforms (only Linux has been tested)
Test with standalone bundle (only release bits were tested)
Various combinations of small commands in Ginkgo tests. See this map. The config, ip, console, oc-env, podman-env subtrees are not tested yet. Many flag combinations are also missing as of now.
Need to designate a focus subset of basic tests to run as an "important" subset when not everything needs to be tested.
Did not update dependency to godog v0.11.0, hence no jUnit output for e2e tests yet.

Warning

Renamed PULL_SECRET_FILE env var to PULL_SECRET_LOCATION to be analogous to BUNDLE_LOCATION var. This Makefile change will affect pipelines.

Proposed changes

Move the integration component of the basic tests to Ginkgo framework in test/ginkgo
Add disk/mem/cpu resize integration tests in Ginkgo.
Condense the Cucumber e2e tests component to adhere to Cucumber design principles. E.g. independent Scenarios.
Produce jUnit files with test results to upload to ReportPortal (or elsewhere).

Testing

Set BUNDLE_LOCATION(if using standalone bundle) and PULL_SECRET_LOCATION variables.
Go to test/ginkgo and trigger the integration tests by typing ginkgo. Check jUnit output in test/ginkgo/results.
To run e2e basic test, modify makefile to only run <os> && basic tags and type make integration from repo dir.

Good start! Let's make the CI green so that we can all iterate on this

I rebased the PR and make it work on CentOS CI but it's very red: https://ci.centos.org/job/codeready-crc-pr/2685/console

Will resubmit in smaller chunks.

Will resubmit in smaller chunks.


DND: visible watermark prevents dropping of files (fixes #15698)

@chrmarti this change puts the watermark into the content element of the editor part and hides it when drop feedback is indicated.
I also removed the has-watermark class because we no longer seem to use it and this change makes a reload with empty editor area smoother because the VS logo is not moving up after the workbench has been loaded.

Thanks @bpasero for looking into this. I wonder if we really want to hide the commands, would making DND work on them make more sense?

@chrmarti I tried to bubble up the DND event from the watermark to the parent container but could not get it to work, if you have a better idea please do it 👍

Would counting the number of drag_enter and drag_leave on the container help in tracking the current state? (See, e.g., https://coderwall.com/p/bwsara/taming-html5-drag-and-drop)

Found a fix that can keep the watermark visible.

Thanks @bpasero !


Installing procedures

Hello guys, @tt-rkim @TT-billteng @blozano-tt
I am working with @zzigler-tt to review our installing docs and procedures and something I noticed is that from the user perspective there are a lot of manual steps to get the software setup.
Is there any front working to develop scripts for making these steps seamless? Something that would take care of installing driver, firmware and any other required applications without depending on the user to navigate between all the different sources to install these individually?
It seems like many of these dependencies are common across TT devices so having them installed as a pack could be beneficial for the user experience. What are your thoughts on that?

I got an update that sysEng is working on this for their applications.
So how is packaging looking like on our end? Are there any upcoming installation steps changes we should take into consideration for our installing doc?

@afuller-TT is doing some key cleanup work and is working on a lot of our packaging story to help make installing metal itself more seamless. This includes more predictable and idiomatic versioning schemes and selecting the distros we support + channels through which we distribute.
Note - as I've mentioned before - there are two use cases we should be optimizing for, which @adifrancescoTT has pointed out in his proposal document:

For hackers/developers of tt-metal who want to be close to the source (are ok with more steps which is necessary for dev)
Plain users / downstream dependencies who don't care (should be simple)

I would consider 1) a done deal.
For 2), this is on our roadmap. I'll lean on my team for this @bkeith-TT I think we were looking at a first half 2025 for being in a good place for things like a debian and 22.04 support for faster installs?


v2.9 Skybell

I saw in the release notes that Skybell was supported for doorbells but with that version and the latest version I am not showing any new devices other than what was already pulled in my case just the garage door and door lock.  I am running the latest version of HA also.

Sorry I have the slimline version I didn't see HD my bad.

All good. I don't think that I'll be able to support an entirely new device right now, but if you're willing to do some light testing I can see if the Slimline can use the code I wrote for the HD. If the config pages are the same it should be an easy change. How does that sound?

I would be willing but fair warning I am in the process of migrating to HA from Hubitat so I don’t really know much but I would be willing to try. Sent from Mail for Windows From: Elahd Bar-ShaiSent: Friday, June 3, 2022 9:37 AMTo: uvjustin/alarmdotcomCc: jriffe6; State changeSubject: Re: [uvjustin/alarmdotcom] v2.9 Skybell (Issue #133) All good. I don't think that I'll be able to support an entirely new device right now, but if you're willing to do some light testing I can see if the Slimline can use the code I wrote for the HD. If the config pages are the same it should be an easy change. How does that sound?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID<EMAIL_ADDRESS>Virus-free. www.avg.com


Acquire API token

Authentication flow seems to work fine. But after authentication many frontends will need to acquire an authorization token to access a specific API. This seems to be an important missing feature, or did I miss something?

I don't follow. The getUser returns the user and it has an access_token property.

Yes it does, but in our case the this token does not contain the claims we need. Therefore we need to acquire an API token to access a specific resource server. You could have many resource servers running many API's all having their own scope.

This library isn't designed to be "multi-tenant" (for lack of a better word). But you could create multiple instances of the UserManager for each, and that might solve your problem.


How to do group or one to one video call in iOS swift?

Hi
I want to implement a voice & video call. Voice call is working fine but in the same way, I want to implement a video call too.
How to do group/single video call like 'whats app' with call kit notification.

Hi @ashishpatelmi,
Sorry for the slow response to your question.

Voice call is working fine but in the same way, I want to implement a video call too.

Was the voice call using the Twilio Voice SDK? At the moment Twilio Video doesn't integrate push notifications. However, you can send VoIP push notifications to Apple yourself and then invoke the CallKit APIs upon receipt of a push.
We have an example which uses the UserNotifications framework to send local notifications and then invoke CallKit. You would need to replace the UserNotifications component with your own push notifications.
Regards,
Chris


Update imaskjs dependency

Describe the feature:
imaskjs is at version 7.1.3 and the version dependency on forge is 6.6.1.  This is preventing me from using the latest imaskjs in my project.

@RobJacobs We'll definitely update the dependency version at some point, but I was curious what is preventing you from updating? Is it just a case of not wanting to include multiple versions of imask in your app?
I ask because if you install imask in your own project, npm should allow for you to use the latest version in your app while Forge will use its own version (unless the versions are compatible via semver). Of course this means you may end up with two versions of imask in the same app, so I was just curious if that was the concern or if there is a functional problem?


Error: config::parse::failed Failed to parse ProtoConfig. plugins.moon: Unknown plugin scope https

What version?
0.37.0
Which command?
moon run
What happened?
This occurred after updating proto to 0.37.0
The command I'm running is moon but it looks like it is caused when proto is parsing the config.  It is reporting https as an unknown plugin scope.  I had to prepend  source: to all of the plugins in .prototools as a workaround.  I expected it to identify https and support it or give better feedback on how to fix the issue.

Trace logs?
Error: config::parse::failed
× Failed to parse ProtoConfig.
╰─▶   × plugins.act: Unknown plugin scope https.
╭─[2:7]
1 │ [plugins]
2 │ act = "https://raw.githubusercontent.com/theomessin/proto-toml-plugins/master/act.toml"
·       ────────────────────────────────────────┬────────────────────────────────────────
·                                               ╰── Fix this
3 │ moon = "https://raw.githubusercontent.com/moonrepo/moon/master/proto-plugin.toml"
╰────
Operating system?
MacOS
Architecture?
arm64

@dngrhm What moon version are you using? moon is coupled to a specific proto version behind the scenes, so I think you need to be using moon >= 1.25.2, or stay on an older proto version.

Thanks! Sry I missed your earlier comment.  Yes, that appears to be the case with the version mismatch.


Feature request - Add an option not to jump to prev/next instance

I often want to search for the word under the cursor without losing my cursor position. I suspect I'm not the only person who does this (http://stackoverflow.com/questions/4256697/vim-search-and-highlight-but-do-not-jump).
Because of the way vim-searchindex maps *, I can't seem to map this in my vimrc. It would be nice if that option were supported within the plugin.

What you are trying to achieve can already by done through the mappings.
First, to override the plugin mappings, you can use autocmd VimEnter *. Second, you can use mappings from the linked thread, just add <Plug>SearchIndex at the end to display the counts.
Putting these together:
autocmd VimEnter * call <SID>OverridePluginSettings()
function! s:OverridePluginSettings()
  nmap <silent> * :let @/='\<'.expand("<cword>").'\>'<CR>:set hls<CR><Plug>SearchIndex
  nmap <silent> # :let @/='\<'.expand("<cword>").'\>'<CR>:let v:searchforward=0<CR>:set hls<CR><Plug>SearchIndex
endfunction

Let me know how that works for you. I know having a dedicated option would be more convenient, but I'm trying to prevent having too many of them.

That snippet you put there works exactly as you said it would - thank you!

The normal behavior of * and # adds the word under the cursor to your search history. With the above commands, * and # don't add the word under the cursor to your search history. Any suggested workarounds?

You could extend the snippet to call histadd() function, thus updating history directly: http://vimdoc.sourceforge.net/htmldoc/eval.html#histadd()


feat: report multiple error messages from rules in TAP format

MailOnline/tap-github-issues#3

Coverage increased (+0.02%) to 96.97% when pulling b70d66b33547995694d9b6c4ab6269b48d857276 on issue-comments into 8585a0c8de092a390247d601422cda6c53db6abd on master.


Save (list of) Transaction Hashes longer

Hello,
Thanks for making such a great tool.
After sending a transaction a pop up appears briefly with a message with a transaction hash at the bottom of the browser window.
It would be nice if this information didn't disappear so fast. You have to be very quick to copy and paste it somewhere else if you want to keep it. It can easily happen that the transaction doesn't happen to show up any time soon in your account (public key) in etherscan.io and this transaction hash is the only thing you have to confirm that it should at least be pending soon.
Ideally the sent transaction hashes should be saved in a list somewhere at least until the browser tab is closed, but at the very least several times longer than the popup is currently shown.
Thanks in Advance.

@dec1 Is this still an issue?

@dec1 with version 3.21.07, it looks like the pop up doesn't close unless you click on the x at the bottom right of the screen. I'm gonna go ahead and close this issue for now. If you have any question or you want to reopen the issue, don't hesitate to reach out.

This is not fixed. The popup disappears on its own (in a second or two) - at least if you paste a signed (offline ) transaction and send it


Ccc

FluidConcat, FluidETC, FluidShannonEntropy, , FluidShannonEntropyKR

Is the complement of this: https://github.com/chriskiefer/flucoma-core/blob/CCC/include/clients/rt/ETCClient.hpp?
These look cool – I will try to compile the core algorithms in Max :)

Is the complement of this: https://github.com/chriskiefer/flucoma-core/blob/CCC/include/clients/rt/ETCClient.hpp?
These look cool – I will try to compile the core algorithms in Max :)

that's right  - it's ETCClient, ShannonEntropyClient, ConcatClient


Unthemable product icons #2

Does this issue occur when all extensions are disabled?: Yes (all other extensions disabled)

Version: 1.89.0
Commit: b58957e67ee1e712cebf466b995adf4c5307b2bd
Date: 2024-05-01T02:09:22.859Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Windows_NT x64 10.0.22621

This is a continuation of https://github.com/microsoft/vscode/issues/208343. I'm in the process of creating a product icon theme and noticed the following icons not being (properly) replaced. They either appear invisible or are unthemed. There may be more icons that suffer from the same problems described below, but I have not been able to find them as of writing this.
Settings icon, used for the settings editor tab
The icon is invisible after applying the theme.
This is due to the following CSS rule overriding the font-family attribute:

Search icon, used for the search editor tab
Same as settings editor tab icon, it's invisible for the same CSS rule overriding the font-family attribute.
More/three dots (codicon-toolbar-more)
Visible in, for example, output and terminal panel views (left next to close icon), editor tabs more actions (where the 'split editor' action icons is), etc.
This doesn't appear to get any overrides at all, so it remains unthemed.
Setting editor item chevron-down dropdown icon
The icon is invisible after applying the theme.
This is due to the a second font-family attribute always being present and overriding the first attribute that correctly retrieves which value font-family should have.

To verify:

extends the product icon theme smaple
with

		"settings-editor-label-icon": {
			"fontCharacter": "\\e000"
		},
		"toolbar-more": {
			"fontCharacter": "\\e000"
		},
		"more": {
			"fontCharacter": "\\e000"
		},
		"search-editor-label-icon": {
			"fontCharacter": "\\e000"
		},

export it as VSIX, import select the VSCode 1.0 Icons theme
verify that the settings editor has now a explorer icon, as well the more icons


Format json file content when download response

Description
When we download a file trough the dowload button at response section, the file is with no formatting. With this contribution, it will be able to get formatted content. Closes issue-1708
How was:

How is now:

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

One of the issues with automatically stringifying JSON responses with indentation is that - it might alter the response text in case of decimals as well lose fidelity of the response contains longint / bigint
A better solution for this is to show the user a modal when downloading json responses - giving them an option to prettify the response before saving to file.

Well, at least, Bruno could mantain the issue open for the future


closes #771 warn on unpaired deletion/insertion

Sorry, I forgot about something else I had wanted implemented. I think I said it in a meeting but didn't write it down.
Let's make a new menu View-->Warnings, and move the following there:

Show DNA base mismatches
Show unpaired insertions/deletions
Show domain name mismatches

Okay, I implemented the View menu re-arrangements in the dev branch.
One thing that's odd is that the warnings stars come in pairs for insertions but not deletions:

Can we change the insertions to be like deletions, where it only shows the warning on the strand lacking the insertion?

We have the warning show on insertions that are of different lengths, so that's why it appears on both strands. Should the warning appear only on one of them in the case of two insertions with different lengths?

We have the warning show on insertions that are of different lengths, so that's why it appears on both strands. Should the warning appear only on one of them in the case of two insertions with different lengths?

Ah, I see. Keep the warning on both if they are different lengths.
But is it possible to slide it out a bit in this case, so it's more visible? It shows up underneath the insertion so is hard to see.

But is it possible to slide it out a bit in this case, so it's more visible?

Ok, should it still be slid out if there's no insertion?

No it's only necessary to keep it from being hidden by the insertion.


[fix-uri] remove '/' from resources uri

When generate the URI for an hydra resource, not add '/' in the beginning of the uri. Guzzle automatically add it, and if you do so he can change the base url.

LGTM

LGTM


SCRD-6374 Remove translation for logging message

The commit includes:

Removed translation of the logging messages in replace server
process
Updated logMessage to append new line '\n' or no new line
based on passed in  params (SCRD-6375)
Removed unused messages.
Allowed error message from action run to show up in the page banner

Ethan has a good idea to allow translation only to English. Will try his idea instead.


other lightning implementations

As far as I see the raspiblitz currently only runs with lnd. is there any reason (other then manpower) to not include c-lightning and have the user prompted which implementation they want to run? If there is no reason I might take the lead to also include c-lightning.

Having both to choose/switch between LND and c-lightning would be great ... related to manpower we just ran with LND in the beginning.
@renepickhardt would be great if you can take the lead on c-lightning integration. We maybe have to see if it makes sense to restructure the shell-scripts a bit for that, so that we dont get too much into a if/then hell. Maybe first step could be to put all LND related shell script fragments into a kind of Lightning-Service Shell-Library: https://bash.cyberciti.biz/guide/Shell_functions_library .. and then in a second step to make this Library dynamically work with LND or c-Lightning were possible. Do you think that makes sense?

I think it does. I realized most of the stuff is shell script which is not necessarily my favorite language. so if you are strong on shell script I would rather have you to restructure the scripts and basically create an abstraction layer for interaction with the lightning node. once that is done I could easily do all the c-lightning related stuff. or we just do it together before , while or after the lightning hackday in berlin (:

I think thats a good task for the hackday :) ... lets go thru the scripts together and see it the steps that are in there for LND make also sense for c-lightning or if it may need even more rearrangement, than just an abstraction layer.

bsss I am thinking all the time that it might by cool to buy a raspiblitz. now I get even more tempted...

Some guide to look into: https://medium.com/@meeDamian/c-lightning-node-on-rbp3-b950660fb835

https://twitter.com/KHS9NE/status/1042837790822727680

@renepickhardt

If there is no reason I might take the lead to also include c-lightning.

Hi Mr Pickhardt.

C-lightning is implemented. Closing this issue for inactivity.

I remember fiatjaf posting about wen eclair on the last blitz release, but if noone will do it, then ok.

Sure and there is the Rust LDK too.
If someone starts to work on these will worth it's own issue. Even it's own project tbh (fork of this one or not).


are there checksums (for releases) ?

Hello,
github runner images team here.
we are looking for securing supply chains when adding software to CI images.
are there checksum available ? or maybe some recommended validation approach.
thanks!

Could you give a bit more info on what you want to checksum? Is it the codeql-action itself? The CodeQL CLI? Both?

we download codeql bundles using https://github.com/actions/runner-images/blob/main/images/win/scripts/Installers/Install-CodeQLBundle.ps1

@ilia-shipitsin since we have multiple artifacts (one artifact for each of the 3 major OSes, and one "universal" artifact) do you have a preference between the following options?

a single checksums file containing checksums and filenames for each of the 4 files, on separate lines (this is what the gh CLI does for example)
one checksum file for each artifact

The standard unix model is one signature per file, since it enables wget $url{,.sig}.


Insert Markers in Trajectory

It'll be useful if we can add markers to waypoints, that will  also stay on the generated trajectory as well.
The reason for that, so it'll be easier to combine subsystems during the path. Currently the way to do start a subsystem in a specific place during auto is by time or by coordinate. It's much more prune to changing/tuning that being made in the trajectory during the season.

I would probably number the markers, so if there are a few, you can easily distinguish between them.

I saw someone asking the same on CD, so I'll suggest a simple way to inplement:

GUI: add "marker" chekbox to waypoint.
Trajectory class:

On trajectory generation: create array of times (for every waypoint with a marker, insert it's time in trajectory).
On sampling: Marker's numeric value will be the number of items in the marker array, that the current time is greater than them.


Show size of tumblr blog before downloading

I want to be able to see the size of the tumblr blog I am downloading before I download it so I know if I have enough space to download the blog.

Would be nice, but I'm not sure if that is even possible..
What could be done instead, is to show simply the total postcount. Given with the post type, a rough approximation could be possible.

That is a good point that it might not be possible, so that is a good alternative to my suggestion!

The size in bytes cannot be shown, as that information isn't available.
For normal blogs the number of posts is shown in the details pane and in the blog list column "number of downloadable items" when adding the blog. More detailed numbers based on post type are only available after a crawl process has finished (or numbers up until then if interrupted). With these numbers you can do a rough estimation.
For blogs of type "hidden" the number of posts cannot be shown when adding the blog.