Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

the the typo error of function name sample_unconditional_model #227

Closed
wants to merge 6 commits into from

Conversation

TommyZihao
Copy link
Contributor

sample_uncoditional_model --> sample_unconditional_model

fix the typo error
fix the typo error
fix the typo error
fix the typo error
@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #227 (e1212d5) into master (c355edb) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
+ Coverage   76.71%   76.75%   +0.04%     
==========================================
  Files         130      130              
  Lines        9421     9421              
  Branches     1899     1899              
==========================================
+ Hits         7227     7231       +4     
+ Misses       1728     1726       -2     
+ Partials      466      464       -2     
Flag Coverage Δ
unittests 76.75% <100.00%> (+0.04%) ⬆️

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

Impacted Files Coverage Δ
mmgen/apis/__init__.py 100.00% <ø> (ø)
mmgen/apis/inference.py 57.50% <100.00%> (ø)
...chitectures/stylegan/generator_discriminator_v2.py 87.77% <0.00%> (+1.11%) ⬆️
...chitectures/stylegan/generator_discriminator_v1.py 89.83% <0.00%> (+1.12%) ⬆️

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 534d12e...e1212d5. Read the comment docs.

@LeoXing1996
Copy link
Collaborator

Thanks for your contribution. Please run pre-commit install before creating PR to avoid the lint errors.

@@ -65,7 +65,7 @@ def main():
if args.sample_cfg is None:
args.sample_cfg = dict()

results = sample_uncoditional_model(model, args.num_samples,
results = sample_unconditional_model(model, args.num_samples,
Copy link
Collaborator

Choose a reason for hiding this comment

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

You may also fix the api name here.

from mmgen.apis import init_model, sample_uncoditional_model # isort:skip # noqa

@plyfager
Copy link
Collaborator

Since this has been implemented in #249 , this pr will be closed.

@plyfager plyfager closed this Feb 22, 2022
LeoXing1996 pushed a commit to LeoXing1996/mmgeneration that referenced this pull request Oct 27, 2022
* Add LIIF.

* Add LIIF.

* modify super init.

* Add licence,optimize code.

* remove debugging codes.

* Modify __init__() function.

* Modify test function of LIIF.

Co-authored-by: 李尹硕 <SENSETIME\liyinshuo@cn0014004493l.domain.sensetime.com>
Co-authored-by: liyinshuo <liyinshuo@sensetime.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants