Skip to content

NAZ LLM EVALS + row_log.py (llama as a judge script) + recent_utils_py (most up to date)#2

Open
nazcol wants to merge 8 commits into
mainfrom
naz
Open

NAZ LLM EVALS + row_log.py (llama as a judge script) + recent_utils_py (most up to date)#2
nazcol wants to merge 8 commits into
mainfrom
naz

Conversation

@nazcol
Copy link
Copy Markdown
Collaborator

@nazcol nazcol commented Apr 4, 2025

No description provided.

@nazcol
Copy link
Copy Markdown
Collaborator Author

nazcol commented Apr 4, 2025

serve request that is erroring below:

vllm serve meta-llama/Meta-Llama-3-8B-Instruct
--gpu-memory-utilization 0.95
--max-model-len 2048
--port 8001
--tensor-parallel-size 1
--max-num-seqs 32

@nazcol nazcol changed the title row_log.py (llama as a judge script) + recent_utils_py (most up to date) NAZ row_log.py (llama as a judge script) + recent_utils_py (most up to date) Apr 4, 2025
@nazcol nazcol changed the title NAZ row_log.py (llama as a judge script) + recent_utils_py (most up to date) NAZ row_log.py (llama as a judge script) + recent_utils_py + LLM EVALS (most up to date) May 1, 2025
@nazcol nazcol changed the title NAZ row_log.py (llama as a judge script) + recent_utils_py + LLM EVALS (most up to date) NAZ LLM EVALS + row_log.py (llama as a judge script) + recent_utils_py (most up to date) May 1, 2025
Copy link
Copy Markdown
Owner

@lisadunlap lisadunlap 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! Mostly minor things and making sure we are updating files instead of making new ones

Comment thread vibe_baseline.py
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

how does this differ from the get_differences.py file? If we are modifying that file let's name it the same thing so we don't muddy the repo

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

they actually are doing the same thing just checked now, I can change the name of mine nevertheless though

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

can we move these to a folder

Comment thread recent_utils_llm.py
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

can we instead update the existing utils_llm file

Comment thread .DS_Store
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

remove

nazcol and others added 3 commits May 1, 2025 19:57
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