Skip to content

[DIAG-635] Enable interfaces to execute MEA deletions #266

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

Merged

Conversation

tomislav-peharda
Copy link
Contributor

Description:
The goal of this PR is to enable SDK interfaces to execute deletions on MEA models, model runs & annotation groups (by data row ids).



def test_model_run_delete(client, model_run_annotation_groups):
# TODO
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm having issues running this test. Basically, it freezes when running and I suppose it's because of model_run_annotation_groups @msokoloff1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed by testing against staging env

Copy link
Contributor

@msokoloff1 msokoloff1 left a comment

Choose a reason for hiding this comment

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

Great job! The code needs to be formatted fyi. Install yapf and then run the following from the root of the project:
yapf labelbox tests -i --verbose --recursive --parallel --style "google"

@msokoloff1 msokoloff1 merged commit 7c32542 into develop Sep 2, 2021
@msokoloff1 msokoloff1 deleted the fs/mea-enable-model-model-runs-annotation-groups-deletions branch September 2, 2021 10:33
msokoloff1 added a commit that referenced this pull request Sep 22, 2021
…annotation-groups-deletions

[DIAG-635] Enable interfaces to execute MEA deletions
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.

2 participants