FrSkyRC / ETHOS-Feedback-Community

Feedback & suggestions are welcomed here for ETHOS by FrSky
188 stars 85 forks source link

Log viewer enhancements #3877

Open Jedsters opened 5 months ago

Jedsters commented 5 months ago

When opening a .csv log file in File Manager you are presented with a Log Viewer, Reading file picture with the option to CLOSE on the right of it, however as best I can work out the CLOSE option doesn't work on the touch screen, although it does stop loading if you press the physical enter button.

Also when you view the log there are red horizontal lines across it but there is no indication as to what values these represent. While I can see that could be problematic if you have selected multiple curves, it would be very useful to quickly see whether you e.g. have a problem or not when 1 curve is selected by showing the values for the red lines as well as perhaps differently coloured horizontal lines for the top and bottom values together with their values.

As an example, see the following RSSI curve from a flight today - do I have a problem or not? The only way to find out is to try and work out which is the lowest point (fairly easy in this case) navigate to it by zooming in far enough that I can reliably touch the appropriate point. Then to check other points, zoom out, navigate to them and again zoom in again.

image

Thanks

miff69 commented 5 months ago

Perhaps my suggestion (https://github.com/FrSkyRC/ETHOS-Feedback-Community/issues/3648) would be a solution to your problem?

Jedsters commented 5 months ago

I like your suggestion and agree that having a min, avg (which average?) and max displayed under the graph would be nice and I'd quite happily accept that as an alternative to indicating on the graph directly the min and max values. I think for one line you could easily accommodate this information at the bottom / top, but am inclined to agree that if there are multiple lines then another way would need to be used to display it.
Actually, looking at the above picture, I reckon there is enough space in the existing box (in this case showing RSSI 89db) to add your request, maybe divide the box into 4 and have current point, min, max and avg all in that box but in smaller non bold text, then you could accommodate all of the information for all of the lines.

Although having a graph without any numbers on the Y axis does reduce its usefulness a lot so I'd still like to see a Y axis showing what values the red lines represents - at least if only one line is being displayed. (Great if there is a way to display Y axis for multiple lines but I have a feeling that it would over complicate the graph trying to get that information onto it.) Reminds me of my ex employer who was notorious for showing us financial graphs with no numbers on the Y axis and a gap at the bottom of the Y axis - gave no information at all other than whether something was going up or down.

Thanks

mirko91 commented 5 months ago

When opening a .csv log file in File Manager you are presented with a Log Viewer, Reading file picture with the option to CLOSE on the right of it, however as best I can work out the CLOSE option doesn't work on the touch screen, although it does stop loading if you press the physical enter button.

Also when you view the log there are red horizontal lines across it but there is no indication as to what values these represent. While I can see that could be problematic if you have selected multiple curves, it would be very useful to quickly see whether you e.g. have a problem or not when 1 curve is selected by showing the values for the red lines as well as perhaps differently coloured horizontal lines for the top and bottom values together with their values.

As an example, see the following RSSI curve from a flight today - do I have a problem or not? The only way to find out is to try and work out which is the lowest point (fairly easy in this case) navigate to it by zooming in far enough that I can reliably touch the appropriate point. Then to check other points, zoom out, navigate to them and again zoom in again.

image

Thanks

Sorry for the question, but how do zoom in? I have an x18se and can't find a way to do so

Jedsters commented 5 months ago
  1. The display can be zoomed in or out by rotating the scroll wheel while holding down the page key.

All nicely described on p.254 of the 1.5.5 Ethos manual

github-actions[bot] commented 2 months ago

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.

Jedsters commented 2 months ago

Has this been addressed in later versions of ethos?