-
Notifications
You must be signed in to change notification settings - Fork 13
Added the TPA/FCA notebook #295
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
Conversation
Spell check passed successfully for 1 notebook(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! Thanks @momchil-flex and @tomflexcompute for your efforts.
- Might say something about where the 8/3 and 4/3 are coming from in the implementation equations (https://en.wikipedia.org/wiki/Kerr_effect#AC_Kerr_effect), and maybe reference the paper https://ieeexplore.ieee.org/document/4299014
- Scaling sigma and tau inversely is a clever way to reach the same steady-state behavior faster. You can see this very nicely from the equation for steady-state Nf. Maybe we could make a similarly clear argument for scaling beta and the waveguide length inversely. There is a nice formula here: https://en.wikipedia.org/wiki/Two-photon_absorption#Absorption_rate.
comes from solving the diffeq dI/dz = -beta I^2 - Can we add an AuxFieldTimeMonitor to record and plot Nf, as an excuse to demonstrate this functionality? And because it is nice to see Nf too.
- I still don't like turning off warnings. Maybe we can leave the warnings for now, and let's separately consider turning off field decay warnings when there are CW sources. Also it says "for the simulations below" but there aren't any?
- The paper specifies a linear optical loss, should we incorporate this into the material model? We should also scale it, as we scaled beta.
This looks really great! Finally we can have a good notebook for this feature...
Thanks @caseyflex , great suggestions!
The warning appears for every simulation data from the batch when the results are loaded.
Yeah I guess we could, it's just pretty small so I don't think it will affect things much, but for completeness I guess we should. |
For FCA in steady state, there is a Beer’s law dependence e^(-alpha x) with alpha proportional to sigma tau beta E^4. So this is also handled correctly with the length and beta rescaling. well I guess the full dependence not assuming small total loss from FCA is a nonlinear diffeq, but I think the scaling argument still holds |
One more suggestion is that maybe instead of plotting the instantaneous flux, you could plot the cycle-averaged flux, or the peak power within each optical period. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. One final comment is you might mention why the aux field is "Nfy", namely that the different components don't interact nonlinearly as a modeling approximation.
@momchil-flex any additional comments? |
Looks good! Maybe just one extra sentence about scaling
|
Added the TPA/FCA notebook
e83ea79
to
fbed217
Compare
Applied some polishing and cosmetic changes to @momchil-flex 's version. Amazing results!