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

Analytics: Add Copilot audit events to the Power BI report #4

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

ncosoroltra
Copy link

No description provided.

Copy link
Collaborator

@jesusfer jesusfer left a comment

Choose a reason for hiding this comment

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

Review the comments regarding a missing table.

[user_id], @StartDate,
SUM([copilot_chats_count]), SUM([copilot_meetings_count]), SUM([copilot_files_count])

FROM dbo.[copilot_activity_log]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This table does not exist in the dbo schema.
You could use a new temporary table that you fill in before this SELECT in order to prepare the metrics to be upserted.

@jesusfer jesusfer changed the title Añadir cambios Copilot Analytics: Add Copilot audit events to the Power BI report May 10, 2024
@jesusfer jesusfer added the enhancement New feature or request label May 10, 2024
@jesusfer jesusfer linked an issue May 10, 2024 that may be closed by this pull request
6 tasks
-- TODO
[Copilot ChatsCount] BIGINT DEFAULT 0,
[Copilot MeetingsCount] BIGINT DEFAULT 0,
[Copilot FilesCount] BIGINT DEFAULT 0
);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use separate words for the metric names. Makes them friendlier and they don't need to be changed later in the report.

Copy link
Collaborator

@jesusfer jesusfer left a comment

Choose a reason for hiding this comment

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

Review metrics in the incorrect table

[Copilot ChatsCount] [bit] NOT NULL DEFAULT 0,
[Copilot MeetingsCount] [bit] NOT NULL DEFAULT 0,
[Copilot FilesCount] [bit] NOT NULL DEFAULT 0

Copy link
Collaborator

Choose a reason for hiding this comment

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

These metrics don't belong to this table. Only true/false metrics belong here.

@jesusfer jesusfer changed the base branch from main to dev May 14, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Copilot events are available in the database but not in the report
2 participants