Skip to content

add notebook generating 3D plots comparing SLSim lenses and SLACS Dinos lenses#354

Open
mia-lamontagne wants to merge 2 commits intoLSST-strong-lensing:mainfrom
mia-lamontagne:3D_SLSim_vs_SLACS_Dinos
Open

add notebook generating 3D plots comparing SLSim lenses and SLACS Dinos lenses#354
mia-lamontagne wants to merge 2 commits intoLSST-strong-lensing:mainfrom
mia-lamontagne:3D_SLSim_vs_SLACS_Dinos

Conversation

@mia-lamontagne
Copy link
Copy Markdown
Contributor

This notebook compares various parameters of simulated SLSim lenses to real observed SLACS Dinos lenses, and displays them as corner plots as well as 3D plots for enhanced visualization.

@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
Copy link
Copy Markdown

codecov Bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.28%. Comparing base (2ab9a2a) to head (8a9c77a).
⚠️ Report is 191 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #354      +/-   ##
==========================================
+ Coverage   97.93%   98.28%   +0.34%     
==========================================
  Files         154       98      -56     
  Lines       10479     6420    -4059     
==========================================
- Hits        10263     6310    -3953     
+ Misses        216      110     -106     

see 139 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nkhadka21
Copy link
Copy Markdown
Collaborator

Hi @mia-lamontagne , thank you very much for this PR! It is failing pre-commit. It would be great if you could fix them.

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