Ticket #10965 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Improve and update LoadFITS documentation

Reported by: Federico M Pouzols Owned by: Federico M Pouzols
Priority: major Milestone: Release 3.4
Component: Documentation Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Wedel

Description

This algorithm was created in #10061 and #10383. Its documentation: http://docs.mantidproject.org/nightly/algorithms/LoadFITS-v1.html

will need several updates. Work still needed on this FITS loader includes changing several headers (#10231) and possibly adding support for different formats. The documentation is also a bit sparse at present.

It should be improved in the following respects, among other things:

  • Explain better how the data is loaded into workspaces (groups, MatrixWorkspace, etc.)
  • Explain also the specificities of this implementation (variants of FITS supported, possible limitations, special headers supported, etc.)
  • Headers that are required, recommended, etc. and their intended usage.

Change History

comment:1 Changed 6 years ago by Federico M Pouzols

  • Summary changed from Improve and update LoadFits documentation to Improve and update LoadFITS documentation

comment:2 Changed 6 years ago by Federico M Pouzols

  • Status changed from new to assigned

comment:3 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from assigned to inprogress

clarify current role of IMAGEKEY, re #10965

Changeset: 06bcb8c53a414a89833d51281f4c192a11bc5868

comment:4 Changed 6 years ago by Federico Montesino Pouzols

proper description of alg + simple doc test, re #10965

Changeset: 85bfb6fb7a5d7d8fc7d78441968ef8c98fc9ce84

comment:5 Changed 6 years ago by Federico Montesino Pouzols

a bit more of info about FITS and headers, re #10965

Changeset: 1977676235d5409d9ff41b5182f32fe5586ecdb5

comment:6 Changed 6 years ago by Federico M Pouzols

We can go ahead with the doc improvements, #10231 will then kill some of the input arguments listed/supported at present.

comment:7 Changed 6 years ago by Federico M Pouzols

  • Blocked By 10231 removed

comment:8 Changed 6 years ago by Federico Montesino Pouzols

simplify doc text, use default=0 for ImageKey, re #10965

Changeset: a27f41b3e946eeb544dfca78cfc50580cd442f09

comment:9 Changed 6 years ago by Federico Montesino Pouzols

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

This is being verified as pull request #471.

comment:10 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:11 Changed 6 years ago by Federico Montesino Pouzols

fix workspace obj name in doc test, re #10965

Changeset: 5ebb5471a7e775c38d6d7b9cb6100b16622e8a05

comment:12 Changed 6 years ago by Federico Montesino Pouzols

clarify text, link WorkspaceGroup and fix ws in test, re #10965

Changeset: 9f10aff09611a0b60e9fc94e8b1324fc63673afd

comment:13 Changed 6 years ago by Federico Montesino Pouzols

fix doc-test keeping _ prefix for FITS logs for now, re #10965

Changeset: afdce0c1cec894a04880841b040137f54f241226

comment:14 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:15 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:16 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:17 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:18 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:19 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:20 Changed 6 years ago by Michael Wedel

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

comment:21 Changed 6 years ago by Michael Wedel

  • Status changed from verifying to closed

Merge pull request #471 from mantidproject/feature/10965_improve_update_LoadFITS_alg_doc

The unit and doc tests for this algorithm pass, both on the build servers as well as locally. The documentation reads well, all the links are working. I did not know FITS at all, now it's clear what kind of data are loaded by the algorithm and what the output workspaces contain.

Full changeset: 43b73d0330fad09aeb2d9020dcaa71fa9b4d52d0

comment:22 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11804

Note: See TracTickets for help on using tickets.