Refactor and clean IsotonicRegression

Refactor and clean IsotonicRegression:

* Rename ``X`` to ``x`` to properly reflect that input is a vector, not matrix
* Refactor the ``y=isotonic_regression(...)`` logic into ``_build_y``
* Refactor the ``interp1d`` logic into ``_build_f``
* Store ``f=interp1d(...)`` so that it is not re-calculated on each call to ``transform``

Sorry about the last one, @agramfort.  Reset and sync'd.
Alright. I think @agramfort thought this line was actually checking for the `out_of_bounds="raise"` case that is actually tested in a separate test, namely `test_isotonic_regression_oob_raise`.