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

Deleting data from the data explorer does not actually remove all vestiges thereof #3015

Closed
butlerpd opened this issue Aug 15, 2024 · 5 comments
Assignees
Labels
Minor Small job SasView 6.0.0 Required for 6.0.0 release

Comments

@butlerpd
Copy link
Member

Describe the bug
Deleting data from the explorer using the Delete Data button loads a pop up stating This operation will remove the checked data from the data explorer. Do you want to continue? Hitting yes removes it from the explorer and any plot that contains only that data (though it will not remove it from plots that contain other data as well). However it seems to keep a record in memory somewhere of at least some of at least the data title. But it is not accessible otherwise. Loading that dataset again from disk (since it is no longer retrievable) appends a [1] at the end of the file title.

However, Deleting the data from the explorer by right clicking on the data set name in the explorer (context menu) and then choosing Delete loads a pop up stating This operation will remove the selected data sets and all the dependents from the data explorer. Do you want to continue? Hitting yes in this case will behave exactly as before (including leaving the data plotted on a plot that contains other data). However, when reloading the data there is no new [x] appended to the title indicating that in this case all vestiges have been removed.

Finally, loading a new project will load a pop up stating This operation will remove all data, plots and analyses from SasView before loading the project. Do you wish to continue? This of course removes everything as it says and reloading one of the datafiles after this also does not append any [x]

Expected behavior

  • language in the three modal popups should be normalized: All should say "removed from SasView" and both the delete options should say "and all dependents"
  • The Delete Data button should behave like the context menu delete.
  • All dependents should in fact be removed even it part of a plot that contains other data.

This last may be harder so a first step could be the first two bullets?

SasView version (please complete the following information):

  • Version: 6.0.0b2

Operating system (please complete the following information):

  • OS: Windows

Additional context
Discovered as part the NIST CNR summer school

@butlerpd butlerpd added Minor Small job SasView 6.0.0 Required for 6.0.0 release labels Aug 15, 2024
@krzywon
Copy link
Contributor

krzywon commented Aug 16, 2024

The first issue, related to appending the [1] to the data, is new for 6.0 (in fact, it was fixed for some version of 5) and should be fixed for the release. The language in the popups could be fixed at the same time for the release version, but it matches the language used in 5.0.6, so only if it is an easy fix.

@rozyczko
Copy link
Member

Selecting a dataset and pressing the Delete Data button is showing the described problem.
Right-clicking on the dataset and selecting Delete works properly (subsequent loads don't append [1])

@rozyczko
Copy link
Member

Fixed the issue and unified the warning messages.

@butlerpd
Copy link
Member Author

Is there a PR to test this with? Happy to review when ready. Sounds like it should be ready?

@rozyczko
Copy link
Member

Fixed the issue and unified the warning messages.

Whoops, looks like I forgot to actually make a PR...
Here it is now:

#3089

krzywon added a commit that referenced this issue Aug 23, 2024
…a-explorer

Fixed querying of the data dict at the removal step. Fixes #3015
@krzywon krzywon closed this as completed Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Small job SasView 6.0.0 Required for 6.0.0 release
Projects
None yet
Development

No branches or pull requests

3 participants