Skip to content
This repository was archived by the owner on Dec 7, 2021. It is now read-only.

feat: Active Learning Updates - #778

Merged
Wallace Breza (wbreza) merged 99 commits into
developfrom
wabrez/active-learning
Apr 24, 2019
Merged

feat: Active Learning Updates#778
Wallace Breza (wbreza) merged 99 commits into
developfrom
wabrez/active-learning

Conversation

@wbreza

@wbreza Wallace Breza (wbreza) commented Apr 19, 2019 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Adds new active learning form
  • Moves active learning settings from project settings to here
  • Refactored and created activeLearningService

@JacopoMangiavacchi @wbreza
Add TensorFlow.js dependencies
0919ce9
@JacopoMangiavacchi @wbreza
Add Toolbar item for AL and load TF.js Model
c102d65
@JacopoMangiavacchi @wbreza
Removed un-needed ToolbarItem subclass
4592e52
@JacopoMangiavacchi @wbreza
Call TF model for prediction and add Regions with Tag name
8a5c832
@PIC123 @wbreza
(WIP) feat: Add auto-labeling and auto-tagging active learning settin…
8faeb6c 
…gs/features (#704)

* Auto-update regions on active learning

* Add active learning settings to project settings
@JacopoMangiavacchi @wbreza
Detect only new regions
04583e0
@JacopoMangiavacchi @wbreza
Remove TF.JS cocoSSD package and move Object Detection code under ele…
1e110f8 
…ctron folder
@JacopoMangiavacchi @wbreza
WIP adding Active Learning provider proxy
32baaef
@JacopoMangiavacchi @wbreza
Test custom Proxy method
0f01dc3
@JacopoMangiavacchi @wbreza
Load TF model in the main electron process
c0ae9d7
@JacopoMangiavacchi @wbreza
WIP: loading model on electron with node.js support and proxy imagedata
87a55b7
@JacopoMangiavacchi @wbreza
Simplify ImageData creation
45d1417
@JacopoMangiavacchi @wbreza
Testing node-canvas
6b64616
@JacopoMangiavacchi @wbreza
Testing pixel-util package
bb0b36a
@JacopoMangiavacchi @wbreza
Converting image to 3D TF Tensor
c1286cb
@JacopoMangiavacchi @wbreza
decode JPG to 3D Tensor
022bf38
@JacopoMangiavacchi @wbreza
Make proxy method async
2d73ee0
@JacopoMangiavacchi @wbreza
Removed passing width/weight to proxy
d9fdd89
@JacopoMangiavacchi @wbreza
Implement TF.io.IOHandler to proxy model loading from filesystem
fe4bd50
@JacopoMangiavacchi @wbreza
Load Model from http url or filesystem electron proxy
da12e44
@JacopoMangiavacchi @wbreza
Dynamic load the model class labels from Json file
39fe192
@JacopoMangiavacchi @wbreza
Load model classes from http url or filesystem proxy
e5b338c
@JacopoMangiavacchi @wbreza
Add toast when loading model
929cff5
@JacopoMangiavacchi @wbreza
Add model path to project settings and config
a853565
@JacopoMangiavacchi @wbreza
Rename and reorder AL project settings
ef81a71
@JacopoMangiavacchi @wbreza
Get Electron installation path
cdf8bf1
@JacopoMangiavacchi @wbreza
Add local pre-trained model and model path type config
002033f
@JacopoMangiavacchi @wbreza
handle file: protocol and distribute model in electron bundle
a9ead2e
@JacopoMangiavacchi @wbreza
Distinguish production vs debug to correctly load default model from …
215559f 
…bundle
@JacopoMangiavacchi @wbreza
Minor fix due to rebase
c511bc6
@wbreza
Wallace Breza (wbreza) changed the base branch from jamangia/rectangle-detection to develop April 19, 2019 23:55
JacopoMangiavacchi
Jacopo Mangiavacchi (JacopoMangiavacchi) approved these changes Apr 23, 2019
View reviewed changes
Comment thread src/services/activeLearningService.ts Outdated
Comment thread src/react/components/pages/editorPage/editorPage.tsx
tbarlow12
Tanner Barlow (tbarlow12) approved these changes Apr 23, 2019
View reviewed changes

@tbarlow12 Tanner Barlow (tbarlow12) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. A few nits and things to update, but looks like a cool feature!

Comment thread src/common/localization/es-cl.ts Outdated
Comment thread src/common/localization/es-cl.ts Outdated
Comment thread src/common/localization/es-cl.ts Outdated
Comment thread src/common/localization/es-cl.ts Outdated
Comment thread src/electron/main.ts Outdated
Comment thread src/providers/activeLearning/objectDetection.ts
Comment thread src/services/activeLearningService.ts Outdated
pjlittle
P.J. Little (pjlittle) reviewed Apr 23, 2019
View reviewed changes
Comment thread src/providers/activeLearning/objectDetection.ts Outdated
pjlittle
P.J. Little (pjlittle) reviewed Apr 23, 2019
View reviewed changes
Comment thread src/providers/activeLearning/objectDetection.ts
pjlittle
P.J. Little (pjlittle) reviewed Apr 23, 2019
View reviewed changes
Comment thread src/providers/activeLearning/objectDetection.ts Outdated
@pjlittle

P.J. Little (pjlittle) commented Apr 23, 2019

Copy link
Copy Markdown
Collaborator

A few long hanging comments -- LGTM!

@wbreza
Wallace Breza (wbreza) merged commit 4431557 into develop Apr 24, 2019
@wbreza
Wallace Breza (wbreza) deleted the wabrez/active-learning branch April 24, 2019 00:46
Wallace Breza (wbreza) added a commit that referenced this pull request Apr 29, 2019
@wbreza
feat: Active Learning Updates (#778)
921dbac 
Adds new active learning form
Moves active learning settings from project settings to here
Refactored and created activeLearningService
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers
@JacopoMangiavacchi Jacopo Mangiavacchi (JacopoMangiavacchi) JacopoMangiavacchi approved these changes
@pjlittle P.J. Little (pjlittle) pjlittle approved these changes
@tbarlow12 Tanner Barlow (tbarlow12) tbarlow12 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@wbreza @pjlittle @JacopoMangiavacchi @tbarlow12 @PIC123