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.

Sign up for GitHub

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

Jump to bottom

Overload interpolator operators #71

Closed
nwolek opened this issue Dec 29, 2015 · 1 comment
Closed

Overload interpolator operators #71

nwolek opened this issue Dec 29, 2015 · 1 comment
Assignees
@nwolek

Comments

@nwolek
Copy link
Member

nwolek commented Dec 29, 2015

Each type should be overloaded to accept four data points as inputs (regardless of wether all four are needed) to allow for easier switching between algorithms.

Related to issues #43 #60 #70
@nwolek nwolek self-assigned this Dec 29, 2015
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: overloading the None option. see issue #71
6bb5f95
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: adding tests of the None option. see issue #71
6e6a00e
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: more tests of low v high delta. need to rethink None v …
61e8119 
…potential Nearest options. issue #71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: separating the None and Nearest ideas into individual o…
936be2f 
…ptions. issue #71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: expanding documentation of Nearest and None. issue #71
8060df4
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: renaming delta for consistency. issue #71
6cb55a5
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: making x numbering more consistent for None and Nearest…
79ad19d 
… options. issue #71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: adding tests for the Nearest option, passes all. issue #71
de0c334
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: adding note that single sample version of operator is l…
8e935eb 
…ikely to never be used. issue #71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: Linear option is now overloaded and passing tests. issue …
2d6fa09 
…#71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: Cosine option is now overloaded and passing tests. issue …
ef1f1f3 
…#71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: Spline changing variable names to be consistent with ot…
0ef525c 
…her algorithms. issue #71
nwolek added a commit that referenced this issue Dec 30, 2015
@nwolek
Interpolator: Hermite changing variable names to be consistent with o…
bb0fc3c 
…ther algorithms. Also a correction to stray delta comment. issue #71
@nwolek nwolek closed this as completed Dec 30, 2015
@nwolek
Copy link
Member Author

nwolek commented Dec 30, 2015

Got a new interpolation option out of this work: Nearest

Cleaned up the documentation and added even more tests. If something goes wrong here, we are going to know about!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nwolek nwolek

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nwolek