compas-dev / compas_fab

Robotic fabrication package for the COMPAS Framework.
https://compas.dev/compas_fab/
MIT License
108 stars 32 forks source link

Add type hints for some of the major classes #423

Closed yck011522 closed 2 weeks ago

yck011522 commented 1 month ago

Addresses #416

What type of change is this?

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

yck011522 commented 1 month ago

This is partly address #416

yck011522 commented 1 month ago

Waiting for #422 to change also the type hints for plan_cartesian_motion. Waiting for #424 to change the type hints in the backend features.

jf--- commented 3 weeks ago

@yck011522 a terrific contribution

yck011522 commented 2 weeks ago

blocked by #422

yck011522 commented 2 weeks ago

Okay everyone I'm messing everything up with renaming a branch. I hope there isn't any opened review here. I'll make a new PR.