Added sub heading to ignite.metrics#2448
Conversation
vfdev-5
left a comment
vfdev-5
left a comment
There was a problem hiding this comment.
Thanks for the draft PR @sayantan1410
Let's improve text
|
@vfdev-5 Hey, made the changes please check !! |
|
@vfdev-5 Hey I have made some changes, let me know if they are alright !! |
|
@sayantan1410 can you reread the docs https://deploy-preview-2448--pytorch-ignite-preview.netlify.app/metrics.html#ignite-metrics and critically estimate how more comprehensive or cumbersome docs are becoming now ? |
|
@vfdev-5 Hey, I tried to make it simpler and consistent, can you please check it once more. |
vfdev-5
left a comment
vfdev-5
left a comment
There was a problem hiding this comment.
Looks much better now, thanks @sayantan1410 !
Just a small nit
|
@DhDeepLIT you were originally behind the request to improve our docs, can you please check if this PR provides enough info ? Thanks a lot ! |
vfdev-5
left a comment
vfdev-5
left a comment
There was a problem hiding this comment.
Thanks @sayantan1410
Let's wait a bit for other comments if any and merge it later.
|
@vfdev-5 yeah, No problem !! |
2 participants
Related to #2437
Added a sub heading to ignite.metrics, however I have not added its link to other pages.
Let me know the changes that I need to make.