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

Simplifying FlightSeparations: removal of some possibilities #87

Closed
jdaw1 opened this issue Mar 21, 2021 · 4 comments
Closed

Simplifying FlightSeparations: removal of some possibilities #87

jdaw1 opened this issue Mar 21, 2021 · 4 comments
Assignees

Comments

@jdaw1
Copy link
Owner

jdaw1 commented Mar 21, 2021

FlightSeparations has possibilities that are ugly and have been superseded.

The basic line construct has straights meeting at sharp turns. These can be softened with the new (March 2021) /Arc control, which replaces the sharp turn with a part-circle.

  • The old (Sep 2010) /Curve inserts a Bézier curve made with the PostScript command curveto, the two middle knot points both being where the straight lines would meet. The end knots points are not necessarily equidistant from this, so the curve can look lopsided. Too often it’s ugly.

  • There’s also a possibly of clinging to the outside of the circle with /Clockwise or /Widdershins (older: Nov 2009). Optically, these more than minimally interfere with the Circlearrays, and so have never looked good. The existence of these necessitated /HorizontalLeftwards, /HorizontalRightwards, /VerticalDown, and /VerticalUp.

Ugly; superseded. Hence the proposal is to remove support for all of:

  • /Curve,
  • /Clockwise,
  • /Widdershins,
  • FlightSeparationArcRadiusControl (which affects only /Clockwise and /Widdershins),
  • /HorizontalLeftwards,
  • /HorizontalRightwards,
  • /VerticalDown, and
  • /VerticalUp.

Objections?

@jdaw1
Copy link
Owner Author

jdaw1 commented Apr 11, 2021

Email from Glenn E., Sat 10 Apr 2021, replying to a question asking whether he objected: “I do not; safe to remove.”

@jdaw1
Copy link
Owner Author

jdaw1 commented Apr 12, 2021

FlightSeparations have been used only rarely. FTR, and as mind food, I find six past uses:

Also, one possible forthcoming use:

@jdaw1
Copy link
Owner Author

jdaw1 commented Apr 18, 2021

Decision: at a convenient moment, I will do this simplification. Doing so will: remove a possible ugliness; make the documentation easier to digest; and make the code itself easier to read and maintain.

@jdaw1 jdaw1 self-assigned this Apr 18, 2021
jdaw1 added a commit that referenced this issue May 28, 2021
Closing issue #87.
jdaw1 added a commit that referenced this issue May 28, 2021
@jdaw1
Copy link
Owner Author

jdaw1 commented May 28, 2021

Done.

@jdaw1 jdaw1 closed this as completed May 28, 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

1 participant