Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

change the order of train, valid and test #103

Merged
merged 1 commit into from
Oct 22, 2019
Merged

change the order of train, valid and test #103

merged 1 commit into from
Oct 22, 2019

Conversation

CunliangGeng
Copy link
Member

To solve the issue #99 .

@CunliangGeng CunliangGeng self-assigned this Sep 20, 2019
@coveralls
Copy link

Pull Request Test Coverage Report for Build 575

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 74.874%
Totals Coverage Status
Change from base Build 571: 0.0%
Covered Lines: 2226
Relevant Lines: 2973
💛 - Coveralls

@CunliangGeng CunliangGeng mentioned this pull request Oct 8, 2019
Merged
@CunliangGeng CunliangGeng mentioned this pull request Oct 21, 2019
Merged
NicoRenaud
NicoRenaud approved these changes Oct 22, 2019
View reviewed changes
Copy link
Member

@NicoRenaud NicoRenaud left a comment

Choose a reason for hiding this comment

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

good catch !

@CunliangGeng CunliangGeng merged commit ccd1d53 into development Oct 22, 2019
@CunliangGeng CunliangGeng deleted the issue99_learning_order branch October 22, 2019 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@NicoRenaud NicoRenaud NicoRenaud approved these changes

Assignees

@CunliangGeng CunliangGeng

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@CunliangGeng @coveralls @NicoRenaud