Skip to content
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

Collapsing nodes #8067

Closed
Tracked by #7685
farmaazon opened this issue Oct 16, 2023 · 11 comments · Fixed by #8686
Closed
Tracked by #7685

Collapsing nodes #8067

farmaazon opened this issue Oct 16, 2023 · 11 comments · Fixed by #8686
Assignees
Labels
-gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints

Comments

@farmaazon
Copy link
Contributor

farmaazon commented Oct 16, 2023

To be done after #7863

Collapsing nodes should work exactly the same as in old GUI: https://github.com/enso-org/enso/blob/develop/app/gui/src/controller/graph.rs#L1074-L1106

@farmaazon farmaazon mentioned this issue Oct 16, 2023
@farmaazon farmaazon added d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints -gui x-on-hold labels Oct 16, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Oct 20, 2023
@Frizi Frizi removed the x-on-hold label Oct 26, 2023
@Frizi
Copy link
Contributor

Frizi commented Oct 26, 2023

Alias analysis is needed to determine inputs/outputs of collapsed nodes and to insert appropriate arguments.

Only support as much as the language server currently allows (e.g. single output)

@farmaazon farmaazon assigned farmaazon and mwu-tow and unassigned farmaazon Nov 6, 2023
@vitvakatu vitvakatu moved this from 📤 Backlog to 🔧 Implementation in Issues Board Dec 7, 2023
@farmaazon farmaazon assigned mwu-tow and vitvakatu and unassigned mwu-tow and vitvakatu Dec 18, 2023
@enso-bot
Copy link

enso-bot bot commented Dec 25, 2023

Ilya Bogdanov reports a new STANDUP for today (2023-12-25):

Progress: Understanding how new ast editing works, working on initial implementation. Unit testing is hard to implement, I might use e2e tests instead. It should be finished by 2023-12-28.

@enso-bot
Copy link

enso-bot bot commented Dec 26, 2023

Ilya Bogdanov reports a new STANDUP for today (2023-12-26):

Progress: Implemented collapsing (except for collapsing inside collapsed nodes) and tested it manually. It still has several issues, like not receiving method info from the engine and invalid connection info for collapsed nodes. It should be finished by 2023-12-28.

@enso-bot
Copy link

enso-bot bot commented Jan 3, 2024

Ilya Bogdanov reports a new 🔴 DELAY for today (2024-01-03):

Summary: There is 8 days delay in implementation of the Collapsing nodes (#8067) task.
It will cause 0 days delay for the delivery of this weekly plan.

4 days are weekends and vacation, 3 days of sick leave

Delay Cause: Being sick, also fixes for corner cases required adding additional API.

@enso-bot
Copy link

enso-bot bot commented Jan 3, 2024

Ilya Bogdanov reports a new STANDUP for today (2024-01-03):

Progress: Finishing the implementation, working on corner cases (collapsing inside other methods, choosing a unique name for collapsed function). Found an issue with node selection mechanism that ignores execution stack. Preplanning and weekly demo calls. Need to adjust positions of newly created nodes. It should be finished by 2024-01-05.

Next Day: Next day I will be working on the same task. Opening the PR.

mergify bot pushed a commit that referenced this issue Jan 4, 2024
Preparations for #8067. The actual implementation requires AST editing.

The implementation resembles the algorithm from the Rust gui.
@enso-bot
Copy link

enso-bot bot commented Jan 5, 2024

Ilya Bogdanov reports a new STANDUP for yesterday (2024-01-04):

Progress: Trying to implement proper positioning for collapsed node and output node of the collapsed function. It does not seem possible with the current API, as Ast editing changes Ast IDs, and there is no way to know the ID of the newly created node in advance. The workaround suggested by Kaz didn’t work, so we decided to proceed without positioning until the issue would be fixed on the Ast API side. It should be finished by 2024-01-05.

@vitvakatu vitvakatu mentioned this issue Jan 5, 2024
5 tasks
@farmaazon farmaazon moved this from 🔧 Implementation to 👁️ Code review in Issues Board Jan 5, 2024
@enso-bot
Copy link

enso-bot bot commented Jan 8, 2024

Ilya Bogdanov reports a new STANDUP for the provided date (2024-01-05):

Progress: Finished polishing the code and opened a PR for reviewers. Implementing proper positioning of the nodes is close to impossible with the restrictions of the current Ast editing API, so it would be postponed for a bit. Starting working on e2e tests for collapsing. It should be finished by 2024-01-05.

@mergify mergify bot closed this as completed in #8686 Jan 12, 2024
mergify bot pushed a commit that referenced this issue Jan 12, 2024
Closes #8067

Also fixes `nodeRects` map, as it no longer stores invlisible nodes.

https://github.com/enso-org/enso/assets/6566674/ba66c99f-df74-497b-8924-dc779cce8ef5

# Important Notes
Positioning of newly created nodes is not handled yet, as it requires fixes in the Ast editing API.
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Jan 12, 2024
@enso-bot
Copy link

enso-bot bot commented Jan 17, 2024

Ilya Bogdanov reports a new STANDUP for the provided date (2023-12-07):

Progress: Starting copying algorithm for collapsing nodes from the old gui. It should be finished by 2024-01-05.

@enso-bot
Copy link

enso-bot bot commented Jan 17, 2024

Ilya Bogdanov reports a new STANDUP for the provided date (2023-12-08):

Progress: Working on collapsed nodes analysis, implementing unit tests for it. It should be finished by 2024-01-05.

@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board Jan 18, 2024
@enso-bot
Copy link

enso-bot bot commented Feb 20, 2024

Ilya Bogdanov reports a new 🔴 DELAY for the provided date (2024-01-10):

Summary: There is 5 days delay in implementation of the Collapsing nodes (#8067) task.
It will cause 0 days delay for the delivery of this weekly plan.

Delay Cause: Waiting on review, other tasks in progress.

@enso-bot
Copy link

enso-bot bot commented Feb 20, 2024

Ilya Bogdanov reports a new STANDUP for the provided date (2024-01-10):

Progress: Addressing review comments It should be finished by 2024-01-10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-gui d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants