Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding script to upload tsv and files to Terra #207

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

snovod
Copy link
Collaborator

@snovod snovod commented Feb 10, 2025

No description provided.

Copy link

Thank you for contributing to the ops-terra-utils repo!
A test suite will now begin running to ensure that no breaking changes are made.
If these tests fail you can have them re-run after making changes by commenting /run_tests

Copy link
Collaborator

@sahakiann sahakiann left a comment

Choose a reason for hiding this comment

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

Looks good!

| **flatten_path** | Use if you want all paths to be in same directory. If not used it will mantain current path as source files (with bucket updated) | Boolean | Yes | N/A |
| **id_column** | Column to be used as the primary key in the Terra table. Must be present in the intput tsv. | String | Yes | N/A |
| **skip_upload_columns** | Pass in comma seperated list (no spaces) of columns you do not want to try copying files in from. This is only helpful if there is columns WITH file paths in it that you do NOT want copied in | String | No | N/A |
| **subdir** | Subdirectory to put files into in new bucket. If flatten path is used all files will be directory in this directory. If flatten is not used then the path structure will stay intact, but all paths will start with `gs://{bucket}/{subdir}/` | String | No | N/A |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like a typo in here. Maybe "If flatten_path is used, all files will be copied into this subdirectory. If flatten_path is not used..."

@snovod
Copy link
Collaborator Author

snovod commented Feb 10, 2025

/run_tests

@snovod snovod merged commit 3d82b4e into main Feb 10, 2025
7 checks passed
@snovod snovod deleted the sn_POD-2450_create_script_to_upload_files branch February 10, 2025 18:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants