`ImagingPlane` API docs claims that excitation lambda can be None in contradiction with the schema

[h-mayorquin]

The signature of ImagingPlane is here:

https://pynwb.readthedocs.io/en/stable/pynwb.ophys.html#pynwb.ophys.ImagingPlane

In the API docs the ImagingPlane indicates that excitation lambda can be equal to None but this is no the case in the schema

https://github.com/NeurodataWithoutBorders/nwb-schema/blob/ade50ef33446beb3c7df4c6f1072ae0e821b5115/core/nwb.ophys.yaml#L216-L217

The API here in pynwb does not allow it either so it seems that this only a documentation issue.

[rly]

See comments in

pynwb/src/pynwb/ophys.py

Lines 134 to 146 in 387f80b

	# Note: device, excitation_lambda, indicator, and location are required arguments.
	# Description was made to be optional in PyNWB 3.1.0, however to avoid breaking API changes the order of
	# the arguments needs to be maintained even though the optional arguments came before the required ones.
	# So in docval these required arguments are displayed as optional when really they are required.
	# This section can be removed when positional arguments are no longer allowed.
	if args_to_set['device'] is None:
	raise ValueError("The 'device' argument is required for ImagingPlane.")
	if args_to_set['excitation_lambda'] is None:
	raise ValueError("The 'excitation_lambda' argument is required for ImagingPlane.")
	if args_to_set['indicator'] is None:
	raise ValueError("The 'indicator' argument is required for ImagingPlane.")
	if args_to_set['location'] is None:
	raise ValueError("The 'location' argument is required for ImagingPlane.")

This will be fixed in PyNWB 4.0 when positional arguments are no longer allowed for ImagingPlane.__init__.

[h-mayorquin]

Thanks, this makes sense.