Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create radar_check.py #2414

Merged

Conversation

tay0thman
Copy link

This Model Checker swiftly verifies the extents of the Revit model. Placing model extents more than 10 miles from the project's internal origin can lead to issues with accuracy, tolerance, performance, and viewport display. This check ensures that the model remains within a 10-mile radius of the internal origin.

This Model Checker swiftly verifies the extents of the Revit model. 
Placing model extents more than 10 miles from the project's 
internal origin can lead to issues with accuracy, tolerance, 
performance, and viewport display. This check ensures that the 
model remains within a 10-mile radius of the internal origin.
@jmcouffin
Copy link
Contributor

Nice, I will have a look tomorrow

@jmcouffin
Copy link
Contributor

Nice entry @tay0thman ! 🍾 You are the first!

Comments:

  • you may want to present digits results in a tabular form for readability and ease in comparison
  • the project base point cannot be placed at more than 10 miles in more recent version of Revit
  • you may want to use the convert unit util to display the distances in the same units as in the UI
  • you may want to limit the amount of objects output in the output windows using linkify as it may greatly slow the tool
  • I tend to prefer print_md over print_html as it is more efficient and improves readability

@jmcouffin jmcouffin merged commit 466aae9 into pyrevitlabs:Preflight-Checks_Hackathon_2024 Oct 16, 2024
@dosymep
Copy link
Member

dosymep commented Oct 16, 2024

@jmcouffin
I just scheduled a review to check tomorrow :D

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.

3 participants