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

[DOCS]: Adds Welcome Section and "What Is Cosmos" Blurb to Home Page #1251

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

cmarteepants
Copy link
Collaborator

@cmarteepants cmarteepants commented Oct 12, 2024

Summary

  • Enhanced the documentation by adding a welcome section and a detailed explanation of what Astronomer Cosmos is to the index / home page
  • Expanded example usage by
    • Explaining how a workflow looks like in DBT and providing a visual of the dbt graph
    • Explaining the role of DbtDag
  • Added a call to action for community involvement via Slack and Github
  • Minor fixes, including adding missing backticks

Changes

  • Updated index.rst (+72 additions, -19 deletions)
  • Added new image: jaffle_shop_dbt_graph.png

Related Issue(s)

Part of #1225

- Updated `index.rst` to enhance the documentation structure
- Included a welcome section and detailed explanation of Cosmos
- Expanded example usage with dbt graph and more usage details
Copy link

netlify bot commented Oct 12, 2024

Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name Link
🔨 Latest commit 555fbb4
🔍 Latest deploy log https://app.netlify.com/sites/sunny-pastelito-5ecb04/deploys/6710098af4df8900084d9746

Copy link

codecov bot commented Oct 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.73%. Comparing base (ae4ec78) to head (555fbb4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1251   +/-   ##
=======================================
  Coverage   95.73%   95.73%           
=======================================
  Files          67       67           
  Lines        3965     3965           
=======================================
  Hits         3796     3796           
  Misses        169      169           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@cmarteepants cmarteepants marked this pull request as ready for review October 15, 2024 18:24
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 15, 2024
@dosubot dosubot bot added the area:docs Relating to documentation, changes, fixes, improvement label Oct 15, 2024
@cmarteepants cmarteepants changed the title docs: add welcome section and cosmos explanation Docs: Adds Welcome Section and "What Is Cosmos" Blurb to Home Page Oct 15, 2024
@cmarteepants cmarteepants mentioned this pull request Oct 15, 2024
9 tasks
@cmarteepants cmarteepants changed the title Docs: Adds Welcome Section and "What Is Cosmos" Blurb to Home Page [DOCS]: Adds Welcome Section and "What Is Cosmos" Blurb to Home Page Oct 15, 2024
Below, you can see what the original dbt workflow looks like in a lineage graph. This graph helps illustrate the
relationships between different models:

.. image:: /_static/jaffle_shop_dbt_graph.png
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
.. image:: /_static/jaffle_shop_dbt_graph.png
.. image:: https://raw.githubusercontent.com/astronomer/astronomer-cosmos/main/docs/_static/jaffle_shop_dbt_graph.png

wydt?

Copy link
Collaborator Author

@cmarteepants cmarteepants Oct 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I tried that, it didn't render (I assume because it wasn't in version control yet). What's the difference between this, and using the relative path so that also works locally?

Copy link
Contributor

@pankajkoti pankajkoti Oct 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think they’re just alternatives to each other. I maybe wrong wrt rst syntax, however, I would consider /_static/jaffle_shop_dbt_graph.png to be an absolute path, and I’m unsure how that works :). I’m wondering if the correct relative path should be ./_static/jaffle_shop_dbt_graph.png instead.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I initially tried https://raw.githubusercontent.com/astronomer/astronomer-cosmos/main/docs/_static/jaffle_shop_dbt_graph.png, I received an error. Intuitively that made sense since the image is new, and does not exist on the main branch.

I then tried ./_static/jaffle_shop_dbt_graph.png, and received the same error. Last attempt was /_static/jaffle_shop_dbt_graph.png, and the docs compiled properly.

As written, the image shows up in the PR preview environment. See here - it's the image for the dbt lineage graph. I'm fine with switching to pulling the image from github main branch if that is what you want, but it means that this PR won't work until after the PR has been merged to main.

Why Should You Use Cosmos?
___________________________

Cosmos makes orchestrating dbt workflows:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Cosmos makes orchestrating dbt workflows:
Cosmos makes orchestrating dbt-core workflows:

Shall we use "Dbt-core" instead of "dbt"?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tatiana / @pankajkoti any thoughts ^^?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not too sure. Maybe we could add a line somewhere that in general when we mean dbt, we're referring to dbt-core workflows throughout the references. no strong opinion.

Copy link

@yanmastin-astro yanmastin-astro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor punctuation and grammar fixes but everything else looks great!

docs/index.rst Outdated Show resolved Hide resolved
docs/index.rst Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
Co-authored-by: yanmastin-astro <[email protected]>
Co-authored-by: yanmastin-astro <[email protected]>
Copy link

@yanmastin-astro yanmastin-astro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more minor formatting fix to remove line break in list

docs/index.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@pankajkoti pankajkoti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Would be nice to confirm if the path to the jaffle_shop_dbt_graph.png image works well. Anyhow, I guess we should be able to quickly iterate through that later in a subsequent PR if we see any issues with it.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:docs Relating to documentation, changes, fixes, improvement lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants