INDIClient version 1.0.12.183

[jconejero]

Hi Team, and especially Klaus,

I have just pushed a complete revision of Klaus' INDIClient module: 48e3476, 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 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.

[kkretzschmar]

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
48e3476,
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
#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
#29, or mute the thread
https://github.com/notifications/unsubscribe/ACDyphM9_JlEGb0tc3_hrL0ZReOm3STJks5qIaFugaJpZM4IuKHR
.

[jconejero]

Quick question: would it be possible to add more servers, besides simbad.u-strasbg.fr, to CoordinateSearchDialog?

[kkretzschmar]

I programmed against the TAP service of the Simbad server which speaks
ADQL. Any server which has such a TAP service with ADQL could be added.
However, I didn't check alternative astronomical databases so far ... so
currentl I dont know...

2016-06-06 19:28 GMT+02:00 Juan Conejero notifications@github.com:

Quick question: would it be possible to add more servers, besides
simbad.u-strasbg.fr, to CoordinateSearchDialog?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#29 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ACDyphhA2YuRjkuTgSS1BPBQv2Hbvlt8ks5qJFjIgaJpZM4IuKHR
.

[jconejero]

No problem. I'll investigate on alternative services. We can of course release it now with Simbad, so no rush here.