Skip to content

Add standin files; use term "processing model", define basic one.#21

Merged
cabo merged 1 commit intomainfrom
standin-file
Mar 2, 2026
Merged

Add standin files; use term "processing model", define basic one.#21
cabo merged 1 commit intomainfrom
standin-file

Conversation

@cabo
Copy link
Copy Markdown
Owner

@cabo cabo commented Mar 2, 2026

No description provided.

@cabo cabo requested a review from marenamat March 2, 2026 15:05
Comment on lines +184 to 191
([^TODO]
The detailed conditions under which certain transformations can be
applied to YANG-CBOR data representations after these have been
generated are TBD; several processing models are conceivable for this.
One such condition might be:
Where information starts out in a legacy representation, these tags
are only used when an Unambiguous Round Trip can be achieved.
are only used when an Unambiguous Round Trip can be achieved.)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would say this:

Transformations of YANG-CBOR data representation after these have been generated, are out of scope of this document.

… and put all these considerations into that separate document: https://github.com/marenamat/ietf-draft-marenamat-netmod-core-yang-transcoding (please send a pull request there adding yourself as an author if you wish)

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I put in a pointer to that document, but coordinating them now is a bit late in the day...
Let's do that in the next iteration.

Comment on lines 243 to +245
For consistency with this specification, an intermediate decoder of a
tag 37 stand-in MUST use lowercase characters in the uuid hex string
generated.
generated ([^TODO] align with processing model).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What about something like "The canonical string representation of tag 37 uses lowercase characters in the uuid hex string."

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Canonical is not really a defined term here -- I would like to keep the MUST in there as one example for how a processing model might treat this.

They already are CBOR tags and thus in the registry, but might
get lost in the bulk of that (and are only identified as YANG-CBOR
stand-in Tags in the specification).
ISSUE: Do we want to have a separate registry for stand-in files?
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe we could have a registry composing the stand-in tables, so that one can see that the standard way to represent e.g. ip-address with a stand-in is a tag 52/54?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I think that is a good point. We haven't really found out how we want these table to compose, which I think is the next item of work; then we might have a better grasp for how we can organize a collection of them.

Copy link
Copy Markdown
Collaborator

@marenamat marenamat left a comment

Choose a reason for hiding this comment

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

Overall good direction, I think that it may be a good time to ask for wg adoption maybe? Or is it still too early?

@marenamat
Copy link
Copy Markdown
Collaborator

note: gonna upload my draft with the original name, we can rename it later

@cabo
Copy link
Copy Markdown
Owner Author

cabo commented Mar 2, 2026

ask for wg adoption maybe? Or is it still too early?

I'm not sure. Let's get a temperature reading from the mailing list...

@cabo cabo merged commit 4659018 into main Mar 2, 2026
2 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.

2 participants