From f202440717b5cfc97c2442425c83bcfbe459aaac Mon Sep 17 00:00:00 2001
From: Vasilis Vryniotis <vvryniotis@fb.com>
Date: Wed, 27 Jan 2021 18:23:20 +0000
Subject: [PATCH 1/5] Moving registration of video methods in Video.cpp and
 removing unnecessary includes.

---
 torchvision/csrc/io/video/Video.cpp            | 16 +++++++++++-----
 torchvision/csrc/io/video/Video.h              | 18 +++++-------------
 torchvision/csrc/io/video/register.cpp         | 14 --------------
 .../csrc/io/video_reader/VideoReader.cpp       | 12 ++++--------
 torchvision/csrc/io/video_reader/VideoReader.h |  4 +++-
 5 files changed, 23 insertions(+), 41 deletions(-)
 delete mode 100644 torchvision/csrc/io/video/register.cpp

diff --git a/torchvision/csrc/io/video/Video.cpp b/torchvision/csrc/io/video/Video.cpp
index c1e19e0b5f5..7f494a47868 100644
--- a/torchvision/csrc/io/video/Video.cpp
+++ b/torchvision/csrc/io/video/Video.cpp
@@ -1,9 +1,6 @@
 #include "Video.h"
-#include <c10/util/Logging.h>
-#include <torch/script.h>
-#include "defs.h"
-#include "memory_buffer.h"
-#include "sync_decoder.h"
+
+#include <regex>
 
 using namespace std;
 using namespace ffmpeg;
@@ -319,3 +316,12 @@ std::tuple<torch::Tensor, double> Video::Next() {
 
   return std::make_tuple(outFrame, frame_pts_s);
 }
+
+static auto registerVideo =
+    torch::class_<Video>("torchvision", "Video")
+        .def(torch::init<std::string, std::string>())
+        .def("get_current_stream", &Video::getCurrentStream)
+        .def("set_current_stream", &Video::setCurrentStream)
+        .def("get_metadata", &Video::getStreamMetadata)
+        .def("seek", &Video::Seek)
+        .def("next", &Video::Next);
diff --git a/torchvision/csrc/io/video/Video.h b/torchvision/csrc/io/video/Video.h
index f7d47c0454c..dfb036ac851 100644
--- a/torchvision/csrc/io/video/Video.h
+++ b/torchvision/csrc/io/video/Video.h
@@ -1,18 +1,10 @@
 #pragma once
 
-#include <map>
-#include <regex>
-#include <string>
-#include <vector>
-
-#include <ATen/ATen.h>
-#include <c10/util/Logging.h>
-#include <torch/script.h>
-
-#include <exception>
-#include "defs.h"
-#include "memory_buffer.h"
-#include "sync_decoder.h"
+#include <torch/types.h>
+
+#include "../decoder/defs.h"
+#include "../decoder/memory_buffer.h"
+#include "../decoder/sync_decoder.h"
 
 using namespace ffmpeg;
 
diff --git a/torchvision/csrc/io/video/register.cpp b/torchvision/csrc/io/video/register.cpp
deleted file mode 100644
index 08902d427b0..00000000000
--- a/torchvision/csrc/io/video/register.cpp
+++ /dev/null
@@ -1,14 +0,0 @@
-#include "Video.h"
-
-namespace {
-
-static auto registerVideo =
-    torch::class_<Video>("torchvision", "Video")
-        .def(torch::init<std::string, std::string>())
-        .def("get_current_stream", &Video::getCurrentStream)
-        .def("set_current_stream", &Video::setCurrentStream)
-        .def("get_metadata", &Video::getStreamMetadata)
-        .def("seek", &Video::Seek)
-        .def("next", &Video::Next);
-
-} // namespace
diff --git a/torchvision/csrc/io/video_reader/VideoReader.cpp b/torchvision/csrc/io/video_reader/VideoReader.cpp
index 06a0521c317..6a98897e0f9 100644
--- a/torchvision/csrc/io/video_reader/VideoReader.cpp
+++ b/torchvision/csrc/io/video_reader/VideoReader.cpp
@@ -1,13 +1,6 @@
 #include "VideoReader.h"
