Skip to content
This repository has been archived by the owner on Jan 29, 2023. It is now read-only.

Add NLTK #56

Closed
wants to merge 1 commit into from
Closed

Add NLTK #56

wants to merge 1 commit into from

Conversation

kklemon
Copy link
Contributor

@kklemon kklemon commented Oct 5, 2018

@ufoym ufoym force-pushed the master branch 2 times, most recently from c77ef77 to 8c5d454 Compare October 11, 2018 08:09
@ufoym
Copy link
Owner

ufoym commented Oct 13, 2018

Thank you for your contribution! Could you provide a set of common deep-learning-based toolkits for natural language processing and add scripts for building images like deepo:all-nlp-py36-cu90?

# all modules with jupyter
for pyver in pyvers:
modules = candidate_modules + ['python==%s' % pyver, 'onnx', 'jupyter']
postfix = 'all-jupyter-py%s' % pyver.replace('.', '')
f.write(get_command(modules, postfix, cuda_ver, cudnn_ver))

@kklemon
Copy link
Contributor Author

kklemon commented Oct 17, 2018

I can, but please also consider my thoughts in #59 regarding the implementation of too many standard pip or apt packages.

@ufoym ufoym closed this Apr 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants