fixing trapezoid() comments for clarity

[NivekT]

Stack from ghstack:

• -> fixing trapezoid() comments for clarity #64592

cc @mruberry @rgommers @heitorschueroff

Differential Revision: D30785663

[facebook-github-bot]

🔗 Helpful links

• 🧪 See artifacts and rendered test results at hud.pytorch.org/pr/64592

💊 CI failures summary and remediations

As of commit aff727c (more details on the Dr. CI page):

---

• 1/1 failures introduced in this PR

---

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages:

win-vs2019-cpu-py3 / test (default, 1, 2, windows.4xlarge) (1/1)

Step: "Run test scripts" (full log | diagnosis details | 🔁 rerun)

2021-09-07T20:24:05.8993421Z ERROR [0.000s]: test_poisson_sample (__main__.TestDistributions)

2021-09-07T20:24:05.8985018Z   File "distributions/test_distributions.py", line 812, in _check_sampler_discrete
2021-09-07T20:24:05.8986009Z     chisq, p = scipy.stats.chisquare(counts[msk], pmf[msk] * num_samples)
2021-09-07T20:24:05.8987077Z   File "c:\jenkins\miniconda3\lib\site-packages\scipy\stats\stats.py", line 6852, in chisquare
2021-09-07T20:24:05.8988039Z     return power_divergence(f_obs, f_exp=f_exp, ddof=ddof, axis=axis,
2021-09-07T20:24:05.8989067Z   File "c:\jenkins\miniconda3\lib\site-packages\scipy\stats\stats.py", line 6694, in power_divergence
2021-09-07T20:24:05.8989926Z     raise ValueError(msg)
2021-09-07T20:24:05.8991106Z ValueError: For each axis slice, the sum of the observed frequencies must agree with the sum of the expected frequencies to a relative tolerance of 1e-08, but the percent differences are:
2021-09-07T20:24:05.8992156Z 0.008265582255680495
2021-09-07T20:24:05.8992410Z 
2021-09-07T20:24:05.8992792Z ======================================================================
2021-09-07T20:24:05.8993421Z ERROR [0.000s]: test_poisson_sample (__main__.TestDistributions)
2021-09-07T20:24:05.8994196Z ----------------------------------------------------------------------
2021-09-07T20:24:05.8994823Z Traceback (most recent call last):
2021-09-07T20:24:05.8995673Z   File "distributions/test_distributions.py", line 1352, in test_poisson_sample
2021-09-07T20:24:05.8996542Z     self._check_sampler_discrete(Poisson(rate),
2021-09-07T20:24:05.8997447Z   File "distributions/test_distributions.py", line 812, in _check_sampler_discrete
2021-09-07T20:24:05.8998480Z     chisq, p = scipy.stats.chisquare(counts[msk], pmf[msk] * num_samples)
2021-09-07T20:24:05.8999607Z   File "c:\jenkins\miniconda3\lib\site-packages\scipy\stats\stats.py", line 6852, in chisquare
2021-09-07T20:24:05.9000563Z     return power_divergence(f_obs, f_exp=f_exp, ddof=ddof, axis=axis,
2021-09-07T20:24:05.9001598Z   File "c:\jenkins\miniconda3\lib\site-packages\scipy\stats\stats.py", line 6694, in power_divergence
2021-09-07T20:24:05.9002450Z     raise ValueError(msg)

---

This comment was automatically generated by Dr. CI (expand for details).

Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

[NivekT]

@NivekT has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

[mruberry]

Neat; thanks @NivekT

[facebook-github-bot]

@NivekT merged this pull request in 1409492.