-
Notifications
You must be signed in to change notification settings - Fork 0
Feat add output schema #17
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
Open
IsabellaARM
wants to merge
11
commits into
main
Choose a base branch
from
feat-add-output-schema
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Features: - Immutable dataclasses (frozen=True) - Full serialization (to_dict/from_dict/to_json/from_json/save/load) - Schema validation with jsonschema library support - UUID and SHA-256 format validation - Pretty-printing support - Serialization tests Resolve: MLIA-1513 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: I478af2354d6a690a9ece3e9f99269bcc97d179b6 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1153954 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1168437
Resolve: MLIA-1517 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: I4a834b23fbb30504d651cce7758a132760426d44 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1153997 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Oluwatamilore Adebayo <oluwatamilore.adebayo@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1168438
Resolve: MLIA-1521 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: I7a39d5326d646f8cf909acd5e4b9e2ff2d63ab83 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156393 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1168439
- Fixed some MD004 errors - Added table of content - Unified optimization tables for a better rendering - Added hyperlink to backend installation Resolve: MLIA-1543 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: Ic34c062149bad0c27c1d2b894bd00161a6e32fe2 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1147609 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Mike Kelly <mike.kelly@arm.com> IP-review: Mike Kelly <mike.kelly@arm.com> Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1168443
Resolve: MLIA-1518 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: I7debdd2d572c12de7ee7e453338989f6c25ca2fe Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156563 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com>
Resolve: MLIA-1516 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: Ic297f500e1316f5e4bc4b7653215b609c325f791 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1166903 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com>
Resolve: MLIA-1520 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: Idcbe9e61f62221ee54c04707e87d94e47a9836a1 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1156534 Tested-by: expkit <svc_expkit@arm.com>
Resolve: MLIA-1585 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: Ibb2cde7c91dd72b89e5e701b1d2aa9faadd9ea89 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1157178 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Maksims Svecovs <maksims.svecovs@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com>
- Advice class: Added metadata field and to_extension_dict() method for structured JSON output with id, message, and metadata fields - JSONReporter: Auto-detects standardized_output from wrapper objects and outputs unified schema format, with advice as extensions Resolve: MLIA-1522 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: Ibcdcec1c9206b82656108941f00f740ba2bcc191 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1166859 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: James Edgar <james.edgar@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com>
- Adapt target configuration file to the new schema - Tests added Resolve: MLIA-1548, MLIA-1549 Signed-off-by: Isabella Gottardi <isabella.gottardi@arm.com> Change-Id: I7bacee2e7c5f4f9bfa4f14123455d8481c336d7d Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1155531 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Maksims Svecovs <maksims.svecovs@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com> (cherry picked from commit 799ed63c75e8492a8bb6e9031c23b8f9bec7f64b)
* Tests added for output_schema.py, output_validation.py and vela/performance.py::to_standarized_output * A hash validation bug fixed in output_validation.py Resolves: MLIA-1523 Change-Id: If33b3809ae6b90b436270da37ced4ffd18e95ed4 Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1172892 Tested-by: expkit <svc_expkit@arm.com> Reviewed-by: Maksims Svecovs <maksims.svecovs@arm.com> IP-review: Maksims Svecovs <maksims.svecovs@arm.com> (cherry picked from commit 1ad866eeff1ed2ae12242005b33db583dc74cce7) Reviewed-on: https://eu-gerrit-2.euhpc.arm.com/c/ml/ecosystem/mlia/+/1172671 Reviewed-by: Isabella Gottardi <isabella.gottardi@arm.com> IP-review: Isabella Gottardi <isabella.gottardi@arm.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.