<div dir="ltr"><div dir="ltr"><div>Hi all, one week to go!  Here is the status:</div><ul><li>11 votes + non-blocking discussion that I've requested to move to a new issue<br></li><li>8 votes + reviews + comments that should be addressed</li><li>11 votes + discussion that I've commented on</li><li>11 votes + reviews + comments that should be addressed</li><li>9 votes</li><li>10 votes + discussion</li><li>9 votes + discussion</li></ul><div>So, none of them are passing, and #1408 and #1507 are in an interesting position.  Paul has suggested comments that Lukas has agreed would be helpful, but they weren't part of the PR at the time the vote started.  We don't have a clear policy here.</div><div><br></div><div>There are three kinds of comments.  1) Those that are obvious and should just be fixed.  Things like typos or easy fixes that wouldn't affect anyone's vote by any rational measure.  2) Those that are substantial enough that they should probably be addressed after the vote, but shouldn't really affect people's votes.  3) Dealbreakers that cause a no vote.</div><div><br></div><div>My suggestion for Lukas:  for 1) just do the fix and for 3) keep the comments unresolved for more discussion.  For 2) we can follow this procedure:</div><div><ul><li>Start a new PR that is based on the existing PR branch, addressing those comments</li><li>Resolve a comment at the bottom of the original PR noting the new PR</li><li>Resolve the comments in the original PR the new PR will address</li></ul><div>Any objections from the committee?</div></div><div>Thanks,</div><div>-Aaron</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 27, 2025 at 3:00 PM Aaron Brewster <<a href="mailto:asbrewster@lbl.gov" target="_blank">asbrewster@lbl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><font face="arial, sans-serif"><span style="color:rgb(31,35,40)">Hi all, we have a bumper crop of votes that were started by today's Telco!</span><span style="color:rgb(31,35,40)">  Please vote on the below PRs using emojis on the comments I linked to below. 👍 for yes, 👎 for no, anything else (for example 👀) to abstain.  We need 14 votes to hit quorum so please review and vote! </span>Voting will close in two weeks.<br></font></div><div><ul><li><span style="color:rgb(31,35,40)"><font face="arial, sans-serif"><a href="https://github.com/nexusformat/definitions/pull/1521#issuecomment-2617047160" target="_blank">Allow for open enumerations #1521</a><br></font></span></li><li><font face="arial, sans-serif"><a href="https://github.com/nexusformat/definitions/pull/1408#issuecomment-2617049939" target="_blank">Fairmat 2024: additions and clarifications in NXbeam #1408</a><br></font></li><li><a href="https://github.com/nexusformat/definitions/pull/1525#issuecomment-2617052302" target="_blank"><font face="arial, sans-serif">NXcomponent as a parent base class #1525 </font></a></li><li><font face="arial, sans-serif"><a href="https://github.com/nexusformat/definitions/pull/1507#issuecomment-2617055267" target="_blank">Implement-NXobject-inheritance #1507</a><br></font></li><li><font face="arial, sans-serif"><a href="https://github.com/nexusformat/definitions/pull/1519#issuecomment-2617058029" target="_blank">Move NXlens_em to base classes #1519</a><br></font></li><li><font face="arial, sans-serif"><a href="https://github.com/nexusformat/definitions/pull/1413#issuecomment-2617062395" target="_blank">Fairmat 2024: several new base classes in NXsample and NXsample_component #1413</a><br></font></li><li><a href="https://github.com/nexusformat/definitions/pull/1528#issuecomment-2617066301" target="_blank"><font face="arial, sans-serif">[resubmission] move NXpid_controller to base_classes #1528</font></a></li></ul><div><span style="font-family:arial,sans-serif">Note as we saw in #1408, a vote can fail due to more discussion and that's ok, so if you've missed some recent Telcos it's fine to jump in!</span></div><div><span style="font-family:arial,sans-serif"><br></span></div><div><span style="font-family:arial,sans-serif">Thanks!</span><br></div><div>-Aaron</div></div></div>
</blockquote></div>
</div>