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.

Sign up for GitHub

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

Jump to bottom

tensorboard #1608

Merged
merged 10 commits into from
May 15, 2023
Merged

tensorboard #1608

merged 10 commits into from
May 15, 2023

Conversation

sauravkdeo
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Feb 16, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@FateScript
Copy link
Member

FateScript commented Feb 17, 2023 •
edited
Loading

Could you plz tell me what is the purpose of this PR? @sauravkdeo

@sauravkdeo
Copy link
Contributor Author

@FateScript
This will add the training loss graph and the learning rate graph to the tensorboard.

@ghsanti
Copy link

ghsanti commented Mar 8, 2023

Interesting, I do not know whether the code is correct, but would certainly be useful to have the loss and maybe also metrics like MSE as well, wouldnt it ?

FateScript
FateScript requested changes Mar 9, 2023
View reviewed changes
Copy link
Member

@FateScript FateScript left a comment

Choose a reason for hiding this comment

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

PTAL @sauravkdeo

yolox/core/trainer.py Outdated Show resolved Hide resolved
@sauravkumar-volt
Copy link

@FateScript any updates on this

@FateScript
Copy link
Member

LGTM.

@FateScript
Copy link
Member

@sauravkdeo workflow failed here, plz lint your code. Thanks.

@sauravkdeo
Copy link
Contributor Author

@FateScript Done

@FateScript
Copy link
Member

FateScript commented Apr 16, 2023 •
edited
Loading

@sauravkdeo Too much code are modified in this PR, plz carefully lint your own modified code but not other part in the file. Thanks.

@FateScript
Copy link
Member

Code still failed with lint @sauravkdeo

@sauravkdeo
Copy link
Contributor Author

@FateScript Done

@sauravkdeo
Copy link
Contributor Author

sauravkdeo commented Apr 30, 2023 •
edited
Loading

@FateScript Can you please merge this?
I am not authorized to merge the PR

@FateScript FateScript merged commit 54a51e1 into Megvii-BaseDetection:main May 15, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FateScript FateScript FateScript approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@sauravkdeo @CLAassistant @FateScript @ghsanti @sauravkumar-volt