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

Add Dimension.label replacing global alias state #1083

Merged
merged 18 commits into from
Jan 30, 2017

Conversation

philippjfr
Copy link
Member

Addresses #1001 and supersedes #1075.

@philippjfr
Copy link
Member Author

This is now passing without any major issues. This needs a lot more unit tests still and @jlstevens promised to write a short tutorial but we're both very happy with how this will work out.

@jbednar
Copy link
Member

jbednar commented Jan 27, 2017

Looks good to me. Some examples of usage would be good.

@philippjfr
Copy link
Member Author

philippjfr commented Jan 27, 2017

Yes, those should be part of the tutorial. I'd actually like to suggest to broaden the scope of that tutorial to discuss Dimensions in general, with a significant section devoted to name and label but also covering the redim method and the difference between range and soft_range.

Another thing I've done in this PR is using strict dimension lookups in user facing APIs, which should result in much better tracebacks when referencing non-existent dimensions in many cases.

@philippjfr philippjfr changed the title Dimension aliases switched Add Dimension.label replacing global alias state Jan 27, 2017
@jlstevens
Copy link
Contributor

Thanks @philippjfr!

I'll shortly write up a notebook tutorial describing aliases, redim and the distinction betweenrange soft_range and add it to this PR.

@jlstevens
Copy link
Contributor

jlstevens commented Jan 30, 2017

I now have part of a Dimensions tutorial but I would like to work on #843 before continuing work on it. I am happy with this PR and would like to merge it now. Are you ok with me adding the Dimensions notebook to a different PR?

@philippjfr
Copy link
Member Author

Sure, let's see how it goes. Better to get it in early to find issues.

@jlstevens jlstevens merged commit bfdad59 into master Jan 30, 2017
@philippjfr philippjfr deleted the dimension_aliases_switched branch February 10, 2017 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants