search

robbert-harms / MDT

Microstructure Diffusion Toolbox
GNU Lesser General Public License v3.0
50 stars 18 forks source link

Confusing error message "Double operator in model listing." #38

Closed lukeje closed 3 years ago

lukeje commented 3 years ago

Using the same model name twice raises an error here: https://github.com/robbert-harms/MDT/blob/79012e86a28f612d632fa299ffc4a120993abc3a/mdt/model_building/trees.py#L110 but the message is ambiguous (I first thought it related to single vs. double floating point). Perhaps "Duplicate operator in model listing." would be more unambiguous?

robbert-harms commented 3 years ago

Hi Luke,

That is a good suggestion.

I changed it and I will put it in the next minor release.

Thanks,

Robbert