OpenChemistry / mongochemdeploy

Scripts to install/deploy the MongoChem server/web client
BSD 3-Clause "New" or "Revised" License
7 stars 9 forks source link

Allow multiple SMILES to be ran through chemml #77

Closed psavery closed 5 years ago

psavery commented 5 years ago

This is still a work in progress. It depends on #68.

cjh1 commented 5 years ago

@psavery What about this one?

psavery commented 5 years ago

A side note about this PR: because we now have lists of calculations running in a single container, one thing that might be beneficial is that containers now have more flexibility when deciding how to run the list of calculations. Currently, they all just loop through the calculations and run them one by one (see here, for instance).

But there is potential for the containers to split up the list of calculations and run each calculation on a different thread (especially if the individual calculations are single-threaded), or maybe even run each calculation on a different compute node or cluster.

cryos commented 5 years ago

There is potential, but we want to avoid making the containers themselves too complex. Some of that might better live a level above them, where there is more visibility of the overall compute resources available, and let the containers themselves be relatively simple running through a serial list.