> On 12 Apr 2023, at 14:14, Tom Lane <t...@sss.pgh.pa.us> wrote:
> 
> Daniel Gustafsson <dan...@yesql.se> writes:
>> I was inclined to spell it out as mcv_frequencies but we use xxx_freqs
>> elsewhere on the same page so keeping it consistent seems better.  The 
>> attached
>> does this as well as adding mcf/mcv as acronyms as previously mentioned 
>> (since
>> they are both tagged as <acronym>).
> 
> mcv_freqs looks good.  I'd write the glossary entries as singular
> (Most Common Frequency, Most Common Value) since our typical usage
> is to pluralize them at the point of use ("MCVs").  Also, just
> expanding the acronym doesn't seem that helpful.  Maybe more like

Pushed with your suggested changes, thanks!

--
Daniel Gustafsson



Reply via email to