BernardoGBS1 commented on PR #68171: URL: https://github.com/apache/airflow/pull/68171#issuecomment-4878617567
Hey @pierrejeambrun, What we are going to do is: - Keep the orthogonal edge routing (to maintain the "routing style" the same as "automatic" mode): We believe that this consistency is important to help maintain the modes coherent, - Discard/Comment out the collision avoidance code: Whereas before we were trying to help the user keep everything organised, we now think that perhaps the user should have the freedom to bunch up nodes on top of each other, - Take a look at the custom edge routing with these new changes and see how these perform on large DAGs. Perhaps the solution was too complex for a viewing aid, we were aiming to provide a refined experience for the user however you are right in pointing out that this is overkill for a non-persistent rearrange mode. These more complex UX features should be implemented down the line if the mode becomes persistent. Furthermore, thank you for spotting the defaultEdgeOptions issue, it can and should be scoped to rearranging only. We'll make the appropriate changes and check if that fixes the issues you pointed out. We started earlier today and hope to have this sorted by either the end of today or tomorrow. Thank you once again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
