Skip to content

Commit 7824381

Browse files
authored
Merge pull request #56 from hirosassa/optional-dependency-import
Move tf dependency to dev-dependencies
2 parents 48a03a8 + 723a7f3 commit 7824381

37 files changed

+920
-733
lines changed

.github/workflows/format.yml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
push:
55
branches: [ master ]
66
pull_request:
7-
branches: [ master ]
7+
88

99
jobs:
1010
formatting-check:
@@ -14,7 +14,9 @@ jobs:
1414

1515
steps:
1616
- uses: actions/checkout@v2
17+
- name: Install dependencies
18+
run: |
19+
pip install toml yapf
1720
- name: run YAPF to test if python code is correctly formatted
18-
uses: AlexanderMelde/yapf-action@master
19-
with:
20-
args: --verbose --style=yapf.ini
21+
run: |
22+
yapf --diff --recursive --verbose $GITHUB_WORKSPACE

README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,8 @@
66
![](https://img.shields.io/pypi/l/redshells)
77

88
Machine learning tasks which are used with data pipeline library "luigi" and its wrapper "gokart".
9+
10+
## Dependencies
11+
12+
You should add `tensorflow = ">=1.13.1,<2.0"` on your app dependencies if you would like to use some models under `redshells/contrib` directory.
13+
`Tensorflow` is not included redshells' dependencies because there are some models not used it.

0 commit comments

Comments
 (0)