Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom Displays #1267

Open
Baidicoot opened this issue Feb 7, 2025 · 1 comment
Open

Custom Displays #1267

Baidicoot opened this issue Feb 7, 2025 · 1 comment

Comments

@Baidicoot
Copy link

I was wondering if there was any play to update the 'active display' logic to support custom display types (i.e. not one of "full", "plain", "rich", "conversation" or "none"). The functionality for this seems quite easy to implement, so some fork could presumably do it pretty easily. Mostly I see three ways you might go about this:

  • Keep the current setup where you have globals for _display_type etc, and update the display initialization functions to read from some registry of displays.
  • Pass in the display to the evaluation as an argument.
  • Otherwise modify the output logic to stream information to a generalized "Output" class, which then decides what to do with it. Display would then subclass this.

Thanks

@jjallaire-aisi
Copy link
Collaborator

Indeed, it wouldn't be hard to make this externally loadable (although honestly the display API wasn't designed for public consumption so might need some revision / clean up). I'm curious though what the scenario for a custom display is (as perhaps it's something we should support internally anyway)

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

No branches or pull requests

2 participants