Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix get/set when shape = [] #251

Closed
wants to merge 631 commits into from
Closed

Conversation

rotu
Copy link
Contributor

@rotu rotu commented Aug 31, 2023

I think this also fixes a pretty big bug with set, where the index would not be properly set for a multi-dimensional array. Recommend adding a test.

mateogianolio and others added 30 commits December 25, 2016 21:41
This badge will show the version on CDNJS!
add CDNJS version badge in README.md
add binary operation function to vector
optimize for loop by pre-calculating size
make callback function takes original data array
mateogianolio and others added 24 commits April 18, 2023 01:35
* Add prettier formatter

* [skip ci] Bump version
* Remove non-null assertions

* 6.1.8
* Remove non-null assertions

* 6.1.8

* Split up iterators into separate files

* 6.1.9
* Remove old benchmarks and add benchmarks to .gitignore

* Add benchmarks to iterators

* Improvement/renaming to bench script

* 6.1.10
…#239)

* Remove next2d and next1d from NDIter and add test case

* 6.1.11
* Improve test for multidimensional iterator

* Improve tests for matrix multiply

* Add blas.ts tests (except gemm)

* Test unoptimized path for add()

* Test unoptimized path for dot()

* Test unoptimized path for inv()

* Test unoptimized path for multiply()
use rimraf instead of `rm -rf`. This makes the build work on Windows.

Co-authored-by: Mateo Gianolio <[email protected]>
@mateogianolio
Copy link
Owner

Sorry, something exploded with github-actions and it automatically closed your pull requests and I can't reopen them.. I'll try to fix it when I have time.

@rotu
Copy link
Contributor Author

rotu commented Sep 7, 2023

no worries! let me know when you are ready and I’ll make my PRs again

@mateogianolio
Copy link
Owner

I have reset the repository now so you can reopen the PRs whenever you are ready!

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.

9 participants