feat(model-ports): add modernbert_sentiment - #211
Open
DarthCeltic wants to merge 2 commits into
Open
Conversation
Recovers an already-converted, already-hosted artifact from earlier this session (GitHub Release model-port-modernbert-sentiment-v1) that never had a model-ports claim filed -- caught during a full re-audit of all open PRs and releases. arch=modern-bert, distinct from bert (distilbert_sst2, already claimed) and from every other bert-family variant (nomic-bert, jina-bert-v2/v3, neo-bert, eurobert -- all still unclaimed). Verified live: built llama-server from a plain CPU-only (GGML_ET=OFF) configuration of the vendored source, loaded the GGUF, and got a real response from POST /rerank (3-way sentiment classifier head loads correctly: negative/neutral/positive). Same verification tier as distilbert_sst2/roberta_sst2 earlier in this campaign. Registered only in artifacts.json, not benchmark_config.json -- same reason as distilbert_sst2: the shared runner only speaks /completion and /v1/chat/completions, not /rerank, so an automated "board": true entry would be misleading without a maintainer adding /rerank support to that protected script.
github-actions
Bot
added
track: model-ports
Member
|
ModernBERT currently has an artifact, recipe and claim but no registered board benchmark or committed deterministic sentiment oracle. Please add the benchmark, pinned test inputs + expected labels/logits and threshold, and a successful ET-SoC1 result. Maintainers will then review/adopt the identity and protected validation contract. |
…ent per maintainer review
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
2 participants
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.
Adds modernbert_sentiment classification port to llama.cpp-et, with a model-ports claim. Artifact was already converted and hosted earlier this session (GitHub Release) but never had a claim filed until this audit caught it.