[ENH]: Add support for multiple preprocessors in the pipeline #263

Merged

synchon merged 6 commits from update/multi-preprocessor into main

2023-10-18 13:13:39 +00:00

synchon commented

2023-10-17 13:09:37 +00:00

(Migrated from github.com)

description of feature/fix
tests added/passed
add an entry for the latest changes

This PR refactors the pipeline internals to add support for multiple preprocessors.

* [x] description of feature/fix * [x] tests added/passed * [x] add an entry for the latest changes This PR refactors the pipeline internals to add support for multiple preprocessors.

github-actions[bot] commented

2023-10-17 13:16:40 +00:00

(Migrated from github.com)

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-10-18 13:18 UTC

[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.4 :---: Preview removed because the pull request was closed. 2023-10-18 13:18 UTC

codecov[bot] commented

2023-10-17 13:16:51 +00:00

(Migrated from github.com)

Codecov Report

Merging #263 (f80d532) into main (9689b02) will decrease coverage by 0.04%.
The diff coverage is 90.47%.

@@            Coverage Diff             @@
##             main     #263      +/-   ##
==========================================
- Coverage   92.38%   92.35%   -0.04%     
==========================================
  Files          86       86              
  Lines        3837     3846       +9     
  Branches      748      753       +5     
==========================================
+ Hits         3545     3552       +7     
- Misses        187      188       +1     
- Partials      105      106       +1

Flag	Coverage Δ
docs	`100.00% <ø> (ø)`
junifer	`92.35% <90.47%> (-0.04%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
junifer/api/functions.py	`96.35% <100.00%> (+0.07%)`	⬆️
junifer/markers/collection.py	`100.00% <100.00%> (ø)`
junifer/api/cli.py	`70.19% <33.33%> (-1.38%)`	⬇️

## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report > Merging [#263](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (f80d532) into [main](https://app.codecov.io/gh/juaml/junifer/commit/9689b026eb7b326e73729bdd6914613dca7da05b?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (9689b02) will **decrease** coverage by `0.04%`. > The diff coverage is `90.47%`. [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/263/graphs/tree.svg?width=650&height=150&src=pr&token=5H21JuZXMw&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #263 +/- ## ========================================== - Coverage 92.38% 92.35% -0.04% ========================================== Files 86 86 Lines 3837 3846 +9 Branches 748 753 +5 ========================================== + Hits 3545 3552 +7 - Misses 187 188 +1 - Partials 105 106 +1 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/263/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [docs](https://app.codecov.io/gh/juaml/junifer/pull/263/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `100.00% <ø> (ø)` | | | [junifer](https://app.codecov.io/gh/juaml/junifer/pull/263/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `92.35% <90.47%> (-0.04%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/api/functions.py](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9hcGkvZnVuY3Rpb25zLnB5) | `96.35% <100.00%> (+0.07%)` | :arrow_up: | | [junifer/markers/collection.py](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9tYXJrZXJzL2NvbGxlY3Rpb24ucHk=) | `100.00% <100.00%> (ø)` | | | [junifer/api/cli.py](https://app.codecov.io/gh/juaml/junifer/pull/263?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9hcGkvY2xpLnB5) | `70.19% <33.33%> (-1.38%)` | :arrow_down: |

fraimondo (Migrated from github.com) approved these changes

2023-10-18 09:14:04 +00:00