-
Notifications
You must be signed in to change notification settings - Fork 95
Fix cummeRbund and various linting errors #638
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
base: main
Are you sure you want to change the base?
Fix cummeRbund and various linting errors #638
Conversation
caused by the removal of the ``file_name`` property in galaxyproject/galaxy#16783 . Fix these errors: ``` Parameter 'x': requires a value, but no legal values defined, Parameter 'y': requires a value, but no legal values defined ``` ``` Parameter 'gene_id': requires a value, but no legal values defined ```
to fix `Mutex creation failed` when importing `later`, xref: r-lib/later#63 cummeRbund 2.24.0 is the first version that doesn't (indirectly) depend on `r-later` and works correctly.
2dd7aa5 to
38e5d6c
Compare
38e5d6c to
ff8ab28
Compare
|
Green! This should fix the weekly CI tests. |
| <output name="output" ftype="txt"> | ||
| <assert_contents><has_text text="cummeRbund finished" /></assert_contents> | ||
| <discovered_dataset designation="maplot-0" ftype="png" file="maplot.png" /> | ||
| <discovered_dataset designation="maplot-0" ftype="png" file="maplot.png" compare="sim_size" /> |
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.
you can use here the more stringent img-comparison asserts
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.
I think I've tried that and the difference was very high for reasons I don't understand.
Co-authored-by: Björn Grüning <[email protected]>
f1d5f3e to
f0984b8
Compare
No description provided.