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

add grid spacing and the ratio of grid spacing to chipsize0 so that adjacent chips can be dependent (finer sampling grid) #23

Merged
merged 5 commits into from
Apr 2, 2021

Conversation

leiyangleon
Copy link
Collaborator

@leiyangleon leiyangleon commented Mar 26, 2021

  • read GridSpacingX from the spacing of the DEM grid, while ChipSize0X is the smallest chip size. They are independent of each other, i.e. GridSpacingX can be either greater than or less than ChipSize0X
  • define the ratio of ChipSize0X to GridSpacingX
  • update NDC filter parameter FracValid to take account of the overlap between adjacent chips when GridSpacingX is less than ChipSize0X (note the overlap is constant when chip size iteratively progresses, thanks to the nested grid design)
  • update NDC filter parameter FiltWidth to cover a consistent area no matter what GridSpacingX is used in relative to ChipSize0X
  • update sparse search sample rate with the ratio of ChipSize0X to GridSpacingX to skip the same distance in the sparse search no matter what GridSpacingX is used in relative to ChipSize0X

@leiyangleon leiyangleon merged commit 9090922 into master Apr 2, 2021
@leiyangleon leiyangleon mentioned this pull request Apr 2, 2021
@jhkennedy jhkennedy deleted the gridspacing_chipsize0_ratio branch September 16, 2022 00:11
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.

1 participant