Tapasya Patki
Tapasya Patki
Hi @morrone Thank you so much for getting the process started! Jessica and I will start working on the comments for the name, Makefile and autoconf soon (early next week),...
@baallan The library uses more than just MSRs (MSRs are only available on Intel systems and AMD systems, for IBM we use OPAL firmware, and for Nvidia and AMD GPUs...
@morrone Sure, updating just the sampler to use your OVIS-JSON library should be possible easily. I probably misunderstood what @baallan was suggesting there. Can you point me to the documentation...
I'm not entirely sure if updating just the LDMS sampler code to parse with OVIS-JSON removes the need for linking in libjansson for the LDMS part. You will still need...
Is there a current norm for other LDMS plugins that use their own JSON libraries?
@morrone We have addressed all your comments and tested this thoroughly, so I marked this PR ready for review.
Hi @morrone, I fixed the typo around `enable_variorum`. Please re-run tests when you get a chance, hopefully they will pass correctly this time :)
@narategithub @morrone @tom95858: I believe this PR ready to go now. Do let us know if there's anything else we need to do for this PR.
Maybe we can use PerfFlowAspect for this. Adding @slabasan as well, we had an initial integration of fluxion and PFA that maybe we can start with
I am aware that @rountree was also running into this as well yesterday. Tagging him so he can follow.