feat: Add, delete and edit rules from Python client

[frascuchon]

Closes #1855

[dvsrepo]

Hi @ufukhurriyetoglu great work. For updating the docs, I suggest the following:

1. Update https://docs.argilla.io/en/latest/guides/techniques/weak_supervision.html to reflect that Rules created from python can (and should) be added to the dataset. This will make everything much more comfortable for users. Also show you can update and delete rules.
2. Update tutorials https://docs.argilla.io/en/latest/tutorials/notebooks/labelling-textclassification-snorkel-weaksupervision.html and https://docs.argilla.io/en/latest/tutorials/notebooks/labelling-textclassification-sklearn-weaksupervision.html . In this case, I would add the rules we use to the dataset. In this way the tutorials become much more reproducible.

In both cases 1 and 2, test that they are working as expected, and take the opportunity to simplify text if you see the opportunity.

[frascuchon]

Not sure if everything is pushed, but some new tests are failing. Could you please take a look https://github.com/argilla-io/argilla/actions/runs/3479400438/jobs/5817898907?

[davidberenstein1957]

@ufukhurriyetoglu could you also take a look at the docs under the reference/webapp. They currently only contain information about weak labelling for text classification and I feel token classification should be added.

https://docs.argilla.io/en/latest/reference/webapp/features.html#weak-labelling

[ufukhurriyetoglu]

@davidberenstein1957 for the webapp/features I checked the post and it seems related to web UI features. This PR is related to python client and UI does not change. As I understand python side is auto generated, we can check if the added modules reflected.

[dvsrepo]

• Please review my changes, especially for typos.
• Remove output cell from https://docs.argilla.io/en/feat-1855-weak-supervision-rules-python-client/tutorials/notebooks/labelling-textclassification-sklearn-weaksupervision.html#Define-rules

[ufukhurriyetoglu]

great! thanks for the review and corrections @dvsrepo

[dvsrepo]

Hi @ufukhurriyetoglu, typically after changes a new review should be requested. This looks good on my side.

@frascuchon do you need to take another look? Otherwise please merge

[frascuchon]

Just some comments to improve the python reference documentation

See here

[codecov]

Codecov Report

Base: 94.59% // Head: 94.56% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (fe74fe6) compared to base (0ff2de7).
Patch coverage: 93.87% of modified lines in pull request are covered.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1884      +/-   ##
===========================================
- Coverage    94.59%   94.56%   -0.03%     
===========================================
  Files          150      150              
  Lines         7044     7090      +46     
===========================================
+ Hits          6663     6705      +42     
- Misses         381      385       +4     

Flag	Coverage Δ
pytest	94.56% <93.87%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/argilla/client/sdk/client.py	94.25% <50.00%> (-2.18%)	⬇️
src/argilla/labeling/text_classification/rule.py	97.05% <95.65%> (-2.95%)	⬇️
src/argilla/client/api.py	97.08% <100.00%> (+0.11%)	⬆️
src/argilla/client/sdk/text_classification/api.py	100.00% <100.00%> (ø)
...c/argilla/client/sdk/text_classification/models.py	100.00% <100.00%> (ø)
...c/argilla/labeling/text_classification/__init__.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.