VIP

VIP copied to clipboard

Improvements to the test suite

2

The coverage will be increased significantly if we include tests for ``vip_hci.metrics.contrcurve`` and ``vip_hci.negfc``.

carlos-gg

ENH: remove `method` argument from OO methods

3

While working with `HCIDataset.recenter`, I was thinking about whether we are giving the user too many parameters at once. `HCIDataset.recenter` aggregates the `preproc/recentering.py::cube_recenter_*` functions, and merges all keyword arguments from...

r4lv

This is related to a todo in `HCIDataset.inject_companions`: https://github.com/vortex-exoplanet/VIP/blob/acb112931b14f962f4eeede8774c0004b61c5d2c/vip_hci/hci_dataset.py#L778 ### current situation For now, `HCIDataset.inject_companions(...)` *modifies* the `self.cube`. ### proposition Most of the methods of `HCIDataset` actually *modify* its state,...

r4lv

For now, all subclasses of `HCIPostProcAlgo` are in one single file (`vip_hci/hci_postproc.py`). Maybe it would make more sense to encapsulate them in their respective submodules, say `vip_hci/pca/postprocalgo.py` for PCA etc.

r4lv

Being the new preferred entry points to the world of VIP, I think the `HCIDataset`, `HCIFrame` and `HCIPostProcAlgo` classes could be renamed to something shorter: that would be easier to...

r4lv

**problem** For now, multiple output-related *things* exist in VIP: - `verbose` parameter to functions - `debug` parameter to functions - timing information - progress bars. **solution** I propose a new...

r4lv

## Description Calling vip_hci.fm.fakecomp.normalize_psf with a psf that has 4 dimensions fails to normalize the psf, does not raise an error and returns None instead. This occurs even if debug...

lwelzel

Firstguess with IFS data takes 12 days to run

3

Hello, I am trying to use the firstguess routine for the IFS data and it took 12 days to run. Is there a way to optimize the sub-routine?

trisha-bhowmik

Calling `psfsub.pca_fullfr.pca` providing a reference cube to trigger RDI raises a NoneType error for the `final_residuals_cube`. However, this variable is not called in the _"4D cube, but no mSDI desired,...

m-vegap

`IndexError` when running ADI contrast curve in HEEPS with VIP

1

While running the demo in the HEEPS environment, we encountered an `IndexError` related to `rad_samp` when attempting to generate the ADI contrast curve. Despite extending the `COMPASS` wavefront file to...

ChrisDelaX