Folder structure and naming methods

As we will potentially have lot of different operators under supg, frozen velocity,

P_1

viscosities, etc, I thought of adding an operator-name and folder-name to the .toml file.

For example, now the forms shear_heating and grad_rho_by_rho_dot_u (with operator name FrozenVelocity) can be grouped under terraneo folder.

Hope this will make things clean and easy to organize!

requested review from @kohl

assigned to @pponkumar

Thanks for taking care of organizing this :) since you switched to the P2Vector implementation - have you removed the old "scalar/componentwise" operators here? I am not sure if we should have both versions because we might lose track at some point. Maybe @he66coqe or @vy28quve have some comments, too.

I guess we cannot have componentwise operators now as hog doesn't support them, for good reason! I can remove them altogether and should be possible to switch to P2Vector in hyteg as well.

But if we need to have them, then the pipeline cannot pass also!

Hopefully it should not be too hard to replace them in hyteg. But I think the HOG should still be able to support the old ops. They are still tested in HOG's hyteg integration tests I think.

Ah okay yes, I missed this part where we call the form differently then it has to be adapted in generate.py also, by passing correct vector component as a vector functions, because the gradient and divergence form doesn't support scalar functions.

I can do this depending on what we decide, maybe we could have both itself, so that it may be useful for certain tests also?

So I am fine with replacing the componentwise stuff with the vector ops. What is important imho is that what is generated by the toml is also checked in here. So if we do not have the compwise ops in the toml they need to be deleted in operators/

added 2 commits

• 724385b5 - Regenerate operators
• 527da0de - Modify generate.py to support updates and folder/op-name changes

Compare with previous version

added 1 commit

• 50e6324d - Modify generate.py to support updates and folder/op-name changes

Compare with previous version

approved this merge request

Will merge once https://i10git.cs.fau.de/hyteg/hyteg/-/pipelines/68572 succeeds!

merged

mentioned in commit 57e32ddc