Skip to content

SNT25 393 - #54

Closed
irautu-bluesquare wants to merge 8 commits into
mainfrom
SNT25-393
Closed

irautu-bluesquare wants to merge 8 commits into
mainfrom
SNT25-393

Conversation

@irautu-bluesquare

Copy link
Copy Markdown
Collaborator
  1. general report notebook (started from the general notebook in the repo):
  • removed NER-specific code
  • adapted naming of input data for plots to remove duplicated dataframes
  • added conditional html output for markdown which is not country-specific but dependent on the existence of disaggregation data (under 5 children and pregnant women)
  1. NER-specific notebook (started from NER notebook in the NER OpenHexa workspace):
  • removed the NER-related conditions
  • created clearer conditions for the other criteria
  • data changed recently: columns in the code were hardcoded in a syntax that only allows hardcoding (no string-input syntax), but column names in the data changed, so input code gave errors; modified the syntax to allow passing them as strings and input them as variables; did the minimum to not have errors, but there would be more changes to be made ideally
  • adapted naming of input data for plots
  • added conditional html output

@sPuntinG

Copy link
Copy Markdown
Contributor

Hey @irautu-bluesquare ! Do you think this is still relevant? Or is there anything we should somehow preserve in the repo?
Else feel free to close this PR :)

@irautu-bluesquare

Copy link
Copy Markdown
Collaborator Author

i think probably @EstebanMontandon fixed these on his side in the meantime
will close this, it's anyway evolved so much in the meantime :)

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