Skip to content

Point plus extended - #95

Merged
nkhadka21 merged 54 commits into
LSST-strong-lensing:mainfrom
nkhadka21:point_plus_extended
Nov 27, 2023
Merged

nkhadka21 merged 54 commits into
LSST-strong-lensing:mainfrom
nkhadka21:point_plus_extended

Conversation

@nkhadka21

Copy link
Copy Markdown
Collaborator

added a point + extended source skeleton.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov

codecov Bot commented Nov 21, 2023 •

Copy link
Copy Markdown

Codecov Report

Merging #95 (f3221a9) into main (d10bb5e) will increase coverage by 0.35%.
The diff coverage is 94.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
+ Coverage   77.65%   78.00%   +0.35%     
==========================================
  Files          27       29       +2     
  Lines        1432     1473      +41     
==========================================
+ Hits         1112     1149      +37     
- Misses        320      324       +4     
Files Coverage Δ
...lsim/Sources/QuasarCatalog/quasar_plus_galaxies.py 100.00% <100.00%> (ø)
slsim/Sources/QuasarCatalog/simple_quasar.py 100.00% <ø> (ø)
slsim/Sources/point_plus_extended_source.py 100.00% <100.00%> (ø)
slsim/Util/param_util.py 97.72% <ø> (ø)
slsim/image_simulation.py 100.00% <100.00%> (+0.66%) ⬆️
slsim/lens_pop.py 96.62% <100.00%> (+0.33%) ⬆️
slsim/Sources/source.py 90.69% <90.00%> (-0.97%) ⬇️
slsim/lens.py 92.08% <85.71%> (ø)
slsim/lsst_science_pipeline.py 14.74% <42.85%> (+0.40%) ⬆️

@nkhadka21

Copy link
Copy Markdown
Collaborator Author

Hi @sibirrer, I resolved all your comments!

@nkhadka21
nkhadka21 requested a review from sibirrer November 24, 2023 22:41

@sibirrer sibirrer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, thank you!

@nkhadka21
nkhadka21 merged commit ab33a21 into LSST-strong-lensing:main Nov 27, 2023
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