Skip to content

Added remapping to ClimaCorePlots#2361

Open
ray-chew wants to merge 1 commit intomainfrom
rc/ccplots
Open

Added remapping to ClimaCorePlots#2361
ray-chew wants to merge 1 commit intomainfrom
rc/ccplots

Conversation

@ray-chew
Copy link
Copy Markdown
Member

@ray-chew ray-chew commented Aug 1, 2025

This feature might be useful for debugging in an infiltrator session.

Might be useful for debugging in an infiltrator session
xguide --> "panel x1"
yguide --> "panel x2"
seriescolor --> :balance
# @Main.infiltrate
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# @Main.infiltrate

# @Main.infiltrate

(unfolded_panels)
# hcoords = Remapping.default_target_hcoords(axes(field))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# hcoords = Remapping.default_target_hcoords(axes(field))

seriescolor --> :balance
title --> "Lat-Lon Remapped Field ($nlat × $nlon)"

return (longpts, latpts, transpose(remapped))
Copy link
Copy Markdown
Member

@akshaysridhar akshaysridhar Aug 5, 2025

Choose a reason for hiding this comment

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

Could you please include an example of the output from your suggested changes and differentiate it from the default "unfolded panel" output in the docs for ClimaCorePlots (I realise the current doc section for Libraries > ClimaCorePlots.jl is empty..) . Thanks @ray-chew, this way we can visually inspect the different options available here. It seems the current tests https://github.com/CliMA/ClimaCore.jl/blob/main/lib/ClimaCorePlots/test/runtests.jl rely heavily on png generation with isfile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ClimaCorePlots enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants