[vtk-developers] vtkOpenFOAMReader is untested

[David E DeMarle]

I for one would love another hackathon!

What can we do to make it trivial to create new tests before hand? Clearly
documenting the steps and the many utilities to call in your test is a good
start.

Then at the hackathon we could take a look at the coverage reports, divy up
the poorly documented regions and boost out coverage to a satisfactory
level. Then we could do what ITK did a while back and have a dashboard day.

David E DeMarle
Kitware, Inc.
R&D Engineer
21 Corporate Drive
Clifton Park, NY 12065-8662
Phone: 518-881-4909


On Thu, Dec 12, 2013 at 1:45 PM, Bill Lorensen <bill.lorensen at gmail.com>wrote:

> Berk,
>
> There are several readers in vtk that are untested. I smell hackathon!!!
>
> vtkPOpenFOAMReader
> vtkFLUENTReader
> vtkPWindBladeReader
> vtkWindBladeReader
> vtkMFIXReader
> vtkLSDynaReader
> vtkPChacoReader
> vtkSLACReader
> vtkMPIImageReader
> vtkPExodusIIReader
> vtkAMREnzoParticlesReader
> vtkMPASReader
> vtkSLACParticleReader
> vtkPostgreSQLToTableReader
> vtkMySQLToTableReader
> vtkGraphReader
> vtkDIMACSGraphReader
> vtkAMRFlashParticlesReader
> vtkPNetCDFPOPReader
> vtkNetCDFCAMReader
> vtkChacoGraphReader
> vtkRTXMLPolyDataReader
> vtkXMLHyperOctreeReader
> vtkAMRFlashReader
> vtkSESAMEReader
> vtkFixedWidthTextReader
> vtkVPICReader
> vtkUGFacetReader
> vtkISIReader
> vtkRISReader
> vtkTreeReader
> vtkNetCDFPOPReader
>
> and many more under covered.
>
> On Thu, Dec 12, 2013 at 1:20 PM, Berk Geveci <berk.geveci at kitware.com>
> wrote:
> > We should test it in VTK. There are other package and development teams
> out
> > there that use that reader. There may be a file or two in the ParaView
> data
> > repo.
> >
> > -berk
> >
> >
> > On Thu, Dec 12, 2013 at 1:04 PM, Bill Lorensen <bill.lorensen at gmail.com>
> > wrote:
> >>
> >> Folks,
> >>
> >> I have started looking at the code coverage in VTK. Only 2/3 of the
> >> code is covered by our tests. This is very low.
> >>
> >> The nearly 4000 lines of code in vtkOpenFOAMReader are uncovered. Can
> >> someone supply a minimal test (or example) and a small dataset that
> >> will exercise this class.
> >>
> >> I believe this class is used in Paraview. Should it be moved into the
> >> Paraview repo?
> >>
> >> Bill
> >>
> >> --
> >> Unpaid intern in BillsBasement at noware dot com
> >> _______________________________________________
> >> Powered by www.kitware.com
> >>
> >> Visit other Kitware open-source projects at
> >> http://www.kitware.com/opensource/opensource.html
> >>
> >> Follow this link to subscribe/unsubscribe:
> >> http://www.vtk.org/mailman/listinfo/vtk-developers
> >>
> >
>
>
>
> --
> Unpaid intern in BillsBasement at noware dot com
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20131212/968db0fc/attachment.html>