[nightcityblade]

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Optimization
• Example
• Documentation

Description

Remove two commented-out sparse-row subtraction snippets from addScalarBC!. The issue discussion confirms that restoring those snippets would reintroduce the implicit-term bug, while the active sparse row replacement remains covered by the existing boundary-condition tests.

Related Issues & Documents

• I have created an Issue that is paired with this PR
• Closes Dead code in MOLE.jl scalarBC.jl #493

QA Instructions, Screenshots, Recordings

• Julia 1.12.6 on macOS: all Julia test sources passed (228/228 assertions), including 14/14 addScalarBC! assertions.
• JuliaFormatter.format("julia/MOLE.jl", overwrite=false, verbose=true) passed.
• git diff --check passed.

No UI changes or screenshots apply.

Keep-open request

• I am requesting maintainer review for keep-open.

Reason: N/A

Added/updated tests?

We encourage you to test all code included with MOLE, including examples.

• Yes
• No, and this is why: the change only removes inert commented-out code; the complete existing Julia test sources were run instead.
• I need help with writing tests

Read Contributing Guide and Code of Conduct

• 📖 I have read the MOLE Contributing Guide: https://github.com/csrc-sdsu/mole/blob/main/CONTRIBUTING.md
• 📖 I have read the MOLE Code of Conduct: https://github.com/csrc-sdsu/mole/blob/main/CODE_OF_CONDUCT.md

[optional] Are there any post deployment tasks we need to perform?

N/A

[optional] What gif best describes this PR or how it makes you feel?

[valeriabarra]

Hello @nightcityblade, we cannot accept contributions from anonymous users (i.e., users that do not have a full display name in their GitHub account).

[github-actions]

This pull request has had no activity for 14 days. It will be marked stale in 14 days if there is no further activity.