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

crow as ii follower commands #354

Merged
merged 2 commits into from Jan 23, 2021
Merged

crow as ii follower commands #354

merged 2 commits into from Jan 23, 2021

Conversation

@trentgill
Copy link
Collaborator

@trentgill trentgill commented Aug 2, 2020

add crow commands for reset & some asllib actions.
add s16ms type to descriptor system for using s16 to represent milliseconds (like teletype ms).

UNTESTED

The API for using crow as a follower has been changed:

ii.self.call = function(args)
  -- 'args' is a table
  -- use `#args` if you want to switch on the count of args (like the old call1, call2)
end

ii.self.query = function(args)
  -- 'args' is a table. use `#args` if you want to switch on arg count.
  return 0 -- here is where you return the value to the leader
end

All other commands & queries are handled directly by the crowlib. They typically should not be redefined by the user, though they can be if so desired. eg. ii.crow.reset() will trigger crow.reset() on the follower, but you could redefine this behaviour with ii.self.reset = function() <do something else here> end.

Will fix #258

… using s16 to represent milliseconds
@trentgill trentgill requested review from tehn and simonvanderveldt Aug 2, 2020
@tehn
tehn approved these changes Aug 4, 2020
@trentgill trentgill merged commit e5c9366 into main Jan 23, 2021
1 check passed
1 check passed
build
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants