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

Additional results and updates to adjoint-optimization tutorial involving waveguide mode converter #2373

Merged
merged 5 commits into from
Jan 20, 2023

Conversation

oskooi
Copy link
Collaborator

@oskooi oskooi commented Jan 12, 2023

Additional results including two new figures for the tutorial Broadband Waveguide Mode Converter with Minimum Feature Size originally added in #2365.

@stevengj
Copy link
Collaborator

stevengj commented Jan 12, 2023

Would be good to check if the constraint is active (i.e. near zero) for the case with a lengthscale of 225 where the Wenchao's a posteriori measurement is returning 275. If the constraint is active, then this may be limited by the accuracy of the constraint functions?

Also, make sure to run Wenchao's ruler code for both the linewidth and line spacing?

@oskooi
Copy link
Collaborator Author

oskooi commented Jan 13, 2023

Would be good to check if the constraint is active (i.e. near zero) for the case with a lengthscale of 225

Looks like the constraint function is not active for the line width and spacing in the final design because the constraint function values are both ~0.001 (i.e., positive). Because the constraint is inactive, this means that the optimizer therefore likely got trapped in a local optima with minimum feature size of 275 nm.

Separately, I have reverted the horizontal axis of the plot of the worst-case reflectance and transmittance back to the imposed constraint from the measured feature size.

@stevengj stevengj merged commit f448ebd into NanoComp:master Jan 20, 2023
@oskooi oskooi deleted the adjsol_tutorial_update branch January 20, 2023 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants