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

process types registry #77

Closed
wants to merge 6 commits into from
Closed

process types registry #77

wants to merge 6 commits into from

Conversation

eagmon
Copy link
Member

@eagmon eagmon commented Dec 18, 2024

This is another possible fix to the cores everywhere. I made a process_type_registry which includes a ProcessType called 'core', which is the default core that Composites and Processes can use. People can add to this core if they want to, or make new cores and then just pass a string into Composite to look up the appropriate core.

I think there may need to be something smarter with how Processes are assigned cores, without having our users needing to thread cores through everywhere -- I don't want Processes to have to do the whole core=core through the super. Can Composite ensure each Process is given the correct core by setting it rather than having to weave it in through the init?

@eagmon eagmon closed this Dec 21, 2024
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.

1 participant