Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install python and moodlemlbackend package #36

Closed

Conversation

dmonllao
Copy link
Contributor

No description provided.

@dmonllao
Copy link
Contributor Author

This patch should be applied to *jessie branches.

@andrewnicols
Copy link
Contributor

andrewnicols commented Dec 10, 2018 via email

@dmonllao dmonllao force-pushed the python-mlbackend-jessie branch from 4d901d1 to 16e4744 Compare December 10, 2018 16:31
@dmonllao
Copy link
Contributor Author

The patch has been amended to clear apt and pip caches.

@dmonllao dmonllao force-pushed the python-mlbackend-jessie branch 6 times, most recently from e1cb40a to 0f2c1d7 Compare January 30, 2019 15:27
@dmonllao dmonllao force-pushed the python-mlbackend-jessie branch from 0f2c1d7 to 5869dac Compare January 30, 2019 15:29
@dmonllao
Copy link
Contributor Author

The tensorflow versions required by our python ML backend can't be installed on jessie (Could not find any downloads that satisfy the requirement tensorflow>=1.0.0,<1.1). This is my conclusion after several tries. I've checked that the correct python version is installed, all the required dependencies... I ran out of ideas.

What would you think about discarding this patch and just applying #35 to stretch?

@scara
Copy link
Contributor

scara commented Jan 30, 2019

Hi @dmonllao,
never played w/ tensorflow, you could try to update pip by itself:

sudo pip install --upgrade pip

BTW if I could vote: +1 to get rid of supporting ML in jessie since stretch images are now available even for PHP 7.2 and 7.3 here in moodle-php-apache.

HTH,
Matteo

@dmonllao
Copy link
Contributor Author

Ok, closing this issue, thanks.

@dmonllao dmonllao closed this Feb 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants