Skip to content

Comments

Kris devel#8

Open
jaseeverett wants to merge 5 commits intomainfrom
kris_devel
Open

Kris devel#8
jaseeverett wants to merge 5 commits intomainfrom
kris_devel

Conversation

@jaseeverett
Copy link
Contributor

No description provided.

ktesturas and others added 5 commits February 12, 2026 09:05
Currently cleaning the scripts. Pushing these version so that the notes I had before (when trying to solve the problem on the number of valid and invalid patches (counting them) are saved. Now that this problem has been solved, these notes shall be removed in the cleaner version.
[ X ] Understand the cost layer in Minpatch
[ X ] Edit original vignette about the Cost [edit the vignette accordingly]
[ X ] Add page on boundary penalty => we must have to make sure it appears in the Vignette
[ X] Add length of perimeter to the table
[ X ] update PU cost in the table
[ X ] make the table as a function
[ X ] Finish the Tasmania Vignette Accordingly
[ X ] recheck pi for pull request
[ X ] experimenting with patch radius
@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

❌ Patch coverage is 10.83123% with 354 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
R/output.R 4.00% 312 Missing ⚠️
R/whittling_functions.R 40.00% 42 Missing ⚠️

📢 Thoughts on this report? Let us know!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants