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

ULMFiT #179

Merged
merged 23 commits into from
Jan 19, 2020
Merged

ULMFiT #179

merged 23 commits into from
Jan 19, 2020

Conversation

aviks
Copy link
Member

@aviks aviks commented Dec 12, 2019

This is an updated version of #168, with most of the work by @ComputerMaestro. I've cleaned it up a bit and made all tests pass (see last commit)

A couple of important points

  • Flux is pinned to 0.9
  • the UMLFiT code is in the module TextAnalysis.ULMFiT

@ComputerMaestro please take a look and verify

@aviks aviks mentioned this pull request Dec 12, 2019
8 tasks
@ComputerMaestro
Copy link
Contributor

I have checked it, it is working properly

@aviks aviks merged commit 912aefb into master Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants