victronenergy / gui-v2

Other
24 stars 9 forks source link

SD card is not detected #1275

Closed blammit closed 3 days ago

blammit commented 2 months ago

When an SDCard is installed and being used for VRM logging, not correctly detected. See screen shot from the v1 and v2 GUI.

image image

blammit commented 1 month ago

@ReinvdZee to advise how this can be handled in gui-v2.

ReinvdZee commented 1 month ago

In gui-v1, the gui's backend connects to some dbus signals: https://github.com/victronenergy/gui/blob/master/platform/storage_events_dbus.cpp

Unmounting is done by requesting vrmlogger to do so: https://github.com/victronenergy/gui/blob/master/qml/MbMountState.qml

VrmLogger already also handles mount events; a storage medium will be used for logging when inserted; even without gui-v1 running.

I tested this on an Ekrano GX (V3.40~36) with gui-v2 running locally and it seems to work properly with SD and USB (both local and wasm display correct values):

image

@blammit is this still an issue?

blammit commented 1 month ago

@nmbath this is in regards to the issue you mentioned here. Could you please re-test with the latest version?

nmbath commented 3 weeks ago

Just checked my CerboGX, remote not locally, I am seeing what I would expect; External Storage Location, Free space, etc. This is on 3.50~13 firmware.

image
blammit commented 2 weeks ago

Thank you, will close this one then.

nmbath commented 1 week ago

@blammit, I think this needs re-opening

Looking more closely at the screen shot I provided it is showing that Storage is internal and not external. I have posted on slack that there are issues. https://victrondevelopment.slack.com/archives/C04RYU0UMDE/p1724420721369989

Summary: GUI when accessing remotely does not display the same as when viewing on Touch 50.

Touch 50 image

Remote

image

Further and not sure this is a GUI thing! If the SDCard is inserted it is found, following a reboot it is not detected.

blammit commented 1 week ago

Re-opening for investigation.

blammit commented 3 days ago

Discussed with @nmbath that he couldn't repro with the latest version.