Skip to content

Conversation

@jsl-models
Copy link
Collaborator

No description provided.

Copy link
Contributor

@josejuanmartinez josejuanmartinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a couple of typos in the python/scala code. Can you check?

.setInputCols(["sentence"])\
.setOutputCol("token")

tokenClassifier = DistilBertForTokenClassification.pretrained("distilbert_token_classifier_typo_detector", "en"))\
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are 2 parenthesis, one only needed.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josejuanmartinez
it is fixed!

.setInputCols(Array("sentence"))
.setOutputCol("token")

val tokenClassifier = DistilBertForTokenClassification.pretrained("distilbert_token_classifier_typo_detector", "en"))\
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two parenthesis, only one needed. And remove also \ in Scala is not needed as you correctly told me some time ago :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josejuanmartinez
it is fixed!

Copy link
Contributor

@josejuanmartinez josejuanmartinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you!

@josejuanmartinez josejuanmartinez merged commit efac917 into models_hub Jan 26, 2022
@maziyarpanahi maziyarpanahi deleted the 2022-01-19-distilbert_token_classifier_typo_detector_en_R3bxMclvDvFZk9qwulf4Kq6r branch September 13, 2022 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants