Skip to content

Add DAG visualization to CROWN configs#369

Open
tvoigtlaender wants to merge 106 commits intomainfrom
add_DAG_vis_PR
Open

Add DAG visualization to CROWN configs#369
tvoigtlaender wants to merge 106 commits intomainfrom
add_DAG_vis_PR

Conversation

@tvoigtlaender
Copy link
Copy Markdown
Contributor

@tvoigtlaender tvoigtlaender commented Mar 31, 2026 •
edited
Loading

Adds an html-based DAG inspection tool and a python function to create the json data necessary for the tool.
Includes search functionality and input/output information on click of nodes/edges.

Intermediate goals:

  • Separation by scope
  • Replacement of I/O nodes
  • drop down menus for scope, era, and sample type
  • improved scope ordering (global above non-global)
  • Type coding of nodes by purpose (cut, define, ...)
  • Jump to option for Input/output elements at the edges/nodes
  • Extended producer information on click
  • Compatibility with friends

@tvoigtlaender
Compatibility with standalone kingmaker container and local sandboxin…
3ca2195 
…g. Remove legacy mapping library. Adapt onnxruntime imports. Remove unused TMVA_onnxruntime import. Update Root version expected by cmake.
@tvoigtlaender
Add missing import
7e7643a
@tvoigtlaender
Update init.sh function to use singularity container over cvmfs LCG s…
34c7661 
…tacks. Compatible with CROWN documentation.
@tvoigtlaender
Add dry run option to init script to only copy in configs
33cd891
@tvoigtlaender
Update init script with updated argument parsing and fixed git status…
0b2e0c1 
… check in container.
@tvoigtlaender
Update init.sh documentation
b5e354a
@tvoigtlaender
Update documentation
6f1427e
@tvoigtlaender
Cleanup
65bb2b9
@tvoigtlaender
update documentation. source replaced by bash as env setup happens in…
e3c32ae 
… the container.
@tvoigtlaender
improve consistency in documentation
2404732
@tvoigtlaender
Replace autogen of dict library with cached libMyDicts.so. Add lib to…
68757ff 
… CROWNlib.
@tvoigtlaender
Split off Dict lib from CROWNlib to make python quantity calls more l…
61470cb 
…ightweight.
@tvoigtlaender
Cleanup
6054aa3
@tvoigtlaender
Move default ccache dir to CROWN/.cache/ccache
e720830
@tvoigtlaender
black
35bea9e
@tvoigtlaender
update CI worflow
fe227bf
@tvoigtlaender
Activate env at container startup for CI pipeline
05368ce
@tvoigtlaender
Add openssh to container
a851f05
@tvoigtlaender
Update local container sartup and git checks
fa7750e
@tvoigtlaender
Update and apply cpp formatting. Remove unused setup_from_tarball.sh
4f200f5
@tvoigtlaender
Remove duplicates
4a8912a
@tvoigtlaender
Update and apply python formatting
619945d
@tvoigtlaender
Add and apply cmake formatting. Add central script that runs (python/…
7d89dc9 
…c++/cmake) formatting scripts
@tvoigtlaender
add c and h files to cpp formatting
8e140d7
@tvoigtlaender
update image in github CI
e80ce6d
@tvoigtlaender
add cmake format checking
2604c5f
@tvoigtlaender
Downgrade github CI checkout for checks
85b1da2
@tvoigtlaender
update container initilization for github CI
f00dbc1
@tvoigtlaender
add ENV variable to dockerfile, even without proper init
2a9aac5
@tvoigtlaender
Add openssh
0fbf929
tvoigtlaender and others added 23 commits April 23, 2026 04:18
@tvoigtlaender
Refactor for less redundant information in json
98c57f3
@tvoigtlaender
Let cytoscape find downstream elements after shift.
c1887a7
@tvoigtlaender
Fix styles
a521495
@tvoigtlaender
Add shift config changes to json. Show input/output quantities in UI …
4793a9a 
…with shift highlighting.
@tvoigtlaender
Add support for shifted input quantities in friend configs.
ab08bf6
@tvoigtlaender
Bugfix
8cbd1de
@tvoigtlaender
Minor style adjustments. Reset button now also resets search and shif…
2b83e94 
…t selection.
@tvoigtlaender
Copy html into DAG_dir at runtime for compatibility with KingMaker
87c4585
@tvoigtlaender
Prevent redundant copying
4cab4a2
@tvoigtlaender
Merge logging
3065be5
@tvoigtlaender
Black
b1efc4e
@tvoigtlaender
Clean up parameters
b3bbd56
@tvoigtlaender
clean up html code
c7d423e
@tvoigtlaender
Add Quantities directly copied from input to output file to Global I/…
799fafa 
…O list
@nshadskiy
Merge branch 'main' into Kingmaker_sandboxing
e84dfac
@nshadskiy
clang formatting
b510b86
@tvoigtlaender
Consistently use 'source' to call init script across all mentions
b47d1b8
@tvoigtlaender
Merge branch 'Kingmaker_sandboxing' into add_DAG_vis_PR
4be4887 
Merge prior PR into later PR
@tvoigtlaender
Redo metadata write to root file. Use root file metadata to validate …
adbeac4 
…quantitiesmap file usage. Export original config name as part of input_quantities_mapping to be used by visualization.
@tvoigtlaender
cleanup
7ee37a6
@tvoigtlaender
Add config source to inputs. Move edge bundling to clientside (To be …
e229b77 
…implemented).
@tvoigtlaender
Bugfix
b0c5a4d
@tvoigtlaender
bugfix
ee87f9c
@tvoigtlaender tvoigtlaender mentioned this pull request May 7, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nshadskiy nshadskiy Awaiting requested review from nshadskiy

@a-monsch a-monsch Awaiting requested review from a-monsch

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tvoigtlaender @nshadskiy