search

aphyr / tesser

Clojure reducers, but for parallel execution: locally and on distributed systems.
871 stars 39 forks source link

tesser.utils/update replacing clojure.core/update #12

Closed andrewsuzuki closed 7 years ago

andrewsuzuki commented 8 years ago

Getting these errors when requiring tesser.simple:

WARNING: update already refers to: #'clojure.core/update in namespace: tesser.utils, being replaced by: #'tesser.utils/update WARNING: update already refers to: #'clojure.core/update in namespace: tesser.core, being replaced by: #'tesser.utils/update WARNING: update already refers to: #'clojure.core/update in namespace: tesser.simple, being replaced by: #'tesser.utils/update

neverfox commented 8 years ago

Also bytes? now too in 1.9.0. The way this was implemented, it was hardcoded to fix 1.7.0 only.

thedavidmeister commented 8 years ago

I'm getting this with math

WARNING: update already refers to: #'clojure.core/update in namespace: tesser.utils, being replaced by: #'tesser.utils/update
WARNING: update already refers to: #'clojure.core/update in namespace: tesser.core, being replaced by: #'tesser.utils/update
WARNING: update already refers to: #'clojure.core/update in namespace: tesser.math, being replaced by: #'tesser.utils/update
WARNING: update already refers to: #'clojure.core/update in namespace: tesser.quantiles, being replaced by: #'tesser.utils/update
WARNING: update already refers to: #'clojure.core/update in namespace: clojure.math.combinatorics, being replaced by: #'clojure.math.combinatorics/update
thedavidmeister commented 8 years ago

I'm getting this in clojure 1.7.0 too.

WARNING: update already refers to: #'clojure.core/update in namespace: clojure.math.combinatorics, being replaced by: #'clojure.math.combinatorics/update

thedavidmeister commented 8 years ago

@aphyr could we just add this? http://spootnik.org/entries/2014/11/03_heads-up-for-clojure-library-writers.html

aphyr commented 8 years ago

1.7's been out long enough. We can probably just drop update.

ktsujister commented 6 years ago

Hi, I was just wondering if you can make release with this change. (with #20)