Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Transform testing and seed bugfix #74

Merged
merged 36 commits into from
Sep 21, 2023
Merged

Transform testing and seed bugfix #74

merged 36 commits into from
Sep 21, 2023

Conversation

harveymannering
Copy link
Contributor

No description provided.

sfmig and others added 30 commits June 5, 2023 18:31
@sfmig
a first draft of a simple test
799237d
@harveymannering
Started building out tests
e236018
@harveymannering
Added pytest pip installs to python install file
4f0e106
@harveymannering
Commenting tests
a964ff0
@harveymannering
Transforms and seed tests
f85fae9
@ruaridhg
Fix linting
8bd2546
@ruaridhg
Fix linting
9a6e327
@ruaridhg
Fix linting
cef3bdd
@ruaridhg
Fix pytest bpy not found error
763d426
@harveymannering
Pre-commit linting changes
657db14
@ruaridhg
Fix pytest bpy not found error
da738be
@ruaridhg
Fix pytest bpy not found error
c6a9b9e
@harveymannering
Trying to make github action work for pytest
2b3ae9d
@harveymannering
Testing removal of pytest fixture
fdc2b2c
@harveymannering
Merge branch 'smg/basic-test' of https://github.com/UCL/Blender_Rando…
88d565a 
…miser into smg/basic-test
@harveymannering
Linting
2507fff
@harveymannering
Trying out a different CI file
019b0da
@harveymannering
Update test.yml
f58ca42
@harveymannering
Update test.yml
49a7a6c
@harveymannering
Create _blender-executable-path.txt
24cfc02
@harveymannering
Fixed linting issues
26df644
@harveymannering
Update test.yml
295ba38
@harveymannering
Constructing a scene for the geometry testing
92e0aa0
@harveymannering
Merge branch 'smg/basic-test' of https://github.com/UCL/Blender_Rando…
e5d5a9b 
…miser into smg/basic-test
@harveymannering
Fixing up the test_randomiser_geometry test
b3f0230
@harveymannering
Removed unneeded seed code
cfdf619
@harveymannering
Removed unnecessary pip installs
2acda15 
pytest works with with these lines, making them pointless
@harveymannering
Revert "Removed unneeded seed code"
1eb9c5d 
This reverts commit cfdf619.
@harveymannering
Rolled back seed change and added geometry test
30b246a
@harveymannering
Changed geometry code to work with testing
f1b6375
ruaridhg
ruaridhg reviewed Sep 21, 2023
View reviewed changes
Copy link
Collaborator

@ruaridhg ruaridhg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Seed bug fixed with seed previous works well and just a couple of doc strings to add to the test functions

@ruaridhg ruaridhg merged commit 5332784 into main Sep 21, 2023
2 of 8 checks passed
@ruaridhg ruaridhg deleted the hgm/transform_tests branch September 21, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ruaridhg ruaridhg ruaridhg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@harveymannering @ruaridhg @sfmig