-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
Document PyKEEN's performance tweaks #56
Conversation
References #55
Update description of TriplesFactory
I revised the first part about the TriplesFactory / ID-based triple representation. It would be nice to properly link to the properties of |
I am not sure whether this is the correct place, but somewhere we should also highlight the risk of manually modifying label-to-ID mappings, and the necessity of keeping the mappings consistent between train/test/validation. |
I made some improvements to the language, and also started to improve the notation (it was pretty confusing before with all of the stars, since RST interpreted them as italics)
@lvermue I made edits for clarity and improved the notation used to be a bit more consistent. Maybe you want to look into the algorithm at the bottom of "Filtering with Index-based Masking" to decide if you think this needs more notation, or if it's okay just as text |
Closes #55
This documentation should be a high-level explanation to non-technical users about what is in PyKEEN that makes it better than others. First thing to start with would be copying over the stuff written for the software paper by @lvermue
Fast OWA(Not implemented yet)Any other ideas?