📦 STGraph v1.1.0 #142
Merged
📦 STGraph v1.1.0 #142
Conversation
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
Just added the docs page for Cora Dataset
Added a table summarizing the graph stats. And added single line documentation for private methods.
STGraphDataset is the base class for all other dataset classes. So far created the private method `_has_dataset_cache()`
Added STGraphStaticDataset as an abstract class for static graph datasets. It inherits from STGraphDataset.
Added _download_dataset(), _save_dataset() and _load_dataset() methods which are used to handle cached graph datasets. These 3 functions will be common for all datasets present in STGraph and hence is implemented within the STGraphDataset base class.
Added more private attributes such as train/test masks/splits. Implemented the logic for _process_dataset() method.
Also added information about how the dataset handling is done inside the docstrings of STGraphDataset class
Just started writing pytests for the CoraDataLoader. So far written the code to test the init method.
HungaryCPDataLoader is part of the temporal graph dataset group.
Added the __init__() functions for HungaryCPDataLoader. Also added lags variable initialization inside STGraphTemporalDataset
Added descriptive docstrings for HungaryCPDataLoader. Made couple of additions to the docstrings of CoraDataLoader. Made assert checks for the parameters passed to the above two dataloaders. Also made an issue related to HungaryCPDataLoader #86 reagarding the node features array.
Initialised the EnglandCovidDataLoader class
Added the various dataset pre-processing methods for EnglandCovidDataLoader.
Added descriptive documentation for EnglandCovidDataLoader along with code to calculate and set the graph attributes gdata field. Made better name choices by changing _get_total_timestamps() to _set_total_timestamps() and so on. Added tests for EnglandCovidDataLoader. It is however incomplete. Added few more documentation and necessary name changes for CoraDataLoader and HungaryCPDataLoader.
Added the METRLADataLoader constructor and method to find the total timestamps of the dataset
The METRLADataLoader has been added with necessary documentation
The MontevideoBusDataLoader has been added. Instead of using assert checks, we will be using if conditions and raising Exceptions if the user is passing any incorrect parameters. Also added the logic for _set_edge_weights() along with documentation for the __init__ method
📑 Adding First Tutorial for GCN
🔨 Remove sphinx_copybutton Dependency Error
|
@nithinmanoj10 I took a quick pass and everything seems good. Feel free to merge if you have tested everything. I just had one question, do we need the |
|
@JoelMathewC Thanks for the review and suggestion. Removed the Tested all the changes locally by running the unit and script tests. Looks like everything is going through properly. Even built the latest docs and everything seems in order. Have published the stgraph v1.1.0 package in Test-PyPI refer here Will test install from Test PyPI locally and run couple of scripts to see if everything is working as required. Once that is complete will publish it officially in PyPI as well |
📝 Quick Changelog Doc Fixes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This is the v1.1.0 version of STGraph that @nithinmanoj10 has been working on. The changelog is present at #143