Since the update to version 3.2.1 of Paraview I'm not able to use the native OpenFOAM Readers in Paraview. So please add the patches to the ebuild. Thank's! Reproducible: Always
Created attachment 138810 [details, diff] OpenFOAM Reader patch To activate the build in capabilities of Paraview for OpenFOAM Files, this patch is needed at compile time. Maybe this can be activated with an USE Flag, so that my ebuild from bug 104257 has an postprocessing tool. The built-in capabilities of Paraview were selected as default in version 2.6.2.
Created attachment 138812 [details, diff] Patch for the OpenFOAM Reader This patch improves the built-in capabilities from Paraview for the OpenFOAM Datastructure very much. I would highly recommend to add this patch to the ebuild, if the openfoam support is compiled with.
Hi Oliver, Thanks much for your patches and I'll try to have a look at them as soon as I find some time. Best, Markus
Markus, In the meantime are you aware that Oliver put an ebuild for OpenFoam on the sunrise overlay? I used the ebuild and it built the beast (and the beast seems to be alive and kicking...)... Maybe time to try to include OpenFoam to the main gentoo tree... Daniel
Hi Daniel, I am aware of it, but haven't had time to even look at it yet :( As soon as I find some time ..... I'll have a closer look. cheers, Markus
Hi Oliver, I just had some time to look at your patches. Since they are quite extensive, would you be able to comment on where they come from. It almost seems to me that this is something that should be reviewed and merged by upstream rather than by us. Best, Markus
Hello Markus, this patches came from the OpenFOAM Mailing list: http://openfoam.cfd-online.com/cgi-bin/forum/show.cgi?tpc=1&post=17466#POST17466 and since the post from yesterday, it seems that these patches were sent upstream but are not yet in the CVS. I fully agree with you, that these patches should be merged into the official tree. Nevertheless, I would highly recommend to use these patches in version 3.2.1 of the paraview ebuild and I hope we don't need these patches in further versions of paraview. Oliver
Hi Oliver, Thanks much for the link! This indeed looks like a solid patch hence it should be fine as part of our paraview ebuild until upstream merges it into their trunk. I've added it to the pareview-3.2.1 ebuild and it should have hit the mirrors by now. You'll need to re-sync and re-emerge paraview to pull it in. Lacking any OpenFOAM dataset I have not yet tested it but the option is there. Please give it a spin and let me know if it works as expected. Thanks, Markus
Markus, Are you sure you put the patch in the gentoo tree? When I emerged, I got the new ebuild but not the patch... Daniel
Daniel, The patch is on the mirrors not in the files/ directory due to its size (patches in files/ should be < 30k uncompressed). Just look for the line epatch "${DISTDIR}"/${P}-OpenFOAM.patch.bz2 in the ebuild. best, Markus
Hello Markus, I have had a quick look on the new paraview and it seems to work as expected. So I think we can close this bug. Oliver BTW: Why don't you call the new ebuild not paraview-3.2.1-r1 or something like that?
The present version works just fine for everybody not caring about OpenFOAM. Hence, there is no reason to force a multi-hour compile onto people for a feature they most likely don't care about. An -r1 in this case would be a real bug fix/enhancement that matters for a significant fraction of people. If paraview would compile in 5 minutes this would have been a different story. Best, Markus
