search

AI-Planning / planning-as-a-service

The newly improved planner (and more) in the cloud.
Apache License 2.0
30 stars 8 forks source link

jsonify our 500 errors. #49

Closed haz closed 10 months ago

jan-dolejsi commented 10 months ago

Hard to say. Sure this will let the 500 prevail? Well, let's try it!

haz commented 10 months ago

It'll still have errors, but at least the return will be json :P. I think that's the right interpretation -- if the flask server crashes down, that should be 500. But if the planner is on the fritz, then I think a json payload makes sense.

Going to merge -- @nirlipo , can you re-deploy?

nirlipo commented 10 months ago

Deployed! I tested Delfi to see if I get the same error that @jan-dolejsi was reporting, but it works fine through the online editor. Just in case I recompiled the images of each worker too.

haz commented 10 months ago

Any server-side indication of what's going on?

nirlipo commented 10 months ago

I'll send you a link with the failed task in the server monitor. The flags for persist were off, so I can't get more info than that it timed out. Looking at previous tasks that had the persist flag on, and delfi failed, it was related to tensorflow.

"stderr": "bash: cannot set terminal process group (1): Inappropriate ioctl for device\nbash: no job control in this shell\nUsing TensorFlow backend.\nError: Could not parse domain file: domain\nReason: Missing ')'.\nError: Could not parse domain file: domain\nReason: Missing ')'.\n",