Fix negative index handling in MultiHeadAttention attention_axes

Author: utsab345

integration_tests/test_multi_head_attention_negative_axis.py

@@ -0,0 +1,22 @@
+import numpy as np
+import keras
+
+def test_attention_axes_negative_indexing_matches_positive():
+    x = np.random.normal(size=(2, 3, 8, 4))
+    
+    mha_pos = keras.layers.MultiHeadAttention(num_heads=2, key_dim=4, attention_axes=2)
+    mha_neg = keras.layers.MultiHeadAttention(num_heads=2, key_dim=4, attention_axes=-2)
+    
+    _ = mha_pos(x, x)
+    _ = mha_neg(x, x)
+    
+    mha_neg.set_weights(mha_pos.get_weights())
+    
+    z_pos, a_pos = mha_pos(x, x, return_attention_scores=True)
+    z_neg, a_neg = mha_neg(x, x, return_attention_scores=True)
+
+    assert z_pos.shape == z_neg.shape
+    assert a_pos.shape == a_neg.shape
+
+    np.testing.assert_allclose(z_pos, z_neg, rtol=1e-5, atol=1e-5)
+    np.testing.assert_allclose(a_pos, a_neg, rtol=1e-5, atol=1e-5)

keras/src/layers/attention/multi_head_attention.py

@@ -378,7 +378,17 @@ def _build_attention(self, rank):
         if self._attention_axes is None:
             self._attention_axes = tuple(range(1, rank - 2))
         else:
-            self._attention_axes = tuple(self._attention_axes)
+            # Normalize negative indices relative to INPUT rank (rank - 1)
+            input_rank = rank - 1
+            normalized_axes = []
+            for ax in self._attention_axes:
+                if ax < 0:
+                    # Normalize relative to input rank
+                    normalized_ax = input_rank + ax
+                else:
+                    normalized_ax = ax
+                normalized_axes.append(normalized_ax)
+            self._attention_axes = tuple(normalized_axes)
         (
             self._dot_product_equation,
             self._combine_equation,
@@ -760,6 +770,12 @@ def _build_attention_equation(rank, attn_axes):
     Returns:
         Einsum equations.
     """
+    # Normalize negative indices to positive indices
+    if isinstance(attn_axes, (list, tuple)):
+        attn_axes = tuple(ax % rank if ax < 0 else ax for ax in attn_axes)
+    else:
+        attn_axes = (attn_axes % rank if attn_axes < 0 else attn_axes,)
+
     target_notation = ""
     for i in range(rank):
         target_notation += _index_to_einsum_variable(i)