-
Notifications
You must be signed in to change notification settings - Fork 0
feat: geometry template #19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Caution Review failedThe pull request is closed. WalkthroughThe changes update the FlatCityBuf schema and corresponding Rust code to support geometry templates and instance encoding. New fields in the Header and CityObject tables, along with added tables and structs (such as GeometryInstance, TransformationMatrix, and DoubleVertex), redefine how geometric data is handled. Several enums and mapping tables have been removed from one schema file and reintroduced with modifications in another, while serialization and deserialization routines are enhanced to process these new elements. Additional tests and minor formatting adjustments complete the update. Changes
Sequence Diagram(s)Serialization FlowsequenceDiagram
participant Writer as FcbWriter
participant Serializer as Serializer Module
participant Builder as FlatBufferBuilder
participant File as Serialized File
Writer->>Serializer: Process CityFeature & CityObject
Serializer->>Builder: Call to_geometry_instance & to_templates_vertices
Builder->>File: Write header with templates and features with geometry instances
Deserialization FlowsequenceDiagram
participant Reader as FcbReader
participant Deserializer as Deserializer Module
participant Buffer as FlatBuffer Data
Reader->>Deserializer: Read header & feature stream
Deserializer->>Buffer: Retrieve templates and geometry instance fields
Deserializer->>Reader: Return decoded CityFeature & CityObject
Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (13)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Here's the code health analysis summary for commits Analysis Summary
|
This change is
Summary by CodeRabbit
New Features
Refactor
Tests
Chore