ats-regression-test error in configure ATS build/install step

[Sergi Molins Rafa]

Hi all

I get an error in the "Configuring ATS build/install."  step regarding the ats-regression-tests python script. Namely, that the python script does not exist. Which in fact the case: I don't see it in the appropriate location. 

I notice it is trying to use /usr/bin/python (which in my case will be 2.7) regardless if I have another python in the PATH, like 3.8 in my case. Not sure if this has anything to do with it.

Thanks

Sergi

-------------------------------------------
--
-- CMake: Configuring ATS build/install.
--
-- ----------------------------------------
-- Adding ats-regression-tests
-- Found PythonInterp: /usr/bin/python (found version "2.7.17")
/usr/bin/python: can't open file '/home/smolins/amanzi-ats/ats-dev/amanzi-build-master-Debug/src/physics/ats/testing/ats-regression-tests/regression_tests.py': [Errno 2] No such file or directory
-- Found ATS tests:
CMake Error at src/physics/ats/testing/CMakeLists.txt:24 (string):
  string sub-command STRIP requires two arguments.


CMake Error at src/physics/ats/testing/CMakeLists.txt:25 (string):
  string sub-command REPLACE requires at least four arguments.

[Coon, Ethan]

Hi Sergi,

 

1. I’ve updated this on master to look for ats-regression-tests/regression-tests.py instead of for the directory (because the directory is there even if the submodule has not been generated).
2. I have to admit I have not fully tested out how bootstrap and git submodules work under a few conditions.  I’m assuming you did a git pull in both Amanzi and ATS and then ran make, which reran cmake/configure?  Did you re-run bootstrap?  Can you rerun bootstrap and let me know if that downloads the ats-regression-tests repo?
3. If that fails, hopefully if you `cd $AMANZI_SRC_DIR/src/physics/ats; git submodule update --init` it should then download the tests repo at $AMANZI_SRC_DIR/src/physics/ats/testing/ats-regression-tests?
4. I’m not entirely sure how to convince cmake to search for python3 and not python2.  Let me see if I can figure that out, and make sure it does not try to run regression_tests.py if it doesn’t have python3.

 

Thanks for your help in debugging this; this is pretty new and hard to test because it all “works for me” but different systems are hard to test…

 

Ethan

 

 

-- 

-------------------------------------------------------------------------

Ethan Coon

Research Scientist

Oak Ridge National Laboratory

 

865-241-1296

https://www.ornl.gov/staff-profile/ethan-t-coon

-------------------------------------------------------------------------

 

--
You received this message because you are subscribed to the Google Groups "Amanzi-ATS Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to ats-users+...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/ats-users/CAP9Hfk5YbosazJLb3X6G19y%2BBEtqR6h7tNkz-ejm8JZsWCoafw%40mail.gmail.com.

[Sergi Molins Rafa]

Hi Ethan

2. I did a clean checkout of Amanzi and used bootstrap.sh as in build_ATS_generic.sh (except for enabling geochemistry, alquimia, etc.) I let bootstrap build the TPLS etc, and pull ATS from the repo (--branch-ats=master)

1. I re-ran the whole thing to test your changes but I got the same error. So it did not download the regression-tests-repo.  

3. This does work. Not completely solves the python issue. One needs python3 with a bunch of things beyond the barebones python3 that comes with Ubuntu and is available as /usr/bin/python3. I think we should be able to tell the build system the location of our preferred distribution.

4. The issue does not have to do with python2. I tried with python3.6 and I got the same error. I tried to suppress the regression tests by using "--disable-reg_tests" with bootstrap.sh but that did not work. Is this the correct approach to disable regression testing with ATS?

Thanks

Sergi

[David Moulton]

Hi Sergi,

    In bootstrap.sh the current git command for cloning/pulling the submodules is

git submodule update --init --remote

Googling around I think we need to add the "--recursive" option here so that it will clone/pull the ats submodule's submodule (the ats-regression repo).  If you're setup to give this a quick try, let me know if it works.  Otherwise I'll try to give it a try this afternoon.

Cheers, David

[Lipnikov, Konstantin]

This worked for me:

git clone --depth=50 https://github.com/amanzi/amanzi.git

cd amanzi

git submodule update --init --recursive

Konstantin

---

From: ats-...@googlegroups.com <ats-...@googlegroups.com> on behalf of David Moulton <moultond...@gmail.com>
Sent: Tuesday, November 17, 2020 11:44:24 AM
To: Amanzi-ATS Users
Subject: Re: [EXTERNAL] ats-regression-test error in configure ATS build/install step

 

To view this discussion on the web visit https://groups.google.com/d/msgid/ats-users/ec3ba888-e0a7-4bba-ad4b-e9c8450b1f7dn%40googlegroups.com.

[Sergi Molins Rafa]

Ok. David's suggestion solved all problems. 

I updated bootstrap.sh and pushed it

https://github.com/amanzi/amanzi/commit/39c0cbd833b7824838cda4726e79faa0b9832482

Thanks all

Sergi

To view this discussion on the web visit https://groups.google.com/d/msgid/ats-users/3a878984747644839091832fc3b13bb1%40lanl.gov.