Skip to content

Conversation

@umbriquse
Copy link

I've come across a use case where I need to change the functionality for batch_updates!, learning_status, and some more functions. I currently just overload the function itself, causing a warning during compilation of my package. It would be ideal if I can perform type dispatch to use the custom functions instead of overloading your functions.

@umbriquse umbriquse changed the title Allow Type Dispatch on AlphaZero.Trainer Allow Type Dispatch on Functions Using AlphaZero.Trainer Feb 1, 2022
@umbriquse umbriquse changed the title Allow Type Dispatch on Functions Using AlphaZero.Trainer [WIP] Allow Type Dispatch on Functions Using AlphaZero.Trainer Feb 3, 2022
@umbriquse umbriquse changed the title [WIP] Allow Type Dispatch on Functions Using AlphaZero.Trainer Allow Type Dispatch on Functions Using AlphaZero.Trainer Feb 3, 2022
@jonathan-laurent
Copy link
Owner

This is a good suggestion, thanks!
In general, I think the current version of AlphaZero.jl misses an opportunity to leverage multiple dispatch for enabling various extensions to the main algorithm. This will be one of my primary goals as I work on the next version.

@umbriquse
Copy link
Author

Will the next version be a completely new package, or a branch/fork that will eventually be merged?

@jonathan-laurent
Copy link
Owner

It will be a branch that will be merged. I haven't started working on it yet but I am writing down some ideas.
Feel free to share suggestions!

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.

2 participants