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

Graph Classifier for ML device selection #197

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

Graph Classifier for ML device selection #197

wants to merge 34 commits into from

Conversation

flowerthrower
Copy link
Collaborator

  • additional circuit features -> graph
  • additional circuit features -> zx claculus graph
  • classifier with different underlying GNN types (Transformer, GCN, GAT)

Patrick Hopf and others added 23 commits January 19, 2024 11:51
👽️ update code to integrate new bench device class
cf4c617
@flowerthrower
Merge branch 'main' into integrate_bench_update
bef9c3e
point dependency to pre-release mqt.bench
f80e490
working gnn with molecule fake data
f10565e
circuit to graph
931c3f5
generate graph training data
b219cdb
working single graph classifier
c10fe73
training notebook with old data
3857259
working with precompiled circuits
4e2d5c2
@flowerthrower
Merge branch 'cda-tum:main' into graph_classifier
0454427
dropout and normalization
8a997cb
restructure and variable activation function
868458e
add variable readout
6082a84
add readout method to classifier
b3b3861
easier modifyable gnn
b49c6ed
first pyzx steps
489674b
@flowerthrower
👽️ update code to integrate new bench device class
787e49b
@flowerthrower
point dependency to pre-release mqt.bench
41f386c
@flowerthrower
imporve coverage
b09f9c5
🐛 fix test
d0f0faa
drop deprecated function
f9e44b8
small adjustements
6c68c90
zx calculus graph
2571dcb
github-advanced-security[bot]
github-advanced-security bot found potential problems Feb 14, 2024
View reviewed changes
src/mqt/predictor/ml/GNNclassifier.py Fixed Show fixed Hide fixed
src/mqt/predictor/ml/GNN.py
]
last_hidden_dim = self.output_dim

self.out_nn = nn.Sequential(nn.Linear(last_hidden_dim, self.output_dim), self.activation_func)

Check failure

Code scanning / CodeQL

Potentially uninitialized local variable Error

Local variable 'last_hidden_dim' may be used before it is initialized.
github-advanced-security[bot]
github-advanced-security bot found potential problems Feb 14, 2024
View reviewed changes
src/mqt/predictor/ml/GNNclassifier.py
}

# Update defaults with provided kwargs
self.set_params(**kwargs)

Check warning

Code scanning / CodeQL

`__init__` method calls overridden method Warning

Call to self.
set_params
Loading
in __init__ method, which is overridden by
method MultiGNNClassifier.set_params
Loading
.
src/mqt/predictor/ml/GNN_zx.py Fixed Show fixed Hide fixed
@flowerthrower flowerthrower marked this pull request as draft June 12, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@flowerthrower