Sync fbsync branch to main (#2210)

* Move tensor to CPU before converting to string for unpickle

Summary:
The loaded tensor could be on CUDA, whose data cannot to converted to a std::string.

This change moves tensor to CPU first. Then data can be converted.

Reviewed By: mortzur

Differential Revision: D48057806

fbshipit-source-id: ad495bb2e84f9aab5120d5462d4d1c381719d014

* Del `(object)` from 10 inc pytorch/tensorboardX/tensorboardX/record_writer.py

Summary: Python3 makes the use of `(object)` in class inheritance unnecessary. Let's modernize our code by eliminating this.

Reviewed By: meyering

Differential Revision: D48958009

fbshipit-source-id: aa42e55a827d83aadf1b10b2de79317c3761ab7a

---------

Co-authored-by: Henry Hu <[email protected]>
Co-authored-by: Richard Barnes <[email protected]>

Author: 3 people

torchtext/csrc/register_torchbindings.cpp

@@ -67,6 +67,7 @@ TORCH_LIBRARY_FRAGMENT(torchtext, m) {
           },
           // __setstate__
           [](torch::Tensor state) -> c10::intrusive_ptr<SentencePiece> {
+            state = state.to(at::kCPU);
             auto* data = static_cast<char*>(state.data_ptr());
             auto numel = state.size(0);
             return c10::make_intrusive<SentencePiece>(std::string(data, numel));

torchtext/data/utils.py

@@ -228,7 +228,7 @@ def _get_ngrams(n):
             yield " ".join(x)
 
 
-class RandomShuffler(object):
+class RandomShuffler:
     """Use random functions while keeping track of the random state to make it
     reproducible and deterministic."""
 

torchtext/vocab/vectors.py

@@ -31,7 +31,7 @@ def _infer_shape(f):
     return num_lines, vector_dim
 
 
-class Vectors(object):
+class Vectors:
     def __init__(self, name, cache=None, url=None, unk_init=None, max_vectors=None) -> None:
         """
         Args: