-
Notifications
You must be signed in to change notification settings - Fork 0
Review of Guideline #38
| Item | Outcome |
|---|---|
| Guideline complies with the Guideline-Template | Yes |
| Name of guideline responsible with affiliation is clearly stated | Yes |
Yes
Yes
Maybe add a few more keywords like application mapping, GPGPU, accelerators etc. since it is more focused towards this.
Insight is well written; starting with the target implementation, the problem statement and why it is necessary to solve it. The implementation method described is a bit general, which is helpful in understanding the issues faced. The guideline is fairly clear and technical in terms of the problem faced but adding more details on data organization and application modification to improve maximum chip utilization and performance would be helpful. The solution proposed is too generic and can be explained more, so it becomes more helpful for other users who want to adapt it.
Does the guideline explicitly refer to the handbook? To which part of the deliverable is it relevant (e.g., chapter of D1.2/D1.3)?
No. It should be mentioned.
Yes, the guideline specifies the work in the project that can benefit.
-
Guideline advice: There are some typos that can be improved.
-
Insights that led to the guideline: see above.
-
Recommended implementation method of the guideline along with a solid motivation for the recommendation: Describe more about the implementation method.
-
Instantiation of the recommended implementation method in the reference platform: Brief and precise.
-
Evaluation of the guideline in reference applications: Briefly explained.
-
References: OK
- 03/08/2018: Guideline created
- 30/11/2018: guideline reviewed.
- 30/01/2019: guideline updated
TULIPP Guideline Wiki