-
Notifications
You must be signed in to change notification settings - Fork 8
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
Camera is hard to control #2
Comments
Hi Jamie, |
Actually, if you have a pointer to some existing code that implements a better camera controller then that would already be very useful. :) |
Hi Wenzel, best |
Hi Jamie -- this looks promising. The only thing that appears to be missing from that example are translations. Is there anything special you would do to support them in an intuitive manner? |
Also, would I have your permission to port this code into NanoGUI (foundation of Tekari), which is BSD-licensed and not MIT-licensed? (It's actually not 100% clear to me what the difference is, they appear to be very similar. But I would not want to change the license of NanoGUI or add source code with a different license.) |
Ah, actually that code (https://github.com/Jam3/orbit-controls) is not by me.. But MIT license seems very permissive ("only requiring preservation of copyright and license notices") so can you not just copy it in? For translations, my first thought is that since Tekari is all about exploring the BSDF "disk", maybe it doesn't need the camera to be "free-roaming", i.e. not allow translation? It might be more intuitive to only allow orbiting and zoom (so the user must resize the window, and/or rotate/zoom to see more detail in some area)? Otherwise it's a bit too easy for the cam to end up in some odd configuration which can be confusing. Though I'm not that familiar with the app so maybe there could be some need for it. (In which case, i'd suggest to add some really obvious way to reset the camera to the default view). best |
The app is great, but the camera could be better. In particular, the mouse wheel (at least on Linux) is too sensitive so you can't properly zoom. It's easy to pan/zoom and completely lose the display, and there doesn't seem to be a "reset". Maybe it would be better to have the camera be a standard "polar" camera which is locked to rotate around the axis, and zooms in a smoother way?
The text was updated successfully, but these errors were encountered: