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

Feature/neural magic extension of the Elyra Tutorial #446

Merged
merged 36 commits into from Oct 5, 2021

Conversation

pacospace
Copy link

Related Issues and Dependencies

Related-To: #297

Francesco Murdaca added 22 commits September 22, 2021 11:53
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@pacospace pacospace requested review from chauhankaranraj and removed request for Shreyanand September 22, 2021 15:15
@sesheta sesheta added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Sep 22, 2021
@pacospace pacospace force-pushed the feature/neural-magic branch 3 times, most recently from a2cdb65 to 2718d8e Compare September 30, 2021 10:18
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
@pacospace pacospace force-pushed the feature/neural-magic branch 6 times, most recently from 1caa94c to 9c64fa5 Compare September 30, 2021 16:46
@sesheta
Copy link

sesheta commented Sep 30, 2021

AICoE CI results

Model and application metrics

The following table shows gathered metrics for model and application on your deployed models.

average_error average_latency average_probability model_version number_requests namespace deployment test URL
0 0.672843 0.991821 pr-446-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/9c64fa561d135a728b8c3b19dc99d00233505e0a/features
0.02 0.643769 0.989049 pr-446-pytorch-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/9c64fa561d135a728b8c3b19dc99d00233505e0a/features

Platform metrics

The following table shows gathered metrics from platform on your deployed models.

CPU max usage Memory max usage model_version
N/A N/A pr-446-inference
N/A N/A pr-446-pytorch-inference

@AICoE AICoE deleted a comment from sesheta Sep 30, 2021
@sesheta
Copy link

sesheta commented Oct 4, 2021

AICoE CI results

Model and application metrics

The following table shows gathered metrics for model and application on your deployed models.

average_error average_latency average_probability model_version number_requests namespace deployment test URL
0 0.679479 0.991821 pr-446-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/9c64fa561d135a728b8c3b19dc99d00233505e0a/features
0.01 0.60259 0.99216 pr-446-pytorch-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/9c64fa561d135a728b8c3b19dc99d00233505e0a/features

Platform metrics

The following table shows gathered metrics from platform on your deployed models.

CPU max usage Memory max usage model_version
N/A N/A pr-446-inference
N/A N/A pr-446-pytorch-inference

@pacospace
Copy link
Author

/retest

1 similar comment
@pacospace
Copy link
Author

/retest

@sesheta
Copy link

sesheta commented Oct 4, 2021

AICoE CI results

Model and application metrics

The following table shows gathered metrics for model and application on your deployed models.

average_error average_latency average_probability model_version number_requests namespace deployment test URL
0 0.677138 0.991821 pr-446-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/519625e90802c18b7e0ea874ab887343cb62ec9b/features
0.02 0.780993 0.989103 pr-446-pytorch-inference 100 opf-ci-pipelines https://github.com/AICoE/elyra-aidevsecops-tutorial/blob/519625e90802c18b7e0ea874ab887343cb62ec9b/features

Platform metrics

The following table shows gathered metrics from platform on your deployed models.

CPU max usage Memory max usage model_version
N/A N/A pr-446-inference
N/A N/A pr-446-pytorch-inference

Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
@sesheta
Copy link

sesheta commented Oct 5, 2021

@pacospace: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
aicoe-ci/prow/mypy 635a13c link /test thoth-mypy-py38

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
Signed-off-by: Francesco Murdaca <fmurdaca@redhat.com>
@pacospace
Copy link
Author

/approve

@sesheta
Copy link

sesheta commented Oct 5, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pacospace

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 5, 2021
@sesheta sesheta merged commit e6cd64a into AICoE:master Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants