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

Create fill_gaps.ipynb #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

gjgilbert
Copy link
Collaborator

jupyter notebook containing a gap filling algorithm

jupyter notebook containing a gap filling algorithm
@theand9
Copy link
Collaborator

theand9 commented Mar 5, 2021

Heyy @gjgilbert how much work is left on the gap-filling algorithm?
If you have the time could you work on it so that we could get jazzhands on JOSS?

@gjgilbert
Copy link
Collaborator Author

@theand9 The gap-filling algorithm should be good to go as is. Just copy/paste the functions in the notebook to wherever you want to implement the wavelets methods that rely on uniform sampling. If something isn't clear or doesn't work, let me know and I'll look into it.

There are likely going to be edge cases where the algorithm fails, but I think the best way to handle that is to address them as they arise. I tried to anticipate as many as I could, but I'm sure something will come up as we test/use the package in real applications.

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.

2 participants