Thomas Germer

Results 140 comments of Thomas Germer

The requirements in `requirements.txt` are already quite broad. I hope they are also broad enough for `install_requires`. Did you find any incompatibility issues? If not, I'd rather keep things as...

Numba version `0.49.0` had been disabled because that specific version contained a bug which was triggered by our KDTree code. Version `0.48.0` did not have that bug. More details in...

> We could specify `numba>=0.48.0,!=0.49.0` That is probably more conservative than necessary. It is likely that PyMatting will also work with earlier versions, although I do not have a list....

I do not understand your question. 1. PyMatting uses images form http://alphamatting.com/datasets.php for testing if PyMatting works correctly. We can not include those images directly in this GitHub repository because...

> Can you show us the kernels you used for cuda_matvec / opencl_matvec ? You can find the code for the other implementations by clicking on the word "here" in...

> I would argue that forcing a block dim of 8 is never a good idea, and this isn't a typical implementation of mat @ vec. We have tested a...

This is Python's syntax for formatting strings. You can read more about it in the [Documentation](https://docs.python.org/3/tutorial/inputoutput.html). In this example, the integer `epoch` will be left-padded with zeros to a length...

> Thanks so much for reaching out. I fan-reacted a bit when I saw that No need, I am just some guy 😄 > so cool this little project got...

> Does the upper limit on `number_of_sorts` _need_ to be five? Could it be 5,000 instead? It could be up to 8. https://github.com/deepmind/alphadev/blob/1a6eac1544c1075ef27814f09f9223cb84537c59/sort_functions_test.cc#L280 I guess higher numbers would be more...

> I changed your code to pre-generate a large array and then sort all the data at once, which gave a completely different result (AlphaDev is now significantly faster). I...