`flagtools.transferFlags` vs. `resampling.concatFlags`
We now have to functions to transfer flags/histories from one variable to another:
- The rather new
flagtools.transferFlags
which adds flags from source to target. This is only a wrapper aroundgeneric.flagGeneric
and therefore has the same limitations (source and target need to have the same index). - The older and more powerful , func=isflagged(K25_)), which adds the history from source to target and also allows for some sort of resampling.
IDK, for me this seems to be redundant and I wonder, if we should merge both functionalities into one function. I would suggest the following:
- Extend
resampling.concatFlags
with:- a default to
method
and suggest"match"
- another parameter like
squueze: bool=True
which determines if the entire history of just the flags should be concatenated
- a default to
- Make
flagtools.transferFlags
a wrapper aroundresampling.concatFlags
- Optionally: deprecate
flagtools.transferFlags
in 2.1 and remove it in 2.2 or 2.3.