Skip to content
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

Final Assembly Rendering #109

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Final Assembly Rendering #109

wants to merge 8 commits into from

Conversation

jmwright
Copy link
Contributor

@jmwright jmwright commented Jan 9, 2025

This PR adds generation of final assembly renders, in a step-by-step fashion.

@jmwright jmwright marked this pull request as ready for review January 9, 2025 22:21
@jmwright
Copy link
Contributor Author

jmwright commented Jan 9, 2025

I think this is as far as I can take the final assembly renders without someone else testing and giving feedback. There are some aspects of the code that I am not crazy about, but some of it is due to technical debt from what we have already done. It does work, as you can see from the assembly renders below. They are in the order they were automatically generated, from top to bottom, showing each sequential assembly step for a configuration involving a RPi 4B, a NUC and another RPi 4B, in that order. The reason the NUC is installed first is that it is a "broad" shelf and must be inserted from the top before the top plate is installed.

final_assembly_step_1_annotated
final_assembly_step_1_assembled
final_assembly_step_2_shelf_2_insertion_annotated
final_assembly_step_2_shelf_2_installed
final_assembly_step_3_annotated
final_assembly_step_3_assembled
final_assembly_step_4_shelf_1_insertion_annotated
final_assembly_step_4_shelf_1_installed
final_assembly_step_4_shelf_3_insertion_annotated
final_assembly_step_4_shelf_3_installed

@julianstirling
Copy link
Collaborator

The renders look perfect! It'll take me a bit of time to find time to go through the code.

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.

2 participants