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

Code Cleanup #23

Merged
merged 29 commits into from
Oct 25, 2023
Merged

Code Cleanup #23

merged 29 commits into from
Oct 25, 2023

Conversation

aMahanna
Copy link
Member

@aMahanna aMahanna commented Oct 5, 2023

Warrants a 1.1.2 release

Note: the diff is large due to the output notebook change from 48160f1

Open In Collab

Note: For notebook testing purposes:

  • Use !pip install git+https://github.com/arangoml/pyg-adapter.git@code-cleanup
  • Use !git clone -b code-cleanup --single-branch https://github.com/arangoml/pyg-adapter.git

No functionality changes

What's been done:

  • copy changes from arangoml/dgl-adapter@6b36a50
  • update rich usage to rely on the rich.live module
  • general code cleanup & comments
  • improve method readability
  • pip install fixes for build.yml & release.yml

.github/workflows/build.yml Outdated Show resolved Hide resolved
@aMahanna aMahanna changed the title Minor Code Cleanup Code Cleanup Oct 12, 2023
@aMahanna aMahanna merged commit bf70b9f into master Oct 25, 2023
7 checks passed
@aMahanna aMahanna deleted the code-cleanup branch October 25, 2023 12:52
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