search

lbolla / EMpy

Electromagnetic Python
MIT License
194 stars 83 forks source link

fix n0_known #9

Closed Ripalda closed 7 years ago

Ripalda commented 7 years ago

I suspect n0_known does not work as intended when multiple wavelengths are used in a calculation, but I have not attempted to fix this, because L. Bolla probably has a clear idea about how he would like to implement this. A fix could be included here or in tm.solve, where RefractiveIndex could be called with one wavelength at a time as argument inside the for w in wls loop.

lbolla commented 7 years ago

Hi! I've pushed a fix: https://github.com/lbolla/EMpy/commit/57409d4b0ac601fca8207ed23210ee9b1f1d45c8

Can you take a look and let me know if it fixes it for you?

Ripalda commented 7 years ago

This seems to work for me. Thank you for your excellent work on this project.