Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
finam
Manage
Activity
Members
Labels
Plan
Issues
31
Issue boards
Milestones
Code
Merge requests
3
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Model registry
Monitor
Service Desk
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
FINAM
finam
Commits
3cf9b5b3
Commit
3cf9b5b3
authored
2 years ago
by
Sebastian Müller
🐈
Browse files
Options
Downloads
Patches
Plain Diff
docs: add note to encourage implementation of _validate and _finalize
parent
d4b109ce
No related branches found
Branches containing commit
No related tags found
Tags containing commit
1 merge request
!156
Component: don't require to overwrite _validate or _finalize
Pipeline
#126290
passed with stages
in 3 minutes
Changes
1
Pipelines
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
docs/source/finam-book/development/components.rst
+8
-0
8 additions, 0 deletions
docs/source/finam-book/development/components.rst
with
8 additions
and
0 deletions
docs/source/finam-book/development/components.rst
+
8
−
0
View file @
3cf9b5b3
...
...
@@ -189,6 +189,10 @@ This is done in :meth:`.TimeComponent._connect`.
After this connection phase, models can validate their state in :meth:`.TimeComponent._validate`. We do nothing there.
.. note::
It is not strictly required to implement `_validate` but it is highly encouraged to do so.
.. code-block:: Python
# imports...
...
...
@@ -315,6 +319,10 @@ In method :meth:`.TimeComponent._finalize`, the component can do any cleanup req
We do nothing special here.
.. note::
It is not strictly required to implement `_finalize` but it is highly encouraged to do so.
.. code-block:: Python
# imports...
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment