Merged
Conversation
…ontext for configuration and package management. Introduced utility functions for imputation and reporting, enhancing modularity and readability. Updated paths and error handling for loading configurations.
…context for configuration and package management. Introduced utility functions for imputation and reporting, improving modularity and readability. Updated paths and error handling for loading configurations across both mean and median imputation pipelines.
…t for configuration and package management. Introduced utility functions for outlier detection, enhancing modularity and readability. Updated paths for loading configurations and improved error handling in reporting notebooks.
…essing. Introduced error handling for configuration loading, validation of required keys, and standardized output path creation. Updated outlier imputation notebooks to utilize these new utilities, improving modularity and readability.
…ared helper scripts. Removed the old bootstrap script and updated paths for loading configuration and utility functions, enhancing modularity and readability across the pipeline.
…ance modularity and readability. Updated helper functions for plotting outliers and added error handling for missing data. Improved configuration loading and path management for better integration across the pipeline.
…s to reflect new naming conventions. This change enhances clarity and consistency across the pipeline.
…ons for improved modularity and clarity. Introduced `prepare_magic_glasses_input` to streamline input handling and context setup, while enhancing logging and error management. Updated notebook to utilize new function for better organization.
…lize routine data loading and validation. Introduced `load_routine_data` function to streamline data retrieval and error handling, enhancing modularity and clarity across multiple pipelines.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It goes with this ticket: https://bluesquare.atlassian.net/browse/SNT25-381
I made as a poc the refractoring of the outliers_imputation pipelines. We moved all the big function in a utils foilder. One file per code and reporting. I also implemented a little change in the snt_utils.R
The idea was to have pipeline specific utils in the pipeline folder and a generic function in utils we can call from wherever we are