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

Fix datasets in topomodelx' test workflow + Merge SNN #161

Open
ninamiolane opened this issue Aug 24, 2023 · 1 comment
Open

Fix datasets in topomodelx' test workflow + Merge SNN #161

ninamiolane opened this issue Aug 24, 2023 · 1 comment
Assignees

Comments

@ninamiolane
Copy link
Collaborator

What?

There is an issue upon loading the datasets from TopoNetX when testing TopoModelX through GitHub Actions.
See details:
pyt-team/TopoNetX#195

We need to fix it.

Why?

Because the merge of TopoModelX's PR 98 is blocked by this:
#98

This PR has the implementation of the SNN model.

Where?

On TopoModelX, this might mean modifying the github workflow file for tests, which is:
.github/workflows/test.yml

On TopoNetX, this might mean modifying how the datasets are loaded.

How?

Many tutorials manage to load and compute with datasets from TopoNetX. Get inspiration from this code and fix the dataset that is not working for the SNN PR 98.

@ninamiolane
Copy link
Collaborator Author

@devendragovil any update on this?

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

No branches or pull requests

2 participants