Closed MayoG closed 4 years ago
Merging #56 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #56 +/- ##
====================================
Coverage 67% 67%
====================================
Files 16 16
Lines 938 938
====================================
Hits 629 629
Misses 309 309
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update b7e8d16...b7e8d16. Read the comment docs.
To make it easier to review this I will divide the code to seperate parts:
pipeline_manager.py = Created the new manager and methods to control the entire pipe
component.py = Deleted the zerorpc server and added new methods to control the queues and routines
routine.py = Added routine type data member, method to get the parameters of each routine and changed the way the runner creating the threads for the routine.
main.py = The file we will execute that will choose whether we want an API or CLI and create the servers that recieve requests.
pipert/contrib/(components|routines).py = Moved the routines from the previous components and added them new methods that routine must implement such as get_constructor_parameters.
cli.py = Create new script file to run to connect to the pipeline and communicate with him
Sorry for this long pull request, didn't expect it to be that big when i started making it.