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

Move coil meshing to MagnetCoil class #49

Open
gonuke opened this issue Mar 4, 2024 · 1 comment
Open

Move coil meshing to MagnetCoil class #49

gonuke opened this issue Mar 4, 2024 · 1 comment
Labels
magnets Related to generation of magnets meshing_feature

Comments

@gonuke
Copy link
Member

gonuke commented Mar 4, 2024

The cubit calls to create a coil mesh should be a method of the MagnetCoil class. The MagnetSet class could then have a mesh generation method that calls the mesh gen on each of the coils.

@gonuke gonuke changed the title Move coil meshing to MeshCoil class Move coil meshing to MagnetCoil class Mar 4, 2024
@gonuke gonuke added this to the First OO upgrades milestone Mar 4, 2024
@gonuke gonuke added the magnets Related to generation of magnets label Mar 4, 2024
@connoramoreno
Copy link
Collaborator

connoramoreno commented Aug 27, 2024

We might reconsider this issue in the context of the modified magnet modeling workflow. Meshing takes place after full coil construction, since the STEP file must be generated via CadQuery first, before being imported into Cubit. Furthermore, there's interest in providing an open-source volumetric meshing workflow (see #140).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
magnets Related to generation of magnets meshing_feature
Projects
None yet
Development

No branches or pull requests

2 participants