<div dir="ltr">I agree. File I/O is not a property of the object but an operation on the object.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 6, 2017 at 10:51 AM, Matthew Leotta <span dir="ltr"><<a href="mailto:matt.leotta@kitware.com" target="_blank">matt.leotta@kitware.com</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">I would not add any file I/O fu actions directly to the Vital type classes.  You can add stream operators (that is, >> and <<) along side the type classes.  I believe homography already has those.  Currently we are putting file I/O functions in separate files in "kwiver/vital/io”.  Look at camera_io.h as an example.  These I/O functions should mostly just validate file object and then call the stream operators. <div><br><div><blockquote type="cite"><div><div class="h5"><div>On Sep 6, 2017, at 10:38 AM, Matt Brown <<a href="mailto:matt.brown@kitware.com" target="_blank">matt.brown@kitware.com</a>> wrote:</div><br class="m_9028378023877067919Apple-interchange-newline"></div></div><div><div><div class="h5"><div dir="ltr">My current interest is in reading/writing a vital/homography from/to disk. Note, this is the simpler homography without other metadata in it (i.e., not homography_f2f). I would like to create a base class  methods save_to_file and read_from_file, but I have a few questions:<br><br><ul><li>Should this go in the base class or in homography_?</li><li>It seems to me that this might be a useful generic facility across all vital types, so maybe we should take a moment and consider what the member function names should be and their scope.<br></li></ul></div></div></div>
______________________________<wbr>_________________<br>Kwiver-users mailing list<br><a href="mailto:Kwiver-users@public.kitware.com" target="_blank">Kwiver-users@public.kitware.<wbr>com</a><br><a href="http://public.kitware.com/mailman/listinfo/kwiver-users" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/kwiver-users</a><br></div></blockquote></div><br></div></div><br>______________________________<wbr>_________________<br>
Kwiver-users mailing list<br>
<a href="mailto:Kwiver-users@public.kitware.com">Kwiver-users@public.kitware.<wbr>com</a><br>
<a href="http://public.kitware.com/mailman/listinfo/kwiver-users" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/kwiver-users</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><font color="#888888"><b>Linus Sherrill - </b></font><font color="#888888">Staff R&D Engineer<br></font><font color="#888888">Kitware, Inc.<br>28 Corporate Drive<br>Clifton Park, NY 12065-8662<br>E: <a href="mailto:linus.sherrill@kitware.com" target="_blank">linus.sherrill@kitware.com</a><br>P: 518.881.4400<br></font></div></div></div></div></div>
</div>