-
Notifications
You must be signed in to change notification settings - Fork 18
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
RESULTS upload fmralign_benchopt/benchopt_run_2023-07-05_12h16m30.parquet #35
base: main
Are you sure you want to change the base?
Conversation
@@ -0,0 +1 @@ | |||
{"pretty_name": "frmalign-benchopt"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you provide a nicer name for the benchmark? 😁 as is, it is hard to know what it is about.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops, it was actually supposed to be fmralign-benchopt
not frmalign-benchopt
. We'll fix and improve that. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is brain-alignment-benchopt
clearer ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the name that will appear on the front page of the results so you should be explicit.
I would say putting fMRI alignment
or Brain MRI alignment
is nice, without the bemchopt and the hyphen?
Note that this should be set as a name
attribute in the Objective
class.
@pbarbarant added more solvers to the benchmark. Is there a way to add those too? Do we just need to push the parquet with all of them? |
Yes you can simply push the parquet. I think if you do a second |
Loading result file for benchmark fmralign_benchopt.