-
Notifications
You must be signed in to change notification settings - Fork 110
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
Score-P #181
base: master
Are you sure you want to change the base?
Score-P #181
Conversation
up to date
update fork
Hey @gmarkomanolis -- Thanks for putting this together! I started reviewing this today, and unfortunately it looks like MiniWeather has moved to a CMake build system after this MR was open as a draft. Would you mind updating the instrumentation example(s) to match how MiniWeather is built now? I started doing so in 81be937. I also noticed that the following two sections are only code snippets: Could you add a blurb about the difference between these two methods? You'll see in 0e1eceb there's quite a bit of table reformatting, which was just to bring things into alignment and emphasize values as in-lined code. This commit also added Score-P into the menus, which required a rename of the .rst file. Thanks again! Happy to finally be getting this one published. |
Preparing the material for Score-P tutorial