🐛 [firebase_messaging] Push notifications not working on Android - Different issues

Bug report
Describe the bug
Since two weeks I try to setup push notifications for iOS and Android devices. On iOS everything works like a charm. I receive backend and in app notifications as well. But for Android I can't get it to work. I receive neither backend nor in app notifications.
If I try to get a background notification, I get the error:
I/flutter ( 9518): FlutterFire Messaging: An error occurred in your background messaging handler: I/flutter ( 9518): type 'List<dynamic>' is not a subtype of type 'List<String>'
... and the app crashes.
If I try to get an In App Notification the onMessage.listen event not fires. But in the console I get a broadcast received for message with the json object of my notification.
Also the error: java.lang.IllegalArgumentException: Service not registered: lt@c879c4d
I'm sending the notification inside a Firebase Cloud function like this:
const payload: admin.messaging.MessagingPayload = {
    notification: {
        titleLocKey: "new_entry_title",
        bodyLocKey: "new_entry_desc",
        titleLocArgs: `["${enteredUserName}", "${enteredUserID}", "${guestbookID}", "new_entry_added"]`,
        bodyLocArgs: `["${enteredUserName}"]`,
        badge: "1",
        sound: "default",
    },
}

return fcm.sendToDevice(userToken, payload, {
    priority: "high",
});

Steps to reproduce

Setup push notifications like to official example
Try to send a notification with the setup above on an Android device

Expected behavior
I expect that the push notifications are working for both devices with one code base

Additional context
I followed the official migration guide to use the new version of firebase_messaging
(https://firebase.flutter.dev/docs/migration)

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 1.22.4, on macOS 11.0.1 20B29 darwin-x64, locale de-DE)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.52.0)
[✓] Connected device (2 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.10.4
Flutter SDK 1.22.4
virtual_guestbook 1.0.0+8

dependencies:
- cached_network_image 2.3.2+1 [flutter flutter_cache_manager octo_image]
- carousel_slider 2.2.1 [flutter]
- cupertino_icons 0.1.3
- dio 3.0.10 [http_parser path]
- easy_localization 2.3.3 [flutter intl shared_preferences args path flutter_localizations]
- file_picker 2.0.10 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface]
- firebase_core 0.5.3 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_database 4.4.0 [flutter firebase_core]
- firebase_messaging 8.0.0-dev.11 [meta flutter firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web]
- firebase_storage 4.0.1 [flutter firebase_core]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_colorpicker 0.3.4 [flutter]
- flutter_inapp_purchase 3.0.0 [http meta flutter platform]
- flutter_inner_drawer 0.5.7+2 [flutter]
- flutter_launcher_icons 0.8.1 [image args yaml path]
- flutter_localizations 0.0.0 [flutter intl characters collection meta path typed_data vector_math]
- flutter_secure_storage 3.3.5 [meta flutter]
- font_awesome_flutter 8.10.0 [flutter]
- gallery_saver 2.0.1 [flutter path_provider http path]
- get 3.17.1 [flutter]
- get_storage 1.3.2 [flutter path_provider get]
- image 2.1.18 [archive xml meta]
- image_gallery_saver 1.6.4 [flutter]
- intl 0.16.1 [path]
- lottie 0.5.1 [flutter archive characters charcode collection logging meta path vector_math]
- modal_bottom_sheet 0.2.0+1 [flutter]
- multi_image_picker 4.7.14 [flutter meta]
- painter 0.4.0 [flutter]
- path_provider 1.6.24 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- persistent_bottom_nav_bar 2.1.0 [flutter]
- pin_code_fields 5.0.1 [flutter]
- provider 4.3.2+2 [flutter nested collection]
- pull_to_refresh 1.6.2 [flutter]
- share 0.6.5+2 [meta mime flutter]
- uuid 2.2.2 [crypto convert]
- video_compress 2.1.0 [flutter]
- video_player 0.11.1+2 [meta video_player_platform_interface video_player_web flutter]
- zefyr 0.12.0 [flutter collection url_launcher quill_delta notus meta quiver_hashcode]

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

transitive dependencies:
- archive 2.0.13 [crypto args path]
- args 1.6.0
- async 2.5.0-nullsafety.1 [collection]
- boolean_selector 2.1.0-nullsafety.1 [source_span string_scanner]
- characters 1.1.0-nullsafety.3
- charcode 1.2.0-nullsafety.1
- clock 1.1.0-nullsafety.1
- collection 1.15.0-nullsafety.3
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- fake_async 1.2.0-nullsafety.1 [clock collection]
- ffi 0.1.3
- file 5.2.1 [intl meta path]
- firebase_core_platform_interface 2.1.0 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.1+1 [firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_messaging_platform_interface 1.0.0-dev.7 [flutter meta firebase_core plugin_platform_interface]
- firebase_messaging_web 0.1.0-dev.2 [flutter flutter_web_plugins meta firebase_core firebase_core_web firebase_messaging_platform_interface js service_worker]
- flutter_blurhash 0.5.0 [flutter meta]
- flutter_cache_manager 1.4.2 [flutter path_provider uuid http path sqflite pedantic clock file rxdart]
- flutter_plugin_android_lifecycle 1.0.11 [flutter]
- flutter_web_plugins 0.0.0 [flutter characters collection meta typed_data vector_math]
- http 0.12.2 [http_parser path pedantic]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- js 0.6.2
- logging 0.11.4
- matcher 0.12.10-nullsafety.1 [stack_trace]
- meta 1.3.0-nullsafety.3
- mime 0.9.7
- nested 0.0.4 [flutter]
- notus 0.1.5 [collection meta quill_delta quiver_hashcode]
- octo_image 0.3.0 [flutter flutter_blurhash]
- path 1.8.0-nullsafety.1
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+4 [flutter]
- path_provider_platform_interface 1.0.3 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+1 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.2 [meta]
- petitparser 3.1.0 [meta]
- platform 2.2.1
- plugin_platform_interface 1.0.3 [meta]
- process 3.0.13 [file intl meta path platform]
- quill_delta 1.1.3 [collection quiver_hashcode]
- quiver 2.1.3 [matcher meta]
- quiver_hashcode 2.0.0
- rxdart 0.24.1
- service_worker 0.2.4 [js]
- shared_preferences 0.5.12+4 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- shared_preferences_linux 0.0.2+4 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+11 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+3 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- sky_engine 0.0.99
- source_span 1.8.0-nullsafety.2 [charcode collection path term_glyph]
- sqflite 1.3.1+1 [flutter sqflite_common path]
- sqflite_common 1.0.2+1 [synchronized path meta]
- stack_trace 1.10.0-nullsafety.1 [path]
- stream_channel 2.1.0-nullsafety.1 [async]
- string_scanner 1.1.0-nullsafety.1 [charcode source_span]
- synchronized 2.2.0+2
- term_glyph 1.2.0-nullsafety.1
- test_api 0.2.19-nullsafety.2 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0-nullsafety.3 [collection]
- url_launcher 5.7.2 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos url_launcher_windows]
- url_launcher_linux 0.0.1+1 [flutter]
- url_launcher_macos 0.0.1+8 [flutter]
- url_launcher_platform_interface 1.0.8 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.4+1 [url_launcher_platform_interface flutter flutter_web_plugins meta]
- url_launcher_windows 0.0.1+1 [flutter]
- vector_math 2.1.0-nullsafety.3
- video_player_platform_interface 2.2.0 [flutter meta]
- video_player_web 0.1.4 [flutter flutter_web_plugins meta video_player_platform_interface]
- win32 1.7.3 [ffi]
- xdg_directories 0.1.2 [meta path process]
- xml 4.5.1 [collection convert meta petitparser]
- yaml 2.2.1 [charcode collection string_scanner source_span]

Is this a known bug?

Not sure if you need this but hopefully it helps.
If you are using cloud functions, and sending a payload that has a 'notification' param in it, the backgroundmessagehandler in Flutter Android will NOT be called.
Example:
const payload = {
            notification: {
                title: titleString,
                body: bodyString,
            },
            data: {
                id: id,
                type: 'type'
            },
        };

^ DOES NOT WORK
CORRECT:
const payload = {
data: {
id: id,
type: type,
title: titleString,
body: bodyString,
},
};
^ WORKS
My flutter side implementation is just as the Readme is on the firebase_messaging pub site.
Thanks

CHECK THIS:
https://firebase.google.com/docs/cloud-messaging/send-message#android_specific_fields

Thanks very much @danielgchap... removing the notification key and adding the data key instead works in general. The app does not crash..and I receive the print() inside the background notification event.
But: The notifications for iOS are not working anymore. Does iOS not need a notification key to show a push notification? If yes, how could I get it work for both platforms? Android needs data, iOS needs notification.. on addition I need the  LocKeys for multilingual notification.
Do I understand something wrong here?

Thanks very much @danielgchap... removing the notification key and adding the data key instead works in general. The app does not crash..and I receive the print() inside the background notification event.
But: The notifications for iOS are not working anymore. Does iOS not need a notification key to show a push notification? If yes, how could I get it work for both platforms? Android needs data, iOS needs notification.. on addition I need the LocKeys for multilingual notification.
Do I understand something wrong here?

Hey @drosa88 how goes my friend.
Yes, I've been working on this a few hours now and got it working great.
What you are sending in the payload from cloud_functions really matters. Here is what I have in mine (with redactions).
Def read the link I put above about specified fields because the keys, etc, in the payload are important.
Here's one of mine as an example:
const payload = {             notification: {                 title: titleString,                 body: bodyString,             },             android: {                 data: {                     id: id,                     type: type,                     title: titleString,                     body: bodyString,                     "click_action": "FLUTTER_NOTIFICATION_CLICK",                     "priority": "high",                 },             },             data: {                 id: id,                 type: type,                 title: titleString,                 body: bodyString,             },             topic: topicString,         };
*** I got rid of my background message handler cuz I don't need it and when you get a notification in Android tray, you open and onResume should pick it up
*** This would NOT work until I added:
"click_action": "FLUTTER_NOTIFICATION_CLICK",
                    "priority": "high",

*** I'm pretty sure my other 'data' object in the payload is redundant but this is just from me playing around,
*** Make sure that your topicString is set, and that you include it in the payload
*** Make sure you are not using 'sendToTopic' but you are using
       admin.messaging().send(payload);
Hope that helps more!

Thanks for your help.
This is my code to send a notification from Firebase Cloud Functions:
const payload: admin.messaging.Message = {
    android: {
        data: {
            clickAction: "FLUTTER_NOTIFICATION_CLICK",
            priority: "high",
        },
    },
    apns: {
        payload: {
            aps: {
                alert: {
                    title: "Test",
                    body: "Body Test",
                    titleLocKey: 'new_entry_title',
                    titleLocArgs: [enteredUserName, enteredUserID.toString(), guestbookID.toString(), 'new_entry_added'],
                    locKey: 'new_entry_desc',
                    locArgs: [enteredUserName],
                },
                sound: "default",
                badge: 1,
            }
        }
    },
    token: userToken
}

