fix: TProfile2D/3D weighted and counts() were broken for N-D profiles

[henryiii]

🤖 AI text below 🤖

Part of #1646

Summary

• TProfile2D/3D weighted property (TProfile2D.py:36-39, TProfile3D.py:40-43): fNcells is an int, so len(self.member("fNcells")) raised TypeError whenever fBinSumw2 existed. Additionally the logic was inverted relative to the correct 1-D version in TProfile.py:237-240. Both files now match the 1-D version: fBinSumw2 is not None and len(fBinSumw2) == self.member("fNcells").

• TProfile2D/3D counts() shape (TProfile2D.py:41-52, TProfile3D.py:45-56): out.reshape(fBinEntries.shape) was a no-op because fBinEntries is stored flat (shape (fNcells,)). The subsequent out[1:-1, 1:-1] / out[1:-1, 1:-1, 1:-1] slices then raised IndexError. The fix applies the same reshape + numpy.transpose pattern already used by values() in both classes, ensuring counts(flow=…).shape == values(flow=…).shape.

• ⚠️ BEHAVIOR CHANGE — TProfile.counts() default flow (TProfile.py:242): Changed default from flow=True to flow=False. Every other counts() in the codebase defaults to flow=False (the abstract Profile.counts at TProfile.py:142, TProfile2D.counts, TProfile3D.counts, TH1.Histogram.counts). The flow=True default was an inconsistency. Maintainer: please confirm this behavior change is acceptable — any downstream code calling h.counts() on a 1-D TProfile without an explicit flow= argument will now receive the no-flow array (length fNbins) instead of the with-flow array (length fNbins + 2).

• Python-2 __ne__ cleanup (TH1.py:85-89, TAxis.py:122-126): Removed explicit __ne__ methods with the comment "Some versions of Python don't automatically negate __eq__". Python 3 auto-derives __ne__ from __eq__, so these are dead code.

Test plan

• weighted property no longer raises TypeError on uproot-issue-227a.root (TProfile2D) and uproot-issue-227b.root (TProfile3D)
• counts(flow=True).shape == values(flow=True).shape for both TProfile2D and TProfile3D
• counts(flow=False).shape == values(flow=False).shape for both TProfile2D and TProfile3D
• Existing TProfile tests still pass (tests/test_0228_read_TProfiles.py, tests/test_1609_tprofiles.py)
• Regression tests added for all fixed behaviors

🤖 Generated with Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.85%. Comparing base (1c06db0) to head (51d2e73).
⚠️ Report is 16 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Files with missing lines	Coverage Δ
src/uproot/behaviors/TAxis.py	87.28% <ø> (+0.62%)	⬆️
src/uproot/behaviors/TH1.py	83.10% <ø> (+0.44%)	⬆️
src/uproot/behaviors/TProfile.py	85.87% <100.00%> (ø)
src/uproot/behaviors/TProfile2D.py	64.06% <100.00%> (+13.26%)	⬆️
src/uproot/behaviors/TProfile3D.py	65.21% <100.00%> (+12.97%)	⬆️

... and 1 file with indirect coverage changes