Skip to content
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

Attempt at Removing Apertures #330

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Attempt at Removing Apertures #330

wants to merge 6 commits into from

Conversation

kjkoeller
Copy link
Owner

This is an attempt at removing apertures/annuli from the list that are not within the data region. There is a discrepancy for what the output light curve data should be and what is given by the program.

This is an attempt at removing apertures/annuli from the list that are not within the data region. There is a discrepancy for what the output light curve data should be and what is given by the program.
This reduces the complexity of the `multiple_ap` function.
This update makes the code much more readable. Includes a way for the code to check to see if apertures are outside the x y coordinates of the image and removes them. It should restart the process photometry analysis automatically but that has not been extensively tested yet.
Adds functions to check for valid files and directories to be used across the plackage.
The program variable `filter` was overriding a built in variable also named `filter` in Python.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant