Matthew Middlehurst
Matthew Middlehurst
There have been a few distance updates recently, I think this would be outdated now? @chrisholder
using this as a test. @aeon-actions-bot assign @chrisholder
@aeon-actions-bot assign @chrisholder
Actually, with forecasting more officially deprecated now, I can probably remove the legacy check function. Can keep for now if people disagree.
I think deprecating a whole module is a big enough of a change to warrant it. I don't know what would be if not. I think it's best just to...
I wouldn't mind adding a test and checking to see if it's broken, but I'm not quite sure what the expected output is. Do you have an example of what...
See changes in #624. This skips the currently failing tests.
I think we have accomplished this somewhat with the `PR_TESTING` setup? I'm sure the individual tests could be improved a lot though.
Commented in #154 as well, I think we have accomplished this somewhat with the `PR_TESTING` setup.
Any reason for the new base class direction? Do they other base classes not have much shared functionality?