Skip to content

Conversation

@Samoed
Copy link
Member

@Samoed Samoed commented Nov 23, 2025

Close #3578

@Samoed Samoed added the new dataset Issues related to adding a new task or dataset label Nov 24, 2025
Copy link
Contributor

@KennethEnevoldsen KennethEnevoldsen left a comment

Choose a reason for hiding this comment

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

Should we just run a model on both datasets to compare

I am kinda unsure how about the differences in the data

Copy link
Contributor

Choose a reason for hiding this comment

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

We need to add superseeded_by

Copy link
Member Author

Choose a reason for hiding this comment

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

V1 is already superseeded_by v2

Copy link
Contributor

Choose a reason for hiding this comment

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

v2 needs a superseeded_by v3

Copy link
Member Author

@Samoed Samoed Nov 25, 2025

Choose a reason for hiding this comment

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

Then we should clenup it as in #2900, but I can't find script for this

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe. Try to run this too

@Samoed
Copy link
Member Author

Samoed commented Nov 25, 2025

Results for sentence-transformers/LaBSE

Results
HebrewSentimentAnalysis 0.616357
HebrewSentimentAnalysis.v2 0.68636
HebrewSentimentAnalysis.v3 0.629297

@KennethEnevoldsen
Copy link
Contributor

hmm would have expected it would have gotten better

@Samoed Samoed mentioned this pull request Dec 6, 2025
@Samoed
Copy link
Member Author

Samoed commented Dec 7, 2025

Results for sentence-transformers/LaBSE

Results
HebrewSentimentAnalysis 0.616357
HebrewSentimentAnalysis.v2 0.68636
HebrewSentimentAnalysis.v3 0.629297
HebrewSentimentAnalysis.v4 0.713456

@KennethEnevoldsen
Copy link
Contributor

Do we need to add v3?

@Samoed
Copy link
Member Author

Samoed commented Dec 7, 2025

Changed v4 to v3

@Samoed Samoed enabled auto-merge (squash) December 7, 2025 13:18
@Samoed Samoed merged commit 3982f33 into main Dec 7, 2025
10 checks passed
@Samoed Samoed deleted the HebrewSentimentAnalysis branch December 7, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new dataset Issues related to adding a new task or dataset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add v2 of HebrewSentimentAnalysis

2 participants