Skip to content

Conversation

@anhappdev
Copy link
Collaborator

  • I added a caption_id to the *.tfrecord files. The caption_id is the id field in the captions_source.tsv.
  • The caption_id will be used as the filename for the image generated by the mobile app.
  • With a caption_id in filename we can calculate any metrics (e.g. IQA-A) offline and map it with the spreadsheet for analytic purpose.

Relevant files are here:
https://github.com/anhappdev/tmp/releases/tag/6

@github-actions
Copy link

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@sonarqubecloud
Copy link

@anhappdev anhappdev marked this pull request as ready for review September 14, 2024 00:15
@anhappdev anhappdev requested a review from a team as a code owner September 14, 2024 00:15
@freedomtan
Copy link
Contributor

@mohitmundhragithub to test it and report if it's OK.

@mohitmundhragithub
Copy link
Contributor

This PR seems to be working fine.

@anhappdev do you want to udpate the link for the dataset for the test dataset as well?

input_path: "https://github.com/anhappdev/tmp/releases/download/6/coco_gen_full.tfrecord"
groundtruth_path: ""
}
tiny {
Copy link
Contributor

Choose a reason for hiding this comment

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

probably we should update the tfrecord file for tiny as well, as the below one will throw an error.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok. The tiny is for integration test. I will update it later,

@anhappdev anhappdev merged commit 723d323 into submission-v4.1 Sep 17, 2024
@anhappdev anhappdev deleted the anh/add-caption-id branch September 17, 2024 23:44
@github-actions github-actions bot locked and limited conversation to collaborators Sep 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants