-
Notifications
You must be signed in to change notification settings - Fork 4
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
Optimization: Better typing across the package #23
Comments
We already have a project tab here https://github.com/orgs/JuliaWGPU/projects/3 We can file an issue where type system needs work and complete it. Keeping this issue active anyways. |
We can pick each example and fix typesystem for each example. This will help us understand the benefits of type stability. I have Tracy integration example which might help us document performance improvements or julia use new chainmarks.jl package. For me Typesystem is a way towards static compiling purpose. |
@arhik Whats the purpose of this variable defined here? I also can't find any instances of |
This is used to convert between conventions used in different frameworks like opengl, webgl, blender, etc ... We cannot use For example: if some framework uses (x, z, -y) as coordinate conventions instead of (x, y, z) we can simply exchange 2 and row of this matrix and also flip sign of y. Then when we multiply [a, b, c] data with this matrix it becomes [a, c, -b]. |
I think we can simply use |
Sorry I misread your comment. I didn't need inverse transform yet. But we will most likely. Its rather common in computer vision and graphics. |
They are both like placeholders for now. I used it some of my own experiments like robotics simulator to load blender data. |
Properly generalizing it will be good feature to add. |
Taking a look at the package code, it'd be nice if we can:
scene.objects
).The text was updated successfully, but these errors were encountered: