Jessica Scheick

Results 206 comments of Jessica Scheick

> Rebased this PR (and https://github.com/icesat2py/icepyx/pull/612 and https://github.com/icesat2py/icepyx/pull/613) and fixed merge conflicts! Thanks, @mfisher87! This PR is pretty close - a few suggestions where we may need to make type...

Looks like you've encountered your own flavor of https://github.com/nsidc/earthaccess/issues/453... We never did figure out what, exactly, was happening between icepyx + earthaccess + IDEs, but after a few debugging sessions...

Just picking up on this thread as I explore how we could replace icepyx.Query with earthaccess.search (https://github.com/icesat2py/icepyx/issues/575). One of the things I'm finding (here temporal is the relevant input, but...

> Looks like this PR is ready-to-go pending https://github.com/icesat2py/icepyx/pull/605 Any idea why this shouldn't go directly into development?

> > > Looks like this PR is ready-to-go pending #605 > > > > > > Any idea why this shouldn't go directly into development? > > This work...

Just wanted to add a note that we're going to port in the remainder of the changes manually for the repo linked above ("current behavior") because we need it for...

Thanks for the fix, @AndreasAugustin! I totally understand (darn those pesky bills!). I look forward to trying it out.

> Looks like something isn't right :) Looking in to it! I'm noticing that the checks ran (successfully, and interestingly included the integration test) after the precommit auto fixes (https://github.com/icesat2py/icepyx/pull/591/commits/06564c11b8083a7c782f215c1867373f023909e5)....

Maybe open PRs are just going to be a bit grumpy at us until we're into ones opened after the actions were implemented?

## Review checklist for @JessicaS11 ### Conflict of interest - [x] I confirm that I have read the [JOSS conflict of interest (COI) policy](https://github.com/openjournals/joss/blob/master/COI.md) and that: I have no COIs...