Fix for #146#147
Conversation
|
This is a temporary fix that shouldn't affect anything else. More thorough testing of existing code will probably be required to prevent this from happening in the future. |
|
I was thinking it might just be missing an |
|
I think the problem with such a dispatch is that it doesn't cover colons? Also, I am not sure which code you are referring to. The given link is for this PR's diff. |
|
Good point. It may need some of these: https://github.com/SciML/SciMLBase.jl/blob/master/src/solutions/solution_interface.jl#L20-L29 |
|
@ChrisRackauckas |
|
Yeah, let's merge and see. Today isn't going to be a clean day. |
Fixes #146
CC: @ChrisRackauckas