Skip to content

Live tube visualization#207

Merged
sneakers-the-rat merged 10 commits into
mainfrom
live-visualization
May 11, 2026
Merged

Live tube visualization#207
sneakers-the-rat merged 10 commits into
mainfrom
live-visualization

Conversation

@sneakers-the-rat
Copy link
Copy Markdown
Collaborator

@sneakers-the-rat sneakers-the-rat commented May 1, 2026

a new cli command noob view my-tube that displays your tube as you write it!

noob_live_editing.mp4

this is the "certified dumbass way to do it" where i am polling with a websocket from inline JS rather than building that into the react component, that'll come next. this is the minimal implementation that just makes it exist, no tests yet becoz not even sure what of this would need to be tested, i guess that the websocket stream works, but i'm gonna hold off on that because we don't have any js tests and that's sorta the real thing we need and that will require a bit of infra. i can port over some of the testing stuff from sciop here because it actually required a bit of doing to get that off the ground.

so what's changed here

  • the package build process now builds the JS stuff and bundles it into the source dist. the wheel is built from the sdist so it ends up in there too. i am a little not thrilled at the size of the bundles it ends up producing, but that is sort of 'how it goes' with bundling js. we want to be able to use this with just a pip install. eventually we might want to take advantage of our monorepo lifestyle and make this into a separate package.
  • cleaned up the vite configs so all the common stuff is in a base config
  • added vite configs for building into the package
  • moved the logic of finding the config file into a separate class method because that need comes up a lot and is also something we need for mio
  • added gui optional dependency group to hold these deps, the gui module fails to load if they are absent
  • and then ya the cli command

give it a whirl, to get it going you need to run the js build command which is specified in the js package npm run watch:package


📚 Documentation preview 📚: https://noob--207.org.readthedocs.build/en/207/

@raymondwjang
Copy link
Copy Markdown
Collaborator

Starting to read through, rest assured it will be done by eod tomorrow!

@raymondwjang raymondwjang self-requested a review May 5, 2026 09:27
@sneakers-the-rat
Copy link
Copy Markdown
Collaborator Author

Oops sorry for leaving CI a mess

Comment thread docs/conf.py
# autodoc-pydantic
autodoc_pydantic_model_show_json_error_strategy = "coerce"
autodoc_pydantic_model_show_json = True
autodoc_pydantic_model_show_json = False
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

it looks like autodoc-pydantic broken something upstream where it no longer handles generic types correctly, had to do this in mio too. package might be unmaintained

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 5, 2026

Merging this PR will not alter performance

✅ 8 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing live-visualization (73cb4c9) with main (f2ef961)

Open in CodSpeed

Footnotes

  1. 8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@raymondwjang
Copy link
Copy Markdown
Collaborator

ah crap i reviewed the wrong pr lol

@raymondwjang
Copy link
Copy Markdown
Collaborator

will get to this within a couple days sorry

@sneakers-the-rat sneakers-the-rat mentioned this pull request May 6, 2026
@sneakers-the-rat
Copy link
Copy Markdown
Collaborator Author

discussed offline a bit, raymond has a new job now!

@sneakers-the-rat sneakers-the-rat merged commit 6b4e337 into main May 11, 2026
37 checks passed
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.

2 participants