accimage not installed in CI skipping a number of tests

[prabhat00155]

Context: Internal tests are failing on test_transforms.py::TestAccImage::test_accimage_crop with the following error:

AttributeError: 'accimage.Image' object has no attribute 'getbands'

It points to the following function:

def get_dimensions(img: Any) -> List[int]:
        if _is_pil_image(img):
>           channels = len(img.getbands())
E           AttributeError: 'accimage.Image' object has no attribute 'getbands'

from functional_pil.py which was added in this #5487.

cc @pmeier

[pmeier]

Could you link a CI run so I can see where we need to install it?

[prabhat00155]

@pmeier Added some context to this issue. This error was not caught by our CI. Also, looks like conda install -c conda-forge accimage from https://github.com/pytorch/accimage#installation doesn't work, so not sure how we would add this to CI.

[pmeier]

I'll have a look. Maybe this a candidate fro deprecation / removal. It seems we are only using it for image loading. For all transformations it needs to be converted into a tensor with ToTensor(), which we also want to deprecate AFAIK.

Could you check internal usages of accimage together with torchvision?

[NicolasHug]

This comment from @fmassa seems to suggest that we actually don't support accimage anymore

We don't have Travis (nor accimage) tests anymore, closing this.

That being said, we still claim on our README that accimage is part of the supported backends. Let's wait for @fmassa 's insights before we decide to fully remove it, or actually tests against it.