Skip to content

Conversation

ldups
Copy link
Contributor

@ldups ldups commented Aug 13, 2025

This PR adds graph wrappers for more of the graph styles included with STK, building off of the previous PRs that added wrappers for a few of the access styles.

@ldups ldups requested a review from a team as a code owner August 13, 2025 20:19
@github-actions github-actions bot added enhancement General improvements to existing features tests Related with improvements of the test suite docs:api Related to API documentation labels Aug 13, 2025
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2025

Codecov Report

❌ Patch coverage is 92.50535% with 70 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.70%. Comparing base (4fac034) to head (6695c18).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
.../extensions/data_analysis/graphs/_common_graphs.py 91.17% 18 Missing ⚠️
...extensions/data_analysis/graphs/aircraft_graphs.py 69.04% 13 Missing ⚠️
...tk/extensions/data_analysis/graphs/chain_graphs.py 74.46% 12 Missing ⚠️
...xtensions/data_analysis/graphs/satellite_graphs.py 87.14% 9 Missing ⚠️
...k/extensions/data_analysis/graphs/sensor_graphs.py 72.72% 6 Missing ⚠️
...ions/data_analysis/graphs/ground_vehicle_graphs.py 88.88% 3 Missing ⚠️
...ions/data_analysis/graphs/launch_vehicle_graphs.py 94.59% 2 Missing ⚠️
.../extensions/data_analysis/graphs/missile_graphs.py 95.74% 2 Missing ⚠️
...extensions/data_analysis/graphs/facility_graphs.py 94.73% 1 Missing ⚠️
...k/extensions/data_analysis/graphs/graph_helpers.py 97.14% 0 Missing and 1 partial ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #794      +/-   ##
==========================================
+ Coverage   89.69%   89.70%   +0.01%     
==========================================
  Files          47       67      +20     
  Lines      187134   188022     +888     
  Branches     1335     1339       +4     
==========================================
+ Hits       167842   168664     +822     
- Misses      18897    18965      +68     
+ Partials      395      393       -2     

☔ 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.

Copy link
Member

@jorgepiloto jorgepiloto left a comment

Choose a reason for hiding this comment

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

Thanks for adding this big feature, @ldups. It is very handy to have all these graphs available for each STK object.

@jorgepiloto jorgepiloto merged commit 3682ad5 into main Aug 29, 2025
58 checks passed
@jorgepiloto jorgepiloto deleted the feat/graph-wrappers branch August 29, 2025 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs:api Related to API documentation enhancement General improvements to existing features tests Related with improvements of the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants