-
Notifications
You must be signed in to change notification settings - Fork 47
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
Adding documentation and changing website layout #688
Conversation
docs/pages/plugins_docs.md
Outdated
- Sets the configuration for tests | ||
- Logs results | ||
- Loads plugins by called LoadClasses | ||
- Starts execution of tests by called one of the [Executor](https://github.com/open-mpi/mtt/tree/master/pylib/Tools/Executor) plugins. The [combinatorial](https://github.com/open-mpi/mtt/blob/master/pylib/Tools/Executor/combinatorial.py) plugin is currently the default plugin; however, the [sequential](https://github.com/open-mpi/mtt/blob/master/pylib/Tools/Executor/sequential.py) plugin can be set by using the ```--executor=sequential``` flag. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The default executor is sequential:
Lines 273 to 277 in 75cd103
# Cli specified executor takes precedent over INI | |
# If there is nothing defined in either use fallback | |
fallback = "sequential" | |
executor = args.executor or testDef.config.get('MTTDefaults', 'executor', fallback=fallback) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok! Fixed it. Thanks for pointing that out.
e55a3c3
to
c163eaf
Compare
docs/README.md
Outdated
It should be noted that the software has been mostly developed internally | ||
to the Open MPI project and will likely experience some growing pains | ||
while adjusting to a larger community. | ||
MTT is divided into multiple phases of execution to split up grabbing content, building content, running content, and reporting results (please refer to the [INI documentation](/mtt/docs/ini_docs.html) to learn more). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
INI documentation URL should be /mtt/pages/ini_docs.html
Signed-off-by: Akshaya Jagannadharao <[email protected]>
c163eaf
to
2f4842f
Compare
Akshaya, this looks really good. I haven't had much time to look over it, but what I have seen so far is good. I will try to look over/fact check over the weekend, but it is likely good to merge. refs #671 |
Adding documentation and changing website layout
Signed-off-by: Akshaya Jagannadharao [email protected]
Website I'm trying to build should look like this one: https://jagaskak.github.io/mtt/
Lots of changes (I captured the more important ones here):
Things to look for: