Add skip_non_updateable arg to write function

[MariusWirtz]

Handy new argument to skip cells that are not updateable.

Fixes #651

[MariusWirtz]

Performance impact of new argument quantified below:

Scenario1: 100k updateable cells + 100k non updateable cells

use_ti	skip_non_updateable	Elapsed Time
TRUE	TRUE	00:00:33
FALSE	TRUE	00:01:25
TRUE	FALSE	00:01:25
FALSE	FALSE	00:01:25

Scenario2: 200k updateable cells

use_ti	skip_non_updateable	Elapsed Time
TRUE	TRUE	00:00:36
FALSE	TRUE	00:01:57
TRUE	FALSE	00:00:14
FALSE	FALSE	00:00:52

All tests run on localhost.

[MariusWirtz]

@dsfomichev, @gbryant-dev
Any thoughts on this approach?

[gbryant-dev]

I like it. I've left a comment just to get your thoughts on one approach over the other 🙂

The performance impact when testing with the first scenario and skip_non_updateable is False is interesting compared to second scenario when all cells are updateable. Presumably, the write failures are slowing the processing down.

[rkvinoth]

The performance metrics looks very useful and provides an idea of when to use this parameter.
Thanks @MariusWirtz

[MariusWirtz]

Thanks for the feedback @gbryant-dev and @rkvinoth

I implemented your suggestions.