Skip to content

Derive File-Accession mappings from AltAccessions - Update for Apollo Epic (GSI-2265) - #65

Merged
TheByronHimes merged 5 commits into
mainfrom
feature/alt_accessions_apollo_GSI-2265
Apr 2, 2026
Merged

Derive File-Accession mappings from AltAccessions - Update for Apollo Epic (GSI-2265)#65
TheByronHimes merged 5 commits into
mainfrom
feature/alt_accessions_apollo_GSI-2265

Conversation

@TheByronHimes

@TheByronHimes TheByronHimes commented Apr 1, 2026

Copy link
Copy Markdown
Member

This PR updates WPS for changes introduced by Apollo - specifically with regard to research data file IDs and how we map them to externally visible accession numbers.

Old way: consume events that have accession: str, file_id: UUID4.
New way: consume events that have id: str, pid: str, type: str, created: datetime, where pid=accession and id=file_id.

Since Apollo's AltAccessions are published for other kinds of entities beyond the file IDs, we filter out any AltAccession event where the "type" field is not AltAccessionType.FILE_ID

@coveralls

coveralls commented Apr 1, 2026

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23894365116

Details

  • 25 of 25 (100.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 86.756%

Totals Coverage Status
Change from base Build 23435349025: 0.1%
Covered Lines: 845
Relevant Lines: 974

💛 - Coveralls

@TheByronHimes
TheByronHimes requested a review from mephenor April 1, 2026 15:14
@TheByronHimes
TheByronHimes marked this pull request as ready for review April 1, 2026 15:14
Comment thread src/wps/core/models.py Outdated
Comment thread src/wps/core/models.py Outdated
@TheByronHimes
TheByronHimes requested a review from mephenor April 2, 2026 09:47
@TheByronHimes
TheByronHimes merged commit ed925a4 into main Apr 2, 2026
9 checks passed
@TheByronHimes
TheByronHimes deleted the feature/alt_accessions_apollo_GSI-2265 branch April 2, 2026 10:33
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