Skip to content

Conversation

@evan-aveva
Copy link
Contributor

Adding intro_cds dataset. Reduced Windtopia dataset with 60 streams.

@evan-aveva evan-aveva requested a review from a team as a code owner April 30, 2025 00:47
@gitstream-cm
Copy link

gitstream-cm bot commented Apr 30, 2025

🚨 gitStream Monthly Automation Limit Reached 🚨

Your organization has exceeded the number of pull requests allowed for automation with gitStream.
Monthly PRs automated: 316/250

To continue automating your PR workflows and unlock additional features, please contact LinearB.

Copy link
Contributor

@afink-aveva afink-aveva left a comment

Choose a reason for hiding this comment

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

Do we really want the intro_cds dataset in this repo? This dataset is specific for the Introduction to Cds/Cvs class and not a true "dataset" that customers would use to explore various objects in Cds (assets, dataviews, etc). So I'm not sure it fits here. I actually had a PR a few weeks ago to add this dataset but abandoned it for this reason.

At the same time, I could also potentially see value in customer's having access to the course dataset after the fact (since they have the workbooks) they could try it on their own time as well if they have a Cds tenant. But again, if so, I'm not sure how they would find their way here to find it.

@derekendres @evan-aveva What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants