Skip to content

Conversation

jiaqianjing
Copy link
Contributor

@jiaqianjing jiaqianjing commented Jun 20, 2021

PR types
New features

PR changes
Database

Description
Add COTE-BD & COTE-MFW #447

jiaqianjing added 2 commits June 21, 2021 09:15
Add COTE-MFW Dataset #447
add annotation of COTE-MFW
@jiaqianjing jiaqianjing changed the title Add COTE-BD Dataset #447 Add COTE-BD & COTE-MFW Dataset #447 Jun 21, 2021
@jiaqianjing jiaqianjing changed the title Add COTE-BD & COTE-MFW Dataset #447 Add COTE-BD & COTE-MFW Dataset Jun 21, 2021
@ZeyuChen ZeyuChen added data Issues about data pipeline and dataset SIG labels Jun 21, 2021
Copy link
Contributor

@smallv0221 smallv0221 left a comment

Choose a reason for hiding this comment

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

In general, we dont recommend using try to handle exceptions unless there are no other ways to do it.

@ZeyuChen
Copy link
Member

@jiaqianjing Could you update the Pull Request according to comments?

@jiaqianjing
Copy link
Contributor Author

@jiaqianjing Could you update the Pull Request according to comments?

wait a moment

Copy link
Contributor Author

@jiaqianjing jiaqianjing left a comment

Choose a reason for hiding this comment

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

@jiaqianjing Could you update the Pull Request according to comments?

done

Copy link
Contributor Author

@jiaqianjing jiaqianjing left a comment

Choose a reason for hiding this comment

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

done

Copy link
Member

@ZeyuChen ZeyuChen left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for your nice contributions!

@ZeyuChen ZeyuChen merged commit 9a45f83 into PaddlePaddle:develop Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Issues about data pipeline and dataset SIG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants