Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Sep 3, 2022

This was logged as null, since the logger didn't know how to handle the MappingView. I simply implemented an __rfm_json_encode__ in this class, as well in the SequenceView.

@codecov-commenter
Copy link

Codecov Report

Merging #2588 (59513c4) into master (eff1c97) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2588   +/-   ##
=======================================
  Coverage   86.67%   86.68%           
=======================================
  Files          59       59           
  Lines       10854    10858    +4     
=======================================
+ Hits         9408     9412    +4     
  Misses       1446     1446           
Impacted Files Coverage Δ
reframe/utility/__init__.py 92.65% <100.00%> (+0.04%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vkarak vkarak merged commit a27b7ca into reframe-hpc:master Sep 6, 2022
@vkarak vkarak deleted the bugfix/log-check-perfvalues branch September 6, 2022 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants