search

moodlehq / moodle-mlbackend-python

Moodle machine learning backend
GNU General Public License v3.0
18 stars 19 forks source link

Add detail about saving payloads to install-venv #52

Closed danmarsden closed 2 years ago

douglasbagnall commented 2 years ago

Looks accurate and useful.

danmarsden commented 2 years ago

@ilyatregubov this is just a change to the docs to describe an extra option for debugging - no code changes here. :-)

danmarsden commented 2 years ago

it's probably worth mentioning we could tidy up the readme in the project to link to this file - I would expect that most people coming to github are likely to want to build this using something similar to the steps documented in install-venv rather than the steps for generating a python package and uploading it to the package index which is likely something only HQ staff will do.

douglasbagnall commented 2 years ago

Also worth noting is there are changes in our production branch: https://github.com/catalyst/moodle-mlbackend-python/blob/catalyst-3.4/INSTALL-venv.md (commit 7a707427c0217e64a339528d8fe0feee149b91bc).

I haven't worked out which changes can be backported here, but I suspect most should be. This document is currently more indicative than accurate.

ilyatregubov commented 2 years ago

Aha. Just some more extra debugging info. Ok will merge soon. No need of new version for this one.