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

[DRAFT] create dataset merge branch #5596

Open
wants to merge 223 commits into
base: feat/argilla-direct-feature-branch
Choose a base branch
from
Open

[DRAFT] create dataset merge branch #5596

wants to merge 223 commits into from

Conversation

frascuchon
Copy link
Member

Description

This is a draft PR to deploy a test space creating datasets from the UI

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (change restructuring the codebase without changing functionality)
  • Improvement (change adding some improvement to an existing functionality)
  • Documentation update

How Has This Been Tested

Checklist

  • I added relevant documentation
  • I followed the style guidelines of this project
  • I did a self-review of my code
  • I made corresponding changes to the documentation
  • I confirm My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

damianpumar and others added 30 commits September 24, 2024 13:10
@damianpumar
✨ First iteration for dataset creation
003918d
@damianpumar
🔥 Add di
ad16756
@damianpumar
📝 Support more than subsets
ea9882f
@damianpumar
💄 Fix lint
4f537fb
@leiyre
include dataset_info in datasetCreation
02b25a6
@leiyre
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
9b7ef93 
…lla-io/argilla into feat/create-dataset-from-ui
@leiyre
update some styles for dataset creation
663ddb7
@leiyre
include new dataset creation page and component
3ec8d61
@leiyre
fix lint
08ace3c
@leiyre
fix lint
6df8ba8
@leiyre
fix resizable styles
b8532dd
@leiyre
remove class
05f3e7a
@leiyre
new configuration components
6495e30
@leiyre
fix lint
94b7930
@leiyre
include subset and split selectors and update styles
78ce04b
@leiyre
use FieldType
7175ddc
@leiyre
create DatasetConfigurationField
a15d17f
@leiyre
include draggable area
dcfc758
@leiyre
include empty question and styles
54eb1d6
@leiyre
update styles
f7699bf
@leiyre
create question card
ac36407
@damianpumar
feat: ✨ Add Metadata creation and improve types
46542c4
@damianpumar
feat: 🔥 Add required text field
206383d 
Add required text field if the dataset has just one field or if does not have.
@damianpumar
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
d768181 
…lla-io/argilla into feat/create-dataset-from-ui
@leiyre
remove question and update styles
c602a18
@leiyre
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
806901a 
…lla-io/argilla into feat/create-dataset-from-ui
@damianpumar
feat: ✨ Add single question as required
afe9879
@damianpumar
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
367989f 
…lla-io/argilla into feat/create-dataset-from-ui
@leiyre
include plus icon
5182769
@leiyre
restructure
aa1abb7
leiyre and others added 30 commits October 17, 2024 16:49
@leiyre
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
e9fc63a 
…lla-io/argilla into feat/create-dataset-from-ui
@jfcalvo
Merge branch 'feat/argilla-direct-feature-branch' into feat/add-hub-d…
e84fb45 
…ataset-import
@jfcalvo
Merge branch 'feat/add-hub-dataset-import' into feat/add-support-for-…
89b64df 
…class-labels-and-casting-rows
@jfcalvo
Merge branch 'feat/add-support-for-class-labels-and-casting-rows' int…
1071519 
…o feat/improve-hub-dataset-image-support
@jfcalvo
feat: add support to import ClassLabel values using -1 no label values
e1da026
@frascuchon
perf: Reduce general transaction time
3c66836
@frascuchon
Update argilla-server/src/argilla_server/bulk/records_bulk.py
3ba9a72
@frascuchon
chore: more revisions
8ee67f0
@frascuchon
chore: Change to refresh=wait_for
c92a45c
@frascuchon
fix: add missing refresh
0a9d22d
@frascuchon
Update argilla-server/src/argilla_server/contexts/datasets.py
6d0edba
@leiyre
Update styles and copy
8820db6
@leiyre
update border color
9df5efa
@jfcalvo
Merge branch 'feat/add-hub-dataset-import' into draft/create-dataset-…
3943b2f 
…merge-branch
@jfcalvo
Merge branch 'feat/add-support-for-class-labels-and-casting-rows' int…
2a78661 
…o draft/create-dataset-merge-branch
@jfcalvo
Merge branch 'feat/improve-hub-dataset-image-support' into draft/crea…
d663e08 
…te-dataset-merge-branch
@jfcalvo
Merge branch 'feat/add-support-to-no-label-values' into draft/create-…
e339572 
…dataset-merge-branch
@jfcalvo
Merge branch 'feat/create-dataset-from-ui' into draft/create-dataset-…
1b3b798 
…merge-branch
@leiyre
remove default value for resize
598755d
@damianpumar
🔥 Little improves
af241ef
@leiyre
prevent white spaces
97e1bf6
@leiyre
show selected column and prevent map when some type are selected
f65c77a
@leiyre
Merge branch 'feat/create-dataset-from-ui' of https://github.com/argi…
e14af1b 
…lla-io/argilla into feat/create-dataset-from-ui
@leiyre
disable button when no workspace
7d7519e
@leiyre
include split selector
4deb5b5
@jfcalvo
Merge branch 'feat/argilla-direct-feature-branch' into draft/create-d…
9d3266a 
…ataset-merge-branch
@jfcalvo
Merge branch 'feat/create-dataset-from-ui' into draft/create-dataset-…
8a25211 
…merge-branch
@frascuchon
chore: Apply arg rename
7f6187f
@jfcalvo
chore: force server to change
d019350
@frascuchon
Merge branch 'perf/argilla-server/reduce-transaction-time' into draft…
ac5241e 
…/create-dataset-merge-branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@frascuchon @damianpumar @leiyre @jfcalvo