Daniel Ching

identify reports wrong bit depth for 32-bit image

2

### Prerequisites - [x] I have written a descriptive issue title - [x] I have verified that I am using the latest version of ImageMagick - [x] I have searched...

**Describe the problem** After building TomoPy for python 3.8, the libtomopy-accel will fail to load at runtime. When building without OpenCV, there is no failed loading of the other libraries....

Unexpected behavior with gridrec and filter_name

2

I experienced unexpected behaviour when using filters and gridrec. Given the following example, ```python import tomopy # set data and theta here for params in [ {'algorithm': 'gridrec'}, {'algorithm': 'gridrec',...

Filters not implemented for FBP

4

According to the [docs](https://tomopy.readthedocs.io/en/latest/api/tomopy.recon.algorithm.html), filters are not implemented for FBP. However, TomoPy does not raise an error when you try to pass `filter_name` or `filter_par` with `algorithm='fbp'`. This would explain...

Use buildsystem provided by pytorch

1

I don't understand the reason for the custom python/make build system in `ptx_annotation.py` and `make.py`, but it was causing some missing symbol errors for me when I tried to load...

Parallel Radon slightly off for simple example

3

Good work on this interesting package. I built 306ae4638f6c8a2ea4621a857e03419174d77df4 and ran the following simple test where I expect that the Radon operator should be perfect at angles 0 and PI/2....

Implement __repr__ for custom classes

3

Recently I discovered the `__repr__` function in python. It seems that it would be useful implement this function such that objects can be recreated from log outputs. http://stackoverflow.com/a/19597196

Performance Benchmarking

1

I've made a template to help with performance benchmarks on this branch carterbox/polytope@ec5fdd01c4e7046ec3e47436b0ec37cfbd1e36af. I chose `timeit` instead of `cProfile` because the output is easier to manage and compare. Though, if...

bounding_box format

16

Why is bounding box stored as a 2-tuple of vertical arrays? For the purpose of applying transformations to the bounding box (#32), I think it would be better to store...

BUG: Don't require absolute path for shared libraries

2

The purpose of this PR is to refactor `c_shared_lib()`, so it doesn't require that ctypes `find_library()` return an absolute path. Apparently on arch linux, ctypes `LoadLibrary` can still sometimes find...