return admin.messaging().send(payload);

iOS: Everything works wonderful
Android: The notification still does not appear. But interesting: The badge on app icon sums up! I followed the Example: notification message with localization options section here: https://firebase.google.com/docs/cloud-messaging/send-message#android_specific_fields
By the way.. I´ve added the following pull request into my dependencies because of the error above: https://github.com/FirebaseExtended/flutterfire/pull/4382
Console output when I move the app to background and trigger a notification:
D/FLTFireMsgReceiver(12559): broadcast received for message
D/ActivityThread(12559): SVC-Creating service: CreateServiceData{token=android.os.BinderProxy@c284c56 className=io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingBackgroundService packageName=de.danielederosa.virtualguestbook intent=null}
D/ActivityThread(12559): SVC-Calling onStartCommand: io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingBackgroundService@4eee4d7, flags=0, startId=1
D/ActivityThread(12559): BDC-Calling onReceive: intent=********, receiver=com.google.firebase.iid.FirebaseInstanceIdReceiver@d8f48ad
D/ActivityThread(12559): SVC-Destroying service: io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingBackgroundService@4eee4d7
D/ActivityThread(12559): SVC-Creating service: CreateServiceData{token=android.os.BinderProxy@32a9630 className=io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingService packageName=de.danielederosa.virtualguestbook intent=null}
D/ActivityThread(12559): SVC-Calling onStartCommand: io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingService@c04c2a9, flags=0, startId=1
D/ActivityThread(12559): SVC-Destroying service: io.flutter.plugins.firebase.messaging.FlutterFirebaseMessagingService@c04c2a9

@drosa88 you beast!
I think you are so close. If I can help more let me know is but I think you got this. Thank you!

@drosa88
What does your onResume show when you receive notification on iOS but not on Android ?

It works now. The issue is almost ridiculous :-)
The problem was my strings.xml file on which I save some strings for localizing the notifications. I used a wrong syntax for string params. Instead of using %1$s I used %1$d which is a decimal number instead of a string. This caused the app to crash while receiving the notification.
So everything is fine in general. The only problem I have: I get a silence notification on Android while the App is in background. The badge sums up and I see the notification in the status bar. But I get no popup when the notification was received.
This are my current settings:
const payload: admin.messaging.Message = {
    android: {
        data: {
            clickAction: "FLUTTER_NOTIFICATION_CLICK",
            priority: "high",
        },
        notification: {
            titleLocKey: "new_guest_title",
            titleLocArgs: [enteredUserName, enteredUserID.toString(), (guestbookID?.val()).toString(), "new_guest_added"],
            bodyLocKey: "new_guest_desc",
            bodyLocArgs: [enteredUserName],
            priority: "high",
            channelId: "high_importance_channel"
        },
    },
    apns: {
        payload: {
            aps: {
                alert: {
                    titleLocKey: "new_guest_title",
                    titleLocArgs: [enteredUserName, enteredUserID.toString(), (guestbookID?.val()).toString(), "new_guest_added"],
                    locKey: "new_guest_desc",
                    locArgs: [enteredUserName],
                },
                sound: "default",
                badge: 1,
            }
        }
    },
    token: userToken
}

return admin.messaging().send(payload);

By the way: With this settings I also don't get foreground notifications because the onMessage event was not triggered. The channelId: "high_importance_channel" is new. I'm playing around with this.
In my main.dart file I create this channel like that:
const AndroidNotificationChannel channel = AndroidNotificationChannel(
  'high_importance_channel', // id
  'High Importance Notifications', // title
  'This channel is used for important notifications.', // description
  importance: Importance.max,
);

const AndroidInitializationSettings initializationSettingsAndroid =
    AndroidInitializationSettings('app_icon');
final InitializationSettings initializationSettings = InitializationSettings(
    android: initializationSettingsAndroid, iOS: null, macOS: null);

final FlutterLocalNotificationsPlugin flutterLocalNotificationsPlugin =
      FlutterLocalNotificationsPlugin();

  await flutterLocalNotificationsPlugin
      .resolvePlatformSpecificImplementation<
          AndroidFlutterLocalNotificationsPlugin>()
      ?.createNotificationChannel(channel);

AWESOME! Good job.
In regards to your 'no popup' Android notification... I don't get one either tbh with you. At this point, I was so disappointed in the lack of concern for backwards comp in these pubs, that I was fine with it.
Sorry. Great job!

@drosa88
Good to know you found the root-cause and resolved the issue. Since your original issue for push notifications not working on Android is now resolved, can this be closed ?
Regarding the other issue you mentioned The only problem I have: I get a silence notification on Android while the App is in background. The badge sums up and I see the notification in the status bar. But I get no popup when the notification was received., you may have to open new issue providing all required details, since tracking that in this issue could lead to confusion.

Thanks very much for your help @danielgchap :-)
@darshankawar Well. We can close this issue. Thank you.

My pleasure. Thank you!!


⚠ AGEG website (redirect) has degraded performance

