Skip to content
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

DGL Refactor Updates #31

Merged
merged 7 commits into from
Oct 26, 2023

Conversation

aMahanna
Copy link
Member

@aMahanna aMahanna commented Oct 15, 2023

Creating a separate PR to avoid cluttering the (already large) diff of #30.

This PR is a mirror to the primary changes made in arangoml/pyg-adapter#23

  • updates rich to support the batching rework introduced in new: adbdgl refactor #30
  • improving adapter.py readability via comments & private methods

@aMahanna aMahanna merged commit 2178ac0 into feature/adapter-refactor Oct 26, 2023
5 checks passed
@aMahanna aMahanna deleted the feature/adapter-refactor-updates branch October 26, 2023 12:09
@aMahanna aMahanna mentioned this pull request Oct 26, 2023
aMahanna added a commit that referenced this pull request Oct 26, 2023
* initial commit (WIP)

* checkpoint

* Update adapter.py

* checkpoint 2

* cleanup

* checkpoint

* checkpoint

* cleanup: `valid_meta`

* mvp: #29

todo: update jupyter notebook to reflect new functionality

* fix: black

* fix: flake8

* Update setup.py

* temp: try for 3.10

* new: 3.10 support

* cleanup: progress bars

* update: documentation

* Update README.md

* new: adbdgl 3.0.0 notebook

* new: address comments

* revive PR

improvements based on pyg-adapter, general code cleanup

* swap python 3.7 support for 3.11

3.7 has reached eol

* fix: PyG typos

* cleanup: udf behaviour (dgl to arangodb)

* fix: rich progress style

* lock python-arango version

* new: notebook output file

* code cleanup

* fix: explicit_metagraph

that took a while to find...

* cleanup function order

* more cleanup

* address comments

* fix: PyG typos

* Update README.md

* fix: typo

* DGL Refactor Updates (#31)

* initial commit

* bump

* cleanup workflows

* Update build.yml

* Update build.yml

* parameter renaming, new `use_async` param

* update release action

* Update README.md

* Update README.md
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.

2 participants