-
Notifications
You must be signed in to change notification settings - Fork 0
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
WIP: User preference prediction openday #53
Open
Baschdl
wants to merge
42
commits into
master
Choose a base branch
from
userpreference-prediction-openday
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 24 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
068510a
Add notebook to load data from open day
Baschdl b0674ee
Add fitting of graph
Baschdl e84ae04
Merge branch 'add-testing-preference-prediction' into userpreference-…
Baschdl 0705252
Correct creation of MetaPathRatingGraph
Baschdl f336836
Add exporting of decision tree graph and add feature names
Baschdl 192bb85
Revert "Add exporting of decision tree graph and add feature names"
Baschdl bd18e05
Add exporting of decision tree graph and add feature names
Baschdl 0b48d1d
Tidy up notebook
Baschdl f2c8358
Move notebook
Baschdl 086de3d
Rename notebook
Baschdl fdbc5cf
Add pruned notebook
Baschdl 7690ee4
Add dynamic rendering of decision tree
Baschdl b821fc3
Allow enabling development mode for server.
GittiHab c3b0d82
Add exploration notebook.
GittiHab ed7a369
Update exploration notebook.
GittiHab df4d490
Add Dockerfile and jupyter notebook config
Baschdl 3a9a539
Rename notebook
GittiHab 6781556
Add regression notebook.
GittiHab c42fcc3
Add accuracy score to regressor.
GittiHab 0b8f930
Add tini to Docker image
Baschdl 619c5a1
Introduce script to create Jupyter Notebook Docker image.
GittiHab d6defb4
Correct script for other Dockerfile
Baschdl 9a05d92
Merge pull request #51 from KDD-OpenSource/jupyter-notebook-extra-doc…
Baschdl ad32a41
Merge branch 'master' into userpreference-prediction-openday
Baschdl 2388005
Fix not initializing with super.
GittiHab f11c646
Updated notebook.
GittiHab 35208f0
Add script to copy notebooks from container to local directory.
GittiHab a12c260
Add missing random state.
GittiHab 48b83b3
Add rnn notebook
GittiHab d91bec1
Refactor domain scoring class.
GittiHab 9512986
Add simple neural networks to notebook.
GittiHab 2bc0905
Add newly rendered image.
GittiHab 9f7eeab
Add rnn regression notebook.
GittiHab 93a08b6
Add n-gram range parameter
Baschdl a5ed39b
Merge branch 'userpreference-prediction-openday' of https://github.co…
Baschdl f794c3b
Add rnn notebook with high score.
GittiHab 733c5a0
Add new structured rnn notebook.
GittiHab e131272
Merge branch 'master' into userpreference-prediction-openday
Baschdl f00d779
Restructured rated datasets
GittiHab 3a622f0
Updated dataset paths in notebooks.
GittiHab 1de630a
Merge branch 'master' into userpreference-prediction-openday
GittiHab e328645
Merge branch 'userpreference-prediction-openday' of https://github.co…
Baschdl File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
FROM server:latest | ||
|
||
RUN apt-get update && apt-get install -y graphviz | ||
RUN pip3 install jupyter | ||
|
||
# Add Tini. Tini operates as a process subreaper for jupyter. This prevents | ||
# kernel crashes. | ||
ENV TINI_VERSION v0.6.0 | ||
ADD https://github.com/krallin/tini/releases/download/${TINI_VERSION}/tini /usr/bin/tini | ||
RUN chmod +x /usr/bin/tini | ||
ENTRYPOINT ["/usr/bin/tini", "--"] | ||
|
||
COPY . /32de-python/ | ||
|
||
EXPOSE 8888 | ||
CMD ["jupyter", "notebook", "--config", "deployment/jupyter_notebook_config.py"] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
#!/bin/bash | ||
docker build -t notebook -f Dockerfile-Notebook ${1:-.} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# Set ip to '*' to bind on all interfaces (ips) for the public server | ||
c.NotebookApp.ip = '*' | ||
c.NotebookApp.password = u'sha1:ba8ffcde0b68:95fa25d7225a3915db1db76799f1695a0483afb4' | ||
c.NotebookApp.open_browser = False | ||
|
||
c.NotebookApp.port = 8888 | ||
c.NotebookApp.allow_root = True |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
#!/bin/bash | ||
docker stop notebook-container | ||
docker rm notebook-container | ||
docker run --name notebook-container \ | ||
--publish=${1:-8888}:8888 \ | ||
-d \ | ||
notebook |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
#!/bin/bash | ||
docker stop notebook-container |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't always yield the accuracy as we say with the RandomForestRegressor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean that this depends on the predictor?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, either we we don't call
self.classifier.score
and instead predict the labels with our classifier and evaluate it with the general scikit-learnaccuracy()
method or we print "Test score is..." but that's more or less useless if you don't know which score this is.