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

boxfill not fully plotted (lat bit white) #627

Closed
doutriaux1 opened this Issue Sep 10, 2014 · 7 comments

Comments

Projects
None yet
3 participants
@doutriaux1
Member

doutriaux1 commented Sep 10, 2014

No description provided.

@doutriaux1 doutriaux1 self-assigned this Sep 10, 2014

@doutriaux1 doutriaux1 added this to the 2.0.0 milestone Sep 10, 2014

@doutriaux1

This comment has been minimized.

Member

doutriaux1 commented Sep 10, 2014

fixed in 627_vcs_bfill_last_bit_white
@aashish24 please review and merge

@aashish24

This comment has been minimized.

Contributor

aashish24 commented Sep 10, 2014

@doutriaux1 Would have been nice to have a test for it. Let's not close it until then.

@aashish24

This comment has been minimized.

Contributor

aashish24 commented Sep 11, 2014

Looking into it now 🐸

@aashish24

This comment has been minimized.

Contributor

aashish24 commented Sep 11, 2014

WOW getting dv3d tests now as well... great job @ThomasMaxwell

@aashish24

This comment has been minimized.

Contributor

aashish24 commented Sep 11, 2014

Looks good.. 👍 but since no test. I am keeping it open ..

@williams13

This comment has been minimized.

Contributor

williams13 commented Sep 11, 2014

Let us know when we can rebuild.

From: Aashish Chaudhary <notifications@github.commailto:notifications@github.com>
Reply-To: UV-CDAT/uvcdat <reply@reply.github.commailto:reply@reply.github.com>
Date: Wednesday, September 10, 2014 6:26 PM
To: UV-CDAT/uvcdat <uvcdat@noreply.github.commailto:uvcdat@noreply.github.com>
Subject: Re: [uvcdat] boxfill not fully plotted (lat bit white) (#627)

WOW getting dv3d tests now as well... great job @ThomasMaxwellhttps://github.com/ThomasMaxwell


Reply to this email directly or view it on GitHubhttps://github.com//issues/627#issuecomment-55207845.

@aashish24

This comment has been minimized.

Contributor

aashish24 commented Sep 11, 2014

Not yet.. @doutriaux1 has to be merge this branch for me: #622

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment