ac2cz / FoxTelem

AMSAT Fox-1 Telemetry Decoder
GNU General Public License v3.0
52 stars 17 forks source link

Graphs no longer re-open on start #742

Closed ac2cz closed 2 years ago

ac2cz commented 2 years ago

If a graph is open when you close FoxTelem it is supposed to re-open when you restart. We do not seem to save the state to the properties file. At least on Ubuntu that is the case.

ac2cz commented 2 years ago

This seems to be more than one bug. I have made sure the open status us saved and the graphs are reopened. It works for most, but there are two situations where it still fails:

  1. When we iterate the tabs we assume they are ModuleType and that fails for Measurement tabs.
  2. When we save the graph status it includes the payload type. But this seems to then be different to the payload type when we try to load at startup from the DisplayModule. e.g. when I open the health tab the rtPayload.getType() returns 1. But when IHU CPU temp is saved it has payload Type 2 and has the string: Graph1002IHUcpuTempopen=true
ac2cz commented 2 years ago

Ok, in line with V3 FOXDB it was setting the payloadType to the layout number in the graph and then saving the data with that. So now I load with the same. Need to confirm that works for a legacy spacecraft

ac2cz commented 2 years ago

This is fixed in 1.12w1 and later.