In 2bd9f74, AGEG website (redirect) (https://ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2558 ms

Resolved: AGEG website (redirect) performance has improved in cb82ad3.


[Bug]: GameCardName component in dialogs will sometimes overlap other elements

Bug Summary
The GameCardName component will sometimes overlap other text in dialogs instead of pushing them aside to make sufficient space
Detailed Description
See image below:

In this picture both the 9♦  and 10♠ are overlapping the other text. Instead, they should push the other text to create enough space for the cream-colored background on the GameCardName without overlapping the other text.
Current Behavior
The GameCardName background will overlap other text on smaller device sizes
Expected Behavior
The GameCardName does not overlap the text and both the text and the card names are legible
Steps to Reproduce

Enter a game
Play a royal
Have opponent play a 2 on your royal
Shrink your window size down so that the dialog text wraps
Observe the GameCardName component overlapping the other text

Hey Can i work on this one ?

@alexKawaljen Yes go for it!


Grammatically incorrect sentence, difficult to understand

The following sentence is included in one of the "note" boxes on this page:

For users that there mailbox is hosted onprem it is expected to have presence delays with a maximun of an hour.

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

ID: a162ef5c-3099-7f58-4178-d29e3ec2b5a7
Version Independent ID: 6ca31e56-052d-6c62-1323-b5aaaae1f04a
Content: User presence in Teams - Microsoft Teams
Content Source: Teams/presence-admins.md
Service: msteams
GitHub Login: @msdmaguire
Microsoft Alias: dmaguire

@Reeced40

@tremeur29 Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Make Lori capabilities secure

Currently no auth is required to start up Listeners or open outgoing connections.

An interesting problem with this.
I'd like to reuse the same capabilities that stdlib net but can't easily as I'd need to bring in the entire package.  I think I'll create primitives with the same names for now as I'm not worried about interop. Still it would be nice to have "universal" capabilities for certain operations in Pony that could be used across different libraries.

It seems like the correct place on TCPConnectionActor to apply the capability is on open(). However, open() is currently used when starting a connection from TCPListenerActor so there would need to be something like _from_accept() that calls into another private method that has what open() currently does so that...

open() requires an auth
TCPListenerActor can call "_from_accept() without an auth
And we still get the current open() functionality


Fix misc breakage

Add clearfix on header-main and header-info
Proper btn-primary class for "Add a Linode"
Checkbox checked use boolean always instead of optional inclusion

Opens #158 to deal with username/logout section.

Coverage remained the same at 79.181% when pulling 1a16558fd680b76529e0e1db17d95bc410466f8f on eatonphil:master into cac780add8f5f7562b1df40a384908bd1deec4a3 on linode:master.

thank you!


feat: Code-as-config for golang

Codecov Report

Merging #56 (c8e2966) into develop (726004d) will decrease coverage by 6.64%.
The diff coverage is 28.40%.

@@             Coverage Diff             @@
##           develop      #56      +/-   ##
===========================================
- Coverage    57.91%   51.27%   -6.65%
===========================================
  Files           23       23
  Lines         1067     1217     +150
===========================================
+ Hits           618      624       +6
- Misses         381      527     +146
+ Partials        68       66       -2

Flag
Coverage Δ

unittests
51.27% <28.40%> (-6.65%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/utils/gomodules.go
0.00% <0.00%> (ø)

pkg/runtime/javascript.go
32.25% <5.71%> (ø)

pkg/runtime/typescript.go
44.87% <8.82%> (ø)

pkg/runtime/python.go
54.76% <16.66%> (ø)

pkg/runtime/golang.go
28.82% <28.82%> (ø)

pkg/runtime/java.go
50.79% <33.33%> (ø)

pkg/build/build.go
35.86% <50.00%> (-0.50%)
:arrow_down:

pkg/runtime/types.go
86.20% <86.20%> (ø)

pkg/stack/options.go
45.94% <100.00%> (ø)

Continue to review full report at Codecov.

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


AudioStore development

As per #311, integrating audio into CC Catalog will require a few different things. This issue hopes to keep track of everything relevant to the development of AudioStore.
(NEW) means a field that was not in ImageStore that was added to AudioStore
Current schema (in audio.py):

Foreign ID
Landing URL
Audio URL
File format (NEW)
Thumbnail URL
Filesize
Duration (NEW)
Samplerate (NEW)
Bitdepth (NEW)
Channels (NEW)
License (like ImageStore, license and license version)
Creator
Creator URL
Title
Collection/Album (NEW)
Type (music, podcast, lecture) (NEW)
Genre (NEW)
Language (NEW)
Metadata
Tags
Provider
Source

Todo (https://github.com/creativecommons/cccatalog/pull/311#pullrequestreview-372895546):

Date

Inappropriate columns for AudioStore that are in ImageStore:

Width
Height
Watermarked

I am uncertain about the "type" field (as in music, podcast, lecture) and if it could be combined with genre in some way, or if we should keep it separate. What do you think @mathemancer ?

Date field blocked by #324

@mariuszskon I think Genre and Type are different.  We could probably come up with a list of 'allowed' types, e.g., music, podcast, lecture.  Then, within music, we'd have the genres:  rock, pop, classical, jazz, etc.  On the other hand, in the 'podcast' type, for genre, we'd have: 'science', 'comedy', 'news', etc.

@brenoferreira Do you have any guidance or ideas based on what you'd like to have available to the front end here?

Based on some of the designs here, it seems very comprehensive to me.
The frontend probably won't look exactly like this, but it'll probably be inspired by it. The only thing missing from the metadata list here are things like music instruments and mood. But I think that's ok

@mathemancer Indeed I see the distinction much more clearly, and it is certainly worth having them separate.

@brenoferreira instruments and mood can be added no problem, but I am concerned that audio sources might not have this information available for ingestion.

yes. that's what I meant. If the source has the info, great. But if not, it
should be fine for now
On Mon, Mar 16, 2020 at 11:33 PM Mariusz Skoneczko<EMAIL_ADDRESS>wrote:

@brenoferreira https://github.com/brenoferreira instruments and mood
can be added no problem, but I am concerned that audio sources might not
have this information available for ingestion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/cccatalog/issues/322#issuecomment-599846329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMTSYST7AKUKEJPCGGO4LRH3ON7ANCNFSM4LG32CIQ
.

--
Thank you
Breno Ferreira
Front-end Engineer
Creative Commons


No data is shown when using the widget renderer from an IDE to a browser

What is the intended/supported behaviour when running the widget renderer from a script in an IDE?
Minimal example: the widget renderer example in the documentation with an added .show() on the last line (since just chart does nothing in non-notebook context).
When trying this from an IDE (e.g. PyCharm), the plot is served on localhost and a browser tab is automatically opened, but no data is received by it (i.e. the 3 plot areas are there but empty).
A warning for the data not being found is issued (although the mentioned file IS there on inspection):
Displaying chart at http://localhost:59133/
WARNING:tornado.access:404 GET /_vegafusion_data/vegafusion-29a8824c1c6cebef5b2f03c6cc02b4539175a65d.feather (::1) 0.00ms

If a data folder is specified (i.e. vf.enable_widget(data_dir = ...) at the beginning of the example), then the no-data behaviour persists but no warning is issued.
Is this behaviour expected and/or correctable?
As a separate note, the example works "fine" (very slowly) with altair_data_server (i.e. replacing vf.enable_widget() with alt.data_transformers.enable('data_server')), thus not pointing to the typical causes of no-data being shown mentioned in the altair documentation.
Versions:

VegaFusion 1.2.4
Altair 4.2.2 (as 5.x currently has compatibility issues with altair_viewer https://github.com/altair-viz/altair_viewer/issues/50 and altair_data_server https://github.com/altair-viz/altair/issues/3048, as well as deprecation warnings and errors with this particular example)
Python 3.10.4 (conda env on Windows 11)

Hi @T-Flet, thanks for the question.
tl;dr The widget renderer is not compatible with altair_viewer, so what you're seeing is expected unfortunately.  One potential workaround would be to save the charts to a standalone HTML file with vf.save_html(chart, "chart.html") and then open the saved file in a browser (See https://vegafusion.io/save.html)
The widget renderer is based on a custom Jupyter Widget, so it will only work in contexts that are compatible with Jupyter Widgets. This includes the Jupyter Notebook, JupyterLab, Visual Studio Code notebook, Colab, and some other notebook environments. But AFAIK, there's not an easy solution for displaying Jupyter widgets in standalone browser windows from a Python script. What makes this tricky at a technical level is that Jupyter widgets require two-way communication between the Python kernel and the browser.
Could you add a little more detail on your desired workflow?

Are you running a Python script with an if __name__ == "__main__": block, or using Pycharm's notebook interface?
Are you hoping to run a script and have a single chart open in a browser at the end?
Do the charts you're building generally have aggregations (e.g. histograms, heatmaps, bar charts, etc) or not (e.g. large scatter plots).

As a workaround for aggregated charts or moderate sized scatter charts (a few tens of thousands of rows) the vf.save_html function should do a good job of evaluating the chart's transforms, removing unused data columns, and writing the result to a standalone HTML file.  Then to automatically open the file in a web browser, the Python webbrowser module could be used.

Thank you for the quick reply; perfectly reasonable and concise explanation.
Unfortunately, the two-way communication is precisely what I am looking for due to dataset sizes affecting plot responsiveness, therefore saving an html does not solve my issue.
My desired workflow is to have an interactive plot showing a slidable and x-y-resizable window of a small set of long (~500K+) time series (some overlaid, some in separate charts); ideally, multiple tabs containing separate such visualisations would be the goal.
Replying to your specific queries:

I was indeed trying the documentation example in a __main__ block in a PyCharm script, this as a prelude to using this setup in a large program which produces multiple datasets which I would like to inspect with these interactive visualisations (i.e. the larger program would call a plotting function for each of the datasets and hang until all plot windows are closed).
Running a script and opening one browser tab at the end is the starting point, but I would like to be able to open multiple ones.
No, none of the datasets require aggregation in their visualisation, only a sliding window so that not all the data needs to be visualised at once.

Altair + VegaFusion's widget renderer seems to be a perfect match for my preferences and needs (grammar-of-graphics plotting + interactivity and avoiding sending all data at once); is there a way to make the above work? If not an exact way, is there something close I could do?

Thanks for the additional details @T-Flet,
The use case of running VegaFusion outside of Jupyter with two-way communication wired up crosses into the territory of a full Python dashboarding toolkit.
Presently, I'm aware of 3 Python dashboarding toolkits that have some support for Jupyter Widgets: Voila, Panel, and Solara.
Currently, Voila works well with VegaFusionWidget, but it requires writing your dashboard in a Jupyter notebook. Panel depends in ipywidget_bokeh, which unfortunately doesn't have support for the binary messages that VegaFusionWidget uses (See https://github.com/bokeh/ipywidgets_bokeh/issues/46).  I tried out Solara yesterday, and ran into an issue, but @maartenbreddels already fixed it in Solara 1.16.1.
From what I understand about your use case (and about Solara), I think Solara is currently the best solution for running VegaFusionWidget from a standalone Python script.

Here's an example:
pip install "vegafusion-jupyter[embed]" solara==1.16.0 vega-datasets altair==5.0.1

Create a file named try_solara.py that constructs a VegaFusionWidget assigned to a variable named chart_widget.
from vega_datasets import data
import altair as alt
import vegafusion as vf

source = alt.UrlData(
    data.flights_2k.url,
    format={'parse': {'date': 'date'}}
)

brush = alt.selection_interval(encodings=['x'])

# Define the base chart, with the common parts of the
# background and highlights
base = alt.Chart(width=160, height=130).mark_bar().encode(
    x=alt.X(alt.repeat('column')).bin(maxbins=20),
    y='count()'
)

# gray background with selection
background = base.encode(
    color=alt.value('#ddd')
).add_params(brush)

# blue highlights on the transformed data
highlight = base.transform_filter(brush)

# layer the two charts & repeat
chart = alt.layer(
    background,
    highlight,
    data=source
).transform_calculate(
    "time",
    "hours(datum.date)"
).repeat(column=["distance", "delay", "time"])

chart_widget = vf.jupyter.VegaFusionWidget(chart)

Then launch it from the command line with:
solara run try_solara.py:chart_widget

And the nice thing about this approach is that you can use all of the built-in ipywidgets as well (like dropdowns, layouts, tabs, etc. See https://ipywidgets.readthedocs.io/en/stable/examples/Widget List.html).
See what you think!

Have you updated to vegafusion/vegafusion-jupyter 1.3.0?  I believe I saw a similar error to your solara error in 1.2.4 at one point, but that it was resolved in 1.3.0.
But now that I wrote this, I just checked and it looks like the conda-forge packages for vegafusion-jupyter 1.3.0 didn't go through. I'll have to look into that.

Ok, vegafusion-jupyter 1.3.0 is now on conda-forge. I don't know that this will fix the solara issue you're seeing, but worth a try.

I am afraid both vegafusion and vegafusion-jupyter were/are 1.3.0, as they were pip-installed (though in a conda env).

I force-reinstalled vegafusion-jupyter and now it works on solara. I did re-check it was 1.3.0 before doing so, just as a sanity check, and it was. I do not know what changed, and I unfortunately did not run it today before the force-reinstall, therefore I do not even know whether that was indeed the solution.
(Not relevant now since it works, but adding verbose = True produced logs of requests sent and times and sizes of responses every time I interacted with the plots).
(Yes, the example was exactly the one you posted.)
The weird no-plot behaviour in VS Code persists, but I am happy to close this issue and open another for it if you prefer (or deem it worthy of one).

That's great to hear! Yeah, I'll close this and you can open another issue on the VSCode issue. Thanks


Upgrade hpke dependency

As we prepared for in #311, we can now upgrade hpke. I've also added zeroization to the serialization/deserialization routines, since otherwise there might be some private key material left on the stack.

Sorry for taking so long to review this. Thank you so much for updating this for me and continuing to support your implementination! Great catch on zeroing the key material on the stack.
The automated builds are broken due to things completely unrelated to these changes. I've verified everything works locally.
Thank you again!

Happy to help!


Turbostat dependency

requires recent version of turbostat
turbostat 2020.03.20 --> KO
turbostat 2024.05.10 --> OK
turbostat needs to be compiled as autonomous binary from Linux Kernel power tools
Note that turbaostat requires libcap-dev

Maybe @ErwanAliasr1 can enlighten us on this, but the tool need (it does not currently) to support more versions if possible, unless some required metrics are not provided.
I would expect hwbench to crash because of a different output format from turbostat

Yeah the parsing is changing pretty often.
I think that could be handled at the same time as issue #11 by having a version check at startup time.

Added a note in https://github.com/criteo/hwbench/pull/14

Solved in https://github.com/criteo/hwbench/pull/20


Stray empty line in umls.txt

The UMLS concordance starts with a single blank line (causing trouble for the kgx transformer)

This is the blank line I add so that Snakemake doesn't time out waiting for the output file to be created >.< I will look into a better way of doing this.

could you touch the file which makes an empty (no newline character) file?

Yup, touching the file instead of adding a blank line! Thanks. Closing.


link navigation and page traversal

This hooks up @cvan's hyperlink work with @dmarcos cursor.
navigate to hyperlinks.html to in action.

WIP, do not merge yet.    fixing a few issues first.

OK, this is ready @dmarcos r?

Can we leave the cube, space and cube2 examples simple as they are and create new examples to illustrate the links?

Would it make sense to create a vr-frame component based on VRLoader that it's an iframe on steroids?

@caseyyee I agree with dmarcos; let's leave cube, cube2 and space as they were so that the examples directory stays relatively clean, and move our demos into a new demo folder. I can take care of that.

OK guys, all the VR handling code has been moved into the components along with the Enter VR button.   The example code is now even cleaner and simpler.


status of this project?

I see that the last commit was in 2017. Is this project still maintained or should it be considered EOL?

tag @vvakame

this project is not active.

I made this repository to archived status. thanks!


[FLINK-24862][Connectors / Hive]Fix user-defined hive udaf/udtf cannot be used normally in hive dialect

What is the purpose of the change
Fix user-defined hive udaf/udtf cannot be used normally in hive dialect
Brief change log

FunctionCatalog#validateAndPrepareFunction method skip validate TableFunctionDefinition

Verifying this change
This change added tests and can be verified as follows:

Added unit test HiveDialectITCase#testTemporaryFunctionUDAF for create temporary udaf function

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot  run azure

@xiangqiao123 Thanks for contribution. I'm glad to review when i'm free.

@rmetzger @Tartarus0zm Thank you for your review.
I have modified the logic to skip validation: skip validate if the input is not instance of UserDefinedFunction.

And for the second question you mention in FLINK-24862 .
I think we don't need to modify the logic of HiveGenericUDTF.
You can implement your UDTF by overriding method public StructObjectInspector initialize(ObjectInspector[] argOIs) just like HiveGenericUDTFTest.TestSplitUDTF does.

Since hive 0.13, many hive UDTF implementations have not  overrided method public StructObjectInspector initialize(ObjectInspector[] argOIs)，and this method is deprecated.
Therefore, flink needs to be compatible with the use of this udtf.

Ok, It makes sense to me.

@xiangqiao123 Thanks for contribution.
We usually solved one problem in one issue.
Could you split this pr into two?

@beyond1920 ok, i have splited this pr into FLINK-24862 and FLINK-24862

@luoyuxia Thank you. I have addressed the comments.

LGTM. @wuchong @JingsongLi  Would you like to help review/merge?


Music track manager

Current status:

MusicTrackManager with a single audio source connected to the music source in the main audio mixer, without cross-fade or multiple layers
Menu and battle music (the latter is very WIP a placeholder while I write some new music)
Fixed music source in mixer not being wired up to the options menu (!)

To do:

[x] Music track scriptable objects
[x] Basic track manager
[x] Fade before track switch (logarithmic AudioMixer fade paraphrased from this tutorial)
[x] Play entire battle theme once, then switch to looping version
[x] Audio mixer groups for the layers
[x] Play layers of music
[ ] Fade in/out layers

Saved for later:

Apply effects to layers (e.g. in response to shooting trumpeters)

Resolves #34

Looks good to me! sunglasses
The tracks sound nice (even for placeholders) and the music control system looks good and runs as expected after testing.

Thanks!

The sound scale in the options menu seem to drop off a bit too fast at the moment, but I would say that is outside the scope of this PR anyway.

I'll look into that. We might need to do some logarithmic shenanigans here as well.


customPositionToValue on Keyboad event

Hi there,
I defined a customPositionToValue for the slider which works just fine when used with mouse drag. But if I use the keyboard to select a slider value the customPositionToValue function seems to be ignored.
I have created a demo on JSFiddle: https://jsfiddle.net/czery/mwy45dxt/
I guess this is not an expected behavior, is it? If so, does anyone have a clue on how to fix this?
Cheers and thanks in advance
czery

customPositionToValue should be used along with customValueToPosition which do the reverse job. However, these functions only work with a constant step, while in your example, your step changes.

Hm, I think I did not get that right... What you mean with "reverse job"? When do I get a value which will be transformed into a position? When will the one or the other function be executed?

The slider to be rendered needs to be able to map a value to a screen position and also (to handle mouse events) it needs to be able to map a screen position to a value.
See the "Custom scale" example on the website.
However, in your case , this won't be enough since you don't have a constant step.

Thanks for the information, @ValentinH
So far so good, I have updated my Fiddle with the "customValueToPosition" function (and I do understand that this will not help me with my issue): https://jsfiddle.net/czery/mwy45dxt/
Is there any way to define a listener for the keyboard event so that I could update the value based on the closest value of the given array? Something like onKeyboardChange: setValue(value.getClosest(stepsArray))
Cheers
czery

No there's only the onChange callback where you can do this. But you won't differentiate keyboard and mouse.

To close this: I managed to resolve my issue by using onStart and onChange to detect if keyboard press was "up" or "down" and then compare the new values against my input array.
Here ist the JSfiddle: https://jsfiddle.net/czery/mwy45dxt/

Thanks for the feedback. Your solution work but is kind-of hackish. Also if you press the keyboard quickly, the result is bugged.
I found a solution by using the stepsArray function and adapting the custom scale functions.
Demo: https://jsfiddle.net/Ltprx6e7/
The important part of the code is the line like val = $scope.stepsArray[val]. Indeed, when using a stepsArray, internally, we manipulate the indexes of the array. Thus the val, minVal and maxVal args passed to the customValueToPosition and the returned value of customPositionToValue are the indexes in the array.

Awesome! Yep, that seems to be cleaner implemented :)
Thanks for your support.


Get SQLx connection pool

PR Info

Closes https://github.com/SeaQL/sea-orm/issues/753

New Features

[x] Added seaql-internal feature
[x] Added DatabaseConnection::get_*_connection_pool() method to borrow the inner SQLx connection pool

Please merge the conflict, I think it is a reasonable middle ground between 'we don't leak implementation details' and what users are asking for

umm although why seaql-internal not sea-orm-internal?

umm although why seaql-internal not sea-orm-internal?

I changed it into sea-orm-internal now


Accidental Jitsi calls take forever to cancel

When a user accidentally presses the call button, a Jitsi widget is added and they are brought to a fullscreen interface for it. Users end up tapping the close button a lot, but the widget doesn't close until Jitsi is done setting up. It should be possible to cancel the call quicker than that.

@turt2live
Did you succeed to reproduce it ?

Yes


Integrating Stable Diffusion v1-4 with UAgents

This example showcases the integration of Stable Diffusion with uAgents. Feedback and further suggestions are welcome!

Please update with the latest main branch to prepare for merge.

done! please check now.


Steam Transport Unreliable Channels not working

Describe the bug
Steam Transport only supports up to 1200 bytes per package in unreliable mode. That number is provided via GetMaxPackageSize() but it still fails to connect. If the NetworkWriter buffer size is changed from 1500 to 1200, it apparently works.
To Reproduce
Steps to reproduce the behavior:

Open the project ( https://www.dropbox.com/s/tzngavyenb35553/SteamBug.zip?dl=1 ), Scene "NetworkTest"
Connect to another peer. Steam needs to be running on both clients. The Transport shows the SteamID that can be used as a connection string after a server has been initialised.
Connect. (It should fail as the channel is configured as unreliable). If you set Breakpoints, you will see that there are packages that Mirror is trying to send that are 1500 byte in size.

Expected behavior
As GetMaxPackageSize() provides 1200 for Unreliable, it should configure the buffer size accordingly before any communication happens.

Regardless of the transport preconfiguring the packet size, it should be throwing an error if the packet is too large as it checks that beforehand, however it is not. Also ServerSend send is returning false as the send is failing due to packet size being too high, this false return should also probably be throwing an error. I spent hours tracing where the issue was coming from and that would have saved a lot of time.

@jasonsurratt found out a way to solve this. While @deltapilot 's comment may still be valid, I am closing this issue!


Never processes the challenge.

Challenge Fill in the Blank with Placeholder Text has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2652.0 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Thanks and happy coding!

Cheers, I feel dumb now, I did double check and look over it, but sometimes 1 letter mistakes can be missed rather easily.
Do you have any metrics on how many people drop the camp at each stage? Maybe some sort of spell check or something on this stage could help retain campers?


[RELAY][WIP] Add Relay evaluator and compiler for TVM runtime system.

This PR implements the evaluation machinery for Relay, including basic passes for lowering and transforming Relay programs to make them suitable for compilation.
Passes

[x] Monomorphization, is a phase similar to template specialization which tries to generate concrete, specialized versions of functions and operators for specific shapes.
[x] LowerOps, is a phase which collects the set of operations used by an expression and converts them into LoweredFunc using the appropriate TVM operator and schedule.
[ ] FuseOps, is a phase which implements operator fusion, and is still currently underdevelopment.

Evaluator
The evaluator is a simple interpreter for Relay programs, which can be used as its reference semantics.
The interpretation of the program is not well-optimized but each operator call uses TVM optimized operators on CPU. The value's of Relay are reflected in the node system allowing the evaluator to be used for testing, and experimentation.
TVM Graph Runtime.
This PR ships a compiler located in to_tvm.py which will convert a Relay program into a TVM graph, as well as using the above passes to generate a tvm::Module enabling users to deploy a subset of Relay programs to the TVM RTS.
Testing
The final piece left on this PR is to polish the tests, and improve operator coverage.

@eqy please help review this

@zhiics Tianqi has been pushing back on implementing a pass manager for the past couple weeks. He feels like we should wait until we have a few more passes and use cases before we go all in on designing the pass manager. I'm happy to build one sooner then later for various reasons which I'm happy to elaborate on.

@eqy @zhiics @tqchen @MarisaKirisame @slyubomirsky @joshpoll this is ready for review if you could all gives some eyeballs, @junrushao1994 @ZihengJiang ?

I am very interested in this PR. Will do tomorrow!

Thanks, @jroesch for the changes, I did a serious round of review.
Besides the changes I suggested, I would recommend us to deliberate about the API choices of the function evaluate. In particular, while it is understandable that env could be a useful thing, it is a bit annoying to users who usually only evaluate an Expr. This is the same reason behind why we put expr in the first argument in most ir_passes and makes env optional. Here is a possible alternative proposal:
env = relay.Interpreter(env) # env is optional
x = relay.var("x")
# evaluate a free expression.
env.evaluate(relay.add(x, x), binds={x: np.zeros((1,))})
# evaluate a function
env.evaluate(f)(np.zeros((1,)))

@junrushao1994 can you help review this, @zhiics @slyubomirsky please take another round of look, we would like to bring this in as it is one of the major enabler of full compilation

@yzhliu @ZihengJiang please also take a look

Thanks, @jroesch @junrushao1994 @joshpoll @zhiics @slyubomirsky @MarisaKirisame , this is now merged


SQL Logger error as reported on the Indigo forum

I think I have discovered the issue - it looks like it might be a problem with the SQL Logger.
The fields causing the problem are e.g. :

port_table_8_rx_bytes :<PHONE_NUMBER>7 (integer)
port_table_0_tx_bytes :<PHONE_NUMBER>6 (integer)
etc

Using Valentina Studio to interrogate the indigo device schema schema for the device, the field types for these fields are defined as integer which can hold a maxim value of -2147483648 to +2147483647. Those values listed above are larger.
The fields need to be of type bigint which can hold -9223372036854775808 to<PHONE_NUMBER>854775807.
Either that or your plugin needs to store them as strings.
I will try and do a fix by changing the filed types to bigint and see what happens. :)

Well, the plugin's not going to change the type.  I do an automatic state generation based on the JSON dict returned by the controller.  If the value in the dict is an int or float, it's going to be a Number type in the state.  Adding exceptions for specific fields isn't really practical.

I wonder if there's a way to reset that counter on a regular basis?

I proably wouldn't want to do that - this looks like an SQL Logger issue.

Changing it to bigint doesn't seem to resolve the issue, so while that might be a solution, it probably requires the SQL Logger to be fixed. Also it looks like I would have to change at least 16 fields (RX & TX) for my 8 port device. If users have 24 or 48 port devices then that would be a headache. Also, if you ever deleted the device, you would have to do all that work again - so not really practical. At the moment, I have temporarily had to disable the plugin as it is filling my log with pages of sql logger error messages. I am getting a screen full for every error as it lists out all the states.

And those are coming from the SQL Logger, so nothing I can do to suppress them.

Correct, I think.
It is the SQL logger trying to insert a record and updating an int field with too big a value (for an int field).
Probably one to fire at Matt. Do you want to do it or should I?

I can't replicate it, so you should.

OK :)

Posted here: https://forums.indigodomo.com/viewtopic.php?f=98&t=24502


docs(@angular-cli): update documentation for ng new

for several options that are saved in .angular-cli.json, I've added an explanation of what setting in the json file controls that options.
added an explanation about what --dry-run will output.
added possible values for --style option

The motivation for this change is that sometimes people would want to change some of the settings set by the cli durig ng new, that they might have not known or cared about, for example, prefix.

Could you also update the command documentation as well?

@delasteve where can I find your docs standards for code?

I was merely asking to update the descriptions on the commands. For example, styles.
<EMAIL_ADDRESS>

Got it.

@delasteve not used to this whole github ecosystem (we use gerrit at work, not working open source), but I've managed to figure this whole deal out. updated the command descriptions as well

Can I get a review on this?

@sumitarora please take a look.

@sumitarora can I get a review on this? It would help me if this goes in, as I want to add similar docs in my other PR

@gioragutt LGTM, thanks!

@filipesilva when will this be merged to master? does this happen only on release? or is it something I have to do? I'm quite new to this, so I'd like some explanation if it's possible. And also any constructive criticism is always appreciated :)

@filipesilva just noticed this was merged. Thanks a lot!

It's in master already but will be published to the wiki next release.

@filipesilva actually, I noticed that the merged that is referenced here is another pull-request. I'm going to reopen the PR and push some updates I've not pushed yet.

Oh duh, sorry, yeah it was approved to be merged but hasn't been yet. My bad.

@filipesilva cool, I was worried for a second :P I'm pulling from master but nothing new is coming

Yes

@filipesilva by the way, i'm gonna update the commit - I've seen a tutorial on youtube where the dude uses lass as his styles, so I guess that's also worth mentioning in the docs


Adding a new advanced sample

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looking at this I do not see more than a new line added, can you explain the changes better?


User manual 0.7.1 section 10.7 -  example code has wrong shape type

In the example code under section 10.7 this is written:
// Create the sphere collision shape
rp3d::decimal radius = rp3d::decimal(3.0)
const rp3d::BoxShape shape(radius);

It says it wants to create a SphereShape but instead creates a BoxShape.
Also later on in the same example it uses shape like this: proxyShape = body->addCollisionShape(&shape, transform, mass); however addCollisionShape takes a non const pointer to shape so defining the shape as const won't compile.

The version v0.8.0 of ReactPhysics3D has been released. The user manual has been updated and this issue is now fixed.
Thanks a lot for reporting this issue.


kde-applications: 17.12.2 -> 17.12.3

Motivation for this change
I could not run nox-reviewas it always crash (the error was something with the garbage collector of nix) :(
@ttuegel @adisbladis @peterhoeg
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test plasma5

Failure on aarch64-linux (full log)
Attempted: tests.plasma5
Partial log (click to expand)
intproto.cpp: In member function 'INT_TEMPLATES_STRUCT* tesseract::Classify::ReadIntTemplates(FILE*)':
intproto.cpp:1011:12: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result [-Wunused-result]
       fread(&Class->font_set_id, sizeof(int), 1, File);
       ~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/bash ../libtool  --tag=CXX   --mode=compile g++ -DHAVE_CONFIG_H -I. -I..  -O2 -DNDEBUG -I../cutil -I../ccutil -I../ccstruct -I../dict -I../viewer  -I/nix/store/r1ravrfmqnsh2fbzadwnscwclyjrw3n8-leptonica-1.74.1/include/leptonica -I/nix/store/x3i8pz2h5k0f8bicxfpijq2l1v4k2rkr-pango-1.40.14-dev/include/pango-1.0 -I/nix/store/0v1df0w5y9hh1c522662wj0xgwsdc054-glib-2.54.3-dev/include/glib-2.0 -I/nix/store/0a7p1pxiv56fxnszw5nfrp8w5zcl6imd-glib-2.54.3/lib/glib-2.0/include -I/nix/store/sxyg4ydnaydyi7v5ik0nqrsh9iih0a7a-cairo-1.14.10-dev/include/cairo -I/nix/store/xy79vwmf7k14a75gkzwak1pdikpfqd0b-freetype-2.7.1-dev/include/freetype2 -I/nix/store/xy79vwmf7k14a75gkzwak1pdikpfqd0b-freetype-2.7.1-dev/include  -g -O2 -std=c++11 -c -o kdtree.lo kdtree.cpp
libtool: compile:  g++ -DHAVE_CONFIG_H -I. -I.. -O2 -DNDEBUG -I../cutil -I../ccutil -I../ccstruct -I../dict -I../viewer -I/nix/store/r1ravrfmqnsh2fbzadwnscwclyjrw3n8-leptonica-1.74.1/include/leptonica -I/nix/store/x3i8pz2h5k0f8bicxfpijq2l1v4k2rkr-pango-1.40.14-dev/include/pango-1.0 -I/nix/store/0v1df0w5y9hh1c522662wj0xgwsdc054-glib-2.54.3-dev/include/glib-2.0 -I/nix/store/0a7p1pxiv56fxnszw5nfrp8w5zcl6imd-glib-2.54.3/lib/glib-2.0/include -I/nix/store/sxyg4ydnaydyi7v5ik0nqrsh9iih0a7a-cairo-1.14.10-dev/include/cairo -I/nix/store/xy79vwmf7k14a75gkzwak1pdikpfqd0b-freetype-2.7.1-dev/include/freetype2 -I/nix/store/xy79vwmf7k14a75gkzwak1pdikpfqd0b-freetype-2.7.1-dev/include -g -O2 -std=c++11 -c kdtree.cpp  -fPIC -DPIC -o .libs/kdtree.o
building of '/nix/store/9gc04rcgzqwdq812vyyjyi77pyghwpa8-tesseract-4.00.00alpha-git-20170410.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/l27093v9i9yz775j6dfiis41yy8mflj2-nixos-test-driver-plasma5.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/gd82353c4my2p1y22rcgr6pj6rc2wcf6-vm-test-run-plasma5.drv': 1 dependencies couldn't be built
error: build of '/nix/store/gd82353c4my2p1y22rcgr6pj6rc2wcf6-vm-test-run-plasma5.drv' failed

Failure on x86_64-linux (full log)
Attempted: tests.plasma5
Partial log (click to expand)
cannot build derivation '/nix/store/73x7hc8811cn55sbwjbgd64kg3j28sqs-system-units.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/dr7yp28bl2nwc0wnnzzysgzgsxkmly4i-user-units.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/d510yjsb2lf03g5p9dnclh3vc1ml40q6-etc.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/s2r7kv6skfpnkhhnk2d4sf0gbm9kbvl0-nixos-system-machine-18.09.git.f4339d2.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/a8sby556jw7849ggind790g3h1l1ya8j-closure-info.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/n2sy2j84a07gqfyh2gvyf1h5wdkkn5ly-run-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/flbph4lj3rv7lfz9mdgzmb67pfx73001-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/i3z9ccanqg6wakwybjbxygvx5c0yzh24-nixos-test-driver-plasma5.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/mgvccy961ahlvym8jqbvi1swdgxz76zv-vm-test-run-plasma5.drv': 1 dependencies couldn't be built
error: build of '/nix/store/mgvccy961ahlvym8jqbvi1swdgxz76zv-vm-test-run-plasma5.drv' failed

On my local machine it worked. I think these sh changes that also broke qt, break plasma too.


Feature: add belongsTo() support

Because I don't want to see value, I want to see relation title. Please support it :)

Sorry I don't understand what you mean, can you give an example?

@milewski
In resource field, commonly I wrote:
BelongsTo::make('My Merchant', 'merchant', \App\Nova\Merchant::class),

My $title = 'display_name'; in Merchant resource class let it display as a sementic meaningful "display_name", not "ID", if I use  Column::make('Merchant ID', 'merchant_id')->checked(), it only allow me to use the actual value in the table, not the related "display_name".

This doesn't work?
Column::make('My Merchant', 'merchant')->checked()

This doesn't work?
Column::make('My Merchant', 'merchant')->checked()

No, merchant is the field of the table, but merchant display name, which I want to use, is not in the table, it's in the relation table that belongsTo() can work.

Im not really getting what you are saying... if you can post your full nova resource file.. would be easier for me to understand what you mean

I find a document may explain my need: https://nova.laravel.com/docs/2.0/resources/relationships.html#belongsto

I know how that belongs to works, what im not understanding is why this example is not working for you:
Column::make('My Merchant', 'merchant')->checked()

Because I use this on my projects and it works fine with relationships, the name of the column shown the column selection is exactly what the name you give to the first argument My Merchant


How to handle OPTIONS request?

i try to use itty-router and deploy it to cloudflare workers. but when there is an OPTIONS request from browser it always return 404 because of router.all('*', () => new Response('404, Not Found!', { status: 404 }));
how can we handle options request to always return 200 response?

thank you for this awesome router :+1:

i found a simple solution like this:

const handleOptions = (req) => new Response('');

const router = Router();

router.routes.push(['OPTIONS', /(.*)/, [handleOptions]]);

if anyone has another solution, please kindly post it here too. i will close this now, thanks.


xls format not supported message

https://github.com/QtExcel/QXlsx/blob/499901003f02da34f7fbc1ec214a24e4dc128b74/QXlsx/source/xlsxzipreader.cpp#L28
At this moment code
Document xlsxR("table.xls");
produce warning
QZip: not a zip file!
I suggest add extra message.

Someone try open .xls file
Program write "warning xls format doesn't support"

.xls is an old binary format and cannot be supported. Please close issue as unrelated

.xls is an old binary format and cannot be supported. Please close issue as unrelated

Please read my first message again. It's not about "let's support.xls", it's about let's improve error messages.


Hudi Write Performance

Concerned about performance. How long should the following mocked-up sample take to write to s3? There are 1,369,765 records and 308 columns. It is taking ~10.5min running in docker container on an t2.xlarge ec2 instance using the datamechanics/spark:3.2.0-hadoop-3.3.1-java-11-scala-2.12-python-3.8-latest image. Any suggestions how to increate performance.  The sample file generated below is just to illustrate our issue.
Steps to reproduce the behavior:

Start docker container
docker run -it datamechanics/spark:3.2.0-hadoop-3.3.1-java-11-scala-2.12-python-3.8-latest  /bin/bash

Download sample file
cd /tmp
wget https://s3.amazonaws.com/nyc-tlc/trip+data/yellow_tripdata_2021-01.csv

Start spark shell
/opt/spark/bin/spark-shell  --packages org.apache.hudi:hudi-spark3-bundle_2.12:0.8.0,org.apache.spark:spark-avro_2.12:3.0.1,org.apache.hadoop:hadoop-aws:2.7.3   --conf 'spark.serializer=org.apache.spark.serializer.KryoSerializer'  --driver-memory 16g

run the following code(replace {bucket_} with a valid bucket):

mport org.apache.spark.sql.SaveMode
import org.apache.spark.sql.functions._
import org.apache.hudi.DataSourceWriteOptions
import org.apache.hudi.config.HoodieWriteConfig
import org.apache.hudi.hive.MultiPartKeysValueExtractor
import org.apache.hudi.QuickstartUtils._
import scala.collection.JavaConversions._
import org.apache.spark.sql.SaveMode._
import org.apache.hudi.DataSourceReadOptions._
import org.apache.hudi.DataSourceWriteOptions._
import org.apache.hudi.config.HoodieWriteConfig._
import org.apache.spark.sql.expressions.Window
import org.apache.spark.sql.functions.row_number
var df = spark.read.option("header","true").csv(file:///tmp/yellow_tripdata_2021-01.csv);
var a = 0;
var b = 0;
// Just constructing a table for testing.
var cols = df.columns;
var num_cols = cols.length;
// duplicating colums to make a larger dataset
for( a <- 1 to 16; b <- 0 to num_cols-1){
     var col_name = cols(b);
     var new_col_name = col_name + "_" + a;
     df = df.withColumn(new_col_name, col(col_name));
};

// going to written to one partition
val w = Window.partitionBy(lit('A')).orderBy(lit('A'))
var df_id = df.withColumn("_id", row_number().over(w)).withColumn("partpath", lit('N'))
val tableName = "hudi_test"
val basePath = "s3a://{bucket_}/hudi_test_table"
val starttime = System.nanoTime
df_id.write.format("hudi").
option(PRECOMBINE_FIELD_OPT_KEY, "_id").
option(RECORDKEY_FIELD_OPT_KEY, "_id").
option(PARTITIONPATH_FIELD_OPT_KEY, "partpath").
option("hoodie.datasource.write.operation","upsert").
option("hoodie.datasource.write.table.type","COPY_ON_WRITE").
option(TABLE_NAME, tableName).
mode(Overwrite).
save(basePath)
val duration = (System.nanoTime - starttime) / 1e9d
print("write time:" + duration )
Expected behavior
Not sure if this is the expected performance that we can expect with this instance size. Or if there are suggestions on how to increase the performance.
Environment Description

Hudi version : 0.8.0 / 0.9.0

Spark version : 3.0.1

Hive version :

Hadoop version : 3.3.1

Storage (HDFS/S3/GCS..) :

Running on Docker? (yes/no) : yes

@codope Thanks for the response. Strange I get 333s when I call the parquet writer directly using the exact snippet provided. Should hudi write and parquet write be similar in performance?  Really new to hudi and not sure if there are some optimization I can make.

Ok. I tried with latest Hudi master. Can you give build from the latest master?
I'll try Hudi 0.9. I think latest EMR already has it.

@codope  I built from master and takes 492 secs. Still seems slow.
We have an internal file(2.6m rows ~300col) takes 16min to load into a new table(one partition). If we dump the same df to parquet(gzip) using pandas it takes 2m 4secs.
should df_id.write.format("parquet").mode(Overwrite).save(parquetBasePath) times be similar to pandas parquet write times?

@p-powell : for immutable use-cases, we recommend setting some configs to get better performance.
https://hudi.apache.org/docs/performance#bulk-insert
let us know if you are looking for more assistance. would be happy to help.


[Feature Request] Automatically select definition in lsp_definitions

After calling FzfLua lsp_definitions on a symbol with one definition, I believe the sole entry should be automatically selected and cursor moved to it (e.g. behave exactl like vim.lsp.buf.definition().

One of the oldest request, take a look at #13.

FYI, it’s also found in the wiki: LSP: jump to location for single result, although incomplete with much room for improvement it has good information for power users, worth taking a look.


[bug] int to number broken

Describe the bug
Prior workflows that used int to number node were working. Now ComfyUI can't find the node.

Reproduction
No response
Expected behavior
No response
Operating System
Linux
Comfy Mode
In a custom virtual env (venv, virtualenv, conda...)
Console output
[comfy_mtb] | INFO -> loaded 71 nodes successfuly
[comfy_mtb] | INFO -> Some nodes (2) could not be loaded. This can be ignored, but go to http://None:8188/mtb if you want more information.

got prompt
Failed to validate prompt for output 181:
* Any To String (mtb) 179:
  - Return type mismatch between linked nodes: input, INT != *
Output will be ignored
Failed to validate prompt for output 180:
* Any To String (mtb) 178:
  - Return type mismatch between linked nodes: input, INT != *
Output will be ignored

Additional context
There are 2 nodes that fail to load, but neither one is int to number, while int to number is just missing completely now.

Thanks for the report, I removed them thinking they weren't used, seems like I was wrong. I'll add them again


Whitespaces should be allowed in user story titles and not lead to an error message

Story
AS A Developer or Customer
I WANT TO Be able to use whitespaces in user story titles
IN ORDER TO Write Descriptive User Story Titles

Expected Behavior
When naming a User Story with whitespaces it should just save like it usually does.
Actual Behavior
It creates a "method/user story" with the title error
Steps to reproduce error
Create a user story in the created user story class and use whitespaces in the title.
Estimations:
Jascha: Since this requires us to check into how the parser works I'd say at least a 5.

5

Estimation: 8

8


feat: add PreEventCallback

Description:
This PR introduces the preEnableIdentityCallback and preCreateIdentityCallback to the ClientOptions object to be called before enabling and creating an identity.
This feature is the starting point of this issue

Do you mind writing up a detailed issue body so future developers can understand the changes made here?

@nplasterer
The purpose of this pull request is to add the optional methods preCreateIdentityCallback and preEnableIdentityCallback, which are received through the ClientOptions and are invoked in the create method. These methods are executed just before creating an identity in the case of preCreateIdentityCallback when calling wallet.createIdentity(), and in the case of preEnableIdentityCallback, they are called both when encrypting or decrypting a keyBundle.
This is necessary for integration with React Native, as on the JavaScript side, we may be listening to events (exposed here) that are triggered once these callbacks are invoked.

Thanks @kele-leanes for the explanation and working code in RN. I'll give this another pass later today on the tests and get it merged for you.


⚠ Bofrost.de has degraded performance

In 4b50323, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2285 ms

Resolved: Bofrost.de performance has improved in d4ddca9 after 8 minutes.


Show properties not ids when joint relations

Describe the problem feature solves
Sometimes it is preferred to see the name or another property (that is not id) of a joint entity to avoid this, and get more human friendly properties to guide ourselves by:

Describe the solution you'd like
I'm using typeorm and nestjs and I am not sure that a solution to this already exists, to reference another field of an entity rather than the id in many-to-one relations.

This is defaulted to: title, name, email, subject if such fields are present in your entity, otherwise primary key is used:
https://github.com/SoftwareBrothers/adminjs/blob/a0d2a17d55af39e03cf3b3f5ab8cd5c09bcaaeb6/src/backend/adapters/property/base-property.ts#L22
You can define your title property by adding isTitle: true in the property which you want to use as title (in your referenced resource options):
https://docs.adminjs.co/PropertyOptions.html#isTitle

Oh, I see! I was looking at the docs all wrong. Thanks!
Also, is there a way tell a dropdown that in case a title is not available use id? Right now it just renders nothing although it is a valid option:

I don't think you can do that at the moment


savepoint monkey-patching is obsolete

With savepoint support coming in 20.1, the customizations this library does around savepoints are no longer needed (for the new crdb version). They seem to break nested transactions.
At least:
https://github.com/cockroachdb/activerecord-cockroachdb-adapter/blob/f14037d3d9d7dbe5eb1ea1f46ef8eb3ca5031107/lib/active_record/connection_adapters/cockroachdb_adapter.rb#L126
https://github.com/cockroachdb/activerecord-cockroachdb-adapter/blob/f14037d3d9d7dbe5eb1ea1f46ef8eb3ca5031107/lib/active_record/connection_adapters/cockroachdb/transaction_manager.rb
Btw, maybe this library is not needed at all any more? I couldn't tell what's in in.

sorry, wrong repo. Not sure how I got here. Opened https://github.com/cockroachdb/activerecord-cockroachdb-adapter/issues/42


[FEATURE] implement all reconstructions within processing step

Problem
Currently, we implement following compute in the processing step of the pipeline:

QLIPP: pixel-wise reconstruction of retardance and deconvolution of phase from brightfield.
Fluorescence Deconvolution: deconvolution of fluorescence alone
PhaseFromBF: deconvolution of phase from brightfield stack alone.
If QLIPP or PhaseFromBF modes are chosen, fluorescence deconvolution is considered a post-processing step.

This has been a) confusing to the user, and b) affects the efficiency of compute and data i/o.
A specific example of this is #81.
Proposed solution
The user should be presented a simple and cleaner set of choices.
One choice is  between 2D vs 3D reconstruction. Most experiments involve imaging a sample that is either thinner than the depth of field or thicker than the depth of field. The spatio-temporal dimensions are consistent across the channels, most of the time.
The second choice is between types of reconstructions. User may need any or all of the following reconstructions.

Retardance and Orientation: pixel-wise or deconvolution.
Phase from Brightfield.
Fluorescence Deconvolution.

Under the hood, the pipelines need to be optimized to enable the implementation of above reconstructions in arbitrary combinations. Some of the above modes should be disabled if the relevant channels are not found in the input.
An idea for clean implementation is that

user can run only one type of reconstruction from UI at a time if the relevant input channels are present in the zarr store.
new channels can be added to an existing zarr store.

Alternatives you have considered, if any
To develop efficient pipelines along the above lines, @talonchandler and @lihaoyeh  have written scripts using waveorder reader, writer, and compute modules. These scripts are also amenable to parallelization with multiprocessing. Once these scripts are developed, we can incorporate them in the pipeline.

Closing. These features are now present on the main branch of recOrder.


match_algorithm preference list

Prefer fzy if it exists, otherwise use fzf:
match_algorithm = { "fzy", "fzf" },

Yeah good idea, but since we only have 2 algorithms, what we should probably do is just fallback to the other one at this point.


Command responses don't match command requests

Hi,
I tried to use this lib on production and got many strange errors
await r.publish(topic, val)
...
response from redis 'OK'
exception: Unable to map 'OK' to int

val = await r.get(self.key())
...
response from redis 1
exception: Unable to map 1 to AnyStr

await r.publish(topic, val)
...
(in the Sentry stack trace)
await self.create_request(b"HELLO", *hello_command_args, decode=False)
...
exception: only (P)SUBSCRIBE / (P)UNSUBSCRIBE / PING / QUIT allowed in this context

also, I had many list index out of range errors.
Any suggestions what it could be and if I can fix it? It looks like responses from other commands.
I used the lib for set, get and publish, subscribe commands
r = await Redis.from_url(
      redis_addr,
      max_connections=10000,
      decode_responses=True,
      protocol_version=2,
)

async def save():
    await r.set('key', 'val', 300)

async def subscribe():
    pubsub = r.pubsub()
    await pubsub.subscribe('topic')
    try:
        while pubsub.subscribed:
            msg = pubsub.get_message(ignore_subscribe_messages=True, timeout=10)
            yield msg
    finally:
        pubsub.close()

This looks pretty interesting/bizarre. Could you share the details of the redis server that you are connecting to?

I use aws elasticache redis 5.0.6 cluster mode off. Also, I use Sanic as webserver.
before I used aioredis lib and didn't have this issue.
I think I don't use the pool connections properly. I used context manager for aioredis:
async with client as conn:
    await conn.set('key', 'val', 300)

Does coredis have something similar?

For the commands you shared in your sample there are some api usage errors.

set only takes two positional arguments (key & value). Everything else should be keyword arguments, I think in your case you want to use ex=300? (reference)
PubSub.get_message() is an async method so you'll need an await. (reference)

5.x should work with RESP2 (which you correctly specified).
coredis doesn't have a similar concept of an async context manager to explicitly reserve a connection from the pool, every time you issue a command with the coredis.Redis client a connection will be reserved for sending the command and released back as soon as possible (in most cases before the response is received so that it can be reused).

exception: only (P)SUBSCRIBE / (P)UNSUBSCRIBE / PING / QUIT allowed in this context
That error implies that somehow the connection reserved by the pubsub listener ended up getting reused for the publish method - which should not be possible but clearly is.

thanks for the fast responses. I use Sanic for graphql websockets and I guess the issue is related to incorrect task canceling during websocket closing.
I'll try to update or change the web framework.

I use Sanic for graphql websockets and I guess the issue is related to incorrect task canceling during websocket closing.

I'm actually pretty curious how the client got into this state because it looks like there definitely is a bug in the cleanup/connection pool reuse code under certain conditions. Is there some sample code that you have which I could use to reproduce this myself?

@shananin just curious, was the issue resolved for you? And if so what change did you have to make?


Remove/Replace Sns's Writer turn type PCollectionTuple

Imported from Jira BEAM-7562. Original Jira may contain additional context.
Reported by: cmach.

Duplicate of https://github.com/apache/beam/issues/19783


XS_AUTHORIZATION_ADMIN authorization error

We are able to launch the "launchpad" app but got the following error when tried to check the prerequisites. We are receiving this authorization error even though the role collection " XS_AUTHORIZATION_ADMIN" has been added to the user through XSA_ADMIN.
"An error occurred while checking your authorization. Please assign the XS_AUTHORIZATION_ADMIN role collection to your user and retry"
anyone facing this issue?

name: uaa
properties:
url: 'https://host:30032/uaa-security'
--

Same here. It seems not possible to create the role collections. When I look into the console, I got the following error messages:

Running on HANA2 SPS4 Rev. 45 and deployed the shine application via git.

I recognized that the build of the user-xsjs node was always creating a xsuaa service with a generated name, e.g. XSA_DEV-8qkhdn0gmoaxjl9o-hana-shine-xsa-shine-uaa and not using the shine-xsuaa I have created as described in the manual. Therefore I deleted this generated service instance and created a new one with the same name and uploaded the xs-security.json from the shine project when creating the service. This solved the issue for me.


feat: add boost.core

add boost.core
need skip url stability check

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (boost.core) have been updated in this PR. Please review the changes.


ypeError: Cannot read property '0' of undefined (row: 11, file: "Code")

I applied all your guide on Scprit Editor.
But when I run the module, It shows "loading"
`{
module: "MMM-GoogleSheets",
header: "Google Sheets",
position: "top_right",
config: {
url: "https://script.google.com/macros/s/AKfycbw8yIKosRs-BZooJ7cFKG4ppCvImCjmgikIZI0ke0EsIs4I6zs/exec",
sheet: "Sheet1",
range: "A1:B1"
}
}

`
What is the problem ? Could you please help me

My Version is :
"name": "magicmirror",
"version": "2.12.0",

@emrahaslan when I run magic mirror with that exact config it works fine. Are you seeing any errors besides it saying "loading" all the time? Are your other modules running okay?

All running fine
On Wed, Jul 29, 2020 at 16:46 ryan-d-williams<EMAIL_ADDRESS>wrote:

@emrahaslan https://github.com/emrahaslan when I run magic mirror with
that exact config it works fine. Are you seeing any errors besides it
saying "loading" all the time? Are your other modules running okay?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ryan-d-williams/MMM-GoogleSheets/issues/4#issuecomment-665986733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGZLIEFXLXDXBC3DPFATXTR6CYG3ANCNFSM4PL6UBIQ
.

I noticed it was @amanzimdwini who commented not @emrahaslan. @emrahaslan let me know if you are still seeing issues.

Yes my other modules are working fine.
It is always says loading.
I tried to solve ans debug this 4 hours but no result
I want to understand where problem is ç, google script or magic mirror
Please help me

Now It's works with white background ! I have to work on stylesheets. thank you


Emojis don't display on Google Chrome in Windows

Hey guys,
Thank you so much for this github - its amazing. I ran into an issue though my emojis are displaying perfectly on my apple computer using chrome. But, on my windows 7 using chrome it comes as black and white or sometimes just an empty box was wondering if anyone has a solution for this. Thank you so much!

what do you see here?

Here they all look perfect :) how can I do this on my end?

by &#127880 you meant &#1F388;, right ? with a semicolon after and a number that is HEX one, right?

or better .. check this source code out:
https://github.com/twitter/twemoji/blob/gh-pages/2/test/preview.html
it's the page you say that it looks perfect.

Thank you so much mate!! This was amazing :smile:

glad to help, too bad GitHub doesn't use twemoji 😉

How did you solve emojis issue on windows ?


Retry to obtain db connection during mojo execution.

 The startup process might be asynchronous and Flyway Maven Plugin will crash immediately.
Solution:
Added an optional parameter 'maxRetries'. The plugin will try to obtain connection every 1 second till connection will be obtained or 'maxRetries' reached. When 'maxRetries' is reached, an exception will be thrown. The parameter 'maxRetries' has default value 0, that means retries won't be performed.

Thanks for the pull request. This is however something that when implemented, should be part of the core and not the Maven plugin. Also it should probably be solved transparently inside a datasource.

You're right. Shall I implement this? Maybe you can guide me in terms of the best place where it should be.
Thanks for your time.


no MBean interface

Just finished implementing HikariCP in an app to replace BoneCP.  The pool configures and runs successfully, but as soon as I try and turn on the JMX monitoring, I see the following.
[2015-06-27 12:00:56.756] [WARN] HikariMBeanElf - Unable to register management beans.
javax.management.NotCompliantMBeanException: com.zaxxer.hikari:type=PoolConfig (ZC_DBA) mbean has no MBean interface for class com.zaxxer.hikari.HikariDataSource
at com.caucho.jmx.AbstractMBeanServer.createMBean(AbstractMBeanServer.java:441)
at com.caucho.jmx.AbstractMBeanServer.registerMBean(AbstractMBeanServer.java:415)
at com.zaxxer.hikari.pool.HikariMBeanElf.registerMBeans(HikariMBeanElf.java:67)
at com.zaxxer.hikari.pool.BaseHikariPool.(BaseHikariPool.java:170)
at com.zaxxer.hikari.pool.HikariPool.(HikariPool.java:60)
at com.zaxxer.hikari.pool.HikariPool.(HikariPool.java:48)
at com.zaxxer.hikari.HikariDataSource.(HikariDataSource.java:80)
at com.zaneray.foundation.dbconnection.HikariCPPool.connect(HikariCPPool.java:79)
[12:00:53.700] {main} Windows 8.1 6.3 amd64
[12:00:53.700] {main} Java(TM) SE Runtime Environment 1.8.0_45-b15, UTF-8, en
And this is using the most recent 2.3.8 release.
(edit)
I suppose this could simply be an issue with an outdated version of Resin also, not the pool or the JVM

As a follow up, I just downloaded the snapshot HikariCP-2.4.0-rc3-20150615.073114-1.jar and do not have this issue.  the JMX Beans are registered correctly.  I'll use that version to continue testing.

Thank you for the report, I will investigate.

Hi,
Using rc4. From this morning, started to get this:
[2015-07-01 09:54:23.330][               ][DEVELOPMENT][WARN ][com.zaxxer.hikari.pool.PoolElf          ][292 ] - Unable to register management beans.
javax.management.NotCompliantMBeanException: MBean class com.zaxxer.hikari.pool.HikariPool does not implement DynamicMBean, and neither follows the Standard MBean conventions (javax.management.NotCompliantMBeanException: Class com.zaxxer.hikari.pool.HikariPool is not a JMX compliant Standard MBean) nor the MXBean conventions (javax.management.NotCompliantMBeanException: com.zaxxer.hikari.pool.HikariPool: Class com.zaxxer.hikari.pool.HikariPool is not a JMX compliant MXBean)
	at com.sun.jmx.mbeanserver.Introspector.checkCompliance(Introspector.java:176) ~[na:1.8.0_40]
	at com.sun.jmx.interceptor.DefaultMBeanServerInterceptor.registerMBean(DefaultMBeanServerInterceptor.java:317) ~[na:1.8.0_40]
	at com.sun.jmx.mbeanserver.JmxMBeanServer.registerMBean(JmxMBeanServer.java:522) ~[na:1.8.0_40]
	at com.zaxxer.hikari.pool.PoolElf.registerMBeans(PoolElf.java:285) ~[HikariCP-2.4.0-rc4-SNAPSHOT.jar:na]
	at com.zaxxer.hikari.pool.HikariPool.<init>(HikariPool.java:137) [HikariCP-2.4.0-rc4-SNAPSHOT.jar:na]
	at com.zaxxer.hikari.HikariDataSource.<init>(HikariDataSource.java:73) [HikariCP-2.4.0-rc4-SNAPSHOT.jar:na]
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) [na:1.8.0_40]
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) [na:1.8.0_40]
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) [na:1.8.0_40]
	at java.lang.reflect.Constructor.newInstance(Constructor.java:422) [na:1.8.0_40]
	at org.springframework.beans.BeanUtils.instantiateClass(BeanUtils.java:147) [spring-beans-4.1.7.RELEASE.jar:4.1.7.RELEASE]
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:122) [spring-beans-4.1.7.RELEASE.jar:4.1.7.RELEASE]
	at org.springframework.beans.factory.support.ConstructorResolver.autowireConstructor(ConstructorResolver.java:267) [spring-beans-4.1.7.RELEASE.jar:4.1.7.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.autowireConstructor(AbstractAutowireCapableBeanFactory.java:1139) [spring-beans-4.1.7.RELEASE.jar:4.1.7.RELEASE]
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1042) [spring-beans-4.1.7.RELEASE.jar:4.1.7.RELEASE]
...
...

This is fixed.  For now, you can continue to use rc3.

Thank you Brett! :-)
-=david=-
On 1 July 2015 at 15:18, Brett Wooldridge<EMAIL_ADDRESS>wrote:

This is fixed. For now, you can continue to use rc3.
—
Reply to this email directly or view it on GitHub
https://github.com/brettwooldridge/HikariCP/issues/334#issuecomment-117691431
.

--
I prefer encrypted and signed messages. KeyID: B20A22F9
Fingerprint: 110A F423 3647 54E2 880F ADAD 1C52 85BF B20A 22F9
"It is not usually until you've built and used a version of the program
that you understand the issues well enough to get the design right." - Rob
Pike, Brian Kernighan.
No trees were harmed in the sending of this message, however, a number of
electrons were inconvenienced.

v2.3.9 is now available with this fix.

Hi Brett!
Thank you for the update :-) I'll wait until 2.4.0 comes out (or the next
RC of 2.4.0). I'm not scheduling a release for a few weeks at least... :-)
-=david=-
On 11 July 2015 at 01:33, Brett Wooldridge<EMAIL_ADDRESS>wrote:

v2.3.9 is now available with this fix.
—
Reply to this email directly or view it on GitHub
https://github.com/brettwooldridge/HikariCP/issues/334#issuecomment-120556191
.

