Skip to content

fix(issue-discovery): ignore zero-share repos#1270

Closed
Desel72 wants to merge 5 commits into
entrius:testfrom
Desel72:fix/issue-discovery-zero-share
Closed

fix(issue-discovery): ignore zero-share repos#1270
Desel72 wants to merge 5 commits into
entrius:testfrom
Desel72:fix/issue-discovery-zero-share

Conversation

@Desel72
Copy link
Copy Markdown
Contributor

@Desel72 Desel72 commented May 14, 2026

Summary

Filters issue-discovery inputs to repositories whose issue_discovery_share is greater than zero, so PR-only repos cannot unlock issue-discovery eligibility or produce raw discovery rows.

Related Issues

Fixes #1267

Type of Change

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

Testing

  • Tests added/updated
  • Manually tested
uv run pytest tests/validator/issue_discovery/test_scan.py -q
uv run ruff check gittensor/validator/issue_discovery/scan.py tests/validator/issue_discovery/test_scan.py
uv run ruff format --check gittensor/validator/issue_discovery/scan.py tests/validator/issue_discovery/test_scan.py

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 14, 2026
@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 19, 2026

Fix conflicts.

Desel72 added 2 commits May 20, 2026 03:36
…-zero-share

# Conflicts:
#	gittensor/validator/issue_discovery/scan.py
#	tests/validator/issue_discovery/test_scan.py
@anderdc
Copy link
Copy Markdown
Collaborator

anderdc commented May 21, 2026

Per-repository eligibility (#1293) already gates issue-discovery eligibility and scoring per repo — a zero-share repo's solved issues only affect that repo's own evaluation, which the allocator skips entirely. The cross-repo unlock this targets no longer occurs on test. Closing.

@anderdc anderdc closed this May 21, 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] issue_discovery_share=0 repos still unlock issue-discovery eligibility

2 participants