Skip to content

Conversation

@JensKrumsieck
Copy link
Member

@JensKrumsieck JensKrumsieck commented Nov 7, 2025

Next steps to do:

  • Add proper view models for workflow and tool and add to state as enum
  • Save changes done in code editor with live update in nodes or error on invalid
  • Improve project handling. Folder valid but no workflow.toml? Ask user if he/she wants s4n init
  • Add new steps from sidebar (drag and drop tools)
  • Shift+A opens dropdown with possibility to add new steps
  • Add second view possibilty in sidebar "only cwl"
  • [ ] Add fields for default values in node editor (?) - will break how we currently handling drawing of edges (later point)
  • Complete styling for edges (todo! macros)
  • Get rid of most of the .unwrap()
  • [ ] Add a (tab?, sidebar right?, dropdown?) view to show infos about currrent open file with possibility to execute (how do other tools do this?)
  • [ ] Add possibility to annotate label, description, intent in view (of above task)
  • Delete button in file list
  • New Workflow creation
  • New Tool create

@JensKrumsieck JensKrumsieck linked an issue Nov 7, 2025 that may be closed by this pull request
@JensKrumsieck JensKrumsieck force-pushed the dioxus branch 2 times, most recently from c0c673e to 9861aba Compare November 10, 2025 09:29
@JensKrumsieck
Copy link
Member Author

JensKrumsieck commented Nov 11, 2025

image current status, workflow can be loaded into graph representation using `petgraph` crate . connections to outputs missing in conversion currently. Nodes can be displayed, and dragged, a bit clunky however.

only connected slots displaying currently, not all of tool however (!)

@JensKrumsieck
Copy link
Member Author

Moving the onmouseXXX events to parent worked to make dragging less clunky, also missing connection have been added:
image

@JensKrumsieck
Copy link
Member Author

Can be merged kinda soon after #202 is merged

@JensKrumsieck
Copy link
Member Author

started with create form:
grafik

after that we'll merge to main

@github-actions
Copy link

Code Coverage

Package Line Rate Health
packages/gui/src 27%
packages/cwl_annotation/src 35%
packages/cwl_core/src 79%
packages/gui/src/components/files 9%
packages/core/src 72%
packages/remote_execution/src 0%
packages/test_utils 83%
packages/gui/src/components/layout 0%
packages/util/src 82%
packages/core/src/parser 93%
packages/gui/src/components 2%
packages/cwl/src 0%
packages/gui/src/components/graph 0%
packages/remote_execution/src/reana 0%
packages/repository/src 97%
packages/cli/src 62%
packages/reana/src 79%
packages/cwl_execution/src/runner 65%
packages/cli/src/commands 62%
packages/cwl_execution/src 64%
Summary 61% (3970 / 6501)

@JensKrumsieck JensKrumsieck marked this pull request as ready for review December 11, 2025 10:00
@JensKrumsieck
Copy link
Member Author

The PoC is prepared to merge into master

@JensKrumsieck JensKrumsieck merged commit 6b48699 into main Dec 11, 2025
13 of 14 checks passed
@JensKrumsieck JensKrumsieck deleted the dioxus branch December 11, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GUI for CWL Creation

3 participants