tjgalvin / flint

BSD 3-Clause "New" or "Revised" License
7 stars 1 forks source link

BANE and Aegean Options + Masking #98

Closed tjgalvin closed 2 months ago

tjgalvin commented 2 months ago

Noticed some strange behavior when deriving clean masks, which prompted the creation of BANEOptions and AegeanOptions to help keep track and drill down into things. Eventually with the aim of including these options in strategy format.

In all of this however it became clear the problem is actually related to not passing the per round MaskingOptions through to the clean mask creation function.

In the banegrid branch I have:

tjgalvin commented 2 months ago

Addressed in #100