Skip to content
This repository has been archived by the owner on Mar 23, 2021. It is now read-only.

fixed Encoder(When using vocab_expansion) and GensenSingle for CPU #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed Encoder(When using vocab_expansion) and GensenSingle for CPU #9

wants to merge 1 commit into from

Conversation

najafmurtaza
Copy link

@najafmurtaza najafmurtaza commented Aug 18, 2020 •
edited
Loading

  1. GensenSingle was loading model directly to GPU instead of checking with self.cuda().
  • When using vocab_expansion
  1. Encoder had bug with try,except where it was moving all tensors to GPU by default. Now added cuda attribute, which moves the data using self.cuda() checks.

@najafmurtaza
fixed Encoder and GensenSingle for CPU
f1c5f3f 
1. GensenSingle was loading model directly to GPU instead of checking with self.cuda().
2. Encoder had bug with try,except where it was moving all tensors to GPU by default. Now added cuda attribute, which moves the data using self.cuda() checks.
@msftgits
Copy link

msftgits commented Aug 18, 2020 •
edited
Loading

CLA assistant check
All CLA requirements met.

@najafmurtaza najafmurtaza changed the title fixed Encoder and GensenSingle for CPU fixed Encoder(When using vocab_expansion) and GensenSingle for CPU Aug 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@najafmurtaza @msftgits