-
-
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
🌌🧐 Macro evaluation #850
🌌🧐 Macro evaluation #850
Conversation
trigger ci
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.
Needs a bit of high-level documentation. can you add some notes on this in understanding_evaluation.rst
(e.g., what issues are caused by micro evaluation? how does macro evaluation work and how does it solve these issues?) Maybe copy-paste some of the content that didn't make it into the metrics manuscript
e1d41fb - feel free to adjust / rephrase |
trigger ci
trigger ci
trigger ci
some parts of the metric description in the understanding evaluation part is incorrect, but also duplicated from the class doc itself, e.g., here, where the section about adjusted mean rank describes the adjusted MRR, and the expectation is incorrect. pykeen/docs/source/tutorial/understanding_evaluation.rst Lines 128 to 160 in 9cb4899
maybe it is better to directly link to the class reference? |
@mberr good idea |
trigger ci
trigger ci
Trigger CI
Trigger CI
Adds an evaluator for computing macro averaged rank-based metrics.
Also enables
docstr-coverage
anddarglint
forpykeen.evaluation
and adds missing docstrings.Dependencies