Update for PySAM 2.0 (#874)

* 'none' to None

* require pysam 2.0 or later

Author: cwhanse

ci/requirements-py35.yml

@@ -24,5 +24,5 @@ dependencies:
     - shapely  # pvfactors dependency
     - siphon  # conda-forge
     - pip:
-        - nrel-pysam
+        - nrel-pysam>=2.0
         - pvfactors==1.0.1

ci/requirements-py36.yml

@@ -24,5 +24,5 @@ dependencies:
     - shapely  # pvfactors dependency
     - siphon  # conda-forge
     - pip:
-        - nrel-pysam
+        - nrel-pysam>=2.0
         - pvfactors==1.0.1

ci/requirements-py37.yml

@@ -24,5 +24,5 @@ dependencies:
     - shapely  # pvfactors dependency
     - siphon  # conda-forge
     - pip:
-        - nrel-pysam
+        - nrel-pysam>=2.0
         - pvfactors==1.0.1

docs/sphinx/source/whatsnew/v0.7.2.rst

@@ -18,6 +18,11 @@ Documentation
 ~~~~~~~~~~~~~
 * Add NumFOCUS affiliation to Sphinx documentation :pull:`862`
 
+Requirements
+~~~~~~~~~~~~
+* nrel-pysam (optional) minimum set to v1.0.0 (:issue:`874`)
+
 Contributors
 ~~~~~~~~~~~~
 * Mark Mikofski (:ghuser:`mikofski`)
+* Cliff Hansen (:ghuser:`cwhanse`)

pvlib/ivtools.py

@@ -92,7 +92,7 @@ def fit_sdm_cec_sam(celltype, v_mp, i_mp, v_oc, i_sc, alpha_sc, beta_voc,
         raise ImportError("Requires NREL's PySAM package at "
                           "https://pypi.org/project/NREL-PySAM/.")
 
-    datadict = {'tech_model': '6parsolve', 'financial_model': 'none',
+    datadict = {'tech_model': '6parsolve', 'financial_model': None,
                 'celltype': celltype, 'Vmp': v_mp,
                 'Imp': i_mp, 'Voc': v_oc, 'Isc': i_sc, 'alpha_isc': alpha_sc,
                 'beta_voc': beta_voc, 'gamma_pmp': gamma_pmp,