search

microsoft / yardl

Tooling for streaming instrument data
https://microsoft.github.io/yardl/
MIT License
29 stars 5 forks source link

Fix #95 - Fix Python optional generic record fields #96

Closed naegelejd closed 8 months ago

naegelejd commented 8 months ago

Nested record fields are now correctly instantiated with proper default values in generated Python code.

johnstairs commented 8 months ago

There is another issue. 

MyRec<T>: !record
  fields:
    a: T*

Does not default the field, whereas this does: 

MyRec: !record
  fields:
    a: int*

(An existing issue, not a regression)

naegelejd commented 8 months ago

There is another issue.

MyRec<T>: !record
  fields:
    a: T*

Does not default the field, whereas this does:

MyRec: !record
  fields:
    a: int*

(An existing issue, not a regression)

I'm going to open a separate issue for this one you discovered ("generic vector default")