Congrats Alan, arrivo un po’ tardi 😁

Non sono molto esperto di XGBoost ma forse potresti postare su #data-science se ho capito bene di che si tratta

Questa cascata di map https://gitlab.com/alanmarazzi/clj-boost/blob/master/src/clj_boost/core.clj#L214 credo possa essere sostituita con base re-seq, tipo:

(for [s (vec res)
          :let [kv (re-seq #"(\S+):([0-9\.]+)" s)]]
      (reduce (fn [m [_ k v]]
                (assoc m (keyword k) (Double. v))) {} kv))

Hey, grazie mille!! Sì, stavo pensando anche io che fosse bruttino, ma pensavo ad un transducer. Anche con re-seq non è male

Sì poi spammerò anche #data-science è che prima volevo capire se riuscivo a fare la build di cljdoc, ma xgboost dà problemi a quanto pare 😅

grande complimenti!!

@UCZA8KXRU THX!

alcune cose: 1. La ns form all’inizio dovrebbe contenere tutti i require. 2. Meglio require [x :refer [y]] di use (che non mi dice che simboli vengono importati nel ns corrente) 3. Un for con dentro un map e’ equivalente al solo for aggiungendo il relativo parametro nei bindings. 4. Se possibile meglio evitare def con IO perche’ vengono eseguite a compile time. Preferisco defn senza args in generale. 5. esiste una keyword :let per for che toglie il bisogno di avere una let form dentro il for. 6. linea 54: C’e’ un do di troppo

grandissimo! ottima code review 🙂

ti devo una birra se mai venissi a berlino 😉

prima o poi 🙂