-
Notifications
You must be signed in to change notification settings - Fork 27
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
add single precision (float32) mode #70
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great stuff! Thank you for implementing it, changes look good apart from some minor interface & variable naming issues.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great stuff! Thank you for implementing it, changes look good apart from some minor interface & variable naming issues.
64d6ef6
to
1102962
Compare
Thank you for your insightful review, and I have made the changes accordingly. If you have any other suggestions, please feel free to let me know. |
LGTM, thanks for the PR @zjzjwang! Will try to do a release on the weekend |
Closes #69
mostly copy from this branch vecterize