Deprecating certain functional APIs and fix an Error msg (#890)

Summary:
- Continue to deprecate stuff based on #163
- Fix a typo

Pull Request resolved: #890

Reviewed By: NivekT

Differential Revision: D41130080

Pulled By: ejguan

fbshipit-source-id: d8c1a6a4732211fcf75a08d5cb5e090a587b555b

Author: ejguan

torchdata/datapipes/iter/load/fsspec.py

@@ -148,10 +148,6 @@ def __len__(self) -> int:
         return len(self.source_datapipe)
 
 
-# Register for functional API for backward compatibility
-IterDataPipe.register_datapipe_as_function("open_file_by_fsspec", FSSpecFileOpenerIterDataPipe)
-
-
 @functional_datapipe("save_by_fsspec")
 class FSSpecSaverIterDataPipe(IterDataPipe[str]):
     r"""

torchdata/datapipes/iter/load/iopath.py

@@ -142,10 +142,6 @@ def __len__(self) -> int:
         return len(self.source_datapipe)
 
 
-# Register for functional API for backward compatibility
-IterDataPipe.register_datapipe_as_function("open_file_by_iopath", IoPathFileOpenerIterDataPipe)
-
-
 @functional_datapipe("save_by_iopath")
 class IoPathSaverIterDataPipe(IterDataPipe[str]):
 

torchdata/datapipes/iter/util/converter.py

@@ -90,7 +90,7 @@ def __getitem__(self, index):
                 self._load_map()
             return self._map[index]  # type: ignore[index]
         except KeyError:
-            raise IndexError(f"Index {index} is valid for IterToMapConverter.")
+            raise IndexError(f"Index {index} is invalid for IterToMapConverter.")
 
     def __len__(self):
         if self._map is not None: