search

KosinskiLab / AlphaPulldown

https://doi.org/10.1093/bioinformatics/btac749
GNU General Public License v3.0
176 stars 39 forks source link

Group unittests #346

Open dingquanyu opened 1 month ago

dingquanyu commented 1 month ago

Re-organised unittest scripts

DimaMolod commented 1 month ago
  1. test_features_with_templates.py should go to github_CI_tests
  2. If you want to run CPU tests using slurm, I suggest you to make a wrapper, similar to the one for testing predictions
  3. I also suggest using pytest everywhere: it works for a unit test-like structures with classes but can also be used in a simplified manner using functions. Additional features like fixtures and parametrization look very beneficial to me.
  4. There are not enough tests, e.g., postprocessing is not tested at all. For padding, I found only this; why need GPU? Why not prepare features on CPU and compare the shapes (and the elements of the array itself)?