Skip to content

Conversation

@trackibs
Copy link

Hi Ariel,

I have created a Dockerfile which installs all the dependencies required to run this script and also updated the Makefile to add some make docker commands.

I renamed the file ariel.csv to root.csv due to it looking for the username :

mv ariel.csv root.csv

Then I run the following make command :

make docker-score-chart

or the abbreviated command is :

make dsc

It will then build the docker container if it doesn't exist and then generate the scores.txt and root.scores.png files.

I was hoping to be able to use your repository to help with doing similar machine-learning but with Irritable Bowel Syndrome (IBS) Symptoms.

Instead of adding in weight we instead add in a digestive symptom score which is calculated by an app called Cara :

The symptom score is calculated on the basis of the symptoms tracked with Cara. The value can be between 0 and 100. The higher the symptom score, the more severe are the digestive problems in the region.

There was a discussion earlier today on Reddit about using Machine Learning to troubleshoot IBS symptoms :
https://www.reddit.com/r/ibs/comments/9f1w46/im_working_on_a_startup_to_find_individual/

and it reminded me of your Weight Loss machine learning repository I looked at a while ago, any way let me know what you think!

Copy link
Owner

@arielf arielf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. Appreciated.
Could you make the following changes?

  1. The docker build needs to be run as root or it'll fail. Can you may it so it gives guidance if you're not root
  2. On standard Ubuntu there's no need for /usr/local, and many systems may not even have /usr/local, let alone bin under it. /usr/bin/python2 exists by default so adding the /usr/local/bin/python2 symbolic link is both unnecessary and likely to fail.
  3. In the original Makefile the variable NAME is already auto-discovered and set to whoever is running the make so not sure about the comment about having data in root.csv. This is not a PR change, just a comment about the comment.

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.

2 participants