Skip to content

DSERV-1291 Scorpion adapter#713

Open
pedrohr wants to merge 15 commits into
devfrom
DSERV-1291-scorpion
Open

DSERV-1291 Scorpion adapter#713
pedrohr wants to merge 15 commits into
devfrom
DSERV-1291-scorpion

Conversation

@pedrohr

@pedrohr pedrohr commented May 22, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@pedrohr pedrohr requested review from mingjiecn and shengchd May 22, 2026 20:18
@pedrohr pedrohr force-pushed the DSERV-1291-scorpion branch from e83c943 to cb75ec9 Compare May 22, 2026 20:21
Comment thread data/adapters/scorpion_adapter.py Outdated

for row in data_csv:
props = {
'_key': row['tf_ensembl'] + '_' + row['target_ensembl'] + '_' + self.LABEL,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

self.LABEL has space in it, not sure if this is good for key generation. Need to replace space, or use other string for example file accession?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

ohh great catch... will replace by underline


igvf_file_accessions:
Scorpion:
- IGVFFI3035IXUW

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Need to also merge in dev branch and generate new files_filesets data file for IGVF source.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

may need to regenerate it again since we don't know which data loading ticket goes in first...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

the only data loading ticket ready to QA is dual-ipa, it's already generated, so I think the JSONLs in this PR are the latest

@mingjiecn mingjiecn self-requested a review June 2, 2026 20:52
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