You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
At current the code is set up to require additional locations to be defined when calling statmake.lib.apply_stylespace_to_variable_font, even if one does not have additional locations to define. Additionally, it looks like the _sanity_check code and _generate_builder_data also assume the presence of additional defined locations.
I went through and added if additional_locations is not None: to my copy of lib.py so now it works ok! Figure you might want to implement it a different way, so thought I'd just file a bug.
The text was updated successfully, but these errors were encountered:
Fair enough. I still think it would be better to have the additional_locations set as optional rather than required (and provide a default state of {} if not explicitly set). That way one doesn't even have to include empty braces if it is unnecessary (which I find a bit annoying 😆 )
At current the code is set up to require additional locations to be defined when calling statmake.lib.apply_stylespace_to_variable_font, even if one does not have additional locations to define. Additionally, it looks like the _sanity_check code and _generate_builder_data also assume the presence of additional defined locations.
I went through and added
if additional_locations is not None:
to my copy of lib.py so now it works ok! Figure you might want to implement it a different way, so thought I'd just file a bug.The text was updated successfully, but these errors were encountered: