forked from AMReX-Codes/amrex
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Eyoo/multicuts #1
Open
ejyoo921
wants to merge
39
commits into
baperry2:test-pr
Choose a base branch
from
ejyoo921:eyoo/multicuts
base: test-pr
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 35 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
df6b6d0
This branch is for visulization of multicut faces/cells. We mainly ad…
ejyoo921 734f7aa
As a baby step, we now can see how many multicuts detected, along wit…
ejyoo921 fb5c787
All the parts are implemented to write plot files of multicut locatio…
ejyoo921 ac65ec6
Merge branch 'AMReX-Codes:development' into eyoo/multicuts
ejyoo921 70b5111
1) Added parmparse called "plt_multiple_cuts" which is false by defau…
ejyoo921 b35cba5
Edited the parmparse option to turn on/off visualization/printing of…
ejyoo921 5c7561e
Erased everything for plotting; We now generate one text file loc_mul…
ejyoo921 92c65fa
We found a 'right' place to produce the plot file in AMReX_EB2_Level.…
ejyoo921 4b4b890
1) Created a multicut called 'multicut_bx'.
ejyoo921 823e97f
Deleted non-necessary printings.
ejyoo921 4007d81
Deleted total_multicuts value
ejyoo921 8e67650
minor changed after Bruce's comments. Run a tutorial input with a rot…
ejyoo921 1a7edb3
We now create separate plot files containing multi-cut locations from…
ejyoo921 5fe3c8e
cleaned up some useless comments. Now writing (i,j,k) and face boxes …
ejyoo921 2fd9f4d
<Cherry-pick due to detached head>
ejyoo921 11e7ddd
Merge branch 'AMReX-Codes:development' into eyoo/multicuts
ejyoo921 1b1a970
Cleanup more no useful comments && using now std::ofstream for text f…
ejyoo921 cc0af83
Removed #ifndef condition for GPU use since we no longer use amrex::P…
ejyoo921 436bb3f
Return to PrintToFile since ofstream is not compatible with CUDA
ejyoo921 0536861
Forgot to add #ifndef back
ejyoo921 9b63533
added AMREX_USE_GPU back -> check with Kestrel. is this even compilin…
ejyoo921 427d166
I just deleted the print txt file part in AMReX_EB2_3D_C.cpp. We may …
ejyoo921 df35481
2D version is added -> how can we chekc?
ejyoo921 bb08807
Whitespace edit
ejyoo921 3c0b9ce
Fixed 2D case error -
ejyoo921 7db7054
Fixed whitespace things
3aeff0e
RE-generate the MF for face-centered one. called multicut_face. 2D is…
ejyoo921 b8179f1
RE-generate the MF for face-centered one. called multicut_face. 2D is…
ejyoo921 0023869
1) fixed 2D bug
ejyoo921 0f19915
Whitespace sytling
fb47a1a
changed the location to call mcx,y,z arrays -> 2D is checked as well
ejyoo921 a0dc9c6
fixed notations in .H files
ejyoo921 048d390
Merge branch 'AMReX-Codes:development' into eyoo/multicuts
ejyoo921 8b4c582
Edited based on Bruce's comments on Sep 6.
ejyoo921 75b419a
I don't understand this pull -
ejyoo921 766e8a9
changed the way we fill the plot file data: Value of 2 is regular, 0 …
ejyoo921 0ba88b9
Merge branch 'AMReX-Codes:development' into eyoo/multicuts
ejyoo921 2304763
Merge branch 'AMReX-Codes:development' into eyoo/multicuts
ejyoo921 8eb29d8
make chage to cell-centered MF
ejyoo921 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bring this outside of the
if (ncuts > 2)
scope. That way ncuts = 2 will show regular cut cells, ncuts > 2 multicuts, and ncuts = 0 cells that are either completely covered or completely fluid.