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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Examples folder refactor #9346

Open
zhouyu5 opened this issue May 22, 2024 · 1 comment
Open

Examples folder refactor #9346

zhouyu5 opened this issue May 22, 2024 · 1 comment

Comments

@zhouyu5
Copy link
Contributor

zhouyu5 commented May 22, 2024

馃洜 Proposed Refactor

Hi, I am a beginner of PyG, and I get myself familiar with PyG through the documentation, now I want to try to run some examples, I felt a little confused on the current example folder in the repo. The following are my current thoughts, I put it here for your reference, looking forward to hear your feedback.

  1. Is it possible to refactor the examples folder, now there are too many separate python scripts in the first level of the example folder, which is not helpful for beginner to get familiar with it, could you please help to tidy it?
  2. There are some official tutorial examples in documentation website, it looks quite helpful for beginners, could you also create a notebook folder, and put these notebooks into it?
    image

Suggest a potential alternative/fix

  1. Classify the python scripts into several folders according to model or datasets, and provide a better readme file under example folder.
  2. Create a notebook folder, and put the colab notebooks code into it.
@rusty1s
Copy link
Member

rusty1s commented May 27, 2024

Thank you for this issue. We always wanted to clean-up the example folder, sorry that it looks confusing to you. I see what I can do. In the meantime, happy to answer any questions regarding examples.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants