Skip to content

Commit e78bb14

Browse files
authored
Merge pull request #349 from uclmr/fix-346
Fix 346
2 parents c762855 + 19d52c6 commit e78bb14

File tree

4 files changed

+17
-19
lines changed

4 files changed

+17
-19
lines changed

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -112,3 +112,5 @@ data/WN18/wordnet-mlj12
112112
tmp/
113113
tests/test_results/fastqa_reader_test/
114114
tests/test_results/dam_reader_test/
115+
116+
data/GloVe/glove.840B.300d.*

requirements.txt

+1-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,7 @@ pyyaml
1313
progressbar2
1414
spacy==1.9
1515
diskcache
16-
17-
pytest==3.3.0
16+
pytest
1817
pytest-runner
1918
pytest-xdist
2019
pytest-pep8

setup.py

+10-14
Original file line numberDiff line numberDiff line change
@@ -27,26 +27,25 @@ def install_torch():
2727

2828

2929
class Install(_install):
30-
def __init__(self, *args, **kwargs):
31-
super().__init__(*args, **kwargs)
32-
3330
def run(self):
3431
_install.do_egg_install(self)
3532
spacy_download_en()
3633
_install.run(self)
3734

3835

3936
class Develop(_develop):
40-
def __init__(self):
41-
super().__init__()
42-
4337
def run(self):
4438
spacy_download_en()
4539
_develop.run(self)
4640

4741

4842
with open('requirements.txt', 'r') as f:
49-
requirements = [l for l in f.readlines() if not l.startswith('http://')]
43+
install_requires = [l for l in f.readlines() if not l.startswith('http://')]
44+
45+
extras_require = {
46+
'tf': ['tensorflow>=1.4.0'],
47+
'tf_gpu': ['tensorflow-gpu>=1.4.0']
48+
}
5049

5150
setup(name='jack',
5251
version='0.1.0',
@@ -61,13 +60,10 @@ def run(self):
6160
'install': Install,
6261
'develop': Develop
6362
},
64-
install_requires=requirements,
65-
extras_require={
66-
'tensorflow': ['tensorflow>=1.4.0'],
67-
'tensorflow_gpu': ['tensorflow-gpu>=1.4.0'],
68-
},
69-
setup_requires=requirements,
70-
tests_require=requirements,
63+
install_requires=install_requires,
64+
extras_require=extras_require,
65+
setup_requires=install_requires,
66+
tests_require=install_requires,
7167
classifiers=[
7268
'Development Status :: 4 - Beta',
7369
'Intended Audience :: Developers',

wercker.yml

+4-3
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,9 @@ build:
1111
code: |
1212
sudo apt-get update
1313
sudo apt-get -y install libtk8.6
14-
pip3 install --upgrade -r requirements.txt
15-
python3 setup.py install
14+
15+
pip install --upgrade -r requirements.txt
16+
pip install -e .[tf]
1617
1718
- script:
1819
name: echo python information
@@ -29,4 +30,4 @@ build:
2930
- script:
3031
name: Code coverage upload
3132
code: |
32-
codecov
33+
codecov

0 commit comments

Comments
 (0)