Move the year check outside of event_selection's trg_pass_no_overlap()

[kmohrman]

Regarding this code:

    # The triggers for 2016 and 2016APV are the same
    if year == "2016APV":
        year = "2016"
    #The trigger list for 2022 and 2022EE are the same
    if year == "2022EE":
        year = "2022"

I think it would be better to handle this on the users' side (where the function is called) rather than inside of this function. The year is used when we use it as a key for the dataset dictionary (but the dataset dictionary is also passed to this function). So the handling of which year goes with which key should go on the user side, not hard coded within this function.