bug : packager generates a solution which goes outside the bounds of the container.

[tyrcho]

as shown in https://github.com/skjolber/3d-bin-container-packing/pull/452/files the packager sometimes generates a solution which goes outside the bounds of the container.

I've provided a new test method to help check the results.

Note : it would be quite helpful to provide a method which generates this "packaging" as a property of the result container !

packaging [26, 62, 63] ([2[0x0x0 31x26x20], 1[32x0x0 63x23x18], 3[0x0x21 33x23x41], 4[34x0x21 63x22x39], 5[0x0x42 29x24x62]])
 should fit into container [44, 56, 70]

[skjolber]

@tyrcho thanks for the bug report, I'll release a fix tomorrow.

[skjolber]

Note : it would be quite helpful to provide a method which generates this "packaging" as a property of the result container !

tyrcho Can you elaborate on this?

[tyrcho]

In most cases, I need to know what is the "occupied space" in the container to give an idea to the carriers of what they will handle. So a method which would compute this volume (as 3 integers) on the Container class would be useful !

…

On Sat, Feb 26, 2022 at 3:52 PM Thomas Skjølberg ***@***.***> wrote: Note : it would be quite helpful to provide a method which generates this "packaging" as a property of the result container ! tyrcho <https://github.com/tyrcho> Can you elaborate on this? — Reply to this email directly, view it on GitHub <#453 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFK6ZD7DR2U47PC62TMDRTU5DSKRANCNFSM5PKYNWDA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[skjolber]

Fixed in latest release. Added "occupied space" as #457.