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

LLNL/mfem #21

Open
2 of 4 tasks
vsoch opened this issue Mar 6, 2022 · 4 comments
Open
2 of 4 tasks

LLNL/mfem #21

vsoch opened this issue Mar 6, 2022 · 4 comments

Comments

@vsoch
Copy link
Collaborator

vsoch commented Mar 6, 2022

  • quick example of usage in README
  • issue / PR templates
  • coverage / format
  • developer environments

Nice job!

https://github.com/vsoch/contributor-ci-llnl/blob/main/_cfa/cfa-mfem-mfem.md

@tzanio
Copy link
Member

tzanio commented Apr 26, 2022

I just saw this, thank you.

Here are some comments on the outstanding TODO items. Please let me know what you think.

Todo

  • Clearly states similar projects, competitors, prior art, etc. → This could be contentious, so IMHO it is better handled through external websites like Wikipedia
  • The project has issue templates to help the user submitting a new issue. → Good point, we should add this
  • The project provides a pull request template to guide the developer through steps. → Good point, we should add this
  • A container is provided for quick usage/interaction. → That will be awesome 😄
  • The code is robustly commented. → We have doxygen, but docs can always be improved
  • The documentation states how to run the tests. → We should improve the discussion in CONTRIBUTING.md
  • The repository reports code coverage which is >= 90 % → We have a codecov check, but it is definitely not above 90% 😞
  • The project has a social media handle or account. → We don't have that, though we probably should.

Done

The software can quickly output a version.

MFEM is a library, and its version is available to apps that include it. You can also see it with make info.

Different versions of documentation for various releases are available.

This is now available at https://docs.mfem.org/, see https://mfem.org/download/.

A support contact or email is provided.

This is listed here: https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#contact-information

The documentation states how to lint and check formatting.

This is mentioned here: https://github.com/mfem/mfem/blob/master/CONTRIBUTING.md#developer-guidelines

Tests checks for formatting.

We do that: https://github.com/mfem/mfem/runs/6178104680?check_suite_focus=true#step:4:6

The README.md has one or more examples of usage.

We have a link to https://mfem.org/examples. It is tricky to provide a simple example for MFEM usage, because even the simplest code https://github.com/mfem/mfem/blob/master/examples/ex0.cpp requires some domain knowledge.

vsoch added a commit to vsoch/contributor-ci-llnl that referenced this issue Apr 26, 2022
@vsoch
Copy link
Collaborator Author

vsoch commented Apr 26, 2022

Thank you @tzanio ! I've been quietly doing this (in preparation for some future effort), and I've updated the mfem page to your specifications, and it's rendered here: https://vsoch.github.io/contributor-ci-llnl/cfa/cfa-mfem-mfem/#/cfa/.

mfem has a very good score overall - out of all the repos only spack is a bit higher (e.g., 81% to 89%).

Since it's a development library, what would you have in mind for a container? if it builds with spack, I could probably throw together a quick container base and instructions that would allow for development. Is there a dummy example we could include alongside it for an interested party?

@tzanio
Copy link
Member

tzanio commented Apr 26, 2022

Thank you @tzanio ! I've been quietly doing this (in preparation for some future effort), and I've updated the mfem page to your specifications, and it's rendered here: https://vsoch.github.io/contributor-ci-llnl/cfa/cfa-mfem-mfem/#/cfa/.

mfem has a very good score overall - out of all the repos only spack is a bit higher (e.g., 81% to 89%).

Thanks @vsoch. I'll report here if/when we make further improvements.

@tzanio
Copy link
Member

tzanio commented Apr 26, 2022

Since it's a development library, what would you have in mind for a container? if it builds with spack, I could probably throw together a quick container base and instructions that would allow for development. Is there a dummy example we could include alongside it for an interested party?

The simplest thing will be to have a build (with Spack) including all examples and miniapps (we have many of these). A more ambitious option is to also enable the support for some of our optional TPLs, e.g. cuda, rocm, petsc, sundials, superlu-dist, etc.

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

No branches or pull requests

2 participants