Skip to content

Comments

feat: Affinity-purified Organelle dataset configs#558

Open
uermel wants to merge 29 commits intomainfrom
uermel/affinity_organelles
Open

feat: Affinity-purified Organelle dataset configs#558
uermel wants to merge 29 commits intomainfrom
uermel/affinity_organelles

Conversation

@uermel
Copy link
Contributor

@uermel uermel commented Dec 18, 2025

Description

Ingestion config files for affinity purified organelle datasets.

…elles

# Conflicts:
#	ingestion_tools/poetry.lock
@uermel uermel requested a review from daniel-ji January 23, 2026 22:35
Copy link
Contributor

@daniel-ji daniel-ji left a comment

Choose a reason for hiding this comment

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

some typos and some swaps and some duplicate text, all metadata related. sources look good!

primary_author_status: true
corresponding_author_status: false
- name: Jonathan Schwartz
ORCID: 0000-0002-3248-9678
Copy link
Contributor

Choose a reason for hiding this comment

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

This is Yue's ORCID I think. I think it might be swapped across many/all files?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😭😭😭😭😭😭😭😭😭😭😭😭😭😭😭

primary_author_status: false
corresponding_author_status: false
- name: Yue Yu
ORCID: 0000-0002-8063-6951
Copy link
Contributor

Choose a reason for hiding this comment

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

And this is Jonathan's? I swear we had some validation to make sure the ORCID matches to the name, I should look into this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😭😭😭😭😭😭😭😭😭😭😭😭😭

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.

2 participants