Skip to content
This repository has been archived by the owner on Nov 14, 2022. It is now read-only.

1419 add reduction script dunder methods ar 1419 #1434

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Dagonite
Copy link
Contributor

@Dagonite Dagonite commented Feb 7, 2022

fixes #1489

@codecov
Copy link

codecov bot commented Feb 7, 2022

Codecov Report

Merging #1434 (5df9cca) into main (0438b8b) will decrease coverage by 0.20%.
The diff coverage is 50.00%.

❗ Current head 5df9cca differs from pull request most recent head ea8bc87. Consider uploading reports for the commit ea8bc87 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1434      +/-   ##
==========================================
- Coverage   91.17%   90.96%   -0.21%     
==========================================
  Files          17       17              
  Lines         793      797       +4     
==========================================
+ Hits          723      725       +2     
- Misses         70       72       +2     
Flag Coverage Δ
backend 90.96% <50.00%> (-0.21%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
autoreduce_qp/queue_processor/reduction/service.py 97.97% <50.00%> (-1.34%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0438b8b...ea8bc87. Read the comment docs.

@Pasarus Pasarus linked an issue Aug 18, 2022 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add __repr__ or __str__ to ReductionScript for easier reading
1 participant