assign_root

[egouden]

https://github.com/openradar/xradar/blob/19487ead4813cd57e3a39d9dfcfe049077b447f5/xradar/io/backends/common.py#L88

Why is the first sweep discarded from the computation?

[kmuehlbauer]

The first group here is the root group and does not contain any sweeps:

https://github.com/openradar/xradar/blob/19487ead4813cd57e3a39d9dfcfe049077b447f5/xradar/io/backends/odim.py#L841-L844

[kmuehlbauer]

We should clearly make this more explicit.

[egouden]

Thank you for the quick reply.

Here is my use case. I have a single cfradial1 file containing 3 volumes. I want to create 3 single volumes by selecting the sweeps and assigning a new root for each volume.

Would it be natural for the ODIM backend to provide only the sweeps to the assign_root function?

[kmuehlbauer]

Here is my use case. I have a single cfradial1 file containing 3 volumes. I want to create 3 single volumes by selecting the sweeps and assigning a new root for each volume.

You could just create everything from scratch (same as _assign_root does).

Would it be natural for the ODIM backend to provide only the sweeps to the assign_root function?

Would work, but as of now the Conventions attribute is forwarded to the root group to keep track of provenience.

[egouden]

I am now providing the root group from the cfradial1 file, as expected by the function.

[kmuehlbauer]

@egouden Is your issue resolved? Can we close?

[egouden]

Yes the issue is resolved.