Skip to content

[com1;com4;com8;com9] Removed Windows-OS-specific conditional imports for multiprocessing#1234

Merged
fso42 merged 1 commit intomasterfrom
removeWinParallelDifference
Feb 5, 2026
Merged

[com1;com4;com8;com9] Removed Windows-OS-specific conditional imports for multiprocessing#1234
fso42 merged 1 commit intomasterfrom
removeWinParallelDifference

Conversation

@fso42
Copy link
Contributor

@fso42 fso42 commented Feb 5, 2026

Reported by @stnfranck

On Windows recent python versions don't need the distinction for parallel threads anymore. This lead to having multiple processes but only on one core.

It runs on multiple cores now...

@fso42 fso42 self-assigned this Feb 5, 2026
@fso42 fso42 added confirmed Something isn't working enhancement New feature or request labels Feb 5, 2026
@qltysh
Copy link
Contributor

qltysh bot commented Feb 5, 2026

Qlty

Coverage Impact

⬆️ Merging this pull request will increase total coverage on master by 0.03%.

Modified Components (1)

RatingComponent% Diff
Coverage rating: C Coverage rating: C
com1DFA100.0%

Modified Files with Diff Coverage (4)

RatingFile% DiffUncovered Line #s
Coverage rating: F Coverage rating: F
avaframe/com4FlowPy/flowCore.py100.0%
Coverage rating: B Coverage rating: B
avaframe/com1DFA/com1DFA.py100.0%
Coverage rating: F Coverage rating: F
avaframe/com8MoTPSA/com8MoTPSA.py0.0%12
Coverage rating: F Coverage rating: F
avaframe/com9MoTVoellmy/com9MoTVoellmy.py100.0%
Total75.0%
🤖 Increase coverage with AI coding...

In the `removeWinParallelDifference` branch, add test coverage for this new code:

- `avaframe/com8MoTPSA/com8MoTPSA.py` -- Line 12

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@fso42 fso42 changed the title refactor(core):Removed Windows-OS-specific conditional imports for multiprocessing [com1;com4;com8;com9] Removed Windows-OS-specific conditional imports for multiprocessing Feb 5, 2026
@fso42
Copy link
Contributor Author

fso42 commented Feb 5, 2026

Standardtest ident

@fso42 fso42 merged commit fc56616 into master Feb 5, 2026
4 checks passed
@fso42 fso42 deleted the removeWinParallelDifference branch February 5, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

confirmed Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant