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

Support FluxMonitor natively in adjoint plugin #1205

Open
tylerflex opened this issue Oct 14, 2023 · 1 comment
Open

Support FluxMonitor natively in adjoint plugin #1205

tylerflex opened this issue Oct 14, 2023 · 1 comment
Assignees

Comments

@tylerflex
Copy link
Collaborator

currently done through FieldMonitor.flux but would be perhaps useful to support the monitor directly, if only to do this under the hood.

@tylerflex
Copy link
Collaborator Author

passing the torch... actually might not be a bad exercise though. Basically what we will want to do is add some kind of "auxiliary monitor" to the forward simulation, handled similar to the grad and grad_eps monitors, but placed at monitor locations that might need field data (ie the FluxMonitors). Then when we construct the adjoint simulation, we swap in the auxiliary monitor's .flux method for the FluxMonitor data and pretend that this is what the user did.

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