Skip to content
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.

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

Optimize planetry recipes for keeping highest resolutions #174

Merged
merged 3 commits into from
Jan 9, 2023

Conversation

PaulWessel
Copy link
Member

See #171 for background. I have solved the issue by determining the exact increment and tile size required for the highest resolution, which gets a name based on the floor of the actual increment. Need floor since 2.56 cannot be rint to 3 since we also will make a 03m grid. Testing this now but seems to work so want to get these into the repo for further testing.

See #171 for background.  I have solved the issue by determining the exact increment and tile size required for the highest resolution, which gets a name based on the floor of the actual increment.  Need floor since 2.56 cannot be rint to 3 since we also will make a 03m grid.
Testing this now but seems to work so want to get these into the repo for further testing.
@PaulWessel PaulWessel added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 9, 2023
@PaulWessel PaulWessel self-assigned this Jan 9, 2023
Copy link
Member

@Esteban82 Esteban82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine. I just update the date on two recipes.

@PaulWessel PaulWessel merged commit f54d064 into master Jan 9, 2023
@PaulWessel PaulWessel deleted the more-planet-incs branch January 9, 2023 16:40
Esteban82 added a commit that referenced this pull request Mar 18, 2023
See #187 for details. 
With #174 a typo was made when editting this file. The variable $REG  was replaced by $IREG which doesn't exist. I changed back to $REG.
@Esteban82 Esteban82 mentioned this pull request Mar 18, 2023
PaulWessel pushed a commit that referenced this pull request Mar 18, 2023
See #187 for details. 
With #174 a typo was made when editting this file. The variable $REG  was replaced by $IREG which doesn't exist. I changed back to $REG.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants