WIP:Pymaketop

[MTLehner]

Description

• restructuring forcefield class in pygromos and making a generic-ff class from which others inherit to remove code duplication and make the usage of ffs more clear.
• adding a python version of make_top to the gromosff class to create topologies in python

Todos

a lot

[lgtm-com[bot]]

This pull request introduces 33 alerts and fixes 104 when merging 7b758e2e8d3553c4183a9ad37d85acebc9ed2711 into 3bb489f7f9c94f3323307c23a435015ad8bd9473 - view on LGTM.com

new alerts:

• 14 for `__eq__` not overridden when adding attributes
• 7 for Unused local variable
• 4 for Modification of parameter with default
• 2 for Variable defined multiple times
• 2 for Wrong number of arguments in a call
• 1 for Unnecessary pass
• 1 for Suspicious unused loop iteration variable
• 1 for Module is imported with 'import' and 'import from'
• 1 for `__init__` method calls overridden method

fixed alerts:

• 24 for Unused local variable
• 21 for Unused import
• 20 for Testing equality to None
• 18 for Except block handles 'BaseException'
• 9 for 'import *' may pollute namespace
• 4 for Modification of parameter with default
• 2 for `__eq__` not overridden when adding attributes
• 2 for Unreachable code
• 2 for Variable defined multiple times
• 1 for Module is imported more than once
• 1 for Suspicious unused loop iteration variable

[lgtm-com[bot]]

This pull request introduces 3 alerts when merging 7b758e2e8d3553c4183a9ad37d85acebc9ed2711 into 7e3a9f4a7d8a9520ad784697b45b0d90dcdf6c2e - view on LGTM.com

new alerts:

• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

[lgtm-com[bot]]

This pull request introduces 3 alerts when merging dc2acad69fdd73feea98bbe04576a7cfd55329b2 into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging a58de3f99ab7b906e25897899de931572b82c87e into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging 51c602ec79e75b0ffa4061d045dde07523ebe841 into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging a1c0d0508e64d48478ad6a565c60dbd2aa29ea7f into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging 52af8504849bf47a002b6679ca2f75fedbaab611 into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging cdb9e066578312604415608f09d859a9406f640c into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 5 alerts and fixes 1 when merging 842bcd15c2f3114b7d777fc5b2e953bd140c4bb6 into 2453683ea340eacf610e1b00ae48e0df5b109532 - view on LGTM.com

new alerts:

• 2 for Wrong name for an argument in a class instantiation
• 1 for Unused local variable
• 1 for Suspicious unused loop iteration variable
• 1 for `__init__` method calls overridden method

fixed alerts:

• 1 for Unused local variable