🐛 [firebase_messaging: ^10.0.4] iOS app icon notification badge always shown

Bug report
Describe the bug
A clear and concise description of what the bug is.
When the iOS App(iOS 14.5) receives a notification and the user clicks the notification to start the App, the App icon badge will not be cleared, even after removing all of the notifications from the notification center.
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 2.0.3, on macOS 11.0.1 20B50 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.59.0)
[✓] Connected device (2 available)

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

dependencies:
- auto_route 1.0.2 [flutter path collection]
- auto_size_text_pk 3.0.0 [flutter]
- cached_network_image 3.0.0 [flutter flutter_cache_manager octo_image]
- country_code_picker 2.0.1 [flutter modal_bottom_sheet collection universal_platform]
- country_icons 2.0.2 [flutter]
- country_pickers 2.0.0 [flutter]
- credit_card_input_form 2.3.0 [flutter provider flip_card]
- crisp 0.1.7 [flutter url_launcher flutter_inappwebview]
- crop 0.5.0 [flutter vector_math]
- cupertino_icons 1.0.3
- dio 4.0.0 [http_parser path]
- email_validator 2.0.1
- firebase_auth 3.0.1 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.4.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 2.0.7 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- firebase_messaging 10.0.4 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_storage 10.0.1 [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_cache_manager 3.0.2 [clock collection file flutter http image path path_provider pedantic rxdart sqflite uuid]
- flutter_dotenv 4.0.0-nullsafety.0 [flutter]
- flutter_emoji_suite 1.0.0 [flutter]
- flutter_local_notifications 5.0.0+4 [flutter flutter_local_notifications_platform_interface platform timezone]
- flutter_riverpod 0.13.0 [collection flutter meta riverpod state_notifier]
- flutter_segment 3.1.2 [flutter flutter_web_plugins meta]
- flutter_slidable 0.6.0 [flutter]
- flutter_sticky_header 0.6.0 [flutter value_layout_builder]
- flutter_typeahead 3.1.1 [flutter flutter_keyboard_visibility]
- flutter_webview_plugin 0.4.0 [flutter]
- fluttertoast 8.0.7 [flutter flutter_web_plugins]
- freezed_annotation 0.14.1 [collection json_annotation meta]
- google_sign_in 5.0.1 [google_sign_in_platform_interface google_sign_in_web flutter meta]
- hasura_connect 3.0.0+1 [http async either_dart meta string_validator uuid dart_websocket collection]
- image 3.0.2 [archive meta xml]
- image_picker 0.7.4 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface image_picker_for_web]
- intl 0.17.0 [clock path]
- json_serializable 4.1.0 [analyzer build build_config collection json_annotation meta path source_gen]
- linked_scroll_controller 0.2.0 [flutter]
- mime_type 1.0.0
- multi_select_flutter 4.0.0 [flutter collection]
- numberpicker 2.1.1 [flutter infinite_listview]
- overlay_support 1.2.1 [flutter async]
- pagination_view 2.0.0-nullsafety.0 [flutter bloc flutter_bloc]
- path 1.8.0
- plugin_platform_interface 2.0.0 [meta]
- rxdart 0.27.0
- shared_preferences 2.0.5 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- shimmer 2.0.0 [flutter]
- stream_chat_flutter 2.0.0-nullsafety.4 [cached_network_image characters chewie collection dio ezanimation file_picker flutter flutter_keyboard_visibility flutter_markdown flutter_portal flutter_slidable flutter_svg http_parser image_gallery_saver image_picker jiffy lottie meta path_provider photo_manager photo_view rxdart scrollable_positioned_list share_plus shimmer stream_chat_flutter_core substring_highlight synchronized url_launcher video_compress video_player video_thumbnail visibility_detector]
- stripe_sdk 5.0.0-nullsafety.0 [flutter http url_launcher mask_text_input_formatter flutter_slidable simple_animations supercharged awesome_card uni_links2 credit_card_validator collection]
- syncfusion_flutter_calendar 19.2.51 [flutter timezone syncfusion_flutter_core syncfusion_flutter_datepicker intl]
- timeago 3.0.2
- universal_platform 1.0.0-nullsafety
- url_launcher 6.0.3 [flutter url_launcher_platform_interface url_launcher_linux url_launcher_macos url_launcher_windows url_launcher_web]
- url_launcher_web 2.0.0 [url_launcher_platform_interface meta flutter flutter_web_plugins]
- uuid 3.0.4 [crypto]
- webview_flutter 2.0.10 [flutter]

dev dependencies:
- auto_route_generator 1.0.2 [build source_gen analyzer path build_runner code_builder dart_style auto_route]
- build_runner 1.12.2 [args async build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style glob graphs http_multi_server io js logging meta mime path pedantic pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_launcher_icons 0.9.0 [args image path yaml]
- 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]
- freezed 0.14.1+1 [analyzer build build_config meta source_gen freezed_annotation]
- lint 1.5.3
- pedantic 1.11.0

