I actually think this should have been two films. I think all the exposition they tried to cram in to get you backstory in the first 45 minutes should have been in a separate film, allowing us to jump off with the rescue mission of Erso and the trip to Jedha. Jyn’s childhood just didn’t fit in the film and I think it made it seem like they couldn’t find a better way to tell you how the Erso’s got to this point.
I liked how they intercut Rebel pilots from ANH into this film, with the nod to needing a new Red 5.
I wonder if Vader’s final scene will play out for us like Yoda’s scene in ATC did, it was kinda cool to see the old Jedi in a saber battle but in subsequent viewings that appeal wore off.