EpistasisLab / Aliro

Aliro: AI-Driven Data Science
https://epistasislab.github.io/Aliro
GNU General Public License v3.0
223 stars 63 forks source link

Python Execution Error on Aliro #595

Closed HyunjunA closed 1 year ago

HyunjunA commented 1 year ago

When attempting to run Python code on Aliro, the following error occurs

Screen Shot 2023-08-14 at 1 56 31 PM
jay-m-dev commented 1 year ago

This issue is fixed on commit https://github.com/EpistasisLab/Aliro/commit/a96a447237d142c8606c7757ab1b9b290b1f5aee.

miguelehernandez commented 1 year ago

Can Jun verify this was fixed to his satisfaction?

HyunjunA commented 1 year ago
image

When I asked a simple question, such as 'print hello,' I encountered an error message that I posted above. I've modified the code as shown below, and it resolved the error. However, I'm not sure if this change will lead to other potential issues.

if (req.body.packages === null && req.body.command !== "freeze") { return res.status(400).json({ message: "No packages provided" }); }

jay-m-dev commented 1 year ago

My mistake. This is a new issue in the /install endpoint. It worked when I tested locally due to a commented out 'packages' parameter that was still passed to the endpoint. I've recreated this new error and will revise the /install endpoint. I'll keep the check for packages.length in the revised solution to allow it to catch empty packages lists.

jay-m-dev commented 1 year ago

@HyunjunA while testing the /install endpoint I found another enhancement that's needed for this and the code run endpoint. Since this issue is for a different reason and for a different endpoint. I've opened issue #597. I'll include the fix for this length undefined error in the fix to #597.