Skip to content

Commit

Permalink
Merge pull request #47 from WMD-group/pre_joss
Browse files Browse the repository at this point in the history
minor doc edits pre-joss
  • Loading branch information
savya10 authored Jul 20, 2023
2 parents 86b2578 + 7f0c17f commit e952003
Show file tree
Hide file tree
Showing 7 changed files with 61 additions and 25 deletions.
12 changes: 8 additions & 4 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
@@ -1,25 +1,29 @@
Change log
==========

v2.0.1
~~~~~~
- Minor documentation and website updates.

v2.0.0
------
~~~~~~
- Reduce the sum over band transitions to the energy mesh (min(-6, -energy_max), max(6, energy_max)) to
make parsing of VASP optics more efficient.
- Switch to semantic versioning.

v23.6.3
-------
~~~~~~~
- Remake PyTASER MP example workflow
- Update PyPI releasing workflow
- Update minor MPRester API issue

v23.6.2
------
~~~~~~~
- Add multiprocessing functionality to dramatically speed up parsing of large kpoint sets / large numbers of
electronic bands, and add example notebook and HPC script for this.

v23.6.1
------
~~~~~~~
- Add functionality to parse orbital derivatives from `VASP` optics calculations, to incorporate oscillator strengths
in the prediction of TAS spectra.
- Add functionality to save and load `Tas` objects to `json` file, to save time on re-analysing later on.
Expand Down
25 changes: 17 additions & 8 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,16 +26,16 @@ experiments. The main features include:
To install the module with pip (recommended):

```
pip install --user pytaser
pip install pytaser
```
To install directly from the git repository:
```
pip install --user git+https://github.com/WMD-group/PyTASER
pip install git+https://github.com/WMD-group/PyTASER
```
To do a manual build and installation:
```
python3 setup.py build
python3 setup.py install --user
python3 setup.py install
```

PyTASER is compatible with Python 3.9+ and relies on a number of open-source packages, specifically:
Expand Down Expand Up @@ -70,14 +70,23 @@ Alternatively, you can set up a file in Python to run it in the command line of
python3 <filename.py>
```

If using your own, locally calculated data, please follow [this](https://pytaser.readthedocs.io/en/latest/dft_examples.html) workflow.
Otherwise, use the MP-integrated workflow [here](https://pytaser.readthedocs.io/en/latest/jdos_examples.html).
If using your own, locally calculated data, please follow [this](https://github.com/WMD-group/PyTASER/blob/main/examples/PyTASER_DFT_Example.ipynb) workflow.
Otherwise, use the MP-integrated workflow [here](https://github.com/WMD-group/PyTASER/blob/main/examples/PyTASER_MP_Example.ipynb).

# Contributing

We appreciate any contributions in the form of a pull request.
Additional test cases/example spectra performed with PyTASER would be welcomed.
Please feel free to reach out to us if there are any questions or suggestions.
We appreciate any contributions in the form of a pull request. Please see the [Contributing](https://pytaser.readthedocs.io/en/latest/contributing.html) documentation for more details.
Additional analysis/example spectra performed with `PyTASER` would be welcomed.

Please feel free to reach out to us via the [Issue Tracker](https://github.com/WMD-group/PyTASER/issues) if there are any questions or suggestions.

# Testing

Unit tests are in the `tests` directory and can be run from the top directory using pytest. Automatic testing is run on the master and develop branches using GH Actions.

Please run tests and add new tests for any new features whenever submitting pull requests.

# Future Work

Future topics we plan to build on:

Expand Down
4 changes: 2 additions & 2 deletions docs/source/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
author = "Savyasanchi Aggarwal"

# The full version, including alpha/beta/rc tags
release = "1.0.0"
release = "2.0.0"

# -- General configuration ---------------------------------------------------

Expand Down Expand Up @@ -70,7 +70,7 @@
"home_page_in_toc": True,
}

html_logo = "_static/docs/PyTASER.png"
html_logo = "_static/PyTASER.png"
html_title = "PyTASER"

# If true, SmartyPants will be used to convert quotes and dashes to
Expand Down
4 changes: 2 additions & 2 deletions docs/source/contributing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,8 @@ How to Make a **Great** Pull Request

We have a few tips for writing good PRs that are accepted into the main repo:

- Use the Google Code style for all of your code. Find an example `here
<https://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html>`__.
- Use the Black Code style for all of your code. Find an example `here
<https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html>`__.
- Your code should have (4) spaces instead of tabs.
- If needed, update the documentation.
- **Write tests** for new features! Good tests are 100%, absolutely necessary
Expand Down
37 changes: 30 additions & 7 deletions docs/source/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
contain the root `toctree` directive.
Welcome to PyTASER's documentation!
===================================

