[ENH]: Add feature_md5 argument to read_transform() #248

Merged
synchon merged 3 commits from update/read_transform_feature_md5 into main 2023-08-03 10:21:37 +00:00
synchon commented 2023-08-03 09:04:33 +00:00 (Migrated from github.com)
  • description of feature/fix
  • tests added/passed
  • add an entry for the latest changes

This PR adds feature_md5 argument to junifer.onthefly.read_transform() to allow accessing storage features by MD5 hashes.

* [x] description of feature/fix * [x] tests added/passed * [x] add an entry for the latest changes This PR adds `feature_md5` argument to `junifer.onthefly.read_transform()` to allow accessing storage features by MD5 hashes.
github-actions[bot] commented 2023-08-03 09:11:45 +00:00 (Migrated from github.com)
PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-08-03 10:27 UTC
[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.4 :---: Preview removed because the pull request was closed. 2023-08-03 10:27 UTC <!-- Sticky Pull Request Commentpr-preview -->
codecov[bot] commented 2023-08-03 09:31:46 +00:00 (Migrated from github.com)

Codecov Report

Merging #248 (3f8febc) into main (dc23879) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #248   +/-   ##
=======================================
  Coverage   93.10%   93.10%           
=======================================
  Files          84       84           
  Lines        3714     3714           
  Branches      722      722           
=======================================
  Hits         3458     3458           
  Misses        160      160           
  Partials       96       96           
Flag Coverage Δ
docs 100.00% <ø> (ø)
junifer 93.10% <100.00%> (ø)

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

Files Changed Coverage Δ
junifer/onthefly/read_transform.py 93.75% <100.00%> (ø)
## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/248?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report > Merging [#248](https://app.codecov.io/gh/juaml/junifer/pull/248?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (3f8febc) into [main](https://app.codecov.io/gh/juaml/junifer/commit/dc238793fbb36d1abea5ef3f2d3d054c7f201279?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) (dc23879) will **not change** coverage. > The diff coverage is `100.00%`. [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/248/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/248?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #248 +/- ## ======================================= Coverage 93.10% 93.10% ======================================= Files 84 84 Lines 3714 3714 Branches 722 722 ======================================= Hits 3458 3458 Misses 160 160 Partials 96 96 ``` | Flag | Coverage Δ | | |---|---|---| | docs | `100.00% <ø> (ø)` | | | junifer | `93.10% <100.00%> (ø)` | | 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 Changed](https://app.codecov.io/gh/juaml/junifer/pull/248?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/onthefly/read\_transform.py](https://app.codecov.io/gh/juaml/junifer/pull/248?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9vbnRoZWZseS9yZWFkX3RyYW5zZm9ybS5weQ==) | `93.75% <100.00%> (ø)` | |
Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
juaml/junifer!248
No description provided.