[vtk-developers] Add pugixml to VTK

[Dan Lipsa]

Hi all,
Pugixml has been added to VTK master.

https://gitlab.kitware.com/vtk/vtk/merge_requests/4597

Dan



On Tue, Jul 17, 2018 at 9:54 AM Dan Lipsa <dan.lipsa at kitware.com> wrote:

> Hi all,
> I have 4 responses so far, all for adding pugixml to VTK. I'll go ahead
> and do it. ParaView, CMB and SMTK would benefit
> as they'll have less maintenance cost and VTK will have access to pugi.
>
> On Fri, Jul 6, 2018 at 9:20 PM Patrick Bergeron <pbergeron at spiria.com>
> wrote:
>
>> Have you guys tried profiling what happens during those 2 minutes?
>>
>> > On Jul 6, 2018, at 17:56, Elvis Stansvik <elvis.stansvik at orexplore.com>
>> wrote:
>> >
>> > 2018-07-06 23:26 GMT+02:00 Jonathan Borduas <
>> jonathan.borduas at caboma.com>:
>> >> Hi,
>> >>
>> >> Here’s a large xml (paraview statefile with 200+ sphere). It weighs
>> 17.5 MB.
>> >> It takes 1-2 minutes to load with pugixml 1.4 on ParaView.
>> >
>>
> Indeed, this seem like a long time. Do you mind filing a bug report and
> attaching the state file. Not sure what is the reason for this.
>
>
>> > Alright. That shows the time of XML parsing is minuscule compared to
>> > other things in that use case. That file parses in < 400 ms with
>> > libxml2 here, and I guess pugixml would be even faster.
>
> >
>> > Elvis
>> >
>> >>
>> >>
>> >>
>> >> Best,
>> >>
>> >>
>> >>
>> >> Jonathan Borduas
>> >>
>> >>
>> >>
>> >> From: vtk-developers <vtk-developers-bounces at public.kitware.com> On
>> Behalf
>> >> Of Dan Lipsa
>> >> Sent: Friday, July 06, 2018 4:59 PM
>> >> To: elvis.stansvik at orexplore.com
>> >> Cc: VTK Developers <vtk-developers at vtk.org>
>> >> Subject: Re: [vtk-developers] Add pugixml to VTK
>> >>
>> >>
>> >>
>> >>
>> >>
>> >>
>> >> Alright. Maybe this could actually benefit from being done in a
>> >> streaming fashion (à la libxml2's reader API)? Or will it be
>> >> beneficial/easier to have access to the whole DOM in memory?
>> >>
>> >>
>> >>
>> >> You are right - it could be implemented in streaming fashion.
>> >>
>> >> I think the trade off is
>> >>
>> >> - streaming: slower than dom, less memory than dom, possibly more
>> >> complicated as you may need to store data for later.
>> >>
>> >> - dom: quickest, more memory than streaming, simplest way to access
>> citygml
>> >> data.
>> >>
>> >>
>> >>
>> >> For now I lean toward dom as our input data is tiled to start with so I
>> >> don't expect really large files. I realize I have to revise the specs
>> of the
>> >> problem:
>> >>
>> >> I'll need to parse many medium size XML files.
>> >>
>> >> Dan
>> >>
>> >>
>> > _______________________________________________
>> > Powered by www.kitware.com
>> >
>> > Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>> >
>> > Search the list archives at:
>> http://markmail.org/search/?q=vtk-developers
>> >
>> > Follow this link to subscribe/unsubscribe:
>> > https://public.kitware.com/mailman/listinfo/vtk-developers
>> >
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Search the list archives at: http://markmail.org/search/?q=vtk-developers
>>
>> Follow this link to subscribe/unsubscribe:
>> https://public.kitware.com/mailman/listinfo/vtk-developers
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://public.kitware.com/pipermail/vtk-developers/attachments/20181011/c8c05ee0/attachment.html>