Element project summary reports#172

Merged

ssjunnebo merged 18 commits intoNationalGenomicsInfrastructure:masterfrom

ssjunnebo:element_project_summary

Aug 11, 2025

Contributor

ssjunnebo commented Aug 7, 2025 •

edited

Loading

Add support for Element project summary reports
Refactor fetching documents from StatusDB
Fix bug that open date is missing from reports

ssjunnebo added 10 commits

May 8, 2025 10:21


          Initial setup for aviti reports

d50773a


          Get correct values from statusdb for aviti

c405c3f


          cleanup

c7bc995


          Only look for FCs in the relevant DB

77587b1


          refactor get_entries

e9f233a


          bugfix: open_date missing in report

aad9363


          cleanup

f941a22


          Correcting logic

e53b7a9


          markdown text formatting for Aviti

407770c


          Add yield and q30 for element

cf4e113

ssjunnebo requested review from FranBonath and aanil

August 7, 2025 13:36

ssjunnebo added the validation label

ssjunnebo changed the title ~~Element project summary~~ Element project summary reports


          Remove print

5383a55

aanil reviewed

View reviewed changes

ngi_reports/utils/statusdb.py Outdated Show resolved Hide resolved

aanil reviewed

View reviewed changes

ngi_reports/utils/statusdb.py Show resolved Hide resolved

aanil reviewed

View reviewed changes

ngi_reports/reports/element_project_summary.py Outdated Show resolved Hide resolved

aanil reviewed

View reviewed changes

ngi_reports/reports/element_project_summary.py Outdated Show resolved Hide resolved

aanil reviewed

View reviewed changes

ngi_reports/utils/entities.py Outdated Show resolved Hide resolved

aanil reviewed

View reviewed changes

ngi_reports/utils/entities.py Show resolved Hide resolved

ssjunnebo added 6 commits

August 8, 2025 09:33


          Move signature to parent class

a210d67


          Expand loop variable names

12637b3


          Cleanup comments

72d1804


          Expand more loop variables

3286a3b


          Move get_project_flowcell to GenericRunConnection

a1b31fd


          Remove name_view

260ff00

aanil reviewed

View reviewed changes

ngi_reports/reports/element_project_summary.py Outdated Show resolved Hide resolved

aanil approved these changes

View reviewed changes

Member

aanil left a comment

👍 Looks good!


          replace most .format() with f-strings

144c055

ssjunnebo merged commit 8589ae2 into NationalGenomicsInfrastructure:master

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels