[ENH]: Add mode as an aggregation function for get_aggfunc_by_name #287

Merged
synchon merged 8 commits from update/mode-for-aggfunc into main 2024-01-12 10:54:59 +00:00
synchon commented 2024-01-10 12:24:22 +00:00 (Migrated from github.com)

Are you requiring a new dataset or marker?

  • I understand this is not a marker or dataset request

Which feature do you want to include?

Add mode as an aggregation function for junifer.stats.get_aggfunc_by_name.

How do you imagine this integrated in junifer?

Like any other aggregation function.

Do you have a sample code that implements this outside of junifer?

No response

Anything else to say?

No response

### Are you requiring a new dataset or marker? - [X] I understand this is not a marker or dataset request ### Which feature do you want to include? Add [mode](https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.mode.html) as an aggregation function for `junifer.stats.get_aggfunc_by_name`. ### How do you imagine this integrated in junifer? Like any other aggregation function. ### Do you have a sample code that implements this outside of junifer? _No response_ ### Anything else to say? _No response_
codecov[bot] commented 2024-01-10 12:25:06 +00:00 (Migrated from github.com)

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1382fd2) 89.45% compared to head (22952cd) 89.45%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #287   +/-   ##
=======================================
  Coverage   89.45%   89.45%           
=======================================
  Files          98       98           
  Lines        4408     4410    +2     
  Branches      849      850    +1     
=======================================
+ Hits         3943     3945    +2     
  Misses        323      323           
  Partials      142      142           
Flag Coverage Δ
docs 100.00% <ø> (ø)
junifer 89.45% <100.00%> (+<0.01%) ⬆️

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

Files Coverage Δ
junifer/stats.py 100.00% <100.00%> (ø)
## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/287?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report All modified and coverable lines are covered by tests :white_check_mark: > Comparison is base [(`1382fd2`)](https://app.codecov.io/gh/juaml/junifer/commit/1382fd205e13d46a97ce238a53b9ba5604ce14a8?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) 89.45% compared to head [(`22952cd`)](https://app.codecov.io/gh/juaml/junifer/pull/287?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) 89.45%. <details><summary>Additional details and impacted files</summary> [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/287/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/287?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #287 +/- ## ======================================= Coverage 89.45% 89.45% ======================================= Files 98 98 Lines 4408 4410 +2 Branches 849 850 +1 ======================================= + Hits 3943 3945 +2 Misses 323 323 Partials 142 142 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/287/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/287/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/287/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `89.45% <100.00%> (+<0.01%)` | :arrow_up: | 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/287?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/stats.py](https://app.codecov.io/gh/juaml/junifer/pull/287?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9zdGF0cy5weQ==) | `100.00% <100.00%> (ø)` | | </details>
github-actions[bot] commented 2024-01-10 12:31:00 +00:00 (Migrated from github.com)
PR Preview Action v1.4.6
Preview removed because the pull request was closed.
2024-01-12 10:59 UTC
[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.4.6 :---: Preview removed because the pull request was closed. 2024-01-12 10:59 UTC <!-- Sticky Pull Request Commentpr-preview -->
fraimondo (Migrated from github.com) approved these changes 2024-01-12 10:54:52 +00: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!287
No description provided.