Skip to content

fix: count current mirror open issues for spam gate#1058

Closed
CaptainTimon wants to merge 1 commit intoentrius:testfrom
CaptainTimon:fix/mirror-open-issue-spam-count
Closed

fix: count current mirror open issues for spam gate#1058
CaptainTimon wants to merge 1 commit intoentrius:testfrom
CaptainTimon:fix/mirror-open-issue-spam-count

Conversation

@CaptainTimon
Copy link
Copy Markdown

@CaptainTimon CaptainTimon commented May 7, 2026

Summary

  • Keep mirror issue scoring bounded by the PR lookback window.
  • Fetch the miner's current mirror issues separately and use that unbounded current-open count for the open-issue spam multiplier.
  • Record the current mirror open issue count on evaluations even when no lookback issues are scoreable.

Close: #929

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • Tests added/updated
  • Manually tested

Validation performed:

  • python3 -m py_compile gittensor/validator/issue_discovery/mirror_scan.py gittensor/classes.py tests/validator/issue_discovery/test_mirror_scan.py
  • git diff --check

Note: pytest could not be run in this local environment because pytest and runtime dependencies such as bittensor are not installed.

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Changes are documented (if applicable)

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label May 7, 2026
@CaptainTimon
Copy link
Copy Markdown
Author

@adamdecaf Could you please review my PR?
Thanks!

Copy link
Copy Markdown

@alpurkan17 alpurkan17 left a comment

Choose a reason for hiding this comment

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

Good fix! Counting current mirror open issues directly addresses the spam gate logic. The approach correctly handles the edge case. Looks good to me.

@CaptainTimon
Copy link
Copy Markdown
Author

@anderdc Could you please review my PR?
Thanks!

@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 7, 2026

Duplicate of #930 which is better scoped/implemented. Closing.

@anderdc anderdc closed this May 7, 2026
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.

[Bug] Mirror issue-discovery spam count ignores open issues older than lookback window

4 participants