Skip to content
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

glossary: create Glossary page (besides Basic Concepts?) #1395

Closed
jorgeorpinel opened this issue Jun 5, 2020 · 39 comments · Fixed by #2595
Closed

glossary: create Glossary page (besides Basic Concepts?) #1395

jorgeorpinel opened this issue Jun 5, 2020 · 39 comments · Fixed by #2595
Assignees
Labels
type: discussion Requires active participation to reach a conclusion. type: enhancement Something is not clear, small updates, improvement suggestions website: eng-doc DEPRECATED JS engine for /doc

Comments

@jorgeorpinel
Copy link
Contributor

jorgeorpinel commented Jun 5, 2020

"Basic Concepts" is an actual proposed section (see #550) and not related (?) specifically to the tooltips/glossary. Furthermore, it's used all over docs, not just in the User Guide.

Should we put it perhaps in content/docs/glossary/ ?


UPDATE: Jump to #1395 (comment)

@jorgeorpinel jorgeorpinel added type: enhancement Something is not clear, small updates, improvement suggestions good first issue Good for newcomers website: eng-blog DEPRECATED JS engine for /blog labels Jun 5, 2020
@shcheklein
Copy link
Member

I'm not sure I understand the proposal and the context (I have some ideas, but not sure we are on the same page). Could you clarify please?

@shcheklein

This comment has been minimized.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 5, 2020

Motivation: I just find it strange that these md metafiles are found in user-guide/basic-concepts/
Also, they are more like glossary terms than basic concepts (there's overlap of course)

@shcheklein
Copy link
Member

The idea behind user-guide/basic-concepts/ structure is to eventually drive both - basic concepts and glossary. Glossary items feeding from some frontmatter entries, or summarizing basic concepts articles in some other way.

I think that should be more or less enough and we won't need a separate glossary section.

@me-heer

This comment has been minimized.

@jorgeorpinel jorgeorpinel added type: discussion Requires active participation to reach a conclusion. and removed good first issue Good for newcomers labels Jun 11, 2020
@jorgeorpinel

This comment has been minimized.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 11, 2020

I see @shcheklein but from what I remember in our discussions the glossary terms are not the same as the basic concepts. There may be some overlap but not a lot I think.

@shcheklein
Copy link
Member

@jorgeorpinel we can try and see, my hope that they will overlap significantly.

@jorgeorpinel
Copy link
Contributor Author

OK so let's do #550 first, and include or confirm this one then.

@jorgeorpinel jorgeorpinel changed the title glossary: move basic-concepts/ out of user-guide/ , probably rename glossary: move basic-concepts/ out of user-guide/ + rename to glossary? Jul 20, 2020
@jorgeorpinel jorgeorpinel changed the title glossary: move basic-concepts/ out of user-guide/ + rename to glossary? glossary: move basic-concepts/ out of user-guide/ + rename to glossary/ ? Jul 20, 2020
@jorgeorpinel
Copy link
Contributor Author

So when I try to create an index in user-guide/basic-concepts/index.md I get this error:

image

I'll use a basic-concepts.md file alonside that dir for now. Cc @rogermparent

@rogermparent
Copy link
Contributor

@jorgeorpinel I just looked into the issue and found it's easy enough to fix, I just need to add an explicit exclusion for the filename index on glossary entries. Do you have a PR with some attempted content changes? If so I can adapt that, otherwise I can make a PR that sets up a dummy index page and we can add content from there.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jul 21, 2020

Thanks for looking into that Roger. So any other md file added there is expected to have frontmatter (otherwise app will crash)? I'm not sure it's the best approach to have a special directory like this in the middle of the content directory. That would feel more like a config or source directory. If it's going to be in content/ I think it should not have such side effects i.e. the frontmatter should be optional — only affecting the files where it's found in.

@rogermparent
Copy link
Contributor

@jorgeorpinel I could either move the glossary's directory out of the content dir or make it so all md files in there without Glossary frontmatter are passed over as glossary entries.

I leave it up to the group which solution to pick, I don't really have a preference here.

@jorgeorpinel
Copy link
Contributor Author

Let's wait until #550 is addressed (I already started with that) to decide here 🙂

@shcheklein
Copy link
Member

I would make it optional since it will be part of the content most likely.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jan 12, 2021

Some comments that support having a glossary page in addition to the Concepts section (from #2083):

collect all known and important terms within the project, the product and its usage. Examples:
NumPy - https://numpy.org/doc/stable/glossary.html
AWS - https://docs.aws.amazon.com/general/latest/gr/glos-chap.html

There are discussions between developers in comments to some issues where different understanding of the same terms takes place. The Glossary page could be a place to refer to as the reference.

@shcheklein
Copy link
Member

Yep, we can build an index page like this from frontmatter headers. And make it an index in the Basic Concepts, or make a separate auto-generated page.

@jorgeorpinel

This comment has been minimized.

@jorgeorpinel jorgeorpinel changed the title glossary: move basic-concepts/ out of user-guide/ + rename to glossary/ ? glossary: create Glossary page (besides Basic Concepts) Jan 12, 2021
@jorgeorpinel jorgeorpinel added the website: eng-doc DEPRECATED JS engine for /doc label Feb 15, 2021
@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Feb 15, 2021

The glossary page proposed here is separate from the Concepts section. It would basically be a single page with all the tooltips dumped into some order. Auto-generated by the engine. The Q is whether we want/need that.

@iesahin

This comment has been minimized.

@jorgeorpinel

This comment has been minimized.

@iesahin iesahin removed their assignment Apr 10, 2021
@casperdcl casperdcl removed the type: discussion Requires active participation to reach a conclusion. label Jun 9, 2021
@casperdcl
Copy link
Contributor

summary: @iterative/docs decided that we'd like the contents of https://github.com/iterative/dvc.org/tree/master/content/docs/user-guide/basic-concepts to be auto-rendered in a single page titled "Glossary" at https://dvc.org/doc/user-guide/glossary.

@rogermparent
Copy link
Contributor

we'd like the contents of https://github.com/iterative/dvc.org/tree/master/content/docs/user-guide/basic-concepts to be auto-rendered in a single page titled "Glossary" at https://dvc.org/doc/user-guide/glossary

Cool! This is certainly well within standard Gatsby stuff, so getting the data onto a specific page should be a breeze. We should be able to use the src/pages directory like we do for a few other static pages (i.e. src/pages/doc/user-guide/glossary). If somehow nested directories don't work here, we should still be able to figure something else out without too much trouble.

Once the page is there, the query should be pretty easy as well- we can model the new query after the existing one we use for the tooltips and render it into a docs page template, then add an entry for the page in the proper slot in the sidebar.

Beyond that, we may want to consider design- I'm all for just reusing the style of existing docs pages, but what does everyone else think?

@casperdcl
Copy link
Contributor

I'm all for just reusing the style of existing docs pages, but what does everyone else think?

agreed, something like:

# Glossary

<b id="bold-term">Bold term</b>: description text

rendered as the engine usually would.

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 9, 2021

We should be able to use the src/pages directory like we do for a few other static pages (i.e. src/pages/doc/user-guide/glossary

Can the glossary contents come from the frontmatter in https://github.com/iterative/dvc.org/tree/master/content/docs/user-guide/basic-concepts ?

I think we also want the glossary to be in https://dvc.org/doc/user-guide/basic-concepts (index page) initially. It would help us advance towards #550. But no super strong opinion on this cc @shcheklein

@jorgeorpinel
Copy link
Contributor Author

p.s. we also talked about automatic back-links to every doc where each term is mentioned as a tooltip (maybe as a collapsible section since there can be a lot). Could be a separate issue.

@casperdcl
Copy link
Contributor

casperdcl commented Jun 10, 2021

I think we also want the glossary to be in dvc.org/doc/user-guide/basic-concepts (index page) initially.

Strong disagree for me since basic concepts != glossary. The fact that the sources are currently in a folder called basic-concepts is misleading and should not (for now) be reflected in the auto-generated page path/title. Fleshing things out for inclusion in a Basic Concepts page should be a separate unrelated issue (#550)

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 10, 2021

It's not that unrelated. Keep in mind that this has been discussed before. We want to keep the glossary and basic concepts source together. What sidebar structure are you proposing @casperdcl ? Considering Basic Concepts.

And is our glossary really a glossary (or what kind is it, rather)? We seem to only have entries for basic concepts anyway. It's kind of a basic concepts glossary (unlike a full technical glossary like https://git-scm.com/docs/gitglossary)

@julieg18
Copy link
Contributor

julieg18 commented Jun 10, 2021

Can the glossary contents come from the frontmatter in https://github.com/iterative/dvc.org/tree/master/content/docs/user-guide/basic-concepts ?

@jorgeorpinel I'm not sure I understand the question. 🤔 Are you' asking if we could attempt to create the page without adding to the pages directory or are you asking if we can reuse the text inside the /content/docs/user-guide/basic-concepts? If it's the first one, there's probably a way to do it. If it's the second, I'm pretty sure we can without any issues!

@casperdcl
Copy link
Contributor

If it's the second, I'm pretty sure that's exactly what my original summary (#1395 (comment)) was stating.

@casperdcl
Copy link
Contributor

casperdcl commented Jun 10, 2021

It's not that unrelated. Keep in mind that this has been discussed before.

Are you arguing with yourself? #1395 (comment) 😆 I was just repeating what I thought were your previous sentiments to avoid debating off-topic stuff.

We want to keep the glossary and basic concepts source together.

yes, if and when we address #550. AFAIK the engineering implementation request here (#1395 (comment)) is fully compatible with the #550 long-term plan. Are you saying you disagree?

What sidebar structure are you proposing @casperdcl ? Considering Basic Concepts.

I... don't understand. Why are we constantly discussing Basic Concepts #550? You even titled this issue "besides Basic Concepts."

Sidebar structure for the Glossary isn't something I wanted to bother @julieg18 / @rogermparent with atm but I was thinking third item under User Guide (below What is DVC? & Project structure since it seems related to both) but fine with putting it elsewhere. If you are asking whether it should also itself be collapsible (like Command Reference) I don't really think so but again no strong opinion.

And is our glossary really a glossary (or what kind is it, rather)? We seem to only have entries for basic concepts anyway. It's kind of a basic concepts glossary (unlike a full technical glossary like git-scm.com/docs/gitglossary)

It is precisely the "full technical glossary" you linked. We do NOT have entries for basic concepts. By "entries for basic concepts" I mean detailed explanations (as opposed to brief tooltip/glossary entries). I suspect by "entries for basic concepts" you mean the limited number of terms, in which case I'd say selecting some (probably just the current terms) to flesh out the hypothetical Basic Concepts for #550 is a different topic (namely, #550 😖), while adding more brief terms to flesh out the current proposed Glossary is also a different (follow-up) issue.

I don't know what a basic concepts glossary is - to me it's an oxymoron. One year ago I think you would've agreed? #1395 (comment)

I suspect we're speaking 2 different languages & have 2 different ideas which means 4 different misunderstandings. Maybe another meeting lol?

The key thing is please state if you find this request wrong: #1395 (comment)

@iesahin
Copy link
Contributor

iesahin commented Jun 13, 2021

I think we need to discuss the relation between BC and glossary (maybe with some presentations from both of you) in the meeting. @jorgeorpinel @casperdcl

@julieg18
Copy link
Contributor

Should we go ahead and start on this issue, or are do we need more discussion on the naming/placement?

@casperdcl
Copy link
Contributor

I believe everyone's agreed on

while all the the subsequent discussion was about minor detail best addressed after we have a first PR?

@jorgeorpinel jorgeorpinel changed the title glossary: create Glossary page (besides Basic Concepts) glossary: create Glossary page (besides Basic Concepts?) Jun 23, 2021
@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 23, 2021

If it's the second, I'm pretty sure we can
Should we go ahead and start on this issue

@julieg18 yes: I meant that we want to use the existing frontmatter content from the files in /content/docs/user-guide/basic-concepts (tooltip texts). Whether or not you need to crate a page or any other element in the engine is up to you 🙂

I think we're good to go with this issue's implementation (ready when you are) like @casperdcl summarized in #1395 (comment). I'm just unsure about the nav (discussion to follow) but we can probably hash it out during the PR review.

I was thinking third item under User Guide...

@casperdc I should've clarified earlier that I'm referring to the URL path / general nav hierarchy, not exact order or collapsibility (BTW I also don't think so).

@jorgeorpinel
Copy link
Contributor Author

jorgeorpinel commented Jun 23, 2021

Are you arguing with yourself? 😆

@casperdc maybe. More precisely with myself from over a year ago (that comment is old). Like I mentioned it has been a long discussion. And still it's not clear since, again, we seem to be building a Basic Concepts Glossary after all.

Are you saying you disagree?

Summarizing, I'm just unsure about the nav at this point. The more I think about it, the more I like the idea of putting the "glossary" in guide/basic-concepts instead. Another benefit is that it will help us get going with #550 which otherwise may take very long to publish.

I think we should try to decide nowish (yes, taking into consideration the future Basic Concepts section) to avoid an unnecessary redirect later.

By "entries for basic concepts" I mean detailed explanations...
need to discuss the relation between BC and glossary in the meeting

Good idea @iesahin let's retro about what we each mean by DVC concepts and glossary. We shall report back ⌛

@jorgeorpinel jorgeorpinel added the type: discussion Requires active participation to reach a conclusion. label Jun 23, 2021
@jorgeorpinel
Copy link
Contributor Author

let's retro about what we each mean by DVC concepts and glossary

Update: we agreed that we can start with all the current tooltips and be flexible about what else to include. But ideally only DVC-specific terms I think. We'll manage it as it comes.

I forgot to discuss about the nav though 🤦 but please ping us from the PR @julieg18 and we'll decide then. Thanks all

@julieg18
Copy link
Contributor

Pull request opened!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: discussion Requires active participation to reach a conclusion. type: enhancement Something is not clear, small updates, improvement suggestions website: eng-doc DEPRECATED JS engine for /doc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants