Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi raftopyannis,
First, thanks for creating jsirontable, it was exactly the kind of library I was looking for in terms of table creation from json data.
But one thing I was after was the ability to create columns and rows from the data itself - a "2d" table.
I see there's been no commits to jsirontable (or any other activity on your github account) for 8 months, but I'm hoping you are still out there somewhere and get a notification of this PR.
Please accept a pull request to add 2d table capability. I've added an example to the example page which creates a table of the number of tracks each artist has at each unique BPM value (just something contrived to use the facilities with the existing data).
Hopefully you like it, I've tried to keep the implementation as similar to existing code as possible, resisted any urge to refactor the code in any way, and make the number of changes as small and self-contained as possible to ease PR. The changes should also mean that no existing functionality should be affected.
If you are generally supportive, I'd love to hear back and then I'll spend some more time testing, and implementing any suggestions you may have.
Regards