Skip to content

Feat/bulk progress display model v2#999

Draft
milovate wants to merge 6 commits into
masterfrom
feat/bulk-progress-display-model-v2
Draft

Feat/bulk progress display model v2#999
milovate wants to merge 6 commits into
masterfrom
feat/bulk-progress-display-model-v2

Conversation

@milovate

Copy link
Copy Markdown
Collaborator

Q/A checklist

  • I have tested my UI changes on mobile and they look acceptable
  • I have tested changes to the workflows in both the API and the UI
  • I have done a code review of my changes and looked at each line of the diff + the references of each function I have changed
  • My changes have not increased the import time of the server
How to check import time?

time python -c 'import server'

You can visualize this using tuna:

python3 -X importtime -c 'import server' 2> out.log && tuna out.log

To measure import time for a specific library:

$ time python -c 'import pandas'

________________________________________________________
Executed in    1.15 secs    fish           external
   usr time    2.22 secs   86.00 micros    2.22 secs
   sys time    0.72 secs  613.00 micros    0.72 secs

To reduce import times, import libraries that take a long time inside the functions that use them instead of at the top of the file:

def my_function():
    import pandas as pd
    ...

Legal Boilerplate

Look, I get it. The entity doing business as “Gooey.AI” and/or “Dara.network” was incorporated in the State of Delaware in 2020 as Dara Network Inc. and is gonna need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Dara Network Inc can use, modify, copy, and redistribute my contributions, under its choice of terms.

milovate and others added 5 commits June 10, 2026 03:37
Replace deprecated bulk progress word wrapping, validate input audio
links before rendering anchors, and derive the card run state inside the
snapshot builder. Keep eval progress rings based on completed evals and
inline redundant one-off helpers in the bulk progress display path.
Refactor the bulk progress card model to ensure ring percentages are clamped between 0 and 100. Update the bulk progress display to replace the deprecated Literal with an Enum for bulk runner states, enhancing clarity and maintainability. Adjust snapshot building to utilize the new state management approach.
- Trim workflow titles before rendering so whitespace-only titles do not create empty links.
Show per-workflow elapsed time beside the active workflow link during
running/stopping, and reorder the complete-state summary to Total runs,
Rows, Credits, Total time. Reuse workflow link-group layout for last
completed meta and inline the live elapsed hook in WorkflowRow. Remove
unused BulkRunner row-group helper.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@milovate, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 49 minutes and 55 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 107ebd94-dc45-4247-b912-03b6a84cc81b

📥 Commits

Reviewing files that changed from the base of the PR and between 19934e2 and 4034645.

⛔ Files ignored due to path filters (2)
  • gooey-gui/app/components/bulkProgress/generated/componentProps.d.ts is excluded by !**/generated/**
  • gooey-gui/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (13)
  • gooey-gui/app/components/bulkProgress/BulkProgressCard.css
  • gooey-gui/app/components/bulkProgress/BulkProgressCard.tsx
  • gooey-gui/app/components/bulkProgress/bulkProgress.types.ts
  • gooey-gui/app/components/bulkProgress/bulkProgressCardModel.ts
  • gooey-gui/app/components/bulkProgress/bulkProgressFormat.ts
  • gooey-gui/app/components/index.ts
  • gooey-gui/package.json
  • pyproject.toml
  • recipes/BulkRunner.py
  • scripts/generate_bulk_progress_props.py
  • widgets/bulk_progress_display.py
  • widgets/bulk_progress_props.py
  • widgets/bulk_progress_state.py

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Define the Python→React prop contract in bulk_progress_props.py and
codegen TypeScript types from JSON Schema. Only componentProps.d.ts is
committed; the schema JSON is a local build artifact gitignored by *.json.

Co-authored-by: Cursor <cursoragent@cursor.com>
@milovate milovate force-pushed the feat/bulk-progress-display-model-v2 branch from 94a424c to 4034645 Compare June 10, 2026 08:47
@milovate milovate self-assigned this Jun 12, 2026
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.

1 participant