-#include <ATen/ATen.h>
-#include <Python.h>
-#include <c10/util/Logging.h>
-#include <exception>
-#include "memory_buffer.h"
-#include "sync_decoder.h"
 
-using namespace std;
-using namespace ffmpeg;
+#include <Python.h>
 
 // If we are in a Windows environment, we need to define
 // initialization functions for the _custom_ops extension
@@ -18,6 +11,9 @@ PyMODINIT_FUNC PyInit_video_reader(void) {
 }
 #endif
 
+using namespace std;
+using namespace ffmpeg;
+
 namespace video_reader {
 
 const AVPixelFormat defaultVideoPixelFormat = AV_PIX_FMT_RGB24;
diff --git a/torchvision/csrc/io/video_reader/VideoReader.h b/torchvision/csrc/io/video_reader/VideoReader.h
index 923a3190977..e8b5cca99a5 100644
--- a/torchvision/csrc/io/video_reader/VideoReader.h
+++ b/torchvision/csrc/io/video_reader/VideoReader.h
@@ -1,3 +1,5 @@
 #pragma once
 
-#include <torch/script.h>
+#include <torch/types.h>
+#include "../decoder/memory_buffer.h"
+#include "../decoder/sync_decoder.h"

From 94a819228a75a0ed9392d25cbd965f2fadb4f92c Mon Sep 17 00:00:00 2001
From: Vasilis Vryniotis <vvryniotis@fb.com>
Date: Wed, 27 Jan 2021 18:35:50 +0000
Subject: [PATCH 2/5] Rename files according to cpp styles.

---
 torchvision/csrc/io/video/{Video.cpp => video.cpp}              | 2 +-
 torchvision/csrc/io/video/{Video.h => video.h}                  | 0
 .../csrc/io/video_reader/{VideoReader.cpp => video_reader.cpp}  | 2 +-
 .../csrc/io/video_reader/{VideoReader.h => video_reader.h}      | 0
 4 files changed, 2 insertions(+), 2 deletions(-)
 rename torchvision/csrc/io/video/{Video.cpp => video.cpp} (99%)
 rename torchvision/csrc/io/video/{Video.h => video.h} (100%)
 rename torchvision/csrc/io/video_reader/{VideoReader.cpp => video_reader.cpp} (99%)
 rename torchvision/csrc/io/video_reader/{VideoReader.h => video_reader.h} (100%)

diff --git a/torchvision/csrc/io/video/Video.cpp b/torchvision/csrc/io/video/video.cpp
similarity index 99%
rename from torchvision/csrc/io/video/Video.cpp
rename to torchvision/csrc/io/video/video.cpp
index 7f494a47868..368d895b4e3 100644
--- a/torchvision/csrc/io/video/Video.cpp
+++ b/torchvision/csrc/io/video/video.cpp
@@ -1,4 +1,4 @@
-#include "Video.h"
+#include "video.h"
 
 #include <regex>
 
diff --git a/torchvision/csrc/io/video/Video.h b/torchvision/csrc/io/video/video.h
similarity index 100%
rename from torchvision/csrc/io/video/Video.h
rename to torchvision/csrc/io/video/video.h
diff --git a/torchvision/csrc/io/video_reader/VideoReader.cpp b/torchvision/csrc/io/video_reader/video_reader.cpp
similarity index 99%
rename from torchvision/csrc/io/video_reader/VideoReader.cpp
rename to torchvision/csrc/io/video_reader/video_reader.cpp
index 6a98897e0f9..182f1fdbd13 100644
--- a/torchvision/csrc/io/video_reader/VideoReader.cpp
+++ b/torchvision/csrc/io/video_reader/video_reader.cpp
@@ -1,4 +1,4 @@
-#include "VideoReader.h"
+#include "video_reader.h"
 
 #include <Python.h>
 
diff --git a/torchvision/csrc/io/video_reader/VideoReader.h b/torchvision/csrc/io/video_reader/video_reader.h
similarity index 100%
rename from torchvision/csrc/io/video_reader/VideoReader.h
rename to torchvision/csrc/io/video_reader/video_reader.h

From 5828cfd168acb22d86174884c1812f86bea8dc15 Mon Sep 17 00:00:00 2001
From: Vasilis Vryniotis <vvryniotis@fb.com>
Date: Wed, 27 Jan 2021 19:01:32 +0000
Subject: [PATCH 3/5] Adding namespaces and moving private methods to anonymous
 namespaces.

---
 torchvision/csrc/io/video/video.cpp           |  11 +-
 torchvision/csrc/io/video/video.h             |   6 +
 .../csrc/io/video_reader/video_reader.cpp     | 200 +++++++++---------
 .../csrc/io/video_reader/video_reader.h       |  54 ++++-
 4 files changed, 171 insertions(+), 100 deletions(-)

diff --git a/torchvision/csrc/io/video/video.cpp b/torchvision/csrc/io/video/video.cpp
index 368d895b4e3..a3dfb3d6221 100644
--- a/torchvision/csrc/io/video/video.cpp
+++ b/torchvision/csrc/io/video/video.cpp
@@ -2,8 +2,10 @@
 
 #include <regex>
 
-using namespace std;
-using namespace ffmpeg;
+namespace vision {
+namespace video {
+
+namespace {
 
 const size_t decoderTimeoutMs = 600000;
 const AVPixelFormat defaultVideoPixelFormat = AV_PIX_FMT_RGB24;
@@ -90,6 +92,8 @@ std::tuple<std::string, long> _parseStream(const std::string& streamString) {
   return std::make_tuple(type_, index_);
 }
 
+} // namespace
+
 void Video::_getDecoderParams(
     double videoStartS,
     int64_t getPtsOnly,
@@ -325,3 +329,6 @@ static auto registerVideo =
         .def("get_metadata", &Video::getStreamMetadata)
         .def("seek", &Video::Seek)
         .def("next", &Video::Next);
+
+} // namespace video
+} // namespace vision
diff --git a/torchvision/csrc/io/video/video.h b/torchvision/csrc/io/video/video.h
index dfb036ac851..ff28fd1d290 100644
--- a/torchvision/csrc/io/video/video.h
+++ b/torchvision/csrc/io/video/video.h
@@ -8,6 +8,9 @@
 
 using namespace ffmpeg;
 
+namespace vision {
+namespace video {
+
 struct Video : torch::CustomClassHolder {
   std::tuple<std::string, long> current_stream; // stream type, id
   // global video metadata
@@ -50,3 +53,6 @@ struct Video : torch::CustomClassHolder {
   DecoderParameters params;
 
 }; // struct Video
+
+} // namespace video
+} // namespace vision
diff --git a/torchvision/csrc/io/video_reader/video_reader.cpp b/torchvision/csrc/io/video_reader/video_reader.cpp
index 182f1fdbd13..2b8e89caa81 100644
--- a/torchvision/csrc/io/video_reader/video_reader.cpp
+++ b/torchvision/csrc/io/video_reader/video_reader.cpp
@@ -2,6 +2,9 @@
 
 #include <Python.h>
 
+#include "../decoder/memory_buffer.h"
+#include "../decoder/sync_decoder.h"
+
 // If we are in a Windows environment, we need to define
 // initialization functions for the _custom_ops extension
 #ifdef _WIN32
@@ -11,11 +14,13 @@ PyMODINIT_FUNC PyInit_video_reader(void) {
 }
 #endif
 
-using namespace std;
 using namespace ffmpeg;
 
+namespace vision {
 namespace video_reader {
 
+namespace {
+
 const AVPixelFormat defaultVideoPixelFormat = AV_PIX_FMT_RGB24;
 const AVSampleFormat defaultAudioSampleFormat = AV_SAMPLE_FMT_FLT;
 const AVRational timeBaseQ = AVRational{1, AV_TIME_BASE};
@@ -413,95 +418,6 @@ torch::List<torch::Tensor> readVideo(
   return result;
 }
 
-torch::List<torch::Tensor> readVideoFromMemory(
-    torch::Tensor input_video,
-    double seekFrameMargin,
-    int64_t getPtsOnly,
-    int64_t readVideoStream,
-    int64_t width,
-    int64_t height,
-    int64_t minDimension,
-    int64_t maxDimension,
-    int64_t videoStartPts,
-    int64_t videoEndPts,
-    int64_t videoTimeBaseNum,
-    int64_t videoTimeBaseDen,
-    int64_t readAudioStream,
-    int64_t audioSamples,
-    int64_t audioChannels,
-    int64_t audioStartPts,
-    int64_t audioEndPts,
-    int64_t audioTimeBaseNum,
-    int64_t audioTimeBaseDen) {
-  return readVideo(
-      false,
-      input_video,
-      "", // videoPath
-      seekFrameMargin,
-      getPtsOnly,
-      readVideoStream,
-      width,
-      height,
-      minDimension,
-      maxDimension,
-      videoStartPts,
-      videoEndPts,
-      videoTimeBaseNum,
-      videoTimeBaseDen,
-      readAudioStream,
-      audioSamples,
-      audioChannels,
-      audioStartPts,
-      audioEndPts,
-      audioTimeBaseNum,
-      audioTimeBaseDen);
-}
-
-torch::List<torch::Tensor> readVideoFromFile(
-    std::string videoPath,
-    double seekFrameMargin,
-    int64_t getPtsOnly,
-    int64_t readVideoStream,
-    int64_t width,
-    int64_t height,
-    int64_t minDimension,
-    int64_t maxDimension,
-    int64_t videoStartPts,
-    int64_t videoEndPts,
-    int64_t videoTimeBaseNum,
-    int64_t videoTimeBaseDen,
-    int64_t readAudioStream,
-    int64_t audioSamples,
-    int64_t audioChannels,
-    int64_t audioStartPts,
-    int64_t audioEndPts,
-    int64_t audioTimeBaseNum,
-    int64_t audioTimeBaseDen) {
-  torch::Tensor dummy_input_video = torch::ones({0});
-  return readVideo(
-      true,
-      dummy_input_video,
-      videoPath,
-      seekFrameMargin,
-      getPtsOnly,
-      readVideoStream,
-      width,
-      height,
-      minDimension,
-      maxDimension,
-      videoStartPts,
-      videoEndPts,
-      videoTimeBaseNum,
-      videoTimeBaseDen,
-      readAudioStream,
-      audioSamples,
-      audioChannels,
-      audioStartPts,
-      audioEndPts,
-      audioTimeBaseNum,
-      audioTimeBaseDen);
-}
-
 torch::List<torch::Tensor> probeVideo(
     bool isReadFile,
     const torch::Tensor& input_video,
@@ -646,6 +562,97 @@ torch::List<torch::Tensor> probeVideo(
   return result;
 }
 
+} // namespace
+
+torch::List<torch::Tensor> readVideoFromMemory(
+    torch::Tensor input_video,
+    double seekFrameMargin,
+    int64_t getPtsOnly,
+    int64_t readVideoStream,
+    int64_t width,
+    int64_t height,
+    int64_t minDimension,
+    int64_t maxDimension,
+    int64_t videoStartPts,
+    int64_t videoEndPts,
+    int64_t videoTimeBaseNum,
+    int64_t videoTimeBaseDen,
+    int64_t readAudioStream,
+    int64_t audioSamples,
+    int64_t audioChannels,
+    int64_t audioStartPts,
+    int64_t audioEndPts,
+    int64_t audioTimeBaseNum,
+    int64_t audioTimeBaseDen) {
+  return readVideo(
+      false,
+      input_video,
+      "", // videoPath
+      seekFrameMargin,
+      getPtsOnly,
+      readVideoStream,
+      width,
+      height,
+      minDimension,
+      maxDimension,
+      videoStartPts,
+      videoEndPts,
+      videoTimeBaseNum,
+      videoTimeBaseDen,
+      readAudioStream,
+      audioSamples,
+      audioChannels,
+      audioStartPts,
+      audioEndPts,
+      audioTimeBaseNum,
+      audioTimeBaseDen);
+}
+
+torch::List<torch::Tensor> readVideoFromFile(
+    std::string videoPath,
+    double seekFrameMargin,
+    int64_t getPtsOnly,
+    int64_t readVideoStream,
+    int64_t width,
+    int64_t height,
+    int64_t minDimension,
+    int64_t maxDimension,
+    int64_t videoStartPts,
+    int64_t videoEndPts,
+    int64_t videoTimeBaseNum,
+    int64_t videoTimeBaseDen,
+    int64_t readAudioStream,
+    int64_t audioSamples,
+    int64_t audioChannels,
+    int64_t audioStartPts,
+    int64_t audioEndPts,
+    int64_t audioTimeBaseNum,
+    int64_t audioTimeBaseDen) {
+  torch::Tensor dummy_input_video = torch::ones({0});
+  return readVideo(
+      true,
+      dummy_input_video,
+      videoPath,
+      seekFrameMargin,
+      getPtsOnly,
+      readVideoStream,
+      width,
+      height,
+      minDimension,
+      maxDimension,
+      videoStartPts,
+      videoEndPts,
+      videoTimeBaseNum,
+      videoTimeBaseDen,
+      readAudioStream,
+      audioSamples,
+      audioChannels,
+      audioStartPts,
+      audioEndPts,
+      audioTimeBaseNum,
+      audioTimeBaseDen);
+}
+
 torch::List<torch::Tensor> probeVideoFromMemory(torch::Tensor input_video) {
   return probeVideo(false, input_video, "");
 }
@@ -655,11 +662,12 @@ torch::List<torch::Tensor> probeVideoFromFile(std::string videoPath) {
   return probeVideo(true, dummy_input_video, videoPath);
 }
 
-} // namespace video_reader
-
 TORCH_LIBRARY_FRAGMENT(video_reader, m) {
-  m.def("read_video_from_memory", video_reader::readVideoFromMemory);
-  m.def("read_video_from_file", video_reader::readVideoFromFile);
-  m.def("probe_video_from_memory", video_reader::probeVideoFromMemory);
-  m.def("probe_video_from_file", video_reader::probeVideoFromFile);
+  m.def("read_video_from_memory", readVideoFromMemory);
+  m.def("read_video_from_file", readVideoFromFile);
+  m.def("probe_video_from_memory", probeVideoFromMemory);
+  m.def("probe_video_from_file", probeVideoFromFile);
 }
+
+} // namespace video_reader
+} // namespace vision
diff --git a/torchvision/csrc/io/video_reader/video_reader.h b/torchvision/csrc/io/video_reader/video_reader.h
index e8b5cca99a5..d27cd6c28af 100644
--- a/torchvision/csrc/io/video_reader/video_reader.h
+++ b/torchvision/csrc/io/video_reader/video_reader.h
@@ -1,5 +1,55 @@
 #pragma once
 
 #include <torch/types.h>
-#include "../decoder/memory_buffer.h"
-#include "../decoder/sync_decoder.h"
+
+namespace vision {
+namespace video_reader {
+
+torch::List<torch::Tensor> readVideoFromMemory(
+    torch::Tensor input_video,
+    double seekFrameMargin,
+    int64_t getPtsOnly,
+    int64_t readVideoStream,
+    int64_t width,
+    int64_t height,
+    int64_t minDimension,
+    int64_t maxDimension,
+    int64_t videoStartPts,
+    int64_t videoEndPts,
+    int64_t videoTimeBaseNum,
+    int64_t videoTimeBaseDen,
+    int64_t readAudioStream,
+    int64_t audioSamples,
+    int64_t audioChannels,
+    int64_t audioStartPts,
+    int64_t audioEndPts,
+    int64_t audioTimeBaseNum,
+    int64_t audioTimeBaseDen);
+
+torch::List<torch::Tensor> readVideoFromFile(
+    std::string videoPath,
+    double seekFrameMargin,
+    int64_t getPtsOnly,
+    int64_t readVideoStream,
+    int64_t width,
+    int64_t height,
+    int64_t minDimension,
+    int64_t maxDimension,
+    int64_t videoStartPts,
+    int64_t videoEndPts,
+    int64_t videoTimeBaseNum,
+    int64_t videoTimeBaseDen,
+    int64_t readAudioStream,
+    int64_t audioSamples,
+    int64_t audioChannels,
+    int64_t audioStartPts,
+    int64_t audioEndPts,
+    int64_t audioTimeBaseNum,
+    int64_t audioTimeBaseDen);
+
+torch::List<torch::Tensor> probeVideoFromMemory(torch::Tensor input_video);
+
+torch::List<torch::Tensor> probeVideoFromFile(std::string videoPath);
+
+} // namespace video_reader
+} // namespace vision

From 9f8ca89768fc5ab65e2fe7cfb881c141d6329b92 Mon Sep 17 00:00:00 2001
From: Vasilis Vryniotis <vvryniotis@fb.com>
Date: Wed, 27 Jan 2021 19:15:40 +0000
Subject: [PATCH 4/5] Syncing method names.

---
 .../csrc/io/video_reader/video_reader.cpp        | 16 ++++++++--------
 torchvision/csrc/io/video_reader/video_reader.h  |  8 ++++----
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/torchvision/csrc/io/video_reader/video_reader.cpp b/torchvision/csrc/io/video_reader/video_reader.cpp
index 2b8e89caa81..1b5df64c93b 100644
--- a/torchvision/csrc/io/video_reader/video_reader.cpp
+++ b/torchvision/csrc/io/video_reader/video_reader.cpp
@@ -564,7 +564,7 @@ torch::List<torch::Tensor> probeVideo(
 
 } // namespace
 
-torch::List<torch::Tensor> readVideoFromMemory(
+torch::List<torch::Tensor> read_video_from_memory(
     torch::Tensor input_video,
     double seekFrameMargin,
     int64_t getPtsOnly,
@@ -608,7 +608,7 @@ torch::List<torch::Tensor> readVideoFromMemory(
       audioTimeBaseDen);
 }
 
-torch::List<torch::Tensor> readVideoFromFile(
+torch::List<torch::Tensor> read_video_from_file(
     std::string videoPath,
     double seekFrameMargin,
     int64_t getPtsOnly,
@@ -653,20 +653,20 @@ torch::List<torch::Tensor> readVideoFromFile(
       audioTimeBaseDen);
 }
 
-torch::List<torch::Tensor> probeVideoFromMemory(torch::Tensor input_video) {
+torch::List<torch::Tensor> probe_video_from_memory(torch::Tensor input_video) {
   return probeVideo(false, input_video, "");
 }
 
-torch::List<torch::Tensor> probeVideoFromFile(std::string videoPath) {
+torch::List<torch::Tensor> probe_video_from_file(std::string videoPath) {
   torch::Tensor dummy_input_video = torch::ones({0});
   return probeVideo(true, dummy_input_video, videoPath);
 }
 
 TORCH_LIBRARY_FRAGMENT(video_reader, m) {
-  m.def("read_video_from_memory", readVideoFromMemory);
-  m.def("read_video_from_file", readVideoFromFile);
-  m.def("probe_video_from_memory", probeVideoFromMemory);
-  m.def("probe_video_from_file", probeVideoFromFile);
+  m.def("read_video_from_memory", read_video_from_memory);
+  m.def("read_video_from_file", read_video_from_file);
+  m.def("probe_video_from_memory", probe_video_from_memory);
+  m.def("probe_video_from_file", probe_video_from_file);
 }
 
 } // namespace video_reader
diff --git a/torchvision/csrc/io/video_reader/video_reader.h b/torchvision/csrc/io/video_reader/video_reader.h
index d27cd6c28af..48c4c841219 100644
--- a/torchvision/csrc/io/video_reader/video_reader.h
+++ b/torchvision/csrc/io/video_reader/video_reader.h
@@ -5,7 +5,7 @@
 namespace vision {
 namespace video_reader {
 
-torch::List<torch::Tensor> readVideoFromMemory(
+torch::List<torch::Tensor> read_video_from_memory(
     torch::Tensor input_video,
     double seekFrameMargin,
     int64_t getPtsOnly,
@@ -26,7 +26,7 @@ torch::List<torch::Tensor> readVideoFromMemory(
     int64_t audioTimeBaseNum,
     int64_t audioTimeBaseDen);
 
-torch::List<torch::Tensor> readVideoFromFile(
+torch::List<torch::Tensor> read_video_from_file(
     std::string videoPath,
     double seekFrameMargin,
     int64_t getPtsOnly,
@@ -47,9 +47,9 @@ torch::List<torch::Tensor> readVideoFromFile(
     int64_t audioTimeBaseNum,
     int64_t audioTimeBaseDen);
 
-torch::List<torch::Tensor> probeVideoFromMemory(torch::Tensor input_video);
+torch::List<torch::Tensor> probe_video_from_memory(torch::Tensor input_video);
 
-torch::List<torch::Tensor> probeVideoFromFile(std::string videoPath);
+torch::List<torch::Tensor> probe_video_from_file(std::string videoPath);
 
 } // namespace video_reader
 } // namespace vision

From 4802cdebf3e71a7132dce30c7436b47ccb3806e3 Mon Sep 17 00:00:00 2001
From: Vasilis Vryniotis <vvryniotis@fb.com>
Date: Wed, 27 Jan 2021 19:25:26 +0000
Subject: [PATCH 5/5] Fixing minor issues.

---
 torchvision/csrc/io/video/video.cpp | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/torchvision/csrc/io/video/video.cpp b/torchvision/csrc/io/video/video.cpp
index a3dfb3d6221..bda9725570e 100644
--- a/torchvision/csrc/io/video/video.cpp
+++ b/torchvision/csrc/io/video/video.cpp
@@ -160,7 +160,7 @@ Video::Video(std::string videoPath, std::string stream) {
   Video::_getDecoderParams(
       0, // video start
       0, // headerOnly
-      get<0>(current_stream), // stream info - remove that
+      std::get<0>(current_stream), // stream info - remove that
       long(-1), // stream_id parsed from info above change to -2
       true // read all streams
   );
@@ -210,9 +210,9 @@ Video::Video(std::string videoPath, std::string stream) {
 
   succeeded = Video::setCurrentStream(stream);
   LOG(INFO) << "\nDecoder inited with: " << succeeded << "\n";
-  if (get<1>(current_stream) != -1) {
+  if (std::get<1>(current_stream) != -1) {
     LOG(INFO)
-        << "Stream index set to " << get<1>(current_stream)
+        << "Stream index set to " << std::get<1>(current_stream)
         << ". If you encounter trouble, consider switching it to automatic stream discovery. \n";
   }
 } // video
@@ -230,8 +230,8 @@ bool Video::setCurrentStream(std::string stream = "video") {
   _getDecoderParams(
       ts, // video start
       0, // headerOnly
-      get<0>(current_stream), // stream
-      long(get<1>(
+      std::get<0>(current_stream), // stream
+      long(std::get<1>(
           current_stream)), // stream_id parsed from info above change to -2
       false // read all streams
   );
@@ -254,8 +254,8 @@ void Video::Seek(double ts) {
   _getDecoderParams(
       ts, // video start
       0, // headerOnly
-      get<0>(current_stream), // stream
-      long(get<1>(
+      std::get<0>(current_stream), // stream
+      long(std::get<1>(
           current_stream)), // stream_id parsed from info above change to -2
       false // read all streams
   );