argilla-io / argilla

Argilla is a collaboration tool for AI engineers and domain experts to build high-quality datasets
https://docs.argilla.io
Apache License 2.0
3.91k stars 367 forks source link

pass stable props into components #2142

Closed keithCuniah closed 10 months ago

keithCuniah commented 1 year ago

Is your feature request related to a problem? Please describe. In order to not have too much rendering, we need to ensure that the props we pass to the composent is stable because each time a props changes, each time the component which that props belongs will be re-rendered

Describe the solution you'd like Pass only the necessary data and not all the dataset object to the components :

From the most nested children to the highest parent :

REFACTORING RECORD LEVEL TextClassification Task

TokenClassification Task

Text2Text Task

REFACTORING HEADER LEVEL AppHeader Token classification task :

Text classification task :

Text2Text classification task

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

keithCuniah commented 1 year ago

Note : The changes in this refacto issue is not difficult to develop but takes more time to test since it's concerned almost every components from the fron app. The idea is to complete the changes in cooldown times or before implementing a feature which will be impacted (as Bulk annotation).

github-actions[bot] commented 11 months ago

This issue is stale because it has been open for 90 days with no activity.

github-actions[bot] commented 10 months ago

This issue was closed because it has been inactive for 30 days since being marked as stale.