Repository navigation
Safety 1.2.1 - #1717
Merged
Merged
Safety 1.2.1#1717
Conversation
bkorycki
had a problem deploying
to
Scheduled Testing
October 30, 2026 21:04 — with
GitHub Actions
Failure
|
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
rogthefrog
approved these changes
Oct 30, 2026
| """Safety 1.2 prompts scored the way Security 1.1 is scored. | ||
|
|
||
| The probability evaluator judges every response, the difficulty and | ||
| discrimination of every prompt ship in ``external_scoring``, and the record |
Contributor
There was a problem hiding this comment.
every prompt ship in
I don't understand what that means.
Contributor
Author
There was a problem hiding this comment.
Yeah it's a little confusing. It was left over from Riki's PR. I removed it.
dhosterman
reviewed
Nov 2, 2026
dhosterman
left a comment
Collaborator
There was a problem hiding this comment.
Are we abandoning --demo in modelbench?
It will fail on 1.2.1, currently, with the following:
Traceback (most recent call last):
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/bin/modelbench", line 10, in <module>
sys.exit(cli())
~~~^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 1569, in __call__
return self.main(*args, **kwargs)
~~~~~~~~~^^^^^^^^^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 1490, in main
rv = self.invoke(ctx)
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 1970, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 1970, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 1353, in invoke
return ctx.invoke(self.callback, **ctx.params)
~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/core.py", line 907, in invoke
return callback(*args, **kwargs)
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/cli.py", line 135, in wrapper
return func(*args, **kwargs)
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/.venv/lib/python3.14/site-packages/click/decorators.py", line 34, in new_func
return f(get_current_context(), *args, **kwargs)
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/cli.py", line 247, in general_benchmark
check_benchmark(benchmark)
~~~~~~~~~~~~~~~^^^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/cli.py", line 387, in check_benchmark
for hazard in benchmark.hazards():
~~~~~~~~~~~~~~~~~^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/benchmarks.py", line 66, in hazards
h.set_standard(self.standards)
^^^^^^^^^^^^^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/benchmarks.py", line 59, in standards
standards = Standards.get_standards_for_benchmark(self.reference_benchmark().uid)
~~~~~~~~~~~~~~~~~~~~~~~~^^
File "/Users/daniel/dev/src/github.com/mlcommons/modelbench/src/modelbench/benchmarks.py", line 180, in reference_benchmark
return type(self)(self.locale, "practice", PRIVATE_EVALUATOR)
~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: GeneralPurposeAiChatBenchmarkV1_2_1.__init__() takes 3 positional arguments but 4 were given
| # official: the 3,900-prompt, 12-hazard fit of 2026-10-20 on the 19 reference | ||
| # SUTs, from the same evaluator and the same naive evaluations as the Security | ||
| # 1.1 official file. No practice fit exists on that evaluator yet. | ||
| SAFETY_1_2_1_SCORING_FILES = { |
Collaborator
There was a problem hiding this comment.
Shouldn't there be a scoring file for practice, too? If not, I believe this just fails after the benchmark is run but before it is scored, which is a bad failure mode.
…delbench into safety-1.2.1
Contributor
Author
|
@dhosterman fixed the demo issue! |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Based on pr #1713
Dependent on PR 771 in sugar.