Fix: SemanticModel.same_model compares model attribute

[MichelleArk]

resolves #9548

Problem

• semantic models were always getting picked up as part of state:modifed because the SemanticModel.same_model was comparing the wrong things to each other (self.model == self.same_model (method)), which always returned false

Solution

• Fix SemanticModel.same_model to actually compare the right attributes!

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
• This PR includes type annotations for new and modified functions

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f732b76) 87.88% compared to head (e572dbd) 87.96%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9549      +/-   ##
==========================================
+ Coverage   87.88%   87.96%   +0.07%     
==========================================
  Files         167      167              
  Lines       22170    22170              
==========================================
+ Hits        19485    19501      +16     
+ Misses       2685     2669      -16     

Flag	Coverage Δ
integration	85.53% <100.00%> (+0.07%)	⬆️
unit	61.90% <100.00%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[QMalcolm]

Makes sense! Thank you for finding and fixing this ❤️