P. L. Lim
P. L. Lim
For 1, streaming is currently not supported. It is an entirely different ticket/epic/whatever. For 2 and 3, as Ricky pointed out, you can still stuff it into NDData. I am...
> optional Not pretty but I guess... still need to know what models to support though. There are many available in JWST, right? https://github.com/spacetelescope/jdaviz/blob/1f57249e8daef6e12ef5ecb0dbc3a56b5ffe6662/jdaviz/configs/imviz/plugins/parsers.py#L22-L27 > This is a CORE WORKFLOW...
> "API translation" layer If we limit this request to ASDF models (instead of arbitrary format), I think it makes sense to have that layer in `asdf-astropy`. There is already...
If you open a feature request for `to_nddata` upstream, please cross link here. Thanks!
Re: decorator -- If you think the performance is irrelevant outside of this narrow set of code, then it is best to keep it super private and then remove as...
> To be clear, are you suggesting I re-implement the decorator in Python only for the sake of delaying the more painful review ? > Sorry I don't get what...
How many users are we antagonizing? If it is just one or two, I will take that risk. I will even open a PR in their repos.
Also if performance is really such a concern to the point you need C-extension for a decorator, biting bullet with `/` would be the most performant.
Units came from another package originally. There were talks in the past about a similar topic but we decided against it. 🤷
To elaborate further, even though on the surface, you only use those few things, under the hood, they also rely on `astropy.utils`, `astropy.config`, and the science state machinery. Splitting them...