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

min_age setting should be applied to Alignment dirs as well as Run #20

Closed
ressy opened this issue Jan 11, 2019 · 0 comments
Closed

min_age setting should be applied to Alignment dirs as well as Run #20

ressy opened this issue Jan 11, 2019 · 0 comments

Comments

@ressy
Copy link
Member

ressy commented Jan 11, 2019

The min_age setting (that ignores run directories until their ctime timestamp becomes old enough) should be applied to the alignment sub-directories inside run directories as well. Otherwise we can still get spurious warnings about missing data on the first pass over incompletely-written directories.

ressy added a commit that referenced this issue Jan 17, 2019
This checks the behavior of a new argument, min_alignment_dir_age, for
the Run constructor.
ressy added a commit that referenced this issue Jan 17, 2019
The new test should still fail at this point (but because assertions
fail, not just an argument mismatch this time).
@ressy ressy closed this as completed in #22 Jan 18, 2019
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

No branches or pull requests

1 participant