Skip to content

Add Makie extension#122

Merged
jellepoland merged 35 commits intomainfrom
makie
Oct 19, 2025
Merged

Add Makie extension#122
jellepoland merged 35 commits intomainfrom
makie

Conversation

@1-Bart-1
Copy link
Member

@1-Bart-1 1-Bart-1 commented Sep 28, 2025

Summary

• Add comprehensive Makie plotting extension for 3D visualization of kite systems
• Implement real-time 3D visualization example with interactive camera controls
• Significantly improve developer documentation and examples
• Remove outdated default manifests and update CI workflows
• Add developer mode features for enhanced debugging capabilities

Changes

• New Features:
• Makie extension (ext/SymbolicAWEModelsMakieExt.jl) with 3D plotting capabilities
• Real-time visualization example (examples/realtime_visualization.jl)
• Developer mode with enhanced logging and debugging tools
• Documentation:
• Major improvements to developer documentation
• Added comprehensive examples and tutorials
• Enhanced getting started guide
• Infrastructure:
• Updated CI/CD workflows for dynamic Julia version support
• Removed outdated manifest files
• Improved package management and dependencies
• Bug Fixes:
• Various plotting and visualization fixes
• Improved zoom and camera controls
• Enhanced VSM (Vortex Step Method) integration

This PR introduces powerful 3D visualization capabilities while maintaining backward
compatibility and improving the overall developer experience.

@1-Bart-1 1-Bart-1 marked this pull request as draft September 28, 2025 09:35
@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2025

Codecov Report

❌ Patch coverage is 1.21951% with 324 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
ext/SymbolicAWEModelsMakieExt.jl 0.00% 244 Missing ⚠️
src/system_structure.jl 4.10% 70 Missing ⚠️
src/SymbolicAWEModels.jl 9.09% 10 Missing ⚠️

📢 Thoughts on this report? Let us know!

@1-Bart-1 1-Bart-1 marked this pull request as ready for review October 16, 2025 15:15
Copy link
Collaborator

@jellepoland jellepoland left a comment

Choose a reason for hiding this comment

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

  • The Getting Started Guide link in the Readme of the Makie branch does not work, seems to be an URL issue (the .md missing?)

For the rest seems all good, nice improvements.
We'll have to change some of these again, once I merged with nested examples.
But that is for later.

Great to see more detail on the developer instructions, and streamlining the plotting usage and multiple .toml setup

@1-Bart-1
Copy link
Member Author

[ ] The Getting Started Guide link in the Readme of the Makie branch does not work, seems to be an URL issue (the .md missing?)

This link won't exist until merged with main, because it is new documentation.

@jellepoland
Copy link
Collaborator

Clear, thanks for the clarification and the updates.

@1-Bart-1 1-Bart-1 requested a review from jellepoland October 19, 2025 14:18
@jellepoland jellepoland merged commit ee81411 into main Oct 19, 2025
13 of 14 checks passed
@1-Bart-1 1-Bart-1 deleted the makie branch October 19, 2025 14:19
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.

3 participants