<div dir="ltr">On Sun, Mar 24, 2013 at 3:17 PM, Paul Mullowney <span dir="ltr"><<a href="mailto:paulm@txcorp.com" target="_blank">paulm@txcorp.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Jed,<br>
<br>
I think I've resolved all of your comments. I have some questions.<br>
<br>
1) Tests for the nightly test harness<br>
<br>
(a) Is there a naming convention for the tests?<br>
I have written two examples named aijcusparse_ex_ilu.c and aijcusparse_ex_icc.c. Is this ok?<br></blockquote><div><br></div><div style>They source code files should be numbered unless we change policy. If we were to change, we would still need some systematic naming so that .gitignore would not be interrupted, perhaps 'ex071-aijcusp-icc.c'. Note that some projects use sequential numbering to indicate "level" and priority of tests (lower numbers run first). For now, just use numbers.</div>
<div style> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
(b) What directory should they live in?<br>
I assume these go in ksp/ksp/examples/tutorials<br></blockquote><div><br></div><div style>If they are well-documented, thus suitable as _tutorials_, they can go in the tutorials directory. If they are _tests_, they should be in examples/tests/.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
(c) How do I ensure that they tests are built and run every night with txpetscgpu package?<br></blockquote><div><br></div><div style>Use TESTEXAMPLES_TXPETSCGPU in conf/test and in the examples makefile. You'll see lots of other packages.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
(d) Where do I put binary matrix files that these tests rely upon?<br></blockquote><div><br></div><div style>Put them somewhere and we'll add them here:</div><div style><br></div><div style><a href="ftp://ftp.mcs.anl.gov/pub/petsc/matrices/">ftp://ftp.mcs.anl.gov/pub/petsc/matrices/</a><br>
</div><div style><br></div><div style>(We could consider nicer ways to maintain these.)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

2) Doing<br>
> git status<br>
shows<br>
    Your branch is ahead of 'origin/pm/aijcusparse-icc' by 2 commits.<br>
<br>
I assume that when I push to the fork, these changes will be uploaded remotely?<br></blockquote><div><br></div><div style>Yes, you can use 'git push --dry-run' to see what would happen if you push. After pushing to your fork, open a pull request.</div>
</div></div></div>