You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey, guys,
Thank you all for this pioneering work in OpenFOAM, it really does much help to both the OpenFOAM and DG community, I believe this work will be popular around the world soon.
I have not go through the code right now but just a glance and wonder why the ThirdParty folder is included in this repository because it is the original part of official OpenFOAM, do you guys have done some changes to this folder? I mean as far as I can see, there is nothing more here if the official OpenFOAM and ThirdParty have been compiled. You know the ParaView package is rather large.
Best wishes.
Yu Cheng
The text was updated successfully, but these errors were encountered:
Sorry, I have mistaken HopeFOAM package after digging into the code, it is a redistribution of original OpenFOAM, not only an extension of OpenFOAM, so everything should be included in the repo.
However, for the sake of maintenance convenience, I doubt if it is possible to separate HopeFOAM with official OpenFOAM since it could be easily upgraded when OpenFOAM is updated to new version. Actually, the OpenFOAM extention LEMOS is a good reference. The Thirdparty-0.1 could be included in the repo to have pacakages like boost, PETSC and SLEPC in it, which will help the users to compile the package.
Hey, guys,
Thank you all for this pioneering work in OpenFOAM, it really does much help to both the OpenFOAM and DG community, I believe this work will be popular around the world soon.
I have not go through the code right now but just a glance and wonder why the ThirdParty folder is included in this repository because it is the original part of official OpenFOAM, do you guys have done some changes to this folder? I mean as far as I can see, there is nothing more here if the official OpenFOAM and ThirdParty have been compiled. You know the ParaView package is rather large.
Best wishes.
Yu Cheng
The text was updated successfully, but these errors were encountered: