Skip to content

Conversation

@will-pang
Copy link
Contributor

Fixing a small typo in the "how_to_get_started" documentation.

I believe for this eval/monitoring method:

trainer.train(
    train_dataloader=train_loader,
    val_dataloader=val_loader,
    epochs=50,
    monitor="pr_auc_samples",
)

We want monitor=pr_auc as prc_auc_samples is for multi label classification, whereas pr_auc is for binary classification (yes/no decease).

Running the current documentation leads the pipeline to error out/fail.

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.

Ah thanks for the catch William! That's on us , we also need to sooner or later update all of these installation instructions as we're aiming to get a full PyHealth 2.0 release later this next week. We're super close to basically revamping the entire package to work with our new backend.

@jhnwu3 jhnwu3 merged commit 528e27e into sunlabuiuc:master Jan 11, 2026
1 check passed
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