Overhaul custom entities page#676
Conversation
Summary of ChangesHello @kmoscoe, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly overhauls the documentation for defining custom entities, now referred to as non-place entities. The primary focus has shifted to the explicit method of entity definition using MCF and CSV files, removing previous implicit schema discussions. The changes also include an expanded section on how to define and use enumerations for entity types, and a clearer explanation of how statistical variables are defined and linked to these entities. The overall structure of the related documentation has been reorganized to improve clarity and user guidance. Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request provides a significant overhaul of the custom entities documentation, making it clearer and more focused on the explicit definition method. The restructuring and addition of an expanded enums section are excellent improvements. I've identified a few minor areas for improvement in the examples and text to enhance clarity and correctness.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
custom_dc/custom_entities.md
Outdated
| # Define custom (non-place) entities | ||
|
|
||
| This page shows you how to define <!--or extend--> custom non-place entities, which may be part of the process to add your data to your local instance. It assumes you are already familiar with the content in [Key concepts](/data_model.html) and [Prepare and load your own data](custom_data.md). | ||
| This page shows you how to define (or extend) custom (non-place) entities, which may be part of the process to add your data to your local instance. It assumes you are already familiar with the content in [Key concepts](/data_model.html) and [Prepare and load your own data](custom_data.md). |
There was a problem hiding this comment.
Should we briefly talk about or link entities and place entities in general before they dive into custom entities here?
There was a problem hiding this comment.
Looks like you have links later on but think about whether it makes sense to talk about it first up.
There was a problem hiding this comment.
It's explained at length in the linked sections, so I'd prefer not to repeat it here.
custom_dc/custom_entities.md
Outdated
| - `typeOf`: For an entity type, this must be `Class`. | ||
|
|
||
| You can additionally define any number of key:value pairs. | ||
| If you need to define custom entity types in MCF (rare), you define them in MCF. You can have a single MCF file or as many as you like. |
There was a problem hiding this comment.
Should have a blurb about why this is rare. One challenge today is that it is difficult to find what the existing types are.
There was a problem hiding this comment.
Again, this is discussed in the other page. I will link to it.
This PR completely rewrites the custom entities page to cover only the explicit method of defining them. It also adds an expanded section on enums and restructures everything.
Staged at http://bullie.svl.corp.google.com:4000