Skip to content

Fix "Sum Total" not addressing unit conversion for others than length/area/volume#2875

Merged
jmcouffin merged 1 commit intopyrevitlabs:developfrom
Wurschdhaud:fix-#2274
Oct 20, 2025
Merged

Fix "Sum Total" not addressing unit conversion for others than length/area/volume#2875
jmcouffin merged 1 commit intopyrevitlabs:developfrom
Wurschdhaud:fix-#2274

Conversation

@Wurschdhaud
Copy link
Copy Markdown
Contributor

Description

  • adds support for project unit conversion for parameters other than hardcoded length/area/volume
  • removes hardcoded entirely by reusing format_value
  • formatted using black

Checklist

Before submitting your pull request, ensure the following requirements are met:

  • Code follows the PEP 8 style guide.
  • Code has been formatted with Black using the command:
    pipenv run black {source_file_or_directory}
  • Changes are tested and verified to work as expected.

Related Issues

If applicable, link the issues resolved by this pull request:


Additional Notes

Include any additional context, screenshots, or considerations for reviewers.


Thank you for contributing to pyRevit! 🎉

…hardcoded length/area/volume

- removes hardcoded entirely by reusing format_value
- formatted using black
@devloai
Copy link
Copy Markdown
Contributor

devloai bot commented Oct 19, 2025

Unable to perform a code review. You have run out of credits 😔
Please upgrade your plan or buy additional credits from the subscription page.

Copy link
Copy Markdown
Contributor

@jmcouffin jmcouffin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in 2024
All Good,
Good one, Thank you @Wurschdhaud

@jmcouffin jmcouffin merged commit 088ae01 into pyrevitlabs:develop Oct 20, 2025
@Wurschdhaud Wurschdhaud deleted the fix-#2274 branch October 20, 2025 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Water and Airflow parameters - Sum total tool unit fix

2 participants