-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Augmentation when shift_row or shift_col is negative. #7
Comments
I'm sorry for the late reply, Let me see that and If you are not in a hurry, I'll come with a solution either tomorrow or the day after tomorrow. |
thanks for spotting this out, it was a bug in the (image) translation function. Please test the new version of the package using devtools::install_github('mlampros/OpenImageR') (it will take some time till the new version of the package is on CRAN) and let me know. |
Hi, unfortunately, I get the same result by the above code. Can you quickly copy & paste it to see if you get the same result? Assuming that you get the same result as I do, my humble guess is that this part is a problem: Lines 875 to 878 in 1ba1a99
It says |
I'm sorry, I totally forgot to change the inequality signs. I fixed it and I'll add some exception handling in the next few days. Please test it and let me know. |
Terrific. Thank you for your work! Quick check code: library(OpenImageR)
object <- matrix(0.5, nrow=3, ncol=4)
Augmentation(object, shift_rows=-1, padded_value=1)
Augmentation(object, shift_cols=-2, padded_value=1)
Augmentation(object, shift_row=-2, shift_cols=1, padded_value=1)
Augmentation(object, shift_row=-2, shift_cols=-3, padded_value=1) |
Hi, I find a weird behavior of
Augmentation
whenshift_rows
orshift_cols
is negative.So, no change in the image at all.
It seems the function discounts
shift_rows
andshift_cols
by 1 when it is negative. That is, it treats -1 as 0, -2 as -1, and so on.The text was updated successfully, but these errors were encountered: