Refactored geoplot.py and added test file with fake data for visualiz… #83
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hi! I’ve refactored the geoplot.py file for better readability and added a test file using fake data to validate the rendering logic. Let me know if any changes are needed. Thanks for reviewing! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request refactors the geoplot.py file to improve code readability and modularity. It also introduces a new test file test_geoplot.py, which contains fake data to help visualize entity rendering on the map. This helps validate that the plotting logic works as expected.
✅ Changes Made
Refactored render method in geoplot.py for clarity.
Added test_geoplot.py to simulate input data for testing visualization.
Ensured compatibility with Cesium entity plotting.
🧪 Purpose
To enhance maintainability and provide a reliable way to test the visualization logic with sample data.