Jessica Scheick

Results 76 comments of Jessica Scheick

> Ok, but I might as well add some unit tests for ATL14 since I've been playing with it recently. Will see what I can do to increase code coverage....

thanks @sjh1024 - this is looking great! I left a few comments, and noticed that a bunch of the new tests are failing. Let me know if you need help...

@all-contributors add @sjh1024 for bug, code, doc, maintenance, test

> 2\. Why not add the spatial class (including validation) directly to `geospatial.py`? I'm wondering if this will make it more streamlined when we get to Could you remind me...

In query.py (circa lines 1010 and the next else), there is a line `geom_filepath=self._geom_filepath` that needs to be updated to `geom_filepath=self._spatial._geom_file`. GitHub won't let me comment directly on the non-altered...

Thanks for getting in touch @jameshgrn! Could you please provide the exact code you are running and which line is running into the error? I put the bounding box and...

I tried to visualize your region using icepyx's `Visualization` module, which calls OA under the hood, and I started running into a no data error. My current guess at what's...

> Hey all! Sure, I'd be into helping out! Would you like to review the Pull Request (#220) @icetianli put together? I will add you to the icesat2py GitHub organization...

@jameshgrn Essentially, yes. [GitHub Docs](https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews) has some great info on PR reviews, including lots of images that show how to do various review tasks. I'd also be happy to walk...