Skip to content

[IMP] evaluation: selectively invalidate chart runtimes#7960

Open
pro-odoo wants to merge 1 commit intomasterfrom
master-dependency-chart-pro
Open

[IMP] evaluation: selectively invalidate chart runtimes#7960
pro-odoo wants to merge 1 commit intomasterfrom
master-dependency-chart-pro

Conversation

@pro-odoo
Copy link
Collaborator

Previously, any UPDATE_CELL or EVALUATE_CELLS
command would invalidate all chart runtimes,
causing every chart to be fully re-computed even
when the edited cell had no relation to any chart.

Chart runtimes are now only invalidated when the
re-evaluated cells actually overlap with the
chart's data ranges (datasets and label range).

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

review checklist

  • feature is organized in plugin, or UI components
  • support of duplicate sheet (deep copy)
  • in model/core: ranges are Range object, and can be adapted (adaptRanges)
  • in model/UI: ranges are strings (to show the user)
  • undo-able commands (uses this.history.update)
  • multiuser-able commands (has inverse commands and transformations where needed)
  • new/updated/removed commands are documented
  • exportable in excel
  • translations (_t("qmsdf %s", abc))
  • unit tested
  • clean commented code
  • track breaking changes
  • doc is rebuild (npm run doc)
  • status is correct in Odoo

@robodoo
Copy link
Collaborator

robodoo commented Feb 19, 2026

Pull request status dashboard

Previously, any UPDATE_CELL or EVALUATE_CELLS
command would invalidate all chart runtimes,
causing every chart to be fully re-computed even
when the edited cell had no relation to any chart.

Chart runtimes are now only invalidated when the
re-evaluated cells actually overlap with the
chart's data ranges (datasets and label range).
@pro-odoo pro-odoo force-pushed the master-dependency-chart-pro branch from 89e681a to db3c7fe Compare February 19, 2026 08:38
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.

2 participants