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

Cookiecut (first time) #91

Merged
merged 14 commits into from
Oct 1, 2024
Merged

Cookiecut (first time) #91

merged 14 commits into from
Oct 1, 2024

Conversation

bobleesj
Copy link

@bobleesj bobleesj commented Sep 30, 2024

Cookiecut for the first time

Screenshot 2024-09-30 at 3 41 11 PM

Docs running nicely.

@bobleesj
Copy link
Author

Ready for review @sbillinge

Half of the changes are due to doc files generated automatically.

@bobleesj bobleesj marked this pull request as ready for review September 30, 2024 19:49
Copy link
Contributor

@sbillinge sbillinge left a comment

Choose a reason for hiding this comment

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

just a couple of inline comments

README.rst Outdated

If you use diffpy.srfit in a scientific publication, we would like you to cite this package as

diffpy.srfit Package, https://github.com/diffpy/diffpy.srfit
Copy link
Contributor

Choose a reason for hiding this comment

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

please have them cite the diffpy-cmi complex modeling paper. Please have them cite this same paper for:
srfit
srreal
structure
utils
...am I forgetting any? not pdfmorph/snmf/fourigui which are separate packages.

Copy link
Author

Choose a reason for hiding this comment

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

Screenshot 2024-09-30 at 8 59 46 PM

will update those repos

README.rst Outdated
complex modeling initiative at the Brookhaven National Laboratory. The
diffpy.srfit sources are hosted at
https://github.com/diffpy/diffpy.srfit.
complex modeling initiative at the Brookhaven National Laboratory.
Copy link
Contributor

Choose a reason for hiding this comment

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

please change this to Columbia University.

@@ -156,7 +163,7 @@ Support and Contribute
If you see a bug or want to request a feature, please `report it as an issue <https://github.com/diffpy/diffpy.srfit/issues>`_ and/or `submit a fix as a PR <https://github.com/diffpy/diffpy.srfit/pulls>`_. You can also post it to the `Diffpy user group <https://groups.google.com/g/diffpy-users>`_.

diffpy.srfit is an open-source software developed as a part of the DiffPy-CMI
complex modeling initiative at the Brookhaven National Laboratory.
complex modeling initiative at Columbia University.
Copy link
Author

Choose a reason for hiding this comment

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

Change to Columbia

@@ -67,9 +67,16 @@ http://diffpy.github.io/diffpy.srfit.
Citation
--------

If you use diffpy.srfit in a scientific publication, we would like you to cite this package as
If you use this program for a scientific research that leads
Copy link
Author

Choose a reason for hiding this comment

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

Added citation

  • made PRs to diffpy.srreal and diffpy.utils, adding citation

@bobleesj
Copy link
Author

bobleesj commented Oct 1, 2024

@sbillinge ready for review

@sbillinge sbillinge merged commit 2836264 into diffpy:cookie Oct 1, 2024
2 checks passed
@bobleesj bobleesj deleted the cookie-recut branch October 1, 2024 17:12
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.

2 participants