svalinn / DAGMC

Direct Accelerated Geometry Monte Carlo Toolkit
https://svalinn.github.io/DAGMC
Other
104 stars 65 forks source link

Compare KDE and TrackLength mesh tallies #43

Closed kldunn closed 11 years ago

kldunn commented 11 years ago

Check current implementations of KDE and TrackLength mesh tallies for redundancy and common functionality that can be moved to MeshTally.

kldunn commented 11 years ago

One example of redundancy is the print and write_results methods, since there appears to be no need to have both methods in both KDE and TrackLength mesh tally classes.

kldunn commented 11 years ago

I have added a few common functions and variables to the MeshTally base class that were involved in the setup process for both KDEMeshTally and TrackLengthMeshTally classes:

kldunn commented 11 years ago

I have combined write_results() and print() methods in the KDEMeshTally and MeshTally implementation as a part of cleaning up the code formatting and adding detailed comments stage. There are still two print methods in TrackLengthMeshTally that can probably be simplified/extended to the MeshTally Base class.

kldunn commented 11 years ago

As part of the cleanup stage of KDEMeshTally, I generalized the end_history() and add_score_to_tally() methods. Both of these can probably be moved up to MeshTally as TrackLengthMeshTally also needs to perform these tasks and I do not think that there is a major difference in their implementations.

gonuke commented 11 years ago

See https://mywebspace.wisc.edu/groups/CNERG/Neutronics/WeeklyMeetings/2013.06.03/MeshTallyRefactorUpdate_kldunn06032013.pdf for some discussion

gonuke commented 11 years ago

This has been replaced by #92