fix: consider length scale in normalization

[severinstrobl]

When examining the length of the normal vector obtained using Newell's method prior to the actual normalization, the length scale has to be taken into account.

Fixes: #96

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.22%. Comparing base (ad9bde3) to head (7b56b79).
Report is 44 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop      #98      +/-   ##
===========================================
+ Coverage    93.30%   95.22%   +1.91%     
===========================================
  Files            1        1              
  Lines          851      816      -35     
  Branches       156      145      -11     
===========================================
- Hits           794      777      -17     
+ Misses          10        4       -6     
+ Partials        47       35      -12     

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

[github-actions]

C++ benchmark results

Benchmark	Reference (351d479)	Current (d447ddb)
hex_overlap_volume_aabb	8.46 ns	8.36 ns: 1.01x faster
normal_newell	66.2 ns	65.5 ns: 1.01x faster
tet_overlap_volume	1.29 us	1.28 us: 1.01x faster
Geometric mean	(ref)	1.01x faster

Benchmark hidden because not significant (3): hex_overlap_volume, regularized_wedge, tet_overlap_volume_aabb

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud