Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Too many upstream warnings #34

Open
andrewsb8 opened this issue Apr 30, 2024 · 2 comments
Open

Too many upstream warnings #34

andrewsb8 opened this issue Apr 30, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@andrewsb8
Copy link
Owner

Current output of pytest: = 42 passed, 1 skipped, 225133 warnings in 177.76s (0:02:57) =

Only 2 or 3 warnings are coming from the explicit output of tests. The rest are coming from dependencies like random and tensorflow probability. Warnings summary:

===================== warnings summary ======================
tests/test_cases/test_check_target.py: 60 warnings
tests/test_cases/test_codon_translations.py: 20 warnings
tests/test_cases/test_ntrials.py: 20 warnings
tests/test_cases/test_optimizers.py: 112 warnings
tests/test_cases/test_random_seed.py: 224864 warnings
tests/test_cases/test_resume_design.py: 12 warnings
tests/test_cases/test_verify_dna.py: 40 warnings
  /home/bandrews/miniconda3/envs/quvax/lib/python3.11/random.py:362: DeprecationWarning: non-integer arguments to randrange() have been deprecated since Python 3.10 and will be removed in a subsequent version
    return self.randrange(a, b+1)

tests/test_cases/test_optimizers.py::test_TFDE
  /home/bandrews/miniconda3/envs/quvax/lib/python3.11/site-packages/tensorflow/python/debug/cli/debugger_cli_common.py:19: DeprecationWarning: module 'sre_constants' is deprecated
    import sre_constants

tests/test_cases/test_optimizers.py::test_TFDE
tests/test_cases/test_optimizers.py::test_TFDE
  /home/bandrews/miniconda3/envs/quvax/lib/python3.11/site-packages/tensorflow_probability/python/__init__.py:57: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    if (distutils.version.LooseVersion(tf.__version__) <

tests/test_cases/test_optimizers.py::test_TFDE
tests/test_cases/test_optimizers.py::test_TFDE
  /home/bandrews/miniconda3/envs/quvax/lib/python3.11/site-packages/tensorflow_probability/python/__init__.py:58: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    distutils.version.LooseVersion(required_tensorflow_version)):

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

Having this many warnings come from tests does not look like tests are actually passing unless one digs into the warnings themselves.

@andrewsb8 andrewsb8 added the enhancement New feature or request label May 1, 2024
@andrewsb8
Copy link
Owner Author

Look into context manager that suppresses some warnings

@andrewsb8
Copy link
Owner Author

Since #40, the number of warnings has gone down to ~5000. While better, it's still too high.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant