File generation in 2.0.0 in a Windows Forms app much slower about graphviz.netwrapper HOT 4 CLOSED

Done @modery

from graphviz.netwrapper.

Could this be specific to starting processes in a winforms application?
When I reproduce your code in a unit test it prints:

Starting: 1/19/2024 2:19:40 PM
Finished: 1/19/2024 2:19:43 PM

And after removing the redundant calls to CreateLayout:

Starting: 1/19/2024 2:20:19 PM
Finished: 1/19/2024 2:20:20 PM

I don't think we do anything exotic in this library, we just start a process and communicate over stdin/stdout. But since you mention you see a console window opening every call, there would likely be some additional overhead somewhere.

Sounds a bit like https://stackoverflow.com/questions/5377423/hide-console-window-from-process-start-c-sharp which recommends setting startInfo.UseShellExecute = false;, but we already do that. Perhaps you could experiment with other options, like ProcessWindowStyle.Hidden and see if that resolves your issue?

from graphviz.netwrapper.

I can confirm that running the same code in a console app takes 2 seconds (as expected).

Tested your suggestion, and added the Rubjerg.Graphviz and GraphvizWrapper projects to a test solution. Compiled as is (so same as 2.0.0), and the behaviour is the same (window pops up, generally it's slow).

Once I added the process.StartInfo.WindowStyle = ProcessWindowStyle.Hidden; to the relevant section in GraphvizCommand.Exec, the window didn't pop up again and generation was as fast as before

         // Redirect the input/output streams
        process.StartInfo.UseShellExecute = false;
        process.StartInfo.RedirectStandardOutput = true;
        process.StartInfo.RedirectStandardInput = true;
        process.StartInfo.RedirectStandardError = true;
        process.StartInfo.WindowStyle = ProcessWindowStyle.Hidden;

Final test was doing the same thing in my actual program, and I can confirm that things worked well once the ProcessWindowStyle.Hidden was set

from graphviz.netwrapper.

Hi @chtenb , will you publish a new release for this? Preferably I wouldn't want to include your code in my own as a submodule, and rather simply use nuget (as I've been doing so far)

from graphviz.netwrapper.