Skip to content

Found a subtle bug caused by data race.#3

Merged
tohtsky merged 2 commits intomainfrom
fix-sync-topic-bug
Jan 10, 2022
Merged

Found a subtle bug caused by data race.#3
tohtsky merged 2 commits intomainfrom
fix-sync-topic-bug

Conversation

@tohtsky
Copy link
Copy Markdown
Owner

@tohtsky tohtsky commented Jan 10, 2022

No description provided.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #3 (114de6d) into main (87609e0) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   95.33%   95.22%   -0.12%     
==========================================
  Files          13       13              
  Lines         900      900              
==========================================
- Hits          858      857       -1     
- Misses         42       43       +1     
Impacted Files Coverage Δ
cpp_sources/child_worker.cpp 100.00% <100.00%> (ø)
cpp_sources/trainer_base.cpp 98.19% <100.00%> (+0.01%) ⬆️
cpp_sources/wrapper.cpp 96.55% <0.00%> (-0.58%) ⬇️

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 87609e0...114de6d. Read the comment docs.

@tohtsky tohtsky merged commit 451c08a into main Jan 10, 2022
@tohtsky tohtsky deleted the fix-sync-topic-bug branch January 10, 2022 15:26
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.

2 participants