Matthew Middlehurst

Results 287 comments of 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?