-
Notifications
You must be signed in to change notification settings - Fork 13
Added and integrated C++ graphium_cpp library, a Python module implem… #510
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
Merged
DomInvivo
merged 54 commits into
datamol-io:graphium_3.0
from
ndickson-nvidia:graphium_cpp
Jul 9, 2024
Merged
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
5ffe261
Added and integrated C++ graphium_cpp library, a Python module implem…
ndickson-nvidia 8286383
Small changes to support not needing label data during data loading
ndickson-nvidia dca9b2b
Removed FakeDataset, FakeDataModule, and SingleTaskDataset. SingleTa…
ndickson-nvidia 8304210
Removed option to featurize using Python, (but didn't delete everythi…
ndickson-nvidia 4ee35d4
Removed newly deprecated options from yaml files
ndickson-nvidia cf23e37
Added support for limiting the number of threads used by prepare_and_…
ndickson-nvidia 5db0e2a
Fixed compiler warning about signed vs. unsigned comparison
ndickson-nvidia c75a452
Fixed Python syntax issues
ndickson-nvidia 4aa1f85
Changed asymmetric inverse normalization type to be implemented using…
ndickson-nvidia c53451a
Fixed compile errors
ndickson-nvidia 268e245
Some simplification in collate.py
ndickson-nvidia e032e8e
Deleting most of the Python featurization code
ndickson-nvidia bdefe89
Implemented conformer generation in get_conformer_features, trying to…
ndickson-nvidia 5298444
Deleted deprecated properties.py
ndickson-nvidia c38aa06
Handle case of no label data in prepare_and_save_data. Also added con…
ndickson-nvidia 86abf21
Changed prepare_data to support having no label data
ndickson-nvidia 80276da
Updated license passed to setup call in setup.py
ndickson-nvidia 9492e62
Changes to get test_dataset.py and test_multitask_datamodule.py passing
ndickson-nvidia d94097c
Removed load_type option from test_training.py, because it's no longe…
ndickson-nvidia 11e6935
Updated comment in setup.py about how to build graphium_cpp package
ndickson-nvidia ff93c2d
Rewrote test_featurizer.py. Fixed bug in mask_nans C++ function, and …
ndickson-nvidia a892068
Removed deprecation warnings and deprecated parameters from datamodul…
ndickson-nvidia 38a5510
Recommended tweaks to extract_labels in multilevel_utils.py
ndickson-nvidia f7771b3
Fixed "else if"->"elif"
ndickson-nvidia 4256839
Rewrote test_pe_nodepair.py to use graphium_cpp
ndickson-nvidia 91c37a3
Rewrote test_pe_rw.py to use graphium_cpp. Comment update in test_pe_…
ndickson-nvidia f347a0d
Rewrote test_pe_spectral.py to use graphium_cpp
ndickson-nvidia 26b5531
Removed tests/test_positional_encodings.py, because it's a duplicate …
ndickson-nvidia 1ded38b
Fixed handling of disconnected components vs. single component for la…
ndickson-nvidia 314d636
Fixed compile warnings in one_hot.cpp
ndickson-nvidia e49b4da
Rewrote test_positional_encoders.py, though it's still failing the te…
ndickson-nvidia f001464
Removed commented out lines from setup.py
ndickson-nvidia 2782fbc
Ran linting on Python files
ndickson-nvidia 77d27b5
Hopefully explicitly installing graphium_cpp fixes the automated test…
ndickson-nvidia cb1df19
Test fix
ndickson-nvidia f3f6a0d
Another test fix
ndickson-nvidia c5c0085
Another test fix
ndickson-nvidia 6dd827f
Make sure RDKit can find Boost headers
ndickson-nvidia 59c84a2
Reimplemented test_pos_transfer_funcs.py to test all supported conver…
ndickson-nvidia 7bc8ade
Linting fixes
ndickson-nvidia 6903243
Fixed collections.abs.Callable to typing.Callable for type hint
ndickson-nvidia 9f38afb
Removed file_opener and its test
ndickson-nvidia 5ab9ca9
Fixed the issue with boolean masking, introduced by `F._canonical_mas…
DomInvivo 9c7504f
Fixed the float vs double issue in laplacian pos encoding
DomInvivo f8358f3
Added comment
DomInvivo 692decc
Fixed the ipu tests by making sure that `IPUStrategy` is not imported…
DomInvivo 8891e66
Update test.yml to only test python 3.10
DomInvivo c2d3c87
Removed positional encodings from the docs
DomInvivo d3d19d7
Merge remote-tracking branch 'origin/dom_unittest' into dom_unittest
DomInvivo 0a1696f
Upgraded python versions in the tests
DomInvivo 50265df
Removed reference to old files now in C++
DomInvivo 58fc2aa
Downgraded python version
DomInvivo 5852467
Fixed other docs broken references
DomInvivo ea9a775
Merge pull request #1 from ndickson-nvidia/dom_unittest
ndickson-nvidia File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.