Skip to content

Various UI updates: remove elkjs add dagre, cost updates, executable script#586

Merged
skrawcz merged 4 commits intomainfrom
stefan/replace-elkjs
Oct 27, 2025
Merged

Various UI updates: remove elkjs add dagre, cost updates, executable script#586
skrawcz merged 4 commits intomainfrom
stefan/replace-elkjs

Conversation

@skrawcz
Copy link
Contributor

@skrawcz skrawcz commented Oct 5, 2025

Various changes/

Changes

  • remove elkjs because of license and replace with dagre
  • update model costs
  • change script to executable for ergonomics.

How I tested this

  • spun up things locally

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

DAGRE is MIT licensed and compatible with apache 2.0

Validated that the UI works.
This wasn't before.
So that UI costs are closer to reality today.
@github-actions
Copy link

github-actions bot commented Oct 5, 2025

A preview of 7b15d12 is uploaded and can be seen here:

https://burr.dagworks.io/pull/586

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/apache/burr/tree/gh-pages/pull/586/

Copy link
Contributor

@zilto zilto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Left some comments. I didn't pull the code to try the UI change. I'm assuming the dagre layout was manually viewed / tested

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm confused why we're getting an empty diff on this file. Was it moved?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For reference, Pydantic is now running a project that keep tracks of costs:https://github.com/pydantic/genai-prices

@elijahbenizzy
Copy link
Contributor

Tested it out seems OK
image

@skrawcz skrawcz merged commit a7caa12 into main Oct 27, 2025
2 checks passed
@skrawcz skrawcz deleted the stefan/replace-elkjs branch October 27, 2025 06:42
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.

3 participants