What do you mean by "aren't running"?

reframe-10x doesn't show that they've run, and the icon of the cloud doesn't change after I click on it, despite the app db being updated

How do you use cloud? What is ::events/toggle-cloud?

cloud is a true or false value that represents whether it's cloudy in a particular spot or not. I'll need to send toggle-cloud later, so sorry but I'm not at my computer

It basically does (update clouds cloudId not)

I mean, how do you use the cloud component? Not just its direct usage, but also its parent component.

I have a clouds component that does (map cloud (range 50))

Could that be it? The map makes a seq of clouds, but are they actually components?

Hard to say without the actual code.

And they are not components in this case - just functions. Sometimes it's ok, but not in this case as updating a single cloud will rerender the whole parent component.

Ok thanks for looking over it with me. Sorry, I'm away from my computer or I would share the code in full

That's fine, I'm in no hurry. :)

This is my code: https://github.com/danielmartincraig/pick-things-up

I got the issue resolved by doing it this way as opposed to doing (map cloud (range 50)):

(defn cloud [cloudId]
  (let [cloud (re-frame/subscribe [::subs/cloud cloudId])]
    [re-com/md-icon-button
     :md-icon-name (if @cloud "zmdi-cloud" "zmdi-cloud-outline")
     :on-click (fn [] (re-frame/dispatch [::events/toggle-cloud cloudId]))]))

(defn clouds []
  [re-com/h-box
   :src      (at)
   :width    "100%"
   :children [(for [cloudId (range 50)]
                [cloud cloudId])]])

for is also lazy, so you probably have a bunch of warnings in the JS console about missing key property. You can do this instead:

(mapv (fn [cloudId] [cloud cloudId]) (range 50))

Since mapv returns a vector, it's not lazy anymore.

Oh that's a great tip, thanks!