Skip to content

Snt25 494 - #103

Merged
irautu-bluesquare merged 35 commits into
mainfrom
SNT25-494
Jul 27, 2026
Merged

Snt25 494#103
irautu-bluesquare merged 35 commits into
mainfrom
SNT25-494

Conversation

@irautu-bluesquare

Copy link
Copy Markdown
Collaborator

Reporting Rate Data Element

In Computation

  • fixed computation of reporting rate by year
  • intermediate results to store yearly data

In Report

  • corrected import of yearly data
  • harmonized plots with rest of pipelines
  • harmonized structure with rest of pipelines
  • made functions for plots
  • changed some plots'
    cod_bdi_examples.zip
    layouts/types

Changed files

  • pipelines/snt_dhis2_reporting_rate_dataelement/code/snt_dhis2_reporting_rate_dataelement.ipynb
  • pipelines/snt_dhis2_reporting_rate_dataelement/reporting/snt_dhis2_reporting_rate_dataelement_report.ipynb
  • pipelines/snt_dhis2_reporting_rate_dataelement/utils/snt_dhis2_reporting_rate_dataelement_report.r

Examples attached

  • COD and BDI reports

@@ -0,0 +1,350 @@
# Helpers for the access to healthcare pipeline report

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.

Is this here by mistake?

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.

yes :D copy-pasta-ed the comment from that pipeline, thanks!

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.

I was a bit puzzled by this file that belongs to a completely different pipeline, but I see that it's a teeny tiny change so I guess it's ok to include it in this pipeline :)

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.

ahh, yes, that's because Frederik couldn't see the transparency around buffers, with 0.3 opacity, so i set it to 0,2 and didn't make a branch for it. should've mentioned it, sorry :D

}


summarise_reporting_rate_by_year <- function(

@sPuntinG sPuntinG Jul 24, 2026

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.

👀 I feel like this is the type of code that should remain in the notebook to be immediately visible (also, it's 2 lines of code and not used elsewhere) and debuggable in case of incongruent results

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.

you're right, it's no longer used! removed, thanks!

@sPuntinG sPuntinG left a comment

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.

Hey! Nice stuff!

I left a few comments in the code. My opinions, nothing blocking so go ahead if you don't agree :)

One additional questions while we're at it, is it possible that this image needs to be made larger (bigger width) - or is it rendering weird on my side only?

Image

@irautu-bluesquare

Copy link
Copy Markdown
Collaborator Author

made the map bigger (the width)

@irautu-bluesquare
irautu-bluesquare merged commit baa1b49 into main Jul 27, 2026
@irautu-bluesquare
irautu-bluesquare deleted the SNT25-494 branch July 27, 2026 09:13
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