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.

Sign up for GitHub

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

Jump to bottom

Fix test workflow and add zero-shot classifiers for texts lists #1

Merged
merged 2 commits into from
Aug 5, 2023
Merged

Fix test workflow and add zero-shot classifiers for texts lists #1

merged 2 commits into from
Aug 5, 2023

Conversation

Dedalo314
Copy link
Collaborator

@Dedalo314 Dedalo314 commented Jul 28, 2023 •
edited
Loading

  • Fix GitHub test workflow to work with poetry
  • Add zeroshot classifiers

What does this PR do?

The are two main changes in this PR:

  • Fix test workflow to be compatible with Poetry.
  • Add zero-shot classifiers for texts lists based on RoBERTa.

Before submitting

  • Did you make sure title is self-explanatory and the description concisely explains the PR?
  • Did you make sure your PR does only one thing, instead of bundling different changes together?
  • Did you list all the breaking changes introduced by this pull request?
  • Did you test your PR locally with pytest command?
  • Did you run pre-commit hooks with pre-commit run -a command?

Did you have fun?

Make sure you had fun coding 🙃

@Dedalo314 Dedalo314 self-assigned this Jul 28, 2023
@Dedalo314
Fix GitHub test workflow to work with poetry
6c48af7 
The workflow was implemented for pip, so when the project was moved to
poetry it broke. Now it has been fixed to use poetry to manage the
dependencies. The test using wandb has been removed to avoid issues in
the CI pipeline due to missing api key.
@Dedalo314
Add zeroshot classifiers
43e515e 
These models allow to find the closest text among a set of
texts, provided a query. One is based on the RoBERTa model, while the
other is based on a sentence-transformers model trained with a
contrastive loss.
@Dedalo314 Dedalo314 marked this pull request as ready for review August 5, 2023 10:13
@Dedalo314 Dedalo314 merged commit b3cff19 into main Aug 5, 2023
6 checks passed
@Dedalo314 Dedalo314 deleted the daedalus branch August 5, 2023 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Dedalo314 Dedalo314

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Dedalo314