-
Notifications
You must be signed in to change notification settings - Fork 188
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Add mention of molecules * Remove n_max This attribute should be removed because: - It is potentially confusing - It cannot be set in SF or the apps layer (something to reconsider?), we were able to set by directly using thewalrus - It anyway defaults to 30, which is rather large - The samples we generated all had a cutoff of 20. This can be inferred by just looking at the data - The new vibronic data samples just use the default of 30 * Add new GraphDataset ABC * Add data for formic * Update file * Provide dataset * Update data * Fix data * Update dataset * Add temperature to attribute * Add new samples and data for formic acid * Add r before docstring * Change d to delta * Apply suggestions from code review Co-Authored-By: ixfoduap <40441298+ixfoduap@users.noreply.github.com> * Correct reference * Add code review correction
- Loading branch information
Showing
9 changed files
with
176 additions
and
47 deletions.
There are no files selected for viewing
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Oops, something went wrong.