Skip to content
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

Components #7

Open
2 of 3 tasks
mousyball opened this issue Jan 21, 2021 · 0 comments · Fixed by #9, #11, #15, #16 or #19
Open
2 of 3 tasks

Components #7

mousyball opened this issue Jan 21, 2021 · 0 comments · Fixed by #9, #11, #15, #16 or #19
Assignees
Labels
features New feature or request

Comments

@mousyball
Copy link
Owner

mousyball commented Jan 21, 2021

NOTE

  • Transform

    • Use config to define the status of transform and output keys.
    • Visualization
  • Dataset

    • Handle with Registry. The purpose is to leave flexibility for customized computation in the dataset.
  • Network

    • Distinguish by different tasks. (detection/segmentation/others...)
    • Loss builder
  • Config and Registry

    • Use config and registry for management.

Features

  • Use fvcore for registry and config
  • Network Builder
  • Network
    • INetwork -> BaseNetwork (w/ a module only defines the layers connection) -> Network (registered network)
    • Wrapper class for communication with trainer
    • BaseNetwork includes a network module that only defines the layer connection.
  • Config
    • Trainer and Network
    • YAML with Inheritance
    • NAME is required for components.
  • Transform Visualization ([feat] Basic arch of transform visualization #20, [refactor] transform visualization #21)

Test

  • Hook setting

Docs

  • Design thinking
  • Simple guide for entire customized setting.

TODOs

@mousyball mousyball added the features New feature or request label Jan 21, 2021
@mousyball mousyball self-assigned this Jan 21, 2021
This was linked to pull requests Jan 27, 2021
@mousyball mousyball removed a link to a pull request Jan 28, 2021
@JanLin0817 JanLin0817 linked a pull request Mar 9, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment