Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

cpsievert

cpsievert pushed 1 commit to chat-markdown-stream posit-dev/shinychat

View on GitHub

cpsievert pushed 1 commit to chat-markdown-stream posit-dev/shinychat
  • py-shiny component is merged now ff013b7

View on GitHub

cpsievert pushed 1 commit to main posit-dev/py-shiny
  • Add `ui.MarkdownStream` (#1782) a781305

View on GitHub

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

View on GitHub

cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny

View on GitHub

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()...

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny

View on GitHub

cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
  • Remove the .ui() entirely from shiny.ui.MarkdownStream 95f271a

View on GitHub

cpsievert pushed 2 commits to markdown-stream-component posit-dev/py-shiny
  • Go back to output_markdown_stream() a99b3a5
  • Update API reference 2496cc0

View on GitHub

cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
  • Take another pass at polishing docstrings 96d6db0

View on GitHub

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...

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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...

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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)

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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)

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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...

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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)

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

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)

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

cpsievert pushed 2 commits to markdown-stream-component posit-dev/py-shiny
  • Take a pass at addressing documentation feedback d091ebd
  • Address JS feedback add7b2f

View on GitHub

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`?

View on GitHub

cpsievert created a review on a pull request on posit-dev/py-shiny

View on GitHub

cpsievert pushed 1 commit to markdown-stream-component posit-dev/py-shiny
  • Missed a couple renamings 37257ec

View on GitHub

Load more