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

Issue 1158 auto magic time labels #1178

Merged
merged 4 commits into from
Mar 27, 2015
Merged

Conversation

doutriaux1
Copy link
Contributor

No description provided.

it was receiving 'default' and hence returning None
now it converts 'default; to boxfill which really is default
1D data are still later converted back to a 1D default
fix #1158
two tests were inserted at the same location
@doutriaux1
Copy link
Contributor Author

@dlonie @aashish24 @chaosphere2112 please review
goes with CDAT/uvcdat-testdata#30

@alliepiper
Copy link
Contributor

test_vcs_lon_axes_freak_out fails for me, but it looks like the baselines just didn't get merged. Otherwise LGTM. I'll merge the other testdata in, too.

alliepiper pushed a commit that referenced this pull request Mar 27, 2015
@alliepiper alliepiper merged commit 2ca2b68 into master Mar 27, 2015
@aashish24
Copy link
Contributor

thanks David!. I am deleting the branch as well.

@aashish24 aashish24 deleted the issue_1158_auto_magic_time_labels branch March 27, 2015 19:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants