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

Update the handling of arXiv/DOI #12349

Closed
wants to merge 2 commits into from

Conversation

laky241
Copy link

@laky241 laky241 commented Jan 2, 2025

fixes #12292

We improved the InspireHEP API so that the cite key is consistent, no matter how you search. Before, if you searched using an arXiv number or DOI, the cite key would show up as a long DOI URL, which wasn't ideal. Now, we’ve added a "texkeys" field that returns the proper cite key across all search types, making it easier to use.

Added a "texkeys" field to the API response for searches using unique identifiers like arXiv numbers.
This ensures that the cite key is consistent, whether you're searching normally or by arXiv number/DOI.
Updated the API to include the correct cite key for all search methods, improving usability.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

JUnit tests are failing. In the area "Some checks were not successful", locate "Tests / Unit tests (pull_request)" and click on "Details". This brings you to the test output.

You can then run these tests in IntelliJ to reproduce the failing tests locally. We offer a quick test running howto in the section Final build system checks in our setup guide.

@@ -41,7 +41,7 @@ void setup() {
@Test
void badFilenameCharWillBeReplacedByUnderscore(@TempDir Path tempDir) throws Exception {

Path invalidFile = tempDir.resolve("?invalid.pdf");
Copy link
Member

Choose a reason for hiding this comment

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

Please revert this change

@@ -25,9 +25,16 @@ public List<IntegrityMessage> check(BibEntry entry) {
List<IntegrityMessage> messages = new ArrayList<>();
for (Field field : fields) {
Optional<String> value = entry.getFieldLatexFree(field);

// Debugging Output
Copy link
Member

Choose a reason for hiding this comment

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

if you want to have debug output just use LOGGER.debug

Copy link
Member

@subhramit subhramit left a comment

Choose a reason for hiding this comment

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

@laky241 hey, welcome to open source!
Before making a pull request, you can ask to be assigned on the issue.
Please take a look at the contributing guidelines and howtos on setting up a local workspace.
We do not encourage newcomers to be heavily reliant on AI (as it takes away the learning opportunity from the contributor and other people who may have been interested), but it is understandable considering you are new to here, so appreciate the effort!

I am closing this PR for now, you can come back anytime to work again on another issue once you have everything set up.

@subhramit subhramit closed this Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use default texkeys when importing from Inspirehep web search with arxiv number
3 participants