Skip to content

Make use of x-jsonld-value and x-jsonld-constants - #108

Merged
jviotti merged 4 commits into
mainfrom
value-constants
Aug 3, 2026
Merged

Make use of x-jsonld-value and x-jsonld-constants#108
jviotti merged 4 commits into
mainfrom
value-constants

Conversation

@jviotti

@jviotti jviotti commented Aug 3, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Review in cubic

jviotti added 4 commits August 2, 2026 22:42
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti
jviotti marked this pull request as ready for review August 3, 2026 12:05
@augmentcode

augmentcode Bot commented Aug 3, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR expands JSON-LD annotations across the schema library to standardize how numeric “quantity” schemas can be expressed in RDF.

Changes:

  • Adds x-jsonld-constants with a fixed QUDT hasUnit IRI for many SI (base/prefixed/derived), non‑SI, and ISO unit schemas
  • Adds x-jsonld-type set to http://qudt.org/schema/qudt/Quantity on those quantity-like schemas
  • Adds x-jsonld-value set to http://qudt.org/schema/qudt/value to indicate where the numeric literal should be emitted
  • Adjusts ISO 6709 latitude/longitude schemas to be plain number types (no longer $ref the degree unit schema) to avoid JSON-LD degree-quantity annotation
  • Documents the latitude/longitude JSON-LD rationale via $comment

Technical Notes: The new annotations consistently bind schema instances to QUDT unit IRIs, enabling downstream JSON-LD tooling to emit typed quantity nodes with fixed units while keeping validation constraints intact.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 235 files

Note: This PR contains a large number of files. cubic only reviews up to 200 files per PR, so some files may not have been reviewed. cubic prioritizes the most important files to review.

Re-trigger cubic

@jviotti
jviotti merged commit cb5f03a into main Aug 3, 2026
3 checks passed
@jviotti
jviotti deleted the value-constants branch August 3, 2026 12:14
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