diff --git a/docs/sphinx/source/whatsnew/v0.5.1.rst b/docs/sphinx/source/whatsnew/v0.5.1.rst index f1ef0be9be..992a7aad3d 100644 --- a/docs/sphinx/source/whatsnew/v0.5.1.rst +++ b/docs/sphinx/source/whatsnew/v0.5.1.rst @@ -1,26 +1,13 @@ .. _whatsnew_0510: -v0.5.1 (?, 2017) ------------------------- +v0.5.1 (October 17, 2017) +------------------------- API Changes ~~~~~~~~~~~ -* - -Bug fixes -~~~~~~~~~ -* Remove condition causing Overflow warning from clearsky.haurwitz -* modelchain.basic_chain now correctly passes 'solar_position_method' - arg to solarposition.get_solarposition -* Fixed: `Variables and Symbols extra references not available `_ -* Removed unnecessary calculations of alpha_prime in spa.solar_position_numpy - and spa.solar_position_loop -* Fixed args mismatch for solarposition.pyephem call - from solarposition.get_solarposition with method='pyephem' - arg to solarposition.get_solarposition (:issue:`370`) -* ModelChain.prepare_inputs and ModelChain.complete_irradiance now - correctly pass the 'solar_position_method' argument to - solarposition.get_solarposition (:issue:`377`) +* `pvsystem.v_from_i` and `pvsystem.i_from_v` functions now accept + resistance_series = 0 and/or resistance_shunt = numpy.inf as inputs + (:issue:`340`) Enhancements ~~~~~~~~~~~~ @@ -32,18 +19,30 @@ Enhancements resistance_series and/or resistance_shunt may still cause issues with the implicit solver (:issue:`340`) -API Changes -~~~~~~~~~~~ -* `pvsystem.v_from_i` and `pvsystem.i_from_v` functions now accept - resistance_series = 0 and/or resistance_shunt = numpy.inf as inputs - (:issue:`340`) +Bug fixes +~~~~~~~~~ +* Remove condition causing Overflow warning from clearsky.haurwitz + (:issue:`363`) +* modelchain.basic_chain now correctly passes 'solar_position_method' + arg to solarposition.get_solarposition (:issue:`370`) +* Fixed: `Variables and Symbols extra references not available + `_ (:issue:`380`) +* Removed unnecessary calculations of alpha_prime in spa.solar_position_numpy + and spa.solar_position_loop (:issue:`366`) +* Fixed args mismatch for solarposition.pyephem call + from solarposition.get_solarposition with method='pyephem' + arg to solarposition.get_solarposition (:issue:`370`) +* ModelChain.prepare_inputs and ModelChain.complete_irradiance now + correctly pass the 'solar_position_method' argument to + solarposition.get_solarposition (:issue:`377`) +* Fixed usage of inplace parameter for tmy._recolumn (:issue:`342`) Documentation ~~~~~~~~~~~~~ * Doc string of modelchain.basic_chain was updated to describe args - more accurately + more accurately. (:issue:`370`) * Doc strings of `singlediode`, `pvsystem.v_from_i`, and `pvsystem.i_from_v` - were updated to describe acceptable input arg ranges + were updated to describe acceptable input arg ranges. (:issue:`340`) Testing ~~~~~~~ @@ -57,3 +56,4 @@ Contributors * KonstantinTr * Will Holmgren * Mark Campanelli +* DaCoEx diff --git a/pvlib/tmy.py b/pvlib/tmy.py index 833ecdd220..111f2bab91 100644 --- a/pvlib/tmy.py +++ b/pvlib/tmy.py @@ -187,7 +187,7 @@ def readtmy3(filename=None, coerce_year=None, recolumn=True): index_col='datetime') if recolumn: - _recolumn(data) # rename to standard column names + data = _recolumn(data) # rename to standard column names data = data.tz_localize(int(meta['TZ']*3600)) @@ -213,7 +213,7 @@ def _parsedate(ymd, hour, year=None): return true_date -def _recolumn(tmy3_dataframe, inplace=True): +def _recolumn(tmy3_dataframe): """ Rename the columns of the TMY3 DataFrame. @@ -251,7 +251,7 @@ def _recolumn(tmy3_dataframe, inplace=True): mapping = dict(zip(raw_columns.split(','), new_columns)) - return tmy3_dataframe.rename(columns=mapping, inplace=True) + return tmy3_dataframe.rename(columns=mapping) def readtmy2(filename):