Skip to content

Fix ChestX-ray14 Example Notebooks#828

Merged
jhnwu3 merged 2 commits intosunlabuiuc:masterfrom
EricSchrock:fix-chestxray14-notebooks
Feb 8, 2026
Merged

Fix ChestX-ray14 Example Notebooks#828
jhnwu3 merged 2 commits intosunlabuiuc:masterfrom
EricSchrock:fix-chestxray14-notebooks

Conversation

@EricSchrock
Copy link
Collaborator

@EricSchrock EricSchrock commented Feb 8, 2026

In #392, I added the chestxray14_binary_classification.ipynb and chestxray14_multilabel_classification.ipynb examples. This PR cleans up three issues with these example notebooks.

  1. PyHealth Install
  2. GitHub Display
  3. Duplicate Output

1. PyHealth Install

Previously, these notebooks installed PyHealth from a branch in my fork. Now that #392 is merged AND 2.0.0 is released, that can be cleaned up.

Before

!rm -rf PyHealth
!git clone https://github.com/EricSchrock/PyHealth.git
%cd PyHealth
!git checkout ChestX-ray14
!pip install -e .

After

%pip install pyhealth ipywidgets

Caution

Is it a surprise that ipywidgets must also be explicitly installed? I get an error in base_dataset.py if I just pip install pyhealth.

2. GitHub Display

I initially generated these notebooks in Google Colab. Apparently, this added a bunch of metadata, which wasn't compatible with the GitHub web UI. Trying to view the notebooks on GitHub would show the following. This PR should fix the issue.

image

3. Duplicate Output

The old versions of the example notebooks printed every output twice, which makes the example much longer and less clear. Apparently, I didn't have Google Colab properly configured to work with the Python logger. This PR fixes the duplicate output.

Copy link
Collaborator

@jhnwu3 jhnwu3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@jhnwu3 jhnwu3 merged commit 4b80f25 into sunlabuiuc:master Feb 8, 2026
1 check passed
@EricSchrock EricSchrock deleted the fix-chestxray14-notebooks branch February 8, 2026 02:56
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