Skip to content

Conversation

@curieshicy
Copy link
Contributor

Pavol,
I followed your instructions and arrived at this pull request step. If you see issues, please free to reject.

Copy link
Member

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

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

good start, but needs some updates.

xINTERPDF
##########

xINTERPDF is a Python GUI program for analyzing X-ray pair distribution function (PDF) data of organic compounds collected at synchrotron and/or laboratory X-ray sources. It uses DiffPy-CMI as a backend for simulation of PDFs. Currently it supports (1) The study of intermolecular interaction (e.g. hydrogen bonds) by subtracting out the scattering signal(s) of single molecule(s) in real space. (2) The PDF model fit of the crystalline organic compound using the method proposed by Prill et al. (*J. Appl. Cryst.*, **48**, 171-178, 2015.) (3) The phase quantification of physical mixtures of organics. (4) Generate Score/Scree plots based on Principle Component Analysis (PCA).
Copy link
Member

Choose a reason for hiding this comment

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


User Guide
-------------
Please see `User manual <https://github.com/curieshicy/xINTERPDF/blob/master/xINTERPDF_User_Guide_20180615.pdf>`__
Copy link
Member

Choose a reason for hiding this comment

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

Please copy the user guide into the static_root/doc/xinterpdf folder and link it as

<../doc/xinterpdf/xINTERPDF_User_Guide_20180615.pdf>`__

It is better to have all documentation served from diffpy.org.

Please see `User manual <https://github.com/curieshicy/xINTERPDF/blob/master/xINTERPDF_User_Guide_20180615.pdf>`__
for more details.

Reference
Copy link
Member

Choose a reason for hiding this comment

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

Should be References

papers:

* Chenyang Shi, “xINTERPDF: a GUI program for analyzing intermolecular pair distribution functions of organic compounds from X-ray total scattering data”, under review, *J. Appl. Cryst.*, 2018.
* Pavol Juhás, Christopher L. Farrow, Xiaohao Yang, Kevin R. Knox, and Simon J. L. Billinge, “Complex modeling: a strategy and software program for combining multiple information sources to solve ill posed structure and nanostructure inverse problems”, *Acta Crystallogr. A*, **71**, 562-568, 2015.
Copy link
Member

Choose a reason for hiding this comment

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

Please replace with abbreviation |citeJuhasAca15|. You will also need to add the following line to the end of the rst file:

.. include:: ../abbreviations.txt

we ask you to acknowledge the use of xINTERPDF by citing the following
papers:

* Chenyang Shi, “xINTERPDF: a GUI program for analyzing intermolecular pair distribution functions of organic compounds from X-ray total scattering data”, under review, *J. Appl. Cryst.*, 2018.
Copy link
Member

Choose a reason for hiding this comment

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

Please add abbreviation |citeShiJac18| into the abbreviations.txt file using the same style as other entries there.

Then cite here as |citeShiJac18|. This will let us reuse it in the Publications page when accepted.

GNUmakefile Outdated
@@ -0,0 +1,2 @@
include Makefile
PUBLISHBRANCH = gh-pages
Copy link
Member

Choose a reason for hiding this comment

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

Please delete this file. It is intended for your working copy only and should not be committed.

Its purpose is to setup preview page under your account rather than to diffpy.org.

@curieshicy
Copy link
Contributor Author

Pavol,
Please take a look at updated version.

@pavoljuhas
Copy link
Member

@curieshicy - I made a few corrections, see 541c76b...172b0c8.
Let me know if this looks good to you.

@pavoljuhas pavoljuhas merged commit df8c43c into diffpy:source Jun 25, 2018
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