-
Notifications
You must be signed in to change notification settings - Fork 183
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
Fix shading #317
Fix shading #317
Conversation
Thank you for this work. The fix looks good to me and makes sense to me. It did not occur to me that a |
Just signed CLA. I will skip ACKNOWLEDGMENTS file for now. |
Alright. (We may still record your contribution in ACKNOWLEDGMENTS eventually by means of #308, I hope that's okay for you.) |
@juniartisu Does this fix work for you as well?/Can you confirm it fixes #316 in your environment? |
@Stewori I will try out the fix and do testing on it and will put update here once it is done. |
@Stewori The fix is good. I am not seeing any issue so far. Than you @lifebarier for investigating and fixing it. |
@lifebarier could you do me a favor and add the line |
@lifebarier @Stewori Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 9 |
@abhi-rck |
Seems to fix shading issue #310 #316