-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Rename "aggregration" edge to something better understandable #962
Comments
Hm, I cannot reproduce this issue. Can you please post some example code? or a screenshot? |
It might be my misunderstanding. If my understanding above is correct, I suggest renaming "aggregation 24 edges" to something else. |
Yes, you understood correctly. "The aggregation edge does not mean that types aggregate but that the edge is an aggregation of multiple different edges. I think you have a valid point and the name of the edge can be missleading. @egraether: what do you think about renaming this edge? |
Yes, if renaming makes it more clear, then we should do it (we are not English native speakers after all). Still, we need a new name for this edge type that works as standalone term. Suggestions:
That's all I can think of. I like bundle edge. Any other suggestions? |
bundle sounds good to me |
Final decision: |
The term aggregation is misleading. Bundled edges is a better definition for this type of edge. closes #962
2020.1.117 version.
when a type is using another type(function etc...) it is showing as aggregation instead of using.
I'm sure it was different in prev version. any idea?
thanks,
Kobi
The text was updated successfully, but these errors were encountered: