Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Blank instrument view with U correction #18959

Merged
merged 3 commits into from Feb 23, 2017
Merged

Blank instrument view with U correction #18959

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9667cea
Refs #18869 Fix blank view with no U correction
samueljackson92 Feb 21, 2017
6dea8af
Merge branch '18869_u_correction_instrument_view' of https://github.c…
samueljackson92 Feb 21, 2017
0d30e7c
clang-format PR18959 6dea8af
Feb 21, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions MantidQt/MantidWidgets/src/InstrumentView/RotationSurface.cpp
View file
Open in desktop
Expand Up @@ -162,6 +162,7 @@ void RotationSurface::init() {
udet.u = applyUCorrection(udet.u);
}
}
updateViewRectForUCorrection();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the text need to be indented so much from line 89 to 164? Otherwise it fixes the problem

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, I tried to fix this in the previous PR but the code has detector info changes which I don't think we want in the patch release so I took that commit out.

}

/** Update the view rect to account for the U correction
Expand Down