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

Camera switching sometimes crashes #177

Open
jspjutNV opened this issue Jun 25, 2020 · 2 comments
Open

Camera switching sometimes crashes #177

jspjutNV opened this issue Jun 25, 2020 · 2 comments
Assignees
Labels
bug Something isn't working defer Defer working on this for a future date/request

Comments

@jspjutNV
Copy link
Contributor

Since #118 added the ability to use different cameras, and particularly the debug camera to look around the scene, I noticed a bug where switching from some additional cameras back to the player camera or debug camera occasionally causes FPSci to crash.

We need to more carefully figure out how to reproduce this bug on the way to fixing it.

@jspjutNV jspjutNV added the bug Something isn't working label Jun 25, 2020
@jspjutNV jspjutNV changed the title Camera Switching sometimes crashes Camera switching sometimes crashes Jun 25, 2020
@jspjutNV jspjutNV modified the milestones: v20.07.01, v20.07.02 Jun 25, 2020
@jspjutNV jspjutNV modified the milestones: v20.07.02, v20.08.01 Jul 10, 2020
@bboudaoud-nv
Copy link
Collaborator

I suggest deferring this issue, as using non-primary cameras is not a standard feature for FPSci users or developers. Also this issue seems to occur infrequently enough that it is difficult to produce a minimum and precise reproducible example for.

@jspjutNV
Copy link
Contributor Author

Using non-primary cameras is a standard feature of G3D applications so we have a regression compared to G3D. I'm fine pushing this to later, but I think it should stay on our list of things we'll fix.

That said, this bug is also relatively minor since it only crashes sometimes, so I don't mind having us take a while to fix it.

@jspjutNV jspjutNV modified the milestones: v20.07.03, v20.08.01 Jul 15, 2020
@jspjutNV jspjutNV modified the milestones: v20.08.01, v20.08.02 Jul 27, 2020
@bboudaoud-nv bboudaoud-nv added the defer Defer working on this for a future date/request label Aug 10, 2020
@jspjutNV jspjutNV modified the milestones: v20.09.01, v20.09.02 Aug 17, 2020
@jspjutNV jspjutNV removed this from the v20.10.01 milestone Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working defer Defer working on this for a future date/request
Projects
None yet
Development

No branches or pull requests

2 participants