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

Test adaptive radius integration #490

Merged
merged 1 commit into from Mar 31, 2015
Merged

Test adaptive radius integration #490

merged 1 commit into from Mar 31, 2015
+12 −1

Conversation

VickieLynch
Copy link
Contributor

Fixes issue 11435

Check that unit test works using AdaptiveQMultiplier.

@FedeMPouzols FedeMPouzols self-assigned this Mar 31, 2015
@FedeMPouzols
Copy link
Contributor

Jenkins, retest this please

@FedeMPouzols FedeMPouzols added the Framework Issues and pull requests related to components in the Framework label Mar 31, 2015
@FedeMPouzols FedeMPouzols added this to the Release 3.4 milestone Mar 31, 2015
@FedeMPouzols
Copy link
Contributor

This extends the unit test IntegratePeaksMD2 to cover new properties that were added in a previous ticket. It all looks good and seems to be working well. We got unrelated test failures on two platforms (two well known friends: LoadFullprofResolutionTest which lately tends to be dizzy, and the MERLIN system test on rhel7). I'd put my hand in the fire that this is just fine.

@FedeMPouzols
Copy link
Contributor

Jenkins, retest this please

2 similar comments
@FedeMPouzols
Copy link
Contributor

Jenkins, retest this please

@VickieLynch
Copy link
Contributor Author

Jenkins, retest this please

FedeMPouzols added a commit that referenced this pull request Mar 31, 2015
@FedeMPouzols
Merge pull request #490 from mantidproject/11435_adaptive_test
84ea0e6 
Test adaptive radius integration
@FedeMPouzols FedeMPouzols merged commit 84ea0e6 into master Mar 31, 2015
@FedeMPouzols FedeMPouzols deleted the 11435_adaptive_test branch March 31, 2015 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@FedeMPouzols FedeMPouzols

Labels
Framework Issues and pull requests related to components in the Framework
Projects
None yet
Milestone
Release 3.4
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@VickieLynch @FedeMPouzols