should fix isse #82 and #83

[darcangelomauro]

No description provided.

[darcangelomauro]

More details in issue #82 and #83. @Misty-W could you check if it works this way?

[Misty-W]

More details in issue #82 and #83. @Misty-W could you check if it works this way?

Thanks @darcangelomauro! I tested this on Daniele's three test cases he posted in the comments on #78. There is a bit more troubleshooting to do, since none of them work as specified just yet. If I remove the cropping line on the first and third I get the final result w/o error messages. If I keep the cropping in, for the first and third cases I get

File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/density_canvas/DensityCanvas.py", line 678, in calculate_detunings
threshold_distances[i] = distances[i][-1][1] #initialize as smallest distance
IndexError: list index out of range

For the second case, both with and without cropping, I get

File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/density_canvas/Lattice.py", line 142, in poisson_disk
ref_radius = radius_density[_index_from_position(ref_point)]
IndexError: index 706 is out of bounds for axis 0 with size 79

The index that is reported out of bounds varies (I assume because of the random selection of points), but the rest of the error remains the same.

[darcangelomauro]

Added a function DensityCanvas.get_lattice() with an optional argument minimal_spacing, which defaults to None. If minimal_spacing is a number x, the function returns the lattice coordinates scaled by a factor such that the minimal distance in the coordinates is x.
The function DensityCanvas.calculate_detunings() now also takes the optional argument minimal_spacing, so that the user can make sure that the qubits will be at a reasonable distance from each other

[Misty-W]

Modified test cases 1 and 3 work with the latest fix.