Skip to content

Conversation

@xarthurx
Copy link
Owner

@xarthurx xarthurx commented Dec 7, 2025

Use the GSP v1.0 version for organizing P/Invoke code from the CPP side.

@xarthurx xarthurx marked this pull request as ready for review December 7, 2025 15:39
Copilot AI review requested due to automatic review settings December 7, 2025 15:39
@xarthurx xarthurx merged commit ab02ce5 into main Dec 7, 2025
2 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the project to use GSP v1.0 architecture, reorganizing the P/Invoke code structure from the C++ side. The changes establish a cleaner separation between core GSP functionality and platform-specific adapters (Rhino), introduce platform-independent geometry types, and refactor the native bridge into a more maintainable extension-based structure.

Key Changes:

  • Introduced platform-independent geometry types (Vec2, Vec3, Mesh) and core utilities (Serializer, Platform, MarshalHelper)
  • Reorganized namespace structure: GSP for core, GSP.Adapters.Rhino for Rhino-specific code, igMesh.Native for native bridge functions
  • Updated version from 0.5.9 to 0.6.0

Reviewed changes

Copilot reviewed 33 out of 35 changed files in this pull request and generated no comments.

Show a summary per file
File Description
igm/*.cs (multiple files) Added new namespace imports for GSP adapters and igMesh native bridge
igm/01_meshInfo.cs Updated native bridge class references from GSP.NativeBridge to igMeshBridge
igm/01_meshIO.cs Code formatting improvements and reorganization
igm/03_meshAdjacency.cs Fixed string indentation alignment
igm/00_igmInfo.cs Version bump to 0.6.0
GeoSharPlusNET/Wrapper.cs Moved to GSP.Adapters.Rhino namespace
GeoSharPlusNET/NativeBridge.cs Refactored to igMeshBridge class using new Platform utilities
GeoSharPlusNET/MeshUtils.cs Updated to use new namespace structure and igMeshBridge
GeoSharPlusNET/Geometry/*.cs New platform-independent geometry types
GeoSharPlusNET/Core/*.cs New core utilities for serialization, marshaling, and platform detection
GeoSharPlusNET/Adapters/Rhino/*.cs New Rhino adapter implementation
GeoSharPlusNET/Extensions/igMesh/*.cs New high-level igMesh utilities and documentation
GeoSharPlusCPP/src/API/BridgeAPI.cpp Updated include path
GeoSharPlusCPP/CMakeLists.txt Corrected comment text

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.

1 participant