search

askap-vast / vast-tools

A collection of tools that are useful for the VAST project and for exploration of results from the VAST Pipeline.
https://vast-survey.org/vast-tools/
MIT License
8 stars 0 forks source link

Added recalc_sources functionality to pipeline #235

Closed ajstewart closed 4 years ago

ajstewart commented 4 years ago

Adds the ability to define a filtered measurements dataframe and recalculate the source statistics.

Fixes #226.

vast-bot commented 4 years ago

Hello @ajstewart! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1376:46: E712 comparison to True should be 'if cond is True:' or 'if cond:'

Comment last updated at 2020-11-19 06:47:34 UTC
ajstewart commented 4 years ago

Hello @ajstewart! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1273:46: E712 comparison to True should be 'if cond is True:' or 'if cond:'

Comment last updated at 2020-11-18 08:38:37 UTC

The is condition has to be ignored as it's inside a pandas filter.