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 model file docs to be more accurate #2148

Merged
merged 3 commits into from
Feb 24, 2023

Conversation

tmc
Copy link
Contributor

@tmc tmc commented Feb 20, 2023

Description

This clears up existing references to "torch.nn.modules" and replaces them with "torch.nn.Module" with links to the docs as appropraite.

Type of change

  • Documentation update

@codecov
Copy link

codecov bot commented Feb 20, 2023

Codecov Report

Merging #2148 (fe1eab3) into master (26089d4) will not change coverage.
The diff coverage is n/a.

❗ Current head fe1eab3 differs from pull request most recent head ac21b1c. Consider uploading reports for the commit ac21b1c to get more accurate results

@@           Coverage Diff           @@
##           master    #2148   +/-   ##
=======================================
  Coverage   53.36%   53.36%           
=======================================
  Files          71       71           
  Lines        3225     3225           
  Branches       56       56           
=======================================
  Hits         1721     1721           
  Misses       1504     1504           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@agunapal agunapal self-requested a review February 23, 2023 19:41
@msaroufim msaroufim merged commit 87359c4 into pytorch:master Feb 24, 2023
morgandu pushed a commit to morgandu/pytorch-serve that referenced this pull request Mar 8, 2023
* Update model file docs to be more accurate

* model-archiver: Improve docs

---------

Co-authored-by: Ankith Gunapal <agunapal@meta.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.

3 participants