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

[OTE_SDK_TESTS] Added input parameters validation and tests. Part 2 Shapes #891

Closed

Conversation

saltykox
Copy link
Contributor

@saltykox saltykox commented Feb 2, 2022

Added input parameters validations for Shapes classes of OTE SDK:
• Ellipse class: initialization and get_evenly_distributed_ellipse_coordinates methods
• Point class: initialization, normalize_wrt_roi and denormalize_wrt_roi_shape methods
• Polygon class: initialization method
• Rectangle class: initialization, is_full_box and crop_numpy_array methods
• Shape class: initialization, intersects, contains_center, get_labels, append_label, set_labels and _validate_coordinates methods
Tests updated according to above changes:
• test_rectangle.py: test_rectangle_is_full_box
Added tests/parameters_validation/test_shapes_input_parameters_validation.py with negative scenarios to check that ValueError exception is raised when unexpected type object is specified in Shape methods

Task: CVS-77259

@saltykox saltykox added API Any changes in OTX API TEST Any changes in tests labels Feb 2, 2022
@saltykox saltykox requested review from LeonidBeynenson, Ilya-Krylov and a team February 2, 2022 06:51
@saltykox saltykox marked this pull request as draft February 2, 2022 06:51
@saltykox saltykox removed the request for review from a team February 2, 2022 06:51
@saltykox saltykox marked this pull request as ready for review February 2, 2022 09:31
@saltykox
Copy link
Contributor Author

saltykox commented Feb 2, 2022

run ote_sdk tests

1 similar comment
@saltykox
Copy link
Contributor Author

saltykox commented Feb 2, 2022

run ote_sdk tests

@saltykox saltykox marked this pull request as draft February 2, 2022 15:06
@saltykox saltykox marked this pull request as ready for review February 2, 2022 15:06
@saltykox saltykox marked this pull request as draft February 7, 2022 12:41
@saltykox saltykox marked this pull request as ready for review February 10, 2022 10:22
@saltykox saltykox marked this pull request as draft February 10, 2022 14:29
@nervana-ff
Copy link

Can one of the admins verify this patch?

@github-actions github-actions bot removed the TEST Any changes in tests label Feb 16, 2022
@Ilya-Krylov
Copy link
Contributor

draft?

@saltykox
Copy link
Contributor Author

This pull request is in the “Draft” state due to unmerged base PR #890, but the approach to check input parameters is ready for review

@github-actions github-actions bot added the ALGO Any changes in OTX Algo Tasks implementation label Mar 25, 2022
@github-actions github-actions bot removed the ALGO Any changes in OTX Algo Tasks implementation label Mar 25, 2022
saltykox added 2 commits April 1, 2022 09:16
…tion_tests

# Conflicts:
#	ote_sdk/ote_sdk/tests/parameters_validation/test_shapes_input_parameters_validation.py
#	ote_sdk/ote_sdk/utils/argument_checks.py
@saltykox saltykox marked this pull request as ready for review April 1, 2022 07:15
@saltykox
Copy link
Contributor Author

saltykox commented Apr 1, 2022

@LeonidBeynenson @Ilya-Krylov please review

@saltykox saltykox added the TEST Any changes in tests label Apr 1, 2022
@saltykox
Copy link
Contributor Author

saltykox commented Apr 1, 2022

run ote_sdk tests

@saltykox
Copy link
Contributor Author

saltykox commented Apr 1, 2022

run ote-test

@github-actions github-actions bot removed the TEST Any changes in tests label Apr 1, 2022
@saltykox saltykox marked this pull request as draft April 5, 2022 07:58
@github-actions github-actions bot added the ALGO Any changes in OTX Algo Tasks implementation label Apr 5, 2022
@github-actions github-actions bot removed the ALGO Any changes in OTX Algo Tasks implementation label Apr 5, 2022
@yunchu
Copy link
Contributor

yunchu commented Nov 18, 2022

closed due to clean up the stale branches

@yunchu yunchu closed this Nov 18, 2022
@yunchu yunchu deleted the vsaltykovx/add_shapes_input_params_validation_tests branch November 18, 2022 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Any changes in OTX API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants