Simplify TensorStats Inheritance

[kylebd99]

This PR is a small cleanup within the TensorStats folder. The goal is to remove TensorDef and instead use inheritance to manage the shared functionality.

[codspeed-hq]

Merging this PR will degrade performance by 11.38%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 4 regressed benchmarks
✅ 28 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Mode	Benchmark	BASE	HEAD	Efficiency
❌	WallTime	test_ops_binary[interpret_galley-add]	787 ms	899 ms	-12.47%
❌	WallTime	test_ops_binary[interpret_galley-multiply]	786.2 ms	888 ms	-11.47%
❌	WallTime	test_ops_reduction[interpret_galley-sum]	419.5 ms	471.4 ms	-11%
❌	WallTime	test_f1_jit[interpret_galley]	11.2 s	12.6 s	-10.56%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

---

_{Comparing kbd-refactor-tensor-stats (c322bda) with main (64775fa)}

[willow-ahrens]

Some of these methods seem like they should be calling super(), but instead they're passing around the parent where the def used to be, which is a little bit of a mismatch in the programming pattern

[willow-ahrens]

This part seems a little strange? I think we should just call super().mapjoin_union?

[kylebd99]

It's a bit tricky because the class is a factory not a statistic, so calling super().mapjoin creates a BaseTensorStats object. We essentially do this within the BaseTensorStatsFactory.mapjoin implementation, then pass it to the _mapjoin_union/_mapjoin_join functions in the child classes. We can instead do that call within every _mapjoin_union/_mapjoin_join implementation though (and I think that may be cleaner).

[kylebd99]

Just made that change