-
Notifications
You must be signed in to change notification settings - Fork 45
Create initial TriBITS tutorial #190
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
Comments
Need to decide on a few things:
|
Create branch
Then just make all of your changes on your local
And, please put |
With the old setup, it was ignoring the setting of ${PROJECT_NAME}_ENABLE_Fortran_DEFAULT to OFF and was trying to find Fortran anyway. We found this while working on #190. I added an automated test to make sure that TribitsHelloWorld (which sets the default to OFF) is not trying to find Fortran. This is a break in backward compatibility but about the only project affected will be Trilinos. When I snapshot TriBITS into Trilinos, I will add back the logic to the Trilinos/CMakeLists.txt file to disable Fortran on Windows (without Cygwin). Build/Test Cases Summary Enabled Packages: Enabled all Packages 0) MPI_DEBUG => passed: passed=239,notpassed=0 (0.41 min) 1) SERIAL_RELEASE => passed: passed=239,notpassed=0 (0.36 min)
This maintains backward compatibility for Trilinos given the TriBITS refactoring that removed this logic. Projects should provide their own logic for if and when they want to try to default enable Fortran or not. Trilinos is a bit unique in this reguard. Other TriBITS projects like CASL VERA, COBRA-TF, MPACT, SCALE, and TribitsHelloWorld, have a more straightforward default in all cases. Also see trilinos#1338.
@fryeguy52, note that I just pushed the commit 959cb9e that should fix the behavior of |
@fryeguy52, I rebased your branch
I updated the script |
Now that the outputs are ignored, it is okay to build these docs in the base directory. (And that is standard latex practice.)
…#190) I cleaned up TribitsHelloWorld/CMakeLists.txt to make it shorter and compare better against RawHelloWorld/CMakeLists.txt. I removed a lot of the comments. These comments are better made in the more general documentation. Hopefully this makes it easier to create simple starter TriBITS projects by. not having so specify MPI as a TPL or hack setting TPL_ENABLE_MPI off. I updated the test of TribitsHelloWorld to not set TPL_ENABLE_MPI and updated TriBITS to set it to FALSE when it is not set already.
This is just an update to the tutorial it is not finished but I want toback up what I have tot he remote
Now that the outputs are ignored, it is okay to build these docs in the base directory. (And that is standard latex practice.)
This is just an update to the tutorial it is not finished but I want toback up what I have tot he remote
* TribitsTutorialHelloWorld.rst has a nearly complete first draft of a basic tutorial for TriBITS * HelloWorldConfigure.output is a file I want to reference in the rst * generate-tutorial.sh is a copy of the generate-*.py for the developer's guide and has not been modified at all
@fryeguy52, I just merged PR #267. Let's try to find someone willing to view this material. Otherwise, I will take a look a little later. |
Since this tutorial is in the tribits/doc/ directory, calling the 'tribits/doc/tribits_tutorial' is redundant. Therefore, I changed the name to 'tribits/doc/tutorial'. The driver script was trying to copy files to cupojoe.srn.sandia.gov. Also, it was removing output files automatically. How can you debug problems if you remove the output files? Added README.md file to state that this is still in develpment and may not build (I can't build it on my Linux machine). This will need some refactoring to make this cleaner, more robust, and easier to work with.
Description:
This story will track the creation of a new TriBITS tutorial.
ToDo: Fill in more scope ...
@fryeguy52 will primarily work on this as part of learning TriBITS.
The text was updated successfully, but these errors were encountered: