Fix interaction between CircularPrioritizedTraces and NStepBatchSampler

[jeremiahpslewis]

• Fixes issue where dispatch on Circular Trajectories was not working for certain samplers, this I believe has been causing CI for Rework the run loop ReinforcementLearning.jl#921 to fail as type dispatch fails to match :priority and :key keys

[jeremiahpslewis]

@HenriDeh now running into an issue where sampleable_inds is length(x) + 2 of the trace length, but it should be length(x) + 1), any ideas why? Capacity for EpisodeBuffer wasn't correctly set.

[codecov]

Codecov Report

Merging #44 (9ee7cda) into main (e3179da) will increase coverage by 1.24%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   70.63%   71.87%   +1.24%     
==========================================
  Files          15       15              
  Lines         630      640      +10     
==========================================
+ Hits          445      460      +15     
+ Misses        185      180       -5     

Files Changed	Coverage Δ
src/common/CircularPrioritizedTraces.jl	84.84% <50.00%> (+0.47%)	⬆️
src/episodes.jl	92.59% <100.00%> (+0.59%)	⬆️
src/samplers.jl	72.00% <100.00%> (+8.11%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[jeremiahpslewis]

@HenriDeh Merged this as it only adds tests and fixes functionality.

[HenriDeh]

Hi @jeremiahpslewis, sorry I was on vacations. I'll catch up when I resume work on Monday.

[jeremiahpslewis]

@HenriDeh I figured as much. Hope you had a great trip!