Fix double definition of openephys warnings

[alejoe91]

No description provided.

[Copilot]

Pull Request Overview

This PR removes a redundant inline import of the warnings module in the OpenEphys extractor’s deprecation warning block.

• Removed the duplicated import warnings inside the if load_sync_channel: block
• Left the deprecation message assignment unchanged

Comments suppressed due to low confidence (2)

src/spikeinterface/extractors/neoextractors/openephys.py:163

• The deprecation message is assigned to warning_message but never emitted. Add a call to warnings.warn(warning_message, DeprecationWarning) (or a logger) after constructing the message to actually surface the deprecation warning.

warning_message = (

src/spikeinterface/extractors/neoextractors/openephys.py:163

• Consider adding a unit test to verify that the deprecation warning is emitted when load_sync_channel is true, ensuring this behavior remains covered.

warning_message = (

[alejoe91]

This actually results in an UnboundLocalVariable error in case setting timestamps fails...

[zm711]

That's really weird. I thought it would just find the package in some sort of python cache.... But doesn't hurt to delete unnecessary import :)

[alejoe91]

@vncntprvst FYI