Update radar_check.py - Rev1#2418
Update radar_check.py - Rev1#2418jmcouffin merged 1 commit intopyrevitlabs:Preflight-Checks_Hackathon_2024from
Conversation
Made the changes below per Jean-Marc's feedback. 1- you may want to present digits results in a tabular form for readability and ease in comparison. Response: Added output.print_table 2- the project base point cannot be placed at more than 10 miles in more recent version of Revit Response: Good to know! I removed the report for the location of the project basept. 3- you may want to use the convert unit util to display the distances in the same units as in the UI Response: I used the unitutils to convert the internal distances to the model distances, however in metric I am having hard time to show the suffix (mm, M, km)....etc 4- you may want to limit the amount of objects output in the output windows using linkify as it may greatly slow the tool Response: Set the limit of linkify to 10 prints per category, if there are more violating objects, then it will show a message "Manual investigation is required" 5- I tend to prefer print_md over print_html as it is more efficient and improves readability Response: I switched most of them to use print_md. also I used pyrevit emojis instead of html colors. please take a look and let me know if this makes sense.
|
@jmcouffin code have some problems 
|
|
@dosymep I merged it too quickly. 🏃 |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24338+2325-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24340+1616-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24340+1645-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24345+0715-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24351+0714-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24354+1726-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24364+2228-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24364+2256-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.24365+2215-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25006+1041-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25009+2113-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25010+1021-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25010+1128-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25013+1201-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25013+1638-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25013+1855-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25013+1857-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25015+1341-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25015+1357-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25024+1520-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25024+1957-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25025+1310-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25030+1056-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25030+1130-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25030+1138-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25031+1700-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25032+1615-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25032+1635-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25032+1841-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25033+1337-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25033+1402-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25034+1241-wip |
|
📦 New work-in-progress (wip) builds are available for 5.0.0.25034+1511-wip |
|
📦 New public release are available for 5.0.0.24174+2300 |
Hello @dosymep
Made the changes below per @jmcouffin feedback.
1- you may want to present digits results in a tabular form for readability and ease in comparison.
Response: Added output.print_table
2- the project base point cannot be placed at more than 10 miles in more recent version of Revit
Response: Good to know! I removed the report for the location of the project basept.
3- you may want to use the convert unit util to display the distances in the same units as in the UI
Response: I used the unitutils to convert the internal distances to the model distances, however in metric I am having hard time to show the suffix (mm, M, km)....etc
4- you may want to limit the amount of objects output in the output windows using linkify as it may greatly slow the tool
Response: Set the limit of linkify to 10 prints per category, if there are more violating objects, then it will show a message "Manual investigation is required"
5- I tend to prefer print_md over print_html as it is more efficient and improves readability
Response: I switched most of them to use print_md. also, I used pyrevit emojis instead of html colors.
Please look and let me know if this makes sense.
When violations are detected:
