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

Make DALI tests compatible with Python 3.12 #5452

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

JanuszL
Copy link
Contributor

@JanuszL JanuszL commented May 2, 2024

  • adds additional replacements for functions used by nose
    and removed in python 3.12

Category:

Other (e.g. Documentation, Tests, Configuration)

Description:

  • adds additional replacements for functions used by nose
    and removed in python 3.12

Additional information:

Affected modules and functionalities:

  • nose test wrapper
  • nose test utils

Key points relevant for the review:

Tests:

  • Existing tests apply
    • all nose tests
  • New tests added
    • Python tests
    • GTests
    • Benchmark
    • Other
  • N/A

Checklist

Documentation

  • Existing documentation applies
  • Documentation updated
    • Docstring
    • Doxygen
    • RST
    • Jupyter
    • Other
  • N/A

DALI team only

Requirements

  • Implements new requirements
  • Affects existing requirements
  • N/A

REQ IDs: N/A

JIRA TASK: DALI-3958

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [14688850]: BUILD STARTED

qa/nose_wrapper/__main__.py Fixed Show fixed Hide fixed
qa/nose_wrapper/__main__.py Fixed Show fixed Hide fixed
qa/nose_wrapper/__main__.py Fixed Show fixed Hide fixed
qa/nose_wrapper/__main__.py Fixed Show fixed Hide fixed
@JanuszL JanuszL force-pushed the make_test_python3.12_compat branch from d171b28 to 087d99c Compare May 2, 2024 09:28
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [14689665]: BUILD STARTED

@JanuszL JanuszL force-pushed the make_test_python3.12_compat branch from 087d99c to 435bb70 Compare May 2, 2024 09:52
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [14689899]: BUILD STARTED

- adds additional replacements for functions used by nose
  and removed in python 3.12
- remove numpy version pinning as the <1.26 is not compatible
  with python 3.12

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@JanuszL JanuszL force-pushed the make_test_python3.12_compat branch from 435bb70 to 570cdb5 Compare May 2, 2024 11:20
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [14691274]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [14691274]: BUILD FAILED



BEOPEN.COM LICENSE AGREEMENT FOR PYTHON 2.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to copy the old licenses, too?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably not, removed.

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
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.

None yet

4 participants