Ticket #2167 (closed: worksforme)

Opened 10 years ago

Last modified 5 years ago

Iteration 27 Doxygen comments

Reported by: Nick Draper Owned by: Nick Draper
Priority: trivial Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description


Change History

comment:1 Changed 10 years ago by Peter Peterson

(In [8360]) Commenting out the Atom and NeutronAtom from doxygen using a preprocessor directive. Refs #2167.

comment:2 Changed 10 years ago by Russell Taylor

Re the commit above, there's a built in way to do this:

/// @cond

at the top, and

/// @endcond

at the bottom. See for example Logger.h

comment:3 Changed 10 years ago by Peter Peterson

(In [8363]) Following Russell's advice. Refs #2167.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [8410]) Refs #2167: runTests.sh paths updated.

comment:5 Changed 10 years ago by Janik Zikovsky

(In [8413]) Refs #2167: CMakeLists amended to add missing files.

comment:6 Changed 10 years ago by Janik Zikovsky

(In [8414]) Refs #2167: More fixes to runTests.sh

comment:7 Changed 10 years ago by Janik Zikovsky

(In [8415]) Refs #2167: What works for me does not work for the build server. Reverting the change.

comment:8 Changed 10 years ago by Janik Zikovsky

(In [8418]) Refs #2167: Doxygen fixes.

comment:9 Changed 10 years ago by Janik Zikovsky

(In [8428]) Refs #2167: monitorTests.py updated for /bin/ output.

comment:10 Changed 10 years ago by Peter Peterson

(In [8436]) Removing executable bit for source files. Refs #2167.

comment:11 Changed 10 years ago by Peter Peterson

(In [8437]) Removing executable bit for source files. Refs #2167.

comment:12 Changed 10 years ago by Peter Peterson

(In [8438]) Removing executable bit for source files. Refs #2167.

comment:13 Changed 10 years ago by Peter Peterson

(In [8439]) Removing executable bit for source files. Refs #2167.

comment:14 Changed 10 years ago by Michael Whitty

(In [8455]) refs #2167 - adjust ANN header to check for IN_MANTID_KERNEL rather than it's own little thing to allow it to build from cmake generated project files

comment:15 Changed 10 years ago by Janik Zikovsky

(In [8459]) Refs #2167: Fix for building Mantidplot if you have two version of qwt.

comment:16 Changed 10 years ago by Michael Whitty

(In [8642]) refs #2167 - fix for Cmake KernelTest's under Windows

comment:17 Changed 10 years ago by Michael Reuter

(In [8677]) Fix typo in include statement. This refs #2167.

comment:18 Changed 10 years ago by Michael Reuter

(In [8686]) Removed unnecessary forward reference. This refs #2167.

comment:19 Changed 10 years ago by Janik Zikovsky

(In [8689]) Refs #2167: Removed LoadInstrumentFromSNSNexus as it is not used anywhere.

comment:20 Changed 10 years ago by Janik Zikovsky

(In [8690]) Refs #2167: Missed a reference to LoadInstrumentFromSNSNexus

comment:21 Changed 10 years ago by Janik Zikovsky

(In [8694]) Refs #2167: Test should not have been un-commented.

comment:22 Changed 10 years ago by Janik Zikovsky

(In [8697]) Refs #2167: ManagedWorkspaceTest failed if run in a different folder.

comment:23 Changed 10 years ago by Janik Zikovsky

(In [8698]) Refs #2167: Fix test for facilities

comment:24 Changed 10 years ago by Janik Zikovsky

(In [8701]) Refs #2167: Fix ManagedWorkspace2D test

comment:25 Changed 10 years ago by Janik Zikovsky

(In [8704]) Refs #2167: Try to make sure the tmp file is deleted before it is created.

comment:26 Changed 10 years ago by Janik Zikovsky

(In [8724]) Refs #2167: CreatePSDBleedMask test fails locally. Made it throw instead of aborting the program, so it plays nicer.

comment:27 Changed 10 years ago by Martyn Gigg

(In [8758]) Refs #2167. Added newer version of Visual Leak Detector.

comment:28 Changed 10 years ago by Peter Peterson

(In [8760]) Adding directory so startup warning message goes away. Refs #2167.

comment:29 Changed 10 years ago by Janik Zikovsky

(In [8799]) Refs #2167: Helps arrayPropertyTest to pass when run in parallel.

comment:30 Changed 10 years ago by Russell Taylor

(In [8801]) Fix failing test. Re #2167.

comment:31 Changed 10 years ago by Janik Zikovsky

(In [8860]) Refs #2167: Used a more straightfoward way to find the # of vectors in creating a workspace = avoid 32-bit overflows. Should replace int's with size_t's.

comment:32 Changed 10 years ago by Martyn Gigg

(In [8870]) Minor update to email reporter for system tests. Re #2167

comment:33 Changed 10 years ago by Janik Zikovsky

(In [8880]) Refs #2167: Fix incorrect memory reporting for Workspace2D's

comment:34 Changed 10 years ago by Janik Zikovsky

(In [8962]) Refs #2167: Possible fix for MantidPlot instrument view of mask workspaces.

comment:35 Changed 10 years ago by Peter Peterson

(In [8966]) Fixing doxygen comments. Refs #2167.

comment:36 Changed 10 years ago by Russell Taylor

(In [8987]) Case fixes so that system tests can run on linux. Re #2167.

comment:37 Changed 10 years ago by Janik Zikovsky

(In [9008]) Refs #2167: Test was failing spuriously.

comment:38 Changed 10 years ago by Janik Zikovsky

(In [9023]) Refs #2167: Made doxygen comment style more consistent: @param instead of \param, etc.

comment:39 Changed 10 years ago by Janik Zikovsky

(In [9024]) Refs #2167: Fixed extra ::

comment:40 Changed 10 years ago by Janik Zikovsky

(In [9025]) Refs #2167: You can't put ClassName:: in front of your function declarations in a header file; I'm surprised this builds under windows.

comment:41 Changed 10 years ago by Michael Whitty

(In [9055]) refs #2167 - some algorithms headers missing from cmakelists

comment:42 Changed 10 years ago by Janik Zikovsky

(In [9064]) Refs #2167: Fixed some compiler warnings (except those in MD* projects).

comment:43 Changed 10 years ago by Janik Zikovsky

(In [9067]) Refs #2167: Possible test fix on ubuntu?

comment:44 Changed 10 years ago by Martyn Gigg

(In [9073]) Added a small txt file listing the Ubuntu package names so that they can be installed from a single shell command. Re #2167

comment:45 Changed 10 years ago by Stuart Campbell

(In [9167]) Remove compiler warnings. refs #2167

comment:46 Changed 10 years ago by Stuart Campbell

(In [9169]) Remove compiler warnings. refs #2167

comment:47 Changed 10 years ago by Janik Zikovsky

(In [9179]) Refs #2167: PlaneImplicitFunctionTest fix.

comment:48 Changed 10 years ago by Janik Zikovsky

(In [9180]) Refs #2167: PlaneImplicitFunctionTest fix.

comment:49 Changed 10 years ago by Stuart Campbell

(In [9190]) Compiler warnings. refs #2167

comment:50 Changed 10 years ago by Janik Zikovsky

(In [9194]) Refs #2167: Quick tool that lists files that you have forgotten to add using svn add command.

comment:51 Changed 10 years ago by Stuart Campbell

(In [9213]) compiler warnings. refs #2167

comment:52 Changed 10 years ago by Janik Zikovsky

(In [9248]) Refs #2167: Timer get elapsed time without resetting made public.

comment:53 Changed 10 years ago by Janik Zikovsky

(In [9252]) Refs #2167: Testviewer shows console output while running a long test, if not in parallel.

comment:54 Changed 10 years ago by Janik Zikovsky

(In [9253]) Refs #2167: Testviewer shows console output while running a long test, if not in parallel.

comment:55 Changed 10 years ago by Michael Whitty

(In [9323]) refs #2167 - cmakelist update for customdialogs to allow windows build

comment:56 Changed 10 years ago by Janik Zikovsky

(In [9366]) Refs #2167: svn tools

comment:57 Changed 10 years ago by Janik Zikovsky

(In [9368]) Refs #2167: test runner tool.

comment:58 Changed 10 years ago by Janik Zikovsky

(In [9378]) Refs #2167: MD fix so it builds on my system.

comment:59 Changed 10 years ago by Janik Zikovsky

(In [9381]) Refs #2167: CheckWorkspacesMatchTest runs in parallel too.

comment:60 Changed 10 years ago by Janik Zikovsky

(In [9387]) Refs #2167: Occasional CheckWorkspacesMatch test failure?

comment:61 Changed 10 years ago by Russell Taylor

(In [9388]) using declarations not allowed in headers. Also fix delete error in test. Re #2167.

comment:62 Changed 10 years ago by Russell Taylor

(In [9389]) Clear print statements that upset tests under scons builds. Re #2167

comment:63 Changed 10 years ago by Janik Zikovsky

(In [9417]) Refs #2167: Couple of fixes to MD stuff.

comment:64 Changed 10 years ago by Russell Taylor

(In [9422]) Fix test. Re #2167

comment:65 Changed 10 years ago by Janik Zikovsky

(In [9425]) Refs #2167: Added parallel interruption checks where they were missing.

comment:66 Changed 10 years ago by Janik Zikovsky

(In [9428]) Refs #2167: Test seems to hang on one of the build servers, not sure why.

comment:67 Changed 10 years ago by Michael Whitty

(In [9442]) refs #2167 - off by one with iterator causes infrequent crash (possibly only in debug mode)

comment:68 Changed 10 years ago by Michael Whitty

(In [9452]) refs #2167 - spenceranalysis unit test update

comment:69 Changed 10 years ago by Janik Zikovsky

(In [9475]) Refs #2167: Fix test.

comment:70 Changed 10 years ago by Stuart Campbell

(In [9478]) Re-insert reporting of differences in data. Seems to have been lost somewhere. refs #2167

comment:71 Changed 10 years ago by Janik Zikovsky

(In [9481]) Refs #2167: Having the same OpenMP critical block in two different algorithms caused a deadlock -> caused by the LoadEventNexus algorithm.

comment:72 Changed 10 years ago by Janik Zikovsky

(In [9490]) Refs #2167: Removed LoadSNSEventNexusTest that was causing a deadlock with LoadEventNexus for some bizarre reason.

comment:73 Changed 10 years ago by Russell Taylor

(In [9491]) Typo. Re #2167.

comment:74 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:75 Changed 10 years ago by Martyn Gigg

(In [9514]) Fix for RPM build script to ensure it doesn't fall over if the debug rpm isn't built. Re #2167

comment:76 Changed 10 years ago by Russell Taylor

(In [9528]) Re #2167.

comment:77 Changed 10 years ago by Michael Whitty

(In [9548]) refs #2167 - some unchecked in additions to osiris parameter files that are necessary.

comment:78 Changed 10 years ago by Nick Draper

  • Status changed from new to verify
  • Owner set to Nick Draper
  • Resolution set to worksforme
  • Milestone changed from Iteration 28 to Iteration 27

comment:79 Changed 10 years ago by Michael Whitty

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

comment:80 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

I have looked at every single one of these changes, and all are good. Yes.

comment:81 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3014

Note: See TracTickets for help on using tickets.