daraghhollman / jupt Goto Github PK

The data shortening doesn't work when only one Waves file is loaded.

Need to add case for when i == 0 and i == len(filesWithInfo)-1

Attempting to run default_config.ini results in the error:

ValueError: 
Lon$_{III}$ ($^\circ}$)
             ^
ParseException: Expected end of text, found '$'  (at char 13), (line:1, col:14) 

Something strange to note is that this did not occur the first time I ran the default config, but has persisted even after a clean install of JUPT.

Escaping the $ symbols as \$ seems to fix the issue.

Currently figure sizes are not set, and it is left for the user to manually size them. This makes it difficult to compare between saved figures

Plotting from 07:35 to 07:40 on 2019-05-29

Printing timeTransformed in junoEphemeris for Waves:

The same for MAG:

Its not fun to wait several minutes for a large plot to complete only to get an error about how you inputted the label for your vertical lines wrong.

Add functions to read and plot Juno/Waves (burst mode) data from PDS

Much of what is printed to the terminal is not useful for the general user. The print outs should be limited to what is important, with the rest being hidden behind a 'verbosity' config parameter or command flag.

Instead of having the user install external tools for features (such as the jupiter_magnetospheres_boundaries repo), it might be possible to include them in this repository as a submodule

This can be currently avoided by manually increasing the timeframe, however it would be nice if a workaround could be found.

As the PDS urls may change time to time, a test should be added at the start of the tool, to test if the url are still active, or not.

Issue in CodeSnippets/cdfExample.py:

unit = cdf_file.varinq("Epoch")["Data_Type_Description"] doesn't work in cdflib 1.0.3.

This is avoidable by using the new syntax unit = cdf_file.varinq("Epoch").Data_Type_Description, however, this conflicts somehow later in the script with astropy throwing an error that the cdf time format doesn't exist:

time = Time(epoch, format="cdf_tt2000")

Temporarily this means we are required to use cdflib==0.4.4 - as adjusted in requirements.txt - although some testing could be carried out to see what is the latest version compatible.

The section "limitations" in the docs still describes the tool as non-function on windows (barring WSL). This needs to be changed.

As JUPT grows in features, the config file rapidly fills with the necessary options for such additions. However, features outside of the basic functionality - while useful when needed - can clutter up the config files from the more basic (and required) options.

For example, vertical lines are not always needed, yet reference to them is required in all config files.

One solution could be to have the tool default to these already default settings and only change them if there are entries in the config file.

Ephemeris data labels (i.e. R_j, lon, lat, ...) are currently positioned manually which makes them out of place for changes in fontsize or number of panels

The JADE panel displays as if the full time is plotted however it instead simply stretches the data available in either high or low res mode to the full width of the panel.