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

Delete dataset bottom in UI #197

Closed
weixuanfu opened this issue May 6, 2019 · 5 comments · Fixed by #611
Closed

Delete dataset bottom in UI #197

weixuanfu opened this issue May 6, 2019 · 5 comments · Fixed by #611

Comments

@weixuanfu
Copy link
Contributor

For now, users cannot delete the dataset or re-upload it (see the screenshot below) if the dataset has been uploaded to PennAI and also cannot change the metafeatures (categorical features or ordinal features) even if users found something wrong after uploading.

I suggest that we should have edit metafeatures bottom and delate dataset bottom in UI.

image

@HyunjunA
Copy link
Contributor

I propose the following idea in relation to the image below. The screenshot I shared displays a delete button for the card.

Image

@HyunjunA
Copy link
Contributor

HyunjunA commented Jul 7, 2023

Jay suggested an idea on the delete button " it may be better to move that delete button so it's not next to the AI button, user may click 'delete' accidentally."

@jay-m-dev
Copy link
Contributor

deleting the dataset entails removing the dataset, along with all related experiments and files attached to the dataset. We'll need a solution that will delete all these interdependent files and documents.

@jay-m-dev jay-m-dev pinned this issue Sep 12, 2023
@jay-m-dev
Copy link
Contributor

jay-m-dev commented Sep 18, 2023

Tasks to be completed for dataset deletion:

  • Add delete icon on GUI (@HyunjunA)
  • Delete dataset from datasets collection
  • Delete dataset file from GridFS
  • Delete linked experiments
  • Delete GridFS files
  • Delete chats and chatlogs
  • Delete linked code executions
    • Delete generated files from code executions

@jay-m-dev
Copy link
Contributor

All tasks to delete in the backend are ready and merged on PR #610. The last step is to implement the frontend.
@HyunjunA, please add the icon and please test that a deleted dataset can be re-uploaded and experiments can be run after re-uploading.

@jay-m-dev jay-m-dev linked a pull request Sep 26, 2023 that will close this issue
@jay-m-dev jay-m-dev unpinned this issue Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants