compiler: Minor tweaks to enable integrated NCU-based profiling#2937
compiler: Minor tweaks to enable integrated NCU-based profiling#2937FabioLuporini wants to merge 2 commits into
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2937 +/- ##
==========================================
- Coverage 83.37% 83.36% -0.02%
==========================================
Files 248 248
Lines 51881 51905 +24
Branches 4479 4479
==========================================
+ Hits 43257 43271 +14
- Misses 7867 7879 +12
+ Partials 757 755 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| configuration.add('profiling', 'basic', list(profiler_registry), impacts_jit=False) | ||
| def profiling_preprocessor(i): | ||
| if isinstance(i, dict): | ||
| return NcuProfiling(i['ncu']) |
There was a problem hiding this comment.
This could use a try/key error for safety
| # Allow unquoted strings as `k:v` values. | ||
| processed.append(i) | ||
| values = processed | ||
| except AttributeError: |
There was a problem hiding this comment.
Not sure why you need processed and values if you just do values = processed
There was a problem hiding this comment.
there is an eval above. This is essentially to avoid the user having to supply quotes in the env var , that is, instead of doing DEVITO_PROFILING="ncu:'ForwardFletcherTTI'" they can just do DEVITO_PROFILING="ncu:ForwardFletcherTTI"
3 participants
Bulk of the code is in PRO