Add FENICS_EPS

Issue #781 resolved
Anders Logg (Chalmers) created an issue

We have DOLFIN_EPS. In line with from fenics import *, we should add an alias FENICS_EPS.

Comments (9)

  1. Anders Logg (Chalmers) reporter

    It's an alias in the same way as we have from fenics/dolfin import *. I don't foresee adding more of these (possibly fenics_parameters.xml). Most of our users are not aware of DOLFIN and other components, they only know FEniCS. Plus it's a tiny amount of code duplication of code that has remained static for years.

  2. Prof Garth Wells

    I don't like the fenics alias either. If a user doesn't know what they're using they should find out. Having two names for the same package is just confusing.

  3. Anders Logg (Chalmers) reporter

    I'm of the opposite opinion. We've emphasized the FEniCS branding for many years. 99% (or some other large fraction) of our users can't name and don't care about any of the components that make up FEniCS, and they shouldn't have to.

  4. Jan Blechta

    Demos have traditional from dolfin import *. Others teach new users to from fenics import *. This is exactly what creates confusion. Either the library is DOLFIN or FEniCS. We should pick one and follow it consistently.

  5. Anders Logg (Chalmers) reporter

    I've been teaching numerous courses and tutorials that use FEniCS and I can tell you what is most confusing is to ask users to import dolfin. No one knows what dolfin is. Having fenics as an alias for most users to use is not confusing. We also import ufl.* into the fenics/dolfin namespace. We could be very strict and silly and import neither but it would make a crappy user experience.

  6. Anders Logg (Chalmers) reporter

    The really logical thing would be to not import ufl.* into dolfin but instead create a new module named fenics which imports ufl.*, dolfin.* and, possibly, mshr.*. What we have now essentially does this without the need for creating a separate top-level repository for the fenics module.

  7. Log in to comment