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

Make TOGL feature complete #21

Open
marcusblake opened this issue Jan 1, 2023 · 10 comments
Open

Make TOGL feature complete #21

marcusblake opened this issue Jan 1, 2023 · 10 comments
Labels
enhancement New feature or request

Comments

@marcusblake
Copy link
Contributor

Currently, the implementation of TOGL ignores the following features:

  • handling higher-order information properly
  • expanding simplicial complexes
  • making use of the dimension of features
@Pseudomanifold Pseudomanifold added the enhancement New feature or request label Jan 1, 2023
@Pseudomanifold
Copy link
Contributor

Some of these are rather tough to handle; this should probably rather become a project on its own. In the original paper, for instance, we are not actually going beyond dimension 1 in terms of features, so I am somewhat loath to mix the re-implementation of TOGL with new feature development. A good starting point would be to see how to capture 1-dimensional information; in my current implementation, I figured that I could get this stuff much more easily by using a lower star filtration.

@uchukwu
Copy link

uchukwu commented Mar 5, 2024

Hi @Pseudomanifold, is this a good place to pick up on where we left off in BorgwardtLab/TOGL#13, or should I open a new ticket?

@Pseudomanifold
Copy link
Contributor

Sure!

@uchukwu
Copy link

uchukwu commented Mar 8, 2024

Thanks! You mentioned that TOGL isn't complete yet during our discussion. What is missing? What features do you need added?

@Pseudomanifold
Copy link
Contributor

Potentially some other aggregation functions would be useful; I also have not tried out how easy it is to include this type of layer into arbitrary GNNs. Except for the aggregation functions, I think this is relatively in sync with the other code, though.

@JerBaf
Copy link

JerBaf commented Jun 7, 2024

Hello! I am also coming from the previous implementation of TOGL. I tried to carefully read the code but I struggle to identify what are the key differences between this implementation and the original one for TOGL. Would it be possible to have a few key points on the major differences between the two code bases?

Best,
Jeremy

@Pseudomanifold
Copy link
Contributor

The main difference is that the current implementation only supports aggregation based on deep sets.

This is what works already:
- simple deep set layer
- simple TOGL implementation with deep set functions
- basic GCN with TOGL

This is what is currently ignored:
- handling higher-order information properly (described in TOGL but not yet implemented)
- expanding simplicial complexes (described in TOGL but not implemented)
- making use of the dimension of features (I'm only using dimension 0 at the moment)

Hope that helps!

@JerBaf
Copy link

JerBaf commented Jun 7, 2024

Okay great thank you very much!

@cypans
Copy link

cypans commented Oct 28, 2024

image
Hello, may I ask if we can backpropagate after continuous homology calculation? Especially on CPU, gudhi does not support Tensors

@cypans
Copy link

cypans commented Oct 28, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants