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

Crashes consistently on Sankey diagrams #8

Open
graphadvantage opened this issue Feb 20, 2021 · 6 comments
Open

Crashes consistently on Sankey diagrams #8

graphadvantage opened this issue Feb 20, 2021 · 6 comments

Comments

@graphadvantage
Copy link

Seems very unstable in the upgraded desktop and using Neo4j 4.2+. Many of the chart types don't work or cant render labels properly.

@adam-cowley
Copy link
Collaborator

Do you have any sample queries that you can share which are crashing the app?

@graphadvantage
Copy link
Author

graphadvantage commented Feb 21, 2021

Using MacOS Catalina 10.15.7
Neo4j 4.2.3, Desktop 1.4.1
Updated Chart App 1.0.7, all permissions granted

encounter problems right away
having created a test dashboard, clicking on Help crashes the app. Restarting the app does not recover the dashboard, and it has to be recreated.

For Sankey, and Vertical Sankey

MATCH (actor:Person)-[:ACTED_IN]-(m:Movie)-[:DIRECTED]-(director:Person)
WITH actor.name AS from, director.name AS to, COUNT(DISTINCT m) AS value
RETURN from,to,value ORDER BY value DESC

image

App crashes and dashboard is lost

Chord does work, but seems to truncate the data

image

image

The bar charts seem to have problems rending axis labels properly:

MATCH (n)
RETURN 1 AS index, labels(n)[0] AS key, COUNT(n) AS value

image

image

image

image

Line Charts not labeled correctly, and disappear when expanded, but the hover tips still render, some kind of canvas issue?

MATCH (n:Movie)
RETURN (n.released) AS key, COUNT(n) AS value ORDER by key

image

Radar charts don't seem to work right, crashes the app

MATCH (n:Movie {title:"The Matrix"})<-[:ACTED_IN]-(actor)
RETURN actor.name AS key, n.released-actor.born AS value ORDER by key

There also seems to be some issue with saving data for the dashboards in general.
I have yet to be able to load a saved dashboard, maybe there's a files permissions issue.

@adam-cowley
Copy link
Collaborator

Perfect, thanks. When the app crashes can you open up the Developer > Developer Tools and check for errors?

@adam-cowley
Copy link
Collaborator

adam-cowley commented Feb 22, 2021

Sankey

It looks like the Sankey charts don't like circular references. The error is thrown by the underlying library so it can't be caught and turns the screen white which is frustrating. If I use this query instead it works:

MATCH (actor:Person)-[:ACTED_IN]-(m:Movie)-[:DIRECTED]-(director:Person)
WHERE actor <> director
WITH actor.name AS from, director.name AS to, COUNT(DISTINCT m) AS value
RETURN from, to, value ORDER BY value DESC

I've added a check to remove circular references.

Help Page

Same problem with the Help page - for some reason the library supplies some invalid styles to a Transition component which throws an error - I've disabled animation which will hopefully solve the issue.

Bar/Line labels

This seems to work fine for me here so there must be some rendering issue somewhere. I'll see if I can replicate it with another browser.

Issues Saving & Opening

Are you opening the app in Neo4j Desktop?

If so, when the app opens it will look for graphpanel-dashboards.json and graphpanel-queries.json file in the current active project (the project with the current active database). Every time you change something, this file should be updated. If you switch active databases then the file may be saved to another project. When the app loads, it checks for files with these names in the active project and loads them into memory. This is done by a GraphQL query - so there may be an error message in the Network tab of Developer Tools.

If you're not using Neo4j Desktop then the information is saved to local storage in the browser.

Feel free to drop me an email on adam at neo4j.com or send me a message on the Slack channel if you want to into more depth on this

@graphadvantage
Copy link
Author

Hi Adam - Thanks for taking a look. I updated using your latest build - still seeing issues. I am launching through the desktop. Help is crashing the app, and updating the Sankey queries to remove the circular references didn't fix the crash issue. I don't see anything coming through in the logs that was informative. I am not seeing the .json files being created in /project folders.

@florianschummer
Copy link

got the same issues as the original author regarding help and missing .json files.

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

3 participants