Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Wrong Rotation Checking #3

Open
miticociuck opened this issue Jan 19, 2016 · 2 comments
Open

Wrong Rotation Checking #3

miticociuck opened this issue Jan 19, 2016 · 2 comments

Comments

@miticociuck
Copy link

In several parts of the Guillotine3D.cs there is a wrong checking of dimensions
.....
else if (box.AllowedRotationsY &&
container.Width >= box.Width &&
container.Length >= box.Width &&
container.Width >= box.Length)
return true;
....

I think it should be:
container.Width >= box.Height &&

@SahibYar
Copy link
Owner

thanks for keeping an eye on the code, yeah that is a mistake there. I will shortly upload the latest commit fixing all the issues you raised.

@miticociuck
Copy link
Author

Hi Sahib,

I didn’t see any activity in the last 3 weeks, and I didn’t receive any news from You.

So I’m asking…. are You still working on this project or You gave up?

Luca

Da: Sahib Yar [mailto:[email protected]]
Inviato: mercoledì 20 gennaio 2016 16:15
A: SahibYar/3D-Bin-Packing [email protected]
Cc: Luca Breda [email protected]
Oggetto: Re: [3D-Bin-Packing] Wrong Rotation Checking (#3)

thanks for keeping an eye on the code, yeah that is a mistake there. I will shortly upload the latest commit fixing all the issues you raised.


Reply to this email directly or view it on GitHub #3 (comment) . https://github.com/notifications/beacon/AP0ynC6k6hg_cp3WQtujwoljXj2PbCq_ks5pb5wJgaJpZM4HH3xY.gif

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants