Sets should exist #69

New Issue

scott · 2025-07-02T17:48:17Z

scott commented

2025-07-02 17:48:17 +00:00

Sets are a useful datatype! And maybe even should be what we use to get keyboard events.

Value needs to be hashable (b/c f64 is not hashable)

Sets are a useful datatype! And maybe even should be what we use to get keyboard events. * [ ] `Value` needs to be hashable (b/c `f64` is not hashable)

scott added the

feature

label 2025-07-02 17:48:24 +00:00

scott added this to the Expanded: CC2 milestone 2025-07-02 17:48:27 +00:00

scott modified the milestone from Expanded: CC2 to Post-CC2

2025-07-06 05:45:02 +00:00

scott commented

2025-07-06 05:46:33 +00:00

Moving this to post-CC. This involves a more subtle problem than I had anticipated: RefCells (and imbl::HashMaps, for some reason) are not Hash. I can get behind the idea that Ludus boxes shouldn't go in data structures, maybe, but dicts most certainly should be able to. So we are without sets until I figure out how to implement Hash for Value.

Moving this to post-CC. This involves a more subtle problem than I had anticipated: `RefCell`s (and `imbl::HashMap`s, for some reason) are not `Hash`. I can get behind the idea that Ludus `box`es shouldn't go in data structures, maybe, but `dict`s most certainly should be able to. So we are without sets until I figure out how to implement `Hash` for `Value`.

scott added a new dependency 2025-07-07 04:48:47 +00:00

#90 `Value` should be `Hash`

Sign in to join this conversation.