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

teletype support for crow #55

Open
trentgill opened this issue Feb 14, 2019 · 2 comments
Open

teletype support for crow #55

trentgill opened this issue Feb 14, 2019 · 2 comments

Comments

@trentgill
Copy link
Collaborator

@trentgill trentgill commented Feb 14, 2019

crow is able to be queried by another i2c leader device. teletype is the clear companion device who would lead crow, so the TT project must be updated with hooks for the actions.

This should be implemented for a single crow device first.

Afterwards we can consider how to handle multiple crow devices. This could be sequential input/output numbers (ie CROW.OUT 5 would refer to the first output of the second device, vs CROW.2.OUT 1 or CROW.OUT 2 1 etc).

Proposed initial functions are listed in lua form: https://github.com/trentgill/crow/blob/aa45d96a322f6c818e84a98bc2d69b58a189ed45/lua/ii.lua#L38
and TT form: https://github.com/trentgill/crow/blob/aa45d96a322f6c818e84a98bc2d69b58a189ed45/lua/ii.lua#L67

All the generic functions should send /receive int16 values.

@trentgill trentgill added this to the 1 milestone Feb 14, 2019
@trentgill trentgill changed the title Add I2C support for crow to teletype teletype support for crow Feb 14, 2019
@trentgill trentgill modified the milestones: 1.0, 1.1 Mar 1, 2019
@trentgill
Copy link
Collaborator Author

@trentgill trentgill commented Mar 1, 2019

edit: none of this is required now that crow has native support for 'teletype volts' in the descriptor language.

/////////////////////////////
to aid integration between crow & TT, a small library of helper functions should be added as a lua file.

purpose is to allow TT familiar users to have a way to treat values in crow the same as they do in TT without worrying about the transformations they hide:
TT.V(x) converts a lua volt (ie float where 1.0 is 1V) to a TT volt (ie 1638.4)
TT.VV(x) as above but for 163.84
TT.N(x) as above but 136.53

note the i2c driver will truncate floating point values, so this lib might need to add 0.5 after confirming the rounding behaviour or TT.

this will be deprecated after #49 is complete.

@scanner-darkly
Copy link
Member

@scanner-darkly scanner-darkly commented May 11, 2020

i can help with the teletype side of things once the supported messages are finalized.

@trentgill trentgill removed this from the 1.0.3 milestone Jun 10, 2020
@trentgill trentgill added this to the Minor milestone Jun 22, 2020
@trentgill trentgill modified the milestones: Minor, 3.0 Jul 15, 2020
@trentgill trentgill added this to Minor in 3.0 Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3.0
Minor
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants