Skip to content

IGVF-3614-encode-measurement - #2164

Merged
ottojolanki merged 2 commits into
devfrom
IGVF-3614-encode-measurement
Jul 27, 2026
Merged

IGVF-3614-encode-measurement#2164
ottojolanki merged 2 commits into
devfrom
IGVF-3614-encode-measurement

Conversation

@ian-whaling

Copy link
Copy Markdown
Contributor

No description provided.

@ian-whaling
ian-whaling requested a review from jenjou July 25, 2026 02:12
Comment thread src/igvfd/schemas/measurement_set.json Outdated
"title": "External identifier",
"description": "Identifier from an external resource that may have 1-to-1 or 1-to-many relationships with IGVF file sets.",
"type": "string",
"pattern": "^(GEO:GSE\\d+|urn:mavedb:\\d{8}-[0a-z]+-\\d+|ENCODE:.+)$"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should the regex for ENCODE be more strict to exactly match an ENCODE accession, or do we not require that?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I can make it specific to experiments, i.e. ENCSR* if that is what you are proposing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

But I did just copy this from the existing regex in reference files

@ian-whaling
ian-whaling force-pushed the IGVF-3614-encode-measurement branch from 7fe07ab to ae8ed90 Compare July 27, 2026 21:32
@ottojolanki
ottojolanki merged commit 9f29226 into dev Jul 27, 2026
8 checks passed
@ottojolanki
ottojolanki deleted the IGVF-3614-encode-measurement branch July 27, 2026 23:14
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