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.

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

Getting the credits correct (hopefully) #113

Merged
merged 21 commits into from
Aug 27, 2020
Merged

Conversation

pc494
Copy link
Member

@pc494 pc494 commented Aug 27, 2020


name: Getting the credits correct (hopefully)
about: Details in #97 and #98


Release Notes

developer change
Summary: Rob Tovey contribution accurately represented.

What does this PR do? Please describe and/or link to an open issue.
Means that Rob is on the credits, get code a little better organised, will be followed with a couple of tidy up PR's, but it would be good to get this in soon to minimise the number of conflicts in other PR's

** ALSO NOTE ** ~ would prefer a squash merge on this one please.

@pc494
Copy link
Member Author

pc494 commented Aug 27, 2020

github makes reviewing errors a lot easier, so I'm going pick a few of them off now.

@pc494 pc494 marked this pull request as draft August 27, 2020 13:47
Copy link
Member Author

@pc494 pc494 left a comment

Choose a reason for hiding this comment

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

Picking off the low hanging style fruit

diffsims/generators/diffraction_generator.py Outdated Show resolved Hide resolved
diffsims/sims/kinematic_simulation.py Outdated Show resolved Hide resolved
diffsims/sims/kinematic_simulation.py Outdated Show resolved Hide resolved
diffsims/sims/kinematic_simulation.py Outdated Show resolved Hide resolved
diffsims/tests/test_sims/test_kinematic_simulation.py Outdated Show resolved Hide resolved
diffsims/utils/discretise_utils.py Outdated Show resolved Hide resolved
diffsims/utils/discretise_utils.py Outdated Show resolved Hide resolved
diffsims/utils/discretise_utils.py Outdated Show resolved Hide resolved
diffsims/utils/discretise_utils.py Outdated Show resolved Hide resolved
diffsims/utils/probe_utils.py Outdated Show resolved Hide resolved
@pc494 pc494 marked this pull request as ready for review August 27, 2020 15:28
@pc494 pc494 removed the status:wip label Aug 27, 2020
@pc494 pc494 requested a review from dnjohnstone August 27, 2020 15:29
@dnjohnstone dnjohnstone merged commit 0f5b65c into pyxem:master Aug 27, 2020
@pc494 pc494 added this to the v0.3.0 milestone Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants