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

metric correlation plot intake source added #464

Merged
merged 3 commits into from
Jul 18, 2024
Merged

Conversation

austinhk
Copy link
Contributor

closes: #455


Description
Issue: Currently we are trying to build an extension for serving visualizations through a Juptyer lab extension. To do this we want to include visualization information in the catalogs. Initially, we created a base visualization data source similar to the existing one for projects and experiments. Now we need to create a concrete implementation for Metric Correlation Plot.
base.py

Solution: For this issue, we added a new MetricCorrelationPlotDataSource that is going to extend VizDataSourceMixin. We will read in the catalog data and metadata from the passed arguments . Then _get_schema creates a metric correlation plot visualization and sets it as the visualization object attribute.

Changes

Modified file viz.py within the intake_rubicon folder. This file holds the code for the new MetricCorrelationPlotDataSource.
Updated the init function within the intake_rubicon folder to now be able to include MetricCorrelationPlotDataSource for module importing purposes.
Updated the setup.cfg file to include the MetricCorrelationPlotDataSource as an Intake driver.

@austinhk austinhk requested review from a team as code owners July 18, 2024 17:29
@CLAassistant
Copy link

CLAassistant commented Jul 18, 2024

CLA assistant check
All committers have signed the CLA.

@austinhk austinhk closed this Jul 18, 2024
@austinhk austinhk reopened this Jul 18, 2024
Copy link
Member

@ryanSoley ryanSoley left a comment

Choose a reason for hiding this comment

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

great work! everything's running properly locally

NOTE TO SELF don't leave setuptools pinned for too long

@ryanSoley ryanSoley merged commit 8c6b527 into capitalone:main Jul 18, 2024
7 checks passed
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.

create intake source for metric list comparisons
3 participants