Skip to content

Commit

Permalink
Merge pull request #44 from efabless/remove_timeout
Browse files Browse the repository at this point in the history
Removed the 300 second timout for thread completion
  • Loading branch information
RTimothyEdwards authored Mar 24, 2024
2 parents 5cab3af + bb94cbb commit 6d293f3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cace/__version__.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
__version__ = '2.1.15'
__version__ = '2.1.16'

if __name__ == '__main__':
print(__version__, end='')
6 changes: 5 additions & 1 deletion cace/common/cace_launch.py
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,11 @@ def cace_launch(dsheet, param):

for result in results:
try:
presult = result.get(timeout=300)
# Note: Previously used result.get(timeout=300).
# This was intended to prevent hung simulations from
# hanging CACE. But with multiprocessing, the timeout
# itself causes CACE to hang, making it less than useless.
presult = result.get()
except Exception as e:
print('simulate_and_measure failed with exception:')
print(e)
Expand Down

0 comments on commit 6d293f3

Please sign in to comment.