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

Potential Bug in setTriggerSource #177

Closed
Alex-Beh opened this issue Aug 7, 2023 · 5 comments
Closed

Potential Bug in setTriggerSource #177

Alex-Beh opened this issue Aug 7, 2023 · 5 comments

Comments

@Alex-Beh
Copy link

Alex-Beh commented Aug 7, 2023

I am using a2A2600-64umBAS and it only provide Line1, Line2 & Line 3.

However from the source code setTriggerSource it only provide Line1, Line3 & Line4. Is it a bug or a different design for different camera?

Relevat part:

  1. CurrentParams.msg
  2. getTriggerSource
@ayguencem
Copy link

Hi Alex, what pin are you using as trigger pin?

@Alex-Beh
Copy link
Author

For my case, there are two GPIO pins available to me and I am using Line 2.

FrancoisPicardDTI added a commit that referenced this issue Oct 17, 2023
… new camera functionalities - aligning with issue #177
@FrancoisPicardDTI
Copy link
Collaborator

Hi @Alex-Beh
The current implementation is due to old camera design, and you're absolutely right, the new cameras imply a new implementation. I fixed it in the humble version:
0 -> Trigger Software / 1 -> Line 1 / 2 -> Line 2 / 3 -> Line 3 / 4 -> Line 4 / 5 -> Action 1 / 6 -> Periodic signal 1
Let me know if it solves your problem.

@Alex-Beh
Copy link
Author

Yes it did. Thanks for supporting this project.

@FrancoisPicardDTI
Copy link
Collaborator

Thank you!

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

No branches or pull requests

3 participants