[Firebase_core]  crash when use physical ios device with Dart-only initialization

Bug report
Describe the bug
I have iphone ios 12.5.5, my app is crashing when use dart-only initialization with real device.
In the simulator my app works fine with dart-only initialization.
The app works fine with manual Installation 'GoogleService-Info.plist' in real device and simulator.
I have this in main.dart:
void main() async {   WidgetsFlutterBinding.ensureInitialized();   await Firebase.initializeApp(     options: DefaultFirebaseOptions.currentPlatform,   ); }
When use Xcode with this problem I get this message :
2022-01-27 10:54:11.419241+0300 Runner[315:10609] Warning: Unable to create restoration in progress marker file *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'The default FirebaseApp instance must be configured before the default Authinstance can be initialized. One way to ensure this is to call FirebaseApp.configure()in the App Delegate'sapplication(:didFinishLaunchingWithOptions:)(or the@mainstruct's initializer in SwiftUI).' terminating with uncaught exception of type NSException 2022-01-27 10:54:11.586464+0300 Runner[315:10609] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'The default FirebaseApp instance must be configured before the default Authinstance can be initialized. One way to ensure this is to callFirebaseApp.configure()in the App Delegate'sapplication(:didFinishLaunchingWithOptions:)(or the@main struct's initializer in SwiftUI).' *** First throw call stack: (0x18bb23180 0x18acfb9f8 0x18ba2d3a0 0x10259d8a0 0x101054298 0x102aaa9a8 0x104d5b6f4 0x104d5cc78 0x104d6a6fc 0x18bab4b20 0x18baafa58 0x18baaefb4 0x18dcb179c 0x1b8341c38 0x10080f4dc 0x18b5728e0) libc++abi.dylib: terminating with uncaught exception of type NSException
Steps to reproduce
Use dart-only initialization with firebase plugins with real iphone ios 12.
Expected behavior
The app is supposed to work fine and without crash in the real device because it works in the simulation as it should.
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.8.1, on macOS 11.6 20G165 darwin-x64, locale ar-SA)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.63.2)
[✓] Connected device (3 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.15.1
Flutter SDK 2.8.1
qatrah_app 1.0.0+1

dependencies:
- cached_network_image 3.2.0 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- cloud_firestore 3.1.6 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cupertino_icons 1.0.4
- dio 4.0.4 [http_parser path]
- dio_http_cache 0.3.0 [crypto dio flutter json_serializable json_annotation path quiver sqflite]
- enum_to_string 2.0.1
- firebase_auth 3.3.5 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_messaging 11.2.5 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_storage 10.2.5 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_launcher_icons 0.9.2 [args image path yaml]
- flutter_rating_stars 1.0.3+4 [flutter]
- google_maps_flutter 2.1.1 [flutter flutter_plugin_android_lifecycle google_maps_flutter_platform_interface]
- image_picker 0.8.4+4 [flutter flutter_plugin_android_lifecycle image_picker_for_web image_picker_platform_interface]
- location 4.3.0 [flutter location_platform_interface location_web]
- moyasar_payment 1.0.0 [flutter webview_flutter]
- provider 6.0.2 [collection flutter nested]
- shimmer 2.0.0 [flutter]
- url_launcher 6.0.18 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]

dev dependencies:
- flutter_lints 1.0.4 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- _fe_analyzer_shared 22.0.0 [meta]
- analyzer 1.7.2 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml pedantic]
- archive 3.1.6 [crypto path]
- args 2.3.0
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.2.1 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.5 [meta path]
- clock 1.1.0
- cloud_firestore_platform_interface 5.4.11 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.6.6 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- collection 1.15.0
- convert 3.0.1 [typed_data]
- cross_file 0.3.2 [flutter js meta]
- crypto 3.0.1 [collection typed_data]
- dart_style 2.1.1 [analyzer args path pub_semver source_span]
- fake_async 1.2.0 [clock collection]
- ffi 1.1.2
- file 6.1.2 [meta path]
- firebase_auth_platform_interface 6.1.10 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.3.6 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core 1.11.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_core_platform_interface 4.2.3 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.5.3 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_messaging_platform_interface 3.1.5 [firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 2.2.6 [firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- firebase_storage_platform_interface 4.0.12 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.2.6 [async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_plugin_android_lifecycle 2.0.5 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- glob 2.0.2 [async collection file path string_scanner]
- google_maps_flutter_platform_interface 2.1.3 [collection flutter meta plugin_platform_interface stream_transform]
- http 0.13.4 [async http_parser meta path]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- image 3.1.0 [archive meta xml]
- image_picker_for_web 2.1.5 [flutter flutter_web_plugins image_picker_platform_interface pedantic]
- image_picker_platform_interface 2.4.1 [flutter http meta plugin_platform_interface cross_file]
- intl 0.17.0 [clock path]
- js 0.6.3
- json_annotation 4.1.0 [meta]
- json_serializable 4.1.4 [analyzer build build_config collection json_annotation meta path source_gen]
- lints 1.0.1
- location_platform_interface 2.3.0 [flutter meta plugin_platform_interface]
- location_web 3.1.1 [flutter flutter_web_plugins http_parser js location_platform_interface meta]
- logging 1.0.2
- matcher 0.12.11 [stack_trace]
- meta 1.7.0
- nested 1.0.0 [flutter]
- octo_image 1.0.1 [flutter flutter_blurhash]
- package_config 2.0.2 [path]
- path 1.8.0
- path_provider 2.0.8 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- path_provider_android 2.0.11 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.7 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.4 [ffi flutter meta path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.4 [flutter meta path_provider_platform_interface]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.4 [ffi flutter meta path path_provider_platform_interface win32]
- pedantic 1.11.1
- petitparser 4.4.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.2 [meta]
- process 4.2.4 [file path platform]
- pub_semver 2.1.0 [collection meta]
- pubspec_parse 1.1.0 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.0.1+1 [matcher]
- rxdart 0.27.3
- sky_engine 0.0.99
- source_gen 1.0.3 [analyzer async build dart_style glob meta path pedantic source_span yaml]
- source_span 1.8.1 [collection path term_glyph]
- sqflite 2.0.1 [flutter sqflite_common path]
- sqflite_common 2.1.0 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.4.3 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0 [collection]
- url_launcher_android 6.0.14 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.0.14 [flutter url_launcher_platform_interface]
- url_launcher_linux 2.0.2 [flutter]
- url_launcher_macos 2.0.2 [flutter]
- url_launcher_platform_interface 2.0.4 [flutter plugin_platform_interface]
- url_launcher_web 2.0.6 [flutter flutter_web_plugins url_launcher_platform_interface]
- url_launcher_windows 2.0.2 [flutter]
- uuid 3.0.5 [crypto]
- vector_math 2.1.1
- watcher 1.0.1 [async path]
- webview_flutter 3.0.0 [flutter webview_flutter_android webview_flutter_platform_interface webview_flutter_wkwebview]
- webview_flutter_android 2.8.2 [flutter webview_flutter_platform_interface]
- webview_flutter_platform_interface 1.8.1 [flutter plugin_platform_interface]
- webview_flutter_wkwebview 2.7.1 [flutter webview_flutter_platform_interface]
- win32 2.3.3 [ffi]
- xdg_directories 0.2.0 [meta path process]
- xml 5.3.1 [collection meta petitparser]
- yaml 3.1.0 [collection source_span string_scanner]

@Shlaayl
This issue has been fixed yesterday:https://github.com/FirebaseExtended/flutterfire/issues/7752, so just keep an eye on latest messaging plugin version to roll out and then you can see the behavior working properly.
Closing this as already fixed.

I updated firebase_messaging from 11.2.5  to 11.2.6 but the problem is still there.


fix path to master.json file in RH SSO Realm Model chapter

Thanks @Ladicek
LGTM. merging.

Merged: https://github.com/openshiftio/appdev-documentation/commit/7bd71191b31519e08838cefa6396d357f35a1097


Support for extra data in enums

Hello! 👋
I've been looking at the enum documentation and have been wondering wether there's an option to add some kind of behavior or extra attributes to enums?
I'm talking about something similar to this: https://github.com/infinum/enumerations#advanced-usage

All types support meta data via Type#meta so you can do this:
require 'dry/types'

module Types
  include Dry::Types.module
end

Status = Types::Strict::Symbol
  .enum(0 => :draft, 1 => :published)
  .meta(draft: 'a draft article', published: 'a published article')

puts Status.inspect
# #<Dry::Types[Enum<Constrained<Definition<Symbol>
#   rule=[type?(Symbol) AND included_in?([0, 1])]>
#   mapping={0=>:draft, 1=>:published}
#   meta={draft: "a draft article", published: "a published article"}>]>

puts Status.meta[:draft]
# "a draft article"

puts Status.meta[:published]
# "a published article"


Feature/core name

Extensions for

Automatic determination of the index name
Extensions for date filters and facets

soley the comment from my last review is now unaddressed

Oh, I overlooked that. The GitHub actions seem to have a problem right now. But the project can be validated properly.


UI doesn't indicate each thread listed in the call stack as having been "PAUSED ON EXCEPTION" on loading core dump

Version of VS Code: 1.67.2
OS: Ubuntu 18.04.6 LTS
Reproducible steps that cause the issue:
Have Visual Studio Code load a core dump in lldb via lldb-vscode (debug adapter for Visual Studio Code available as part of the LLDB project, which, to the best of my knowledge, is unpublished) by employing an attach configuration akin to the following –
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Post-mortem Debug",
            "type": "lldb-vscode",
            "request": "attach",
            "program": "${workspaceFolder}/a.out",
            "coreFile": "${workspaceFolder}/a.out.core"
        }
    ]
}

What I expect to see, versus what I actually see:
All threads in the process are stopped with reason exception –
...

{"body":{"allThreadsStopped":true,"preserveFocusHint":false,"reason":"exception","threadCausedFocus":true,"threadId":848},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":843},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":844},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":845},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":846},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":847},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":849},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":850},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":851},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":852},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":853},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":854},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":855},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":856},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1881},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1882},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1883},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1884},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1886},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1887},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1888},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1889},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":true,"preserveFocusHint":true,"reason":"exception","threadId":1890},"event":"stopped","seq":0,"type":"event"}