--
I prefer encrypted and signed messages. KeyID: B20A22F9
Fingerprint: 110A F423 3647 54E2 880F ADAD 1C52 85BF B20A 22F9
"It is not usually until you've built and used a version of the program
that you understand the issues well enough to get the design right." - Rob
Pike, Brian Kernighan.
No trees were harmed in the sending of this message, however, a number of
electrons were inconvenienced.


Append cluster name to natgateway names

Towards https://github.com/giantswarm/roadmap/issues/3294
Why do we need this breaking change?
https://kubernetes.slack.com/archives/CEX9HENG7/p1720180477056679
Trigger e2e tests
/run cluster-test-suites

Upgrade tests are supposed to fail.
/run cluster-test-suites

Upgrade tests are supposed to fail.
/run cluster-test-suites


Need a screen to indicate wait for the shell to open when launching kubectl

Need a UI indicate while waiting for shell to open when launching kubectl

Message already exists in the window in the upper right. The message was quite dark though and not called out, so I changed the color to the warning text color which highlights it a lot more. If we'd like to address the design further maybe we should do it post 2.5?

ON rancher-master-head - commit id: 4ac3a87 ui tag: latest2

Dark mode

Light mode:

I'm going to leave the connecting message yellow, this is how the dashboard functions when showing the connecting message in the shell. I believe it pulls the users attention to the message. After connecting it will show the message in black, again this is how the dashboard functions.

