Jessica Scheick
Jessica Scheick
> We'll have to investigate a bit on our end (NSIDC). I'll let @JessicaS11 weigh in about how/whether to handle a case like this within icepyx. Thanks for looking into...
Thanks @jrenrut! Unfortunately we don't have CI running a lot of tests on the read module yet, but I have a script I can run locally to test this fix...
> @JessicaS11 that script sounds like a great candidate for an integration test! Do you think it can be automated? I can take that on. Thanks! It could definitely be...
> Do we need tests for this? Funny you should ask - it turns out this functionality wasn't tested to begin with, so @eni-awowale and I are actually working on...
Reference note: QC filtering specifically is covered [on this page](https://argovis.colorado.edu/apiintro)
> Special thanks to Chuck Daniels at Development Seed (@chuckwondo) for this work! > > In this workflow, integration tests will fail when triggered by a person without permissions. The...
> Just a minor clarification: regardless of whether or not the PR is from a fork, as long as the author has write permission to the original repo (i.e., maintainers),...
> but I think not having automated integration tests runs on PRs is a fairly high cost to pay. Agreed, so I'm wondering if we're using terminology the same way....
@all-contributors add @chuckwondo for infra
@mfisher87 Any idea if this was rebased after merging #604? Trying to determine where to start for fixing merge conflicts and moving this forward.