kkretzschmar
commented
8 years ago Hi Juan, looks great!! I'll test it on windows but I don't expect any problems here. However it turned out that the setup with the windi server sometimes behaves different. I'd like to know where problems can occur. Maybe the wheather allows me to test also on real stars.
I am not sure how excatly the current park option really works. As far as I remember this option is for setting a custom park position in combination with TELESCOPE_PARK_OPTION -> PARK_WRITE_DATA. For example, if your telescope needs a special park positioin so that the roof of your small backyard observator can be closed (as in my case), then you first move the telescope to your park position, then set TELESCOPE_PARK_OPTION to PARK_CURRENT and then set TELESCOPE_PARK_OPTION to PARK_WRITE_DATA to write the encoder data on the file system which is important to calculate the real telescope position in RA and DEC. I am not sure but we should test on real stars first I think, or did you already test this option?
2016-06-04 18:01 GMT+02:00 Juan Conejero notifications@github.com:
Hi Team, and especially Klaus,
I have just pushed a complete revision of Klaus' INDIClient module: 48e3476 https://github.com/PixInsight/PCL/commit/48e3476b61035b51df65c983e7e236b590b4847f, where we have a new, fully operational INDIMount process. In first place, congratulations to Klaus for his excellent work. The main changes with respect to Klaus' latest pull request #28 https://github.com/PixInsight/PCL/pull/28 are:
-
Redesigned INDINewPropertyItem class with more flexible template constructors.
New template member functions INDIClient::SendNewPropertyItem() and INDIClient::MaybeSendNewPropertyItem() for simplified sending of device properties, including bulk properties.
Extensive use of new PCL resources for sexagesimal numeric conversion and representation. See String::SexagesimalToDouble(), String::TrySexagesimalToDouble(), String::ParseSexagesimal(), String::TryParseSexagesimal() and String::ToSexagesimal() member functions (and their IsoString counterparts) in pcl/String.h and pcl/String.cpp. See also the new DecimalToSexagesimal() and SexagesimalToDecimal() global functions in pcl/Math.h.
Redesigned CoordinateSearchDialog class with more powerful online search and interactive information visualization capabilities.
Redesigned INDIMountInterface class.
Improved AbstractINDIMountExecution (and derived classes) for better GUI interaction.
Changed class member and enumeration identifiers for coherence with PCL coding style guidelines.
Changed exported parameter identifiers to be more coherent with existing JavaScript interfaces. Of course, I have updated the relevant test scripts accordingly.
New IMCCommand::ParkDefault command to park the mount at the current park coordinates (TELESCOPE_PARK_POSITION property). The IMCCommand::Park command now parks at the p_targetRA and p_targetDec parameter values of INDIMountInstance.
Bug fixes and performance improvements.
I have tested this version and it seems to work extremely well. It passes all tests without any problems. There are a couple of very minor issues I'm going to fix, but it is ready for release IMO. Let me know if you agree with the changes.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PixInsight/PCL/issues/29, or mute the thread https://github.com/notifications/unsubscribe/ACDyphM9_JlEGb0tc3_hrL0ZReOm3STJks5qIaFugaJpZM4IuKHR .
jconejero
Hi Team, and especially Klaus,
I have just pushed a complete revision of Klaus' INDIClient module: https://github.com/PixInsight/PCL/commit/48e3476b61035b51df65c983e7e236b590b4847f, where we have a new, fully operational INDIMount process. In first place, congratulations to Klaus for his excellent work. The main changes with respect to Klaus' latest pull request https://github.com/PixInsight/PCL/pull/28 are:
I have tested this version and it seems to work extremely well. It passes all tests without any problems. There are a couple of very minor issues I'm going to fix, but it is ready for release IMO. Let me know if you agree with the changes.