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

Show epsilon plot #243

Closed
Tracked by #242
joamatab opened this issue Nov 21, 2023 · 6 comments · Fixed by #256
Closed
Tracked by #242

Show epsilon plot #243

joamatab opened this issue Nov 21, 2023 · 6 comments · Fixed by #256

Comments

@joamatab
Copy link
Contributor

joamatab commented Nov 21, 2023

I find the epsilon plots more useful than current plots in ComponentModeller

before you can see nitride above

image

now you can see one or the other

image

@yaugenst

@joamatab joamatab mentioned this issue Nov 21, 2023
3 tasks
@yaugenst
Copy link
Collaborator

yaugenst commented Nov 21, 2023

i believe the second plot is plotting the wrong x slice, both layers should show up

you can see the monitor in the yz plot, which means you are plotting at either the left or right border of the simulation, where there is only a single layer with polygons

@joamatab
Copy link
Contributor Author

i see,

this looks better now
image

@yaugenst
Copy link
Collaborator

if you want to see both layers you need to plot somewhere other than the ports

@joamatab
Copy link
Contributor Author

yes, the main issue is that td.ComponentModeler does not have plot_eps method

@yaugenst
Copy link
Collaborator

yaugenst commented Nov 21, 2023

and it shouldn't because the component modeler does not know about materials, this is intentional

ah sorry i confused it with the base component. for the td.ComponentModeler, you can extract individual simulations like this:

sim = modeler.sim_dict[sim_key]
sim.plot_eps(...)

i.e., you can extract a simulation and plot epsilon for each of them. maybe this functionality can be included in the component modeler on tidy's side yeah

@joamatab
Copy link
Contributor Author

joamatab commented Dec 1, 2023

fixed by #256

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

Successfully merging a pull request may close this issue.

2 participants