Jitter noise PR

[sambhavnoobcoder]

Issue Description

This pull request addresses a bug in the Switch Transformers architecture where the jitter noise (intended to be applied only for routing decisions) was also being unintentionally applied to the expert inputs.

Fixes : #33969

Problem Statement

In Switch Transformers, a small amount of jitter noise is added to the inputs at routing time to ensure route diversity. However, these jittered inputs were incorrectly passed along to the experts, which contradicts the original paper’s design and led to unexpected discrepancies in outputs.

Root Cause

It was discovered that the code used the same hidden states for both routing and expert processing. When jitter noise was enabled in training mode, it directly modified these states in place, causing the experts to receive noisy inputs.

Implementation

1. We now clone the original hidden states before applying jitter noise.
2. A separate copy is used exclusively for computing router logits and probabilities.
3. The unchanged hidden states are then fed to the experts to maintain the original semantics.

Screenshot

Test Cases

1. test_router_training_mode
   • Objective: Ensures that jitter noise is only applied during training.
   • Checks that outputs differ between consecutive runs (due to noise) but original inputs remain unchanged.

2. **test_router_jitter_noise_separation **
   • Objective: Verifies that jitter noise affects only the router’s internal computations and not the expert inputs.
   • Confirms the logits differ when jitter is applied, while the main input stays the same.

3. test_expert_inputs_consistency
   • Objective: Asserts that all expert inputs remain consistent, even when jitter is applied during training.
   • Uses a forward hook on the first expert to capture its inputs across multiple runs and compares them.

With these changes and test additions, we ensure that Switch Transformers adhere to the original design while preserving backward compatibility and correctness.

cc : @ArthurZucker

original pr refrence : #35847

[github-actions]

[For maintainers] Suggested jobs to run (before merge)

run-slow: switch_transformers

[sambhavnoobcoder]

@ArthurZucker can you please have a look at this ? this a copy of an already pre approved branch #35847 , looking to get this merged .