Closed ajstewart closed 4 years ago
Hello @ajstewart! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
@ddobie I think this should be merged as is.
The loading of jobs will need to be changed again, but the two epoch calculation is going to be moved to the pipeline: https://github.com/askap-vast/vast-pipeline/pull/305 also to load the whole pilot survey uses too much memory. I am experimenting with using Vaex to overcome this problem.
But at least these fixes will make the current process faster.
Release candidate 3 of version 2.0.0.
Addresses #207. Fixes #209.