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

Add generate_computed_file Command #915

Closed
avrohomgottlieb opened this issue Oct 2, 2024 · 1 comment · Fixed by #936
Closed

Add generate_computed_file Command #915

avrohomgottlieb opened this issue Oct 2, 2024 · 1 comment · Fixed by #936
Assignees

Comments

@avrohomgottlieb
Copy link
Contributor

avrohomgottlieb commented Oct 2, 2024

Context

We want Batch jobs to be able to handle one computed_file at a time, consisting of a project or sample and a download_config.

Problem or idea

This will entail adding a generate_computed_file method to the loader module, and adding a new generate_computed_file management command. This ticket should handle tests for the new loader::generate_computed_file method as well.

As well, a dispatch_to_batch command should be added, which either takes a project id and iterates over all project (and sample) + download config combinations and submits them to Batch.

Tests should also be created, which accomplish the following:

  • generate_computed_file command:
    • validate that the project_id / sample_id exists
    • validate that the download config name exists
  • loader::generate_computed_file:
    • validate if a download config applies to the project/sample

Solution or next step

...

@avrohomgottlieb
Copy link
Contributor Author

avrohomgottlieb commented Nov 1, 2024

Addressed in PR #936

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 a pull request may close this issue.

1 participant