Skip to content
This repository has been archived by the owner on May 4, 2019. It is now read-only.

Internal API changes due to 10525 #152

Merged
merged 3 commits into from
Jun 22, 2015
Merged

Conversation

mbauman
Copy link
Contributor

@mbauman mbauman commented Jun 4, 2015

There were a few internal APIs that I changed in JuliaLang/julia#10525. This is a start at fixing some of your uses of them. I think I got everything except a few assumptions about how Base.checkbounds will work with NAType, so this fails in a few places where you expect checkbounds to throw an NAException -- it is now a MethodError.

There is a lot more that you could do to adjust to a post-10525 world. Let me know if you have any questions along the way.

There were a few internal APIs that I changed in JuliaLang/julia#10525.  This is a start at fixing some.  I think I got everything except a few assumptions about how Base.checkbounds will work with NAType, so this fails in a few places where you expect checkbounds to throw an NAException -- it is now a MethodError.

There is a lot more that you could do to adjust to a post-10525 world.  Let me know if you have any questions along the way.
@mbauman
Copy link
Contributor Author

mbauman commented Jun 4, 2015

Actually, these should be accompanied with a Compat entry to allow things to continue to work on 0.3.

@mbauman
Copy link
Contributor Author

mbauman commented Jun 4, 2015

Ah, got it. Now the only remaining question is how to support these usages in 0.3. (Tsk tsk for using internal APIs). :)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 80.76% when pulling 36a5611 on mbauman:mb/10525 into b5d39c3 on JuliaStats:master.

@mbauman
Copy link
Contributor Author

mbauman commented Jun 11, 2015

I'm not sure that there's a sensible way to use Compat here since these aren't exported methods. And there were only two other packages that got tripped up on this (that weren't cascading failures).

CC: @davidanthoff for ExcelReaders.jl, @kbarbary for FITSIO.jl (FITSIO might be just the change to checkbounds)

This PR has the information you'll need to get your packages up and running again after JuliaLang/julia#10525 changed some internal APIs regarding indexing.

@mbauman mbauman changed the title WIP: Internal API changes due to 10525 Internal API changes due to 10525 Jun 11, 2015
@kbarbary
Copy link

@mbauman Thanks for the heads up, glad to see 10525 merged!

@alyst alyst mentioned this pull request Jun 14, 2015
@alyst
Copy link
Member

alyst commented Jun 14, 2015

Oh, good that I have found this PR! I was almost desperate because of the cryptic failure in DataFrames (JuliaData/DataFrames.jl#810). It would be nice to see this merged soon.

@axsk
Copy link

axsk commented Jun 22, 2015

Indeed #10525 broke my use of DataFrames completely. Thank you for this PR 👍, hope it will be merged soon

@nalimilan
Copy link
Member

Any reason not to merge? The reduction in coverage appears to be due to the parts conditional on the Julia version (as expected).

simonster added a commit that referenced this pull request Jun 22, 2015
Internal API changes due to 10525
@simonster simonster merged commit e7172be into JuliaStats:master Jun 22, 2015
@simonster
Copy link
Member

Thanks! Sorry for the slow merge.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants