Skip to content
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

Pack locations #8

Open
luenib opened this issue Jun 23, 2017 · 4 comments
Open

Pack locations #8

luenib opened this issue Jun 23, 2017 · 4 comments
Labels

Comments

@luenib
Copy link

luenib commented Jun 23, 2017

I have tested your class and it works great every time. Very good work.

I actually have a questions: where do I get the location of the boxes inside the container? I have manually placed my boxes in the contained guided by the results provided by your class. I know it works, but I haven't been able to find the line of code that sets the coordinates for each box.

Thanks a lot.

@mdeboer
Copy link
Member

mdeboer commented Jun 23, 2017

Thanks @luenib, appreciated!

To be fair, it's been years since I put this together. I had a quick glance but it seems it needs some work to get the coordinates (eg. exact layout). I will play around with this and see if I can generate a graphical "map" of the boxes in the container. This would not only make things easier to debug but also would help you pack I suppose.

@luenib
Copy link
Author

luenib commented Jun 23, 2017

Thanks for the quick response.

I'm using isomer.js for the graphic representation of the packs inside the container. Nothing fancy but it works very well. You can take a look at it here: https://jsfiddle.net/luenib/6op8ockr/

The packs are hard coded by I think I can feed an array with the packs (and coordinates) returned by your class.

Thanks again.

@mdeboer
Copy link
Member

mdeboer commented Jun 24, 2017

@luenib awesome, that saves me so much work and probably makes debugging a lot easier (at least for me lol). I'll see with I can do and will try to make it work with isomer. Unfortunately can't give a time estimate but I'll keep you posted.

@luenib
Copy link
Author

luenib commented Jun 25, 2017

Thank you. All the effort you have put in this work and all the future effort are greatly appreciated.

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

No branches or pull requests

2 participants