Skip to content

Snt25 659 outliers (IQR) - #119

Merged
EstebanMontandon merged 6 commits into
mainfrom
SNT25-659-outliers
Sep 16, 2026
Merged

EstebanMontandon merged 6 commits into
mainfrom
SNT25-659-outliers

Conversation

@EstebanMontandon

Copy link
Copy Markdown
Collaborator

Third cleaning (no functional changes) --> only a fix for imputation function printing infinite warnings.

Scripts touched:
snt_dhis2_outliers_imputation_iqr.ipynb

Report:
snt_dhis2_outliers_imputation_iqr_report.ipynb

.R files touched:
snt_utils.r --> (small fixes and added a new path in snt_init function to "figures" folder to be available during reporting)
snt_dhis2_outliers_imputation_iqr.r
snt_dhis2_outliers_imputation_iqr_report.r

Re-arranged try: block in the python pipeline snt_dhis2_outliers_imputation_iqr (no functional changes)

@EstebanMontandon

Copy link
Copy Markdown
Collaborator Author

@sPuntinG I resolved the merge conflicts, this branch is ready to be Squash and merged.

@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.

Ok cool, all seems good!

I re-run the notebooks in the SNT Testing ws (BFA data) with all modified files (copied the whole ./code folder and the whole ./snt_dhis2_outliers_imputation_iqr folder in the ws).
Then also run via pipeline (latest version from template, without the "Pull scripts").

This seems enough of a test since the changes are all in the nb setup, utils and engeneering side -- no changes to core calculations.

@EstebanMontandon
EstebanMontandon merged commit 2abedac into main Sep 16, 2026
@EstebanMontandon
EstebanMontandon deleted the SNT25-659-outliers branch September 16, 2026 14:25
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