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

Tangents on curves #57

Open
tobydriscoll opened this issue Jan 15, 2015 · 1 comment
Open

Tangents on curves #57

tobydriscoll opened this issue Jan 15, 2015 · 1 comment

Comments

@tobydriscoll
Copy link
Member

Tangents can be handy and might be required in some circumstances. But they place a burden on user constructed curves that may have no payoff in every case. We should make them optional, and then check if one exists wherever it is about to be used.

@ehkropf
Copy link
Collaborator

ehkropf commented Jan 19, 2015

Agreed.

I propose the base curve class throw an error with message ident CMT:NotDefined in its tangent definition. Classes that need the tangent from a curve should use the try/catch mechanism and check for this error. This would be an example of using the error message idents discussed in #55.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants