Merged
Conversation
…eadmission Removed readmission task definition, as it is not applicable to any publicly available dataset as stated here.
…RBERT_disk Add a warning to the README for CEHR-BERT disk usage.
More ruff changes
❌ 1 Tests Failed:
View the full list of 1 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
- Fix doctest in conftest.py: use pytest.UsageError instead of _pytest.config.exceptions.UsageError - Update MIMIC-IV dataset.yaml for MIMIC-IV-MEDS 0.1.0: replace MEDS_cohort_dir with MEDS_output_dir and add root_output_dir - Bump cehrbert to 1.4.9 (1.3.5 had numpy/pandas conflict) and add tf-keras to fix Keras 3 compatibility with transformers Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add uv as a project dependency and use it in install_venv() instead of stdlib venv + pip. This avoids the need for the python3.X-venv system package and enables future support for creating venvs with different Python versions (e.g. uv venv --python 3.11 for models that need older packages). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use 2-space indent for bash continuation lines to match CI's mdformat-shfmt formatting. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Pass --clear to uv venv so it tolerates pre-existing directories (the test fixture writes model_name.txt inside the venv dir before install_venv runs) - Pin all mdformat plugin versions in .pre-commit-config.yaml and add shfmt-py so the shfmt binary is bundled in the hook environment, making formatting deterministic across local and CI - Update CLAUDE.md bash indentation to match pinned shfmt output Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This fixes some test and PR issues, and makes the venv installation more robust and fast -- there is an outstanding failure with meds-tab which is an upstream issue.
- Bump meds-tab from 0.1 to 0.2.0, which no longer pins scipy==1.6.1 (fixes the build failure on Python 3.11+) - Remove hydra-optuna-sweeper==1.3.0.dev0 pin (meds-tab 0.2.0 pulls in its own compatible version) - Remove hydra.sweeper.max_failure_rate override which is not supported by hydra-optuna-sweeper 1.2.0 Closes #274 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…-v0.2 Upgrade meds-tab to 0.2.0 (fixes scipy build failure)
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
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.
No description provided.