Update metadata

[XX-Yin]

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:

• https://github.com/AllenNeuralDynamics/dynamic-foraging-task/pull/1051
• [x] "Lick sensor left" and "Lick sensor right" are not daq_names for your rig and thus can't be listed as under daq_names. (Removed the behavior from the data stream and added the behavior to the stimulus epoch instead. And no daq names required.)
• [x] Camera names don't match the names in the rig. They must match. There is a utility function you can use to make sure that the names align between rig and session.
• [x] Reward delivery cannot be null. How are you delivery rewards without it? (We already have the code to generate the reward delivery, and we are waiting for the SOP to collect the reference position and reference area. )
• [x] Your session_end_time is before the end of one of your data streams. The end time should be the last time of all data streams and stimulus epochs.
• [x] Missing stimulus_device_name
• [x] These output parameters and task parameters duplicate almost the entire metadata schema. Can we please please please not do this? Very seriously, this is going to make our DocDB painful and it ultimately undermines having the schema in the first place.
• [x] eg. calibrations should be in the calibration field
• [x] frame rate should be in the device (also, check the version of the schema you're using. The cameras in your device class have a max_frame_rate which was changed a while ago) (The max_frame_rate has been removed and replaced by frame_rate)
• [x] Your frame_num item specifically is blocking upload to DocDB. You can't have fields with ".csv" in the name. This information should live in the quality_control class (which was only recently added to the schema, but that's where it should live) (These fields should go to the qc and they are removed here)
• [x] adding software name to the stimulus_epochs
• [x] mismatch harp names

  What issues or discussions does this update address?

  resolved https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/510

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

  No

  Describe any manual update steps for task computers

  No

  Was this update tested in 446/447?

  On my own computer

[alexpiet]

Other than the name issue, this seems good to me. Can you share a new rig.json and session.json for approval from Saskia?

Are there any other issues left to implement, or is this ready to go?

[XX-Yin]

Other than the name issue, this seems good to me. Can you share a new rig.json and session.json for approval from Saskia?

Are there any other issues left to implement, or is this ready to go?

It's ready for review. I will generate a new session and rig metadata tomorrow for review

[alexpiet]

@XX-Yin Can you add details to the PR about what issues were fixed in the metadata? Probably just copying the itemized list from the issue is sufficient.