Fix units for virtual packet properties in MonteCarloTransportState

[sarthak-dv]

📝 Description

Type: 🪲 bugfix

Fixed the units in properties for virtual packets in MonteCarloTransportState.

Resolves #2540

🚦 Testing

How did you test these changes?

• Testing pipeline
• Other method (describe)
• My changes can't be tested (explain why)

☑️ Checklist

• I requested two reviewers for this pull request
• I updated the documentation according to my changes
• I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

[tardis-bot]

*beep* *bop*

Hi, human.

The docs workflow has succeeded ✔️

Click here to see your results.

[sarthak-dv]

@wkerzendorf @andrewfullard Please check this change, as this seems an fundamental one

[andrewfullard]

Thank you for this fix! Looks like some copy-paste got us here. Your changes are correct, I think.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.73%. Comparing base (f515d16) to head (7b18b38).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2541   +/-   ##
=======================================
  Coverage   68.73%   68.73%           
=======================================
  Files         165      165           
  Lines       14001    14001           
=======================================
  Hits         9624     9624           
  Misses       4377     4377           

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

[AlexHls]

Looks good to me. Only formatting needs to be fixed.

[andrewfullard]

Yes please run black on your changes

[sarthak-dv]

@andrewfullard
I've run the black formatter, please check