Merge pull request #21 from nzw0301/master

Fix link to Github repo

docs/index.rst

@@ -1,5 +1,5 @@
 
-:github_url: https://github.com/pytorch/pytorch
+:github_url: https://github.com/PetrochukM/PyTorch-NLP
 
 Welcome to Pytorch-NLP's documentation!
 =======================================

tests/datasets/test_multi30k.py

@@ -1,13 +1,15 @@
 import os
 
 import mock
+import pytest
 
 from torchnlp.datasets import multi30k_dataset
 from tests.datasets.utils import urlretrieve_side_effect
 
 multi30k_directory = 'tests/_test_data/multi30k'
 
 
+@pytest.mark.skip(reason="Unsafe dataset host (SSL: CERTIFICATE_VERIFY_FAILED)")
 @mock.patch("urllib.request.urlretrieve")
 def test_multi30k_dataset_row(mock_urlretrieve):
     mock_urlretrieve.side_effect = urlretrieve_side_effect

torchnlp/datasets/multi30k.py

@@ -16,7 +16,7 @@ def multi30k_dataset(directory='data/multi30k/',
                          'http://www.quest.dcs.shef.ac.uk/wmt16_files_mmt/training.tar.gz',
                          'http://www.quest.dcs.shef.ac.uk/wmt16_files_mmt/validation.tar.gz',
                          'http://www.quest.dcs.shef.ac.uk/wmt16_files_mmt/mmt16_task1_test.tar.gz'
-                     ]):
+                     ]):  # pragma: no cover
     """
     Load the WMT 2016 machine translation dataset.
 
@@ -26,6 +26,10 @@ def multi30k_dataset(directory='data/multi30k/',
     German human translation. As test data, we provide a new set of 1,000 tuples containing an
     English description.
 
+    Status:
+        Host ``www.quest.dcs.shef.ac.uk`` forgot to update their SSL
+        certificate; therefore, this dataset does not download securely.
+
     References:
         * http://www.statmt.org/wmt16/multimodal-task.html
         * http://shannon.cs.illinois.edu/DenotationGraph/