so-wise / weddell_gyre_clusters

Unsupervised classification of Weddell Gyre profiles
MIT License
2 stars 1 forks source link

Minor revisions: figures and text #123

Closed DaniJonesOcean closed 1 year ago

DaniJonesOcean commented 1 year ago

Minor comments on text:

Suggestions for figure improvements:

DaniJonesOcean commented 1 year ago

Minor comments on text:

Line 39: do you mean to say “30°E, 70°E”?

Changed

Line 40: 60°S

Added

Line 112: suggest to include the reference for the data collection here

Added

Line 119: paper

Changed

Lines 124-125 and 198-204: In my opinion, these are unnecessary introductions to subsections, as was also pointed out by one reviewer. Please re-consider whether preambles are really needed.

Removed

Line 78 and 224: „in latitude and longitude“ seems unnecessary

Removed

Line 247: perhaps better say “thickness of the water column”

???

Line 331: parenthesis and period missing

Added

Line 372: „becomes“ is used twice in this sentence

Fixed

Line 446: an

Fixed

Section 3.5 in particular as well as other text passages: the word “relatively” is used somewhat inflationary

Great point! That word was heavily overused. Reduced to just four instances now.

---> Text improvements done

DaniJonesOcean commented 1 year ago

Suggestions for figure improvements:

Discrete colorbars: This may also be a matter of taste and hence suggestions rather than requests, but discrete increments generally provide more quantitative detail in the figures. While some of your figures employ discrete colorbars, this could be used to improve figs 2, 4-8 and 12 as well.

Excellent point. Now revised.

Fig 3: Why are different colors used for the profiles?

For easier comparison with other figures; each class gets its own colour

Fig11: the tickmarks on the colorbar do not quite agree with the increments

Fixed.

Fig 12,13: “latitude”, “longitude” labels are missing

For consistency with the other plots, we've left these off

Fig 13: remove right-hand y-axis labels

Done

Fig 14: The panels are missing grids and need more detailed captions

Yes, the caption did need to be more detailed, thanks for catching that. It has been updated. Figure updated.

FigA1: very small fonts, consider using only one ylabel “depth (m)” on the left hand side and remove all other labels, as the panels overall share identical y-axes. Perhaps include titles with larger fonts inside of the panels

Modified.

DaniJonesOcean commented 1 year ago

Resolved by

https://github.com/so-wise/weddell_gyre_clusters/pull/124