search

zhafen / galaxy-dive

A general analysis package for analyzing hydrodynamic simulations of galaxy formation.
MIT License
0 stars 0 forks source link

Allow more flexible masking #12

Closed zhafen closed 6 years ago

zhafen commented 6 years ago

Originally reported by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)

Current masking is actually pretty flexible already, but what would be great is to choose what masks to combine together. I'll implement this by adding another dictionary to the DataMasker from which masks can be pulled on demand.

Things to be updated:

  1. DataMasker.mask_data needs to be able to store the masks, including an appropriate name.
  2. DataMasker.mask_data should check that no mask is already stored.
  3. DataMasker.get_total_mask() should be able to pull from the masks.
  4. DataMasker.get_masked_data() should be able to pull from the masks.
zhafen commented 6 years ago

Original comment by Zachary Hafen (Bitbucket: zhafen, GitHub: zhafen)

Resolved Issue #12!