Skip to content

Conversation

@Palomena105
Copy link

Description

The PolyGNN implementation for OpenHGNN,updated by ZhangKaiyi.

Changes

Modify openhgnn/config.ini
Modify openhgnn/config.py
Modify openhgnn/dataset/init.py
Modify openhgnn/models/init.py
Modify openhgnn/tasks/init.py
Modify openhgnn/trainerflow/init.py

Add openhgnn/dataset/PolyGNNDataset.py
Add openhgnn/models/PolyGNN.py
Add openhgnn/tasks/PolyGNN.py
Add openhgnn/trainerflow/PolyGNN.py

Add testPolyGNN.py # example of running

	modified:   openhgnn/config.py
	new file:   openhgnn/dataset/PolyGNNDataset.py
	modified:   openhgnn/dataset/__init__.py
	new file:   openhgnn/models/PolyGNN.py
	modified:   openhgnn/models/__init__.py
	new file:   openhgnn/tasks/PolyGNN.py
	modified:   openhgnn/tasks/__init__.py
	new file:   openhgnn/trainerflow/PolyGNN.py
	modified:   openhgnn/trainerflow/__init__.py
	modified:   openhgnn/config.py
	modified:   openhgnn/dataset/__init__.py
	new file:   testPolyGNN.py
	modified:   openhgnn/dataset/PolyGNNDataset.py
	modified:   openhgnn/dataset/__init__.py
	modified:   openhgnn/models/PolyGNN.py
	modified:   openhgnn/trainerflow/PolyGNN.py
	modified:   testPolyGNN.py
@CfromBU CfromBU self-requested a review September 15, 2025 10:05
Copy link
Collaborator

Choose a reason for hiding this comment

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

不要删除原来的配置,只增加现有的配置即可

Copy link
Collaborator

Choose a reason for hiding this comment

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

MyDataset是否冗余?

Copy link
Author

Choose a reason for hiding this comment

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

MyDataset用于辅助生成训练集、验证集和测试集,并用于GraphDataLoader中生成三个dataloader

Copy link
Collaborator

Choose a reason for hiding this comment

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

去除不必要的注释

Copy link
Collaborator

Choose a reason for hiding this comment

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

test脚本不需要提交

还原了漏掉的配置
还原了遗漏的配置
还原了原有的配置
还原了原有的配置
删除了不必要的注释,并还原了原有配置
还原了原本的配置
Copy link
Collaborator

Choose a reason for hiding this comment

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

#138
这部分多一个空格,导致缩进问题

删除了多余的空格
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