Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

alpha argument of ElasticTransform() should completely avoid negative values, giving error and the doc should have the explanation. #8880

Open
hyperkai opened this issue Jan 24, 2025 · 0 comments

Comments

@hyperkai
Copy link

📚 The doc issue

Setting 1000 and -1000 to alpha argument of ElasticTransform() gets the same kind of results as shown below:

from torchvision.datasets import OxfordIIITPet
from torchvision.transforms.v2 import ElasticTransform

my_data = OxfordIIITPet(
    root="data"
)

import matplotlib.pyplot as plt

def show_images2(data, main_title=None, a=50, s=5, f=0):
    plt.figure(figsize=(10, 5))
    plt.suptitle(t=main_title, y=0.8, fontsize=14)
    for i, (im, _) in zip(range(1, 6), data):
        plt.subplot(1, 5, i)
        et = ElasticTransform(alpha=a, sigma=s, fill=f) # Here
        plt.imshow(X=et(im)) # Here
        plt.xticks(ticks=[])
        plt.yticks(ticks=[])
    plt.tight_layout()
    plt.show()

show_images2(data=my_data, main_title="alpha1000_data", a=1000) # Here
show_images2(data=my_data, main_title="alpha-1000_data", a=-1000) # Here

Image

Image

Suggest a potential alternative/fix

So, alphe argument should completely avoid negative values, giving error and the doc should have the explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant