-
Notifications
You must be signed in to change notification settings - Fork 9
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
Inconsistent treatment of AperturePhotometry <--> AutoPhotometry #370
Comments
Maybe I would rather add back the aperture config to the initial config file. It seems more usable. However, it can become confusing if things are specified twice (python and config file). I would rather fail quick giving a descriptive error message in that case. i.e.
The detail here is that when there is no Python config file, the detection image is kind of "duplicated". Internally there is still Since units are generally given in detection-frame pixels, that's why check-images are generated twice: the aperture in your detection frame, and the aperture projected into the measurement frame. Which happens to be the identity projection. This is the explanation. It may be worth adding a method that can be checked so the properties know to generate things |
Adding back aperture config to the ASCII config is certainly the better solution. Many users do not need to work with measurement images but still are interesting having aperture photometry. |
Aperture photometry on the detection image is possible. The name for the aperture parameter and the output-properties point to the detection image. |
The output-properties AperturePhotometry and AutoPhotometry are treated differently:
I would be more consistent to either making measurement images mandatory also for AutoPhotometry or (re-)introducing using the detection image for AperturePhotometry when there are no measurement images (which would require introducing an aperture parameter).
The confusion extends also to the checkimages. In my example with only the detection image and giving:
--check-image-aperture=aper.fits --check-image-auto-aperture=autoaper.fits
results in four images:
For both photometries there is one checkimage referring to the detection image and one referring to the measurement images. This is rather confusing.
The text was updated successfully, but these errors were encountered: