search

DynamicsAndNeuralSystems / pycatch22

python implementation of catch22
https://time-series-features.gitbook.io/catch22/python
GNU General Public License v3.0
75 stars 15 forks source link

Fix for Installing on Windows #23

Closed FixItChris closed 8 months ago

FixItChris commented 8 months ago

This is to fix #20 - Windows systems don't have a config variable called CFLAGS so the previous fix for #5 would throw errors.

I've added a fix that checks if this situation occurs, and if so installs the package with just the -std=c99 flag. For the people who didn't have installation issues before it should still install the same way.

benfulcher commented 8 months ago

Thanks @FixItChris ! @jmoo2880 can you have a quick look and check this works. We'll soon want some basic form of unit testing on this package also.