Conversation
Codecov Report
@@ Coverage Diff @@
## dev #76 +/- ##
==========================================
- Coverage 82.27% 82.09% -0.18%
==========================================
Files 37 37
Lines 897 905 +8
Branches 108 110 +2
==========================================
+ Hits 738 743 +5
- Misses 159 162 +3
Continue to review full report at Codecov.
|
6cfb7b6 to
ec4fd8a
Compare
|
So, I think I have covered all the topics that we discussed regarding the docs. Just have that example with the multiple-inputs in the two-mappers pipeline that maybe can be removed from this PR. What do you think about it @thejmazz ? I made a disclaimer on the script anyway... |
|
Hmm just noticed that |
docs/Forkception.md
Outdated
| >``` | ||
| > This will end up in three different branches (or lineages) with three | ||
| >leaves: | ||
| > `task0 --> task1 --> task4`, `task0 --> task2 --> task4` and `task0 --> |
There was a problem hiding this comment.
name these forked tasks task2A, task2B, task2C
(to be consistent with idea that fork is for multiple "alternatives" to a specific task (e.g. different params)
rather than different tasks
| join(task2, task3, fork(task4, task5)) | ||
| ), | ||
| task6 | ||
| ) |
There was a problem hiding this comment.
add labeled diagram (not the d3 screenshot) for this pipeline
This PR intends to improve current bionode-watermill documentation.