Expand All @@ -25,6 +26,7 @@ Welcome to PyTASER's documentation!
.. image:: https://img.shields.io/badge/code%20style-black-000000.svg
:target: https://img.shields.io/badge/code%20style-black-000000.svg


============
Introduction
============
Expand All @@ -39,13 +41,15 @@ experimental spectra. The main features include:
* Consideration of non-magnetic and magnetic materials
* Capability to input calculated bandstructures and density of states inputs with support for https://materialsproject.org.

==========
Background
===========
==========

TAS is a powerful pump-probe tool to characterise the excited states of materials. It can be used to understand microscopic processes in photochemical and electrochemical transformations, including phenomena such as electron trapping and carrier recombination.

The drawback is that TAS spectra are difficult to interpret, especially for crystals where the specific valence and conduction band structure can give rise to complex features. Our goal here is to predict TAS features from first-principles starting from the most simple models of static excitations through to the kinetics of relaxation of the excited state back to the ground state.

============
Installation
============

Expand All @@ -68,6 +72,7 @@ To do a manual build and installation:
python3 setup.py build
python3 setup.py install
===================================
Developer’s installation (optional)
===================================

Expand All @@ -93,6 +98,7 @@ Install the code with the command:
This command tries to obtain the required packages and their dependencies and install them automatically.

============
Dependencies
============

Expand All @@ -102,6 +108,7 @@ Dependencies
* `numpy <https://numpy.org/>`__, `scipy <https://scipy.org/>`__ for data structures and unit conversion
* `matplotlib <https://matplotlib.org/>`__ for plotting the spectra

=============
Visualisation
=============

Expand All @@ -112,25 +119,41 @@ Alternatively, you can setup a ``Python`` file to run it in the command line of
python3 <filename.py>
If using your own, locally calculated data, please follow `this workflow <https://github.com/WMD-group/PyTASER/blob/main/examples/PyTASER_DFT_Example.ipynb>`__. Otherwise, please use the MP-integrated workflow as shown `here <https://github.com/WMD-group/PyTASER/blob/main/examples/PyTASER_MP_Example.ipynb>`__.

============
Contributing
============

We appreciate any contributions in the form of a pull request.
Additional test cases/example spectra performed with ``PyTASER`` would be welcomed.
Please feel free to reach out to us if there are any questions or suggestions.
We appreciate any contributions in the form of a pull request. Please see the `Contributing <https://pytaser.readthedocs.io/en/latest/contributing.html>`__ Documentation for more details.
Additional analysis/example spectra performed with ``PyTASER`` would be welcomed.

Please feel free to reach out to us via the `Issue Tracker <https://github.com/WMD-group/PyTASER/issues>`__ if there are any questions or suggestions.

=======
Testing
=======

Unit tests are in the ``tests`` directory and can be run from the top directory using pytest. Automatic testing is run on the master and develop branches using Github Actions.

Please run tests and add new tests for any new features whenever submitting pull requests.

===========
Future Work
===========

Future topics we plan to build on:

* Incorporating finite-temperature effects (particularly for indirect bandgaps)
* Description of more complex optical processes (e.g. stimulated emission)
* Direct treatment of pump-probe time delay
* Direct treatment of pump-probe time delay and relaxation kinetics
* Incorporating spin-flip processes for spin-polarised systems
* Description of defective crystals

================
Acknowledgements
================

Developed by Savyasanchi Aggarwal and Alex Ganose. Aron Walsh helped to design the project. Thanks to group members for their support, especially Seán Kavanagh, Youngwon Woo, Anahita Manchala and Liam Harnett-Caulfield.
Developed by Savyasanchi Aggarwal, Seán Kavanagh, Youngwon Woo and Alex Ganose. Aron Walsh designed and led the project. Thanks to group members for their support, especially Anahita Manchala and Liam Harnett-Caulfield.


.. toctree::
Expand Down
2 changes: 1 addition & 1 deletion examples/PyTASER_MP_Example.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@
"id": "e859d8ac-ed7c-4346-81cb-2dafe03db618",
"metadata": {},
"source": [
"# GaAs"
"## GaAs"
]
},
{
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

setup(
name="pytaser",
version="2.0.0",
version="2.0.1",
description="TAS prediction tool",
url="https://pytaser.readthedocs.io/en/latest/",
author="Savyasanchi Aggarwal",
Expand Down

0 comments on commit e952003

Please sign in to comment.