Skip to content

Query.LocationPoint - support DirectShape - #1688

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

Query.LocationPoint - support DirectShape#1688
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.

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