odoo/o-spreadsheet#8911

Created by BI, Lucas Lefèvre (lul)

label

odoo:master-faster-visitMatchingRanges-lul

head

391f58fa4b48a887b1f1aa4307b4b97abf6f4f45

Master faster visit matching ranges lul

Description:

Bunch tool

cells imported in
  6ed4851d7:  Mean: 2650.00 ms, StdErr: 6.15 ms
  1e2fdf985:  Mean: 2643.66 ms, StdErr: 5.65 ms → ⚫ (vs prev: -0%, Δ=-6.34 ms, combined StdErr=8.35 ms, |Δ|/SE=-0.8x; n=20)
  0ac99c06d:  Mean: 2647.70 ms, StdErr: 4.96 ms → ⚫ (vs prev: +0%, vs baseline: -0%, Δ=+4.03 ms, combined StdErr=7.52 ms, |Δ|/SE=0.5x; n=20)

evaluate all cells
  6ed4851d7:  Mean: 2381.09 ms, StdErr: 9.10 ms
  1e2fdf985:  Mean: 2025.87 ms, StdErr: 7.22 ms → 🟢 (vs prev: -15%, Δ=-355.23 ms, combined StdErr=11.61 ms, |Δ|/SE=-30.6x; n=20)
  0ac99c06d:  Mean: 1902.54 ms, StdErr: 5.89 ms → 🟢 (vs prev: -6%, vs baseline: -20%, Δ=-123.33 ms, combined StdErr=9.32 ms, |Δ|/SE=-13.2x; n=20)

Model created in
  6ed4851d7:  Mean: 5886.37 ms, StdErr: 10.83 ms
  1e2fdf985:  Mean: 5519.48 ms, StdErr: 8.33 ms  → 🟢 (vs prev: -6%, Δ=-366.89 ms, combined StdErr=13.66 ms, |Δ|/SE=-26.9x; n=20)
  0ac99c06d:  Mean: 5398.08 ms, StdErr: 7.92 ms  → 🟢 (vs prev: -2%, vs baseline: -8%, Δ=-121.40 ms, combined StdErr=11.49 ms, |Δ|/SE=-10.6x; n=20)


Legend:
  ⚫: no measurable change |Δ|/SE < 2
  🔴: slower Δ > 0 and |Δ|/SE >= 2
  🟢: faster Δ < 0 and |Δ|/SE >= 2

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