Ticket #3911 (closed: wontfix)

Opened 9 years ago

Last modified 5 years ago

Eliminate the use of HET15869 in unit tests

Reported by: Russell Taylor Owned by: Robert Whitley
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

Ahead of the move to git for revision control, it would be good to prune the size of the AutoTestData directory and this 2 file is among the largest in there.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 Changed 9 years ago by Anders Markvardsen

  • Owner changed from Anyone to Robert Whitley

As Nick explained, to help Russell with move to git end of this week

comment:3 Changed 9 years ago by Robert Whitley

  • Status changed from assigned to accepted

comment:4 Changed 9 years ago by Robert Whitley

In [15281]:

Refs #3911. I have eliminated the files use in half of the tests. The other half require a file with one to many mapping so I'm waiting to hear back from Freddie to see if there is a smaller file that uses this form of mapping.

comment:5 Changed 9 years ago by Russell Taylor

HRPD has one-many mapping (i.e. >1 detector pixels contribute to a spectrum) so you should be able to use HRP38692a.nxs that Karl introduced earlier today.

comment:6 Changed 9 years ago by Robert Whitley

Unfortunately the tests require a raw file so using the file stated above wouldn't work.

comment:7 Changed 9 years ago by Robert Whitley

  • Status changed from accepted to verify
  • Resolution set to wontfix

Nick has spoken to Freddie about a smaller one-many mapping raw file but it seems that we already have one of the smallest (HET15869).

comment:8 Changed 9 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:9 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

verified that best efforts were made, but this file is the smallest we can find for the tests.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4758

Note: See TracTickets for help on using tickets.