Skip to content

Conversation

harwiltz
Copy link
Contributor

I encountered a dependency issue when trying to run some experiments that appears to be caused by an update to the CircularArrayBuffers package. My project environment had CircularArrayBuffers@0.1.7 installed and was causing the following error when I tried to run the JuliaRL_IQN_CartPole experiment (among others):

ERROR: MethodError: no method matching _buffer_frame(::CircularArrayBuffers.CircularVectorBuffer{Float32, Vector{Float32}}, ::Matrix{Int64})

You might have used a 2d row vector where a 1d column vector was required.
Note the difference between 1d column vector [1,2,3] and 2d row vector [1 2 3].
You can convert to a column vector with the vec() function.
Closest candidates are:
  _buffer_frame(::CircularArrayBuffers.CircularArrayBuffer, ::Int64) at ~/.julia/packages/CircularArrayBuffers/FWC6A/src/CircularArrayBuffers.jl:70
  _buffer_frame(::CircularArrayBuffers.CircularArrayBuffer, ::AbstractVector{<:Integer}) at ~/.julia/packages/CircularArrayBuffers/FWC6A/src/CircularArrayBuffers.jl:80

The problem resolves itself as soon as I downgrade CircularArrayBuffers.

This is probably more like a band-aid than a permanent fix, its probably just a simple change somewhere. But I'm not very familiar with CircularArrayBuffers so I thought this fix could be helpful in the meantime.

@findmyway
Copy link
Member

Thanks for reporting. Should be fixed in JuliaReinforcementLearning/CircularArrayBuffers.jl#11

Please update the dependency to the v0.1.8 once JuliaRegistries/General#56499 get merged.

@findmyway
Copy link
Member

@all-contributors please add @harwiltz for bug

@allcontributors
Copy link
Contributor

@findmyway

I've put up a pull request to add @harwiltz! 🎉

@harwiltz
Copy link
Contributor Author

@findmyway great, will do. Thanks!

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