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.

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

[FEATURE] add __repr__ to argilla.settings._resource.SettingsProperties #5380

Open
davidberenstein1957 opened this issue Aug 5, 2024 · 4 comments
Assignees
Labels
good first issue Indicates a good issue for first-time contributors status: help wanted Indicates that a maintainer wants help on an issue or pull request
Milestone

Comments

@davidberenstein1957
Copy link
Member

davidberenstein1957 commented Aug 5, 2024

Is your feature request related to a problem? Please describe.

These are used for error messaging but don't show the relevant information.

Describe the solution you'd like

I would like to see it as normal printed settings for the dataset.

Describe alternatives you've considered

NA.

Additional context

There might be other classes that need a __repr__.

@davidberenstein1957 davidberenstein1957 added good first issue Indicates a good issue for first-time contributors status: help wanted Indicates that a maintainer wants help on an issue or pull request labels Aug 5, 2024
@davidberenstein1957 davidberenstein1957 added this to the v2.1.0 milestone Aug 5, 2024
@phershbe
Copy link

phershbe commented Aug 6, 2024

Cool project!! I would like to work on this if it is okay. I am pretty inexperienced but this should be self-explanatory. I could check for other classes too but let me start with this. 🙂

@davidberenstein1957
Copy link
Member Author

Hi @phershbe, that would be very welcome. You can tak e a look here to see how you can contribute but feel free to ask for help when needed. https://docs.argilla.io/latest/community/contributor/

@davidberenstein1957
Copy link
Member Author

@phershbe are you still interested in tackling this?

@phershbe phershbe removed their assignment Aug 26, 2024
@phershbe
Copy link

@DavidBerenstein Sorry about this. Yeah I am, I spent quite a bit of time and went through the code base and checked out the __repr__ methods for different classes. I guess that in this case it is going to want to mention the properties but I didn't feel like I knew enough to ask a question and didn't want to submit a really sloppy pull request. I unassigned myself a few minutes ago, I will study whatever is in the pull request when it gets made. Sorry again about not letting you know before.

@nataliaElv nataliaElv modified the milestones: v2.1.0, v2.2.0 Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Indicates a good issue for first-time contributors status: help wanted Indicates that a maintainer wants help on an issue or pull request
Projects
None yet
Development

No branches or pull requests

3 participants