Skip to content
This repository has been archived by the owner on Feb 21, 2022. It is now read-only.

Dpc refel coord fix #3

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

Conversation

cyruscycheng21
Copy link
Contributor

-The previous coordinate change mapping stretched out the simplex that the nodes reside on, which is undesirable.
-The reference element was passed to DualSet as a simplex, it is now passed as the original hypercube

These bug fixes are introduced in this pull request.

wence- and others added 30 commits October 4, 2018 11:21
Squashes more deprecation warnings.
No derivative tabulations yet.
Banish expensive sympy and use fast numpy.
Performance could be improved further, but it is already much faster
than Lagrange elements.  (Although, Lagrange elements are less
relevant for high-order than Bernstein elements.)
Add Bernstein element

Approved-by: Lawrence Mitchell <wence@gmx.li>
Approved-by: rckirby <robert_kirby@baylor.edu>
Remove reliance on numpy.matrix

Approved-by: Chris Richardson <chris@bpi.cam.ac.uk>
updates for latest pytest

* Updates for pytest version

* Update pipelines

Approved-by: Garth Wells <gnw20@cam.ac.uk>
cyruscycheng21 and others added 20 commits February 8, 2019 12:07
* toriesz:
  Address review comments
  Fix a typo and flake8
  Add some documentation for to_riesz
  make point dict and deriv dict loops look alike in to_riesz
  bug fix in derivatives for to_riesz
  fix typo
  Address dham's concerns on dual_set.py
  remove some code, comments
  pep 8, remove some old code
  Switch to new to_riesz method
  general/abstract to_riesz, lift to dual
  Merged in chris/update-pytest (pull request #53)
This reverts commit 1375329, reversing
changes made to daacf42.

This breaks the Bell element, and also FacetBubble.
Discontinuous Serendipity element for cuboids (DPC)
This was referenced Jun 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants