Ticket #8064 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Load Event Monitors does not load sample information for ISIS data

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: major Milestone: Release 3.0
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #7643 Tester: Jay Rainey

Description

In ticket #7985 we have solved the problem of not loading sample information for event workspace.

But, loading monitor still shows the same problem:

run = '19682'
ws = LoadNexus(Filename=run)
ws_ev, mon = LoadEventNexus(Filename=run, LoadMonitors=True)
print ws.sample().getGeometryFlag() 
print ws_ev.sample().getGeometryFlag()
print mon.sample().getGeometryFlag()

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Blocking 7643 added

(In #7643) The monitors showed wrong sample information.

comment:2 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

Move loadISIScompatibility to be static

In order to allow loadNexusMonitors to call it.

re #8064

Changeset: 80a9b19d223508ef8f4d65c600a87fa88e1ceb11

comment:3 Changed 7 years ago by Gesner Passos

LoadNexusMonitors load isis sample information

re #8064

Changeset: 31d39cbc5b3e6fc3edc617cc1b15228b8f966581

comment:4 Changed 7 years ago by Gesner Passos

Tester:

execute the script above and check that they return the same number (3). (cycle_13_2)

comment:5 Changed 7 years ago by Gesner Passos

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:6 Changed 7 years ago by Jay Rainey

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

comment:7 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

Output generated by the script returns the same results (3). Code changes seem fine.

Last edited 7 years ago by Jay Rainey (previous) (diff)

comment:8 Changed 7 years ago by Jay Rainey

Merge remote-tracking branch 'origin/bugfix/8064_monitor_sample_info'

Full changeset: e8091efa75cead75644b2e0d7155ce55ddf2174c

comment:9 Changed 7 years ago by Gesner Passos

Merge branch 'bugfix/8064_monitor_sample_info' into bugfix/7643_sans2d_fixing_reduction

Full changeset: 55540157185d7c1edc0e7339da20151a8b67b10e

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8909

Note: See TracTickets for help on using tickets.