Forward args from apply_and_annotate to apply_test#60
Forward args from apply_and_annotate to apply_test#60StefRe wants to merge 1 commit intotrevismd:masterfrom
Conversation
so you and use the shortcut convenience apply_and_annotate instead of apply_test and annotate individually
|
Hello @StefRe , I think rather keep the simple |
|
No problem - I just came across it when answering the SO question and already forgot about it. It's not a necessity, but once we have
Oh, I hadn't considered that. In this particular case, however, only one of them takes var-key args, so the solution would be straightforward here: def apply_and_annotate(self, line_offset=None, line_offset_to_group=None, num_comparisons='auto', **stats_params):
"""Applies a configured statistical test and annotates the plot"""
self.apply_test(num_comparisons, **stats_params)
return self.annotate(line_offset, line_offset_to_group)But again, no big deal for me - you can leave it as is and close the PR. It was more a spontaneous change request than a real necessity. |
so you can use the shortcut convenience
apply_and_annotatemethod with parameters instead of callingapply_testandannotateindividually.See also https://stackoverflow.com/a/72445947/3944322.