...

So, I'd expect the UI to indicate each thread listed in the call stack as having been PAUSED ON EXCEPTION. However, I see that the UI indicates only Thread # 23 as having been PAUSED ON EXCEPTION, and others as having been PAUSED –

What's curious, though, is that if allThreadsStopped is reported as false in each of the pertinent stopped events, then the UI indicates all threads in the call stack as having been PAUSED ON EXCEPTION –
...

{"body":{"allThreadsStopped":false,"preserveFocusHint":false,"reason":"exception","threadCausedFocus":true,"threadId":848},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":843},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":844},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":845},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":846},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":847},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":849},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":850},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":851},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":852},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":853},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":854},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":855},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":856},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1881},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1882},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1883},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1884},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1886},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1887},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1888},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1889},"event":"stopped","seq":0,"type":"event"}

{"body":{"allThreadsStopped":false,"preserveFocusHint":true,"reason":"exception","threadId":1890},"event":"stopped","seq":0,"type":"event"}

...

Kindly let me know in case any additional information is required.

Initial investigation shows that this is from https://github.com/microsoft/vscode/blob/a803dc359aa1ed48c7acf074dd81d6af1525c8c8/src/vs/workbench/contrib/debug/browser/debugSession.ts#L892
I think it essentially clears the previous pause reason when processing the allThreadsStopped. I think this could keep the previous stoppedDetails on the thread. But, isn't it odd to send allThreadsStopped if you are going to send individual stopped events for every thread anyway?

@roblourens, I imagine that that's an implication of the design. Currently, LLDB supports only process-centric debugging. So, when any thread in a process stops, all the other threads are stopped too. I reckon that that's why lldb-vscode reports allThreadsStopped as true for every stopped event. However, AFAICT, lldb-vscode maintains neither a list nor the states of the processes and threads under debug, and relies on pertinent events broadcast by LLDB to report stopped events. Now, if I'm not wrong, then all the threads in a core dump are stopped. So, I'd expect LLDB to broadcast a pertinent event for each thread in a core dump, and lldb-vscode to consequently report stopped events for all the threads.

Sure, that's fine. Seems correct for vscode to support this the way you're using it.

@roblourens, thanks! 😁
BTW, is it too early for any work on this to be planned for a specific release? I only ask so that I can offer our customers a ballpark estimate of when they can expect the fix for this to become available. 😬

How does next month sound? 😄

And once the PR above is merged, and we have resumed shipping new Insiders after this week's release (should be by the end of the week, or next Monday), I'd appreciate you verifying that the issue is actually fixed

@roblourens, sure, I'd be happy to! Thanks a lot for looking into this! 😁

@roblourens, the UI indicates each thread listed in the call stack as having been PAUSED ON EXCEPTION in the latest release of Visual Studio Code Insiders.


Add NPM/yarn finest usage into Readme.md

Added better inclusion of multi.js installed via packages asper request on this ISSUE

Hi!
I'm sorry about the late reply to this. This is great and a much-needed improvement to the readme. I've adjusted the phrasing a bit and also removed the version bump as this is only a documentation change.
Thanks!

Hi!
I'm sorry about the late reply to this. This is great and a much-needed improvement to the readme. I've adjusted the phrasing a bit and also removed the version bump as this is only a documentation change.
Thanks!


Document functions available for the user

pacmacs-start
pacmacs-quit
pacmacs-score
pacmacs-up
pacmacs-down
pacmacs-left
pacmacs-right
etc.

All of this functions must have the doc string.


doc(Lab4, Lab5)

split README.md and integrated with current pages framework

Manually merged in a42fdb02691ce8008e498947e62114c86821e1c7


refactor(p2p)!: Extract TCP transport

Closes https://github.com/cometbft/cometbft/issues/4301
This PR refactors the peer-to-peer (p2p) module by extracting the TCP transport into its own component, laying groundwork for potential QUIC transport integration. It introduces a breaking change to the Go API, particularly in the handling of peer connections and transport interfaces.
Important: no logic has been changed; just moving stuff and updating interfaces.
Changes
Public API

Updated Transport interface

// Transport emits and connects to Peers. The implementation of Peer is left to
// the transport. Each transport is also responsible to filter establishing
// peers specific to its domain.
type Transport interface {
	// NetAddr returns the network address of the local node.
	NetAddr() na.Addr

	// Accept waits for and returns the next connection to the local node.
	Accept() (net.Conn, *na.Addr, error)

	// Dial dials the given address and returns a connection.
	Dial(addr na.Addr) (net.Conn, error)

	// Cleanup any resources associated with the given connection.
	//
	// Must be run when the peer is dropped for any reason.
	Cleanup(conn net.Conn) error
}

New Go packages (extracted from p2p)

netaddress - network address
nodeinfo - node's info
nodekey - ID of the node
transport/tcp - TCP transport

Internalized Go packages

fuzz - fuzzing connection

#go-api-breaking

A general fast comment here: if we don't rename GetChannels() reactor's method and the type that stores it, we have much less changes and maybe is not (that) API breaking?

A general fast comment here: if we don't rename GetChannels() reactor's method and the type that stores it, we have much less changes and maybe is not (that) API breaking?

It will be API-breaking anyway because I had to extract many packages. But if you think the renaming is unjustified, I can revert it to the "channels" concept.

nice job!!


Make it possible to create various segmentation tasks and switch between them

Suggestion from @laurentletg
Currently, a use case is defined with a config file. If users would like to create multiple tasks (eg: segmentation of brain MS lesions, segmentation of brain tumors, etc.) it would be nice for the user to be able to generate multiple config files and then select the desired one from the Slicer module.
This would probably require changing the module name in the CMake file, python filename (and script), and .ui file (unless there is a way to do this directly in the extension wizard) -

This is now supported. The configuration files are saved in the output folder. Therefore, a user is able to continue working on multiple projects by simply selecting one of the project's output folder and the correct configurations will be applied to the extension. In addition, this output folder can be shared between members of a same team for collaboration and/or revision.
I have made some videos describing the new configuration GUI, here is the link!

Relying on @delphinepilon  comment above. Issue solved.


[Bug report] Doris web e2e failed when only 8GB memory in docker

Version
main branch
Describe what's wrong
When I setting 8GB memory in the Docker, then doris web e2e failed.
When I setting 16GB memory in the Docker, then doris web e2e success.

Error message and/or stacktrace
OpenJDK 64-Bit Server VM (25.402-b06) for linux-aarch64 JRE (1.8.0_402-8u402-ga-2ubuntu1~22.04-b06), built on Feb 27 2024 07:25:38 by "buildd" with gcc 11.4.0
Memory: 4k page, physical 8137636k(171532k free), swap 1048572k(1048048k free)
CommandLine flags: -XX:-CriticalJNINatives -XX:InitialHeapSize=130202176 -XX:MaxHeapSize=1073741824 -XX:+PrintGC -XX:+PrintGCTimeStamps -XX:+UseCompressedClassPointers -XX:+UseCompressedOops -XX:+UseParallelGC
0.217: [GC (Allocation Failure)  32768K->4093K(123904K), 0.0032893 secs]
0.330: [GC (Allocation Failure)  36861K->4303K(156672K), 0.0024209 secs]
How to reproduce
Setting 8 GB memory in the dorcker, and run doris web e2e test
Additional context
No response

@zhoukangcn Could you help to fix this issue?

@LauraXia123 I will fix this issue later.
I have a little problem, what's the minum docker memory?
We'd best add the limit in Docker environment checker and add to documents.
Do you have any advice? @xunliu

@LauraXia123 I try to fix this issue in #3883
But I cannot reproduce this failure in my local machine with 8G memory in docker.
Can you help to test with image docker.io/zhoukangcn/gravitino-ci-doris:0.1.4
you need to update in file integration-test/build.gradle.kts and catalogs/catalog-jdbc-doris/build.gradle.kts

@LauraXia123 as #3883 has been merged. Please try again.

This has been fixed.


Resolve cases of NOASSERTION

Type: Incorrect
Summary:
Resolve cases of NOASSERTION
Details:
All of the files referenced were incorrectly declared as "NOASSERTION".
They are actually all under Amazon Software License (ASL) which is not a valid SPDX license.
See: https://aws.amazon.com/asl/
Resolution:
Updated the declared license of all references files in accordance with their content/headers. Because the Amazon Software License is not a valid SPDX license, files were declared as "OTHER".
Affected definitions:

amazon-kinesis-producer 0.12.11

Copyright attributions are being added to scope of curation unintentionally due to issue https://github.com/clearlydefined/service/issues/677. They should be the same as their original values prior to curation, so no harm done in this case.


Invalid links

On the TextBuffer page the link in the main description paragraph is improperly rendered.

Includes the TextTag page as well.

Thanks for the feedback!

These are actually all for the TextWidget class that doesn't seem to exist in the docs. So maybe the docs are trying to auto-link it, but not finding the reference, leave it as plain text.

I think yes. But since this doc is the one from the current crates.io version, these issues might be resolved since then.

Its plain text in Gtk-3.0.gir on real page https://developer.gnome.org/gtk3/stable/GtkTextBuffer.html links to https://developer.gnome.org/gtk3/stable/TextWidget.html.
We can replace [xxx][url] to link, but not sure it we need do that.

I suppose the way to fix this would be to make the struct TextBuffer docs manual and put a correct link there. Eventually that page could be translated to Rust and hosted on gtk-rs.org

Isn't it better to wait for the next release? We can add it manually on the doc repo at that moment.

The docs repo has so much machine-generated stuff that we can't hope to also manage manual patches. We could look into making the docs generator repwrite links from [xxx][url] to [xxx](https://developer.gnome.org/gtk3/stable/url).

No I mean, we already have the doc overriding. Couldn't it the same here as well?

Ah, that what I meant by manual docs.

Oh my bad. Then we agree . :smiley:

I think I'll do it. I never really work on the doc repo.

When changing [xxx][url] to [xxx](https://developer.gnome.org/gtk3/stable/url) we need check for links, that we already have in doc (example for RadioButton we need link to your site not gnome's).

The first link I posted on TextBuffer has been fixed, but the link on TextTag is still incorrect.

Both of my original examples here have fixed links now, so I'm going to go ahead and close this.


Use this library

I'm trying to use this library but i can't find any tutorial for using this framework  with another project.Please give me some suggestion about that.

i import file except view and controller but still getting error:
CGPDFPageRef page = CGPDFDocumentGetPage(document, 1);
Scanner *scanner = [Scanner scannerWithPage:page];
1 undeclare idenifier 'document'
2 no known class method for selector 'scannerWithPage:'


test: missing e2e tests on the dart_frog_auth examples

Description
Add e2e tests to the dart_frog_auth examples
Requirements

[ ] e2e tests on the dart_frog_auth examples
[ ] enable workflow jobs running those tests

related: https://github.com/VeryGoodOpenSource/dart_frog/pull/820


Search Function - some translations are missing

Q
A

Bug?
no

New Feature?
no

Sulu Version
1.3.0

Actual Behavior
Some Translations are missing in the search function. When I open the drop down menu there is no translation for the following translation keys:
search-overlay.index.page_sulu_io.title
search-overlay.index.page_spitex_intranet.title
Expected Behavior
There should be a translation for the mentioned translation keys.
Steps to Reproduce

Click on the search icon
Open the drop down menu next to the search bar

Just tested it, and it is working for me:

Can you try to purging the index with the massive:search:purge and reindex with the massive:search:reindex command? I think the keys you are talking about could be leftovers from old webspaces.
Could you also check if the request /admin/search/indexes returns the correct data? For the webspaces there should be an entry like this:
[{
    "indexName":"page_sulu_io",
    "name":"sulu.io",
    "securityContext":"sulu.webspaces.sulu_io",
    "contexts":[]
}]

Since there isn't more information coming I am closing this issue.


License

Hi @nikis05, I'm not sure what your plans are for this crate / repository, but I was wondering if you could include a license? It would make it easier for people. Thanks for this.

Sure, I will add a MIT license when I’m at computer


[joiner-adv] initial implementation of Joiner Advertisement

This PR implements Thread based Joiner Advertisement from Joiner side, including:

A new MeshCop TLV: Joiner Advertisement TLV which includes IEEE OUI and the Advertisement Data
New APIs to set Joiner Advertisement which will be sent along with Discovery Request messages

@bukepo @librasungirl @jwhui any comments?


KCEF initialization is not covered in the documentation

I've encountered problem on Windows. The WebView shows blank screen for all possible usage scenarios. After some debugging, I found the problem: KCEF returns the state NEW and webview then just shows nothing without errors. In order to fix that I had to add
KCEF.init(
    builder = {
        progress {
            onDownloading {
                println("Download progress: $it%")
            }
        }
        release(true)
    }
)

at the start of the application. It is covered here. But webview does not seem to do any initialization or check if the framework is initialized.
I think that the component should check if the client is available and initialize it properly. In any case, the initialization should be covered in the documentation.

@SPC-code Thanks for your feedback. Have you checked this doc for desktop setup?

I will place the reminder in a more noticeable location to prevent it from being overlooked.


Add system uptime metric

What are you trying to achieve?
I want to add a metric to the semantic conventions that will describe the system uptime. How about system.uptime?
Additional context.
This is reported by Telegraf as uptime field of the system metric (in seconds).
Here's a related proposal on the hostmetrics receiver to add this metric: https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/14130.

Q. Is there any plan to do it?

@kernelpanic77 offered to do it here https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/31627#issuecomment-2133882411 :)

Looks like we have an agreement here. Just need someone to submit a PR

Is there any way we can generalize this to be an "uptime" of any entity, not just of "system"? What if we make this an uptime metric with the Resource describing what it is about (e.g. Resource can have "host.name=foo" to indicate that it is an uptime of a host).

Is there any way we can generalize this to be an "uptime" of any entity, not just of "system"? What if we make this an uptime metric with the Resource describing what it is about (e.g. Resource can have "host.name=foo" to indicate that it is an uptime of a host).

I suppose we could do it, I wonder what others think.
The uptime attribute name would not be namespaced, unlike system.uptime that is namespaced to system. Looking at the Attributes Registry, it doesn't look like we currently have any non-namespaced attributes in the semantic conventions. Is that true?

Is there any way we can generalize this to be an "uptime" of any entity, not just of "system"?

I think this is part of a broader discussion taking place in https://github.com/open-telemetry/semantic-conventions/issues/1161 (system.uptime vs process.uptime vs container.uptime). The main benefit of using the metric without namespace seems to be dashboards correlation and avoiding deduplication. But it comes at the cost of implying resource attributes to corresponding metrics (not sure if this is possible in semconv), for example, the uptime metric should always be linked to either host.name, process.pid or container.id.
During the System Semantic Conventions SIG (20/06/2024) we agreed on keeping the metrics in namespaces (even if there are duplications) due to:

The potential for minute differences between the meanings of seemingly identical metrics between the different contexts
The namespaces also semantically represent the reporting source, making query scenarios more clear (i.e. "I want all my operating system process metrics" or "I want all my jvm metrics" has a clear separation due to the metrics reported from each source all having their respective namespaces)


Bug: Page not found in CONTRIBUTING.md for https://github.com/MathJax/MathJax-testing

Issue Summary
There is no page for the testing framework at https://github.com/MathJax/MathJax-testing
Steps to Reproduce

Open CONTRIBUTING.md from "Please review the guidelines for contributing to this repository."
Jump to "testing and quality assurance"
When you click the following link you get 404 (Jedi Octocat):
"If you want to get involved with testing MathJax, there is a set of QA Documentation in our testing framework.."

This is a bug because we should not see the missing page error.
(and then I can not get involved in testing MathJax ;) )
Technical details

MathJax Version: GitHub page CONTRIBUTING.md
Client OS: Linux
Browser: Firefox 38.6.1 ESR

Thanks for catching that. It's https://github.com/mathjax/MathJax-test by the way.

Fixed in develop.


Update to 20180923

Hi Daniel,
Thank you for packaging OpenRA!
OpenRA 20180923 has been released so this is a version bump to the latest release.

I changed the source url because the *-source.tar.bz2 downloads are already pre-configured with the correct in-game version strings (see openRA wiki)
I also made the online multiplayer functionality working in the snap (I got the 'Failed to query server list.' error before).
To fix this the mono certstore is now populated before starting the game.

Thanks, I've merged the request so this should build shortly.

Just a heads up, it seems that the package on the snapcraft store is still at the old version. Maybe something went wrong with the build?

thanks for the reminder. I've fixed the build and released it.


Address Ember 2.1 deprecations

Application becomes first arg to initializer function

This would also require a dist/build if accepted. @ckung


Release 2021-07-21

Included:

azure-verticals-agrifood-farming

Docs Build status updates of commit 8371c3e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

package.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Adds a script to audit our package.json

The governance audit task does the same thing. I don't see why we need this.

We can use this to fail builds on PR

the governance task can also fail builds, see https://github.com/microsoft/azuredatastudio/pull/8096/files#diff-42a49d406e55711ca10c2213569ab204R65

Does that task search all the various package.jsons under our project?

Yes is checks every package.json, every lock file and every node nodule


⚡Added toggles for player and embed colors

Added toggle parameters for enabling/disabling player color and custom color embeds.

Nice one, thanks!


Sorting / UI / Login

Sorry if this has been asked (I couldn't find any mention but search terms can vary so much...) but my library is flooded with hundreds of "things" and I was wondering if there was a way to filter just what is installed?
There is a little search input above the list in library view and a button? beside that with no label. I click on that and check Installed under App State but it doesn't work. The App Type filters some things but not completely or correctly.

Did some digging this filter might work but opensteam doesn't link to the library correctly. Even when I manually add the library directory it still doesn't work i.e. can't launch anything either.

I also noticed most the menu entries at the top (view friends etc) have no entries if clicked on. I get it if it's not implemented but normally you'd "gray" them if that was the case or at least have a place holder menu entry that just says not implemented yet. (lots of text is cut off too Insta, Enable compatibility for t, please be patient as th, with a beta ke, heck cod)
Another odd thing was it seemed to import one single solitary category but not the others.
Lastly logging in for the first time after inputting the security code from email you can't progress. You have to cancel everything and restart. Perhaps that should be noted in the UI until it's fixed?

Hi. There's lots of issues with the Qt/C++ version. This issue is due to us using app tags, as it's the closest thing to categories I could find. However, it turns out app tags were left unused after the new library ui update, and instead valve made yet another system for storing all the new library data. The C# version handles the library data correctly (no library GUI yet though). And the login is also much improved in that version.

As for filters, the new library will support dynamic collections, which is also what ValveSteam uses for it's search.

When will the C# build be buildable?

When the pull request gets merged it'll be as workable as the old Qt based one at minimum (but with broken features actually working). Then I'll add in other important features.

Hey I just noticed you wanted a better logo. I do A/V work so if you toss me a general idea I could mock some things up. Also if you need dings, boops, and buzzers for audio cues in the UI toss me a list and I'll make them.

Sounds cool. I like the general design of the current one where it mimics steam's logo, but we should color it red and make it not be of a copyright similarity. The piston would need to change somewhat for that reason, maybe just a solid white fill or something? I quite like the color red and will eventually also build an Avalonia theme for the app around it (for now it just uses Avalonia defaults, which do look great, but are rather generic looking)
As for the sounds? Maybe when I add something similar to Steam's big picture mode or just a "controller mode" for the main UI (this is pretty low on the priority list though, might take a couple years or so to get here)
Currently I'm focusing on getting the library done, then some sort of way to host HTML pages inside the client to use for store and community pages (which disable when launching a game, unlike steam where they stay a resource hog...).
Then the bulk of the dev effort for getting it workable should be over, with just adding minor things like a settings page.
Then I can start with all the fun and new stuff like friends lists, concommand console and big picture mode/controller support.

Heh I was bored earlier so I whipped up a few things. Just a couple of things, Error, Uh-Oh, Attention, Launch, Download Complete (kinda cornball heh)
Open-Steam-Sounds.zip
I'll give the logo a think. I had an idea earlier but I'm unsure if I can get it clean and readable but I'll see what I can come up with.

So you want flat single color, but something that doesn't brush against trade infringement but like the current one that could be trade infringement heh. The only idea I had like that (so far) was a piston, no steam, no lever arms. Sounds like either way you just want the current one made more exactly.

I quite fancy the gradient. The piston could also be replaced with something else steam engine related (ideally something that would fit into the logo, but it could also be the piston but just ending abruptly (so that it looks like it extends beyond the background circle)

For now I've just cleaned up the original with in circle and "breakout" styles in white and black. I'll try the piston in a bit but in the end it will be in the repo and you can just use what you want. Check out the big picture audio and let me know if there are any other audio cues you need.

That piston design is gorgeous, that could definitely be the new logo. Can't review the new sounds yet but will soon. The original sounds you made sounded great as well though.
Is it ok if take that iteration of it and use it in the README and app (in the C# branch)?

Go ahead, I set the license to MIT so it's compatible.

Brilliant, will add the logo in the next commit.
Also, brilliant big picture entering jingle. I have audacity installed so I can cut silences out if needs be.
Will also credit you in the README and app credits, and when I eventually decide to make a Discord server I'll give you a "Logo Designer" and "Sound designer" role there if you want to.

The audio is trimmed as tight as it goes. I'm an audio engineer so you won't need to touch that. I just added a header version.

There's no fonts chosen yet, but if you want to you can also find a good free to use font. Also, yeah guess that audio bit makes sense, you don't want it abruptly ending, just that I couldn't hear anything towards the end.

Most people probably won't hear the tail. I'm in a studio and even with my engineering monitors it's still best to just watch the meters. I may fool a bit later on. Free/Open is already tricky for me in the font world. I've got like 10K installed but few are open heh.

Hmm, Some sans font should be good enough for now until typefaces and design language is decided. If you want to, you could make banners for OpenSteamworks.Client and OpenSteamworks as well (First one is a C# library which allows you to make your own steam client UI's and general client behaviour like logging in, and OpenSteamworks is just a simple wrapper over the native APIs provided by steamclient.dll which is aimed at making tools which make use of Steam's Client APIs like external compat tool managers)

I just updated the header version with a nice open font ;) (TASA) It's not 100% what I'd go for but it works.

I assume the header svg isn't supposed to cut off?

Hmm, and I'm planning to eventually split this into three repos:

this repo, OpenSteamClient (has ClientUI and ClientConsole)

Would be the main repo for non developers, description: "Community-made Steam Client alternative"

OpenSteamworks (has OpenSteamworks)

Description: "Library for interfacing with a running Steam Client"

OpenSteamworks.Client (has OpenSteamworks.Client)

Description: "Library for creating your own Steam Client"

What are you viewing the svg with? Remember it's SVG, if you don't have the right fonts installed (which it looks like you don't) it won't render correctly, hence why I added png renders.

Ah, that makes sense. Will use the renders in the README then.
Oh, and I quite like Nitrox's style of banner, ignoring the fact that it's a gif (Icon, name, description, fancy background). But we could also do something like

(quick mockup of the layout only, GH doesn't like max-width and max-height tags)

    OpenSteamClient

    A community-made, open-source Steam client.

Though we at the moment don't have any images to show for the background so let's just use the gradient, but maybe darker red or a black/gray gradient.

This could become a huge discussion heh. Format and application can get insane. Print vs web, accessibility, design choices vs possible user choices (light/dark webpage eats like images).

Yep. Though having the project name and description as text should help accessibility, and using a plain old table should make light/dark work well.

Well that's what alt/text tag in the html is for ;)


Add example extractors

Implemented and tested these extractors:

Invoice Extractor (using Donut)
Language Extractor (provided full-text, chunks the text and predicts the language for each chunk)
Identity Extractor (hashes the input-bytes into a vector using sha256, useful for quick duplicate-checks)

How to reproduce:
(1) Package the extractor into a docker image cargo run extractor package --dev -v --config-path extractors/simple_invoice_parser.yaml
(2) Run the extractor, mounting any files required
docker run \
--mount type=bind,source="$(pwd)/data",target=/indexify/data \
-it \
--rm \
-e RUST_LOG=debug \
yenicelik/simple-invoice-parser extractor extract \
--file data/...

cargo run can be replaced by ./target/debug/indexify or indexify if the binaries were added to the PATH.
For maintainers & contributors:
These steps are usually not needed when working only extractor! Currently, the extractor-base image is pulled from DockerHub, due to how BuildKit works (it does not use a local registry). Unfortunately, it seems tedious to resolve this (see https://github.com/moby/buildkit/issues/2343). If you need to modify the rust code to run the extractors, please run

create an account for dockerhub
modify the Makefile credentials at the top of the Makefile
run make build-base-extractor-push, this will build the image, and push it to dockerhub
modify dockerfiles/Dockerfiles.extractor to use your own FROM ..., (i.e. FROM yenicelik/indexify-extractor-base instead FROM diptanu/indexify-extractor-base).
Then you can go to the "How to reproduce" section above

Please make sure to merge https://github.com/diptanu/indexify/pull/146 before this

Will move the extractors here https://github.com/tensorlakeai/indexify-extractors and remove them from diptanu/indexify


Security Needs

Need to implement Helmet
Need to talk with security folks about pen testing.
Hardware security requirements for iot devices.
Autoboot script of limited user account/hardware lockout?

I start working on this, I do Cyber Security pen testing on the side, along with my web/mobile app development.

I start working on this, I do Cyber Security pen testing on the side, along with my web/mobile app development.
:octocat:

what are the rules of engagement, for the security program, there is only so much I can do on my local machine, what kinds of attacks can I do for testing?


mkl spring update 1

solve  resnet / inception-v3  do not coverage for memory reuse
bn support global stats

Signed-off-by: lingyan<EMAIL_ADDRESS>

@piiswrong  tested already been done ,please review thanks!


bug: Use of incorrect flag --tf-vars

Describe the bug
This is related to fix #651.
The implementation of the fix considered --tf-vars as the flag to be used to pass a terraform.tfvars file, but the correct flag is -var-file.  Terraform does not recognize --tf-vars of -tf-vars.
On the other side, terraform plan/apply works fine when you use the flag -var-file.
To Reproduce
Steps to reproduce the behaviour:

Create any terraform code
Create a terraform.tfvars file
Run a plan command for your TF code passing the variables terraform.tfvars file:
terraform plan --tf-vars terraform.tfvars
See error:

Expected behavior
The expected behaviour would be a plan result, that can only be achieved following the correct sintax:
terraform plan -var-file terraform.tfvars

System Info

tfsec version: v1.15.0
terraform version: v1.1.6
OS: Tested both locally on Windows 11 and using ubuntu-latest in Azure DevOps

Please let me know if you need any additional information.
Thank you;
Diogo Maul

Hey @diogomaul  - thanks for pointing this out.
I'm reluctant to remove the flag for fear of breaking existing flows in people's CI. I've added --var-file as a duplication of the flag to offer an alternative more consistent with Terraform

Hi @owenrumney, this is awesome! And yes, you are right regarding breaking this for people that already use the other (old?) flag.
Thank you so much for the quick turnaround!

Hey @diogomaul  - this is finally available in v1.19.0 https://github.com/aquasecurity/tfsec/releases/tag/v1.19.0


[DOCFIX] Update aws emr doc

add link to aws cli cmd for emr create-clusters
describe alternative method to reference json configuration file

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.5 please


Create topic branch set for proto3

Following the process defined here, though I am a collaborator in this repo so I suppose I could go ahead and just create a branch myself. Who is in charge of creating these? @skeenan maybe?

I've created the protobuf branch here.

Yes, creating branch yourself was the right thing to do.

On 10 Nov 2015, at 02:17, CH Albach<EMAIL_ADDRESS>wrote:
Following the process defined here https://github.com/ga4gh/schemas/blob/master/CONTRIBUTING.md#topic-branches, though I am a collaborator in this repo so I suppose I could go ahead and just create a branch myself. Who is in charge of creating these? @skeenan https://github.com/skeenan maybe?
—
Reply to this email directly or view it on GitHub https://github.com/ga4gh/schemas/issues/486.


Added to the acceptable date formats.

Added 19 date formats including the month in "long" format.

Thank you @kpaape for your work! This PR is merged to master and will be released in 1.3 version.


[DE-665]: GET /_api/transaction

Introduces list_transactions() to list the running stream transactions

Codecov Report

Merging #283 (fde83e4) into main (744a6e9) will decrease coverage by 0.50%.
Report is 18 commits behind head on main.
The diff coverage is 93.33%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
- Coverage   99.01%   98.51%   -0.50%
==========================================
  Files          26       26
  Lines        3958     4040      +82
==========================================
+ Hits         3919     3980      +61
- Misses         39       60      +21

Files Changed
Coverage Δ

arango/client.py
98.41% <ø> (ø)

arango/http.py
100.00% <ø> (ø)

arango/database.py
99.22% <87.50%> (-0.64%)
:arrow_down:

arango/collection.py
97.41% <92.92%> (-1.09%)
:arrow_down:

arango/exceptions.py
100.00% <100.00%> (ø)

arango/request.py
94.44% <100.00%> (ø)

arango/utils.py
87.50% <100.00%> (-12.50%)
:arrow_down:

... and 2 files with indirect coverage changes


Docs tell user to install missing debian package

The docs at https://github.com/HyphaApp/hypha/blob/main/docs/setup/deployment/stand-alone.md reference postgresql-server-dev-12 but this package is not installable from the standard repos of any currently available version of Debian.  Perhaps the recent Debian release shuffled it out of the repos.
To Reproduce
Search for postgresql-server-dev-12 on https://packages.debian.org:
You have searched for packages that names contain postgresql-server-dev-12 in all suites, all sections, and all architectures.

Sorry, your search gave no results

See https://packages.debian.org/search?keywords=postgresql-server-dev-12&searchon=names&suite=all&section=all
Alternatively, try to find it in apt or install it.  You'll get an error message: "No packages found"
Expected behavior
When the documentation instructs me to install a debian package, I expect it to be in the standard repos or to be told what PPA to pull it from.
Priority

High (keeping you from completing day-to-day tasks)

I'm not sure if one cannot complete the install without this package, but I assume not, so I think this bug is fairly high priority, as new users will block on it.
Affected roles

Staff (technical staff installing hypha)
Developers

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Hypha tells users to install an old version of postgres.  At some point, more postgres-related Debian packages might stop being available.  At some point, should hypha consider documenting installation using a more recent version of postgres?
Also, I haven't checked the docker scripts, but I wonder if they might also fail on the same point.

These docs were written for Ubuntu 18.04LTS, we really need to update them.


pull request for e2e test at 12/18/2016 20:56:18

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


completed whiteboard - Binary Search

Binary Search

[x] - Completed whiteboard
[x] - Checked code on computer
[x] - Passed all tests
[x] - Uploaded photo of whiteboard


parse filename with quote

Browser sent header:
Content-Disposition: form-data; name="file"; filename="100129-03"100129.jpg"
It parsed as:
originalFilename: '"100129.jpg'
But expected:
originalFilename: '100129-03"100129.jpg'

looks like we should use this module: https://github.com/jshttp/content-disposition
to parse content-disposition instead of using the naive regex that is currently in place.

So there are a few "quirks" for parsing the CD header in multipart vs http (because... browsers :/). If you plan to add it, I believe I at least started hacking on adding a "multipart quirks mode" and can wrap it up ASAP :)

Nice! Yeah if you do that we should use it in multiparty.

Cool. That was the whole reason I was doing in the first place :) But now I have motivation since it seems you're likely to end up using it, haha :D

At the very least I'm happily using it for groove basin: https://github.com/andrewrk/groovebasin/blob/master/package.json#L24
:-)

nice. I think this might be a good candidate for a minor version bump since the change in parsing behavior might be considered "risky".

Ah, yea, I suppose so, lol. What about the change on waiting to emit the error on part until after we actually handed part to the user to attach a listener (wow, a mouthful)? 07fabc525ddd72a6b9d9dc60073f04a7121e2e04
Seems like a patch release would be good, or just roll up into the minor?

hmm on second thought maybe we can call these both bugfixes. if there's another bug revealed, we'll fix those too.

damn we're almost down to 0 issues. that's almost motivating me to make it the rest of the way.

lol, yea, all that's left in the issues are feature requests. Any objection to the current master being 4.1.2?

no objection :)


Fix #10449, buffer.getBytes(...) not change a file channel position

Motivation:
Regression appeared after making changes in fix #10360 .
The main problem here that buffer.getBytes(buffer.readerIndex(), fileChannel, fileChannel.position(), localsize)
doesn't change channel position after writes.
Modification:
Manually set position according to the written bytes.
Result:
Fixes #10449 .

@netty-bot test this please

Lgtm

@netty-bot test this please


Feat/allow file validations

This PR is an evolution of waelbeso one it includes:
-Suggested @mileszim changes
-Updated README with options & events.
-Added new useful props (style & accept)
-Catch the jQuery event 'fileuploadprocessfail'. @mileszim The cloudinary library does not catch this event

Great work @AdriaRios, thank you!


Temurin >17 does not support ppc64le

Hello,
the latest support of platform ppc64le is image 17, after that despite there are the jdk available for later versions the docker images are not.
I see that they were built few months ago:

Why are they not supported anymore?
Can you push back also such images?

@roy20021 Unfortunately we were not able to complete testing for Java 20 for that platform.  We will be making sure that Java 21 is complete


Add Nikita Lazarev-Zubov's blog on Medium

Nikita Lazarev-Zubov on Medium
iOS development, architecture in Swift and general topics
https://medium.com/@lazarevzubov

Thank you


Don't log errors when a closed channel is detected

In HTTPServerResponse, channel and handler are weak references to the
underlying NIO channel and the channel handler. These weak vars are used
as a signaling mechanism to indicate a closed connection. While writing
responses back to the underlying channel, we might detect both of this as
nils, which simply means they were deallocated because, most likely, the
underlying connection was closed. This condition does not qualify to be
an error condition.

Codecov Report

Merging #186 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #186   +/-   ##
=======================================
  Coverage   68.35%   68.35%
=======================================
  Files          20       20
  Lines        1324     1324
=======================================
  Hits          905      905
  Misses        419      419

Flag
Coverage Δ

#KituraNet
68.35% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

Sources/KituraNet/HTTP/HTTPServerResponse.swift
53.5% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


MapView.getBoundingBox returns strange values when the map wraps vertically

related #521
specifically, north and south latitude are the same value

#334  #183  #201 all related issues

so when replication (map wrapping around the poles and date time) is on, this is somewhat expected behavior. A simple math check + isReplicationEnabled + zoom level check will get an approximate check


changing the style of ride page

Hey @07sumit1002 can you please assign me this under vsoc'24

these are the screenshots how it will look like

@Krishna100604 I will assign it to you but can you mention this issue in the Discord cab-rental channel so that it will be easy for me to give points

https://discord.com/channels/1234877876382208082/1252619076522475530

https://discord.com/channels/1234877876382208082/1252619076522475530

okk

I have assigned the points you can check


Aggregation framework for $sample does not support sizes larger than collection size

When I try to use $sample with a size of 1 on an empty collection I get an IndexError as the sequence for random is empty. Normally, MongoDB would return nothing in this situation.
In addition, when using $sample, if a size of 5 is used with a collection size of only 1. The aggregation framework should only return the 1 document but it seems that your code may potentially return the 1 document five times.

Hi @DarwinLYang, thanks for reporting. I'll fix it ASAP.


chore: update code formatting tools

Resolves #95
This PR updates this codebases' code formatting tools to look like this:

Prettier is used for code formatting
ESLint is used for code linting

The ESLint configuration is based on eslint-config-standard, so we continue to use the configuration of Standard, but it also includes eslint-config-prettier in order to turn off all stylistic lint rules that may conflict with Prettier. I think this is a nice compromise.
Also included in this PR is lint-staged, which will execute the following commands on staged files during a precommit hook:

when the README is staged, runs doctoc in order to keep the table of contents up-to-date
when a .js file is staged, runs Prettier to format the file and runs eslint --fix to lint the file
when a .json file is staged, runs Prettier to format the file

Since the ESLint configuration has been updated to include the Prettier configuration, I have reformatted all .js and .json files by running prettier --write **/*.+(js|json).
This PR touches a lot of files, but all tests continue to pass, so I'm 👍  on bringing this change in. I'm going to leave it open for a little bit and think about it. @xjamundx if you have thoughts on if this is a 👍  / 👎  change, please let me know.

I'm going to merge this and follow up with better documentation in a CONTRIBUTING file (#84). I think consistent code formatting from all contributors that's enforced multiple ways (IDE, precommit hook, ESLint) is a net positive.


Saving the jump in replay

I have no idea how to save the jump in replay, after I did what I wanted to do I pressed F6, then did the command jumpIn_replaySave, but then nothing happened. Am I doing something wrong or is it just not working for me?

just save the replay like you normally would


Header is not a Control

Control components can take focus of input events and in this case the Header component can steal the touch events on mobile making option selection inconsistent.

https://doc.qt.io/qt-6/qml-qtquick-controls2-control.html#event-handling Here is more details about the specifics related to the issue with events.

PR

So I think we can remove the change done in OptionsButton.qml for this PR.

Ill take another look at this. My memory was that making Header no longer a Control changed how the margins were working primarily due to issues with setting "Layout.preferredWidth: 0". Setting width to 0 has some strange consequences with layouts.

Ok, I'm seeing the same thing as  Shashwat. Going to remove changes to the OptionButton.qml


Implement retriable API events in injected Reconcilers

/area API
/kind feature
I like the way generated Reconcilers can extract an event from the returned error and record that event via the injected EventRecorder. 👍
Unfortunately, there is no way to return an "error" event and retry the reconciliation. What I want is to be able to return an actual error while still being able to provide a better classification than "InternalError".
I suggest we support wrapping events inside a specific error type, similarly to how controller.NewPermanentError creates a special type of error, and take that as a hint that the reconciliation should be retried after the event is recorded.
I'm gonna look into this over the weekend unless someone objects.
/assign

cc @n3wscott
This sounds reasonable to me :D


Nucleo board signal generation

To do

[ ] DAC signal generation
[ ] Sine wave generation
[ ] Wave frequency switching
[ ] Can receiving

Codecov Report

Merging #144 (052cbb7) into master (70e5929) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   38.94%   38.94%
=======================================
  Files          66       66
  Lines        3782     3782
=======================================
  Hits         1473     1473
  Misses       2309     2309

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 70e5929...052cbb7. Read the comment docs.

If you want to run this it's a little cursed rn, you need to make a new project in STM32IDE with the .ioc file then replace Core with the one in this pull request. Will try to find a nicer way to do it, but for now this is the easiest way


add miri test in CI

#18

It takes too much time, I don't think my free GitHub plan is enough for this, I need to run a private VPS for this CI test. then we can merge it.

It is not necessary, if it is public you have all the time you want.

I think we got 2000 minutes per month and darwin tests are using 2x per minute.
I'm updating the project regularly and I'm always running Miri test on my system before merge.
I'm not sure that 2000 minutes will be enough as described in : https://github.com/pricing

It is true! Github give us the copilot and not the minutes. Well, we close it to see if Microsoft releases more minutes...


add unit test cases for core/pkg/ingress/controller/annotations

Just what like the descriptions of two commits, thank you!

This change is

Coverage increased (+0.2%) to 43.894% when pulling 767591fa1847d66bedd6bc8529c447c87caf6c49 on chentao1596:annotations_unittest into a7af8409c89f959369e58c719d93a9dff58b4160 on kubernetes:master.

/lgtm

@chentao1596 thanks!


Missing HTTP trigger on Azure Functions Runtime

I have already installed Azure Functions Runtime and created some functions. I wonder why the HTTP trigger is not available when selecting a template.
For V1 functions I can choose between:

Timer trigger
Queue trigger
Service Bus Queue trigger
Blob trigger
Event Hub trigger
SQL Queue trigger

For V2 Functions there are just three templates available:

Timer trigger
Queue trigger
Blob trigger.

So my question is, do I have to install more templates manually as needed?
Thanks in advance.

Thanks for the feedback! We are currently investigating and will update you shortly.

@apercova I am seeing the HTTP trigger in V2 when checking my Visual Studio:

Can you make sure your visual studio and functions SDK is up to date?

Hi @MicahMcKittrick-MSFT
Let me try to explain my issue.
I followed Azure Functions Runtime on-premise installation steps as described here:
functions-runtime-install
When Runtime was installed, I logged in to the portal, then created a subscription with the default template and created Two Apps: One for version 1 and another one for version 2.
Below you can see the options that I have at creating a new Function, both in V1 and V2.
These options are the same as I described before.
As you can see, there is no option to create an HTTP trigger from the portal.

@apercova got it! Thank you for clarifying that. Are you able to use the search box on the top and look for the HTTP trigger?

I'm quite sure it's not possible. As you can see, the Language and Scenario filters are set on "All".
Search box just allow me filtering between available options shown above.

Interesting...
@apwestgarth do you know the answer to this?

Hi,
as this is a preview product only a limited number of triggers and templates per Functions runtime version were validated for this release.  HTTP Triggers are not available in Azure Functions Runtime preview 2 and were not available in Azure Functions Runtime preview 1 either.  You cannot install additional triggers and templates.
Thanks
Andrew

Thanks @apwestgarth! Glad to hear we were not just missing something obvious :)
@apercova I will close this out now that we know they are simply not available at this point in time. Let me know if you have any additional questions I can address.

@apwestgarth, @MicahMcKittrick-MSFT thanks for your help.
I don't have any additional questions regarding this.

I just installed it too and was wondering where it was.  Thanks for the clarification ...


Add subscribe button

Summary
Frontend & Database Update to Allow User Subscription to "DormScoop"
Questions

I am not sure whether the API should be set up that way. I had to allow a GET request to sustain user subscription status so that when a subscribed user refreshes the page they will see "Unsubscribe"
I did that using a useEffect. However that has lagging, so I used a slightly hacky method to hide the button until things are loaded, and have some empty padding to keep spacing consistent
I refactored the loading.gif out to display a random tip with it. I couldn't figure out how to sync the randomness so I went with a deterministic tip for now.

For random loading tips, have you checked https://nextjs.org/docs/messages/react-hydration-error?
We can probably disable SSR on <Loading/> or use useEffect to apply the randomness.
For getting the subscription status, the current hacky way looks good to me. Have you considered setting some placeholder text on the button (e.g. "loading subscription status") instead of hiding it?
Everything else lgtm.


x86_64 and armeabi-v7a prebuilt support

Hi @Lzhiyong
Thanks for this project - trying to compile for armv7 locally at the moment, but have stumbled upon issues with Ninja getting into a CMake configuration loop (the cache remains dirty, even after 100 tries).
Would you mind adding x86_64 & armeabi-v7a prebuilts as well to the latest release?

I have uploaded.

Hi @Lzhiyong
With arm & x86_64 (didn't test aarch64) fastboot binaries, I see the following output:
./fastboot devices
> 884861c3	�����N�
./fastboot devices
> 884861c3	�����
./fastboot devices
> 884861c3	����x�y�

And cannot decode as utf-8 as is.
Did you also perhaps observe this? (the text in question should be fastboot here)

I have recompiled and uploaded but no tested.


[Feature Request] Add support for termux git for Android

As isomorphic-git is very unstable, I suggest there being an option to use termux with git installed, instead of it.

There are already a bunch of issues on the subject. As I understood it, there is no way for obsidian-git to get access to Termux. The most promising approach seems to be switching to wasm-git, but there appear to remain a few roadblocks. See this issue https://github.com/Vinzent03/obsidian-git/issues/572. There wasn't much activity on it recently but as I got interested in the issue I'll probably get a shot at it soon

There are already a bunch of issues on the subject. As I understood it, there is no way for obsidian-git to get access to Termux. The most promising approach seems to be switching to wasm-git, but there appear to remain a few roadblocks. See this issue https://github.com/Vinzent03/obsidian-git/issues/572. There wasn't much activity on it recently but as I got interested in the issue I'll probably get a shot at it soon

I actually researched it a bit and there does seem to be a way for 3rd party apps to interface with Termux. The only issue I see, is that you'd always have to have a Termux session open, that Obsidian can communicate to. Technically you can fix this by making a daemon that starts on boot (using Termux) which automatically runs the commands it is sent through Obsidian. While these two options would take quite a bit of setting up, I think quite a few users (including me) would rather set it up once and have it function, than keep facing issues with isomorphic-git. Also if this was just an option the users could weigh the pros and cons themselves.

This involves sending intents and needs extra permissions in the manifest of Obsidian, which is not possible for a plugin. I'm not aware of a way to access native features on mobile.

This involves sending intents and needs extra permissions in the manifest of Obsidian, which is not possible for a plugin. I'm not aware of a way to access native features on mobile.

Ah ok. I guess I'll wait for the WASM-git implementation.


Please support haddock-library 1.8.x and Cabal 3.x

Nixpkgs recently switched to LTS 15.x, which now comes with ghc-8.8.x by default. I'm sure we can jump through some hoops to keep stylish-cabal building, but it my favorite solution would be if there was an update of the package that supports recent versions of its dependencies.

Hmm, after actually trying it, it turns out we can't jump through hoops, because ghc-8.8.x cannot compile Cabal-2.4.x. So we really do need an update.


Fix elevator block name

Changes elevator messages based on the configured elevator block material.
Before change: all messages refer to the elevator block as "gold block," regardless of what it is configured to be.
After change: all messages refer to the correct elevator block material

Duplicate of #17

Oh, oops

Thank you for contributing regardless, anything is always appreciated


bug: setfocus with viewChild

Bug Report
Ionic version:

[x] 5.x
Current behavior:

It does not have setFocus method when you access ion-input element as ionInput;
  @ViewChild('testInput', {read: IonInput}) testInput: IonInput;

 ngAfterViewInit(): void {
   console.log((this.testInput as IonInput))
   console.log((this.testInput as IonInput).setFocus())
 }

output:
IonInput {z: {…}, el: {…}, ionInput: {…}, ionChange: {…}…}
el: r
ionBlur: Observable
ionChange: Observable
ionFocus: Observable
ionInput: Observable
z: NgZone

Expected behavior:

It should have setFocus() to invoke focus programmatically.
Steps to reproduce:

Related code:
https://stackblitz.com/edit/ionic-angular-v5-y2wscm?file=src%2Fapp%2Fapp.component.html

insert short code snippets here

Other information:

Ionic info:

5

Thanks for the issue. I can only reproduce this when using StackBlitz (a standalone app does not exhibit this behavior) so this is likely a StackBlitz bug. We will look into bringing this up with the StackBlitz team. Thanks!


How to do Response.BinaryWrite in ASP.NET 5, Thank you.

Is something like this will work for you?
https://github.com/aspnet/KestrelHttpServer/blob/dev/samples/LargeResponseApp/Startup.cs

not sure it will work, I am talking about content in zip, pdf, jpeg format. from dynamically generated binary content byte[].

WriteAsync takes a byte[].

does it? how come vs2015 only show string as the first parameter? I will double check, sorry about this issue.

Response.Body is a Stream, Streams have Write(byte[], offset, length)

Body is not like a action, i did not even look at it. so if i gave stream to Body, i do not need to call any function? it will automatically write?
Thank you,
Gary

I'm not sure what you mean.
context.Response.Body.Write(buffer, 0, buffer.Length);

lol. sorry i am not in front of computer, i am in bed. i thought it is something like Body = steam. it is all good after seeing ur code. Are u guys in Redmond? it is late night now. ty for the hardwork.
Good night.
Gary

Sleep when you're dead :smile:


BugFix: truncate to earliest Offset during DST overlap

Background:
Instants that are before and after a dst overlap got truncated differently.
Basically instants that where during the overlap period got truncated to the earlier offset, and those
that were after, to a later offeset.
This will prevent this, and will only truncate to the earliest possible offset.

Looking really good. Nothing to add!


Label as Quilt supported

This can be used under Quilt so please mark this mod as Quilt supported in Modrinth.

Done. I' marked latest builds for Fabric as Quilt-compatible, all following ones will be marked automatically.


Service Already Present

OS: Windows 10
Version of IPFS Desktop 2.12.4

Describe the bug
By mistake I deleted Pinata from my Pinning Services in IPFS and now I can't add it again. When I try adding Pinata I get a message "Service Already Present" (see attached file). I have filed a support question, but nothing has happened..
To Reproduce
Steps to reproduce the behavior:

Go to Settings in IPFS.
Click on Add Service
Select Pinata.
Enter API endpoint and Secret Access Token (the JWT-code)
Message Service Already Present is shown.
Pinata is not added as Pinning Service-

Expected behavior
I expect to be able to see Pinata as one of the Pinning Services.
Screenshots

Additional context
Add any other context about the problem here.

I don't know what the issue is, but try giving it the nickname Pinata1 or something like that

This is duplicate of https://github.com/ipfs/ipfs-webui/issues/1816, let's continue there.


Add support for scheduled jobs

Trello: https://trello.com/c/ZaOJUZZB/233-5-automate-data-collection-on-a-daily-basis

I'm going to go via a Jenkins approach, rather than using cron/whenever as it seems that its difficult to get the jobs to run once, and the deployment details leak in to the app (e.g. govuk_setenv in config/schedule.rb).


[Enhancement]: Use single median frame for all segments

Instrument
NIRSpec (Stages 1-3), NIRCam (Stages 1-3), MIRI (Stages 1-3)
What is your suggestion?
When processing Stage 3 data in batches, a median frame is computed for each batch.  This sometimes leads to flux offsets in the light curves between batches.  By default, the code should only use the median frame from the first batch.  Users should provide an optional argument if they want a median frame computed for each batch.
Error traceback output
No response
What operating system are you using?
No response
What version of Python are you running?
No response
What Python packages do you have installed?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@kevin218 I'm adding this right now. Any preferences on what we call the optional ECF parameter that overrides the new default behaviour and computes a seperate median frame for each batch?

@kevin218 I'm adding this right now. Any preferences on what we call the optional ECF parameter that overrides the new default behaviour and computes a seperate median frame for each batch?

Nothing comes to mind at the moment. Feel free to suggest something here.

What about meta.independent_batches or something like that, set to False by default?


Link fix related to 14108

Changing accidental contentRootPath link to Webroot link on Generic Host doc.
This fix should have been in https://github.com/aspnet/AspNetCore.Docs/pull/14108#discussion-diff-321499728R267     Discovered just as it was merged however so fixing here.
Fixes issue https://github.com/aspnet/AspNetCore.Docs/issues/10559

Thanks Tom!


moose should not loose data on restart in dev

A long description of a data loss scenario I ran into:
Last night, I used the latest build to construct a test scenario for work on this migration PR.  I ran into an unexpected situation and suspected I was missing something.
Here's what I did.

I pushed records into my local Clickhouse DB into UserActivity_0_0, which worked as expected.
I then modified the data model for UserActivity by adding a new field, and I updated the package.json version and project.toml version with the new git commit ID.

All good there.

I restarted the moose docker-compose by shutting down the stack and restarting with moose dev
I then inserted new records into UserActivity_0_1, and that worked as expected.

I wasn't expecting that my records from UserActivity_0_0 would be missing and migrated to UserActivity_0_1.  I understand there's a trigger in place, but I thought that was a copy and not a move operation.

After running a test I determined that the data loss occurs during a simpler path:

Create a new moose project
Push records to the newly created UserActivity_0_0 clickhouse table by using the senddata.sh script or curl if you like.  I sent 10 records in my test.
Now simply edit the app/datamodels/models.prisma file and update the UserActivity data model.
At this point the file watcher detects a change to the model and drops the UserActivity_0_0 table causing the data loss of my 10 records.


Appropriate 'Illegal Type Annotation' Error?

 Issue
I do not know enough to say whether the following is a reasonable limitation or not. If it is a reasonable limitation I would argue both cases should error.

 Environment data

Language Server version: 2023.3.25
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda): 3.11.2
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet
"""Doc string."""
from typing import TypedDict

class Good(TypedDict):
    """Doc string."""
    test: dict
    dict: dict

class Bad(TypedDict):
    """Doc string."""
    dict: dict
    test: dict

a: Bad = {'dict': {}, 'test': {}}
b: Good = {'dict': {}, 'test': {}}
print(a, b)

 Expected behavior
The definition of 'Bad' should not cause an error if 'Good' does not.
 Logs
```
[Info  - 10:39:05] (364157) Pylance language server 2023.3.25 (pyright 68a8e98a) starting
[Info  - 10:39:05] (364157) Server root directory: /home/shapedsundew9/.vscode/extensions/ms-python.vscode-pylance-2023.3.25/dist
[Info  - 10:39:05] (364157) Starting service instance "scratch"
[Info  - 10:39:05] (364157) Auto-indent enabled
[Info  - 10:39:05] (364157) No configuration file found.
[Info  - 10:39:05] (364157) No pyproject.toml file found.
[Info  - 10:39:05] (364157) Setting pythonPath for service "scratch": "/home/shapedsundew9/Projects/venv/bin/python3"
[Warn  - 10:39:05] (364157) stubPath /home/shapedsundew9/Projects/scratch/typings is not a valid directory.
[Info  - 10:39:05] (364157) Assuming Python version 3.11
[Info  - 10:39:05] (364157) Assuming Python platform Linux
[Info  - 10:39:05] (364157) Searching for source files
[Info  - 10:39:05] (364157) Found 1 source file
[Info  - 10:39:06] (364157) Background analysis(1) root directory: /home/shapedsundew9/.vscode/extensions/ms-python.vscode-pylance-2023.3.25/dist
[Info  - 10:39:06] (364157) Background analysis(1) started
[Info  - 10:39:06] (364157) Indexer background runner(2) root directory: /home/shapedsundew9/.vscode/extensions/ms-python.vscode-pylance-2023.3.25/dist (index)
[Info  - 10:39:06] (364157) Indexing(2) started
[Info  - 10:39:07] (364157) scanned(2) 174 files over 1 exec env
[Info  - 10:39:11] (364157) [IDX(2)] Long operation: index execution environment /home/shapedsundew9/Projects/scratch (3696ms)
[Info  - 10:39:11] (364157) [IDX(2)] Long operation: index packages /home/shapedsundew9/Projects/scratch (3703ms)
[Info  - 10:39:11] (364157) indexed(2) 123 files over 1 exec env
[Info  - 10:39:11] (364157) Indexing finished(2).
```

This behavior is as designed. The evaluation of type annotations in pyright (the static type checker that underlies pylance) attempts to mirror the evaluation at runtime.
The evaluation of type annotations at runtime depends on a number of factors:

Is the type annotation expression enclosed in quotes? If so, its evaluation is deferred.
Do you have from __future__ import annotations at the top of the source file? If so, all type annotations are assumed to be deferred, as though they were all enclosed in quotes.
Are you using Python 3.12 (prerelease)? In Python 3.12, the from __future__ import annotations behavior will likely become default, so pyright assumes its presence if you manually set the version to Python 3.12.

If type annotation evaluation is not deferred, then it is evaluated in code flow order. In your example, the first dict annotation is evaluated prior to defining the local symbol dict, so it evaluates to builtins.dict which is a legal type annotation. The second dict annotation evaluates to the local variable dict, which is not a legal type annotation.
In short, the error you're seeing is not a bug. It's by design.

Thank you for the explanation. I thought I was going to improve my 'wheat to chaff' ratio on issues raised with this one but alas...though I feel my mental model has been incrementally improved. Thx.


Added @labeshsharma as a contributor

Added my name as a contributor

@all-contributors add @labheshwar for content.


Do not create an output folder during test run

Coverage remained the same when pulling 471c61a2dbe8251e484082d37e2ff0c7798097a8 on test-do-not-create-output into 6ec5e32ded23f52d351f7d5f5498d67c3a1cd008 on master.


Remove coloredlogs from the interface

rail_delight uses the coloredlogs package to make its logs more readable. Unfortunately there's a bug in coloredlogs that means that this causes the global log level to be set to DEBUG. This is especially annoying for matplotlib which can the start listing all the fonts it finds on your system.
This removes the call to coloredlogs.

The errors here are actually with the base Delight module.  This parallel PR should fix them if merged:
https://github.com/LSSTDESC/Delight/pull/11


Config extraction & documentation

Another sub-crate with config dumping functionality:

Helper to debug-dump the config to logs.
A --dump-config option, that would parse all configuration, dump it (in toml, probably?) and exit.
(tricky) --describe-config that examines the structure of the Cfg based on its Deserialize and documents it somehow. If not possible, adding a new trait might be needed.

Released just few hours ago


Using express in meteor deployments, is it secure?

Hi, for Meteor applications you are using a standalone express server that is ruinning on port 4000. How secure is this to deploy to the live app? What if the app crashes? I have several deployments on Galaxy, thinking of using GraphQl, but that express server scares me a bit. Will it be detected by Meteor app and crash as well? This way Galaxy would start a new instance with no issues. Thanks for any advice.

I guess, my question is, can apollo server run in-proc with meteor appllication in order to assure the stability of the application? My naive way of doing that was:
WebApp.connectHandlers.use("/graphql", apolloServer(async () => {
    return {
      graphiql: true,
      pretty: true,
      schema,
      resolvers
    };
}));

Which of course does not work ... but something in this ballpark would be great!

Asking the Galaxy team about it!

@stubailo did you have any response from the galaxy team? I think I got enough knowledge to start bringing this into production. In combination with redux it's pure awesomeness. But the stability is priority num. 1 for now. Thanks.

Oh yeah - I checked up with then and only one port is externally accessible. So there shouldn't be any security concerns with running a different port as an implementation detail.

Great, thanks.


post-processing

Dear author, how can I understand the post-processing part in the decode_preds function？
# pose-processing
for n in range(coords.size(0)):
    for p in range(coords.size(1)):
        hm = output[n][p]
        px = int(math.floor(coords[n][p][0]))
        py = int(math.floor(coords[n][p][1]))
        if (px > 1) and (px < res[0]) and (py > 1) and (py < res[1]):
            diff = torch.Tensor([hm[py - 1][px] - hm[py - 1][px - 2], hm[py][px - 1]-hm[py - 2][px - 1]])
            coords[n][p] += diff.sign() * .25
coords += 0.5
preds = coords.clone()

@zhao181 Hi, I can try to explain it, but  I' m not the author, so maybe I'll make some mistakes in some
details.
As you know, our input image is 256x256 pixel, while the sizes of output heatmaps are 64x64, which is only a quarter of the original image. Then it comes to a problem when you transform your output to your predicted coordinates.
Let's take an example, the ground-truth coordinate of a certain landmark is (242,242), then we transform it into a heatmap(64x64), we just divide it by 4, so it turns to (60.5,60.5). However, the index of an image variable must be a int , so we just take it as (60,60). Suppose our network can predict heatmaps very accurately, then the heatmap will be a gaussian heatmap with its center on (60,60), then if you just multiply it by 4, you can only get (240,240), which will have 2 pixels error on each direction.
How can we solve this problem? First, it takes some tricks when generating heatmap ground-truth, you can look into the code in lib/dataset/* and lib/util/transform.py. Second, you can make the predicted heatmap output some float number. In your above code, hm is a predicted heatmap of one landmark of one sample, "coords" is the sorted index of the heatmap, so you can see px and py is just the output we need( if you don't care about the error I said). Next, the author checks if the px and py are not on the boundaries of the heatmap( in case it throws errors in the following operation).  Then the author checks the symmetry of the heatmap. For example, if the left side hm[py - 1][px-2] is larger than the right side hm[py - 1][px], then maybe we should move the center to left a little ( the reason why we take the py-1-th row is in py-th row the lefe side is surely smaller than the right side, and it has no help to the finetune of coords),  otherwise, we can see that now the px is a pretty good center(because no matter we choose which row, the px-th pixel is still larger), then we move to right a little, why? because we transform (240/241/242）into 64, you can see in most cases (i.e. 241/242) we better turn 64 into 64.25, even though 64 is exactly the ground-truth.
As for the line "coords += 0.5", I  believe it's related to the ground-truth generating processing but  i'm not sure. @wondervictor Would you please to explain it?

do you know it now?

No I don't know

2020.4.23. do you understand now.......

Anyone can help?

It is a common practice to compensate for the quantization effect. Since the landmark is encoded in a heatmap of lower resolution (generally 1/4th) than the original image, when recovering the coordinates after inference, a sub-pixel shift towards the next highest pixel value is used.
From Stacked Hourglass Networks for Human Pose Estimation, pg 8:

To improve performance at high precision thresholds the prediction is offset by a quarter of a pixel in the direction of its next highest neighbor before transforming back to the original coordinate space of the image.

Some details are also provided in Distribution-Aware Coordinate Representation for Human Pose Estimation in Coordinate Decoding section.
Hope this helps!

@kuldeepbrd1  really thanks for your help.

It is a common practice to compensate for the quantization effect. Since the landmark is encoded in a heatmap of lower resolution (generally 1/4th) than the original image, when recovering the coordinates after inference, a sub-pixel shift towards the next highest pixel value is used.
From Stacked Hourglass Networks for Human Pose Estimation, pg 8:

To improve performance at high precision thresholds the prediction is offset by a quarter of a pixel in the direction of its next highest neighbor before transforming back to the original coordinate space of the image.

Some details are also provided in Distribution-Aware Coordinate Representation for Human Pose Estimation in Coordinate Decoding section.
Hope this helps!

Thanks a lot

It is a common practice to compensate for the quantization effect. Since the landmark is encoded in a heatmap of lower resolution (generally 1/4th) than the original image, when recovering the coordinates after inference, a sub-pixel shift towards the next highest pixel value is used.
From Stacked Hourglass Networks for Human Pose Estimation, pg 8:

To improve performance at high precision thresholds the prediction is offset by a quarter of a pixel in the direction of its next highest neighbor before transforming back to the original coordinate space of the image.

Some details are also provided in Distribution-Aware Coordinate Representation for Human Pose Estimation in Coordinate Decoding section.
Hope this helps!

Hi，thanks for the reference. Why do we need to shift 0.5 pixel in here ?

any directio

In my own understanding, the +0.5 is for the interpolation to decrease the final scaled  error. The 1/4 offeset is considering the heatmap , but do you thnk the +0.5 and 0.25 are dpendent on experience, that is something hyper-parameters？

any directio

@kuldeepbrd1  In my own understanding, the +0.5 is for the interpolation to decrease the final scaled  error. The 1/4 offeset is considering the heatmap , but do you thnk the +0.5 and 0.25 are dpendent on experience, that is something hyper-parameters？

I have a very interesting result here with the same so-called post-processing on my own dataset. In the training and evaluation process, the `NME' first drop and then increase at the very begining 5th epoch. However, the landmark heatmap loss always decrease. I couldn't figure it out. Why? Anyone have any idea?


make Tensor zero inlinable

This is causing a "cannot partition generic function" error when compiling the TensorFlow lib in Piper. It's surprising to me that it's not also causing this error in the cmake build, but I haven't investigated why.

@swift-ci please test tensorflow


Minor changes in the documentation (customization page)

Thank you. I will merge it in v1.5.8 (Hopefully the last minor release in v2)


python310Packages.pyunifiprotect: 4.8.3 -> 4.9.0

Diff: https://github.com/briis/pyunifiprotect/compare/refs/tags/v4.8.3...v4.9.0
Changelog: https://github.com/AngellusMortis/pyunifiprotect/releases/tag/v4.9.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

home-assistant-component-tests.unifiprotect
python310Packages.pyunifiprotect
python311Packages.pyunifiprotect
unifi-protect-backup


VSCode searching for exception information in the wrong path for pyzmq

System Information
Version: 1.41.0 (system setup)
Commit: 9579eda04fdb3a9bba2750f15193e5fafe16b959
Date: 2019-12-11T18:37:42.077Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18362
Steps to Reproduce:

Create new folder and create new python virtual environment venv.
activate venv
pip install pyzmq  (Using version 18.1.1)
Copy paste the following code in a file and run

# echoClient.py
import sys
import time
import zmq

if __name__ == "__main__":
    if len(sys.argv) != 2:
        sys.stderr.write("Add port as argument")
        sys.exit(1)

    context = zmq.Context()
    socket = context.socket(zmq.REQ)
    socket.connect("tcp://localhost:%d" % int(sys.argv[1]))
    socket.setsockopt(zmq.LINGER, 0)

    print("Sending ...")
    times = 0
    while True:
        times += 1
        msg = "Message".encode('utf-8')
        send_OK = socket.poll(1, zmq.POLLOUT)
        socket.send(msg)   # <-- Will crash here if Server is not connected
        #  Get the reply.

        if socket.poll(10, zmq.POLLIN):
            message_recd = socket.recv()
            print("Received %s" % message_recd)

            if msg == message_recd:
                print("ECHO OK")

            else:
                print("ECHO NOT OK")

        else:
            print("Timeout")

        time.sleep(1)

    # close socket and exit
    socket.close()
    context.term()

This code will crash if a listener is not connected because of the internal state machine mechanisms of a zmq REQ socket, that is you cannot send() twice without recv() after every send().
The issue is that VSCode seems to look for the exception information in .\zmq\backend\cython\checkrc.pxd when it should be looking in the site-packages folder. The correct path is .\venv\Lib\site-packages\zmq\backend\cython\checkrc.pxd
Error message - Unable to open 'checkrc.pxd': Unable to read file (Error: File not found (c:\pyprojects\echoclient\zmq\backend\cython\checkrc.pxd)).

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

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

Only the python extension is enabled. Rest were disabled during test.

VS Code has no built-in Python functionality. Please file this issue against the Python extension.

I see. Thanks.


when set rollupTypes: true, the import is redundant

Describe the bug

these import maybe come from different files, so how to merge them
Reproduction

Steps to reproduce
No response
System Info
"version": "4.0.3",

Validations

[X] Read the FAQ.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is a behavior of @microsoft/api-extractor, also it's unnecessary to merge these imports and it will work normally.