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

SketchUp to Unreal does not properly handle block definitions/instances. #96

Open
mtalbottasg opened this issue Sep 16, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@mtalbottasg
Copy link

mtalbottasg commented Sep 16, 2024

I posted this on the speckle forum but haven't heard a response so I'm going to put it here as an issue...

Unfortunately, something seems broken with the SketchUp to Unreal Workflow:

It appears to have two bugs:

  1. Block Instances are not loading the instance transform so every group or block is loaded at 0,0,0.
  2. Materials also seem to be not loaded if they are applied at the instance level.

I have tried in Sketchup 2023 and 2024 and I’ve also trying in Unreal 5.3 and 5.4.

I am using the SketchUp Connector 2.20.0 and the Unreal Connector 2.19.0

@JR-Morgan
Copy link
Member

Since you've identified the problem with the Transforms (see forum thread)

Would you feel confident enough to submit a fix on the Unreal side, It may help speed things up a little since both @oguzhankoral and I are quite busy this week.
I think a simple check for matrix OR value should be easy to add.

If not no worries, thanks for the help debugging this.

@JR-Morgan JR-Morgan added the bug Something isn't working label Sep 16, 2024
@JR-Morgan
Copy link
Member

JR-Morgan commented Sep 16, 2024

The Material issue we can treat as separate, I have a suspicion that we're not expecting this to work in Unreal. But just want to confirm with Team what support looks like in the other connectors.

@JR-Morgan
Copy link
Member

Just confirmed with the team, we do not expect materials applied on components to work for the Unreal connector.

@mtalbottasg
Copy link
Author

Would you feel confident enough to submit a fix on the Unreal side

Unfortunately, I just don't know enough about C++ syntax to confidently write the OR statement.

That said, I'm very comfortable in the SketchUp ruby connector. @oguzhankoral Here's an updated \speckle_connector\src\speckle_objects\other\transform.rb The only change is just renaming "value" to "matrix". I tested it with the Unreal connector and everything is working.

It's probably still a good idea to add the OR to the unreal connector for backwards compatibility by this fix for the sketchup connector should do the trick.

@oguzhankoral
Copy link
Member

@mtalbottasg thanks for bringing your own fix. I am happy you made it work for yourself till we fix and test it before creating another hotfix version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants