-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix broken tests in Better Tracing & Skeletonisation Merger branch #850
Closed
1 of 6 tasks
Tracked by
#800
Comments
30 tasks
This was referenced Sep 24, 2024
Current state of failed tests is shown below (not sure if this is an improvement or not 😆 !).
|
This was referenced Oct 1, 2024
Tests are all passing on |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Whilst working on writing unit tests for
maxgamill-sheffield/800-better-tracing
which addresses #800 I found that the tests suite does not complete as there are a number of breaking changes.In total there are 61 tests fail, output with basic details are below...
Many of these are
unexpected keyword argument
which suggests one of a few things...Common keyword arguments that crop up repeatedly in errors and possible causes/problems and other types of errors are listed below.
skeletonisation_method
- which is a new argument introduced in this branch but not updated in many test files (by far the most common cause of errors/failures).countNeighbours
- looks like an old function of thegenTracingFuncs
module that has been moved/renamed/removed.AttributeError: 'dnaTrace' object has no attribute 'mol_is_circular'. Did you mean: 'mol_is_circulars'?
- looks like a renaming issue where plural is now used, should be easy to correct.KeyError : 'green_black' is not a known colormap name
- a new colormap introduced in some manner, probably needs to be defined intopostats/theme.py
although it does have a newblu_purp_green
colormap included so perhaps what is required isgreen_black
>blu_purp_green
(personally I would favor not using abbreviations tab-auto-complete saves typing and full names are easier to remember than abbreviations soblue_purple_green
).IndexError
- crops up intest_processing.py::test_process_scan_align_grainstats_dnatracing
test_plottingfuncs.py
do not match the existing images this will need investigating and resolving.The text was updated successfully, but these errors were encountered: