-
Notifications
You must be signed in to change notification settings - Fork 18
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
Normal form vs Normalized #31
Comments
I think replacing our homegrown names with more commonly used names is a good thing. I think we should keep the old names as aliases to the new names for some time. The aliases could generate warnings if used. I am not so sure what names to choose, however.
[1] http://dx.doi.org/10.1137/090752286 |
Currently at it on the |
There is one more thing that I find confusing:
|
What about the wording for the documentation? |
Yes, I think that's OK unless you have a better suggestion
Sent from TypeApp
…On Oct 20, 2017, 19:32, at 19:32, Milan Holzaepfel ***@***.***> wrote:
What about the wording for the documentation? `bond leg` becomes
`virtual leg`? Do we keep the name `physical leg`?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#31 (comment)
|
Fixed in the refactor branch with fe3e95a and all the preceding commits. |
I am slowly getting angry with my past self for calling the cannoncial form as normal form (and, hence, introducing functions such as
MPArray.normalize(...)
. This will be a big change in the API, but I think we should think about a new naming scheme.And while we are making ourselves unhappy, I would also think about renaming
bdims
toranks
andpdims
todims
. I think the distinction would help readability a littleThe text was updated successfully, but these errors were encountered: