search

PaNOSC-ViNYL / SimEx

Start-to-end photon experiment simulation platform
https://simex.readthedocs.io/
GNU General Public License v3.0
26 stars 25 forks source link

py dependencies cleanup #172

Open CFGrote opened 5 years ago

CFGrote commented 5 years ago

Hi Carsten,

I’ve installed SIMEX on Maxwell successfully. However, during the installing, I’ve met the following python modules which are not listed in the document (https://eucall-software.github.io/simex_platform/#software-dependencies):

breathe
pint

Are they actually needed? For the breathe module, pyaudio ,which needs extra libraries, is dependent. If they are not actually needed, we could ignore them to make the installation simpler.

Best, Juncheng

CFGrote commented 5 years ago

breathe is only needed to built the documentation, so could move it to a new requirements.txt in Source/doc/source. while on it, we could then also move Source/doc/source to doc/ under the main simex directory ...

pint is the module that takes care of physical units and we need it.

see also #171

JunCEEE commented 5 years ago

OK, then I understand. It seems not necessary to change the file structure. Just an update of the documentation should be enough.