Maxim Zaks
Maxim Zaks
I implemented a dict alternative, which uses a pattern I described in my proposal. https://github.com/mzaks/compact-dict/blob/main/test_generic_dict.mojo
It’s compile time evaluation of a value. How about: ceval It is short, novel and IMHO spells out nicely. In my talk about Mojo alias keyword is the first thing...
@JoeLoser I don't think the bug is uni-code related, as if the `,` char is removed from the string the dict does not crash, hence it feels to me that...
@JoeLoser the bug is back in Mojo 24.1.1, but only reproduce able on Intel i7. On Apple M1, everything works fine.
Hi @JoeLoser I figured out the issue. It’s because of the random probing implemented here https://github.com/modularml/mojo/blob/457378c3c5bafa287d3fc6be3a913e71e2233231/stdlib/src/collections/dict.mojo#L678 For the test case I provided the probing needs 11 rounds to find an...
@bethebunny I created a PR with a fix, which includes a hash function independent unit test. I fixed Bug 3 by insisting on `perturb` being a UInt64. I fixed Bug...
Actually if the sorting algorithm should land in `builtin/sort.mojo` than there is no need to wait. I actually started on it, but it is a bit unfortunate that we have...
Yeah this repo will be deprecated and https://github.com/mzaks/Entitas-Lang should be the new home. I will start working on it next week.
Seems like compile time evaluation is quite volatile right now. I tried something very simple and it still didn't work: ``` from collections import Optional struct Dict[T: CollectionElement](Sized): var keys:...
Minimal example to reproduce the issue: https://github.com/mzaks/mojo/tree/feature/minimal-example-of-test-crash-for-new-hasher