Rob Marrowstone

Updates the test to verify that imported with undefined text are not equivalent to: other imported symbols with undefined text and different SIds, local symbols with undefined text, or symbols...

This change fixes a bug in the Ion TextBinary conversion where an Ion stream of multiple top-level values would be transcoded as an Ion list. In the process of fixing...

Symbols lack SIds when C-extension is enabled

2

What I expect: ``` >>> from amazon.ion.simpleion import loads >>> loads('$1') IonPySymbol(text='$ion', sid=1, location=ImportLocation(name='$ion', position=1)) >>> >>> >>> ``` What happens: ``` >>> from amazon.ion.simpleion import loads >>> loads('$1') IonPySymbol(text='$ion',...

Many of the calls to the Python C API in ioncmodule are not checked for NULL return values, which would indicate an error, for example in `ionc_add_to_container`. For the most...

Clean Code to Remove Python 2 bits

3

Python2 has been past EOL for almost 3 years (was Jan 1 2020, time of writing Dec 1 2022). This library has not supported it for some time now. There...

__repr__ for s-exp is misleading

1

The __repr__ for s-expressions is indistinguishable from that of lists: ``` >>> simpleion.loads('()') [] ```

The __repr__ for an object is expected to return an unambiguous representation that may be used to reconstruct an instance of the object. Annotations are currently not part of the...

When loading a large decimal with the c-extension (which has a limited decimal size), if the decimal is too large the error thrown is not helpful: ``` >>> simpleion.loads('49.89999999999999857891452847979962825775146484375') Traceback...

There is currently no description for this package in pypi.

Using ion-java 1.9.6 (scala) ``` "it" should "provide helpful error with malformed timestamp" in { ionSystem.singleValue("2019-01-01T00:00:00.000") } ``` Throws an error (as expected), but the message is misleading: ``` a...