Ticket #7611 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Mock-up for new ALC Analysis interface

Reported by: Anders Markvardsen Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.2
Component: Muon Keywords:
Cc: Blocked By:
Blocking: Tester: Jay Rainey

Description

Together with Muon instrument scientist create mockup for new interface

Change History

comment:1 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

First version of ALC Analysis mock-up.

Refs #7611

Changeset: d20e95862bedb293502d9c01a04ba29622723a95

comment:2 Changed 7 years ago by Arturs Bekasovs

Results of the last meeting:

Scientists agreed on the suggestion to keep plots in the interface and make it possible to export them to the workspaces in the ADS.

Feature request: make it possible to set last_run to "Auto" which will make the interface to observe changes in the directory of the first_run and append any new files to the data set. Is a great enhancement and will need to look into that as soon as basic functionality is working. (#9214)

Addition: need to add missing parameters from the AsymmetryByLogValue, such as integration start_x and end_x and integration type - integral, differential. (Was added in the new version of the mock-up)

Feature request: on the loading step, make it possible to play with parameters without reloading the files. This means keeping all the original data in memory, which might not be feasible. Wait for Stephen or Jamie to provide a large data set to play with it. (#9215)

Addition: in addition to specifying data sections for polynomial function, specify some points which will be used for fit instead of the original data. This might be done after section-based approach is implemented, and used as another "mode". Anders suggested a spline fit function could be used for that. (#9216)

Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:3 Changed 7 years ago by Arturs Bekasovs

  • Milestone changed from Release 3.0 to Release 3.1

comment:4 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from Muon: mockup for new PlotAsymmetryByLogValue to Muon: Mock-up for new ALC Analysis interface

New interface will use PlotAsymmetryByLogValue, but is not a replacement for it'c current interface.

comment:5 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from Muon: Mock-up for new ALC Analysis interface to [Muon] Mock-up for new ALC Analysis interface

comment:6 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from [Muon] Mock-up for new ALC Analysis interface to Mock-up for new ALC Analysis interface

comment:7 Changed 7 years ago by Arturs Bekasovs

  • Milestone changed from Release 3.1 to Backlog

Still waiting for scientists feedback on that one, and it doesn't seem to be very urgent.

comment:8 Changed 7 years ago by Arturs Bekasovs

Refs #7611. No need to store rendered mock-up in the VCS

Changeset: 0db27b74a2572f996340f3a61befdf066ca8b1f8

comment:9 Changed 7 years ago by Arturs Bekasovs

Refs #7611. New version of Muon ALC mock-ups

Changeset: 0aadb7c0ccff90ab3a032448d09e585d2f104532

comment:10 Changed 7 years ago by Arturs Bekasovs

  • Status changed from inprogress to verify
  • Resolution set to fixed
  • Milestone changed from Backlog to Release 3.2

I've sent renders of these mock-up to Stephen Cottrell and he responded:

I've now spoken with Jamie regarding the ALC interface, and believe we're both happy that you go ahead and implement the interface based on your attached mock-ups.

Which means I can close this ticket and open another one for the actual implementation. Implementation ticket: #9213.

Tester: Check that the mock-up was really added. :-) Then go to the comment:2 and verify that all the scientists' suggestions were addressed or will be addressed in the appropriate tickets.

Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:11 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying
  • Tester set to Jay Rainey

comment:12 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

Separate tickets have been opened for each feature/issue noted by the muon scientists in their meeting with Arturs (comment:2). The updated mock-up has been added to the documents repo.

As the scientists are happy with the mockups, and have gave the go ahead for implementation this ticket is now void. Closing.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8456

Note: See TracTickets for help on using tickets.