Skip to content

Add option to specify camera axis direction #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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marius-ne
Copy link

Hello!

I have added a feature that allows the user to specify the direction that the camera (boresight) axis is pointing to, either in the positive or negative z directions.

The original code had -z by default which I found surprising because in my line of work, +z is the norm. So I'm adding this for other people who might have the same experience.

I am open for suggestions and if you agree @xt4d I will add some docstrings to the methods to explain the new option along with the other, pre-existing arguments. I would also like to add being able to specify -/+ x/y axes for the camera axis in the future.

Note: One has to be careful when specifying -z as an option on the command line, the string needs to be properly escaped and passed to the option like this: --camera_axis="-z"

Thanks!

PS: Here an illustration that shows the new feature:
demonstration

@marius-ne
Copy link
Author

marius-ne commented May 8, 2025

Resolves #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants