Update pick_point() in pyrevitlib\pyrevit\revit\selection.py#3018
Update pick_point() in pyrevitlib\pyrevit\revit\selection.py#3018jmcouffin merged 7 commits intopyrevitlabs:developfrom
Conversation
Implemented the creation of a work plane for the current view. Without this, the PickPoint() function does not work. This is relevant for ceiling plans, where a work plane is sometimes not assigned during creation.
|
Unable to trigger custom agent "Code Reviewer"You have run out of credits 😔 |
There was a problem hiding this comment.
Pull request overview
This PR adds automatic work plane (SketchPlane) creation to the pick_point() function in pyrevitlib's selection utilities. The change addresses an issue where PickPoint() fails in views without an assigned work plane, particularly ceiling plans.
Changes:
- Adds logic to detect when a view lacks a SketchPlane and automatically creates one based on the view's direction and origin
- Implements multi-language support for the transaction name using translations for English, French, German, Spanish, and Russian
- Wraps the SketchPlane creation in a transaction to ensure proper database modification
Some corrections based on comments
Fix 'English_USA' language
To avoid potential future naming conflicts between DB and pyrevit.revit.
|
Please let me know when it is ready for review @Denver-22 |
|
@jmcouffin, it is ready |
… creation and cleanup - Enhanced error handling during SketchPlane creation and deletion. - Added support for Chinese_Simplified translation. - Ensured temporary work planes are removed after use to maintain document integrity.
|
Not that simple in the end.. thks @Denver-22 |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+1003-wip |
|
@jmcouffin |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+2108-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26017+2119-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2117-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2124-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26022+2138-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26023+2136-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26023+2141-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26025+1329-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26025+1418-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2037-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2039-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2101-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2136-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2147-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26030+2212-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1043-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1111-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1304-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1323-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1433-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1538-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1543-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1553-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1612-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1624-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1738-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1743-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1829-wip |
|
📦 New work-in-progress (wip) builds are available for 5.3.1.26032+1937-wip |
|
📦 New work-in-progress (wip) builds are available for 6.0.0.26032+1956-wip |
|
📦 New work-in-progress (wip) builds are available for 6.0.0.26032+2005-wip |
|
📦 New work-in-progress (wip) builds are available for 6.0.0.26032+2008-wip |
|
📦 New public release are available for 6.0.0.26032+2040 |
|
📦 New public release are available for 6.0.0.26032+2040 |
Implemented the creation of a work plane for the current view. Without this, the PickPoint() function does not work.
This is relevant for ceiling plans, where a work plane is sometimes not assigned during creation.
P.S. The translation for French, German, and Spanish was done using a translator. I apologize for any errors.