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

params() is no longer supported in Flux #996

Closed
halyusuf25 opened this issue Oct 24, 2023 · 1 comment
Closed

params() is no longer supported in Flux #996

halyusuf25 opened this issue Oct 24, 2023 · 1 comment

Comments

@halyusuf25
Copy link

halyusuf25 commented Oct 24, 2023

When trying to run any example experiment with neural net i got the following error:
ERROR: UndefVarError: params not defined

tracing back the error and found that it always come from the Flux function params, for example when running the JuliaRL_BCQD_CartPole.jl experiment the error comes from neural_network_approximator object and more specifically from this function
function Base.copyto!(dest::ActorCritic, src::ActorCritic) Flux.loadparams!(dest.actor, params(src.actor)) Flux.loadparams!(dest.critic, params(src.critic)) end

After some google search i found that params() function was supported in Flux <= version 0.12 but it was removed in version 0.13
https://fluxml.ai/Flux.jl/stable/training/reference/#Flux.params
i tried "using Flux: params" but it gives me the same error...

im creating this issue because this change is impacting almost all experiments examples.

@HenriDeh
Copy link
Member

I'm closing since this is a duplicate of #982

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