handsontable / handsontable/hyperformula
Support dynamic parameters for OFFSET function
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 2.8k
- Forks
- 171
- Avg merge
- 1d 22h
- Merged PRs (30d)
- 7
Description
Description
The goal of this task is to remove all limitations of the OFFSET function:
- dynamic parameters e.g.
=OFFSET(A1, F16, F17) - passing range as first parameter e.g.
=OFFSET(A1:B1, 0, 0) - printing OFFSET as a result of
getCellFormula - other limitiations that migh be discovered
Implementation
OFFSET function is hardcoded in HF parser but it should be treated as a regular function. Big refactor is needed for that.
Note
Discussed during HyperFormula Summit: https://docs.google.com/document/d/1xwNKiA4O-gRLNFlw4mu03S3XJXhsR3JzgVeC772ooKc/edit (internal doc)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the OFFSET implementation in the HyperFormula parser and how getCellFormula handles it. The work is done when OFFSET supports dynamic parameters, ranges as its first parameter, and is printed correctly by getCellFormula, while any related limitations discovered during the refactor are addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend, compilers
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100