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

Standard python package layout #1

Closed
jcfr opened this issue Sep 15, 2015 · 3 comments
Closed

Standard python package layout #1

jcfr opened this issue Sep 15, 2015 · 3 comments

Comments

@jcfr
Copy link
Collaborator

jcfr commented Sep 15, 2015

@fedorov Assuming the package won't depend on Slicer, I would suggest to organize it a standard python package.

See https://github.com/pypa/sampleproject and https://packaging.python.org/en/latest/distributing/

@jcfr jcfr added the question label Sep 15, 2015
@fedorov
Copy link
Collaborator

fedorov commented Sep 16, 2015

@jcfr this is what I thought I was doing - but apparently I was following a guide not so comprehensive as the one you mentioned. Yes, I agree with you, we should do it right. I will look into the guidelines you referenced. Thank you!

@fedorov fedorov self-assigned this Sep 16, 2015
@jcfr
Copy link
Collaborator Author

jcfr commented Sep 16, 2015

One of the idea is to have the setup.py at the base of the repository. It means the content of source would be moved up. And the temp folder would moved into radiomics-temp along side the radiomics one.

You would then have something like :

<root>
   |-- setup.py
   |-- radiomics
   |          |---- __init__.py
   |          |---- base.py
   |          [...]
   |-- radiomics-temp
   |         |----- RadiomicsFeaturesLib
   |                         [...]
   |-- bin
   |       |--- helloRadiomics.py

@fedorov
Copy link
Collaborator

fedorov commented Oct 6, 2015

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

No branches or pull requests

2 participants