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

Ignore Nextclade cache on new Nextclade version or dataset version #457

Closed
joverlee521 opened this issue Jul 8, 2024 · 2 comments · Fixed by #466
Closed

Ignore Nextclade cache on new Nextclade version or dataset version #457

joverlee521 opened this issue Jul 8, 2024 · 2 comments · Fixed by #466
Assignees
Labels
enhancement New feature or request

Comments

@joverlee521
Copy link
Contributor

We've had a history of caching issues (#456, #392) come up whenever there's a new release of Nextclade or SARS-CoV-2 Nextclade dataset and we forget to manually upload the *.renew files to trigger the full re-run.

The workflow should automatically ignore the Nextclade cache if it encounters a new Nextclade version or dataset version.

@joverlee521 joverlee521 added the enhancement New feature or request label Jul 8, 2024
@joverlee521
Copy link
Contributor Author

Also prompted by discussion in blab/forecasting project to surface Nextclade/dataset version in the sequence counts generated by forecasts-ncov/ingest

@corneliusroemer
Copy link
Member

Good idea, yes.

Shouldn't be hard to set renew files programmatically based on equality or not of the columns in the existing files with the current software and dataset version!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants