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

Error if there is only one small cluster #36

Closed
canbax opened this issue Aug 4, 2021 · 5 comments
Closed

Error if there is only one small cluster #36

canbax opened this issue Aug 4, 2021 · 5 comments
Assignees

Comments

@canbax
Copy link
Member

canbax commented Aug 4, 2021

I'm using the below layout options.

{
    "name": "cise",
    "clusters": [
        [
            "n2",
            "n1045152",
            "n1076797",
            "n1045573",
            "n1045238"
        ]
    ],
    "randomize": false,
    "animate": "end",
    "refresh": 10,
    "animationDuration": 500,
    "fit": true,
    "padding": 30,
    "nodeSeparation": 10,
    "idealInterClusterEdgeLengthCoefficient": 1.4,
    "allowNodesInsideCircle": true,
    "maxRatioOfNodesInsideCircle": 0.1,
    "springCoeff": 0.45,
    "nodeRepulsion": 4500,
    "gravity": 0.25,
    "gravityRange": 3.8,
    "packComponents": true
}

with the graph attached as a JSON file
small-graph.txt

I get the error below.

core.js:6210 ERROR TypeError: Cannot read property 'getInnerNodePushCount' of undefined
at CiSELayout.clusterEnlargementCheck (cytoscape-cise.js:2053)
at CiSELayout.runSpringEmbedderTick (cytoscape-cise.js:2031)
at Layout.tick (cytoscape-cise.js:3735)
at tick (cytoscape-cise.js:4396)
at multitick (cytoscape-cise.js:4420)
at Layout.run4state (cytoscape-cise.js:4211)
at Layout.run (cytoscape-cise.js:4229)

Please also note that I'm calling eles.layout on all the existing elements.

@smarek
Copy link

smarek commented Aug 6, 2021

Do you use master or devel branch? I've just hit this with allowNodesInsideCircle: true, on devel, not happening with this setting set to false

@hasanbalci
Copy link
Contributor

We make the development in develop branch and from time to time we merge it to master branch and release the extension. This issue is currently valid for develop branch.

@smarek
Copy link

smarek commented Aug 6, 2021

@hasanbalci thank you for confirmation, i've just been trying to test your solution in #28 and hit this, so I wanted to point out the possible scope of this issue.

herencalik added a commit that referenced this issue Aug 17, 2021
@ugurdogrusoz
Copy link
Contributor

Seems fixed. @canbax please verify and close.

@ugurdogrusoz ugurdogrusoz assigned canbax and unassigned ugurdogrusoz Aug 17, 2021
@canbax
Copy link
Member Author

canbax commented Aug 17, 2021

Thank you. I no longer see this error.

@canbax canbax closed this as completed Aug 17, 2021
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

5 participants