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 magnet cutting to MagnetCoil class #50

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

Move magnet cutting to MagnetCoil class #50

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

Comments

@gonuke
Copy link
Member

gonuke commented Mar 4, 2024

The process of cutting a magnet coil might make sense to do with each magnet individually rather than as a set.

This probably warrants some discussion to confirm if this is a wise change, but it further focuses on the operations to be per magnet coil and thus allows a richer set of possible actions.

@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

We might reconsider this issue in the context of the modified magnet modeling workflow. It may still be worth doing, but would look differently in that new context.

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
Projects
None yet
Development

No branches or pull requests

2 participants