greetings! Is there a way to have reduced in areduce? For now I use usual reduce, but I have an uneasy feeling about this.

@misha Do (source areduce). You can see that it is a fairly simple macro.

It keeps going until the indices are consumed.

@mfikes are you suggesting I should just loop/recur?

Well, it sounded like you were asking whether you could get areduce to terminate early

I sort of did, yes :)

I hoped there were some combination of reduced and some higher level function/macro for reducing arrays and terminating earlier.

falling back on loop is fine, though

@misha If it makes you feel less uneasy, reduce calls array-reduce which is a loop/recur honoring reduced?

true! thanks

yeah, areduce dates back before reduce was optimized like it is today - now it doesn't need to call seq on the last arg

so, there is no point in using areduce anymore? @noisesmith

I'm not sure of that - but I suspect it

if you have performance problems, make sure you profile, then figure out what actually needs fixing, then benchmark your improvement

too often you see someone make a 70% improvement on something that takes 3% of runtime, while ignoring a simple fix that would fix the thing taking 50% of their runtime...

Yeah, if you are reducing a lot of arrays and you don't need early termination, areduce might be faster

http://blog.fikesfarm.com/posts/2017-11-18-clojurescript-performance-measurement.html if it helps... (especially with respect to testing :advanced code in different JS engines)

I am interoping heavily with paper.js right now, and just being in the "as close to js everywhere as possible" mode. Reduces amount of decisions I have to make right now (no performance concerns yet)

"as close to js as possible" is js :/

to an extent

not saying you should switch to js, just that optimizing before you know what needs to be sped up is not an ideal use of time

@misha in your case I don’t see why reduce + reduced won’t work for you

I don’t see what areduce would offer here

@dnolen it makes sense now, when I know reduce calls array-reduce under the hood, and saw code of both. thanks!

the next major release of ClojureScript is looming - it’s pretty massive

but probably the thing we’re looking most forward to is how much easier it is to use

https://github.com/clojure/clojurescript-site/blob/0dfc95afe1ef0e5dc5976c0d974f2b4c9fc369c4/content/guides/quick-start.adoc

This WIP for the new Quick Start which is significantly more straight forward than what we had before - please give it a try

you should be able to work through this, we cut a pre-release (1.10.126) so that this Quick Start can be tested

this is how people will build their first impressions so very interested in making this as compelling as possible

I’ve started a thread on http://clojureverse.org for this https://clojureverse.org/t/clojurescript-1-10-x-new-quick-start-feedback-please/1679, feel free to jump in

I have a .clj macro file, it has error. (:require-macros [panaeolus.macros :refer [definstrument]] :reload) I don't see the :reload working, is this expected? Other ways to reload a macro namespace? I'm useing node repl.

@hlolli I believe there’s still an edgecase around triggering reloads for macro nses

probably something simple, but somebody needs to look into that

@dnolen ok, then I continue to restart the repl. Not big deal 🙂

clj -m cljs.main

love it!

🙂

@bhauman great idea here was to make the browser REPL the default, and then of course I was like why don’t we just open a browser - duh! 🙂

FYI it appears clara rules does not work with 1.10.126, lots of warnings about Wrong number of args (3) passed to MapEntry and Use of undeclared Var schema.core/MapEntry in its schema.core ns. fails at runtime too. if it's just a var that was moved/renamed then i assume itll need to be fixed on their end

right probably a conflict with cljs.core/MapEntry now

for the first error, I bet it's something like this

+cljs.user=> ([:a 0] 2 :not-found)
:not-found

never mind, not that, that's only two args :/

yeah I don’t know about that anyway

but we’ll see how many people are relying on weird tricks

hi all

i'm trying to use the new npm-deps feature, i'm using cljs 1.10.126

this is in my project.clj: :npm-deps {:react "16.2.0" :react-router-dom "4.2.2"}

i also have install-deps set to true

when i run lein figwheel, it creates the package.json and also npm packages under node_modules

however when i would like to require the libs, this is what i get: https://gist.github.com/mengu/78f4220ef19f7892af2867635b39582c

does anyone have any idea?

Question: in clj it's easy to extend a protocol to all types of maps (ArrayMap, HashMap...) by extending it to the interface java.util.Map. That's nice because it spares you the hassle of extending it to each map type separately. As far as I can tell, I can't do the same in cljs by extending the protocol to cljs.core/IMap, because there's not a reified interface for cljs protocols. Is that correct? And is there any other way to do the equivalent in cljs?

Has anyone come across errors like Caused by: java.lang.RuntimeException: Unable to find static field: STRING_KEY in class com.google.javascript.rhino.Token ?