-
Notifications
You must be signed in to change notification settings - Fork 848
Description
Describe the issue linked to the documentation
Currently in some older parts of the pytorch-forecasting, we still have Google style of docstrings, but I think we should move it numpydoc style now.
Some areas I could find the issues:
-
_timeseries.py- @karthikch947 is working on this starting from this line:def save(self, fname: str) -> None: -
pytorch_forecasting/layers/_attention- most files - @omkar-334 is working on this -
pytorch_forecasting/layers/_decomposition/_series_decomp.py@Meet-Ramjiyani-10 -
pytorch_forecasting/layers/_embeddings/- most files - @Soham-47 is working on this [DOC] migration of docstrings to numpydoc style in embeddings layer #2079 -
pytorch_forecasting/layers/_encoders/- most files - @Skvmqq [DOC] Migrate encoder docstrings from Google to NumPy style #2121 -
pytorch_forecasting/layers/_filter/- most files - @AyushDineshRathi [DOC] Migration of docstrings in layers/_filter to NumPydoc style #2116 -
pytorch_forecasting/layers/_output/- most files @Meet-Ramjiyani-10 -
pytorch_forecasting/metrics/distributions.py@Meet-Ramjiyani-10 [DOC] migrate distributions.py docstrings to numpydoc style #2118 -
point.py- @karthikch947 is working on this -
quantile.py@amruth6002 [DOC] migrate nhits and quantile docstrings to numpydocstyle #2111 -
pytorch_forecasting/models/base/_base_model.py@AyushDineshRathi -
_nhits.py@amruth6002 [DOC] migrate nhits and quantile docstrings to numpydocstyle #2111 -
nn- most files @AyushDineshRathi -
tuning.py@Siddhazntx [DOC] Convert TFT tuning and utils docstrings to numpydoc style #2097 -
_utils.py@Siddhazntx [DOC] Convert TFT tuning and utils docstrings to numpydoc style #2097
Suggest a potential alternative/fix
Move to numpydocstyle of docstrings.