Skip to content

Conversation

@stefmolin
Copy link
Owner

@stefmolin stefmolin commented Feb 16, 2025

Fixes #218
Fixes #274 (hopefully)

Describe your changes

  • Switch from tqdm to rich for progress tracking
  • Set up pytest-xdist to distribute tests over multiple CPU cores
  • Reworked CLI to parallelize morphing over specified number of workers when there is more than one job and more than one worker requested. Passing 0 will use all available cores.
  • Updated docs and test suite for new additions
  • Set matplotlib backend to Agg in actions to hopefully fix tkinter issue on Windows runners
Screenshot 2025-02-16 at 12 38 07 PM

Checklist

  • Test cases have been modified/added to cover any code changes.
  • Docstrings have been modified/created for any code changes.
  • All linting and formatting checks pass (see the contributing guidelines for more information).

@github-actions github-actions bot added documentation Improvements or additions to documentation ci/cd Relating to local development or CI/CD testing Relating to the testing suite morpher Work relating to morpher module cli Work relating to the cli packaging Relating to packaging labels Feb 16, 2025
@codecov
Copy link

codecov bot commented Feb 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.58%. Comparing base (ace8539) to head (f3b67af).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #273      +/-   ##
==========================================
+ Coverage   98.54%   98.58%   +0.03%     
==========================================
  Files          80       81       +1     
  Lines        1991     2043      +52     
  Branches      113      121       +8     
==========================================
+ Hits         1962     2014      +52     
  Misses         26       26              
  Partials        3        3              
Files with missing lines Coverage Δ
src/data_morph/cli.py 100.00% <100.00%> (ø)
src/data_morph/morpher.py 100.00% <100.00%> (ø)
src/data_morph/progress.py 100.00% <100.00%> (ø)
tests/test_cli.py 100.00% <100.00%> (ø)
tests/test_morpher.py 100.00% <100.00%> (ø)

@stefmolin stefmolin changed the title Run in parallel Run in parallel via CLI Feb 16, 2025
@stefmolin stefmolin merged commit 023af7c into main Feb 16, 2025
23 checks passed
@stefmolin stefmolin deleted the run-in-parallel branch February 16, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd Relating to local development or CI/CD cli Work relating to the cli documentation Improvements or additions to documentation morpher Work relating to morpher module packaging Relating to packaging testing Relating to the testing suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Frequent _tkinter.TclError when running test suite on Windows runners (GitHub Actions) Run multiple morphs in parallel

2 participants