Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Simon Chu's Critique Post #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sychoo
Copy link

@sychoo sychoo commented Feb 18, 2021

Copy link
Owner

@bvasiles bvasiles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, albeit a bit too brief and mechanical. Write it more like a blog post geared at convincing someone that this paper was worth reading.


# Overview of Study Design and Methods

The study uses quasi-experiments to perform the study on two classes of subjects, a group of synthetic networks, and emergency management systems (EMS) -- a large-scale industry system.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What makes this a quasi-experiment? Can you be more specific?


None of the research in optimizing configuration for multiple criteria addressed the online (dynamic-reconfiguring) SDN, instead, most of them addressed offline, and in the domain of system design or architecture.

## Hook
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The hook is more about articulating why filling the gap is important. Your text reads more like "gap" (what is unknown given the existing literature) than "hook" (why is filling said gap important). Take a look again at the Lingard readings listed here.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please edit to clarify

@bvasiles bvasiles self-assigned this Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants