Skip to content
This repository was archived by the owner on Nov 21, 2025. It is now read-only.

ENH: Adds BioViL-T Linear Classifier (Chexpert Classes)#860

Draft
ozan-oktay wants to merge 20 commits intomainfrom
ozoktay/biovilt_linear_classifier
Draft

ENH: Adds BioViL-T Linear Classifier (Chexpert Classes)#860
ozan-oktay wants to merge 20 commits intomainfrom
ozoktay/biovilt_linear_classifier

Conversation

@ozan-oktay
Copy link
Contributor

  • Adds the capability to instantiate a linear classifier on top frozen BioViL-T image embeddings.
  • Note that the model is not trained in an end-to-end fashion.

@codecov
Copy link

codecov bot commented Mar 30, 2023

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.39%. Comparing base (190e8d4) to head (f9e99fa).
⚠️ Report is 57 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

Flag Coverage Δ
hi-ml-multimodal ∅ <100.00%> (∅)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...al/src/health_multimodal/image/model/pretrained.py 85.00% <100.00%> (+2.14%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ozan-oktay ozan-oktay requested review from Shruthi42 and fepegar March 30, 2023 15:04
@fepegar
Copy link
Contributor

fepegar commented Mar 30, 2023

Shall we review now or wait until it's not a draft anymore?

@fepegar
Copy link
Contributor

fepegar commented Apr 20, 2023

Can you please take a look at the failing tests and the merge conflicts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants