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

Undo button not loading correct cached table #277

Open
marshallmcdonnell opened this issue Oct 1, 2019 · 0 comments
Open

Undo button not loading correct cached table #277

marshallmcdonnell opened this issue Oct 1, 2019 · 0 comments
Assignees
Labels
GUI: Bragg / Rietveld Tab The Bragg / Rietveld plotting tab Priority: Medium Medium priority feature / fix ReductionType: IDL Related to the legacy IDL-based reduction from Joerg Type: Bug Bug in application; Crashes; Requires break-fix work

Comments

@marshallmcdonnell
Copy link
Member

If you:

  • Load a table (in --mode idl)
  • Right-click to Clear a row
  • Right-click to Undo a row

you will notice that not all the fields are repopulated. Doing this cycle over an over reveals that the cached state of the table is out of sync, reloading fields in rows or entire rows it shouldn't be.

The methods [save_table and load_table] for this functionality is located at addie.processing.idl.undo_handler.UndoHandler

@marshallmcdonnell marshallmcdonnell added Type: Bug Bug in application; Crashes; Requires break-fix work GUI: Bragg / Rietveld Tab The Bragg / Rietveld plotting tab ReductionType: IDL Related to the legacy IDL-based reduction from Joerg Priority: Medium Medium priority feature / fix labels Oct 1, 2019
@marshallmcdonnell marshallmcdonnell self-assigned this Oct 1, 2019
@marshallmcdonnell marshallmcdonnell added Priority: High High priority feature / fix Priority: Medium Medium priority feature / fix and removed Priority: Medium Medium priority feature / fix Priority: High High priority feature / fix labels Oct 1, 2019
@marshallmcdonnell marshallmcdonnell added this to To do in Version 5.2 release via automation Oct 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI: Bragg / Rietveld Tab The Bragg / Rietveld plotting tab Priority: Medium Medium priority feature / fix ReductionType: IDL Related to the legacy IDL-based reduction from Joerg Type: Bug Bug in application; Crashes; Requires break-fix work
Projects
Development

No branches or pull requests

1 participant