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

Umbrella issue -- parallelism improvements #427

Open
8 of 22 tasks
elijahbenizzy opened this issue Nov 19, 2024 · 0 comments
Open
8 of 22 tasks

Umbrella issue -- parallelism improvements #427

elijahbenizzy opened this issue Nov 19, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request umbrella Group of issues together

Comments

@elijahbenizzy
Copy link
Contributor

elijahbenizzy commented Nov 19, 2024 •
edited
Loading

All the improvements to parallelism that we want to make post the first release:

  • UI improvements
  • Task creation/management
    • Add API for task IDs
      • Add API
      • Wire through to UI
      • Add documentation
  • Guard-rails around async problems
    • Ensure that generators that are both async and not async work as expected
  • Wiring through persisters
  • Execution improvements
    • Reevaluate execution API
    • Add guard-rails for executors (E.G. with async, etc...)
    • Allow customization of execution capabilities on a per-task basis
  • Ensure recursion works -- multiple levels deep
@elijahbenizzy elijahbenizzy added documentation Improvements or additions to documentation enhancement New feature or request umbrella Group of issues together labels Nov 19, 2024
elijahbenizzy added a commit that referenced this issue Nov 22, 2024
@elijahbenizzy
Parallelism UI improvements -- see #427
a0241af 
1. Displays sub-applications inline
2. Adds additional tooling on hierarchy of traces
3. A few other miscellaneous fixes around highlighting/table spacing

Note that we change from having the sequence ID in the URL
to having the point in the highlighted state in the URL. This allows us
to have multiple applications at once.

The code can be a little messy, and there are a few more places we
should probably have this distinction made, but for now this really
improves the experience with parallelism
elijahbenizzy added a commit that referenced this issue Nov 22, 2024
@elijahbenizzy
Parallelism UI improvements -- see #427
9d9d430 
1. Displays sub-applications inline
2. Adds additional tooling on hierarchy of traces
3. A few other miscellaneous fixes around highlighting/table spacing

Note that we change from having the sequence ID in the URL
to having the point in the highlighted state in the URL. This allows us
to have multiple applications at once.

The code can be a little messy, and there are a few more places we
should probably have this distinction made, but for now this really
improves the experience with parallelism
elijahbenizzy added a commit that referenced this issue Nov 22, 2024
@elijahbenizzy
Parallelism UI improvements -- see #427
606194a 
1. Displays sub-applications inline
2. Adds additional tooling on hierarchy of traces
3. A few other miscellaneous fixes around highlighting/table spacing

Note that we change from having the sequence ID in the URL
to having the point in the highlighted state in the URL. This allows us
to have multiple applications at once.

The code can be a little messy, and there are a few more places we
should probably have this distinction made, but for now this really
improves the experience with parallelism
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation enhancement New feature or request umbrella Group of issues together
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@elijahbenizzy