Skip to content

Format code examples in the user documentation#1640

Merged
rahulporuri merged 18 commits into
masterfrom
doc/format-code-examples-docs
May 5, 2021
Merged

Format code examples in the user documentation#1640
rahulporuri merged 18 commits into
masterfrom
doc/format-code-examples-docs

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

@rahulporuri rahulporuri commented May 3, 2021

This PR formats/cleans up the code examples in the traitsui user documentation. The code examples used to live in the root-level directory \examples\tutorials\doc_examples\ and the relevant ones have now been moved to \docs\source\traitsui_user_manual\examples. At the moment, none of the examples in examples\tutorials\doc_examples are used - and will be removed in a future PR. The moved examples have been formatted/cleaned up.

Poruri Sai Rahul added 16 commits May 3, 2021 16:12
	modified:   docs/source/traitsui_user_manual/adapters.rst
	modified:   docs/source/traitsui_user_manual/advanced_view.rst
	modified:   docs/source/traitsui_user_manual/custom_view.rst
	modified:   docs/source/traitsui_user_manual/factories_advanced_extra.rst
	modified:   docs/source/traitsui_user_manual/factories_basic.rst
	modified:   docs/source/traitsui_user_manual/factory_intro.rst
	modified:   docs/source/traitsui_user_manual/handler.rst
	modified:   docs/source/traitsui_user_manual/view.rst
and literalinclude the example files in the documentation rather than
manually duplicating the information in the docs again

this commit only moves examples and removes duplicates from factory
intro

	new file:   docs/source/traitsui_user_manual/examples/default_trait_editors.py
	new file:   docs/source/traitsui_user_manual/examples/mixed_styles.py
	modified:   docs/source/traitsui_user_manual/factory_intro.rst
	deleted:    examples/tutorials/doc_examples/examples/default_trait_editors.py
	deleted:    examples/tutorials/doc_examples/examples/mixed_styles.py
	new file:   docs/source/traitsui_user_manual/examples/array_editor.py
	modified:   docs/source/traitsui_user_manual/factories_basic.rst
	deleted:    examples/tutorials/doc_examples/examples/array_editor.py
	new file:   docs/source/traitsui_user_manual/examples/enum_editor.py
	modified:   docs/source/traitsui_user_manual/factories_basic.rst
	deleted:    examples/tutorials/doc_examples/examples/enum_editor.py
	modified:   docs/source/traitsui_user_manual/examples/array_editor.py
	new file:   docs/source/traitsui_user_manual/examples/instance_editor_selection.py
	modified:   docs/source/traitsui_user_manual/factories_basic.rst
	deleted:    examples/tutorials/doc_examples/examples/instance_editor_selection.py
	new file:   docs/source/traitsui_user_manual/examples/key_bindings.py
	modified:   docs/source/traitsui_user_manual/factories_advanced_extra.rst
	deleted:    examples/tutorials/doc_examples/examples/key_bindings.py
	new file:   docs/source/traitsui_user_manual/examples/tree_editor.py
	modified:   docs/source/traitsui_user_manual/factories_advanced_extra.rst
	deleted:    examples/tutorials/doc_examples/examples/tree_editor.py
	modified:   docs/source/traitsui_user_manual/custom_view.rst
	new file:   docs/source/traitsui_user_manual/examples/wizard.py
	modified:   examples/tutorials/doc_examples/examples/configure_traits_view_buttons.py
	deleted:    examples/tutorials/doc_examples/examples/wizard.py
	modified:   docs/source/traitsui_user_manual/advanced_view.rst
	new file:   docs/source/traitsui_user_manual/examples/default_traits_view.py
	new file:   docs/source/traitsui_user_manual/examples/multiple_views.py
	deleted:    examples/tutorials/doc_examples/examples/default_traits_view.py
	deleted:    examples/tutorials/doc_examples/examples/multiple_views.py
	modified:   docs/source/traitsui_user_manual/advanced_view.rst
	new file:   docs/source/traitsui_user_manual/examples/multi_object_view.py
	deleted:    examples/tutorials/doc_examples/examples/multi_object_view.py
	new file:   docs/source/traitsui_user_manual/examples/configure_traits_view_group.py
	modified:   docs/source/traitsui_user_manual/view.rst
	deleted:    examples/tutorials/doc_examples/examples/configure_traits_view_group.py
	new file:   docs/source/traitsui_user_manual/examples/configure_traits_view.py
	modified:   docs/source/traitsui_user_manual/view.rst
	deleted:    examples/tutorials/doc_examples/examples/configure_traits_view.py
