Skip to content

✨ Create or append descriptors to global IDs#9

Merged
znatty22 merged 23 commits intomainfrom
upsert-global-ids
Feb 14, 2025
Merged

✨ Create or append descriptors to global IDs#9
znatty22 merged 23 commits intomainfrom
upsert-global-ids

Conversation

@znatty22
Copy link
Contributor

@znatty22 znatty22 commented Feb 7, 2025

Closes #6

To do

  • Upsert and download single descriptor cmd
  • Upsert batch of descriptors cmd
  • Download descriptors cmd
  • Upsert and download batch descriptors cmd
  • Generate fake global id file cmd
  • Check if state column has been implemented (not yet)
  • Check if descriptor appending has been implemented (yes)
  • Integration tests
  • Unit tests

@znatty22 znatty22 added the feature New feature or request label Feb 7, 2025
@znatty22 znatty22 self-assigned this Feb 7, 2025
@znatty22 znatty22 marked this pull request as ready for review February 12, 2025 23:55
@znatty22 znatty22 requested a review from haynescd February 12, 2025 23:55
Copy link
Collaborator

@haynescd haynescd 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

Comment on lines +68 to +71
if study_global_id:
s_id = study_global_id
else:
s_id = dewrangle_study_id
Copy link
Collaborator

Choose a reason for hiding this comment

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

don't have to change but I prefer this when they are so simple

s_id = study_global_id if (study_global_id) else dewrangle_study_id

@znatty22 znatty22 merged commit 5a7b0f8 into main Feb 14, 2025
3 checks passed
@znatty22 znatty22 deleted the upsert-global-ids branch February 14, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create or append a descriptor(s) to existing global ID(s)

2 participants