Skip to content

Ripsnet tutorial - #90

Open
VincentRouvreau wants to merge 4 commits into
GUDHI:masterfrom
VincentRouvreau:ripsnet
Open

Ripsnet tutorial#90
VincentRouvreau wants to merge 4 commits into
GUDHI:masterfrom
VincentRouvreau:ripsnet

Conversation

@VincentRouvreau

@VincentRouvreau VincentRouvreau commented Aug 4, 2026

Copy link
Copy Markdown
Contributor
  • Replaces ripsnet tutorial #59
  • Includes ripsnet gudhi-devel#587 (that won't be in main gudhi project)
    • User manual is copied at the beginning of the tutorial ina dedicated section
    • RipsNet code is in tutorials/gudhi_extra directory and can be called by from gudhi_extra.tensorflow import RipsNet
  • Fix PersistenceImage and Landscape that were called with transform, but were requiring a fit_transform call
  • In RipsNet.call method, I replaced inputs = tf.keras.layers.InputLayer(input_shape=(None, self.input_dim), dtype="float32", ragged=True)(pointclouds) (was failing) with inputs = tf.cast(pointclouds, dtype="float32")
  • In RipsNet.__init__ constructor, I uncomment the exception that is thrown when perm_op is unknown (useful from a user point of view)
  • Before constructing the dataset with create_multiple_circles, I added a call to numpy.random.seed(42) for reproductible results (and a comment)

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@MathieuCarriere MathieuCarriere self-assigned this Aug 4, 2026
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