Skip to content

Commit

Permalink
Do not require PDDLStream for tests, switch to main repo
Browse files Browse the repository at this point in the history
  • Loading branch information
sea-bass committed Mar 17, 2024
1 parent f989895 commit a0bef61
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 15 deletions.
3 changes: 1 addition & 2 deletions setup/setup_pddlstream.bash
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,8 @@ then
fi

# Clone and build PDDLStream
# TODO: Once PDDLStream works with Python 3.10, go back to the original repo.
pushd $SCRIPT_DIR/../dependencies
git clone https://github.com/sea-bass/pddlstream.git
git clone https://github.com/caelan/pddlstream.git
pushd pddlstream
touch COLCON_IGNORE
git submodule update --init --recursive
Expand Down
5 changes: 5 additions & 0 deletions test/planning/test_pddlstream_manip.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,16 @@
"""
Tests for PDDLStream planning with manipulation streams.
"""

import importlib
import numpy as np
import os
import pytest
import threading

if importlib.util.find_spec("pddlstream") is None:
pytest.skip(allow_module_level=True, reason="PDDLStream not available")

from pyrobosim.core import Robot, World
from pyrobosim.gui import start_gui
from pyrobosim.manipulation import GraspGenerator, ParallelGraspProperties
Expand Down
5 changes: 5 additions & 0 deletions test/planning/test_pddlstream_nav.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,14 @@
"""
Tests for PDDLStream planning with navigation streams.
"""
import importlib
import os
import pytest
import threading

if importlib.util.find_spec("pddlstream") is None:
pytest.skip(allow_module_level=True, reason="PDDLStream not available")

from pyrobosim.core import Robot, World
from pyrobosim.gui import start_gui
from pyrobosim.navigation import ConstantVelocityExecutor, PathPlanner
Expand Down
9 changes: 0 additions & 9 deletions test/test_dependencies.py

This file was deleted.

6 changes: 2 additions & 4 deletions test/test_pyrobosim.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,12 @@
Basic sanity checks for pyrobosim module.
"""

import sys
import importlib
from importlib.metadata import version


def test_import():
import pyrobosim

assert "pyrobosim" in sys.modules
assert importlib.util.find_spec("pyrobosim")


def test_version():
Expand Down

0 comments on commit a0bef61

Please sign in to comment.