🚀 feat(inspect): Geti Inspect

[ashwinvaidya17]

📝 Description

• Merge feature branch into main

✨ Changes

Select what type of change your PR is:

• 🚀 New feature (non-breaking change which adds functionality)
• 🐞 Bug fix (non-breaking change which fixes an issue)
• 🔄 Refactor (non-breaking change which refactors the code base)
• ⚡ Performance improvements
• 🎨 Style changes (code style/formatting)
• 🧪 Tests (adding/modifying tests)
• 📚 Documentation update
• 📦 Build system changes
• 🚧 CI/CD configuration
• 🔧 Chore (general maintenance)
• 🔒 Security update
• 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

Before you submit your pull request, please make sure you have completed the following steps:

• 📚 I have made the necessary updates to the documentation (if applicable).
• 🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
• 🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

[Copilot]

Pull request overview

Copilot reviewed 234 out of 415 changed files in this pull request and generated 3 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Importing a private function (prefixed with _) from another module is an anti-pattern and breaks encapsulation. Consider making this function public or refactoring the test to use the public API.

Suggested change

	from api.endpoints.trainable_models_endpoints import _get_trainable_models # noqa: PLC2701
	def test_list_trainable_models(fxt_client):
	_get_trainable_models.cache_clear()
	# Mock anomalib.models.list_models to return a predictable set
	with patch("api.endpoints.trainable_models_endpoints.list_models", return_value={"padim", "patchcore"}):
	def test_list_trainable_models(fxt_client):
	# Mock anomalib.models.list_models to return a predictable set
	with patch("api.endpoints.trainable_models_endpoints.list_models", return_value={"padim", "patchcore"}):
	with patch("api.endpoints.trainable_models_endpoints.list_models", return_value={"padim", "patchcore"}):

[Copilot]

Error message contains 'TODO' indicating incomplete implementation. Replace with a proper error message like 'Job ID is required for cancellation' or ensure jobs always have IDs before this point.

Suggested change

	throw Error('TODO: jobs should always have an ID');
	throw Error('Job ID is required for cancellation');

[Copilot]

[nitpick] CSS custom property is defined but not used in the component's styles. Either use this property in the CSS or remove it to avoid confusion.

Suggested change

'--zoom-scale': zoom.scale,

[Copilot]

Pull request overview

Copilot reviewed 234 out of 417 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

[nitpick] Comment uses 'train loop' but should clarify this is the training worker's run loop.

Suggested change

	# Run the train loop
	# Run the training worker's run loop

[Copilot]

Pull request overview

Copilot reviewed 227 out of 417 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Importing a private function (prefixed with _) from another module violates encapsulation. Consider exposing this function publicly or refactoring the test to use the public API.

[Copilot]

Pull request overview

Copilot reviewed 228 out of 425 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Empty error message in useSetZoom hook. The error message should be descriptive to help developers understand what went wrong, e.g., 'useSetZoom must be used within a ZoomProvider'.

Suggested change

	throw new Error('');
	throw new Error('useSetZoom must be used within a ZoomProvider');

[Copilot]

The opacity is set both in the style prop and in the animate prop. Since the animate prop controls the opacity transition, the inline style is redundant and could cause conflicts. Remove the inline style and rely solely on the animate prop.

Suggested change

style={{ opacity: inferenceOpacity }}

[Copilot]

Pull request overview

Copilot reviewed 229 out of 448 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

[nitpick] CSS custom property value should be converted to a string. While JavaScript may coerce the number, explicit conversion with .toString() or template literal is more reliable and matches CSS custom property best practices.

Suggested change

	'--zoom-scale': zoom.scale,
	'--zoom-scale': `${zoom.scale}`,

[Copilot]

Pull request overview

Copilot reviewed 229 out of 448 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The fire-and-forget task creation could lead to unhandled exceptions being silently ignored. Consider storing task exceptions and logging them. For example: task = asyncio.create_task(training_service.train_pending_job()); task.add_done_callback(lambda t: logger.error(f'Task failed: {t.exception()}') if t.exception() else None)

Suggested change

	running_tasks.add(asyncio.create_task(training_service.train_pending_job()))
	task = asyncio.create_task(training_service.train_pending_job())
	task.add_done_callback(
	lambda t: logger.error(f"Task failed: {t.exception()}", exc_info=True) if t.exception() else None
	)
	running_tasks.add(task)

[Copilot]

Pull request overview

Copilot reviewed 231 out of 454 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 230 out of 454 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.