search

dsavransky / EXOSIMS

Simulator for exoplanet direct imaging space missions
BSD 3-Clause "New" or "Revised" License
25 stars 35 forks source link

replace interp2d #375

Closed theabdulrehman99 closed 3 months ago

theabdulrehman99 commented 3 months ago

Description

This pull request addresses the transition away from using interp2d for interpolation purposes. The change replaces the use of interp2d with RectBivariateSpline for regular grid interpolation and bisprep/bisplev combo for scattered interpolation. This transition is necessary as interp2d has been deprecated.

Changes Made

Replaced the usage of interp2d with RectBivariateSpline for regular grid interpolation. Implemented the bisprep/bisplev combo for scattered interpolation. Added handling to switch between the two methods based on the lengths of the input arrays.

Reasoning

interp2d is deprecated. The new implementation ensures compatibility with future updates and removes reliance on deprecated functionality.

Testing

Verified that the transition preserves the results obtained from interp2d exactly. Tested the regular grid interpolation using RectBivariateSpline. Tested the scattered interpolation using the bisprep/bisplev combo. Checked that the code handles both regular grid and scattered data cases correctly.