PlasmaControl / DESC

Stellarator Equilibrium and Optimization Suite
MIT License
87 stars 22 forks source link

Change plot argument from `eq` to `things` to accomodate plotting things that require more than one object #953

Open dpanici opened 5 months ago

dpanici commented 5 months ago

From Daniel:

Idea: to better match the Objectives API, we could change the first argument eq to things, which would normally be the equilibrium but in this case would be the tuple (eq, field). Instead of adding a new kwarg. And this would also be more general for adding new plot quantities in the future.

dpanici commented 2 months ago

887