Skip to content

[AI0021L] model-eval notebook#59

Open
jbenignocardoso wants to merge 3 commits intoRHOAI2.25from
model-eval
Open

[AI0021L] model-eval notebook#59
jbenignocardoso wants to merge 3 commits intoRHOAI2.25from
model-eval

Conversation

@jbenignocardoso
Copy link

No description provided.

@jbenignocardoso jbenignocardoso marked this pull request as ready for review March 10, 2026 16:55
Copy link
Collaborator

@alxlenc alxlenc left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Contributor

@sffrench sffrench left a comment

Choose a reason for hiding this comment

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

Quick review complete.

"source": [
"### 1. Install and import libraries\n",
"\n",
"**LMEval** is the industry-standard framework for evaluating language models across standardized benchmarks.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"**LMEval** is the industry-standard framework for evaluating language models across standardized benchmarks.\n",
"**LMEval** is the industry standard framework for evaluating language models across standardized benchmarks.\n",

"### 4. Evaluate compressed model\n",
"\n",
"The INT8 quantized model reduces weights from 16-bit to 8-bit. \n",
"Manually load it with `transformers` because LMEval doesn't directly support the `compressed-tensors` format."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"Manually load it with `transformers` because LMEval doesn't directly support the `compressed-tensors` format."
"Manually load it with `transformers` because LMEval does not directly support the `compressed-tensors` format."

@sffrench sffrench assigned jbenignocardoso and unassigned sffrench Mar 16, 2026
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.

3 participants