Skip to content

Add hotkey to assign empty label #1414

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.

Sign up for GitHub

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
merged 4 commits into from
Oct 22, 2025
Merged

Add hotkey to assign empty label #1414

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
16b1eac
Add hotkey to assign empty label
jpggvilaca Oct 21, 2025
715bc96
Copilot comment
jpggvilaca Oct 21, 2025
55b6ea0
Address comment
jpggvilaca Oct 22, 2025
8316289
Address comment
jpggvilaca Oct 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions ...c/pages/annotator/components/main-content/labels-shortcuts/labels-shortcuts.component.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import { useCallback, useMemo } from 'react';

import { Flex, Tooltip, TooltipTrigger } from '@geti/ui';
import { isEmpty } from 'lodash-es';
import { useHotkeys } from 'react-hotkeys-hook';
import { useLocalStorage } from 'usehooks-ts';

import { Annotation } from '../../../../../core/annotations/annotation.interface';
Expand All @@ -15,6 +16,8 @@ import { useProjectIdentifier } from '../../../../../hooks/use-project-identifie
import { LOCAL_STORAGE_KEYS } from '../../../../../shared/local-storage-keys';
import { getId, getIds, hasEqualId } from '../../../../../shared/utils';
import { AnnotationToolContext, ToolSettings, ToolType } from '../../../core/annotation-tool-context.interface';
import { useAnnotatorHotkeys } from '../../../hooks/use-hotkeys-configuration.hook';
import { HOTKEY_OPTIONS } from '../../../hot-keys/utils';
import { getOutputFromTask } from '../../../providers/task-chain-provider/utils';
import { useTask } from '../../../providers/task-provider/task-provider.component';
import { LabelsHotkeys } from '../../labels/labels-hotkeys/labels-hotkeys.component';
Expand All @@ -39,6 +42,7 @@ const hasNotAnomalousNorLocalLabels = (annotation: Annotation) =>
export const LabelsShortcuts = ({ labels, annotationToolContext, isDisabled = false }: LabelsShortcutsProps) => {
const { selectedTask, tasks, setDefaultLabel } = useTask();
const { projectId } = useProjectIdentifier();
const { hotkeys } = useAnnotatorHotkeys();
const {
isDrawing,
annotations: annotationsScene,
Expand Down Expand Up @@ -111,6 +115,21 @@ export const LabelsShortcuts = ({ labels, annotationToolContext, isDisabled = fa
return isDrawingSmartTool ? smartToolHandler(label) : clickHandler(label);
};

const emptyLabel = labels.find((label) => isEmptyLabel(label));

useHotkeys(
hotkeys['empty-label'],
(event) => {
event.preventDefault();

if (emptyLabel && selectedAnnotations.length > 0) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is the check being made on the selected annotations? For detection and segmentation the empty label can also be when the user has no annotations selected - for local annotation projects we always replace the whole scene with a single "full image" annotation that only has the empty label.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

True! I wasnt 100% sure. Updated :)

handleSelectLabelShortcut(emptyLabel);
}
},
HOTKEY_OPTIONS,
[emptyLabel, selectedAnnotations, handleSelectLabelShortcut, hotkeys]
);

return (
<div aria-label='Label shortcuts' role='list'>
{!isDisabled && (
Expand Down
1 change: 1 addition & 0 deletions .../annotator/components/primary-toolbar/hot-keys-button/use-availability-of-hotkeys.hook.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ export const useAvailabilityOfHotkeys = (domains: DOMAIN[]): Record<HotKeyAction
close: true,
hideAllAnnotations: true,
accept: true,
['empty-label']: true,
[ToolType.EditTool]: false, // we do not have hotkey for edit tool
[ToolType.SelectTool]: true,
[SelectingToolType.SelectionTool]: false, // we do not have hotkey for selection tool
Expand Down
4 changes: 3 additions & 1 deletion web_ui/src/pages/annotator/providers/annotator-provider/utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ export type HotKeyActions =
| 'pasteAnnotation'
| 'close'
| 'accept'
| 'hideAllAnnotations';
| 'hideAllAnnotations'
| 'empty-label';

export type Hotkeys = Record<HotKeyActions, string>;

Expand Down Expand Up @@ -76,6 +77,7 @@ export const DefaultHotkeys = (
close: 'escape',
hideAllAnnotations: 'a',
accept: 'enter',
['empty-label']: `n`,
[ToolType.SelectTool]: 'v',
[ToolType.SSIMTool]: 'd',
[ToolType.RITMTool]: 'i',
Expand Down
Loading