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.

Sign up for GitHub

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

Jump to bottom

Concave Areas not calculated properly in LOM #5

Closed
rowanc1 opened this issue Nov 26, 2013 · 3 comments
Closed

Concave Areas not calculated properly in LOM #5

rowanc1 opened this issue Nov 26, 2013 · 3 comments
Labels
mesh
Milestone
Discretize Package

Comments

@rowanc1
Copy link
Member

rowanc1 commented Nov 26, 2013

area calculation (i.e. 'volume' in 2D) is not correct if the cell is concave.

Currently averaging the four parallelograms, and I think this works out to give the area of the convex hull of the cell.

Need to write test cases and explore this issue.
rowanc1 pushed a commit that referenced this issue Nov 4, 2015
Merge pull request #5 from simpeg/MapUpdates
f4e210a 
Map updates (there are still serious(?) problems with TDEM), but not with these updates.
@lheagy lheagy added the mesh label Mar 2, 2016
rowanc1 added a commit that referenced this issue Jun 22, 2016
@rowanc1
Merge pull request #5 from simpeg/MapChanges
640457c 
Updates
@lheagy
Copy link
Member

lheagy commented Sep 21, 2016

@rowanc1 : is this still an issue?

@rowanc1
Copy link
Member Author

rowanc1 commented Sep 21, 2016

I believe so yes.

@rowanc1 rowanc1 modified the milestone: Discretize Package Jan 6, 2017
@rowanc1 rowanc1 mentioned this issue Jan 17, 2017
Open
@rowanc1
Copy link
Member Author

rowanc1 commented Jan 17, 2017

This issue was moved to simpeg/discretize#17

@rowanc1 rowanc1 closed this as completed Jan 17, 2017
jcapriot pushed a commit that referenced this issue May 17, 2021
@xiaolongw1223
Merge pull request #5 from simpeg/master
33e29dd 
Merge from original repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
mesh
Projects
None yet
Milestone
Discretize Package
Development

No branches or pull requests
2 participants
@rowanc1 @lheagy