Skip to content

schema: computed dimensions via expanded ODD chaining (#45) - #44

Merged
duncdrum merged 4 commits into
masterfrom
dp-dimensions-normalized
Aug 31, 2026
Merged

duncdrum merged 4 commits into
masterfrom
dp-dimensions-normalized

Conversation

@duncdrum

@duncdrum duncdrum commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Schema changes for computed physical dimensions and layout encoding. Design rationale and full example set: #45.

Decision: expand adds parallel sibling blocks marked @subtype='computed' (same physical @type on dimensions; layout uses @type='catalogue'). Machine-readable values live on empty height/width/depth/dim children (@quantity, @unit, optional @atLeast/@atMost) or computed layout/@writtenLines. Cataloguer blocks are unchanged.

ODD chaining: expand-only rules live in tei-betamesaheft-expanded.xml (chained from compiled authoring ODD). Authoring tei-betamesaheft.xml keeps cataloguer constraints and emits a nonfatal schematron warning if @subtype appears on dimensions in source files.

Rejects: in-place text rewrite (BetMasWeb#104), dimensions[@type='computed'], duplicate height in one block, @corresp linkage.

Changes

  • tei-betamesaheft-expanded.xmldimensions @subtype='computed' + exemplum; layout att.typed + exemplum (refs TEI design: computed physical dimensions (parallel @subtype siblings) #45)
  • tei-betamesaheft.xml — nonfatal warning if @subtype on dimensions in source; open @subtype datatype only so warning can fire without RNG hard-fail
  • .gitignore / build.xml — align with sibling repos

Prerequisites before expand deploy

  • BetMasWeb viewItem.xqm must skip [@subtype='computed'] blocks before display (otherwise duplicate dimensions/layout in MS view).

Not in this PR

  • Expand implementation
  • collection.xconf retype
  • Caller changes

Test plan

@duncdrum duncdrum changed the title docs(schema): normalized dimensions TEI design spike schema: normalized dimensions derived siblings (#45) Aug 31, 2026
@duncdrum
duncdrum force-pushed the dp-dimensions-normalized branch from 132b444 to 96b2fc6 Compare August 31, 2026 11:22
duncdrum and others added 2 commits August 31, 2026 14:15
Ignore local tooling paths (.worktrees/, .codacy/, .cursor/, etc.) and
keep them out of the packaged schema xar.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add dimensions @subtype='normalized' with parallel-block exemplum.
Add layout att.typed (type='catalogue', subtype='normalized') and exemplum.
Expand adds parallel siblings for machine-readable @quantity values;
design rationale and fixture checklist in #45.

Co-authored-by: Cursor <cursoragent@cursor.com>
@duncdrum
duncdrum force-pushed the dp-dimensions-normalized branch from 01d33d4 to b4b97eb Compare August 31, 2026 12:15
@duncdrum
duncdrum marked this pull request as ready for review August 31, 2026 13:17
Move @subtype=computed encoding rules to tei-betamesaheft-expanded.xml;
authoring ODD keeps cataloguer constraints and emits a nonfatal warning
if @subtype appears on dimensions in source files.

Co-authored-by: Cursor <cursoragent@cursor.com>
@duncdrum duncdrum changed the title schema: normalized dimensions derived siblings (#45) schema: computed dimensions via expanded ODD chaining (#45) Aug 31, 2026
@duncdrum
duncdrum marked this pull request as draft August 31, 2026 13:36
TEI ODD sch:report content cannot contain doc elements (att/gi).

Co-authored-by: Cursor <cursoragent@cursor.com>
@duncdrum
duncdrum marked this pull request as ready for review August 31, 2026 13:57
@duncdrum

Copy link
Copy Markdown
Contributor Author

ok this should be workable for a new expanded schema

@duncdrum
duncdrum merged commit e326b13 into master Aug 31, 2026
2 checks passed
@duncdrum
duncdrum deleted the dp-dimensions-normalized branch August 31, 2026 14:21
@duncdrum duncdrum added this to the Jinntec_Phase_Docker milestone Sep 4, 2026
duncdrum added a commit that referenced this pull request Sep 4, 2026
Bare classes after #44 replaced stock att.global. Use odd2odd merge
(classes/@mode=change + memberOf mode=add) so @cert/@corresp stay.

Co-authored-by: Cursor <cursoragent@cursor.com>
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