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 Smoothing Period in ADX #1019

Open
myalgomate opened this issue Feb 10, 2023 · 9 comments
Open

Add Smoothing Period in ADX #1019

myalgomate opened this issue Feb 10, 2023 · 9 comments
Labels
enhancement New feature or request

Comments

@myalgomate
Copy link
Contributor

In Continuation of the Discussion related to ADX.

#1015 (reply in thread)

@myalgomate myalgomate added the enhancement New feature or request label Feb 10, 2023
@myalgomate
Copy link
Contributor Author

Hi Dave
Hope you are doing well.
I just wanted to know if you could help me with the above issue.

Thanks in advance.

Regards,
Harsh

@DaveSkender
Copy link
Owner

DaveSkender commented Feb 20, 2023

Hi Dave Hope you are doing well. I just wanted to know if you could help me with the above issue.

Yes. I've put it in the backlog, so it is something I intend to implement, but not immediately. Given other more widely needed features, I do not expect to work on this anytime soon.

If you need it sooner, please consider contributing the code in a Pull Request. If you decide to take this one on, please let me know so I can reserve it for you.

@myalgomate
Copy link
Contributor Author

Hi Dave,

Sorry, but I didn't understand what you were saying.?
It would be really great if you can add smoothing parameters as arguments in ADX.

Thanks

@DaveSkender
Copy link
Owner

DaveSkender commented Feb 28, 2023

I cannot work on this now.
It’s not a priority.
I may work on it later.

@myalgomate
Copy link
Contributor Author

okay, Thank you.
Please do at your convenience.

@kresch23
Copy link

@DaveSkender I also need the Smoothing parameter in the ADX function. Can I possibly support that we can implement this?

@DaveSkender
Copy link
Owner

Can I possibly support that we can implement this?

If you want to create an overload method here and submit a pull request, I’d be able to review it. Contributions are greatly appreciated.

@kresch23
Copy link

@DaveSkender I have looked at the code and am apparently already confused about the structure. My knowledge is probably not sufficient :/ I would be willing to pay for you to implement this, would that be an option?

@DaveSkender
Copy link
Owner

I would be willing to pay for you to implement this, would that be an option?

Yes. Many of our sponsorship tiers come with prioritization of work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

3 participants