On master-head - commit id: 27000be
Behavior is seen as described here - https://github.com/rancher/rancher/issues/28569#issuecomment-699245604


fix 2527, customize page name

#2527

[ ] I've read and followed the contributing guide on how to create great pull requests.
[ ] I've updated the relevant documentation for any new or updated feature.
[ ] I've linked relevant GitHub issue with "Closes #".
[ ] I've added a visual demonstration in the form of a screenshot or video.

@apedroferreira May I ask you how I can update this image ?

I ran this but  didn't any happen

We're solving those warnings in https://github.com/mui/mui-toolpad/pull/2920
Would you mind updating the documentation and mention this new option? https://mui.com/toolpad/concepts/page-properties/

We're solving those warnings in #2920
Would you mind updating the documentation and mention this new option? https://mui.com/toolpad/concepts/page-properties/

ok no problem

 @apedroferreira May I ask you how I can update this image ?

This is fine, it just means that this page changed.
I can approve these changes on Github as part of the review process, I'm just waiting for the rest of the changes:

Show the page display names in the sidebar navigation
Move displayName to node.attributes
Update documentation

 @apedroferreira May I ask you how I can update this image ?

This is fine, it just means that this page changed. I can approve these changes on Github as part of the review process, I'm just waiting for the rest of the changes:

