Results 21 comments of Shadab Shaikh

as far as the code on line 42 and 43 of autocomplete.py goes, it only predicts extensions of the current word and not the next word.

For `line 1213`, which is better(checking with os.path.split(path) ): `if os.path.split(save_location)[1] : ` or `if os.path.split(save_location)[1] != '' :`

So basically you want to add the trailing pathname seperator whilst passing argument(alternative_location) on line 166 itself using os.path.join? Because right now. on line 166 we are not explicitly adding...

I understand the robustness part. But according to the code , in lines 1213 and 1214 we are checking for a trailing '/' and if there isnt we are appending...

Ok @So-Cool . Actually the line of code which I sent in the earlier comment does it all i.e. there wont be any need to explicitly check for the trailing...

[https://github.com/greninja/cuckooml/commits/filesystem_path_concatenation](url) Link to the commit.

https://github.com/greninja/cuckooml/commit/6859d6e9ef0d54740243076ce99b66338838d69c

Any issues with the commit? @So-Cool

can we add a 'raw_input()' prompting for 'if the user wants to use plotting or not' and import the required libraries only on a 'yes'?

The checking for imports of plotting libraries is taken care by the try and catch block. As far as giving a plotting switch is concerned, don't the "figures" argument in...