Prevent math.import(...) from overriding units

[bornova]

Would it be possible to extend the override: false option in math.import(...) to units as well?

For example, if a function named N is imported using math.import({ N: function () {...} }), it overrides the newton unit N. Would be great to have an option to prevent this.

[josdejong]

That makes a lot of sense!

I've created a PR here: #3225 it's a breaking change though so we'll have to wait until the first next major release to merge and publish it.

As a workaround, you could create your own wrapper function around math.import and check whether an import is a unit using math.Unit.isValuelessUnit(...) and if so, throw an exception.

[bornova]

Thanks @josdejong! I worked around it by importing as factory function with a custom name instead. So instead of directly importing function N, I am importing as myfunc.N. This way I got to keep both N unit and the 'N' function imported from an external library. Looking forward to the next major release.

[josdejong]

Ah, that is indeed a neat way to prevent this issue, thanks for sharing!