René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[MNT]: Refactor grouped_bar orientation logic using shared common_kwargs","articleBody":"### Summary\n\nIn the Axes.grouped_bar() implementation, the code currently passes repetitive keyword arguments separately for vertical and horizontal orientations.\n\nTim Hoffmann suggested a more maintainable approach by introducing a shared common_kwargs dictionary to consolidate repeated parameters such as align, label, color, and hatch.\n\n\nCurrent Code Snippet:: \n\n```\nif orientation == \"vertical\":\n    bc = self.bar(lefts, hs, width=bar_width, align=\"edge\",\n                  label=label, color=color, hatch=hatch_pattern, **kwargs)\nelse:\n    bc = self.barh(lefts, hs, height=bar_width, align=\"edge\",\n                   label=label, color=color, hatch=hatch_pattern, **kwargs)\n```\n\nProposed Refactor::\n```\ncommon_kwargs = dict(align=\"edge\", label=label, color=color, hatch=hatch_pattern)\nif orientation == \"vertical\":\n    bc = self.bar(lefts, hs, width=bar_width, **common_kwargs, **kwargs)\nelse:\n    bc = self.barh(lefts, hs, height=bar_width, **common_kwargs, **kwargs)\n```\n\nBenefits\nEliminates redundant code across orientation branches.\nReduces likelihood of introducing inconsistencies or errors during future updates.\nMakes the code cleaner and easier to maintain.\nRemoves the need for a separate orientation-specific test since the logic path becomes symmetrical.\n\nReference:\nSuggestion from @timhoffm in PR ENH: Add broadcasted hatch to grouped_bar #30683 \n\n\n\n### Proposed fix\n\n```\ncommon_kwargs = dict(align=\"edge\", label=label, color=color, hatch=hatch_pattern)\nif orientation == \"vertical\":\n    bc = self.bar(lefts, hs, width=bar_width, **common_kwargs, **kwargs)\nelse:\n    bc = self.barh(lefts, hs, height=bar_width, **common_kwargs, **kwargs)\n```","author":{"url":"https://github.com/ilakkmanoharan","@type":"Person","name":"ilakkmanoharan"},"datePublished":"2025-11-05T02:15:44.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/30729/matplotlib/issues/30729"}