Skip to content

Safety 1.2.1 - #1717

Merged
bkorycki merged 13 commits into
mainfrom
safety-1.2.1
Nov 3, 2026
Merged

bkorycki merged 13 commits into
mainfrom
safety-1.2.1

Conversation

@bkorycki

Copy link
Copy Markdown
Contributor

Based on pr #1713
Dependent on PR 771 in sugar.

@bkorycki
bkorycki requested a review from a team as a code owner October 30, 2026 21:04
@github-actions

Copy link
Copy Markdown

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

Comment thread src/modelbench/benchmarks.py Outdated
"""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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

every prompt ship in

I don't understand what that means.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah it's a little confusing. It was left over from Riki's PR. I removed it.

@dhosterman dhosterman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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 = {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

@bkorycki
bkorycki deployed to Scheduled Testing November 2, 2026 21:08 — with GitHub Actions Active
@bkorycki
bkorycki deployed to Scheduled Testing November 3, 2026 19:22 — with GitHub Actions Active
@bkorycki

bkorycki commented Nov 3, 2026

Copy link
Copy Markdown
Contributor Author

@dhosterman fixed the demo issue!

@bkorycki
bkorycki requested a review from dhosterman November 3, 2026 19:22
@bkorycki
bkorycki merged commit a23357d into main Nov 3, 2026
3 checks passed
@bkorycki
bkorycki deleted the safety-1.2.1 branch November 3, 2026 20:32

This branch was successfully deployed

1 active deployment
Scheduled Testing — 2a5d42cd Deployed Nov 3, 2026 by bkorycki via build #2649
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.

4 participants