transitive dependencies:
- _fe_analyzer_shared 19.0.0 [meta]
- analyzer 1.3.0 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml pedantic]
- archive 3.1.2 [crypto path]
- args 2.0.0
- async 2.5.0 [collection]
- awesome_card 1.1.5 [flutter]
- bloc 7.0.0 [meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.0.0 [analyzer async convert crypto glob logging meta path]
- build_config 0.4.7 [checked_yaml json_annotation meta path pubspec_parse yaml]
- build_daemon 2.1.10 [built_collection built_value http_multi_server logging pedantic path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.0.0 [analyzer build crypto graphs logging path package_config pool pub_semver stream_transform]
- build_runner_core 6.1.12 [async build build_config build_resolvers collection convert crypto glob graphs logging meta path package_config pedantic pool timing watcher yaml]
- built_collection 5.0.0
- built_value 8.0.4 [built_collection collection fixnum]
- characters 1.1.0
- charcode 1.2.0
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- chewie 1.0.0 [cupertino_icons flutter video_player wakelock]
- cli_util 0.3.0 [meta path]
- clock 1.1.0
- code_builder 3.7.0 [built_collection built_value collection matcher meta]
- collection 1.15.0
- convert 3.0.0 [typed_data]
- credit_card_type_detector 2.0.0-nullsafety.0
- credit_card_validator 2.0.0-nullsafety.0 [credit_card_type_detector]
- crypto 3.0.0 [collection typed_data]
- dart_style 2.0.0 [analyzer args path pub_semver source_span]
- dart_websocket 1.0.0
- either_dart 0.1.0-nullsafety.1
- equatable 2.0.2 [collection meta]
- ezanimation 0.5.0 [flutter]
- fake_async 1.2.0 [clock collection]
- ffi 1.0.0
- file 6.1.0 [meta path]
- file_picker 3.0.3 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface]
- firebase_auth_platform_interface 6.0.0 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.0.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.0.1 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.1.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_crashlytics_platform_interface 3.0.6 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_platform_interface 3.0.3 [firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 2.0.3 [firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- firebase_storage_platform_interface 4.0.0 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.0.0 [async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- fixnum 1.0.0
- flip_card 0.5.0 [flutter]
- flutter_bloc 7.0.0 [flutter bloc provider]
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_inappwebview 5.3.2 [flutter]
- flutter_keyboard_visibility 5.0.2 [meta flutter_keyboard_visibility_platform_interface flutter_keyboard_visibility_web flutter]
- flutter_keyboard_visibility_platform_interface 2.0.0 [flutter meta plugin_platform_interface]
- flutter_keyboard_visibility_web 2.0.0 [flutter_keyboard_visibility_platform_interface flutter_web_plugins flutter]
- flutter_local_notifications_platform_interface 3.0.0 [flutter plugin_platform_interface]
- flutter_markdown 0.6.2 [flutter markdown meta path]
- flutter_plugin_android_lifecycle 2.0.1 [flutter]
- flutter_portal 0.4.0 [flutter]
- flutter_svg 0.22.0 [flutter meta path_drawing vector_math xml]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- glob 2.0.1 [async collection file path pedantic string_scanner]
- google_sign_in_platform_interface 2.0.1 [flutter meta quiver]
- google_sign_in_web 0.10.0 [google_sign_in_platform_interface flutter flutter_web_plugins meta js]
- graphs 1.0.0
- http 0.13.1 [http_parser meta path pedantic]
- http_multi_server 2.2.0 [async]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- image_gallery_saver 1.6.9 [flutter]
- image_picker_for_web 2.0.0 [image_picker_platform_interface meta flutter flutter_web_plugins]
- image_picker_platform_interface 2.0.1 [flutter meta http plugin_platform_interface]
- infinite_listview 1.1.0 [flutter]
- io 0.3.5 [meta path string_scanner]
- jiffy 4.1.0 [intl]
- js 0.6.3
- json_annotation 4.0.1
- logging 1.0.1
- lottie 1.0.1 [archive characters charcode collection flutter logging meta path vector_math]
- markdown 4.0.0 [args charcode meta]
- mask_text_input_formatter 2.0.0-nullsafety.2 [flutter]
- matcher 0.12.10 [stack_trace]
- meta 1.3.0
- mime 1.0.0
- modal_bottom_sheet 2.0.0 [flutter]
- nested 1.0.0 [flutter]
- octo_image 1.0.0+1 [flutter flutter_blurhash]
- package_config 2.0.0 [path]
- path_drawing 0.5.1 [vector_math meta path_parsing flutter]
- path_parsing 0.2.1 [vector_math meta]
- path_provider 2.0.1 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 2.0.0 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 2.0.0 [flutter]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.0 [path_provider_platform_interface meta path flutter ffi win32]
- petitparser 4.0.2 [meta]
- photo_manager 1.2.1 [flutter]
- photo_view 0.11.1 [flutter]
- platform 3.0.0
- pool 1.5.0 [async stack_trace]
- process 4.2.1 [file path platform]
- provider 5.0.0 [collection flutter nested]
- pub_semver 2.0.0 [collection]
- pubspec_parse 1.0.0 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.0.1 [matcher]
- riverpod 0.13.1 [collection freezed_annotation meta state_notifier]
- scrollable_positioned_list 0.2.0-nullsafety.0 [flutter collection]
- share_plus 2.1.2 [meta mime flutter share_plus_platform_interface share_plus_linux share_plus_macos share_plus_windows share_plus_web]
- share_plus_linux 2.0.3 [share_plus_platform_interface file flutter meta url_launcher]
- share_plus_macos 2.0.2 [share_plus_platform_interface flutter]
- share_plus_platform_interface 2.0.1 [flutter meta mime plugin_platform_interface]
- share_plus_web 2.0.4 [share_plus_platform_interface url_launcher flutter flutter_web_plugins meta]
- share_plus_windows 2.0.3 [share_plus_platform_interface flutter meta url_launcher]
- shared_preferences_linux 2.0.0 [flutter file meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 2.0.0 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 2.0.0 [flutter]
- shared_preferences_web 2.0.0 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 2.0.0 [shared_preferences_platform_interface flutter file meta path path_provider_platform_interface path_provider_windows]
- shelf 1.1.0 [async collection http_parser path stack_trace stream_channel]
- shelf_web_socket 0.2.4+1 [shelf stream_channel web_socket_channel]
- simple_animations 3.1.1 [flutter supercharged pedantic]
- sky_engine 0.0.99
- source_gen 1.0.0 [analyzer async build dart_style glob meta path pedantic source_span]
- source_span 1.8.0 [charcode collection path term_glyph]
- sqflite 2.0.0+3 [flutter sqflite_common path]
- sqflite_common 2.0.0+2 [synchronized path meta]
- stack_trace 1.10.0 [path]
- state_notifier 0.7.0 [meta]
- stream_channel 2.1.0 [async]
- stream_chat 2.0.0-nullsafety.2 [async collection dio equatable freezed_annotation http_parser json_annotation logging meta mime rxdart uuid web_socket_channel]
- stream_chat_flutter_core 2.0.0-nullsafety.3 [collection flutter meta rxdart stream_chat]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- string_validator 0.2.0-nullsafety.0
- substring_highlight 1.0.33 [flutter]
- supercharged 2.0.0 [supercharged_dart flutter]
- supercharged_dart 2.0.0
- syncfusion_flutter_core 19.2.51 [vector_math flutter]
- syncfusion_flutter_datepicker 19.2.51 [flutter intl syncfusion_flutter_core]
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.2.19 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timezone 0.7.0 [path]
- timing 0.1.1+3 [json_annotation]
- typed_data 1.3.0 [collection]
- uni_links2 0.6.0+2 [uni_links2_platform_interface uni_links_web2 flutter]
- uni_links2_platform_interface 1.0.0+2 [plugin_platform_interface flutter]
- uni_links_web2 0.1.0+2 [uni_links2_platform_interface flutter flutter_web_plugins]
- url_launcher_linux 2.0.0 [flutter]
- url_launcher_macos 2.0.0 [flutter]
- url_launcher_platform_interface 2.0.2 [flutter plugin_platform_interface]
- url_launcher_windows 2.0.0 [flutter]
- value_layout_builder 0.3.1 [flutter]
- vector_math 2.1.0
- video_compress 3.0.0 [flutter]
- video_player 2.1.5 [flutter flutter_test meta video_player_platform_interface video_player_web]
- video_player_platform_interface 4.1.0 [flutter meta flutter_test]
- video_player_web 2.0.1 [flutter flutter_web_plugins meta video_player_platform_interface]
- video_thumbnail 0.3.3 [flutter]
- visibility_detector 0.2.0 [flutter]
- wakelock 0.4.0 [flutter meta wakelock_macos wakelock_platform_interface wakelock_web]
- wakelock_macos 0.1.0+1 [flutter flutter_web_plugins wakelock_platform_interface]
- wakelock_platform_interface 0.2.1+1 [flutter meta]
- wakelock_web 0.2.0+1 [flutter flutter_web_plugins js wakelock_platform_interface]
- watcher 1.0.0 [async path pedantic]
- web_socket_channel 2.1.0 [async crypto stream_channel]
- win32 2.0.5 [ffi]
- xdg_directories 0.2.0 [meta path process]
- xml 5.0.2 [collection meta petitparser]
- yaml 3.1.0 [collection source_span string_scanner]

Hi @maavimartinez
Please see https://github.com/FirebaseExtended/flutterfire/issues/4145#issuecomment-739571979
Thank you


Investigate XYZ export issues

Describe the bug
User exports Miranda data to XYZ format including a variable named "pressure". Output he gets is...
  1991232
visit export chunk 0
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000

The ? char is  what XYZ writes when it doesn't have an atomic element name. It would be better to say "unknown atomic element" or something but we'd need to take care because I think that string is consumed in the XYZ reader and we'd need to adjust the reader to deal with that new string.
The next 3 columns are supposed to b x, y and z coordinates (z is always zero for 2D outputs). The fact that they are all the same here suggests that ofstream precision and/or notation are preventing seeing all significant digits of the intput. I adjusted this in #3380.
The next column after the coordinates should be the user's pressure variable. It isn't there or perhaps it is printing but all blanks for some reason (again maybe precision/notation issue on the stream -- thats just a guess). Could be pilot error too.

Helpful additional information
(Please click check boxes AFTER submitting ticket)

[ ] Did VisIt crash?
[x] Did you get wrong results?

Impact

[ ] High - User productivity significantly degraded
[x] Medium - User productivity partially degraded
[ ] Low - User productivity trivially degraded

Likelihood

[ ] High - occurrence is not specific to platform, data, config, a combination of operators, queries, etc.
[ ] Medium - neither low nor high likelihood
[x] Low - occurrence is specific to combination of multiple factors such as configuration, data, platform, etc.

To Reproduce
I tried to reproduce this behavior in exporting something like wave.silo + pressure and was not able to. I suspect the problems the user is seeing is related to the data range of their input (Miranda) or maybe the GetTuple1 call being used to get values at nodal positions isn't returning the expected data type (double). That could happen, if maybe the input VTK objects read from the Miranda (or blueprint) plugin somehow produced coordinate data in format other tha float or double. Again, a guess.
Priority
This is for a very patient ASC user team whose been having trouble trying various aveneues to export data from VisIt for one of their customers. I think we need to try to fix asap.

According to the ticket description, it seems that the "definition of done" can be broken down into 3 parts:

Instead of printing a question mark, the writer should print "unknown atomic element". Update the writer and reader accordingly.
Confirm that PR #3380 fixes the precision/notation problem. If not, investigate and fix the issue.
Investigate why the pressure variable is not appearing and fix it.

@markcmiller86 Since we are unable to reproduce the issue, I think it would be best if I could access the user's data directly.

User is scullard1 and olson45. Please contact directly. From 04Apr19 email...
Ok, this is 2 million points and I suspect (some of) the spatial coordinate extents are bigger than fit in 6 digits, the default precision for C++ ofstream. Also, it looks like we're using default *notation* there too which isn't scientific or fixed but something else. Thats easily fixable.

The ? issue is ignorable, it is an artifact of the format originally designed for atomic nuclei positions

Why we're not also getting the variable (pressure) values is a mystery now. I tried a test and get them. They would appear in the 4-10th columns of the output.

Mark

User wrote:

For example, if I try to export the pressure variable output by Miranda, the file visit_ex_db.xyz looks like this:

  1991232
visit export chunk 0
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000
?       -100000 -100000 4000

It continues like this with some different coordinates here and there, but no actual pressures appear to have made it into the file. To generate this, I have clicked Export Database, selected XYZ, and selected the variable I want under the variable pull-down menu. Aside from that, it's not clear what else needed to be done.

User is scullard1 and olson45. Please contact directly. From 04Apr19 email...

Just emailed the users for their data. Thanks for the extra info!

The next column after the coordinates should be the user's pressure variable. It isn't there or perhaps it is printing but all blanks for some reason (again maybe precision/notation issue on the stream -- thats just a guess). Could be pilot error too.

The XYZ writer only looks for point data from the dataset. It does not look for cell data. I believe this is by design (since each variable is printed with the nodal coordinate), so if variables are not coming out then it may be because the user is attempting to export cell-centered data. I will know more when they respond to my email.

Well, so for point meshes, which is all XYZ plugin supports I think, cell-centered (or zone-centered) and point-centered (or node-centered) essentially mean the same thing.
However, in VTK, there is a potential impendance problem if the mesh consists only of points and no cells. This can happen if there were no calls to create the VTK_VERTEX cells. These are cells that consist of just a single point. In some ways, they are kinda sorta redundant. But, to be mathematically rigorous, they are indeed needed to fully define a point mesh. The trouble is, I don't think this rigor is followed everywhere in VisIt.
Encountering the need for cell-centered data for a mesh consisting only of points and no cells can probably present problems.
I think the right answer should be to ensure the mesh has the cells also. I dunno if that should be part of the XYZ exporter, the reader, or both.

But, to be mathematically rigorous, they are indeed needed to fully define a point mesh.

@markcmiller86 Can you elaborate on this? Why is it necessary for a point mesh to have an equivalent and redundant cell mesh?

Investigate why the pressure variable is not appearing and fix it.

XYZ writer looks for variables from the dataset's pointdata here. nvars is calculated in a loop over the point data here. So, if a variable is only cell-centered, the XYZ writer will not see it. This is what is happening in the reported use case here. The data is not point-centered.

Confirm that PR #3380 fixes the precision/notation problem. If not, investigate and fix the issue.

Not sure why this is still happening. XYZ writer is not the only one that has this. For example, I exported to vtk and saw the same thing.

Why is it necessary for a point mesh to have an equivalent and redundant cell mesh?

A simple example...its concievable that there are some points that are not intended to be part of the mesh. This can definiately happen in non-point meshes where, for example, you have a vtkPoints object containing some points that are not referenced by any cell.
Next, I dunno but I think VTK may require the redundent cells in order for the object to be a compliant unstructured grid object. I mean, it may be that a vtkUnstructuredGrid object that returns non-zero for GetNumberOfPoints() but zero for GetNumberOfCells() is considered an empty mesh.
Ultimately, a mesh is collection of entities (knitted together) over which things like field evaluation, interpolation, continuity, etc. are possible to characterize. These kinds of operations have meaning only over neighborhoods which are ultimately specified by the cells of the mesh. So, IMHO, from a purely theoretical perspective, I think it may make sense to decouple to characterization of topologically zero dimensional entities (the points as cells) from the list of tuples (the points as points). Though, other than the comments above, I am at a loss to suggest any practical operations as examples where that distinction might actually be relevant.

@markcmiller86 thanks for the explanation. Based on what you've said here, it seems that the XYZ plugin is not designed to handle meshes because the Writer never outputs any cell data. It only outputs point data.

Well, in XYZ writer, that is XYZ file format. XYZ file format is designed only for points and so the cells VTK might need can be inferred from them when the time comes to instantiate it as VTK (e.g during a read back).
Maybe this all boils down to possibility that user has both point data and cell data fields and when they export a cell-data field, if the cells that  field is defined on are not VTK_VERTEX cells, then the field needs to be re-centered to the nodes before exporting. Might be nice if the writer could just that automagically but maybe thats too high cost/benefit. Next best thing might be a warning from the writer indicating cell data was requested for a points-only format and so those variables need to be re-centered via expressions to export them. Thats all a guess on my part though.

Well, in XYZ writer, that is XYZ file format. XYZ file format is designed only for points and so the cells VTK might need can be inferred from them when the time comes to instantiate it as VTK (e.g during a read back).

I see what you're saying. So then the XYZ format does not natively distinguish between point data that you want as a mesh and point data that you want to not be connected to any mesh.

when they export a cell-data field, if the cells that field is defined on are not VTK_VERTEX cells, then the field needs to be re-centered to the nodes before exporting. Next best thing might be a warning from the writer indicating cell data was requested for a points-only format and so those variables need to be re-centered via expressions to export them.

I spoke with the users who were having this problem and they had found a work around using tec plot. The feature to automatically detect and convert would be nice, although I am not familiar enough with vtk yet to estimate how much work that would be. I propose that for this ticket, I create a warning for cell only data. I will create a separate feature ticket for the automatic conversion of cell to point data by re-centering. Thus, the feature ticket can be evaluated and prioritized as needed.

That sounds fine.

So, I just did this test...

Open globe.silo
PC plot dx
Export DB to XYZ

You get the nodal coordinates but no 4th column for dx

Define expression "dx_nodal" as "recenter(dx, "nodal")"
PC plot of dx_nodal
Export to XYZ

You get the 4th column.

So, a warning message about not being able to handle non-node centered variables along with a hint to recenter them to the nodes would be very useful.

If, however, the cell-data is all defined on VTK_VERTEX elements, then we should write those to the file witout issue.

So, I just did this test...

This is the same thing that I found to happen.

If, however, the cell-data is all defined on VTK_VERTEX elements, then we should write those to the file without issue.

Just to be clear, this is the case where there is no point data but there is cell data and all the cells are VTK_VERTEX, then they should essentially be treated the same as points as far as the Writer is concerned?

Well, if there is no point data, we have no coordinates. So, no not that case. Sorry if I sound confusing (I probably am confused ;). But, imagine a spatially 2D dataset arrives in the XYZ writer requesting both point and cell-centered variables to be written. In general, the collection of cells could consist of VTK_TRIANGLE and VTK_QUAD cells and we'd have to do a recentering there in the writer to fix that if we wanted to write the data to the XYZ format.
However, if the cells were all VTK_VERTEX, then they are effectively point-centered variables already and we can write those to XYZ format without any work.

Got it, thanks for the clarification!

I took a quick look at the expression system's recentering code. It might be pretty easy to do the following...

refactor that code to src/avt/Pipeline/Data/avtCommonDataFunctions.C
change expression system to use it from its new place
Add logic in XYZ plugin to use it there to recenter any cell-centered variables during export by fiat
Issue warning (not error message) when variables are recentered during export there

@visit-dav how can I add a test for how something is written? The existing tests in /tests/databases seem to be testing how something is read, but I want to test how something gets written, and also test if a warning message gets issued. Is this possible?

Well, your in a bit of luck because XYZ format is ascii. So, the whole file can be represented as a string of text, right? So, you could do it as a TestText() kind of test where the baseline is the XYZ file you expect to produce.
If it wasn't an ascii format. Then, the way I'd do it is write the exported data and then read that data back in and do various Test() operations (e.g. images) on it to ensure the round-trip, which includes the writer's behavior, passes muster. Does that make sense?


CB-3133. Resource role assignment is not cleaned up after environment deletion

added ums resource delete step for the environment (non critical error - happens right before finish state)

few notes:

only internal iam user can call notify resource deletion ums api call (turned out during testing against real UMS)
resource crn is not passed through environment steps in the event objects, so i calling environment service find by id (which is also happening in another flow steps, like ClusterDefinitionDeleteHandler)...maybe would make sense fill resource crn in the events in the future, then these findbyid calls would be not required. (cc @attilapalfi92 )


ios-device-objc sometimes fails on CI

See https://ci.realm.io/job/objc_pr/2669/configuration=Release,swift_version=2.1.1,target=ios-device-objc/consoleFull
...
▸ Compiling RLMMultiProcessTestCase.m
▸ Compiling RLMPredicateUtil.m
▸ Compiling RLMTestCase.m
▸ Compiling RLMTestObjects.m
▸ Compiling SchemaTests.mm
▸ Compiling TransactionTests.m
▸ Compiling UnicodeTests.m
▸ Compiling UtilTests.mm
▸ Touching iOS
▸ Signing build/DerivedData/Realm/Build/Products/Release-iphoneos-static/iOS\ static\ tests.xctest
writeDictToFile:1278 ==== Successfully wrote Manifest cache to /var/folders/bj/n03qf0s968d07kcp9bc4v_vw0000gl/C/com.apple.DeveloperTools/All/Xcode/EmbeddedAppDeltas/bcbd05ba8f51351b8193eaae7dc6c8ae/07510d12c09c53e44b3682dd0367934f82cfab9c/ManifestCache.plist
/var/folders/bj/n03qf0s968d07kcp9bc4v_vw0000gl/T/0CC28208-B7C4-47F0-9BB4-002722197738-5370-000006D0E9402828/tmp/XCTest.framework: replacing existing signature
/var/folders/bj/n03qf0s968d07kcp9bc4v_vw0000gl/T/0CC28208-B7C4-47F0-9BB4-002722197738-5370-000006D0E9402828/tmp/IDEBundleInjection.framework: replacing existing signature
2016-01-04 18:05:27.808 xcodebuild[5370:42943]  iPhoneConnect: Failed to copy test bundle dSYM from /Users/realm/workspace/objc_pr/configuration/Release/swift_version/2.1.1/target/ios-device-objc/tightdb_objc/build/DerivedData/Realm/Build/Products/Release-iphoneos-static/iOS static tests.xctest.dSYM to /var/folders/bj/n03qf0s968d07kcp9bc4v_vw0000gl/T/0CC28208-B7C4-47F0-9BB4-002722197738-5370-000006D0E9402828/tmp/iOS static tests.xctest.dSYM: error Error Domain=NSCocoaErrorDomain Code=260 "The file “iOS static tests.xctest.dSYM” couldn’t be opened because there is no such file." UserInfo=0x7f962b8ea620 {NSFilePath=/Users/realm/workspace/objc_pr/configuration/Release/swift_version/2.1.1/target/ios-device-objc/tightdb_objc/build/DerivedData/Realm/Build/Products/Release-iphoneos-static/iOS static tests.xctest.dSYM, NSUnderlyingError=0x7f962bf828a0 "The operation couldn’t be completed. No such file or directory"}

Testing failed:
	Test target iOS static tests encountered an error (Error copying unit test bundle to container path /private/var/mobile/Applications/AC4B1D6F-40E7-403E-974A-DF12B10F2146 for app TestHost error: Permission denied)
** TEST FAILED **

I believe this is now fixed. It was a permissions problem on the iOS devices attached to VMs 1 & 2 of the host 1 CI machine. Deleting the test host files seems to have resolved it.


[Bug][github_graphql] Graphql query need the retry logic

Search before asking

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

What happened

It seems to wait too short or github meets some problem. When retrying it, it will work.
What you expected to happen
Graphql query needs the retry logic. Also, It should not panic.
How to reproduce

run github graphql query

Anything else
No response
Version
main
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Although added retry logic, not all error can be delt:
time="2022-10-28 05:18:10" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #1 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F17EC5:FCE69E:635AF533` when reporting this issue.\"\n      }\n   ]\n}\n""
time="2022-10-28 05:18:23" level=info msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] github graphql init success with remaining 4164/5000 and will reset at 2022-10-27 21:18:23 +0000 UTC"
[GIN] 2022/10/28 - 05:18:36 | 200 |   31.355417ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:18:36 | 200 |   18.416166ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:19:23" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #2 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F19F5C:FD08DB:635AF59F` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:19:38 | 200 |   22.373708ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:19:38 | 200 |   20.077084ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:20:37" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #3 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F1BBB9:FD26C7:635AF5CE` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:20:37 | 200 |     23.5265ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:20:37 | 200 |   23.763833ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:21:23" level=info msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] github graphql init success with remaining 4992/5000 and will reset at 2022-10-27 22:19:10 +0000 UTC"
[GIN] 2022/10/28 - 05:21:38 | 200 |    26.24125ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:21:38 | 200 |   24.536333ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:21:55" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #4 graphql calling after 60s
        caused by: unexpected EOF"
[GIN] 2022/10/28 - 05:22:37 | 200 |   21.906959ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:22:37 | 200 |   28.820083ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:23:10" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #5 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F1FF4B:FD6D9F:635AF661` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:23:36 | 200 |   34.385416ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:23:36 | 200 |   20.084459ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:24:26" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #6 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F21B23:FD8B2C:635AF6AF` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:24:36 | 200 |   11.190708ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:24:36 | 200 |   23.658667ms |             ::1 | GET      "/pipelines/128/tasks"
[GIN] 2022/10/28 - 05:25:35 | 200 |   23.564541ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:25:35 | 200 |   24.842292ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:25:39" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #7 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F234E7:FDA682:635AF6FB` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:26:33 | 200 |   36.531208ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:26:34 | 200 |   26.269541ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:26:52" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #8 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F24C9B:FDBFC8:635AF746` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:27:30 | 200 |    5.714917ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:27:30 | 200 |   21.093709ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:28:05" level=warning msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] retry #9 graphql calling after 60s
        caused by: non-200 OK status code: 502 Bad Gateway body: "{\n   \"data\": null,\n   \"errors\":[\n      {\n         \"message\":\"Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include `62A6:6738:F26556:FDDA1C:635AF78F` when reporting this issue.\"\n      }\n   ]\n}\n""
[GIN] 2022/10/28 - 05:28:25 | 200 |      41.077ms |             ::1 | GET      "/pipelines/128"
[GIN] 2022/10/28 - 05:28:25 | 200 |    26.34825ms |             ::1 | GET      "/pipelines/128/tasks"
time="2022-10-28 05:29:05" level=error msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] [CollectPr] ended Graphql collector execution with error
        caused by: graphql query failed"
time="2022-10-28 05:29:05" level=error msg=" [pipeline service] [pipeline #128] [task #11248] [github_graphql] [CollectPr] the first error of them
        caused by: graphql query failed
        Wraps: (2) got error when querying GraphQL (from the 10th retry)
        Error types: (1) *hintdetail.withDetail (2) *errors.errorString"
time="2022-10-28 05:29:05" level=error msg=" [task service] task failed
        caused by: Error running task 11248.
        Wraps: (2) subtask CollectPr ended unexpectedly
        Wraps: (3) graphql query failed
        Error types: (1) *hintdetail.withDetail (2) *hintdetail.withDetail (3) *errors.errorString"
time="2022-10-28 05:29:05" level=error msg=" [pipeline service] [pipeline #128] run tasks failed
        caused by: Error running task 11248."


can not fully reproduce the test results of the open source weight

Hello, thanks for your outstanding work!
I tested the open source weight: wisdomik/Quilt-Llava-v1.5-7b. Based on my test results, I guess the weight is trained with LLaVA chckpoint, 7B Language Model and stage 1 trains 0 epoch and stage 2 trains 3 epochs. Unfortunately, there is a test metric that is quite different from what you documented in your paper, and that is the test results on the closed set of Quilt-VQA w/ red circle. My test result was 71.3 and you recorded 77.78.
I am looking forward to your reply! Trank you a milion!

Hello, thanks for your outstanding work!
I tested the open source weight: wisdomik/Quilt-Llava-v1.5-7b. Based on my test results, I guess the weight is trained with LLaVA chckpoint, 7B Language Model and stage 1 trains 0 epoch and stage 2 trains 3 epochs. Unfortunately, there is a test metric that is quite different from what you documented in your paper, and that is the test results on the closed set of Quilt-VQA w/ red circle. My test result was 71.3 and you recorded 77.78.
I am looking forward to your reply! Trank you a milion!

Hi,
How did you evaluate the model? In quilt_eval.py, where is the 'answer-file-llava-zeorshot.jsonl'? If I set --anchor as None, I only get 'yes/no accuracy = 62.9738'.

I also encounter the same issue. Do you have any solution now?

Add a prompt “Please choose from the following two options: [Yes, No]” may help


DM-1138: Store and retrieve last configuration

On a command message it will store a complete list of everything being forwarded in a Kafka topic.
On restart it will retrieve the last set of settings and start them forwarding.
This is all optional (enabled by defining the storage topic) and the retrieval on start-up can be skipped using a flag.
System tests will be done in a separate PR.

Codecov Report

Merging #48 into master will decrease coverage by 1.10%.
The diff coverage is 81.13%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   93.08%   91.97%   -1.11%
==========================================
  Files          10       11       +1
  Lines         347      399      +52
  Branches       45       49       +4
==========================================
+ Hits          323      367      +44
- Misses         18       26       +8
  Partials        6        6

Impacted Files
Coverage Δ

forwarder/update_handlers/fake_update_handler.py
93.10% <50.00%> (-6.90%)
:arrow_down:

forwarder/update_handlers/ca_update_handler.py
91.17% <60.00%> (-2.48%)
:arrow_down:

forwarder/update_handlers/pva_update_handler.py
91.78% <60.00%> (-2.34%)
:arrow_down:

forwarder/configuration_store.py
88.88% <88.88%> (ø)

forwarder/handle_config_change.py
92.30% <100.00%> (+0.30%)
:arrow_up:

forwarder/parse_config_update.py
90.90% <0.00%> (+3.03%)
:arrow_up:


Formatting/README

Just was getting familiar with git again and made a few formatting changes and added a README.

Thanks !


query returning 404 on valid endpoint

@apollo/client: 3.8.10
@apollo/experimental-nextjs-app-support: 0.6.0
I have a simple query that has 1 string variable that is defined only when the component is mounted
My query looks something like
const GET_SENTIENCE5_HOLDER = gql`
  query sentience5($id: String!) {
     ...
    }
  }
`

and my hook looks like
const { data } = useSuspenseQuery(
    GET_SENTIENCE5_HOLDER,
    address
      ? {
          variables: {
            id: address,
          },
          queryKey: ['sentience5', address],
        }
      : skipToken
  )

I'm using skipToken to only run the query when the address is defined
I've confirmed that the problem is not the endpoint nor the query because if instead of the variable I pass a direct string it works perfectly.
If someone knows what might be the problem here I would love the help, thank you

@phryneas  I think so because if I do something like
const { data } = useSuspenseQuery(
    GET_SENTIENCE5_HOLDER,
     {
          variables: {
            id: 'test',
          },
          queryKey: ['sentience5'],
        }
  )

it works, so I ruled out that the HttpLink was not the problem but something related to my variable and the way I was using skipToken (although I followed the documentation to this argument, and why I am confused with the error)

@phryneas it is a 404 error from GraphQL, so can I assume the error is not in my skipToken logic?

Definitely not - it's something in how your server reacts to this specific query, so I'd look into the server.

Thanks, I'll take a look


[workbox-build] getManifest should return filePaths instead of manifestEntries

Hi,
Library Affected:
workbox-build
Browser & Platform:
all browsers
Issue or Feature Request Description:
Since 5.0.0, docs indicate that getManifest()'s return type is:
{count: number, filePaths: Array of string, size: number, warnings: Array of string}

That is, filePaths, instead of the old manifestEntries from v4.
However, as of 5.1.2, the source still returns the result of getFileManifestEntries without changing the manifestEntries to filePaths.
The actual behavior does not match the documentation.
Furthermore, @types/worbox-build include filePaths in getManifest()'s return type, and therefore are incompatible with the library.

Thanks for pointing this out! The getManifest() JSDoc was inaccurate, and will  be updated to reflect that the return value of getManifest() is an object with a manifestEntries property, not filePaths.
We don't publish @types/worbox-build, but I'm assuming that they're generated based on our JSDoc, so presumably fixing the JSDoc will propagate that fix forward.


[docs] Updating default poll.intervall.ms in PG connector docs

Hey @jpechane, just noticed this inconsistency in the docs. The actual default is 500ms indeed.

Applied to main (=2.1) and branches 2.0 and 1.9.  Branch 2.1 will be fast-forwarded when synchronizing with main during next 2.1 pre-release.

Thanks, Chris!

Message ID: @.***>


runtime schema-driven untagged deserialization

    return value;

Missing implementation for Bonded. Need to deserialize (unmarshall) Bonded contents. Do you plan to do this in the next commit?

Refers to: java/core/src/main/java/org/bondlib/BondedBondType.java:137 in 95a2ac8. [](commit_id = 95a2ac859d3cbd643866228434759781c810deaa, deletion_comment = False)

One of the examples is still broken, and I'm not sure what the correct behavior is. C# exception messages coming in a different fixup from a different machine.

Failing example was due to some futzing I was doing with it before handed it off. Sorry!

    return value;

I didn't see a reply to the comment regarding Bonded yet. It's not clear to me how an untagged payload containing Bonded struct (marshalled) is deserialized. The current code disregards the schema. Can you please add a TODO at least?

In reply to: 328975137 [](ancestors = 328975137)

Refers to: java/core/src/main/java/org/bondlib/BondedBondType.java:137 in a402c6b. [](commit_id = a402c6bf28e7d2ce17af960dfb96701ae9e30227, deletion_comment = False)

Forgot to update golden files. Fixed.

Made a mistake during the fixup. Fixed that.


Version 1.9.0 not running on Google Cloud Function

Attempting to use version 1.9.0 of puppeteer on Google Cloud Function is returning the following error.

Error: Failed to launch chrome! /srv/node_modules/puppeteer/.local-chromium/linux-594312/chrome-linux/chrome: error while loading shared libraries: libXcursor.so.1: cannot open shared object file: No such file or directory TROUBLESHOOTING: https://github.com/GoogleChrome/puppeteer/blob/master/docs/troubleshooting.md at onClose (/srv/node_modules/puppeteer/lib/Launcher.js:339:14
at Interface.helper.addEventListener (/srv/node_modules/puppeteer/lib/Launcher.js:328:50
at emitNone (events.js:111:20
at Interface.emit (events.js:208:7
at Interface.close (readline.js:370:8
at Socket.onend (readline.js:149:10
at emitNone (events.js:111:20
at Socket.emit (events.js:208:7
at endReadableNT (_stream_readable.js:1064:12
at _combinedTickCallback (internal/process/next_tick.js:138:11)
I have reverted back to 1.7.0 currently and that is still working.

It worked for me yesterday, and now it's broken, maybe some issue with firebase?
Error: Failed to launch chrome!
/srv/node_modules/puppeteer/.local-chromium/linux-579032/chrome-linux/chrome: error while loading shared libraries: libXcursor.so.1: cannot open shared object file: No such file or directory

TROUBLESHOOTING: https://github.com/GoogleChrome/puppeteer/blob/master/docs/troubleshooting.md

    at onClose (/srv/node_modules/puppeteer/lib/Launcher.js:299:14)
    at Interface.helper.addEventListener (/srv/node_modules/puppeteer/lib/Launcher.js:288:50)
    at emitNone (events.js:111:20)
    at Interface.emit (events.js:208:7)
    at Interface.close (readline.js:370:8)
    at Socket.onend (readline.js:149:10)
    at emitNone (events.js:111:20)
    at Socket.emit (events.js:208:7)
    at endReadableNT (_stream_readable.js:1064:12)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)

Tries today. Still broken for me.
Error: Failed to launch chrome! /srv/node_modules/puppeteer/.local-chromium/linux-594312/chrome-linux/chrome: error while loading shared libraries: libXcursor.so.1: cannot open shared object file: No such file or directory TROUBLESHOOTING: https://github.com/GoogleChrome/puppeteer/blob/master/docs/troubleshooting.md at onClose (/srv/node_modules/puppeteer/lib/Launcher.js:339:14) at Interface.helper.addEventListener (/srv/node_modules/puppeteer/lib/Launcher.js:328:50) at emitNone (events.js:111:20) at Interface.emit (events.js:208:7) at Interface.close (readline.js:370:8) at Socket.onend (readline.js:149:10) at emitNone (events.js:111:20) at Socket.emit (events.js:208:7) at endReadableNT (_stream_readable.js:1064:12) at _combinedTickCallback (internal/process/next_tick.js:138:11)

Dupe of #3422 - should be fixed right now.

@aslushnikov I'm still seeing this with 1.10.0 deployed to a Firebase Function
TimeoutError: Timed out after 30000 ms while trying to connect to Chrome! The only Chrome revision guaranteed to work is r599821     at Timeout.onTimeout (/srv/node_modules/puppeteer/lib/Launcher.js:353:14)     at ontimeout (timers.js:482:11)     at tryOnTimeout (timers.js:317:5)     at Timer.listOnTimeout (timers.js:277:5)

Exactly the same error message for me, version 1.10 in Node 8 Cloud Functions.
Hangs on this line of code:
const browser = await puppeteer.launch({args: ['--no-sandbox']});

Oddly enough, this is happening for me on my work desktop but not on my home desktop. Both environments are identical.


Test: Shell Integration API

Refs: #145234

[x] macOS @meganrogge
[ ] linux @justschen
[ ] windows @dbaeumer

Complexity: 5
Create Issue

The Terminal.shellIntegration API has been finalized this iteration. To test this:

Clone and open the extension sample https://github.com/microsoft/vscode-extension-samples/tree/main/shell-integration-sample
Test to make sure the extension functionality works - open the custom profile via the dropdown on the terminal:

Explore the API and its docs. Here are the entry points: Terminal.shellIntegration, onDidChangeTerminalShellIntegration, onDidStartTerminalShellExecution, onDidEndTerminalShellExecution
Exercise other functionality of the API, the sample only uses onDidChangeTerminalShellIntegration and executeCommand(string, string[])

@Tyriar I tried for one hour to get shell integration work for me on my Windows machine but no luck. Whatever I do, I never received any onDidStartTerminalShellExecution, onDidChangeTerminalShellIntegration or onDidEndTerminalShellExecution events, so the sample never worked. Here is what I tried:

setting explicit "terminal.integrated.shellIntegration.enabled": true
Added if ($env:TERM_PROGRAM -eq "vscode") { . "$(code-insiders --locate-shell-integration-path pwsh)" } to my profile, following the advise from https://code.visualstudio.com/docs/terminal/shell-integration#_manual-installation
turned my own profile script off since I have a custom Prompt function.

Any hint on how to enable this

@dbaeumer sounds like it should be working with that setup. Try installing pwsh 7 and using that if you haven't, Windows PowerShell (5) which is shipped with Windows should work but maybe that's why?

@Tyriar I use pwsh 7.4.5. Anything I can do (logging) to find out why it is not working?

@dbaeumer when you hover the tab you should see a long command line like this:

Make sure you don't have terminal.integrated.windowsEnableConpty set.
If that doesn't help enable a trace log and check the pty host output channel when you create a terminal. You should see a [trace] [RPC Request] PtyService#createProcess log which will have some config associated with it like this:
{"shellIntegration":{"enabled":true,"suggestEnabled":true,"nonce":"80e12edd-a671-4229-a4b9-028ae4b8f609"},"windowsEnableConpty":true,"windowsUseConptyDll":true,"environmentVariableCollections", ...

That would be helpful

Cool 👏 , I think this will be powerful


Update docs with new Get Image Media API

Similar to the Modality API issue for media access, we need to make sure that the url property of ImageResource is a documented API. A new API method GET /images/{id}/media was created for this purpose.
The OpenAPI spec, the ReDoc page and the Wiki must be updated with this new API call.

The OpenAPI spec and Redoc files have been updated with this new API. The SDK was also updated.
@jtetro2, we have only the Wiki to update and we can close this issue.

@JohnGoyette, the WiKi has been updated, per your comments.

I do not see the added API in the Wiki.

@JohnGoyette,
The Get Image Media API, and your other suggestions regarding additional sidelinks (here and in the Modality API WiKi), and Response information for the Filters APIs, are incorporated.

Looks good. Thanks, @jtetro2


How do I start the extension?

Many major Chrome plug-ins (eg. Lazarus, SessionBuddy, React Developer Tools) have icons that appear next to the URL bar and make it possible to easily start them.  But ChromeRestClient doesn't have such an icon, and I couldn't find anywhere on the Chrome store, the extension's webpage, or the GitHub page that says how to start it up.
Could you please consider documenting this very basic bit of info somewhere, or better yet add an icon to make the extension easy to access?  I know this must seem incredibly obvious to you, but to someone using the extension for the first time it's not, and right now I'm having to open the extensions page every time I want to us the rest client :(
Thanks.

Hi,
Thank you for issue report.
I can't do much about this. It's a Chrome team issue. Apps are not extensions and do not have icon in the browser. You can run it from the new tab page clicking first "App" button:

An then selecting the app:

Clicking right button on the app icon there will brings a context menu. There's an option to create a shortcut to the app. Options there depends on your OS.
Nothing much I can do more. I'll add proper description to the store's description.

Nothing much I can do more.

I'll add proper description to the store's description.

Thanks for at least doing that much!

I'm guessing that's why they are removing support for Chrome Apps next year...


SnippetsList equivalent?

Are there plans to add the SnippetsList in?

Not likely. The usual answer applies--I don't believe there is a universal way to do this where everybody is happy. That should be handled in an application-specific way.

No problem, I'll create a custom one then. Is there a way to get the (x,y)
coordinates of the caret in the editor?
On Thu, Jul 2, 2015 at 4:28 PM, jacobslusser<EMAIL_ADDRESS>wrote:

Not likely. The usual answer applies--I don't believe there is a universal
way to do this where everybody is happy. That should be handled in an
application-specific way.
—
Reply to this email directly or view it on GitHub
https://github.com/jacobslusser/ScintillaNET/issues/54#issuecomment-118157901
.

--
Ryan O'Day
Business Email<EMAIL_ADDRESS>Business Website: www.bionetworkscorp.net

CharPositionFromPoint, CharPositionFromPointClose, PointXFromPosition, and PointYFromPosition.
That should be what you need if you wanted to display a custom popup, buy my recollection is that the last version of ScintillaNET simply used user completion / auto completion lists to display a list of snippets.

Thank you once again.


✨ EditorConfig Support

Description
EditorConfig is a useful file format for keeping code style the same across IDE’s. It has built-in support in numerous editors and is a fairly simple spec.
https://editorconfig.org/
All supported config options: https://github.com/editorconfig/editorconfig/wiki/EditorConfig-Properties
Alternatives Considered
No response
Additional Context
This would require workspace settings overrides, since the config would be workspace-specific.
Screenshots
No response

Duplicate of #766

Closing #766 in favor of this issue.


Using Microsoft Rush with SPFx-Fast-Serve

I started with your suggestion about Lerna, and got more interested in Microsoft Rush. I just want to make this post to see if there is any improvements that could be made in the process what I have so far. Its not really an issue about fast-serve, but just wanted to get your opinion (or others).
What got me interested with Rush. (Some may be possible with Lerna, but I haven't invested as much time with Lerna)

It creates one node_modules and creates links for all the packages (Lerna as well can achieve this with hoist)
It has a built in "rush check" command that can be used to track difference in package versions across all the solutions
When purging and installing node_modules freshly, it moves the existing node_modules to separate recycler folder, and starts a job to clean up the existing node_modules. The time taken to remove node_modules is one of the big issues and Rush helps with it.
Configurations to create custom bulk commands and global commands.
Able to build / run commands only to specific project (and all it dependencies)
SPFx already uses the rushstack and expect more support for SPFx in Future.

Considering the below as what I wanted to achieve (With multiple webpart / library projects)

Make dependency management easy across all projects in a workspace (upgrading to newer versions)
Reduce package installation times
Use SPFx-Fast-Serve during development time
Make package creation easy
Avoiding manual "npm link"
If possible allow someone else to use the traditional gulp commands and not depend on Rush
With many number of web part solutions, its difficult to keep track of port numbers if we have to maintain different port number for each. Hence all non-library solutions will use same port number, and only one "non"-library solution will be served at a time, which is also meaningful.

Refer Below Repo (which kind of works, but not in best possible way)
Sample Repo
My Observations (with NPM only)

With library components added using "../SPFx-Library"

Installing Dependencies: Faced issues. Have tried many things with "preferredVersions" in "common-versions.json" and "cyclicDependencyProjects" in rush.json
Without this, approach its not possible to meet points 5 & 6
This would be ideal approach

With Library components added using version number

Installing Dependencies: Works
Build Package: Works
Npm Run Serve using Rush's Bulk command: Not working as expected, as Rush is waiting for the "npm run serve" of library component to finish in-order to start the next one. As gulp serve waits for ever, it doesn't go past Library. Not sure how Lerna handles this with "Run Parallel" (This would be next ideal approach)
Npm Run Serve on the two projects works individually fine. (as in Sample Repo)

Feel free to close the thread if required, as it not really something Fast-Serve could do about. I just wanted to check with you, since I was not able to find any other sources of information (w.r.t SPFx) other than the two links that you have provided about Lerna and Rush

One step forward: Parallel execution is working with ignoreDependencyOrder set. Though console logs are not appearing from all the projects that are being executed. I have raised a request in Rush repo...
https://github.com/microsoft/rushstack/issues/1863

That's a matter of personal preferences and opinion, whether to use one library or another. I just want to share my thoughts on why I would not select rush and select lerna as a multipackage manager for SPFx (again, that's just how I see it):

fast and simple, has zero prod dependencies (rush has deep dependency tree, however maybe it has more features)
lerna has better adoption and widely used. If you have questions\troubles, most likely you will find the answer faster. Also, check out downloads count on npm over time:

rush doesn't work with local workbench
rush doesn't have any "killer" features, which make it brilliant when compared to lerna (at least for me)

Taking your list "what I wanted to achieve" only #2 is not possible with lerna, because lerna will install all dependencies separately.

Yes its indeed personal choice, and also depends on the type of projects as well. Localhost is not important for us, as we mostly work with data from SharePoint directly and not into setting up mock data / tests as of now.
Anyways, I will close this thread for now. Will update here if I get any progress in Rush. Thanks.

BTW, you can develop against your real SharePoint data from local workbench without mocking. Check out sp-rest-proxy project.

Wow.. u give me a lot of good info.. I will have a look into it some time later..

Just a follow up about Rush... Adding packages to single solution takes considerable time as its linking for other projects as well.


AuthMetadataProcessor: don't start a ThreadPool for non-blocking

Fixes #16855
Without this, N threads(one per core) end up waiting
on a cond var that is never updated
at DynamicThreadPool::ThreadFunc()

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @nathan-prat isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

@nathan-prat : can you sign the CLA? I can't run tests on this until you do

About GPR_ASSERT: thread_pool_ is init in the ctor when IsBlocking(), so the assert is indeed a bit redundant(I had not noticed it was a runtime assert)
About the CLA: there is something wrong with the signature. I have opened a ticket.
NOTE: this is company CLA, so it may take some time to sort this out.

@nathan-prat : thanks for the update. We'll merge when we get a signed CLA. I hope your process works out.


compatibility with keras's cudnn saved model?

Hi
I'm new to machine learning and so far, I'n training a cudnn lstm model using tensor flow 1.13.1, question is, can i load a saved model using your library into c# and predict the result or not? if yes, is there an example for this or can you please guide me on how to do it?
if no, is there any solution for this? i am aware of tensorsharp but it looks like the version they use is tensor 1.12 with cuda 9 (tensor 1.13 uses cuda 10)
thanks

Hi Jiraiyah,
Not yet - but we will add this to the list for such a feature would be very useful.
Thanks!


Fix import error in python3

Add code to check python version in runtime to fix import error in
python3 environment.

Thanks for this -- but I am closing this request since the latest update to imutils includes full Python 3 support.


feat(dbm-services): spider_priv

关联研发信息
Pull Request: https://github.com/TencentBlueKing/blueking-dbm/pull/55
开发分支: feat/spider_priv
目标分支: master

分支污染废弃


Fix unable to parse each.value object issue

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Currently yor cannot parse existing tags like tags = each.value.tags. This pr fix this issue.

Hi @nimrodkor, we'd like to use yor in our projects but we've encountered some bugs, would you please review this pr when you have time? Thanks!

Greetings @nimrodkor & @gruebel, any chance to merge this pr? Thanks!

Thanks @gruebel for the update!


OpenJ9 heads up, mixed refs builds soon to be enabled for jdk11

FYI that OpenJ9 is soon ready to switch to use mixed refs builds for jdk11. These builds combine compressed and XL into a single build. Once the mixed refs build is enabled, the platform doesn't require a separate XL build.
The mixed ref build can be enabled via configure with the option --with-mixedrefs. The OpenJ9 project has enabled them for jdk11 nightly testing. Atm there is a problem to be resolved, https://github.com/eclipse/openj9/issues/11819, but once this is resolved then AdoptOpenJDK should enable these builds for jdk11 in time for the April 11.011 release.
Note a prereq to enabling mixed refs is cmake. Only jdk11 has cmake enabled for all platforms atm (see https://github.com/AdoptOpenJDK/openjdk-build/issues/2424).

Note the mixed refs build could be enabled now for jdk11 if you like, but it will cause some special.system testing to fail as described in the previous comment. I will update here when the problem is resolved.

We are not running special.system target currently (as its a lot of extra modes / resources) so we should be good to switch and not see those test failures.

I can open ad PR the build script changes as I have already tested them. I assume there will be more changes required for website etc.

I think work is complete.

Yup, mixed refs builds were enabled by default for all platforms and versions. See https://github.com/ibmruntimes/openj9-openjdk-jdk11/pull/382 and the referenced PRs.


Scala 2.13 support

Scala 2.13 is finally available. :tada:
Since we are using scapegoat in our projects, we need a release for 2.13.
This PR is migrating the inspections to 2.13 while keeping compatibility with 2.11 and 2.12.
breaking changes:

String interpolations are no longer recognized, since they are macro expanded (see FastStringInterpolator.scala:23)
JavaConversions are gone in 2.13, test code compilation fails with object JavaConversions is not a member of package collection
scala.Seq is now aliased to scala.collections.immutable.Seq instead of scala.collections.Seq - makes the (ineptly named) PredefSeqIsMutable inspection obsolete
scala.Traversable is now an alias for scala.collection.Iterable and it is deprecated - makes the PredefTraversableIsMutable obsolete

Awesome work.

1.3.9 has been released with 2.13 support.


Documentation

Some functionalities this library provides aren't always intuitive therefore some docstrings would be very helpful.

Thanks for your feedback! There are indeed very limited documentations for TorchSparse right now. We designed our API to be as close to PyTorch as possible. However, there is also a few backend APIs that is harder to understand. We will add more documentation in the future (but it will not in our top priority as our focus is more on efficiency). If you find anything in the library that is unclear, please do not hesitate to post them in the issues.

Can you explain a bit about these methods (except maybe conv3d, this I kind of understood from the use cases in torch-points3d and spvnas): https://github.com/mit-han-lab/torchsparse/blob/a50506a904f336dfe8f96b3393f7e9ac2b8f1950/torchsparse/nn/modules/conv.py#L10
I can guess the functionality from their names but I want to be sure. Also can you specify the shape of the input tensor and output tensor for each of these case (where applicable / possible offcourse).

@digital-idiot torchsparse.nn.modules.conv3d is a normal sparse convolution. The parameters follow from a dense pytorch convolution. The functional version takes the following inputs:
def conv3d(inputs, # SparseTensor for N input points
                   # C = coordinates of shape (N, 4) for C[:,0:3] as quantized coordinates and C[:,3] as batch indices
                   # F = features of shape (N, x) for x>0 features per point
           kernel, # Volumetric kernel of shape (kernel_size**3, in_channels, out_channels)
           kernel_size, # length of kernel along one dimension
           bias=None,	# rest of the args are the same as non-functional
           stride=1,
           dilation=1,
           transpose=False)

'ToBEVConvolution', 'ToBEVReduction', 'ToDenseBEVConvolution'  are just object detection helper functions as I understand (https://github.com/mit-han-lab/spvnas/issues/13).

Thanks @CCInc. I understood conv3d and partly ToDenseBEVConvolution. What I don't understand so far are ToBEVConvolution, and ToBEVReduction. Below is my own implementation of converting voxelized SparseTensor to dense 5D tensor. If I understood it correctly, a voxel may contain numerous points, thus an aggregation function or pooling when doing this. I don't see that in any of the  ToDenseBEVConvolution, ToBEVReduction or ToBEVConvolution. That is why I am curious know about these three functions.
import torch_scatter

def voxel_to_grid(x, shape, offset=None, pooling='avg'):

    coords = x.C[:, :-1].cpu().numpy()
    assert coords.shape[-1] == len(shape)
    if offset:
        coords -= offset

    flat_indexes = np.ravel_multi_index(
        multi_index=(coords[:, ax] for ax in range(len(shape))),
        dims=shape,
        mode='raise',
        order='C'
    )
    flat_indexes = torch.tensor(
        data=flat_indexes, dtype=torch.long, device=x.F.device
    )
    grid_shape = (*shape, x.F.shape[-1])
    fgrid_shape = (reduce(operator.mul, grid_shape[:-1], 1), grid_shape[-1])
    fgrid = torch.zeros(
        size=fgrid_shape,
        dtype=x.F.dtype,
        device=x.F.device
    )
    if pooling.lower() == 'avg':
        fgrid = torch_scatter.scatter_mean(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    elif pooling.lower() == 'max':
        fgrid = torch_scatter.scatter_max(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    elif pooling.lower() == 'min':
        fgrid = torch_scatter.scatter_min(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    else:
        raise NotImplementedError(
            'Unknown Pooling Operation: {}'.format(pooling)
        )
    grid = torch.reshape(input=fgrid, shape=grid_shape)
    return grid

The previous implementation was incomplete, it did not consider the batch dimension. Below is the new one. @CCInc , @zhijian-liu can you have a quick look and let me know if the approach is somewhat correct?
def voxel_to_grid(
    x,
    vol_shape,
    index_offset=None,
    pooling='avg'
):
    """
    Converts SparseTensor with quantized coordinates to dense grid Tensor (5D)
    :param x: SparseTensor with quantized coordinates
    :param vol_shape: Volumetric shape (H, W, Z)
    :param index_offset: Offset of the index (X, Y, Z, B)
    :param pooling: Type of pooling {'avg', 'max', 'min'}
    :return: Tensor with shape (H, W, Z, B, C)
    TODO: Add an option for initializer (such as uniform, normal, xavier etc.)
    """
    # This is just to make use of `numpy.ravel_multi_index` (Yes, I am lazy!)
    coords = x.C.cpu().numpy()
    if index_offset:
        coords -= index_offset
    batch_size = coords[:, -1].max() + 1
    flat_indexes = np.ravel_multi_index(
        multi_index=coords,
        dims=(*vol_shape, batch_size),
        mode='raise',
        order='C'
    )
    # Tensor with raveled index with matching rows size
    flat_indexes = torch.tensor(
        data=flat_indexes, dtype=torch.long, device=x.F.device
    )

    # This is the vol_shape of output tensor (H W Z B C)
    grid_shape = (*vol_shape, batch_size, x.F.shape[-1])

    # Flattened shape (N, C): N = H × W × Z × B
    fgrid_shape = (reduce(operator.mul, grid_shape[:-1], 1), grid_shape[-1])
    fgrid = torch.empty(
        size=fgrid_shape,
        dtype=x.F.dtype,
        device=x.F.device
    )

    # pooling / aggregation to ensure single feature for each voxel
    # Relies on https://pytorch-scatter.readthedocs.io for the pooling ops
    if pooling.lower() == 'avg':
        fgrid = torch_scatter.scatter_mean(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    elif pooling.lower() == 'max':
        fgrid = torch_scatter.scatter_max(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    elif pooling.lower() == 'min':
        fgrid = torch_scatter.scatter_min(
            src=x.F,
            index=flat_indexes,
            dim=0,
            out=fgrid
        )
    else:
        raise NotImplementedError(
            'Unknown Pooling Operation: {}'.format(pooling)
        )
    grid = torch.reshape(input=fgrid, shape=grid_shape)
    # TODO: Rearrange the axis here as required
    return grid

If you've already voxelized the coords using one of the helper functions like sparse_quantize, it should remove the duplicates automatically. SparseTensor objects would usually contain unique coord ids. If your goal is just to create a dense representation from sparse coords, then one approach would be to create a (H,W,D,C) tensor and assign C based on the sparse X,Y,Z coords, i.e.
sparse = SparseTensor(feats, coords)
dense = tensor.ones(H,W,D,B,C)
dense[sparse.C] = sparse.F

If your goal is just to create a dense representation from sparse coords, then one approach would be to create a (H,W,D,C) tensor and assign C based on the sparse X,Y,Z coords, i.e.
sparse = SparseTensor(feats, coords)
dense = tensor.zeros(H,W,D,B,C)
dense[sparse.C] = sparse.F

@CCInc thanks for clarifying. Yes, sparse_quantize is mentioned in the README of this repository. But the implementation of the voxelization funtions initial_voxelize and point_to_voxel from spvnas are seems quite different from sparse_quantize here. These two functions relies on sphash, sphashquery, and spvoxelize functions from torchsparse. This is because in the SPVNAS paper : section 3.3  they (@zhijian-liu ) specified:

we propose to use the GPU hash table to accelerate the sparse voxelization and devoxelization. Specifically, we first construct a hash table for all activated points in the sparse voxelized tensor S, which can be completed in O(n) time. After that, we iterate over all points in T, and for each point, we use its voxelized coordinate as the key to query the corresponding index in the sparse voxelized tensor. As the lookup over the hash table requires O(1) time [37], this query step will in total take O(m) time. Therefore, the total time of coordinate indexing will be reduced from O(mn) to O(m+n).

My intention was to use initial_voxelize and point_to_voxel instead of sparse_quantize which seem to be more efficient. In that  respect, I was wondering if

it should remove the duplicates automatically.

is valid or not.

Sorry for the late reply! These functions deal with the duplicate coordinates using the default behavior of torch.cuda.float.SparseTensor, which can be seen from https://pytorch.org/docs/stable/sparse.html:
PyTorch sparse COO tensor format permits uncoalesced sparse tensors, where there may be duplicate coordinates in the indices; in this case, the interpretation is that the value at that index is the sum of all duplicate value entries.

@CCInc, thank you so much for following up on this.
@digital-idiot, your voxel_to_grid implementation looks correct to me.


La 384 cleanups

Issue: LA-384

@hughsaunders Adapted, could you have a look at the english again?

recheck_cit_trusty_leapfrogupgrade_swift -- tempest issue server got down.

See https://github.com/rcbops/rpc-openstack/pull/2444


Move away from find_packages() to explicitly declaring the package.

Closes #87. The find_packages() function was picking up the tests/ directory and polluting site-packages with a tests/ directory. :hushed:

@jaitaiwan I think this one might be a priority to ship along with the fixes I've brushed up tonight. I started a 2.0 milestone and tagged PRs that I think we need to look into or get merged.
What are your thoughts on pushing out a 1.9 release and then 2.0 when PT3K is done along with the other PRs? (I also have finished auditing all of the issues. Lots of duplicates.)

Agreed. LGTM


[Optimize] Health check scraping website page like "/health" or "/metrics" will contaminate the website visit statistics

Hi:
As I said in issue title, exporter scrape page like /health or /metrics/format/json will contaminate website visit statistics, Those visits have no relationship with real service.
I deem that this could be optimize in ngx-vts-module, but could we do sth in this exporter?

This could be solved by proper nginx config. 😃
And as far as I'm concerned, nginx-vts-module should have directory to do this stuff.

@hnlq715
I had try to find this directory in ngx-vts-module, but not yet >_<

@hnlq715
Hi , I have found that the directories as below, thanks a ton

vhost_traffic_status_bypass_limit
vhost_traffic_status_bypass_stats


Delete file

Issue Type: Bug
We were working with my team and suddenly the whole file was deleted
Extension version: 1.0.2236
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:22.494Z)
OS version: Linux x64 5.3.0-53-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz (8 x 2749)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
7, 5, 4

Memory (System)
7.62GB (0.65GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
cinnamon

XDG_CURRENT_DESKTOP
X-Cinnamon

XDG_SESSION_DESKTOP
cinnamon

XDG_SESSION_TYPE
x11

Thanks for reaching out. Can you give us a bit more detail about your scenarios ? Were you the host or guest ?
Your logs may provide us with helpful information. You can access them via the Live Share: Export Logs command from the command palette (ctrl + shift + p) and attach them to this issue.

@daytonellwanger assigning to you but feel free to reassign if it is not your area

In addition to @setaskin 's questions, have you seen this happen before? Were the contents of the file deleted? Or was the file actually deleted?


How to enable the panel?

According to the installation page on the plugin overview page the panel should somehow be addable in a dashboard. However I do not see a way to add it anywhere.
When I look at the home page of my Grafana instance (version 6.1.4) the Influx Admin is listed under "Installed Panels", but I cannot do anything with it on that page.
When I create a new dashboard and add a panel there, there is no option to select the admin panel under the "Visualisation" section either.
Where is it supposed to show up?

Same problem here. How to add it to the dashboard?

Go to a (new) dashboard, add a new panel and click "Choose Visualization".

If the plugin is installed, you can choose the option "Influxdb Admin".

I can not find the panel in Grafana 5. Is it hiding somewhere else?

You need to install the InfluxDB Admin panel: https://grafana.com/grafana/plugins/natel-influx-admin-panel

Still same issue, installed but not visible from Visualisations


Huggingface has marked your model as suspicious

It looks very interesting though. Would you mind uploading a safetensors version?
Thanks

Hi, @ForgetfulWasAria , we have converted the checkpoint in HF to safetensors format.

Thank you!


Patched apk fails to install on android 12

Recently the quest updated to android 12 and that probably means changes in how apps are structured / the andorid manifest is structured...
Questpatcher is unable to install a patched apk:
adb: failed to install C:\Users\redbr\AppData\Local\Temp\QuestPatcher\patching\patched.apk: Failure [-124: Failed parse during installPackageLI: Targeting R+ (version 30 and above) requires the resources.arsc of installed APKs to be stored uncompressed and aligned on a 4-byte boundary]

Manually installing the app from the patching directory yields this error:
.\patched.apk: 1 file pushed, 0 skipped. 126.7 MB/s (936195836 bytes in 7.049s)
Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES: Failed collecting certificates for /data/app/vmdl833537255.tmp/base.apk: Failed to collect certificates from /data/app/vmdl833537255.tmp/base.apk: META-INF/BSQUEST.SF indicates /data/app/vmdl833537255.tmp/base.apk is signed using APK Signature Scheme v2, but no such signature was found. Signature stripped?]

Fixed in latest release


Crowdsec Support

Hi Folks,
Is Crowdsec being officially supported by the latest image? Asking because I see it mentioned in v2.9.17 released this Mar 30 but there are no other mentions besides the ones at the fork by lepresidente @ https://github.com/LePresidente/docker-nginx-proxy-manager.
Thanks!

I don't see why you wouldn't be able to proxy crowdsec.
There isn't really such a thing as "official software support", as npm is really just a fancy configuration interface for nginx.

@cloudybyte you need to make some changes on the nginx configuration and install a bouncer that is not easy. There is another developer that is making the configuration, his nick is lepresidente, but he is making a fork from another fork, not the official image, and some things are changed, like the mount point for the data folder, and some features are not working, like generating certificates from cloudflare.
It could be nice to have the crowdsec bouncer included on the official image.


feat: kafka sink should populate key from reflection

supports providing .proto files for key
supports autogenerating a proto message at runtime with a single field as key without .proto file.

Sample config for using .proto file
name: date-kafka-recipe
source:
  type: date
sinks:
  - name: kafka
    config:
      brokers: localhost:9092

      # kafka topic name
      topic: test-3

      # Use this path to find value that will be used to set kafka key proto
      # message.
      # Only supports root level key at the moment
      # Should be a string
      key_path: ".Action"

      # name of the kafka key proto message
      proto_message: odpf.meta.CustomKey

      # proto to use for building key
      proto_file: absolute_path_to___key.proto

key.proto
syntax = "proto3";

package odpf.meta;

option go_package = "github.com/odpf/proton/meta/custom";

message CustomKey {
  string Action = 1;
}

If we remove proto_file from config, it will autogenerate the proto at runtime.

All of this is without stencil. We can add stencil support as well to avoid providing .proto files maybe in stencil JAM. 🕺

neat!!
Im thinking if we should allow user to use a default proto key defined by us.
since right now Meteor handles it owns proto model, technically we could also decide the proto key from that model.
and if user wants to override it, we let them use those configs.
this would also allow us to not having to define the proto key everytime on every recipe that sinking to kafka.
wdyt? @kushsharma

neat!!
Im thinking if we should allow user to use a default proto key defined by us.
since right now Meteor handles it owns proto model, technically we could also decide the proto key from that model.
and if user wants to override it, we let them use those configs.
this would also allow us to not have to define the proto key every time on every recipe that sinking to Kafka.
wdyt? @kushsharma

+1 Do we need even need the proto file and name provided. Meteor already has the struct. It does not have to worry about unmarshalling proto. unless some extractor is marshaling the data and sending bytes.

neat!!
Im thinking if we should allow user to use a default proto key defined by us.
since right now Meteor handles it owns proto model, technically we could also decide the proto key from that model.
and if user wants to override it, we let them use those configs.
this would also allow us to not having to define the proto key everytime on every recipe that sinking to kafka.
wdyt? @kushsharma

Do you mean having single protobuf for all kafka keys in messages?

neat!!
Im thinking if we should allow user to use a default proto key defined by us.
since right now Meteor handles it owns proto model, technically we could also decide the proto key from that model.
and if user wants to override it, we let them use those configs.
this would also allow us to not have to define the proto key every time on every recipe that sinking to Kafka.
wdyt? @kushsharma

+1 Do we need even need the proto file and name provided. Meteor already has the struct. We can just provide the cofigs for folks to override the keys. It does not have to worry about unmarshalling proto. unless some extractor is marshaling the data and sending bytes.

What meteor struct are you referring to? Sorry didn't get it, can you give an example?

neat!!
Im thinking if we should allow user to use a default proto key defined by us.
since right now Meteor handles it owns proto model, technically we could also decide the proto key from that model.
and if user wants to override it, we let them use those configs.
this would also allow us to not having to define the proto key everytime on every recipe that sinking to kafka.
wdyt? @kushsharma

Do you mean having single protobuf for all kafka keys in messages?

right now in our metadata model, we also have proto key defined here.
we could use that, or @rohilsurana actually brought up an idea that we could use the model message proto as the proto key itself.
So when building the key, we would use the same proto that we use for the metadata, but instead of populating all the value, we would only populate fields that we want to mark as the key (default to Urn).

So when building the key, we would use the same proto that we use for the metadata, but instead of populating all the value, we would only populate fields that we want to mark as the key (default to Urn).

I like this idea, let me do this instead as the default behavior of autogenerating a new message from scratch. I guess we can keep the .proto option for users who want to use a different proto for key?

I guess we can keep the .proto option for users who want to use a different proto for key?

Since it is already there I guess we can keep it, it would be helpful if we decided to introduce custom proto for sinking to kafka.

Yeah, i am also okay with having only one way. We can choose to remove this custom proto option until someone requests it

@kushsharma  @StewartJingga LGTM


How to customize text color?

What OS?

Mac

Description
I have looked for the way to change the table's text color, but didn't find one.
Is this possible?
The default color is too light for my light background.

This is currently not possible. Would the text color have to change the text of controls (buttons, ...) too?

It might be helpful for some people, but I do not feel the need to do so. The effort to customize it would outweigh the benefits.

I've added options to change the table text color darkThemeTextColor and lightThemeTextColor. Lets see if this is enough.
Will be released with the next update.

Additional note: Any valid css color will work. When the color is invalid, the default color is used.


[Bug] Adding Folder To Workspace Expands All Folders

Environment Details

VSCode Version: 1.31.1
OS Version: Windows 10 Pro 64bit

Steps to Reproduce:

Adding Folder To Workspace Expands All Folders

Expected Result

Adding Folder To Workspace Only Expands That Folder

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

I can not reproduce.
Can you reproduce with vscode insiders https://code.visualstudio.com/insiders/
If yes, what are the exact steps you add folders, and what fodlers exactly get expanded

Update
It also happens when a folder is removed from the workspace. If looks like the workspace refresh/update event messes up.

Fixed in latest insiders


Hs stable

@rufelo these are development branches corresponding to our environments, please do not open PRs for them


[WIP] Added support for LinearEqualityToPenalty converter accepting a list of penalty terms

Summary
Fixes #1294
Added support for LinearEqualityToPenalty converter accepting a list of penalty terms (#1294)
* now penalty can either be a single float (backward compatible) or a list of float that applies to linear constraints correspondingly
* updated docstring for the change
For the tests:
* added peanlty list version for all existing test
* added test for exception when length of penalty list does not match number of constraints
Details and comments
Please let me know if any change is needed! Thanks!

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

Looks good. Can we have support for int and np.array as well? Also numpy int and float types might be used, but for them the error message should be fine if implementing is tedious.

I think int makes sense. It seems np.int and np.float are equivalent to python built in int and float type so as long as we support int and float the np version will work. If we support int, do also wnat to support mixed int and float in a list?
If we want the penalty "list" to be as compatible as possible (i.e. compatible with all iterable objects), maybe we can just construct a list out of it and test the element type.
So, with int and float (and mixing) and general iterable object support:
        if self._penalty is None:
            penaltys = self._auto_define_penalty()
        elif isinstance(self._penalty, (float, int)):
            penaltys = [self._penalty] * num_constraint
        else:
            try:
                penaltys = list(self._penalty)
            except TypeError as e:
                raise QiskitOptimizationError('Unsupported penalty type: {}'.format(
                    type(self._penalty)))
            if not all(isinstance(penalty, (float, int)) for penalty in penaltys):
                raise QiskitOptimizationError('Unsupported penalty type')

This looks very good now :) Thank you.

Also, it seems that it is also failing Mypy testing.

qiskit/optimization/converters/linear_equality_to_penalty.py:69: error: Incompatible types in assignment (expression has type "List[float]", variable has type "float")

What is this testing? How can I test this locally? BTW, I have run all unittest.

Also, I would appreciate some feedback on unittest. Do my tests make sense? What else do I need to put in unittest? I suppose I should test mixed int float list. Should I also test np.array? np array with mixed int float? Other unsupported type?

If you have run from qiskit-aqua folder: pip install -r requirements-dev.txt
You can run: make mypy
That is what the CI is using

For the mypy type test, I tried to annoate penalty as we supports more type:
def __init__(self, penalty: Union[float, int, Iterable, None] = None) -> None:

@property
def penalty(self) -> Union[float, int, Iterable, None]:

@penalty.setter
def penalty(self, penalty: Union[float, int, Iterable, None]) -> None

But I am still getting many errors like this:

error: Incompatible types in assignment (expression has type "List[float]", variable has type "float")

I would appreciate help on proper type hints.

Given the move/refactor that is about to take place, and given this has been around for some while I am thinking it should be closed. If it turns out its something that is wanted then it can be replicated if needs be. Thoughts/objections?

Given the move/refactor that is about to take place, and given this has been around for some while I am thinking it should be closed. If it turns out its something that is wanted then it can be replicated if needs be. Thoughts/objections?

Qiskit Optimization has now been moved/refactored into its own repository https://github.com/Qiskit/qiskit-optimization As such, apart from bug fixes that are deemed necessary, the code here is in effect frozen. If this function is still deemed worthwhile, given it has not been worked on for while, then an issue can be raised on the new repo to discuss ahead of any implementation - I'm noting that the details here were still under discussion.

Qiskit Optimization has now been moved/refactored into its own repository https://github.com/Qiskit/qiskit-optimization As such, apart from bug fixes that are deemed necessary, the code here is in effect frozen. If this function is still deemed worthwhile, given it has not been worked on for while, then an issue can be raised on the new repo to discuss ahead of any implementation - I'm noting that the details here were still under discussion.


Fixed conflicting module name in go.mod

current go.mod file has the module name github.com/Obayanju/paystack-go when it should be github.com/rpip/paystack-g0 because the repo exists on rpip's githu account

LGTM

@Obayanju can we please have this fixed?

@elvis-chuks  Thanks for opening a  PR! Sorry, this is coming late. I'll take a  look this weekend.

@metaclips Yes, I'll take a look this weekend.


Fix issue preventing lowering of PF's MTU

When the MTU set in the SRIOV Network Node Policy is lower than the actual MTU of the PF, it cannot be changed due to a faulty condition. This triggers the reconcile loop for the Node state indefinitely, preventing the configuration from completing.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 39.956%

Files with Coverage Reduction
New Missed Lines
%

controllers/drain_controller.go
1
68.06%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
5114

Relevant Lines:
12799

💛  - Coveralls

I see that there was a reason for never allowing to lower the PF's MTU: https://github.com/k8snetworkplumbingwg/sriov-network-operator/pull/489
@SchSeba @adrianchiris What's your take on this?
I see two options here: either allow to lower the MTU of the PF or also update the condition here

Thanks @almaslennikov. Another way to hit this issue is when the user manually increases the mtu after having configured a policy with a lower mtu. We have a test (should reconcile managed VF if status changes) in test_sriov_operator.go for the opposite case (manually decreasing mtu).
I think we should go for the second option, aligning NeedToUpdateSriov with which is actually being applied in order to avoid the loop. Please consider adding a test for this scenario as well.

IMO we should update NeedToUpdateSriov  so we support the use-case of a PF having a different (higher) MTU than VF as requested in spec.

agree if the MTU on the PF is higher the NeedToUpdateSriov should return false

/test-e2e-all

Hi @almaslennikov please check the unit-tests

/test-e2e-all

/test-e2e-all

LGTM


fix: remove unused function

Can the linter check for this?

Can the linter check for this?

In theory, yes. I used (Vulture)[https://github.com/jendrikseipp/vulture]. It's not 100% reliable, so I would advise against it.


ControlsFX Notification doesn't work with AtlantaFX

Hi!
During the development process, I noticed that after connecting AtlantaFX styles, Notification from the Controls FX packages did not work.
мая 02, 2024 2:11:12 AM javafx.scene.CssStyleHelper calculateValue
WARNING: Caught 'java.lang.ClassCastException: class java.lang.String cannot be cast to class javafx.scene.paint.Paint (java.lang.String is in module java.base of loader 'bootstrap'; javafx.scene.paint.Paint is in module<EMAIL_ADDRESS>of loader 'app')' while converting value for '-fx-background-color' from rule '*.notification-bar>*.pane' in stylesheet jar:file:///C:/Users/jahng/.m2/repository/org/controlsfx/controlsfx/11.1.2/controlsfx-11.1.2.jar!/org/controlsfx/control/notificationpopup.css

Perhaps someone has already encountered this error, if you can tell me how to fix it (which style element to replace or correct), it would be great.
If you can tell me how to use the Notification provided in AtlantaFX without binding to any Root, and how it is presented in the ControlsFX package, I will be doubly grateful :)

ControlsFX uses Modena (JavaFX bundled theme) colour variables. These are not present in the AtlantaFX stylesheet, hence the error.
If you want to display desktop notifications like ControlsFX, you can refactor/adapt NotificationPopupHandler. It basically creates a Popup, puts a notification node into it and displays it using calculated screen coordinates depending on the selected position.