Closed prjemian closed 2 months ago
A built ZIP file of the docs are uploaded as an artifact: https://github.com/bluesky/hklpy/actions/runs/8890336016
Here's a screen view:
Ready for review, no real changes to content, just presentation, aimed at making it easier to navigate.
@ambarb - Comments?
@padraic-shafer - Thanks for the feedback. I agree there is still too much information on the home page. Rather than adding sections, I'd like to keep the page at a very high level. I'll build your phrasing into the overview text, while keeping it oriented to the needs of the crystallographer who is not familiar with package names and libraries. The intent, eventually, is to expand from the Hkl library to other computational backends.
home page (some cards have been moved to new User Guide page)
home page (some cards have been moved to new User Guide page)
I find that much easier to digest. Nicely done!
@rodolakis has been guiding me to say the same thing (in documentation) with fewer words. Applying that here.
Newly-created User Guide gathers some of content pulled from home page, and groups as @padraic-shafer suggested:
Install page completely revamped. Most other pages left alone.
Overhaul of appearance, not substance.
Reviewers, get ZIP file from https://github.com/bluesky/hklpy/actions/runs/8901753697
@prjemian the new layout/design looks great. I spent more time exploring and finding the spec-to-bluesky was not something I had seen before. I imagine this is part of your objective.
unrelated to this review, I have two comment for consideration:
d_object.move((h, k, l))
description that people should not use that within a RE script. they should use bps.mv()
@prjemian the new layout/design looks great. I spent more time exploring and finding the spec-to-bluesky was not something I had seen before. I imagine this is part of your objective.
:100:
unrelated to this review, I have two comment for consideration:
- maybe it would be good to add in the
d_object.move((h, k, l))
description that people should not use that within a RE script. they should usebps.mv()
Good point. I'll add this.
- does reflex work like fourc's reflex (in that the lattice parameters are refined )?
Have not seen lattice parameter refinement in the libhkl code. I'll look for it.
@ambarb - Found it. SPEC's reflex
is affine
in libhkl. See https://blueskyproject.io/hklpy/sample.html#hkl.sample.HklSample.affine
The only advice I have now is to consult the libhkl documentation. Start with section 1.5.2:
It is also possible to calculate U without the knowledge of crystalline parameters. It is then necessary to refine the parameters. This is the same as minimizing a function. We will use the simplex method to find this minimum and therefore adjust the set of crystal parameters as well as the orientation matrix.
Also section 1.5.4. That part of the docs are in French.
To help with review, I pushed the new docs to the web site: https://blueskyproject.io/hklpy/
(It's easy for me to push the old version to the web site. We're not ready, yet, to support multiple versions of the docs on the web site. Probably that will come sometime this year.)
This is great, I like the way it progressed and the landing page developed above. I followed all links, and nothing is dead or misdirecting.
Only 1 comment: The release history confuses me. I think it makes sense to have place holders in the comments, so that changes don't get lost. But History probably shouldn't include the future.
History probably shouldn't include the future
I tend to agree. I see value in documenting changes in the repo in public, for those who work out of the main branch. But for now, all future changes are noted as comments.
I'm hearing consensus overall to these changes but no one has approved. I plan to merge tomorrow if there is nothing else to be changed.