Waztom / xchem-CAR

5 stars 0 forks source link

Auto script generator error #61

Open GitGud93 opened 2 years ago

GitGud93 commented 2 years ago

image

The four problems encountered when trying to use the auto script generator.

image

A look at the error text in the terminal on the right.

Any suggestions? Not sure how it came to the conclusion we are maxing out the volume.

Waztom commented 2 years ago

Hi @GitGud93 will go through the screenshots in the order you posted:

  1. The 'opentrons' import error is because CAR has not installed the 'opentrons' Python package. The script generated is to be run via the OpenTrons app. Including the 'opentrons' package is something we plan to do, but for other reasons - to validate the scripts generated.
  2. The overfilling logic is something that we are working on. If you run fewer compounds through, then it should work. @OliviaSimpson - if there is not already one, please create an issue for the overfilling logic.

You mentioned four errors - could you please provide more details for the other two?

GitGud93 commented 2 years ago

Hi @Waztom the four errors I was referring to were just the four issues listed under problems in the first screenshot. Thanks a bunch for this, much appreciated!

If I understand correctly does that mean If I install the package then it should generate the script? Because as you can see the script wasn't generated at all. (I opened the file where it was supposed to be generated and this is what you are viewing along with the terminals)

Waztom commented 2 years ago

@GitGud93 No worries! For now - do not think it's necessary for you to install the 'opentrons' package. The OpenTrons app will read the script as it is without installing extra packages etc. The script was not generated because of the overfilling error - if you try with fewer reactions or at a smaller scale, then it should produce something. Let me know if you still do not see a script file being generated with less reactions and/or smaller scale.