[Bug] Following the "Integrating with existing projects" in the docs results in broken binary for MacOS

Describe the bug
Following the https://cjycode.com/flutter_rust_bridge/integrate.html#integrating-with-existing-projects section, I am getting a binary that fails to run:
[  +20 ms] dyld[88711]: Library not loaded: '/usr/local/lib/rust.dylib'
[        ]   Referenced from: '/Users/eran/Projects/mc/ledger-py/flutter-signer/flutter_signer_app/build/macos/Build/Products/Debug/flutter_signer_app.app/Contents/MacOS/flutter_signer_app'
[        ]   Reason: tried: '/usr/local/lib/rust.dylib' (no such file), '/usr/lib/rust.dylib' (no such file)
[   +2 ms] Error waiting for a debug connection: The log reader stopped unexpectedly, or never started.
[        ] Error launching application on macOS.
[        ] "flutter run" took 13,235ms.

(My crate is named rust).
Codegen logs with RUST_LOG=debug environment variable
See https://gist.github.com/eranrund/5fe6476f75d529e54f5aecdb6536365c

To Reproduce
Start with a simple Flutter project and go through the steps in https://cjycode.com/flutter_rust_bridge/integrate.html#integrating-with-existing-projects
I am on an M1 Mac running Monterey 12.6
Expected behavior
No response
Generated binding code
bridge_generated.h:

#include <stdbool.h>
#include <stdint.h>
#include <stdlib.h>

typedef int64_t DartPort;

typedef bool (*DartPostCObjectFnType)(DartPort port_id, void *message);

typedef struct WireSyncReturnStruct {
  uint8_t *ptr;
  int32_t len;
  bool success;
} WireSyncReturnStruct;

void store_dart_post_cobject(DartPostCObjectFnType ptr);

void wire_greet(int64_t port_);

void free_WireSyncReturnStruct(struct WireSyncReturnStruct val);

static int64_t dummy_method_to_enforce_bundling(void) {
    int64_t dummy_var = 0;
    dummy_var ^= ((int64_t) (void*) wire_greet);
    dummy_var ^= ((int64_t) (void*) free_WireSyncReturnStruct);
    dummy_var ^= ((int64_t) (void*) store_dart_post_cobject);
    return dummy_var;
}

bridge_generated.dart:
// AUTO GENERATED FILE, DO NOT EDIT.
// Generated by `flutter_rust_bridge`@ 1.51.0.
// ignore_for_file: non_constant_identifier_names, unused_element, duplicate_ignore, directives_ordering, curly_braces_in_flow_control_structures, unnecessary_lambdas, slash_for_doc_comments, prefer_const_literals_to_create_immutables, implicit_dynamic_list_literal, duplicate_import, unused_import, prefer_single_quotes, prefer_const_constructors, use_super_parameters, always_use_package_imports, annotate_overrides, invalid_use_of_protected_member, constant_identifier_names, invalid_use_of_internal_member

import 'dart:convert';
import 'dart:async';
import 'package:flutter_rust_bridge/flutter_rust_bridge.dart';

import 'package:meta/meta.dart';
import 'package:meta/meta.dart';
import 'dart:ffi' as ffi;

abstract class Rust {
  Future<String> greet({dynamic hint});

  FlutterRustBridgeTaskConstMeta get kGreetConstMeta;
}

class RustImpl implements Rust {
  final RustPlatform _platform;
  factory RustImpl(ExternalLibrary dylib) => RustImpl.raw(RustPlatform(dylib));

  /// Only valid on web/WASM platforms.
  factory RustImpl.wasm(FutureOr<WasmModule> module) =>
      RustImpl(module as ExternalLibrary);
  RustImpl.raw(this._platform);
  Future<String> greet({dynamic hint}) {
    return _platform.executeNormal(FlutterRustBridgeTask(
      callFfi: (port_) => _platform.inner.wire_greet(port_),
      parseSuccessData: _wire2api_String,
      constMeta: kGreetConstMeta,
      argValues: [],
      hint: hint,
    ));
  }

  FlutterRustBridgeTaskConstMeta get kGreetConstMeta =>
      const FlutterRustBridgeTaskConstMeta(
        debugName: "greet",
        argNames: [],
      );

// Section: wire2api

  String _wire2api_String(dynamic raw) {
    return raw as String;
  }

  int _wire2api_u8(dynamic raw) {
    return raw as int;
  }

  Uint8List _wire2api_uint_8_list(dynamic raw) {
    return raw as Uint8List;
  }
}

// Section: api2wire

// Section: finalizer

class RustPlatform extends FlutterRustBridgeBase<RustWire> {
  RustPlatform(ffi.DynamicLibrary dylib) : super(RustWire(dylib));
// Section: api2wire

// Section: finalizer

// Section: api_fill_to_wire

}

// ignore_for_file: camel_case_types, non_constant_identifier_names, avoid_positional_boolean_parameters, annotate_overrides, constant_identifier_names

// AUTO GENERATED FILE, DO NOT EDIT.
//
// Generated by `package:ffigen`.

/// generated by flutter_rust_bridge
class RustWire implements FlutterRustBridgeWireBase {
  /// Holds the symbol lookup function.
  final ffi.Pointer<T> Function<T extends ffi.NativeType>(String symbolName)
      _lookup;

  /// The symbols are looked up in [dynamicLibrary].
  RustWire(ffi.DynamicLibrary dynamicLibrary) : _lookup = dynamicLibrary.lookup;

  /// The symbols are looked up with [lookup].
  RustWire.fromLookup(
      ffi.Pointer<T> Function<T extends ffi.NativeType>(String symbolName)
          lookup)
      : _lookup = lookup;

  void store_dart_post_cobject(
    DartPostCObjectFnType ptr,
  ) {
    return _store_dart_post_cobject(
      ptr,
    );
  }

  late final _store_dart_post_cobjectPtr =
      _lookup<ffi.NativeFunction<ffi.Void Function(DartPostCObjectFnType)>>(
          'store_dart_post_cobject');
  late final _store_dart_post_cobject = _store_dart_post_cobjectPtr
      .asFunction<void Function(DartPostCObjectFnType)>();

  void wire_greet(
    int port_,
  ) {
    return _wire_greet(
      port_,
    );
  }

  late final _wire_greetPtr =
      _lookup<ffi.NativeFunction<ffi.Void Function(ffi.Int64)>>('wire_greet');
  late final _wire_greet = _wire_greetPtr.asFunction<void Function(int)>();

  void free_WireSyncReturnStruct(
    WireSyncReturnStruct val,
  ) {
    return _free_WireSyncReturnStruct(
      val,
    );
  }

  late final _free_WireSyncReturnStructPtr =
      _lookup<ffi.NativeFunction<ffi.Void Function(WireSyncReturnStruct)>>(
          'free_WireSyncReturnStruct');
  late final _free_WireSyncReturnStruct = _free_WireSyncReturnStructPtr
      .asFunction<void Function(WireSyncReturnStruct)>();
}

typedef DartPostCObjectFnType = ffi.Pointer<
    ffi.NativeFunction<ffi.Bool Function(DartPort, ffi.Pointer<ffi.Void>)>>;
typedef DartPort = ffi.Int64;

### OS

MacOS 12.6

### Version of `flutter_rust_bridge_codegen`

1.51.0

### Flutter info

```shell
[✓] Flutter (Channel stable, 3.3.9, on macOS 12.6 21G115 darwin-arm, locale en-US)
    • Flutter version 3.3.9 on channel stable at /Users/eran/Projects/mc/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b8f7f1f986 (8 days ago), 2022-11-23 06:43:51 +0900
    • Engine revision 8f2221fbef
    • Dart version 2.18.5
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/eran/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13E113
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • 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 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.1.4)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • 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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 107.0.5304.121

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

• No issues found!

Version of clang++
Apple clang version 13.1.6 (clang-13<IP_ADDRESS>), Target: arm64-apple-darwin21.6.0
Version of ffigen
No response
Additional context
No response

It seems like linking with the static library and not the dylib fixes this issue.

Is this related or not?
https://github.com/fzyzcjy/flutter_rust_bridge/issues/754
https://github.com/fzyzcjy/flutter_rust_bridge/issues/815

Honestly I am not sure. But it appears that just using the static library works so maybe the instructions need to change to reflect that.

Honestly I am not sure. But it appears that just using the static library works so maybe the instructions need to change to reflect that.

Agreed, since I have had several issues relating to dynamic linking on MacOS at the template as well. I'm still preoccupied during this time of year but I might take a look later this month if nobody's put in a PR yet.

I am facing the same problem (M1 macOS Ventura 13.0.1) and I only want to add that the application runs if I open XCode by double clicking on macos/Runner.xcworkspace (not Runner/xcodeproj) - and build and run the application from XCode. But of course it should work via "flutter run".
THANK YOU VERY MUCH FOR FLUTTER_RUST_BRIDGE. It's awesome (once it is set-up correctly).

the same problem with me.
os: M1 macOS Ventura 13.1
flutter:  3.3.10
the problem is when run command flutter run -d macos  it well tell you Library not loaded: '/usr/local/lib/xxx.dylib.
but if you open Xcode and click run button, then everything is ok.
and i found solution in #851
copy rust-> flutter_rust_bridge_example.xcodeproj-> project.pbxproj from the with_flutter to own project, and replace the lib name inside.
it's work!!
I think maybe it's a problem with cargo xcode version?

It turns out that cargo-xcode 1.4.3 works but 1.5.0 fails.
Reinstall cargo-code with cargo install<EMAIL_ADDRESS>does work.

I found out that in order to use cargo-xcode 1.5.0 you need to add DYLIB_INSTALL_NAME_BASE = "$(TARGET_BUILD_DIR)"; to buildSettings in your $crate.xcodeproj -> project.pbxproj file.
For example:
CARGO_TARGET_DIR = "$(PROJECT_TEMP_DIR)/cargo_target"; /* for cargo */
CARGO_XCODE_FEATURES = ""; /* configure yourself */
DYLIB_INSTALL_NAME_BASE = "$(TARGET_BUILD_DIR)";

Thank you @fl1pflops, this saved me from pulling out my hair

For further readers, this may be related: https://github.com/fzyzcjy/flutter_rust_bridge/pull/1227


Also it would be nice if there was an example of a desk toy application (like the old NEKO.EXE).

Solution

Created an example with googly eyed Bevy logo under examples/games/desktoy.rs.

Changelog

Added "Desk Toy" game example showcasing window transparency and hit test.

In case it wasn't clear: I genuinely think we should include this. This is a nice well-scoped example demonstrating a new flavor of app, and I think that there's real pedagogical and marketing value to the fact that it's also extremely funny.

I think this should be named desk_toy.rs.

I tried making the primary window fullscreen but macOS puts it into its own
workspace (even if it’s borderless).
On Wed, 6 Mar 2024 at 7:45 PM SpecificProtagonist @.***>
wrote:

Nice! Testing this on Cinnamon, there are some problems with desktop mode
though:

Pressing space triggers the toggle every frame. This isn't this PR's
fault though; looks like #12273
https://github.com/bevyengine/bevy/issues/12273
Clicking the logo doesn't focus the window (also not this PR's fault)
the window size stays the same instead of being full-screened, so I
can only move the logo in the window's bounds

—
Reply to this email directly, view it on GitHub
https://github.com/bevyengine/bevy/pull/12331#issuecomment-1981317646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPHHM6TBSYWYCMIDMM6PODYW5B4HAVCNFSM6AAAAABEIHREZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBRGMYTONRUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

@tjamaan ping me once you've tweaked the physics and I'll merge this in :)

cargo fmt is failing :)

cargo fmt is failing :)

I have automatic format on save configured and I also ran cargo fmt manually. Can you share the error?

@alice-i-cecile I've toned down the bounciness, cleaned up the code as suggested, and renamed the file to desk_toy.rs


hash-slinger: 3.1 -> 3.2

Description of changes
https://github.com/letoams/hash-slinger/releases/tag/3.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 177553 run on x86_64-linux 1

1 package built:

hash-slinger


Change transition for mobile at element level

This is:

Question

Wondering what the property is to change animation direction at an element level (e.g., div) using media queries.
For example:
@media screen and (max-width: 800px) {
.element {
data-aos="fade-up"
}
}

You can use an attribute CSS selector to get this done!
Let's say you wanted to select all .element elements with the data-aos="fade-up" property. You would use the following rule.
@media screen and (max-width: 800px) {
.element[data-aos="fade-up"] {
/* Insert changes here */
}

You may also need to target specifically once they're finished animating, so you'd use something like this:
@media screen and (max-width: 800px) {
.element.aos-animate[data-aos="fade-up"] {
/* Insert changes here */
}

That doesn't seem to do what I need.
For example, if I have elements that are set to fade up and want to change to fade-left via the media query, I can't do that:
@media screen and (max-width: 800px) { .element[data-aos="fade-up"] { data-aos="fade-left" } }

You can add like this-
If you want to change fade up to down
Example-
@media screen and (max-width: 800px) {
.element.[data-aos="fade-up"] {
transform: translateY(50px);
}


Create validation rule for rejecting charge updates with effective date AFTER stop date (ac6)

As gaps are not allowed in a Charge's timeline, hence when a charge is stopped, any attempts to update it with an effective date after the stop date, must be rejected.

Acceptance criteria

[x] 1 Given a charge is stopped
When the owner submits a request containing the same charge but with an effective date after the registered stop date
Then the request is rejected.

[x] 2 The rejection message received by the market participant contains error code D14 and text
"Charge ID <senderProvidedChargeId> has been stopped and thus cannot be updated as per <startDateTime>."

[x] 3 Update validations.md in GitHub with this new rule.

Please use thise reasoncode:

Given the persisted state of the charge is stopped 31-12-2021

When we try to update the charge 01-01-2022

Then we will get the following error presented


Update TransferPokemon to use new Inventory Class

Short Description:
Use the new Inventory Class for Transfer Pokemon task

This looks good to me, @douglascamata can you review too please?
:+1:

@TheSavior seems ok. @achretien did you test it very well? This is a critical task.

I did but feel free to ask more testers on Slack.

@achretien we need a dry mode to test, otherwise ppl will be afraid, lol

Should I not release pokemon if test mode enable for exemple ?

@achretien I think that's great for testing such a critical feature.

added the test mode

👍
LGTM


Unable to use binaries option to sign additional binaries

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Packager Version:

Not relevant

Electron Version:

Not relevant

Operating System:

Not relevant

Last Known Working Electron Packager version::

Not relevant

Expected Behavior
I should be able to use binaries in osxSign to sign additional binaries.
Actual Behavior
The TypeScript definition explicitly removes the binaries key using Omit which results in this error:
forge.config.ts:15:7 - error TS2322: Type '{ binaries: string[]; optionsForFile: () => { entitlements: string; }; }' is not assignable to type 'true | OsxSignOptions'.
  Object literal may only specify known properties, and 'binaries' does not exist in type 'OsxSignOptions'.

To Reproduce
const config: ForgeConfig = {
  ...
  packagerConfig: {
    osxSign: {
      binaries: ["./path/to/extra/binary"],
    },
  },
};

Additional Information
binaries has been omitted since the OsxSignOptions type was introduced in a2a3ae559d402a0ce36a00cd0e9ceab6bcaba2db

Up. The type explicitly omits the binaries:

In the other place the docs say something different:

I can't find it in the code but is binaries stripped somewhere? Even if I defined that array, I see in the logs (running DEBUG=electron-osx-sign* yarn electron-forge make --arch=universal) that it is not supplied:
2024-07-18T15:02:12.807Z electron-osx-sign Signing application...
 > Application: /var/folders/6g/_ypptv3n2fs80_zc0xz8p1d40000gn/T/electron-packager/tmp-FbK8DD/[REDACTED].app
 > Platform: darwin
 > Additional binaries: undefined <--- HERE
 > Identity: ***

@voldemortensen Did you have any luck with this?

For posterity.
I had problems with node-hid binaries. After a long investigation I found they were not signed and as shown on the screenshot from logs above, specifying osxSign.binaries didn't work. Somehow binaries are not passed to @electron/osx-sign as the type suggests. I did try to find where it happens in the source code but failed.
Anyway, I was able to sign the native binaries during the afterCopy hook:
const forgeConfig = {
  packagerConfig: {
    afterCopy: [
      (buildPath, electronVersion, platform, arch, callback) => {
        try {
          console.log('Build path:', buildPath);

          // Only run signing on macOS
          if (platform === 'darwin') {
            const binaries = [];

            if (arch === 'x64' || arch === 'universal') {
              binaries.push(
                path.join(
                  buildPath,
                  'node_modules/node-hid/bin/darwin-x64-123/node-hid.node',
                ),
                path.join(
                  buildPath,
                  'node_modules/node-hid/prebuilds/HID-darwin-x64/node-napi-v3.node',
                ),
              );
            }

            if (arch === 'arm64' || arch === 'universal') {
              binaries.push(
                path.join(
                  buildPath,
                  'node_modules/node-hid/bin/darwin-arm64-123/node-hid.node',
                ),
                path.join(
                  buildPath,
                  'node_modules/node-hid/prebuilds/HID-darwin-arm64/node-napi-v3.node',
                ),
              );
            }

            binaries.push(
              path.join(
                buildPath,
                'node_modules/node-hid/build/Release/HID.node',
              ),
            );

            binaries.forEach((binaryPath) => {
              if (fs.existsSync(binaryPath)) {
                console.log(`Signing binary: ${binaryPath}`);
                execSync(
                  `codesign --deep --force --verbose --sign "${process.env.SIGNING_IDENTITY}" "${binaryPath}"`,
                  {
                    stdio: 'inherit',
                  },
                );
              } else {
                console.error(`Binary not found: ${binaryPath}`);
              }
            });
          }

          callback();
        } catch (error) {
          callback(error);
        }
      },
    ],
  },
};

It still feels to me that I'm doing something wrong here 🙈 but at least that works. Otherwise, I was getting an error like this: "not valid for use in process: mapping process and mapped file (non-platform) have different Team IDs" while interacting with node-hid in the app.

For posterity. I had problems with node-hid binaries. After a long investigation I found they were not signed and as shown on the screenshot from logs above, specifying osxSign.binaries didn't work. Somehow binaries are not passed to @electron/osx-sign as the type suggests. I did try to find where it happens in the source code but failed.
Anyway, I was able to sign the native binaries during the afterCopy hook:
const forgeConfig = {
packagerConfig: {
afterCopy: [
(buildPath, electronVersion, platform, arch, callback) => {
try {
console.log('Build path:', buildPath);
      // Only run signing on macOS
      if (platform === 'darwin') {
        const binaries = [];

        if (arch === 'x64' || arch === 'universal') {
          binaries.push(
            path.join(
              buildPath,
              'node_modules/node-hid/bin/darwin-x64-123/node-hid.node',
            ),
            path.join(
              buildPath,
              'node_modules/node-hid/prebuilds/HID-darwin-x64/node-napi-v3.node',
            ),
          );
        }

        if (arch === 'arm64' || arch === 'universal') {
          binaries.push(
            path.join(
              buildPath,
              'node_modules/node-hid/bin/darwin-arm64-123/node-hid.node',
            ),
            path.join(
              buildPath,
              'node_modules/node-hid/prebuilds/HID-darwin-arm64/node-napi-v3.node',
            ),
          );
        }

        binaries.push(
          path.join(
            buildPath,
            'node_modules/node-hid/build/Release/HID.node',
          ),
        );

        binaries.forEach((binaryPath) => {
          if (fs.existsSync(binaryPath)) {
            console.log(`Signing binary: ${binaryPath}`);
            execSync(
              `codesign --deep --force --verbose --sign "${process.env.SIGNING_IDENTITY}" "${binaryPath}"`,
              {
                stdio: 'inherit',
              },
            );
          } else {
            console.error(`Binary not found: ${binaryPath}`);
          }
        });
      }

      callback();
    } catch (error) {
      callback(error);
    }
  },
],

},
};
It still feels to me that I'm doing something wrong here 🙈 but at least that works. Otherwise, I was getting an error like this:

not valid for use in process: mapping process and mapped file (non-platform) have different Team IDs

while interacting with node-hid in the app. The reality wasn't that the Team IDs were different. It's the node-hid binaries that were not signed at all.

@tniezurawski Thanks for sharing! This solution was super helpful. I agree though that it feels like there's something wrong... feels like the tool should support this feature and I'm missing some configuration.
Did you had any troubles with this approach? Did you find a better alternative?


Perl extensions in urxvt crash or freeze

I have urxvt loading some standard perl scripts for clickable links, yet when I do actually click links, urxvt appears to freeze up, and the urxvtd process in the task manager shows as suspended.
! Perl extensions
URxvt.perl-ext-common:      default,matcher
URxvt.matcher.button:       1
URxvt.urlLauncher:          links2

The same thing happens with choosing, for example, Firefox.  Is this a Perl problem, Xming problem, or WSL problem?
For the record, doing
links2 <url>
and
firefox <url>
both do correctly launch the url in the correct app.
Also, launching urxvt with the "kuake" perl extension simply quits it instantaneously.

@fpqc - maybe confirm your urxvt scenario works in April update. This one went chirping crickets two years ago.

@therealkenc Oh I have not the foggiest idea of how I had it set up at the time. Since the WSLBridge came out, I have had no interest in running a native-through-X terminal emulator (vcxsrv has weird layer-ordering problems with the Windows layering system that makes quake-style functionality seriously broken).
I also went back into academia in October, so I have a lot less time to test these things while pretending to work… 😉
I've got it on my radar. I'll try to test it on a sleepless night or something.

I've got it on my radar. I'll try to test it on a sleepless night or something.

Make it mostly "or something". There's too good a chance it works now to take the effort, and there is an even better chance you are the only one who tried the "kuake perl extension" (whatever that is) with urxvt on WSL.
The only reason I pinged you was (a) [mostly] because it was you, and (b) on the chance you knew whether urxvt perl extensions work now offhand. Closed by fiat is good enough for this one.


feat(nuxi): add quiet mode support

🔗 Linked issue
resolves #18656
resolves https://github.com/nuxt/nuxt/issues/18497
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This adds a basic quiet option that can be set in config or passed via -q or --quiet option in nuxi that will set vite logLevel to silent.
TODO:

(unrelated to this PR) investigate why we have no logging in webpack at the moment

For review:

consider whether to default always to false (vs enabling in CI/test/without TTY)

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

For next steps, as we planned when we paired, @Baroshem will be either opening a fresh PR with further changes or opening a PR to this one...

closing in favour of https://github.com/nuxt/nuxt/pull/19369


MSAL Token Issue | Error: Token Renewal Time out error

Library

[x]<EMAIL_ADDRESS>
Framework

[ ] React

Description
We  have a  SPFx(React) solution and MSAL.js v1.4.
Initial page start / authentication works however, post token expiry(1 hour) MSAL is not regenerating the token and we are getting “Token Renewal Time out error”.
Also as per implementation we have set the configuration to “LocalStorage” as then the token cache will be shared amongst the other tabs in same browser(Edge Chromium and Google Chrome).
Error Message
Token Renewal Time out error
MSAL Configuration
const msalConfig = {

            authority: 'https://login.microsoftonline.com/<tenantID>',

            clientId: '<Azure Ad App Client Id>', //PROD
            navigateToLoginRequestUrl: true,

           redirectUri: () => 'https://xx.com'
            };
          const myMSALObj= new MSAL.UserAgentApplication({
            auth:msalConfig,
            cache:{
              cacheLocation:'localStorage',
              storeAuthStateInCookie: true
            },
            system:{
                tokenRenewalOffsetSeconds:300,
                logger: new MSAL.Logger(()=>{

                },{
                    level:MSAL.LogLevel.Verbose,
                    piiLoggingEnabled:false,
                    correlationId:"1234"
                }
                )
            }
          });
          const accessTokenRequest: MSAL.AuthenticationParameters = {
            scopes: ["<Azure Ad App Client Id>/user.read", "<Azure Ad App Client Id>/user_impersonation"],
                          loginHint: this._pageContext.user.email,
             authority: "https://login.microsoftonline.com/<tenantID>"
           };
           var token = await (await myMSALObj.acquireTokenSilent(accessTokenRequest)).accessToken
           console.log('Token from Msal 1.4', token)
           localStorage.setItem('Token',token)
           console.log('Storing into localStorage',localStorage.getItem('Token'))

Reproduction steps
After 1 hour of token expiry navigating from Tab 1 to Tab 2 of same browser.
Expected behavior
The Token should generate and data should be pulled on the screen.
Identity Provider

[x] Azure AD

Browsers/Environment

[ ] Chrome
[ ] Edge

@jasonnutter
Regression

[ ] Did this behavior work before?No
Version:

Security

[ ] Is this issue security related? No

Source

[ ] Customer request

@tnorling I have provided all the information as was there in the template anything specific you are looking for do let me know?

@yoursnikhilsinha Timeouts can happen for a number of reasons. Can you start by reviewing our documentation about this particular error and let me know if you need more help?

@tnorling yeah we checked the aspect mentioned in the document which we have gone through and as you mentioned about acquireTokenPopup.We have set up redirect to a blank html page also we cant use acquireTokenPopup as the user will not put their credential again for this. Please do let me know if you have any other recommendations.

@yoursnikhilsinha Can you clarify, are 3rd party cookies disabled in your browser?

@tnorling yes it is disabled.

Unfortunately 3rd party cookies are required for the silent flow to work. Your only options are:

Enable 3rd party cookies
Call an interactive method (acquireTokenPopup or acquireTokenRedirect)
Upgrade to<EMAIL_ADDRESS>

@tnorling as you have mentioned I have enable the 3rd party cookie and will let you know the outcome. But is it really required to do item 2 as told earlier that user will not prefer to relog in after the Token Expire .And for item 3 we updated from version 1.3 to 1.4.6 how does version upgrade to 2.x help here.

@yoursnikhilsinha<EMAIL_ADDRESS>uses hidden iframes to do silent token renewal which rely on 3rd party cookies. If these are disabled the only way to renew your token is to call an interactive method.
Version 2.x uses the Auth Code Flow with PKCE which utilizes a refresh token and a POST request instead of the hidden iframe so it does not use hidden iframes until the refresh token expires (24 hours).
You can read more about the limitations of 3rd party cookie blocking and how the Auth Code Flow mitigates this here and you'll find a migration guide from version 1 to version 2 here

@tnorling I see your point to update to version 2.x but what I read in the link your shared says its for only Single Page Application but ours is a web application not SPA, so do you think this would be a limitation and I may get into same situation or something else if we uppgrade.Also are you saying that Token expiry is 24 hours with version 2.x but with previous version it is 1 hour can you please confirm both the point.

@yoursnikhilsinha

what I read in the link your shared says its for only Single Page Application but ours is a web application not SPA

Both msal 1.x and 2.x are only for Single Page Applications. What we mean by that is that your application does not have a server component. If your application does have a server component you should use a server side library for auth.

Also are you saying that Token expiry is 24 hours with version 2.x but with previous version it is 1 hour

In both 1.x and 2.x, access token expiry is 1 hour. In version 2 we also use a refresh token that has a 24 hour expiry and can be used to obtain a new 1 hr access token. Version 1 does not use a refresh token and thus is reliant on hidden iframes to renew access tokens and this scenario is broken without 3rd party cookies.

Closing, as no further action needed at this time.


removes node_modules, adds 'valid-url' dependency to .gitignore and p…

…ackage.json

Node modules folder still included......
----- Reply message -----
From: "Matt Senate"<EMAIL_ADDRESS>To: "JohnMcLear/ep_hide_referrer"<EMAIL_ADDRESS>Subject: [ep_hide_referrer] removes node_modules, adds 'valid-url' dependency to .gitignore and p… (#14)
Date: Fri, Aug 7, 2015 20:47
…ackage.json

You can view, comment on, or merge this pull request online at:
https://github.com/JohnMcLear/ep_hide_referrer/pull/14
Commit Summary

removes node_modules, adds 'valid-url' dependency to .gitignore and package.json

File Changes

M .gitignorehttps://github.com/JohnMcLear/ep_hide_referrer/pull/14/files#diff-0 (1)
M node_modules/valid-url/package.jsonhttps://github.com/JohnMcLear/ep_hide_referrer/pull/14/files#diff-1 (6)
M package.jsonhttps://github.com/JohnMcLear/ep_hide_referrer/pull/14/files#diff-2 (4)

Patch Links:

https://github.com/JohnMcLear/ep_hide_referrer/pull/14.patch
https://github.com/JohnMcLear/ep_hide_referrer/pull/14.diff

—
Reply to this email directly or view it on GitHubhttps://github.com/JohnMcLear/ep_hide_referrer/pull/14.

ping @JohnMcLear

Node modules folder still included......

@JohnMcLear: It seems that the module valid-url is more in use since https://github.com/ether/ep_hide_referrer/pull/13, this PR can be closed.


Sanitycheck device-testing & CI related enhancements

Some enhancements related to using sanitycheck for device-testing and CI systems.
The idea here is that we'd have a builder CI system that only builds what tests are runnable via (--device-testing --build-only -M).  The -M will keep the minimal artifacts that we need to allow --device-testing --test-only to function on another system that has the HW/boards connected to it.

@PerMac ping


SCIM administration of access controls

There is some work to be done to map the SCIM standard to the Medplum access control mechanisms. Rather than individual, templatized "policies" and "rules", SCIM uses the concepts of "permissions" and "groups", so there is a design and implementation exercise needed to align these.

Closing this one as "Won't Fix" as at present it seems like the SCIM concepts aren't rich enough to model what's needed for access policies.  Can revisit if there is a crisper linkage.


Deprecation Warnings in 2022.7.0 for the Number Component

What version of Music Assistant has the issue?
2022.6.10
The problem
Receiving Deprecation Warnings in the Logs
Logger: homeassistant.components.number
Source: components/number/init.py:462
Integration: Number (documentation, issues)
First occurred: 3:21:41 AM (6 occurrences)
Last logged: 3:21:41 AM
Entity None (<class 'custom_components.mass.number.CrossfadeDurationEntity'>) is using deprecated NumberEntity features which will be unsupported from Home Assistant Core 2022.10, please report it to the custom component author.
Entity None (<class 'custom_components.mass.number.VolumeNormalizationTargetEntity'>) is using deprecated NumberEntity features which will be unsupported from Home Assistant Core 2022.10, please report it to the custom component author.
How to reproduce
It just shows up in the log on Core Restart
Relevant log output
Logger: homeassistant.components.number
Source: components/number/init.py:462
Integration: Number (documentation, issues)
First occurred: 3:21:41 AM (6 occurrences)
Last logged: 3:21:41 AM
Entity None (<class 'custom_components.mass.number.CrossfadeDurationEntity'>) is using deprecated NumberEntity features which will be unsupported from Home Assistant Core 2022.10, please report it to the custom component author.
Entity None (<class 'custom_components.mass.number.VolumeNormalizationTargetEntity'>) is using deprecated NumberEntity features which will be unsupported from Home Assistant Core 2022.10, please report it to the custom component author.
Additional information
No response
What version of Home Assistant Core are your running
2022.7.0
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

This is fixed in the 2022.7 version of MA of which a first beta has been released yesterday.

Guess I should have looked in Releases... Great software BTW... Thank You!


Initial commit of telecon tool features/uses

capture some notes
play with table format a bit

@maherou if you have some time, if you could have a look at this PR and see if there are features missing you think should be included or features there you think aren't too relevant. This is based on conversation from yesterday. You can inspect the rendered markdown at this link

Here's what we're using. Feel free to take whatever you think is useful:
Capabilities

Whiteboard
Shared whiteboard
Annotations
Screen sharing
Desktop/application sharing
Recording/playback
Calendar integration
Scheduled meetings
Group chat
Individual chat
Breakout sessions
Speaker tracking
Gallery view
Video pinning
Outside users
File sharing
Reactions/polling
Virtual backgrounds
Personal meeting IDs
Meeting limit
Meeting time limit
Other

Security

Random meeting IDs
Encryption
Meeting locking
Pariticpant removal

A/V Features

Max participants
Audio quality
Video quality
Dial-in available
Dial-in linked to video feed (e.g. "Call me")

Platforms

Windows
MacOS
Linux
iOS
Android
Browser

Licensing

Basic
Intermediate
Enterprise

User experiences

Maximum number of participants successfully experienced
Maximum number of participants successfully hosted
Years of experience as user
Reliability as experienced by real users

Ease of Use

International phone numbers
Easy phone entry for participants
Easy video entry for participants
Call back support
Easy entry without individual host approval

@markcmiller86 I can help you with curated content formatting etc. That way,  you can focus on the content.

@markcmiller86, can we add a row for remote keyboard and mouse control?  That is critical for remote pair-programming.  I know that WebEx and Skype for Business supports that but I don't think that Bluejeans does.

@markcmiller86, might also be nice to add a column for Skype (for Business) since that is that is the current required solution for SNL.

The overview is nice.
One feature that may be helpful to add is how scalable a solution is, eg. is it good for upto 10 people, 50 people, 100 or more people.
It may also be helpful to include open source options such as

Jitsi
Apache Open Meetings
Big Blue Button
OpenVidu

Since WebRTC makes streaming video over the web easier than it was before, video chat applications can also work for small group meetings, as for example done in:

Nextcloud

Some of these can be readily deployed, for example see https://aws.amazon.com/marketplace/pp/Code-Creator-Jitsi-Meet-Video-Conferencing/B081QH7V6V  (link is an example not an endorsement)
Other solutions include

Eztalks
Ding Talk
Google Hangouts

@bartlettroscoe will work on Skype column. Don't work on it until you get heads up from him that he is done.

Building own solution may also be possible using for example

https://www.alibabacloud.com/product/apsaravideo-for-live
https://aws.amazon.com/solutions/live-streaming-on-aws/
https://cloud.google.com/solutions/media-entertainment/use-cases/live-streaming/
https://docs.microsoft.com/en-us/azure/media-services/latest/live-streaming-overview

@elaineraybourn found this great PDF we should somehow included either here or as an additional CC
VirtualConferences_GuideToBestPractices_CURRENT.pdf

Hey @markcmiller86, by the way, I've had several requests to share this ACM document with a broader audience--so what do you think about me creating a resource page such as https://bssw.io/items/resources-for-maximizing-remote-working for virtual conferences? Would this augment your efforts, or should I wait until your article is live to point folks to that link? I'm flexible.

@elaineraybourn ... please feel free to move foward as another resource.

@markcmiller86 sounds good. I'll work on a pointer that perhaps mention ACM as having good resources for virtual events like conferences--so slightly expanding the topic so it doesn't distract from your article. I'll work with @rinkug for guidance on how to handle.

Well, I am curious...if this is going to take a week to put in place, I guess I'd prefer we just include it in what I am putting together. I am committing some more changes to that PR this PM. Maybe its close enough to be read to host with the intention of adding to it (e.g. evolving it) as BSSw-EB decided would be appropriate.

@betterscientificsoftware/editorial-board I need the following help to complete this...

Can anyone with experience with any of these tools for the use-cases described fill in any boxes in the subjective quality section (bottom of table)? I've filled in from my experiences with Zoom (recent) and WebEx (past 4+ years).
Can anyone pick a row (or rows) to try to fill in? The way I have done this is to submit the same search query on Google using their URL search keywords but with time restricted to last month and different product names as in...

https://www.google.com/search?q=security+issues+with+webex+video+conferencing+software&as_qdr=m
https://www.google.com/search?q=security+issues+with+zoom+video+conferencing+software&as_qdr=m
https://www.google.com/search?q=security+issues+with+gotomeeting+video+conferencing+software&as_qdr=m

Oh, and, of course, I need someone to a) review what is there...there are many typos I think and maybe b) suggest additional links to point to from each sub-sub-section heading.

@markcmiller86, @rinkug, @curfman,
It occurred to me that since I have been looking into these tools recently and found out that:

MS is dropping support for Skype for Business in July 2021

Sandia will be switching from Skype for Business in to MS Teams in 2021

Skype for Business has a lot of problems and missing features compared to other tools already listed

that perhaps we should not bother providing a column in this table for Skype for Business since no one in their right mind would select Skype for Business given other choices.
But perhaps we should at least mention that Skype for Business exists and mention that it is currently being used by some institutions and that the current plan is for MS to end support for Skype for Business in July 2021.  Then we should suggest that teams that might be interested in Skype for Business should instead look at MS teams?
What do you think?

I can't recall who suggested we needed the Skype for Business column in the first place. So, its fine removing it. That said, 2021 is a long way in the future and I can envision some subset of our readership having an interest in it. All of the information you provided above can be included as product footnotes (as you can see there for other products).

I can't recall who suggested we needed the Skype for Business column in the first place.

@markcmiller86 it was me :-(

All of the information you provided above can be included as product footnotes (as you can see there for other products).

Right, but how about we just have a section that mentions other tools not included in the table and provide a few bullets about them below that in the footnotes?
After some thought, I think we should only be including columns in that table for products that we think people should seriously consider using if they have a choice.  (The issue is that some people don't really have a choice in many cases as it is dictated by their company.)

@bartlettroscoe I see you most recent commit there and read the footnootes...AFAICT that second note about browser behavior is pretty much true across the board. A browser often requires an extension or plugin installed (but that is sometimes less involved for a powerless user than installing a whole application) and typically then does not support all the sharing features.

BTW, does everyone know how to use GitHub as a WYSIWYG for markdown files?  You can do that through a separate branch in your GitHub fork and then you can be editing the Markdown file on your locally machine and then just do:
$ git commit --amend -a --no-edit \
   && git push -f rab-github mcm86-cc-teletools-4-remote-work:mcm86-cc-teletools-4-remote-work

You can see the WIP rendering at:

https://github.com/bartlettroscoe/betterscientificsoftware.github.io/blob/mcm86-cc-teletools-4-remote-work/CuratedContent/TelecomToolsForRemoteWork.md

Otherwise, I can't image the nightmare it must be trying to edit a Markdown file like this.

You can also do one of the following...

Edit the file directly on GitHub from any browser and use the Preview tab.
Use the markdown extension in Google chrome to render locally called Markdown Viewer available in the Google Chrome Store.

Use the markdown extension in Google chrome to render locally called Markdown Viewer available in the Google Chrome Store.

@markcmiller86, thanks for the tip.  It takes a little configuration but the "Mardown Viewer" Chrome extension looks pretty nice.  Thanks!

This is done and published.


Features/4.0.0

Thanks for helping out :+1:
Before submitting a pull request, please have a quick read through the contribution guidelines and provide the following information, where appropriate replace the [ ] with a [X]
The issue or feature being addressed

link to the issue/feature using its github issue number #number (if an issue/feature does not exist, please create it first)

Details on the issue fix or feature implementation

provide some details here

Confirm the following

[ ] I have ensured that I have merged the latest changes from the dev branch
[ ] I have successfully run a local build
[ ] I have included unit tests for the issue/feature
[ ] I have included the github issue number in my commits

Is this stable enough to use now when targeting .NET Core 3.x?

I've just started using in production. Will push a stable release soon hopefully


max_tokens_exceeded_by_camel

Thanks to nice project !
I wanted to run this project but I got this Error while running run.py.
Is there any way to avoid this error?
RuntimeError: error in ChatAgentResponse, info:{'id': None, 'usage': None, 'termination_reasons': ['max_tokens_exceeded_by_camel'], 'num_tokens': 16398}

Thank you for your feedback. Could you please provide the log file (which is under the /WareHouse path)

Thank you for your respond. This is log file.
othello_v2_DefaultOrganization_20230908103617.log

The reason for this error is that agents write a long code, and in each review turn it will see the current code and all old versions, which makes the context exceed the limits of 16384 tokens in GPT3.5 Turbo.
I just made a little fix on the complete token calculation, which may resolve the token exceed problem (does not exceed but reports exceed error) in multi-turn chatting, but it does not solve your problem since in your log the token does exceed the limit. Here are some suggestions:

Try again! I just repeated your task prompt and did not encounter an exceeded problem, the log is here: test_DefaultOrganization_20230908101138.log
Use GPT4_32k, which has a context of 32768 tokens. Money always solves the problem :)
Lower the chatting turn in the review Phase.
Close the self-improve. Self-improve can make prompts with higher quality but it often proposes too many requirements and makes agents write long and complex codes.
In your task prompt try some expressions like "write a simple software" or "no more than 50 lines in each code file"
We may consider using git diff messages in the review Phase, which would greatly reduce the tokens.

Thank you for your quick fix and helpful suggestions !
I' ll try again!

i got new error while running.
This error seems to be caused by a previous modification.
Traceback (most recent call last):
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/tenacity/__init__.py", line 382, in __call__
    result = fn(*args, **kwargs)
  File "/home/jovyan/ghq/github.com/OpenBMB/ChatDev/camel/utils.py", line 145, in wrapper
    return func(self, *args, **kwargs)
  File "/home/jovyan/ghq/github.com/OpenBMB/ChatDev/camel/agents/chat_agent.py", line 191, in step
    response = self.model_backend.run(messages=openai_messages)
  File "/home/jovyan/ghq/github.com/OpenBMB/ChatDev/camel/model_backend.py", line 69, in run
    response = openai.ChatCompletion.create(*args, **kwargs,
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/openai/api_resources/chat_completion.py", line 25, in create
    return super().create(*args, **kwargs)
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/openai/api_resources/abstract/engine_api_resource.py", line 153, in create
    response, _, api_key = requestor.request(
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/openai/api_requestor.py", line 298, in request
    resp, got_stream = self._interpret_response(result, stream)
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/openai/api_requestor.py", line 700, in _interpret_response
    self._interpret_response_line(
  File "/cache/pypoetry/virtualenvs/chatdev-JP5cuhws-py3.10/lib/python3.10/site-packages/openai/api_requestor.py", line 763, in _interpret_response_line
    raise self.handle_error_response(
openai.error.InvalidRequestError: -145 is less than the minimum of 1 - 'max_tokens'

Could you please share the log? It seems that the num_max_completion_tokens is less than 0, which means the number of tokens sent to GPT still exceeds the limit.

The log is here:
othello_v2_DefaultOrganization_20230908135830.log

It is the same problem, the agent write too many codes and in the review Phase it exceeds the limit. You can check out how many tokens are sent to and received from LLM in the log. The messages are like OpenAI_Usage_Info . For example, the last token statistic in your log is:
``
[2023-08-09 14:05:29 INFO] [OpenAI_Usage_Info Receive]
prompt_tokens: 16248
completion_tokens: 10
total_tokens: 16258
``
In the next turn the chatting exceeds the limit of 16384
I found that in your log the review loop for 5 times, you can lower this number, the default is 3.

Thank you for your reaction!
I' ll try various configs!


SCRUM-4634: production release/v0.36.0

i thought we were doing alpha to beta today?

Beta is deployed now!.
This is to deploy on Monday if Chris signs up beta, but it could wait if needed

ahh ok


Implementation of rare_sat_satpoint is not precise

Currently only the first sat of any sat range is checked against common and stored, but IMO it can happen subsequent sats of sat range can also be rare.
In other words, the result of rare_sat_satpoint may not be complete. This may cause accidental transfer of rare sats.

Rare sats are always in the beginning of a range because of the way they are mined. Sat ranges can only be split and not joined, so rare sats will always be the first in the range.


Support elemMatch for nullable iterable

I required it for this to work: item::someProperty / SomePropertyType::collection elemMatch document

Thank you


Rename AckRequest to AckMessageRequest and move it from the signal group to the message group

By the way, we may create a new request model "AckNotificationRequest" in the future.

Won't fix. The clients don't need to ack messages anymore because of https://github.com/turms-im/turms/issues/289


Enable default behavior is to turn on language server for Developer Build

This is something we'll need to do immediately after July release to ensure users using the developer build end up using the new language server.

What happened to https://github.com/Microsoft/vscode-python/pull/2215

Closing this, as this isn't necessary.


Vienna causes a font download request

Per this forum thread:

Hoping someone can provide an answer (and adjustments) to this issue.
I have Vienna 3.1.10 on a MacBook Pro running 10.12.5. Vienna is now asking to download fonts when going to certain webpages from the links.
Thanks,
bf

I can confirm this issue, I have seen it once before. My guess is that a web-view is causing this, based also on this report at StackOverflow. I do not know whether this is particular to WebView as opposed to WKWebView.

The two I see all the time are this one and one whose name is something like "Gung Seo."  I get these a half dozen times a day each.
A forum response asked for particular URLs that do this.  Here's one:
https://www.standard.co.uk/news/uk/rspca-called-to-rescue-lizard-which-turns-out-to-be-a-dirty-sock-a3635086.html
I would like to be able to tell Vienna just to deny these requests and shut up about them.

Seems like the problem does not appear anymore on macOS 10.13 (High Sierra) ?

Answering to myself : no, it still appears.


ino-markdown-editor: Link-Button is not working

Description
As a user, I want to add a link to a certain text in the ino-markdown-editor.
What I expect:

Write a text
Select the text where I want to provide a link to
Click the link button (🔗)
I'm able to somehow provide a link to the selected text passage (usually handled by a popover)

Current state
We currently got a link button that's missing the functionality above. Here's an example:

Acceptance criteria:

The user is able to provide a link by clicking the link icon in the ino-markdown-editor

More
Draw inspiration from Github, Gitlab, Jira, Gdocs, etc. (anything that has a rich text editor).

Currently we don't have a toolbar when in markdown mode. Is that something we want to implement at some point?
I will implement the modal for the WYSIWYG mode.


Create new on other than default connection

Laravel Version: beta (used this proposed fix: https://github.com/laravel/framework/pull/16103 scheduled for laravel 5.4)
PHP Version: 7.0.1
Database Driver & Version: Mysql 5.6

The model create method keeps using the default db.
We are using artisan tinker, and tried the following:
\App\FeatureGroup::on('otherConnection')->getModel()->create(['name'=>'test']);
// a new row is inserted in the default db, but should be in the otherConnection db
// firstOrCreate does the same :(

/* OUR WORKAROUND */

$featureGroupModel = new \App\FeatureGroup;
$featureGroupModel->setConnection('otherConnection');
$featureGroupModel->save();

$featureGroupModel->id; // the given id is from the new table row in otherConnection db :)

// or

\App\FeatureGroup::on('otherConnection')->getModel()->firstOrNew(['name'=>'test'])->save();

Please upgrade to the latest php 7.0.x or 7.1.x.

@reno1979 that fix wasn't related to creating model, just querying related models on the parent connection.
I've submitted a proposal to add a connection argument to the connection method, not sure if it'll pass though since you can already do it by constructing the instance yourself.
https://github.com/laravel/framework/pull/17392

@themsaid Thank you for looking in to this. Constructing a new instance is our current work arround. It just seems odd that the create methods always falls back to the default connection settings.


Mongodb as datasource - how to do that?

Hi,
I am trying to use cyclotron([http://www.cyclotron.io][1]) to work with my database in mongodb.
The configuration settings doc seems to ask for custom mongodb.
But while creating data source there is no option to select mongodb.
I would like to
a) Select my database
b) Select collection
c) Set parameters for widgets to know what to plot.
Previously I had a custom stack running nodejs + mongodb + jquery to do the plotting. Wherein nodejs has the business logic as to exchange data based on query send from frontend.
I am looking to replace the whole stack so that with minimum programming plots can be created.
Regards

Hi @mrudulp.  Unfortunately, MongoDB isn't one of the built-in Data Sources in Cyclotron, despite using MongoDB as the backend.
This is mainly due to the fact that MongoDB doesn't have a good way to execute string queries.  By this I mean, a user would need to put a MongoDB query string into the Cyclotron UI, and then some generic code would need to translate that string into the required MongoDB driver calls to provide that data.  The last I looked, even things like JDBC for MongoDB was not in a very robust state.
We could implement a limited Data Source, which only supports picking a database, collection, and (optional) filters.  Would this satisfy your needs, without the ability to run arbitrary queries or aggregations?
Alternately, with a little bit of programming you could add some new REST endpoints to the Cyclotron service to expose your MongoDB data as JSON.  This would require some programming to setup each query, but would let you use any MongoDB driver features.


Use own Table class

Let's say I have custom MyTable class (similar to \Phinx\Migration\Db\Table) in project and I want Migration to use it. Do I need to create new Migration class and rewrite table method like this
namespace App\Migrations\Migration;

use App\Migrations\Db\MyTable;
use Migrations\AbstractMigration;

class MyMigration extends AbstractMigration {
    public function table($tableName, $options = array())
    {
        return new MyTable($tableName, $options, $this->getAdapter());
    }
}

OR is there better way how to "inject" custom Table class without creating custom MyMigration class and rewritting table method?

Currently, this is the best way.
Adding them through some kind of configurable factory could be useful.
PR welcome.


Area light surfaces still appear

Perhaps due to non MIS rays?

This is just the area light helper


nixos/transmission: add to bindpaths watch-dir if need.

Motivation for this change
Noticed I got errors with watch-dir ever since hardening. cc @ju1m / @Lassulus .
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@doronbehar Indeed support for watch-dir is missing, thanks. I cannot test this PR right now, but I suppose that:

Because the user's settings may not define it, a default value must be added for watch-dir-enabled.
Also, a sensible default like "${cfg.home}/Torrents" or else, may be set for watch-dir.
Maybe using BindReadOnlyPaths= (instead of BindPaths=) would be enough for watch-dir to work correctly.
A rule r ${cfg.settings.watch-dir}, guarded by cfg.settings.watch-dir-enabled must also be added in transmission-daemon's AppArmor profile.

Because the user's settings may not define it, a default value must be added for watch-dir-enabled
Also, a sensible default like "${cfg.home}/Torrents" or else, may be set for watch-dir.

There's an upstream default for watch-dir which is "${cfg.home}/watchdir" and it's false by default. I copied most of what you did with incomplete-dir for watch-dir, according to upstream's defaults, so I hope it should be OK now.

Maybe using BindReadOnlyPaths= (instead of BindPaths=) would be enough for watch-dir to work correctly.

No, because after Transmission is adding a torrent, it renames the file with the .added suffix, so it should need write permissions as well.

A rule r ${cfg.settings.watch-dir}, guarded by cfg.settings.watch-dir-enabled must also be added in transmission-daemon's AppArmor profile.

Agreed. I added this plus other checks based on incomplete-dir.

@doronbehar Great :) I still can't test it, but it looks like there is a missing ''} in the AppArmor profile there: https://github.com/NixOS/nixpkgs/commit/42dd740ffe0230a91ad22f39932d497b6d9f1764#diff-3b9521332ddaab17070757e70eb2ff9bR421

Correct. I fixed that and tested the transmission.nix test and it succeeded.

Well then, LGTM, thanks for this PR @doronbehar :)


Cap arity of core category-specific functions to three

Background
See PR review comment.
Approach

Apply to category-specific functions such as:

buildXCategoryValueViews (e.g. buildSelectCategoryValueViews, buildCuratedOntologyCategoryView).
onFilterxCategory (e.g. onFilterSelectCategory, onFilterCuratedOntologyCategory).

Apply to select, range, curated ontology and multi-panel categories.

Closing, to be addressed with #5877.


Move ESM Patternfly generator to Spog UI

Context

We need Patternfly Charts in Spog UI but unlike other components like buttons or tables, Charts rely on JS libraries.
https://github.com/trustification/trustification/pull/644 has introduced a way of creating Donut charts importing spog/ui/crates/donut/js/main.js

What this PR is for?

This PR moves the code I had locally, for generating the file spog/ui/crates/donut/js/main.js
All changes of this PR are done into the folder spog/ui/patternfly-charts. All we need to do to update/generate spog/ui/crates/donut/js/main.js is to execute:

npm ci && npm run build

This PR is just for us to know how the spog/ui/crates/donut/js/main.js file is created. Otherwise, the other guys won't know how that file has been generated or where it has been taken from. This PR has no impact on current code/functionality of Spog UI


[Bug]: You can’t open the application “vibe” because this application is not supported on this Mac.

What happened?
You can’t open the application “vibe” because this application is not supported on this Mac.
Steps to reproduce

step one...
step two...

What OS are you seeing the problem on?
MacOS
Relevant log output
MAC 14.3 (23D56)

@chikosan
Do you have Apple silicon CPU (eg. M1) or Intel?

Intel 👍

Fixed in latest version (v0.0.5)
Please follow the Downloads section at vibe#Downloads
You should download specific installer for Intel and not for Apple Silicon

Feel free to reopen if it still doesn't work


update ecl geo_lookup usage

Waiting on https://github.com/PX4/ecl/pull/851.

I rebased on master to get this in since it came up in an estimation discussion.

It doesn't fit v2 by 845 bytes: https://github.com/PX4/Firmware/pull/15207/checks?check_run_id=917970228

I rebased on master to get this in since it came up in an estimation discussion.

Thanks, I was actually neglecting it because it's mixed with a month of ecl/EKF changes, but it's good to know I need to fight for some flash first.

I am currently including this patch when doing ecl dev work. When https://github.com/PX4/ecl/pull/870 is merged it should help with the flash space issue

I am currently including this patch when doing ecl dev work. When PX4/ecl#870 is merged it should help with the flash space issue

Thanks Paul. If needed immediately before sympy we'll have just enough flash to squeeze this in shortly. https://github.com/PX4/Firmware/pull/15445

I switched to the latest ecl version with the SymPy covariance https://github.com/PX4/ecl/pull/870 already merged. Let's see if that all fits.

Thanks everyone.


improve appearance of names in dropdowns

This alters CodapFlowSelect so that it only displays the option's underlying value if there are multiple options that share the exact same title. I.e., in a situation in which the value is necessary to disambiguate between like titles. This greatly reduces the length of the dropdowns and removes clutter in most situations (when titles are indeed unique).
I also consolidated the "generate unique name" functionality that was required in both codapPhone utils and certain transformations under one function instead of having multiple that effectively do the same thing.

Definitely makes the interface slicker, so possibly a good thing to merge soon.
I did notice that I had to remove line 27 of CollectionSelector.tsx to get it to run. Should I commit that change?

Yes please, that must've gotten overlooked when I merged to resolve conflicts


re: remove comments && deprecated codes

This PR is related to the ticket https://smarttokenlabs.atlassian.net/jira/software/c/projects/TKS/boards/13?modal=detail&selectedIssue=TKS-380&assignee=6295de5db407cc0069ff3164


Recommend add IgnoreOnInsert and IgnoreOnUpdate attribute that limit insert or update.

When update db by dtos with [IgnoreOnInsert] or [IgnoreOnUpdate], this is very useful!

@leshin could you provide a detailed example?

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.


Implement nbf claim in Uri Signing Plugin

Optional nbf claim is now implemented. If a request comes in before time specified by the nbf claim, the token is rejected.

[approve ci]


Revamp the replay details page.

There's still work to do on this page, but I think it's already an improvement.

The main goal of this redesign was to introduce more visual hierarchy on the page. In other words, I want more important or interesting information to be clearer and closer to the top of the page. That's why I added the main score box, which makes the score jump out and also presents the most interesting metadata, as well as a simpler replay details table which includes only the info most interesting for scoring runs (score) and survival runs (lives and bombs).
Known issues with the page:

It doesn't look great for replays other than full game runs of Windows games.
The SVG icons used as badges are ugly.
The css on the cards themselves could be improved; I don't love the way they look right now. I tried a different background color but couldn't find something I liked.
Adding badges will likely make players more interested in editing their replay to fix the metadata if they didn't tag it as LNN (or whatever), but we don't have that feature yet. The same is true if and when we add "no third category"-type annotations.

I'm not sure how much I like that the box at the top right shows the same information as the "Stage details" box. Maybe all the stage properties should be converted to use icons, or Stage details should have some of the information taken out. I'm also not sure about showing lifes specifically in the top right box, since some games don't have a "lifes" system to speak of.
What I do however like, is how this could look with showing the cards in th18 or the items in th143

The top box does show bombs too if that's information we have for the game; it just isn't in the screenshot above because it's mof. eg

It's not in this PR, but I figured we could show motivation and perfect freeze for GFW. I considered putting PIV in there too but decided not to, just because it's irrelevant for survival games (whereas on the other hand, resource counts are still interesting for most scoring runs).
Having two tables is not ideal, but I am not sure a better way to handle it. raviddog added a bunch of information collected from replays that most people won't care about, so I wanted to present the most interesting data front and center. But it would be a shame to hide that data entirely from the site; some of it is still interesting, and at the very least, it's cool that we have it. We could put it on another web page entirely, but that seemed like too big of a task for this PR, and I'm not sold on it as an idea in the first place.
A bigger deal (and something I'm not sure about) is what to do with runs other than full game runs of Windows games. That's a definite minority of runs, so I kinda figured we could merge this PR and then improve the situation later.
I totally agree that a good part about this layout is that we can add boxes for specific things (like UM cards) that wouldn't be relevant for most runs.

I've been putting off responding to this because css scary, but I'll try to put some time away to try other colors out.

Added some light background and shadow elements, what do you think? (note: not uploaded to this PR yet)

oh I should upload a screenshot of the whole page, not just the replay details part, because I've found screenshots of just one part look unnaturally dreary because they miss the banner and borders

here's an attempt at a dark mode style— I think it looks better than the one with the white borders, at least

The purple highlights on dark mode help so much! It fits a lot better with the site's theme. I still think that light mode feels too monochromatic.

Per popular demand, I got rid of the clear icon :)
I'll try to bring it back at some point, of course :p But hopefully with a different icon.


CORE-30186 Support an ID Sync container config

Description

Here's the config:
"query": { "identity":
{ "containerId": 1 }}
Related Issue

CORE-30186
Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All tests pass and I've made any necessary test changes.
[x] I have run the Sandbox successfully.

Be sure to update https://github.com/AdobeDocs/alloy-docs/blob/master/fundamentals/configuring-the-sdk.md


test: Add retries to curl command

See commit msgs.

test-me-please

Coverage increased (+0.06%) to 37.015% when pulling 036087e1d0dbbf8ba2cd2718b6825e8bdf3c6faf on christarazi:pr/christarazi/add-retry-curl-istio-test into 749d03af85ff8740039d518dd4d189ea5f08931b on cilium:master.


Highlight the corresponding breakpoint when stopping on a breakpoint in the debugger

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:26:20.018Z)
OS Version: Linux x64 4.4.0-96-generic
Extensions:

Extension
Author (truncated)
Version

Bookmarks
ale
0.16.0

python
don
0.7.0

Go
luk
0.6.66

cpptools
ms-
0.13.1

perforce
sle
2.2.1

Currently when stopping on a breakpoint it's hard to tell which one is it in the list of breakpoints (there's no visual clue). I'm primarily interested in the C++ debugger, but this could be relevant to all debugger engines.

Out-of-scope? Was the tag added by mistake?

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Please change the textures of the Univite items.

This fast switching rainbow is really bad IMO. Could you at least slow it down?

No


Translated few words in hindi

Fixes #1126
Changes:
Reference page (संदर्भ) contains foundation , Local Storage,Vector,Random,Interaction in English language they should be in hindi in hindi version.
Screenshots of the change:
Before

After

@Himanshu664 I think you are using some formatter that is formatting the file so it shows some unintended changes kindly look into it

@shaharyar-shamshi I made changes as suggested please review commit.

@shaharyar-shamshi I made recent commit as per changes suggested

@limzykenneth please merge this PR.

Looks good. Thanks!


shipper: better error message when thanos.shipper.json invalid

[na] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Improve diagnostics in error from shipper.ReadMetaFile when thanos.shipper.json is invalid JSON
Replace usages of ioutil in shipper tests, as it's deprecated
Add three testcases for shipper.ReadMetaFile

Verification
I added a testcase corresponding to my change.

Please let me know if I should add a changelog entry with this type of change to an error message.


Support Puppet 6

Thanks, Mate. I got the vibe that 4.10 was pretty much the current "extended support" minimum, too. I'll have another PR for Elastic Stack 7 today. I think we should increment the Puppet 4 version and our own major version in that.


fix syntax error  in sendgrid_email.js

In the future, we should separate messages subjects and content from the class to the translation system


YAML key getting cased incorrectly after using yaml.load

I have some yaml that looks similar to this:
True:
  isTrue: true

however, after using yaml.load to load this yaml, the resulting JSON is converted to:
{
  "true": {
    "isTrue": true
  }
}

Is there a way to preserve the casing on the True: object besides adding quotes around it like so: "True:"? Thanks.

Is there a way to preserve the casing on the True: object besides adding quotes around it like so: "True:"? Thanks.

No. Why do you need that? true and True have the same exact value (boolean truth), and all information about original markup is lost on load().
Well actually, there's a way. You can use custom schema instead of boolean to load and serialize that value however you want. It depends on your use-case.

My program that I am using converts the key name (in this case True) to a string that gets used later for other processing. So when the yaml is loaded and True gets turned to true for the key in the JSON, it then causes issues for me since an object im trying to find an item of true on doesnt have it, since it has True only. Thanks for the info on the custom schema, i'll give that a look.

It works with failsafe schema for example:
> yaml=require('js-yaml')
> yaml.dump(yaml.load('True'))
'true\n'
> yaml.dump(yaml.load('True',{schema:yaml.FAILSAFE_SCHEMA}),{schema:yaml.FAILSAFE_SCHEMA})
'True\n'

It doesn't have integers and such, but you can add them (see docs).

Awesome, thanks a lot for the help!


[Central_v2] As a teacher, I can create a question and save it, so I can capture a problem area my students are having.

Here is the prototype: https://www.figma.com/proto/PadzU8GqyhFLCjIt7rQkpE/RightOn-Apps?page-id=11957%3A373842&node-id=23498-140318&node-type=frame&viewport=640%2C-496%2C0.06&t=EYXWdlFUWpFz842D-1&scaling=scale-down&content-scaling=fixed&starting-point-node-id=23498%3A141965&show-proto-sidebar=1


loading break due to undefined symbol about constexpr const char*

Problem

when build with bazel build --config opt -c dbg //tensorflow/tools/pip_package:build_pip_package, there is a symbol not found error around kLowerUsingSwitchMergeAttr  when a python program loads the output dynamic linked libraries.

Root Cause

this is related to the new C++ standard about constexpr const char* as discussed here, and my local C++ compiler does not support this yet

Solution

add definitions in .cc file and the problem is gone

I don't know why the standard build options (without -c dbg) do not break though ...

@imzhenyu-dev thank you for your contribution, please sign CLA.


sql.ScanType Type error

What version of Go are you using (go version)?
1.18
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE="on"
GOARCH="arm64"
GOBIN=""
GOCACHE="~/Library/Caches/go-build"
GOENV=" ~/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="~/.gvm/pkgsets/go1.20.4/global/pkg/mod"
GONOPROXY="*.byted.org"
GONOSUMDB="*.byted.org"
GOOS="darwin"
GOPATH="~/.gvm/pkgsets/go1.20.4/global"
GOPRIVATE="*.byted.org"
GOPROXY="https://go-mod-proxy.byted.org,https://proxy.golang.org,direct"
GOROOT="~.gvm/gos/go1.20.4"
GOSUMDB="sum.golang.google.cn"
GOTMPDIR=""
GOTOOLDIR="~/.gvm/gos/go1.20.4/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.20.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/d8/6s9cc0d52zvbt66b2nzwqwx40000gn/T/go-build4191857230=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
i want to use gorm.Rows().ColumnTypes() get field ScanType
What did you expect to see?
When field type is bigint(20) unsigned, it will return sql.NullInt64 which can't scan uint64 data. i think this is a bug.

package go1.20.4/database/sql/sql.go

@seankhliao Yes, how do I respond to this problem?


Small bug in Heater status

I just noticed that the heater shows Off (uppercase O) or on (lowercase O).  I think both should be upper, right?

I believe that is an issue with the Home Assistant frontend.  The relevant lines of code in this integration are here https://github.com/cryptk/haomnilogic-local/blob/main/custom_components/omnilogic_local/water_heater.py#L104 and I am using the STATE_ON and STATE_OFF const values (as an integration should).  It's up to the frontend how it wants to render it though.


BUG - inaccurate creation date for accounts created via free account page

Describe the bug
 ago sub text under account shows inaccurate time stamp (4 years ago) for accounts created via https://www.telos.net/signup
To Reproduce
Steps to reproduce the behavior:

Go to https://explorer.telos.net/account/telostest222 (account created Jul 4th, 2022, or create a new account via https://www.telos.net/signup)
see 'created ... 4 years ago' sub text

Expected behavior
should display accurate creation timestamp
Screenshots

Additional Notes:
likely related to #215

@donnyquixotic this seems fixed?


Enable Curse Words

First off, thanks for the library, it has worked very well and helped me a great deal.
However, while looking through the transcripts for some videos (Michael Reeves for context), I noticed that "[ __ ]" was in the place where I assume curse words would be located.  Is there any way to get curse words to show up in these locations? I am using the basic get_transcript command.

Hi @Lavarider, actually I never notices this. Could you maybe share the ID of a video where this is the case? Does YouTube also show it like that when you enable subtitles on their Web-Player?
Without having this tested, I could imaging that YouTube censors curse-words when not being logged in (or logged in and under 18 or something like that). So maybe if you use a cookie from a logged in session you'll get uncensored subtitles. Check this out to find out how to use cookies.
Would be great if you could provide some feedback whether this works or not! 👍

One video where this is the case is "I built a Surgery Robot" at this link: https://www.youtube.com/watch?v=A_BlNA7bBxo. The closed captions between 1 minute and 1:10 are probably the most helpful for reference.
Youtube does have this "feature" when I enable subtitles on their web player. However, this is always enabled in the web player closed captions, both when I try and watch youtube signed out and when I signed in on an account registered as 30 years old. This leads me to believe changing cookies won't do much. Additionally, this Reddit post: https://www.reddit.com/r/youtube/comments/iurox3/youtube_literally_censoring_swears_in/ implies this is a problem for everyone.

Wow, looks like you're right. There actually seems to be no way to get an uncensored transcripts. That's kinda ridiculous 😄
But unfortunately if there's no way to get these transcripts in the web-player, then there's also no way to get them with this module 😞
So I'll unfortunately have to close this as there is not much we can do here.


Fixing how the decorator works (now it wont need an argument)

New usage:
@route('/')
@login()
def home(auth):
    user = auth.get_user(request.environ)
    return "Home page {}".format(pformat(user))

I don't believe the demo in the readme has been update to reflect this... I have the latest code (from pip) and still get this error.

I'm using the bottle-auth in openmining[1] and I am not in error
[1] https://github.com/avelino/mining/blob/master/manage.py#L57

That's great, but the demo code in the README doesn't work. It should be updated.

@adammhaile upgrade example source https://github.com/avelino/bottle-auth#example

Thanks!


Web admin logs overwrites ctrl-F

I want to find something in page and having this default to search is very annoying (and the reason I use find in page is because the search isn't robust enough yet).

this should presumably be an issue on ss14.admin

I can change it to ctr + shift + f but using the browser search probably won't help you.
Bug pjb about merging my PR for it because I wan't to improve it but I can't because the PR isn't merged yet.
Also, can you tell me how the search isn't robust enough yet if it isn't about it crashing?

This is my mockup of how I want to improve the web admin logs

There's a lot of little things but the 2 main ones  are I can't search by round and the date column is cut off so I have no idea if is a time is AM or PM.


Minor bugs with Import Individual Nodes Widgets

I found a couple of minor bugs with the Import Individual Nodes Widgets feature.
Restoring default names to imported nodes
When you drop an image on a node, the name is restored to its default.
https://github.com/user-attachments/assets/ff88ebd8-e3cc-45e0-aef1-c7bb076df699

Warning when you drop models from Model Library
Model Library introduced a dropping feature as well: You can drop models on compatible loaders.
But it seems that your feature is "conflicting" in some way, and everytime you drop a model, this warning appears:
"No exact match found in workflow. Would you like to replace the whole workflow?"
You can either confirm or not, it doesn't matter: The model will be replaced as intended. This is more of an annoying thing.
https://github.com/user-attachments/assets/fc6ad99e-624d-4f7b-a856-8c52f3a08a4e

Thanks for the report.


[etlas] Fix the new-build command

The new-build command currently has several problems:

It builds an uberjar even though the uberjar flag is not on.
It seems to not apply patches correctly causing build failures.

Now implemented on master! Closing.


chore(doc): use cargo add to instruct how to install this crate as dependency

cargo add was introduced recently to Cargo and it works well to install the latest version of this crate as dependency.
Since the version is included in the TOML snippets in README.md, they need to be updated when the major version of this crate is bumped. By using cargo add, we no longer need to take care about it and users can easily install this crate as dependency by copy&paste the command line.

Right, good point!


"import pandas" changes NumPy error-handling settings

A simple import pandas apparently does a np.seterr(all='ignore'): https://github.com/pydata/pandas/blob/23eb483d17ce41c0fe41b0bfa72c90df82151598/pandas/compat/numpy/init.py#L8-L9
This is a problem when using Pandas as a library, particularly during testing: a test (completely unrelated to Pandas) that should have produced warnings can fail to do so just because some earlier test happened to import a library that imported a library that imported something from Pandas (for example). Or a test runner that's done a np.seterr(all='raise') specifically to catch potential issues can end up catching nothing, because some Pandas import part-way through the test run turned the error handling off again.
I'm working around this right now by wrapping every pandas import in with np.errstate():. For example:
with np.errstate():
    from pandas import DataFrame

But that's (a) ugly, and (b) awkward if the pandas import is in a third-party library out of your control.
Please consider removing this feature!
Code Sample, a copy-pastable example if possible
Enthought Canopy Python 2.7.11 | 64-bit | (default, Mar 18 2016, 14:49:17)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy, pandas
>>> 1.0 / numpy.arange(10)
array([        inf,  1.        ,  0.5       ,  0.33333333,  0.25      ,
        0.2       ,  0.16666667,  0.14285714,  0.125     ,  0.11111111])

Expected Output
I expected to see a RuntimeWarning from the division by zero above, as in:
Enthought Canopy Python 2.7.11 | 64-bit | (default, Mar 18 2016, 14:49:17)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy
>>> 1.0 / numpy.arange(10)
__main__:1: RuntimeWarning: divide by zero encountered in divide
array([        inf,  1.        ,  0.5       ,  0.33333333,  0.25      ,
        0.2       ,  0.16666667,  0.14285714,  0.125     ,  0.11111111])

output of pd.show_versions()
>>> pandas.show_versions()

INSTALLED VERSIONS
------------------
commit: None
python: 2.7.11.final.0
python-bits: 64
OS: Darwin
OS-release: 13.4.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_GB.UTF-8

pandas: 0.18.0
nose: 1.3.7
pip: 8.1.1
setuptools: 20.6.7
Cython: None
numpy: 1.10.4
scipy: 0.17.0
statsmodels: None
xarray: None
IPython: 4.1.2
sphinx: None
patsy: None
dateutil: 2.5.2
pytz: 2016.3
blosc: None
bottleneck: None
tables: None
numexpr: None
matplotlib: 1.5.1
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: 4.4.1
html5lib: 0.999
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None
jinja2: 2.8
boto: 2.39.0

This came up somewhat recently in https://github.com/pydata/pandas/issues/12464, but I don't think anything came out of it.

@mdickinson this is very very common to do. The entire point of pandas is to turn things to NaN AND propogate them.
It IS possible to turn this off by having practially every deferral option to numpy wrap this in a context manager (rather than setting the global).
It would'nt be that invasive (could do it with a decorator), but adds complexity, may be somewhat non-performant, and its a fair amount of work.
If you want to give this a stab, then i'll reopen.

The entire point of pandas is to turn things to NaN AND propogate them.

Sure, and that's fine when you're using Pandas to do data analysis at a command line. It's a long way from fine when you're using Pandas as a library for a couple of small tasks (in this particular case, flexible reading from .csv files) within a larger application. Then you're in the situation where one of your application dependencies has unilaterally and silently turned off NumPy warnings for the entire application, oblivious to the needs of that application or any of its many other libraries. That's just rude. :-) Deciding to turn off NumPy warnings (or any warnings, for that matter) globally is a decision that should be made at application level, not at the level of one particular library.
It's interesting to compare with the seaborn library, where import seaborn changes several matplotlib settings. Again, that's a useful thing to do for the purposes of visual exploration from the command line, but you want to avoid that import-time side-effect when using seaborn as a library in a larger application. The seaborn folks have provided a seaborn.apionly module for that exact purpose, so that you do import seaborn.apionly as sns and any changes that have been made to the matplotlib settings elsewhere in your app are unaffected. Would something like this be possible for Pandas?

well this setting has existed since pandas inception. as I layed out above I don't see much gain in changing this. If you would like to make an attempt by all means.

Sure, and that's fine when you're using Pandas to do data analysis at a command line. It's a long way from fine when you're using Pandas as a library for a couple of small tasks (in this particular case, flexible

your statement is very odd. Virtually the entire user base uses pandas for data analysis. Want to drop back to some exception handling in numpy w/o using pandas seems dubious at best.

I agree that this behavior is very annoying.  Most of the time, my only use of pandas is to have a (great) implementation of groupby, and do not want other errors to pass silently.  Likewise, because seaborn depends on pandas, it is easy to end up with this change even without explicitly importing pandas.
Two possible solutions may be

Provide pandas.apionly, similar to seaborn.apionly.  The problem is that this is going to take a while to propagate down to other packages that depend on pandas.
Only change the errstate if at an interactive prompt (as defined by the presence of sys.ps1, not sys.flags.interactive, as the latter is unset when in IPython).

I do not believe that unilaterally modifying numpy's error-handling is a common practice. If you proposed that over at numpy-discussion, I think you would get some pushback. Importing numpy.ma used to do this for basically the same reasons, and we eventually received enough complaints about it to fix it. If you would like to propose that pandas' default settings are better than numpy's (and I am rather more sympathetic to that idea now than I was in the past), then let's change numpy's defaults so that we have consistent, documented behavior across the board.
Part of the reason that you have seen only a few complaints about this behavior since pandas' inception is that it changes behavior silently. It took a couple of years after the same behavior's inception in numpy.ma for us to notice it and fix it too.
Having an application that has both data analysis (where NaN may mean missing data) and moderately-complicated numerical algorithms (where NaNs can appear from perfectly-clean inputs due to domain violations and other numerical errors) is not at all dubious. Most nontrivial data analysis tasks have both. Cleaning missing data first before doing the heavy numerical work of data analysis/statistics/machine learning is quite common. That's why numpy has the ability to control the FPE-handling at a pretty fine grain.

well @rkern this has been in place from < 2012 AFAICT. This IS possible, but performance of the context managers would need to be considered, and much more importantly someone to do it; are you volunteering?

In progress.
In so doing, I found another reason to avoid the fire-and-forget np.seterr() in favor of more fine-grained np.errstate(). pandas assumes that no one else has changed the settings. If someone uses with np.errstate(invalid='raise'): then binary comparisons will be silently wrecked.
[~]
|1> import pandas

[~]
|2> df = pandas.DataFrame(dict(x=[np.nan, 1, 2]))

[~]
|3> df
     x
0  NaN
1  1.0
2  2.0

[~]
|4> df < 0
       x
0  False
1  False
2  False

[~]
|5> df > 0
       x
0  False
1   True
2   True

[~]
|6> with np.errstate(invalid='raise'): print df < 0
      x
0  True
1  True
2  True

[~]
|7> with np.errstate(invalid='raise'): print df > 0
      x
0  True
1  True
2  True

ok thanks @rkern that would be awesome!
yeah I have seen that inverse case as well.

@jreback how do we envision this interacting with the internals refactor? This is kind of a gray-zone on the defined behavior right now, I don't think we would want to commit to anything that could change with the refactor.
As an example:
s = pd.Series([1, 2, 3])
with np.errstate(invalid='raise'): s.reindex(range(4)) > 1

That casts to float and uses the np.nan marker for NaN, so it raises. Post internals-refactor that should still be an int dtype, so it won't use the np.nan missing values, so presumably it wouldn't raise, unless pandas explicitly does so. Should we give Wes a heads up?

no @TomAugspurger I don't think it will have much of an effect. IF its done, then computation that is deferred to numpy will use a context manager to wrap the err state. That's it, as it is one now. Ideally one would have a function like:
def compute_numpy(name, arr, *args, **kwargs):
    with np.errstate(invalid='raise'):
          return getattr(arr, name)(*args, **kwargs)

something like that. There would have to be some logic in this function to handle some of this but its not a big deal. It will mostly be isolated.

Just hit this problem myself, I'm definitely in favor of a fix. In this case, Pandas was a dependency of another package I was using---and indeed I wasn't even using the Pandas-dependent functionality---but I was bitten by this anyway. Thanks to @rkern for working on this.

This is great. @rkern thank you for the hard work!


Escape config vars on windows

People seem to keep running into username being picked up by the default Node.js USERNAME env var on windows. We should probably at least escape this one and set it to //username when the generator is run on Windows.
There are likely a few other config values we want to escape. Maybe all of them on Windows???

The default field is email so I'm not sure if we need to change anything in the generator although we might want to look into disabling the auto expansion in feathers-configuration?

I think the main problem this got missed was that the documentation for it got lost somewhere. I added it back in https://github.com/feathersjs/feathers-configuration/pull/43


How to config timeout or disable in  pipeline v0.32.4

I install tekton pipeline v0.32.4, PipelineRun aborted when timeout of 1 hour.
PipelineRun "gitlab-pipelinerun-98xsx" failed to finish within "1h0m0s"
How to config timeout or disable it?

There are a number of ways to configure the timeout, including on the PipelineRun, the tasks, or by changing the default timeout value.
Docs for Pipelines v0.32: https://tekton.dev/vault/pipelines-v0.32.x/pipelineruns/#configuring-a-failure-timeout
Docs for latest Pipelines release: https://tekton.dev/docs/pipelines/pipelineruns/#configuring-a-failure-timeout
The docs includes information on each of these options, including the global default-timeout-minutes.

Feel free to reopen if there still open questions on this.


Added disk selection and exclusion of deactivated disks

https://github.com/wefixit-AT/oVirtBackup/issues/69

This error was caused by a missing parameter in the
given in config_example.cfg
# an array of exclude disks id,exampel ["6d811aa25-52a1-45c3-9f40-1c7c868182c9", "6e78d7c2-5786-4dz2-a966-fd308c23f08e"] disks_id_exclude: []