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

Compatibility with Julia 0.4 #13

Closed
waldyrious opened this issue Jan 7, 2017 · 6 comments
Closed

Compatibility with Julia 0.4 #13

waldyrious opened this issue Jan 7, 2017 · 6 comments

Comments

@waldyrious
Copy link
Collaborator

waldyrious commented Jan 7, 2017

Currently testing in 0.4 fails: https://github.com/waldyrious/Tau.jl/pull/1

To fix this, Compat macros will almost certainly be needed.

See #12 for context.

waldyrious added a commit that referenced this issue Jan 7, 2017
Restoring compatibility with 0.4 is tracked in #13.
@giordano
Copy link
Member

giordano commented May 9, 2017

Is support for Julia 0.4 still wanted? If not, the two @vectorize_1arg call can be removed.

@waldyrious
Copy link
Collaborator Author

waldyrious commented May 9, 2017

I would like to try it, as a personal exercise (I haven't used Compat.jl yet) but I wouldn't want to compromise the code's readability or simplicity too much for that.

Considering that 0.6-rc1 has been tagged, I suppose it makes sense to start considering Julia's past, current, and next releases to be 0.5, 0.6 and 0.7, respectively. So if you think it makes sense to do this, feel free to go ahead -- no objections on my part.

@giordano
Copy link
Member

giordano commented May 9, 2017

Personally I'd go ahead, the package will continue to work on Julia 0.4 anyway and if necessary bug fixes can be backported. There won't be new features for that version of Julia, but I don't think this simple package is going to receive a great load of new functions.

Anyway, from what I can see, the only test that doesn't pass on Julia 0.4 is the first one, @test_broken tau == 2*pi. Just change it?

@waldyrious
Copy link
Collaborator Author

Anyway, from what I can see, the only test that doesn't pass on Julia 0.4 is the first one, @test_broken tau == 2*pi. Just change it?

Is that all? Wow, I'm embarrassed to have missed that. I suppose we could just wrap that test with an if VERSION < v"0.5-" check, and do @test tau ≠ 2*pi in the else block, right?

@giordano
Copy link
Member

giordano commented May 9, 2017

Is that all?

Actually no. I just discovered an inconsistent behavior between Julia 0.4 and 0.5 in complex sin/cos. Will submit a PR in a few minutes.

Anyway, I would always test tau ≠ 2*pi The fact that tau is not equal to 2*pi makes this package useful, otherwise would be redundant.

@waldyrious
Copy link
Collaborator Author

Ok, let's close this when 0.6 is officially released. The badges in the readme will need to be updated then.

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

No branches or pull requests

2 participants