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.

Can you apply this changes? Thank you!


## Description

This model was imported from `Hugging Face` and it's been fine-tuned on emotion [dataset](https://huggingface.co/nlp/viewer/?dataset=emotion), leveraging `Bert` embeddings and `BertForSequenceClassification` for text classification purposes.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think here the link should be to the model (nateraw/bert-base-uncased-emotion) not the dataset

Copy link
Contributor

Choose a reason for hiding this comment

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

@josejuanmartinez
I changed the link

## Benchmarking

```bash
Val. Accuracy : 0.931
Copy link
Contributor

Choose a reason for hiding this comment

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

Val. accuracy is not a useful metric, but I took a look and the author didn't share any other. Can you add the following?

NOTE: The author didn't share Precision / Recall / F1, only Validation Accuracy was shared as [Evaluation Results](https://huggingface.co/nateraw/bert-base-uncased-emotion#eval-results).

Validation Accuracy: 0.931 

Copy link
Contributor

Choose a reason for hiding this comment

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

@josejuanmartinez
I added the text you suggested.

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, thanks!

@josejuanmartinez josejuanmartinez merged commit 11015ff into models_hub Jan 17, 2022
@maziyarpanahi maziyarpanahi deleted the 2022-01-14-bert_sequence_classifier_emotion_en_6NRaOrTmDo55msheHfCfVItQ 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