You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is a strong and thoughtful first research contribution — thank you. The distinction between validation and scoring is especially useful, and the 8-dimension rubric gives DRP a credible path toward measurable auditability.
Before merge, please make these focused changes:
Remove the CONTRIBUTING.md change from this PR. A contributor path has already been merged into main, so this part now duplicates existing content. This PR should stay focused on docs/AUDITABILITY_SCORING.md.
Add explicit N/A handling and normalized scoring. Some dimensions do not apply to every record. For example, a root decision may have no parent links, and a decision that replaces nothing should not be penalized for missing supersession. Please define the final score as earned points / applicable maximum × 100 and explain when a dimension is marked N/A.
Update the PR description. Please replace the template text with a short summary, mark this as a documentation/research change with no protocol semantics change, and include Closes #21.
Small cleanup:
fix Breath of alternatives to Breadth of alternatives in the summary table;
make clear that automated or NLP checks are proposed future implementations, not guarantees of the current repository.
Once these are addressed, I will re-review quickly. This is valuable work and I would like to get it merged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
One or two sentences. What changes and why.
Spec impact
docs/SPEC.md.docs/SPEC.md(no behavioral change).docs/SPEC.md. List affected sections below.If this is a normative change, describe:
CHANGELOG.mdentry.Validation and testing
schema/drp.schema.jsonupdated if the machine-checkable shape changed.tools/drp_validator.pyupdated if a new invariant was introduced.fixtures/valid/.fixtures/invalid/.tests/.python3 -m pytest tests/passes locally.Breaking change?
Checklist
main.CHANGELOG.mdupdated under[Unreleased].VERSIONupdated if this PR cuts a release.