Ticket #4362 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Fix LoadGrouping bug on Mac.

Reported by: Robert Whitley Owned by: Robert Whitley
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

In MuonAnalysis, on the grouping tab, click load grouping file and try loading a grouping file.

Mantid will fail.

The message suggests that it is looking for tempMuonAnalysisGrouping.xml and not the actual file.

"Unable to open file: in tempMuonAnalysisGrouping.xml"

Change History

comment:1 Changed 9 years ago by Robert Whitley

N.B. This happens when loading a random file like untitled.xml

comment:2 Changed 9 years ago by Robert Whitley

Found problem area (I think IO_MuonGrouping.cpp)

  std::ofstream outFile(filename.c_str());
  if (!outFile)
  {
    throw Mantid::Kernel::Exception::FileError("Unable to open file:", filename);
  }

comment:3 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 32 to Iteration 33

Moved to iteration 33 at iteration 32 code freeze

comment:4 Changed 9 years ago by Robert Whitley

  • Milestone changed from Iteration 33 to Iteration 32

comment:5 Changed 9 years ago by Martyn Gigg

Refs #4362. Add the instrument/Grouping directory to installers.

Changeset: 917bda630a28316ab441a2245a9e648217cb3c33

comment:6 Changed 9 years ago by Anders Markvardsen

Save temp file in writable location on all platforms. Re #4362

Changeset: b21f98559dd3c696ea8fdbbd9e888b3049d0261c

comment:7 Changed 9 years ago by Anders Markvardsen

Fix to save to better dir. Re #4362

Changeset: 3ff9b97496e4a2fe12d4fee934d6ac45589c3c7c

comment:8 Changed 9 years ago by Anders Markvardsen

I meant this should work. Re #4362

Changeset: 0b5bdbdd0c01fb1cb45bd4cd4333134cd124b513

comment:9 Changed 9 years ago by Anders Markvardsen

Apologize for this another fix to re #4362

Changeset: 6e3da6e72c5fff889a3547ff3538dec1a33c00da

comment:10 Changed 9 years ago by Robert Whitley

  • Status changed from new to accepted

comment:11 Changed 9 years ago by Robert Whitley

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

comment:12 Changed 9 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:13 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

This now works

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5209

Note: See TracTickets for help on using tickets.