Skip to content

Rename find_canonical_root -> find_uqps_root locator API - #3

Closed
sherryzyh wants to merge 1 commit into
mainfrom
refactor/rename-canonical-locator-to-uqps
Closed

Rename find_canonical_root -> find_uqps_root locator API#3
sherryzyh wants to merge 1 commit into
mainfrom
refactor/rename-canonical-locator-to-uqps

Conversation

@sherryzyh

Copy link
Copy Markdown
Owner

Summary

Renames the prkit project-locator helper for the renamed consumer project (formerly canonical_answer_protocol, now uqps / uncertainty_quantification_via_physics_semantics).

  • find_canonical_rootfind_uqps_root (and its __all__ entry + the find_repo_root dispatcher)
  • env var PRKIT_CANONICAL_ROOTPRKIT_UQPS_ROOT
  • hardcoded sibling-directory lookups repointed from canonical_answer_protocoluncertainty_quantification_via_physics_semantics
  • sampling_backfill.py import/call + fallback path updated
  • doc reference updated

Unrelated domain uses of "canonical" (unit canonicalization, canonical_text, the --canonical-sampling-root baseline flag) are intentionally left untouched.

Why

The consumer project was renamed to align with the NeurIPS draft "Uncertainty Quantification for Open-Ended LLM Physics Reasoning via Physics Semantics." The old locator hardcoded a directory name that no longer exists.

Verification

  • pytest: 1378 passed, 1 skipped (incl. tests/prkit/core/test_project_env.py)
  • pre-commit (ruff/black/mypy) green
  • Cross-repo: find_uqps_root() resolves to the renamed sibling directory

🤖 Generated with Claude Code

Renames the project-locator helper and its env var
(PRKIT_CANONICAL_ROOT -> PRKIT_UQPS_ROOT), and repoints the hardcoded
sibling-directory lookups to uncertainty_quantification_via_physics_semantics
(the renamed canonical_answer_protocol project). Leaves unrelated
domain uses of 'canonical' (unit canonicalization, canonical_text,
the --canonical-sampling-root baseline flag) untouched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@sherryzyh

Copy link
Copy Markdown
Owner Author

Superseded by #4. The decoupling removes the renamed locator (find_uqps_root) entirely, so this standalone rename no longer makes sense on its own — it's folded into #4 as the first commit. Closing in favor of #4.

@sherryzyh sherryzyh closed this Jun 16, 2026
@sherryzyh
sherryzyh deleted the refactor/rename-canonical-locator-to-uqps branch June 16, 2026 04:25
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.

1 participant