Skip to content

fix paper completeness check false warning#1049

Merged
zazabap merged 4 commits intomainfrom
fix-completeness-check
Apr 14, 2026
Merged

fix paper completeness check false warning#1049
zazabap merged 4 commits intomainfrom
fix-completeness-check

Conversation

@isPANN
Copy link
Copy Markdown
Collaborator

@isPANN isPANN commented Apr 14, 2026

Summary

  • Fix completeness check using covered-rules.get() which only sees state before the current location
  • Changed to covered-rules.final() to see all rules in the document
  • Eliminates false warning about 4 "missing" rules that were actually defined later in the file

🤖 Generated with Claude Code

The warning falsely reported 4 missing rules because .get() only sees
state accumulated before the current location. Rules defined after the
check were invisible. Using .final() sees all rules in the document.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.94%. Comparing base (f2a3648) to head (db1b534).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1049   +/-   ##
=======================================
  Coverage   97.94%   97.94%           
=======================================
  Files         979      979           
  Lines      100890   100890           
=======================================
  Hits        98816    98816           
  Misses       2074     2074           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

isPANN and others added 3 commits April 14, 2026 12:11
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@zazabap zazabap self-requested a review April 14, 2026 07:03
Copy link
Copy Markdown
Collaborator

@zazabap zazabap left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread docs/paper/reductions.typ
]
#v(1.5em)
#text(size: 10pt, style: "italic")[arXiv:2604.xxxxx]
#text(size: 10pt, style: "italic")[#link("https://arxiv.org/abs/2604.11535")[arXiv:2604.11535]]
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.

Great.

Comment thread README.md
## Citation

If you find this project useful in your research, please cite:

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.

Great fix.

@zazabap zazabap merged commit e38b1b5 into main Apr 14, 2026
5 checks passed
@isPANN isPANN deleted the fix-completeness-check branch April 14, 2026 09:46
zazabap added a commit to zazabap/problem-reductions that referenced this pull request Apr 14, 2026
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.

2 participants