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

Script for automated workflow #78

Merged
merged 44 commits into from
Jul 30, 2024
Merged

Conversation

Misty-W
Copy link
Contributor

@Misty-W Misty-W commented Jun 5, 2024

Fixes #75 and #77.

Create a script to

  • Load input files
  • Call functions to slice the density and find the locations of water molecules
  • Save the output

@Misty-W Misty-W linked an issue Jun 5, 2024 that may be closed by this pull request
@Misty-W Misty-W force-pushed the 75-script-for-automated-workflow branch from cb156ef to d72c039 Compare June 11, 2024 20:36
@Misty-W Misty-W marked this pull request as ready for review June 11, 2024 20:36
@Misty-W Misty-W force-pushed the 75-script-for-automated-workflow branch from 4c80d35 to f071651 Compare June 18, 2024 22:53
@Misty-W Misty-W force-pushed the 75-script-for-automated-workflow branch from 799f24f to 810a314 Compare June 26, 2024 00:32
@DanieleQubit
Copy link
Collaborator

Testing docker + automated script for analog emulation: WIP

@Misty-W
Copy link
Contributor Author

Misty-W commented Jun 26, 2024

@DanieleQubit please make sure you update your clone to the latest commit on this branch, since I just reverted a change I made yesterday, experimenting with grid scaling in slicing.

@DanieleQubit
Copy link
Collaborator

Hi, here my first test, which results in a python error message (see later in this message):

  1. 3DRISM density used attached here inside the .zip
    2a15_ph4.zip
  2. the analog parameter file (in the zip too):

"7 7.3585242543498035 124.66370614359172 3000
10 30
poisson-disk 0 0 4 6 5
filter gaussian-laplace 0.5
crop 4.70 -12.0 5.0 5.0
2.702 -11.26 -9.197 3.202 -11.260 -9.197 2.702 -10.76 -9.197
2.702 -11.26 -9.697 3.202 -11.260 -9.697 2.702 -10.76 -9.697"

  1. the automated flow .py modified is in the zip too.

I have this error:
Screenshot from 2024-06-27 18-13-00

in the same screenshot you can also see the command I used to run the simulation:
"docker exec aquapointer_test /bin/bash -c 'python ./aquapointer/analog/automated_flow.py'"

@Misty-W
Copy link
Contributor Author

Misty-W commented Jul 8, 2024

@DanieleQubit @darcangelomauro I fixed the error Daniele reported and the one I subsequently encountered (ordering of the slicing planes), but now there are issues when creating the lattice from the density slices.
The first and third examples from Daniele "hang" at the lattice creation step but run through without errors when I remove the cropping line from the settings file. On Daniele's second example (minus the cropping line), I get this error:

Traceback (most recent call last):
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/analog/automated_flow.py", line 6, in
from aquapointer.slicing import density_file_to_grid, density_slices_by_planes
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/init.py", line 8, in
from aquapointer.analog import analog, automated_flow, density_mapping, qubo_solution, utils
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/analog/automated_flow.py", line 132, in
locations = rism_to_locations(
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/analog/automated_flow.py", line 65, in rism_to_locations
[c.set_poisson_disk_lattice(spacing) for c in canvases]
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/analog/automated_flow.py", line 65, in
[c.set_poisson_disk_lattice(spacing) for c in canvases]
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/density_canvas/DensityCanvas.py", line 349, in set_poisson_disk_lattice
lattice = Lattice.poisson_disk(
File "/Users/mistywahl/Documents/GitHub/aquapointer/aquapointer/density_canvas/Lattice.py", line 135, in poisson_disk
ref_radius = radius_density[_index_from_position(ref_point)]
IndexError: index 82 is out of bounds for axis 0 with size 79

I've set up a call for tomorrow to discuss and will continue to troubleshoot in the meantime.

@Misty-W Misty-W linked an issue Jul 10, 2024 that may be closed by this pull request
@FarLab FarLab merged commit 94c826a into main Jul 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Dockerfile Script for automated workflow
3 participants