change units to SI for radii array in Bolin table_1.ipynb

[AgnieszkaZaba]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.22%. Comparing base (824a94d) to head (9c8cd53).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1493   +/-   ##
=======================================
  Coverage   85.22%   85.22%           
=======================================
  Files         376      376           
  Lines        9266     9266           
=======================================
  Hits         7897     7897           
  Misses       1369     1369           

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

[slayoo]

@AgnieszkaZaba, the original code seems more intuitive as it matches what's in the paper's table column 1

[AgnieszkaZaba]

@AgnieszkaZaba, the original code seems more intuitive as it matches what's in the paper's table column 1

So maybe rename it to radii_cm?

[slayoo]

@AgnieszkaZaba, the original code seems more intuitive as it matches what's in the paper's table column 1

So maybe rename it to radii_cm?

Since we multiply the values by si.cm, what get stored into the variable is in metres, hence such name would be misleading.

[AgnieszkaZaba]

Closing as unnecessary:)