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 support for selective dilation #39

Open
dvolgyes opened this issue Feb 7, 2022 · 1 comment
Open

add support for selective dilation #39

dvolgyes opened this issue Feb 7, 2022 · 1 comment

Comments

@dvolgyes
Copy link

dvolgyes commented Feb 7, 2022

Description

It would be very nice to replace the current dilation options
with a more intelligent one.
Currently the number of dilations can be specified,
and the dilations always increase.
There are multiple problems with this, but the two largest ones:

  • you cannot set the dilation
  • you cannot set the order

So instead of the current integer input it would be nice to accept two kind of inputs:

  • integer, which has the same meaning as currently
  • tuple, where the length of the tuple means the number of dilation steps,
    and the numbers represent the pixel distance / dilation.
    E.g. 1,2,4,1 means 4 blocks, with dilation of 1,2,4 and 1.

There are good arguments that a final refinement could be very helpful
to remove small scale noise. Also, the receptive field is better tunable
this way.
The implementation is relatively straightforward.

@fepegar
Copy link
Owner

fepegar commented Mar 19, 2022

Hi, @dvolgyes. Sounds good. Would you like to contribute with a PR?

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

2 participants