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: Updating paths for Gains and CollectionMetadata #271

Merged
merged 5 commits into from
Sep 17, 2024

Conversation

manasaV3
Copy link
Contributor

@manasaV3 manasaV3 commented Sep 13, 2024

Relates to chanzuckerberg/cryoet-data-portal#898

Description

Frames

  • Update the location of .mdoc to /<dataset-id>/<run-name>/Frames

Gains

  • Update gain files to be stored in /<dataset-id>/<run-name>/Gain
  • Update the naming of the gain file to be <name-in-source>.<ext>

@manasaV3 manasaV3 requested review from jgadling, Bento007 and uermel and removed request for jgadling September 13, 2024 22:31
@manasaV3 manasaV3 changed the title Updating paths for Gains and CollectionMetadata feat: Updating paths for Gains and CollectionMetadata Sep 13, 2024
@@ -23,6 +23,11 @@ annotations:
file_format: csv
glob_string: particle_lists/{run_name}_fas.csv
shape: Point
collection_metadata:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is the code for collection_metadata support not part of this PR?

Copy link
Contributor Author

@manasaV3 manasaV3 Sep 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! the collection_metadata test file got missed in being tracked.

Copy link
Contributor

@jgadling jgadling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like there's an extra file in thIs PR?

@manasaV3 manasaV3 merged commit 2014169 into main Sep 17, 2024
6 checks passed
@manasaV3 manasaV3 deleted the mvenkatakrishnan/multi_tomo_paths1 branch September 17, 2024 19:37
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.

3 participants