align/resample sets flags to unflagged
unfortunately i approved the MR before i wrote the comment
The following discussion from !658 (merged) should be addressed:
-
@palmb started a discussion:
in the current case (which was not changed by the MR !658 (merged)) all flags are explicitly set to UNFLAGGED. besides that i like this behavior, because of idea new data - new flags it might be unexpected to others. up to now this was shadowed by the early exit of shift, which i guess was quite often used by our pipelines/users. We might want to set NaN instead of UNFLAGGED, to make align be transparent (except for the meta), as it was in the shift-case before.
Edited by David Schäfer