-
Notifications
You must be signed in to change notification settings - Fork 36
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
Make code pip-installable #1
Comments
Hi Cthoyt, This is an excellent suggestion for improving! With a few more changes coming soon to the repo., we will certainly add a pip install extension! Thank you!! |
@akshat998 thanks for the feedback. I guess you mean that you will take care of it after you make some improvements? Otherwise I'd be happy to send a PR |
Yup! - - we will add this after the extension :) |
Hi @akshat998 , any news on this issue? |
@ratthachat: thanks for checking in on this! Unfortunately, I have not had the time to implement this feature. @cthoyt: I was wondering if it might be possible for your to help here? A PR would be amazing :) Thank you for the reminder! |
TBH, I've lost interest. I was bummed to see so much advertisement about stoned-selfies without more attention payed to packaging and reproducibility earlier on :/ |
It would be great for potential users of STONED-SELFIES if it were possible to install the code with pip, and later from PyPI. Would you be willing to accept a PR that does this?
It would involve a bit of re-organize the code into the
src/
layout, the addition of asetup.py
andsetup.cfg
file, and the addition of aMANIFEST.in
file if thedata/
folder should be distributed with the package.The text was updated successfully, but these errors were encountered: