-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
Packaging issue #227
Comments
Sounds like you might have found an edge case where the algorithm picks a wrong orientation. I hate these 😅 |
@dvdoug thanks for the quick response. Is there a way to fix it? Are you planning to do so at an early date? |
Hi @kolevich There is no perfect box-packing algorithm in existence, so like all such algorithms BoxPacker is essentially a set of heuristics which due to their very nature will sometimes pick the non-optimal* result. It's usually fairly easy for me to track down which heuristic causes any particular sub-optimal result, the difficult part is coming up with a new, tweaked version of it that works better for the particular scenario at hand without regressing other cases in the test suite....
|
I've had a look at this, sadly haven't found a revised heuristic yet that works yet without regressing more cases than it improves |
Hello,
We have an issue with BoxPacker when trying to pack 11 items in a single box.
item:
box 1:
We are pretty sure it's a bug because a) our end-user states it works in a real-life b) it works fine when we split this box into 2 and trying to pack 3 and 8 items in there, like this:
box 2:
box 3:
Would appreciate it if you check this and give your comments.
The text was updated successfully, but these errors were encountered: