-
Notifications
You must be signed in to change notification settings - Fork 26
Tradeoff of defining types as subobjects #86
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
base: master
Are you sure you want to change the base?
Conversation
adomani
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for writing this!
Co-authored-by: damiano <adomani@gmail.com>
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
…g-types-as-subobjects
|
Can you mention that the subobject approach has sometimes performance issues? Like what happened for the ring of integers (see #12386). Also, I think that it is worth mentioning that the anonymous constructor can make the projections' name issue less important. |
|
#12386 was in fact the impetus for me to write this blogpost, but it seems that the issue has been fixed by making the discrimination keys more thorough, see #86 (comment) I am not sure why anonymous dot notation is relevant here. |
1 similar comment
|
#12386 was in fact the impetus for me to write this blogpost, but it seems that the issue has been fixed by making the discrimination keys more thorough, see #86 (comment) I am not sure why anonymous dot notation is relevant here. |
|
I mean |
|
Ah sure, but anonymous constructor notation is useless when working with highly nested structures such as subobjects. Eg to build a subring one needs to do |
Vierkantor
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My concerns are addressed, this is good to go as far as I'm concerned. I'll merge in a few days unless we get objections.
Zulip