-
Notifications
You must be signed in to change notification settings - Fork 71
Conduit Device Support #1358
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
Draft
JustinPrivitera
wants to merge
165
commits into
develop
Choose a base branch
from
task/JustinPrivitera/05_01_24/device_support
base: develop
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.
Draft
Conduit Device Support #1358
Changes from 132 commits
Commits
Show all changes
165 commits
Select commit
Hold shift + click to select a range
864e52f
I'm really thinking about GPUs this time
JustinPrivitera ab961e3
oops
JustinPrivitera cede461
start
JustinPrivitera b3b5246
I know what I'm doing
JustinPrivitera 8343fdc
adasdas
JustinPrivitera 1950d8a
thanks cyrus
JustinPrivitera 28bed36
trying to get it to build
JustinPrivitera c2c37c5
COMPILE!
JustinPrivitera 921695e
maybe we allow non raja open mp
JustinPrivitera 2a6c51b
these changes are from 2 weeks ago
JustinPrivitera 56e72a7
have more mystery changes
JustinPrivitera c86066a
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera ca41133
myth busted
JustinPrivitera ea27d0d
help what do i do next
JustinPrivitera 08eb646
sigh
JustinPrivitera a0c64f7
new source files
JustinPrivitera a7b2191
exec array src
JustinPrivitera e6d04c0
I did it I made GPUs work
JustinPrivitera df1e13d
It works, it just needs to compile
JustinPrivitera d8fd62c
correction to sync and replace
JustinPrivitera 02d74a1
im really dong it
JustinPrivitera 44572a3
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera 50b591d
starting to add the screte sauce
JustinPrivitera 3325854
thx cyrus
JustinPrivitera 4757552
replace goes away
JustinPrivitera 1b00fd3
new world of node linked accessor
JustinPrivitera 73f3283
working through use with cases
JustinPrivitera 0c5aef2
use with
JustinPrivitera 1190e41
im doing data array
JustinPrivitera 2c81fd1
have a header
JustinPrivitera 4ff36e0
exec array done besides special methods
JustinPrivitera 6cf6a0a
data array special sauce
JustinPrivitera 837048c
have more setters
JustinPrivitera eb47dd4
intense compiler battles
JustinPrivitera 428ac39
get python out of here
JustinPrivitera 553c4c4
solved the dependency issue
JustinPrivitera 28ec695
battle for the soul of this nation
JustinPrivitera 9922a7c
fixed horrible dependency issue
JustinPrivitera 4028f18
this is for cyrus
JustinPrivitera e14613f
im in jail
JustinPrivitera 47e6f62
world hunger - solved
JustinPrivitera f0bb334
world peace - done
JustinPrivitera e48b995
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera fc7bbd7
restore node note
JustinPrivitera eea3e9a
notes fgrom meeting
JustinPrivitera 3705428
exec policy rewrite
JustinPrivitera a51da75
memory manager
JustinPrivitera 6e12f87
clean up execution
JustinPrivitera 675db88
compiler jail
JustinPrivitera dfa6c1c
commented out things so now we compile
JustinPrivitera ac41a1f
maybe the answer
JustinPrivitera ddaf819
bringing more things back online
JustinPrivitera 92d67f2
our problems may be solvesd
JustinPrivitera df52f73
compiler? I barely know her!
JustinPrivitera 425169e
they call me mr compiler
JustinPrivitera 2011e38
let em play ref
JustinPrivitera 3a5447a
defualt policy
JustinPrivitera edeba86
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera 7bc4a38
clean up policies
JustinPrivitera e689e74
fix foralls in topo md
JustinPrivitera 64ce7e3
big moves
JustinPrivitera 6d76471
I don'
JustinPrivitera fb40758
finished rewrite of topo md
JustinPrivitera 08794e6
nice
JustinPrivitera 7bc61c8
the end of an era
JustinPrivitera 48ce5af
oh yeah cmake
JustinPrivitera a42f91b
compiles
JustinPrivitera 2ce692a
all the old tests pass
JustinPrivitera aafbaff
cpp magic tests + openmp
JustinPrivitera 8f58145
woohoo
JustinPrivitera 9f4565a
compile
JustinPrivitera 658a967
note
JustinPrivitera 2d21363
gimme set methods
JustinPrivitera 0c78216
livchat couldn't save me
JustinPrivitera 0505b3a
todo
JustinPrivitera 0484304
back online
JustinPrivitera ec0019e
add some comments
JustinPrivitera e5ffe26
eliminating todos
JustinPrivitera 06c9bde
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera 72513b6
I eat const nodes for breakfast
JustinPrivitera 8f47fa0
life is better with const_cast
JustinPrivitera d0ccdf8
sorry for my node sins
JustinPrivitera 396adee
it will be time consuming to bring all the tests online
JustinPrivitera ef058d1
another day another test
JustinPrivitera f2f83e2
test for set_using_exec_array
JustinPrivitera f396712
you can now ask the exec environment what is supported
JustinPrivitera d3647fd
rewrote test using new exec enviornment checks
JustinPrivitera 2d9a193
more tests
JustinPrivitera 4a6838b
fix compile errors and tests pass
JustinPrivitera 457798f
summary stats test
JustinPrivitera 61bd505
all the tests are online
JustinPrivitera d03d2a8
fix typo
JustinPrivitera 850d002
test going from data array to exec array
JustinPrivitera f16dd28
data accessor to exec array test
JustinPrivitera 0754148
tests for data array to be constructed from exec array and exec accessor
JustinPrivitera 0973268
exec accessor tests
JustinPrivitera 75f7527
tests for everything to data accessor
JustinPrivitera 836d092
we have tests for all sets between data/exec arrays/accessors
JustinPrivitera 925c0ba
strawman
JustinPrivitera c94d508
conduit device check macro
JustinPrivitera c9ac3d4
we're not ready for the strawman
JustinPrivitera 1db647e
tag dispatching approach to avoid partial specialization error
JustinPrivitera fc917ed
fix device error macro
JustinPrivitera e316b2d
works now just need it to compile
JustinPrivitera bd90c8e
add active space method
JustinPrivitera 8ce35a7
fix compile error
JustinPrivitera 856e27d
deleting the strawman helps us compile
JustinPrivitera 8387d13
as_whatever_exec_array methods
JustinPrivitera fb8229f
more as_whatever_array methods
JustinPrivitera f465abc
hooking things up in Value class
JustinPrivitera a1140f3
hooking things up in ConstValue class
JustinPrivitera 63cd5ee
update exec accessor tests
JustinPrivitera d1fdfb7
update exec array tests
JustinPrivitera 41bc3b6
SFINAE
JustinPrivitera b054c5a
OWIE
JustinPrivitera c2a6082
Merge branch 'develop' into task/JustinPrivitera/05_01_24/device_support
JustinPrivitera 8447dfd
Merge ExecutionArray/Accessor with DataArray/Accessor (#1473)
JustinPrivitera e8681e8
merge develop
JustinPrivitera c419d2c
replace old for_all with forall
JustinPrivitera 5697007
attempt to fix MSVC compilation errors
JustinPrivitera 03f7080
merge develop
JustinPrivitera 2be1395
fix merge damage
JustinPrivitera b853f3a
necessary cmake steps
JustinPrivitera ae7f890
codex helps with HIP 2 (#1564)
JustinPrivitera b6902c3
Codex cleans up device support (#1565)
JustinPrivitera db26262
rename ExecutionPolicy when it is a typename and add some style comments
JustinPrivitera bf8b572
codex add reducers and atomics (#1569)
JustinPrivitera e5c3997
Device Support: codex implements the strawman (#1570)
JustinPrivitera eee9601
comments
JustinPrivitera e4fe170
cmake for cyrus
JustinPrivitera a35d444
add note
JustinPrivitera 459a744
codex versus cyrus
JustinPrivitera 7d9c0a9
sorry cyrus
JustinPrivitera c255e8c
blt enables languages for us
JustinPrivitera fcb7fcc
cyrus wins
JustinPrivitera c7eb6cd
remove redundant set target line
JustinPrivitera ffa88a7
cmake helper
JustinPrivitera 03ff245
fixes for cyrus
JustinPrivitera dbd8112
add decorators
JustinPrivitera ce0a3b0
Apply suggestion from @JustinPrivitera
JustinPrivitera f70b33c
fight codex to remove the decorator header
JustinPrivitera 3ba8fad
Merge branch 'task/JustinPrivitera/05_01_24/device_support' of github…
JustinPrivitera 458c327
Apply suggestion from @JustinPrivitera
JustinPrivitera f86da9b
Apply suggestion from @JustinPrivitera
JustinPrivitera 1fa25ad
Apply suggestion from @JustinPrivitera
JustinPrivitera 1be83ee
Apply suggestion from @JustinPrivitera
JustinPrivitera a6f9e03
Apply suggestions from code review
JustinPrivitera 9a04bd5
make cmake function take an argument
JustinPrivitera 966160c
merge
JustinPrivitera a142b5d
make cmakedefines more like openmp
JustinPrivitera 484a4fb
remove undefs
JustinPrivitera fb05f75
Apply suggestion from @JustinPrivitera
JustinPrivitera 2e7082e
we don't need memory manager
JustinPrivitera 091a891
parallel policy added
JustinPrivitera 59aa4a5
Device Support: Caliper Annotations + Actually Run On Device (#1581)
JustinPrivitera c752079
fix warnings from smoke test
JustinPrivitera 2de500a
fix conflicts
JustinPrivitera d704a8c
argument to control array sizes to influence timings
JustinPrivitera 6c74a2e
comments and changes
JustinPrivitera ba2840f
Device Support PR feedback (#1589)
JustinPrivitera 805d9f2
Apply suggestion from @JustinPrivitera
JustinPrivitera ffe5e78
comment
JustinPrivitera 41863a8
Merge branch 'task/JustinPrivitera/05_01_24/device_support' of github…
JustinPrivitera fbb9473
Device Support PR Feedback: Fix atomics and reducers (#1592)
JustinPrivitera 8b13210
clean up macro story
JustinPrivitera 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8340,7 +8340,7 @@ void polyhedral_face_centers_normals(const IndexAccessor subelements_connectivit | |
| Vector *allFaceNormalsPtr = allFaceNormals.data(); | ||
|
|
||
| // Compute face centers and normals. | ||
| conduit::execution::for_all<ExecPolicy>(0, totalNumFaces, [=](conduit::index_t f) { | ||
| conduit::execution::forall<ExecPolicy>(0, totalNumFaces, [=](conduit::index_t f) { | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am confused by this
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. wherever this method is called, we want to take a look at the template and consider switching to a policy object choice. |
||
| const int NUM_VERTS = 4; | ||
| const auto size = subelements_sizes[f]; | ||
| const auto offset = subelements_offsets[f]; | ||
|
|
@@ -8457,7 +8457,7 @@ void polyhedral_elem_centers(const IndexAccessor elements_connectivity, | |
| allElemCenters.resize(totalNumElems); | ||
| Vector *allElemCentersPtr = allElemCenters.data(); | ||
| const Vector *allFaceCentersPtr = allFaceCenters.data(); | ||
| conduit::execution::for_all<ExecPolicy>(0, totalNumElems, [=](conduit::index_t i) { | ||
| conduit::execution::forall<ExecPolicy>(0, totalNumElems, [=](conduit::index_t i) { | ||
| const auto size = elements_sizes[i]; | ||
| const auto offset = elements_offsets[i]; | ||
| Vector center {}; | ||
|
|
@@ -8580,7 +8580,7 @@ static void polyhedral_to_hexes(const conduit::Node &n_topo, conduit::Node &n_ou | |
| const Vector *allFaceCentersPtr = allFaceCenters.data(); | ||
| const Vector *allFaceNormalsPtr = allFaceNormals.data(); | ||
| const Vector *allElemCentersPtr = allElemCenters.data(); | ||
| conduit::execution::for_all<ExecPolicy>(0, nElem, [=](conduit::index_t i) { | ||
| conduit::execution::forall<ExecPolicy>(0, nElem, [=](conduit::index_t i) { | ||
| constexpr int FORWARD = 1; | ||
| constexpr int BACKWARD = -1; | ||
| // Determine face orientations with respect to this element. | ||
|
|
||
This file contains hidden or 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 hidden or 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 hidden or 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 hidden or 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.
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.
Uh oh!
There was an error while loading. Please reload this page.