Ticket #7239 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Link NeXus PV plugins to NeXus libraries

Reported by: Michael Reuter Owned by: Michael Reuter
Priority: minor Milestone: Release 3.0
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description

The plugins that handle reading MD NeXus files into ParaView are broken due to the change in #6201. Looks like we need to link directly to the NeXus libraries now.

Change History

comment:1 Changed 7 years ago by Michael Reuter

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Michael Reuter

Refs #7239. NexusPeaksReader plugin needs to link against NeXus libs.

Changeset: 34c54e7c7a221b36dec689157f18e37a5d413b20

comment:3 Changed 7 years ago by Michael Reuter

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

Branch can be found ​here.

To test:

On RHEL6, read any MD NeXus file (like SEQ_MDEW.nxs) into ParaView and it should not crash.

comment:4 Changed 7 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:5 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Release 2.5.4

comment:6 Changed 7 years ago by Stuart Campbell

  • Status changed from verifying to reopened
  • Resolution fixed deleted

Fails on all RHEL6 machines we test on - except for Michaels!

comment:7 Changed 7 years ago by Stuart Campbell

  • Milestone changed from Release 2.5.4 to Release 2.6

comment:8 Changed 7 years ago by Michael Reuter

Seems to be an issue with the system-level threading libraries that is not present on my machine.

comment:9 Changed 7 years ago by Nick Draper

  • Component changed from VATES to User Interface

comment:10 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

comment:11 Changed 7 years ago by Michael Reuter

  • Status changed from reopened to inprogress

Refs #7239. NexusPeaksReader plugin needs to link against NeXus libs.

Changeset: aff2524d1cedc2a83a48f579da17509793c1abd6

comment:12 Changed 7 years ago by Nick Draper

  • Milestone changed from Backlog to Release 3.0

moved to Release 3.0 as these all seem to be active

comment:13 Changed 7 years ago by Michael Reuter

  • Priority changed from blocker to minor
  • Keywords PatchCandidate removed
  • Status changed from inprogress to verify
  • Resolution set to fixed
  • Summary changed from Cannot read MD NeXus files in ParaView on RHEL6 to Link NeXus PV plugins to NeXus libraries

Well, this change managed to make it onto master, so I'm closing the ticket even though the problem isn't fixed.

comment:14 Changed 7 years ago by Stuart Campbell

  • Status changed from verify to verifying

comment:15 Changed 7 years ago by Stuart Campbell

  • Status changed from verifying to closed

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8085

Note: See TracTickets for help on using tickets.