itaps-parallel Questions
Devine, Karen D
kddevin at sandia.gov
Tue Nov 4 12:44:19 CST 2008
I am updating the iMeshP.h file to Doxygenize (Doxygenate? Dox?) the
function prototypes. I found a number of questions, which are listed below.
I also made some small changes that I thought were harmless; let me know if
they cause you trouble. These changes are still in my local copy, as I
didn't want to disrupt the examples development prematurely.
Also, I am not finished with the doxygen work; I will likely have more
questions later this week.
Please reply soon with your comments; otherwise, I'll make a decision on the
questions that need to be resolved before SC.
Questions:
- In iMeshP_exchEntArrToPartsAll, what are implementations using for the
command_code? Should I make this an enumerated type? Or just "yes/no" for
migration? (Yes == migrate entity, no == create only a copy).
- In iMeshP_exchEntArrToPartsAll, how are implementations interpreting the
update_ghost argument?
- In iMeshP_exchEntArrToPartsAll, what adjacencies are being migrated with
the entities in the arguments? Downward adjacencies only? Is the ownership
of the downward adjacencies automatically changed to something that "makes
sense"? (I think Ting raised this issue before but I don't recall that we
resolved her questions.)
- How is the offset array in iMeshP_replaceOnPartBdry used? I guessed the
following; Onkar, let me know if it is correct. If it is correct, do we
need offset_size?
offset (In) Index into new_entities; old_entities[i]
is replaced by new_entities[offset[i]] to
new_entities[offset[i+1]-1].
Changes:
- Function name changes for consistency with our conventions:
iMeshP_Wait -> iMeshP_waitForRequest
iMeshP_WaitAll -> iMeshP_waitForAllRequests
iMeshP_WaitAny -> iMeshP_waitForAnyRequest
iMeshP_WaitEnt -> iMeshP_waitForRequestEnt
iMeshP_Test -> iMeshP_testRequest
iMeshP_load -> iMeshP_loadAll
iMeshP_save -> iMeshP_saveAll
iMeshP_createGhostEnts -> iMeshP_createGhostEntsAll
iMeshP_deleteGhostEnts -> iMeshP_deleteGhostEntsAll
- Removed iMeshP_Status from the interface; it was defined only as a
handle, and we didn't give the apps a way to access any of its data.
- Removed some "const" keywords where I thought they didn't make sense.
Karen
More information about the itaps-parallel
mailing list