replace various uses of math.sqrt with more concise math.hypot #40416

New Issue

Lawrence D'Oliveiro · 2014-05-29T04:14:36+02:00

2014-05-29 04:14:36 +02:00

This patch replaces long-winded calls of the form

math.sqrt(x * x + y * x)

with the more concise

math.hypot(x, y)

Also in the Node Tools addon, I noticed a couple of instances of

sqrt(x * x)

which could be written more concisely as

abs(x)

sqrt-hypot.patch

This patch replaces long-winded calls of the form ``` math.sqrt(x * x + y * x) ``` with the more concise ``` math.hypot(x, y) ``` Also in the Node Tools addon, I noticed a couple of instances of ``` sqrt(x * x) ``` which could be written more concisely as ``` abs(x) ``` [sqrt-hypot.patch](https://archive.blender.org/developer/F91788/sqrt-hypot.patch)

Lawrence D'Oliveiro commented

2014-05-29 04:14:36 +02:00

Changed status to: 'Open'

Lawrence D'Oliveiro commented

2014-05-29 04:14:36 +02:00

Added subscriber: @ldo

Lawrence D'Oliveiro commented

2014-05-29 04:15:44 +02:00

I’m not sure who to assign this to; the module owner list doesn’t mention anybody for addons.

I’m not sure who to assign this to; the [module owner list ](http://wiki.blender.org/index.php/Dev:Doc/Process/Module_Owners/List) doesn’t mention anybody for addons.

Bastien Montagne self-assigned this 2014-05-29 09:11:21 +02:00

Bastien Montagne commented

2014-05-29 09:11:43 +02:00

That’s a patch easy to handle, perfect for me! :)

Bastien Montagne commented

2014-05-29 10:17:22 +02:00

Committed as 97039067da, thanks for the patch!

Committed as 97039067da, thanks for the patch!

Bastien Montagne commented

2014-05-29 10:26:52 +02:00

Changed status from 'Open' to: 'Resolved'

Bastien Montagne closed this issue

2014-05-29 10:26:52 +02:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

replace various uses of math.sqrt with more concise math.hypot #40416