uncomplicate / neanderthal

Fast Clojure Matrix Library
http://neanderthal.uncomplicate.org
Eclipse Public License 1.0
1.06k stars 56 forks source link

Conflict `sv` in `native` and `linalg` module #131

Closed yangchenyun closed 1 year ago

yangchenyun commented 1 year ago

While following the tutorial, there is an error to import linalg with native module. https://dragan.rocks/articles/17/Clojure-Linear-Algebra-Refresher-Vector-Spaces

Not sure whether it is safe to only mix the use sv from linalg on native constructors.

blueberry commented 1 year ago

These are two different functions from two different namespaces.

yangchenyun commented 1 year ago

This is an issue reported for the example usage in tutorial.

Steve Yang 杨晨昀

On Tue, Apr 18, 2023 at 12:18 AM Dragan Djuric @.***> wrote:

Closed #131 https://github.com/uncomplicate/neanderthal/issues/131 as completed.

— Reply to this email directly, view it on GitHub https://github.com/uncomplicate/neanderthal/issues/131#event-9032256901, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAH6XS2BDCR3PIE7YH77LXDXBY52VANCNFSM6AAAAAAXCCFPNU . You are receiving this because you authored the thread.Message ID: @.***>

blueberry commented 1 year ago

Thank you for reporting that. Unfortunately, I don't have resources to edit old blog articles outside of Neanderthal's code base in case some feature change introduces such slight inconsistencies in the text of the tutorial.