Merge pull request #137 from openscm/save-read

Fix write read circularity

Author: lewisjared

CHANGELOG.rst

@@ -4,6 +4,8 @@ Changelog
 master
 ------
 
+- (`#137 <https://github.com/openscm/scmdata/pull/137>`_) Fix :meth:`scmdata.ScmRun.to_csv` so that writing and reading is circular (i.e. you end up where you started if you write a file and then read it straight back into a new :obj:`scmdata.ScmRun` instance)
+
 v0.7.6
 ------
 

src/scmdata/run.py

@@ -1850,7 +1850,7 @@ def to_csv(self, fname: str, **kwargs: Any) -> None:
         fname
             Path to write the file into
         """
-        self.timeseries().reset_index().to_csv(fname, **kwargs)
+        self.timeseries().reset_index().to_csv(fname, **kwargs, index=False)
 
     def reduce(self, func, dim=None, axis=None, **kwargs):
         """

tests/integration/test_io.py

@@ -0,0 +1,14 @@
+import os.path
+
+import scmdata
+import scmdata.testing
+
+
+def test_read_write_circularity(test_scm_df_monthly, tmpdir):
+    tempfile = os.path.join(tmpdir, "test.csv")
+
+    test_scm_df_monthly.to_csv(tempfile)
+
+    read = scmdata.ScmRun(tempfile)
+
+    scmdata.testing.assert_scmdf_almost_equal(read, test_scm_df_monthly)