Autotype not working on Garuda Linux

Sorry if this is another case of me not reading the documentation correctly, but from my understanding, once you select an entry in the autotype window, it's supposed to automatically type that out in the set text boxes, correct?
On my system, selecting and entering just closes the auto-type window. It does not type out anything.
I also noticed there are key shortcuts (by default, u for username and p for password) that can be used to copy either the username or password of the selected entry into the clipboard, but again, it does nothing on my system.
Video
https://youtu.be/VIrCcl2wDVs
Logs
flatpak
flatpak run com.quexten.Goldwarden
/app/bin/monitors/dbus_autofill_monitor.py:4: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '4.0') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk
IS daemon running False
running daemon
Failed err None
Flatpak Config directory: /home/zany130/.var/app/com.quexten.Goldwarden/config/goldwarden.json

[INF] [19:49] [Goldwarden > Keyring] >>> Creating new memguard keyring

[INF] [19:49] [Goldwarden > Agent] >>> Agent listening on /home/zany130/.var/app/com.quexten.Goldwarden/data/goldwarden.sock...

Blocking, press ctrl+c to continue...

[INF] [19:49] [Goldwarden > SSH] >>> SSH Agent listening on /home/zany130/.var/app/com.quexten.Goldwarden/data/ssh-auth-sock

[WRN] [19:49] [Goldwarden > Agent] >>> Could not monitor idle: org.freedesktop.DBus.Error.ServiceUnknown

autostart enabled..!?
(':1.15', '/org/freedesktop/portal/desktop/request/1_468/com/quexten/Goldwarden/13804143', 'org.freedesktop.portal.Request', 'Response', GLib.Variant('(ua{sv})', (0, {'background': <true>, 'autostart': <true>})), None)
/app/bin/settings.py:34: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  self.ssh_row.set_icon_name("emblem-default")
[INF] [19:49] [Goldwarden > Pinentry] >>> Asking for pin |Unlock Goldwarden|Enter the vault PIN|

[INF] [19:49] [Goldwarden > Pinentry] >>> Got pin from user

[INF] [19:49] [Goldwarden > Keyring] >>> Unlocking keyring with account key

[INF] [19:49] [Goldwarden > Auth] >>> Refreshing token

[INF] [19:49] [Goldwarden > Auth] >>> Refreshing using refresh token

[INF] [19:49] [Goldwarden > Auth] >>> Token refreshed

[INF] [19:49] [Goldwarden > Bitwarden API] >>> Performing full sync...

[INF] [19:49] [Goldwarden > Websocket] >>> Connected to websocket server...

[INF] [19:50] [Goldwarden > Bitwarden API] >>> Sync successful, initializing keyring and vault...

[INF] [19:50] [Goldwarden > Bitwarden API] >>> Reading  0 org keys...

[INF] [19:50] [Goldwarden > Bitwarden API] >>> Initializing keyring from user symmetric key...

[INF] [19:50] [Goldwarden > Keyring] >>> Unlocking keyring with account key

[INF] [19:50] [Goldwarden > Bitwarden API] >>> Clearing vault...

[INF] [19:50] [Goldwarden > Bitwarden API] >>> Adding 563 ciphers to vault...

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:50] [Goldwarden > Websocket] >>> Invalid message received, length too short

/app/bin/autofill.py:6: PyGIWarning: Notify was imported without specifying a version first. Use gi.require_version('Notify', '0.7') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk, Adw, GLib, Notify
/app/bin/autofill.py:80: DeprecationWarning: Gtk.Widget.hide is deprecated
  self.history_list.hide()
/app/bin/autofill.py:108: DeprecationWarning: Gtk.Widget.get_style_context is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
/app/bin/autofill.py:108: DeprecationWarning: Gtk.StyleContext.add_class is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
[INF] [19:50] [Goldwarden > Systemauth] >>> Checking permission for goldwarden with session type com.quexten.goldwarden.accessvault

[INF] [19:50] [Goldwarden > Biometrics] >>> Checking biometrics for com.quexten.goldwarden.accessvault

[INF] [19:50] [Goldwarden > Biometrics] >>> Biometrics result: true

[INF] [19:50] [Goldwarden > Systemauth] >>> Permission granted, creating session

[INF] [19:50] [Goldwarden > Pinentry] >>> Asking for approval |Approve List Credentials|zany130 on main.py>python3>goldwarden is trying access all credentials|

[INF] [19:50] [Goldwarden > Pinentry] >>> Got approval from user

/app/bin/autofill.py:64: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  action_row.set_icon_name("dialog-password")
/app/bin/autofill.py:46: DeprecationWarning: Gtk.Widget.show is deprecated
  self.history_list.show()
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.

(autofill.py:1320): libnotify-WARNING **: 19:51:03.787: Running in confined mode, using Portal notifications. Some features and hints won't be supported
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
enter
secure.hulu.com
[WRN] [19:51] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:51] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:51] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:51] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:52] [Goldwarden > Websocket] >>> Invalid message received, length too short

Operating System: Garuda Linux
KDE Plasma Version: 5.27.10
KDE Frameworks Version: 5.114.0
Qt Version: 5.15.12
Kernel Version: 6.7.0-3-cachyos (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 5600X 6-Core Processor
Memory: 31.3 GiB of RAM
Graphics Processor: AMD Radeon RX 6700 XT

Sorry if this is another case of me not reading the documentation correctly, but from my understanding, once you select an entry in the autotype window, it's supposed to automatically type that out in the set text boxes, correct?

Never place yourself at fault. If it's mis-understandable, then the documentation is bad ;)
Right now, when you hit enter on an entry, the autotype portal should pop up. What DE are you using? Hyperland or sway? Do those implement the remote desktop portal? (this is used to perform the auto-typing. I only know about KDE and Gnome having this implemented)

Ok, in your video it seems like you never hit enter. The documentation and ux here needs some work. If you do hit enter when having selected an entry, what happens?

Yes, I hit enter at the end of the video. It just closes the pop-up, and that's it.
Looking at the logs, this seems relevant
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
enter
secure.hulu.com

however I seem to have that file
file /run/user/1000/wayland-0
/run/user/1000/wayland-0: socket

file /run/user/1000/wayland-0
…wayland-0  …wayland-0.lock

Sorry, took me a while to circle back around to this, I saw you added a video to the original post, but it seems to be private, could you change it to unlisted?

@zany130 Could you try on quexten/goldwarden/actions/runs/7580997577 and see what logs you get when autotyping?

that gives me the following when I try to  do the autotype
flatpak run com.quexten.Goldwarden
/app/bin/monitors/dbus_autofill_monitor.py:4: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '4.0') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk
IS daemon running False
running daemon
Failed err None
Flatpak Config directory: /home/zany130/.var/app/com.quexten.Goldwarden/config/goldwarden.json

[INF] [19:15] [Goldwarden > Keyring] >>> Creating new memguard keyring

[INF] [19:15] [Goldwarden > Agent] >>> Agent listening on /home/zany130/.var/app/com.quexten.Goldwarden/data/goldwarden.sock...

Blocking, press ctrl+c to continue...

[INF] [19:15] [Goldwarden > SSH] >>> SSH Agent listening on /home/zany130/.var/app/com.quexten.Goldwarden/data/ssh-auth-sock

[WRN] [19:15] [Goldwarden > Agent] >>> Could not monitor idle: org.freedesktop.DBus.Error.ServiceUnknown

autostart enabled..!?
(':1.15', '/org/freedesktop/portal/desktop/request/1_258/com/quexten/Goldwarden/16482974', 'org.freedesktop.portal.Request', 'Response', GLib.Variant('(ua{sv})', (0, {'background': <true>, 'autostart': <true>})), None)
/app/bin/settings.py:34: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  self.ssh_row.set_icon_name("emblem-default")
[INF] [19:16] [Goldwarden > Pinentry] >>> Asking for pin |Unlock Goldwarden|Enter the vault PIN|

[INF] [19:16] [Goldwarden > Pinentry] >>> Got pin from user

[INF] [19:16] [Goldwarden > Keyring] >>> Unlocking keyring with account key

[INF] [19:16] [Goldwarden > Actions] >>> Unlocking vault...

[INF] [19:16] [Goldwarden > Auth] >>> Refreshing token

[INF] [19:16] [Goldwarden > Auth] >>> Refreshing using refresh token

[INF] [19:16] [Goldwarden > Auth] >>> Token refreshed

[INF] [19:16] [Goldwarden > Actions] >>> Token refreshed

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Performing full sync...

[INF] [19:16] [Goldwarden > Websocket] >>> Connected to websocket server...

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Sync successful, initializing keyring and vault...

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Reading  0 org keys...

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Initializing keyring from user symmetric key...

[INF] [19:16] [Goldwarden > Keyring] >>> Unlocking keyring with account key

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Clearing vault...

[INF] [19:16] [Goldwarden > Bitwarden API] >>> Adding 563 ciphers to vault...

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

/app/bin/autofill.py:6: PyGIWarning: Notify was imported without specifying a version first. Use gi.require_version('Notify', '0.7') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk, Adw, GLib, Notify
/app/bin/autofill.py:80: DeprecationWarning: Gtk.Widget.hide is deprecated
  self.history_list.hide()
/app/bin/autofill.py:108: DeprecationWarning: Gtk.Widget.get_style_context is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
/app/bin/autofill.py:108: DeprecationWarning: Gtk.StyleContext.add_class is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
[INF] [19:17] [Goldwarden > Systemauth] >>> Checking permission for goldwarden with session type com.quexten.goldwarden.accessvault

[INF] [19:17] [Goldwarden > Biometrics] >>> Checking biometrics for com.quexten.goldwarden.accessvault

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[INF] [19:17] [Goldwarden > Biometrics] >>> Biometrics result: true

[INF] [19:17] [Goldwarden > Systemauth] >>> Permission granted, creating session

[INF] [19:17] [Goldwarden > Pinentry] >>> Asking for approval |Approve List Credentials|zany130 on main.py>python3>goldwarden is trying access all credentials|

[WRN] [19:17] [Goldwarden > Websocket] >>> Invalid message received, length too short

[INF] [19:17] [Goldwarden > Pinentry] >>> Got approval from user

/app/bin/autofill.py:64: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  action_row.set_icon_name("dialog-password")
/app/bin/autofill.py:46: DeprecationWarning: Gtk.Widget.show is deprecated
  self.history_list.show()
enter
secure.hulu.com
Flatpak Config directory: /home/zany130/.var/app/com.quexten.Goldwarden/config/goldwarden.json
[INF] [19:18] [Goldwarden > Autotype] >>> Starting to Type String
[INF] [19:18] [Goldwarden > Autotype] >>> Selecting Devices
[INF] [19:18] [Goldwarden > Autotype] >>> Starting Session
[ERR] [19:18] [Goldwarden > Autotype] >>> Error starting session: Invalid session
[INF] [19:18] [Goldwarden > Autotype] >>> Performing Typing

/app/bin/autofill.py:6: PyGIWarning: Notify was imported without specifying a version first. Use gi.require_version('Notify', '0.7') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk, Adw, GLib, Notify
/app/bin/autofill.py:80: DeprecationWarning: Gtk.Widget.hide is deprecated
  self.history_list.hide()
/app/bin/autofill.py:108: DeprecationWarning: Gtk.Widget.get_style_context is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
/app/bin/autofill.py:108: DeprecationWarning: Gtk.StyleContext.add_class is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
[INF] [19:18] [Goldwarden > Systemauth] >>> Checking permission for goldwarden with session type com.quexten.goldwarden.accessvault

[INF] [19:18] [Goldwarden > Biometrics] >>> Checking biometrics for com.quexten.goldwarden.accessvault

[INF] [19:18] [Goldwarden > Biometrics] >>> Biometrics result: true

[INF] [19:18] [Goldwarden > Systemauth] >>> Permission granted, creating session

[INF] [19:18] [Goldwarden > Pinentry] >>> Asking for approval |Approve List Credentials|zany130 on main.py>python3>goldwarden is trying access all credentials|

[INF] [19:18] [Goldwarden > Pinentry] >>> Got approval from user

/app/bin/autofill.py:64: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  action_row.set_icon_name("dialog-password")
/app/bin/autofill.py:46: DeprecationWarning: Gtk.Widget.show is deprecated
  self.history_list.show()
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.

(autofill.py:2717): libnotify-WARNING **: 19:18:26.206: Running in confined mode, using Portal notifications. Some features and hints won't be supported
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy username
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
copy password
Failed to connect to a Wayland server: No such file or directory
Note: WAYLAND_DISPLAY is set to wayland-0
Note: XDG_RUNTIME_DIR is set to /run/user/1000
Please check whether /run/user/1000/wayland-0 socket exists and is accessible.
enter
secure.hulu.com
Flatpak Config directory: /home/zany130/.var/app/com.quexten.Goldwarden/config/goldwarden.json
[INF] [19:18] [Goldwarden > Autotype] >>> Starting to Type String
[INF] [19:18] [Goldwarden > Autotype] >>> Selecting Devices
[INF] [19:18] [Goldwarden > Autotype] >>> Starting Session
[ERR] [19:18] [Goldwarden > Autotype] >>> Error starting session: Invalid session
[INF] [19:18] [Goldwarden > Autotype] >>> Performing Typing

/app/bin/autofill.py:6: PyGIWarning: Notify was imported without specifying a version first. Use gi.require_version('Notify', '0.7') before import to ensure that the right version gets loaded.
  from gi.repository import Gtk, Adw, GLib, Notify
/app/bin/autofill.py:80: DeprecationWarning: Gtk.Widget.hide is deprecated
  self.history_list.hide()
/app/bin/autofill.py:108: DeprecationWarning: Gtk.Widget.get_style_context is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
/app/bin/autofill.py:108: DeprecationWarning: Gtk.StyleContext.add_class is deprecated
  self.history_list.get_style_context().add_class("boxed-list")
[INF] [19:19] [Goldwarden > Systemauth] >>> Checking permission for goldwarden with session type com.quexten.goldwarden.accessvault

[INF] [19:19] [Goldwarden > Biometrics] >>> Checking biometrics for com.quexten.goldwarden.accessvault

[INF] [19:19] [Goldwarden > Biometrics] >>> Biometrics result: true

[INF] [19:19] [Goldwarden > Systemauth] >>> Permission granted, creating session

[INF] [19:19] [Goldwarden > Pinentry] >>> Asking for approval |Approve List Credentials|zany130 on python3>python3>goldwarden is trying access all credentials|

[INF] [19:19] [Goldwarden > Pinentry] >>> Got approval from user

/app/bin/autofill.py:64: DeprecationWarning: Adw.ActionRow.set_icon_name is deprecated
  action_row.set_icon_name("dialog-password")
/app/bin/autofill.py:46: DeprecationWarning: Gtk.Widget.show is deprecated
  self.history_list.show()
enter
secure.hulu.com
Flatpak Config directory: /home/zany130/.var/app/com.quexten.Goldwarden/config/goldwarden.json
[INF] [19:19] [Goldwarden > Autotype] >>> Starting to Type String
[INF] [19:19] [Goldwarden > Autotype] >>> Selecting Devices
[INF] [19:19] [Goldwarden > Autotype] >>> Starting Session
[ERR] [19:19] [Goldwarden > Autotype] >>> Error starting session: Invalid session
[INF] [19:19] [Goldwarden > Autotype] >>> Performing Typing

[WRN] [19:19] [Goldwarden > Websocket] >>> Invalid message received, length too short

Hmmm, so from the logs it seems like everything works up to the point of using the remote-deskop portal.  What portal are you using? The KDE portal should support this and should make a prompt pop up to ask you for remote-desktop permission.

What portal are you using?

How do I check this?

Autotype might be fixed now. It seems the KDE portal is out of spec for a method, instead of defaulting to "all device access" it defaults to "permission denied" when not explicitly noting devices.
https://github.com/quexten/goldwarden/actions/runs/7629310368

At least it works now on Kubuntu using kde portal on wayland.

Yup works on my system as well guess this can be closed

Now to fix all the other KDE-related bugs ;) Thanks for reporting&testing!


05/04/2018 02:55 : Create Pull Request for branch master -> prTesting1_2018050402

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Add returncode to subprocess.CalledProcessError output

This is for slightly better output as discussed in https://github.com/modelmat/sphinxcontrib-drawio/issues/69. It appears not all return codes have stderr output associated with them, so this will help in those rare cases.
Apparently there are ways to get more information about the return code, such as this line in subprocess.py, but I didn't know how finicky that would be and I left it as the bare return code.
Fixes #69

@jdillard Looks like the line length is too long so if you can fix that I'll merge this.

Oops! I think I got it right this time.

Now to figure out how to push a release... I've forgotten.

Ha I know the feeling! I have to put a MAINTENANCE.md file in my projects with instructions for myself so I don't forget (there is probably a better way though).

Should be released now. Ignore the fact CI says it failed to release, it seems to upload it twice for unknown reason.

I see it https://pypi.org/project/sphinxcontrib-drawio/. You rock! Thanks so much!


upgrade from RDFS2020 style to RDFSEd2Beta style

NC 2.3 uses the older RDFS2020 style, CGMES 3.0 uses the new RDFSEd2Beta style.
Upgrading to RDFSEd2Beta will harmonize data and simplify SPARQL Updates.
@Sveino Is there a formal spec for RDFSEd2Beta? I've seen an "RDF usage guide", please link it
@Sveino and @griddigit-ci, let's list all differences:

[ ] New cim and eu namespaces: can be seen in https://github.com/Sveino/Inst4CIM-KG/issues/13
[ ] rdfs:Class -> owl:Class
[ ] rdf:Property -> owl:ObjectProperty, owl:DatatypeProperty
[ ] cims:dataType cim:String -> rdfs:range xsd:string etc: https://github.com/Sveino/Inst4CIM-KG/issues/28
[ ] cims:inverseRoleName -> owl:inverseOf: https://github.com/Sveino/Inst4CIM-KG/issues/26
[ ] cims:multiplicity cims:M:*..1 -> owl:FunctionalProperty: https://github.com/Sveino/Inst4CIM-KG/issues/30.

But it's better to also leave cims:multiplicity. It's reflected in SHACL but keeping it gives easier access to this important info

[ ] Bring all meta-props back to cims: namespace (this is a regression): https://github.com/Sveino/Inst4CIM-KG/issues/10.

https://github.com/Sveino/Inst4CIM-KG/issues/4 is also related

[ ] Fix enumerations (this is not a difference, but still nice to fix): https://github.com/Sveino/Inst4CIM-KG/issues/7, https://github.com/Sveino/Inst4CIM-KG/issues/11
[ ] Fix enumeration members (this is a regression) https://github.com/Sveino/Inst4CIM-KG/issues/45
[ ] Fix packages (this is not a difference, but still nice to fix):  https://github.com/Sveino/Inst4CIM-KG/issues/12
[ ] Multipliers and Units (attached to QuantityKinds) are styled differently, see section ## Actual Multipliers and Units. Affects https://github.com/Sveino/Inst4CIM-KG/issues/38
[ ] rdfs:comment -> skos:definition for individuals (eg cim:UnitSymbol.W a owl:NamedIndividual)

https://github.com/Sveino/Spec4CIM-KG/blob/develop/docs/Profile/UML2RDF.md is the spec for UML conversion, which also lists the differences. (Note: the link is private, and the markdown is converted from docx, so it's not yet perfect).
@Sveino and @griddigit-ci do you have more tasks (checkboxes) to add?


Tooltip Visiblity parameter missing

Hello,
Is there way to get the tooltip (in FlaUI the HelpText property of AutomationElement) visibilty? I already had tests that found the helptext but that the tooltip was not visible. I want to catch that during automated tests and not during manual.
With kind regards,
Kevin

The tooltip is a special think. It is like a dropdown, a window either directly on the desktop or on the root of the application. Best way to get the tooltip-window is to use FlauInspect and use the hover mode and try to catch the tooltip window when it is open. Then you know where it is and what title it has and then you can try to find that window and get the content to check if your tooltip is visible or not.


Add new minutes

Signed-off-by: Hugues de Valon<EMAIL_ADDRESS>

Only Too Many Requests errors.

Only Too Many Requests errors.


Streaming with Monix's Observable

Looks like we have all the pieces in place to make this happen.

hi! What's a preferred implementation to address the issue that a monix Observable has a kind F[_], but in Finch a stream needs to be S[_[_], _]?
Some class ObservableWrapper[F[_]: Effect, A](val observable: Observable[A]) to mimic it?

@alexeygorobets How does Iterant relate to Observable?  It seems to be in proper kind.

Sorry, somehow missing that we need Iterant.
Can you please explain then why this issue is called so?

Oh the original plan I had in mind was to integrate with Observable, but, as you noted, it doesn't have the right kind. I was wondering if Iterant could serve as well as a replacement for Observable.

Thanks! I will take it then.


Fix: Tabs widget height update loop

Description
On AutoLayout canvas, TabsWidget height keeps flickering on every widget operation.
Cause:

Clash between positionUtils and AutoHeight.
PositionUtils attempts to check for minSize violation and updates the TabsWidget height to 30 rows (minheight).
Since the Tab canvas is empty, AutoHeight tries to set the Height of the parent widget to accomodate the minHeight required by the canvas ~ 19 rows.
This loop occurs on every widget operation.

Solution:

Update AutoHeight logic to consider widget's minHeight in its calculations.

Fixes # (issue)
(https://github.com/appsmithorg/appsmith/issues/22329)
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual (Follow the steps provided in the issue ticket)

Checklist:
Dev activity

[ x ] My code follows the style guidelines of this project
[ x ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ x ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test

Auto Height already has a minDynamicHeight that it takes into consideration when computing the heights. Could we use this property instead, so that the AutoHeight computations are the source of truth?
PositionUtils should ideall handle the sizes in the horizontal direction.
Let's discuss the code flow and understand why we need to compute the heights in the PositionUtils, and see if we can rely on one of the features for the height computations, and prevent the other feature from triggering.


Restructure settings file to allow settings per development environment

Proposed changes
Splits settings files for local development, testing, etc.
This commit doesn't add production settings, as we still don't know what is required for production.
Fixes issue: #56
Related issue: #130
Checklist
Change the space in the box to an x for those you have completed. You can also fill these out after creating the pull request. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your change.

[x] I have read the contribution guidelines, because I wrote them.
[x] I have linked any relevant existing issues/suggestions in the description above (include #??? in your description to reference an issue, where ??? is the issue number)
[x] The pull request is requesting a merge into the develop branch
[x] I have added necessary documentation (if appropriate)

@JackMorganNZ have you noted somewhere to delete that redundant step?


Enable selective sync

Today on cf-for-k8s, when I vendir sync --use-directory to replace some remote code with local dev copies, it takes 30s to redownload all the dependencies. I really only need to move the files from the one directory. Could this be made faster?

@cwlbraa good point. cf-for-k8s definitely got beefier.

covered by https://github.com/k14s/vendir/releases/tag/v0.7.0.
eg vendir sync -d config/_ytt_lib/github.com/cloudfoundry/cf-k8s-networking=~/workspace/cf-k8s-networking to just touch that directory.


various: Fix issues causing pipeline failures

Fix merge problem that is causing pipeline failures
Signed-off-by: Jared Baumann<EMAIL_ADDRESS>

The following west manifest projects have been modified in this Pull Request:

Name
Old Revision
New Revision
Diff

hal_nxp
https://github.com/zephyrproject-rtos/hal_nxp/commit/3ee6020efc1f8323d0fdc85615d3477161f0aa1f
https://github.com/zephyrproject-rtos/hal_nxp/commit/d957472002a3758154f402e982d95ce90cf58e12
<EMAIL_ADDRESS>

hal_stm32
https://github.com/zephyrproject-rtos/hal_stm32/commit/83cf59e4309e5f1758a76c0513c437e0fbcee40b
https://github.com/zephyrproject-rtos/hal_stm32/commit/b916f9a01c665194fbbdd6a67a0f37b4e0a7e78c (main)
<EMAIL_ADDRESS>

mbedtls
https://github.com/zephyrproject-rtos/mbedtls/commit/7fed49c9b9f983ad6416986661ef637459723bcb
https://github.com/zephyrproject-rtos/mbedtls/commit/6e166050075688fd3cf3d0cb3fc34a1c52df2496 (zephyr)
<EMAIL_ADDRESS>

nrf_hw_models
https://github.com/zephyrproject-rtos/nrf_hw_models/commit/85ecf8e3bda805fa1405db5048d77e2980c976f0
https://github.com/zephyrproject-rtos/nrf_hw_models/commit/d8f125e1988221ebfea5944fc3118f1780787ccc (master)
<EMAIL_ADDRESS>

picolibc
https://github.com/zephyrproject-rtos/picolibc/commit/04ada5951cbaf8e7b17f8226ce31cb6837c28ba7
https://github.com/zephyrproject-rtos/picolibc/commit/98e59b70d6ef8eb18f7dccc3d39cecaa0658fa4f (main)
<EMAIL_ADDRESS>

psa-arch-tests
https://github.com/zephyrproject-rtos/psa-arch-tests/commit/f4fc2442b8e29e2a03d9899e46e5a3ea3df8c2c9
https://github.com/zephyrproject-rtos/psa-arch-tests/commit/24ed42e34e03ebbc945a7204819c1471cce2bda8 (master)
<EMAIL_ADDRESS>

tf-m-tests
https://github.com/zephyrproject-rtos/tf-m-tests/commit/bcb53bccccdc05c713aade707e7a8ddad35c210f
https://github.com/zephyrproject-rtos/tf-m-tests/commit/ce57ade67f4e70724975a3bbadeea488f4b9effc (master)
<EMAIL_ADDRESS>

trusted-firmware-m
https://github.com/zephyrproject-rtos/trusted-firmware-m/commit/13abde213930ffdf600cf8f01113da8a5fdd9d9c
https://github.com/zephyrproject-rtos/trusted-firmware-m/commit/5503c15bd4026e3ea4264a62d70395af34e16742 (master)
<EMAIL_ADDRESS>

Note: This message is automatically posted and updated by the Manifest GitHub Action.


BUG: Spell Points arent able to be used if the actor has 0 spell slots

The upcasting portion of it bugs out- I'm guessing it uses the cause is that the module uses what current spell slots the actor has in order to arrive at what levels you can cast the spell at, which is what most of my testing came to.

why do you have no slots? what is the usecase? anyway you should at least have 1 slot just to allow the module deduct the level of the spell.


[Feature Request] Azure.Core provide an API to configure the RequestFailedDetailsParser

In Azure Core, RequestFailedException will not include the response content or headers in the message when the IsError property of the response is false.
https://github.com/Azure/azure-sdk-for-net/blob/8f5cf842761758ec06ff2fbd52c03468b05d09e1/sdk/core/Azure.Core/CHANGELOG.md?plain=1#L84
We received a request in Language-Dotnet channel, they want to see the error message from RequestFailedException
https://teams.microsoft.com/l/message/19:7b87fb348f224b37b6206fa9d89a105b@thread.skype/1734990642330?tenantId=72f988bf-86f1-41af-91ab-2d7cd011db47&groupId=3e17dcb0-4257-4a30-b843-77f47f1d4121&parentMessageId=1734990642330&teamName=Azure SDK&channelName=Language - DotNet&createdTime=1734990642330
Current solution:
we will ask customer to write customized RequestFailedDetailsParser and set it to Pipeline (customize the client constructor and re-build the pipeline with the customized RequestFailedDetailsParser).
Several DPG SDKs have adopt this solution.
Problem:
This is mgmt-plane SDK which use ArmClient which cannot be customized to accept RequestFailedDetailsParser.
And write customized code is heavy for customer.
Possible solution option:
And since more than one service required to retrieve the content of RequestFailedException, Azure.Core may be can provide an API to construct the Pipeline with cutsomized parser (just like pipeline policy)  when create client just like pipeline policy
There will be an optional field RequestFailedDetailsParser? RequestFailedDetailsParser in ClientOptions. When build pipeline, it will set the requestFailedDetailsParser to pipeline.
public abstract class ClientOptions
{
    ...
    public RequestFailedDetailsParser ? RequestFailedDetailsParser { get; set; }
  ...
}

When customer want to retrieve content from RequestFailedException, they can write their own RequestFailedDetailsParser and set to ClientOptions when create client.

@chunyu3 :  I don't follow the scenario here.  We intentionally do not include the content or headers when IsError is not set to avoid leaking potentially sensitive information.   Please explain the full end-to-end scenario of the use case and why accessing the raw response is not adequate.    Further, please provide details on why ArmClient cannot use the existing Core API to manage the parser.
//cc: @ArthurMa1978, @m-nash

This feature ask is related to https://github.com/Azure/azure-sdk-for-net/issues/45220#issuecomment-2260718461 , if customer want to have the details of the response then they can specify the RequestFailedDetailsParser when constructing the RequestFailedException.

I'm not convinced this is the right path forward.   Please open an email thread with full end-to-end scenario details for discussion and send to the Azure Core v-team.


Update language service slug

Use the correct slug name and follows Python.

The issue has been addressed in https://github.com/Azure/azure-sdk-tools/pull/3702


Drop size 0 arrays in stack

Fixes https://github.com/dask/dask/issues/4928
Was doing some cleanup of da.stack ( https://github.com/dask/dask/pull/4975 ) ( https://github.com/dask/dask/pull/4976 ) after the recent blogpost (since we reference using da.stack there). Was curious to see what it would take to solve issue ( https://github.com/dask/dask/issues/4928 ) and found it was pretty easy. Anyways here is the fix.
Note: Requires PR ( https://github.com/dask/dask/pull/4976 ) be merged first.

[x] Tests added / passed
[x] Passes flake8 dask

Thoughts on adding a small test?

Happy to take a look. Given this is waiting on PR ( https://github.com/dask/dask/pull/4976 ) to be merged first, will hold off until that happens.

I've pushed a test here that I think should work. It relies on some dev versions though.

Also added a test that works with stable versions.

Ok should be good now. If we can include this in this release, that would be nice. Though if that's not possible, that's ok.

Is this critical for something?  Ideally it's nice for PRs to be in master
for a bit to flush out the possibility of errors.  Merging things just
before release makes me a little nervous.
On Mon, Jun 24, 2019 at 10:51 PM jakirkham<EMAIL_ADDRESS>wrote:

Ok should be good now. If we can include this in this release, that would
be nice. Though if that's not possible, that's ok.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/4978?email_source=notifications&email_token=AACKZTA5MVJTSXHJYQHI6GLP4EXWNA5CNFSM4H2FZ2NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOGBYQ#issuecomment-505176290,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTHVJAIH6J3YPLATCGLP4EXWNANCNFSM4H2FZ2NA
.

Just to reiterate, it's up to you to merge or not. Am not trying to for the issue. 🙂

Planning to merge end of day if no comments.

Thanks all! 😄


support scheduledJob with locks.

Support jobs that cannot have overlapping execution at the same time. #7
Description of changes:
Add ScheduledJobs with optional parameter of lock.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

LGTM thanks for the changes Jinsoo!


Add Python 3.10 to CI matrix

Currently fails due to "ray" constraints.
ERROR: Could not find a version that satisfies the requirement ray<=1.7.0 (from versions: none)

And "ray" doesn't have Python 3.10 support yet: e.g. https://github.com/ray-project/ray/issues/19116
Also, is it worth dropping Python 3.7 from the CI under NEP 29?

thanks for the contribution @jamesmyatt !

And "ray" doesn't have Python 3.10 support yet: e.g. ray-project/ray#19116

K, yeah I think we can exclude the modin extra in CI like this until ray supports 3.10.

Also, is it worth dropping Python 3.7 from the CI under NEP 29?

Let's do it! Would you mind creating a new issue for dropping 3.7?

hey @jamesmyatt, would you mind updating the CI to conditionally exclude the modin tests when the python version = 3.10?
https://github.com/pandera-dev/pandera/blob/master/.github/workflows/ci-tests.yml#L169-L181

hey @jamesmyatt friendly ping on this PR. Let me know if you need any help bringing this to the finish line. Basically what's left to do is:

rebase changes onto dev branch and change PR target branch to dev.
exclude python 3.10 from CI matrix for the ray CI tests.

Sorry I haven't had time. I've updated the target and resolved the conflicts from dev. Would you like it to be properly rebased?

Sorry I still need to skip the ray tests.

@cosmicBboy There's an issue with this PR that I can't see a good way to avoid trying to install ray on Python 3.10. I've tried something hacky but it doesn't work. Any thoughts?

@jamesmyatt working on your branch to clean up CI and nox build... it's sort of a mess 😅

after much trial and error reproducing the errors in CI, managed to get an updated build that seems to be more stable in terms of (i) installing geopandas and (ii) using miniconda to install dependencies ones in CI runtime and using pytest instead of nox to run unit tests of modules. Will need to figure out why nox isn't working for certain combinations in the CI matrix.
gonna merge this now @jamesmyatt!

Thanks @cosmicBboy. You have quite a complicated build in terms of dependencies etc. Well done for getting it working.


excalidraw canvas group selection reveal bug

When I group selection graphics and delete the grouped graphics, a strange symbol suddenly appears on the excalidraw canvas. When I click excalidraw canvas, it disappears
Steps To Reproduce
1、Create multiple graph
2、Drag box selection
3、Right click and click group selection
4、delete

I can confirm that I have encountered the same issue described by @Wlivans. When performing the steps outlined in the initial report, I observed the appearance of a strange symbol on the Excalidraw canvas after deleting grouped graphics. Clicking anywhere on the canvas does indeed make the symbol disappear.


Question: warnings in the left panel?

Our team has a differs from Prettier's opinions in a couple small areas, so I set out to implement prettier-eslint using a VS Code extension (which this one seems to support).
Both the "Format document" and auto-formatting on save work perfectly in VS Code—everything formats properly with prettier-eslint, first using prettier, than picking up my additional eslint rules.
What I'm missing, though, is warnings on the left panel of the editor when my code differs from prettier (before I either save or manually format it). Is that possible with this extension? If not, do you have a suggestion of how to integrate this plugin with prettier-eslint in a way where all the proper prettier/eslint changes are warned correctly in the editor?
(I realize this question is 50/50 about the extension and prettier-eslint itself)

If you have eslint rules, eslint extension should warn you about those errors. At least those covered by eslint and your plugins/configs
We could compare the current text with this extensions output.

Performance?
UX?

I feel this is out of our scope and more a linter thing.
What do you call left panel?

Sorry, I meant the left panel in VS Code where it's standard to see warnings (like eslint, etc).
So it sounds like the VS Code extension does not (by design) show any prettier "lint" errors in the editor itself—it's purely for manually, or automatically on save, formatting the code through prettier or prettier-eslint 👍
That's all I was curious about... I'll have to continue exploring how to get these two things working together.

If you haven't, you should check eslint extension. A linter :-)
I may suggest  https://github.com/prettier/eslint-plugin-prettier

Yes, that's the ESlint plugin I've been using.
Once I added that, I got warnings for all the linting issues, but then when I ran the plugin formatter in VS Code, it defaulted back to formatting prettier rules only (my additional ESlint rules seem to be ignored/overridden).
I don't want to go outside the scope of this project, but I'm basically curious how to use this in tandem with prettier-eslint, and have all the same rules show lint warnings in VS Code before formatting.

Have you enabled prettier.eslintIntegration?

Can you share your eslint config?

Yes, I have the eslintIntegration enabled. Here's a GIF example of what's going wrong:

File in the state that I want based on eslint rules
Observe prettier error (via eslint config) - prettier rules seem to override eslint "space-before-function-paren" rule
Save file (vscode-prettier should run prettier-eslint)
Observe file is re-formatted with just plain prettier formatting, ignoring additional eslint rules
Observe new eslint "space-before-function-paren" error

.eslintrc.json
{
    "extends": "eslint:recommended",
    "parser": "babel-eslint",
    "root": true,

    "env": {
        "browser": true,
        "commonjs": true,
        "es6": true,
        "node": true,
        "mocha": true
    },

    "plugins": ["react", "prettier"],

    "parserOptions": {
        "ecmaFeatures": {
            "jsx": true
        },
        "sourceType": "module"
    },

    "rules": {
        "prettier/prettier": [
            "error",
            {
                "singleQuote": true,
                "bracketSpacing": false,
                "printWidth": 100,
                "tabWidth": 4
            }
        ],
        "indent": [
            "error",
            4,
            {
                "SwitchCase": 1
            }
        ],
        "quotes": ["error", "single"],
        "semi": "error",
        "curly": "error",
        "strict": "error",
        "eqeqeq": "error",
        "one-var": ["error", "never"],
        "guard-for-in": "error",
        "no-floating-decimal": "error",
        "no-nested-ternary": "error",
        "no-multi-spaces": "error",
        "radix": "error",
        "keyword-spacing": "error",
        "space-before-blocks": "error",
        "space-before-function-paren": [
            "error",
            {
                "named": "never",
                "anonymous": "always",
                "asyncArrow": "always"
            }
        ],
        "spaced-comment": [
            "error",
            "always",
            {
                "exceptions": ["-"]
            }
        ],
        "wrap-iife": "error",
        "eol-last": "error",
        "handle-callback-err": "error",
        "comma-dangle": "error",
        "no-console": "off",
        "no-unsafe-negation": "error",
        "quote-props": [
            "error",
            "as-needed",
            {
                "keywords": true,
                "unnecessary": false
            }
        ],
        "valid-jsdoc": [
            "error",
            {
                "requireReturn": false
            }
        ],
        "semi-spacing": "error",
        "dot-notation": "error",
        "space-infix-ops": "error",
        "comma-spacing": "error",
        "new-cap": "error",
        "no-extra-bind": "error",
        "no-shadow": "error",
        "func-call-spacing": "error",
        "no-loop-func": "error",
        "key-spacing": "error",
        "no-trailing-spaces": "error",
        "camelcase": "error",

        "react/display-name": "error",
        "react/jsx-boolean-value": "error",
        "react/jsx-equals-spacing": ["error", "never"],
        "react/jsx-no-undef": "error",
        "react/jsx-uses-react": "error",
        "react/jsx-uses-vars": "error",
        "react/no-did-mount-set-state": "error",
        "react/no-did-update-set-state": "error",
        "react/no-multi-comp": "error",
        "react/no-unknown-property": "error",
        "react/prop-types": "error",
        "react/react-in-jsx-scope": "error",
        "react/self-closing-comp": "error",
        "react/sort-prop-types": "error",
        "react/wrap-multilines": "error"
    }
}

I'm realizing this might be more of an issue between the interplay of eslint-plugin-prettier and prettier-eslint, but I thought I'd start here since I'm saving via VS Code.

To me this seems to be conflicting rules. I could reproduce it with
{
  "env": {
    "es6": true
  },
  "parserOptions": {
    "ecmaFeatures": {
      "jsx": true
    }
  },
  "plugins": [
    "prettier"
  ],
  "rules": {
    "prettier/prettier": [
      "error",
      {
        "singleQuote": true,
        "bracketSpacing": false,
        "printWidth": 100,
        "tabWidth": 4
      }
    ],
    "space-before-function-paren": [
      "error",
      {
        "named": "never",
        "anonymous": "always",
        "asyncArrow": "always"
      }
    ]
  }
}

call(function() {});

Sorting rules differently doesn't change the outcome

Note: with the formater, prettier wins :-)

Obviously if I remove eslint-plugin-prettier and only use the extension, the formatting works correctly (since it runs prettier then eslint --fix). But then I don't get the linting help along the way.
I'm not sure how to use the two at the same time :/

I've tried to debug this quickly,
I see two troubles with this case:
call( function     ()) -> prettier -> call(function() {}) -> eslint -> error: missing space -> eslint --fix -> call(function() {}) (no fix)
And the fact that what ever you fix, it will conflict with the other rule.

eslint-plugin-prettier and prettier-eslint contradict by definition.

eslint-plugin-prettier will create an error for every code fragment that doesn't match prettier's output.
prettier-eslint will run eslint --fix after prettier --write, thus writing non-prettier code to disk.

You should remove eslint-plugin-prettier. As far as I know there is no eslint-plugin-prettier-eslint, which is what you're really after.

^ haha precisely. I almost mentioned that eslint-plugin-prettier-eslint is the equivalent.
For now, I've added all the rules to my .eslintrc that I want to "lint/warn" in VS Code, even the ones that come by default within prettier's opinions already (not ideal). This way I get the linting hints, but my ESlint config is far more bloated than it really needs to be.

I decided to give this a crack: https://github.com/prettier/eslint-plugin-prettier/pull/38

Seems answered.


uwsgi env to build it with python 3.11.5 or higher

Checklist before submitting

[ ] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

This was for trial purpose, but no longer needed.


added standard resolution types

@platelminto tests are passed. Only issue is unable to detect QHD due to ignore case in, matches = list(re.finditer(pattern, clean_name, re.IGNORECASE))  instead it's identified as qHD which is 540p.
Do you know any other way to distinguish, identify both? If not, It's fine for me to have other standard resolutions types.

For example torrent title:
www.1TamilBlasters.lat - Thuritham (2023) [Tamil - 2K QHD AVC UNTOUCHED - x264 - AAC - 3.4GB - ESub].mkv
this need to be like 1440p

I'm also waiting for this PR too

Should now be available in 2.8.1! I've opened an issue for qHD case-sensitivity, so for now it defaults to QHD. Thank you for opening this PR!
(Sorry if I screwed up the commits and stuff, if you can next time make a PR into dev, not master!)

No worries. I forgot to change the target branch.


[SPARK-33084][CORE][SQL] Add jar support ivy path

What changes were proposed in this pull request?
Support add jar with ivy path
Why are the changes needed?
Since submit app can support ivy, add jar we can also support ivy now.
Does this PR introduce any user-facing change?
User can add jar with sql like
add jar ivy:://group:artifict:version?exclude=xxx,xxx&transitive=true
add jar ivy:://group:artifict:version?exclude=xxx,xxx&transitive=false

core api
sparkContext.addJar("ivy:://group:artifict:version?exclude=xxx,xxx&transitive=true")
sparkContext.addJar("ivy:://group:artifict:version?exclude=xxx,xxx&transitive=false")

How was this patch tested?
Added UT

Test build #129507 has started for PR 29966 at commit 51daf9a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34113/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34113/

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34113/
Test FAILed.

Test build #129507 has finished for PR 29966 at commit 51daf9a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129507/
Test FAILed.

Test build #129520 has started for PR 29966 at commit d6e8caf.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34124/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34124/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34124/
Test PASSed.

Test build #129520 has finished for PR 29966 at commit d6e8caf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129520/
Test PASSed.

Test build #129551 has started for PR 29966 at commit 169e1f8.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34157/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34157/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34157/
Test PASSed.

Test build #129551 has finished for PR 29966 at commit 169e1f8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129551/
Test PASSed.

cc @dongjoon-hyun  As I have mentioned in  https://issues.apache.org/jira/browse/SPARK-29288, make this pr support ivy path like  https://issues.apache.org/jira/browse/HIVE-9664

Got it. Thank you, @AngersZhuuuu

Test build #131522 has started for PR 29966 at commit b3e3211.

Test build #131522 has finished for PR 29966 at commit b3e3211.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131522/
Test FAILed.

Test build #131524 has started for PR 29966 at commit 9161340.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36125/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36127/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36125/

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36125/
Test FAILed.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36127/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36127/
Test PASSed.

Test build #131524 has finished for PR 29966 at commit 9161340.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131524/
Test FAILed.

retest this please

Test build #131531 has started for PR 29966 at commit 9161340.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36134/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36134/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36134/
Test PASSed.

Test build #131531 has finished for PR 29966 at commit 9161340.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131531/
Test PASSed.

Test build #131604 has started for PR 29966 at commit 9161340.

Ah, I see. This looks useful. I think we need to document it. Otherwise, user cannot notice it. https://spark.apache.org/docs/3.0.1/sql-ref-syntax-aux-resource-mgmt-add-jar.html

Ah, I see. This looks useful. I think we need to document it. Otherwise, user cannot notice it. https://spark.apache.org/docs/3.0.1/sql-ref-syntax-aux-resource-mgmt-add-jar.html

Yea, will update the guide toody

Ah, I see. This looks useful. I think we need to document it. Otherwise, user cannot notice it. https://spark.apache.org/docs/3.0.1/sql-ref-syntax-aux-resource-mgmt-add-jar.html

Updated.

+CC @xkrogen

Test build #131617 has started for PR 29966 at commit 0e3c1ec.

Test build #131604 has finished for PR 29966 at commit 9161340.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131604/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131617/

Test build #131632 has started for PR 29966 at commit 63e877b.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36205/

Test build #131638 has started for PR 29966 at commit b60ba1e.

Test build #131632 has finished for PR 29966 at commit 63e877b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131632/

Test build #131638 has finished for PR 29966 at commit b60ba1e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36218/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131638/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36233/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36239/

gentle ping @maropu  @xkrogen Sorry for my late reply since busy work, have updated to merge code using DependencyUtils, since it's origin access privilege is  deploy package, but now it's. not only. used in deploy  so remove to utils package

Test build #131896 has started for PR 29966 at commit 7f878c2.

Test build #131896 has finished for PR 29966 at commit 7f878c2.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131896/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36493/

Test build #131897 has started for PR 29966 at commit d2c1950.

Test build #131897 has finished for PR 29966 at commit d2c1950.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131897/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36494/

Test build #131901 has started for PR 29966 at commit 2200076.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36498/

Test build #131901 has finished for PR 29966 at commit 2200076.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131901/

Test build #131921 has started for PR 29966 at commit 875d8a7.

Test build #131921 has finished for PR 29966 at commit 875d8a7.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131921/

Test build #131922 has started for PR 29966 at commit e921245.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36518/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36519/

Test build #131922 has finished for PR 29966 at commit e921245.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131922/

Test build #131989 has started for PR 29966 at commit 8c5cb7c.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36588/

Test build #131989 has finished for PR 29966 at commit 8c5cb7c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131989/

Test build #132004 has started for PR 29966 at commit f460974.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36603/

Build finished.

Test build #132004 has finished for PR 29966 at commit f460974.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

For other reviewers, could you put the screenshot of the updated doc in the PR description?

For other reviewers, could you put the screenshot of the updated doc in the PR description?

Thanks! Could you move the screenshot into the PR description?

Thanks! Could you move the screenshot into the PR description?

Updated

Split UT to make it clear and add more detail case and use ivy path with simply dependency to speed up test.

My last concern is the @xkrogen comment above (This PR is similar to #21339)
https://github.com/apache/spark/pull/29966#pullrequestreview-537856115 Anyone could review this? @srowen @HeartSaVioR ?

Test build #132017 has started for PR 29966 at commit bdc5035.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36616/

Test build #132021 has started for PR 29966 at commit 9c22882.

Test build #132017 has finished for PR 29966 at commit bdc5035.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class IvyProperties(

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132017/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36620/

Test build #132021 has finished for PR 29966 at commit 9c22882.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132021/

Test build #132057 has started for PR 29966 at commit b69a62e.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36656/

Test build #132057 has finished for PR 29966 at commit b69a62e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132057/

Test build #132170 has started for PR 29966 at commit ebe1c9c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36771/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36771/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36771/

Test build #132170 has finished for PR 29966 at commit ebe1c9c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132170/

@maropu are there any more blockers from your side? I'd love to get this into the 3.1 release if possible.
Last re-ping to @HeartSaVioR @srowen @gaborgsomogyi in case they have any comments since I think we are nearing completion on this PR.

Test build #132254 has started for PR 29966 at commit 6034fb2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36855/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36855/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36855/

Test build #132254 has finished for PR 29966 at commit 6034fb2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132254/

retest this please

Test build #132273 has started for PR 29966 at commit 6034fb2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36874/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36874/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36874/

Test build #132273 has finished for PR 29966 at commit 6034fb2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132273/

retest this please

Test build #132279 has started for PR 29966 at commit 6034fb2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36880/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36880/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36880/

Test build #132279 has finished for PR 29966 at commit 6034fb2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132279/

@maropu are there any more blockers from your side? I'd love to get this into the 3.1 release if possible.

I'm neutral on this, but we don't basically merge a new feature into the release branch (v3.1) after the branch-cut. cc: @HyukjinKwon @dongjoon-hyun @gatorsmile

Thanks for this work, @AngersZhuuuu . Btw, are you planning to support a ivy schema in --packages, too?

Thanks for this work, @AngersZhuuuu . Btw, are you planning to support a ivy schema in --packages, too?

Currently we support --packages with maven as --package group:module:version,etc...  and will download all transitive dependency jars, it's better to control each jars's transitive parameters.  Support ivy schema  URI can be more flexible, I will work on this after  this merged and refactor related code https://github.com/apache/spark/pull/29966#discussion_r531917374 https://github.com/apache/spark/pull/29966#discussion_r533868828

Okay, at least it makes it Hive compliant per HIVE-9664. I am okay.

Wait..seems we all ignore how can we pass these Jars to executor side....

Test build #132321 has started for PR 29966 at commit afea73f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36921/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36921/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36921/

Test build #132321 has finished for PR 29966 at commit afea73f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132321/

retest this please

Test build #132337 has started for PR 29966 at commit afea73f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36937/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36937/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36937/

Test build #132337 has finished for PR 29966 at commit afea73f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132337/

I'm neutral on this, but we don't basically merge a new feature into the release branch (v3.1) after the branch-cut.

Yeah, I was thinking we had one more day before the branch cut -- had my dates a little off. Looks like we missed it. Thanks for checking @maropu

Wait..seems we all ignore how can we pass these Jars to executor side....

Good catch! Approach LGTM, one minor comment.

Test build #132740 has started for PR 29966 at commit 13000f2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37341/

Test build #132744 has started for PR 29966 at commit d53f302.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37341/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37341/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37344/

Test build #132740 has finished for PR 29966 at commit 13000f2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132740/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37344/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37344/

Test build #132744 has finished for PR 29966 at commit d53f302.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132744/

Test build #132768 has started for PR 29966 at commit d53f302.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37370/

Test build #132768 has finished for PR 29966 at commit d53f302.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132768/

retest this please

Test build #132792 has started for PR 29966 at commit d53f302.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37393/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37393/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37393/

Test build #132792 has finished for PR 29966 at commit d53f302.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132792/

Test build #132819 has started for PR 29966 at commit 57c351d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37421/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37421/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37421/

Test build #132819 has finished for PR 29966 at commit 57c351d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132819/

retest this please

Test build #132831 has started for PR 29966 at commit 57c351d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37433/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37433/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37433/

@maropu or @mridulm , would you be able to help merge this if there are no more outstanding concerns?

Test build #132831 has finished for PR 29966 at commit 57c351d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/132831/

@maropu or @mridulm , would you be able to help merge this if there are no more outstanding concerns?

Gentle ping @maropu @mridulm  Is this pr ok? hope this merge and start work on  https://github.com/apache/spark/pull/29966#discussion_r531917374

This PR itself almost looks fine to me, but I'm not familiar with this feature in Hive. So, I think it's best to have someone familiar with it checking it before merging. cc: @viirya @wangyum

Test build #133198 has started for PR 29966 at commit 8c53b83.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37797/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37797/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37797/

Test build #133198 has finished for PR 29966 at commit 8c53b83.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133198/

Test build #133210 has started for PR 29966 at commit 4048c5b.

Test build #133210 has finished for PR 29966 at commit 4048c5b.

This patch fails to build.
This patch merges cleanly.
This patch adds the following public classes (experimental):
          assert(e.contains(\"Can not load class 'org.apache.spark.examples.sql.Spark33084\"))

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133210/

Test build #133211 has started for PR 29966 at commit 2ffb431.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37808/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37808/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37811/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37808/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37811/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/37811/

Test build #133211 has finished for PR 29966 at commit 2ffb431.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133211/

Test build #133276 has started for PR 29966 at commit 6bd41cd.

Hi, @AngersZhuuuu. This PR looks mostly ready as @maropu wrote. I left a few comments.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37874/


fix: deserialize ColumnSynchronize.precision correctly from "columnPrecision"

resolves #308

Coverage decreased (-0.07%) to 81.253% when pulling a33fad5369f908f70b7d472323a0d5989b9a9d85 on jmi-colsync into 0317c0b488fda119b2b41a583b7d43f75444706a on master.

LGTM


iio node fails to initialize in time before called

Issue Description
Would like to use node constant/empty to tick sensor read gyro() twice. But it fails since iio node haven't finished initialized before the TICK
Steps to reproduce

refer sample from https://github.com/solettaproject/soletta/wiki/Logic-constructs-on-Soletta-flows and compose gyroscope sensor data read code as following:
#!/usr/bin/env sol-fbp-runner
tick_1(constant/empty)
tick_2(constant/empty)
gyro(iio/gyroscope:iio_device="mpu6050 create,i2c,platform/80860F41:05,0x68,mpu6050 0",buffer_size=20)
tick_1 OUT -> TICK gyro
tick_2 OUT -> TICK gyro

run it using command: export SOL_LOG_LEVEL='DBG' && ./test_mpu6050_tick_3.fbp

Board information
Host OS and architecture:
Target OS and platform:
Ostro/MinnowMax
Logs
Since pastebin temporarily unavailable from my side, I put log here:
INF:sol-platform ./src/lib/common/sol-platform-linux-common.c:960 sol_platform_impl_load_locales() The locale file (/etc/locale.conf) was not found in the system.
INF: ./src/shared/sol-util-file.c:419 sol_util_iterate_dir() Could not open dir [/etc//soletta/boards/] to iterate: No such file or directory
DBG:sol-platform ./src/lib/common/sol-platform.c:193 sol_platform_get_board_name() detected board name=intel-minnow-max-linux_gt_3_17
DBG:sol-platform ./src/lib/common/sol-platform.c:216 sol_platform_get_board_name() using board name=intel-minnow-max-linux_gt_3_17
INF:sol-pin-mux ./src/lib/common/sol-pin-mux.c:82 _load_mux() Could not load platform pin multiplexer '//usr/lib/soletta/modules//pin-mux//intel-minnow-max-linux_gt_3_17.so': //usr/lib/soletta/modules//pin-mux//intel-minnow-max-linux_gt_3_17.so: cannot open shared object file: No such file or directory
DBG:sol-update ./src/lib/common/sol-update.c:120 get_first_module_on_dir() Invalid update module dir: //usr/lib/soletta/modules//update/
DBG:sol-update ./src/lib/common/sol-update.c:152 load_update_module() No update module to load
DBG:sol-mainloop ./src/lib/common/sol-mainloop.c:187 sol_init() Soletta 0.0.1 on linux-4.1.15-yocto-standard initialized
DBG:sol-mainloop ./src/lib/common/sol-mainloop.c:229 sol_run() run
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='constant/empty' resolved using resolver=builtins_resolver
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='constant/empty' resolved using resolver=parse-state-resolver-with-declares
DBG:sol-flow ./src/lib/flow/sol-flow-builder.c:519 sol_flow_builder_add_node() Node tick_1 added: type=0x7fcd883570a0, opts=0x62a940.
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='constant/empty' resolved using resolver=builtins_resolver
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='constant/empty' resolved using resolver=parse-state-resolver-with-declares
DBG:sol-flow ./src/lib/flow/sol-flow-builder.c:519 sol_flow_builder_add_node() Node tick_2 added: type=0x7fcd883570a0, opts=0x62ae60.
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:114 sol_flow_resolve() could not resolve module for id='iio/gyroscope' using resolver=builtins_resolver
DBG:sol-flow ./src/lib/flow/sol-flow-resolver-conffile.c:210 _resolver_conffile_get_module() module named 'iio' loaded from '//usr/lib/soletta/modules//flow//iio.so'
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='iio/gyroscope' resolved using resolver=conffile
DBG:sol-flow ./src/lib/flow/sol-flow-resolver.c:118 sol_flow_resolve() module for id='iio/gyroscope' resolved using resolver=parse-state-resolver-with-declares
DBG:sol-flow ./src/lib/flow/sol-flow-node-options.c:773 fill_options_with_named_options() Parsed option "iio_device" member #0 name="iio_device", type="string", offset=8, size=8
DBG:sol-flow ./src/lib/flow/sol-flow-node-options.c:773 fill_options_with_named_options() Parsed option "buffer_size" member #1 name="buffer_size", type="int", offset=16, size=4
DBG:sol-flow ./src/lib/flow/sol-flow-builder.c:519 sol_flow_builder_add_node() Node gyro added: type=0x7fcd8281b7a0, opts=0x658df0.
DBG:sol-flow ./src/lib/flow/sol-flow-builder.c:950 sol_flow_builder_get_node_type() Node type 0x658ff0 created
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-test_mpu6050_tick_3-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-test_mpu6050_tick_3.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-test_mpu6050_tick_3-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-test_mpu6050_tick_3.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: ./sol-flow.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: /etc//soletta//sol-flow-test_mpu6050_tick_3-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: /etc//soletta//sol-flow-test_mpu6050_tick_3.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: /etc//soletta//sol-flow-intel-minnow-max-linux_gt_3_17.json
DBG: ./src/shared/sol-conffile.c:724 _load_json_from_paths() Trying to load conffile: /etc//soletta//sol-flow.json
DBG:sol-flow ./src/lib/flow/sol-flow-static.c:242 connect_nodes() Making 2 connections.
WRN: ./src/lib/io/sol-iio.c:1090 sol_iio_device_trigger_now() device == NULL
WRN:sol-flow-iio ./src/modules/flow/iio/nodes.c:192 gyroscope_tick() Could not read channel values
WRN: ./src/lib/io/sol-iio.c:1090 sol_iio_device_trigger_now() device == NULL
WRN:sol-flow-iio ./src/modules/flow/iio/nodes.c:192 gyroscope_tick() Could not read channel values
DBG: ./src/lib/io/sol-iio.c:1385 create_or_resolve_device_address_dispatch() IIO device creation/resolving dispatching command: mpu6050
WRN: ./src/lib/io/sol-iio.c:357 set_buffer_size() Could not set IIO device buffer size
DBG: ./src/lib/io/sol-iio.c:710 sol_iio_open() iio device created. device0 - buffer_enabled: 1 - manual_trigger: 1 - trigger_name: mpu6050-dev0 - trigger_id: 0
INF: ./src/lib/io/sol-iio.c:852 channel_get_offset() Could not get offset for channel [in_anglvel_x] in device0. Assuming 0
DBG: ./src/lib/io/sol-iio.c:969 sol_iio_add_channel() channel [in_anglvel_x] added. scale: 0.001065 - offset: 0 - storagebits: 16 - bits: 16 - mask: 65535
INF: ./src/lib/io/sol-iio.c:852 channel_get_offset() Could not get offset for channel [in_anglvel_y] in device0. Assuming 0
DBG: ./src/lib/io/sol-iio.c:969 sol_iio_add_channel() channel [in_anglvel_y] added. scale: 0.001065 - offset: 0 - storagebits: 16 - bits: 16 - mask: 65535
INF: ./src/lib/io/sol-iio.c:852 channel_get_offset() Could not get offset for channel [in_anglvel_z] in device0. Assuming 0
DBG: ./src/lib/io/sol-iio.c:969 sol_iio_add_channel() channel [in_anglvel_z] added. scale: 0.001065 - offset: 0 - storagebits: 16 - bits: 16 - mask: 65535
WRN:sol-flow ./src/lib/flow/sol-flow-static.c:337 flow_send_do() Error packet '5 (Could not read channel values)' sent from './test_mpu6050_tick_3.fbp (0x6590e0)' was not handled
WRN:sol-flow ./src/lib/flow/sol-flow-static.c:337 flow_send_do() Error packet '5 (Could not read channel values)' sent from './test_mpu6050_tick_3.fbp (0x6590e0)' was not handled

Hi @yma11,
Please, check if #1715 fix the issues. Note however that I added a note to sample you refer to: it may not always work. Some sensors need a time between two consecutive readings.

@yma11: have you had the time to check that for us? We will assume it's fixed soon and you'll have to re-open if that's not the case. Thanks.

@glima: sorry to reply you late but can you tell me how can I verify the fix works when this PR in open status? can I get a soletta image which includes this fix? thanks.

@yma11, yes: https://github.com/solettaproject/soletta/releases/tag/v1_beta19

@yma11 I'll close this issue. If you happen to find it again, feel free to reopen it.


Cleanup just_entrypoint* comments

Cleanup just_entrypoint.sh and just_entrypoint_user_functions comments as per PR #153

Duplicate of #154


Quickstart: riscof testlist: argument requirements unclear

According to https://riscof.readthedocs.io/en/stable/installation.html#running-riscof riscof testlist doesn't need the arguments --suite and --env but this seems to be incorrect since the command riscof testlist --config=config.ini results in this output:
error: the following arguments are required: --suite, --env
usage: riscof testlist [-h] [--work-dir PATH] [--config PATH] --suite PATH
                       --env PATH

optional arguments:
  --config PATH    The Path to the config file. [Default=./config.ini]
  --env PATH       The Path to the custom env directory.
  --suite PATH     The Path to the custom suite directory.
  --work-dir PATH  The Path to the work-dir.
  -h, --help       show this help message and exit

But as you see the help text is a bit of a mix, both listing them as required and optional.

@kaddkaka Yes the quickstart is a little dated. We are working improving the docs on this branch. Will have them updated and merge soon.
Also the help-command doesn't yet distinguish between required and non-required arguments. I think we will be moving to click based cli to ensuer a better help screen.

https://riscof.readthedocs.io/en/plugin-doc-update/installation.html#quickstart following this should solve your problems

Thanks @neelgala this updated documentation suggests this command:
riscof testlist --config=config.ini --suite=riscv-arch-test/riscv-test-suite/ --env=riscv-arch-test/riscv-test-suite/env
But what path is actually --suite=riscv-arch-test/riscv-test-suite/ ? For example for the spike template, what should the command look like?
riscof testlist --config=config.ini --suite=??? --env=spike/env

the branch with docs has been merged . Closing this issue.


Update notebooks snapshots and cypress

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Notebooks CI is fixed as seen here: https://github.com/ps48/dashboards-observability/actions/runs/7566656455/job/20604374203
The current CI failure are due to commits added in the main last PR and some upstream changes will create a separate PR to fix the CI changes.


Sort order of nodes

I'm testing your newest branch. I found that when there are more than 10 childs in one node, the sorting goes wild. I guess that .sort function orders by string in the data-nodeid, so it goes from 0.13.1 to 0.13.10 and 0.13.11 instead of 0.13.1 to 0.13.2 and so on....

For me I fixed it by filling up the id with zeros for making it work with up to 999 items:

I addid this line in the initialization:
node.nodeId = (parent && parent.nodeId) ? parent.nodeId + '.' + this._zeroFill(node.index,3) :
(level - 1) + '.' + this._zeroFill(node.index,3);

and adding this function:
	Tree.prototype._zeroFill = function zeroFill( number, width ) 	{ 	  width -= number.toString().length; 	  if ( width > 0 ) 	  { 		return new Array( width + (/\./.test( number ) ? 2 : 1) ).join( '0' ) + number; 	  } 	  return number + ""; // always return a string 	}
I'm not really a developer, so I don't feel, that this can be a real solution, but I wanted to point of the issue. Love the work put into it though!
Best Regards
Charles

Thank you. This is indeed why my sort seemed off.  For a solution I ended up pulling the function by LeJared from http://stackoverflow.com/questions/6832596/how-to-compare-software-version-number-using-js-only-number and updating _sortNodes as follows:
    Tree.prototype._sortNodes = function () {
        return $.map(Object.keys(this._nodes).sort(cmpVersions), $.proxy(function (value, index) {
            return this._nodes[value];
        }, this));
    };


build(deps): avoid loading json 2.7.0

Avoid loading release 2.7.0 of the json gem as it contains a backward-compatibility issue.
(Basically, on 2.7.0 you can't do JSON.dump(a:1, b:2, c:3) but you need to do JSON.dump({ a:1, b:2, c:3 })
Refrences:

Issue => https://github.com/flori/json/issues/553
PR => https://github.com/flori/json/pull/554 and https://github.com/flori/json/pull/556

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Not sure if I should add a Changelog entry for this 🤔

Tests for RuboCop pass with the matrix of JSON 2.7.0 and Ruby 2.7, and it works. This means that instead of avoiding the version of JSON on the RuboCop side, it seems best to handle it on applications and client side. Thank you.


There is a dead link to get PossessProperty xml schema

This link is dead:
http://schemas.xmlsoap.org/ws/2005/05/identity/right/PossessProperty

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

ID: ee00e326-b334-8b60-2283-e508275208e2
Version Independent ID: 14e26578-4f61-3776-30d0-04832caa6a56
Content: Rights.PossessProperty Property (System.IdentityModel.Claims)
Content Source: xml/System.IdentityModel.Claims/Rights.xml
Service: unspecified
Product: .net
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@karelz can you help us find who owns this API?

Right now the URL http://schemas.xmlsoap.org/ws/2005/05/identity redirects to http://docs.oasis-open.org/imi/ns/identity-200810.

@brentschmaltz can you comment on this? Do we need to update the product or obsolete this API?

@brentschmaltz can you comment on this?

Ping @brentschmaltz. Is it an issue that now all the URI links are broken for these kind of URLs used in .NET Framework?

@JRAlexander @mairaw it is an issue, but I would prioritize a fix as very low. This particular issue is referring to a property on the first claims model that arrived out of WCF in 2006.
That model was replaced with System.Security.Claims.Claim in 2010.
If something was missing in S.S.C.Claim, that would have a higher priority.


[Event Parameter Request] Table 37 Sales Line - procedure UpdateVATAmounts

Dear ALAppExtensions Team,
We would like to request the following change
procedure UpdateVATAmounts()
begin
  ...............

  **//--> change request
  OnAfterUpdateVATAmounts(Rec,Currency);
  //<-- change request**
end;

[IntegrationEvent(false, false)]
local procedure OnAfterUpdateVATAmounts(var SalesLine: Record "Sales Line"; Currency: Record Currency)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46663.


Reverse DNS for internal IPs.

The URL referencing this has no information on getting it working.  How do we generate PTR records for our internal DNS from Azure DHCP? A solution to this was promised 2 years ago and I haven't seen anything.

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

ID: a7b1b215-0434-145b-cd11-95d0f32bc015
Version Independent ID: d328a868-a151-1985-2b62-f3884f7ad76c
Content: Name resolution for resources in Azure virtual networks
Content Source: articles/virtual-network/virtual-networks-name-resolution-for-vms-and-role-instances.md
Service: virtual-network
GitHub Login: @rohinkoul
Microsoft Alias: rohink

@hawkbox Thanks for the feedback. As is stated, this is only possible if you are using a custom DNS server, and not Azure's DNS server. This is typically done using a reverse lookup zone on the DNS server.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

If you would like us to add this feature in the future, please leave your feedback at Azure Networking's feedback.azure.com page!


Latency alerts include no unit of measurement

Alert Description
Host Read Latency (29438) exceeds the configured monitoring threshold.
Please include the unit of measurement in the alert as I do not think this is in 'ms' as stated in the documentation.

I believe this is corrected in <IP_ADDRESS>.

It was not corrected in the latest release. I will create an RFE to add this in the parameter description.
Latency metrics are in microseconds (usec).

Will be corrected in next release.


[FEATURE]: support for gradient clipping by value

Describe the feature
Is there any plan for supporting clip_grad_value in Pytorch?

Can you post more information about the PyTorch clip_grad_value? How to use it? Maybe some code snippet is more helpful.

Hi @haofanwang , its torch version shall work well with ColossalAI. In addition, we have our own clip_grad_norm in colossalai.utils.common for the correctness of model parallelism.


[mesheryctl] Unhelpful error when token is missing

See mesheryctl Command Tracker for current status of commands.
Current Behavior

Running mesheryctl perf list without a token in .meshery folder throws this error:

Desired Behavior

This error is not helpful as it does not give any information to the user on how to get a token. Asking the user to provide the token through the --token flag instead of authenticating and automatically downloading the token to the .meshery folder would also result in a bad UX.
Instead it should show:
mesheryctl system login to login to Meshery and grab a token automatically.
Link to a doc which talks about authenticating to local/remote provider.
Environment

Host OS: Mac
Meshery Server Version: edge-latest
Meshery Client Version: edge-latest
Platform: Docker

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Hey @navendu-pottekkat I want to give it a try !

🚀  Too Fast, Too Furious, Too Bawse!

This sounds interesting

@midepeter I believe @piyushsingariya has fixed this issue in the PR linked above.

Just saw that...

Cool