NWChemEx / NWChemEx/SimDE

Document how to add a new ao integral type

Open
#38 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
0
Forks
0
Avg merge
10h 7m
Merged PRs (30d)
4

Description

This is a note that we need a page on how to add a new ao integral property type. Adding one includes:

  • inheriting from NCenter
  • update ao_integrals_fwd.hpp
  • add header to ao_integrals.hpp
  • add type to test_common_TA.hpp

Also should have a section on how to add one templated on the number of centers.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the documentation structure and read ao_integrals_fwd.hpp, ao_integrals.hpp, and test_common_TA.hpp to understand the listed integration points. Add a page covering how to add a new ao integral property type, including the templated number-of-centers case; done means all listed steps are documented clearly.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
documentation
Issue type
Documentation
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.