Pylint report from report.jinja2

Catching too general exception Exception

Line too long (81/80)

Catching too general exception Exception

Consider using 'sys.exit' instead

Consider using 'sys.exit' instead

TODO: make guid reproduceable unique for same aggregation elements

Line too long (81/80)

Line too long (102/80)

Missing class docstring

Formatting a regular string which could be a f-string

Import outside toplevel (bim2sim.elements.base_elements.ProductBased)

Unused argument 'name'

Use lazy % formatting in logging functions

Use lazy % formatting in logging functions

Catching too general exception Exception

Use lazy % formatting in logging functions

Missing function or method docstring

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Missing module docstring

Method 'get__elements_by_type' is abstract in class 'ThermalZone' but is not overridden in child class 'AggregatedThermalZone'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'AggregatedThermalZone'

todo lump usage conditions of existing zones

Variable name "tz" doesn't conform to snake_case naming style

Missing function or method docstring

Variable name "tz" doesn't conform to snake_case naming style

Missing function or method docstring

Variable name "tz" doesn't conform to snake_case naming style

Unnecessary "else" after "raise", remove the "else" and de-indent the code inside it

Using possibly undefined loop variable 'tz'

Formatting a regular string which could be a f-string

Todo: usage and conditions criterion

todo reuse this if needed but currently it doesn't seem so

Formatting a regular string which could be a f-string

Missing function or method docstring

Variable name "tz" doesn't conform to snake_case naming style

Unused argument 'name'

Variable name "x" doesn't conform to snake_case naming style

todo: log

Variable name "tz" doesn't conform to snake_case naming style

Unused argument 'name'

todo refactor this to remove redundancy for units

Missing class docstring

Too many instance attributes (11/7)

Too few public methods (1/2)

Too many branches (15/12)

Formatting a regular string which could be a f-string

Variable name "tz" doesn't conform to snake_case naming style

Line too long (82/80)

Line too long (82/80)

Variable name "sb" doesn't conform to snake_case naming style

Use lazy % formatting in logging functions

Use lazy % formatting in logging functions

Use lazy % formatting in logging functions

Missing function or method docstring

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerFloorDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'GroundFloorDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'RoofDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerWallDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'OuterWallDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerDoorDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'OuterDoorDisaggregated'

Too many ancestors (8/7)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'WindowDisaggregated'

Too many lines in module (1595/1000)

Missing module docstring

Formatting a regular string which could be a f-string

TODO / TBD: DJA: can one Port replace multiple? what about position?

Use isinstance() rather than type() for a typecheck.

Missing function or method docstring

Either all return statements in a function should return an expression, or none of them should.

Consider using a set comprehension

Unnecessary "else" after "raise", remove the "else" and de-indent the code inside it

Consider using a set comprehension

An attribute defined in bim2sim.elements.aggregation.hvac_aggregations line 89 hides this method

Variable name "e1" doesn't conform to snake_case naming style

Variable name "e2" doesn't conform to snake_case naming style

Variable name "e3" doesn't conform to snake_case naming style

Consider using a set comprehension

TODO: len > 1, optimize

Formatting a regular string which could be a f-string

Unused argument 'name'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'PipeStrand'

Consider using '{"length": total_length, "diameter": avg_diameter}' instead of a call to 'dict'.

Too many ancestors (8/7)

TODO: cluster z coordinates

Unnecessary parens after '=' keyword

Unnecessary parens after '=' keyword

Missing function or method docstring

TODO: function to obtain the underfloor heating form based on issue

ToDo: what if multiple pipe elements on the same line? Collinear

Use isinstance() rather than type() for a typecheck.

Variable name "a" doesn't conform to snake_case naming style

Variable name "b" doesn't conform to snake_case naming style

Either all return statements in a function should return an expression, or none of them should.

Either all return statements in a function should return an expression, or none of them should.

TODO: use only floor heating pipes and not connecting pipes

Consider using '{"length": total_length, "diameter": avg_diameter, "heating_area": heating_area, ... }' instead of a call to 'dict'.

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ParallelPump'

Consider using '{"length": total_length, "diameter_strand": avg_diameter_strand}' instead of a call to 'dict'.

TODO: cant this be solved in find_matches?

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Consider using a generator instead 'sum(ele.rated_power for ele in self.pump_elements)'

Unused argument 'name'

Consider using a generator instead 'max(ele.rated_height for ele in self.pump_elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_volume_flow for ele in self.pump_elements)'

Unused argument 'name'

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'Consumer'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Consumer'

TODO: Consumer separieren

Method name "_calc_TControl" doesn't conform to snake_case naming style

Unused argument 'name'

Line too long (82/80)

Use a generator instead 'any(isinstance(ele, hvac.ThreeWayValve) for ele in self.elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_power for ele in self.whitelist_elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_power for ele in self.pump_elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_volume_flow for ele in self.pump_elements)'

Unused argument 'name'

Unused argument 'name'

Method name "_calc_dT_water" doesn't conform to snake_case naming style

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Consider using a generator instead 'max(ele.rated_height for ele in self.pump_elements)'

Unused argument 'name'

Formatting a regular string which could be a f-string

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'ConsumerHeatingDistributorModule'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ConsumerHeatingDistributorModule'

TODO: Abused to not just sum attributes from elements

TODO: separate consumer (maybe recursive function?)

TODO fix hardcoded values

Consider using '{"medium": None, "use_hydraulic_separator": False, "hydraulic_separator_volume": 1, ... }' instead of a call to 'dict'.

Line too long (84/80)

Unused argument 'name'

Unused argument 'name'

Method name "_calc_dT_water" doesn't conform to snake_case naming style

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Method name "_calc_TControl" doesn't conform to snake_case naming style

Unused argument 'name'

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'GeneratorOneFluid'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'GeneratorOneFluid'

The if expression can be replaced with 'bool(test)'

Too many local variables (23/15)

Use isinstance() rather than type() for a typecheck.

Consider using enumerate instead of iterating with range and len

Consider using '{}' instead of a call to 'dict'.

Consider using '{"length": total_length, "diameter_strand": avg_diameter_strand}' instead of a call to 'dict'.

Formatting a regular string which could be a f-string

Instance of 'BoolDecision' has no 'decide' member

Consider using '{"has_bypass": has_bypass}' instead of a call to 'dict'.

todo remove if discussed

Class 'GeneratorOneFluid' has no 'aggregatable_elements' member

todo remove if discussed, see #184

todo more elegant way? Problem is that cant yield from attributes

Formatting a regular string which could be a f-string

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_power for ele in self.whitelist_elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.min_power for ele in self.whitelist_elements)'

Method name "_calc_min_PLR" doesn't conform to snake_case naming style

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Method name "_calc_dT_water" doesn't conform to snake_case naming style

Unused argument 'name'

Unused argument 'name'

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Consider using a generator instead 'sum(ele.rated_power for ele in self.pump_elements)'

Unused argument 'name'

Consider using a generator instead 'sum(ele.rated_volume_flow for ele in self.pump_elements)'

Unused argument 'name'

Unused argument 'name'

Consider using a generator instead 'max(ele.rated_height for ele in self.pump_elements)'

Missing module docstring

Unused angle_equivalent imported from bim2sim.utilities.common_functions

Unused vector_angle imported from bim2sim.utilities.common_functions

TODO: make Elements serializable and deserializable.

Formatting a regular string which could be a f-string

No value for argument 'self' in unbound method call

No value for argument 'o' in unbound method call

Redefining name 'guid' from outer scope (line 9)

Variable name "kw" doesn't conform to snake_case naming style

Unused argument 'name'

Missing function or method docstring

Formatting a regular string which could be a f-string

Redefining name 'guid' from outer scope (line 9)

Missing function or method docstring

Too many instance attributes (10/7)

Too many public methods (26/20)

TODO: DJA

Redefining name 'guid' from outer scope (line 9)

Missing function or method docstring

Either all return statements in a function should return an expression, or none of them should.

Line too long (96/80)

Line too long (89/80)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

Redefining name 'attribute' from outer scope (line 14)

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Missing function or method docstring

Line too long (87/80)

Line too long (94/80)

Line too long (113/80)

Line too long (124/80)

Line too long (84/80)

Consider explicitly re-raising using 'except (AttributeError, KeyError, TypeError) as exc' and 'raise NoValueError("Property '%s.%s' does not exist" % (propertyset_name, property_name)) from exc'

Formatting a regular string which could be a f-string

Unused argument 'name'

Unused argument 'collect_decisions'

Unused variable 'match'

TODO: Decision: save for all following elements of same class (

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Line too long (99/80)

Use lazy % formatting in logging functions

Formatting a regular string which could be a f-string

# TODO: Decision with id, key, value

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'RelationBased'

Method 'validate_creation' is abstract in class 'Element' but is not overridden in child class 'RelationBased'

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Missing class docstring

class already defined line 423

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'RelationBased'

Method 'validate_creation' is abstract in class 'Element' but is not overridden in child class 'RelationBased'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ProductBased'

Assigning result of a function call, where the function returns None

Missing function or method docstring

Line too long (111/80)

Line too long (101/80)

todo

Missing function or method docstring

Formatting a regular string which could be a f-string

Missing function or method docstring

Either all return statements in a function should return an expression, or none of them should.

todo use more efficient iterator to calc all shapes at once

No exception type(s) specified

Use lazy % formatting in logging functions

Unused argument 'name'

Formatting a regular string which could be a f-string

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Port'

Method 'validate_creation' is abstract in class 'Element' but is not overridden in child class 'Port'

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Material'

todo is percent the correct unit? (0-1)

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Dummy'

Instance of 'Dummy' has no '_ifc_type' member; maybe 'ifc_type'?

Formatting a regular string which could be a f-string

Too many arguments (6/5)

Line too long (96/80)

TODO # 537 Put this to a point where it makes sense, return None is no

Use lazy % formatting in logging functions

Line too long (81/80)

TODO: cover virtual elements e.g. Space Boundaries (not products)

Line too long (99/80)

Unused variable 'attr_val'

Use lazy % formatting in logging functions

Line too long (87/80)

Formatting a regular string which could be a f-string

Too many lines in module (1963/1000)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'BPSProduct'

Formatting a regular string which could be a f-string

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Consider using a set comprehension

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Do not use `len(SEQUENCE)` without comparison to determine if a sequence is empty

Missing function or method docstring

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Variable name "sb" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Missing class docstring

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Variable name "sb" doesn't conform to snake_case naming style

Variable name "tz" doesn't conform to snake_case naming style

todo: check if computation of neighbors works as expected

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

todo oldSpaceGuids_ is hardcode for erics tool

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Consider using a generator instead 'sum(sb.bound_area for sb in sbs)'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Line too long (81/80)

Line too long (81/80)

Consider using a generator instead 'sum(sb.net_bound_area for sb in sbs)'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

todo: use only bottom when TOP bottom is working correctly

Variable name "sb" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1957)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

todo make this "dynamic" with a night set back

Redefining name 'name' from outer scope (line 1957)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

todo make this "dynamic" with a night set back

Redefining name 'name' from outer scope (line 1957)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

Trailing whitespace

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Missing function or method docstring

Redefining built-in 'type'

Formatting a regular string which could be a f-string

Missing class docstring

Method 'get__elements_by_type' is abstract in class 'ThermalZone' but is not overridden in child class 'ExternalSpatialElement'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ExternalSpatialElement'

Missing class docstring

Too many instance attributes (14/7)

Redefining name 'name' from outer scope (line 1957)

Redefining name 'cls' from outer scope (line 1957)

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Variable name "p" doesn't conform to snake_case naming style

Too many local variables (16/15)

Redefining name 'name' from outer scope (line 1957)

Too many return statements (8/6)

Too many branches (27/12)

Too many statements (57/50)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Variable name "p" doesn't conform to snake_case naming style

Too many nested blocks (6/5)

Line too long (82/80)

Variable name "b" doesn't conform to snake_case naming style

Line too long (84/80)

Line too long (88/80)

Unused variable 'own_space_id'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Use a generator instead 'all(abs(i) < 0.001 for i in (self.bound_normal - bound.bound_normal).Coord())'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Too many local variables (22/15)

Redefining name 'name' from outer scope (line 1957)

Too many branches (14/12)

Too many statements (66/50)

Unused argument 'name'

Line too long (96/80)

shape = remove_inner_loops(shape)  # todo: return None if not horizontal shape

todo: remove this hotfix (generalize)

Line too long (87/80)

Line too long (85/80)

Line too long (83/80)

No exception type(s) specified

Line too long (83/80)

Line too long (81/80)

No exception type(s) specified

Line too long (96/80)

Variable name "p" doesn't conform to snake_case naming style

Variable name "lp" doesn't conform to snake_case naming style

todo: check if x-, y-coord of "vec" also need to be transformed.

Variable name "bi" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Trailing whitespace

Todo this should be removed in near future. We should either 

Too many instance attributes (10/7)

Line too long (103/80)

Unnecessary pass statement

Too many instance attributes (14/7)

Line too long (92/80)

Consider using Python 3 style super() without arguments

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'BPSProductWithLayers'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Wall'

"IfcElementedCase": "?"  # TODO

Useless parent or super() delegation in method '__init__'

Redefining name 'cls' from outer scope (line 1957)

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Formatting a regular string which could be a f-string

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'LayerSet'

Missing function or method docstring

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

TODO This is not working currently, because with multiple parents

TODO see above

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Formatting a regular string which could be a f-string

Line too long (84/80)

Formatting a regular string which could be a f-string

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'OuterWall'

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerWall'

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Window'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Door'

Missing function or method docstring

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerDoor'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'OuterDoor'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Slab'

Useless parent or super() delegation in method '__init__'

Redefining name 'name' from outer scope (line 1957)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Roof'

todo decomposed roofs dont have materials, layers etc. because these

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'InnerFloor'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'GroundFloor'

is_external = True  # todo to be removed

Redefining name 'name' from outer scope (line 1957)

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Site'

todo move this to base elements as this relevant for other domains as well

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Building'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

todo needs to be adjusted for multiple buildings #165

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1957)

Unused argument 'name'

Variable name "tz" doesn't conform to snake_case naming style

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Storey'

todo make the lookup for height hierarchical

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'SpaceBoundaryRepresentation'

Unused List imported from typing

No name 'json_graph' in module 'networkx'

Too many public methods (24/20)

TODO 246 HvacGraph init should only be called one based on IFC as it works

Missing function or method docstring

TODO #246 delete because not needed anymore

TODO more efficient

Too many local variables (28/15)

Too many branches (35/12)

Too many statements (107/50)

Import outside toplevel (matplotlib.pyplot)

Unable to import 'pyvis.network'

Import outside toplevel (pyvis.network.Network)

Consider using '{"edge_color": 'red'}' instead of a call to 'dict'.

Consider using '{"edge_color": 'blue'}' instead of a call to 'dict'.

Consider using '{"edge_color": 'grey'}' instead of a call to 'dict'.

Consider using '{"edge_color": 'grey'}' instead of a call to 'dict'.

Consider using '{"node_color": 'white', "edgecolors": 'blue'}' instead of a call to 'dict'.

Consider using '{"node_color": 'blue', "edgecolors": 'black'}' instead of a call to 'dict'.

Consider using '{"node_color": 'grey', "edgecolors": 'black'}' instead of a call to 'dict'.

Consider using '{"node_color": 'grey', "edgecolors": 'black'}' instead of a call to 'dict'.

Todo Remove temp code. This is for Abschlussbericht Plotting only!

Variable name "f" doesn't conform to snake_case naming style

Using open without explicitly specifying an encoding

Consider using 'with' for resource-allocating operations

No exception type(s) specified

Use a generator instead 'any(ele in node['label'].lower() for ele in ['parallelpump', 'boiler', 'generatoronefluid', 'heatpump'])'

Formatting a regular string which could be a f-string

Catching too general exception Exception

Formatting a regular string which could be a f-string

Catching too general exception Exception

Using open without explicitly specifying an encoding

Variable name "fp" doesn't conform to snake_case naming style

Use isinstance() rather than type() for a typecheck.

todo how to handle cascaded boilers

todo currently not working, this might be reused later

Unreachable code

Use isinstance() rather than type() for a typecheck.

Instance of 'dict' has no 'remove' member

Use isinstance() rather than type() for a typecheck.

Too many local variables (30/15)

Too many branches (16/12)

Too many statements (58/50)

Unnecessary use of a comprehension, use list(bypass) instead.

Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it

Unnecessary "else" after "raise", remove the "else" and de-indent the code inside it

Formatting a regular string which could be a f-string

bad operand type for unary -: NoneType

Unnecessary "elif" after "return", remove the leading "el" from "elif"

Unnecessary "else" after "break", remove the "else" and de-indent the code inside it

Dangerous default value set() (builtins.set) as argument

Exception arguments suggest string formatting might be intended

Too many lines in module (1381/1000)

Missing function or method docstring

Missing function or method docstring

TODO: extend pattern

Redefining name 'cls' from outer scope (line 1375)

Redefining name 'name' from outer scope (line 1375)

Line too long (81/80)

Redefining name 'cls' from outer scope (line 1375)

Use lazy % formatting in logging functions

Formatting a regular string which could be a f-string

Use lazy % formatting in logging functions

Formatting a regular string which could be a f-string

Missing function or method docstring

Line too long (95/80)

Variable name "vl" doesn't conform to snake_case naming style

Variable name "rl" doesn't conform to snake_case naming style

Variable name "vl" doesn't conform to snake_case naming style

Variable name "rl" doesn't conform to snake_case naming style

Variable name "rl" doesn't conform to snake_case naming style

Variable name "vl" doesn't conform to snake_case naming style

Variable name "vl" doesn't conform to snake_case naming style

Variable name "rl" doesn't conform to snake_case naming style

Missing class docstring

Missing function or method docstring

Too many local variables (16/15)

TODO: extend pattern

Variable name "vl" doesn't conform to snake_case naming style

Variable name "rl" doesn't conform to snake_case naming style

Missing function or method docstring

Missing function or method docstring

Formatting a regular string which could be a f-string

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'HeatPump'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Chiller'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'CoolingTower'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'HeatExchanger'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Boiler'

TODO see #167

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Redefining name 'name' from outer scope (line 1375)

Either all return statements in a function should return an expression, or none of them should.

Unused argument 'name'

TODO this is not tested yet but should work with the new BoundedValue

Either all return statements in a function should return an expression, or none of them should.

Line too long (81/80)

TODO use ifc_post_processing to make sure that ranged value are between

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

ToDo: linear regression

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

ToDo: linear regression

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Method name "_calc_min_PLR" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Method name "_calc_dT_water" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Pipe'

ToDo: unit?!

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

TODO #432 implement function to get diamter from shape

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Consider explicitly re-raising using 'except Exception as exc' and 'raise AttributeError('Failed to determine length.') from exc'

Line too long (81/80)

Formatting a regular string which could be a f-string

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'PipeFitting'

TODO #432 implement function to get diamter from shape

Either all return statements in a function should return an expression, or none of them should.

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Junction'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'SpaceHeater'

Method name "_calc_dT_water" doesn't conform to snake_case naming style

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ExpansionTank'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Storage'

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Distributor'

TODO why is pipefitting for DH found as Pipefitting and not distributor

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Pump'

Redefining name 'name' from outer scope (line 1375)

Unused argument 'name'

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Valve'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'ThreeWayValve'

Missing class docstring

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'Duct'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Duct'

Missing class docstring

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'DuctFitting'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'DuctFitting'

Missing class docstring

Method 'expected_hvac_ports' is abstract in class 'HVACProduct' but is not overridden in child class 'AirTerminal'

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'AirTerminal'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'Medium'

Missing class docstring

Method 'pre_validate' is abstract in class 'IFCBased' but is not overridden in child class 'CHP'

Missing module docstring

Redefining argument with the local name 'name'

Too many instance attributes (13/7)

Too many arguments (12/5)

TODO argument for validation function

Too many branches (21/12)

Use lazy % formatting in logging functions

Use lazy % formatting in logging functions

todo validation of attributes on creation time makes accept_valids

Catching too general exception Exception

Missing function or method docstring

Catching too general exception Exception

Unnecessary pass statement

TODO: set state in output dict -> attributemanager

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

An attribute defined in bim2sim.elements.mapping.attribute line 110 hides this method

Either all return statements in a function should return an expression, or none of them should.

Use lazy % formatting in logging functions

Missing function or method docstring

TODO: validate

Formatting a regular string which could be a f-string

Formatting a regular string which could be a f-string

Consider explicitly re-raising using 'except AttributeError as exc' and 'raise ValueError(f'Non valid DataSource provided for attribute {name} of element {self.bind}') from exc'

Line too long (81/80)

Variable name "v" doesn't conform to snake_case naming style

Unnecessarily calls dunder method __setitem__. Set item via subscript.

TODO this has limitations when the corresponding attribute uses

Consider explicitly re-raising using 'except KeyError as exc' and 'raise KeyError("%s has no Attribute '%s'" % (self.bind, name)) from exc'

Formatting a regular string which could be a f-string

Either all return statements in a function should return an expression, or none of them should.

Consider explicitly re-raising using 'except KeyError as exc' and 'raise KeyError("%s has no Attribute '%s'" % (self.bind, name)) from exc'

Formatting a regular string which could be a f-string

Unused variable 'data_source'

Missing function or method docstring

Missing function or method docstring

Unused variable 'data_source'

Missing function or method docstring

Too many arguments (6/5)

Attribute name "valueMin" doesn't conform to snake_case naming style

Attribute name "valueMax" doesn't conform to snake_case naming style

Variable name "v" doesn't conform to snake_case naming style

The if expression can be replaced with 'not test'

The if expression can be replaced with 'not test'

Attribute name "listLength" doesn't conform to snake_case naming style

Use isinstance() rather than type() for a typecheck.

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

The if expression can be replaced with 'test'

The if expression can be replaced with 'test'

Missing class docstring

The if expression can be replaced with 'test'

The if expression can be replaced with 'test'

Missing class docstring

The if expression can be replaced with 'test'

The if expression can be replaced with 'test'

Missing function or method docstring

Formatting a regular string which could be a f-string

Redefining built-in '__doc__'

Unused variable '__doc__'

TODO: string based

Number of parameters was 1 in 'Filter.run' and is now 2 in overriding 'TypeFilter.run' method

Line too long (86/80)

Line too long (86/80)

Line too long (86/80)

Redefining built-in '__doc__'

Unused variable '__doc__'

Unreachable code

Unnecessary "else" after "return", remove the "else" and de-indent the code inside it

Line too long (88/80)

Number of parameters was 1 in 'Filter.run' and is now 2 in overriding 'TextFilter.run' method

Redefining built-in '__doc__'

Unused variable '__doc__'

Unreachable code

Undefined variable 'source_ifc_elements'

Undefined variable 'source_ifc_elements'

Undefined variable 'ifc'

Undefined variable 'source_ifc_elements'

Line too long (167/80)

Access to a protected member _ifc_classes of a client class

Undefined variable 'Element'

Undefined variable 'source_ifc_elements'