-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change output file naming scheme #403
Conversation
@ajstewart can you take a look at this before I update of the docs etc? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good to me!
I don't know the answer but is the find_sources
script ok with it? Is there anything there that will break when it attempts to save all the cutouts?
Co-authored-by: Adam Stewart <ajstewart@users.noreply.github.com>
Co-authored-by: Adam Stewart <ajstewart@users.noreply.github.com>
It seems to be okay - I've run a few test queries without any issues |
This pull request changes the naming scheme for output files (
.fits
,.png
,.ann
and.reg
) to allow multiple files to be saved for each epoch. Files built from pipeline data will be saved in the formSOURCENAME_INDEX.ext
, while files built from regular data will be saved in the formSOURCENAME_FIELDNAME_SBID.ext
This also changes the way that various functions inside
vastools.source.Source
are called, from being called based on epoch to being called based on index, and removing references to epochs in titles.