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

update config tutorials #465

Merged
merged 2 commits into from
Feb 7, 2021
Merged

update config tutorials #465

merged 2 commits into from
Feb 7, 2021

Conversation

jin-s13
Copy link
Collaborator

@jin-s13 jin-s13 commented Feb 6, 2021

fix #303

@jin-s13 jin-s13 requested a review from innerlee February 6, 2021 15:26
@codecov
Copy link

codecov bot commented Feb 6, 2021

Codecov Report

Merging #465 (8c0db83) into master (291b43e) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #465      +/-   ##
==========================================
+ Coverage   80.70%   80.72%   +0.02%     
==========================================
  Files         137      137              
  Lines        9156     9156              
  Branches     1469     1469              
==========================================
+ Hits         7389     7391       +2     
  Misses       1444     1444              
+ Partials      323      321       -2     
Flag Coverage Δ
unittests 80.72% <ø> (+0.02%) ⬆️

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

Impacted Files Coverage Δ
mmpose/datasets/datasets/face/face_base_dataset.py 77.77% <ø> (ø)
.../datasets/datasets/fashion/fashion_base_dataset.py 83.63% <ø> (ø)
mmpose/datasets/datasets/hand/hand_base_dataset.py 93.63% <ø> (ø)
...datasets/datasets/top_down/topdown_coco_dataset.py 87.27% <ø> (ø)
...datasets/datasets/top_down/topdown_mpii_dataset.py 40.65% <ø> (ø)
mmpose/datasets/pipelines/mesh_transform.py 95.45% <0.00%> (+1.13%) ⬆️

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 291b43e...f16632d. Read the comment docs.


If the value to be updated is a list or a tuple. For example, the config file normally sets `workflow=[('train', 1)]`. If you want to
change this key, you may specify `--cfg-options workflow="[(train,1),(val,1)]"`. Note that the quotation mark \" is necessary to
support list/tuple data types, and that **NO** white space is allowed inside the quotation marks in the specified value.
Copy link
Contributor

Choose a reason for hiding this comment

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

why spaces are not allowwd?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

seems like an upstream problem (mmcv)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

BTW, this part is copied from mmaction2

@innerlee innerlee merged commit 2110116 into master Feb 7, 2021
@innerlee innerlee deleted the updates branch February 7, 2021 04:39
@jin-s13 jin-s13 mentioned this pull request Feb 8, 2021
11 tasks
rollingman1 pushed a commit to rollingman1/mmpose that referenced this pull request Nov 5, 2021
* a little bit faster confusion matrix

* add changelog
shuheilocale pushed a commit to shuheilocale/mmpose that referenced this pull request May 6, 2023
* update tutorials

* add 0_config.md
ajgrafton pushed a commit to ajgrafton/mmpose that referenced this pull request Mar 6, 2024
* update tutorials

* add 0_config.md
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.

Add a README file to explain the settings in the config files
2 participants