-
Notifications
You must be signed in to change notification settings - Fork 17
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
Incorporate ITF1788 tests into the test suite #65
Comments
This could be by making the ITF1788 suite a (test) dependency of Can we just copy over the files that ITF1788 outputs and add them to our test collection? (And then update when necessary.) |
I guess this depends on the license; I really have no idea. Some numeric tests I implemented (#56 and #63) check if the result using EDIT: corrected a misprint... |
|
@oheim Once #56 and #63 are merged, I will push an update to your repo of the Regarding our tests, we can still check compatibility (off-line) with ITF1788, and maybe add a line to README.md (and in the docs) about the actual status, or a gist. |
I just saw #67; I'll do that list later today. |
Updated Julia API pushed; see oheim/ITF1788@964a67c |
@lbenet sounds great! You are really making progress. Please note that you might want to give lower priority to the operations that are only “recommended” in the standard document, compared to the required operations. |
Thanks for the advice; too late, I realized that :-) Incidentally, can you explain me what is the |
It has been planned to include the following function in the standard together with its interval extension: |
Having discussed a bit with @lbenet, I think the best thing to do is to directly include the |
cc @lbenet |
Looks good. Some minor remarks:
|
Thanks very much, @oheim. I'll implement your first 3 suggestions. For the last suggestion, we can certainly include the An alternative would just be to include the whole of your repository -- or, at least, the part that's relevant for Julia -- inside ours. |
So far, just the |
Can we somehow incorporate the relevant ITF1788 tests directly into our test suite?
I believe the Apache license is compatible with MIT?
Cc @oheim
The text was updated successfully, but these errors were encountered: