AlignmentSubsystemForDrivers

  • Posts: 216
  • Thank you received: 120
Jasem, I did not figure out what was wrong with them. I have to say that I have forgotten almost all the math I learned in college, and I never was any good at linear algebra.

I do want to add the tests to libindi if possible (to serve as documentation as well as test the code), but I'm getting sporadic SEGFAULTs when running the tests. It seems like ctest is sending data to stdin, which the driver is trying to parse as data from indiserver, causing issues. I've tried closing stdin in the test main function, but it doesn't seem to do the trick. Any ideas?
3 years 4 months ago #65138

Please Log in or Create an account to join the conversation.


×

INDI Library v2.0.7 is Released (01 Apr 2024)

Bi-monthly release with minor bug fixes and improvements

  • Posts: 216
  • Thank you received: 120

Rendering Error in layout Message/Item: array_keys(): Argument #1 ($array) must be of type array, null given. Please enable debug mode for more information.

Please Log in or Create an account to join the conversation.

The driver's stdin/stdout are used to exchange INDI XML messages. It seems the other tests we have in place now run fine as is with gmock/gtest?
3 years 4 months ago #65144

Please Log in or Create an account to join the conversation.

  • Posts: 216
  • Thank you received: 120

Rendering Error in layout Message/Item: array_keys(): Argument #1 ($array) must be of type array, null given. Please enable debug mode for more information.

Please Log in or Create an account to join the conversation.

  • Posts: 216
  • Thank you received: 120
I figured out the issue with the test. I wasn't calling ISGetProperties in the test scope's constructor. Once that was added, the crazy xml messages went away, and I no longer get the segfault.
3 years 4 months ago #65150

Please Log in or Create an account to join the conversation.

  • Posts: 216
  • Thank you received: 120
After some cleanup, I've put in a PR that adds the tests and the new methods. I figure we can continue the discussion there as to whether we want to include them.

github.com/indilib/indi/pull/1303
3 years 4 months ago #65151

Please Log in or Create an account to join the conversation.

Time to create page: 0.788 seconds