Skip to content

Increase utility test coverage#29

Open
mickaelbegon wants to merge 15 commits into
pyomeca:mainfrom
mickaelbegon:pyomeca-main/increase-coverage
Open

Increase utility test coverage#29
mickaelbegon wants to merge 15 commits into
pyomeca:mainfrom
mickaelbegon:pyomeca-main/increase-coverage

Conversation

@mickaelbegon

@mickaelbegon mickaelbegon commented May 7, 2026

Copy link
Copy Markdown

Summary

  • add fast utility tests for Fourier approximation helpers
  • cover solution pickle export duplicate-time handling
  • cover pulse intensity sliding-window constraint padding and trimming

Note

Verification

  • black . --exclude "external/*" --check -l120
  • PYTHONPATH=/private/tmp/pyomeca-bioptim-fdafe4d:. MPLBACKEND=Agg OMP_NUM_THREADS=1 OPENBLAS_NUM_THREADS=1 MKL_NUM_THREADS=1 VECLIB_MAXIMUM_THREADS=1 /Users/mickaelbegon/miniconda3/envs/cocofest-bayes/bin/python -m pytest tests/shard1/test_utility_coverage.py -q
  • PYTHONPATH=/private/tmp/pyomeca-bioptim-fdafe4d:. MPLBACKEND=Agg OMP_NUM_THREADS=1 OPENBLAS_NUM_THREADS=1 MKL_NUM_THREADS=1 VECLIB_MAXIMUM_THREADS=1 /Users/mickaelbegon/miniconda3/envs/cocofest-bayes/bin/python -m pytest --cov-report term-missing --cov=cocofest tests/shard1/test_models_dynamics_without_bioptim.py tests/shard1/test_ivp.py tests/shard1/test_fes_dynamics.py tests/shard1/test_utility_coverage.py -q

This change is Reviewable

@mickaelbegon mickaelbegon force-pushed the pyomeca-main/increase-coverage branch from 75a37e1 to 37937b3 Compare May 8, 2026 12:53
@mickaelbegon mickaelbegon force-pushed the pyomeca-main/increase-coverage branch from 37937b3 to cc76233 Compare May 8, 2026 13:15
@mickaelbegon mickaelbegon force-pushed the pyomeca-main/increase-coverage branch from cc76233 to 1df5f35 Compare May 9, 2026 17:09
@mickaelbegon mickaelbegon force-pushed the pyomeca-main/increase-coverage branch from 1df5f35 to 9975404 Compare May 18, 2026 11:58
@mickaelbegon

Copy link
Copy Markdown
Author

I rebuilt this branch on top of the cleaned bioptim 3.4 migration PR so the diff is now limited to the added utility coverage test only.

Current scope:

  • tests/shard1/test_utility_coverage.py

Validation on my side:

  • python -m pytest tests/shard1/test_utility_coverage.py -q -> 5 passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant