Skip to content
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

Use np.random instead of random #213

Merged
merged 3 commits into from
Feb 26, 2021
Merged

Use np.random instead of random #213

merged 3 commits into from
Feb 26, 2021

Conversation

innerlee
Copy link
Contributor

Increase consistency

Signed-off-by: lizz <lizz@sensetime.com>
Signed-off-by: lizz <lizz@sensetime.com>
Signed-off-by: lizz <lizz@sensetime.com>
@codecov
Copy link

codecov bot commented Feb 26, 2021

Codecov Report

Merging #213 (4fde769) into master (48e39a9) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
- Coverage   82.28%   82.27%   -0.01%     
==========================================
  Files         145      145              
  Lines        6699     6697       -2     
  Branches      998      998              
==========================================
- Hits         5512     5510       -2     
  Misses       1083     1083              
  Partials      104      104              
Flag Coverage Δ
unittests 82.27% <100.00%> (-0.01%) ⬇️

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

Impacted Files Coverage Δ
mmedit/datasets/pipelines/augmentation.py 99.16% <100.00%> (-0.01%) ⬇️
mmedit/datasets/pipelines/crop.py 96.81% <100.00%> (-0.02%) ⬇️

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 48e39a9...7a7d5aa. Read the comment docs.

@innerlee innerlee merged commit ebb9b37 into master Feb 26, 2021
@innerlee innerlee deleted the zz/rand branch February 26, 2021 11:12
Yshuo-Li pushed a commit to Yshuo-Li/mmediting that referenced this pull request Jul 15, 2022
* Use np.random instead of random

Signed-off-by: lizz <lizz@sensetime.com>

* Fix randint range

Signed-off-by: lizz <lizz@sensetime.com>

* Fix randint range

Signed-off-by: lizz <lizz@sensetime.com>
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.

None yet

2 participants