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

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

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

Comments

@hyperkai
Copy link

hyperkai commented Jan 24, 2025

📚 The doc issue

sigma argument of ElasticTransform() doesn't like negative values as I show in this issue.

And, setting 0 and -100 to sigma argument of ElasticTransform() gets the same 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_images(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_images(data=my_data, main_title="sigma0_data", s=0) # Here
show_images(data=my_data, main_title="sigma-100", s=-100) # Here

Image

Image

Suggest a potential alternative/fix

So, sigma 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