Added comparison function as histogram operation (with colorized + info based string return)

[gohil-jay]

[henryiii]

Suggested change

	return "The axes [" + "\033[91m" + str(list(self.axes)) + " and " + str(list(hist2.axes)) + "\033[0m" + "] are not equal."
	return f"The axes [\033[91m{list(self.axes)} and {list(hist2.axes)}\033[0m] are not equal."

F-strings were made for this. :) Also I'd pull out the color formatting into a variable, like red and reset. Actually, there are lots of concerns with color formatting (needs to be disablable and such), at least rich can auto-format this, so probably leave the color formatting off for the moment.

[henryiii]

I bet if you do import rich; rich.print(h.compare(h2)) this would still be colorful. ;)

[gohil-jay]

Yes I was going to do something similar (with termcolor), but then though I'd do it without importing a library..

[gohil-jay]

@henryiii, pre-commit throws few errors... Any feedback/suggestion?

[henryiii]

It's complaining because you are using the result of re.search without checking to see if it has a result first. I think the best fix, though, is to use ._storage_type, which contains the storage type of the histogram, rather than trying to parse it out of the repr.

[gohil-jay]

I've been looking for something like _storage_type since the beginning (couldn't find it on docs or book here)! But alright now it should work :)

[gohil-jay]

Seems alright now (?)

[henryiii]

Note: the goal is to support pytest_assertrepr_compare https://docs.pytest.org/en/latest/how-to/assert.html#defining-your-own-explanation-for-failed-assertions