<div dir="ltr">Yeah I realized that it was in the merge commit but not the actual commit. Sorry for the confusion.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 10, 2013 at 10:42 AM, Tobias Pietzsch <span dir="ltr"><<a href="mailto:pietzsch@mpi-cbg.de" target="_blank">pietzsch@mpi-cbg.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Mark<div><br></div><div>Ok, if it doesn't harm performance it's fine with me. I had missed the note in the merge commit...</div>

<div><br></div><div>best regards,</div><div>Tobias</div><div><div class="h5"><div><br></div><div><div><div>On Jul 8, 2013, at 3:46 PM, Mark Hiner <<a href="mailto:hiner@wisc.edu" target="_blank">hiner@wisc.edu</a>> wrote:</div>

<br><blockquote type="cite"><div dir="ltr"><div><div>Hi Tobias,<br><br>The <a href="https://github.com/scifio/scifio/blob/master/scifio/src/main/java/io/scif/img/cell/SCIFIOCell.java" target="_blank">SCIFIOCell</a> class needs to be serializable to allow the library we're using to persist opened cells to disk, and thus its superclass(es) needs to be serializable with non-final fields. <br>



<br>The only potential alternative I can think of would be to fully duplicate AbstractCell's fields and logic in SCIFIOCell... but that would still require the ArrayDataAccess to be serializable (do you see that as a potential problem, or only AbstractCell?).<br>



<br></div>In the <a href="https://github.com/imagej/imglib/commit/7bc7dde9b095d6e3544d5f781716c0dc06de439b" target="_blank">merge commit</a> I mentioned we ran the PerformanceBenchmark with/without implementing Serializable and saw no difference. We thought that was sufficient for merging the changes, but I'm sorry if we circumvented discussion.<br>



<br></div>If there are other tests you'd like to see run, or would like me to investigate duplicating AbstractCell's code, let me know.<br><br>Thanks,<br>Mark<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">



On Wed, Jul 3, 2013 at 10:40 AM, Tobias Pietzsch <span dir="ltr"><<a href="mailto:pietzsch@mpi-cbg.de" target="_blank">pietzsch@mpi-cbg.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Hi Mark,<br>
<br>
we just noticed the merge of the cell-serialization branch, that makes AbstractCell Serializable and therefore final field non-final.<br>
Actually Stephan Saalfeld noticed, I would have missed it probably. It would be nice if such changes could be discussed before they are merged to master.<br>
Did you do any tests to evaluate the performance impact of un-finalizing the fields.<br>
What is this needed for and are there other alternatives to achieve it instead of making AbstractCell Serializable.<br>
<br>
best regards,<br>
Tobias</blockquote></div><br></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br></div>