oemof / tespy

Thermal Engineering Systems in Python (TESPy). This package provides a powerful simulation toolkit for thermal engineering plants such as power plants, district heating systems or heat pumps.
https://tespy.readthedocs.io
MIT License
272 stars 85 forks source link

New feature: self documenting models #243

Closed fwitte closed 3 years ago

fwitte commented 3 years ago

This PR introduces self documenting TESPy models. The goal is facilitating documentation of your TESPy models and to improve transparency of the documentation of such models. The feature will be shipped with the upcoming release but will still be in development.

pep8speaks commented 3 years ago

Hello @fwitte! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 27:80: E501 line too long (88 > 79 characters) Line 28:80: E501 line too long (88 > 79 characters)

Line 35:80: E501 line too long (96 > 79 characters) Line 36:80: E501 line too long (106 > 79 characters) Line 37:80: E501 line too long (95 > 79 characters) Line 38:80: E501 line too long (101 > 79 characters) Line 42:80: E501 line too long (93 > 79 characters) Line 43:80: E501 line too long (89 > 79 characters)

Line 37:80: E501 line too long (96 > 79 characters) Line 38:80: E501 line too long (106 > 79 characters) Line 39:80: E501 line too long (108 > 79 characters) Line 40:80: E501 line too long (114 > 79 characters) Line 44:80: E501 line too long (106 > 79 characters) Line 45:80: E501 line too long (102 > 79 characters)

Line 41:80: E501 line too long (90 > 79 characters) Line 43:80: E501 line too long (94 > 79 characters) Line 44:80: E501 line too long (106 > 79 characters) Line 45:80: E501 line too long (95 > 79 characters) Line 46:80: E501 line too long (99 > 79 characters) Line 47:80: E501 line too long (93 > 79 characters) Line 48:80: E501 line too long (92 > 79 characters) Line 49:80: E501 line too long (92 > 79 characters) Line 50:80: E501 line too long (95 > 79 characters) Line 54:80: E501 line too long (93 > 79 characters) Line 55:80: E501 line too long (89 > 79 characters) Line 56:80: E501 line too long (87 > 79 characters) Line 57:80: E501 line too long (87 > 79 characters)

Line 27:80: E501 line too long (81 > 79 characters)

Line 39:80: E501 line too long (90 > 79 characters) Line 49:80: E501 line too long (95 > 79 characters)

Line 21:80: E501 line too long (81 > 79 characters) Line 40:80: E501 line too long (98 > 79 characters) Line 47:80: E501 line too long (102 > 79 characters) Line 49:80: E501 line too long (82 > 79 characters) Line 50:80: E501 line too long (80 > 79 characters) Line 51:80: E501 line too long (89 > 79 characters)

Line 28:80: E501 line too long (98 > 79 characters) Line 29:80: E501 line too long (96 > 79 characters) Line 33:80: E501 line too long (102 > 79 characters) Line 34:80: E501 line too long (86 > 79 characters) Line 35:80: E501 line too long (91 > 79 characters) Line 36:80: E501 line too long (89 > 79 characters) Line 37:80: E501 line too long (89 > 79 characters)

Line 18:80: E501 line too long (81 > 79 characters) Line 38:80: E501 line too long (98 > 79 characters) Line 42:80: E501 line too long (102 > 79 characters) Line 43:80: E501 line too long (86 > 79 characters) Line 44:80: E501 line too long (91 > 79 characters) Line 45:80: E501 line too long (89 > 79 characters) Line 46:80: E501 line too long (89 > 79 characters)

Line 38:80: E501 line too long (81 > 79 characters) Line 39:80: E501 line too long (83 > 79 characters) Line 51:80: E501 line too long (111 > 79 characters) Line 52:80: E501 line too long (108 > 79 characters) Line 53:80: E501 line too long (105 > 79 characters) Line 54:80: E501 line too long (110 > 79 characters) Line 306:80: E501 line too long (110 > 79 characters) Line 307:80: E501 line too long (119 > 79 characters)

Line 37:80: E501 line too long (111 > 79 characters) Line 38:80: E501 line too long (108 > 79 characters) Line 39:80: E501 line too long (100 > 79 characters)

Line 37:80: E501 line too long (111 > 79 characters) Line 38:80: E501 line too long (108 > 79 characters) Line 39:80: E501 line too long (98 > 79 characters)

Line 32:80: E501 line too long (85 > 79 characters) Line 33:80: E501 line too long (94 > 79 characters) Line 34:80: E501 line too long (93 > 79 characters) Line 356:80: E501 line too long (80 > 79 characters)

Line 25:80: E501 line too long (85 > 79 characters) Line 26:80: E501 line too long (94 > 79 characters) Line 27:80: E501 line too long (93 > 79 characters)

Line 29:80: E501 line too long (111 > 79 characters) Line 30:80: E501 line too long (108 > 79 characters) Line 31:80: E501 line too long (105 > 79 characters) Line 32:80: E501 line too long (110 > 79 characters)

Line 39:80: E501 line too long (90 > 79 characters) Line 40:80: E501 line too long (94 > 79 characters) Line 41:80: E501 line too long (101 > 79 characters) Line 42:80: E501 line too long (99 > 79 characters) Line 43:80: E501 line too long (100 > 79 characters) Line 52:80: E501 line too long (88 > 79 characters) Line 53:80: E501 line too long (93 > 79 characters) Line 54:80: E501 line too long (89 > 79 characters) Line 55:80: E501 line too long (112 > 79 characters)

Line 41:80: E501 line too long (94 > 79 characters) Line 42:80: E501 line too long (81 > 79 characters) Line 43:80: E501 line too long (86 > 79 characters) Line 44:80: E501 line too long (84 > 79 characters)

Line 37:80: E501 line too long (94 > 79 characters)

Line 38:80: E501 line too long (94 > 79 characters) Line 40:80: E501 line too long (80 > 79 characters)

Line 33:80: E501 line too long (94 > 79 characters)

Line 2092:80: E501 line too long (81 > 79 characters) Line 2299:80: E501 line too long (88 > 79 characters)

Line 20:80: E501 line too long (81 > 79 characters)

Line 326:80: E501 line too long (80 > 79 characters) Line 338:80: E501 line too long (80 > 79 characters)

Comment last updated at 2021-01-27 09:42:54 UTC
fwitte commented 3 years ago
lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging 81145ae302f9e703ff4a9faa5c9d94588152f13f into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 14 alerts and fixes 1 when merging 01abd9aea11133402a313f72e3d63e18513dadef into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 20 alerts and fixes 1 when merging 75a303494bb2c4ddf7fa58d9bc0b3875c9b86682 into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 33 alerts and fixes 1 when merging b604ae398fa4ae636edaf7a4261d1d3ccb247530 into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 4 alerts and fixes 1 when merging 2eea6adf6b4663b854ba6b862fea13b6fd2faf9a into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 3 alerts and fixes 1 when merging d918f10de55b14f81a20668e9d234bd392f4d8e7 into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 3 alerts and fixes 1 when merging 0b51ae0ef757589221c73f8b34a04cec738b6fa7 into 4aecead1d7e3c0ac743d9e383f2af72c6d8f6ee6 - view on LGTM.com

new alerts:

fixed alerts: