Potential issue while upgrading from .25.1 to .25.2 #102
Comments
|
Hi Thomas, I will try to reproduce the issue. Matthias |
|
Don't try too hard. Might be a false alarm, as he at some point installed a dev version. |
|
No, not a false alarm. A real issue. Thanks a lot for testing!!! I did not see that myself (yet) because I did not try to upgrade (yet). I have fixed that plus made the application safe against failure when reading the config. Config errors are reported but ignored otherwise. Thanks again and best regards, Matthias |
|
Thank you for figuring this out. We really appreciate it. LG |
Kazzz-S
pushed a commit
to Kazzz-S/klayout
that referenced
this issue
Mar 21, 2018
The reason was that for 0.25.1 "macro-editor-font-size" was allowed
to be an empty string (the default). Which meant: take default application
font size. In 0.25.2 this now was required to be a number and 0 was
the default for "auto" font size.
Two changes:
- The default is back to empty string ("0" is still allowed as default)
- The application was made safe against broken configuration files: an
error is printed to the log, but apart from that the application
will work (the configuration value is ignored however).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A labmate reported that when he installed the new klayout, he got that “Got an empty string where a real number was expected” exception.
This could be solved by deleting the
~/.klayout/klayoutrcfile.Can anyone help upgrade from a clean .25.1 to a .25.2 to check whether this will be a major issue for end users? Otherwise, more in the long term, we have to change klayout code so that it's failsafe with regards to "corrupted" klayoutrc.
The text was updated successfully, but these errors were encountered: