search

openml / OpenML

Open Machine Learning
https://openml.org
BSD 3-Clause "New" or "Revised" License
666 stars 90 forks source link

Example for performance measures not correct #2

Closed berndbischl closed 11 years ago

berndbischl commented 11 years ago

See: http://expdb.cs.kuleuven.be/expdb/api/index.php#openml_evaluation_measures

Naming format is wrong here, measure names should be lower case.

Also: Why have this twice anyway? Probably best to remove the example output, just provide a link to the api call, this gives all needed info.

Also: area_under_ROC_curve should probably be area_under_roc_curve

joaquinvanschoren commented 11 years ago

Yes, we already noticed this. The api documentation page must be updated. I'll discuss with Jan tomorrow whether we want to remove the examples from the documentation page (replace the example by a link to an example api call).

Cheers, Joaquin

On 27 Jan 2013, at 19:31, berndbischl notifications@github.com wrote:

See: http://expdb.cs.kuleuven.be/expdb/api/index.php#openml_evaluation_measures

Naming format is wrong here, measure names should be lower case.

Also: Why have this twice anyway? Probably best to remove the example output, just provide a link to the api call, this gives all needed info.

Also: area_under_ROC_curve should probably be area_under_roc_curve

— Reply to this email directly or view it on GitHub.

janvanrijn commented 11 years ago

Yesterday the API docs have been updated.

berndbischl commented 11 years ago

ROC is still in uppercase in some measure names

joaquinvanschoren commented 11 years ago

ROC has been replaced by roc in all measures. API-docs fixed. Can I close the issue?

berndbischl commented 11 years ago

Yeah

Von Samsung Mobile gesendet

-------- Ursprüngliche Nachricht -------- Von: Joaquin Vanschoren notifications@github.com Datum:
An: joaquinvanschoren/OpenML OpenML@noreply.github.com Cc: berndbischl bernd_bischl@gmx.net Betreff: Re: [OpenML] Example for performance measures not correct (#2)

ROC has been replaced by roc in all measures. API-docs fixed. Can I close the issue?

— Reply to this email directly or view it on GitHub.

berndbischl commented 11 years ago

Acually it is still not update here

http://expdb.cs.kuleuven.be/expdb/api/index.php#openml_evaluation_measures

Also: Why have this twice anyway? Probably best to remove the example output, just provide a link to the api call, >this gives all needed info.

joaquinvanschoren commented 11 years ago

Hmm, forgot to push the update. Done now. I do agree that the explicit XML can be replaced by an example call. That would mean less maintenance...

Cheers, Joaquin

On 31 Jan 2013, at 19:07, berndbischl notifications@github.com wrote:

Acually it is still not update here

http://expdb.cs.kuleuven.be/expdb/api/index.php#openml_evaluation_measures

Also: Why have this twice anyway? Probably best to remove the example output, just provide a link to the api call, >this gives all needed info.

— Reply to this email directly or view it on GitHub.