Skip to content

Conversation

@ydcjeff
Copy link
Contributor

@ydcjeff ydcjeff commented Mar 24, 2021

This PR

  • added simple README.md and requirements.txt to both gan and image classification templates
  • changed the template files extensions to .pyi to have syntax highlighting in IDE and on GitHub.
  • changed the sidebar.py to _sidebar.py not to confuse with template files.
  • a few naming refactors
  • changed base template to _base to have some generic reusable jinja template for ignite handlers, loggers, distributed training settings, hydra, ... (common things around templates will go into _base)

This PR is more viewable locally as some are isort and black formatted.

@ydcjeff
Copy link
Contributor Author

ydcjeff commented Mar 24, 2021

Tests are passing on colab, merging
Feel free to review.

@ydcjeff ydcjeff merged commit e062047 into master Mar 24, 2021
@ydcjeff ydcjeff deleted the refactor/readme-req branch March 24, 2021 17:16
@@ -0,0 +1,91 @@
# Image Classification Template by Code-Generator

This template is ported from [PyTorch-Ignite MNIST example](https://github.com/pytorch/ignite/blob/master/examples/mnist/mnist.py).
Copy link
Member

Choose a reason for hiding this comment

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

It is not MNIST but CIFAR10 right ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, Thanks for pointing out

@ydcjeff ydcjeff added this to the v0.1.0 milestone Mar 25, 2021
@ydcjeff ydcjeff added the enhancement New feature or request label Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

2 participants