enso icon indicating copy to clipboard operation
enso copied to clipboard

Auto layout doesn't take the connections into account when placing nodes

Open farmaazon opened this issue 6 months ago • 6 comments

For example, when creating "Geospatial analysis" example and removing metadata, all nodes seem to be laid out backwards (so all connections are going up)

Image

farmaazon avatar Feb 05 '24 09:02 farmaazon

Note: Just put them from top to bottom as in file. This ensures topological order.

farmaazon avatar Feb 06 '24 15:02 farmaazon

Adam Obuchowicz reports a new STANDUP for today (2024-02-07):

Progress: Made a fix for old nodes being visible for a moment after switching project. Also implemented small fixes for boolean. It should be finished by 2024-02-12.

Next Day: Next day I will be working on the same task. Merge PR and plan next tasks

enso-bot[bot] avatar Feb 07 '24 16:02 enso-bot[bot]

This seems to be fixed by c811a5ae8ba4a512eb74ece2f9f6cb332c3bcdd4.

Before this commit, I observed that removing the metadata of a program results in the graph being updated with a reversed layout, as in the screenshot above. After this commit, this behaviour is gone and the layout is correct.

New projects without metadata seem to be laid out correctly before and after this commit.

MichaelMauderer avatar Feb 20 '24 13:02 MichaelMauderer

Let's yet investigate if the nodes are actually sorted, or we get the proper order by chance.

farmaazon avatar Feb 20 '24 15:02 farmaazon

Michael Mauderer reports a new 🔴 DELAY for today (2024-02-20):

Summary: There is 10 days delay in implementation of the Auto layout doesn't take the connections into account when placing nodes (#8963) task. It will cause 0 days delay for the delivery of this weekly plan.

Delay Cause: There was a previous standup reported on this issue by Adam. This task was newly picked up now by me.

Possible solutions: Never put tasks back in the backlog or allow this to be better reported for enso bot.

enso-bot[bot] avatar Feb 20 '24 15:02 enso-bot[bot]

Michael Mauderer reports a new STANDUP for today (2024-02-20):

Progress: Opened PR for previous task. Started investigasting issue and found it resolved on develop. After standup discussion we decided to continue investigation into the root cause, which likely still exists. It should be finished by 2024-02-21.

Next Day: Next day I will be working on the #8963 task. Find root cause and eitehr fix it or confirm this is no problem.

enso-bot[bot] avatar Feb 20 '24 15:02 enso-bot[bot]

Michael Mauderer reports a new STANDUP for yesterday (2024-02-21):

Progress: Identified root cause, and the place where the order of node updates is determined. This is now reliably in the correct order and added new documentation pointing out why and how this is happening, ensuring this is noted and not broken in the future. It should be finished by 2024-02-21.

Next Day: Next day I will be working on the #8995 task. Start next task.

enso-bot[bot] avatar Feb 22 '24 16:02 enso-bot[bot]