Darsh Ranjan

Results 13 comments of Darsh Ranjan

There's an efficient loader for large elements via memory-mapping, but it requires: - Binary mode, not ASCII - No list properties - Underlying file can be memory-mapped If those requirements...

By the way, here's the logic for deciding which reader it can use: https://github.com/dranjan/python-plyfile/blob/9f8e8708d3a071229cf292caae7d13264e11c88b/plyfile.py#L638-L656 The middle branch, where `_can_mmap(stream) and not self._have_list` evaluates to `True` and `text` is `False`, is...

Unfortunately elements that contain list properties can't be memory-mapped.

The simplest way is to set the `text` attribute of the `PlyData` instance to `True`.

Is this the file: [test.txt](https://github.com/dranjan/python-plyfile/files/2752488/test.txt) For completeness, could you post a minimal test script to reproduce this error? I realize I haven't tested this on Python 3.7, so I'm working...

Okay, thanks. I was able to reproduce the error with the file you provided. The error appears to stem from the line ending convention ("\r", or Mac-style), which seems to...

Thanks for the report, @peendebak, and sorry for not getting back sooner. From what I can determine, the `__version__` specification comes from the deferred [PEP-396](https://www.python.org/dev/peps/pep-0396/). Doing a quick and unscientific...

@athompson673 - wouldn't the standard way to do that be with a version specifier (like `plyfile>=1.0`) in the `setup.py` script or `requirements.txt` file? Anyway, I did a bit of searching...

Exactly, @smoluck. I'm going to close this, since the issue seems to have been resolved.

Not that anyone was likely holding their breath for a proper fix for this issue...but it should be fixed on `master` now. It hasn't been packaged for PyPI yet, since...