Closed SimonBiggs closed 2 years ago
Hey! Sorry for the delay. So far it looks good, however, black
is failing. To fix it run:
black .
and commit changes.
Didn't mean to close it, wrong button.
Merging #78 (4a7cd52) into master (5231806) will decrease coverage by
0.08%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #78 +/- ##
==========================================
- Coverage 90.42% 90.34% -0.09%
==========================================
Files 63 63
Lines 2267 2279 +12
==========================================
+ Hits 2050 2059 +9
- Misses 217 220 +3
Impacted Files | Coverage Δ | |
---|---|---|
pypeln/process/queue.py | 93.02% <100.00%> (-3.37%) |
:arrow_down: |
pypeln/task/queue.py | 95.90% <100.00%> (-0.07%) |
:arrow_down: |
pypeln/thread/queue.py | 93.75% <100.00%> (-0.25%) |
:arrow_down: |
pypeln/utils.py | 91.35% <100.00%> (+1.96%) |
:arrow_up: |
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 5231806...4a7cd52. Read the comment docs.
Sorry for the delay.
No need to be sorry :)
All good, thanks @SimonBiggs! Will do a release tonight with the updates.
thanks @SimonBiggs!
My pleasure :)
Will do a release tonight with the updates.
No need to rush on my behalf, I've been installing from my own git branch.
I've been installing from my own git branch.
Nice! I assume its within another poetry project or did poetry figure out how to support pip install git+...
?
Yeah a separate poetry project. I put the following in my toml file:
pypeln = { git = "https://github.com/RadiotherapyAI/pypeln", rev = "7727021b4775fb3ec26a84f8fc45547412d23720" }