-
Notifications
You must be signed in to change notification settings - Fork 213
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
indi-ipx800v0.5 #999
indi-ipx800v0.5 #999
Conversation
IPX800_v0.5 release
ISwitch RelaisInfoS[11] {}; | ||
ISwitch Relais1InfoS[11], Relais2InfoS[11],Relais3InfoS[11],Relais4InfoS[11],Relais5InfoS[11],Relais6InfoS[11],Relais7InfoS[11],Relais8InfoS[11] {}; | ||
ISwitchVectorProperty RelaisInfoSP[8] {}; | ||
|
||
ISwitch DigitalInputS[10] {}; | ||
ISwitch Digital1InputS[10], Digital2InputS[10], Digital3InputS[10], Digital4InputS[10], Digital5InputS[10], Digital6InputS[10], Digital7InputS[10], Digital8InputS[10]; | ||
ISwitchVectorProperty DigitalInputSP[8] {}; | ||
|
||
ISwitch Relay1StateS[2],Relay2StateS[2],Relay3StateS[2],Relay4StateS[2],Relay5StateS[2],Relay6StateS[2],Relay7StateS[2],Relay8StateS[2] ; | ||
ISwitchVectorProperty RelaysStatesSP[8] ; | ||
ISwitch Digit1StateS[2],Digit2StateS[2],Digit3StateS[2],Digit4StateS[2],Digit5StateS[2],Digit6StateS[2],Digit7StateS[2],Digit8StateS[2] ; | ||
ISwitchVectorProperty DigitsStatesSP[8] ; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are these for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the driver does not have any hard dependencies on third party libraries, it is candidate for inclusion in indi-core instead of indi-3rdparty. Therefore please submit it to to INDI core instead.
Hi,
those switches are used
- RelaisInfosS, and DigitalInputs : to configure the ipx (and so to let driver manage Engine Power Management, ...). The plan was to update the label of RelaysStates and DigitsStates in "status" tab, depending of the function selected...
- RelaysStates and DigitsStates just displayed the states of Relays and Digits, now it's quite redundant with Inputs and Outputs Interface, will thing what to do with this.
Arnaud.
Envoyé avec la messagerie sécurisée [Proton Mail.](https://proton.me/mail/home)
Le dimanche 3 novembre 2024 à 05:18, Jasem Mutlaq ***@***.***> a écrit :
… @knro commented on this pull request.
---------------------------------------------------------------
In [indi-ipx800/indi_ipx800.h](#999 (comment)):
> + ISwitch RelaisInfoS[11] {};
+ ISwitch Relais1InfoS[11], Relais2InfoS[11],Relais3InfoS[11],Relais4InfoS[11],Relais5InfoS[11],Relais6InfoS[11],Relais7InfoS[11],Relais8InfoS[11] {};
+ ISwitchVectorProperty RelaisInfoSP[8] {};
+
+ ISwitch DigitalInputS[10] {};
+ ISwitch Digital1InputS[10], Digital2InputS[10], Digital3InputS[10], Digital4InputS[10], Digital5InputS[10], Digital6InputS[10], Digital7InputS[10], Digital8InputS[10];
+ ISwitchVectorProperty DigitalInputSP[8] {};
+
+ ISwitch Relay1StateS[2],Relay2StateS[2],Relay3StateS[2],Relay4StateS[2],Relay5StateS[2],Relay6StateS[2],Relay7StateS[2],Relay8StateS[2] ;
+ ISwitchVectorProperty RelaysStatesSP[8] ;
+ ISwitch Digit1StateS[2],Digit2StateS[2],Digit3StateS[2],Digit4StateS[2],Digit5StateS[2],Digit6StateS[2],Digit7StateS[2],Digit8StateS[2] ;
+ ISwitchVectorProperty DigitsStatesSP[8] ;
What are these for?
—
Reply to this email directly, [view it on GitHub](#999 (review)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AXW4X7HPRHEB2LL4NLCME63Z6WP2JAVCNFSM6AAAAABRB47JF2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIMJRG43TONBZGY).
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
No description provided.