Update foraging eff

[micahwoodard]

Pull Request instructions:

• Please follow the update protocol
• Answer the questions below in detail. Your responses will be emailed to experimenters.
• If the experimenters must do anything new, provide detailed step by step instructions on the wiki
• If computer maintainers need to manually update anything, provide detailed step by step instructions
• Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
• If you use the keyword "skip email" in the title, it will skip the email updates
• Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
• Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

• adds aind-dynamic-foraging-basic-analysis as a dependency and uses compute_foraging_efficiency to calculate foraging efficiency

What issues or discussions does this update address?

• resolves Problem in computing foraging efficiency online #272

Describe the expected change in behavior from the perspective of the experimenter

• None

Describe any manual update steps for task computers

• None

Was this update tested in 446/447?

• 446/7

Does this update impact downstream processing by adding new saved files, or changing their format? If so, have you documented changes?

• None

[micahwoodard]

@hanhou I'm unsure if I used the correct values for the random_number arg

[alexpiet]

This repo is on Pypi, and has release tags

[alexpiet]

So is this one aind-dynamic-foraging-models

Which you might want to also install from pypi because we recently fixed a dependency issue between foraging models and basic-analysis. I'm not sure if the way you have it now will create a problem, but I think its safer to use pypi

[alexpiet]

@hanhou can you review how the foraging efficiency is being called?