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.

Sign up for GitHub

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

Jump to bottom

create_workspace() should take kwargs instead of dict #35

Closed
Stiivi opened this issue Apr 23, 2012 · 0 comments
Closed

create_workspace() should take kwargs instead of dict #35

Stiivi opened this issue Apr 23, 2012 · 0 comments
Labels
usability
Milestone
0.9

Comments

@Stiivi
Copy link
Member

Stiivi commented Apr 23, 2012

Current state: backends.*.create_workspace(model, config) takes a config dictionary with configuration. This is fine for "non-user" calls of this function, such as from server. There might be situation when it is desired to create workspace programmatically and it is more convenient to be able to set arguments as of any other function.

Solution:: The signature should be backends.*.create_workspace(model, **config)
@Stiivi Stiivi closed this as completed in 30c13dd May 10, 2012
@Stiivi Stiivi mentioned this issue May 11, 2012
Closed
code6 pushed a commit to code6/cubes that referenced this issue Mar 13, 2014
@Stiivi
Backend options are in [workspace], no more configurable (closes Data…
7aff671 
…Brewery#52); Create workspace accepts keyword arguments instead of dict (closes DataBrewery#35)

Added depreciation warnings, still accepts [backend] and [db];
reflected documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
usability
Projects
None yet
Milestone
0.9
Development

No branches or pull requests
1 participant
@Stiivi