Skip to content

Exclude time windows#104

Open
aleberti wants to merge 26 commits intoastro-transients:masterfrom
aleberti:exclude_time_windows
Open

Exclude time windows#104
aleberti wants to merge 26 commits intoastro-transients:masterfrom
aleberti:exclude_time_windows

Conversation

@aleberti
Copy link
Contributor

Add feature to exclude time windows.

@aleberti aleberti marked this pull request as draft December 18, 2025 15:33
@aleberti
Copy link
Contributor Author

For the moment the time windows are read as command line arguments and put into a list. As discussed via Slack with @mseglar , probably it is better to have the excluded windows read from a file, and pass the name to e.g. add_parsed_args like for the subtraction pointing file. So for the moment I put the PR in draft state, and then implement the reading of such a file. But not today ;)

@aleberti
Copy link
Contributor Author

Now added reading the time windows to be excluded from a file.

@aleberti aleberti marked this pull request as ready for review December 19, 2025 14:25
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ aleberti
❌ pre-commit-ci[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

2 participants