[GitHub] climate pull request: CLIMATE-508: added a class 'Downscaling' and...

[cgoodale <gi...@git.apache.org>]

Github user cgoodale commented on the pull request:

    https://github.com/apache/climate/pull/112#issuecomment-78422029
  
    Hey @huikyole I was looking into the open issues and I saw this one and I have to agree with @MJJoyce  that example code would be great.  Even a simple example of how the API you defined in the class would be called.
    
    From what I see in the code I "think" it would be something like the following:
    
    # the basic way a user would import the Class
    >>> from ocw.statistical_downscaling import Downscaling
    # simple class instantiation
    >>> my_downscaler = Downscaling(ref, present, future)
    # Then we use the defined downscaler to return values
    >>> downscaled_present, modeled_future = my_downscaler.Delta_addition()
    >>> 
    
    Am I even close here?  Perhaps a unittest would also help convey how the code would be used.
    
    I am not sure I know enough about Downscaling, but all the return documentation is the same:
    :returns: downscaled model_present and model_future
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---