Skip to content

Conversation

@CZDanol
Copy link
Collaborator

@CZDanol CZDanol commented Dec 9, 2025

This schema is intended to be used in other OpenPrintTag as well to provide a standardized decoded representation of data.

BFW-8025

Change representation of the ColorRGBAField in rec_info.

BFW-8025
Pass unnown fields as a raw CBOR hex string instead of the decoded CBOR representation.
This should improve interoperability and opens the possibility of passing the unknown fields verbatim in the future without decoding them at all (not possible yet with the cbor library).

BFW-8025
Silly variable name collision

BFW-8025
Without UUID, the tag is still valid, just doesn't have an uuid.

BFW-8025
- Change validate to produce a yaml report instead of asserting
- Move things from opt_check that should be in validate (checks not tied to the field semantics)
- If --opt-check is set, also do validation

BFW-8025
@CZDanol CZDanol force-pushed the 8025-opt-json-schema branch 3 times, most recently from 82f017a to a62d44e Compare December 9, 2025 10:53
ondratu
ondratu previously approved these changes Dec 9, 2025
@@ -1,6 +1,8 @@
# Examples
The OpenPrintTag specification comes with [a set of utilities written in Python]({{repo}}/tree/main/utils) that serve as a baseline/reference implementation.

Copy link

Choose a reason for hiding this comment

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

Nechceš to nějak sjednotit s tím dalším textemL Protože hned v dalším odstavci se píše, že rec_info je utilita která umí parsovat data z NFC....

ondratu
ondratu previously approved these changes Dec 9, 2025
ljanecek
ljanecek previously approved these changes Dec 10, 2025
This schema is intended to be used in other OpenPrintTag as well to provide a standardized decoded representation of data.

BFW-8025
Also add a test case with colo

BFW-8025
@CZDanol CZDanol force-pushed the 8025-opt-json-schema branch from 439d6da to 828951d Compare December 12, 2025 08:09
@CZDanol CZDanol merged commit 7ce315d into prusa3d:main Dec 12, 2025
4 checks passed
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.

4 participants