-
Notifications
You must be signed in to change notification settings - Fork 229
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
compiler: Fix issue 2194 #2212
compiler: Fix issue 2194 #2212
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2212 +/- ##
=======================================
Coverage 87.10% 87.11%
=======================================
Files 228 228
Lines 40783 40810 +27
Branches 7465 7471 +6
=======================================
+ Hits 35525 35551 +26
Misses 4653 4653
- Partials 605 606 +1
|
devito/ir/support/basic.py
Outdated
|
||
# Writing (reading) over an entire dimension, reading (writing) | ||
# from one point | ||
|
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.
nitpicking, I wouldn't normally say it, but since this is your learning phase and since this might be the first of many PRs... :
- Drop blank line;
- Move the "For example ..." on the same line as "# from one point. For example:" (after the for example, it's ok to break line)
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.
In doing this, please git rebase and squash
782463b
to
47a39ea
Compare
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.
Thanks, GTG
27bf8bf
to
1a11d18
Compare
1a11d18
to
c3abf22
Compare
No description provided.