Closed MohanVashist1 closed 4 years ago
This looks awesome. Just one question, did you test the program again with both stocks and crypto?
Apologies, I should have included this in the PR but yes I did test it here is the output: For the command:
python detection_engine.py --top_n 25 --min_volume 5000 --data_granularity_minutes 60 --history_to_use 14 --is_load_from_dictionary 0 --data_dictionary_path 'dictionaries/data_dict.npy' --is_save_dictionary 1 --is_test 0 --future_bars 0
For the command:
python detection_engine.py --top_n 25 --min_volume 5000 --data_granularity_minutes 60 --history_to_use 14 --is_load_from_dictionary 1 --data_dictionary_path 'dictionaries/data_dict.npy' --is_save_dictionary 0 --is_test 0 --future_bars 0 --output_format 'CLI'
For the command:
python detection_engine.py --top_n 25 --min_volume 500 --data_granularity_minutes 60 --history_to_use 14 --is_load_from_dictionary 0 --data_dictionary_pata_source binance --stock_list cryptos.txt
Also note that I will be watching the Github issues for any problems that may arise. Also, feel free to contact me if you find anything else.
Thanks, Mohan mohan.vashist@mail.utoronto.ca
This is awesome. Thank you so much. Merging it now.
Appreciate the kind words :)
This PR is to support Crypto-Currencies listed on Binance. With some simple modifications you can add support for more data sources/exchanges with ease.
List of changes: