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

ci: debug RTFD build #2859

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

ci: debug RTFD build #2859

wants to merge 9 commits into from

Conversation

Borda
Copy link
Member

@Borda Borda commented Dec 3, 2024

What does this PR do?

The name of the builder is: html
Warning, treated as error:
/home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/src/torchmetrics/audio/dnsmos.py:docstring of torchmetrics.audio.dnsmos.DeepNoiseSuppressionMeanOpinionScore:10:Exception occurred in plotting deep_noise_suppression_mean_opinion_score-1
 from /home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/docs/source/audio/deep_noise_suppression_mean_opinion_score.rst:
Traceback (most recent call last):
  File "/home/docs/.asdf/installs/python/3.9.20/lib/python3.9/site-packages/matplotlib/sphinxext/plot_directive.py", line 552, in _run_code
    exec(code, ns)
  File "<string>", line 4, in <module>
  File "/home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/src/torchmetrics/metric.py", line 550, in wrapped_func
    update(*args, **kwargs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/src/torchmetrics/audio/dnsmos.py", line 122, in update
    metric_batch = deep_noise_suppression_mean_opinion_score(
  File "/home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/src/torchmetrics/functional/audio/dnsmos.py", line 232, in deep_noise_suppression_mean_opinion_score
    onnx_sess = _load_session(f"{DNSMOS_DIR}/{'p' if personalized else ''}DNSMOS/sig_bak_ovr.onnx", device, num_threads)
  File "/home/docs/checkouts/readthedocs.org/user_builds/torchmetrics/checkouts/2848/src/torchmetrics/functional/audio/dnsmos.py", line 106, in _load_session
    infs = InferenceSession(path, providers=["CPUExecutionProvider"], sess_options=opts)
  File "/home/docs/.asdf/installs/python/3.9.20/lib/python3.9/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 419, in __init__
    self._create_inference_session(providers, provider_options, disabled_optimizers)
  File "/home/docs/.asdf/installs/python/3.9.20/lib/python3.9/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 480, in _create_inference_session
    sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
onnxruntime.capi.onnxruntime_pybind11_state.InvalidProtobuf: [ONNXRuntimeError] : 7 : INVALID_PROTOBUF : Load model from /home/docs/.torchmetrics/DNSMOS/DNSMOS/sig_bak_ovr.onnx failed:Protobuf parsing failed.
make: *** [Makefile:19: html] Error 2

ref: https://readthedocs.org/projects/torchmetrics/builds/26426250/

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?
PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃


📚 Documentation preview 📚: https://torchmetrics--2859.org.readthedocs.build/en/2859/

@Borda Borda requested a review from lantiga as a code owner December 3, 2024 11:35
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 3, 2024
@Borda Borda changed the title debug RTFD build [wip] ci: debug RTFD build Dec 3, 2024
@Borda Borda marked this pull request as draft December 3, 2024 11:59
Copy link

codecov bot commented Dec 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99%. Comparing base (f1d8f53) to head (fad3910).

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #2859      +/-   ##
=========================================
+ Coverage      69%     99%     +30%     
=========================================
  Files         346      14     -332     
  Lines       19129     176   -18953     
=========================================
- Hits        13227     175   -13052     
+ Misses       5902       1    -5901     

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation topic: Audio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant