Closed wanghy-anl closed 5 months ago
@wanghy-anl this looks good to me for now, and we can look at a more holistic way of finding the paths as part of the installation work this year. I restarted the tests to capture some recent reorganization, but it those pass I can merge this if you're okay with it.
@PaulTalbot-INL Thanks Paul for looking into this PR. Yes, it would be helpful to figure out a new way to find the paths. I am totally fine with merging this to HERON. Feel free to proceed after the tests are done. Thanks, Haoyu
Pull Request Description
What issue does this change request address?
This is not urgent, it can wait after the https://github.com/idaholab/raven/pull/2235 gets merged.
Without installing FARM plugin through
raven/scripts/install_plugins.py -s FARM
, HERON might occasionally pick up FARM (maybe from previous installation, or existing entries insys.path
?) via the lineimport FARM.src
inHERON/src/validators/Factory.py
. A tentative solution is proposed in this PR, but it may not be the best solution:farm_loc
usinghutils.get_farm_loc
, and then append it to thesys.path
;sys.path
contains another entry of "FARM", will it cause any confusion?What are the significant changes in functionality due to this change request?
import FARM.src
andfarm_loc = FARM.__path__
,farm_loc = hutils.get_farm_loc(raven_path=raven_path)
to specify the location of FARM.It would be perfect if we can force the code to search the
farm_loc
only, and importFARM_SISO
andFARM_MIMO
from that folder. However, I am unsure how to do this properly...Any advices are appreciated!
Thanks, Haoyu
For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.