Deprecation: remove AssemblerBase::reset_sparsity prior to 1.5 release.

Issue #285 resolved
Jan Blechta created an issue

AssemblerBase::reset_sparsity is going to be deprecated in 1.4 release and should be removed after 1.4 release. Probably also deprecated_bool class should be removed. Affected files:

grep -r reset_sparsity dolfin/ site-packages/ demo/ test/
grep -r bool_deprecated dolfin/ site-packages/ demo/ test/
dolfin/swig/fem/pre.i
dolfin/swig/fem/post.i
dolfin/fem/AssemblerBase.h
site-packages/dolfin/fem/assembling.py

Comments (11)

  1. Jan Blechta reporter
    • changed status to open
    1. You forgot assembling.py.

    2. What about bool_deprecated in AssemblerBase.h? Shouldn't we remove this also?

  2. Prof Garth Wells

    @blechta I fixed assembler.py in another branch (by accident).

    We need to coordinate the removal of deprecated functions better. I started branches for some functions before the release (so I could merge into development branches that would not be merged into master before 1.4). Cleaning this all up with multiple developers making the same changes will be a pain.

  3. Martin Sandve Alnæs

    Removal of the deprecated arguments other than reset_sparsity in assembler.py needs to be done in concert with a bunch of other issues in the ufl/dolfin intersection that I've been alone at handling. If you only removed reset_sparsity that's an easy merge.

  4. Jan Blechta reporter

    What about removing illogically placed AssemblerBase.h/bool_deprecated class. It can be anytime recovered when needed...

  5. Log in to comment