Skip to content

Add normalize parameter to gauxc_basisset_new_from_shells#199

Open
franzpoeschel wants to merge 1 commit intowavefunction91:skalafrom
franzpoeschel:basisset-add-normalize-parameter
Open

Add normalize parameter to gauxc_basisset_new_from_shells#199
franzpoeschel wants to merge 1 commit intowavefunction91:skalafrom
franzpoeschel:basisset-add-normalize-parameter

Conversation

@franzpoeschel
Copy link
Copy Markdown

This is the C function bound to Fortran here. Leaving the normalize param unspecified here seems like undefined behavior.

/**
 * @brief Create a new BasisSet instance from arrays of shells.
 * @param status Status object to capture any errors.
 * @param shells Pointer to an array of GauXCShell.
 * @param nshells Number of shells in the array.
 * @param normalize Whether to normalize the basis functions.
 * @return Handle to the created BasisSet.
 */
extern GauXCBasisSet gauxc_basisset_new_from_shells(
   GauXCStatus* status,
   GauXCShell* shells,
   size_t nshells,
   bool normalize
);

@franzpoeschel franzpoeschel force-pushed the basisset-add-normalize-parameter branch from 6458d3a to c9aab15 Compare April 27, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant