Skip to content

import, delete, add and update traces for report #1227

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

Merged
merged 11 commits into from
Jun 1, 2022

Conversation

gmalinve
Copy link
Contributor

In this PR new methods have been implemented:

  • Import trace from file
  • Delete traces from report
  • Add traces to report
  • Update traces in report

New tests.

@gmalinve gmalinve self-assigned this May 27, 2022
@gmalinve gmalinve linked an issue May 27, 2022 that may be closed by this pull request
gmalinve added 2 commits May 30, 2022 09:50
# Conflicts:
#	_unittest/test_12_PostProcessing.py
@codecov
Copy link

codecov bot commented May 30, 2022

Codecov Report

Merging #1227 (a651abc) into main (eba23d1) will decrease coverage by 0.16%.
The diff coverage is 90.32%.

@@            Coverage Diff             @@
##             main    #1227      +/-   ##
==========================================
- Coverage   77.79%   77.62%   -0.17%     
==========================================
  Files          95       96       +1     
  Lines       37471    37772     +301     
==========================================
+ Hits        29149    29320     +171     
- Misses       8322     8452     +130     

Copy link
Collaborator

@maxcapodi78 maxcapodi78 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gmalinve gmalinve merged commit 5249ca5 into main Jun 1, 2022
@gmalinve gmalinve deleted the add_remove_traces_report branch June 1, 2022 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Import/Delete/Add/Update traces in report
3 participants