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

Update DOI to include paper #102

Merged
merged 3 commits into from
Dec 11, 2014
Merged

Update DOI to include paper #102

merged 3 commits into from
Dec 11, 2014

Conversation

stuartcampbell
Copy link
Collaborator

Change doi to point to paper.

@stuartcampbell
Update MantidVersion.cpp.in
5d37c05 
Change doi to point to paper.
@NickDraper
Copy link
Contributor

Don't merge this yet! I agree we want to reference the paper, but as well as the version specific doi as opposed to instead. The author list of the paper is fixed, whereas the version specific doi contains all of those that have contributed to that release, which may well include people who were not on the paper, but they do deserve inclusion.

I'll try to make a change tomake it look something like: -
please cite Mantid or the paper .

@stuartcampbell stuartcampbell changed the title Update MantidVersion.cpp.in Update DOI to include paper Dec 11, 2014
@NickDraper
Copy link
Contributor

This should be fine for testing and merging now.

@peterfpeterson peterfpeterson added this to the Release 3.3 milestone Dec 11, 2014
AndreiSavici added a commit that referenced this pull request Dec 11, 2014
@AndreiSavici
Merge pull request #102 from mantidproject/feature/10759_doi
bd07ebc 
Update DOI to include paper
@AndreiSavici AndreiSavici merged commit bd07ebc into master Dec 11, 2014
@AndreiSavici AndreiSavici deleted the feature/10759_doi branch December 11, 2014 21:06
martyngigg added a commit that referenced this pull request Feb 17, 2015
@martyngigg
Revert "Pull request #102 from mantidproject/feature/9843_integrate_s…
3dfa3e9 
…ysmonitor_widget"

This reverts commit 8efc448, reversing
changes made to fbe8a6f.
@Matthew-Andrew Matthew-Andrew mentioned this pull request Apr 12, 2018
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@AndreiSavici AndreiSavici

Labels
None yet
Projects
None yet
Milestone
Release 3.3
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@stuartcampbell @NickDraper @peterfpeterson @AndreiSavici