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

[BUG] ArgillaTrainer imports are not working correctly #3345

Closed
davidberenstein1957 opened this issue Jul 5, 2023 · 0 comments · Fixed by #3346
Closed

[BUG] ArgillaTrainer imports are not working correctly #3345

davidberenstein1957 opened this issue Jul 5, 2023 · 0 comments · Fixed by #3346
Assignees
@frascuchon @Amelie-V
Labels
area: api Indicates that an issue or pull request is related to the Fast API server or REST endpoints area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior

Comments

@davidberenstein1957
Copy link
Member

Describe the bug
Import for the ArgillaTrainer w.r.t. TaskMapping are incorrect.

Stacktrace and Code to create the bug
N.A.

Expected behavior
A clear and concise description of what you expected to happen.

Environment:

  • Argilla Version [e.g. 1.0.0]: 1.12
  • ElasticSearch Version [e.g. 7.10.2]:
  • Docker Image (optional) [e.g. argilla:v1.0.0]:

Additional context
N.A.
@davidberenstein1957 davidberenstein1957 added type: bug Indicates an unexpected problem or unintended behavior ux area: ui Indicates that an issue or pull request is related to the User Interface (UI) area: api Indicates that an issue or pull request is related to the Fast API server or REST endpoints labels Jul 5, 2023
@davidberenstein1957 davidberenstein1957 mentioned this issue Jul 5, 2023
Merged
7 tasks
@davidberenstein1957 davidberenstein1957 linked a pull request Jul 6, 2023 that will close this issue
fix: resolved import errors ArgillaTrainer #3346
Merged
7 tasks
davidberenstein1957 added a commit that referenced this issue Jul 6, 2023
@davidberenstein1957 @frascuchon
fix: resolved import errors ArgillaTrainer (#3346)
78933b6 
<!-- Thanks for your contribution! As part of our Community Growers
initiative 🌱, we're donating Justdiggit bunds in your name to reforest
sub-Saharan Africa. To claim your Community Growers certificate, please
contact David Berenstein in our Slack community or fill in this form
https://tally.so/r/n9XrxK once your PR has been merged. -->

# Description

The imports for `ArgillaTrainer` where not setup correctly.

Closes #3345 

**Type of change**

- [X] Bug fix (non-breaking change which fixes an issue)

**How Has This Been Tested**

N.A.

**Checklist**

- [X] I added relevant documentation
- [X] follows the style guidelines of this project
- [X] I did a self-review of my code
- [X] I made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [X] I have added relevant notes to the CHANGELOG.md file (See
https://keepachangelog.com/)

---------

Co-authored-by: Francisco Aranda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@frascuchon frascuchon

@Amelie-V Amelie-V

Labels
area: api Indicates that an issue or pull request is related to the Fast API server or REST endpoints area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Milestone
No milestone
3 participants
@frascuchon @davidberenstein1957 @Amelie-V