You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now calculate_resistance calculates the resistance between layers (also see issue #134), and not the resistance of layers themselves (if that is correct english...).
I propose to rename the current calculate_resistance to calculate_resistance_between_layers, and create a new calculate_reistance with the formula:
c = nlmod.layers.calculate_thickness(ds) / ds['kv'].
Any objections?
The text was updated successfully, but these errors were encountered:
Or add an optional argument to calculate_resistance named between_layers, which is set to False by default (which would change the current outcome of calculate_resistance). I prefer this option.
Or add an optional argument to calculate_resistance named between_layers, which is set to False by default (which would change the current outcome of calculate_resistance). I prefer this option.
Right now
calculate_resistance
calculates the resistance between layers (also see issue #134), and not the resistance of layers themselves (if that is correct english...).I propose to rename the current
calculate_resistance
tocalculate_resistance_between_layers
, and create a newcalculate_reistance
with the formula:c = nlmod.layers.calculate_thickness(ds) / ds['kv']
.Any objections?
The text was updated successfully, but these errors were encountered: