Skip to content

chore(weights): drop the linked-issue bonus for sparkinfer - #1687

Merged
LandynDev merged 1 commit into
entrius:testfrom
ai-hpc:chore/sparkinfer-remove-issue-bonus
Aug 11, 2026
Merged

chore(weights): drop the linked-issue bonus for sparkinfer#1687
LandynDev merged 1 commit into
entrius:testfrom
ai-hpc:chore/sparkinfer-remove-issue-bonus

Conversation

@ai-hpc

@ai-hpc ai-hpc commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #1682.

Summary

  • Add a scoring block to gittensor-ai-lab/sparkinfer pinning standard_issue_multiplier and maintainer_issue_multiplier to 1.0.

Why

#1682 carried the K3 settings over to gittensor-ai-lab/sparkinfer but left both issue multipliers unset, so the repo silently inherited the platform defaults from gittensor/constants.py:

Knob Before (inherited default) After
scoring.standard_issue_multiplier 1.33 1.0
scoring.maintainer_issue_multiplier 1.66 1.0

Under those defaults, _calculate_issue_multiplier in gittensor/validator/oss_contributions/mirror/scoring.py scores a PR with a valid linked issue 33–66% above an otherwise identical PR with no linked issue. That is a bonus SparkInfer does not want: the repo already sets issue_discovery_share to 0.0, so the issue channel is intended to be off here, and PR value is already expressed through the eval:* label multipliers applied by the trusted label pipeline. Leaving the linked-issue bonus on adds a scoring axis unrelated to the measured performance win, and rewards linking an issue rather than the contribution itself.

1.0 is neutral (no bonus) and is the floor of the [1, 5] range enforced by _validate_scoring_configs, so this is the minimum the registry permits. Every other knob — emission_share, maintainer_cut, eligibility, labels, and the platform-default time decay restored in #1682 — is unchanged.

Validation

  • jq empty gittensor/validator/weights/master_repositories.json
  • uv run --extra dev pytest tests/validator/test_load_weights.py (64 passed)
  • uv run --extra dev pytest tests/validator/oss_contributions/mirror/ (117 passed)
  • Resolved config confirms the override lands: resolve_scoring returns standard_issue_multiplier=1.0, maintainer_issue_multiplier=1.0.

Follow-up to entrius#1682. That PR carried the K3 settings over to
gittensor-ai-lab/sparkinfer but left scoring.standard_issue_multiplier
and scoring.maintainer_issue_multiplier unset, so the repo inherited the
platform defaults (1.33 / 1.66) and a PR that links an issue scored
33-66% above an identical PR that doesn't.

SparkInfer already sets issue_discovery_share to 0.0, so the issue
channel is meant to be off for this repo. Pin both multipliers to 1.0
(neutral, the floor of the [1, 5] validated range) so PR scoring rests
only on the eval:* label multipliers from the trusted label pipeline.
@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label Aug 9, 2026
@LandynDev
LandynDev merged commit 087ff9a into entrius:test Aug 11, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants