before proceeding, i still have few remarks:
1) in the docstring of get_value_text() please add the meaning of the params and returned value
2) please document the function start_log_process()
3) the code of inline_process_old_data() and inline_process_new_data() seems really similar. I wonder if we couldn't factorize that code. Morover, having those functions inside another function make the code a bit unreadable.
4) a general effort of documenting/commenting is needed because the code is really difficult to understand.
Next time will be the good one, i'm pretty sure of it! (otherwise, the bug is totally fixed by now and the module seems to run perfectly!)
hi again Naresh,
before proceeding, i still have few remarks: process_ old_data( ) and inline_ process_ new_data( ) seems really similar. I wonder if we couldn't factorize that code. Morover, having those functions inside another function make the code a bit unreadable. commenting is needed because the code is really difficult to understand.
1) in the docstring of get_value_text() please add the meaning of the params and returned value
2) please document the function start_log_process()
3) the code of inline_
4) a general effort of documenting/
Next time will be the good one, i'm pretty sure of it! (otherwise, the bug is totally fixed by now and the module seems to run perfectly!)
Regards,
Quentin