startinpy

startinpy copied to clipboard

We should throw specific exceptions when possible as per the [PEP8](https://peps.python.org/pep-0008/#programming-recommendations) guidelines which state that ```python try: ... except Exception: ... ``` blocks are a code smell.

DimitrisMantas

hugoledoux

Creating a tin within a certain threshold error

5

I was wondering if you are considering the possibility of having some threshold error parameter so that the insertion of points would occur until a certain threshold error has been...

mllobera

so that `copy.deepcopy(dt)` is possible some tips there: https://github.com/PyO3/pyo3/issues/100#issuecomment-1220672112

hugoledoux

v0.11.0 beta: OverflowError: Python int too large to convert to C long

1

When I use int32 or uint32 in attributes (for example: `dt.set_attributes_schema(np.dtype([('intensity', np.uint32)]))` and want to print the attributes using `dt.attributes[1:]` I get: `OverflowError: Python int too large to convert to...

djdan7

> The algorithm is based on flips > The ears are filled by flipping Found in [howitworks.md lines 13 and 17](https://github.com/hugoledoux/startinpy/blob/master/docs/howitworks.md?plain=1#L13). Is this referring to the `Lawson Flip Algorithm`? If...

kylemann16

It's a bit non-DRY but that can be ok. If CI gets more complicated, it might be worth refactoring to a matrix-based setup.

gadomski

```python import startinpy ls = [] dt = startinpy.DT() dt.insert(ls, insertionstrategy="BBox") print(dt) ``` ==> ```bash thread '' panicked at /Users/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/startin-0.8.2/src/lib.rs:1337:59: called `Option::unwrap()` on a `None` value note: run with `RUST_BACKTRACE=1`...

hugoledoux