From: Nicolas Hug <nicolashug@fb.com>
Date: Thu, 2 Sep 2021 10:48:44 +0000 (-0700)
Subject: Fix list() and help() torchhub functions for Windows (#63773)
X-Git-Tag: accepted/tizen/8.0/unified/20231005.095509~483
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aedd70fcfe62a8bfb726b609f16edb8448a74299;p=platform%2Fupstream%2Fpytorch.git

Fix list() and help() torchhub functions for Windows (#63773)

Summary:
This PR Fixes the help() and list() torchhub functions which were probably failing for Windows since the `/` OS separator was hardcoded.

Before merging this I need to double check whether the CI actually runs the corresponding tests on Windows or not

Pull Request resolved: https://github.com/pytorch/pytorch/pull/63773

Reviewed By: zou3519

Differential Revision: D30695664

Pulled By: NicolasHug

fbshipit-source-id: fac328163fd05db804a8186ae28f22b3cc3a6404
---

diff --git a/torch/hub.py b/torch/hub.py
index 4cfbc83..bcd53f7 100644
--- a/torch/hub.py
+++ b/torch/hub.py
@@ -277,7 +277,8 @@ def list(github, force_reload=False, skip_validation=False):
 
     sys.path.insert(0, repo_dir)
 
-    hub_module = import_module(MODULE_HUBCONF, repo_dir + '/' + MODULE_HUBCONF)
+    hubconf_path = os.path.join(repo_dir, MODULE_HUBCONF)
+    hub_module = import_module(MODULE_HUBCONF, hubconf_path)
 
     sys.path.remove(repo_dir)
 
@@ -307,7 +308,8 @@ def help(github, model, force_reload=False, skip_validation=False):
 
     sys.path.insert(0, repo_dir)
 
-    hub_module = import_module(MODULE_HUBCONF, repo_dir + '/' + MODULE_HUBCONF)
+    hubconf_path = os.path.join(repo_dir, MODULE_HUBCONF)
+    hub_module = import_module(MODULE_HUBCONF, hubconf_path)
 
     sys.path.remove(repo_dir)