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

Error if Convolution Window is Larger than the Image #170

Open
robelgeda opened this issue Jan 2, 2023 · 1 comment
Open

Error if Convolution Window is Larger than the Image #170

robelgeda opened this issue Jan 2, 2023 · 1 comment
Labels
Bug Something isn't working Enhancement New feature or request

Comments

@robelgeda
Copy link
Contributor

If convolution window is larger than the image, there is an error about broadcasting arrays of different sizes. The two options to fixing this is are:

  1. Raise an exception when the model is declared. This is probably not good because the model can be fit to multiple image data with different sizes.
  2. Crop the window to fit image space.

It seems like option 2 is the option that makes the most sense.

@robelgeda robelgeda added the Bug Something isn't working label Jan 2, 2023
@robelgeda robelgeda added this to the v0.5.0 milestone Jan 11, 2023
@Bhavesh012
Copy link

I passed this error by keeping the size as an odd no. while making the cutouts. And my psf image size is also in odd no.s.

@robelgeda robelgeda removed this from the v0.5.0 milestone Aug 21, 2023
@robelgeda robelgeda added Enhancement New feature or request Bug Something isn't working and removed Bug Something isn't working labels Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants