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

[BUG] ring / radial layouts intermittently do not render as rings #632

Open
DataBoyTX opened this issue Jan 7, 2025 · 1 comment
Open
Assignees
Labels

Comments

@DataBoyTX
Copy link
Contributor

Describe the bug

User reported an issue where the radial layouts sometimes do not render, and it creates more of force-directed layout instead of radial. I was able to reproduce this with our demo notebook after running the notebook several times.

To Reproduce

Run the following pygraphistry demo notebook until the issue appears:
layout_categorical_ring.ipynb

Expected behavior
ring layout should also appear

Actual behavior
looked more like force-directed layout than ring

Screenshots
Screenshot (1323)

Browser environment (please complete the following information):

Chrome is up to date
Version 131.0.6778.205 (Official Build) (64-bit)

Edition	Windows 11 Home
Version	24H2
Installed on	‎11/‎15/‎2024
OS build	26100.2605
Experience	Windows Feature Experience Pack 1000.26100.36.0

Graphistry GPU server environment
AWS AMI - v2.41.17

PyGraphistry API client environment

pyg v0.35.3

@lmeyerov
Copy link
Contributor

lmeyerov commented Jan 7, 2025

I think there are potentially a couple of diff issues here:

  • initial layout when setting play=0 is still via FDL instead of passed in

    • race on loading, e.g., tick-before-play=0? or a dropped message?
    • initial positions look 'close' to passed in, vs a random init seed / FDL one, suggesting the race
  • axis don't render

    • axis are technically an encoding...
    • ... and if encodings are slow to load, we skip and proceed ...
    • .. and even those axis are 'fast' encodings, they may sequenced after slower non-axis ones?

@DataBoyTX DataBoyTX added the p2 label Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants