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

"Diamond" like inheriting structure with Processes and Recipes #126

Open
aalbino2 opened this issue Dec 6, 2024 · 1 comment
Open

"Diamond" like inheriting structure with Processes and Recipes #126

aalbino2 opened this issue Dec 6, 2024 · 1 comment

Comments

@aalbino2
Copy link
Collaborator

aalbino2 commented Dec 6, 2024

I'm fiduring out while tweaking the merge_sections function that we have some problematic classes:

  • EtchingRecipe
  • AnnealingRecipe
  • and so on

Those are never really used because what we do in the code is:

MyEtchingRecipe(MyEtching, Recipe)

not

MyEtchingRecipe(EtchingRecipe)

So i propose to get rid of those and explain in the Recipe class how to use it,

is it a topi for datamodeling @ka-sarthak @hampusnasstrom ?

@hampusnasstrom
Copy link
Collaborator

That sounds very interesting! Yes, would be great to hear about it on Wednesday :)

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

No branches or pull requests

2 participants