Skip to content

update breathe#49407

Closed
mattip wants to merge 1 commit intopytorch:masterfrom
mattip:breathe
Closed

update breathe#49407
mattip wants to merge 1 commit intopytorch:masterfrom
mattip:breathe

Conversation

@mattip
Copy link
Copy Markdown
Contributor

@mattip mattip commented Dec 15, 2020

Fixes #47462, but not completely.

Update breathe to the latest version to get fixes for the "Unable to resolve..." issues. There are still some build errors, but much fewer than before.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Dec 15, 2020

💊 CI failures summary and remediations

As of commit 1b15c2a (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

This comment has been revised 3 times.

Copy link
Copy Markdown
Contributor

@glaringlee glaringlee left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for updating.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@glaringlee has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@vermeeren
Copy link
Copy Markdown

fyi there was a regression discovered, likely with the ellipsis ... syntax. breathe-doc/breathe#608

This wasn't caught in our tests and also not in any of the projects I manually tried it with, likely a very specific edge case. You may want to keep an eye on this as there should be bugfix release for this soon.

@mattip
Copy link
Copy Markdown
Contributor Author

mattip commented Dec 15, 2020

Thanks for the heads up. I am subscribed to the breathe repo.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 15, 2020

Codecov Report

Merging #49407 (1b15c2a) into master (5a5e576) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #49407      +/-   ##
==========================================
- Coverage   80.56%   80.41%   -0.16%     
==========================================
  Files        1875     1875              
  Lines      202701   202701              
==========================================
- Hits       163307   163000     -307     
- Misses      39394    39701     +307     

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@glaringlee merged this pull request in acd72e7.

hwangdeyu pushed a commit to hwangdeyu/pytorch that referenced this pull request Jan 6, 2021
Summary:
Fixes pytorch#47462, but not completely.

Update breathe to the latest version to get fixes for the "Unable to resolve..." issues. There are still some build errors, but much fewer than before.

Pull Request resolved: pytorch#49407

Reviewed By: izdeby

Differential Revision: D25562163

Pulled By: glaringlee

fbshipit-source-id: 91bfd9e9ac70723816309f489022d72853f5fdc5
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Fixes pytorch#47462, but not completely.

Update breathe to the latest version to get fixes for the "Unable to resolve..." issues. There are still some build errors, but much fewer than before.

Pull Request resolved: pytorch#49407

Reviewed By: izdeby

Differential Revision: D25562163

Pulled By: glaringlee

fbshipit-source-id: 91bfd9e9ac70723816309f489022d72853f5fdc5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

at::from_blob documentation is broken

5 participants