search

mrilikecoding / StreamPoseML

pose estimation / ML model wrapper and annotation utility
MIT License
2 stars 2 forks source link

JOSS paper #14

Closed imcatta closed 1 week ago

imcatta commented 8 months ago

Hi! I'm one of the reviewers of your JOSS submission openjournals/joss-reviews#6392 This PR is intended to keep track of things you might want to revise in the paper.

  1. While the pipeline shown in Figure 1 is very interesting and common to various applications of ML pose estimation, I think some elements of the last two steps are very context specific, resulting in a restriction of the otherwise broad generability of the workflow. In particular, they are: (a) inference on videos captured via the web, since one can classify videos from other sources; (b) sending results via Bluetooth.
  2. [Line 34] Missing reference to the figure.
  3. [Line 44] A reference to Google's Teachable Machine should be added.
  4. [Figure 1] The text in the figure is small compared to the font size of the rest of the manuscript.
mrilikecoding commented 1 week ago

Hello @imcatta, thanks, and apologies for the very long delay. I believe I have addressed this now. I ended up removing the figure for a few reasons in favor of a text list. I added the reference to teachable machine. Additionally, I noticed the paper action was deprecated so updated that and the paper seems to be compiling now.