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

[BUG (soft)] fillholes_pushpull maybe should be aliased as fillholes_pullpush #4068

Open
alister-chowdhury opened this issue Dec 3, 2023 · 0 comments

Comments

@alister-chowdhury
Copy link
Contributor

alister-chowdhury commented Dec 3, 2023

Hey Larry,

So this isn't really a problem in practice, but it seems the algorithm is referred to as "pull push" by atleast a few papers.

I just noticed that in OIIO we refer to it as "push pull".

(Push and pull seems to be an unrelated thing that I won't pretend I understand: https://home.ubalt.edu/ntsbarsh/business-stat/opre/PushPull.pdf)

Does this matter?
Probably not, nobody else has noticed and it's clear what the intention is, certainly wouldn't be worth breaking existing code over.

But I figured it was probably atleast letting you know.

Many thanks,
Alister

@alister-chowdhury alister-chowdhury changed the title [BUG (soft)] fillholes_pushpull should be named fillholes_pullpush [BUG (soft)] fillholes_pushpull maybe should be aliased as fillholes_pullpush Dec 3, 2023
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

No branches or pull requests

1 participant