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

feat: Active Learning Updates #778

Merged
merged 99 commits into from
Apr 24, 2019
Merged

feat: Active Learning Updates #778

merged 99 commits into from
Apr 24, 2019

Conversation

wbreza
Copy link
Contributor

@wbreza wbreza commented Apr 19, 2019

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

JacopoMangiavacchi and others added 30 commits April 19, 2019 13:45
…gs/features (#704)

* Auto-update regions on active learning

* Add active learning settings to project settings
@wbreza wbreza changed the base branch from jamangia/rectangle-detection to develop April 19, 2019 23:55
Copy link
Contributor

@tbarlow12 tbarlow12 left a comment

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!

src/common/localization/es-cl.ts Outdated Show resolved Hide resolved
src/common/localization/es-cl.ts Outdated Show resolved Hide resolved
src/common/localization/es-cl.ts Outdated Show resolved Hide resolved
src/common/localization/es-cl.ts Outdated Show resolved Hide resolved
src/electron/main.ts Outdated Show resolved Hide resolved
src/services/activeLearningService.ts Outdated Show resolved Hide resolved
@pjlittle
Copy link
Collaborator

A few long hanging comments -- LGTM!

@wbreza wbreza merged commit 4431557 into develop Apr 24, 2019
@wbreza wbreza deleted the wabrez/active-learning branch April 24, 2019 00:46
wbreza added a commit that referenced this pull request Apr 29, 2019
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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants