make_fixed_length_epochs() should allow for custom specification of the event ID

Currently, make_fixed_length_epochs() assigns all events the ID 1.

make_fixed_length_events(), on the other hand, already supports user-defined IDs.

It would be good to add a parameter to make_fixed_length_epochs(), allowing users to pass a custom event ID.

This was brought up on the forum, https://mne.discourse.group/t/fixed-epochs-and-fixed-events/3702

[mscheltienne]

@hoechenberger Not sure either about the issue encountered by OP on discourse. I was ticking on:

I have different events with different lengths of time. I want to make each second of each event as an epoch for classification purposes later.

For me, my description above is not really an issue. I can think of one or 2 different ways to do it, and your proposition would work like a charm. I was wondering if this scenario was encountered often, especially in online paradigms or in paradigms that have an alternation of long phases; in which case, a specific function to create this type of fixed-length epochs could be a good addition to MNE. As OP wrote on Discourse:

Any suggestion that makes my life easier?

---

Anyway, that said, and since I polluted a bit too much the discussion feed of this issue, I’ll add the additional parameter to make_fixed_length_epochs() to change the event ID; at least this small upgrade won’t be left out because of my post 😉

[hoechenberger]

Closed via #9782