Closed Moanalengkeek closed 10 months ago
@Moanalengkeek please request review with the arrow buttons under review when I should have another look ;)
PS. also added you to the repo so the CI runs automatically (before I had to manually start the tests so it makes sense adding you after all ;) )
Thanks! Will do. I also tied to find where the tests failed, but it seems to be in a place I didn't change. Would you have any resources for best troubleshooting of such tests?
Hi @Moanalengkeek and everyone.
First of all, thanks to @gomezzz for the support in providing an exact revision. Thanks to you for your contribution to PASEOS.
I think your implementation generally looks good. Supporting a mesh is for long-term vision, but I would stick to a Cuboid, as Pablo suggested, for the next steps.
Before approving this PR, please improve the comments in the code. A few things shall be clarified:
In terms of reference frame, in the frame of the next PR you should create a new directory inside PASEOS, providing utilities to transform your frame to one attached to Earth, which is needed for the pointing vector coordinates.
Moved to #195, which does not need a fork.
Description
Adding a geometric model to the model. This is done with two possibilities; option 1 is to model a basic cuboid with width, length, and height. Option 2 is to import an .obj file, and to use the mesh from that.
Summary of changes
Resolved Issues
How Has This Been Tested?
Tested by the two tests mentioned added in actor_builder_test.py, as well as running a simple code that generates an actor and creates the geometry
Related Pull Requests