Skip to content

Commit

Permalink
Fix PyPI README (#136)
Browse files Browse the repository at this point in the history
  • Loading branch information
paddyroddy authored Apr 5, 2023
1 parent 91a6153 commit 1db25f0
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
- name: Build distributions
run: python -m build

- name: Publish package to PyPI
- name: Publish package to TestPyPI
uses: pypa/gh-action-pypi-publish@release/v1
with:
password: ${{ secrets.TEST_PYPI_API_TOKEN }}
Expand Down
16 changes: 8 additions & 8 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ meshes using `plotly`.
## Installation

The recommended way to install `SLEPLET` is via
[pip](https://pypi.org/project/pip/)
[pip](https://pypi.org/project/pip)

```sh
pip install sleplet
Expand All @@ -33,7 +33,7 @@ pip install -e .

This will install two scripts `sphere` and `mesh` which can be used to generate
the figures in
[the associated papers](https://astro-informatics.github.io/sleplet/#paper-figures).
[the associated papers](https://astro-informatics.github.io/sleplet#paper-figures).

### Supported Platforms

Expand All @@ -43,9 +43,9 @@ Windows is not currently supported as `SLEPLET` relies on
[pyssht](https://pypi.org/project/pyssht) and
[pys2let](https://pypi.org/project/pys2let) which do not work on Windows.
These can hopefully be replaced with
[s2fft](https://github.com/astro-informatics/s2fft/) and
[s2fft](https://github.com/astro-informatics/s2fft) and
[s2wav](https://github.com/astro-informatics/s2wav) in the future when they
are available on [PyPI](https://pypi.org/).
are available on [PyPI](https://pypi.org).

## Example Usage

Expand Down Expand Up @@ -73,7 +73,7 @@ sleplet.plotting.PlotSphere(
).execute()
```

![Slepian Wavelet j=2](./documentation/slepian_wavelets_south_america_3B_2jmin_2j_L128_res512_real.png)
![Slepian Wavelet j=2](https://github.com/astro-informatics/sleplet/blob/main/documentation/slepian_wavelets_south_america_3B_2jmin_2j_L128_res512_real.png)

### CLI Usage

Expand All @@ -84,19 +84,19 @@ region of a Homer Simpson mesh for a per-vertex normals field.
mesh homer -e 3 2 0 -m slepian_wavelet_coefficients -u -z
```

![Slepian Mesh Wavelet Coefficients j=2](./documentation/slepian_wavelet_coefficients_homer_3B_2jmin_2j_zoom.png)
![Slepian Mesh Wavelet Coefficients j=2](https://github.com/astro-informatics/sleplet/blob/main/documentation/slepian_wavelet_coefficients_homer_3B_2jmin_2j_zoom.png)

## Documentation

See here for the [documentation](https://astro-informatics.github.io/sleplet).
This includes demonstrations of the figures from the associated papers along
with the API documentation. Further examples are included in the
[examples folder](./examples).
[examples folder](https://github.com/astro-informatics/sleplet/tree/main/examples).

## Community Guidelines

We'd love any contributions you may have, please see the
[contributing guidelines](./CONTRIBUTING.md).
[contributing guidelines](https://github.com/astro-informatics/sleplet/blob/main/CONTRIBUTING.md).

## Citing

Expand Down
4 changes: 1 addition & 3 deletions documentation/DOCUMENTATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,7 @@ such it is recommended to stick to the powers of two up to `L=128`.
When it comes to selecting a Slepian region the order precedence is
[polar cap region](https://doi.org/10.1111/j.1365-246X.2006.03065.x) >
[limited latitude longitude region](https://doi.org/10.1109/TSP.2016.2646668) >
arbitrary region,
[as seen in the code](https://github.com/astro-informatics/sleplet/blob/main/src/sleplet/utils/region.py).
The default region is the `south_america` arbitrary region.
arbitrary region. The default region is the `south_america` arbitrary region.

- `POLAR_GAP`: for a Slepian `polar cap region`, when set in conjunction with
`THETA_MAX` but without the other `PHI`/`THETA` variables
Expand Down

0 comments on commit 1db25f0

Please sign in to comment.