Option "get all" in Add Layer --> Time Step crashes

[Jens Rodmann]

When adding another layer ("Add Layer") and selecting "get all" from the Time Step, the user gets the following error message:

Error getting the data
Invalid cadence specified. Cadence must be greather than 0.

Proposed solution:
JHelioviewer should set the cadence to the smallest possible value.

Thanks for having a look at this one...

Jens Rodmann

[Keith Hughitt (keith-hughitt)]

Thanks for reporting the issue! It should also be possible to fix this from Helioviewer.org's side just have anything less than or equal to 0 return all images in the range.

[Daniel Mueller (dmueller-esa)]

The current release of JHV sends a "cadence=-1" in the API request if a user selects "get all" as cadence. However, this does not work with the API as-is ("Error getting the data. Invalid cadence specified. Cadence must be greather than 0.").

@Jeff: Could you maybe modify the API to make 'cadence=-1' work, e.g. by having the API return all images if cadence <= 0 ?

[Jeff Stys (jstys-z)]

I have updated the 'getJPX ' API action to accept 'cadence' parameter values <= 0.

The cadence gets set to 1 second internally, which effectively means return all available images.

If this results in a calculated number of frames that exceeds HV_MAX_JPX_FRAMES, then the cadence is automatically re-calculated to fit within HV_MAX_JPX_FRAMES. Note that HV_MAX_JPX_FRAMES is currently defined as in the settings/Config.ini file as:

    max_jpx_frames = 1000

Please let me know if this resolves the issue.