Right now Regions2D.mask requires region_ids: List as input without a default value.
I think we can add None as default so that Regions2D.mask has the same behavior as .plot and .close_region that defaults to region_id=None and plot or close all regions in this case.
And note #134 for changing region_ids to just region_id.
This is a relatively small change, so I suggest having it in v0.1.0.
leewujung
commented
9 months ago
Right now
Regions2D.maskrequiresregion_ids: Listas input without a default value.I think we can add
Noneas default so thatRegions2D.maskhas the same behavior as.plotand.close_regionthat defaults toregion_id=Noneand plot or close all regions in this case.And note #134 for changing
region_idsto justregion_id.This is a relatively small change, so I suggest having it in v0.1.0.