-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
🐍 🛤️ Python result tracker #681
Conversation
trigger ci
trigger ci
trigger ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trigger ci
Trigger CI @mberr be careful on the placing of the spaces with code blocks - you can always try building the docs locally to check that they render properly
trigger ci
@@ -50,8 +50,7 @@ def end_run(self, success: bool = True) -> None: | |||
|
|||
|
|||
class PythonResultTracker(ResultTracker): | |||
""" | |||
A tracker which stores everything in Python dictionaries. | |||
"""A tracker which stores everything in Python dictionaries. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pykeen readme
does not like empty first lines 😕
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha this is obviously my doing. We could add a string strip in there so you can continue to write docstrings this way
This PR adds a simple in-memory result tracker, which stores parameters and metrics in Python dictionaries.