Also move the configure_traits_view_buttons file - which i missed before

	renamed:    examples/tutorials/doc_examples/examples/configure_traits_view_buttons.py -> docs/source/traitsui_user_manual/examples/configure_traits_view_buttons.py
	new file:   docs/source/traitsui_user_manual/examples/handler_override.py
	modified:   docs/source/traitsui_user_manual/handler.rst
	deleted:    examples/tutorials/doc_examples/examples/handler_override.py
	modified:   integrationtests/test_all_examples.py
	modified:   docs/source/traitsui_user_manual/custom_view.rst
	modified:   docs/source/traitsui_user_manual/factories_basic.rst
	modified:   docs/source/traitsui_user_manual/view.rst
@rahulporuri rahulporuri changed the title [WIP] Doc/format code examples docs Format code examples in the user documentation May 4, 2021
@rahulporuri
Copy link
Copy Markdown
Contributor Author

another "illegal instruction" related segfault


test_base_url_changed (traitsui.tests.editors.test_html_editor.TestHTMLEditor) ... Fatal Python error: Illegal instruction

Current thread 0x00007f6411a02740 (most recent call first):
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/traitsui/testing/_gui.py", line 36 in process_cascade_events
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/traitsui/testing/tester/ui_tester.py", line 116 in create_ui
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/contextlib.py", line 88 in __exit__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/traitsui/tests/editors/test_html_editor.py", line 249 in test_base_url_changed
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/case.py", line 605 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/case.py", line 653 in __call__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 122 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 84 in __call__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 122 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 84 in __call__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 122 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 84 in __call__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 122 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/suite.py", line 84 in __call__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/runner.py", line 176 in run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/main.py", line 256 in runTests
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/main.py", line 95 in __init__
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/unittest/__main__.py", line 18 in <module>
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/execfile.py", line 183 in run_python_file
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/execfile.py", line 113 in run_python_module
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/cmdline.py", line 627 in do_run
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/cmdline.py", line 483 in command_line
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/cmdline.py", line 756 in main
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/site-packages/coverage/__main__.py", line 8 in <module>
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/runpy.py", line 85 in _run_code
  File "/home/runner/work/traitsui/traitsui/.edm/envs/traitsui-test-3.6-pyside2/lib/python3.6/runpy.py", line 193 in _run_module_as_main
Command '['edm', 'run', '-e', 'traitsui-test-3.6-pyside2', '--', 'python', '-X', 'faulthandler', '-W', 'default', '-m', 'coverage', 'run', '-p', '-m', 'unittest', 'discover', '-v', 'traitsui']' returned non-zero exit status 252.
/usr/bin/bash /home/runner/work/_actions/GabrielBB/xvfb-action/v1/cleanup.sh
No xvfb processes to kill

@rahulporuri
Copy link
Copy Markdown
Contributor Author

@aaronayres35 i'm unable to find the exact issue where we're keeping track of the test failures

@aaronayres35
Copy link
Copy Markdown
Contributor

xref: #1551

Copy link
Copy Markdown
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

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

LGTM

I think some of these examples can be pulled up into examples/demo/StandardEditors, but that can be done separately.
Also a few nit picky comments

Comment thread docs/source/traitsui_user_manual/advanced_view.rst Outdated
#
# Thanks for using Enthought open source!

# tree_editor.py -- Example of a tree editor
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe this should end up in traitsui/examples/demo/StandardEditors eventually. Same with many of the others.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i don't know if there already exists a tree editor (or an array editor) example in "StandardEditors" but yeah, these can be moved there.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

correction - there already exists a "TreeEditor_demo" which is more or less a copy of this file - and the differences need to be reconciled and then this file can probably be removed.

@@ -1,39 +0,0 @@
# Copyright (c) 2007, Enthought, Inc.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like these examples are getting deleted in this PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

nope. this array_editor example got moved to docs\source\traitsui_user_manual\examples\array_editor.py. In some cases, git recognizes that I simply moved the file and made some changes. In other cases, git doesn't recognize this and thinks i deleted some files and created some from scratch - coincidentally with almost the same content.

@@ -0,0 +1,42 @@
# (C) Copyright 2004-2021 Enthought, Inc., Austin, TX
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should almost definitely end up in traitsui/examples/demo/StandardEditors

Comment thread docs/source/traitsui_user_manual/examples/enum_editor.py Outdated
Comment thread docs/source/traitsui_user_manual/examples/enum_editor.py Outdated
Poruri Sai Rahul added 2 commits May 5, 2021 15:44
after creating it first

	modified:   docs/source/traitsui_user_manual/advanced_view.rst
	new file:   docs/source/traitsui_user_manual/examples/default_traits_view2.py
	modified:   docs/source/traitsui_user_manual/examples/enum_editor.py
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.

2 participants