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

Documentation build is failing #403

Closed
rflamary opened this issue Sep 27, 2022 · 5 comments · Fixed by #402
Closed

Documentation build is failing #403

rflamary opened this issue Sep 27, 2022 · 5 comments · Fixed by #402

Comments

@rflamary
Copy link
Collaborator

Describe the bug

Documentation build is currently failing. For a while it was because of a too long example when using pytorch but in recent build (in PR) the following error has been reported on circleCI:

Sphinx-Gallery successfully executed 45 out of 47 files subselected by:

    gallery_conf["filename_pattern"] = '/plot'
    gallery_conf["ignore_pattern"]   = '__init__\\.py'

after excluding 0 files that had previously been run (based on MD5).


Extension error:
Here is a summary of the problems encountered when running the examples

Unexpected failing examples:
/home/circleci/project/examples/unbalanced-partial/plot_UOT_barycenter_1D.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/circleci/project/examples/unbalanced-partial/plot_UOT_barycenter_1D.py", line 130, in <module>
    ax = pl.gcf().gca(projection='3d')
TypeError: gca() got an unexpected keyword argument 'projection'


/home/circleci/project/examples/barycenters/plot_barycenter_1D.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/circleci/project/examples/barycenters/plot_barycenter_1D.py", line 109, in <module>
    ax = plt.gcf().gca(projection='3d')
TypeError: gca() got an unexpected keyword argument 'projection'

It seems to come from a use of matplotlib gca that is not possible anymore.

To Reproduce

Steps to reproduce the behavior:

Build the documentation

Screenshots

Code sample

Expected behavior

Doc should build on circleCI and locally ;)

Environment (please complete the following information):

CirclCI

Additional context

@agramfort
Copy link
Collaborator

agramfort commented Sep 27, 2022 via email

@rflamary
Copy link
Collaborator Author

I know @agramfort it's not a good practice, we did not fix the version in the requirement.txt, but it is usually pretty stable. They remove the parameter togca() for 3d projection in 3.6 mid september...

@rflamary
Copy link
Collaborator Author

It was definitely a good way to detect problems for recent versions though

@agramfort
Copy link
Collaborator

agramfort commented Oct 11, 2022 via email

@rflamary
Copy link
Collaborator Author

that is what @ncassereau-idris did in his PR now everything works again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants