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.

Sign up for GitHub

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

Jump to bottom

Feat/add dag viz #213

Open
wants to merge 16 commits into
base: release/3.4
Choose a base branch
from
Open

Feat/add dag viz #213

wants to merge 16 commits into from

Conversation

everdyke
Copy link
Collaborator

  • Added a Workflow DAG viz to the Workflow details page (by TaskTemplate)
  • Add a Task DAG viz to the Task Details page

@everdyke
Break it out in to tabs
07418e3
@everdyke
backend route is working
931ef0d
@everdyke
Data plumbing is now working
1509227
@everdyke
fix merge conflicts
46fd051
@everdyke
Handle use case where there is only one node in the tasktemplate dag
35fc598
@everdyke
Everything currently working. Need to show more information in both o…
77d3fbd 
…f the DAGs
@everdyke
Pull the popover in to it's own component. Have progress bar and wf D…
471ac74 
…AG call that component
@everdyke
Add loading spinner to Workflow DAG and Task DAG if node and edges li…
8a94977 
…sts are empty
@everdyke
change out circular progress bar
ca7897a
@everdyke
Show task status in information pop up, and update FSM information
1773e1a
@everdyke
Change the Task ID to be a link in the Task information modal
44c33df
@everdyke
Modal with dummy data pops up when node is clicked
6d0534a
@everdyke
Information pop up is workfing for Task DAG. Need to clean up still
b253f19
@everdyke
fix type errors
715b4e9
@everdyke
Fix linting
ff92880
@github-actions GitHub Actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  jobmon_server/src/jobmon/server/web/routes/v3/fsm
  workflow.py 591-667
Project Total  

This report was generated by python-coverage-comment-action

@github-actions GitHub Actions
Copy link

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 4562288. ± Comparison against base commit 7db189d.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LimingXuIHME LimingXuIHME Awaiting requested review from LimingXuIHME LimingXuIHME is a code owner

@mlsandar mlsandar Awaiting requested review from mlsandar mlsandar is a code owner

@geoffreyphipps-uw geoffreyphipps-uw Awaiting requested review from geoffreyphipps-uw geoffreyphipps-uw is a code owner

@wcbzero wcbzero Awaiting requested review from wcbzero wcbzero is a code owner

@blsmxiu47 blsmxiu47 Awaiting requested review from blsmxiu47 blsmxiu47 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@everdyke