Skip to content

Query.LocationPoint - support DirectShape - #1688

Merged
michal-pekacki merged 1 commit into
developfrom
Revit_Toolkit-#1687-DirectShapeLocationPoint
Aug 28, 2026
Merged

michal-pekacki merged 1 commit into
developfrom
Revit_Toolkit-#1687-DirectShapeLocationPoint

Conversation

@michal-pekacki

@michal-pekacki michal-pekacki commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor

NOTE: Depends on

Issues addressed by this PR

Closes #1687

Fixed location point of DirectShape - use bbox instead of physical bounds.

Test files

installer

Changelog

Additional comments

@michal-pekacki michal-pekacki self-assigned this Aug 19, 2026
@michal-pekacki michal-pekacki added the type:bug Error or unexpected behaviour label Aug 19, 2026
@michal-pekacki

Copy link
Copy Markdown
Contributor Author

@BHoMBot check installer

1 similar comment
@sakanni

sakanni commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@BHoMBot check installer

@bhombot-ci

bhombot-ci Bot commented Aug 19, 2026

Copy link
Copy Markdown

@sakanni to confirm, the following actions are now queued:

  • check installer

@michal-pekacki

Copy link
Copy Markdown
Contributor Author

@BHoMBot check versioning

@bhombot-ci

bhombot-ci Bot commented Aug 19, 2026

Copy link
Copy Markdown

@michal-pekacki to confirm, the following actions are now queued:

  • check versioning

@adam-sobieski adam-sobieski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested on model, works fine

@michal-pekacki

Copy link
Copy Markdown
Contributor Author

@BHoMBot check ready-to-merge

@bhombot-ci

bhombot-ci Bot commented Aug 27, 2026

Copy link
Copy Markdown

@michal-pekacki to confirm, the following actions are now queued:

  • check ready-to-merge

There are 307 requests in the queue ahead of you.

@michal-pekacki

Copy link
Copy Markdown
Contributor Author

@BHoMBot check ready-to-merge

@bhombot-ci

bhombot-ci Bot commented Aug 28, 2026

Copy link
Copy Markdown

@michal-pekacki to confirm, the following actions are now queued:

  • check ready-to-merge

There are 577 requests in the queue ahead of you.

@michal-pekacki
michal-pekacki merged commit 459560d into develop Aug 28, 2026
36 of 38 checks passed
@michal-pekacki
michal-pekacki deleted the Revit_Toolkit-#1687-DirectShapeLocationPoint branch August 28, 2026 12:11
@bhombot-ci

bhombot-ci Bot commented Aug 28, 2026

Copy link
Copy Markdown

The check ready-to-merge has already been run previously and recorded as a successful check. This check has not been run again at this time.

@BHoMBot BHoMBot mentioned this pull request Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Error or unexpected behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LocationPoint for DirectShape doesn't work well

3 participants