-
Notifications
You must be signed in to change notification settings - Fork 3
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: 3D Renderer and Preview Export of Thumbnails #87
Merged
Merged
Changes from 10 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
9cc66fe
init rendering work from @net-cscience-raphael (see Cineast)
rahelarnold98 9fa71ca
rendering of 3d models with embedded textures completed
rahelarnold98 25a279c
3D rendering and extracting of a preview finally works🥳, clean up fol…
rahelarnold98 f362c99
deleted duplicated and no longer used files
rahelarnold98 9dfa0a3
more cleanup
rahelarnold98 8a21952
deleted unused files
rahelarnold98 09a75c6
minor cleanup changes
rahelarnold98 ccac2de
3d model preview can now be a gif of different views
rahelarnold98 8678d62
removed print statements used while debuggung
rahelarnold98 b2d3e55
rendering window is hidden
rahelarnold98 2b15d7d
renamed model to model3d to be more specific
rahelarnold98 237c538
renamed ModelContent to Model3DContent to be more specific
rahelarnold98 f50b53d
removed deprecated and unused functions from core.model.mesh
rahelarnold98 a1ee346
changed OptimizerOptions to class
rahelarnold98 8714eca
removed empty class
rahelarnold98 335b167
Adds facility for external rendering.
ppanopticon 170f676
Adds facility for external rendering.
ppanopticon 641453f
Fixes combination of four images in ModelPreviewExporter.
ppanopticon 9427706
Removes superfluous Renderer interface.
ppanopticon 1fecf4f
ExternalRenderer now re-creates the external process on demand.
d9e67f8
rewrite of Java code to Kotlin
rahelarnold98 39f4ac7
changed GLMaterial to data class
rahelarnold98 fad75ca
changed Texture to data class
rahelarnold98 cc627ca
changed RenderOptions to data class
rahelarnold98 f6fd74c
changed RenderActions to enum
rahelarnold98 7e8d974
Camera is now a data class
rahelarnold98 70f3a36
removed joml dependency in core module
rahelarnold98 2a28b39
merged dev in feature branch
rahelarnold98 7f276a2
Minor optimisation as proposed by @lucaro.
ppanopticon 8aecbef
Attempt at better error reporting.
ppanopticon 16affa6
The -XstartOnFirstThread is only used on MacOS.
ppanopticon ad07a8f
Improves error reporting.
ppanopticon 10042ba
GitHub action script now creates a screen (in an attempt to get GLFW …
ppanopticon b02197e
Removes logging.
ppanopticon 6c47ac2
Removes change, since it doesn't seem to work.
ppanopticon 50e609d
Dependency clean-up.
ppanopticon cba45f5
Adds display to ubuntu container and runs tests with display.
ppanopticon 6961a83
Adds test reporting to CI/CD.
ppanopticon 7a14cee
Merge branch 'dev' into feature/3d-preview
ppanopticon 1d719a6
DISPLAY=:1 is now integrated into ExternalRenderer invocation of proc…
ppanopticon eaa3821
Fix.
ppanopticon 02dfef1
Merged
1247b9c
Downgrades Protobuf dependency, since it seems to be incompatible wit…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
9 changes: 4 additions & 5 deletions
9
...e/model/content/element/Model3DContent.kt → ...ore/model/content/element/ModelContent.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,7 +8,7 @@ import org.vitrivr.engine.core.model.content.impl.cache.CachedAudioContent | |
import org.vitrivr.engine.core.model.content.impl.cache.CachedContent | ||
import org.vitrivr.engine.core.model.content.impl.cache.CachedImageContent | ||
import org.vitrivr.engine.core.model.content.impl.cache.CachedTextContent | ||
import org.vitrivr.engine.core.model.mesh.Model3D | ||
import org.vitrivr.engine.core.model.mesh.texturemodel.Model | ||
import org.vitrivr.engine.core.model.metamodel.Schema | ||
import java.awt.image.BufferedImage | ||
import java.io.IOException | ||
|
@@ -117,7 +117,7 @@ class CachedContentFactory : ContentFactoriesFactory { | |
return content | ||
} | ||
|
||
override fun newMeshContent(model3D: Model3D): Model3DContent { | ||
override fun newMeshContent(model: Model): ModelContent { | ||
check(!this.closed) { "CachedContentFactory has been closed." } | ||
TODO() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should be properly implemented before merging this PR. |
||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 changes: 0 additions & 28 deletions
28
vitrivr-engine-core/src/main/kotlin/org/vitrivr/engine/core/model/mesh/Material.kt
This file was deleted.
Oops, something went wrong.
199 changes: 0 additions & 199 deletions
199
vitrivr-engine-core/src/main/kotlin/org/vitrivr/engine/core/model/mesh/Mesh.kt
This file was deleted.
Oops, something went wrong.
68 changes: 0 additions & 68 deletions
68
vitrivr-engine-core/src/main/kotlin/org/vitrivr/engine/core/model/mesh/Model3D.kt
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ModelContent
is very generic; the previous name was more descriptive. I'd change it back.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done👍