Skip to content

add logger#101

Merged
arjunsavel merged 3 commits intomainfrom
warnings_for_nulls
Mar 3, 2025
Merged

add logger#101
arjunsavel merged 3 commits intomainfrom
warnings_for_nulls

Conversation

@arjunsavel
Copy link
Owner

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 3, 2025

Codecov Report

Attention: Patch coverage is 29.06977% with 61 lines in your changes missing coverage. Please review.

Project coverage is 69.75%. Comparing base (c0854dd) to head (8f24064).

Files with missing lines Patch % Lines
src/scope/input_output.py 11.11% 40 Missing ⚠️
src/scope/logger.py 46.43% 15 Missing ⚠️
src/scope/run_simulation.py 53.85% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   70.63%   69.75%   -0.88%     
==========================================
  Files          13       14       +1     
  Lines         984     1028      +44     
==========================================
+ Hits          695      717      +22     
- Misses        289      311      +22     
Flag Coverage Δ
unittests 69.75% <29.07%> (-0.88%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

This was referenced Mar 3, 2025
@arjunsavel arjunsavel merged commit 8f80fd5 into main Mar 3, 2025
3 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.

2 participants