Agisilaos Kounelis

Results 31 comments of Agisilaos Kounelis

@eddelbuettel if you see my last commit, we were basically never testing col sizes greater than zero. My change cause the test to fail with the message: ``` WriterBase: Invalid...

After https://github.com/TileDB-Inc/TileDB-Py/pull/1959, this should be resolved.

> LGTM but please try setuptools_scm before merging so we don't have to update manually. Some other changes were made because newer versions of setuptools and setuptools_scm are needed. For...

> Overall LGTM, but could you please do a test build by pushing this branch as `release-test-sc35414`? Then after ~1hr you should have wheels available [here](https://dev.azure.com/TileDB-Inc/CI/_build?definitionId=1) (just invited you to...

> LGTM, just make sure that you can still produce wheel, so please: > > 1. Create and install wheel > 2. Create and install sdist > 3. In all...

Error comes from libtiledb. Should we address it there? `FAILED tiledb/tests/test_query_condition.py::QueryConditionTest::test_qc_enumeration - tiledb.cc.TileDBError: TileDB internal: Enumeration value not found for field 'attr2'`

It fails without 2.21 (or at least with 2.20...) --- ``` git checkout viviannguyen/sc-38443/dataframe-value-filter-on-enum-dict-column pip install -r requirements_dev.txt python setup.py develop python -c "import tiledb; print(tiledb.libtiledb.version())" ``` **(2, 21, 0)**...

Also tested it with 2.21.0 (not pre-release) and it works.

Should be 2 commands after https://github.com/TileDB-Inc/TileDB-Py/pull/1988: ``` pip install -e '.[test]' -C skbuild.cmake.define.TILEDB_PATH=/home/tiledb/dist python -m pytest tiledb ```