-
Notifications
You must be signed in to change notification settings - Fork 269
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
Add ability to run this project in Docker #9
Open
uzakov
wants to merge
26
commits into
SamPom100:master
Choose a base branch
from
uzakov:master
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 all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
df9cdd5
Added Dockerfile in order to run this project easier and faster, expa…
8ed6c12
Enhanced description on how to run it and what is required
f974456
Update market_scanner.py
NihilistPenguin 7574904
Add files via upload
NihilistPenguin 6f5cb6c
Update grapher.py
NihilistPenguin ae0a979
Add files via upload
NihilistPenguin 2f4fd42
Update README.md
NihilistPenguin 29c29bc
Update README.md
NihilistPenguin 4c1770b
Update README.md
NihilistPenguin cbf1b4e
Update grapher.py
NihilistPenguin f9e20da
Update grapher.py
NihilistPenguin eb859ec
Update market_scanner.py
NihilistPenguin 2fd7709
add figures directory with demo images
1d9ddc8
add figures directory with sample images
24eca53
Delete .gitignore
NihilistPenguin e845186
Update README.md
18b5427
Update README.md
65c26cf
Update README.md
9fb4250
Merge pull request #16 from tcosculluela/master
21933ae
Revert "Parallelization"
SamPom100 f2b9e36
Merge pull request #20 from SamPom100/revert-16-master
4893e10
Merge pull request #18 from NihilistPenguin/master
bd6508b
Revert "Add various functionality "
SamPom100 1032b62
Merge pull request #21 from SamPom100/revert-18-master
df21531
As per PR comments removed duplicate code and made it clearer about r…
439401a
Merge upstream
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
FROM python:3 | ||
|
||
RUN pip install --upgrade pip && \ | ||
pip install --no-cache-dir nibabel pydicom matplotlib pillow && \ | ||
pip install --no-cache-dir med2image | ||
|
||
# As per https://docs.docker.com/develop/develop-images/dockerfile_best-practices/ | ||
|
||
COPY requirements.txt /tmp | ||
RUN pip install --requirement /tmp/requirements.txt | ||
COPY . /tmp/ | ||
|
||
RUN mkdir /opt/wsb/ | ||
COPY grapher.py market_scanner.py stocklist.py /opt/wsb/ | ||
|
||
RUN mkdir /opt/wsb/data | ||
COPY data /opt/wsb/data |
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
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 |
---|---|---|
|
@@ -5,4 +5,3 @@ numpy==1.18.4 | |
mplcursors==0.3 | ||
python_dateutil==2.8.1 | ||
tqdm==4.48.0 | ||
joblib |
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
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.
Consider using smaller image, there's no need to have all of python to run this.
python:3-slim-buster is around 48MB compressed.
python:3 is around 320MB compressed.