-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extensible definition dictionary #47
Comments
This might be a reason to get rid of the DSL and define the operators using a builder-syntax in-code... haven't thought it through fully though. |
I think that's a separate issue. When planning this, I thought that we could simply split our XML definitions (e.g. I'm not sure if we'll ever want to completely replace all our |
I don't think that's nearly as elegant, if I'm honest... keeping everything in code makes the extensibility aspect really easy. |
Alright, let's think about that a little more. I'd like to settle things like this before releasing the "stable" version 1.0, that should have a stable and extensible API we won't accidentally break. |
Agreed. I know abandoning the XML definition format may seem like a bit of a waste now, but I think it could be the optimal way forward, especially as far as extensibility is concerned. The DSL could grow to unwieldy proportions otherwise... |
To avoid polluting our main formula dictionary (i.e.
PredefinedTexFormulas.xml
) with various localized formula names (like this), I propose to add a dictionary extension mechanism.Users should be able to provide theirs own dictionary files that will be merged with the existing definitions.
Also, these Russian localized definitions should be extracted from the default dictionary and should became opt-in functionality (I still want to redistribute them alongside WPF-Math because personally I need those, and I think they're generally useful for people around the world).
And don't forget to add end-user documentation explaining how to make and apply these definition files!
The text was updated successfully, but these errors were encountered: