[pointer_interceptor] Does not intercept iFrame element with youtube video

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

flutter create bug
copy and paste sample code
flutter pub add pointer_interceptor
flutter run

Expected results
Any click on the youtube video should be intercepted and discarded.
Actual results
Clicks on the video are reaching the iFrame element as PointerInterceptor doesn't exists.
Code sample
Code sample

import 'dart:html';
import 'dart:ui' as ui;

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

void main(List<String> args) {
  runApp(const IframeScreen());
}

class IframeScreen extends StatefulWidget {
  const IframeScreen({super.key});

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

class _IframeScreenState extends State<IframeScreen> {
  Widget? _iframeWidget;

  final IFrameElement _iframeElement = IFrameElement();

  @override
  void initState() {
    super.initState();

    _iframeElement.src = 'https://www.youtube.com/embed/XawP1i314WM';

    // ignore: undefined_prefixed_name
    ui.platformViewRegistry.registerViewFactory(
      'iframeElement',
      (int viewId) => _iframeElement,
    );

    _iframeWidget = HtmlElementView(
      key: UniqueKey(),
      viewType: 'iframeElement',
    );
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: PointerInterceptor(
            debug: true,
            child: SizedBox(
              height: 500,
              width: 500,
              child: _iframeWidget,
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
pointer-interceptor-issue.webm

Logs
Logs
I couldn't get this. Sorry guys and girls.

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.7.11, on macOS 13.2.1 22D68 darwin-arm64, locale en-AR)
    • Flutter version 3.7.11 on channel stable at /Users/eugeniotesio/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f72efea43c (3 weeks ago), 2023-04-11 11:57:21 -0700
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/eugeniotesio/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    ! Some Android licenses not accepted. To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.0

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

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] IntelliJ IDEA Community Edition (version 2021.3.2)
    • 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.77.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.62.0

[✓] Connected device (3 available)
    • iPhone 14 (mobile) • 84140C70-3D86-4F47-BDEA-A8F7CD75715C • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 13.2.1 22D68 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 112.0.5615.137

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

Thanks for the detailed report.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.7.12, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.7.12 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (3 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.11.0-1.0.pre.44, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.11.0-1.0.pre.44 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ebf3ea1755 (2 hours ago), 2023-05-03 23:25:08 -0400
    • Engine revision da45a21708
    • Dart version 3.1.0 (build 3.1.0-72.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

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

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

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

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

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

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

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

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

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

! Doctor found issues in 1 category.

I think the issue is that the iframe sits on top of the interceptor. Can you check the HTML DOM? To fix it you can put them in a stack with the iframe as the first child and the pointer interceptor as the second child.

cc @ditman - maybe another point towards adding a feature where one can choose where the pointer interceptor goes, above or below.

Linking to the issue you're referencing: https://github.com/flutter/flutter/issues/121272

I think the issue is that the iframe sits on top of the interceptor. Can you check the HTML DOM? To fix it you can put them in a stack with the iframe as the first child and the pointer interceptor as the second child.

@eugenio-d-tesio I think Yegor is right. The PointerInterceptor is supposed to wrap child that need to sit ON TOP of HTMLElementViews.
In this case, you're using the HtmlElementView as the child of the platform view (so the platform view ends behind its child).
The current workaround for this, as Yegor described, is to Stack the PointerInterceptor on top of your HtmlElementView, like this:

Proposed main.dart
import 'dart:html';
import 'dart:ui' as ui;

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

void main(List<String> args) {
  runApp(const IframeScreen());
}

class IframeScreen extends StatefulWidget {
  const IframeScreen({super.key});

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

class _IframeScreenState extends State<IframeScreen> {
  Widget? _iframeWidget;

  final IFrameElement _iframeElement = IFrameElement();

  @override
  void initState() {
    super.initState();

    _iframeElement
      ..src = 'https://www.youtube.com/embed/XawP1i314WM'
      ..style.border = 'none'
      ..style.width = '100%'
      ..style.height = '100%';

    // ignore: undefined_prefixed_name
    ui.platformViewRegistry.registerViewFactory(
      'iframeElement',
      (int viewId) => _iframeElement,
    );

    _iframeWidget = HtmlElementView(
      key: UniqueKey(),
      viewType: 'iframeElement',
    );
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: Stack(
            alignment: AlignmentDirectional.center,
            children: [
              // First the YT video in its square SizedBox
              SizedBox.square(
                dimension: 500,
                child: _iframeWidget,
              ),
              // Then the PointerInterceptor (on top)
              PointerInterceptor(
                debug: true, // optional
                child: const SizedBox(
                  width: 250,
                  height: 500,
                  // No need for children to just block interaction.
                ),
              ),
            ],
          )
        ),
      ),
    );
  }
}


onJoinChatChannelResponse not firing

A few days ago valve updated dota2 and my lobby-bot broke.
When i send joinChat (CMsgDOTAJoinChatChannel) message, GC do not send me CMsgDOTAJoinChatChannelResponse.

Need somebody to confirm this bug against the node-steam1.1.0 branch. If it's still an issue we should resolve it for the 1.0.0 release.

It's working on the node-steam1.1.0 branch.

Thanks @devilesk
In that case I'll close the issue, the fix will ship with 1.0.0.


Model citations

Draft to discuss metadata model and citations

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4022

This has served its purpose and the models have been reviewed and the place that Citations is being added is approved.


Videos won't register as played/watched/seen and won't show in the video history

Script version
5.6, also seen in 5.5 already
Describe the bug
At the moment (at least going on for 3 days) I keep seeing the same videos I already watched in my YouTube feeds (frontpage, recommendations). There's no hint they've been watched and no time stamp has been saved. Most videos also don't show up in the video history. When disabling the script it seems to work properly and the videos I tested it with haven't shown up again in the feeds yet and have appeared in the history.
Expected behavior
YouTube registers when a video has been watched, saves time stamp, displays "seen/watched" label on the video thumbnail, shows watched videos it in the YouTube history.
Desktop (please complete the following information):

OS: Windows 11, Version 10.0.22621 Build 22621
Browser: Chrome
Version: 126.0.6478.62 (also previous version, since I received this update for Chrome just a few minutes ago)

Excuse me, why has this been closed without a comment?


Add change to enable GitHub Issues notifier to create issues based on a subset of checks.

Overview

I would like the GitHub Issues notifier to be able to notify against a subset of checks for a given accreditation.

Requirements

If specific checks are configured within a given accreditation GHIssuesNotifier should only create issues for those specific checks.

Approach

Add logic to GHIssuesNotifier.__generate_issues to look for a non-empty checks configuration element within a given accreditation and then only generate issues for the specified checks if it's found.

Security and Privacy
N/A
Test Plan
N/A

See also #37


(feat) JSON file to NEW/GEN project/files

Would be great if we could configure a JSON file to do the following:

Create new project (ng new)
Generate all types (ng generate ...)
Option for JSON file per module (ng generate module and all its contained generate commands in one JSON file that is referenced from the "project" JSON file)

Additionally, would be nice if we could Reverse Generate the JSON file from project/files (so we can create the JSON file from a previously generate project/files - to use for incremental gen updates)
@kenhansen01 has a PR (https://github.com/angular/angular-cli/pull/1039) for some of this and hoping it makes it to master.

Closing together with #1039, reasoning mentioned https://github.com/angular/angular-cli/pull/1039#issuecomment-253889085


Do you want a logo?

I'm thinking a swift shaped cloud or something like that.

Hi, yes. Swift shaped cloud sounds good to me. Thank you.

Pick one (or none, and suggest changes) and I'll clean up a .svg for you:

oop missed one

All of them look great. Can you create pull request by adding fifth one to images folder as logo.png? Also it would be great if you could make it rounded to match app icon style like in the following link. Thanks!
https://developer.apple.com/design/resources/#macos-apps


Add Vagrantfile

This PR adds a Vagrantfile, adds a .gitignore, removes the dependency on elasticsearch, and fixes the repo url in setup.py.

:+1:


Switch: adding liveview binding has no effect

Hello!
Firstly great job on porting this great library to LiveVIew. A quick question: how can I effectively attach a binding to the component (i.e. phx-click or phx-change)?
Best,
Jakub

Sorry for late response.
For basic components, you can bind event as usual:
          <.button phx-click={JS.push("update")}>
            Close from back-end
          </.button>

For other components like sheet, dialog, etc., they don't allow event binding.
If you have any specific issue, pls open another one.


⚠ Spryr has degraded performance

In 85df374, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3530 ms

Resolved: Spryr performance has improved in 02b3c60 after 28 minutes.


Weave CNI Installation Link Moved Permanently

What you expected to happen?
When I visit the documentation for the kube addon for Weavenet, the instructions should be up to date and should work even for the most recent versions of Kubernetes.
What happened?

The first code block from the documentation link above is copied here verbatim.
kubectl apply -f "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')"

As of the time I am submitting this issue, this command returns a permanent redirect. The body of the message suggests using k8s/v1.10/net.yaml, as the endpoint, however Kubernetes latest release is now on v1.15.1. If you change v1.10 out for a newer version, the same permanent redirect is returned to v1.10. If I use v1.10 anyway, the new endpoint doesn't support the query argument syntax used to do custom configuration as described further down on the documentation page.
How to reproduce it?

Run curl "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')" on an up-to-date Kubernetes cluster and observe the redirect response.
Summary
I'm guessing this is just a matter of the docs going stale and requiring a refresh. Most google-able guides online specify the same command as the docs, and the redirect message doesn't seem like it's telling me the right thing. I wasn't sure where to bring this up, so if my question belongs elsewhere please point me in the right direction.

As of the time I am submitting this issue, this command returns a permanent redirect. The body of the message suggests using k8s/v1.10/net.yaml

If you are using kubectl apply -f  then its should just apply. Are you getting message about Redirect.

Run curl "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')" on an up-to-date Kubernetes cluster and observe the redirect response.

if you use it with -L option it should bypass the redirect message and generate manifest.

If I use v1.10 anyway, the new endpoint doesn't support the query argument syntax used to do custom configuration as described further down on the documentation page.

what error you are getting. It runs fine againest v1.15.1 for me
curl  "https://cloud.weave.works/k8s/v1.10/net.yaml?k8s-version=$(kubectl version | base64 | tr -d '\n')&disable-npc=true"
apiVersion: v1
kind: List
items:
  - apiVersion: v1
    kind: ServiceAccount
    metadata:
      name: weave-net

If you are using kubectl apply -f then its should just apply. Are you getting message about Redirect.

I make a point of grabbing the yaml with curl and inspecting it before applying it to my cluster. I suspect most enterprise kubernetes admins would do the same. I never considered that the behavior of the endpoint would be different between curl and kubectl.

if you use it with -L option it should bypass the redirect message and generate manifest.

Yes, this workaround bypasses the redirect. That said, I ran into this while performing upgrades. I'm ready to move to the latest and greatest version of the endpoint!

what error you are getting. It runs fine againest v1.15.1 for me

The error I was getting was due to confusion around the endpoint. The old url was
https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')

the new one as suggested by the redirect is
https://cloud.weave.works/v1.10/net.yaml

To a naive weave and kubectl user like me, this suggests that the way the kubernetes version is specified has moved from a URL parameter (old way) to part of the path to the endpoint (new, redirect way). I see from your example that I need to specify both v1.10 in the URL and then specify my kubernetes version as a part of the base64 encoded parameter.
The specification of multiple version numbers in the endpoint and in the URL parameters all near v1.1x really confused me. Is this documented anywhere? If not, I volunteer to submit a PR to try to clarify this in the docs for future users like myself (assuming the docs are open source on GitHub).


Merge efforts

@Maximkulkin, thank you for taking this to the next level
I fully agree that my code looked messy and that is no surprise since I actually learned coding doing this project :-).
I propose that we join forces and together we bring this even further
I'll contact you to see how we could do that.
Again, kudos for investing!
HacK

actually, since you do not have a mail address visible, can you contact mine to make a backchannel?


Move PrimitiveBuilder constraints to where clause

small follow up from #1630

Let's just implement the builder ourselves, these constraints are annoying


nrzled.NewSPI: also add padding in the front

Some SPI devices take a while to set up for transfer, and while doing
so, show the first bit they're about to send.
This causes the first LED to always interpret the first symbol as a
logical 1. Zapping out the whole strip will leave the first LED green,
and the first pixel will always be slightly green-ish:

Prevent this, by applying the same 3 bit padding we have at the end to
the front:

Test failures seem unrelated:

Run gosec -include=G104 -fmt=golint -quiet ./...
2022/05/14 12:15:59 internal error: package "errors" without types was imported from "command-line-arguments"
Error: Process completed with exit code 1.

Thanks! Will take a look in ~half a day. CI failures look unrelated, will take a look separately.

I fixed CI. I'll still squash since I can't run the checks on tip of tree.

https://github.com/periph/devices/actions/runs/2340193976 worked!

🎉

Released v3.6.14: https://github.com/periph/devices/releases/tag/v3.6.14
I was long overdue.

ugh, forgot to git fetch before tagging, let me create v3.6.15 :(


[now-cli] Render an error message when the --token content is invalid

Fixes #3334.
$ node ~/now-cli/dist/index.js --token $'he˚ll,o\n'
> Error! You defined "--token", but its contents are invalid. Must not contain: "\n", ",", "˚"
> More details: https://err.sh/now/invalid-token-value

Codecov Report

Merging #3335 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3335      +/-   ##
==========================================
- Coverage   13.76%   13.75%   -0.01%
==========================================
  Files         267      267
  Lines       10446    10452       +6
  Branches     1262     1262
==========================================
  Hits         1438     1438
- Misses       8937     8943       +6
  Partials       71       71

Impacted Files
Coverage Δ

src/index.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


bridge usdc from eth to Nervos always show pending

bridge usdc from eth to Nervos.
It has been 7 hours and it still shows: 1. lock asset on Ethereum (confirmed) 2. mint asset on Nervos (pending). Then at Yokai Bridge, we can't see that usdc has reached ckb. It only took a few minutes to cross the chain before, but this time it was really stuck.
eth-tx:https://etherscan.io/tx/0x7fd02597716e4520f7685509f37059cdacce98a2cae3cd4170b67bd838b07386

@DT1990 It sounds like your query would be best dealt with by the support team.
Please see the link below to our dedicated support line:
Support :Help Center
Note: Click on the live chat icon at the bottom corner of the page to start a conversation


blt_add_target_compile_flags can now append flags to INTERFACE targets

I tested this within BLT (on the toss3 gcc host config) by adding the following flags in tests/internal/CMakeLists.txt:
# Add flag to a regular library target
blt_add_target_compile_flags(TO example FLAGS " -finstrument-functions")
# Add list of flags to a library target
blt_add_target_compile_flags(TO example FLAGS "-Wundef" "-ftrapv")

# Add list of flags to an interface target (in two ways)
blt_add_target_compile_flags(TO blt_header_only FLAGS "-Wswitch-enum" "-Wfloat-equal")
blt_add_target_compile_flags(TO blt_header_only FLAGS -finstrument-functions -ftrapv)

I confirmed that the new macro doesn't overwrite the flags on the second call and that flags are transitively sent to dependents.
I am not planning to commit these lines to the blt internal tests since the added flags are meaningless in this context and I did not define them for other compilers.

We can also apply similar logic to blt_add_target_definitions and blt_add_target_link_flags to make these work with INTERFACE targets.


Publish 4/30/2020 3:37 PM PST

Docs Build status updates of commit 66d7518:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/msonlinebackup/Rename-OBVolume.md
:white_check_mark:Succeeded

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


VoxelShape -> Shape

Hot Take Of The Week™
There is nothing inherintly "voxel" about the implementation, it just represents a collection of points that form into a shape of sorts. The only point where voxels are introduced as a concept are in the third-party methods provided in the Block class that divide given points by 16.0 (the standardized 16x16x16 voxel-per-cube concept) before constructing a shape. But outside of this, in the primary classes and types, there is no application of this concept and they are purely points representing a shape that can be of any dimension or resolution. I do recognize that Mojang refers to them as VoxelShape's within String's and that Yarn may have a tendency to respect Mojang's naming decisions, but I feel this is one of various cases where the name could easily be simplified as the prefix can either be seen as redundant or conceptually incorrect. Another thought on this is that a case in Advancement where Mojang's toString actually names it SimpleAdvancement has been recognized as incorrect and that Mojang actually call the class Advancement, and this could very well be the case for VoxelShape too. Mojang in fact only refer to it as VoxelShape 2 times in the codebase: The toString of VoxelShape, and an IllegalArgumentException message in ArrayVoxelShape. Meanwhile also in VoxelShape is an UnsupportedOperationException message merely describing it as a shape ("No bounds for empty shape."). I don't have much more to put here so I'll just conclude by reminding everyone that this is a "hot take" and I am not pushing for this change, it is merely a wildcard suggestion.

During FAP fry referred to it as VoxelShape multiple times. I think it's pretty safe to assume. Shall we close?

This was created with the knowledge that Mojang calls it VoxelShape. It was intended as a simplification of type concept. Averaged, the votes lean towards a decline of this change currently, though, so I will close it based on that.


spacy. gold

Hi,
I running the example (ok, with my data), but have this error report
----> 4 from spacy.gold import GoldParse
      5
      6 from .util import is_special_token

ModuleNotFoundError: No module named 'spacy.gold'

It appears spacy > 3.x does not know that module. I reinstalled with spacy 2.3.3, and had the following error
"ValueError: [E003] Not a valid pipeline component. Expected callable, but got 'sentencizer' (name: 'None').[E004] If you meant to add a built-in component, use create_pipe: nlp.add_pipe(nlp.create_pipe('sentencizer'))"
Could you give a pointer where to start?
thanks,
Andreas
running python 3.8

Hi Tim,
I'll try it all again in a clean virtual environment (I'm sometimes lazy). Unfortunately I can't share the data, but I'll take the examples.
Best,
Andreas

When I use your data and code on a clean Python 3.8 environment everything works as expected and I get the following output:
2022-02-26 14:10:27,457 - KeyphraseVectorizer - INFO - It looks like the selected spaCy pipeline is not downloaded yet. It is attempted to download the spaCy pipeline now.
Collecting en-core-web-sm==3.2.0
  Downloading https://github.com/explosion/spacy-models/releases/download/en_core_web_sm-3.2.0/en_core_web_sm-3.2.0-py3-none-any.whl (13.9 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 13.9/13.9 MB 13.2 MB/s eta 0:00:00
Requirement already satisfied: spacy<3.3.0,>=3.2.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from en-core-web-sm==3.2.0) (3.2.2)
Requirement already satisfied: preshed<3.1.0,>=3.0.2 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.0.6)
Requirement already satisfied: spacy-loggers<2.0.0,>=1.0.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (1.0.1)
Requirement already satisfied: srsly<3.0.0,>=2.4.1 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.4.2)
Requirement already satisfied: spacy-legacy<3.1.0,>=3.0.8 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.0.9)
Requirement already satisfied: murmurhash<1.1.0,>=0.28.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (1.0.6)
Requirement already satisfied: blis<0.8.0,>=0.4.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (0.7.5)
Requirement already satisfied: pathy>=0.3.5 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (0.6.1)
Requirement already satisfied: jinja2 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.0.3)
Requirement already satisfied: tqdm<5.0.0,>=4.38.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (4.62.3)
Requirement already satisfied: requests<3.0.0,>=2.13.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.27.1)
Requirement already satisfied: numpy>=1.15.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (1.22.2)
Requirement already satisfied: thinc<8.1.0,>=8.0.12 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (8.0.13)
Requirement already satisfied: typer<0.5.0,>=0.3.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (0.4.0)
Requirement already satisfied: pydantic!=1.8,!=1.8.1,<1.9.0,>=1.7.4 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (1.8.2)
Requirement already satisfied: wasabi<1.1.0,>=0.8.1 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (0.9.0)
Requirement already satisfied: catalogue<2.1.0,>=2.0.6 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.0.6)
Requirement already satisfied: setuptools in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (60.9.3)
Requirement already satisfied: packaging>=20.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (21.3)
Requirement already satisfied: cymem<2.1.0,>=2.0.2 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.0.6)
Requirement already satisfied: langcodes<4.0.0,>=3.2.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.3.0)
Requirement already satisfied: pyparsing!=3.0.5,>=2.0.2 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from packaging>=20.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.0.7)
Requirement already satisfied: smart-open<6.0.0,>=5.0.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from pathy>=0.3.5->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (5.2.1)
Requirement already satisfied: typing-extensions>=<IP_ADDRESS> in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from pydantic!=1.8,!=1.8.1,<1.9.0,>=1.7.4->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (4.1.1)
Requirement already satisfied: charset-normalizer~=2.0.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from requests<3.0.0,>=2.13.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.0.12)
Requirement already satisfied: certifi>=2017.4.17 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from requests<3.0.0,>=2.13.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2021.10.8)
Requirement already satisfied: idna<4,>=2.5 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from requests<3.0.0,>=2.13.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (3.3)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from requests<3.0.0,>=2.13.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (1.26.8)
Requirement already satisfied: click<9.0.0,>=7.1.1 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from typer<0.5.0,>=0.3.0->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (8.0.4)
Requirement already satisfied: MarkupSafe>=2.0 in /opt/anaconda3/envs/kv-test/lib/python3.8/site-packages (from jinja2->spacy<3.3.0,>=3.2.0->en-core-web-sm==3.2.0) (2.1.0)
Installing collected packages: en-core-web-sm
Successfully installed en-core-web-sm-3.2.0
✔ Download and installation successful
You can now load the package via spacy.load('en_core_web_sm')
['new diptyque candle' 'waste' 'coffee table' 'tex paclitewhat' 'trim'
 'blades' 'ability' 'magic keyboard' 'mother' 'magazines' 'lunch'
 'north america' 'kitchen sink' 'person' 'system' 'opportunities' 'form'
 'hours' 'grapeseed oils' 'share' 'kind' 'rest' 'spiceworks' 'trend'
 'company' 'ohms' 'chief executive' 'many compliments' 'urban outfitters'
 'online shopping cart' 'episodes' 'drone' 'category' 'hollywood reporter'
 'foreign buyers' 'class' 'length zipper' 'equity' 'thing' 'hood'
 'urban style' 'iron construction' 'kid' 'course' 'survey' 'couples'
 'genetic material' 'july' 'hyperion' 'shuttle bus' 'trays' 'fence'
 'capacity' 'good' 'event' 'free dream pillows' 'podcasts' 'books'
 'dakkan' 'performance' 'ipads' 'northeast divisional director' 'call'
 'razor bumps' 'act' 'sacrifices' 'belle' 'inch macbook'
 'actual physical design' 'anything beach' 'spiceworks network' 'bag'
 'phantom' 'flowery' 'natural seagrass tray' 'car' 'hand' 'smith'
 'biggest impact' 'feelings' 'furniture' 'food' 'jeans' 'scissor' 'trails'
 'use tool' 'sunflower' 'site amenities' 'lot' 'great freedom'
 'tjx president' 'weeks' 'pool' 'set' 'sense' 'razor' 'pro review'
 'cloths' 'many layers' 'comb' 'company sizes' 'interest' 'lenses'
 'separate dna strands' 'plan' 'lightest option' 'total' 'manufacturing'
 'point' 'home lending advisor' 'template' 'peter jianette' 'dresser'
 'bikini trimmer' 'hint' 'happy experience' 'car mode' 'way' 'bed' 'field'
 'degrees' 'length' 'conundrum' 'gen z' 'durran' 'end' 'harm' 'black hole'
 'butterfly mechanism' 'enquiry levels' 'decline' 'mortgage'
 'certain angles—90 degrees' 'note' 'healthcare' 'sized businesses'
 'combo' 'brain' 'belt bag' 'wind' 'treat' 'woven seagrass' 'line'
 'rest ohm' 'life' 'back' 'code forbes200' 'angle' 'international buyer'
 'cozy vibes' 'protection' 'everyday ensemble' 'enquiries' 'drop'
 'snowboard industry' 'diy instructions' '2l gore' 'housing market'
 'ruffled blouse' 'pound' 'case' 'considerable acclaim' 'luxury markets'
 'go' 'many bells' 'simplified menu' 'season' 'daily usewaterproofing'
 'electric bikini trimmer' 'eject button' 'moisture' 'moonlit skincare'
 'respect' 'whistles' 'non - profits' 'tanager' 'imperfections' 'day'
 'industries' 'mitochondrial dna' 'dreams' 'elements' 'terms' 'mtdna'
 'pounds' 'skin guards' 'finance' 'duo' 'warm floral scent'
 'technology buyers' 'prime central london' 'respondents' 'macbook'
 'parka' 'shower' 'ipad pro' 'mini' 'cold person' 'playstation'
 'switch mechanism' 'home' 'shoppers' 'durable men' 'ebay' 'email'
 'typical time' 'restaurant' 'weight' 'lifespan' 'variety' 'ernie herrman'
 'shick hydro' 'agents' 'space' 'huge financial commitment' 'hdmi cable'
 'best option' 'tuberose candle' 'ohm' 'side' 'picking' 'snags'
 'polyester' 'ways' 'europe' 'midnight shift' 'much extra storage'
 'introduction' 'horse' 'style jacket' 'fall' 'plenty' 'denim cutoffs'
 'top' 'warm sleepers' 'ages' 'chase' 'head' 'consumption' 'coachella'
 'tunein app' 'ounces' 'duty piece' 'current smart keyboard'
 'millennial shoppers' 'long car trip' 'years' 'grooves' 'example'
 'temperature' 'daybeds' 'lunch coolers' 'hot tub' 'pins' 'versatility'
 'friends' 'people' 'piece' 'flexibility' 'pro' 'financial freedom'
 'government' 'mag model' 'specific section'
 'microfiber eyeglass cleaning cloths' 'corset' 'multiblade razor'
 'backpack' 'oz' 'performance goggle leader' 'options' '[ chase' 'journey'
 'long story' 'storage' 'cotton lining' 'job title' 'new contact'
 'education' 'enterprises' 'therm' 'shop' 'review'
 'interchangeable lens model' 'table' '%' 'durability'
 'hypoallergenic moisturizing' 'view' 'mitochondria' 'holes' 'job'
 'effect' 'elevator pitch' 'fragrant lavender scent' 'testing' 'analysis'
 'affordable drone' 'jojoba' 'significant amount' 'texture' 'race' 'ski'
 'larger frames' 'person sleep system' 'miter gauge']

Process finished with exit code 0

It seems that your issue is caused by the spacy-pytorch-transformers package, which is outdated and replaced by spacy-transformers by now. KeyphraseVectorizers does not require any of them. I would therefore suggest either upgrading the package, or even better, deleting it to fix the issue. A clean new environment setup should usually not include spacy-pytorch-transformers or  spacy-transformers. KeyphraseVectorizers does not install any of them either because  none of them is included in the requirements.
When trying to install spacy-pytorch-transformers after KeyphraseVectorizers I even get the following dependency error:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
keyphrase-vectorizers 0.0.7 requires spacy>=3.0.1, but you have spacy 2.1.9 which is incompatible.

Installing spacy-pytorch-transformers automatically downgrades spaCy to 2.1.9 which is incompatible with KeyphraseVectorizers and thus may cause the issue. So both packages together are incompatible.

Since I did not hear anything back, I consider this issue as solved.

Hi Tim,
apologies for not getting back. The solution was in the spaCy installation
procedures. I discussed that with Ines and the problem is solved.
thanks again,
Andreas
On Mon, Mar 28, 2022 at 3:07 PM Tim Schopf @.***> wrote:

Since I did not hear anything back, I consider this issue as solved.
—
Reply to this email directly, view it on GitHub
https://github.com/TimSchopf/KeyphraseVectorizers/issues/3#issuecomment-1080568719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFO2PW3C7L42P2HZQFSQFMDVCGOJTANCNFSM5O6TLIEQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


[BEAM-12931] Allow for DoFn#getAllowedTimestampSkew() when checking the output timestamp

A DoFn may emit elements with a timestamp up to DoFn#getAllowedTimestampSkew() before the current element's timestamp. This change implements this change for timer's as well. Now a timer may have an output timestamp up to DoFn#getAllowedTimestampSkew() before the current element's timestamp. Before this change a timer's output timestamp could not be before the current output element.
Additional Context: https://lists.apache.org/thread.html/r7554658114ddde86c5d82e1c39fe7e1ef587fe926b8e406d1130d501%40<dev.beam.apache.org>
@reuvenlax

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Is there an associated JIRA?

I think you have to update the similar check in FnApiDoFnRunner

Added checks in FnApiDoFnRunner which was not checking for skew on timers and not checking timestamps at all for normal emit. PTAL and make sure it makes sense and I can prob add some tests in FnApiRunner.
PTAL @reuvenlax @lukecwik Thanks!

Won't this allow for infinite skew since if have a timer at X and skew of -1 then the first time the timer is processed you can output at time X-1 and when it gets scheduled again you can now output at X-2 since the the new timers timestamp is X-1?

So my understanding of the reason for these checks is to stop people from doing the wrong thing without realizing it. We don't even take any different action based on this variable. It seems okay to apply this to each specific output timestamp and let you skew more if you chain timers in this fashion.
On a more practical note, there's reasons why you might want a timer to output an earlier element if you've properly set up watermark holds. There's currently no way to do that so we need some allowance. It would probably be better if we could constrain skew from the first output timestamp but I don't think that's available in the later timers, right?
If you disagree with the approach, I can bring this up on the email thread for others to chime in in case they are not checking here.

PTAL, @lukecwik @reuvenlax

@lukecwik PTAL, still need to check that everything runs internally but otherwise should be good.

Run Java PreCommit


System.TypeInitializationException

I'm trying to utilize the library in an ASP.NET Core 2.1 (preview) application but continue to get exceptions when running the following within a method in my data access layer / repository:
 var filter = new Filter<Persona>();
 filter.By("LastName", Operation.Contains, "Alex");  // exception here

And when I run this line (from above):
ASP.NET Core crashes with the following:
"exceptionType": "System.TypeInitializationException",
"stackTrace": " in ExpressionBuilder.Helpers.OperationHelper..ctor()\r\n in ExpressionBuilder.Operations.Operation..cctor()",

Could you give me some help? Thanks in advice!

I got the same issue

Hi @snill93, @Hobbes1987,
Sorry for the late reply.
The project currently doesn't support .Net Core, but I'll be working on that.
I'll try to keep you guys posted.
Cheers.

Hi @snill93, @Hobbes1987,
I created a pre-release package at NuGet.org with support for .Net Core.
Would you mind checking that out and leaving a comment here to let me know if it's working correctly now?
Many thanks,
David

An unhandled exception occurred while processing the request.
AmbiguousMatchException: Ambiguous match found.
System.RuntimeType.GetMethodImplCommon(string name, int genericParameterCount, BindingFlags bindingAttr, Binder binder, CallingConventions callConv, Type[] types, ParameterModifier[] modifiers)
TargetInvocationException: Exception has been thrown by the target of an invocation.
System.RuntimeTypeHandle.CreateInstance(RuntimeType type, bool publicOnly, bool wrapExceptions, ref bool canBeCached, ref RuntimeMethodHandleInternal ctor)
TypeInitializationException: The type initializer for 'ExpressionBuilder.Helpers.OperationHelper' threw an exception.
ExpressionBuilder.Helpers.OperationHelper..ctor()
TypeInitializationException: The type initializer for 'ExpressionBuilder.Operations.Operation' threw an exception.
ExpressionBuilder.Operations.Operation.get_Contains()
filter.By(queryField.Field, ExpressionBuilder.Operations.Operation.Contains, queryField.Value);

Get the same exception with EqlausTo.
.net core 2.2 in asp core project.

Same here for core 2.2 and equalsTo

@dbelmont no plans on improving support for .net core? I'm also using .net core 2.2

I've had the same issue.
After debugging it seems the problem for me was a nuget package that didn't specify its dependencies.
The nuget package that caused problems referenced another nuget package, but didn't actually use anything in that nuget package, causing our main project to not load in the "required" .dll or load in a different version of the .dll.
The code used to fetch all types used in ExpressionBuilder (to initialize the operations) also searches for the types in the missing (or having an incorrect version) .dll, causing the exception.
Either way, you should catch the System.Reflection.ReflectionTypeLoadException if you're having problems and look what library is giving troubles.

I'm getting the same error
System.TypeInitializationException: The type initializer for 'ExpressionBuilder.Operations.Operation' threw an exception. ---> System.TypeInitializationException: The type initializer for 'ExpressionBuilder.Helpers.OperationHelper' threw an exception. ---> System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.Reflection.AmbiguousMatchException: Ambiguous match found.
I'm using Xamarin Forms on IOS targeting .net standard 2.1 (tried 2.0 also to no avail)

It looks like this is already fixed in the repo. Could you push a new version to NuGet?
#27


Rebrand

Rebrand the Readme, Keymaps, and Menu items.

Gonna go ahead and merge this one to avoid any issues with anyone who does decide to download this version of the package.


icd: Add external memory properties

Updates external memory properties to be able to write tests for this issue: https://github.com/KhronosGroup/Vulkan-ValidationLayers/issues/7790
Tested VVL with this icd update here: https://github.com/artem-lunarg/Vulkan-ValidationLayers/commits/artem-test-mock/

CI Vulkan-Tools build queued with queue ID 160656.

CI Vulkan-Tools build # 1412 running.

CI Vulkan-Tools build # 1412 passed.


When run in Windows 10, Obsidian import creates only documents from 'root' directory

Description
On Windows importing an Obsidian Vault (via markdown import pod) does not import documents from anything except the top level 'root' directory.
To Reproduce
Steps to reproduce the behavior:

Create fresh Obsidian vault, one document in root, one in subfolder
Initialise fresh dendron vault
Configure import pod
Execute import
Observe that document from subfolder is not imported

Expected behavior
Expected import of document from sub-folder with appropriate hierarchy file name.
Screenshots
Obsidian Vault supposed to be imported

Dendron vault after import (also showing log file)

versions

OS: Windows

VSC
Version: 1.51.1 (user setup)
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-10T23:34:32.027Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0

Dendron 0.15.0

Additional context
Import works as expected when run in Crostini/Chromeos
Dendron Log file
Log file shown in dendron screenshot above

Can no longer repro this. @spacejunky - let me know if you still see it and please reopen with fresh logs.


Hooks do not trigger events and windows starts lagging.

Hello folks,
I currently work with the Steelseries GameSense SDK to make my own effects etc. for my keyboard and mouse. To light up my mouse and keyboard on clicks and presses, I use this library.
Unfortunately, the mouse and keyboard events don't get triggered. Also, my mouse starts lagging, and keyboard input gets delayed.
The lag and the delay only stay for about half a minute. I suspect that windows removes the hooks because it detects the lag.
I also tried this example program and everything works fine there.
Here is the code:
private static readonly IKeyboardMouseEvents GlobalHook = Hook.GlobalEvents();

static InputManager()
{
    Logger.Log("Starting...", Logger.Type.Info);
    GlobalHook.KeyDown += KeyEvent;
    GlobalHook.MouseDownExt += MouseEvent;
    Logger.Log("Ready!", Logger.Type.Info);
}

And the event functions:
private static void KeyEvent(object sender, KeyEventArgs eventArgs)
{
    Logger.Log(eventArgs.KeyCode.ToString());
}

private static void MouseEvent(object sender, MouseEventArgs eventArgs)
{
    Logger.Log(eventArgs.Button.ToString());
}

You can find the whole class (and the project) here.
The constructor is the only thing that gets executed in the program.
What I found regarding the lag is that the event function has to be fast. That cannot be the problem in my case because the Logger.Log() function is fast, and the lag also occurs when using Console.WriteLine().
As I said, the example program runs fine. I tried copying the example code, but that made no difference. The only real difference between my program and the example program is that the example uses .Net Core but I use .Net Framework (4.8). Could that be the reason? If it is the reason, is there any way to use the library with .Net Framework?
I look forward to any help.

Answered thanks to Charlieface.
I did not call Application.Run(new ApplicationContext()).
A working example looks the following (on [usage example](private static void Main(string[] args))):
private IKeyboardMouseEvents m_GlobalHook;

public void Subscribe()
{
    // Note: for the application hook, use the Hook.AppEvents() instead
    m_GlobalHook = Hook.GlobalEvents();

    m_GlobalHook.MouseDownExt += GlobalHookMouseDownExt;
    m_GlobalHook.KeyPress += GlobalHookKeyPress;
    Application.Run(new ApplicationContext());
    Console.ReadLine();
    Unsubscribe()
}

private void GlobalHookKeyPress(object sender, KeyPressEventArgs e)
{
    Console.WriteLine("KeyPress: \t{0}", e.KeyChar);
}

private void GlobalHookMouseDownExt(object sender, MouseEventExtArgs e)
{
    Console.WriteLine("MouseDown: \t{0}; \t System Timestamp: \t{1}", e.Button, e.Timestamp);

    // uncommenting the following line will suppress the middle mouse button click
    // if (e.Buttons == MouseButtons.Middle) { e.Handled = true; }
}

public void Unsubscribe()
{
    m_GlobalHook.MouseDownExt -= GlobalHookMouseDownExt;
    m_GlobalHook.KeyPress -= GlobalHookKeyPress;

    //It is recommened to dispose it
    m_GlobalHook.Dispose();
}

Question on Stackoverflow:
https://stackoverflow.com/questions/67827007/windows-hooks-do-not-trigger-events-and-windows-starts-lagging-using-globalmou/67828935?noredirect=1#comment119891333_67828935


Release Datetime show the current datetime for package details

Hi @tyler-sommer,
i have seen, that the release date is not correct. It is the current datetime and not the releasedate from the package.

Regards,
Bjoern

Thanks for the bug report!

The only problem with release datetime is in views, where release_date variable is called instead of releaseDate, that is declared in Composer\Package\Package.php.
Change code on line 34 in views/Plugin/Satis/Inventory/view.html.twig from:
{{ package.release_date|date('Y-m-d H:i:s') }}

to:
{{ package.releaseDate|date('Y-m-d H:i:s') }}

Have a nice day. :)

Fixed in #106


[Bug] Docker container startup：Error while recovering ledger error code: -10

Search before asking

[X] I searched in the issues and found nothing similar.

Version
Operating system: macOS
Mirror label：3.0.0
Minimal reproduce step
Caused by: javax.ws.rs.InternalServerErrorException: HTTP 500 {"reason":"\n --- An unexpected error occurred in the server ---\n\nMessage: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n\nStacktrace:\n\norg.apache.pulsar.broker.service.BrokerServiceException$PersistenceException: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n\tat org.apache.pulsar.broker.service.BrokerService$2.openLedgerFailed(BrokerService.java:1747)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl.lambda$asyncOpen$8(ManagedLedgerFactoryImpl.java:421)\n\tat java.base/java.util.concurrent.CompletableFuture.uniExceptionally(CompletableFuture.java:990)\n\tat java.base/java.util.concurrent.CompletableFuture$UniExceptionally.tryFire(CompletableFuture.java:974)\n\tat java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510)\n\tat java.base/java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:2162)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl$2.initializeFailed(ManagedLedgerFactoryImpl.java:416)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl$1.lambda$operationComplete$2(ManagedLedgerImpl.java:438)\n\tat org.apache.bookkeeper.common.util.SingleThreadExecutor.safeRunTask(SingleThreadExecutor.java:137)\n\tat org.apache.bookkeeper.common.util.SingleThreadExecutor.run(SingleThreadExecutor.java:113)\n\tat io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)\n\tat java.base/java.lang.Thread.run(Thread.java:833)\nCaused by: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n"}
2023-11-21 11:50:28             at org.glassfish.jersey.client.JerseyInvocation.convertToException(JerseyInvocation.java:960) ~[org.glassfish.jersey.core-jersey-client-2.34.jar:?]
2023-11-21 11:50:28             at org.glassfish.jersey.client.JerseyInvocation.access$700(JerseyInvocation.java:82) ~[org.glassfish.jersey.core-jersey-client-2.34.jar:?]
What did you expect to see?
Normal startup
What did you see instead?
Caused by: javax.ws.rs.InternalServerErrorException: HTTP 500 {"reason":"\n --- An unexpected error occurred in the server ---\n\nMessage: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n\nStacktrace:\n\norg.apache.pulsar.broker.service.BrokerServiceException$PersistenceException: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n\tat org.apache.pulsar.broker.service.BrokerService$2.openLedgerFailed(BrokerService.java:1747)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl.lambda$asyncOpen$8(ManagedLedgerFactoryImpl.java:421)\n\tat java.base/java.util.concurrent.CompletableFuture.uniExceptionally(CompletableFuture.java:990)\n\tat java.base/java.util.concurrent.CompletableFuture$UniExceptionally.tryFire(CompletableFuture.java:974)\n\tat java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510)\n\tat java.base/java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:2162)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl$2.initializeFailed(ManagedLedgerFactoryImpl.java:416)\n\tat org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl$1.lambda$operationComplete$2(ManagedLedgerImpl.java:438)\n\tat org.apache.bookkeeper.common.util.SingleThreadExecutor.safeRunTask(SingleThreadExecutor.java:137)\n\tat org.apache.bookkeeper.common.util.SingleThreadExecutor.run(SingleThreadExecutor.java:113)\n\tat io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)\n\tat java.base/java.lang.Thread.run(Thread.java:833)\nCaused by: org.apache.bookkeeper.mledger.ManagedLedgerException: Error while recovering ledger error code: -10\n"}
2023-11-21 11:50:28             at org.glassfish.jersey.client.JerseyInvocation.convertToException(JerseyInvocation.java:960) ~[org.glassfish.jersey.core-jersey-client-2.34.jar:?]
2023-11-21 11:50:28             at org.glassfish.jersey.client.JerseyInvocation.access$700(JerseyInvocation.java:82) ~[org.glassfish.jersey.core-jersey-client-2.34.jar:?]
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Could you provider some logs in the bk side ?

Because it is this exception Error while recovering ledger error code: -10. The -10 in bookkeeper means recovery failed.
Recovery failure has various reasons. You can try using bin/bookkeeper shell ledgermetadata -l <ledgerid> to check the ledger metadata to see if it is in recovery. Then using bin/bookkeeper shell readledger -fe 0 -le 1 -l <ledgerid> -force-recovery to execute the recovery to see what happens in that ledger. You may meet no such ledger or ledger read timeout. If it shows no such ledger or no such entry, that means the ledger is deleted or there is something wrong with the bookkeeper. You can set autoSkipNonRecoverableData to true to skip it.

I'm experiencing the same issue: pulsar would just break every few days (possibly after a laptop restart). Could this be related to a new Docker for Mac version? This wasn't happening until somewhat recently.


python311Packages.clevercsv: 0.8.0 -> 0.8.1

Diff: https://github.com/alan-turing-institute/CleverCSV/compare/refs/tags/v0.8.0...v0.8.1
Changelog: https://github.com/alan-turing-institute/CleverCSV/blob/refs/tags/v0.8.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

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

88 packages built:

ansible (python310Packages.ansible-core)
ansible-later
ansible-later.dist
ansible-lint
ansible-lint.dist
ansible.dist (python310Packages.ansible-core.dist)
ansible_2_13
ansible_2_13.dist
ansible_2_14
ansible_2_14.dist
appdaemon
appdaemon.dist
clevercsv (python310Packages.clevercsv)
clevercsv.dist (python310Packages.clevercsv.dist)
deepdiff (python310Packages.deepdiff)
deepdiff.dist (python310Packages.deepdiff.dist)
home-assistant-component-tests.apple_tv
home-assistant-component-tests.litterrobot
kargo
kargo.dist
molecule (python310Packages.molecule)
molecule.dist (python310Packages.molecule.dist)
python310Packages.ansible
python310Packages.ansible-compat
python310Packages.ansible-compat.dist
python310Packages.ansible-kernel
python310Packages.ansible-kernel.dist
python310Packages.ansible-runner
python310Packages.ansible-runner.dist
python310Packages.ansible-vault-rw
python310Packages.ansible-vault-rw.dist
python310Packages.ansible.dist
python310Packages.atsim_potentials
python310Packages.atsim_potentials.dist
python310Packages.pyatv
python310Packages.pyatv.dist
python310Packages.pylitterbot
python310Packages.pylitterbot.dist
python310Packages.pytest-ansible
python310Packages.pytest-ansible.dist
python310Packages.pytest-testinfra
python310Packages.pytest-testinfra.dist
python310Packages.qcodes
python310Packages.qcodes-contrib-drivers
python310Packages.qcodes-contrib-drivers.dist
python310Packages.qcodes-loop
python310Packages.qcodes-loop.dist
python310Packages.qcodes.dist
ttp (python310Packages.ttp)
ttp.dist (python310Packages.ttp.dist)
python311Packages.ansible
python311Packages.ansible-compat
python311Packages.ansible-compat.dist
python311Packages.ansible-core
python311Packages.ansible-core.dist
python311Packages.ansible-kernel
python311Packages.ansible-kernel.dist
python311Packages.ansible-runner
python311Packages.ansible-runner.dist
python311Packages.ansible-vault-rw
python311Packages.ansible-vault-rw.dist
python311Packages.ansible.dist
python311Packages.atsim_potentials
python311Packages.atsim_potentials.dist
python311Packages.clevercsv
python311Packages.clevercsv.dist
python311Packages.deepdiff
python311Packages.deepdiff.dist
python311Packages.molecule
python311Packages.molecule.dist
python311Packages.pyatv
python311Packages.pyatv.dist
python311Packages.pylitterbot
python311Packages.pylitterbot.dist
python311Packages.pytest-ansible
python311Packages.pytest-ansible.dist
python311Packages.pytest-testinfra
python311Packages.pytest-testinfra.dist
python311Packages.qcodes
python311Packages.qcodes-contrib-drivers
python311Packages.qcodes-contrib-drivers.dist
python311Packages.qcodes-loop
python311Packages.qcodes-loop.dist
python311Packages.qcodes.dist
python311Packages.ttp
python311Packages.ttp.dist
sublime-music
sublime-music.dist


color-changing favicon

https://css-tricks.com/svg-favicons-in-action/

The favicon chosen for this project is colored and works on both light and dark backgrounds.


Docker Image

Hey! Is there an image available to reproduce the environment for this project? I'm having some trouble getting this code working, specifically with functorch (says that vmap does not exist) and tcnn(tcnn not found error -tinycudann does exist though).  Still, I think there's a CUDA version issue too w tinycudann. I imagine an image would make this a lot easier.
On a related note, would love to see nerfbusters use the latest release of nerfstudio if possible - as it's quite tough to integrate this into existing pipelines which use the latest release.

