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

Quick work-around; unused max cells parameter #217

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Commits on Jul 22, 2016

  1. The 'max number of cells' parameter in the refinement section does no…

    …t work for my simple code that uses the poisson problem interface. I have not found the root cause. This commit is a quick work-around that instead adds a new parameter max_cells to the pi-DoMUS interface. Such a parameter seems at least as relevant as already existing parameters such as 'kelly threshold' and 'max iterations', so maybe this is actually what we want. This 'work-around' commit violates the concept of 'do not repeat yourself', so I would call this an open issue.
    Alex Zimmerman committed Jul 22, 2016
    Configuration menu
    Copy the full SHA
    cfb6343 View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2016

  1. Moved the return statement up so that the Kellythreshold message does…

    …n't repeat at each time step.
    Alex Zimmerman committed Jul 28, 2016
    Configuration menu
    Copy the full SHA
    296be93 View commit details
    Browse the repository at this point in the history