Revew: Document is readable and matches Yang for 1.1. 
Status: 1 issue

Is it possible that deployments for the filter-mode be 
require both INCLUDE and EXCLUDE (see pages. 6, 7 and grouping state-group-attribute)? 

The text says "INCLUDE" or "EXCLUDE".  It would be good to know why the authors felt
consider the "choice" statement where both "INCLUDE" and "EXCLUDE" would not be possible. 

Is this due to the normal situation for IGMP-proxy and MLD-Proxy in most deployed? 
What would happen if both were needed?  How can choice be part of an augmentation? 

Also, this review did not consider the automated tools review.