Skip to content

feat: cap breakdown#545

Merged
zlatanpham merged 5 commits intomasterfrom
feat/cap-breakdown
Sep 9, 2025
Merged

feat: cap breakdown#545
zlatanpham merged 5 commits intomasterfrom
feat/cap-breakdown

Conversation

@R-Jim
Copy link
Collaborator

@R-Jim R-Jim commented Sep 4, 2025

What's this PR do?

  • Add breakdown for cap

@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

🤖 Automatic Formatting Applied

Files formatted: 1
Changes made:

  • ✅ Prettier formatting applied

View workflow run: View Run

Copy link
Collaborator

@monotykamary monotykamary left a comment

Choose a reason for hiding this comment

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

At a high level, I don't think it's missing anything. However, it is a lot of bullet points. Although this makes the article sparse and easily skimmable, it's quite hard to connect ideas and derive meaning from the bullet points without leaving the reader to second guess. How you choose which bullet points to convert to prose is up to you.

This breakdown needs a bit of visuals to make it easier to imagine how the technical pieces work together. You can use their demo videos as a gif or plaster the whole thing somewhere in the introduction:

result.mp4

In the technical parts, it would be really nice to have some animations/visualizations for the timelines, muxing, etc. Something like this would work: https://chatgpt.com/canvas/shared/68b96f7360e48191b624780d05b54a54

mux.mp4

@R-Jim R-Jim requested a review from monotykamary September 8, 2025 10:09
Copy link
Collaborator

@monotykamary monotykamary left a comment

Choose a reason for hiding this comment

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

FLORA-Add person left-36a7659d

@zlatanpham zlatanpham merged commit 8abacae into master Sep 9, 2025
1 check passed
@zlatanpham zlatanpham deleted the feat/cap-breakdown branch September 9, 2025 07:30
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

Comments