Prevents refractive index dispersions from being summed#113
Conversation
|
@MarJMue If you can come up with a better name than |
|
Maybe it's weird because it is negating it. We could also switch and add additivity by an subclass with Edit: See #114 |
|
I found a better way. I will implement it as |
MarJMue
left a comment
There was a problem hiding this comment.
I think the implementation is good as is. And we can easily expand if needed. 👍
Fixes #110
Also keeps the DispersionSum flat, i.e. adds the dispersions to the internal DispersionSum list instead of creating
DispersionSum(DispersionSum(Dispersion, Dispersion), Dispersion.I think with this fix we could also release 0.15.1 to provide python 3.11 support and also to fix this bug in the published package.