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

Add Feature: Documentation Page for XAS Plugin #614

Merged
merged 7 commits into from
Feb 12, 2024

Conversation

PNOGillespie
Copy link
Contributor

Overview

This PR adds a complete documentation set for using the XAS plugin of AiiDALab-QE, covering both calculation setup and post-processing of the final spectra using the Plotly-derived widget provided with the plugin. The instructions should adequately cover all features of the XAS App using a self-contained, robust, and well-tested example.

This will be followed up by a video demonstration of this example, as required for BIG-MAP project key deliverables, to accompany the documentation.

Note that the documentation example uses Li2CO3 as the structure, which has been added to the "From Examples" list in src/aiidalab_qe/app/static/, so there shouldn't be any issues in the future with missing links to important files.

Adds a full example for performing XAS calculations of Li2CO3 using
the new XAS plugin. This covers workflow setup, with helpful suggestions
to improve performance, as well as how to effectively utilise the
post-processing tools provided.
Applies various improvements to the text of the documentation to improve
readability and clarify certain points
Copy link

codecov bot commented Feb 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (98ebc43) 77.48% compared to head (8f67e6b) 77.48%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #614   +/-   ##
=======================================
  Coverage   77.48%   77.48%           
=======================================
  Files          55       55           
  Lines        4011     4011           
=======================================
  Hits         3108     3108           
  Misses        903      903           
Flag Coverage Δ
python-3.10 77.48% <ø> (ø)
python-3.8 77.51% <ø> (ø)
python-3.9 77.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@superstar54 superstar54 self-requested a review February 1, 2024 11:43
Reformats the XAS documentation source code such that each sentence is
on a separate line (where possible)
@PNOGillespie
Copy link
Contributor Author

Hi @superstar54, I reformatted the source code as you asked so that each sentence is on a separate line. Obviously this wasn't possible for lists, because the syntax doesn't let you do that, but hopefully that makes review easier.

Adds a note at the start of the guide specifying which version of the QE
App must be installed to access the XAS plugin, referring to the
relevant How-To guide on installing/upgrading the App for instructions
and specifying that the plugin is currently in a prerelease version.
@PNOGillespie PNOGillespie marked this pull request as ready for review February 1, 2024 15:44
Copy link
Member

@superstar54 superstar54 left a comment

Choose a reason for hiding this comment

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

Nice docs! Very clear and rigorous. I only has one small request, please see the comment.

Comment on lines 22 to 26
If installing `via` the terminal, simply give the version number using the correct `PEP 440 version specifier`_\ :

.. code-block:: console

$ aiidalab install quantum-espresso>=v24.04.0a1
Copy link
Member

Choose a reason for hiding this comment

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

There seems to be a bug in the aiidalab install. This command does not work as expected. I would suggest removing this part. We try to avoid the user using the command line if they can do it in the GUI.

Removes the CLI installation instructions from the XAS documentation due
to known issues with the `aiidalab install` command
@PNOGillespie
Copy link
Contributor Author

Ok @superstar54, I've removed the CLI installation instructions. Hopefully this is now good-to-go.

Just out of curiosity (and also in case this needs to be documented properly), is there already an issue page for the problem with aiidalab install?

@superstar54
Copy link
Member

superstar54 commented Feb 12, 2024

Just out of curiosity (and also in case this needs to be documented properly), is there already an issue page for the problem with aiidalab install?

In case of ">=", one needs to include " or '

- aiidalab install quantum-espresso>=v24.04.0a1
+ aiidalab install "quantum-espresso>=v24.04.0a1"

@superstar54 superstar54 self-requested a review February 12, 2024 14:02
@superstar54 superstar54 merged commit 983845a into aiidalab:main Feb 12, 2024
14 checks passed
@PNOGillespie PNOGillespie deleted the feature/docs/xas branch February 12, 2024 14:06
PNOGillespie added a commit to PNOGillespie/aiidalab-qe that referenced this pull request Feb 14, 2024
Fixes an oversight from aiidalab#614
where the lithium carbonate example was not added to `Examples` in `__init__.py`
and was thus not selectable as a structure.
superstar54 pushed a commit that referenced this pull request Feb 14, 2024
Fixes an oversight from #614
where the lithium carbonate example was not added to `Examples` in `__init__.py`
and was thus not selectable as a structure.
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