Fix: setPlotValue() early return preventing logic execution

[s1dhu98]

Description

Fixes the early return statement in the setPlotValue() function that prevented all plotting logic from executing.

Changes

• Removed bare return statement on line 146 that was blocking function logic

Fixes

Closes #890

Summary by CodeRabbit

• Bug Fixes
  • Fixed plotting behavior that was previously skipped on initial invocations, ensuring consistent visualization updates.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[netlify]

✅ Deploy Preview for circuitverse ready!

Name	Link
🔨 Latest commit	3f836ba
🔍 Latest deploy log	https://app.netlify.com/projects/circuitverse/deploys/697b710b6d7d1f0008755e02
😎 Deploy Preview	https://deploy-preview-891--circuitverse.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 45 (🟢 up 1 from production) Accessibility: 73 (no change from production) Best Practices: 92 (no change from production) SEO: 82 (no change from production) PWA: - View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

Walkthrough

An early return statement was removed from the setPlotValue() function in testbenchInput.js. Previously, this statement caused the function to exit immediately, making all subsequent logic unreachable. With this removal, the function now executes its intended behavior to compute elapsed time from the stopwatch and process plot value tracking operations during each invocation.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing an early return statement from setPlotValue() that was preventing logic execution.
Linked Issues check	✅ Passed	The PR directly addresses issue #890 by removing the early return statement from setPlotValue() as required, restoring the function's ability to execute its logic.
Out of Scope Changes check	✅ Passed	The PR contains only the necessary change to fix the issue: removal of one early return statement from setPlotValue(). No extraneous modifications are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[s1dhu98]

@Nihal4777 please review the changes and merge them if possible

[Radhikaa-chauhan]

@s1dhu98 Please update the PR description to follow the standard template and include valid proof for the changes before requesting a review. Thanks!