New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix formatting in IntervalQueryParam class #223

Closed

andamian opened this issue Apr 21, 2020 · 0 comments

Assignees

Labels

bug

Milestone

Contributor

andamian commented Apr 21, 2020

Couple of issues with the formatter:

it doesn't handle correctly the (low, high)*unit case
it returns an error if the interval low>end. However some units conversions (spectral for example) might result in invalid intervals. The fix is to check the interval only for default units and autocorrect the other cases.

andamian added the bug label

andamian added this to the v1.1 milestone

andamian self-assigned this

This was referenced Apr 21, 2020

Fixed interval checks #224

Closed

Fixed formatter in IntervalQueryParam #225

Merged

andamian closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment