Open SirNate0 opened 3 months ago
@SirNate0, thank you for the feedback. We'll look into adding names for the dispersion parameters where we can. The indentation aspect I'm not so sure about, but we'll bear that in mind.
Just out of interest, what kind of systems are you interested in studying with refellips?
Sorry for taking so long to get back to you. I was trying to model a bilayer of less standard semiconductor films (AlN, and ScAlN). In the end I went back to using CompleteEASE to do it (I've used it before, but I wanted to give this package a try analyzing the data from home rather than at the work computers with CompleteEASE instealled).
Are there any features that you think need to be added?
It seems the various ScattererSE derived models do not specify the names for their parameters, which makes it very hard to read the output when printing out the structure's parameters as there are many parameters that are just named
''
.As an example, I think just adding this to
__init__
forclass TaucLorentz
mostly fixes the problem.For TaucLorentz, at least, this still leaves the
sequence_to_parameters
individual Parameter objects without names, but the Parameters groups is at least named something besidesNone
. As a futher improvement, the sub-Parameters can be indented relative to their parent, though that requires modifying the refnx/parameter.py code.For example, with this code instead of the current in
class Parameters
:the output looks like