Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
cpsievert pushed 1 commit to chat-markdown-stream posit-dev/shinychat
- py-shiny component is merged now ff013b7
cpsievert closed a pull request on posit-dev/py-shiny
Add `ui.MarkdownStream`
This PR extracts the streaming markdown capabilities of `ui.Chat()` into another component: `ui.MarkdownStream()`. See the new files for example usage: * `shiny/api-examples/MarkdownStream/app-e...cpsievert created a comment on a pull request on posit-dev/py-shiny
Merging so this available to others at work week. Will file issues to handle remaining feedback
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Ok, that's fair, done in [a99b3a5](https://github.com/posit-dev/py-shiny/pull/1782/commits/a99b3a5bc4a0cdbc403a897f4de08de8473a90be). It does make me wonder though -- should we change `chat_ui()...
cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
- Fix add_example usage 187817b
cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
- Remove the .ui() entirely from shiny.ui.MarkdownStream 95f271a
cpsievert pushed 2 commits to markdown-stream-component posit-dev/py-shiny
cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
- Take another pass at polishing docstrings 96d6db0
cpsievert created a review comment on a pull request on posit-dev/py-shiny
I get where you're coming from. I think it could technically work since there's not currently any important state management going on, but it makes me feel uneasy that it would handcuff future desi...
cpsievert created a review comment on a pull request on posit-dev/py-shiny
> but we should also have content_type as an argument in stream() I agree, but unfortunately it wouldn't be support that combined with `.stream(clear=False)` since you could end up with a mix of...
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Done in [d091ebd](https://github.com/posit-dev/py-shiny/pull/1782/commits/d091ebdedb97b6f222f8284c5ac05695d1942b42)
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Addressed in [d091ebd](https://github.com/posit-dev/py-shiny/pull/1782/commits/d091ebdedb97b6f222f8284c5ac05695d1942b42)
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Addressed as a note in [d091ebd](https://github.com/posit-dev/py-shiny/pull/1782/commits/d091ebdedb97b6f222f8284c5ac05695d1942b42) If you feel strongly about `.append()`, would you mind filing a...
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Done in [add7b2f](https://github.com/posit-dev/py-shiny/pull/1782/commits/add7b2f48526adc67b8340ef77c2e76e8d4ac825)
cpsievert created a review comment on a pull request on posit-dev/py-shiny
Done in [add7b2f](https://github.com/posit-dev/py-shiny/pull/1782/commits/add7b2f48526adc67b8340ef77c2e76e8d4ac825)
cpsievert pushed 2 commits to markdown-stream-component posit-dev/py-shiny
cpsievert created a review comment on a pull request on posit-dev/py-shiny
If we call it `output_markdown_stream()`, would you expect there to be a `@render.markdown_stream`?
cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
- Missed a couple renamings 37257ec