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.

Sign up for GitHub

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

Jump to bottom

Update GNN example readme #283

Merged
1 commit merged into from
Jul 19, 2022
Merged

Update GNN example readme #283

1 commit merged into from
Jul 19, 2022

Conversation

pdmack
Copy link
Contributor

@pdmack pdmack commented Jul 18, 2022

chardet package now required by stellargraph
Also, switching to mamba for speed and conda seemed to struggle solving env.
Not sure if we should start pinning these?

@pdmack
Update gnn example readme
9be8989 
`chardet` package now required by stellargraph
also, switching to mamba for speed and conda seemed to struggle solving env
@pdmack pdmack added dependencies Pull requests that update a dependency file doc Improvements or additions to documentation 3 - Ready for Review labels Jul 18, 2022
@pdmack pdmack requested a review from a team as a code owner July 18, 2022 16:23
@pdmack pdmack added the non-breaking Non-breaking change label Jul 18, 2022
@pdmack pdmack changed the title Update gnn example readme Update GNN example readme Jul 18, 2022
cwharris
cwharris approved these changes Jul 19, 2022
View reviewed changes
Copy link
Contributor

@cwharris cwharris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cwharris
Copy link
Contributor

@gpucibot merge

@ghost ghost merged commit c3a3ddf into branch-22.08 Jul 19, 2022
@ghost ghost deleted the pdmack-patch-2 branch July 19, 2022 15:44
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cwharris cwharris cwharris approved these changes

Assignees

@cwharris cwharris

Labels
dependencies Pull requests that update a dependency file doc Improvements or additions to documentation non-breaking Non-breaking change
Projects
Morpheus Boards
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pdmack @cwharris