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

#163: Show latest metric only #165

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pippinlee
Copy link

JIRA Ticket link:

#163

@choi-calvin
Copy link
Contributor

Sorry I'm a bit out of the loop on this. Was this the recommended UX solution from the MLE's?

@pippinlee
Copy link
Author

Hopefully this summarizes well: when logging a metric continuously throughout the training, MLE's primarily look for the last metric when inspecting a job run. So even though continuous logging of a metric would create a list, we can ignore all metrics except the last as all other metrics are process details and would make for a tougher time reading results.

@choi-calvin
Copy link
Contributor

Sorry let me rephrase, the conversation in the ticket seems inconclusive. Were there any conclusions made?

@mohammedri
Copy link

@choi-calvin the conclusion is to change the functionality of log_metrics such that only the most recent metric published to it is shown in the UI.

This is the task to be able to do so

@choi-calvin
Copy link
Contributor

Got it, thanks for the clarification

@mohammedri
Copy link

@pippinlee OK to merge this?

@pippinlee
Copy link
Author

Let's hold off on merging till UAT pipeline is green: https://jenkins.shehanigans.net/job/foundations-uat/

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.

None yet

3 participants