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

Terms should throw an error on bad coeff type #47

Closed
guyer opened this issue Sep 19, 2014 · 1 comment
Closed

Terms should throw an error on bad coeff type #47

guyer opened this issue Sep 19, 2014 · 1 comment

Comments

@guyer
Copy link
Member

@guyer guyer commented Sep 19, 2014

FaceTerm and NthOrderDiffusionTerm objects do not work correctly if their coefficients are not either FaceVariables or simple numbers. CellTerm objects require a coefficient that is either a CellVariable or a simple number. Presently, if a CellVariable is supplied to a FaceTerm or a FaceVariable is supplied to a CellTerm, they fail rather cryptically in solve(). They should throw an error on __init__().

Imported from trac ticket #48, created by jguyer on 06-27-2005 at 17:05, last modified: 09-02-2005 at 10:32

@guyer
Copy link
Member Author

@guyer guyer commented Sep 19, 2014

Logged In: YES user_id=121894 Terms now either coerce coefficients to a desired type or throw an error at instantiation.

Trac comment by jguyer on 09-02-2005 at 10:32

@guyer guyer closed this Sep 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant