I like it.
Orekit uses the OrekitMessages
class for the format string so that the text can be translated.
I would also tend towards more precision in the reported values as I’ve seen some issues that differ by only a very small amount. I can do this after you push if you would like. The MessageFormat
syntax is not intuitive.
And perhaps it would be worthwhile to add that information to any exception that occurs during event finding, so catch RuntimeException
instead of MathRuntimeException
. What do you think?
Yes, you can push it and I will copy it. It is one of the maintenance headaches with Orekit.