cycle diagram #1

[Mayank77maruti]

Progress Till now (Purpose achieved after almost 60 tries and bunch of diff approaches)

closes #1578
/claim #1578

[Mayank77maruti]

@alixander PTAL
Thanks

[alixander]

No the path should be perfectly circular like in my PR (#2362).

But the paths should start/stop at the shape bounds

[Mayank77maruti]

[Mayank77maruti]

Hey @alixander PTAL
Thanks

[Mayank77maruti]

@alixander Friendly ping
Thank You

[alixander]

@Mayank77maruti

1. Do not ping more than once in 72 hours. Doing it one hour apart is just impatient.
2. Does the test result look adequate to you? The lines are clearly disconnected from the shapes.

Please maintain a high standard for your submissions. I appreciate your attempts but will be blocking future submissions if the standard is not raised.

[Mayank77maruti]

I am so sorry.
I will make sure I will not ping and will make sure to maintain high standards.
Again , I am sorry.

[Mayank77maruti]

Todays Progress :-

The arrow head orientation is getting a little tricky trying to figure out how to do it.
Thank YOU

[alixander]

Nice that looks close.

The arrow head orientation is getting a little tricky trying to figure out how to do it.

I think the arrowhead just points in the direction of the last 2 points. So it's likely the last 2 points are really close to each other and not going in the direction of the arc

[Mayank77maruti]

Hey @alixander Please do have a look. I think the purpose have been achieved.

Sorry for pining too early. If i have to improve any thing please do let me know.
Thank YOU.

[alixander]

Will take a closer look at the code later but the output looks right and is promising. 👍

[alixander]

When I add more cases, it starts to fail strangely:

The solution does not seem robust. Please update the test to be and make sure it's generalizable to other cases.

1: "" {
  shape: cycle
  a -> b -> c -> d
}
2: "" {
  shape: cycle
  a -> b -> c
}
3: "" {
  shape: cycle
  a -> b
}
4: "" {
  shape: cycle
  a -> b -> c -> d -> e -> f
}
5: "" {
  shape: cycle
  a -> b -> c -> d -> e
}