proposed fix for issue #61 (xy-plot no longer sets labels) #75

Merged
merged 1 commit into from May 17, 2012

Conversation

Projects
None yet
2 participants
Contributor

mknoszlig commented May 12, 2012

the parameter fn create-xy-plot is added to the argument list once in the xy-plot macro and also in the xy-plot* function (which is called by the macro), resulting in an uneven number of entries in the opts seq - this leads to a key/value mixup in the resulting options map. in consequence, the options are not applied correctly to the xy-chart - i've removed create-xy-plot from the argument list that's being concatenated inside the xy-plot macro.

link to issue: liebke#61

duplicate use of create-xy-plot as parameter in xy-plot macro and xy-…
…plot* fn breaks option map structure, removed from macro.

liebke added a commit that referenced this pull request May 17, 2012

Merge pull request #75 from mknoszlig/master
proposed fix for issue #61 (xy-plot no longer sets labels)

@liebke liebke merged commit ac72d00 into incanter:master May 17, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment