Closed mmcdermott closed 3 months ago
The changes enhance the functionality and flexibility of several scripts within the project. Notable updates include expanded spell-checking options, improved data loading processes, and the introduction of new transformation capabilities for reordering measurements. Configuration files have been refined to align with these updates, ensuring clarity and consistency. Overall, these modifications streamline workflows and enrich the project's capabilities.
Files/Paths | Change Summary |
---|---|
.pre-commit-config.yaml |
Expanded codespell ignore words list to include "crate" alongside "ehr". |
MIMIC-IV_Example/joint_script.sh |
Added optional argument do_unzip to enhance file processing flexibility. |
MIMIC-IV_Example/pre_MEDS.py |
Removed load_raw_mimic_file , replaced with dynamic file reading using get_supported_fp , and improved error handling. |
eICU_Example/pre_MEDS.py |
Removed rootutils import and initialization, simplifying directory management. |
pyproject.toml |
Removed rootutils from dependencies; added support for MEDS_transform-reorder_measurements . |
src/MEDS_transforms/... |
Various parameter renames (e.g., code_modifier_columns to code_modifiers ), improved function signatures, and streamlined logic. |
src/MEDS_transforms/configs/preprocess.yaml |
Renamed configuration parameter from code_modifier_columns to code_modifiers . |
src/MEDS_transforms/configs/... |
Introduced reorder_measurements.yaml file to define measurement reordering rules. |
src/MEDS_transforms/utils.py |
Added get_smallest_valid_uint_type function for determining optimal unsigned integer types. |
tests/... |
Added tests for reorder_measurements functionality and enhanced utility functions for testing. |
sequenceDiagram
participant User
participant JointScript
participant PreMEDS
participant Config
participant Transform
User->>JointScript: Run script with do_unzip argument
JointScript->>PreMEDS: Process files based on configuration
PreMEDS->>Config: Read dynamic file format
Config->>Transform: Apply transformations
Transform->>User: Return processed data
🐇 In a world of code and lines,
A rabbit hops through changing times.
With spells corrected, files in sync,
New paths appear, and stars now blink.
So let us cheer for every mod,
For coding's joy is truly broad! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
:warning: Please install the to ensure uploads and comments are reliably processed by Codecov.
Attention: Patch coverage is 97.26027%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 92.48%. Comparing base (
e1e6260
) to head (9a5de23
). Report is 415 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
src/MEDS_transforms/mapreduce/mapper.py | 95.16% | 3 Missing :warning: |
...MEDS_transforms/transforms/reorder_measurements.py | 97.56% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Tests
Chores