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

Offset Reduction #1151

Draft
wants to merge 38 commits into
base: master
Choose a base branch
from
Draft

Offset Reduction #1151

wants to merge 38 commits into from

Conversation

mroethlin
Copy link
Contributor

Technical Description

A rough implementation of the work around described in the google doc about path dependency. The implementation is complete in the sense that there is SIR, IIR representation of the offset, dawn4py hooks and code generation in both backends. However, no safety is provided:

  • There are no facilities to check that the right amount of offsets was passed. Passing too many offsets will lead to ignored values, passing too few will lead to a segfault (in the best case).
  • There is no type checking for the sparse field being offset.

DO NOT MERGE

mroethlin and others added 30 commits December 12, 2019 14:45
@mroethlin
fixed VarDeclStmt visitor
1d5e8b1
@mroethlin
Merge remote-tracking branch 'origin/master'
556e5c6
@mroethlin
Merge remote-tracking branch 'origin/master'
092f7eb
@mroethlin
Merge remote-tracking branch 'origin/master'
f262804
@mroethlin
Merge remote-tracking branch 'origin/master'
3865d4d
@mroethlin
Merge remote-tracking branch 'origin/master'
83f4680
@mroethlin
Merge remote-tracking branch 'origin/master'
bf0a6d2
@mroethlin
Merge remote-tracking branch 'origin/master'
a317ec9
@mroethlin
Merge remote-tracking branch 'origin/master'
2b243c4
@mroethlin
Merge remote-tracking branch 'origin/master'
1e6c81a
@mroethlin
Merge remote-tracking branch 'origin/master'
1b53bd1
@mroethlin
updating git hook ignore list for new codegen tests
3a50412
@mroethlin
Merge remote-tracking branch 'origin/master'
6550116
@mroethlin
Merge remote-tracking branch 'origin/master'
d933ffe
@havogt @mroethlin
Use experimental::fs if fs is not supported (#640)
2662ecb 
## Technical Description

Use std::experimental::filesystem if std::filesystem is not supported, i.e. gcc < 8.
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
c383e3f
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
72b547e
@mroethlin
Merge remote-tracking branch 'origin/master'
328a0cf
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
8d560dc
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
d25dab1
@mroethlin
wip
bca7232
@mroethlin
rough but complete implementation
26e6347
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
b252516
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
1400f05
@mroethlin
small improvements, fix stupid bug when constructing offset dequeue
311c440
@mroethlin
Merge branch 'master' of https://github.com/MeteoSwiss-APN/dawn
a14e95e
@mroethlin
handling the case where lower AND upper level is end
695687f
@mroethlin
sparse full field indexing bug fixed
0b910bb
@mroethlin
only check errors in debug build
52e407b
@mroethlin
merge master
4900e0f
@mroethlin mroethlin changed the title Offset reduction Offset Reduction Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mroethlin @havogt @actions-user