Skip to content

Commit eb54f18

Browse files
authored
Merge 30e115b into b295029
2 parents b295029 + 30e115b commit eb54f18

File tree

3 files changed

+20
-30
lines changed

3 files changed

+20
-30
lines changed

azure-pipelines.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,3 +88,9 @@ jobs:
8888
- script: |
8989
python -m pytest benchmarks pl_examples -v --maxfail=2 --durations=0
9090
displayName: 'Testing: extended'
91+
92+
- script: |
93+
python setup.py install --user --quiet
94+
bash pl_examples/run_ddp-example.sh
95+
pip uninstall -y pytorch-lightning
96+
displayName: 'Examples'

pl_examples/run_ddp-example.sh

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
#!/bin/bash
2+
3+
ARGS_DEFAULT=" --default_root_dir %(tmpdir)s --max_epochs 1 --batch_size 32 --limit_train_batches 2 --limit_val_batches 2"
4+
ARGS_EXTRA_DDP=" --gpus 2 --accelerator ddp"
5+
ARGS_EXTRA_AMP=" --precision 16"
6+
7+
python pl_examples/basic_examples/simple_image_classifier.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP}
8+
python pl_examples/basic_examples/simple_image_classifier.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP} ${ARGS_EXTRA_AMP}
9+
python pl_examples/basic_examples/backbone_image_classifier.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP}
10+
python pl_examples/basic_examples/backbone_image_classifier.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP} ${ARGS_EXTRA_AMP}
11+
python pl_examples/basic_examples/autoencoder.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP}
12+
python pl_examples/basic_examples/autoencoder.py ${ARGS_DEFAULT} ${ARGS_EXTRA_DDP} ${ARGS_EXTRA_AMP}

pl_examples/test_examples.py

Lines changed: 2 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -38,17 +38,7 @@
3838
--accelerator dp \
3939
"""
4040

41-
ARGS_DP_AMP = ARGS_DP + """
42-
--precision 16 \
43-
"""
44-
45-
ARGS_DDP = ARGS_DEFAULT + """
46-
--gpus 2 \
47-
--accelerator ddp \
48-
--precision 16 \
49-
"""
50-
51-
ARGS_DDP_AMP = ARGS_DEFAULT + """
41+
ARGS_AMP = """
5242
--precision 16 \
5343
"""
5444

@@ -61,7 +51,7 @@
6151
]
6252
)
6353
@pytest.mark.skipif(torch.cuda.device_count() < 2, reason="test requires multi-GPU machine")
64-
@pytest.mark.parametrize('cli_args', [ARGS_DP, ARGS_DP_AMP])
54+
@pytest.mark.parametrize('cli_args', [ARGS_DP, ARGS_DP + ARGS_AMP])
6555
def test_examples_dp(tmpdir, import_cli, cli_args):
6656

6757
module = importlib.import_module(import_cli)
@@ -72,24 +62,6 @@ def test_examples_dp(tmpdir, import_cli, cli_args):
7262
module.cli_main()
7363

7464

75-
# ToDo: fix this failing example
76-
# @pytest.mark.parametrize('import_cli', [
77-
# 'pl_examples.basic_examples.simple_image_classifier',
78-
# 'pl_examples.basic_examples.backbone_image_classifier',
79-
# 'pl_examples.basic_examples.autoencoder',
80-
# ])
81-
# @pytest.mark.skipif(torch.cuda.device_count() < 2, reason="test requires multi-GPU machine")
82-
# @pytest.mark.parametrize('cli_args', [ARGS_DDP, ARGS_DDP_AMP])
83-
# def test_examples_ddp(tmpdir, import_cli, cli_args):
84-
#
85-
# module = importlib.import_module(import_cli)
86-
# # update the temp dir
87-
# cli_args = cli_args % {'tmpdir': tmpdir}
88-
#
89-
# with mock.patch("argparse._sys.argv", ["any.py"] + cli_args.strip().split()):
90-
# module.cli_main()
91-
92-
9365
@pytest.mark.parametrize(
9466
'import_cli', [
9567
'pl_examples.basic_examples.simple_image_classifier',

0 commit comments

Comments
 (0)