Skip to content

first pass on llm example - #160

Merged
s-sajid-ali merged 4 commits into
mainfrom
ml_ai
Aug 27, 2025
Merged

first pass on llm example#160
s-sajid-ali merged 4 commits into
mainfrom
ml_ai

Conversation

@RobJY

@RobJY RobJY commented Aug 26, 2025

Copy link
Copy Markdown
Contributor

I can add other pages before we merge this PR, but I thought we might want to do it sooner rather than later, to keep others from duplicating work. Let me know what you think @s-sajid-ali

Also, please let me know if you see any corrections I need to make or have any suggestions. Thanks!

@RobJY RobJY self-assigned this Aug 26, 2025
@github-actions

github-actions Bot commented Aug 26, 2025

Copy link
Copy Markdown
Contributor
PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-08-27 14:51 UTC

@s-sajid-ali

Copy link
Copy Markdown
Member

Only comment is to name the page, "Run a HF model" or similar as we may add another page on running a model with vLLM later.

@RobJY

RobJY commented Aug 26, 2025

Copy link
Copy Markdown
Contributor Author

Sounds good. I made the requested change. Let me know if you think of anything else.

@RobJY
RobJY requested a review from s-sajid-ali August 26, 2025 17:52
@RobJY

RobJY commented Aug 26, 2025

Copy link
Copy Markdown
Contributor Author

Apologies, I thought I set you as a reviewer when I made the PR, @s-sajid-ali.

@s-sajid-ali

Copy link
Copy Markdown
Member

No worries, this is good to go now. If you have a moment, maybe add the script to https://github.com/NYU-RTS/rts-docs-examples ?

@s-sajid-ali
s-sajid-ali merged commit b764c66 into main Aug 27, 2025
5 checks passed
@s-sajid-ali
s-sajid-ali deleted the ml_ai branch August 27, 2025 14:51
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