Skip to content

Commit e374338

Browse files
AlignAndFocusPowderSlim: Only load the 6 banks since everything else assumes we have 6 (#39906)
### Description of work The prototype algorithm [AlignAndFocusPowderSlim](https://docs.mantidproject.org/nightly/algorithms/AlignAndFocusPowderSlim-v1.html) assume we are only loading 6 banks. So for now don't try loading any other banks. As started in the docs "hard coded for 6 particular groups". This is to address the issue of new files having additional banks. This algorithm is under heavy development and we need it to just work for testing. <!-- Please provide an outline and reasoning for the work. If there is no linked issue provide context. --> <!-- If issue raised by user. Do not leak email addresses. **Report to:** [user name] --> ### To test: <!-- Include sufficient instructions for someone unfamiliar with the application to test. Ok to refer back to instructions in the issue. --> This should now be able to load newer files like `VULCAN_260351`. *This does not require release notes* because this is a prototype algorithm that is now doing what the docs say. <!-- REMEMBER: - Add labels, milestones, etc. - Ensure the base of this PR is correct (e.g. release-next or main) - Add release notes in separate file as per ([guidelines](https://developer.mantidproject.org/Standards/ReleaseNotesGuide.html)), or justify their absence: *This does not require release notes* because <fill in an explanation of why> -->
1 parent f2a4bf5 commit e374338

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

Framework/DataHandling/src/AlignAndFocusPowderSlim.cpp

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,10 +335,11 @@ void AlignAndFocusPowderSlim::exec() {
335335
H5::H5File h5file(filename, H5F_ACC_RDONLY, Nexus::H5Util::defaultFileAcc());
336336
if (itClassEntries != allEntries.end()) {
337337
this->progress(.17, "Reading events");
338-
const std::set<std::string> &classEntries = itClassEntries->second;
338+
// const std::set<std::string> &classEntries = itClassEntries->second;
339339

340340
// filter out the diagnostic entries
341341
std::vector<std::string> bankEntryNames;
342+
/*
342343
{
343344
const std::regex classRegex("(/entry/)([^/]*)");
344345
std::smatch groups;
@@ -356,6 +357,12 @@ void AlignAndFocusPowderSlim::exec() {
356357
}
357358
}
358359
}
360+
*/
361+
362+
// hard coded for VULCAN 6 banks
363+
for (size_t i = 1; i <= NUM_HIST; ++i) {
364+
bankEntryNames.push_back("bank" + std::to_string(i) + "_events");
365+
}
359366

360367
// each NXevent_data is a step
361368
const auto num_banks_to_read = bankEntryNames.size();

0 commit comments

Comments
 (0)