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

Simplify Fluid Book Unit Tests #9664

Closed
abhinavarora opened this issue Apr 4, 2018 · 1 comment
Closed

Simplify Fluid Book Unit Tests #9664

abhinavarora opened this issue Apr 4, 2018 · 1 comment
Assignees

Comments

@abhinavarora
Copy link
Contributor

abhinavarora commented Apr 4, 2018

A very common issue that users of Fluid find are that our book unit tests (https://github.com/PaddlePaddle/Paddle/tree/develop/python/paddle/fluid/tests/book) are not very simple for new users. Our book unit tests have tests for CPU/GPU, train/inference and single machine/distributed in the same file. This makes the file very lengthy.

Also, some tests have multiple models in the same file such as test_recognize_digits.py. This has MLP and Conv models in the same file and can end up confusing users. Hence we should make sure that we split all the tests into more granular self-contained tests. This will make it easier for new users to understand Fluid.

@abhinavarora abhinavarora created this issue from a note in Complete Fluid (TODO) Apr 4, 2018
@abhinavarora
Copy link
Contributor Author

  • test_fit_a_line
  • test_image_classification
  • test_label_semantic_roles
  • test_machine_translation
  • test_recognize_digits
  • test_recommender_system
  • test_understand_sentiment
  • test_word2vec

@abhinavarora abhinavarora changed the title Simplify Fluid Unit Tests Simplify Fluid Book Unit Tests Apr 5, 2018
Complete Fluid automation moved this from TODO to Done Aug 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

5 participants