Follow-up from review discussion in #387.
to_cfradial1 currently hardwires the output version:
cfradial1_ds.attrs["Conventions"] = "Cf/Radial"
cfradial1_ds.attrs["version"] = "1.2"
This was hardwired before the refactor too, so #387 keeps it as-is. Opening this so we can discuss whether users would want to export to other CfRadial1 versions (1.3/1.4, see https://ncar.github.io/CfRadial/).
Things to work out:
- Which versions are worth supporting, and what actually differs between 1.2 / 1.3 / 1.4 in terms of variables and attributes.
- Whether this should be a
version= keyword on to_cfradial1 or inferred from something else.
- What the default should stay as (presumably
"1.2").
cc @kmuehlbauer
Follow-up from review discussion in #387.
to_cfradial1currently hardwires the output version:This was hardwired before the refactor too, so #387 keeps it as-is. Opening this so we can discuss whether users would want to export to other CfRadial1 versions (1.3/1.4, see https://ncar.github.io/CfRadial/).
Things to work out:
version=keyword onto_cfradial1or inferred from something else."1.2").cc @kmuehlbauer