Ticket #1678 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Nexus: Add a copy of NXS C++ API in Nexus project.

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

NexusFile.cpp will be next to NexusFileIO.cpp until it gets cleaned up, in a later iteration.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

(In [6399]) Refs #1678. Attempting to add the official Nexus C++ API files into the MantidNexus project.

comment:3 Changed 10 years ago by Janik Zikovsky

(In [6437]) Refs #1588, #1678. Had a nasty namespace clash between NeXus and Mantid::NeXus in the tests. Renamed the NeXus cpp api namespace to NeXusAPI. LoadLogsFromSNSNexus written and tested on one file. Added size() method to Property objects and sub-objects.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [6445]) Refs #1678: Fix missing include in test.

comment:5 Changed 10 years ago by Martyn Gigg

(In [6475]) Fix symbol visibility define in Nexus CPP API. Re #1678

comment:6 Changed 10 years ago by Martyn Gigg

(In [6478]) Remove a preprocessor check that shouldn't be there as far as I can see. Re #1678

comment:7 Changed 10 years ago by Janik Zikovsky

(In [6547]) Refs #1678: Renamed NeXus c++ API back to NeXus and used ::NeXus scoping operator to avoid namespace clash :)

comment:8 Changed 10 years ago by Janik Zikovsky

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

comment:9 Changed 10 years ago by Michael Reuter

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

comment:10 Changed 10 years ago by Michael Reuter

  • Status changed from verifying to closed

I have used the internalized NeXus API for reading monitor data out of NeXus files, so this does work.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2525

Note: See TracTickets for help on using tickets.