ProGamerGov
ProGamerGov
It sounds like you're attempting to maximize the image embeddings by searching text embeddings. Researchers did that here: https://distill.pub/2021/multimodal-neurons/ But people were running into issues trying to figure out how...
Another recorded set of failures: ``` =================================== FAILURES =================================== _____________ Test.test_softmax_classification_batch_zero_baseline _____________ self = def test_softmax_classification_batch_zero_baseline(self) -> None: num_in = 40 input = torch.arange(0.0, num_in * 3.0, requires_grad=True).reshape(3, num_in) baselines...
I've come across some issues: * Doc variables with the lowercase `any` may be seen as referencing the `any()` function rather than the type. So, instead of `any`, we should...
I've also noticed a major mistake in formatting code snippets in docstrings. The other Captum modules all use this incorrect format that doesn't show up on the website: * \`code_highlight\`:...
@NarineK So for Callable and Any, I am planning on using this function in the `conf.py` [here](https://github.com/pytorch/captum/blob/master/sphinx/source/conf.py) to ensure that each type is hyperlinked by intersphinx. Whether or not we...
I ended up going with the uppercase versions of 'Callable' & 'Any' as they are going to be hyperlinked to `typing.Callable` & `typing.Any`, rather than their lowercase function versions. For...
@NarineK I threw in the bool linking for future cases that may arise, but I can remove it. The list and tuple linking though does not work for every case...
@NarineK I've already changed the plural versions of the types like `ints` to `int`, and `floats` to `floats`. It's grammatically incorrect, but its the Google style Python docstring format for...
@aobo-y I would like to add that having a proper standard across Captum will make it easier to adopt any future standards. Stuff like regex and other text replacement tools...
I got the `boolean` -> `bool` and `string` -> `str` changes implemented into the PyTorch master branch: https://github.com/pytorch/pytorch/pull/82410 I also got the plural `ints` -> `int` changes implemented into the...