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

Release v0.6.0 #106

Merged
merged 17 commits into from
Dec 8, 2023
Merged

Release v0.6.0 #106

merged 17 commits into from
Dec 8, 2023

Conversation

ConnorStoneAstro
Copy link
Member

Making PR now so we can track merge compatability

ConnorStoneAstro and others added 8 commits December 1, 2023 12:16
…rary Development Guide (#103)

* feat: Setup packaging and Github Codespaces dev environment (#26)

* Update README.md

Add SSEC badge

* Move project to src/ directory

* Update Codespaces link

* Update README.md

* Update README.md

Update Codespaces link

* Update devcontainer

* ci: Create cd.yml for building and pushing package

* Update caustic name to caustics

* Add PyLint and Black formatting features to devcon

* Closes issue uw-ssec#16

* Updated test/ -> tests/

* Move project to src/ directory

* Update README.md

* Update README.md

Update Codespaces link

* Create noxfile.py

* Add .readthedocs.yaml

* Create .git_archival.txt

* Create .gitattributes

* Create .pre-commit-config.yaml

* Add sphinx and myst-parser support

* style: pre-commit fixes

* Update docs

* style: pre-commit fixes

* Update .github Actions

* Update environment.yml
- Add jupyter-book

* Create CONTRIBUTING.md

* Update README.md

* chore: resolve conflicts

* chore: Remove sources

* test: remove test lenses

* test: fix test_simulator

* chore: Set min python to 3.9

* fix: change pytest to point to tests/

* build: update dep list to use reqs.txt

* ci: Don't fail fast

* chore(deps): Pin astropy < 6

* fix: change instances of caustic to caustics

* fix: Perform pre-commit

* fix: update path to conf.py

* update README

* style: pre-commit fixes

* docs: Add contributor graph

* chore: Setup dynamic versioning

* ci: Update fetch depth to 0 to get correct version

* Update README.md

Update Codespaces button to new repo name

---------

Co-authored-by: Cordero Core <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* build: remove dir sources (#27)

* fix: Add project urls to empty slots

* ci: update yaml configuration for CI (#28)

* ci: update yaml configuration for CI

* style: pre-commit fixes

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Update README.md

Added issue to contribution section

* Update conf.py

Co-authored-by: Connor Stone, PhD <[email protected]>

* Update pixelated_convergence.py

Co-authored-by: Connor Stone, PhD <[email protected]>

* Update test_parametrized.py

Co-authored-by: Connor Stone, PhD <[email protected]>

* Update test_parametrized.py

Co-authored-by: Connor Stone, PhD <[email protected]>

* Update cd.yml

* Update cd.yml

* Update cd.yml

* Update README.md

* docs: Grab version from package __version__

* Update README.md

* Add nox dependency

* Update noxfile.py

* Update CONTRIBUTING.md

Temporarily removing instructions for `docs`

* Remove `docs` instructions (temp)

* Add tutorials and update nox instructions

* Update `nox` instructions in CONTRIBUTING.md (#3)

* Updated local nox instructions

* Remove tutorials

* Update nox install instruction and noxfile

---------

Co-authored-by: Cordero Core <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Connor Stone, PhD <[email protected]>
Copy link

codecov bot commented Dec 1, 2023

Codecov Report

Attention: Patch coverage is 90.62241% with 113 lines in your changes missing coverage. Please review.

Project coverage is 88.70%. Comparing base (c8d51f7) to head (dfd7a28).
Report is 164 commits behind head on main.

Files with missing lines Patch % Lines
src/caustics/lenses/base.py 81.81% 34 Missing ⚠️
src/caustics/utils.py 81.92% 32 Missing ⚠️
src/caustics/lenses/singleplane.py 41.66% 21 Missing ⚠️
src/caustics/sims/lens_source.py 87.01% 10 Missing ⚠️
src/caustics/cosmology.py 91.56% 7 Missing ⚠️
src/caustics/parametrized.py 85.71% 3 Missing ⚠️
src/caustics/lenses/external_shear.py 96.55% 1 Missing ⚠️
src/caustics/lenses/nfw.py 99.06% 1 Missing ⚠️
src/caustics/lenses/tnfw.py 99.13% 1 Missing ⚠️
src/caustics/lenses/utils.py 0.00% 1 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   88.37%   88.70%   +0.32%     
==========================================
  Files          35       36       +1     
  Lines        1772     1885     +113     
==========================================
+ Hits         1566     1672     +106     
- Misses        206      213       +7     

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

@lsetiawan lsetiawan changed the title Dev Release v0.6.0 Dec 1, 2023
The link to the Scientific Python guide changed
Update Scientific Python link
The tests include a simulator run on a lens and source model. As well as some Jacobian functionality which ensures differentiability is working
Remove link to incorrect project from eScience. Update test badge to CI and Docs badge to readthedocs.
ConnorStoneAstro and others added 4 commits December 8, 2023 11:39
- Adding gaussian quadrature integrator in utils
- Using quad integration for lens_source built-in simulator
- Adding test for quad integration
This is a PR from uw-ssec#25.

This improved the documentation to use Jupyter Book instead of vanilla
sphinx. Google docstring style has been updated to Numpy docstring style.

---

Ref: uw-ssec#11
Original author: https://github.com/sophietao127

- docs: Add tutorials
- Added index.md to tutorials
- Updated _config.yml and _toc.yml
- docs: Lint docs and update notebooks

---------

Co-authored-by: Landung 'Don' Setiawan <[email protected]>
Copy link
Collaborator

@lsetiawan lsetiawan left a comment

Choose a reason for hiding this comment

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

LGTM

@ConnorStoneAstro ConnorStoneAstro merged commit e032bd9 into main Dec 8, 2023
24 of 25 checks passed
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.

4 participants