ultravioletrs / cocos

Cocos AI - Confidential Computing System for AI
https://ultraviolet.rs/cocos.html
Apache License 2.0
25 stars 9 forks source link

ALGO STDOUT-STDERR #179

Closed SammyOina closed 4 months ago

SammyOina commented 4 months ago

What type of PR is this?

What does this do?

Which issue(s) does this PR fix/relate to?

Have you included tests for your changes?

Did you document any new/modified feature?

Notes

dborovcanin commented 4 months ago

Let's evaluate this approach:

However, there are some drawbacks to this approach:

Having this in mind, it's worth exploring if there is a way to "fake" the filesystem for those algorithms that require files for logging. @SammyOina Can you check it? Also, try and test how this approach works with Docker.

dborovcanin commented 4 months ago

Finally, I think it would be an acceptable requirement (though definitely not ideal, ofc) for the algo provider to write all the logs to the stderr, since we do not support a file system.

dborovcanin commented 4 months ago

We will keep the file approach, so this PR can be closed. We will return to it once we have a working and fully functional demo with files.