maxspahn / maxspahn/motion_planning_scenes

Python version

Open
#40 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
11
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Python 3.10 allows dataclasses to become keyword_only, see this

This would allow a base class to handle optional parameters such as color, mass, and orientation and put the functionality in the base class. While child classes can handle required classes such as geometry.

This will lower redundant code.

I think we must now wait for casadi to support python 3.10 as mentioned here casadi/casadi#2877

after that, we could (not should) move toward python 3.10

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

First check the CasADi dependency status in casadi/casadi#2877 and confirm whether Python 3.10 is supported. Then review the project's dataclass definitions and assess centralizing optional parameters in a base class while leaving geometry-specific requirements to child classes; done means an agreed, working migration path with redundant code reduced.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.