Skip to content

Urban Mental Health report#2449

Open
claire-simpson wants to merge 15 commits intonatcap:feature/urban-health-modelfrom
claire-simpson:feature/2419-umh-report
Open

Urban Mental Health report#2449
claire-simpson wants to merge 15 commits intonatcap:feature/urban-health-modelfrom
claire-simpson:feature/2419-umh-report

Conversation

@claire-simpson
Copy link
Copy Markdown
Contributor

Description

Fixes #2419

Checklist

  • Updated HISTORY.rst and link to any relevant issue (if these changes are user-facing)
  • Updated the user's guide (if needed)
  • Tested the Workbench UI (if relevant)

@claire-simpson claire-simpson changed the title Feature/2419 umh report Urban Mental Health report Mar 20, 2026
else:
mappable = ax.imshow(arr, cmap=cmap, **imshow_kwargs)
fig.colorbar(mappable, ax=ax, **colorbar_kwargs)
config.plot_on_axis(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Wanted an extension hook without changing plot_raster_list or duplicating too much code, but curious what you think of this. It also doesn't touch the nominal plots, so maybe there is a better design here? Curious what you think.

NEAR_ZERO_RANGE = (-0.01, 0.01)

@dataclass(config=ConfigDict(arbitrary_types_allowed=True))
class SpecialRangeRasterPlotConfig(RasterPlotConfig):
Copy link
Copy Markdown
Contributor Author

@claire-simpson claire-simpson Apr 3, 2026

Choose a reason for hiding this comment

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

I stuck this class here to avoid cluttering raster_utils, as I think this is a somewhat niche use case that doesn't necessarily need to be available to other reporter modules. Perhaps there is a better location for it though?

@claire-simpson claire-simpson marked this pull request as ready for review April 3, 2026 00:53
@davemfish davemfish requested a review from emilyanndavis April 3, 2026 12:10
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

Successfully merging this pull request may close these issues.

1 participant