cartopy icon indicating copy to clipboard operation
cartopy copied to clipboard

Utilise Scaler class in GSHHSFeature

Open kaedonkers opened this issue 6 years ago • 12 comments

Rationale

Utilise Scaler object in GSHHSFeature like it is used in NaturalEarthFeature.

Implications

Code more readable and scale_from_extent now abstracted out to Scaler class.

kaedonkers avatar Aug 09 '18 10:08 kaedonkers

After more testing it seems I've broken GSHHSFeature. I'll dig a bit more...

kaedonkers avatar Aug 09 '18 14:08 kaedonkers

Any progress here?

QuLogic avatar Oct 16 '18 06:10 QuLogic

I will not get a chance to fix this in time for the v0.17 release so will push it back to v0.18.

kaedonkers avatar Nov 16 '18 09:11 kaedonkers

@kaedonkers, any update on trying to get this in for the v0.18 release?

greglucas avatar Feb 23 '20 02:02 greglucas

@greglucas Realistically, I probably won't get around to finishing this in time for the release. Go ahead with the release and if I get it in before you release then happy days.

kaedonkers avatar Mar 09 '20 14:03 kaedonkers

I re-ran the CI and there are failures all over regarding pip. Is this a known issue or have I messed up?

kaedonkers avatar Mar 09 '20 15:03 kaedonkers

I think you may have been unlucky, I don't see any changes you've made that would have caused any issues with installation.

greglucas avatar Mar 22 '20 00:03 greglucas

Thanks for looking. I've rebased the changes, let's see if that helps.

kaedonkers avatar Mar 23 '20 09:03 kaedonkers

Thanks for looking. I've rebased the changes, let's see if that helps.

kaedonkers avatar Mar 23 '20 09:03 kaedonkers

Uh oh! It looks like something went wrong in the rebase because you're bringing in a ton of commits and conflicts now. Try and give it another shot?

greglucas avatar Apr 02 '20 03:04 greglucas

Tagging the rc today, so I'm pushing this back to 0.19.

QuLogic avatar Apr 27 '20 03:04 QuLogic

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 13 committers have signed the CLA.

:white_check_mark: greglucas
:white_check_mark: pelson
:x: QuLogic
:x: ajdawson
:x: ocefpaf
:x: shevawen
:x: dopplershift
:x: htonchia
:x: zbruick
:x: pharshalp
:x: kacmak7
:x: kaedonkers
:x: Mark Wieczorek


Mark Wieczorek seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Apr 08 '24 16:04 CLAassistant