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

Burr tracking client #18

Merged
merged 4 commits into from
Feb 16, 2024
Merged

Burr tracking client #18

merged 4 commits into from
Feb 16, 2024

Conversation

elijahbenizzy
Copy link
Contributor

@elijahbenizzy elijahbenizzy commented Feb 15, 2024 •
edited
Loading

Currently, this:

  • Writes a log file, given a project
  • Writes the application state out
  • Uses pydantic models

TODO:

  • Add test for infinite loop this fixed (it didn't update PRIOR_ACTION in one case)
  • Adds test for Burr tracker
  • Get the pydantic models separated out
  • Separate out the tracking piece into multiple components
  • Add separate install dependencies
  • Add testing for the post_application_construct hook
  • Add TODOs for making async with instructions (that's next, but for now I'm not doing this)
  • Break into three commits, if possible
    • Fixes the infinite loop bug
    • Adds post_application_construct hook
    • Implements Burr tracker
  • Add documentation
    • Concepts
    • Reference

@elijahbenizzy elijahbenizzy mentioned this pull request Feb 15, 2024
Closed
25 tasks
@elijahbenizzy elijahbenizzy force-pushed the burr-tracker branch 3 times, most recently from 64bb632 to ca7818f Compare February 15, 2024 22:23
@elijahbenizzy elijahbenizzy changed the title WIP for burr tracking Burr tracking client Feb 15, 2024
@elijahbenizzy elijahbenizzy marked this pull request as ready for review February 15, 2024 23:48
@elijahbenizzy
Adds application graph + fixes bug in application
abd7c0a 
This should really be two separate commits, but they're too intertwined,
so I'm merging them into one. We have:

1. A new API for application graph
2. fixed a bug in which some actions would infinite loop due to the lack
   of prior state, only in the function-based actions case
@elijahbenizzy
Adds PostApplicationCreate hook
e3fe108 
This will allow the tracker to log the state machine and provide other
capabilities.
@elijahbenizzy
Adds basic tracking component to Burr.
3549d2f 
This is local, and leverages with_tracker() in the application builder.
It uses pydantic to track models, this will be shared between the
tracker and the UI.
@elijahbenizzy
Adds documentation on local tracking
a95da5b 
A WIP but I wanted to make it first-class.
@elijahbenizzy elijahbenizzy merged commit f830c05 into main Feb 16, 2024
5 checks passed
@elijahbenizzy elijahbenizzy deleted the burr-tracker branch February 16, 2024 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skrawcz skrawcz skrawcz approved these changes

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
@elijahbenizzy @skrawcz