torchMoji
torchMoji copied to clipboard
Housekeeping - upgrade to Torch 1.3.1
Thanks for open-sourcing this work!
I'm afraid I couldn't get the dependencies to resolve without conflicts when following the README instructions, so this PR upgrades the dependency.
Since the README suggested installing PyTorch with Conda originally, I've included an environment.yml
that will hopefully make this easier for future users.
To verify the changes I've run:
-
nosetests -v -a '!slow'
-
examples/text_emojize.py
-
examples/finetune_youtube_last.py
These all work as expected, although I can't guarantee this fixes every incompatibility.
@cbowdon This PR was helpful, thanks! I recommend that it be merged in.
Thanks for your work on this, @cbowdon!
Would be great if this could be merged in.
@thomwolf: Thanks for porting this from the source Keras implementation, and your continuing work with HuggingFace (big fan of transformers
and tokenizers
👏 )! When you have a chance, can you please review this?
Hi! Thank you for your work on Torchmoji! Why hasn't this branch been merged? It's too bad to be stuck with torch 1.0.1