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.

Sign up for GitHub

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

Jump to bottom

Fix warnings in test_visualizer.py #1039

Merged
merged 2 commits into from
Jun 12, 2023
Merged

Fix warnings in test_visualizer.py #1039

merged 2 commits into from
Jun 12, 2023

Conversation

vinnamkim
Copy link
Contributor

@vinnamkim vinnamkim commented Jun 7, 2023 •
edited
Loading

Summary

  • Ticket no. 112846
  • Close plt.Figure each time the test teardowns
  • Replace deprecated function usage: np.random.random_integers() ->
    np.random.randint()

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

@vinnamkim vinnamkim added the BUG Something isn't working label Jun 7, 2023
@vinnamkim vinnamkim added this to the 1.4.0 milestone Jun 7, 2023
@vinnamkim
Fix warnings in test_vsiaulizer.py
c5268ba 
 - Close plt.Figure each time the test teardowns
 - Replace deprecated function usage: np.random.random_integers() ->
   np.random.randint()

Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
@vinnamkim vinnamkim marked this pull request as ready for review June 7, 2023 06:24
@vinnamkim vinnamkim requested review from a team as code owners June 7, 2023 06:24
@vinnamkim
Update CHANGELOG.md
f264013 
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
@vinnamkim vinnamkim requested a review from yunchu June 7, 2023 23:33
@vinnamkim vinnamkim merged commit 313446d into openvinotoolkit:develop Jun 12, 2023
@vinnamkim vinnamkim deleted the bugfix/fix-warnings-in-test-visualizer branch June 12, 2023 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yunchu yunchu yunchu approved these changes

@sooahleex sooahleex sooahleex approved these changes

@chuneuny-emily chuneuny-emily Awaiting requested review from chuneuny-emily chuneuny-emily is a code owner automatically assigned from openvinotoolkit/datumaro-admins

@JihwanEom JihwanEom Awaiting requested review from JihwanEom JihwanEom is a code owner automatically assigned from openvinotoolkit/datumaro-admins

@wonjuleee wonjuleee Awaiting requested review from wonjuleee wonjuleee is a code owner automatically assigned from openvinotoolkit/datumaro-admins

@bonhunko bonhunko Awaiting requested review from bonhunko bonhunko is a code owner automatically assigned from openvinotoolkit/datumaro-admins

Assignees
No one assigned
Labels
BUG Something isn't working
Projects
None yet
Milestone
1.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@vinnamkim @yunchu @sooahleex