-
Notifications
You must be signed in to change notification settings - Fork 28
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
Provide example data and example code to also help newcomers #4
Comments
Yah, the fit doesn’t matter that much
On Fri, Mar 13, 2020 at 7:55 PM longyang ***@***.***> wrote:
I dig around the diffpy-cmi and pdfgui tutorials, and only found CdSe NP
data, which is not so well fitted by bulk model. Is this figure ok?
[image: plot]
<https://user-images.githubusercontent.com/16789768/76669994-9a6a1580-6564-11ea-83ab-4b2dc51100cb.png>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABAOWUMRE6VQ2J3MQAAYUKTRHLBYNANCNFSM4LHJ26QQ>
.
--
Professor Simon Billinge
Columbia University
|
actually, I don't think this issue is worth the effort. I was referring to an issue that doesn't appear to be here....I will make it.... |
this issue is already finished. please close. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
following @CJ-Wright it would be even more helpful if we could have some PDF data in an examples folder or sthg and some code that can build a few standard/common PDF plots so that we can see what they look like. Something like nanoarticle data, nanoparticle calculated and offset difference curve, maybe with a label (a) and axis labels etc.
The style needed when we embed in a pdf document is a bit different from when we do a live plot too, so we may need two style sheets. That can come in a later PR
The text was updated successfully, but these errors were encountered: