cyREST icon indicating copy to clipboard operation
cyREST copied to clipboard

Incomplete results for Get Layout Parameters

Open AlexanderPico opened this issue 5 years ago • 3 comments

Compare the results from CyREST:

http://localhost:1234/v1/apply/layouts/force-directed/parameters

and typing into Automation Panel:

help layout force-directed

The CyREST set is missing many paramaters

AlexanderPico avatar Jun 01 '19 22:06 AlexanderPico

This also appears to affect the PUT commad to set layout parameters, only accepting the subset that it recognizes and returning an error for the other legitimate parameters.

AlexanderPico avatar Jun 01 '19 22:06 AlexanderPico

This is definitely an issue. Layouts use some pretty complex Tunable gymnastics, which are hidden behind very specialized interpreters for the swing dialog and the command line interface. Here's what I've come up with so far:

The current implementation ignores @ContainsTunables annotations, which results in being unable to set nested tunable sets.

If we get beyond this, there's ALSO some special case tunable handling behind nodeList, edgeAttribute and nodeAttribute, which accounts for other missing parameters.

I am part way to getting the nested tunables sorted out, but the special cases may need to interface a little differently.

dotasek avatar Sep 25 '19 22:09 dotasek

Any updates on this issue?

AlexanderPico avatar Apr 13 '21 01:04 AlexanderPico