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.1 #125

Merged
merged 18 commits into from
Jan 4, 2024
Merged

Release v0.6.1 #125

merged 18 commits into from
Jan 4, 2024

Conversation

ConnorStoneAstro
Copy link
Member

No description provided.

lsetiawan and others added 4 commits December 8, 2023 14:26
* Attempt to add light/dark logo for website

* Remove big logo from main page
* style: Add codespace whitelist and perform spell check (#34)

* style: Update line length config and fix E501 errors (#35)

* style: Fix eof line .codespell-whitelist

* style: Remove * imports, fixes F403 PEP violation (#36)

* style: Expand imports for data module

* style: Fix undefined variables (F821), etc. (#37)

* style: Update pre-commit hooks and last lintings (#38)

* style: Revert Doubt to Dout (#39)

* style: Lint fix _config.yml
Copy link

codecov bot commented Dec 12, 2023

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (be58a9a) 88.70% compared to head (a0e76f6) 89.51%.

Files Patch % Lines
src/caustics/data/__init__.py 0.00% 4 Missing ⚠️
src/caustics/lenses/base.py 94.33% 3 Missing ⚠️
src/caustics/lenses/multiplane.py 95.45% 2 Missing ⚠️
src/caustics/lenses/utils.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   88.70%   89.51%   +0.81%     
==========================================
  Files          36       36              
  Lines        1885     1955      +70     
==========================================
+ Hits         1672     1750      +78     
+ Misses        213      205       -8     

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

@lsetiawan lsetiawan changed the title Merge dev to main Release v0.6.1 Dec 12, 2023
@lsetiawan
Copy link
Collaborator

I think there are a few more documentation related work about black formatting and stuff that @uwcdc is currently working on, so let's hold off in releasing this clean up until he's done with that. He can chime in on what that entails. Thanks!

ConnorStoneAstro and others added 3 commits December 13, 2023 11:30
* Mark lenses equations to skip formatting

* Mark fmt skip for rest of caustics

* Fix super long equations

* bug test_kappa swapped z_l and z_s

* Pseudojaffe potential in units of arcsec maybe arcsec squared

---------

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

uwcdc commented Dec 13, 2023

I think there are a few more documentation related work about black formatting and stuff that @uwcdc is currently working on, so let's hold off in releasing this clean up until he's done with that. He can chime in on what that entails. Thanks!

I agree with @lsetiawan on delaying the release until I'm finished because I'm working on some awesome docs-related features that should be included in this release...

  • Implementing units (for docs only) and datatype validation
  • Fixing the mypy-related errors
  • Docstring equation templates

ConnorStoneAstro and others added 11 commits December 19, 2023 14:36
* Add multiplane time delay

* include shapiro/geometric options for both ThinLens and multiplane time delay

* Add docstring information. Add tests that time delays are real

* Tidy up time delay function names, and docstrings

* Merge time delay functions. Address citations/comments by @AlexandreAdam

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* chore(devcontainer): Fix codespaces configuration (#43)

* fix: Updated devcontainer to simple CPU only (#50)

* fix: Updated devcontainer to have options

* fix: Remove cuda option for now... doesn't really work

* Update .devcontainer/cpu/devcontainer.json

Removed args from old image configuration.
…re with kwargs (#136)

* New version of unpacked decorator, handles kwargs better

* Convert all lenses to new function signature

* fix: unit tests now run successfully

* fix: update tutorials to new params system

* test: add test for new unpack method

* docs: Add docstring for unpack to explain valid calls

* docs: Add tutorial for parameters

* Make `pack` fully internal when calling function decorated with `unpack`

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add minimal lensing example, update basic intro tutorial

* Add minimal example to home page of website

* Update minimal example to include vmap and jacobian

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…angle, convergence (#138)

* feat: add _null_params for basic lens evaluation

* test: add test for consistency between lens potential and deflection angle

* test: add potential to convergence test

* fix: lower tolerance for NFW test

* TNFW potential now correct

* PseudoJaffe now passes tests

* PixelatedConvergence now kind of works with low tolerance
@ConnorStoneAstro ConnorStoneAstro merged commit 7c342ee into main Jan 4, 2024
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