Skip to content

Commit

Permalink
more informative test logging output and switch to fp64 in workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
thehrh committed Aug 27, 2024
1 parent a50eef2 commit 17ba2fa
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pythonpackage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@ jobs:
run: |
pip install pytest
PISA_FTYPE=fp32 PISA_TARGET=cpu MPLBACKEND=agg ./pisa_tests/test_example_pipelines.py -v
- name: Test PISA services
- name: Test PISA services, double precision
run: |
PISA_FTYPE=fp32 PISA_TARGET=cpu MPLBACKEND=agg ./pisa_tests/test_services.py -v
PISA_FTYPE=fp64 PISA_TARGET=cpu MPLBACKEND=agg ./pisa_tests/test_services.py -v
- name: Test PISA imports and unit tests, double precision
run: |
pip install pytest
Expand Down
5 changes: 4 additions & 1 deletion pisa_tests/test_services.py
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ def test_services(
)
continue

logging.debug(PFX + f"Starting test for service {stage_dot_service}...")
logging.info(PFX + f"Starting test for service {stage_dot_service}...")
ntries += 1

# if service module import successful, try to initialise the service
Expand Down Expand Up @@ -313,6 +313,7 @@ def test_services(

#if service.data is not None: # we should never be in this state here
if service.calc_mode is None:
logging.debug(PFX + "Setting calc_mode ...")
try:
service.calc_mode = 'events'
except ValueError:
Expand All @@ -330,6 +331,7 @@ def test_services(
continue

if service.apply_mode is None:
logging.debug(PFX + "Setting apply_mode ...")
try:
service.apply_mode = 'events'
except ValueError:
Expand Down Expand Up @@ -357,6 +359,7 @@ def test_services(
continue

try:
logging.debug(PFX + "Setting up and running service...")
run_service_test(service)
logging.info(PFX + f"{stage_dot_service} passed the test.")
nsuccesses += 1
Expand Down

0 comments on commit 17ba2fa

Please sign in to comment.