Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interpolators #46

Open
wants to merge 32 commits into
base: dev1.7
Choose a base branch
from
Open

Interpolators #46

wants to merge 32 commits into from

Conversation

findesgh
Copy link
Contributor

Implementation of interpolator classes for BPASS output grids.

@HeloiseS HeloiseS marked this pull request as ready for review July 26, 2020 20:18
@findesgh findesgh changed the title Interpolators WIP: Interpolators Jul 30, 2020
hoki/interpolators.py Outdated Show resolved Hide resolved
@findesgh findesgh marked this pull request as draft August 21, 2020 20:22
@findesgh findesgh changed the base branch from csp to dev1.6 October 6, 2020 13:41
@findesgh findesgh marked this pull request as ready for review March 8, 2021 10:14
@findesgh findesgh changed the title WIP: Interpolators Interpolators Mar 8, 2021
@findesgh
Copy link
Contributor Author

findesgh commented Mar 8, 2021

Finally all done from my side. Sorry it took so long.

The scipy dependency is still in there. Don't know how easy it is to replace that with numpy, haven't had time to look into that.

I'm working on an example notebook for the tutorials.

@cbyrohl
Copy link

cbyrohl commented Feb 3, 2022

It would be great to have this functionality merged. Let me know if I can be of help.

@findesgh
Copy link
Contributor Author

@HeloiseS I see that you have added scipy as a dependency - on the release branch a least - so this PR would not introduce a new dependency anymore. I'm happy to change the target branch and resolve any arising conflicts, if that helps.

@HeloiseS
Copy link
Owner

Totally! Do that and send me an email at hfstevance@gmail.com if I don't respond. I'm on holiday I might not see those messages come through

@findesgh findesgh changed the base branch from dev1.6 to dev1.7 February 19, 2022 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants