BUG: get_dbgap_accession_as_parts does not correctly parse a full dbgap accession

[SpencerAxelrod]

gen3.external.nih.utils.get_dbgap_accession_as_parts does not seem to parse a full dbgap accession.

Example:

get_dbgap_accession_as_parts("phs001961.v2.p1.c1")

returns

{
'phsid': 'phs001961', 
'phsid_number': '001961', 
'participant_set': '', 
'participant_set_number': '',
'version': 'v2', 
'version_number': '2', 
'consent': '', 
'consent_number': ''
}

with participant set and consent group information missing

[AkshataAthreya]

Hi! I’m Akshata — a CS undergrad working mainly with Python and data analysis. I recently interned at Siemens Gamesa (now Siemens Energy), where I worked on Python automation and unit testing.

I’m exploring the uc-cdis codebase as I plan to apply for Google Summer of Code 2026, and I wanted to ask if this issue would be suitable for a first-time contributor. The bug description and example are very clear, and I’d be happy to take a stab at fixing it and adding tests if that’s helpful.

Please let me know if there are any guidelines or context I should be aware of before starting. Thanks!

[WilsonnnTan]

Hi, I'm Wilson 👋
Recently, I'm exploring uc-cdis gen3 SDK codebase as my prep for GSoC 2026 and I would like to work on this problem.
Thanks!

[WilsonnnTan]

Hi, I triaged this issue since there hasn’t been recent maintainer activity on this issue, I investigated it independently to clarify the bug and found that the root cause appears to be in the parser regex logic.