Show the page display names in the sidebar navigation
Move displayName to node.attributes
Update documentation

I got  it ,I'll fix it late

@apedroferreira I'm going to write the document after the code is fully correct.

next week I'll write down, this week I'm little busy

just be sure.
does it need to display display name in preview??

@apedroferreira I wrote the doc，maybe it's not very good, welcome point out any problems

just be sure. does it need to display display name in preview??

Yes, in any mode the sidebar should should the page display names - that's the main intention of this feature.


fix: double click is required for opening reaction details

🎯 Goal
With the new Angular reaction details UI it took two clicks on mobile to open the reactions (first was to open message options, second to open the modal). To fix that clicking on a reaction icon won't open the message options.
🛠 Implementation details
It uses the new :has selector which doesn't yet have complete browser support so we use the @supports syntax  to only use the selector in browsers that support it. For other browsers the users will have to click twice to open the message reactions.
⚠ Autoprefixer might need to be updated in older applications that use the React/Angular SDK: https://getstream.slack.com/archives/C06CF5TKRGA/p1704989315231429
🎨 UI Changes
The message options not displayed when hovering over a reaction icon:
React sample app screenshots:
Before:

After:

Thanks for the review @MartinCupela


Document staking mechanism in substrate

The current staking mechanism spreads across multiple modules in the substrate.

Seating
Council
Staking

has dependencies for staking.
It will be nice to  have an overview of this

Will do this at same time as https://github.com/paritytech/substrate/issues/1709


Accordions, HEADING role.

Just like in web we expect accordions to have heading role. For disclosure group, we can achieve this by .accessibilityElement(children: .contain)
and then .accessibilityLabel("The disclosure group text").
This will add the heading role, to the disclosure group.

@MayurPatel0 I don't think that code actually works, can you build a demo with a VoiceOver screenshot to prove it does?

Closing until a demo can be provided as I don't think this code works.


Nested helpers in .hbs not supported.

While testing journey with 'paperleaf' theme, I found out that "Nested Helpers" are not working properly in comments like :
{{!-- jQuery needs to come before `{{ghost_foot}}` so that jQuery can be used in code injection --}}

It fires the following error: :
#### compileTemplateFromFile( /home/x/go/src/github.com/kabukky/journey/content/themes/paperleaf/default.hbs )
#### compileTemplate(name: default )
panic: runtime error: slice bounds out of range [855:774]

goroutine 1 [running]:
github.com/kabukky/journey/templates.findHelper(0xc000168500, 0x4d3, 0x4d3, 0xc0000f8800, 0x8, 0x8, 0xc000168500, 0x4d3, 0x4d3, 0x0, ...)
        /home/x/go/src/github.com/kabukky/journey/templates/generation.go:97 +0xb24

In this case, the faulty line is only a comment, so deleting it for the.hbs solves the problem.
Olivier

The Casper theme also causes this panic. If fixing this bug isn't a priority, perhaps the docs should be updated not to recommend installing Casper.

Hi.
Is the project still maintained and alive ?
I can't see much commits in the past day/month
Thanks


Marker colours are hard to discern during the videos

See screenshot:

it's hard to tell the difference between bronze and silver tokens

in 360p this is worse

Fixed by #389.


In select componen, can't enter text in search text

Describe the bug
The 6.6.8 version worked fine for me, but since I update to 6.11.21, I can't enter text in search text  in select component.
Also, in www.blazor.zone/selects I don't see "OnSearchTextChanged" in the list of events, anymore.
I'm using multiselect component and it works fine for typing text. (the only problem with this component is that when I select a check box, the whole line becomes colored and the color of check boy is equal to the color of the whole line)
To Reproduce
<Select TValue="string"
Items="serialParametersNew"
PlaceHolder="Not selected"
OnSelectedItemChanged="ParameterSelectedChange1"
@bind-Value="@ParameterSelected1"
ShowSearch="true"
OnSearchTextChanged="@OnSearch"></Select>

Exceptions (if any)
I wish I can simply type and search for items.
Further technical details

BootstrapBlazor version 6.11.21
.Net 6.0
The IDE: VS 2022, 17.3.4

@Irajjj thanks for contacting us. please try the latest version v6.11.22


Support transparency at the semantic layer

Prerequisite for unions, since Flatbuffers has many union-specific hardcoded limitations that are very annoying for us.
Fixes #2526

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested demo.rerun.io (if applicable)

PR Build Summary
Docs preview
Examples preview

This PR removed fbs.scalars.Float32. However, structure like Angle (union { float, float}) requires such a wrapper over a float, as flatc requires unions to only contain table/struct/string. Was this an oversight or is there another way to handle Angle and similar?

This PR removed fbs.scalars.Float32. However, structure like Angle (union { float, float}) requires such a wrapper over a float, as flatc requires unions to only contain table/struct/string. Was this an oversight or is there another way to handle Angle and similar?

You can use private local types + transparency whenever you hit one of flatbuffers artificial limitations (including but not limited to scalars in unions), e.g. for Angle you can do something like this:
struct _Float32 (transparent, order: 0) {
  value: float (order: 0)
}

union Angle (order: 100) {
  degrees: _Float32 (transparent, order: 1),
  radians: _Float32 (transparent, order: 2),
}


🚀 Feature: Scaffolder task list permissions

🔖 Feature description
The scaffolder task read permissions do not allow the creation of complex permissions. Currently, it's only on/off with the taskReadPermission, and no possibility to add conditions.
🎤 Context
We would like to limit the listed tasks so that:

The user can see his own runs
The template owner can see all runs of that template
If the user doesn't have read access to the template entity, the task should be hidden

The number 3 is not really that big of an issue as the template name seems to be hidden in the task list if the user does not have access to it but it still allows user to see the output + logs of the run.
✌ Possible Implementation
Add additional conditions for the taskReadPermission. At least the following rules should be available:

isTaskCreator - should filter based on the createdBy column using the current user entity ref
hasCreatedBy - should filter based on the createdBy column in the SerializedTask
hasTemplateEntityRef -  should filter based on the spec.templateInfo.entityRef in the spec column of SerializedTask. This requires some JSON query magic
hasTemplateAccess - should also filter by template entityRef but include only template references the user has access to. Probably requires a catalog call to get the entity refs.

Additionally, some other rules that could be done:

hasStatus - should filter by task status

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

PR #25969 is relevant however there are still some changes requested