Skip to content

Conversation

nielstron
Copy link
Contributor

Negative values were allowed before as well so it is only consequential to allow subtracting from 0. This simplifies edge case handling for tools that simply want to operate on Values. Is there any inherent rule/implementation in the ledger that lead to the previous error situation?

Negative values were allowed before as well so it is only consequential
to allow subtracting from 0
@nielstron nielstron requested a review from cffls February 21, 2024 12:00
@cffls
Copy link
Collaborator

cffls commented Feb 24, 2024

IIRC, I added the check because I didn't think net negative asset was possible in a finalized output. I am fine with this change. We should also add a check in txbuilder for negative asset if there isn't one today.

@nielstron nielstron merged commit 7db66cf into Python-Cardano:main Mar 2, 2024
@nielstron nielstron mentioned this pull request Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants