jgliss / pyplis

Python toolbox for the analysis of UV SO2 camera data
GNU General Public License v3.0
7 stars 5 forks source link

Py3 cleanup #26

Closed johannjacobsohn closed 5 years ago

johannjacobsohn commented 5 years ago

Some lint cleanup and wording commits, an odd pytest fix (fed120a) caused by a bug in astropy, improve error handling in tests scripts (c88c817) which would otherwise run happily and report no problems even if tests failed, as well as a fixed requirements file for tox (c28dd7c).

Also replaced from_csv with read_csv (fc70375) as the first has been deprecated and removed:

jgliss commented 5 years ago

@johannjacobsohn thanks for this PR and for further testing. Can you maybe specify a bit more detailed your system setup and what tests were failing in ex06_doas_calib.py before you increased the tolerance level of the tests (which test variables, and which values did you get)? I am asking because, as you know, the calibration is crucial for the emission-rate retrieval and I think we should aim for getting the same results, no matter which platform or python version, etc., within a tolerance level, that we consider sufficient. I am afraid, we'll forget about it, if I merge this now ;)

johannjacobsohn commented 5 years ago

@jgliss that is true, I switched to conda and reverted b523bbf, which has disappeared by itself (?). Also replaced from_csv w/ read read_csv becaused it crashed with newer versions of pandas/py3

jgliss commented 5 years ago

Thanks @johannjacobsohn. pandas.read_csv is backwards compatible with older pandas versions, right?

johannjacobsohn commented 5 years ago

Thanks @johannjacobsohn. pandas.read_csv is backwards compatible with older pandas versions, right?

yes