[BUG] Wrong assertion in integer_subbyte.h

[Algy]

Describe the bug
I got an Assertion failure while I tried 55_hopper_int4_fp8_gemm

This was due to the following incorrect assertion logic at [this line]:(

cutlass/include/cutlass/integer_subbyte.h

Line 96 in 8aa95db

assert(value < upper_bound);

):

      [[maybe_unused]] constexpr int lower_bound = -(1 << (Bits - 1));
      [[maybe_unused]] constexpr int upper_bound = (1 << (Bits - 1)) - 1;
      assert(value >= lower_bound);
      assert(value < upper_bound); # HERE

The assertion should be assert(value <= upper_bound) as the upper_bound is inclusive.

[foreverlms]

I got the same error and this assertion should be what you said. I think it should be:

cutlass/include/cutlass/integer_subbyte.h

Lines 95 to 96 in 8aa95db

	assert(value >= lower_bound);
	assert(value < upper_bound);

:

  assert(value > lower_bound);
  assert(value <= upper_bound); 

[Algy]

@foreverlms the lower bound is inclusive as well since the domain of an integer is [-2**(Bits-1), 2**(Bits-1) - 1].
For example the domain of 8-bit integer (i.e., Bits=8) is [-128, 127].