-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Hi,
I think the topic of prediction merger success is very interesting in that both what it reveals about successful/failed mergers and its applications in the field of finance. After reading your report, I have the following feedback:
What you did well:
- The paper was neatly sectioned and utilized an appendix to de-clutter the main report with tables and large figures.
- I like how you guys tried out a lot of different types models to see which one would work instead of just one model with different parameters and regularizers.
Things to improve on:
- Overall the report was very terse and would benefit a lot from explaining more of your approach (e.g why could you group all non "Completed" and "Pending" statuses as "Failed, or why you chose 0 as your NA value). Furthermore, more discussion on why you chose certain models and their parameters can be enlightening and also help the reader try to recreate your results.
- Even though you guys made several nice plots, there were no analysis on them aka why was it important for you to include those plots? Even a simple figure label with a short caption would go a long way!
- I think that all of the foreshadowing about future work in the paper should all be moved to the final section. That way your main report talks about what you've done without any distractions, and your future work section can be more fleshed out.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels