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

DOI improvements #1447

Merged
merged 16 commits into from
May 25, 2016
Merged

DOI improvements #1447

merged 16 commits into from
May 25, 2016

Conversation

stefan-kolb
Copy link
Member

@stefan-kolb stefan-kolb commented May 24, 2016

First batch of improvements of the DOI retrieval logic.

more to come: #1424.

@stefan-kolb stefan-kolb changed the title [WIP] DOI improvements DOI improvements May 24, 2016
@stefan-kolb stefan-kolb added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label May 24, 2016
}

private static double editDistanceIgnoreCase(String a, String b) {
return METRIC_DISTANCE.distance(a.toLowerCase(), b.toLowerCase());
Copy link
Contributor

Choose a reason for hiding this comment

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

Use a locale when using toLowerCase.

@simonharrer
Copy link
Contributor

LGTM

import net.sf.jabref.model.database.BibDatabase;
import net.sf.jabref.model.entry.BibEntry;

public class CrossrefFetcherTest {
Copy link
Member

Choose a reason for hiding this comment

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

Rewrite this as a real JUnit test?

Copy link
Member Author

Choose a reason for hiding this comment

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

We already have tests for the Crossref fetcher. This is a "tool" to analyze the accuracy of the algorithm.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a short JavaDoc to describe the purpose and the usage of the tool?

Copy link
Member Author

Choose a reason for hiding this comment

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

Should I remove it from the git repo?

Copy link
Member

Choose a reason for hiding this comment

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

I don't know 😉 If it will be useful for further improvements leave it - if it has already done its part remove it.

@stefan-kolb
Copy link
Member Author

Thanks for your comments 👍

@stefan-kolb stefan-kolb merged commit 4a6f191 into master May 25, 2016
@stefan-kolb stefan-kolb deleted the doi branch May 25, 2016 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants