Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@rgieseke
Copy link
Contributor

Error was

/layercake/src/_modules/arrowUtils.js:62:33
Error: Expected 1-2 arguments, but got 3. 
            data = data.map((d, i) => {
                        return [xValue.call(data, d, i), yValue.call(data, d, i)];
                });

/layercake/src/_modules/arrowUtils.js:62:58
Error: Expected 1-2 arguments, but got 3. 
            data = data.map((d, i) => {
                        return [xValue.call(data, d, i), yValue.call(data, d, i)];
                });

I simplified the expression to directly call xValue/yValue.

@mhkeller
Copy link
Owner

I don't know why that was so over-engineered! Looks like that code hadn't been touched in four years and likely longer since that's just when I ported this code from the Svelte 3 site to this repo.

@mhkeller mhkeller merged commit 7f2eb0a into mhkeller:main Aug 19, 2025
6 checks passed
@rgieseke
Copy link
Contributor Author

Yea, Copilot had something that it might be useful if the this context would change or something, maybe it came from such a use-case.

@rgieseke rgieseke deleted the simplify-arrow-utils branch August 19, 2025 16:25
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