Wen Bo Li

Fixing https://github.com/openconfig/goyang/issues/131 is a prerequisite.

Same proposal as #179

Thanks for filing this issue Yash! I reproduced this problem when there is more than one type in the union definition, since for a singly-typed union, ygot demotes the type...

Adding 1.0.0 since backwards incompatibility is on the table. I can't think of a backwards compatible solution due to the "polymorphic" nature of enum values. It's possible to make a...

Actually, I think UNSET might be better, since it might be used for logging. Seeing `UNSET` is probably better than the empty string.

Since `TogNMINotifications` doesn't use the schema to determine the value to marshal, doesn't it mean that this is not operational? https://github.com/openconfig/ygot/blob/c6ce7e55aa79fe2cdd2f5c655b882b9c98f71fd3/ygot/render.go#L287 If we have a typedef enum used in multiple...

#172 confirms that this is not in operation in Go. It is also of dubious operation in proto due to the possibility of re-uses of enum typedefs. I think the...

Just clarifying here that this PR is dependent on how #101 and specifically #172 are resolved.

Yep, thanks for linking

From the ideas in the link as well as here it looks like using `string` to represent binary seems the most feasible. I think it's worth considering which one of...