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

feat: Add CopilotUsageChecker class for data analysis #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DevOps-zhuang
Copy link
Contributor

The code changes include adding a new file CopilotUsageChecker.ts which contains the implementation of the CopilotUsageChecker class. This class is responsible for performing various checks on the data, such as checking for missing dates, empty breakdowns, and zero activity days. The class also provides a method runChecks() to run all the checks and return the results.

image

The code changes include adding a new file `CopilotUsageChecker.ts` which contains the implementation of the `CopilotUsageChecker` class. This class is responsible for performing various checks on the data, such as checking for missing dates, empty breakdowns, and zero activity days. The class also provides a method `runChecks()` to run all the checks and return the results.
@DevOps-zhuang
Copy link
Contributor Author

DevOps-zhuang commented Jul 2, 2024

Data quality of the usage data #54

@karpikpl
Copy link
Collaborator

Hey @DevOps-zhuang - do you know what's the reason dates are missing? is that an API issue or by design?
can you update your PR with latest main and change the warning so it doesn't go away after few seconds? :)

<div class="copy-container">
<v-btn @click="checkDataQuality">Check Data Quality</v-btn>
<transition name="fade">
<div v-if="showDataMessage" :class="{'copy-message': true, 'error': isError}">{{ message }}</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe we shouldn't use the 'copy message' class

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, it is because of the css class we are using. and I am redesigin it by adding persistent data save and fetch here, will update it togeterh, thanks for your pointing out it.

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.

3 participants