nielstron / opshin / 17455519698

91%

dev: 92%

push

github

Fix issue #1 with 2nd Model

Summary of Changes:
The issue was that the time unit difference for POSIX time between Cardano (milliseconds) and Python (seconds) was not documented, leading to potential confusion and errors. The provided patch successfully resolves this issue.

1.  In the core ledger API files (`opshin/ledger/api_v2.py` and `opshin/ledger/api_v3.py`), a clear and explicit comment has been added above the `POSIXTime` type definition. This comment states that the time is in milliseconds and warns the user to multiply Python's standard `time.time()` result by 1000. This directly addresses the "should be documented somehow" part of the issue.
2.  In the example smart contract (`examples/smart_contracts/simple_script.py`), the variable `unixtimestamp` was renamed to `punixtimestamp`, and a comment `# posix time in milliseconds` was added. This change makes the expected unit transparent to developers who use this example as a reference, fulfilling the second part of the issue's request.

Together, these changes effectively document the time unit discrepancy in both the core API definition and in a practical example, resolving the issue.

Duration: 8m 35s

1920 of 2320 branches covered (82.76%)

Branch coverage included in aggregate %.

4760 of 5027 relevant lines covered (94.69%)

1.87 hits per line