Start testing for documentation building#217
Conversation
7fef25b to
6e20dfb
Compare
6e20dfb to
2bc5bf6
Compare
dd64342 to
9f1c898
Compare
The accessors framework is no longer present, so we don't need an absolute_temperature accessor
22605bb to
7fefdbe
Compare
Lines aren't supposed to have unexpected indentation
7fefdbe to
f88becd
Compare
93b105e to
31a849d
Compare
8190bc2 to
a615e6c
Compare
a615e6c to
5675193
Compare
5675193 to
91c6c87
Compare
962bc10 to
7fc00b4
Compare
8ea3468 to
54e92cc
Compare
0a5eecf to
c6a2a7e
Compare
| """ | ||
| Calculate I(|Q|) by circularly averaging 2D data between 2 radial limits. | ||
| r""" | ||
| Calculate I($|Q|$) by circularly averaging 2D data between 2 radial limits. |
There was a problem hiding this comment.
Should this one be $I(|Q|)$?
|
|
||
|
|
||
|
|
||
| DDDDDDDDDDDDD NNNNNNNN NNNNNNNN tttt |
There was a problem hiding this comment.
I'm surprised to see changes in this file - isn't it autogenerated?
There was a problem hiding this comment.
Good catch. On the refactor branch, I've reworked everything so that si.py and units.py don't need to be autogenerated, but I forget that that hadn't been ported over to master. I've made the appropriate changes to _autogen_warning.py and then regenerated si.py and units.py
|
|
||
|
|
||
|
|
||
| DDDDDDDDDDDDD NNNNNNNN NNNNNNNN tttt |
There was a problem hiding this comment.
Similarly this file - are these changes expected, isn't this file autogenerated?
There was a problem hiding this comment.
Agreed, see above.
1445f48 to
cb6fc1c
Compare
There was a problem hiding this comment.
Our agent can fix these. Install it.
No quality gates enabled for this code.
Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.
The goal for this PR is to resolve #214 and both fix the warnings with the building of the docs and ensure via the CI that the warnings do not return.
The following changes are included in this PR:
absolute_temperature.pywas removed, as it is part of the deprecated Accessor infrastructure and was not used anywhere.si.pyandunits.pymetadata.py,manipulations.py,binning.py,averaging.pyandmodel_requirements.py. Specifically, sphinx use pipes for keyword substitution while LaTeX uses it for absolute value. Similarly, sphinx uses underscores for hyperlinking while LaTeX uses it for subscripts. I escaped the control characters for sphinx and they should be passed cleanly on to LaTeX, but this is the part of this PR that I'm the least happy withconf.pyexplicitly ignore error messages when these cannot be found.type, Sphinx seems to have trouble figuring out which class to reference (or whether to reference the builtin functiontype). Lines 48-54 ofconf.pysimplify sphinx's resolver so that it doesn't get confused about references totype. I'm also particularly happy about this solution, but it was the only method I could find short of renaming every class member and breaking the readers in SasView.