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

Feature/t3d pipeline package #65

Merged
merged 33 commits into from
Nov 2, 2022
Merged

Feature/t3d pipeline package #65

merged 33 commits into from
Nov 2, 2022

Conversation

TomSimons
Copy link
Collaborator

No description provided.

…n for direct interaction. Added sample scene
Copy link
Collaborator

@sambaas sambaas left a comment

Choose a reason for hiding this comment

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

  • De samples map is nog niet importeerbaar in de Package Manager, en is zichtbaar nog zichtbaar in Unity in de package structuur (geen ~ als suffix) dus heb de samples scene nog niet kunnen testen.

  • Er bestaat al een Poly2Mesh in het project, die valt nu onder de selectiontools.
    Is het een idee om dit pakket en de selectiontools een losse dependecy te geven op een 'MeshManipulation' package, of iets in die trend? De Poly2Mesh library is nagenoeg hetzelfde, alleen hebben we nog een toevoeging gedaan zodat je ook polygonen een dikte / extrude kan geven

…DPipeline and SelectionTools now depend on this. Used version from SelectionTools because this version had some additions.
…ror. it will give a warning that any visualisation will create empty meshes.

- Removed deleted script from Scene
- added todo for MultiPoint and MultiLineString mesh creation
@TomSimons
Copy link
Collaborator Author

Punt 2 is aangepast, punt 1 lukt niet bij mij, hij verwijdert de hele samples map dan in git.

@sambaas sambaas merged commit 37ad87a into main Nov 2, 2022
@sambaas sambaas deleted the feature/T3DPipeline-package branch November 2, 2022 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants