Documentation of subset is misleading

[m93a]

In the documentation for the subset function, it says that:

Parameter index — An index containing ranges for each dimension

That is quite misleading, because the function expects a list of indices, not a range. Compare:

import { subset, index, range } from 'mathjs'
const M = [[1],[2],[3]]

subset(M, index([0,2],0)) // → [[1],[3]]
subset(M, index([0,1,2],0)) // → [[1],[2],[3]]
subset(M, index(range(0,2),0)) // → [[1],[2],[3]]

[arulandu]

@m93a Can I work on this issue as my first contribution to this repository?

[josdejong]

Thanks for your offer @Claeb101 , would be great if you could improve the docs 👍

[josdejong]

Docs are improved in v10.3.0