Commit Graph

20 Commits (9ba060c0f1c0b7bc085d887ef64b685b5c54faff)

Author SHA1 Message Date
Rick Carlino 9ba060c0f1 Fix set_servo_angle, get_sequence 2020-01-23 15:06:04 -06:00
Rick Carlino e1e8adfa00 Fix set_servo_angle, get_sequence 2020-01-23 15:02:15 -06:00
Rick Carlino f2790ff886 Fix read/write pin sys call tests 2020-01-23 14:46:23 -06:00
Rick Carlino e4af444ece TODO: Fix FarmbotCeleryScript.SysCallsTest. 50 tests, 11 failures 2020-01-22 21:03:46 -06:00
Rick Carlino 0c22021d66 TODO: Fix FarmbotCeleryScript.SysCallsTest. 49 tests, 13 failures 2020-01-22 20:51:48 -06:00
Rick Carlino ac7bf38be9 Fix SysCalls.point tests 2020-01-22 20:27:55 -06:00
Rick Carlino c9206ab484 TODO: Fix FarmbotCeleryScript.SysCallsTest. 47 tests, 16 failures 2020-01-22 19:56:29 -06:00
Rick Carlino a8c4887841 Run mix format 2020-01-17 16:02:00 -06:00
Rick Carlino 26d31012fb Test: get_point_group test success case 2020-01-17 15:40:25 -06:00
Rick Carlino 12c55fb60f Test: PointGroup test failure case 2020-01-17 15:28:13 -06:00
Rick Carlino 1c00c543e3 Rename some test helpers 2020-01-17 13:42:58 -06:00
Rick Carlino 5722bea421 Huge formatting update (sorry) 2020-01-17 09:58:53 -06:00
Rick Carlino 845e69ce7e
Re-enable set_servo_angle
* set_servo_angle works.
* Tests for set_servo_angle in  FarmbotCeleryScript.Syscalls
2019-12-27 11:45:26 -06:00
connor rigby 0cbb4ba784
Fix named pins causing celeryscript errors
NamedPin values were encoded as tuples. This changes it to
pattern matchable structs, and implements a `to_string` behaviour
for those structs
2019-09-13 16:03:24 -07:00
Connor Rigby 00d0897672
Dry up celery_script compilation
This adds much more type checking and error
handling in the compiled celery_script.
Also removes the hack/shortcut in the StubSysCalls
module.
2019-09-13 16:03:23 -07:00
Connor Rigby 14ff862176
Allow named_pin to return metadata
This will allow doing special thing contextually depending
on the type of pin being read.
2019-09-13 16:03:22 -07:00
Connor Rigby db54e2117d
Update find_home to not take a speed arg 2019-09-13 16:03:18 -07:00
connor rigby d842c3a123
🎉 Implement Variables in Regimen 2019-09-13 16:03:17 -07:00
Connor Rigby ef3225878d
Change FarmbotCeleryScript Namespace 2019-09-13 16:03:16 -07:00
Connor Rigby 4114e26804
Implement new CeleryScript Runtime environment.
This is obviously a rather large change warranting an essay describing
it.

A Brief overview
Basically the old implementation had quite a few down sides preventing
it from really working as intended, especially with the addition of the
variables feature. Here is the shortlist of things that needed
addressing:

* No scoping between sequences.
What this essentially means is that a sequence that executes another
sequence is unable to add data to the calle. This is important for using
Variables.

* Error recovery
certain nodes have a high likelyhood of failing such as anything that
interfaces the firmware. Much focus was spent ensuring that errors would
be recoverable when desired.

* Complexity of control flow asts versus action asts.
Nodes such as `if` will always work in the same way regardless of the
state of the rest of the system meaning there is no reason for it to
have a special implementation per environment. on the other hand
`move_absolute` is bound to a specific part of the system. Seperating
these concerns allows for better testing of each piece independently.

A More In Depth overview
The core of this change resolves around 1 really big change resulting in
many more small changes. This change is the CeleryScript `compiler`. The
TLDR of this system is that now CeleryScript ASTs are deterministicly
compiled to Elixir's AST and executed. Doing this has some big benifits
as described below.

1) CeleryScript "runtime" environment is now much simpiler in favor of a somewhat
complex "compile time" environment. Basically instead of EVERY single
CeleryScript AST having a custom runtime implementation, only a subset
of ASTs that require external services such as the Firmware, Database,
HTTP, etc require having a runtime implementation. This subset of ASTs
are called `SysCalls`. Also the runtime implementations are compiled to
a single function call that can be implemented instead of needing to
have a contextual environment and making decisions at runtime to
evaluate variables and the like.

2) Static analysis is now possible. This means an incorrectly crafted
sequence can be validated at compile time rather than getting half way
through a sequence before finding the error.

3) Having the "external services" separated leads to better plugability.
There is now a behaviour to be implemented for the subset of syscalls
that are system specific.
2019-09-13 16:03:15 -07:00