Eric Chen

In ion-python-benchmark-cli, If an input file's data format and the desired benchmark format are the same, the tool will copy the original input file as a temporary file and use...

Benchmark-cli read command should support --format option (format conversion feature).

1

Currently ion-python-benchmark-cli read mode doesn't support `--format` option. For example, given a binary ion file. Command `python-benchmar-cli read --format ion_text` will returns the benchmark time used for text format, and...

SimpleIon READ ERROR, 10 unit tests failed consistently.

4

There are some read issues when using the simpleion module. Example failed CI/CD workflow - https://github.com/amazon-ion/ion-python/actions/runs/4410728439/jobs/7728500690 1 failed unit test within ion-python is: ``` test_loads_unicode_utf8_conversion(): ``` 9 failed tests from...

C extension memory leak.

1

We fixed couple memory leak issues and didn't see this kind of issues recently, but It's possible that underlying C extension still has memory leak. Open this issue in case...

The tool currently use `MB` for `file_size` and `seconds` for `execution time`. For example, other benchmark tools such as ion-java-benchmark-cli uses `ms/op` for the execution time. Need to standardize them.

The benchmark tool generates memory usage peak right now. This issue wants to improve the memory profiling feature by two areas, 1. It would be good to adds more metrics...

Right now, the options output in the output table is unclear. For example, (simpleion, ion_binary, file) represents that `--api` is simpleion, `--format` is ion_binary and `--io-type` is file. This needs...

Why the ion-python C extension cannot work correctly with ion-test-driver?

1

Refer to https://github.com/amzn/ion-test-driver/issues/24, when we use ion-python 0.9.x for ion-test-driver, it throws error > amazon.ion.exceptions.IonException: IERR_INVALID_ARG assertion failed: len == 1 What happened there?

Currently ion-python-benchmark-cli `event based` APIs are basically `pure python implementation dump/load` APIs. So, below two commands are same. ``` python-benchmark-cli read --api event python-benchmark-cli read --api simple_ion --c-extension False ```...

ivmInAnnotationWrapper.10n failed CI/CD with pypy

1

Test file [bad/ivmInAnnotationWrapper.10n](https://github.com/amazon-ion/ion-tests/blob/master/iontestdata/bad/ivmInAnnotationWrapper.10n) failed with `pypy` but works on CPython. ``` ---------- test_basic[BAD - bad/ivmInAnnotationWrapper_10n ] p = @parametrize(*chain( _basic_params(_T.GOOD, _GOOD_SUBDIR), _basic_params(_T.GOOD, _GOOD_TIMESTAMP_SUBDIR), _basic_params(_T.BAD, _BAD_SUBDIR), _basic_params(_T.BAD, _BAD_TIMESTAMP_SUBDIR), _basic_params(_T.BAD, _BAD_UTF8_SUBDIR), ))...