itaps-parallel New version of iMeshP.h committed
Devine, Karen D
kddevin at sandia.gov
Tue Nov 4 14:00:44 CST 2008
I just committed a new version of iMeshP.h to the SVN repository at RPI.
The changes in this version are listed below, along with my "To Do" list.
Changes:
- Reformatted many, many, many comments for Doxygen. (Anyone feel like
proofreading??)
- Changed function names 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.
- Removed redundant argument num_ghost_rules from iMeshP_getGhostEntInfo.
- In iMeshP_pushTags and its siblings, change the tag argument to a
source-tag argument; added a destination tag argument.
====================================================================
Out-standing questions and things to do:
- 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].
- Resolve Jason's recent questions about creating ghost entities.
- Add functions to either pull or accumulate tag data as proposed by Jason.
- Update iMeshP functions to reflect changes in iMesh, if any, to
iMesh_getAllVtxCoords and iMesh_getVtxCoordIndex as outlined by Jason.
Karen
More information about the itaps-parallel
mailing list