Skip to content

Conversation

jlapeyre
Copy link
Contributor

@jlapeyre jlapeyre commented Dec 7, 2018

  • For example, charA(q, k::Integer) returns a single
    floating-point value.

  • The test suite prints the distance from the expected result
    for each test.

Note This also breaks the previous interface. So any code depending on this package would have to be changed. (The change is very small.) Having k as a positional argument rather than a keyword argument allows a cleaner interface.

* For example, `charA(q, k::Integer)` returns a single
floating-point value.

* The test suite prints the distance from the expected result
for each test.
@jlapeyre jlapeyre changed the title Add methods for integer-valued order Add methods for Integer-typed order Dec 8, 2018
@jlapeyre
Copy link
Contributor Author

jlapeyre commented Dec 8, 2018

I changed the title from Add methods for integer-valued order to Add methods for Integer-typed order. The methods already accepted integer-valued k. The PR changes the interface. It now accepts not only an AbstractRange of integers, but also a single Integer.

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