Skip to content

Fix ICOHPDistance plotter #401

Merged
naik-aakash merged 1 commit intomainfrom
fix_icohp_distance_plotter
Jun 13, 2025
Merged

Fix ICOHPDistance plotter #401
naik-aakash merged 1 commit intomainfrom
fix_icohp_distance_plotter

Conversation

@naik-aakash
Copy link
Copy Markdown
Collaborator

@naik-aakash naik-aakash commented Jun 13, 2025

Incase on Non orbital resolved ICOHPLIST / ICOOPLIST / ICOBILIST files, icohp distance plotter failed with TypeError as dict is None in such case.

@naik-aakash naik-aakash added the bug Something isn't working label Jun 13, 2025
@naik-aakash naik-aakash requested a review from JaGeo June 13, 2025 09:44
@naik-aakash naik-aakash marked this pull request as draft June 13, 2025 09:44
@naik-aakash naik-aakash marked this pull request as ready for review June 13, 2025 09:47
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.27%. Comparing base (5386931) to head (326be0d).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #401   +/-   ##
=======================================
  Coverage   97.27%   97.27%           
=======================================
  Files          10       10           
  Lines        3123     3124    +1     
=======================================
+ Hits         3038     3039    +1     
  Misses         85       85           

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

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

@naik-aakash naik-aakash merged commit 81c0f39 into main Jun 13, 2025
23 checks passed
@naik-aakash naik-aakash deleted the fix_icohp_distance_plotter branch December 20, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants