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.

Sign up for GitHub

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

Jump to bottom

Use sf rather than sp in polymask() #253

Merged

thomased merged 3 commits into master from replace-sp

Sep 24, 2023

Collaborator

Bisaloo commented Sep 10, 2023 •

edited

Loading

I need to double check all works fine on a real case but feel free to already have a look whenever convenient.

Fix #252

Bisaloo added 3 commits

September 10, 2023 12:22


          Add basic test for polymask()

d68c082


          Use sf rather than sp

8c06458


          Improve test and comments

22df25c

Bisaloo changed the title ~~Add basic test for polymask()~~ Use sf rather than sp in polymask()

Collaborator

thomased commented Sep 10, 2023

Lovely - thanks! I've had a further mess around with it, and it seems to be doing all that's needed. Appreciate it.

thomased merged commit 07031ce into master

5 of 7 checks passed

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

Labels

None yet