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

Using comet-mbr for Multi-Model Translation Ranking: Questions About Input Format and GPU Disabling #161

Open
Aicha-cher opened this issue Aug 14, 2023 · 3 comments
Labels
bug Something isn't working question Further information is requested

Comments

@Aicha-cher
Copy link

Aicha-cher commented Aug 14, 2023

I'm attempting to utilize the following command: comet-mbr -s [SOURCE].txt -t [MT_SAMPLES].txt --num_sample [X] -o [OUTPUT_FILE].txt for the purpose of ranking multi-model translations. However, I have a couple of questions:

Could you provide clarification on the expected format of the [MT_SAMPLES].txt file? How should the distinct translations be structured within it?
I tried disabling the GPU using the --gpus=0 option, but it appears to be ineffective. Is there a different approach I should take to achieve this?

  • OS: iOS
  • Packaging pip
@Aicha-cher Aicha-cher added the question Further information is requested label Aug 14, 2023
@Aicha-cher Aicha-cher changed the title [QUESTION] Using comet-mbr for Multi-Model Translation Ranking: Questions About Input Format and GPU Disabling Aug 14, 2023
@ricardorei ricardorei added the bug Something isn't working label Aug 16, 2023
@ricardorei
Copy link
Collaborator

Hey! There is a bit more information about data format here.

You are right, the MBR command is hardcoded to use a single GPU.

I assigned a bug label to this issue until its solved.

@Aicha-cher
Copy link
Author

Thanks, @ricardorei! I followed the same approach a few days back. I locally commented out this line because the flag wasn't being considered in the code. Everything worked perfectly after that.

@Aicha-cher
Copy link
Author

Another suggestion if possible it to add the data format to the readme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants