Function with "ActiveCell.Offset(x, y).value" not calculating cells automatically?

You should be using Application.Caller rather than ActiveCell.

If val1, val2 & valOffset are variables assigned to cells you should pass them to the UDF as Ranges rather than Integers.

edit: below is not accurate... in my defence it's late
If declared as Ranges they will become precedents of the UDF and as a result when altered they will cause the UDF to recalculate negating need for UDF to be made volatile.

Please Login or Register  to view this content.

edit: below incorrect - covered by romperstompers post below re: indirect reference by virtue of Offset (which if changed would not cause recalc)
If the variables are constants and can not be tied to Ranges you will need to make the UDF volatile so as to minimise risk of calculation error - this brings with it additional overhead.

Of course whether or not you need the UDF in the first instance is open to debate.

No. Volatile UDFs recalculate whenever any calculation takes place, even if it does not affect them, which is why they should be avoided if possible. You need to make your UDF volatile for it to work properly because you effectively have an indirect reference to another cell.

Thanks R.

@skywalker, a lot of what I said re: Volatility should be disregarded in the context of your UDF - I've amended my post to that effect (ie added comments where talking nonsense)

Yes - as I said, you need to make your function volatile. You can do this by adding:

Please Login or Register  to view this content.

as the first line inside the function code.