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

feat: Fetch logs of enqueue_runs.py script #171

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

Conversation

daniel-ji
Copy link
Contributor

Creates a script that can retrieve log files of all jobs scheduled in an enqueue_runs.py run. Modifies enqueue_runs.py slightly to make this possible.

@daniel-ji daniel-ji requested review from uermel and manasaV3 July 26, 2024 00:23
@daniel-ji daniel-ji marked this pull request as draft July 26, 2024 16:20
@daniel-ji daniel-ji marked this pull request as ready for review August 3, 2024 04:12
@daniel-ji
Copy link
Contributor Author

Note: this script has been helpful to Utz and definitely to me when we had to ingest data, definitely worthwhile to merge

@daniel-ji daniel-ji requested a review from jgadling August 3, 2024 04:14
daniel-ji and others added 11 commits August 2, 2024 21:25
* Refactoring method name

* Refactoring unavailable ts placeholder

* Updating name
* Adding deposition support for annotations

* Updating test schema

* Setting method link field to json

* fixing tests
* Adding deposition tables

* Adding timestamps to tables

* Adding timestamps to s3

* Adding dir prefixes

* Refactoring method name

* Refactoring unavailable ts placeholder

* Updating name

* Revert timestamp addition for now

* minor refactor

* fix revert permissions

* Updating all entities to use deposition

* Updating existing tests

* Adding tests for depositions

* Updting seed sql files

* Updating the comments text

* Fixing broken seed sql

* Update documentation

* Adding deposition relationship to graphql
* Updating Jensen configs to include deposition metadata

* Clean up

* Updating exception message
* Add migration commands to modify comments

* Comment out relationship comments

* Attempt to add relationship comments

* Format

* Address PR comments

* Remove relationship comments

* Add may change to ID descrioption
daniel-ji and others added 8 commits August 12, 2024 14:02
* Port over cryoet-portal s3 data validation scripts created by @uermel

* remove some of utz validation scripts from pr

* make annotation tests separate

* fix pytest

* fix pytest again

* fix pytest for real

* delete yarn.lock

* refactor s3 data validation data.py

* more s3 data validation refactoring

* merge refactor fixes from s3-data-validation-annotations
@jgadling jgadling changed the title Fetch logs of enqueue_runs.py script feat: Fetch logs of enqueue_runs.py script Sep 25, 2024
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.

5 participants