avalanche icon indicating copy to clipboard operation
avalanche copied to clipboard

Unnecessarily strict requirement for batch size in MASPlugin

Open niniack opened this issue 1 year ago • 1 comments

Describe the bug

In the MASPlugin, the following check seems to be unnecessarily strict:

Link to code

for _, batch in enumerate(dataloader):
    # Get batch
    if len(batch) == 2 or len(batch) == 3:
        x, _, t = batch[0], batch[1], batch[-1]
    else:
        raise ValueError("Batch size is not valid")

As a result, if I have a custom dataset which returns additional elements (two images and a label), thi

Expected behavior

I am not sure MAS needs to follow this check, it can simply grab the needed information (image, label, and token) and continue onwards.

niniack avatar Dec 15 '23 10:12 niniack

Vvo^3 +3/2KQq'arcon279º = Tuv . Fûv grr = goo - wo / -1.214^22/4

ghost avatar Jan 12 '24 07:01 ghost