Skip to content

Conversation

MiguelAFH
Copy link
Collaborator

Added support for phi-3.5-mini-instruct for StanfordHealthCare through Azure.
Refactored the code to support llama and deepseek models and removed model-specific clients.

@MiguelAFH MiguelAFH requested a review from yifanmai March 31, 2025 21:38
@MiguelAFH MiguelAFH self-assigned this Mar 31, 2025
Copy link
Collaborator

@yifanmai yifanmai left a comment

Choose a reason for hiding this comment

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

Looks good, thanks.

@@ -35,8 +35,9 @@ def get_medcalc_bench_spec() -> RunSpec:
input_noun=None,
newline_after_input_noun=False,
output_noun="Answer only the requested quantity without units. No explanation needed",
max_tokens=10,
max_tokens=4000,
Copy link
Collaborator

Choose a reason for hiding this comment

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

FYI this change is unrelated to the models.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Great catch, fixed on latest push!

@MiguelAFH MiguelAFH merged commit fb61149 into main Mar 31, 2025
8 checks passed
@MiguelAFH MiguelAFH deleted the medhelm-phi branch March 31, 2025 22:59
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