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

Get rid of global parameter call operator #1256

Merged
merged 13 commits into from
Apr 5, 2019
Merged

Get rid of global parameter call operator #1256

merged 13 commits into from
Apr 5, 2019

Conversation

anstaf
Copy link
Contributor

@anstaf anstaf commented Apr 3, 2019 •
edited
Loading

  • global_accessor_with_parameters goes away
  • global_accessor is just an alias to accessor now (with zero extent, in intent and 0 as a number of dimensions)
  • a few typedefs are renamed to match the SID concept

@anstaf anstaf changed the title Even more changes some more changes Apr 3, 2019
@anstaf
Copy link
Contributor Author

anstaf commented Apr 3, 2019

launch strgrid

@anstaf
Copy link
Contributor Author

anstaf commented Apr 3, 2019

launch icgrid

@anstaf
Copy link
Contributor Author

anstaf commented Apr 3, 2019

do perftest

@anstaf
Copy link
Contributor Author

anstaf commented Apr 4, 2019

launch strgrid

@anstaf
Copy link
Contributor Author

anstaf commented Apr 4, 2019

launch icgrid

@anstaf anstaf requested a review from havogt April 5, 2019 11:12
@anstaf anstaf changed the title some more changes Get rid of global parameter call operator Apr 5, 2019
@havogt
Copy link
Contributor

havogt commented Apr 5, 2019

launch strgrid

@havogt
Copy link
Contributor

havogt commented Apr 5, 2019

launch perftest

@havogt havogt merged commit c984a6b into GridTools:master Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@havogt havogt havogt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@anstaf @havogt