Skip to content

Conversation

@RHeckerIntel
Copy link
Collaborator

Adds ability to import models that were exported or downloaded (or converted yourself).

After you select select a folder, if it contains a project.json, meaning it was probably exported from openvino test drive, it will just import it.

If it does not contain a valid project.json. it will show a short form to fill in name and task.

image image

Implementation details:

Models are stored by folder path in the config.json. When you remove a model that is stored that way it will simply remove the reference and leave the files be.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has checked 525 files.

Valid Invalid Ignored Fixed
251 1 273 0
Click to see the invalid file list
  • lib/importers/model_directory_importer.dart
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

@@ -0,0 +1,60 @@
import 'dart:convert';

Choose a reason for hiding this comment

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

Suggested change
import 'dart:convert';
// Copyright (c) 2024 Intel Corporation
//
// SPDX-License-Identifier: Apache-2.0
import 'dart:convert';

@codecov
Copy link

codecov bot commented Mar 14, 2025

Codecov Report

Attention: Patch coverage is 16.00000% with 231 lines in your changes missing coverage. Please review.

Project coverage is 31.33%. Comparing base (3bdbb3d) to head (e71bb19).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
lib/pages/import/widgets/directory_import.dart 0.00% 102 Missing ⚠️
lib/pages/import/widgets/geti_import.dart 0.00% 47 Missing ⚠️
lib/pages/import/widgets/import_model_dialog.dart 3.03% 32 Missing ⚠️
lib/importers/model_directory_importer.dart 0.00% 20 Missing ⚠️
lib/project.dart 35.71% 9 Missing ⚠️
lib/config.dart 73.91% 6 Missing ⚠️
lib/providers/project_provider.dart 0.00% 5 Missing ⚠️
lib/deployment_processor.dart 66.66% 3 Missing ⚠️
lib/utils.dart 57.14% 3 Missing ⚠️
lib/importers/geti_deployment.dart 0.00% 1 Missing ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   32.00%   31.33%   -0.67%     
==========================================
  Files         140      143       +3     
  Lines        5706     5882     +176     
==========================================
+ Hits         1826     1843      +17     
- Misses       3880     4039     +159     
Flag Coverage Δ
integrationtests 16.84% <9.09%> (-0.31%) ⬇️
unittests 20.84% <10.21%> (-0.48%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RHeckerIntel RHeckerIntel merged commit ec67902 into main Apr 9, 2025
12 of 14 checks passed
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