Added Epsilion Greedy#219
Added Epsilion Greedy#219JoeHolt wants to merge 1 commit intonextml:masterfrom JoeHolt:epsilion-greedy
Conversation
| default: 0.5 | ||
| optional: true | ||
|
|
||
| pT: |
There was a problem hiding this comment.
Why is this required? The YAML file is for algorithm inputs/outputs. pT is internal to the algorithm.
There was a problem hiding this comment.
I could not get butler to work/save parameters unless the variable in the yaml. I am not sure of the 'correct' way to get butler working.
There was a problem hiding this comment.
Hm... that's not how butler is supposed to work. It looks like this should work if this is removed beacuse pT is initialized. If it doesn't, could you give a traceback?
There was a problem hiding this comment.
I went back and removed the arg in myApp.yaml and it continued to work as expected. I am not sure what I was doing at the time of writing that...
|
|
||
| # Update t for next run | ||
| newT = t + 1 | ||
| butler.algorithms.set(key='pt', value=newT) |
There was a problem hiding this comment.
I think there's a typo here (pt vs pT). But maybe we could provide a longer name, maybe answers_received? That'd be a little easier to maintain.
There was a problem hiding this comment.
I agree, a more descriptive name would be better.
2 participants
Added basic epsilion greedy to CardinalBandits