-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
Optimizer error #590
Comments
Happy to look into this, do you have an illustration that shows how the 29 fit in? |
Hi! |
Hi @dvdoug , do you think that modifying the code can be done by me or does it require further work? |
I did take a look and the algorithm tweak I thought might fix it didn't, but did help with some other cases. I do accept PRs so if you want to have a go, please do feel free |
I have this box:
420x420x260
And this item:
167x132x55
In another algorithm it tells me that there can be 29 items in the box (and it is actually true), but in this algorithm only 28.
The text was updated successfully, but these errors were encountered: