You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Have you considered submitting this for inclusion in Crystal's standard library before 1.0? I found your module after missing a core bag/multiset type like Bag in Perl 6 or Counter in Python.
The text was updated successfully, but these errors were encountered:
Notably, Perl 6's Bag is actually immutable, while BagHash is mutable (just like its Set and SetHash). I wonder whether it's worth having both versions in Crystal.
Though I like this shard, adding new data structures to the stdlib is quite sensitive. We need to make sure that any change we make to the API of a data structure is consistent across all of them, and adding new ones will amplify the workload on each of those steps. This is especially true with parallelism being the next big step.
I'd suggest adding this to the standard library actually after 1.0, once we have a stable API that we like, and we can ensure that any new data structures we add conform to it. For now, I think that having this structure as a shard and listed in awesome-crystal works fine.
Have you considered submitting this for inclusion in Crystal's standard library before 1.0? I found your module after missing a core bag/multiset type like Bag in Perl 6 or Counter in Python.
The text was updated successfully, but these errors were encountered: