[ENH]: Consistency of logging #239

Merged

synchon merged 20 commits from update/logging-consistency into main

2026-06-15 09:02:42 +00:00

synchon commented

2025-11-21 10:50:56 +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?

Logging level "INFO" should log all parameters.

Currently, some steps log parameters and some other don't. When level is INFO, this should be general.

Example: in fMRIPrepConfoundRemover, filtering parameters are logged only in DEBUG level.

How do you imagine this integrated in junifer?

Go one by one thorugh the datagbrabbers/preprocessing/markers/storage and check mostly the "init".

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? Logging level "INFO" should log all parameters. Currently, some steps log parameters and some other don't. When level is INFO, this should be general. Example: in `fMRIPrepConfoundRemover`, filtering parameters are logged only in DEBUG level. ### How do you imagine this integrated in junifer? Go one by one thorugh the datagbrabbers/preprocessing/markers/storage and check mostly the "init". ### Do you have a sample code that implements this outside of junifer? _No response_ ### Anything else to say? _No response_

codecov[bot] commented

2025-11-21 10:51:49 +00:00

(Migrated from github.com)

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8851c3c) to head (36cac57).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #239   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            1         1           
=========================================
  Hits             1         1

Flag	Coverage Δ
docs	`100.00% <ø> (ø)`

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

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/239?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report :white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 100.00%. Comparing base ([`8851c3c`](https://app.codecov.io/gh/juaml/junifer/commit/8851c3c154e97e6ad82d894a287a2be122015041?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)) to head ([`36cac57`](https://app.codecov.io/gh/juaml/junifer/commit/36cac57d1a9536c133723acc6cfc1de8cab648d1?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)). :warning: Report is 1 commits behind head on main. <details><summary>Additional details and impacted files</summary> [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/239/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/239?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #239 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 1 1 Lines 1 1 ========================================= Hits 1 1 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/239/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/239/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `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. </details> <details><summary> :rocket: New features to boost your workflow: </summary> - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. </details>

fraimondo (Migrated from github.com) approved these changes

2026-06-03 14:24:12 +00:00