Skip to content

Remove the userdemo section in examples #25881

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

Closed
wants to merge 6 commits into from
Closed

Remove the userdemo section in examples #25881

wants to merge 6 commits into 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.

@QuLogic
Copy link
Member

QuLogic commented Aug 3, 2024

I'm going to close this for lack of response, and since #28655 will replace it.

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