Skip to content
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

Missing/incomplete Documentation #174

Closed
lewisfish opened this issue Mar 22, 2024 · 1 comment
Closed

Missing/incomplete Documentation #174

lewisfish opened this issue Mar 22, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@lewisfish
Copy link

Hi MCDC team,

Some of my concerns on the documentation of the project.

Major points:

  • The API is not fully documented. For example cell, universe, lattice, tally, eigenmode, setting, iQMC, are among some of the functions not documented. Please add doc strings for all user-space functions detailed in the docs. I would also highly suggest adding doc strings to internal functions as well.

Minor points:

  • Following through the user guide, I've noticed that the reference solution isn't explictly given in the docs. It might be a good idea to add either an explicit link to this or as you've done already for the rest of the example, add the full code from the reference solution.
  • It might be a good idea to match the MPI runnning instructions given in the readme with those in the docs. For example, adding the bit about mpiexec/mpirun and how to control the number of cores used.
  • It might be due to me not being an expert in neutron transport, but the examples are a bit opaque to me. It would be helpful if there was a blurb or readme attached to each example explaning what experiment/setup is being simulated or even what parts of MCDC are being shown off. It might also be good to give the user a hint on whether to use Numba and/or MPI for each example as not to waste their time
  • On line 303 of user.rst there is a typo "statical" should be statistical.

openjournals/joss-reviews#6415

@jpmorgan98 jpmorgan98 added the documentation Improvements or additions to documentation label Mar 23, 2024
@jpmorgan98 jpmorgan98 mentioned this issue Mar 23, 2024
4 tasks
@jpmorgan98
Copy link
Collaborator

Major Points

  • Agreed, the fix to this is in the draft PR, waiting on some of my collages to add descriptions for functions I am not sure about

Minor points

  • Link added
  • Instructions changed to match
  • Many of these examples are provided to show a demonstration of an input deck for a given system. We modeled our input system to be similar to other production codes (MCNP and OpenMC) so we expect it to make more sense to folks who are familiar with those systems. Sometime soon we plan on expanding our documentation to include descriptions that you recommended. I have added some notes in some of the more complex input decks for recommended particle counts. This is also the reason for low particle counts in highly complex problems as you found in Broken examples #173
  • Fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants