Dimitrie/cnx 510 revit parameter exports perfection

[didimitrie]

Fixes a bunch of tickets from revit parameters.

[linear]

CNX-510 Revit - Parameter Exports Perfection

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 37 lines in your changes missing coverage. Please review.

Project coverage is 8.67%. Comparing base (188f23f) to head (1e53700).
Report is 11 commits behind head on dev.

Files with missing lines	Patch %	Lines
...tShared/ToSpeckle/Parameters/ParameterExtractor.cs	0.00%	34 Missing ⚠️
...hared/ToSpeckle/Raw/MaterialQuantitiesToSpeckle.cs	0.00%	2 Missing ⚠️
...verters.RevitShared/RevitRootToSpeckleConverter.cs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##             dev    #300      +/-   ##
========================================
- Coverage   8.72%   8.67%   -0.05%     
========================================
  Files        232     232              
  Lines       4641    4666      +25     
  Branches     542     551       +9     
========================================
  Hits         405     405              
- Misses      4219    4244      +25     
  Partials      17      17              

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

[clairekuang]

Generally looks fine to me. Approving, but the object type for the param name dictionary took a bit to understand: passing a tuple in the case of ElementType to be parsed later - more comments wouldn't hurt here