RouteChange, MediaServiceLost/Reset and SilenceSecondaryAudioHint for iOS

This is based on the commented out code already in the repository.
I have tested the RouteChangeNotification and MediaServicesWereLost/Reset quite well, but haven't found a way to test the SilenceSecondaryAudioHintNotification.

Awesome! This was the next thing on my todo list (after finishing porting audio_service to media2).
I will take a deeper look when I get a chance, but do you think we have enough now to make AudioSession.becomingNoisyEventStream work on iOS?

Glad that i could help! You may want to look if i was successful following the naming conventions.
The becomingNoisyEvent on Android is equivalent to AVAudioSessionRouteChangeNotification: AVAudioSessionRouteChangeReason.oldDeviceUnavailable.
We could send an event to the AudioSession.becomingNoisyEventStream  or have users listen to the RouteChangeNotificationStream.

Just to give some of my thinking behind the design, AudioSession provides exposes a unified API for features that exist in some form on multiple platforms, while AVAudioSession and AndroidAudioManager provide full access to platform-specific features that may not exist on other platforms.

Here is a pretty clean Swift solution for checking if the old route is a headphone route:
https://developer.apple.com/documentation/avfoundation/avaudiosession/responding_to_audio_session_route_changes

I had this very page open among my 100 tabs waiting for me to get back to, and I was reminded why I let 100 tabs build up ;-)
To implement becomingNoisyEventStreamwe'll need AVAudioSessionRouteDescription which is not a particularly straightforward API to wrap in Dart, because some of the nested objects within this data structure will need access to the method channel.
So what I would suggest for this PR is to leave the AVAudioSession specific streams out of AudioSession but keep them in AVAudioSession which apps can access directly if needed.

Thank you for the insight into the design! I will make these changes in the morning 👍
Could i also add a getter for AVAudioSession and AndroidAudioManager to access all the juicy stuff?
One thought... Given the complexity of the AVAudioSessionRouteDescription I am wondering if you would like to take a shortcut there and just check if the previousRoute is a headphone route, and if so pass an event to `becomingNoisyEventStream. Or if you think it would be more useful to implement it fully?

Could i also add a getter for AVAudioSession and AndroidAudioManager to access all the juicy stuff?

I was originally planning to provide getters like this, but actually both classes have factory constructors that always return the same singleton instances, so an app can just access AVAudioSession() directly.

One thought... Given the complexity of the AVAudioSessionRouteDescription I am wondering if you would like to take a shortcut there and just check if the previousRoute is a headphone route, and if so pass an event to `becomingNoisyEventStream. Or if you think it would be more useful to implement it fully?

It might be possible to implement a minimal sub-fragment of AVAudioSessionRouteDescription that's necessary to support becomingNoisyEventStream, and pass that through the method channel in "onRouteChange". I'll leave some comments on your commit to indicate where. This may mean that my commented out code for the stream might not be adequate:
  //final _routeChangeSubject = PublishSubject<AVAudioSessionRouteChangeReason>();

since what I was missing here was the route description pulled out of userInfo.

I will take a look at this again tomorrow, but have to run out atm 👍

I did a little research and I'm going to make a few changes to this PR.
This example here is confusing and is just giving a code example to check if the previous route was a headphone route:
https://developer.apple.com/documentation/avfoundation/avaudiosession/responding_to_audio_session_route_changes
But according to this guide here and verified by testing, AVPlayer pauses on .oldDeviceUnavailable  no matter what the PortType is in the AVAudioSessionRouteDescription for AVAudioSessionRouteChangePreviousRouteKey:
https://developer.apple.com/library/archive/documentation/Audio/Conceptual/AudioSessionProgrammingGuide/HandlingAudioHardwareRouteChanges/HandlingAudioHardwareRouteChanges.html
So basicly we just need to pass an event to AudioSession.becomingNoisyEventStream when we have a routeChange with .oldDeviceUnavailable.
I'll submit it in a few hours :)

I recall reading that AVPlayer automatically pauses and also noticed this behaviour - I'd like to actually disable this behaviour if I could so that I could handle it manually and therefore consistently on iOS and Android, but I haven't figured out a way to configure that aside from building my own player from scratch using AVAudioEngine.
Regarding the route change logic, my main concern is that each new API addition needs to be designed with backwards-compatibility in mind. Since there is ultimately an extra piece of information in the userInfo that will eventually be included in the route change event, if this is added later, it will be a breaking API change. So at the very least, the routeChangeStream needs to be of some object that encapsulates both the reason and the description. If the description is ultimately unncessary for now, then at the very least, we still need this class even if it just has one field (the reason), so that it will be backwards compatible. Maybe something like this:
class AVAudioSessionRouteChange {
  final AVAudioSessionRouteChangeReason reason;
  // add everything else later
}

final _routeChangeSubject = PublishSubject<AVAudioSessionRouteChange>();
Stream<AVAudioSessionRouteChange> get routeChangeStream => _routeChangeSubject.stream;

That sounds very logical and a valid concern. I made the changes to follow the API design.
Ahh yeah I often feel like AVPlayer is a blackbox and sometimes works in mysterious ways.
Also thanks for taking the time to go through all this with me. I have definitely learned a lot. 👍

Looks good! I've just merged it. Thank you very much for contributing to this code.

Thank you for taking the time to go through it! 🎉

Just FYI, I added a whole bunch of missing methods and data structures for iOS over the weekend, so it would now be possible to add the missing userInfo keys for route change notifications. Specifically, the previous route. Although I think rather than communicate this over the platform channel on each notification, it should be possible for the Flutter side to just cache the current route and use it as the next previous route.
I've also implemented the corresponding features on Android now and updated the example to display a list of connected input/output devices. First each platform defines its own device info data structures, and then I try to layer a cross-platform data structure containing the common attributes over the top of that.


Add enclosingType to InputModelProperty

Contributes to https://github.com/microsoft/typespec/issues/4091

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp.Input


queue_processors: Fix UnboundLocalError in QueueProcessingWorker.

consume_time_seconds wasn't properly defined at the beginning, and in
some cases the finally: block could be entered with it still undefined.

I guess in particular the issue is a BaseException being thrown, e.g. when the worker is restarted?

Merged as a7039c815e338c003dfb1bc8b39724e0671d25eb after explaining the issue more fully.  Thanks @mateuszmandera!


Error build release if using rxdart dependency

compile error:
compiler message: file:///Users/ik/.pub-cache/hosted/pub.dartlang.org/rxdart-0.17.0/lib/src/observable.dart:1909:33: Error: The method 'retype' isn't defined for the class 'dart.async::Stream<#lib1::Observable::T>'.
compiler message: Try correcting the name to the name of an existing method, or defining a method named 'retype'.
compiler message:       new Observable<R>(_stream.retype<R>()); // ignore: deprecated_member_use
compiler message:                                 ^
Compiler terminated unexpectedly.

How reproduce:

create new project
include dependency rxdart: "^0.17.0"
use Observable in some class: example

import 'package:rxdart/rxdart.dart';
....
  void _incrementCounter() {
    Observable.just("");
    setState(() {
      _counter++;
    });
  }
....

build apk, flutter clean, flutter run --release
compile error, crashes apk on launch

[✓] Flutter (Channel master, v0.5.6-pre.64, on Mac OS X 10.13.2 17C205, locale ru-RU)
    • Flutter version 0.5.6-pre.64 at /Users/ik/Repositories/flutter
    • Framework revision def7634ba0 (20 hours ago), 2018-06-25 13:38:08 +0200
    • Engine revision 549c855e89
    • Dart version 2.0.0-dev.63.0.flutter-4c9689c1d2

[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/ik/Library/Android/sdk
    • Android NDK at /Users/ik/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[!] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    ! CocoaPods out of date (1.5.0 is recommended).
        CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
        For more info, see https://flutter.io/platform-plugins
      To upgrade:
        brew upgrade cocoapods
        pod setup

[✓] Android Studio
    • Android Studio at /Applications/Android Studio 3.2 Preview.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b02)

[✓] Android Studio (version 3.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b01)

[✓] IntelliJ IDEA Community Edition (version 2018.1.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 25.0.2
    • Dart plugin version 181.4892.1

[✓] Connected devices (2 available)
    • Redmi Note 5A Prime • 5ca7d459                             • android-arm64 • Android 7.1.2 (API 25)
    • iPhone X            • D3E1B07B-3AD0-4713-830C-111CAAA54581 • ios           • iOS 11.4 (simulator)

You need to upgrade to rxdart 0.18.0

Yh me too. It was just a version problem.


Feedstock meta.yml defined dependencies require update to match current state

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Issue
@mxr-conda
In current state, meta.yml defined dependencies do not match what is defined in adversarial-robustness-toolbox==1.18.1.  This is causing both adversarial-robustness-toolbox to fail installing as well as downstream consumers to be impacted when installing within python 3.11 environments.
Example:
Could not solve for environment specs
The following packages are incompatible
├─ adversarial-robustness-toolbox 1.18.1**  is installable and it requires
│  └─ scikit-learn >=0.22.2,<1.1.0  with the potential options
│     ├─ scikit-learn 0.22.2.post1 would require
│     │  └─ python >=3.6,<3.7.0a0 , which can be installed;
│     ├─ scikit-learn 0.22.2.post1 would require
│     │  └─ python >=3.7,<3.8.0a0 , which can be installed;
│     ├─ scikit-learn 0.22.2.post1 would require
│     │  └─ python >=3.8,<3.9.0a0 , which can be installed;
│     ├─ scikit-learn [0.23.0|0.23.1|...|0.24.2] would require
│     │  └─ python_abi 3.6.* *_cp36m, which can be installed;
│     ├─ scikit-learn [0.23.0|0.23.1|...|1.0.2] would require
│     │  └─ python_abi 3.7.* *_cp37m, which can be installed;
│     ├─ scikit-learn [0.23.0|0.23.1|...|1.0.2] would require
│     │  └─ python_abi 3.8.* *_cp38, which can be installed;
│     ├─ scikit-learn [0.23.2|0.24.0|0.24.1|0.24.2] would require
│     │  └─ python_abi 3.6 *_pypy36_pp73, which can be installed;
│     ├─ scikit-learn [0.23.2|0.24.0|...|1.0.2] would require
│     │  └─ python_abi 3.9.* *_cp39, which can be installed;
│     ├─ scikit-learn [0.24.1|0.24.2|1.0|1.0.1|1.0.2] would require
│     │  └─ python_abi 3.7 *_pypy37_pp73, which can be installed;
│     └─ scikit-learn [1.0.1|1.0.2] would require
│        └─ python_abi 3.10.* *_cp310, which can be installed;
└─ python 3.11**  is not installable because it conflicts with any installable versions previously reported.

In the above example, a conda environment using python=3.11 and adversarial-robustness-toolbox=1.18.1 will fail to solve due to the stale constraint definition of scikit-learn.
Installed packages
Unable to list due to unsolvable tree.  Simple test environment for reproducing issue can be made with:
name: art-test
channels:
  - conda-forge
  - nodefaults
dependencies:
  - python=3.11
  - adversarial-robustness-toolbox=1.18.1

Environment info
n/a

Resolving this issue and merging #48 supersedes #42 (which should be closed when my pull request is completed).

@mxr-conda
I am a team member that works with the development and maintenance of https://github.com/IBM/heart-library (Which has member crossover with adversarial-robustness-toolbox).  I am willing to take a maintenance role on this feedstock if you would like to add me.


Add dockable-mcs to cdnjs

Is your feature request related to a problem? Please describe.
Dockable-mcs is not available in libman library manager for .net core web apps
Describe the solution you'd like
Add dockable-mcs to cdnjs
Describe alternatives you've considered
Using npmjs

JSDelivr can be used instead of cdnjs
You can find it here: JSDelivr
Add this to the libman.json file to include draggable-mcs in your root folder
{
  "provider": "jsdelivr",
  "library"<EMAIL_ADDRESS>  "destination": "wwwroot/lib/draggable-mcs/",
  "files": [
    "dist/css/draggable-mcs.css",
    "dist/css/draggable-mcs.css.map",
    "dist/css/draggable-mcs.min.css",
    "dist/js/draggable-mcs.js",
    "dist/js/draggable-mcs.js.map",
    "dist/js/draggable-mcs.min.js",
    "dist/js/draggable-mcs.min.js.map"
  ]
}


Use a custom type for our static CLI client.

Before this change, we used the fosite.DefaultOpenIDConnectClient{} struct, which implements the  fosite.Client and fosite.OpenIDConnectClient interfaces. For a future change, we also need to implement some additional optional interfaces, so we can no longer use the provided default types. Instead, we now use a custom staticclientt.PinnipedCLI{} struct, which implements all the requisite interfaces and can be extended to handle the new functionality (in a future change).
The only user-observable change is that the storage format has been simplified. Instead of a full JSON serialization of the client, we now store only the client ID ({"id": "pinniped-cli"}). This is backwards compatible when existing refresh sessions or other stored values are read by the new code. There will be a brief (~60s) period of unavailability during upgrades due to older Supervisor pods being unable to correctly deserialize the new JSON.
Release note:
NONE

Going to take another shot at this with a smaller change in https://github.com/vmware-tanzu/pinniped/pull/673.


Template Matrix/Vector to allow any type.

Describe the solution you'd like
Allow matrix and vector to be a double/float/etc.

Float were changed to double. No need for lesser accuraccy.


Alter check for hipRAND to check CMakelists exists

I believe that the failed test on Windows CI is unrelated to this change.


Two Puzzling Issues in Linux

System Information:
Operating System: Arch Linux
KDE Plasma Version: 6.0.3
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.3
Kernel Version: 6.8.2-arch2-1 (64-bit)
Graphics Platform: X11
Processors: 8 × Intel® Xeon® CPU E3-1240L v5 @ 2.10GHz
Memory: 31.3 GiB of RAM
Graphics Processor: NVIDIA GeForce GT 730/PCIe/SSE2

Issue One:
The following Rust code snippets behave differently, which is quite perplexing:
let mut board = arboard::Clipboard::new().unwrap();
board.set_text("hello".to_owned()).unwrap();

versus
arboard::Clipboard::new().unwrap().set_text("hello".to_owned()).unwrap();

Issue Two:
It might be a KDE issue or something related to asynchronous operations:
let mut board = arboard::Clipboard::new().unwrap();
board.set_text("hello".to_owned()).unwrap();
panic!()

This code does not work as expected, but adding a sleep before the panic:
let mut board = arboard::Clipboard::new().unwrap();
board.set_text("hello".to_owned()).unwrap();
sleep(Duration::from_millis(100));
panic!()

makes it work. I suspect that it might take some time for the information to be copied to the clipboard.
Regarding the duration of the sleep, it generally needs to be more than 5 ms on my system, which I assume is related to system performance.
Otherwise, the clipboard remains unchanged, or I end up with an empty clipboard.
Translated by Copilot

I've also meet the issue2, and maybe #143 and #114 are related to this


GDScript: Add warnings that are set to error by default (take 2)

This is the same as the reverted #72487 with the pointed issues fixed, plus tests for these to avoid future regressions. Also added a check for cast for the warning about get_node() without @onready (more complex expressions are not caught, no warning will be generated).

Adds a list of default levels for all warning so they can be set individually.
Add warnings set by default to error for:

Using get_node() without @onready.
Using @onready together with @export.
Inferring a static type with a Variant value.
Overriding a native engine method.

Adjust how annotations to ignore warnings are treated so they also apply to method parameters.
Clean up a bit how ignored warnings are set. There were two sets but only one was actually being used.
Set all warnings to the WARN level for tests, so they they can be properly tested.
Fix enum types in native methods signatures being set to int.
Fix native enums being treated as Dictionary by mistake.
Make name of native enum types use the class they are defined in, not the direct super class of the script. This ensures they are always equal even when coming from different sources.
Fix error for signature mismatch that was only showing the first default argument as having a default. Now it shows for all.

Fix https://github.com/godotengine/godot/issues/46382
Close https://github.com/godotengine/godot/issues/55024

CC @maiself @Chaosus @HungryProton @tapir2342
Would be great if you could test this PR as you got regressions in your project from the previous version.
CC also some with known big 4.0 codebases: @KoBeWi @unfa

Just finished testing and was able to open my project without crashes or other trouble, fix the new script errors, and everything is working fine. :+1:

Tested with my project and it doesn't seem to cause any unwanted errors or warnings.

Works on my project as well.

I tested this PR and found no errors. Except Using get_node() without @onready.: the warning only appears for $ and get_node, not similar methods (get_node_or_null, find_child). But I think it's ok.

Yep, seems working correctly in my project.

Thanks!

hi just a question to core function overriding.
Warn is Error now

Overriding a native engine method.

What this means?
Now will this break my mock/spy implementation because by nature a mock/spy overrides all functions to be mockable?
Is there an annotation to disable such errors?
And what about the desire of many users to override a core function?
Will there be a new annotation @override to enable this?

Warn is Error now
* Overriding a native engine method.

What this means? Now will this break my mock/spy implementation because by nature a mock/spy overrides all functions to be mockable? Is there an annotation to disable such errors?

@MikeSchulze You can annotate a method with @warning_ignore("native_method_override"), or you can change debug/gdscript/warnings/native_method_override project setting so it won't be treated as an error.

And what about the desire of many users to override a core function?
Will there be a new annotation @override to enable this?

There is no such functionality in Godot. To be overridable each engine method needs to be implemented in the engine itself as virtual, and that's not going to happen for performance reasons. Godot exposes other means to hook into engine behavior, such as signals, notifications, and existing virtual methods. Each of them has a well-defined use case to explain the need for its existence, and more can be added if there is a similarly good reason for it. A proposal needs to be made for each case.

@MikeSchulze Why it's done:

The engine doesn't call your "overrides" in most cases (unless it calls them via call()). This behavior is also present in 3.x, it's not a bug, it can't be fixed without a huge performance hit, it would introduce a lot of new bugs due to the violation of the engine's expectations when the user overrides the method incorrectly.
Unlike 3.x, the override call may not happen even in GDScript, due to type collisions (in some cases, GDScript calls a native method for optimization, unless you work around it with a cast or call()). For this reason, non-virtual native methods should never be "overridden", even if you know about the previous point and don't worry about it.

See #55024 and #72363 for details. A few tips:

You can disable or ignore this warning, but correct behavior is not guaranteed.
This warning does not apply to native virtual methods and any custom methods. You can override them.
If you need to customize some engine method, you should create a wrapper with a different name, not "override" it.
I guess if users don't override native non-virtual methods then you won't need to mock them?

@dalexeev I know about the changes in Godot4, from code security I am also 100% with you.

If you need to customize some engine method, you should create a wrapper with a different name, not "override" it.

This will not help me, for a spy you need to track the interactions of the original functions, so using different names will not help.

I guess if users don't override native non-virtual methods then you won't need to mock them?
It is important to define custom return values for a given function INCLUDING the core functions.

I don't want to discuss here whether overrides should be allowed or not.
But it is a fact, in Godot3 I could realize spy and mock because the overshadowing of functions was possible, no matter if bug or not.
For me the question is how I can now realize a spy and mock in Godot4.
Normally this is done via code injection.
I would be grateful here for tips on how to collect without functions override interactions on core functions.

@MikeSchulze this should work the same as 3.x the only difference is that it is an error by default. You can turn it off (or change to warning only) in the project settings. You can also disable case-by-case using the @warning_ignore("native_method_override") annotation (as mentioned by @kleonc above).
The only thing in Godot 4 is that there are more call optimizations. So when the values are typed the compiler will take a faster route and call the engine method directly, which behaves as if it was called by the engine itself (and thus ignore overrides). This can be worked around by casting to the specific script type that has the function or casting to Variant to use a dynamic dispatch which will use the override.
As mentioned multiple types, there's not really any performant way to support this case.

I don't want to discuss here whether overrides should be allowed or not.
But it is a fact, in Godot3 I could realize spy and mock because the overshadowing of functions was possible, no matter if bug or not.

Well, if you were exploiting a bug, you can't really expect that the bug won't ever be fixed. So in fact this is a discussion about whether you can override or not.
Regarding how to do what you want, I'm not really sure it's possible from GDScript alone. It might need patches to the engine code itself. If you can find a feasible way to do it, we can definitely integrate in the engine.

@vnen  Thanks for the detailed answer. 👍
Casting to a Variant is interesting, I'll have to test that when I get a chance. That would definitely help me when a user of a mock casts to the original class.
Speaking of engine patches, would it be possible to inject code into the engine via the GdExtention?
That would also be interesting to integrate a code coverage tool.

I believe GDExtension will have the same problem. For virtual methods you can override with GDExtension, otherwise they won't always be called.


GstWebRTC does not work with MacOS 10.12.6

Hello, thank you for making this material available.
I have tried compiling the example webrtc-sendrecv.py but I did not succeed.
I Tried:
import gi
gi.require_version('Gst', '1.0')
from gi.repository import Gst
gi.require_version('GstSdp', '1.0')
from gi.repository import GstSdp
gi.require_version('GstWebRTC', '1.0')
from gi.repository import GstWebRTC

I get:
raise ValueError('Namespace %s not available' % namespace)
ValueError: Namespace GstWebRTC not available
I installed gst-python with "brew install gst-python"
brew info gst-python
gst-python: stable 1.14.1 (bottled)
Python overrides for gobject-introspection-based pygst bindings
https://gstreamer.freedesktop.org/modules/gst-python.html
/usr/local/Cellar/gst-python/1.14.1 (12 files, 395KB) *
Poured from bottle on 2018-06-23 at 03:19:10
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/gst-python.rb
==> Dependencies
Required: gst-plugins-base ✔, pygobject3 ✔
Recommended: python ✔
Optional: python@2 ✔
What could be wrong?
It's strange that Gst and Gst Sdp work while Gst WebRtc does not

The WebRTC bindings would require files from gst-plugins-bad, i.e. https://github.com/Homebrew/homebrew-core/blob/master/Formula/gst-plugins-bad.rb (same as the others require files from gstreamer core and gst-plugins-base). I'm not sure if Homebrew is including those, but if you don't have gst-plugins-bad installed please do so and try again.
There should be a GstWebRTC-1.0.typelib provided by that one.

I solved this issue. The reason of this issue is that when install gst-plugins-bad by homebrew,  GstWebRTC-1.0.typelib is not generated. So I generate it manually as follows

install these.

brew install libffi
brew install gobject-introspection

add export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:/usr/local/opt/libffi/lib/pkgconfig to PKG_CONFIG_PATH

clone gst-plugins-bad from https://github.com/GStreamer/gst-plugins-bad and run ./autogen.sh

move to gst-libs/gst/webrtc directory and run make then run make install

I think this is workaround solution and I'm looking for install GstWebRTC-1.0.typelib by homebrew.

In case it hasn't been spotted, the comments here talk about how 1.14.1 (the latest version that Homebrew has) isn't enough for Python WebRTC to work:
https://github.com/centricular/gstwebrtc-demos/commit/9a264159453b7ab60e77a41feb90d8f1ac6032a1
I don't think homebrew can be persuaded to checkout the 1.14 branch, but you could upgrade streamer/bad/ugly to the latest dev (1.15) using --head. Or wait for 1.14.2.

I installed gstreamer/bad/ugly using --head but python example not work in same issue.
https://github.com/centricular/gstwebrtc-demos/issues/26#issue-335134893

Using --head did not work. The version continues to be 1.14.1.
I had the same problem using apt-get on Ubuntu.

same issue.

After install  gstreamer/base/bad/good  all  from head,  still have this issuse.

Hi guys,
I have managed to fix GstWebRTC-1.0.typelib installation issue by adding (brew edit gst-plugins-bad):
depends_on "gobject-introspection" => :build
--enable-introspection=yes

as in:
https://github.com/Homebrew/homebrew-core/blob/master/Formula/gst-plugins-base.rb
and reinstalling the gst plugins
brew reinstall --build-from-source gst-plugins-base
brew reinstall --build-from-source gst-plugins-bad

@LucianZala  Hi,  can you give more details about where add that two lines?

@LucianZala  Thanks,  it works.

Hello. Thank you for the work you have done on WebRTC and congratulations on the fairly recent merge into GStreamer 1.4.x. As a regular user of GStreamer via command line and C/C++ I am having a difficult time getting started with WebRTC. I am trying to follow the Centricular demo and this blog, but there does not seem to be a concise/direct step-by-step guide. My goal is to create a server that streams to a mobile client, but I would happily start with any web client. I am on a Mac but have Docker experience. Is there a getting started guide that I have not yet found?

I think we can close this as homebrew has been updated.
@dmanpearl Have you looked at the README.md and http://blog.nirbheek.in/2018/02/gstreamer-webrtc.html ?

Hello. Thank you for the work you have done on WebRTC and congratulations on the fairly recent merge into GStreamer 1.4.x. As a regular user of GStreamer via command line and C/C++ I am having a difficult time getting started with WebRTC. I am trying to follow the Centricular demo and this blog, but there does not seem to be a concise/direct step-by-step guide. My goal is to create a server that streams to a mobile client, but I would happily start with any web client. I am on a Mac but have Docker experience. Is there a getting started guide that I have not yet found?

yes there is no real good guide on how it works, try the tutorials point webrtc tutorial and come back to this. It helped me a lot. and if you have questions ask


fix(ci): restore master and release CI workflow permissions.

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area CI
What this PR does / why we need it:
https://github.com/falcosecurity/falco/pull/3299 broke master CI triggering (if you see latest master commits did not trigger master CI, only codeql and scorecard).
This is an attempt to fix them.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.39.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

it would be great to understand why we cannot apply the permissions on this but I understand that we cannot have the master CI broken

We can actually, but we can't set smaller permissions set on the caller workflow when some called workflows (eg: reusable_publish_X workflows) require larger permissions set.


OCR-assisted Game Script / Text Dump matching

Implemented dropdown select with 4 best matches by looking at the entire log file. To improve performance, do regional scan of the previous match first (go forward and go backward ~100 lines before continuing with the rest of the file).

https://github.com/mathewthe2/Game2Text/issues/21#issuecomment-843996270
Remaining:

Set character minimum required for auto-matching (5 or up)
exact match to highlighted word

Implemented dropdown select with 4 best matches by looking at the entire log file. To improve performance, do regional scan of the previous match first (go forward and go backward ~100 lines before continuing with the rest of the file).

regional matching
https://github.com/mathewthe2/Game2Text/commit/5201c22e6d3cf9b03824a51a6693226b0719c7e4


Number format exceptions

Hi,
several of these messages appear in mycontroller log file after one of the my nodes sent an invalid sensor value (nan, -nan, inf):
2016-01-25 14:02:52,441 ERROR [Thread-3] [org.mycontroller.standalone.mysensors.MessageMonitorThread:73] RawMessage[NodeId:9,ChildSensorId:1,MessageType:1,Ack:0,SubType:38,Payload:-nan] throws exception while processing!,
java.lang.NumberFormatException: For input string: "-nan"
	at sun.misc.FloatingDecimal.readJavaFormatString(Unknown Source) ~[na:1.8.0_71]
	at sun.misc.FloatingDecimal.parseDouble(Unknown Source) ~[na:1.8.0_71]
	at java.lang.Double.parseDouble(Unknown Source) ~[na:1.8.0_71]
	at java.lang.Double.valueOf(Unknown Source) ~[na:1.8.0_71]
	at org.mycontroller.standalone.NumericUtils.getDouble(NumericUtils.java:48) ~[mycontroller-standalone-0.0.2-alpha6-single.jar:na]
	at org.mycontroller.standalone.mysensors.ProcessRawMessage.recordSetTypeData(ProcessRawMessage.java:588) ~[mycontroller-standalone-0.0.2-alpha6-single.jar:na]
	at org.mycontroller.standalone.mysensors.ProcessRawMessage.messageTypeSelector(ProcessRawMessage.java:95) ~[mycontroller-standalone-0.0.2-alpha6-single.jar:na]
	at org.mycontroller.standalone.mysensors.MessageMonitorThread.processRawMessage(MessageMonitorThread.java:61) [mycontroller-standalone-0.0.2-alpha6-single.jar:na]
	at org.mycontroller.standalone.mysensors.MessageMonitorThread.run(MessageMonitorThread.java:87) [mycontroller-standalone-0.0.2-alpha6-single.jar:na]
	at java.lang.Thread.run(Unknown Source) [na:1.8.0_71]

I also noticed some missing information about the node in the Action Board (webgui) after that.

@marceloaqno I hope your sensor sends V_VOLTAGE value. It should be a double value. MyController tries to convert -nan to Double value and failed. I believe this error won't effect your node details.
What exact information missing on action board?

@jkandasa yes, V_VOLTAGE values (it's a S_MULTIMETER type). Node 9 has 13 sensors, but only 7 of them are showing.

@marceloaqno can you send me the complete mycontroller.log? It might give some clue to narrow down this issue.

@jkandasa sure, https://drive.google.com/file/d/0B_-1ZYkXkahacGMxS0lxcTI0Q3M/view?usp=sharing

@marceloaqno Hi it looks like you are using alpha5 version of database with alpha6 version. There is schema changes in alpha6 version, It won't support for previous version of database. You have to start from fresh.

@jkandasa I did that (delete the file mycontroller.h2.db) after upgrading to alpha6.
Once I saw those lines on the log:
org.h2.jdbc.JdbcSQLException: Column "reachable" not found [42122-176]

I thought starting from scratch would be the case, I just kept the logs.

@marceloaqno did you note the time stamp of the log? Is it new log?

This expected issue(Number format exceptions) if we send unreadable measurement data.


cmd/cgo: type error when using pointers to C functions

If I try to pass a C function pointer back to C from Go, I get a type error.  It appears that cgo is mapping the function pointer type to *[0]byte (since there isn't a direct way to express C function-pointer types in the Go type system), but it isn't inserting the correct type conversions when referring to function-pointer values:
cgocallback/main.go:
package main

/*
#include <stdio.h>

static void invoke(void (*f)()) {
	f();
}

void print_hello() {
	printf("Hello, !");
}
*/
import "C"

func main() {
	C.invoke(C.print_hello)
}

bcmills:~$ go build cgocallback
# cgocallback
src/cgocallback/main.go:17: cannot use _Cgo_ptr(_Cfpvar_fp_print_hello) (type unsafe.Pointer) as type *[0]byte in argument to _Cfunc_invoke

The workaround is to define a typedef for the function pointer type an explicitly convert to the typedef, but it would be nice if that workaround were not necessary.
package main

/*
…
typedef void (*closure)();
*/
import "C"

func main() {
	C.invoke(C.closure(C.print_hello))
}

bcmills:~$ go version
go version devel +2bbfa6f746 Thu Mar 9 15:36:43 2017 -0500 linux/amd64

(CC: @ianlancetaylor @thanm)

func main() {
	C.invoke((*[0]byte)(C.print_hello))
}

func main() {
	C.invoke((*[0]byte)(C.print_hello))
}

this work for me

solved my question ! TKS !


Session.window in sesh.toml

What would you like sesh to do?
Hi Josh
First of all thanks for a cool session manager! I want to hear if you have thought about implementing session windows in the sesh.toml file such that you can automatically create sessions with multiple windows inside. It could have the following example structure
  [[session]]
  name = "sesh"
  path = "~"
  startup_command = "nvim ."

  [[session.window]]
  session = "sesh"
  name = "zsh"

  [[session.window]]
  session = "sesh"
  name = "Service1"
  startup_command = "nix develop && task service1"

Let me know what you think or if this is too big of a feature request for now.
I have been quite inspired by your dotfiles (especially gp.nvim) in my Nix dotfiles setup but hope to see sesh as part of home-manager soon as well. 😉

I like this idea, I think it's worth exploring, thanks for the idea!


feat: Add ecr login

What does this PR do?
get ecr token to pull images/charts from public ECR
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

Resolves #

Test Results

Additional Notes

@bryantbiggs  i tested locally. it works.
it needs data "aws_ecr_authorization_token" "ecr_token" {}
and local-exec

Should the ecr login use us-east-1 region?

thats a good point - I think we will have to show users how to pass this in, and make that as easy as possible (expose as top level variables) so that we don't embed the provider into the module

so to clarify, what I would propose is:

Revert this change
Add variables for repository_username and repository_password and pass those down to the helm configs
Update the example and documentation to call this out for folks. Show that they will need the data source and then how to pass its outputs into the module inputs


aggregate clause after append clause

What happened?
PRQL compiler version:0.9.2
This issue is similar to an existing one, https://github.com/PRQL/prql/issues/2680. But I'm not sure if the root causes of those two issues are the same. So I open a new one.
I found that the aggregate clause also affects the translation of the append clause.
In the following example, assuming tb1 and tb2 have the same columns, we append tb2 to tb1, and get the minimum c0. But in the SQL code, it selects the target column before the union operation. Then the union operation fails.
error message from the database: "each UNION query must have the same number of columns"
PRQL input
from tb1
append tb2
aggregate {new_col1 = min c0}

SQL output
WITH table_0 AS (
  SELECT
    c0
  FROM
    tb1
  UNION
  ALL
  SELECT
    *
  FROM
    tb2
)
SELECT
  MIN(c0) AS new_col1
FROM
  table_0

-- Generated by PRQL compiler version:0.9.2 (https://prql-lang.org)

Expected SQL output
WITH table_0 AS (
  SELECT
    c0
  FROM
    tb1
  UNION
  ALL
  SELECT
    c0
  FROM
    tb2
)
SELECT
  MIN(c0) AS new_col1
FROM
  table_0

MVCE confirmation

[X] Minimal example
[ ] New issue

Anything else?
No response

Yes. I have spent less time on the append-like table operations, so I'll defer to others.
I note that writing it out more explicitly works:
let x = (
  from tb2
  select c0
)

from tb1
select c0
append x

WITH x AS (
  SELECT
    c0
  FROM
    tb2
)
SELECT
  c0
FROM
  tb1
UNION
ALL
SELECT
  *
FROM
  x

-- Generated by PRQL compiler version:0.9.2 (https://prql-lang.org)

Though we have quite poor error messages (which also don't show in the playground) if anything is forgotten:
let x = (
  from tb2
  select c0
)

from tb1
# select c0
append x

Error: Unknown name

I think this can be closed: compiler version 0.11.1 gives exactly this SQL output


[android] App crashing in release mode as soon as we enable proguard

The build works fine in the debug mode, but as soon as we enable the proguard, we have started getting the following exception and app is crashing right away, can someone suggest the right proguard rules to avoid this issue.
java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.String.equals(java.lang.Object)' on a null object reference
  at com.amplitude.reactnative.AmplitudeReactNativeModule.setServerZone(Unknown Source:7)
  at java.lang.reflect.Method.invoke(Native Method)
  at com.facebook.react.bridge.JavaMethodWrapper.invoke(Unknown Source:148)
  at com.facebook.react.bridge.JavaModuleWrapper.invoke(Unknown Source:147)
  at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
  at android.os.Handler.handleCallback(Handler.java:873)
  at android.os.Handler.dispatchMessage(Handler.java:99)
  at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(Unknown Source:0)
  at android.os.Looper.loop(Looper.java:193)
  at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(Unknown Source:37)
  at java.lang.Thread.run(Thread.java:764)

I have tried adding below proguard rules on top of what mentioned in the documentation to avoid any obfuscation of classes which might be stopping to invoke this method
-keep class com.amplitude.** {*;}

But it does not seems to be working.

Do you have -keep class com.google.android.gms.ads.** { *; } in your proguard rules?

Yes i have explicitly added this rule to my pro-guard rules.


Add guidance for conditional reveal error messages

Fixes #863.
This PR updates our radios guidance to tell users how to add an error message to a conditionally revealed question.
We're adding this content because users occasionally ask about this issue. For example, in the radios discussion and on the original issue itself.

@calvin-lau-sig7 Cheers for the collab! Have added in the changes. Can I get one more review to make sure things look ok? :)


Develop use-cases for TornadoVM

The TornadoVM repository contains a module with several examples. Include more use cases for acceleration with TornadoVM, such as a graphical version for NBODY, filter for photography, block-chain. etc.

Sorry, I couldn't find an example of blockchain. Can you provide it?

Hi @chenyu2016 , that's what this issue is about. I will rephrase it because it seems that is not clear.
What I meant is that, in the current version of TornadoVM, there is a package dedicated to examples. This issue is open to adding more examples that use TornadoVM for hardware acceleration. Some examples of potential use cases are: computational phography, crypto, block-chain, etc. , on top of the existing ones.

AI inference and training in different apps and services
ray-tracing rendering in CAD/photo/movie apps and games
discrete modeling in apps for termo/aero/hydro dynamic physics, game of life, etc.


Sharebot issue

Whenever I run the sharebot and enter lets say around 100000, it will randomly stop at 1k-10k

Any errors?

hm

probably there are a error with threads
when a error occurs with threads it will try to stop all threads
i put a try on the line 111 & 118 when script create threads
download the new script


Type error to do with BaseLanguageModel and CallOptions

Introduced here I believe: https://github.com/langchain-ai/langchainjs/pull/3709
0.0.203 is a working version.
Stack trace:
node_modules/@langchain/core/dist/language_models/base.d.ts:122:31 - error TS2420: Class 'BaseLanguageModel<RunOutput, CallOptions>' incorrectly implements interface 'BaseLanguageModelInterface<RunOutput, CallOptions>'.
  Property 'CallOptions' is missing in type 'BaseLanguageModel<RunOutput, CallOptions>' but required in type 'BaseLanguageModelInterface<RunOutput, CallOptions>'.

122 export declare abstract class BaseLanguageModel<RunOutput = any, CallOptions extends BaseLanguageModelCallOptions = BaseLanguageModelCallOptions> extends BaseLangChain<BaseLanguageModelInput, RunOutput, CallOptions> implements BaseLanguageModelParams, BaseLanguageModelInterface<RunOutput, CallOptions> {
                                  ~~~~~~~~~~~~~~~~~

  node_modules/@langchain/core/dist/language_models/base.d.ts:104:62
    104 export interface BaseLanguageModelInterface<RunOutput = any, CallOptions extends BaseLanguageModelCallOptions = BaseLanguageModelCallOptions> extends RunnableInterface<BaseLanguageModelInput, RunOutput, CallOptions> {
                                                                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    'CallOptions' is declared here.

Found 1 error in node_modules/@langchain/core/dist/language_models/base.d.ts:122

I'm having the same issue, can't use latest version. any workaround besides of using 0.0.203?

Workaround: Add "skipLibCheck": true to your tsconfig.json
{
  "compilerOptions": {
    .....
    "skipLibCheck": true,
  }
}

having same problem, using workaround. FYI @jacoblee93 something strange going on here...

Thanks for the heads up and sorry for missing this - the workaround is good but is there a code example I can use to repro this?
If you have multiple packages installed do the instructions here fix it?
https://js.langchain.com/docs/get_started/installation#installing-integration-packages

@mieslep perhaps?

Thanks for the heads up and sorry for missing this - the workaround is good but is there a code example/package.json/tsconfig.json that I can use to repro this?
If you have multiple packages installed do the instructions here fix it?
https://js.langchain.com/docs/get_started/installation#installing-integration-packages

I've had no luck following the linked instructions. Here's what you can use to reproduce the errors:
package.json
{
	"type": "module",
	"scripts": {
		"broken": "tsc -m nodenext main.ts",
		"works": "tsc -m nodenext --skipLibCheck main.ts"
	},
	"dependencies": {
		"@langchain/openai": "latest"
	},
	"devDependencies": {
		"typescript": "latest"
	}
}

main.ts
import { ChatOpenAI } from "@langchain/openai";

How about a tsconfig.json file @5-tom?

I think #3953 fixes this! Can you try locking your core version to 0.1.11?
I was occasionally able to repro another typing bug caused by a polyfill overriding a global stream in that OpenAI's SDK uses. Please reopen another issue if you see this.


Wanted to generate password related to user preference

Wanted to generate password related to user preference so it is easier for user to remember
for eg. asking user email id, his/her first & last name and date of birth then, generating random password by combining mail, name and dob

Go ahead.
But don't modify this UI.
Instead create a separate page for it. Let's Have this feature as an option.

sure

please assigned it under hacktoberfest


Feature - Change Remaining References to Lead School

Context
Trello
We need to rename every reference to lead_school and its variations to lead_partner. This will be a big change, so let's use this branch as a feature branch until we're ready.
Changes proposed in this pull request
Rename all lead_school references to lead_partner, fixing any broken tests and removing code where it makes sense.
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

Thanks @kennyevil, you're right on both counts, but they'll be addressed in later PRs. I'm happy to keep this open, so we can see all the changes together before merging to main. Or maybe it's best to merge now, what do you think?

@adamjcolvin Let's merge it now, you've got the second one open so it can be sorted out there.


Add theme selector UI

Currently the app uses a hardcoded theme (in rewrite) of light theme with orange accents. Add a theme selector UI to allow the user to change the theme and accent colours.

Is it possible, on W10, to use theme colors? ie, those used by the system?

It might be. I'm not quite sure, but the UI framework I'm using does support custom themes.

Implemented with commit 363179a

Much obliged!


[BACKLOG-32493] Update 'Bulk load from MySQL into file' job entry to …

…use enhanced VFS browser. Update to use Save To Dialog

:white_check_mark: Build finished in 10m 50s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1125, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Make typescript compile again

After reading #284 I decided to take a stab at seeing if I can get Typescript to work (again).
Although this PR is mergable, I consider it a bit of a Work in Progress. But I think the discussion about maybe switching to Typescript works better if you have some actual code to point at. :)
Things I did:

Removed building of the site.min.js from the gulpfile. It isn't used anywhere in the site and it was confusing that you have a site.js (which is used) that doesn't correspond with site.min.js, but is actually just a small part of that file's contents.
Fixed tsconfig.json so it doesn't give an error anymore and actually compiles/transpiles stuff.
Added compiled/transpiled versions of the files in wwwroot/ts.
Renamed wwwroot/js/site.js to wwwroot/js/site.ts as a sample. You can see that the now generated site.js closely matches the previous version.
Tweaked wwwroot/js/site.ts a bit to sprinkle some Typescript goodness on it. (And to get rid of the many warnings it was giving.)
Added Typings for jquery and bingmaps. Had to reference the latter one manually from site.ts as it delivers it .d.ts in a non-standard way.
Put the devDependencies in packages.json in alphabetical order, updated gulp-typescript, added the typescript dependency and dropped the now unused gulp-minify dependency.
Made a new gulp build task that transpiles the .ts files (and incorporates the old task that minifies the site.css file), so people who don't use Visual Studio can use it as well.
Aliased the old gulp min task to gulp build for backwards compatibility in case people depend on the old name.

Questions I have:

Are the files in webroot/ts used? I can't find any reference to them and they were missing their .js counterparts
Do we want bundling and minification for the resulting .js files?
What settings do we want to use in tsconfig.json?
Do we want the generated .js files in sourcecontrol?

Might want to exclude the .map files as they are generated artifacts and will change every build.

@BBosman thanks for this! As per @stimms suggestion, let's remove the map files from the PR and perhaps even add an exclude in the gitignore for *.map under any related JS trees. That should seal this one up.

I dropped the *.map files and added an entry for them in the .gitignore file.

@MisterJames can you review after the edit per your and @stimms comment? Thanks

:shipit:


Exception using IN/OUT in remote query

OrientDB Version: 3.1.5
Java Version: 8.0.222.hs-adpt
OS: OS 11 (Big Sur)
Expected behavior
Executing a query that includes LET with traversals to IN and OUT and combinations returns the results for following the edges in the methods.
Actual behavior
Exception on server side
either java.lang.ClassCastException: com.orientechnologies.common.collection.OMultiCollectionIterator cannot be cast to java.util.Collection or Error while binary response serialization com.orientechnologies.orient.core.exception.OSerializationException: Impossible serialize value of type class com.orientechnologies.orient.core.record.impl.OEdgeToVertexIterable with the Result binary serializer depending on the query being executed.
Steps to reproduce
I have added 2 new unit tests to com.orientechnologies.orient.server.query.RemoteQuerySupportTest
  @Test
  public void testQueryWithOut() {
    session.command("create class testScriptWithRidbagsV extends V");
    session.command("create class testScriptWithRidbagsE extends E");
    session.command("create vertex testScriptWithRidbagsV set name = 'a'");
    session.command("create vertex testScriptWithRidbagsV set name = 'b'");

    session.command(
            "create edge testScriptWithRidbagsE from (select from testScriptWithRidbagsV where name = 'a') TO (select from testScriptWithRidbagsV where name = 'b');");

    String script = "select name, $o.name from testScriptWithRidbagsV let $o=OUT(\"testScriptWithRidbagsE\")";

    OResultSet rs = session.query(script);

    rs.forEachRemaining(x -> System.out.println(x));
    rs.close();
  }

  @Test
  public void testQueryWithOutThenIn() {
    session.command("create class testScriptWithRidbagsV extends V");
    session.command("create class testScriptWithRidbagsE extends E");
    session.command("create vertex testScriptWithRidbagsV set name = 'a'");
    session.command("create vertex testScriptWithRidbagsV set name = 'b'");

    session.command(
            "create edge testScriptWithRidbagsE from (select from testScriptWithRidbagsV where name = 'a') TO (select from testScriptWithRidbagsV where name = 'b');");

    String script = "select name, $p.name from testScriptWithRidbagsV let $p=OUT(\"testScriptWithRidbagsE\").IN(\"testScriptWithRidbagsE\")";

    OResultSet rs = session.query(script);

    rs.forEachRemaining(x -> System.out.println(x));
    rs.close();
  }

Note that this error does not occur if the query is performed in the GUI of the database.

Hi @pkendall64
Thank you very much for reporting, I'll check it ASAP and let you know
Thanks
Luigi

Hi @pkendall64
Sorry for the late reply, I have a fix for this in a branch https://github.com/orientechnologies/orientdb/tree/no_rs_metadata_serialization
I just have to validate it and make sure it does not break any of the existing functionalities and then I'll merge it
Thanks
Luigi


IMPORTANT! Security Vulnerability Detected in your Repository

Hey there!
We have detected a security vulnerability in your repository. Please contact us at REDACTED to get more information on how to fix this issue.
Best regards,
Github Security Team

This is so pathetic to try to scam users on GitHub of all places.


23 Related Safe Addresses (Report by d3centra)

Related Safe Addresses
0x4a61E2d459f24378fAc26721705BB4a974DF4e93
0x3Bef9cFfEEe4da69fBA9c9b47CC9A75c50e62e8b
0x9937c074e94eCd7D7C23B3b963106eb47fA9235B
0xacc819aBcfaaD2b8E0fFbd701a86d72cfd10407e
0xea166EFbCc7Ca9bD86d665512C488CdDcA5Bdae3
0x7d027B807444dA6252a187B044DE3a196120E897
0x01069fF7f59514Ade22E8F682B7334676C00c56c
0x68A54972e4faeb548C23214FC949197b85058d0E
0xc802b29dE31cdb42B5406AADa2f8DAe24121D248
0xc6101C2595db8bb9179De79ADB44F0ED6683D419
0x3086AB78F3Ec3298545346bD9Dc167ebE1539Ae1
0x673F7Db60E99EAf80e26a2F0e3c716B0e8d15dfD
0x0982eb56Be2F16DE53D76f75FBAC4dF2D69b9CA7
0x85C1217921751f146Df95dcb6a8E8e9f85554D54
0x426c90fe3Dc43AC291Ef849CD3c74dEbB5CCD23a
0x08B8D1D7C92357EEBE892A51e3185B770A58f485
0x174C99D6fFE6e4b7f39d2f33fA73ab3D23E168d5
0x83C6Efecd8084722475f23708E7a260EE8Db52c4
0x744f1Cb0e1baDaDAA951C7e9Db8218F04B70716B
0x7deDc6BfBa089cA3618A1E76c4a2908BF1cD2751
0xd43936F2B187c9C66000a3e8a2E195464603027E
0x2Fc9E83c5175E997A4e9Aaf7E7C3F13742d3B214
0xdaf4eE10Bfe98d6337A1483465FE5849dDa74E3F
Reasoning
0x66deb0894b0949eb3d34176201ab0219161a3b07 created 23 Safes which are eligible for the airdrop.
And while Safes with the same creator aren't necessarily airdrop farming, let's look into what makes this an obvious case:
All of the 23 Safes currently don't store any value, neither ETH nor any other ERC-20 tokens.
The Safes were created in batches, sometimes 3 or 4 only minutes apart of each other:
Feb-24-2021 10:47:28 PM - 10:52:31 PM +UTC: 4 Safes created 1 2 3 4
Mar-04-2021 09:20:49 PM - 09:27:22 PM +UTC: 3 Safes created 5 6 7
Mar-11-2021 06:47:52 PM - 06:50:59 PM +UTC: 3 Safes created 8 9 10
Mar-16-2021 04:53:26 PM +UTC: 1 Safe created 11
Mar-18-2021 09:40:26 PM - 09:51:42 PM +UTC: 4 Safes created 12 13 14 15
Mar-25-2021 06:21:16 PM - 06:27:56 PM +UTC: 4 Safes created 16 17 18 19
Apr-05-2021 05:26:51 PM - 05:35:27 PM +UTC: 4 Safes created 20 21 22 23
Methodology
The reported addresses were discovered by checking for irregularities in the Safe creations through the Safe: Proxy Factory 1.1.1 contract.
I checked for the addresses that had the biggest # of Safe creations each month. In March, 2021 part of this group was the address in question (0x66d) and it is not a relayer.
I then manually looked at the value stored in the safes (which is always the same = 0), the # of Safe Transactions and the date on which the safes were created:

Safe Address
Safe Date Created
Value Stored in Safe
# of Safe Transactions
Tokens Allocated

0x4a61E2d459f24378fAc26721705BB4a974DF4e93
Feb-24-2021 10:47:28 PM +UTC
0
6
1740.292256517797

0x3Bef9cFfEEe4da69fBA9c9b47CC9A75c50e62e8b
Feb-24-2021 10:49:41 PM +UTC
0
1
1157.1912257310473

0x9937c074e94eCd7D7C23B3b963106eb47fA9235B
Feb-24-2021 10:51:25 PM +UTC
0
1
1157.1912257310473

0xacc819aBcfaaD2b8E0fFbd701a86d72cfd10407e
Feb-24-2021 10:52:31 PM +UTC
0
1
1094.8005854228127

0xea166EFbCc7Ca9bD86d665512C488CdDcA5Bdae3
Mar-04-2021 09:20:49 PM +UTC
0
6
1992.0934870522992

0x7d027B807444dA6252a187B044DE3a196120E897
Mar-04-2021 09:25:54 PM +UTC
0
1
1543.2062545890276

0x01069fF7f59514Ade22E8F682B7334676C00c56c
Mar-04-2021 09:27:22 PM +UTC
0
3
1543.2062545890276

0x68A54972e4faeb548C23214FC949197b85058d0E
Mar-11-2021 06:47:52 PM +UTC
0
5
2052.4987624688993

0xc802b29dE31cdb42B5406AADa2f8DAe24121D248
Mar-11-2021 06:48:51 PM +UTC
0
1
1545.4994600959192

0xc6101C2595db8bb9179De79ADB44F0ED6683D419
Mar-11-2021 06:50:59 PM +UTC
0
8
1705.3028295758409

0x3086AB78F3Ec3298545346bD9Dc167ebE1539Ae1
Mar-16-2021 04:53:26 PM +UTC
0
1
1828.2827059211722

0x673F7Db60E99EAf80e26a2F0e3c716B0e8d15dfD
Mar-18-2021 09:40:26 PM +UTC
0
3
1669.0246953503643

0x0982eb56Be2F16DE53D76f75FBAC4dF2D69b9CA7
Mar-18-2021 09:43:39 PM +UTC
0
2
1604.0916393510074

0x85C1217921751f146Df95dcb6a8E8e9f85554D54
Mar-18-2021 09:50:36 PM +UTC
0
2
1532.2166170885466

0x426c90fe3Dc43AC291Ef849CD3c74dEbB5CCD23a
Mar-18-2021 09:51:42 PM +UTC
0
1
1318.26942057808

0x08B8D1D7C92357EEBE892A51e3185B770A58f485
Mar-25-2021 06:21:16 PM +UTC
0
5
1965.2966422781794

0x174C99D6fFE6e4b7f39d2f33fA73ab3D23E168d5
Mar-25-2021 06:22:48 PM +UTC
0
1
1709.2186411870518

0x83C6Efecd8084722475f23708E7a260EE8Db52c4
Mar-25-2021 06:26:30 PM +UTC
0
5
1799.5960828049303

0x744f1Cb0e1baDaDAA951C7e9Db8218F04B70716B
Mar-25-2021 06:27:56 PM +UTC
0
1
1504.5297477943118

0x7deDc6BfBa089cA3618A1E76c4a2908BF1cD2751
Apr-05-2021 05:26:51 PM +UTC
0
13
2423.3307684057577

0xd43936F2B187c9C66000a3e8a2E195464603027E
Apr-05-2021 05:27:52 PM +UTC
0
2
1784.6282125229552

0x2Fc9E83c5175E997A4e9Aaf7E7C3F13742d3B214
Apr-05-2021 05:34:47 PM +UTC
0
3
1863.6461724109904

0xdaf4eE10Bfe98d6337A1483465FE5849dDa74E3F
Apr-05-2021 05:35:27 PM +UTC
0
1
1680.9759087839225

Safe Address
0x81058ff64a2D765E73fC04c6a19E051701D101C8

I checked just the first 3

https://gnosis-safe.io/app/eth:0x4a61E2d459f24378fAc26721705BB4a974DF4e93/transactions/history
https://gnosis-safe.io/app/eth:0x3Bef9cFfEEe4da69fBA9c9b47CC9A75c50e62e8b/transactions/history
https://gnosis-safe.io/app/eth:0x9937c074e94eCd7D7C23B3b963106eb47fA9235B/transactions/history

All Safes have been storing quote sizable amounts of USDT. Store of value is a legitimate use of the Safe.


cli-go: rewrite some tests.bats with go tests (#108)

#108
Please review and merge if you like. Moar tests coming soon.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #456   +/-   ##
=======================================
  Coverage   86.39%   86.39%
=======================================
  Files          29       29
  Lines         272      272
  Branches       26       26
=======================================
  Hits          235      235
  Misses         32       32
  Partials        5        5

Flag
Coverage Δ

#webui
86.39% <ø> (ø)
: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 6fffb33...989f2c5. Read the comment docs.

@innokenty that sounds really reasonable, however I have a couple of proposals.
Can we move integration tests to a separate go project?
We can automate starting up zally service, downloading binary builds of zally and testing them etc.
(e.g. https://github.com/zalando-incubator/zally/blob/master/cli/integration_tests/tests.bats#L65)
What do you think?

Why would we write them in Go then? Could be bash. Or tests.bats 😉
These test now invoke the app as a go function, but from the very outside entrypoint so to say. SO the benefit is you can debug it, or tweak some options. Makes sense? Otherwise let's just stay with .bats file because that's exactly what it does: invokes the compiled zally file with params and checks the console output, doesn't it?

@innokenty good point and I don't have an immediate answer. Let me talk to @maxim-tschumak first :-)

@innokenty @netme At some point, we will have integration tests for the front end as well. Does it make sense to have the same tech stack for them? I believe not, and that's why I prefer to have CLI integration tests in Golang as a part of CLI component.

@maxim-tschumak Do I understand you correct, that you agree with the approach proposed in this PR and you would like me to rewrite the rest of the tests.bats this way?

@innokenty Yes, I do :) and thanks!

Let's start with this approach then. +1

Then I'll continue, must be not a lot of work now 👍
You can either merge, or wait for a couple more commits today.

Let's wait for couple of more commits, will be easier to see the bigger picture.

Squashed 👆
Ready for merge ✅

He-he, GitHub has some troubles. Will check the PR once GH will be back online.

👍

:+1:


Update year

Thank you, would you also be so kind and update the year in the README and LICENSE file?


post-forecaster refactor: check all the tags, especially univariate

Post-refactor #955, we need to go through all the tags again.
@mloning has raised the issue that the "univariate" flag may not be correct - I thought it means whether y is univariate, but it actually means whether the estimator can deal with X and/or uses it.

added this as a sub-task in #955


[BUG] Showing the status bar error when using 4-spaces tabs off

This "bug" seems to be created from a change that was made in response to #143
With the new update for Outliner plugin v1.2.5 I am getting this error saying Outliner only works with four-space tabs. I do have it set to 4 spaces, and I have tabs turned off (uses 4 spaces).

Here are my settings:

It seems @vslinko in that other issue you are maybe intending to show this error if you have tabs turned off (using spaces instead) even if you are using 4 spaces? This would be very unfortunate because then are you saying Outliner plugin only supports using Tabs? That is a deal breaker for a lot of people like me who choose to use spaces instead of Tabs in all their documents/notes.
Did I read it correctly that the only reason you warn about tabs turned off and using 4 spaces is because the indent level lines don't work? That is an obsidian core thing... the way the HTML is styled they do not have a separate HTML element for each indent (tab) so when using spaces you cannot style the indent level lines (which is fine). At minimum I'd suggest being able to turn this warning off otherwise users will likely no use the plugin if they have to see the red error all the time in their status bar.
Thanks for all your hard work and I look forward to hearing your response!

@GitMurf Hi!
This warning only applies to Outliner styles, not to manipulation of lists. You can hide this warning by disabling "Improve the style of your lists".
Alternatively, I could add a setting like "Hide Obsidian styles warning". What do you think about this?

Yes adding the setting to "hide obsidian styles warning" would be good because it actually works great with 4 spaces and tabs turned off. Everything in the outliner works as expected including the "Improve the style of your lists". I want to use that feature but not if I have to have the red warning in the status bar all the time ;) Thanks!

Personally I think the warning is unnecessary when using 4 spaces and tabs turned off. For me everything works/looks fine with the "improve the styles of your lists".

@GitMurf Thank you for your contribution! The setting should work in https://github.com/vslinko/obsidian-outliner/releases/tag/1.2.7


Link to source repository of plugin

It would be great to add a field where we can link to the source code repository of a certain plugin (in case it is open source).

Added ability to add source, blog, article, and homepage url's to each plugin. Will be pushed with 0.4.2.


Correct types for event functions

Is your feature request related to a problem?
It's really annoying to have to cast HTMLInputElement or something like that everytime that i have to use a value of the input, for example.
<input
      value={store.value}
      onInput$={(event) => (store.value = (event.target as HTMLInputElement).value)}
/>

Describe the solution you'd like
The solution would be types being inferred automatically :
<input
      value={store.value}
      onInput$={(event) => (store.value = event.target.value)} // (e) type as HTMLInputElement
/>

Describe alternatives you've considered
Didn't think any
Additional context
I would like to solve this issue, but i am having trouble on finding where those types are

I think the file you're looking for is here.
https://github.com/BuilderIO/qwik/blob/main/packages/qwik/src/core/render/jsx/types/jsx-qwik-attributes.ts

@nnelgxorz haven't found anything about those events such as onInput$, am i missing something?

Yeah, it's a little confusing in there.
They are generated based here:
https://github.com/BuilderIO/qwik/blob/main/packages/qwik/src/core/render/jsx/types/jsx-qwik-attributes.ts#L168
based on this type:
https://github.com/BuilderIO/qwik/blob/bd1b72c9397bdb7bc6f2d9e3028f4388d00560e1/packages/qwik/src/core/render/jsx/types/jsx-qwik-attributes.ts#L8

Thanks @nnelgxorz , this weekend i'll be looking for this

I think this is fixed by https://github.com/BuilderIO/qwik/pull/1827

yes! fixed!


ACLs not getting created properly when having Julie manage principles with Confluent Cloud

When having Julie manage principals with Confluent Cloud, the ACLs are not created properly. Also, the service account is created with format User:SA_NAME instead of just SA_NAME.
To Reproduce

Create a topology file with the following contents:

---
context: com.abc
lob: tech
appgroup: payment
projects:
  - name: julie
    topics:
      - name: test-payment
        schemas:
          value.schema.file: "./schemas/example.avsc"
          value.record.type: "Payment"
        config:
          replication.factor: 3
          num.partitions: 1
        consumers:
          - principal: "User:julie-test"
        producers:
          - principal: "User:julie-test"

Include the following ccloud config in the connection properties file:

julie.enable.principal.management=true
ccloud.environment=env-xxxxx
ccloud.cluster.api.key=xxxxx
ccloud.cluster.api.secret=xxxxx
ccloud.cloud.api.key=xxxxx
ccloud.cloud.api.secret=xxxxx
topology.builder.ccloud.kafka.cluster.id=lkc-xxxxx
ccloud.cluster.url=https://xxxxx:443
topology.builder.access.control.class=com.purbon.kafka.topology.roles.CCloudAclsProvider

Run Julie with the above topology file.

Expected behavior
The expected behavior is for the ACL to be created for the resource id associated with the test-julie account and for the service account name to be test-julie.
Screenshots
Service Account:

ACLs:

Runtime (please complete the following information):

OS: MacOS 10.15.7
JVM version: AdoptOpenJDK 11
Version: 4.1.0

Additional context
If you try the ACL create operation via the Confluent Cloud REST API, the API does not create the ACL unless the numeric service account ID is specified.
The Confluent Cloud ACL API expects a principal with format principalType:principalName. If you send a request in this format, the API will return a 201 but doesn't actually create the ACL. If the numeric service account ID is specified in place of "principalName", it works.
https://docs.confluent.io/cloud/current/api.html#operation/createKafkaV3Acls
Confluent support was able to replicate the issue with the API on their end, and have escalated it internally. They also requested for us to create an issue here.

Any timeline on when this bug will be fixed?

@MichaelBeausoleil we learned from a support case with Confluent Cloud that the documentation for the create ACL API is incorrect. The API only accepts the integer account ID and Confluent is working on updating the documentation https://docs.confluent.io/cloud/current/api.html#operation/createKafkaV3Acls.
@purbon  ☝

Does that mean that I just need ro change the references in the descriptor file or that there still needs to be a code change in JulieOps?
If it is the former can you please provide and example of what needs to change in the descriptor files?
If it is the later is there a timeliness for the code correction?
Thanks for the quick response.

@shan916 can you provide a reference to the Confluent support case so I can ask Comfluent about its status?

Hi folks, this is currently a work in progress to improve on the Confluent Cloud API. In the meantime, I will try to figure out if possible we can do something here to solve this problem in this layer. Sorry for the inconviniences, as you know this is not an official Confluent Product, and I do my best to keep it up for the latest changes and improvements.

Hi,
I wanted to note in the thread that I found a suitable workaround for this. I am working on a solution and will be doing a release as soon as possible, ETA end of this week beginning of next.

Thank you for the update.

This should be fixed with #444

Translation is integrated now in the API usage
It is configurable, so whenever this bug is fixed, a user can disable it when not required anymore.
If translation failed, JulieOps will raise an Exception.

Hope that helps!

Hi, quick update on this issue, you will find a release with a workaround for this issue at https://github.com/kafka-ops/julie/releases/tag/v4.1.1
hope it helps you, let me know how it goes.

@purbon thank you very much for working on this fix.  I will be trying this out today.

Thanks, please let me know how it goes. Remember that in the new version
4.1.1 you need to disable principal management if using the cloud,
otherwise JulieOps is going to try to create your ServiceAccount as defined
in the topology.  You can do this by putting
https://github.com/kafka-ops/julie/blob/master/example/julieops-confluent-cloud.properties#L18
as false.
-- Pere
On Tue, Feb 8, 2022 at 3:58 PM Michael Beausoleil @.***>
wrote:

@purbon https://github.com/purbon thank you very much for working on
this fix. I will be trying this out today.
—
Reply to this email directly, view it on GitHub
https://github.com/kafka-ops/julie/issues/438#issuecomment-1032701001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQXPBGIERBZVRJTXJTQNLU2EVQVANCNFSM5LNWN6UQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Pere Urbon-Bayes
Software Architect
https://twitter.com/purbon
https://www.linkedin.com/in/purbon/

I apologize for the delay on my response.  The new version is working well.  Thank you again for your efforts.

Thanks Michael, happy it is working right now for you.


Hypridle ignores dbus_inhibit on Hyprland version 0.40.0

I updated Hyprland to version 0.40.0 and ignore_dbus_inhibit stopped working. On 0.39.0 it works fine. The logs on both versions are the same:

[LOG] Registered timeout rule for 10s:
on-timeout: loginctl lock-session
on-resume:
[LOG] Registered timeout rule for 1800s:
on-timeout: loginctl suspend
on-resume:
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]    > Bound to ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v4
[LOG]   | got iface: wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: zwlr_export_dmabuf_manager_v1 v1
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: wp_viewporter v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: zwlr_layer_shell_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zxdg_exporter_v1 v1
[LOG]   | got iface: zxdg_importer_v1 v1
[LOG]   | got iface: zxdg_exporter_v2 v1
[LOG]   | got iface: zxdg_importer_v2 v1
[LOG]   | got iface: wp_single_pixel_buffer_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: wl_output v4
[LOG] found 2 rules
[LOG] wayland done, registering dbus
[LOG] Using dbus path /org/freedesktop/login1/session/_32
[LOG] Idled: rule 5620f6777710

Hypridle version is 0.1.1
Distribution is gentoo
Config:

general {
lock_cmd = ~/.config/bemenu/lockscreen.fish
before_sleep_cmd = loginctl lock-session
}
listener {
timeout = 300
on-timeout = loginctl lock-session
}
listener {
timeout = 1800
on-timeout = loginctl suspend
}

And waybar idle inhibit module does not work too :((

fixed in git, this is a dupe


feat: add support for gfx1100 devices

Thread ref: https://github.com/RadeonOpenCompute/ROCm/issues/1880

Hi, Thanks for the PR!
I'll review. There are likely a few more places we'll need to specify the new arch.
I also may be a bit before we can merge as there are some needed miopen updates.

@jayfurmanek
Greetings. Sorry to bother you, but the wheels from CI doesn't work on as indicated here:

https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/issues/2142

And I would like to know when tensorflow-rocm will ship with Navi 31 support?


Add suggestions for how to fix when encountering phone number format errors

When you try to send to an international phone but don't use the + or appropriate country code, Twilio will reject the message send. We should give some hints to the user when we relay the error message back to them.

This should be fixed by the above commit.


Investigate Consequences_Assist.DataGridRowItem

I saw a few instances of classes that were implementing this data grid row item class. I got rid of the one that I was working with directly. I am not sure what its purpose is, but I am guessing we don't need to be implementing this class anywhere in FDA. If we have time we should search these out and investigate and get rid of them.

@Brennan1994 @codymccoy you guys mentioned this issue during our meeting yesterday. Do we plan to address this?

I was able to delete 5 classes that were not being used including the data grid row item mentioned in the title of this issue.
ConsequencesAssist will still be used in FDA for the file and folder browser.


Precision@K

I am currently working with an implicit feedback dataset and trying to evaluate a model based on the precision@K measure. I am having difficulties understanding the implemented precision calculation in a case when the list of recommendations consists only of top K recommendations.
@Evfro Can you please clarify if the calculated precision is a precision@K or a regular precision measure?

Hi.
Yes, currently, all metrics are evaluated @ K, which is by default set to 10. There are several ways to control the value of K in Polara.

You can specify K individually for each recommender model by setting the topk attribute, e.g.

model.topk = <your value>

Alternatively, you can globally redefine the default value of K for current session via the defaults module (make sure to do that before creating any recommender model!):

from polara.recommender import defaults
defaults.topk = <your value>

Finally, you can also provide the value of K at the evaluation time:

model.evaluate(topk=<your value>)

Note that if you have already generated recommendations with some old value of K (e.g., by accessing the .recommendations attribute of the model or by calling model.evaluate()) and decided to change it to a new value K' > K, then recommendations will be automatically recomputed. Conversely, for K' < K the recommendations list will be simply truncated without any recomputations.
Also note that you can automate running experiments for different values of K via Polara's pipelines, especially using the topk_test method, as shown in this tutorial.


Doc: Update PBFT glossary

This PR contains separate commits for the following changes:

Remove math terms, because f, n, p, and v are adequately defined in the "Architecture" section.

Remove entry for Block duration.

Add three new terms: Consensus API, Consensus engine, Transaction family.

Remove entries for Replica and Server; replace with a sentence under Node.

Update View entry with info about regular view changes.

Edit definitions to clarify and fix grammar. This commit also changes "Primary" to "Primary node"
(ditto "Secondary node") and mentions the original PBFT paper's terms for primary and secondary.

Alphabetize entries (in a separate commit with no technical or wording changes).

@jsmitchell : I added a commit that corrects the definition for "Message".


New Service: wdio-applitools-service

Proposed changes
Add a new service for better integration with Applitools.
Types of changes

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

Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
This is work in progress.
Reviewers: @webdriverio/technical-committee

Codecov Report

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

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   91.83%   91.83%
=======================================
  Files          79       79
  Lines        1628     1628
  Branches      352      352
=======================================
  Hits         1495     1495
  Misses        109      109
  Partials       24       24

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 b28a64c...7fbd10b. Read the comment docs.

Thanks for the feedback. Applied suggested changes.


[Feature] Support .net core 2.0

add new target

related to https://github.com/GHPReporter/Ghpr.Core/issues/54

Hi @elv1s42, when are you planning to roll this out please? We would like to use this tool to create Nunit reports but out test project is build on .netCore 2.2 and we are using https://github.com/nunit/docs/wiki/NUnitLite-Runner to execute our tests (hence there is no .addins file etc.).

Hi @mcopjan !
Yes, this one is in progress.
At the moment I have .Core version of the Ghpr NuGet package (not published yet), but there may be some issues with the way Ghpr package will be discovered by NUnit Engine. You check another thread for more details here:
https://github.com/nunit/nunit-console/issues/487#issuecomment-511121027
For now I don't have much time to investigate this deeper, so most likely I'll publish the existing Ghpr code with .Core target framework and try to fix possible issues in the next release.
Thank you

Done


Mouse controls not working

The mouse controls (scroll, select, etc) are simply not working. They work fine if I checkout d2277a376a616b98f30c46f4cc641eaebd81b53b, but not in the next one (8be0f9c52981bd6b1e845198ee89d7347f4c635b). All other controls work just fine.
Linux Mint 18.0, Terminator 0.97 (GTK3 version), libvte 0.42.5
Also tested in gnome-terminal 3.18.3

Sorry about this, it seems I was careless in making commit 8be0f9c. Should be fixed now.


Note that the tests won't pass on master until we merge this travis.yml file, but they do pass on the branch so I think it's ready to go.


Fix macOS plugin naming

Fixes #486.
The name of the Swift class and platforms.macos.pluginClass in pubspec.yaml must match.

UP!

@juliansteenbakker Please can you update pub.dev?

@mogol, Do we have any adjustment expectations?


Kick 9bab0f03-a725-4a13-9ab1-196e46cd80ed to completed

This submission has been exported.

Thanks @aaclan-ebi ! I was gonna update the ticket information but since it's finished, I'll just move it to done


Bring this module into the new world

what

This builds off of the work from @chess-seventh and @ivanmartos to bring this module up-to-date with the latest and greatest Cloud Posse practices:
Adds 0.12+ support
Updates tests to use new Make files + go modules
Adds github workflows / templates
Adds context.tf
Updates example to follow the more consistent Cloud Posse example/complete pattern.
Removes old cruft: codefresh tests and .travis.yml

why

This hasn't been updated since 0.11, but folks are still looking to use it (myself included)

references

Closes #3
Closes #5
Closes #6

Also, this is passing testing: https://cln.sh/QdiFp7
@osterman I'm digging that new docker/test make target! This was first chance to get to use it and it made that whole process a lot easier than my first foray into ya'lls tf test-harness work. I vote we add instructions for that to the README template!


Maintain selected row’s background color in row selection

Description
As we highlight the selected row in grey background color, it looses its original color during selection. This PR aims to change the selection color for each row.
Relevant Technical Choices

Update selection classes

Testing Instructions

Additional Information:

Screenshot/Screencast
https://github.com/GoogleChromeLabs/ps-analysis-tool/assets/6297436/126d74da-5f06-4064-9cc4-16771e2c4a66

Checklist

[ ] I have thoroughly tested this code to the best of my abilities.
[ ] I have reviewed the code myself before requesting a review.
[ ] This code is covered by unit tests to verify that it works as intended.
[ ] The QA of this PR is done by a member of the QA team (to be checked by QA).

Fixes #

Closing in favour of https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/556


define stream throws error if it is declared inside a class

For example:
class FunkyLogger:
    define stream myLittleLogger.
end class.

gets parsed as this:

Registered: https://github.com/usagi-coffee/tree-sitter-abl/issues/94

Fixed with version 0.0.44 of tree-sitter-abl


more than one object named xxxxx

on click the one of the btns:
# Traceback (most recent call last):
#   File "C:/Users/Administrator/Documents/maya/scripts\maya_sniffer.py", line 424, in on_clicked
#     cmds.select(path)
# ValueError: More than one object matches name: P_Ship_Alfida_Tex_Box2450ShapeOrig
Copied 'P_Ship_Alfida_Tex_Box2450ShapeOrig' to clipboard

Thanks, pull-request is welcome. Probably just a matter of returning the full path from cmds.ls rather than the default shortest path.


Fix #3 by using getLibraryFromClassname.

Signed-off-by: Rene Jochum<EMAIL_ADDRESS>

I dont think that this right - the namespace is not the classname, or even the class package.  EG you could have a namespace of "abc" and an application class of "abc.def.ghi.Application".  The namespace is defined in the Manifest.json, so we have to search for matches until we narrow down the library.
(PS Ive got back late after a couple of days of near-zero signal so just catching up but spending time on qx this holiday weekend)

ok


added jwt and enhance home page design

@CodiumAI-Agent /review

PR Reviewer Guide 🔍

 ⏱ Estimated effort to review [1-5]
3

 🧪 Relevant tests
No

 🔒 Security concerns
No

 ⚡ Key issues to review

Password Hashing: The password hashing logic is now performed in the pre('save') middleware for the user schema. Ensure that this change does not introduce any issues with password management, such as hashing the password multiple times inadvertently.

Validation Logic: The addition of ValidateLogin in auth.js and changes to error handling in users.js should be thoroughly tested to ensure that the validation works as expected and that the error messages are clear and informative.

Route Changes: The route for logging in has been changed in Login.jsx to include /add at the end. Confirm that this new route is correctly configured on the server side and that it doesn't conflict with other routes.


Prototype integration of code quality tools.

Take the spot bug, and check style rules from java, integrate that using Gradle plugin, run it for API 29+. (In the upcoming semester, we should get it down to API 19)

Gradle based sports bug + check style is here: https://github.com/anuchandy/az-android-vnext/tree/master/config

Gradle based sports bug + check style is here: https://github.com/anuchandy/az-android-vnext/tree/master/config


populate feedback with submitted primary appraisal

Only populate when the appraisals are confirmed/submitted by lead assessor
If they get unsubmitted, edited, resubmitted - then update the feedback accordignly
If RAG is G, populate Key Strength, else populate Information to strengthen the application
Overall verdict goes to Overall Summary
Add extra help copy to the section header

Looks good :+1:


Detect that we are running in docker-on-mac and automatically reduce …

…timer frequency due to expensive
calls to sample time. This might reduce timeout accuracy from 1ms to 10ms, but that is fine for development.
docker-on-mac will never be supported as production environment.
@arnej27959 and @hmusum PR
@jobergum FYI

@arnej27959 echo removed. PTAL

💯 🥇


fix small buffer for 16k wpp stream;  2)  clear vector when re-allocate buffer

There two changes are all cherry-picked from internal branch.

@weizhu-intel , please give a better name to this PR

Currently I don't see any issues in CI tests after last force-push.


foreign-toplevel-management: only report parent toplevel to clients that requested version 3

In #2419, no version check was added before calling zwlr_foreign_toplevel_handle_v1_send_parent, so clients unaware of version 3 blow up when they receive an event they don't know anything about:
interface 'zwlr_foreign_toplevel_handle_v1' has no event 7

cc @dkondor

Not something I thought of, thanks for spotting this!
I made a quick fix here: https://github.com/dkondor/wlroots/commit/d2722d2207194dfcca8e2fad60cd6cfb6829bfed -- should I open a separate PR? (I've tested this with the code in examples)

Yes, that sounds good, thanks. (Preemptive nit, spacing after the if and before the (. :)


Footer issues megathread

[ ] Don't forget to link to the repo in the footer. I think we somehow missed this on the Hurricane viz - we should link to GH repo by default I would think. w/ a GH logo

[ ] Footer aesthetic

IE is having issues with the footer images
Not sure why, but writing this to remind myself later

Lower CO should be good now, not sure about IE.

@ldecicco-USGS have you looked at the footer on IE lately? is this still an issue?

I think this is the same as in matthew, it's stretching the pictures:

@mwernimont see #181

One more thing remaining is to left-justify the two svgs in here. I think @mwernimont is working on that.


Trying to train transformer like the NMT Keras Transformer, for passage summarization task. The model's masked accuracy goes upto ~30% but the transformer's output is very poor. Could you help me understand what I might be doing wrong?

Click to expand!
Issue Type
Performance
Source
source
Tensorflow Version
tf 2.8
Custom Code
Yes
OS Platform and Distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
NMT task on the keras transformer as showed in the tutorials (https://www.tensorflow.org/text/tutorials/transformer) performs well. But I'm unable to get decent performance on a passage summarization task on the `cnn_dailymail` dataset. I can't understand what I might be doing wrong.

Standalone code to reproduce the issue
https://colab.research.google.com/drive/12sbAGKESj8qynO-pb1K6gh-CZIvCxzfo#scrollTo=TlOaOjwwWN4K

Relevant log output
No response

Hi @thewinterknight14,
Iam unable to access the provided colab file. I request you to please convert it into Github gist and share the same with us.
Thankyou!

https://github.com/thewinterknight14/frost_queen/blob/master/transformer_passage_summarization.ipynb

Hi @thewinterknight14 ,
As per the gist you attached i understand that you need to improve the performance of your model right ?. For such issues we have dedicated TF Forum where you can post the issues and ask for suggestions from larger community.Iam confident you may get some help there.
Thankyou!


Fix #212 #214

This should fix a bug with TS3.6 found in #212 and the bug reported in #214.
This should be working fine, but I'm currently preparing a conference talk, so I don't have the time to test this super-excessively and the bug fix for #214 is something I'm not feeling 100% sure of if I did everything right.
@dragomirtitian, could I ask you to take a look that I didn't do anything incredibly stupid there?

Great, thanks!
Out of curiosity for the still mostly-TS-clueless: any chance you could walk through how you went about debugging this problem?

Hm, that failing build was to be expected. @markerikson, would you be okay if I added an any-cast in that failing line?

Great, thanks!
Out of curiosity for the still mostly-TS-clueless: any chance you could walk through how you went about debugging this problem?

I went to the PayloadActionCreator type and replaced each possible result with a string, to see which path it goes. It went to ActionCreatorWithPayload, so I did the same thing there, only to see that it took a wrong branch in IsUnknownOrNonInferrable - and so on.

Ah, ye olde "brute force logging" approach :)
Yeah, I'm okay with whatever works to actually resolve the problem.

Argh, failing due to prettier issues. I really have to stop procastinating, I'll fix that up tomorrow. Alternatively, @markerikson could you please just push a prettier fix?

At work atm, but yeah, will do so later.

Not my work laptop, and vscode defaults to some other autoformat behaviour than prettier. But it does something. So I always forget to install the plugin, because it looks right. Fixed that on two machines and today, wrote the fix on my third machine :D

Thanks for the fast solution!


traceback with activerecord 5.2.0

Seeing this with current latest activerecord
$ bundle exec rest-assured :memory:
Traceback (most recent call last):
	12: from /var/tmp/bundle/bin/rest-assured:23:in `<main>'
	11: from /var/tmp/bundle/bin/rest-assured:23:in `load'
	10: from /var/tmp/bundle/gems/rest-assured-3.0.1/bin/rest-assured:73:in `<top (required)>'
	 9: from /var/tmp/bundle/gems/rest-assured-3.0.1/bin/rest-assured:73:in `require'
	 8: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/application.rb:13:in `<top (required)>'
	 7: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/application.rb:14:in `<module:RestAssured>'
	 6: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/application.rb:16:in `<class:Application>'
	 5: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/application.rb:16:in `include'
	 4: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/config.rb:44:in `included'
	 3: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/config.rb:71:in `setup_db'
	 2: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/config.rb:113:in `migrate_db'
	 1: from /var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/config.rb:108:in `block in migrate_db'
/var/tmp/bundle/gems/rest-assured-3.0.1/lib/rest-assured/config.rb:104:in `block in migrate_db': undefined method `migrate' for ActiveRecord::Migrator:Class (NoMethodError)

AR5: works, AR5.2: broken? Not much of a semantic versioning, is it?
Anyway, will fix at some point. Feel free to fix & pr to have it sooner :)


Emphasis container inside emphasis container provides inconsistent results

UWP
The text is butted up to the edge without any padding, that's definitely wrong

JS
JS correctly has padding, should this be the behavior? Or should emphasis be ignored since it's already inside an emphasis container? Note that you have to change the host config so that emphasis is semi-transparent to see any difference

WPF
Text is also butted up to the edge just like UWP

{
	"type": "AdaptiveCard",
	"body": [
		{
			"type": "Container",
			"items": [
				{
					"type": "Container",
					"style": "emphasis",
					"items": [
						{
							"type": "ColumnSet",
							"columns": [
								{
									"type": "Column",
									"style": "emphasis",
									"items": [
										{
											"type": "TextBlock",
											"horizontalAlignment": "Left",
											"size": "Small",
											"text": "Confirmation Number"
										},
										{
											"type": "TextBlock",
											"id": "confirmationNumber",
											"horizontalAlignment": "Left",
											"spacing": "Small",
											"size": "Small",
											"text": "0987654321"
										}
									],
									"width": "stretch"
								},
								{
									"type": "Column",
									"style": "emphasis",
									"items": [
										{
											"type": "TextBlock",
											"horizontalAlignment": "Right",
											"size": "Small",
											"text": "PIN"
										},
										{
											"type": "TextBlock",
											"id": "pinNumber",
											"horizontalAlignment": "Right",
											"spacing": "Small",
											"size": "Small",
											"text": "1234"
										}
									],
									"width": "stretch"
								}
							]
						}
					]
				}
			]
		}
	],
	"$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
	"version": "1.0"
}

I do like the padding approach visually. @matthidinger / @RebeccaAnne / @almedina-ms / @nesalang / @paulcam206 thoughts? Is taking the JS approach forward do-able on the other platforms?

@andrewleader assigning to you until we can address the spec ambiguity if that's alright.


Freeze in Smart TVs browsers

Hello,
Using p2pml.hlsjs.Engine.isSupported() does not work in some Smart TV browsers I have tested (Samsung Tizen, LG, AOC, Philips, etc). The browser freezes when this line is reached. I had to create a separate page specifically for TVs, with regular player only (no-P2P).
Perhaps isSupported() method could be improved in this aspect, determining with more simple (non WebRTC related) commands if a browser is not capable of P2P due to the use of an old JS engine - before doing the WebRTC check itself?

Can you check what exactly makes the browsers freeze? It doesn't call any browser functions but just checks if API actually exists:
public static isSupported(): boolean {
    const browserRtc = (getBrowserRTC as Function)();
    return (browserRtc && (browserRtc.RTCPeerConnection.prototype.createDataChannel !== undefined));
}

module.exports = function getBrowserRTC () {
  if (typeof window === 'undefined') return null
  var wrtc = {
    RTCPeerConnection: window.RTCPeerConnection || window.mozRTCPeerConnection ||
      window.webkitRTCPeerConnection,
    RTCSessionDescription: window.RTCSessionDescription ||
      window.mozRTCSessionDescription || window.webkitRTCSessionDescription,
    RTCIceCandidate: window.RTCIceCandidate || window.mozRTCIceCandidate ||
      window.webkitRTCIceCandidate
  }
  if (!wrtc.RTCPeerConnection) return null
  return wrtc
}

A Panasonic TV's browser can't play live stream with p2p-media-loader 0.6.2(hls) but can play with 0.6.1(hls), Maybe ES2017 standard is not support with this tv browser!
Could you release p2p-media-loader with ES6(ES2015)? Thanks!

Can you get the error message from the TV's browser console?

Me too. Only buffering on smart Tv.

@renatofrota Would you be so kind as to provide example on how you solved HLS streaming on Tizen, webOS... without p2p?