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

order_by_cols: '[' is not a valid choice for this field #1100

Closed
lihemin opened this issue Sep 13, 2016 · 8 comments
Closed

order_by_cols: '[' is not a valid choice for this field #1100

lihemin opened this issue Sep 13, 2016 · 8 comments
Labels
validation:required A committer should validate the issue

Comments

@lihemin
Copy link

lihemin commented Sep 13, 2016

add order_by_cols to 0.9.1 from master cause error :order_by_cols: '[' is not a valid choice for this field

@xrmx
Copy link
Contributor

xrmx commented Sep 13, 2016

@lihemin Could you please update the report with:

  • which version of caravel you are using
  • what you are doing and what you get, if it's an error please post the backtrace from the log

@xrmx xrmx added the validation:required A committer should validate the issue label Sep 14, 2016
@xrmx
Copy link
Contributor

xrmx commented Sep 21, 2016

One week without feedback, closing.

@xrmx xrmx closed this as completed Sep 21, 2016
@jasonshah
Copy link

@xrmx I see this issue as well.
I have created a slice that is a table view with 3 "non-grouped-by" columns and 1 ordered column ("collectionid [desc]").
The slice itself executes correctly when viewing it in isolation.
However, when I add it to a Dashboard and open the Dashboard, I see the same error message in red:

order_by_cols: '[' is not a valid choice for this field

The only thing that looks like an error in the webserver output appears when I refresh the slice:

/dev/temp/caravel/venv/lib/python2.7/site-packages/sqlalchemy/sql/compiler.py:575: SAWarning: Can't resolve label reference u'collectionid'; converting to text() (this warning may be suppressed after 10 occurrences)
  util.ellipses_string(element.element))

I see no errors when I refresh the Dashboard.
(Is there another location to find error logs, other than the Terminal window in which I ran caravel runserver?)

I'm running Caravel 0.10.0, installed via pip on OS X.

@xrmx
Copy link
Contributor

xrmx commented Oct 3, 2016

@jasonshah could you please retry with latest git master?

@xrmx xrmx reopened this Oct 3, 2016
@jasonshah
Copy link

jasonshah commented Oct 12, 2016

I'm unable to follow the "Making your own build" section from http://airbnb.io/caravel/installation.html, because I get a dependency error on npm install (per this issue). Right now I have to stick to the prepackaged versions distributed through pip...

@xrmx
Copy link
Contributor

xrmx commented Oct 13, 2016

@jasonshah 0.11 is more or less what was master at the time :)

@jasonshah
Copy link

While I wasn't able to test it on the exact same test case (changes in the datasource and what not), I attempted to replicate this in a slightly different manner in 0.11, and it seems to work there with no errors.

@mistercrunch
Copy link
Member

Closing for now, reopen if you see this on 0.12.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation:required A committer should validate the issue
Projects
None yet
Development

No branches or pull requests

4 participants