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

Add saving plot as tiff #591

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Add saving plot as tiff #591

merged 1 commit into from
Oct 22, 2024

Conversation

timtmok
Copy link
Contributor

@timtmok timtmok commented Oct 16, 2024

Address posit-dev/positron#4690

PR for plot comm update: posit-dev/positron#5044

Regenerate the comms and add TIFF saving support.

Comment on lines +469 to +473
/// This event is used to signal that the stored messages the front-end
/// replays when constructing multi-output plots should be reset. This
/// happens for things like a holoviews extension being changed.
#[serde(rename = "clear_webview_preloads")]
ClearWebviewPreloads,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there some reason this is getting added in now alongside this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this was from a previous update to the comm definition used to generate code, and it showed up now?
https://github.com/posit-dev/positron/blame/a0e53f8b220ebdcc815cf144ea141313954924d9/positron/comms/ui-frontend-openrpc.json#L450

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this came up after regenerating so I think it needed an update.

@timtmok timtmok merged commit 0ebe049 into main Oct 22, 2024
6 checks passed
@timtmok timtmok deleted the feature/plot-tiff-support branch October 22, 2024 13:23
@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants