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.

Sign up for GitHub

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

Jump to bottom

Cylindrical Adjoint Documentation Suggestion for Monitors #1974

Closed
hammy4815 opened this issue Mar 2, 2022 · 1 comment · Fixed by #2027
Closed

Cylindrical Adjoint Documentation Suggestion for Monitors #1974

hammy4815 opened this issue Mar 2, 2022 · 1 comment · Fixed by #2027
Labels
enhancement

Comments

@hammy4815
Copy link
Contributor

I've been running optimizations of a Microdisk in cylindrical coordinates using a near2far objective argument. With the setup below I was facing NaN's in my adjoint run (which I further debugged and found the source NaN's in objective.py during the call to self._monitor.swigobj.near_sourcedata.

MicrosoftTeams-image

After a series of trial and error, I realized the problem was having my monitor at r=0. Moving it away from the origin fixed the issue. It could be useful to note this somewhere so future users don't encounter the same issue. Thanks!
@smartalecH
Copy link
Collaborator

(cc @mochen4)

@mochen4 mochen4 mentioned this issue Mar 30, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

allow cylindrical adjoint near2far at r=0 mochen4/meep
2 participants
@smartalecH @hammy4815