Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

py dependencies cleanup #172

Open
CFGrote opened this issue Jul 26, 2019 · 2 comments
Open

py dependencies cleanup #172

CFGrote opened this issue Jul 26, 2019 · 2 comments
Assignees

Comments

@CFGrote
Copy link
Collaborator

CFGrote commented Jul 26, 2019

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
Copy link
Collaborator Author

CFGrote commented Jul 26, 2019

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

@CFGrote CFGrote assigned CFGrote and ghost Jul 26, 2019
@CFGrote CFGrote added this to TODO in OpenPMD in SIMEX Jul 26, 2019
@JunCEEE
Copy link
Collaborator

JunCEEE commented Jul 26, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

2 participants