ISISNeutronMuon / ISISNeutronMuon/MDANSE

[ENHANCEMENT] Investigate removing IOutputVariable and subclasses

Open
#698 0 comments 1 reaction 0 assignees View on GitHub
enhancement Technical Debt
Dominant language
Python
Stars
29
Forks
8
Avg merge
2d 12h
Merged PRs (30d)
9

Description

**Is your feature request related to a problem? Please describe.**
The `MDANSE.Framework.OutputVariables.IOutputVariable` and subclasses (`LineOutputVariable`, `SurfaceOutputVariable` and `VolumeOutputVariable`) and associated `OutputData` object seem to be vestigial objects.

**Describe the solution you'd like**
It may be possible to either remove them entirely or replace them with standard [`xarray`](https://docs.xarray.dev/en/stable/) data. `xarray` integrates seamlessly with `dask` (see #443), `h5py`, `matplotlib` and [`pint`](https://pint.readthedocs.io/en/stable/) and provides the metadata features which `IOutputVariable` attempts to include.

**Describe alternatives you've considered**
N/A

**Additional context**
N/A

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.