Added generic 1d nearest neighbor helper function

[ahaberlie]

As requested, created generic version of function discussed in:

#99

Subsequent commits were done to fix whitespace/formatting issues after lint failed for initial commit.

[dopplershift]

Numpy docstring standard says this needs to start with Parameters: https://raw.githubusercontent.com/numpy/numpy/master/doc/EXAMPLE_DOCSTRING.rst.txt

[dopplershift]

Thanks for putting this in--I just picked a few minor nits above.

I'm also trying to decide: would this make more sense under something like metpy.calc.tools ? It seems like this function could be more generally useful outside plotting--plus currently metpy.plots.util doesn't have any public API functions.

Regarding the test coverage, can you add a test function as well? It helps to make sure it's working correctly now and, more importantly, doesn't break in the future. It can be really simple like:

import numpy as np
from numpy.testing import assert_array_equal
def test_resample_nn():
    a = np.arange(5.)
    b = np.array([2.1, 3.8])
    truth = np.array([2, 4])
   asset_array_equal(truth, resample_nn_1d(a, b))

(I hope I got that right.) It should go in something like test_tools.py under metpy/calc, assuming that's where we put the function.

Anyhow, I hope I didn't scare you off with the requests. 😄

[ahaberlie]

No problem! I understand needing to keep up the integrity of the codebase. I will incorporate your points as soon as I have some time.

[dopplershift]

FYI, we just moved from nose to pytest for testing, so don't rely on anything in nose.tools when you add a test. It shouldn't really matter to you, but I just wanted to give you a heads up.

[dopplershift]

Thanks for getting this in shape.

For future reference, it's better not to merge master into your branch, but rather rebase your branch onto master (assuming it's up-to-date) using:

git rebase master <mybranch>

This just keeps the history cleaner.

Thanks for the contribution!