Refs #29877 -- Replaced http://www.diveinto.org/ with secure URL.

Wikipedia has it as http://...
Are we definitely all s now, despite no caching? If so, this seems reasonable.

Wikipedia has it as http://...

Fixed :smile:

Are we definitely all s now, despite no caching? If so, this seems reasonable.

I think we should be, yes. IMO, all external references should prefer a secure link if it exists. It provides more privacy to doc users and avoids an unnecessary redirect. I know I prefer my traffic be encrypted. It helps ensure I'm receiving an unaltered document from the requested source.

Given one domain has expired I'm also a bit worried we'd just be inviting certificate errors...

All external references have some risk of going stale. This is true whether they use http:// or https://. If the external reference is authoritative or useful, that is a risk we'll need to accept. Though, I do agree that certificate expiration adds another element to this risk. I think if we can trust these references enough to link to them, we should trust them to maintain their domain and certificate. If the references can't be held to that standard, we should be cautious linking to them.

Yes. I’m half inclined to think dropping these links is the way to go, given that they’re essentially unmaintained.
👍🏽 From me.


New web project throws an error using CanvasKit only

Steps to Reproduce

Start a new project on Android Studio and debug on Chrome (web)
Expected results:
App should work fine
Actual results:
Getting this error inside Android Studio:
Error: Unexpected null value.
    at Object.throw_ [as throw] (http://localhost:56433/dart_sdk.js:5342:11)
    at Object.nullCheck (http://localhost:56433/dart_sdk.js:5675:30)
    at _engine._PointerAdapter.new.[_getSanitizer] (http://localhost:56433/dart_sdk.js:165972:28)
    at http://localhost:56433/dart_sdk.js:166031:42
    at http://localhost:56433/dart_sdk.js:165990:16
    at http://localhost:56433/dart_sdk.js:165689:11
    at <anonymous>
    at http://localhost:56433/dart_sdk.js:50591:13
    at Generator.next (<anonymous>)
    at runBody (http://localhost:56433/dart_sdk.js:39177:34)
    at Object._async [as async] (http://localhost:56433/dart_sdk.js:39208:7)
    at http://localhost:56433/dart_sdk.js:50582:20
    at new Promise (<anonymous>)
    at Object._invokeExtension [as invokeExtension] (http://localhost:56433/dart_sdk.js:50579:12)
    at <anonymous>:1:31

And this one on Google Chrome's console
errors.dart:187 Uncaught Error: Unexpected null value.
    at Object.throw_ [as throw] (errors.dart:236)
    at Object.nullCheck (operations.dart:518)
    at _engine._PointerAdapter.new.[_getSanitizer] (pointer_binding.dart:487)
    at pointer_binding.dart:557
    at pointer_binding.dart:505
    at pointer_binding.dart:214
    at <anonymous>
    at developer_patch.dart:88
    at Generator.next (<anonymous>)
    at runBody (async_patch.dart:84)
    at Object._async [as async] (async_patch.dart:123)
    at developer_patch.dart:80
    at new Promise (<anonymous>)
    at Object._invokeExtension [as invokeExtension] (developer_patch.dart:79)
    at <anonymous>:1:31

If I change the web renderer it works fine

Interestingly enough, if I run flutter run --verbose it also works fine
So I'm not sure if Android Studio runs something different?

Logs

[  +79 ms] executing: sysctl hw.optional.arm64
[  +14 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[  +12 ms] executing: [/Users/jan/flutter/] git -c log.showSignature=false log
-n 1 --pretty=format:%H
[  +18 ms] Exit code 0 from: git -c log.showSignature=false log -n 1
--pretty=format:%H
[        ] f8cd24de95b16b5a1ce6ebc0716154271fbf6252
[        ] executing: [/Users/jan/flutter/] git tag --points-at
f8cd24de95b16b5a1ce6ebc0716154271fbf6252
[  +21 ms] Exit code 0 from: git tag --points-at
f8cd24de95b16b5a1ce6ebc0716154271fbf6252
[        ] 1.27.0-4.0.pre
[  +42 ms] executing: [/Users/jan/flutter/] git rev-parse --abbrev-ref
--symbolic @{u}
[  +11 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/dev
[        ] executing: [/Users/jan/flutter/] git ls-remote --get-url origin
[  +12 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +48 ms] executing: [/Users/jan/flutter/] git rev-parse --abbrev-ref HEAD
[  +13 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] dev
[   +6 ms] executing: sw_vers -productName
[  +12 ms] Exit code 0 from: sw_vers -productName
[        ] macOS
[        ] executing: sw_vers -productVersion
[  +12 ms] Exit code 0 from: sw_vers -productVersion
[        ] 11.2.1
[        ] executing: sw_vers -buildVersion
[  +12 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 20D74
[  +58 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping
update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required,
skipping update.
[  +40 ms] executing: /Users/jan/Library/Android/sdk/platform-tools/adb devices
-l
[  +12 ms] executing: sysctl hw.optional.arm64
[   +5 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[        ] executing: xcrun xcodebuild -version
[ +100 ms] Exit code 0 from: xcrun xcodebuild -version
[   +1 ms] Xcode 12.4
           Build version 12D4e
[   +2 ms] executing: xcrun xcdevice list --timeout 2
[   +3 ms] xcrun simctl list --json devices
[        ] executing: xcrun simctl list --json devices
[  +39 ms] List of devices attached
[  +88 ms] {
             "devices" : {
               "com.apple.CoreSimulator.SimRuntime.iOS-14-4" : [
                 {
                   "dataPath" :
                   "\/Users\/jan\/Library\/Developer\/CoreSimulator\/Devices\/6A
                   C26487-617B-438C-B82E-89FE9A2CC071\/data",
                   "logPath" :
                   "\/Users\/jan\/Library\/Logs\/CoreSimulator\/6AC26487-617B-43
                   8C-B82E-89FE9A2CC071",
                   "udid" : "6AC26487-617B-438C-B82E-89FE9A2CC071",
                   "isAvailable" : true,
                   "deviceTypeIdentifier" :
                   "com.apple.CoreSimulator.SimDeviceType.iPhone-12",
                   "state" : "Shutdown",
                   "name" : "iPhone 12"
                 },
                 {
                   "dataPath" :
                   "\/Users\/jan\/Library\/Developer\/CoreSimulator\/Devices\/AF
                   BBE4FB-B202-406D-AE89-2D770B3D00A2\/data",
                   "logPath" :
                   "\/Users\/jan\/Library\/Logs\/CoreSimulator\/AFBBE4FB-B202-40
                   6D-AE89-2D770B3D00A2",
                   "udid" : "AFBBE4FB-B202-406D-AE89-2D770B3D00A2",
                   "isAvailable" : true,
                   "deviceTypeIdentifier" :
                   "com.apple.CoreSimulator.SimDeviceType.iPad-Pro--9-7-inch-",
                   "state" : "Shutdown",
                   "name" : "iPad Pro (9.7-inch)"
                 }
               ],
               "com.apple.CoreSimulator.SimRuntime.iOS-13-5" : [

               ],
               "com.apple.CoreSimulator.SimRuntime.watchOS-7-2" : [

               ],
               "com.apple.CoreSimulator.SimRuntime.tvOS-14-3" : [

               ]
             }
           }
[+2717 ms] [
                      {
                        "modelCode" : "iPhone9,2",
                        "simulator" : false,
                        "modelName" : "iPhone 7 Plus (Model 1661, 1784, 1785,
                        1786)",
                        "error" : {
                          "code" : -13,
                          "failureReason" : "",
                          "underlyingErrors" : [
                            {
                              "code" : 3,
                              "failureReason" : "",
                              "description" : "iPhone de Jan Lubeck is locked.",
                              "recoverySuggestion" : "To use iPhone de Jan
                              Lubeck with Xcode, unlock it.",
                              "domain" : "DVTDeviceIneligibilityErrorDomain"
                            }
                          ],
                          "description" : "iPhone de Jan Lubeck is not
                          connected",
                          "recoverySuggestion" : "Xcode will continue when
                          iPhone de Jan Lubeck is connected.",
                          "domain" : "com.apple.platform.iphoneos"
                        },
                        "operatingSystemVersion" : "14.0.1 (18A393)",
                        "identifier" :
                        "d167f1a8b5eed7cd0312f630a2da37135e6788f3",
                        "platform" : "com.apple.platform.iphoneos",
                        "architecture" : "arm64",
                        "interface" : "usb",
                        "available" : false,
                        "name" : "iPhone de Jan Lubeck",
                        "modelUTI" : "com.apple.iphone-7-plus-2"
                      },
                      {
                        "simulator" : true,
                        "operatingSystemVersion" : "14.4 (18D46)",
                        "available" : true,
                        "platform" : "com.apple.platform.iphonesimulator",
                        "modelCode" : "iPhone13,2",
                        "identifier" : "6AC26487-617B-438C-B82E-89FE9A2CC071",
                        "architecture" : "x86_64",
                        "modelUTI" : "com.apple.iphone-12-1",
                        "modelName" : "iPhone 12",
                        "name" : "iPhone 12"
                      },
                      {
                        "simulator" : true,
                        "operatingSystemVersion" : "14.4 (18D46)",
                        "available" : true,
                        "platform" : "com.apple.platform.iphonesimulator",
                        "modelCode" : "iPad6,4",
                        "identifier" : "AFBBE4FB-B202-406D-AE89-2D770B3D00A2",
                        "architecture" : "x86_64",
                        "modelUTI" : "com.apple.ipad-pro-9point7-a1674-b9b7ba",
                        "modelName" : "iPad Pro (9.7-inch)",
                        "name" : "iPad Pro (9.7-inch)"
                      }
                    ]
[   +2 ms] Error: iPhone de Jan Lubeck is not connected. Xcode will continue
when iPhone de Jan Lubeck is connected. (code -13)
[   +3 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping
update.
[   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required,
skipping update.
[   +2 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping
update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required,
skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required,
skipping update.
[  +70 ms] Multiple devices found:
[  +50 ms] macOS (desktop) • macos  • darwin-x64     • macOS 11.2.1 20D74
darwin-x64
[   +1 ms] Chrome (web)    • chrome • web-javascript • Google Chrome
88.0.4324.182
[   +1 ms] [1]: macOS (macos)
[        ] [2]: Chrome (chrome)
[   +2 ms] Please choose one (To quit, press "q/Q")
[        ] :
[+2792 ms] 2
[  +14 ms] Skipping pub get: version match.
[ +110 ms] Generating
/Users/jan/Projects/flutter/flutter_html_test/android/app/src/main/java/io/flutt
er/plugins/GeneratedPluginRegistrant.java
[  +84 ms] Launching lib/main.dart on Chrome in debug mode...
[ +111 ms] Updating assets
[  +59 ms] Waiting for connection from debug service on Chrome...
[  +54 ms] <- reset
[   +5 ms] /Users/jan/flutter/bin/cache/dart-sdk/bin/dart --disable-dart-dev
/Users/jan/flutter/bin/cache/artifacts/engine/darwin-x64/frontend_server.dart.sn
apshot --sdk-root /Users/jan/flutter/bin/cache/flutter_web_sdk/ --incremental
--target=dartdevc --debugger-module-names --experimental-emit-debug-metadata
-DFLUTTER_WEB_AUTO_DETECT=true --output-dill
/var/folders/6d/7fb8q3_576358cs38040dm8h0000gn/T/flutter_tools.O6TeKI/flutter_to
ol.6ccp15/app.dill --libraries-spec
file:///Users/jan/flutter/bin/cache/flutter_web_sdk/libraries.json --packages
/Users/jan/Projects/flutter/flutter_html_test/.dart_tool/package_config.json
-Ddart.vm.profile=false -Ddart.vm.product=false --enable-asserts
--track-widget-creation --filesystem-root
/var/folders/6d/7fb8q3_576358cs38040dm8h0000gn/T/flutter_tools.O6TeKI/flutter_to
ols.0kluNe --filesystem-scheme org-dartlang-app --initialize-from-dill
build/c7922d95bf4a2462b75c84a97c312edb.cache.dill.track.dill --platform
file:///Users/jan/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk.dill
--no-sound-null-safety
[  +11 ms] <- compile org-dartlang-app:/web_entrypoint.dart
[+13456 ms] Waiting for connection from debug service on Chrome... (completed in
13.5s)
[        ] Synced 29.2MB.
[        ] <- accept
[        ] Caching compiled dill
[  +62 ms] Using Google Chrome 88.0.4324.182

[ +377 ms] [CHROME]:2021-02-22 22:52:45.210 Google Chrome[15962:167418] Error
loading
/Library/CoreMediaIO/Plug-Ins/DAL/ImmersedCamera.plugin/Contents/MacOS/ImmersedC
amera:
dlopen(/Library/CoreMediaIO/Plug-Ins/DAL/ImmersedCamera.plugin/Contents/MacOS/Im
mersedCamera, 262): no suitable image found.  Did find:
[   +1 ms] [CHROME]:
/Library/CoreMediaIO/Plug-Ins/DAL/ImmersedCamera.plugin/Contents/MacOS/ImmersedC
amera: code signature in
(/Library/CoreMediaIO/Plug-Ins/DAL/ImmersedCamera.plugin/Contents/MacOS/Immersed
Camera) not valid for use in process using Library Validation: mapping process
and mapped file (non-platform) have different Team IDs
[        ] [CHROME]:
/Library/CoreMediaIO/Plug-Ins/DAL/ImmersedCamera.plugin/Contents/MacOS/ImmersedC
amera: stat() failed with errno=9
[ +118 ms] [CHROME]:
[        ] [CHROME]:DevTools listening on
ws://<IP_ADDRESS>:58633/devtools/browser/21bff92b-f9cc-456e-8e3a-745ba56ece10
[ +750 ms] DwdsInjector: Received request for entrypoint at
http://localhost:58620/main_module.bootstrap.js
[   +6 ms] MetadataProvider: Loading debug metadata...
[   +8 ms] MetadataProvider: Loaded debug metadata (no sound null safety)
[   +6 ms] DwdsInjector: Injected debugging metadata for entrypoint at
http://localhost:58620/main_module.bootstrap.js
[+1267 ms] ChromeProxyService: Initializing expression compiler for
main_module.bootstrap.js with sound null safety: false
[ +315 ms] DevHandler: Debug service listening on
ws://<IP_ADDRESS>:58652/de-Gpxl3WD4=/ws

[   +8 ms] Debug service listening on ws://<IP_ADDRESS>:58652/de-Gpxl3WD4=/ws
[        ] Running with unsound null safety
[        ] For more information see
https://dart.dev/null-safety/unsound-null-safety
[   +2 ms] 🔥  To hot restart changes while running, press "r" or "R".
[        ] For a more detailed help message, press "h". To quit, press "q".

[✓] Flutter (Channel dev, 1.27.0-4.0.pre, on macOS 11.2.1 20D74 darwin-x64,
    locale en-US)
    • Flutter version 1.27.0-4.0.pre at /Users/jan/flutter
    • Framework revision f8cd24de95 (6 days ago), 2021-02-16 11:24:17 -0800
    • Engine revision 1d537824d6
    • Dart version 2.13.0 (build 2.13.0-30.0.dev)

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

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.0

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.53.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.1 20D74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 88.0.4324.182

Hi @jlubeck
Can you please provide a minimal complete reproducible code sample?
Thank you

@pedromassangocode it happens with the default template when starting a new project on Android Studio:
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

I just had this same issue yesterday (https://github.com/flutter/flutter/issues/76690). @yjbanov can you please confirm that https://github.com/flutter/flutter/issues/76680 is the same as this one and it was fixed by https://github.com/flutter/flutter-intellij/pull/5286?
Thank you

Hi @jlubeck
Based on my message above, this seems to be currently fixed on master.
Can you please confirm?
Thank you

Yup! Fixed on my side! Thanks


dotenvx ext scan produces gitleaks detect -v 2>&1 error

When running a scan with either the dotenvx ext scan or the older dotenvx scan I get this error:

Command failed: gitleaks detect -v 2>&1

Note: gitleaks is already installed via homebrew.

i can't reproduce this yet. where is your gitleaks installed - can you run which gitleaks ?
also, if you run gitleaks detect -v 2>&1 directly, what output do you get on your machine? (please don't share anything sensitive)


Problem editor overlay should auto dismiss after problem is rectified.

VSCode: 1.48.2
Observed Behaviour:
When I rectify a problem in code, the overlay that was present at the line does not auto dismiss when the problem is resolved and the file is saved. I subsequently have to press esc key to dismiss the overlay.
Desired Behaviour:
Once the problem is resolved, and the file is saved, the overlay should immediately dismiss.

When I rectify a problem by editing or moving the cursor the problem hover immediately disappears:

Are you seeing different behaviour?

Sorry, to clarify there is a different problem dialog that is inlined in the editor.
-create a problem on a line of code
-press F8 to jump to next problem.
-a different overlay dialog will show.
-rectify the problem
-the dialog remains.
This dialog should behave like the dialog you have depicted in your post.

@jrieken Currently we show the error peek view as stale. What do you think about closing the peek view when it becomes stale?

Today it's the way it is to ensure a stable editing experience. Image an error widget shows above your cursor location and while you type you fix the error it shows. The whole contents and cursor would jump upwards when hiding the widget.

Makes sense.
Closing this as won't fix.


CON-5228: use MediaServiceInterface instead of concrete Implementation

Coverage remained the same at 42.226% when pulling 4cdceb11c48ec91cc684ca1b52435f974e30f9d2 on CON-5228 into 13b183025afd76596597ea379e12d8a1d7b3cf2e on master.


Add validation so gids are really guids

The validation that guids are unique should probs live in the uuid mixin so there's that, also, we should have that validation so it's guids not gids, thanks to Batman whose joke that was, anyway. This requires as well a schema change to validate that our guids are in fact unique.

@miq-bot assign @bdunne
@miq-bot add_label enhancement
i mean, it is? because i doubt it's actually a bug anywhere? don't even know what to label this, yarps

How does this play with #18502 ?

@bdunne, in https://github.com/ManageIQ/manageiq/pull/18502#discussion_r261613491 wrote:
I don't think any of these classes allow the user to set the guid
irb(main):009:0> ApplicationRecord.descendants.select { |i| i.ancestors.include?(UuidMixin) }.collect { |i| i.base_class.name }.uniq
=> ["MiqServer", "MiqRegion", "MiqWorker", "AutomateWorkspace", "ConditionSet", "Condition", "Job", "LifecycleEvent", "ScanItem", "CustomButtonSet", "CustomButton", "ExtManagementSystem", "Host", "VmOrTemplate", "Service", "MiqAction", "MiqActionSet", "MiqAeWorkspace", "MiqAlertSet", "MiqAlert", "MiqEventDefinition", "MiqEventDefinitionSet", "MiqPolicySet", "MiqPolicy", "MiqWidgetSet", "MiqWidget", "ScanItemSet", "ChargebackRate", "ServiceTemplate", "Provider"]

guid is a column, so it can always be set.  Unless we choose to hide the setter?  However, if we hide the setter, then the `.dup` case that this PR is trying to handle won't work.

What happens if the user creates an object with :guid => nil explicitly?

@bdunne, in #18502 (comment) wrote:
I don't think any of these classes allow the user to set the guid
irb(main):009:0> ApplicationRecord.descendants.select { |i| i.ancestors.include?(UuidMixin) }.collect { |i| i.base_class.name }.uniq
=> ["MiqServer", "MiqRegion", "MiqWorker", "AutomateWorkspace", "ConditionSet", "Condition", "Job", "LifecycleEvent", "ScanItem", "CustomButtonSet", "CustomButton", "ExtManagementSystem", "Host", "VmOrTemplate", "Service", "MiqAction", "MiqActionSet", "MiqAeWorkspace", "MiqAlertSet", "MiqAlert", "MiqEventDefinition", "MiqEventDefinitionSet", "MiqPolicySet", "MiqPolicy", "MiqWidgetSet", "MiqWidget", "ScanItemSet", "ChargebackRate", "ServiceTemplate", "Provider"]

guid is a column, so it can always be set.  Unless we choose to hide the setter?  However, if we hide the setter, then the `.dup` case that this PR is trying to handle won't work.

In this PR, I don't think that's a concern anymore since there is now a validation that guid can't be nil

I like this.  Should the merge of this coincide with the merge of schema data migration to fix any guids that aren't unique?  If we merge this first, could we get validation errors on existing databases?

I don't think the simultaneous merge is critical for this, but I think they should be merged around the same time.  I think the likelihood that someone will have duplicate GUIDs is fairly low.

I agree, I'll approve this but we should probably try to merge this the same few days as the schema PR.    There's just so many models involved, I'd rather we get the data migration nearly ready before we pull the trigger.

Do I need migration for the addition of the guids in this pr to providers?

It's probably a separate migration, but a good idea to have it in the same PR as the other migration

@miq-bot assign @Fryguy

@d-m-u Was this resolved by another PR?

No, I gave up on it because 👻

🎉 🎉 🎆
77 days!!
😆
thanks @bdunne

77 days!!

Yeah, sorry I was away for most of that.  Thanks for taking care of this.

@bdunne @d-m-u I think this PR broke when you do a .select and don't include a guid in the select clause (which I guess answers the question here: https://github.com/ManageIQ/manageiq/pull/18492/files#r262661011).
The attributes now always have a guid in it, and even worse it's a new wrong guid.
[1] pry(main)> v = Vm.where(:id => 2).first; nil
  Vm Load (2.2ms)  SELECT  "vms".* FROM "vms" WHERE "vms"."type" IN ('Vm', 'VmServer', 'ManageIQ::Providers::InfraManager::Vm', 'ManageIQ::Providers::CloudManager::Vm', 'VmXen', 'ManageIQ::Providers::Kubevirt::InfraManager::Vm', 'ManageIQ::Providers::Redhat::InfraManager::Vm', 'ManageIQ::Providers::Microsoft::InfraManager::Vm', 'ManageIQ::Providers::Vmware::InfraManager::Vm', 'ManageIQ::Providers::Amazon::CloudManager::Vm', 'ManageIQ::Providers::Azure::CloudManager::Vm', 'ManageIQ::Providers::Google::CloudManager::Vm', 'ManageIQ::Providers::Openstack::CloudManager::Vm', 'ManageIQ::Providers::Vmware::CloudManager::Vm') AND "vms"."template" = $1 AND "vms"."id" = $2 ORDER BY "vms"."id" ASC LIMIT $3  [["template", "f"], ["id", 2], ["LIMIT", 1]]
  Vm Inst Including Associations (1103.2ms - 1rows)
=> nil
[2] pry(main)> v.guid
=> "ab21cf27-5438-4fb9-92b3-522d966db0fa"
[3] pry(main)> v.attributes.slice("id", "name", "guid")
=> {"id"=>2, "name"=>"LG_592", "guid"=>"ab21cf27-5438-4fb9-92b3-522d966db0fa"}

[4] pry(main)> v2 = Vm.where(:id => 2).select(:id, :name).first; nil
  Vm Load (0.8ms)  SELECT  "vms"."id", "vms"."name" FROM "vms" WHERE "vms"."type" IN ('Vm', 'VmServer', 'ManageIQ::Providers::InfraManager::Vm', 'ManageIQ::Providers::CloudManager::Vm', 'VmXen', 'ManageIQ::Providers::Kubevirt::InfraManager::Vm', 'ManageIQ::Providers::Redhat::InfraManager::Vm', 'ManageIQ::Providers::Microsoft::InfraManager::Vm', 'ManageIQ::Providers::Vmware::InfraManager::Vm', 'ManageIQ::Providers::Amazon::CloudManager::Vm', 'ManageIQ::Providers::Azure::CloudManager::Vm', 'ManageIQ::Providers::Google::CloudManager::Vm', 'ManageIQ::Providers::Openstack::CloudManager::Vm', 'ManageIQ::Providers::Vmware::CloudManager::Vm') AND "vms"."template" = $1 AND "vms"."id" = $2 ORDER BY "vms"."id" ASC LIMIT $3  [["template", "f"], ["id", 2], ["LIMIT", 1]]
  Vm Inst Including Associations (2.2ms - 1rows)
=> nil
[5] pry(main)> v2.guid
=> "1a9e9816-672e-490a-a5c9-153b525dbb50"
[6] pry(main)> v2.attributes
=> {"id"=>2, "name"=>"LG_592", "guid"=>"1a9e9816-672e-490a-a5c9-153b525dbb50"}

[7] pry(main)> v.guid == v2.guid
=> false

First saw this in a repo failure here: https://travis-ci.org/RedHatCloudForms/cfme-cloud_services/jobs/532975173

@bdunne asked me if default_value_for was doing this, but I tried with a different defaulted column in a completely other model, and it works as expected...
See https://github.com/ManageIQ/manageiq/blob/master/app/models/miq_group.rb#L35, which defaults the :group_type column to "user".
[1] pry(main)> MiqGroup.where(:id => 1).first.group_type
=> "tenant"
[2] pry(main)> MiqGroup.where(:id => 1).select(:id).first.group_type
	 5: from (activerecord-<IP_ADDRESS>) lib/active_record/attribute_methods/read.rb:36:in `__temp__7627f65707f547970756'
	 4: from (activerecord-<IP_ADDRESS>) lib/active_record/attribute_methods/read.rb:66:in `_read_attribute'
	 3: from (activerecord-<IP_ADDRESS>) lib/active_record/attribute_set.rb:44:in `fetch_value'
	 2: from (activerecord-<IP_ADDRESS>) lib/active_record/attribute.rb:196:in `value'
	 1: from (activerecord-<IP_ADDRESS>) lib/active_record/attribute_set.rb:44:in `block in fetch_value'
(activerecord-<IP_ADDRESS>) lib/active_record/attribute_methods/read.rb:66:in `block in _read_attribute': missing attribute: group_type (ActiveModel::MissingAttributeError)
[3] pry(main)> MiqGroup.where(:id => 1).select(:id).first.attributes
=> {"id"=>1}


TCPHandler Errors

I have a 3 node cluster running on Kubernetes in Azure that is spewing about 6,000 logs per hour regarding TCPHandler, of which 2,000 are <Warning> TCPHandler: Client has not sent any data.\n. I originally reported this in the Go driver - https://github.com/kshvakov/clickhouse/issues/102, but I still see these logs when the client is disconnected.
I can't tell from the logs if these communication errors are from internal communication or not. I suspect that they may be related to MySQL dictionary connections, but I'm not sure how to verify that.
My cluster otherwise seems to be performing well, with queries running quickly and no errors in operation. My tables use ReplicatedReplacingMergeTree, with no shards or distributed tables.
What is the best way to find out more information? I am running on the yandex/clickhouse-server:1.1.54380 Docker image and using ZooKeeper 3.5.3beta.
error.log
2018.04.19 18:38:25.329715 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:26.672138 [ 6750 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:27.913569 [ 6749 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:32.519611 [ 6752 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:35.329742 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:42.915973 [ 3583 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:45.329743 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:46.676854 [ 6752 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:47.523200 [ 3681 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:55.329787 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:38:57.527671 [ 3583 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:39:02.921512 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:39:05.329785 [ 3681 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:39:06.681526 [ 6750 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:39:07.531955 [ 6752 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:39:12.923069 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.

log
2018.04.19 18:32:37.426676 [ 6750 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:58719
2018.04.19 18:32:37.825411 [ 3343 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:49876
2018.04.19 18:32:41.591676 [ 6745 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:55638
2018.04.19 18:32:42.427051 [ 6750 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:32:42.427087 [ 6750 ] <Information> TCPHandler: Done processing connection.
2018.04.19 18:32:42.825456 [ 3343 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:32:42.825525 [ 3343 ] <Information> TCPHandler: Done processing connection.
2018.04.19 18:32:45.329679 [ 6749 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:44156
2018.04.19 18:32:45.329926 [ 6749 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:32:45.329953 [ 6749 ] <Information> TCPHandler: Done processing connection.
2018.04.19 18:32:46.592199 [ 6745 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:32:46.592246 [ 6745 ] <Information> TCPHandler: Done processing connection.
2018.04.19 18:32:55.329507 [ 6749 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:44200
2018.04.19 18:32:55.329688 [ 6749 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:32:55.329711 [ 6749 ] <Information> TCPHandler: Done processing connection.
2018.04.19 18:32:56.595929 [ 3681 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:55722
2018.04.19 18:32:57.429842 [ 6750 ] <Trace> TCPHandlerFactory: TCP Request. Address: <IP_ADDRESS>:58814
2018.04.19 18:33:01.596887 [ 3681 ] <Warning> TCPHandler: Client has not sent any data.
2018.04.19 18:33:01.596999 [ 3681 ] <Information> TCPHandler: Done processing connection.

TCP Request. Address: <IP_ADDRESS>:58719
It's really strange IP -- <IP_ADDRESS>, usually it's been used for icmp hole punching.
Check your firewall settings, it seems you allow tcp connections from the public network.

It's really strange IP -- <IP_ADDRESS>, usually it's been used for icmp hole punching.
I took out my public IP and replaced it with <IP_ADDRESS> when reporting this issue.

Check your firewall settings, it seems you allow tcp connections from the public network.
That's a good idea, I'll try closing my load balancer and see if that eliminates the errors.

After running without any external access for a few hours, there was no difference in the internal log errors, so it wasn't just rogue bots trying to connect. It must be internal.

May be some health checks? I.e. for example loadbalancer checking yhat port
is opened?
ср, 9 мая 2018 г., 20:35 Derek Perkins<EMAIL_ADDRESS>
After running without any external access for a few hours, there was no
difference in the internal log errors, so it wasn't just rogue bots trying
to connect. It must be internal.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yandex/ClickHouse/issues/2332#issuecomment-387834541,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABelA1FS1T3K2f1S23V56zBNOl9MiHP9ks5twzbogaJpZM4T3NL5
.

@filimonov Thanks for the suggestion, that is absolutely it, I can't believe I didn't realize it sooner.
Right now, the health check is opening a TCP connection to port 9000. Is there a better way to make sure that ClickHouse is working properly that won't throw errors?

try to 8123 instead of 9000
$ curl http://localhost:8123
Ok.

That worked great, thanks!


Release 1.2.0

The indentation looks weird now. Can you fix it ?

I was too quick on the release trigger. :(
We will fix it later!

Thanks :)


Early Crafting Results often not displaying

If I reload the addon it seems to work to display early crafting results but if I zone it seems to break it until I reload the addon

Private Server or retail?

Cannot speak for OP, but I am getting missing craft results and often times double posts on HorizonXI.


Refactor 'Recent Transactions' view

Ensure relevant transactions always appear in the detail view.

@JamesSmartCell unclear on the meaning of this PR, can you write clearly please?

Ensure transactions relevant to the user's wallet appear in the transaction detail view (the new view that was designed showing the short transaction history with send/receive at the bottom). I noticed during EDcon that when there's only occaisional activity on the account the transactions didn't always appear there, even though the balance updated. This is most likely because of a lag between infura sensing the balance update and etherscan logging the transaction. Best to play it safe and continually update the transaction list from the network when in focussed view.


How to get field value?

In Java Code：
public static final String SDK_VERSION = "1.2.0"
how to use androguard to get" SDK_VEDRSION" the value of this field。

You mean like:
a,d,dx = AnalyzeAPK("apk")
d.get_class("Lyour/class/path;").get_fields()[0].show()

This would print out the first field in Lyour/class/path; class.

How to get the string "1.2.0"？

I think it should be:
d.get_class("Lyour/class/path;").get_fields()[0].get_init_value().get_value()

You coud try to call the reset() Method on the Session (e.g. CONF["SESSION"].reset())
At least for me that helped sometimes. But still, the RAM gets fuller and fuller. I also want to investiagte in this, but I'm still on the way to make androguard useable with python2/3


Extend API for Editing and Deleting

Hello,
since we use Cabot to run a lot of checks on a lot of machines, we have batch scripts to quickly generate checks and services as needed.
It would be very helpful if the API did not only supply creation of new, but also editing and deletion of existing checks, services and instances.

@JeanFred I thought this was already possible?

@JeanFred - could you please give an update?
We cannot find the possibility to edit or delete existing checks via the API documentation.
Thanks!

Sorry about the lag @LXDavid. I just checked, you can DELETE checks via the API but only if you are a staff user.
I’m not sure how much sense that makes − what you’re allowed to do via the web UI, you should be able to do via the API really.

I don't see a way to edit either... re-posting a json payload creates a duplicate resource with a different primary key.

@cmclaughlin which version are you using ?
I'm on 0.10.2 and I only receive

"detail": "Method "PATCH" not allowed."
From the api so i can't update.

As you say re-posting a json payload creates a duplicate resource with a different primary key.

Strike my last comment...
I was using PATCH on /api/http_checks/?format=json&id=$id instead of /api/http_checks/$id/?format=json
I think this can be closed.


Fix Jinja2 breaking docs

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[ ] HISTORY.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added

[ ] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

pins Jinja2 below its latest few versions

Does this PR introduce a breaking change?
No.
Other information:

Coverage remained the same at 91.584% when pulling bbaed90887ec9b61fb058207cd282c75d0cad143 on temp_fix_docs into 5803bf2fb07ba942dede7c1d99abfaaa4b346acb on master.


Simulation fix

PR to fix bug and re-enable simulation mode.  Addresses issue first raised by @adam-erickson , and noted in Issue #114, by including a slight IO refactor in order to utilize the SpectrumFile class for simulation input data cubes.

Nice.


Extended ButtonTheme, RoundedRectangleBorder, gallery buttons demo

Added a ButtonTheme.fromButtonThemeData constructor and a ButtonTheme copyWith() method. This makes it easier to build a ButtonTheme that's derived from the inherited one.
Added support for lerping from RoundedRectangleBorder to StadiumBorder.
Added a feature to the gallery buttons demo that changes the buttons' shape from the default rounded rectangle to a stadium. Because this is done by tweaking the current button theme, the shape change animates.

LGTM other than adding the new lerp.


Effect bonuses to saving throws are combined as strings before being applied

Originally in GitLab by @javalar
Steps to reproduce:

Create a new effect on a character which adds +1 to saving throws (data.bonuses.abilities.save) and activate it.
Create a second effect on the same character which does the same and activate it.

Expected behaviour:
Character has a +2 bonus to saving throws.
Actual behaviour:
Character has +11 bonus to saving throws.
This is probably because the bonus numbers are saved as strings and added together (as strings) before adding to a number (where the string is implicitly cast to a number before being added).

This was changed several versions ago now.


Players shown as blue blox, pls fix

players are blue boxes how can i fix this

not planned to fix, it is just proof of concept web minecraft client


Investigate why numpy.unique is so slow on Series input

Only true anecdotally (I've witnessed np.unique(s.values) being a lot faster but didn't have time to profile)

Ha, turns out was using __builtin__.sum. Major failure


Game board doesn't show if the snake is not given a name on the game server (if left blank)

If I create a game without naming my snake on the game server, the board doesn't show.

Uncaught Error: Ran into a `Debug.crash` in module `Game.State`

This was caused by the `case` expression between lines 49 and 55.
One of the branches ended with a crash and the following value got through:

    Err "Expecting a String at _.content.board.snakes[0].name but instead got: null"

The message provided by the code author is:

    {
    "content": {
        "status": "suspend",
        "board": {
            "width": 20,
            "turn": 0,
            "snakes": [
                {
                    "taunt": "You've just been ERASED!!",
                    "tailType": "fat-rattle",
                    "name": null,
                    "id": "056e2d42-5a68-4fc1-8f3c-ae982ea06a4e",
                    "health": 100,
                    "headUrl": null,
                    "headType": "sand-worm",
                    "coords": [
                        [
                            3,
                            3
                        ],
                        [
                            3,
                            3
                        ],
                        [
                            3,
                            3
                        ]
                    ],
                    "color": "#6495ED"
                }
            ],
            "id": "a67e48fe-ce3c-441e-81c0-1c2493dba7a3",
            "height": 20,
            "gameId": 1,
            "food": [
                [
                    1,
                    6
                ]
            ],
            "deadSnakes": []
        }
    }
}
    at Game.elm:1540
    at decodeTick (Game.elm:17523)
    at updateBroadcast (Game.elm:17530)
    at Function.func (Game.elm:17559)
    at A2 (Game.elm:92)
    at Object.callback (Game.elm:5006)
    at step (Game.elm:5641)
    at work (Game.elm:5699)

Yes this is correct, the name is a required field, but we don't have validation on the form yet.

This is out in the server image now.


update readme and dockerfile for memory pressure module.

Instead of accepting number of cores and number of threads per core, you can use ProcessorArch.getProcessorArchInformation(). Once this is done, then SystemLoad.java can accept only [min-cpu-usage-pressure], [max-cpu-usage-pressure] and [memory-bytes-pressure]
Update the README to include instructions to run SystemLoad.java.
Update the Dockerfile to include the line (commented) to RUN SystemLoad.java.

pull request #8  closed.


[iOS] Load image downloaded previously

I have two types of local files:

Files of the app. Route is like /var/containers/Bundle/Application/8B28041B-1575-46D5-A884-E817818FB7E9/MyApp.app/viewer/index.html (RNFS.DocumentDirectoryPath)
Files downloaded and stored by the user in the app. Route is like /var/mobile/Containers/Data/Application/74C447A6-65DE-44FF-8CB0-79D4E09D59CC/Documents/myImage.jpg (RNFS.MainBundlePath)

As soon as I try to reference a downloaded file from RNFS.MainBundlePath, e.g.
<img src="...." />
I get this error:
Failed to load resource: The operation couldn’t be completed. Operation not permitted
<WebView
  javaScriptEnabled
  allowFileAccess
  originWhitelist={['*']}
  source={{ uri: `file://${rootPath}/index.html`, baseUrl: rootPath }}
/>

Using:
"react-native-webview": "^5.5.0"
"react-native-webview-bridge": "^0.40.1"

I'm having the same issue. It seems to work on https://facebook.github.io/react-native/docs/webview but this will be removed soon.

@EstebanBP @Liestambeur We are facing the same issue in iOS. We save the files downloaded in the documents dir. If we load it using the webview that is part of the react-native core it works well but it fails to load the files for react-native-webview package.

Hey @EstebanBP , any luck on this? I'm having this problem for weeks now and can't find a way to make it work.
In my case, I'm downloading some files in the Cache folder, the HTML seams to load ok (seeing it on safari web inspector) but js files that are on different folders spam this same error.
I'll appreciate any info on this. Thanks.

I'm having the same issue.


fix(router) various fixes and improvements

This initially started as a proper fix for #2073 (raised by @javeme), since the initial fix proposed was incomplete (missing tests and undesired Nginx variable) and the fix was quite urgent (currently affecting all 0.10.0 RCs), this supersedes the proposed fix. Additionally, I discovered another issue with the router's priorities that were off-base in some edge-case.
Full changelog

avoid HTTP method to supersede non-plain URI
preserve upstream_url path component if exists
prefer ngx_re API to ensure JIT

Bonus:

simpler return values for exec() API

Issues resolved
Fix #2073
Supersede #2074
Note: do NOT squash for better future tracking in case those commits introduce an issue.

this might also explain some of the test failures with oauth2 and rate limiting

Does this fix the issue with <api>/oauth2/token not being reachable when using uris instead of hosts?

@DonMartin76 Would appreciate if you try it out and see if it does.

@thibaultcha It does not fix it. Steps to reproduce:
curl -d 'name=hello&uris=/mockbin&upstream_url=http://mockbin.org' http://localhost:8001/apis
curl -d 'name=oauth2&config.enable_client_credentials=true' http://localhost:8001/apis/hello/plugins
curl -d 'username=someuser' http://localhost:8001/consumers
curl -d 'name=oauth2&client_id=some_id&client_secret=some_secret&redirect_uri=http://dummy.org' http://localhost:8001/consumers/someuser/oauth2
curl -d 'grant_type=client_credentials&client_id=some_id&client_secret=some_secret' --insecure https://localhost:8443/mockbin/oauth2/token

The last curl must return an access token. The equivalent when using hosts works:
curl -d 'name=olleh&hosts=binmock.org&upstream_url=http://mockbin.org' http://localhost:8001/apis
curl -d 'name=oauth2&config.enable_client_credentials=true' http://localhost:8001/apis/olleh/plugins
curl -d 'username=otheruser' http://localhost:8001/consumers
curl -d 'name=oauth2&client_id=other_id&client_secret=other_secret&redirect_uri=http://dummy.org' http://localhost:8001/consumers/otheruser/oauth2
curl -H 'Host: binmock.org' -d 'grant_type=client_credentials&client_id=other_id&client_secret=other_secret' --insecure https://localhost:8443/oauth2/token

Well, this is out of the scope of this PR then. Thanks for checking still.

Np. Is there a bug for the above behaviour already? Or should I file a new issue?

This could actually be due to how the router now works. I am not the least familiar with the OAuth2 plugin but I'll take a look.

👍

@DonMartin76 Mind trying out #2086?


Add required Web.config items for porting

While porting a .NET Framework project DI was not able to find the provider for Microsoft.Data.SqlClient without the changes to Web.Config.
I'm not sure if there should be additional text to explain that this is applicable to .NET Framework only...

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: harrygreenhough sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add tests for topic utils

This should add nearly 💯 % test coverage for the slack_topics.topics.utils module. I say nearly because there isn't currently any coverage tool used with pytest, so I don't know for sure. I recommend pytest-cov if you do want to add it.

Travis-ci job appears to have fail on token assertion. Running reillysiemens:tests locally builds and passes tests.


[HIVEMALL-40] Load xgboost-formatted data via Java ServiceLoader

What changes were proposed in this pull request?
In the mater, we need to directly set an explicit implementation class for loading xgboost-formatted data. This pr modified code to resolve the class via Java ServiceLoader.
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/HIVEMALL-40
How was this patch tested?
Added a test to resolve an implementation class to load xgboost-formatted data in XGBoostSuite.scala.

Coverage increased (+0.3%) to 36.14% when pulling d0cbf3db2cb4ce6bcca88a8ed45488a016e9960e on maropu:HIVEMALL-40 into 1801a62c15f4466c331456f6e2d7102b715d4a63 on apache:master.

Coverage increased (+0.3%) to 36.14% when pulling d0cbf3db2cb4ce6bcca88a8ed45488a016e9960e on maropu:HIVEMALL-40 into 1801a62c15f4466c331456f6e2d7102b715d4a63 on apache:master.

LGTM. Please merge and close this PR.

Merged


NXOS bootstrap tests

Converted nxos_bootstrap to python object
Added unit tests for nxos_bootstrap

Coverage increased (+12.8%) to 66.232% when pulling 23b56ab84c643ef57a45a5494ab3a2458147b7ef on nxos_bootstrap_tests into 80735475c589e56426074a7f0882f8f1e04b31d5 on master.

Coverage increased (+12.8%) to 66.232% when pulling 3e552580993b47adb79776737921d2467d77b5d8 on nxos_bootstrap_tests into 80735475c589e56426074a7f0882f8f1e04b31d5 on master.


to hide the start and end characters

hi, is there any method to hide the start "{" and end "}" characters when with the following rule? thanks.
          DecoratorRule.between(
            start: "{",
            end: "}",
            style: TextStyle(
              color: Colors.blue,
            ),
          ),

Not at the moment, but I can add it as a feature.

@wmhseir3 I just added this feature on version 1.0.6 removeMatchingCharacters
DecoratorRule.between(
   start: "{",
   end: "}",
   style: TextStyle(
      color: Colors.blue,
   ),
   removeMatchingCharacters: true
),

you are more than just great. thanks. it does removeMatchingCharacters.


Update index.php

:star: PR built and available in a preview environment tomandco-cloudshell-tutorial-pr-1 here

:star: PR built and available in a preview environment tomandco-cloudshell-tutorial-pr-1 here


Yarn Package Version Upgrades

Fixes some issues encountered on homestead deployments, and generally just installs newer versions of packages.

:tada: This PR is included in version 4.14.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Feature Request] Custom "resolver" support

Is your feature request related to a problem? Please describe.
Go and Java gRPC implementations support custom resolvers so users can fail over to different IPs/hosts, but core does not support this. Tonic does not support it directly but does suggest a connector can be used.
Describe the solution you'd like
temporal_client::ClientOptions should allow some form of custom connector or resolver callback or something that can allow us to customize the connection that is occurring but without being too complicated hopefully. Could accept a connector but have helper/wrapper for simple resolver connector so that we don't have to mess w/ complexities of connectors.
This request may overlap with #378.

It looks like introducing https://github.com/TrueLayer/ginepro might be the right next step here. I'm looking into it.

Hrmm, there still may be value in general purpose resolver override and not specific to what that library does. I wonder if we can take lessons from that library yet still provide general resolver customization.

In case it's useful in the future, there's a WIP branch integrating ginepro here: https://github.com/dandavison/temporalio-sdk-core/tree/565-ginepro


Powerful timeouts

Currently there's a single timeout option that sets a deadline for the entire request to complete.
This is suboptimal for large files and slow connections, because it doesn't distinguish between the server not responding at all, and the file downloading reliably, but slowly.
In reality there are multiple stages of the request that can time out:

DNS timeout
TCP/IP connection timeout
File upload
Waiting for HTTP response headers
Waiting for each chunk of HTTP response body

I'm thinking of hooking into more of these steps to allow having shorter timeouts as appropriate, e.g. DNS and TCP/IP connection never take more than a few seconds, even if the whole request needs minutes to download.
What should be the API for this? Should we try to interpret existing .timeout(2000) API in some clever way?

👎 for clever API interpretations and overloading. My first thought would be leave .timeout(2000) as an alias for the entire request completing including the full body, and introduce APIs like .resolveTimeout(500), .connectTimeout(1000), .headerTimeout(3000), .chunkTimeout(4000).

By overloading, do you mean .timeout(Number) vs .timeout({connect:…, chunk:…})?

Also, should we have any defaults?

Yes, overloading meaning a single API function call that has multiple meanings or behaviors depending on arguments provided. I think in general that's an antipattern and having distinct API functions that do 1 clear thing is preferable, even if it means the API surface is bigger (it's just being honest about the actual API surface).
I don't mind .timeout({connect: 42, chunk: 97}) that much though. It's not terrible. It just requires good validation and I think it's easier to document and more obvious when you typo .conectTimeout(42) and get an obvious exception vs calling .timeout({conect: 42}) and maybe having to debug why the timeout seems to not be taking effect as expected. If the code detects unexpected option names and throws an exception immediately, it's close enough I think.
I think we should have no defaults and do what the runtimes/node/oses do by default.

The MVP of response timeout seems to work well enough, so I'll leave it at that.


dcmaudit - need a way to turn rectangles on/off, and show NER source

[x] Turn on/off the redaction rectangles, to see the text underneath or to verify if it's a false positive
[ ] Show the source of a redacted rectangle so the user can see if it was due to: OCR, UltrasoundRegion
[ ] Show the PII status of a redacted rectangle so the user can see if it is PII (from NER), not PII (from NER), not PII (from whitelist), not known
[ ] Show rectangles which were not redacted because of a false positive due to being large but few characters (huge font). Currently these are filter out and not reported so cannot be shown in the GUI but if that was changed and they were available but marked as not PII due to bad OCR then they could be shown (just to check that the huge font detection is working correctly).

The first one has been completed via https://github.com/SMI/dicompixelanon/commit/b1f8e34cf13b307534da22fedb2da931e159f1cd


Migration of Quick Start Documentation from HTML to Markdown

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Migrated Quick Start Documentation from HTML to Markdown.
Original HTML files preserved
Created separate folders for both HTML and Markdown files

Why should this Pull Request be merged?

Contains Quick Start Documentation in Markdown format

What testing has been done?

No testing is required, only review

FYI, the .lvproj file is modified: Source/FPGA Addon.lvproj
If this is intended, no problem with me. I just want to make sure you know that the change is there and revert it, if it's unintended.

Yep, I already had a discussion with Paul about this. Should be good ;)


Training Stage Loss

Dear author，
Thanks for your great work. Recently I do the code reproduction and fine a small issue which I'm not sure. When I finish the stage1 training and start to train the stage2 I found that the loss will be negative in some cases. After checking the loss code part I thought it will not be negative theoretically. May you know what the problem is or it is not a problem due to my poor knowledge.

The loss includes KLDivLoss which may get a negative loss

Okay, I got it. Thanks for your reply


When running in the iOS simulator, real adverts are displayed even when isTesting = true.

Hi
Great plug-in. Finding it really useful.
One thing I have noticed is that when isTesting = true, I still see real adverts when running in the iOS simulator.
I have done some research and have the following suggestion to contribute.
The release notes for "SDK for AdX Users on iOS" (see https://developers.google.com/mobile-ads-sdk/docs/adx/ios/rel-notes) say
7.2.1	2015‑04‑30
"Simulators once again load live ads by default. Add kGADSimulatorID to the testDevices property on GADRequest to get test ads on simulators."
I modified __buildAdRequest in CDVAdMobAds.m near line 704 so it looks like
request.testDevices =
[NSArray arrayWithObjects:
admobDeviceId, kGADSimulatorID, @"dabe...snip...2b3e",
nil];
Now when isTesting = true, I see test adverts when using the iOS simulator as well as when running on my iPad (UUID beginning dabe etc).
This is not my area of expertise, but I wonder if a change along the lines of what I have suggested might be worth considering.
Best wishes
Mike

I'm getting this as well.  From my console:
<Google> To get test ads on this device, call: request.testDevices = @[ kGADSimulatorID ];

It would be great if that chunk of figured out it's own device id (whether its simulator or not) and passed the required information.

Once uppon a time it was there, then I removed, don't ask me why. Now it's there again. Will see if any issues make me remember why I removed 😄 Thanks anyway!


fix broken float type precision conversion

it is not the case that ln(10) = log_2(10)
@beikov

But WTH is up with those HSQLDB failures, surely they can't be related to this change?

I'd say don't mind these failures. These seem related to some DST changes. Did DST change 3 hours ago in the TZs we test for? :D
FWIW, I would love to get rid of the HSQL dialect from hibernate-core, but that's a different discussion.

I'd say don't mind these failures.

Yeah, agreed. Just ran the tests on a local hsqldb and nothing failed.

Did DST change 3 hours ago in the TZs we test for? :D

Well the same thought occurred to me, but Dec 13 seems like a strange date to change clocks....

Screw it, I'm merging, main was already failing.


Dart extension doesn't work at all

Autocomplete or hover doesn't work at all. On editor open, this shows up in the notifications:

Thanks for logging the issue, @daaniiieel !
Will take a look and see what's blocking - one issue is that our version of the extension-host is a bit behind (Onivim is on 1.47.0, Dart requires 1.50.0) - been meaning to get it up-to-date - thats likely the source of the proposed-api warning.
Related issue: https://github.com/onivim/oni2/issues/2411

I've updated the extension host here, and then tracking the remaining issues here: #2411
I recommend in that issue setting dart.previewLsp to true for best results - it seems like the LSP behaves much better with Onivim than the previous diagnostics service.
I'll close this to track the remaining issues in #2411


[Ticket] Create Sign In and Sign Out Navigation Component

Description

Create a component called Navbar that takes in a session prop. If the session exists, it should render a welcome, along with the user's name or email, along with a signout button
If it doesn't, then it should have a sign in button.
You can make a folder called components under pages to do this.
Contact Manu for the design.

Acceptance Criteria

[ ] Sign in and sign out should work as expected
[ ] Design should work on a multitude of browsers / mobile layouts

Other Notes

Refer to Next Auth to understand what sessions are: https://next-auth.js.org/tutorials/securing-pages-and-api-routes

Can i grab this issue?

@mgupta8143 Could you post a link to the Figma design / send it on Slack when you get the chance?


scientific bam tests

Adds a scientific test comparing alignments in the optimus aligned/tagged bam output with expectations built from Cell Ranger.

retest this please


Can't install Tenserflow on ubuntu 16.04 via pip?

Exception:
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/pip/basecommand.py", line 215, in main
status = self.run(options, args)
File "/usr/local/lib/python2.7/dist-packages/pip/commands/install.py", line 342, in run
prefix=options.prefix_path,
File "/usr/local/lib/python2.7/dist-packages/pip/req/req_set.py", line 784, in install
**kwargs
File "/usr/local/lib/python2.7/dist-packages/pip/req/req_install.py", line 851, in install
self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
File "/usr/local/lib/python2.7/dist-packages/pip/req/req_install.py", line 1064, in move_wheel_files
isolated=self.isolated,
File "/usr/local/lib/python2.7/dist-packages/pip/wheel.py", line 345, in move_wheel_files
clobber(source, lib_dir, True)
File "/usr/local/lib/python2.7/dist-packages/pip/wheel.py", line 316, in clobber
ensure_dir(destdir)
File "/usr/local/lib/python2.7/dist-packages/pip/utils/init.py", line 83, in ensure_dir
os.makedirs(path)
File "/usr/lib/python2.7/os.py", line 157, in makedirs
mkdir(name, mode)
OSError: [Errno 13] Permission denied: '/usr/local/lib/python2.7/dist-packages/tensorflow_gpu-0.12.1.dist-info'
NOTE: Only file GitHub issues for bugs and feature requests.  All other topics will be closed.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Environment info
Operating System:
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
If installed from binary pip package, provide:

A link to the pip package you installed:
The output from python -c "import tensorflow; print(tensorflow.__version__)".

If installed from source, provide

The commit hash (git rev-parse HEAD)
The output of bazel version

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
What other attempted solutions have you tried?
Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).

You'll want to run as root.
OSError: [Errno 13] Permission denied: '/usr/local/lib/python2.7/dist-packages/tensorflow_gpu-0.12.1.dist-info'

Did you use the sudo pip? It said that not permitted, so I think that maybe you didn't use it

Closing issue due to inactivity.


Firefox drag issue, still

Hi.
I'm using angular 1.4.8 together with ng-sortable 1.3.6 to provide reordering of a image collection. Also I use the latest firefox version. Using this combination I'm seeing the same effects as mentioned here: https://github.com/a5hik/ng-sortable/issues/61
I made a small gif that demonstrates the problem. If I try to drag the image, the no-drag-area sign appears and it doesn't start the drag operation. Also the dragStart callback isn't fired. When I release the mouse button, it automatically starts the drag operation as it should be and also the dragStart callback fires.

It seems to have something todo with img tags. Using a normal div works as expected - like in the demos. Also using chrome is no problem with images.
Here's the code snippet:

If also read somewhere putting draggable="false" to the img tag should fix this. Indeed that seems to work most of the time. But even there, sometimes it fails as well. That happens, if I first select the image with a normal click. Then it selects the image with a blue overlay. If I try to drag that selected image, it fails the same way as without using draggable=false. Dragging an previously unselected image, works as expected.

Any clue what's going on here?

I had the same issue and the solution was to remove the image.
You can create a div with the width and height of the image and set the image as background with CSS


your examples dont work with latest wrangler, miniflare3

Hi I tried the code, and your code has an error on the wrapper
==> im doing a simple fetch handler, and returning hello world

I will try to have a look at this in the next few days. I always test with the example in the source directly, but not with the example in the README (which was a copy & paste of the code example, but diverged to keep it simple)

@evanderkoogh I think this issue is more than just the examples ... I now seem to always get the Uncaught (async) RangeError: Maximum call stack size exceeded when developing locally with wrangler on any project with this library, although they do work when deployed to production ...

I am seeing the same on wrangler (v2) locally. I haven't tested it in production.

Hey everyone.. apologies for the lack of response here.. this is a duplicate of #5. As I have mentioned in there, I have found the cause of the issue, which is in wrangler. I am working on a fix there and will have a look to see if I can do a work-around in this code.
@paulgibbs, there miniflare 2 issue has been solved in version 1.0.0-alpha.3

duplicate of #5


Ignore empty strings

I noticed this with comint input rings where empty strings are contained. In default completion UI empty strings are omitted, as well.

Updated the changelog


storage: Add perf statistics for scan detail

What problem does this PR solve?
Add perf statistics for scan detail, so we can investigate which slow operation of which CF affects the command duration.
The metrics would be like this

What is changed and how it works?
Record perf context diff every time calling next/prev/seek/seek_for_perv of cursor.
Related changes

Need to cherry-pick to the release branch

Check List

Manual test (add detailed scripts or steps below)

Side effects

Performance regression

Consumes more CPU (maybe)

Release note

/bench

/bench

/bench +tpch

/bench +tpch +sysbench +tpcc

@Little-Wallace the results show no noteworthy regression. I think it's acceptable.

Maybe using a single "tombstone" property for all cursor operation (seek + next + ...) is enough for our scenaro? Also do you think it is necessary to use PerfContext API directly, considering that PerfStatisticsInstant::delta() calculates a lot of unused peoperties for this PR?

@breeswish You mean like this?
let before = PerfContext::get().get_internal_delete_skipped_count();
iter.seek();
self.statistatics.seek_tombstone += PerfContext::get().get_internal_delete_skipped_count() - before;

PerfStatisticsInstant::delta() just counts 7 properties, not sure if reducing 7 to 1 would make any great difference. Anyway, it's easy to change, I'll change it later.
    pub internal_key_skipped_count: usize,
    pub internal_delete_skipped_count: usize,
    pub block_cache_hit_count: usize,
    pub block_read_count: usize,
    pub block_read_byte: usize,
    pub encrypt_data_nanos: usize,
    pub decrypt_data_nanos: usize,

/bench +tpcc +sysbench

PTAL @Little-Wallace @breeswish

@Connor1996 Yes, exactly!
Note that now scan details are also send back to TiDB so that it can be recorded in places as Statements and Slow Queries. How about changing kvproto.ScanDetailV2 as well to include this newly introduced metrics? For simplicy consideration, maybe merging xxx_tombstone to tombstone altogether when returning back to TiDB is enough.

@breeswish ScanDetailV2 already includes a field named rocksdb_delete_skipped_count

ping @Little-Wallace @breeswish

/merge

/merge

/test

/merge

/test

/merge


Where is the pre-trained models for the demo ?

Hi, SeanChenxy, thanks for your impressive work.
I have tried the demo according to your instructions, but I can't find the pre-trained models. Does it mean that I need to use the models that are trained by myself ?

Hi, you would want to download the files you need from Google drive.

Hi, you would want to download the files you need from Google drive.

Oh, I have found it, thanks !


Show package version under application settings

No easy way to determine what version of headscale-ui is running.

closed with #7


Crash in com.Metal.CompletionQueueDispatch

I'm trying to use MoltenVK with libplacebo on macOS and running into issues when resizing the window causing random crashes in the Metal Completion Dispatch Queue:
Crashed Thread:        2  Dispatch queue: com.Metal.CompletionQueueDispatch

Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Application Specific Information:
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
abort() called

Full crashlog can be found here.
The full source code for the application can be found here.

@ePirat
Can you post, to the cloud somewhere, the libplacebo.dylib library you are using? That will allow us to build your app without having to install everything needed to build libplacebo.dylib.

@billhollings Sure, I've updated my project to make that easier, download and extract 3rdParty.zip and put the 3rdParty folder in the MoltenPlacebo repo root, next to the MoltenPlacebo and  MoltenPlacebo.xcodeproj, the Xcode project should then find the libraries.
Hope that helps.

It might be related to this bug?

@billhollings Any update on this?

@ePirat Thanks for the prod. I'll look at it as soon as I can.

@ePirat This is fixed in PR #652 . Please retest and close this issue if it fixes your issue.

Yes, I can't reproduce anymore with latest master, so it seems #652 fixed it! Thanks a lot!


Failed to load resource: the server responded with a status of 401 ()

After I created the Chrome extension using this JS framework, it gives me the following error in the console.

The link goes to this json:
{
"error": {
"errors": [
{
"domain": "global",
"reason": "required",
"message": "Login Required",
"locationType": "header",
"location": "Authorization"
}
],
"code": 401,
"message": "Login Required"
}
}
Does it mean that I have to authenticate when using this?

Hi eduard9x, it seems like you do need to authenticate in order to utilize the extension. Maybe send the authentication info with the API call?

This looks Google-chat/voice related, and should not be a considered a problem with this library. My guess is this can be reproduced without this library injected at all.
Since there has been no activity or comments on this issue since it was filed, I'm going to close it.
Please re-open if you still feel you have something you need resolved.


Create Parameter Set Tools

This is going to be based upon a function created by KirkMunro on thread: Add / amend syntax displayed from Get-Command -syntax. (PS, need to make sure to credit KirkMunro in the documentation).
The output generated by this function is difficulkt to read, we need to sprinkle some Krayon magic dust on it and pretty up the out put, in particular highlighting the parameter set name, so they can be easily picked out by eye.
I would like to develop this into a fully fledged ParameterSet debugging tool. One nice addition would be enable the user to specify a function and a parameter list to see which parameter set is invoked. I know you can effectively do this by invoking the function for real, but we might not want to do this especially if the function is a state changing one.
When PowerShell can't determine the parameter set for a particular invoke set, we need to see the list of parameter sets that PowerShell doesnt have enough information for to choose which one.
We also need functionality that will show if there are any parameter sets that have the same set of parameters, this will also help in debugging parameter set issues.
Might have to split this functionality out into different commands depending on further analysis.

Some notes:
For both of these, the type is optional, switch parameters don't have any type
[-Pattern] <type>: optional parameter
-underscore <type>: mandatory parameter

mandatory param name => krayola.AFFIRM-COLOURS
optional param name => krayola.VALUE-COLOURS
command name => green
parameter set name => dark green
table column titles => krayola.KEY-COLOURS
table column titles => krayola.KEY-COLOURS
table mandatory param value => krayola.AFFIRM-COLOURS
table optional param value => krayola.VALUE-COLOURS
table value default => gray

A couple of artciles regarding proprty expansion/parameter-set processing:

https://devblogs.microsoft.com/scripting/use-the-get-command-powershell-cmdlet-to-find-parameter-set-information/
https://blogs.msmvps.com/jcoehoorn/blog/2017/10/02/powershell-expandproperty-vs-property/

other functions to create:

Test-CommandInvoke: identifies the parameter set for an invoke, if ambiguous should show candidate list. This would require the user to provide actual parameters.

Add the following extra parameters to Get-ParameterSetInfo:

Sets: an array of parameter set names: provides a filtering mechanism
Indentical: switch to show which parameter sets have the same set of parameters and are thus a cause of errors

I just noticed a potential bug in the -syntax property of Get-Command. So far, I've only notived this on Rename-Many, but it might be prevalent on other commands; the inconsistency of the type psotion may also be related to this issue. This was made more noticeable by the fact that Get-ParameterSetInfo colours its output:

If you look closely, you'll notice the following points

The type info for each parameter in the syntax statement is inconsistently displayed. For 'Pattern' its of the form [-Param ] (because Get-ParameterSetInfo fixes it) but with gcm its [-Param] <type>; ie the type is outside of the square brackets
The syntax statement shows that Pattern is optional (its written in white and it is surrounded in square brackets. However, this is wrong because for this particular parameter set, Pattern is mandatory. This is confirmed by the contets of the table displayed. In it, Pattern is highlighted in red and the Mandatory flag is set

If you run:

gcm Rename-Many -syntax

you'll see this:
λ gcm rename-many -Syntax

Rename-Many [-Pattern] <array> -underscore <FileSystemInfo> [-Copy <array>] [-With <string>] [-Start] [-End] [-Paste <string>] [-Drop <string>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Rename-Many [-Pattern] <array> -underscore <FileSystemInfo> -End [-With <string>] [-Paste <string>] [-Drop <string>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Rename-Many [-Pattern] <array> -underscore <FileSystemInfo> -Start [-With <string>] [-Paste <string>] [-Drop <string>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Rename-Many [-Pattern] <array> -underscore <FileSystemInfo> -Anchor <array> [-Relation <string>] [-Copy <array>] [-With <string>] [-Paste <string>] [-Drop <string>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Rename-Many -underscore <FileSystemInfo> -Append <string> [-Copy <array>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Rename-Many -underscore <FileSystemInfo> -Prepend <string> [-Copy <array>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

But the problem with this is that these parameter sets are not decorated with the parameter name; hence me building these parameter sets tools.
If you run Get-CommandDetails (only the 'ReplaceWith' parameter set is shown):

'Rename-Many' | Get-CommandDetail

Parameter Set: ReplaceWith (Default)

Syntax: Rename-Many [-Pattern] <array> -underscore <FileSystemInfo> [-Copy <array>] [-With <string>] [-Start] [-End] [-Paste <string>] [-Drop <string>] [-File] [-Directory] [-Except <string>] [-Include <string>] [-Whole <string>] [-Condition <scriptblock>] [-Top <int>] [-Transform <scriptblock>] [-Context <psobject>] [-Diagnose] [-WhatIf] [-Confirm] [<CommonParameters>]

Parameters:
Name       Type            Mandatory   Pos PipeValue PipeName Alias
----       ----            ---------   --- --------- -------- -----
Pattern    Array                True     0     False    False
underscore FileSystemInfo       True named      True    False
Copy       Array               False named     False    False
With       String              False named     False    False
Start      SwitchParameter     False named     False    False
End        SwitchParameter     False named     False    False
Paste      String              False named     False    False
Drop       String              False named     False    False
File       SwitchParameter     False named     False    False
Directory  SwitchParameter     False named     False    False
Except     String              False named     False    False x
Include    String              False named     False    False i
Whole      String              False named     False    False
Condition  ScriptBlock         False named     False    False
Top        Int32               False named     False    False
Transform  ScriptBlock         False named     False    False
Context    PSObject            False named     False    False
Diagnose   SwitchParameter     False named     False    False

you can see the errorneous way that Pattern is displayed:
 [-Pattern] <array>

ie the type is outside of the square brackets.
Get-ParameterSetInfo recognises this and repairs the output.

Argh, Get-ParameterSetInfo should be called Show-ParameterSetInfo.


How do I animate a line chart?

After updating the state with new data I would like to animate a line chart.
I tried to do something like this:
data={
    [
      {x: new Animated.Value(0), y: 10},
      {x: new Animated.Value(3), y: 15},
      {x: new Animated.Value(6), y: 9},
    ]
  }

But x is expected to be a number. How can I accomplish this?

Did you manage to find a solution? I'm having the same problem

Unfortunately not, I am still wondering how to do it

I ended up moving to victory native charts. It's a much better project that was animation support and customization.

With so many open issues, I have decided to go with react-native-gifted-charts.
It comes with built-in animation support.
The link below has a gallery of many chart previews:
Gallery


Remove Endpoint from Client.request

Fixes #712

Codecov Report

Merging #714 (8c2c50f) into main (ad64f9d) will decrease coverage by 0.05%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #714      +/-   ##
==========================================
- Coverage   48.15%   48.10%   -0.06%
==========================================
  Files          68       68
  Lines        2033     2031       -2
  Branches       48       51       +3
==========================================
- Hits          979      977       -2
  Misses       1054     1054

Impacted Files
Coverage Δ

zio-http/src/main/scala/zhttp/service/Client.scala
69.38% <50.00%> (-1.21%)
:arrow_down:

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 ad64f9d...8c2c50f. Read the comment docs.


deps: staticcheck

Closing this, see my comment here:

https://github.com/perkeep/perkeep/pull/1405#issuecomment-894727096


chore: move to kyverno/kuttl

This PR moves to kyverno/kuttl.

@eddycharly Does this mean Kyverno is moving to a more permanent fork of kuttl? I hope we can sort this out over time! WDYT about moving kuttl out of kudobuilder? The kudobuilder ecosystem seems quite abandoned. WDYT @kensipe

@erikgb I hope we can continue to contribute to kuttl but for now we need to move forward so we decided to go with a fork.

@erikgb I hope we can continue to contribute to kuttl but for now we need to move forward so we decided to go with a fork.

I totally understand the why, but hope we can "join forces" for the future.

Definitely, if you have suggestions to make contributing easier let us know. Currently, the lack of maintainers make it difficult to propose changes, get feedback, etc...


Too Low HOTA and IDF1

Hi, I'm trying to test on my custom dataset.
However, HOTA and IDF1 scores are too low.
What do you think I should check?

Thanks.

There is not enough information here to help sorry. Why do you think they are too low?

Hi, I'm trying to test on my custom dataset. However, HOTA and IDF1 scores are too low. What do you think I should check?

Thanks.

Hello, may I ask if you have solved this problem

Hi, I'm trying to test on my custom dataset. However, HOTA and IDF1 scores are too low. What do you think I should check?

Thanks.

Hello, may I ask if you have solved this problem

Hi. I found that the annotations are not perfect. So there was the mismatch between detections and ground truths. I think these low scores are normal.


Could be nice to add direct calling to the REPLs

First of all, it's really nice to have a terminal running down on the screen. However, it would be really convenient if I can directly open the REPLs (python, ruby, etc.), like SublimeRepl, and not have to type the instruction into the terminal itself.

This can be done in one of two ways in the Terminal-Plus settings:

Change Shell Override to the REPL of your choice.
or
Change Shell Arguments to the REPL of your choice, assuming the REPL exists in your PATH directories.

Did this work for you? If so, please close the issue.

@jeremyramin Sorry if I wasn't clear, I actually meant to open each with their unique hotkey or something.
For example, alt+p opens python, alt+r opens ruby, alt+c opens normal console etc..
So I'm wondering if there is a way to assign the open console/terminal commands to different key sequences, and pass args to that command so that opens the indicated repl.

@zmLGBBM Ohh okay. Sorry, I did misunderstand. That feature does not currently exist and would have to be added. I could see this being implemented in several ways:

Add a configuration file for custom buttons in the toolbar that run custom commands. (This would let you program the REPLs you want and let you put commands you use frequently in the top bar.)

Add a configuration file for custom Atom commands such as terminal-plus:python that would run the commands you specify (python in this case). You would have to add 'alt+p' : 'terminal-plus:python' to your keymap.

I think these could be done but I would have to give them a try and see what limitations I would run into.

@jeremyramin Great! I think some of the sublime repl plug-in's features are really cool, but don't know if they can be implemented in atom.
Thanks for the package btw!

@zmLGBBM I'll take a look at sublime repl before I get around to adding this to get a better idea of what you are looking for.
No problem, I'm glad you like it.

@jeremyramin I've tried the Shell Override setting but it does not work.
To reproduce this:

Go to Terminal-Plus settings and type C:\Windows\System32\cmd.exe in Shell Override.
Try to open terminal by click the little + button in the bottom left corner.
Open Developer tools, and here's the error it reported:

Unable to start terminal process. Win32 error code: 267 Error: Unable to start terminal process. Win32 error code: 267
    at Error (native)
    at Server.<anonymous> (C:\Users\Clive\.atom\packages\terminal-plus\node_modules\pty.js\lib\pty_win.js:65:9)
    at emitOne (events.js:77:13)
    at Server.emit (events.js:169:7)
    at Pipe.onconnection (net.js:1404:8)

And it seems not helpful to clear the override setting, it would still produce such error. The only way to solve this is to re-install Terminal-Plus.

That is not good. This could be a result of the recent update to winpty. I will have to try this out and figure out what is wrong.

I too experienced this issue on my first launch and then it went away. I'm not sure how to get it back, but I hope this does not happen for everyone...

@jeremyramin Well, it kinda work now. Really strange problem.

@zmLGBBM Have you seen it again or only that time?

@jeremyramin I think I find out what caused this bug. It's because invalid (to os, but not atom) folder were opened.
To reproduce this:

Open a fresh atom, with no file or folder opened
Open Settings panel
Make sure the title of the window says: "atom://config - atom"
Try to open Terminal-plus
Boom, you got a bug ;)

@zmLGBBM Oh you're right, that is probably it. I'll have to filter out this path when setting the present working directory. I believe this puts Linux and Mac users at the root of their system since it is an invalid path. For Windows, it causes that 267 error. Thank you!

@jeremyramin No problem, happy to help with this amazing package!

v0.10.0 should fix the 267 error code.

I don't know if this can be done but be able to write commands directly could be the solution. For example:
terminal-plus:insert-text->command to run

This way you would be able to create toolbar buttons  or keyboard shortcuts for anything.
This is my particular case. I want to run a C# compiler from a toolbar button:
  {
    type: "button"
    iconset: "ion"
    icon: "bug"
    callback: ["terminal-plus:insert-text->mcs myproject.csproj"]
    tooltip: "Build C# project"
    show: "C#"
  }

@Zeioth I will probably have something along those lines with a CSON file. I'm still deciding the best way to implement this. What I have so far:
compileProject:
  trigger: "terminal-plus:compile-project"
  command: "mcs myproject.csproj"
  button:
    icon: "sync"
    tooltip: "Build C# project"
    order: 0
  languages: ["C#"]

The button would be an optional field. This would let people define an atom-command and/or a button for their custom commands.
I see that you had a iconset field. Iconsets would mean I would have to include the major icon sets with my project. I do not have a stance on that, but that seems like I would have to package a number of iconsets. I will probably just use GitHub's Octicons since that is built into Atom.
I also see you have a show field. Would that mean the button would only show if the current textEditor language is C#? I did not think of that but it is a good idea. There could be an optional languages field. The only problem I see with this is the possible lag this would introduce with having to go through all of the defined buttons to see which ones need to be placed. Keep in mind, this would run every time the active pane item changes.

The code I pasted is from flex-tool-bar package. This package is a configuration tool for tool-bar and is already prepared to use some predefined iconsets, so you don't have to worry about it.

Oh okay. Then in your case you would just leave the button and languages option out and define the custom atom-command. Then in the flex-tool-bar package you could just use the command you defined in Terminal-Plus (i.e. terminal-plus:compile-project).

Yes that is. I think this is the only way atom allows to do it.  It would be cool to be able to do something like.
'''
terminal-plus:run-command -> string
'''
Where string is the command you want to directly run.
'''
terminal-plus:run-command -> string
'''
Where the string is the command you want to run.

Unfortunately you are right. Atom does not allow for me to specify input for the package commands. Though I already have an idea for writing this that would not be too bad as far as implementing the every command goes. I will not know if this is even possible until I get a chance to write it out.

@jeremyramin @Zeioth Well, since atom would not let you to specify input for commands, why not just have a config file in the root of the project you're working on, and when you run terminal-plus: run-command it will read that config file and give you a list of things that can be done, just like how Visual Studio Code does with tasks.
I think that should be better since now you don't have to define global commands for every only project-wide commands that might lead to a very long and not related commands showed in specific project. Now when people tell terminal-plus to run tasks, it will get the config file and list possible tasks for user to choose.
Example of terminal-plus.config:
compileProject:
  trigger: "terminal-plus:run-command"
  command: "mcs ./myproject.csproj"
  description:
    icon: "task"
    name: "Build MyProject"
    order: 0

@zmLGBBM So basically implement one command that displays the possible defined tasks for a project? I guess that could work but that would draw away from your request for letting you define alt+p as let's say exec python.
Either way, you make a valid point that every project has a different file to compile and therefore a different command to run. I could see there being a global config file and a project config file. I wish I could offer the idea to the rest of the user group and find out what they think. I'd like some more input on this.

@jeremyramin Well, that should, but can be solve with this too. It's a great idea to have a terminal-plus.global.cson in the user config directory that holds all global tasks that a user would like to run, and a terminal-plus.project.cson in the root of each project.
Like in global.cson:
pythonRepl:             // this is how to run python repl directly
  trigger: "terminal-plus:run-command"
  command: "/path/to/python"
  shell: true              // that means this would offer a terminal plus shell
  key-map: "alt+p"   // it would be nice to have a hotkey defined, but not necessory
  description:
    icon: "python"
    name: "Run Python"
    order: 0

And now in project.cson:
compileProject:
  trigger: "terminal-plus:run-command"
  command: "mcs ./myproject.csproj"
  shell: false                   // this will only give the output of such command but not a shell
  description:
    icon: "task"
    name: "Build MyProject"
    order: 0

This two settings things works well with Sublime users, since it gives user the chance to have different settings in each of the projects. But it would be nicer to have a opinion about how atom users think of it.
PS: The best way to implement this kind of functionality is definitely not in terminal-plus. I do think one package should be focused on what is it offer. You might want to implement this terminal-plus: run-command command and start a new package, something like a Tasks Manager, to let user call that command within it (from trigger section). And that trigger section would not have to be the terminal-plus command, it can be any command that atom supports.


alienfx-config.cmd

Win 11, M18 R2, AFX version 8.6.1, alienfx-config.cmd date<PHONE_NUMBER>59

I ran alienfx-config.cmd:

D:\Program Files\Utilities\AlienFXTools>alienfx-config.cmd backup Usage: alienfx-config [command] Commands are:         backup  - save registry config to files. 4 config file will be created into current folder.         restore - load config files back to registry.         delete  - remove all tools configuration from registry. Doing backup... The operation completed successfully. The operation completed successfully. ERROR: The system was unable to find the specified registry key or value. ERROR: The system was unable to find the specified registry key or value. Done.

In my D:\Program Files\Utilities\AlienFXTools directory, I have these two new files:

afx-fans.reg
afx-lights.reg
Contents of afx-fans.reg:
`Windows Registry Editor Version 5.00
[HKEY_CURRENT_USER\Software\Alienfan]
"StartAtBoot"=dword:00000001
"StartMinimized"=dword:00000001
"LastPowerStage"=dword:00000000
"UpdateCheck"=dword:00000000
"DisableAWCC"=dword:00000000
"KeyboardShortcut"=dword:00000000
"KeepSystemMode"=dword:00000001
"DPTF"=dword:00000000
"PollingRate"=dword:00000535
"Boost-0"=hex:65,69,10
"Boost-1"=hex:65,67,23
"Boost-2"=hex:65,c5,0f
"Boost-3"=hex:65,4e,1a
[HKEY_CURRENT_USER\Software\Alienfan\Powers]
"Power-0"="Manual"
"Power-160"="Level 160"
"Power-161"="Level 161"
"Power-162"="Level 162"
"Power-163"="Level 163"
"Power-164"="Level 164"
[HKEY_CURRENT_USER\Software\Alienfan\Sensors]
"Fan-0-257"=hex:00,00,23,00,23,05,29,05,29,0a,33,0a,41,28,4b,64,64,64
"Fan-1-257"=hex:00,00,23,00,23,05,29,05,29,0a,33,0a,41,28,4b,64,64,64
"Fan-2-262"=hex:00,00,23,00,23,05,2a,05,2a,0a,33,0a,3c,28,46,64,64,64
"Fan-3-262"=hex:00,00,23,00,23,05,2a,05,2a,0a,33,0a,3c,28,46,64,64,64
"SensorName-0-0"="ESIF sensor 1"
"SensorName-0-1"="ESIF sensor 2"
"SensorName-0-2"="ESIF sensor 3"
"SensorName-0-3"="ESIF sensor 4"
"SensorName-0-4"="ESIF sensor 5"
"SensorName-0-5"="ESIF sensor 6"
"SensorName-0-6"="ESIF sensor 7"
"SensorName-0-7"="ESIF sensor 8"
"SensorName-0-8"="ESIF sensor 9"
"SensorName-1-1"="CPU Internal Thermistor"
"SensorName-1-6"="GPU Internal Thermistor"
`
Contents of afx-lights.reg (no configuration for these):
`Windows Registry Editor Version 5.00
[HKEY_CURRENT_USER\Software\Alienfx_SDK]
`

If these are the only two files I should have given my configuation, then eliminate the two error messages. If not, I will be happy to help discern the issue.

I will update to the just released <IP_ADDRESS>. As expected, I see that the alienfx-config.cmd file did not change date. So no need to retest.

It should be 4 files, but seems like you never run FX and Mon. So errors are OK in this case.
Interesting. EMPTY SDK config. So you don't define lights.
cmd is a simple script to dump/remove/load registry configuration. So it not changes most of the time.

The only suspicious is empty SDK record. The rest are ok.

Error messages to the end user are not OK if they are not actually errors. :-) I did not thing wrong, and it is not an error to have not ever run those two apps. Simply check for the condition and skip extract if not applicable. And make sure the user knows that there should only be as many files as used applications. Poof, done. ;-P
Correct, I have not used lighting on this laptop since to my knowledge my M18 R2 is not supported. And I have not had a chance to collect the information for you.
P.S. Stupid github page won't let me remove the emoji I put on your response by mistake.


Fix pre-verify check

This appears to be a typo that is causing pre_verify to return None when it should be returning OK

@dbluhm - how did that manifest?  We're trying to decide if we need an immediate release because of this.  Thanks!

Working through our toolbox demos and I tried a presentation with only a predicate on an age attribute. It showed up as this error:

Sorry to hear about that.  We need to figure out how to prevent that in the future.
We want to have trust in the updates we release.  Something for us to ponder.

No worries; thanks for responding quickly and getting a release out!