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.

Sign up for GitHub

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

Jump to bottom

Clean up code and match Blender guidelines #6

Open
travnick opened this issue Jun 22, 2013 · 0 comments
Open

Clean up code and match Blender guidelines #6

travnick opened this issue Jun 22, 2013 · 0 comments
Labels
Cleanup

Comments

@travnick
Copy link
Owner

Blender uses PEP 8 for python code.
http://www.python.org/dev/peps/pep-0008/
http://www.blender.org/documentation/blender_python_api_2_69_7/info_best_practice.html

Code has to be cleaned up and refactored.
Long functions should be split up names must be reviewed and changed.
@ghost ghost assigned travnick Jun 22, 2013
travnick added a commit that referenced this issue Jun 22, 2013
@travnick
[#6] Code clean up in area of hotfix.
abfd1f5
travnick added a commit that referenced this issue Jul 13, 2013
@travnick
[#6] Import organising, added self in several places.
53afd0a
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Import organising, added self in several places.
c3c3b3f
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] extracted __export_library_controllers from export
09d455b
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Deleted commented out code.
89f52a4
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Extracted doc as class privete field.
4e8f308
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] export function splitted up. Fixed up 'self' param issue.
e916032
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Extracted __export_library_visual_scenes from export()
020ff35
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Code cleaning, veriable renames.
d946fee
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Extracted __export_library_animation_clips_and_animations from e…
3ea4666 
…xport.
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Extracted __export_library_effects and __export_library_materials
c38e62d
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Exctracted __export_library_geometries and __export_scene
b09e55f
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] God export method splitted up. Small bugfixes.
c7f245f
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Deleted commented out code, unused variables, small refactoring.
b046157
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Removed unecessary "if" statement
c2033e9
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] Removed unnecessary len(), refactored name generation for IDREF.
32f522d
travnick added a commit that referenced this issue Jul 20, 2013
@travnick
[#6] variables renamed, removed statements without effect.
8d2c548
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] Code cleanning and refactoring.
714075d
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] Deleted commented out code, unused variables, small refactoring.
c79c716
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] Removed unecessary "if" statement
55c14b1
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] Removed unnecessary len().
e413f4d
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] variables renamed, removed statements without effect.
4bac9e2
travnick added a commit that referenced this issue Jul 24, 2013
@travnick
[#6] Methods moved to match call order.
cd9e6f3
travnick added a commit that referenced this issue Aug 3, 2013
@travnick
[#6] Fixed few PEP-8 issues.
cb766b9
travnick added a commit that referenced this issue Aug 20, 2013
@travnick
[#6] In library_images there are now only images for exported objects.
a2fe2ac
travnick added a commit that referenced this issue Aug 20, 2013
@travnick
[#6] Little simplification of __export_library_effects.
8bf2845 
Now it uses __get_materials_for_selected_objects routine.
travnick added a commit that referenced this issue Aug 20, 2013
@travnick
[#6] Variables renamed
69ce243
travnick added a commit that referenced this issue Sep 2, 2013
@travnick
[#6] Code formatted. Deleted unused vairiable.
971bf6a
travnick added a commit that referenced this issue Sep 20, 2013
@travnick
[#6] Removed unused and commented out code.
5578d73
Stardidi added a commit that referenced this issue Jan 3, 2014
@Stardidi
[#28] Automatically invert green channel on _ddn images [#6] Improved…
04c875b 
… handling of already saved .tif images
Stardidi added a commit that referenced this issue Jan 3, 2014
@Stardidi
[#28] Automatically invert green channel on _ddn images [#6] Improved…
68a528c 
… handling of already saved .tif images
travnick added a commit that referenced this issue Jan 10, 2014
@travnick
[#6] Fixed blender version in bl_info bug. dae file now contains time…
5b48310 
… of creation.
dwhite5914 pushed a commit that referenced this issue Jun 4, 2014
[#46][#19][#6]Thorough code cleanup of init.py
e400112 
- Operators named with Blender convention.
- Classes/Function renamed for readability/clarity.
- String joining performance fixed.
- Reorganization for flow/Broke up code.
- Deleted some old and irrelevant comments.
- General code cleanup.
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Some more code cleanup of init.py
856bbb6 
Some more code cleanup...
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Even more code cleanup of init.py
1570b8e 
- changed class names with '_' in names to fit StudlyCaps convention.
- functions named with joined_lower convention.
- fixed some minor naming inconsistencies.
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Thorough code cleanup of add.py
110f59a 
- Classes/Functions renamed for readability/clarity.
- String joining performance fixed.
- Reorganization for flow/Broke up code.
- Deleted some old and irrelevant comments.
- General code cleanup.
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Some small tweaks
2975f3a 
- General code cleanup.
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Thorough code cleanup of export.py
2c61b12 
- Classes/Functions renamed for readability/clarity.
- String joining performance fixed.
- Reorganization for flow/Broke up code.
- Deleted some old and irrelevant comments.
- General code cleanup.
- Spelling/Grammar.
- Small correction to dds_converter.py
dwhite5914 pushed a commit that referenced this issue Jun 16, 2014
[#46][#19][#6]Small Correction
f59e6c0 
Tested this and it didn't work so I've reverted to the old code which
does work.
travnick added a commit that referenced this issue Jun 17, 2014
@travnick
Merge pull request #52 from travnick/feature/code_cleanup
30d87fb 
[#46][#19][#6]Thorough code cleanup of init.py, add.py, export.py, and small tweaks to the others
dwhite5914 added a commit that referenced this issue Mar 9, 2015
@dwhite5914
[#96][#98][#6]PEP 8 Cleanup
25bd181
dwhite5914 added a commit that referenced this issue Feb 14, 2016
@dwhite5914
[#6][#124]General Cleanup
d7d3904
dwhite5914 pushed a commit that referenced this issue Feb 14, 2016
Merge pull request #140 from travnick/feature/autopep8_cleanup
e19149d 
[#6][#124]General Cleanup
dwhite5914 added a commit that referenced this issue Feb 14, 2016
@dwhite5914
[#6][#124]Remove deprecated features
85da48c 
- Material naming is now automatic.
dwhite5914 pushed a commit that referenced this issue Feb 14, 2016
Merge pull request #141 from travnick/feature/remove_deprecated_features
cd93bba 
[#6][#124]Remove deprecated features
dwhite5914 added a commit that referenced this issue Feb 16, 2016
@dwhite5914
[#6][#98][#124]Improved Robustness and Error Handling
28ee95d 
- Removed outdated scripting module.
dwhite5914 pushed a commit that referenced this issue Feb 16, 2016
Merge pull request #145 from travnick/feature/improve_robustness
be5f200 
[#6][#98][#124]Improved Robustness and Error Handling
dwhite5914 added a commit that referenced this issue Mar 13, 2016
@dwhite5914
[#6]Bone geometry utils
f086be9 
Makes bone geometry and physics code bits easier to read.
dwhite5914 added a commit that referenced this issue Mar 13, 2016
@dwhite5914
[#6] Small correction
c332c8a
dwhite5914 added a commit that referenced this issue Mar 13, 2016
@dwhite5914
[#6][#98]Function cleanup
c1ceae1
dwhite5914 added a commit that referenced this issue Mar 14, 2016
@dwhite5914
[#6][#98]Function cleanup fixes
d055078
dwhite5914 pushed a commit that referenced this issue Mar 15, 2016
Merge pull request #175 from travnick/feature/function_cleanup
2e9c4d8 
[#6][#98]Function cleanup
dwhite5914 added a commit that referenced this issue Mar 19, 2016
@dwhite5914
[#6][#98]Small Change
9bcde81
dwhite5914 pushed a commit that referenced this issue Mar 19, 2016
Merge pull request #179 from travnick/feature/function_cleanup
717fe1d 
[#6][#98]Small Change
feedbackex added a commit to feedbackex/CryBlend that referenced this issue Jun 25, 2016
@feedbackex
pep8 for 5.2 travnick#6
ff0a390
@dwhite5914 dwhite5914 changed the title Clean up the code and match Blender guideline. Clean up code and match Blender guidelines Aug 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Cleanup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@travnick @SsoJz4frBm9 @dwhite5914