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

Updates for ml-uf3 pull request #1

Merged
merged 11 commits into from
Mar 28, 2024
Merged

Updates for ml-uf3 pull request #1

merged 11 commits into from
Mar 28, 2024

Conversation

akohlmey
Copy link

This provides a bunch of mostly "organizational" changes:

  • add ML-UF3 package to CMake presets where applicable
  • update CODEOWNERS file
  • reformatting and spell checking documentation updates
  • move example to examples/PACKAGES/uf3 and add reference logs
  • change potential files in examples to have .uf3 suffix
  • entering the number of atom types is redundant. Atom::ntypes should be used.
  • the special case for pair_coeff wildcards is not needed. The utils::bounds() function handles non-wildcard arguments just fine

@monk-04
Copy link
Member

monk-04 commented Mar 28, 2024

Approving!!

@monk-04 monk-04 merged commit a26c281 into uf3:ml-uf3 Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants