Skip to content
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

Table x/y axes change depending on number of rows columns #2338

Closed
that0n3guy opened this issue Apr 8, 2016 · 4 comments
Closed

Table x/y axes change depending on number of rows columns #2338

that0n3guy opened this issue Apr 8, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@that0n3guy
Copy link

If this is a bug report, please fill in the blanks:

  • I am using the chrome browser.
  • My computer's OS is window 10.
  • I'm running Metabase docker on aws
  • My database is mysql.
  • My Metabase version is v0.16.1 built on 2016-04-05.

This one is hard to explain w/o a video. https://goo.gl/7ExkM6

Word description: If there are more columns than rows in a table, it dynamically switches them. Here is 2 columns (categories) and 4 months (this is correct):

image

Here is 5 columns (categories) and still 4 months (this is wrong):
image

@agilliland
Copy link
Contributor

Yeah, we've discussed setting it such that if one of the dimensions is time then it should be placed on the x-axis. Currently I believe we always put the largest dimension on the x-axis, which is not ideal in this situation.

@that0n3guy
Copy link
Author

Could also do a dropdown select next to the "visualization" selector.

@camsaul camsaul changed the title Table x/y access changes depending on number of rows columns Table x/y axes changes depending on number of rows columns Apr 11, 2016
@camsaul camsaul changed the title Table x/y axes changes depending on number of rows columns Table x/y axes change depending on number of rows columns Apr 12, 2016
@tlrobinson tlrobinson self-assigned this Apr 18, 2016
@tlrobinson
Copy link
Contributor

Postgres test case:

SELECT a, b, c
FROM (VALUES
  (now(), 'a', 1),
  (now() + INTERVAL '1 day', 'a', 2.2),
  (now(), 'b', 2.6),
  (now() + INTERVAL '1 day', 'b', 1.2),
  (now(), 'c', 0.5),
  (now() + INTERVAL '1 day', 'c', 1)
) AS t(a,b,c);

@agilliland
Copy link
Contributor

Lets move forward and set the default behavior to always put the Date/Time dimension on the x-axis. Beyond that fix we can track a separate issue to consider making this fully configurable by the user via #2371

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants