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

Problems with Apex #1

Closed
thegallier opened this issue Oct 5, 2019 · 16 comments
Closed

Problems with Apex #1

thegallier opened this issue Oct 5, 2019 · 16 comments

Comments

@thegallier
Copy link

getting error whern training

utils line 196, in
"binary": BinaryProcessor
NameError: name 'BinaryProcessor' is not defined

@ThilinaRajapakse
Copy link
Owner

It should be fixed now. Please update to the latest version (0.2.0).

@thegallier
Copy link
Author

thegallier commented Oct 5, 2019 via email

@thegallier
Copy link
Author

thegallier commented Oct 5, 2019 via email

@ThilinaRajapakse
Copy link
Owner

Found a few more bugs. I think that's all of them though. Do let me know if there are other issues.

@thegallier
Copy link
Author

thegallier commented Oct 5, 2019 via email

@ThilinaRajapakse
Copy link
Owner

ThilinaRajapakse commented Oct 6, 2019

That's on their end. I think the recommended way to install Apex is from the repo. I could add a note to the installation instructions but I don't want to clutter it too much. We'll see if other people raise the same issue. Thank you for letting me know!

@samarthsarin
Copy link

Can you please add the note to installation of apex.

@ThilinaRajapakse
Copy link
Owner

Ok, I'll add it then.

@thegallier
Copy link
Author

thegallier commented Oct 7, 2019 via email

@samarthsarin
Copy link

samarthsarin commented Oct 7, 2019 via email

@ThilinaRajapakse
Copy link
Owner

image

Added this line to the setup instructions.

@samarthsarin
Copy link

samarthsarin commented Oct 7, 2019 via email

@RabbitJackTrade
Copy link

Just to add a note on Apex here. The default setting is 'fp16': True,. If you don't install it and try to train a model, an error message will result. So I would suggest changing this line in the example to:
model = TransformerModel('roberta', 'roberta-base', args=({'fp16': False})
for those who don't install Apex.

@ThilinaRajapakse
Copy link
Owner

Well, it does give a pretty clear error message if Apex is not installed. I think it makes more sense to leave Apex on by default because there's no indicator to show that it is off.

@ThilinaRajapakse ThilinaRajapakse changed the title followed medium Problems with Apex Oct 10, 2019
@RabbitJackTrade
Copy link

Hey, you're the boss! :)

@ThilinaRajapakse
Copy link
Owner

Ideas are always welcome!

ThilinaRajapakse pushed a commit that referenced this issue Dec 28, 2019
ThilinaRajapakse pushed a commit that referenced this issue May 10, 2020
ThilinaRajapakse pushed a commit that referenced this issue Jun 25, 2020
Pulling to update from source.
ThilinaRajapakse pushed a commit that referenced this issue Jul 11, 2021
Apply some markdown formatting to README.md
ThilinaRajapakse pushed a commit that referenced this issue Jan 20, 2022
…_model.py

Fix typo in question_answering_model.py
ThilinaRajapakse pushed a commit that referenced this issue Sep 18, 2022
fixed seq2seq save function to include marian tokenizer files
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

No branches or pull requests

4 participants