Fix rascal summary

[rodinaarssen]

The Rascal summary now uses physical locations instead of logical ones

[jurgenvinju]

That's cool.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46%. Comparing base (2b63337) to head (aaeb9a9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##              main   #2630   +/-   ##
=======================================
  Coverage       46%     46%           
+ Complexity    6658    6653    -5     
=======================================
  Files          795     795           
  Lines        65856   65856           
  Branches      9871    9871           
=======================================
+ Hits         30643   30649    +6     
+ Misses       32847   32842    -5     
+ Partials      2366    2365    -1     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[PaulKlint]

Looks fine to me. Good that you fixed the tests (although I don't think we run them by default)

For the record: if ever performance issues will occur for makeSummary, we might opt to only convert the result of makeSummary and not the complete TModel