Ticket #7305 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

LoadVTK Docs

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By: #7252
Blocking: Tester: Michael Reuter

Description (last modified by Nick Draper) (diff)

Documentation is not helpful enough, and this algorithm will be used by scientists outside of the usual facilities.

  • Summary documentation required
  • Explanation of different modes in description, as well as the current binary layout expected
  • Pros and cons of setting AdaptiveBinned On/Off
  • Custom usage example
  • Better property documentation required

Change History

comment:1 Changed 7 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Owen Arnold

refs #7305. Add docs for usage, description, summary and props.

Also move memory check into MDHisto routine, where it belongs. Memory check is based on a predictive MDHistoWorkspace size, so gives false results for MDEventWorkspaces. Also switched AdaptiveBinned property default to on, since this is the preferred loading route.

Changeset: c462ea069b28206bece5bf65d4c654a3c26a9476

comment:3 Changed 7 years ago by Owen Arnold

  • Blocked By 7252 added

comment:4 Changed 7 years ago by Owen Arnold

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

Tester: There have been some minor code changes, but it would make sense to retest these features using the fly.vtk file (available on request or on Babylon5, see #7252). Unit tests should all be running properly, as these have changed. The main thing to test is that the online documentation http://www.mantidproject.org/LoadVTK is consistent with the LoadVTK algorithm.

comment:5 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:6 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Everything seems to be in order here and looks good.

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from VATES to Framework
  • Description modified (diff)

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8151

Note: See TracTickets for help on using tickets.