Skip to content

Commit 4812b07

Browse files
authored
Ephys Improvements (#979)
* initial commit * examples * Update procedures.py * adding `default=` field * linters * adding `default` to any optional field missing it
1 parent 264ffde commit 4812b07

File tree

7 files changed

+17
-17
lines changed

7 files changed

+17
-17
lines changed

examples/bergamo_ophys_session.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,6 @@
162162
"headframe_registration": null,
163163
"reward_delivery": null,
164164
"reward_consumed_total": null,
165-
"reward_consumed_unit": "microliter",
165+
"reward_consumed_unit": "milliliter",
166166
"notes": null
167167
}

examples/ephys_session.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -388,6 +388,6 @@
388388
"headframe_registration": null,
389389
"reward_delivery": null,
390390
"reward_consumed_total": null,
391-
"reward_consumed_unit": "microliter",
391+
"reward_consumed_unit": "milliliter",
392392
"notes": null
393393
}

examples/mri_session.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,6 @@
153153
"headframe_registration": null,
154154
"reward_delivery": null,
155155
"reward_consumed_total": null,
156-
"reward_consumed_unit": "microliter",
156+
"reward_consumed_unit": "milliliter",
157157
"notes": "There was some information about this scan session"
158158
}

examples/multiplane_ophys_session.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,6 +282,6 @@
282282
"headframe_registration": null,
283283
"reward_delivery": null,
284284
"reward_consumed_total": null,
285-
"reward_consumed_unit": "microliter",
285+
"reward_consumed_unit": "milliliter",
286286
"notes": null
287287
}

examples/ophys_session.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,6 @@
8787
"headframe_registration": null,
8888
"reward_delivery": null,
8989
"reward_consumed_total": null,
90-
"reward_consumed_unit": "microliter",
90+
"reward_consumed_unit": "milliliter",
9191
"notes": null
9292
}

src/aind_data_schema/core/procedures.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@ class Anaesthetic(AindModel):
277277
type: str = Field(..., title="Type")
278278
duration: Decimal = Field(..., title="Duration")
279279
duration_unit: TimeUnit = Field(default=TimeUnit.M, title="Duration unit")
280-
level: Decimal = Field(..., title="Level (percent)", ge=1, le=5)
280+
level: Optional[Decimal] = Field(default=None, title="Level (percent)", ge=1, le=5)
281281

282282

283283
class OtherSubjectProcedure(AindModel):
@@ -583,7 +583,7 @@ class Surgery(AindModel):
583583
description="First and last name of the experimenter.",
584584
title="Experimenter full name",
585585
)
586-
iacuc_protocol: Optional[str] = Field(None, title="IACUC protocol")
586+
iacuc_protocol: Optional[str] = Field(default=None, title="IACUC protocol")
587587
animal_weight_prior: Optional[Decimal] = Field(
588588
default=None, title="Animal weight (g)", description="Animal weight before procedure"
589589
)

src/aind_data_schema/core/session.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ class ManipulatorModule(DomeModule):
208208
"""A dome module connected to a 3-axis manipulator"""
209209

210210
primary_targeted_structure: str = Field(..., title="Targeted structure")
211-
other_targeted_structure: Optional[List[str]] = Field(None, title="Other targeted structure")
211+
other_targeted_structure: Optional[List[str]] = Field(default=None, title="Other targeted structure")
212212
targeted_ccf_coordinates: List[CcfCoords] = Field(
213213
default=[],
214214
title="Targeted CCF coordinates",
@@ -217,11 +217,11 @@ class ManipulatorModule(DomeModule):
217217
...,
218218
title="Manipulator coordinates",
219219
)
220-
bregma_coordinates: Optional[Coordinates3d] = Field(None, title="Bregma coordinates")
221-
surface_z: Optional[Decimal] = Field(None, title="Surface z")
220+
bregma_coordinates: Optional[Coordinates3d] = Field(default=None, title="Bregma coordinates")
221+
surface_z: Optional[Decimal] = Field(default=None, title="Surface z")
222222
surface_z_unit: SizeUnit = Field(SizeUnit.UM, title="Surface z unit")
223-
dye: Optional[str] = Field(None, title="Dye")
224-
implant_hole_number: Optional[int] = Field(None, title="Implant hole number")
223+
dye: Optional[str] = Field(default=None, title="Dye")
224+
implant_hole_number: Optional[int] = Field(default=None, title="Implant hole number")
225225

226226

227227
class FiberModule(ManipulatorModule):
@@ -511,7 +511,7 @@ class StimulusEpoch(AindModel):
511511
Field(discriminator="stimulus_type"),
512512
]
513513
]
514-
] = Field(None, title="Stimulus parameters")
514+
] = Field(default=None, title="Stimulus parameters")
515515
stimulus_device_names: List[str] = Field(default=[], title="Stimulus devices")
516516
speaker_config: Optional[SpeakerConfig] = Field(default=None, title="Speaker Config")
517517
light_source_config: Optional[List[LIGHT_SOURCE_CONFIGS]] = Field(
@@ -582,7 +582,7 @@ class Session(AindCoreModel):
582582
headframe_registration: Optional[Affine3dTransform] = Field(
583583
None, title="Headframe registration", description="MRI transform matrix for headframe"
584584
)
585-
reward_delivery: Optional[RewardDeliveryConfig] = Field(None, title="Reward delivery")
586-
reward_consumed_total: Optional[Decimal] = Field(None, title="Total reward consumed (uL)")
587-
reward_consumed_unit: VolumeUnit = Field(VolumeUnit.UL, title="Reward consumed unit")
588-
notes: Optional[str] = Field(None, title="Notes")
585+
reward_delivery: Optional[RewardDeliveryConfig] = Field(default=None, title="Reward delivery")
586+
reward_consumed_total: Optional[Decimal] = Field(default=None, title="Total reward consumed (mL)")
587+
reward_consumed_unit: VolumeUnit = Field(VolumeUnit.ML, title="Reward consumed unit")
588+
notes: Optional[str] = Field(default=None, title="Notes")

0 commit comments

Comments
 (0)