Ticket #5923 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Extract Version Number from Paraview

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.3
Component: VATES Keywords:
Cc: martyn.gigg@… Blocked By:
Blocking: Tester: Dereck Kachere

Description (last modified by Owen Arnold) (diff)

This information will be critical to usage of newer versions of Paraview. We will need Mantid and Paraview versions to match.

Martyn has found that the Paraview version can be extracted into cerr (see attachement). This will be the basis for getting this working.

It would also be best to construct a ParaView version file similar to MantidVersion.cpp.in. That way we can compile-in a compatible Paraview version number against Mantid.

Attachments

version.cpp​ (811 bytes) - added by Owen Arnold 8 years ago.

Change History

Changed 8 years ago by Owen Arnold

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

  • Description modified (diff)

comment:3 Changed 8 years ago by Owen Arnold

refs #5923. Uses version number in query.

This works, but needs a little refactoring to improve readability.

Changeset: 40a81b427f7a89e6605a04ccb73603ebd68e41c0

comment:4 Changed 8 years ago by Owen Arnold

refs #5923. Refactor large method.

Changeset: 2bd626d1af463899692f91d031e6cb23e9f0e87d

comment:5 Changed 8 years ago by Owen Arnold

refs #5923. Fix warning.

Changeset: 8342d9df65ecbd9b456494c02dd0494b5f9ebb9e

comment:6 Changed 8 years ago by Owen Arnold

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

comment:7 Changed 8 years ago by Owen Arnold

Tester: You could try to following. Modify the ParaViewVersion.cpp.in file in the following ways (below) and then re-generating and building kernel, then starting mantidplot and watching the log notifications.

  • Change to another version number, for example "3.14" -> Should print notification and not make vates features available
  • Change to a major only version string, for example "3" -> Similar output as above
  • Change to a random string, for example "bob" -> Similar output as above
  • Change back to the current version of Paraview "3.10", and should say "Paraview available"

comment:8 Changed 8 years ago by Owen Arnold

refs #5923. Always create version cpp file.

Changeset: b6b7cbdac0c4bb00588cf8d575a66682dc77788b

comment:9 Changed 8 years ago by Dereck Kachere

  • Status changed from verify to verifying
  • Tester set to Dereck Kachere

comment:10 Changed 8 years ago by Dereck Kachere

  • Status changed from verifying to closed

Followed testing instructions and Paraview is only available when the version set is the current one, "3.10"

comment:11 Changed 8 years ago by Martyn Gigg

Re-implemented Voigt as a peak function. Refs #5923

Changeset: cf7de44d8cde3aab34815998966eeb29f09c20aa

comment:12 Changed 8 years ago by Owen Arnold

refs #5923. Uses version number in query.

This works, but needs a little refactoring to improve readability.

Changeset: 40a81b427f7a89e6605a04ccb73603ebd68e41c0

comment:13 Changed 8 years ago by Owen Arnold

refs #5923. Refactor large method.

Changeset: 2bd626d1af463899692f91d031e6cb23e9f0e87d

comment:14 Changed 8 years ago by Owen Arnold

refs #5923. Fix warning.

Changeset: 8342d9df65ecbd9b456494c02dd0494b5f9ebb9e

comment:15 Changed 8 years ago by Owen Arnold

refs #5923. Always create version cpp file.

Changeset: b6b7cbdac0c4bb00588cf8d575a66682dc77788b

comment:16 Changed 8 years ago by Martyn Gigg

Re-implemented Voigt as a peak function. Refs #5923

Changeset: cf7de44d8cde3aab34815998966eeb29f09c20aa

comment:17 Changed 8 years ago by Martyn Gigg

Re-implemented Voigt as a peak function. Refs #5923

Changeset: cf7de44d8cde3aab34815998966eeb29f09c20aa

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6769

Note: See TracTickets for help on using tickets.