Skip to content

Conversation

@AChenQ
Copy link
Collaborator

@AChenQ AChenQ commented Sep 8, 2021

No description provided.

@coveralls
Copy link

coveralls commented Sep 8, 2021

Pull Request Test Coverage Report for Build 1220191058

  • 6 of 21 (28.57%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 86.362%

Changes Missing Coverage Covered Lines Changed/Added Lines %
tensorbay/client/profile.py 6 21 28.57%
Files with Coverage Reduction New Missed Lines %
tensorbay/client/profile.py 2 40.98%
Totals Coverage Status
Change from base Build 1220174494: -0.1%
Covered Lines: 6130
Relevant Lines: 7098

💛 - Coveralls

@AChenQ AChenQ requested a review from linjiX September 8, 2021 11:10
@AChenQ AChenQ force-pushed the T18299_multi_process branch from c5c6ba6 to 3bacab1 Compare September 8, 2021 12:11
@AChenQ AChenQ force-pushed the T18299_multi_process branch 5 times, most recently from acf8abd to ecf3dcb Compare September 10, 2021 05:12
"""
if multiprocess:
self.manager = Manager()
self.summary = self.manager.dict()
Copy link
Contributor

Choose a reason for hiding this comment

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

should we set manger and summary private?

@AChenQ AChenQ force-pushed the T18299_multi_process branch from ecf3dcb to 7e05c99 Compare September 10, 2021 06:25
@AChenQ AChenQ force-pushed the T18299_multi_process branch from 7e05c99 to 6c73081 Compare September 10, 2021 06:28
@AChenQ AChenQ merged commit 2595cb7 into Graviti-AI:main Sep 10, 2021
@AChenQ AChenQ deleted the T18299_multi_process branch September 10, 2021 06:28
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.

3 participants