OpticalModel has a method to plot itself, but you created a method in OpticalSystem that does somewhat the same.
This is, first a very inefficient thing to do. Second, it seems wierd to me. You ask OpticalSystem to plot an OpticalModel by giving him an instance of OpticalModel instead of just picking such an instance and asking it to plot itself.
Unless you see somethign I don't see I suggest to remove the plotModel method in OpticalSystem
OpticalModel has a method to plot itself, but you created a method in OpticalSystem that does somewhat the same.
This is, first a very inefficient thing to do. Second, it seems wierd to me. You ask OpticalSystem to plot an OpticalModel by giving him an instance of OpticalModel instead of just picking such an instance and asking it to plot itself.
Unless you see somethign I don't see I suggest to remove the
plotModelmethod in OpticalSystem