-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
CVAT-3D Milestone1 #2534
CVAT-3D Milestone1 #2534
Changes from 5 commits
a40d8dd
6bcac4a
db27800
7f89251
0fd3742
3be8ef9
6357751
6baeed3
d0c9b4f
13cd5ff
5be4b3d
9175c07
a9972e1
0877de4
da61f90
5e302c6
2fd3282
9432068
5675d7c
d14226a
733e4bb
fc5cf64
45d276f
6c0d765
86c7494
7c3017d
768a17e
6879381
f0217ca
2af7627
4b61eb4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,6 +15,7 @@ | |
format: initialData.ext, | ||
version: initialData.version, | ||
enabled: initialData.enabled, | ||
dimension: initialData.dimension | ||
}; | ||
|
||
Object.defineProperties(this, { | ||
|
@@ -58,6 +59,16 @@ | |
*/ | ||
get: () => data.enabled, | ||
}, | ||
dimension: { | ||
/** | ||
* @name dimension | ||
* @type {string} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. probably should be There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Made changes accordingly. |
||
* @memberof module:API.cvat.classes.Loader | ||
* @readonly | ||
* @instance | ||
*/ | ||
get: () => data.dimension, | ||
}, | ||
}); | ||
} | ||
} | ||
|
@@ -74,6 +85,7 @@ | |
format: initialData.ext, | ||
version: initialData.version, | ||
enabled: initialData.enabled, | ||
dimension: initialData.dimension | ||
}; | ||
|
||
Object.defineProperties(this, { | ||
|
@@ -117,6 +129,16 @@ | |
*/ | ||
get: () => data.enabled, | ||
}, | ||
dimension: { | ||
/** | ||
* @name dimension | ||
* @type {string} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. probably should be There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Made changes accordingly. |
||
* @memberof module:API.cvat.classes.Loader | ||
* @readonly | ||
* @instance | ||
*/ | ||
get: () => data.dimension, | ||
}, | ||
}); | ||
} | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -974,6 +974,7 @@ | |
use_zip_chunks: undefined, | ||
use_cache: undefined, | ||
copy_data: undefined, | ||
dimension: undefined | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missed trailing comma There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Made changes accordingly. |
||
}; | ||
|
||
let updatedFields = { | ||
|
@@ -1452,6 +1453,16 @@ | |
updatedFields = fields; | ||
}, | ||
}, | ||
dimension: { | ||
/** | ||
* @name enabled | ||
* @type {string} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. probably should be module:API.cvat.enums.TaskDimension There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Made changes accordingly. |
||
* @memberof module:API.cvat.classes.Loader | ||
* @readonly | ||
* @instance | ||
*/ | ||
get: () => data.dimension, | ||
}, | ||
}), | ||
); | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,15 +19,17 @@ interface Props { | |
menuKey: string; | ||
dumpers: any[]; | ||
dumpActivities: string[] | null; | ||
taskDimension: string; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Enum There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Corresponding above, something like this:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Changed code accordingly. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets' use unified name here DimensionType (like in Python code) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @bsekachev can we use cvat-core/src/enums.js here? (it might be a hard link of 2 separate modules, I believe) |
||
} | ||
|
||
export default function DumpSubmenu(props: Props): JSX.Element { | ||
const { taskMode, menuKey, dumpers, dumpActivities } = props; | ||
const { taskMode, menuKey, dumpers, dumpActivities, taskDimension } = props; | ||
|
||
return ( | ||
<Menu.SubMenu key={menuKey} title='Dump annotations'> | ||
{dumpers | ||
.sort((a: any, b: any) => a.name.localeCompare(b.name)) | ||
.filter((dumper: any): JSX.Element => dumper.dimension === taskDimension) | ||
bsekachev marked this conversation as resolved.
Show resolved
Hide resolved
|
||
.map( | ||
(dumper: any): JSX.Element => { | ||
const pending = (dumpActivities || []).includes(dumper.name); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missed trailing comma
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made changes accordingly.