-
Notifications
You must be signed in to change notification settings - Fork 13
Conversation
Haven't tested but reading the code it looks good to me. Think you can go ahead and merge. Awesome contribution :) |
ah sorry i should've clicked at squash and merge |
I can't see relationships option when I am in advanced query. Can someone help me with this? |
Hey @krishnapraveen7 , Unfortunately this feature has never been released due to a lack of time to maintain the project. I'll try and create a tarball if you want to give it a try, but I won't be able to create an official release anytime soon (would require more tests). |
Thank you @stellasia. I would really appreciate if I can get a tarball. |
Can't attach the tarball here, format is not supported... I have uploaded here for now: https://drive.google.com/file/d/1GOoY97XfjsrCDKfuEA3t3CZoSRHuvDgW/view?usp=sharing Be aware this is a development version, not well tested, so some bugs and regressions are possible, but the relationship feature is there. |
closes #15
New PR that adds relationships visualization addressing review comments from #93
I'd appreciate a review but if not i'm ready to merge it on my own.
Advanced queries supported:
Added arrowheads to display relationship directions:
The same thing with nodes:
Hiding relationships option for point layer type: