microsoft / anomalydetector

SR-CNN
MIT License
257 stars 70 forks source link

Issue with "batch_size" in main.py #25

Open JanWaldhorn opened 4 years ago

JanWaldhorn commented 4 years ago

I find your approach very exciting and wanted to try out your code. As a basis I use Anaoconda on a Win 10 PC. After the installation I had some trouble (not recognized _anomaly_kernel_cython.c )which could be solved by compiling the cpython module (see here).

please use the following command to compile the cpython module. python setup.py build_ext --inplace

Originally posted by @guinao in https://github.com/microsoft/anomalydetector/issues/19#issuecomment-642618752

When trying to run the main.py script, the following error message appears

File "main.py", line 17, in <module> detect_anomaly(sample, THRESHOLD, MAG_WINDOW, SCORE_WINDOW, 99, DetectMode.anomaly_only) File "main.py", line 9, in detect_anomaly sensitivity=sensitivity, detect_mode=detect_mode) TypeError: __init__() missing 1 required positional argument: 'batch_size'

Can anyone give me a hint what value the variable batch_size should have?

Many thanks in advance

manikTharaka commented 4 years ago

I also ran into some issues like this. Looks like the maintainers made a few changes to the method signatures. For the batch_size parameter I would try with a value smaller than the series length that you are inputting. For instance if the series length is 1000 I would try with a batch size of 100.

JanWaldhorn commented 3 years ago

Thanks a lot for the hint!