[fix]replacement the abs() with unsigned_abs()

[maldiohead]

abs() may be lead to panic in the debug mode and optimized code will return i64::MIN without a panic. so need use unsigned_abs() in here

Description

The absolute value of i64::MIN cannot be represented as an i64, and attempting to calculate it will cause an overflow. This means that code in debug mode will trigger a panic on this case and optimized code will return i64::MIN without a panic. so
in here need replace abs() with unsigned_abs()

Applicable issues

• fixes #

Additional info (benefits, drawbacks, caveats)

Checklist

• Test coverage for new or modified code paths
• Changelog is updated
• Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)
• New clarity functions have corresponding PR in clarity-benchmarking repo
• New integration test(s) added to bitcoin-tests.yml

[CLAassistant]

All committers have signed the CLA.

[wileyj]

@maldiohead can you rebase against develop and we'll take a look?

[maldiohead]

@wileyj Hey, I have rebase to the develop branch

[jbencin]

We should search and see if abs() is used elsewhere in the code also

[wileyj]

this is interesting - the docker image part of ci (which is a required check before merge) is choking on the parentheses.
docker doesn't like using special chars as the image tag, and since we tag the images with the branch this step is failing.

2 options @maldiohead - you'll either need to adjust your branch name (basically remove the parentheses), or i can poke around the ci workflow, maybe there's substitution i can do on the branch name before building an image.

the first option would be faster to merge this, but the second option is probably needed regardless.