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

Remove the userdemo section in examples #25881

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Sigma-Verma
Copy link
Contributor

@Sigma-Verma Sigma-Verma commented May 13, 2023

PR summary

Closes #25800

To address this issue, the proposed fix is to convert the examples in the userdemo section from literalinclude to figure directives if they are still used in the documentation. This will allow the examples to be displayed as figures rather than code snippets
To address this issue, the proposed fix is to convert the examples in the userdemo section from literalinclude to figure directives if they are still used in the documentation. This will allow the examples to be displayed as figures rather than code snippets

PR checklist



To address this issue, the proposed fix is to convert the examples in the userdemo section from literalinclude to figure directives if they are still used in the documentation. This will allow the examples to be displayed as figures rather than code snippets
@Sigma-Verma Sigma-Verma changed the title #25800 Remove the userdemo section in examples #25800 May 13, 2023
@jklymak jklymak changed the title Remove the userdemo section in examples #25800 Remove the userdemo section in examples May 13, 2023
Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

This appears to contain the changes from #25726 and should be rebased to be on top of the merged version.

@story645
Copy link
Member

Given that this change would still require that the userdemo section exists, I'm wondering if @timhoffm meant that you should use the ..plot directive with the code inlined in the tutorial.

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.

[MNT]: Remove the userdemo section in examples
4 participants