Skip to content

Improve performance on generating layers view

Sarah Groff Hennigh-Palermo requested to merge 327260-memoize-group-lookup into master

What does this MR do?

This resolves a concern from !58646 (merged) about caching / memoizing the groups once they've been looked up. I looked into the performance implication of Lodash's comparison in memoize to see if it was reasonable to use that rather than tracking updates ourselves and since it uses Map, it seemed reasonable.

Screenshots (strongly suggested)

No visual updates.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related to #327260 (closed)

Merge request reports