Skip to content
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.

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

Adds tests quant parsing; Adds unbounded ranges #5

Merged
merged 2 commits into from
Jul 6, 2018
Merged

Conversation

cirocosta
Copy link
Owner

@cirocosta cirocosta commented Jul 6, 2018

#2


commit 1f94258d9b3691b057595b6030f869911147b194

  Previously it wasn't possible to make use of unbounded ranges in the
  CLI: --range 1 would be equivalent to --range 1,0.
  After this comming, such ranges are possible: --range 1 is equivalente
  to --range 1,math.MathFloat64.

  The commit also adds tests for the parsing method given that it's not
  trivial.

commit fe04829d7724a0734c8fcd2ae97442a460b2e536  
  Applies negative value constraint to range parse

Ciro S. Costa added 2 commits July 6, 2018 09:12
Previously it wasn't possible to make use of unbounded ranges in the
CLI: --range 1 would be equivalent to --range 1,0.
After this comming, such ranges are possible: --range 1 is equivalente
to --range 1,math.MathFloat64.

The commit also adds tests for the parsing method given that it's not
trivial.
@cirocosta cirocosta merged commit c33b7f4 into master Jul 6, 2018
@cirocosta cirocosta deleted the fix-quantize branch July 6, 2018 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant