Unify hash_mc_evictions.py (meta-pytorch#3866)

Summary:

Context
---------

The goal is to remove hash_mc_evictions to unify internal with OSS.

Implementation
------------------

- The only difference between internal and OSS, is one assertion statement.
- Internal is imported in over 100 different places.
- Inside Internal:
    - Added a simple depreciated warning, and imported from OSS, in this case.
    - Removed the internal tests: `fb/modules/tests/test_hash_mc_evictions.py`
- Changed all internal imports for fb/ within TREC to be OSS.
    - External imports will be removed in a seperate diff

Differential Revision: D96242594

Author: Ali-Tehrani

torchrec/modules/hash_mc_evictions.py

@@ -64,8 +64,8 @@ def gen_threshold(self) -> int:
 class HashZchSingleTtlScorer(HashZchEvictionScorer):
     def gen_score(self, feature: JaggedTensor, device: torch.device) -> torch.Tensor:
         assert (
-            self._config.single_ttl is not None and self._config.single_ttl > 0
-        ), "To use scorer HashZchSingleTtlScorer, a positive single_ttl is required."
+            self._config.single_ttl is not None
+        ), "To use scorer HashZchSingleTtlScorer, a single_ttl is required."
 
         return torch.full_like(
             feature.values(),