Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC, CI: refguide may vary locs #19525

Merged
merged 1 commit into from Nov 15, 2023

Conversation

tylerjereddy
Copy link
Contributor

[skip cirrus] [skip actions]

* add `# may vary` to two doc examples in attempt
to repair refguide-check circleCI fails like this
one:
https://app.circleci.com/pipelines/github/scipy/scipy/23372/workflows/09f8210c-c813-44a7-88ec-7377cb1882d8/jobs/78647

* I actually see completely different issues with refguide locally,
so wasn't able to repro + fix what we see in the CI locally

[skip cirrus] [skip actions]
@tylerjereddy tylerjereddy added Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure labels Nov 15, 2023
@ev-br
Copy link
Member

ev-br commented Nov 15, 2023

The relevant CI run is green, the fix LGTM, so let's merge it quickly to unblock main. Thank you Tyler.

@ev-br ev-br merged commit 2a6129f into scipy:main Nov 15, 2023
3 of 5 checks passed
@ev-br ev-br added this to the 1.12.0 milestone Nov 15, 2023
@tylerjereddy tylerjereddy deleted the treddy_circle_refguide_shim branch November 15, 2023 18:43
@ev-br
Copy link
Member

ev-br commented Nov 15, 2023

I actually see completely different issues with refguide locally

Care to fix up or open an issue? Some things are just transients and may well depend on e.g.the linalg library (like one of the ones fixed here: the doctest was relying on specific phase choice of the eigenvector components, and this indeed may vary).

tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request Nov 15, 2023
* see: scipy#19525 (comment)

* this allows the refguide check to pass
for me locally; I use `openblas-0.3.23`
from `spack` locally, maybe that's part
of the difference

[skip ci] [skip circle]
ev-br pushed a commit that referenced this pull request Nov 16, 2023
* DOC, MAINT: more refguide fixes

* see: #19525 (comment)

* this allows the refguide check to pass
for me locally; I use `openblas-0.3.23`
from `spack` locally, maybe that's part
of the difference.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Items related to the CI tools such as CircleCI, GitHub Actions or Azure Documentation Issues related to the SciPy documentation. Also check https://github.com/scipy/scipy.org
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants