-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Notebook test ci #212
Merged
Merged
Notebook test ci #212
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
15c1fd4
[workflow] added test ci for notebooks
Tigul 6be5275
[examples] Refactored action designator example
Tigul 76ce3c5
[process module] Fixed typo
Tigul 6b9bbf2
[examples] Fixed examples
Tigul afededa
[location desig/example] Fixed location designator example
Tigul c20a919
[language] Fixed return in repeat
Tigul 2151b0d
[examples] Fixed imports
Tigul c5eda48
Merge branch 'dev' of github.com:cram2/pycram into notebook-test-ci
Tigul 0d0d53d
[examples] Added CI database to improving actions
Tigul d657cae
[ci] Changed name of notebook test ci
Tigul 11f9a36
[ci] Added setuptool requirements
Tigul 460e0fb
[ci] Update to 22.04
Tigul 994a91c
[ci] Update dependency install
Tigul 13977a8
[ci] Added install of jupytext and treon
Tigul 11bf2d0
[ci] Final fix for ci
Tigul dfcd6aa
[exxamples] Some fixes
Tigul ee6740e
[examples] Fixed intro example
Tigul 57ed827
[examples] Fixed more errors
Tigul 7712d49
[ci] Added cleanup to delete leftover old dir before exec
Tigul 79f5801
[ci] Added update of pycram source files
Tigul 7fcbb3c
[bullet world] Check if Display is connected
Tigul 1682856
[bullet world] Better check
Tigul cb8a1cf
[examples] Errors that appeard in ci
Tigul 50e2983
[examples] more fixes
Tigul dde07d0
[ci] ADded libpq-dev because dependencies
Tigul f35a511
[examples] Fixed from test branch
Tigul 6c7f60e
[ci] Added libpq-dev for psycopg2 dependency
Tigul 78d7269
[examples] Added robustness
Tigul 31a52fb
[example] New seed for cram plan tutorial
Tigul b085b81
[general] removed test code
Tigul 385ea0d
[example] new troso val
Tigul File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,83 @@ | ||
name: Test Jupyter Notebook Examples | ||
|
||
on: | ||
push: | ||
branches: | ||
- dev | ||
- master | ||
pull_request: | ||
branches: | ||
- master | ||
- dev | ||
|
||
# ---------------------------------------------------------------------------------------------------------------------- | ||
|
||
defaults: | ||
run: | ||
shell: bash | ||
working-directory: . | ||
|
||
jobs: | ||
test-and-build: | ||
name: Test Jupyter Notebooks | ||
runs-on: ubuntu-20.04 | ||
container: | ||
image: "pycram/pycram:dev" | ||
|
||
steps: | ||
|
||
- name: Checkout PyCRAM | ||
uses: actions/checkout@v3 | ||
with: | ||
path: "ros/src/pycram" | ||
repository: ${{ github.repository }} | ||
ref: ${{ github.ref }} | ||
submodules: "recursive" | ||
|
||
# For debugging | ||
- name: Setup tmate session | ||
uses: mxschmitt/action-tmate@v3 | ||
if: ${{ github.event_name == 'workflow_dispatch' && inputs.debug_enabled }} | ||
|
||
- name: Update PyCRAM source files | ||
run: | | ||
rm -rf /opt/ros/overlay_ws/src/pycram/* | ||
cd /opt/ros/overlay_ws/src/pycram | ||
rm -rf .git .github .gitignore .gitmodules .readthedocs.yaml | ||
cp -r /__w/${{ github.event.repository.name }}/${{ github.event.repository.name }}/ros/src/pycram /opt/ros/overlay_ws/src | ||
|
||
# ---------------------------------------------------------------------------------------------------------------- | ||
|
||
- name: Install python dependencies | ||
run: | | ||
pip3 install --upgrade pip --root-user-action=ignore | ||
cd /opt/ros/overlay_ws/src/pycram | ||
pip3 install -r requirements.txt | ||
sudo apt-get install -y libpq-dev | ||
pip3 install -r requirements-resolver.txt | ||
|
||
# ---------------------------------------------------------------------------------------------------------------- | ||
|
||
- name: Install Jupytext & Jupyter notebook | ||
run: | | ||
pip3 install jupytext treon | ||
|
||
# ---------------------------------------------------------------------------------------------------------------- | ||
- name: Convert Notebooks | ||
run: | | ||
cd /opt/ros/overlay_ws/src/pycram/examples | ||
rm -rf tmp | ||
mkdir tmp | ||
jupytext --to notebook *.md | ||
mv *.ipynb tmp && cd tmp | ||
|
||
- name: Source and start ROS | ||
run: | | ||
source /opt/ros/overlay_ws/devel/setup.bash | ||
roslaunch pycram ik_and_description.launch & | ||
|
||
- name: Run tests | ||
run: | | ||
source /opt/ros/overlay_ws/devel/setup.bash | ||
roscd pycram/examples/tmp | ||
treon --thread 1 -v --exclude=migrate_neems.ipynb |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldnt this be part of the jb build pipeline?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean the pipeline for the documentation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No this is specifically so that treon can do the testing for the notebooks