eurec4a / flight-phase-separation

Collection of manually edited flight segments for all platforms participating in EUREC4A.
1 stars 6 forks source link

Segment kinds #21

Open bjorn-stevens opened 3 years ago

bjorn-stevens commented 3 years ago

Would it be possible to rename the radar_calibration_wiggles rather radar_calibration_roll, and radar_calibration_tilted rather radar_calibratio_pearl. There reason is that the former is a roll maneuver, this is also common for calibrating the gust probes, and I would actually simply call this a roll maneuver. The pearl is short circle, sometimes longer, and is a meaningful maneuver even if not calibrating the radar. For tis reason one could be a bit more general and say: pearl, and roll maneuvers.

d70-t commented 3 years ago

I'd recommend not to do this. The kinds are a form of identifier and are likely used at some places already (i.e. for seraching for the respective segments). If we'd remove any kinds from existing, published segments, we might break some existing code.

We probably could add your suggested kinds as additional kinds to the existing segments. That should be backwards-compatible. We could then mark the old kinds as deprecated in the readme, such that people should prefer to use the new kinds in future. If we do this, we should probably think a bit and broadly agree on which common kind names we really like to have. Another option would be to keep both. I.e. to have radar_calibration_wiggles for those specific maneuvers and a more generic roll_maneuver which could be applied to all radar_calibration_wiggles as well as to other, non-radar specific roll_maneuvers.