jepegit / cellpy

extract and tweak data from electrochemical tests of cells
MIT License
88 stars 30 forks source link

JOSS Review - Installation #273

Closed MaximevdHeijden closed 8 months ago

MaximevdHeijden commented 10 months ago

Dear authors,

The installations manual can be clarified and polished, especially because there are three locations with information on the installation:

Moreover, I tried the installation on Windows and using Linux software on Windows: Windows: I tried installing cellpy using a virtual environment and the base, using a virtual environment it resulted in the following error: “CondaSSLError: OpenSSL appears to be unavailable on this machine”, which I could not install with conda. Whereas when installing cellpy using the base it gave the mentioned obdc error, which was not solved by running “conda install -c conda-forge pyodbc”.

Linux: The installation works well using a virtual environment; however the installation of jupyter notebook in the dependency list is missing.

I suggest you create only one page for the installation with multiple options: for beginners/standard and for using windows/linux.

This issue is related to https://github.com/openjournals/joss-reviews/issues/6236

juliawind commented 9 months ago

Installation notes updated and summarized all in one place.