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

diffusionTerm._test() requires PySparse #289

Closed
guyer opened this issue Sep 19, 2014 · 5 comments
Closed

diffusionTerm._test() requires PySparse #289

guyer opened this issue Sep 19, 2014 · 5 comments

Comments

@guyer
Copy link
Member

guyer commented Sep 19, 2014

trunk/fipy/terms/diffusionTerm.py#L76

>>> from fipy.matrices.pysparseMatrix import _PysparseMeshMatrix as SparseMatrix

doesn't work if PySparse isn't installed.

Isn't there a more general way to do this?

Imported from trac ticket #385, created by guyer on 11-15-2011 at 10:06, last modified: 09-30-2013 at 21:44

@wd15
Copy link
Contributor

wd15 commented Sep 19, 2014

Fixed with r4966.

Trac comment by wd15 on 11-18-2011 at 15:09

@guyer
Copy link
Member Author

guyer commented Sep 19, 2014

Reverted r4966 as it conflicts with r4945 about to be merged

Trac comment by guyer on 11-18-2011 at 16:48

@guyer
Copy link
Member Author

guyer commented Sep 19, 2014

Resolved with r4972

Trac comment by guyer on 11-18-2011 at 17:09

@fipymigrate
Copy link

In 23c59ed:

#CommitTicketReference repository="fipy" revision="23c59ed824d39abaa536d46d455060f4f93be25e"
* Fixed issue #289, `diffusionTerm._test()` requires PySparse
 * Now uses default matrix class



git-svn-id: svn+ssh://code.matforge.org/fipy/trunk@4966 d80e17d7-ff13-0410-a124-85740d801063

Trac comment by Daniel Wheeler daniel.wheeler@nist.gov on 06-12-2013 at 14:14

@guyer
Copy link
Member Author

guyer commented Sep 19, 2014

Marking milestone

Trac comment by guyer on 09-30-2013 at 21:44

@guyer guyer closed this as completed Sep 19, 2014
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

3 participants