-
Notifications
You must be signed in to change notification settings - Fork 21
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
Create dense borefield function #270
Create dense borefield function #270
Conversation
Thank you @wouterpeere. For the minimum spacing |
Hi, What about implementing both, because I think the first solution is easier to handle for the user. Considering the first solution the width and length of the borefield is the multiplication of the number of boreholes times the distance. The same distance on diagonal is indeed the denser solution. Here my code as suggestion:
|
@tblanke I like the idea of having both options. I would prefer having two functions instead of the Different spacings in both directions (there is probably a better name for this function)
Dense field with equal distance along the diagonals
|
Hi @tblanke Good idea indeed! @MassimoCimmino I will implement the code you wrote above and add some tests. Best, |
@all-contributors |
I've put up a pull request to add @wouterpeere! 🎉 I've put up a pull request to add @tblanke! 🎉 |
@all-contributors |
I've put up a pull request to add @tblanke! 🎉 |
I wrote another function to create a 'dens borefield' where the boreholes are packed as densely as possible with a minimum distance B. Instead of creating a regular grid, where all the boreholes are underneath each other, this function moves every uneven row to the right, so it's y-coordinate can be lower.
This creates borefields like: