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

Issue559 prototype #211

Draft
wants to merge 144 commits into
base: main
Choose a base branch
from
Draft

Conversation

SimonDold
Copy link
Contributor

This prototype is meant to decide how we want to tackle the component interaction problem (issue559).

Major changes to the old design:

  1. Replace the "options" constructor with actual parameters (Unpacking of options objects happens in the create_component function from the features to call those constructors).

  2. Features are of the type TaskIndependentXYZ instead of XYZ. These classes are factories and have a get_task_specific function that recursively generates the task dependent object of type XYZ, unless this object was generated already. If it was it returns the already generated component instead of generating an equal one again.

  3. There are new base classes "Component" and "TaskIndependentComponent".

@FlorianPommerening FlorianPommerening marked this pull request as draft July 8, 2024 11:50
@SimonDold SimonDold force-pushed the issue559-prototype branch from 89f74f3 to a874b37 Compare July 12, 2024 07:58
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