(1) Update docstring for cols variable in spatial.py/plot_bands f(x) for spelling and default value and (2) Update formatting of variables in plot_rgb and plot_bands functions. about earthpy

[lwasser]

Hey @yojihigh . thank you for this issue. Just a note that you can link to a line of code like this:
https://github.com/earthlab/earthpy/blob/master/earthpy/spatial.py#L290 this is valuable especially given often the code changes as pull requests are merged

Your issue is a bit unclear. Given there are other people updating docstrings for functions in spatial.py can you kindly specify 2 functions that you'd like to update? if that is what you'd like to do.

We are asking people to follow the rasterio function documentation format for each function.
See example here

rasterio formats functions as follows

"""Display a raster or raster band using matplotlib.
     Text here if needed to describe the function

    Parameters
    ----------
    source : array or dataset object opened in 'r' mode or Band or tuple(dataset, bidx)
        If Band or tuple (dataset, bidx), display the selected band.
        If raster dataset display the rgb image
        as defined in the colorinterp metadata, or default to first band.
    with_bounds : bool (opt)
        Whether to change the image extent to the spatial bounds of the image,
        rather than pixel coordinates. Only works when source is....

Please have a look and ping me when you have a more specific update proposal! i just want to avoid merge conflicts on these pull requests.

[yojihigh]

Hey @lwasser Updated my proposed changes with additional detail. Please let me know if this makes it clearer!

[lwasser]

hey @yojihigh this is looking very good! please go ahead and submit your PR! please be sure however to UPDATE YOUR FORK first because there have been a lot of changes today! When you submit your PR please ping me for a review. I suggest you try to do this TONIGHT!!

[yojihigh]

Pull Request created and submitted to resolve this issue
#94

[lwasser]

Closing this as the PR was merged. Thank you again @yojihigh