Hey @jaidevshriram, so unfortunately we aren't providing an image for this environment. I'd refer to nerfstudio for getting the environment running (there is some docker support there, though I haven't really tested it), and then you can install this repo with its dependencies. If you wish to use the current version of nerfstudio rather than our branch, you can reference the PR for making code changes. If we make further integrations into the nerfstudio repo, we will update the Nerfbusters documentation. Thanks!


Docker simulator not working, built in DB configuration problem

ISSUE TYPE

Bug Report

COMPONENT NAME

Issue with Docker based simulator

CLOUDSTACK VERSION

4.13 SNAPSHOT, "latest" image tag

CONFIGURATION

Docker CE 18.09.6
OS / ENVIRONMENT

Ubuntu 18.04.2 LTS
SUMMARY

Can't access UI via localhost:8080: 503 error Service not available.
According to docker logs there's a database configuration problem:
Caused by: java.sql.SQLException: Access denied for user 'cloud'@'localhost' (using password: YES)
STEPS TO REPRODUCE

As README says: run Docker container and access it.
docker run --name simulator -p 8080:8080 -d cloudstack/simulator

EXPECTED RESULTS

Simulator UI is accessible

ACTUAL RESULTS

HTTP ERROR: 503

Problem accessing /client/. Reason:

    Service Unavailable

Seems like deployment scripts assume there should be a DB user named "cloud" but it is not in the database delivered with the simulator:

mysql> SELECT user, authentication_string FROM mysql.user;
+------------------+-------------------------------------------+
| user             | authentication_string                     |
+------------------+-------------------------------------------+
| root             |                                           |
| mysql.session    | *THISISNOTAVALIDPASSWORDTHATCANBEUSEDHERE |
| mysql.sys        | *THISISNOTAVALIDPASSWORDTHATCANBEUSEDHERE |
| debian-sys-maint | *2505A97210CB9BFDFA8B5C7453A7800A968ECAF7 |
+------------------+-------------------------------------------+
4 rows in set (0.00 sec)

Looks like fixed in https://github.com/apache/cloudstack/commit/9abb703a41a48fc16bc8153451b3c403d4522454
Please reopen if against latest master this is still reproducible

Experiencing this with docker simulator on latest

I'm having the same issue with the latest 4.15 code. Simulator seems to be totally unmainted on Docker as per this.
After building the ACS I had to manually run ssh-keygen command to produce a id_rsa.cloud key and restart the simulator container. Now, I'm getting a null-pointer exception as below. The full log is here: sim.log
[0m INFO  [o.a.c.s.m.m.i.DefaultModuleDefinitionSet] (main:null) (logid:) Loading module context [root-ca] from URL [jar:file:/acs/client/target/cloud-client-ui-<IP_ADDRESS>-SNAPSHOT.jar!/META-INF/cloudstack/bootstrap/spring-bootstrap-context-inheritable.xml]
[0m INFO  [o.a.c.s.l.CloudStackExtendedLifeCycle] (main:null) (logid:) Configuring CloudStack Components
[0m [WARNING] Failed startup of context o.e.j.m.p.JettyWebAppContext@10f192d8{/client,file:///acs/client/target/classes/META-INF/webapp/,UNAVAILABLE}{file:///acs/client/target/classes/META-INF/webapp/}
[0m org.springframework.context.ApplicationContextException: Failed to start bean 'cloudStackLifeCycle'; nested exception is java.lang.NullPointerException
[0m     at org.springframework.context.support.DefaultLifecycleProcessor.doStart (DefaultLifecycleProcessor.java:181)
[0m     at org.springframework.context.support.DefaultLifecycleProcessor.access$200 (DefaultLifecycleProcessor.java:54)
[0m     at org.springframework.context.support.DefaultLifecycleProcessor$LifecycleGroup.start (DefaultLifecycleProcessor.java:356)

@rhtyd Could you pls take a look

Hi @tampler yes the docker support is not officially maintained, I think some of my colleagues may have some idea, cc @weizhouapache @DaanHoogland @sureshanaparti

@tampler
could you please test with <IP_ADDRESS> ?
issues should have been fixed by #5507

Hi @weizhouapache @rhtyd
I've pulled Wei's image from here. I was able to boot it. I'll be looking at it these days.
Wei, could you pls upload it to the official ACS account here And drop obsolete images not to confuse people.

are here any updates? what could be blockers? the fix works!

@synergiator
I think this has been resolved by #5507 .

@weizhouapache the fix makes possible creating your own container from the upstream codebase and this is great. However, the official images haven't been delivered yet where you'd expect them.

Side note: the alternate Docker Hub image works, but only the basic DC deployment. Advanced configuration example scripts seem to fail.


Release 2.0.3

Questions
Answers

Description?
Release

Type?
release

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/PrestaShop#19508

How to test?
Module test

I marked https://github.com/PrestaShop/statsvisits/pull/3 as "bug" so release-drafter should ackowledge this


Can not run a pod in a Kubernetes cluster with wasmedge-shim

My Kubernetes version: v1.22.1
I install the containerd-wasmedge-shim/v0.3.0 from the release page and I cp them to /usr/local/bin.
This is my /etc/containerd/config.toml:
        [plugins."io.containerd.grpc.v1.cri".containerd.runtimes.wasmedge]
          runtime_type = "io.containerd.wasmedge.v1"

I can run a wasm application directly using containerd:
root@master-153:~/projects# ctr run --rm --runtime io.containerd.wasmedge.v1  --platform=wasi/wasm  docker.io/wasmedge/example-wasi:latest wasm-example
Random number:<PHONE_NUMBER>
Random bytes: [33, 62, 23, 84, 147, 22, 145, 117, 20, 27, 180, 1, 152, 25, 212, 224, 12, 11, 130, 167, 203, 124, 76, 238, 176, 210, 155, 192, 187, 215, 24, 251, 170, 203, 107, 136, 207, 54, 29, 11, 45, 36, 13, 182, 107, 90, 197, 145, 110, 213, 35, 16, 156, 61, 158, 29, 240, 71, 61, 211, 232, 115, 90, 205, 40, 134, 24, 21, 64, 250, 189, 215, 26, 38, 115, 131, 199, 46, 60, 237, 101, 183, 202, 154, 9, 175, 58, 193, 79, 110, 61, 20, 192, 178, 230, 76, 48, 8, 105, 89, 170, 174, 150, 231, 72, 34, 194, 234, 8, 124, 238, 211, 46, 204, 20, 43, 150, 121, 244, 44, 142, 105, 223, 133, 115, 159, 157, 146]
Printed from wasi: This is from a main function
This is from a main function
The env vars are as follows.
PATH: /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
The args are as follows.
/wasi_example_main.wasm
File content is This is in a file

But when I try to run a pod in k8s, it does not work.
I have applied a RuntimeClass resource like this:
apiVersion: node.k8s.io/v1
kind: RuntimeClass
metadata:
  name: wasmedge
handler: wasmedge

And this is my pod's yaml:

apiVersion: v1
kind: Pod
metadata:
  name: wasi-example-pod
spec:
  containers:
  - name: example-container
    image: docker.io/wasmedge/example-wasi:latest
  restartPolicy: Never
  runtimeClassName: wasmedge

But the kubectl describe shows that:
  Normal   Scheduled               7s    default-scheduler  Successfully assigned default/wasi-example-pod to master-153
  Warning  FailedCreatePodSandBox  7s    kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = RuntimeHandler "wasmedge" not supported

I really want to know why... If someone need more information, please do not hesitate to comment.

Have you checked if your k8s cluster has the shim binary in the /usr/local/bin?
RuntimeHandler "wasmedge" not supported is telling me that containerd cannot find the wasmedge shim binary in PATH.

Have you checked if your k8s cluster has the shim binary in the /usr/local/bin?
RuntimeHandler "wasmedge" not supported is telling me that containerd cannot find the wasmedge shim binary in PATH.

Thx for replying.
I am very sure that the shim binary is in PATH.
However, I reset my k8s cluster's version from 1.22.1 to 1.28.1, it works. I don't why this could happen, but I am so happy to see it works :)

Happy to see the issue was resolved! Closing it


⚠ Bofrost.at has degraded performance

In b3087bb, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1168 ms

Resolved: Bofrost.at performance has improved in 84250ba after 6 minutes.


Reporting build conclusion back to Dependabot build check run

Current dependencies on/for this PR:

main

PR #647   👈

This comment was auto-generated by Graphite.


Handle Enter/Return press

Closes #248

Checked and it looks like tab is already handled like you'd expect. So pressing enter/return just clicks 'OK' for you.


 - is there a way to give an accordion a header

Bug, feature request, or proposal:
Not sure if this feature exists or not.
What is the expected behavior?
I was hoping to give a <mat-accordion> a header.
What is the current behavior?
Unsure if it's possible to give a <mat-accordion> a header.
What is the use-case or motivation for changing an existing behavior?
I am having trouble with <mat-table> so I am using <mat-accordion> but I'd like to get a header if possible. I might have to use  if I can't get a header with <mat-accordion>.

This is not a good question for SO, can you link to the docs for this issue? Other people can search for this issue and then find the link in the docs.


Complete Tinker's Construct Support ~ It's kind of wierd?

Mekanism is already mostly compatible with Tinkers, however it doesn't appear to support all the liquids that the Smeltery can create. Such as Molten Obsidian for example.
~~There also seems to be some small issue with sending liquids back into the Smeltery, although there appears to be 0 issues with taking liquids out.
Pulling lava from the Smeltery into a Basic Tank works, but pulling lava from the same tank into a pipeline~~
Actually, it only appears that these fluids don't fill pipes if they don't have a destination. Iron and Gold will show flowing through the pipes if there is no available tank.

Doesn't sound like a bug on my end.  There's no specific Tinkers Construct support, which means if some things work and some things don't, it's not my fault.


ROBOT_EXIT_ON_FAILURE not functional inside "Call Method" calls

If a method called using "Call Method" raises a fatal exception (ROBOT_EXIT_ON_FAILURE=TRUE), fatal errors do not stop test execution.

Could you elaborate?

I have a class that I instantiate. Specific methods of this class are critical, and upon failure point to a complete failure of the test jig itself, not the DUT. Therefore, when these methods fail, test execution needs to end so I can service the jig. To implement this critical failure I have defined a fatal error exception, wherein ROBOT_EXIT_ON_FAILURE=True. However, if these methods are called using the Call Method keyword, the exception is raised, the current test fails, but the next test is started.
Other functions that raise this error that are not part of a class and as such do not use Call Method to invoke will completely stop test execution as intended when this fatal exception is raised.

Here is an example demonstrating the bug(?):
class FooEx(RuntimeError):
    ROBOT_EXIT_ON_FAILURE = True

class thing(object):
    def foo(self):
         raise FooEx()

MY_THING = thing()

### test.robot
*** Settings ***
Variables    var.py

*** Test cases ***
TC
    Call method    ${MY THING}     foo

TC2
    No operation```

When running, the first test case fails but second test case passes.

This example also works if you replace `Call method` with `Evaluate    $MY_THING.foo()`

This works as expected:
```### test2.robot
*** Settings ***
Library    var.thing

*** Test cases ***
TC
    foo

TC2
    No operation```

This behaviour in RF 3.0

Could you please provide an example?

The reason for the behavior is that Call Method and Evaluate both catch the exception and report it further and they don't check ROBOT_EXIT_ON_FAILURE or any other such special attribute. Those attributes are only checked by the core framework when a keyword it calls raises an exception. In these cases keywords raise exceptions but those exceptions don't contain any special attributes.
My current thinking is that the current behavior is fine. Various other keywords also directly or indirectly call methods of the objects passed to them as arguments, and all of them checking some special attributes would complicate the code unnecessarily. Using Call Method is not generally even recommended, it's typically better to create dedicated keywords instead.
If you can provide some good use cases showing that this behavior is really desired, we can consider adding that functionality. Even in that case I wouldn't call it a bug but an enhancement request for the selected keywords.

Well, in my case I have a objects that maintain a device IP and various other attributes to get and set SNMP OIDs. If one of these get or sets fail, I want test execution to stop.

Is there a reason you don't expose this functionality as keywords instead of using Call Method?

This is what I've gone with now
But the original setup was that different objects of this class housed the device specific information (IP etc)
and the handles to those devices were instantiated and called with call method.

Objects containing device info sounds good but I don't understand how that would require you to use Call Method. You can easily pass those objects to your custom keywords too.

I must be misunderstanding something.
Object 1: Device 1
Object 2: Device 2
I want Object 1 oid response. Call Method on object 1 "Get response"
I want Object 2 oid response. Call method on object 2 "Get response"
Without call method how can I run get response on the two different objects instantiated from this class that houses the IP?

You create a keyword that does it. Something like this:
def get_response(device):
    return device.get_response()

Shouldn't the documentation be enhanced, though? To me, it seems to pretty directly give the impression that ROBOT_EXIT_ON_FAILURE always stops execution.

Execution does stop if you raise an exception from a keyword with that attribute set. In this case its Call Method that raises the exception because calling a method fails.

So, I need to create a wrapper function to take the object and return its get_response, To me that wrapper function is just sidestepping Call Method, but is inherently just doing the same thing (taking an object and calling a method from that object)
If methods inside Call Method can't critically fail as all other keywords, what is the use Call Method in the first place in an environment where particular methods are critical?

You should always design libraries to that tests using them are as clear as possible. Using
    ${response} =    Get Response    ${device}

is much cleaner than
    ${response} =    Call Method    ${device}    get_response

Call Method is there to help in cases where you need to call some method of some object once and creating a library just for that purpose is an overkill. You should never design your tests so that you need to use it all the time.
If you underlying use case is stopping execution if the environment is not OK, it's easiest that you simply create a keyword like Environment Should Be OK that raises an exception stopping the whole execution if it's not.

After thinking this a bit more, I'm even more convinced that Call Method shouldn't handle this situation. Closing.

But in the case where the environment can go down at any point, this seems redundant to call Environment Should Be OK before any outbound communication.
I'll rethink the design of the test library.
Thanks


Email automation - Email - After a successful deposit payment hold is processed - Biz owner gets

Email should go out from<EMAIL_ADDRESS>email address. to Business Owner email
and BCC a copy to<EMAIL_ADDRESS>Please note the bullet point indentation
Subject: LettuceSocial - We Received Your Collaboration Request
Hi {name},
We are super excited about our collaboration and looking forward to helping you grow {business name} through local social media advocacy by LettuceSocial nano-influencers.
Below is what comes next to kick off {business name}'s collaboration.

LettuceSocial will notify you (via email and text) within 3 business days if the nano-influencer ({creator Instagram handle}) is willing and able to collaborate with you. The 10% deposit you made will only be charged once the nano-influencer accepts the offer. Until then, it will remain on hold. In case the nano-influencer declines the offer, the hold on your 10% deposit will be released.
LettuceSocial will build the business page for {business name} on the LettuceSocial website (LettuceSocial will reach out with the information needed to finalize the business page).
Agree on what free offering {business name} can provide to the nano-influencer. This is important for two reasons:

The nano-influencer will be able to meet with you/your general manager to learn about your story as a local business owner, your services, and what aspects of the business/services need to be highlighted.
The nano-influencer gets to experience your product/services and create authentic content.

LettuceSocial will connect you with your nano-influencer to coordinate the first visit.

Please feel free to reach out to us with any questions.
Looking forward to working with you on this exciting collaboration.
Best regards,
LettuceSocial Team
Follow us on Instagram

@mohsenxad
are you asking me to create an email template or you doing it?
if you want me to do it, do you need me to do HTML or Visual Builder?

tested - done


How to prevent lirc from building?

Hi,
unfortunately I'm mixing and matching versions of meta-rpi and meta-oe. Because of this lirc bbappend is failing. Is there a way to just tell yocto not to build this recipe? I don't need it

you could mask it with BBMASK something like
BBMASK .= "recipes-connectivity/lirc"

might work. but you might have to refine the regexp


Source code for ArduCAM_host_v1.0.exe

Is the source code for ArduCAM_host_v1.0.exe available somewhere? Sorry if this is in your depot and I just can't locate it.

you can find it under (ArduCAM host application demonstration):
http://www.arducam.com/arducam-host-application-demonstration/#more-813

@Pajama, Please find the source code from www.arducam.com/downloads/app/ArduCAM_Mini_C_sharp_20150413.zip

Thanks!


Support for LNK2 WiFi

Hello and many thanks for this project!
I took this project as base for an implementation for the FHEM home automation system written in Perl:
fhem-rainbird
Now Rain Bird has released a new WiFi stick Rain Bird LNK2 WiFi.
Sadly my implementation can't communicate to the device using "tunnelSip" as methode.
The responses for all commands using "tunnelSip" always are:
{"error":{"code":-32602,"message":"Invalid parameters"},"id":43,"jsonrpc":"2.0","result":{"text":"NoResult"}}
Communication using the methodes "getNetworkStatus", "getSettings", "getWifiParams" is working.
Now I'm asking myself if my implementation is faulty or if the LNK2 stick has a different structure of the commands!?
Did someone manage to communicate to a LNK2 WiFi with pyrainbird?

I tested the irrigation test script with LNK2 successfully.

Thank you a lot for your quick response!


feat: Add numpy support for Duration

General comment: in the date_shift openEO process it is called period instead of duration, should we align on that?
https://processes.openeo.org/#date_shift
The code itself seems fine!

General comment: in the date_shift openEO process it is called period instead of duration, should we align on that?
https://processes.openeo.org/#date_shift

Hmm, true - I was following https://github.com/Open-EO/openeo-processes/blob/master/meta/subtype-schemas.json#L129, the subtype is called duration. Either way, I have no strong opinion about this atm, so think I'll just keep it like this for now - can always rename if the need arises!


A way to destroy the auto update function that ties to the scrolling

In this demo, I have a button that triggers a floating tooltip when mouse enter, and remove the tooltip when mouse leave. Obviously I want the onComputed() to run when the tooltip is showing. However, there doesn't seem to be an easy way to turn it off on-demand when the tooltip is hidden. As a result, you can see the console message keeps printing.
I know you mentioned in https://github.com/fedorovvvv/svelte-floating-ui/issues/8 that autoUpdate() is unavoidable when the tooltip is out of view. But what I'm asking is a way to completely reset the whole floating-ui back to before the first call to floatingContent().
Or is this request more appropriate for the upstream?

        <button on:mouseenter={() => (showTooltip = true)} on:mouseleave={() => (showTooltip = false)} use:floatingRef>Hello</button>

        {#if showTooltip}

Tooltip is a long one

        {/if}

Hiii✨
You may be able to dynamically update autoUpdate via update
Example:
$: update({
 autoUpdate: showTooltip
})

According to the type definition, UpdatePosition specifically doesn't take autoUpdate, right? I tested your snippet and it does not work.
On the other hand, if I update(onComputed: () => {}), it no longer executes the old real onComputed, but still, this empty function is executed hundreds of times every time user scrolls, which is suboptimal.

It's weird😥
I'll try to fix that in future versions, thanks🙏
It will be necessary to stop autoUpdate at destroy


Links to task sequence variables point to wrong page

The two notes around SMSTSMPListRequestTimeout are pointing to ‘Task sequence built-in variables’ (https://docs.microsoft.com/en-us/sccm/osd/understand/task-sequence-built-in-variables)  , but should Point to ‘Task sequence action variables’ (https://docs.microsoft.com/en-us/sccm/osd/understand/task-sequence-action-variables#BKMK_InstallApplication )

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

ID: 1187d55d-d8b3-ebc8-37e8-8de14807be4e
Version Independent ID: daff4a99-10ac-b424-f08a-9024407e667a
Content: Task sequence steps - Configuration Manager
Content Source: sccm/osd/understand/task-sequence-steps.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @sigurd-de for the feedback, I have plans on reworking these articles, which aligns with this request. :)

I fixed this with an internal update to this article that's currently in review. Along with partial resolution of issue #551, I created a new article: https://docs.microsoft.com/sccm/osd/understand/task-sequence-variables, which combines the two previous variable articles into a single, comprehensive reference. It should publish live by the end of this week. I'm closing this issue.


Dbradish add cosmos DB sample links

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

#sign-off on link accuracy.
@kairu-ms , please confirm that I have put the long-summary property in the right place.

Add @evelyn-ys for awareness

This task was completed in PR https://github.com/Azure/azure-cli-extensions/pull/4854/files


Feature Request : Add the ability to invoke commands (AutoPilot Reset, Wipe)

Hello - Great tool, thank you. Any discussions or thoughts on expanding the functionalities during the offboarding process? It would be great if we could pass various device actions during the offboarding or before the offboarding. I'd be happy to add to this as well via pull request.

Hello @memile123,
thanks for the request. This is a good idea but was not sure if people would missclick on wipe etc. and reset their devices by mistake. I think that we should approach this by creating a second script that has those features. I am thinking about something like Intune Offboarding "Advanced".
Besides these topics I have limited time. I would appreciate all pull requests that I will check before merging it.
What do you think?
Best regards
Ugur

@ugurkocde Apologies on the late reply here but that sounds great. I or others that see this need might be able to jump in an help.

This will be a feature of v3. Need to rewrite the code base to make them work.


remove origins from cdn endpoint udpate parameters

Hi @csmengwan, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (qinwa). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


fix(toml): remove lib.plugin key support and make it warning

What does this PR try to resolve?
Remove lib.plugin key, making it an "unused key" warning.
Remove some of the tests, which should look useless (I hope I'm understanding this

[x] Remove key, and related tests.
[x] Adjust the documentation about the plugin.
[ ] Some of the comments and function names have not yet finished being modified.

part of #13629
Closes #13246

@bors try

:hourglass: Trying commit a969971632a16625f72b320845213d03fe99898b with merge 7da4ff38acf173c760f362a25e3e38f3407ba13f...

:sunny: Try build successful - checks-actions
Build commit: 7da4ff38acf173c760f362a25e3e38f3407ba13f (7da4ff38acf173c760f362a25e3e38f3407ba13f)

@bors r+
Thanks!

:pushpin: Commit a969971632a16625f72b320845213d03fe99898b has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit a969971632a16625f72b320845213d03fe99898b with merge 54d9c3306bd06250d03f29a4c500e9d5296c70d5...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 54d9c3306bd06250d03f29a4c500e9d5296c70d5 to master...

Something was off about the timing of this and CI is now reporting a breaking change.  We need to bump the version of cargo-util-schemas.
See https://github.com/rust-lang/cargo/actions/runs/9449610811/job/26026286907?pr=14037


Include "user profile" info in Passport object

1. Update Passport model
spec: http://passportjs.org/docs/profile
notes:

displayName should equal User.username
emails must contain User.email
photos should contain the gravatar url currently generated in user
all other fields should behave as they currently do
toJSON should strip password and tokens

2. Update UserController

findOne queries should always populate the Passport object

I am going to implement this differently.


Can't move ends of line objects inside of Trust Boundary frame

Describe the bug:
When line objects (i.e., Data Flow, Trust Boundary lines) are inside of a Trust Boundary frame, manipulation of the line ends causes the manipulated end to be placed at a position other than where the cursor was on release of the left mouse button.
Expected behaviour:
Line ends are placed where the mouse cursor is at the time that the left mouse button is released.
Environment:

Version: <IP_ADDRESS>
Platform: Desktop App
OS: Windows

To Reproduce:

On a STRIDE diagram, drag a Data Flow line to the canvas.
Drag a Trust Boundary frame to the canvas and resize it so that the Data Flow line is fully enclosed in the Trust Boundary.
Click on one end of the Data Flow line and drag it slightly to another position within the Trust Boundary.
Note that the Trust Boundary highlights yellow (as if something were being anchored to it) once the line end begins to be repositioned.
Note that - upon release of the line end - the end will position somewhere else (it looks like the middle of the frame) within the Trust Boundary.
Note that selecting only the Trust Boundary and deleting it will delete both objects.

Many thanks for this @sszeller , I had not see this bug before and I cna easily reproduce it with your instructions
I have marked it as a priority bug, and I hope it can be worked on soon
The drawing package AntV/X6 has its quirks, and needs some more configuration changes to the diagram components

This is fixed in pull request 'Migrate to version 2.x antv/x6' #1099, where the dataflow is still manageable even after an attempt to associate it with a non-existent component within a trust boundary


Google Maps shows the wrong locality in the plus code address

I changed my address on google maps but the plus code doesnot change whit.
How can i change de plus code?

The plus code is attached to a specific area. On Google Maps, the area is roughly 13mx13m. If your new address is within that same area as the old one, it will still have the same plus code.
If this is not the case (your new address is far away from the old one), could you describe more what you mean? Are you talking about the plus code in the left panel of Google Maps?

here:
https://www.google.com/maps/place/Rocha+Engenharia/@-27.0996503,-48.9226397,18.75z/data=!4m5!3m4!1s0x0:0x689338c56f1ceec4!8m2!3d-27.0999247!4d-48.922615?hl=pt-BR
the correct is: "Centro 1, Brusque-sc, and not "Aymoré, Guabiruba-sc"
Can you help please?

"Are you talking about the plus code in the left panel of Google Maps?"
yes.

Thanks for clarifying. How did you change your address? Was it through Google My Business?

You're wellcome, yes I change through GMB. Thanks for helping.

At the moment, the locality used in the plus code does not come from the address, but from the best guess for that location (given the plus code constraints). This is because some localities are too large, and some are so small and obscure that they aren't well known.
(Remember that ""W32G+2X Aimoré, Guabiruba" means the nearest W32G+2X to Aimoré, Guabiruba. It doesn't have to be within it, just near it.)
In your case, you moved your business. The plus code changed, but because it was a short distance, the locality in the plus code address doesn't update right away. It will be recomputed in a couple of days, and then the plus code address will be "W32G+2X Brusque, SC, Brasil" (like all the neighbouring businesses). In the meantime it still works - it will get someone to your new location.
All of that said, this is something we're looking at and planning to address (if you'll forgive the pun) by reusing the locality from your street address where possible.
(I've edited the issue title slightly to reflect the problem.)


Crash on random seed

Whenever I try to land on a moon, I get stuck on the random seed screen
Here's the log:
LogOutput.log

Thank you for reporting the issue.
This part is probably an issue with compatibility with other mods.
Could you send me the code of the mods you have used?
So I can test it more thorough and maybe find a way to fix the issue.
This problem does not occur with mods that I use to test, so I cannot know what the specific issue is now.
But I can guess either an incompatibility with mods that adds moons or something else.

Sorry for late response, here is my profile code :
018ef7c6-964b-e896-5342-ac4e1bc8d920

That is all fine I will take a look at this later.
If you want a quick fix for this, then you need to disable the body spawning in the config for the Remnant mods.
You can do this by going to the config section GeneralBody and setting Body spawn rarity  value to 0.
This bug should be fixed on the next update.

This is bug on my end, I forgot to take in mind that the prop version of the bodies are not compatible with the random suits version yet. So setting it to scrap should fix the issue as well.
This bug should be fixed on the next update.
Thank you for reporting the bug.

The bug should be fixed now, but ill leave this issue open for a week to be sure.


feat(worker): add HoleCounter module to geometry

Overview

add HoleCounter module to geometry

What I've done
What I haven't done
How I tested
Screenshot
Which point I want you to review particularly
Memo

Summary by CodeRabbit

New Features

Introduced a new HoleCounter processor that counts the number of holes in a geometry and adds it as an attribute.

Enhancements

Added the hole_count() method to various geometric entities including points, lines, and polygons, enabling hole counting functionality.

Bug Fixes

Updated error handling to include errors related to HoleCounter processing.

@coderabbitai summary

@coderabbitai help

@coderabbitai review


refactor releases project

preload all feature reviews to increas performance
no longer use release query to simply structure

@dideler please review

LGTM, just some minor comments. Nice work!

ping @dideler Chris approved the task, please merge.

Thanks @andistuder, ready for :rocket:


Changed button type names

aka fixing @shreyshrey1's broken code :))))) 👍

My code is perfect thank you very much


Add a means of more type-safe insert and updates

Since inserts and updates operate on a single table, provide a means of constructing both of these in a way that's not just raw ContentValues creation.

Use https://github.com/square/sqldelight.